mandoline: Add back/forward support and UI.

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/view_manager/public/cpp/scoped_view_ptr.h"
 #include "components/view_manager/public/cpp/view.h"
 #include "components/view_manager/public/cpp/view_tree_connection.h"
 #include "components/web_view/frame.h"
 #include "components/web_view/frame_connection.h"
 #include "components/web_view/frame_devtools_agent.h"
 #include "components/web_view/frame_tree.h"
+#include "components/web_view/navigation_entry.h"
 #include "components/web_view/pending_web_view_load.h"
 #include "mojo/application/public/cpp/application_impl.h"
 #include "mojo/converters/geometry/geometry_type_converters.h"
 #include "url/gurl.h"
 
 namespace web_view {
 namespace {
 
 bool EnableRemoteDebugging() {
   return base::CommandLine::ForCurrentProcess()->HasSwitch(
@@ skipping 42 matching lines @@
   }
 
   FrameTreeClient* frame_tree_client = frame_connection->frame_tree_client();
   const uint32_t content_handler_id = frame_connection->GetContentHandlerID();
   frame_tree_.reset(new FrameTree(content_handler_id, content_, this,
                                   frame_tree_client, frame_connection.Pass(),
                                   client_properties));
   content_->Embed(view_tree_client.Pass());
 }
 
+void WebViewImpl::UpdateBackForwardEnableState() {
+  client_->BackForwardChanged(back_list_.size() > 0, forward_list_.size() > 0);

[msw, 2015/09/04 22:05:00]

nit: use !empty?

+}
+
 ////////////////////////////////////////////////////////////////////////////////
 // WebViewImpl, WebView implementation:
 
 void WebViewImpl::LoadRequest(mojo::URLRequestPtr request) {
+  if (current_page_request_) {
+    // TODO(erg): This doesn't deal with redirect chains. If you navigate to a
+    // site, and it 300s, we put both the url which caused the 300 and the
+    // target url here, when we should not add the redirect url to the back
+    // list.
+    back_list_.push_back(current_page_request_.Pass());
+  }
+  UpdateBackForwardEnableState();
+
+  current_page_request_.reset(new NavigationEntry(request.Pass()));
+
   pending_load_.reset(new PendingWebViewLoad(this));
-  pending_load_->Init(request.Pass());
+  pending_load_->Init(current_page_request_->AsURLRequest());
 }
 
 void WebViewImpl::GetViewTreeClient(
     mojo::InterfaceRequest<mojo::ViewTreeClient> view_tree_client) {
   mojo::ViewTreeConnection::Create(this, view_tree_client.Pass());
 }
 
+void WebViewImpl::GoBack() {
+  if (back_list_.size() == 0)

[msw, 2015/09/04 22:05:00]

nit: empty

+    return;
+
+  // Take the current page request and put it in the forward list.
+  forward_list_.push_back(current_page_request_.Pass());
+
+  mojo::URLRequestPtr new_request = back_list_.back()->AsURLRequest();
+  back_list_.resize(back_list_.size() - 1);
+
+  UpdateBackForwardEnableState();
+  client_->TopLevelNavigate(new_request.Pass());
+}
+
+void WebViewImpl::GoForward() {
+  if (forward_list_.size() == 0)

[msw, 2015/09/04 22:05:00]

nit: empty

+    return;
+
+  back_list_.push_back(current_page_request_.Pass());
+
+  mojo::URLRequestPtr new_request = forward_list_.back()->AsURLRequest();
+  forward_list_.resize(forward_list_.size() - 1);
+
+  UpdateBackForwardEnableState();
+  client_->TopLevelNavigate(new_request.Pass());
+}
+
 ////////////////////////////////////////////////////////////////////////////////
 // WebViewImpl, mojo::ViewTreeDelegate implementation:
 
 void WebViewImpl::OnEmbed(mojo::View* root) {
   // We must have been granted embed root priviledges, otherwise we can't
   // Embed() in any descendants.
   DCHECK(root->connection()->IsEmbedRoot());
   root->AddObserver(this);
   root_ = root;
   content_ = root->connection()->CreateView();
@@ skipping 70 matching lines @@
 ////////////////////////////////////////////////////////////////////////////////
 // WebViewImpl, FrameDevToolsAgentDelegate implementation:
 
 void WebViewImpl::HandlePageNavigateRequest(const GURL& url) {
   mojo::URLRequestPtr request(mojo::URLRequest::New());
   request->url = url.spec();
   client_->TopLevelNavigate(request.Pass());
 }
 
 }  // namespace web_view