mandoline: Make sure omnibox gets updated when navigating back/forwards.

components/web_view/web_view_impl.cc

 // Copyright 2015 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 "components/web_view/web_view_impl.h"
 
 #include "base/command_line.h"
 #include "components/devtools_service/public/cpp/switches.h"
 #include "components/mus/public/cpp/scoped_view_ptr.h"
 #include "components/mus/public/cpp/view.h"
@@ skipping 157 matching lines @@
 
 void WebViewImpl::LoadingStateChanged(bool loading, double progress) {
   client_->LoadingStateChanged(loading, progress);
 }
 
 void WebViewImpl::TitleChanged(const mojo::String& title) {
   client_->TitleChanged(title);
 }
 
 void WebViewImpl::NavigateTopLevel(Frame* source, mojo::URLRequestPtr request) {
-  client_->TopLevelNavigate(request.Pass());
+  client_->TopLevelNavigateRequest(request.Pass());
 }
 
 void WebViewImpl::CanNavigateFrame(Frame* target,
                                    mojo::URLRequestPtr request,
                                    const CanNavigateFrameCallback& callback) {
   FrameConnection::CreateConnectionForCanNavigateFrame(
       app_, target, request.Pass(), callback);
 }
 
 void WebViewImpl::DidStartNavigation(Frame* frame) {}
 
 void WebViewImpl::DidCommitProvisionalLoad(Frame* frame) {
   navigation_controller_.FrameDidCommitProvisionalLoad(frame);
 }
 
 ////////////////////////////////////////////////////////////////////////////////
 // WebViewImpl, FrameDevToolsAgentDelegate implementation:
 
 void WebViewImpl::HandlePageNavigateRequest(const GURL& url) {
   mojo::URLRequestPtr request(mojo::URLRequest::New());
   request->url = url.spec();
-  client_->TopLevelNavigate(request.Pass());
+  client_->TopLevelNavigateRequest(request.Pass());
 }
 
 ////////////////////////////////////////////////////////////////////////////////
 // WebViewImpl, NavigationControllerDelegate implementation:
 
 void WebViewImpl::OnNavigate(mojo::URLRequestPtr request) {
+  client_->TopLevelNavigationStarted(request->url);

[sky, 2015/09/21 14:59:46]

Are you sure you don't want to notify from OnLoad?

[sadrul, 2015/09/23 04:32:14]

That looks like a better place. Made this change. Thanks

   pending_load_.reset(new PendingWebViewLoad(this));
   pending_load_->Init(request.Pass());
 }
 
 void WebViewImpl::OnDidNavigate() {
   client_->BackForwardChanged(navigation_controller_.CanGoBack()
                                   ? ButtonState::BUTTON_STATE_ENABLED
                                   : ButtonState::BUTTON_STATE_DISABLED,
                               navigation_controller_.CanGoForward()
                                   ? ButtonState::BUTTON_STATE_ENABLED
                                   : ButtonState::BUTTON_STATE_DISABLED);
 }
 
 }  // namespace web_view