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

components/web_view/web_view_impl.cc

Index: components/web_view/web_view_impl.cc
diff --git a/components/web_view/web_view_impl.cc b/components/web_view/web_view_impl.cc
index 84675d14f52f6670d1f8c0b979dd63f8bf2fd6c1..03075bea761f9e82964678ef6dadf65b4113ff18 100644
--- a/components/web_view/web_view_impl.cc
+++ b/components/web_view/web_view_impl.cc
@@ -175,7 +175,7 @@ void WebViewImpl::TitleChanged(const mojo::String& title) {
 }
 
 void WebViewImpl::NavigateTopLevel(Frame* source, mojo::URLRequestPtr request) {
-  client_->TopLevelNavigate(request.Pass());
+  client_->TopLevelNavigateRequest(request.Pass());
 }
 
 void WebViewImpl::CanNavigateFrame(Frame* target,
@@ -197,13 +197,14 @@ void WebViewImpl::DidCommitProvisionalLoad(Frame* frame) {
 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());
 }