Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(36)

Side by Side Diff: components/test_runner/web_frame_test_proxy.h

Issue 2211283004: Drop test-only WebFrameClient params/functions (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: test fix Created 4 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef COMPONENTS_TEST_RUNNER_WEB_FRAME_TEST_PROXY_H_ 5 #ifndef COMPONENTS_TEST_RUNNER_WEB_FRAME_TEST_PROXY_H_
6 #define COMPONENTS_TEST_RUNNER_WEB_FRAME_TEST_PROXY_H_ 6 #define COMPONENTS_TEST_RUNNER_WEB_FRAME_TEST_PROXY_H_
7 7
8 #include <memory> 8 #include <memory>
9 #include <utility> 9 #include <utility>
10 10
(...skipping 203 matching lines...) Expand 10 before | Expand all | Expand 10 after
214 Base::didDetectXSS(insecure_url, did_block_entire_page); 214 Base::didDetectXSS(insecure_url, did_block_entire_page);
215 } 215 }
216 216
217 void didDispatchPingLoader(const blink::WebURL& url) override { 217 void didDispatchPingLoader(const blink::WebURL& url) override {
218 // This is not implemented in RenderFrameImpl, so need to explicitly call 218 // This is not implemented in RenderFrameImpl, so need to explicitly call
219 // into the base proxy. 219 // into the base proxy.
220 test_client()->didDispatchPingLoader(url); 220 test_client()->didDispatchPingLoader(url);
221 Base::didDispatchPingLoader(url); 221 Base::didDispatchPingLoader(url);
222 } 222 }
223 223
224 void willSendRequest( 224 void willSendRequest(blink::WebLocalFrame* frame,
225 blink::WebLocalFrame* frame, 225 blink::WebURLRequest& request) override {
226 unsigned identifier, 226 Base::willSendRequest(frame, request);
227 blink::WebURLRequest& request, 227 test_client()->willSendRequest(frame, request);
228 const blink::WebURLResponse& redirect_response) override {
229 Base::willSendRequest(frame, identifier, request, redirect_response);
230 test_client()->willSendRequest(frame, identifier, request,
231 redirect_response);
232 } 228 }
233 229
234 void didReceiveResponse(unsigned identifier, 230 void didReceiveResponse(const blink::WebURLResponse& response) override {
235 const blink::WebURLResponse& response) override { 231 test_client()->didReceiveResponse(response);
236 test_client()->didReceiveResponse(identifier, response); 232 Base::didReceiveResponse(response);
237 Base::didReceiveResponse(identifier, response);
238 }
239
240 void didChangeResourcePriority(unsigned identifier,
241 const blink::WebURLRequest::Priority& priority,
242 int intra_priority_value) override {
243 // This is not implemented in RenderFrameImpl, so need to explicitly call
244 // into the base proxy.
245 test_client()->didChangeResourcePriority(identifier, priority,
246 intra_priority_value);
247 Base::didChangeResourcePriority(
248 identifier, priority, intra_priority_value);
249 }
250
251 void didFinishResourceLoad(blink::WebLocalFrame* frame,
252 unsigned identifier) override {
253 test_client()->didFinishResourceLoad(frame, identifier);
254 } 233 }
255 234
256 blink::WebNavigationPolicy decidePolicyForNavigation( 235 blink::WebNavigationPolicy decidePolicyForNavigation(
257 const blink::WebFrameClient::NavigationPolicyInfo& info) override { 236 const blink::WebFrameClient::NavigationPolicyInfo& info) override {
258 blink::WebNavigationPolicy policy = 237 blink::WebNavigationPolicy policy =
259 test_client()->decidePolicyForNavigation(info); 238 test_client()->decidePolicyForNavigation(info);
260 if (policy == blink::WebNavigationPolicyIgnore) 239 if (policy == blink::WebNavigationPolicyIgnore)
261 return policy; 240 return policy;
262 241
263 return Base::decidePolicyForNavigation(info); 242 return Base::decidePolicyForNavigation(info);
(...skipping 32 matching lines...) Expand 10 before | Expand all | Expand 10 after
296 return test_client()->runFileChooser(params, completion); 275 return test_client()->runFileChooser(params, completion);
297 } 276 }
298 277
299 private: 278 private:
300 DISALLOW_COPY_AND_ASSIGN(WebFrameTestProxy); 279 DISALLOW_COPY_AND_ASSIGN(WebFrameTestProxy);
301 }; 280 };
302 281
303 } // namespace test_runner 282 } // namespace test_runner
304 283
305 #endif // COMPONENTS_TEST_RUNNER_WEB_FRAME_TEST_PROXY_H_ 284 #endif // COMPONENTS_TEST_RUNNER_WEB_FRAME_TEST_PROXY_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698