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

Side by Side Diff: ios/web/navigation/navigation_manager_delegate.h

Issue 2741413007: Refactoring Reload in NavigationManager and CRWWebController. (Closed)
Patch Set: Addressed high level comments Created 3 years, 9 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 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 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 IOS_WEB_NAVIGATION_NAVIGATION_MANAGER_DELEGATE_H_ 5 #ifndef IOS_WEB_NAVIGATION_NAVIGATION_MANAGER_DELEGATE_H_
6 #define IOS_WEB_NAVIGATION_NAVIGATION_MANAGER_DELEGATE_H_ 6 #define IOS_WEB_NAVIGATION_NAVIGATION_MANAGER_DELEGATE_H_
7 7
8 #include <stddef.h> 8 #include <stddef.h>
9 9
10 #import "ios/web/public/navigation_manager.h" 10 #import "ios/web/public/navigation_manager.h"
11 11
12 @class CRWWebController;
13
12 namespace web { 14 namespace web {
13 15
14 struct LoadCommittedDetails; 16 struct LoadCommittedDetails;
15 class WebState; 17 class WebState;
16 18
17 // Delegate for NavigationManager to hand off parts of the navigation flow. 19 // Delegate for NavigationManager to hand off parts of the navigation flow.
18 class NavigationManagerDelegate { 20 class NavigationManagerDelegate {
19 public: 21 public:
20 virtual ~NavigationManagerDelegate() {} 22 virtual ~NavigationManagerDelegate() {}
21 23
(...skipping 10 matching lines...) Expand all
32 34
33 // Informs the delegate that a navigation item has been changed. 35 // Informs the delegate that a navigation item has been changed.
34 virtual void OnNavigationItemChanged() = 0; 36 virtual void OnNavigationItemChanged() = 0;
35 37
36 // Informs the delegate that a navigation item has been commited. 38 // Informs the delegate that a navigation item has been commited.
37 virtual void OnNavigationItemCommitted( 39 virtual void OnNavigationItemCommitted(
38 const LoadCommittedDetails& load_details) = 0; 40 const LoadCommittedDetails& load_details) = 0;
39 41
40 // Returns the WebState associated with this delegate. 42 // Returns the WebState associated with this delegate.
41 virtual WebState* GetWebState() = 0; 43 virtual WebState* GetWebState() = 0;
44
45 // Returns the CRWWebController associated with this delegate.
46 virtual CRWWebController* GetWebController() = 0;
kkhorimoto 2017/03/15 17:16:12 This is no longer needed, right?
liaoyuke 2017/03/15 17:32:39 Done.
42 }; 47 };
43 48
44 } // namespace web 49 } // namespace web
45 50
46 #endif // IOS_WEB_NAVIGATION_NAVIGATION_MANAGER_DELEGATE_H_ 51 #endif // IOS_WEB_NAVIGATION_NAVIGATION_MANAGER_DELEGATE_H_
OLDNEW
« no previous file with comments | « ios/chrome/browser/ui/browser_view_controller.mm ('k') | ios/web/navigation/navigation_manager_impl.mm » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698