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

components/web_view/pending_web_view_load.h

 // 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.
 
 #ifndef COMPONENTS_WEB_VIEW_PENDING_WEB_VIEW_LOAD_H_
 #define COMPONENTS_WEB_VIEW_PENDING_WEB_VIEW_LOAD_H_
 
+#include <string>
+
 #include "base/memory/scoped_ptr.h"
 #include "mojo/services/network/public/interfaces/url_loader.mojom.h"
 
 namespace web_view {
 
 class FrameConnection;
 class WebViewImpl;
 
 // PendingWebViewLoad holds the state necessary to service a load of the main
 // frame. Once the necessary state has been obtained the load is started.
 class PendingWebViewLoad {
  public:
   explicit PendingWebViewLoad(WebViewImpl* web_view);
   ~PendingWebViewLoad();
 
   void Init(mojo::URLRequestPtr request);
 
   scoped_ptr<FrameConnection> frame_connection() {
     return frame_connection_.Pass();
   }
 
   bool is_content_handler_id_valid() const {
     return is_content_handler_id_valid_;
   }
 
+  const std::string& pending_url() const { return pending_url_; }
+
  private:
   void OnGotContentHandlerID();
 
   WebViewImpl* web_view_;
 
+  std::string pending_url_;

[sky, 2015/09/23 15:31:44]

Can we make this a GURL?

[sadrul, 2015/09/27 05:29:39]

Done. (the new TopLevelNavigationStarted() interface is still using strings
since it's in a mojom. I briefly considered using a URLRequest there, but
decided against it).

   bool is_content_handler_id_valid_;
 
   scoped_ptr<FrameConnection> frame_connection_;
 
   DISALLOW_COPY_AND_ASSIGN(PendingWebViewLoad);
 };
 
 }  // namespace web_view
 
 #endif  // COMPONENTS_WEB_VIEW_PENDING_WEB_VIEW_LOAD_H_