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

Side by Side Diff: blimp/client/public/blimp_navigation_controller.h

Issue 2058263002: Tied up BlimpNavigationController to NavigationFeature (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@blimp_core
Patch Set: Addressed code review comments Created 4 years, 5 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 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 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 BLIMP_CLIENT_PUBLIC_BLIMP_NAVIGATION_CONTROLLER_H_ 5 #ifndef BLIMP_CLIENT_PUBLIC_BLIMP_NAVIGATION_CONTROLLER_H_
6 #define BLIMP_CLIENT_PUBLIC_BLIMP_NAVIGATION_CONTROLLER_H_ 6 #define BLIMP_CLIENT_PUBLIC_BLIMP_NAVIGATION_CONTROLLER_H_
7 7
8 #include "base/macros.h" 8 #include "base/macros.h"
9 #include "url/gurl.h" 9 #include "url/gurl.h"
10 10
11 namespace blimp { 11 namespace blimp {
12 namespace client { 12 namespace client {
13 13
14 class BlimpContentsObserver; 14 class BlimpContentsObserver;
15 15
16 // The BlimpNavigationController maintains the back-forward list for a 16 // The BlimpNavigationController maintains the back-forward list for a
17 // BlimpContents and manages all navigation within that list. 17 // BlimpContents and manages all navigation within that list.
18 // 18 //
19 // Each BlimpNavigationController belongs to one BlimpContents, and each 19 // Each BlimpNavigationController belongs to one BlimpContents, and each
20 // BlimpContents has exactly one BlimpNavigationController. 20 // BlimpContents has exactly one BlimpNavigationController.
21 class BlimpNavigationController { 21 class BlimpNavigationController {
22 public: 22 public:
23 virtual ~BlimpNavigationController() = default; 23 virtual ~BlimpNavigationController() = default;
24 24
25 // Calls to LoadURL informs the engine that it should start navigating to the 25 // Calls to LoadURL informs the engine that it should start navigating to the
26 // provided |url|. 26 // provided |url|.
27 virtual void LoadURL(const GURL& url) = 0; 27 virtual void LoadURL(const GURL& url) = 0;
28 28
29 virtual void Reload() = 0;
David Trainor- moved to gerrit 2016/07/25 15:57:32 Do these require documentation?
shaktisahu 2016/07/26 01:25:27 Done.
30 virtual bool CanGoBack() const = 0;
31 virtual bool CanGoForward() const = 0;
32 virtual void GoBack() = 0;
33 virtual void GoForward() = 0;
34
29 // Retrieves the URL of the currently selected item in the navigation list. 35 // Retrieves the URL of the currently selected item in the navigation list.
30 virtual const GURL& GetURL() = 0; 36 virtual const GURL& GetURL() = 0;
31 37
32 protected: 38 protected:
33 BlimpNavigationController() {} 39 BlimpNavigationController() {}
34 40
35 private: 41 private:
36 DISALLOW_COPY_AND_ASSIGN(BlimpNavigationController); 42 DISALLOW_COPY_AND_ASSIGN(BlimpNavigationController);
37 }; 43 };
38 44
39 } // namespace client 45 } // namespace client
40 } // namespace blimp 46 } // namespace blimp
41 47
42 #endif // BLIMP_CLIENT_PUBLIC_BLIMP_NAVIGATION_CONTROLLER_H_ 48 #endif // BLIMP_CLIENT_PUBLIC_BLIMP_NAVIGATION_CONTROLLER_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698