Tied up BlimpNavigationController to NavigationFeature

blimp/client/public/contents/blimp_navigation_controller.h

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef BLIMP_CLIENT_PUBLIC_CONTENTS_BLIMP_NAVIGATION_CONTROLLER_H_
 #define BLIMP_CLIENT_PUBLIC_CONTENTS_BLIMP_NAVIGATION_CONTROLLER_H_
 
 #include "base/macros.h"
 #include "url/gurl.h"
 
 namespace blimp {
 namespace client {
 
 class BlimpContentsObserver;
 
 // The BlimpNavigationController maintains the back-forward list for a
 // BlimpContents and manages all navigation within that list.
 //
 // Each BlimpNavigationController belongs to one BlimpContents, and each
 // BlimpContents has exactly one BlimpNavigationController.
 class BlimpNavigationController {
  public:
   virtual ~BlimpNavigationController() = default;
 
   // Calls to LoadURL informs the engine that it should start navigating to the
   // provided |url|.
   virtual void LoadURL(const GURL& url) = 0;

[Kevin M, 2016/07/29 21:05:33]

Should we add parameters for tabs here too?

[shaktisahu, 2016/07/29 22:57:31]

No, the plan is that navigation controller will not know about the tab id. The
params passed to this function will change in future as more  details about the
implementation is worked out.

 
+  // Reloads the current entry.
+  virtual void Reload() = 0;
+
+  // Navigation relative to the "current entry".
+  virtual bool CanGoBack() const = 0;
+  virtual bool CanGoForward() const = 0;
+  virtual void GoBack() = 0;
+  virtual void GoForward() = 0;
+
   // Retrieves the URL of the currently selected item in the navigation list.
   virtual const GURL& GetURL() = 0;
 
  protected:
   BlimpNavigationController() {}
 
  private:
   DISALLOW_COPY_AND_ASSIGN(BlimpNavigationController);
 };
 
 }  // namespace client
 }  // namespace blimp
 
 #endif  // BLIMP_CLIENT_PUBLIC_CONTENTS_BLIMP_NAVIGATION_CONTROLLER_H_