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

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

Issue 26316005: Move out DidStartProvisionalLoad from WebContentsImpl into Navigator. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fixes based on Charlie's latest review. Created 7 years 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 | Annotate | Revision Log
« no previous file with comments | « content/browser/frame_host/navigator.h ('k') | content/browser/frame_host/navigator_impl.h » ('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_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 "content/public/browser/invalidate_type.h"
9
8 namespace content { 10 namespace content {
9 11
10 class RenderFrameHost; 12 class RenderFrameHost;
11 13
12 // A delegate API used by Navigator to notify its embedder of navigation 14 // A delegate API used by Navigator to notify its embedder of navigation
13 // related events. 15 // related events.
14 class NavigatorDelegate { 16 class NavigatorDelegate {
15 // TODO(nasko): This class will be used to dispatch notifications to 17 public:
16 // WebContentsImpl, such as DidStartProvisionalLoad and 18 // The RenderFrameHost started a provisional load for the frame
17 // NotifyNavigationStateChanged. Longer term, most of the 19 // represented by |render_frame_host|.
18 // NavigationControllerDelegate methods will likely move here. 20 virtual void DidStartProvisionalLoad(
21 RenderFrameHostImpl* render_frame_host,
22 int64 frame_id,
23 int64 parent_frame_id,
24 bool is_main_frame,
25 const GURL& validated_url,
26 bool is_error_page,
27 bool is_iframe_srcdoc) {}
28
29 // Notification to the Navigator embedder that navigation state has
30 // changed. This method corresponds to
31 // WebContents::NotifyNavigationStateChanged.
32 virtual void NotifyChangedNavigationState(InvalidateTypes changed_flags) {}
19 }; 33 };
20 34
21 } // namspace content 35 } // namspace content
22 36
23 #endif // CONTENT_BROWSER_FRAME_HOST_NAVIGATOR_DELEGATE_H_ 37 #endif // CONTENT_BROWSER_FRAME_HOST_NAVIGATOR_DELEGATE_H_
OLDNEW
« no previous file with comments | « content/browser/frame_host/navigator.h ('k') | content/browser/frame_host/navigator_impl.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698