Tied up BlimpNavigationController to NavigationFeature

blimp/client/core/contents/blimp_navigation_controller_delegate.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_CORE_CONTENTS_BLIMP_NAVIGATION_CONTROLLER_DELEGATE_H_
 #define BLIMP_CLIENT_CORE_CONTENTS_BLIMP_NAVIGATION_CONTROLLER_DELEGATE_H_
 
 #include "base/macros.h"
 #include "url/gurl.h"
 
 namespace blimp {
 namespace client {
 
 // Interface for objects embedding a BlimpNavigationControllerImpl to provide
 // the functionality BlimpNavigationControllerImpl needs.
 class BlimpNavigationControllerDelegate {
  public:
   virtual ~BlimpNavigationControllerDelegate() = default;
 
-  // Invoked whenever a new URL has been loaded.
-  virtual void NotifyURLLoaded(const GURL& url) = 0;
+  // Informs the delegate that navigation state has been changed.

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

was/has changed

no need to use the passive voice with "been"

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

Done.

+  virtual void OnNavigationStateChanged() = 0;
 
  protected:
   BlimpNavigationControllerDelegate() {}
 
  private:
   DISALLOW_COPY_AND_ASSIGN(BlimpNavigationControllerDelegate);
 };
 
 }  // namespace client
 }  // namespace blimp
 
 #endif  // BLIMP_CLIENT_CORE_CONTENTS_BLIMP_NAVIGATION_CONTROLLER_DELEGATE_H_