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

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

Issue 2661743002: PlzNavigate: Invoke didStartProvisionalLoad() when the renderer initiates a navigation in startLoad( (Closed)
Patch Set: Remove CHECK for redirect chain as the redirects may not be populated for provisional loads for ren… Created 3 years, 10 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 78 matching lines...) Expand 10 before | Expand all | Expand 10 after
89 void loadURLExternally(const blink::WebURLRequest& request, 89 void loadURLExternally(const blink::WebURLRequest& request,
90 blink::WebNavigationPolicy policy, 90 blink::WebNavigationPolicy policy,
91 const blink::WebString& suggested_name, 91 const blink::WebString& suggested_name,
92 bool replaces_current_history_item) override { 92 bool replaces_current_history_item) override {
93 test_client()->loadURLExternally(request, policy, suggested_name, 93 test_client()->loadURLExternally(request, policy, suggested_name,
94 replaces_current_history_item); 94 replaces_current_history_item);
95 Base::loadURLExternally(request, policy, suggested_name, 95 Base::loadURLExternally(request, policy, suggested_name,
96 replaces_current_history_item); 96 replaces_current_history_item);
97 } 97 }
98 98
99 void didStartProvisionalLoad(blink::WebLocalFrame* frame) override { 99 void didStartProvisionalLoad(blink::WebDataSource* data_source) override {
100 test_client()->didStartProvisionalLoad(frame); 100 test_client()->didStartProvisionalLoad(data_source);
101 Base::didStartProvisionalLoad(frame); 101 Base::didStartProvisionalLoad(data_source);
102 } 102 }
103 103
104 void didReceiveServerRedirectForProvisionalLoad( 104 void didReceiveServerRedirectForProvisionalLoad(
105 blink::WebLocalFrame* frame) override { 105 blink::WebLocalFrame* frame) override {
106 test_client()->didReceiveServerRedirectForProvisionalLoad(frame); 106 test_client()->didReceiveServerRedirectForProvisionalLoad(frame);
107 Base::didReceiveServerRedirectForProvisionalLoad(frame); 107 Base::didReceiveServerRedirectForProvisionalLoad(frame);
108 } 108 }
109 109
110 void didFailProvisionalLoad( 110 void didFailProvisionalLoad(
111 blink::WebLocalFrame* frame, 111 blink::WebLocalFrame* frame,
(...skipping 185 matching lines...) Expand 10 before | Expand all | Expand 10 after
297 return test_client()->runFileChooser(params, completion); 297 return test_client()->runFileChooser(params, completion);
298 } 298 }
299 299
300 private: 300 private:
301 DISALLOW_COPY_AND_ASSIGN(WebFrameTestProxy); 301 DISALLOW_COPY_AND_ASSIGN(WebFrameTestProxy);
302 }; 302 };
303 303
304 } // namespace test_runner 304 } // namespace test_runner
305 305
306 #endif // COMPONENTS_TEST_RUNNER_WEB_FRAME_TEST_PROXY_H_ 306 #endif // COMPONENTS_TEST_RUNNER_WEB_FRAME_TEST_PROXY_H_
OLDNEW
« no previous file with comments | « components/test_runner/web_frame_test_client.cc ('k') | content/browser/frame_host/navigator_impl.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698