Tied up BlimpNavigationController to NavigationFeature

blimp/client/core/blimp_contents_impl.cc

 // 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.
 
 #include "blimp/client/core/blimp_contents_impl.h"
 
 #include "base/memory/ptr_util.h"
 #include "base/supports_user_data.h"
 #include "blimp/client/public/blimp_contents_observer.h"
 
@@ skipping 40 matching lines @@
 }
 
 void BlimpContentsImpl::AddObserver(BlimpContentsObserver* observer) {
   observers_.AddObserver(observer);
 }
 
 void BlimpContentsImpl::RemoveObserver(BlimpContentsObserver* observer) {
   observers_.RemoveObserver(observer);
 }
 
-void BlimpContentsImpl::NotifyURLLoaded(const GURL& url) {
-  FOR_EACH_OBSERVER(BlimpContentsObserver, observers_, OnURLUpdated(url));
+void BlimpContentsImpl::OnNavigationStateChanged() {
+  FOR_EACH_OBSERVER(BlimpContentsObserver, observers_,
+                    OnURLUpdated(navigation_controller_.GetURL()));

[David Trainor- moved to gerrit, 2016/07/25 15:57:31]

Shouldn't this be OnNavigationStateChanged()?  Should we remove OnURLUpdated?

[shaktisahu, 2016/07/26 01:25:27]

I think so. But then should BlimpContentsObserver have a pointer to
BlimpContents?

[David Trainor- moved to gerrit, 2016/07/27 16:55:00]

Why?  Does it need one?

 }
 
 }  // namespace client
 }  // namespace blimp