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

Side by Side Diff: content/public/browser/web_contents_observer.h

Issue 868673002: OOPIF: report both original and destination rfh in AboutToNavigateRenderFrame. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rebaselined Created 5 years, 11 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 | « content/browser/web_contents/web_contents_impl.cc ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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_PUBLIC_BROWSER_WEB_CONTENTS_OBSERVER_H_ 5 #ifndef CONTENT_PUBLIC_BROWSER_WEB_CONTENTS_OBSERVER_H_
6 #define CONTENT_PUBLIC_BROWSER_WEB_CONTENTS_OBSERVER_H_ 6 #define CONTENT_PUBLIC_BROWSER_WEB_CONTENTS_OBSERVER_H_
7 7
8 #include "base/process/kill.h" 8 #include "base/process/kill.h"
9 #include "base/process/process_handle.h" 9 #include "base/process/process_handle.h"
10 #include "content/common/content_export.h" 10 #include "content/common/content_export.h"
(...skipping 100 matching lines...) Expand 10 before | Expand all | Expand 10 after
111 virtual void RenderViewHostChanged(RenderViewHost* old_host, 111 virtual void RenderViewHostChanged(RenderViewHost* old_host,
112 RenderViewHost* new_host) {} 112 RenderViewHost* new_host) {}
113 113
114 // This method is invoked after the WebContents decides which RenderFrameHost 114 // This method is invoked after the WebContents decides which RenderFrameHost
115 // to use for the next browser-initiated navigation, but before the navigation 115 // to use for the next browser-initiated navigation, but before the navigation
116 // starts. It is not called for most renderer-initiated navigations, and it 116 // starts. It is not called for most renderer-initiated navigations, and it
117 // does not guarantee that the navigation will commit (e.g., 204s, downloads). 117 // does not guarantee that the navigation will commit (e.g., 204s, downloads).
118 // 118 //
119 // DEPRECATED. This method is difficult to use correctly and should be 119 // DEPRECATED. This method is difficult to use correctly and should be
120 // removed. TODO(creis): Remove in http://crbug.com/424641. 120 // removed. TODO(creis): Remove in http://crbug.com/424641.
121 virtual void AboutToNavigateRenderFrame(RenderFrameHost* render_frame_host) {} 121 virtual void AboutToNavigateRenderFrame(RenderFrameHost* old_host,
122 RenderFrameHost* new_host) {}
122 123
123 // This method is invoked after the browser process starts a navigation to a 124 // This method is invoked after the browser process starts a navigation to a
124 // pending NavigationEntry. It is not called for renderer-initiated 125 // pending NavigationEntry. It is not called for renderer-initiated
125 // navigations unless they are sent to the browser process via OpenURL. It may 126 // navigations unless they are sent to the browser process via OpenURL. It may
126 // be called multiple times for a given navigation, such as a typed URL 127 // be called multiple times for a given navigation, such as a typed URL
127 // followed by a cross-process client or server redirect. 128 // followed by a cross-process client or server redirect.
128 virtual void DidStartNavigationToPendingEntry( 129 virtual void DidStartNavigationToPendingEntry(
129 const GURL& url, 130 const GURL& url,
130 NavigationController::ReloadType reload_type) {} 131 NavigationController::ReloadType reload_type) {}
131 132
(...skipping 240 matching lines...) Expand 10 before | Expand all | Expand 10 after
372 void ResetWebContents(); 373 void ResetWebContents();
373 374
374 WebContentsImpl* web_contents_; 375 WebContentsImpl* web_contents_;
375 376
376 DISALLOW_COPY_AND_ASSIGN(WebContentsObserver); 377 DISALLOW_COPY_AND_ASSIGN(WebContentsObserver);
377 }; 378 };
378 379
379 } // namespace content 380 } // namespace content
380 381
381 #endif // CONTENT_PUBLIC_BROWSER_WEB_CONTENTS_OBSERVER_H_ 382 #endif // CONTENT_PUBLIC_BROWSER_WEB_CONTENTS_OBSERVER_H_
OLDNEW
« no previous file with comments | « content/browser/web_contents/web_contents_impl.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698