Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Delta Between Two Patch Sets: src/plugin/PluginWbPassThrough.h

Issue 29332980: Issue #1484 - Use the currently distributed version of PassthroughAPP unaltered
Left Patch Set: new version of PassthroughAPP Created Dec. 22, 2015, 4:56 p.m.
Right Patch Set: use OnStartEx() instead of OnStart() Created Dec. 23, 2015, 3:18 p.m.
Left:
Right:
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
Right: Side by side diff | Download
« no previous file with change/comment | « src/plugin/PluginStdAfx.h ('k') | src/plugin/PluginWbPassThrough.cpp » ('j') | no next file with change/comment »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
LEFTRIGHT
(no file at all)
1 /* 1 /*
2 * This file is part of Adblock Plus <https://adblockplus.org/>, 2 * This file is part of Adblock Plus <https://adblockplus.org/>,
3 * Copyright (C) 2006-2015 Eyeo GmbH 3 * Copyright (C) 2006-2015 Eyeo GmbH
4 * 4 *
5 * Adblock Plus is free software: you can redistribute it and/or modify 5 * Adblock Plus is free software: you can redistribute it and/or modify
6 * it under the terms of the GNU General Public License version 3 as 6 * it under the terms of the GNU General Public License version 3 as
7 * published by the Free Software Foundation. 7 * published by the Free Software Foundation.
8 * 8 *
9 * Adblock Plus is distributed in the hope that it will be useful, 9 * Adblock Plus is distributed in the hope that it will be useful,
10 * but WITHOUT ANY WARRANTY; without even the implied warranty of 10 * but WITHOUT ANY WARRANTY; without even the implied warranty of
(...skipping 47 matching lines...) Expand 10 before | Expand all | Expand 10 after
58 /* [in] */ LPCWSTR szHeaders, 58 /* [in] */ LPCWSTR szHeaders,
59 /* [in] */ DWORD dwReserved, 59 /* [in] */ DWORD dwReserved,
60 /* [out] */ LPWSTR* pszAdditionalHeaders); 60 /* [out] */ LPWSTR* pszAdditionalHeaders);
61 61
62 STDMETHODIMP OnResponse( 62 STDMETHODIMP OnResponse(
63 /* [in] */ DWORD dwResponseCode, 63 /* [in] */ DWORD dwResponseCode,
64 /* [in] */ LPCWSTR szResponseHeaders, 64 /* [in] */ LPCWSTR szResponseHeaders,
65 /* [in] */ LPCWSTR szRequestHeaders, 65 /* [in] */ LPCWSTR szRequestHeaders,
66 /* [out] */ LPWSTR* pszAdditionalRequestHeaders); 66 /* [out] */ LPWSTR* pszAdditionalRequestHeaders);
67 67
68 HRESULT OnStart(LPCWSTR szUrl, IInternetProtocolSink* pOIProtSink, 68 HRESULT OnStartEx(IUri* pUri, IInternetProtocolSink* pOIProtSink,
sergei 2016/02/26 19:53:56 There is still Start method which calls OnStart, i
Eric 2016/05/19 16:10:19 It doesn't appear to be called at all. I verified
69 IInternetBindInfo* pOIBindInfo, DWORD grfPI, HANDLE_PTR dwReserved, 69 IInternetBindInfo* pOIBindInfo, DWORD grfPI, HANDLE_PTR dwReserved,
70 IInternetProtocol* pTargetProtocol); 70 IInternetProtocol* pTargetProtocol);
71 71
72 HRESULT OnRead(void* pv, ULONG cb, ULONG* pcbRead); 72 HRESULT OnRead(void* pv, ULONG cb, ULONG* pcbRead);
73 73
74 STDMETHODIMP ReportProgress( 74 STDMETHODIMP ReportProgress(
75 /* [in] */ ULONG ulStatusCode, 75 /* [in] */ ULONG ulStatusCode,
76 /* [in] */ LPCWSTR szStatusText); 76 /* [in] */ LPCWSTR szStatusText);
77 77
78 STDMETHODIMP ReportResult( 78 STDMETHODIMP ReportResult(
(...skipping 37 matching lines...) Expand 10 before | Expand all | Expand 10 after
116 116
117 // derived from IInternetProtocol 117 // derived from IInternetProtocol
118 STDMETHODIMP Read(/* [in, out] */ void* pv,/* [in] */ ULONG cb,/* [out] */ ULO NG* pcbRead) override; 118 STDMETHODIMP Read(/* [in, out] */ void* pv,/* [in] */ ULONG cb,/* [out] */ ULO NG* pcbRead) override;
119 119
120 bool m_shouldSupplyCustomContent; 120 bool m_shouldSupplyCustomContent;
121 }; 121 };
122 122
123 typedef PassthroughAPP::CMetaFactory<PassthroughAPP::CComClassFactoryProtocol, W bPassthroughProtocol> MetaFactory; 123 typedef PassthroughAPP::CMetaFactory<PassthroughAPP::CComClassFactoryProtocol, W bPassthroughProtocol> MetaFactory;
124 124
125 125
LEFTRIGHT

Powered by Google App Engine
This is Rietveld