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

Side by Side Diff: content/browser/frame_host/navigator_delegate.h

Issue 1374053002: Remove AboutToNavigateRenderFrame, issue custom notification for DevTools. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rebased, win test fix Created 5 years, 2 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 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_BROWSER_FRAME_HOST_NAVIGATOR_DELEGATE_H_ 5 #ifndef CONTENT_BROWSER_FRAME_HOST_NAVIGATOR_DELEGATE_H_
6 #define CONTENT_BROWSER_FRAME_HOST_NAVIGATOR_DELEGATE_H_ 6 #define CONTENT_BROWSER_FRAME_HOST_NAVIGATOR_DELEGATE_H_
7 7
8 #include "base/strings/string16.h" 8 #include "base/strings/string16.h"
9 #include "content/public/browser/invalidate_type.h" 9 #include "content/public/browser/invalidate_type.h"
10 #include "content/public/browser/navigation_controller.h" 10 #include "content/public/browser/navigation_controller.h"
(...skipping 79 matching lines...) Expand 10 before | Expand all | Expand 10 after
90 const FrameHostMsg_DidCommitProvisionalLoad_Params& params) {} 90 const FrameHostMsg_DidCommitProvisionalLoad_Params& params) {}
91 91
92 virtual void SetMainFrameMimeType(const std::string& mime_type) {} 92 virtual void SetMainFrameMimeType(const std::string& mime_type) {}
93 virtual bool CanOverscrollContent() const; 93 virtual bool CanOverscrollContent() const;
94 94
95 // Notification to the Navigator embedder that navigation state has 95 // Notification to the Navigator embedder that navigation state has
96 // changed. This method corresponds to 96 // changed. This method corresponds to
97 // WebContents::NotifyNavigationStateChanged. 97 // WebContents::NotifyNavigationStateChanged.
98 virtual void NotifyChangedNavigationState(InvalidateTypes changed_flags) {} 98 virtual void NotifyChangedNavigationState(InvalidateTypes changed_flags) {}
99 99
100 // Notifies the Navigator embedder that it is beginning to navigate a frame.
101 virtual void AboutToNavigateRenderFrame(
102 RenderFrameHostImpl* old_host,
103 RenderFrameHostImpl* new_host) {}
104
105 // Notifies the Navigator embedder that a navigation to the pending 100 // Notifies the Navigator embedder that a navigation to the pending
106 // NavigationEntry has started in the browser process. 101 // NavigationEntry has started in the browser process.
107 virtual void DidStartNavigationToPendingEntry( 102 virtual void DidStartNavigationToPendingEntry(
108 const GURL& url, 103 const GURL& url,
109 NavigationController::ReloadType reload_type) {} 104 NavigationController::ReloadType reload_type) {}
110 105
111 // Opens a URL with the given parameters. See PageNavigator::OpenURL, which 106 // Opens a URL with the given parameters. See PageNavigator::OpenURL, which
112 // this forwards to. 107 // this forwards to.
113 virtual void RequestOpenURL(RenderFrameHostImpl* render_frame_host, 108 virtual void RequestOpenURL(RenderFrameHostImpl* render_frame_host,
114 const OpenURLParams& params) {} 109 const OpenURLParams& params) {}
(...skipping 13 matching lines...) Expand all
128 // throbber stopping. 123 // throbber stopping.
129 virtual void DidStopLoading() {} 124 virtual void DidStopLoading() {}
130 125
131 // The load progress was changed. 126 // The load progress was changed.
132 virtual void DidChangeLoadProgress() {} 127 virtual void DidChangeLoadProgress() {}
133 }; 128 };
134 129
135 } // namspace content 130 } // namspace content
136 131
137 #endif // CONTENT_BROWSER_FRAME_HOST_NAVIGATOR_DELEGATE_H_ 132 #endif // CONTENT_BROWSER_FRAME_HOST_NAVIGATOR_DELEGATE_H_
OLDNEW
« no previous file with comments | « content/browser/devtools/render_frame_devtools_agent_host.cc ('k') | content/browser/frame_host/navigator_impl.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698