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

Side by Side Diff: content/renderer/render_frame_impl.h

Issue 1480043002: [WIP] Call WebTestProxyBase::CheckDone() in didStopLoading() Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase. Created 4 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
« no previous file with comments | « components/test_runner/web_test_proxy.cc ('k') | content/renderer/render_frame_impl.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 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 CONTENT_RENDERER_RENDER_FRAME_IMPL_H_ 5 #ifndef CONTENT_RENDERER_RENDER_FRAME_IMPL_H_
6 #define CONTENT_RENDERER_RENDER_FRAME_IMPL_H_ 6 #define CONTENT_RENDERER_RENDER_FRAME_IMPL_H_
7 7
8 #include <stddef.h> 8 #include <stddef.h>
9 #include <stdint.h> 9 #include <stdint.h>
10 10
(...skipping 248 matching lines...) Expand 10 before | Expand all | Expand 10 after
259 void WasShown(); 259 void WasShown();
260 void WidgetWillClose(); 260 void WidgetWillClose();
261 261
262 // Start/Stop loading notifications. 262 // Start/Stop loading notifications.
263 // TODO(nasko): Those are page-level methods at this time and come from 263 // TODO(nasko): Those are page-level methods at this time and come from
264 // WebViewClient. We should move them to be WebFrameClient calls and put 264 // WebViewClient. We should move them to be WebFrameClient calls and put
265 // logic in the browser side to balance starts/stops. 265 // logic in the browser side to balance starts/stops.
266 // |to_different_document| will be true unless the load is a fragment 266 // |to_different_document| will be true unless the load is a fragment
267 // navigation, or triggered by history.pushState/replaceState. 267 // navigation, or triggered by history.pushState/replaceState.
268 void didStartLoading(bool to_different_document) override; 268 void didStartLoading(bool to_different_document) override;
269 void didStopLoading() override; 269 void didStopLoading(blink::WebLocalFrame*) override;
270 void didChangeLoadProgress(double load_progress) override; 270 void didChangeLoadProgress(double load_progress) override;
271 271
272 AccessibilityMode accessibility_mode() { 272 AccessibilityMode accessibility_mode() {
273 return accessibility_mode_; 273 return accessibility_mode_;
274 } 274 }
275 275
276 RendererAccessibility* renderer_accessibility() { 276 RendererAccessibility* renderer_accessibility() {
277 return renderer_accessibility_; 277 return renderer_accessibility_;
278 } 278 }
279 279
(...skipping 925 matching lines...) Expand 10 before | Expand all | Expand 10 after
1205 #endif 1205 #endif
1206 1206
1207 base::WeakPtrFactory<RenderFrameImpl> weak_factory_; 1207 base::WeakPtrFactory<RenderFrameImpl> weak_factory_;
1208 1208
1209 DISALLOW_COPY_AND_ASSIGN(RenderFrameImpl); 1209 DISALLOW_COPY_AND_ASSIGN(RenderFrameImpl);
1210 }; 1210 };
1211 1211
1212 } // namespace content 1212 } // namespace content
1213 1213
1214 #endif // CONTENT_RENDERER_RENDER_FRAME_IMPL_H_ 1214 #endif // CONTENT_RENDERER_RENDER_FRAME_IMPL_H_
OLDNEW
« no previous file with comments | « components/test_runner/web_test_proxy.cc ('k') | content/renderer/render_frame_impl.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698