Honors requests to navigate in existing frame for the root

mandoline/tab/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 "mandoline/tab/web_view_impl.h"
 
 #include "components/view_manager/public/cpp/view.h"
 #include "components/view_manager/public/cpp/view_manager.h"
 #include "mandoline/tab/frame.h"
 #include "mandoline/tab/frame_connection.h"
@@ skipping 119 matching lines @@
 
 void WebViewImpl::ProgressChanged(double progress) {
   client_->ProgressChanged(progress);
 }
 
 void WebViewImpl::RequestNavigate(Frame* source,
                                   mandoline::NavigationTargetType target_type,
                                   Frame* target_frame,
                                   mojo::URLRequestPtr request) {
   // TODO: this needs security checks.
-  if (target_type == mandoline::NAVIGATION_TARGET_TYPE_EXISTING_FRAME) {
-    if (target_frame && target_frame != frame_tree_->root() &&
-        target_frame->view()) {
+  if (target_type == mandoline::NAVIGATION_TARGET_TYPE_EXISTING_FRAME &&
+      target_frame != frame_tree_->root()) {
+    if (target_frame && target_frame->view()) {
       NavigateExistingFrame(target_frame, request.Pass());
       return;
     }
     DVLOG(1) << "RequestNavigate() targeted existing frame that doesn't exist.";
     return;
   }
   client_->TopLevelNavigate(request.Pass());
 }
 
 ////////////////////////////////////////////////////////////////////////////////
 // WebViewImpl, private:
 
 void WebViewImpl::NavigateExistingFrame(Frame* frame,
                                         mojo::URLRequestPtr request) {
   scoped_ptr<FrameConnection> frame_connection(new FrameConnection);
   mojo::ViewManagerClientPtr view_manager_client;
   frame_connection->Init(app_, request.Pass(), &view_manager_client);
   frame->view()->Embed(view_manager_client.Pass());
   FrameTreeClient* frame_tree_client = frame_connection->frame_tree_client();
   frame_tree_->CreateOrReplaceFrame(frame, frame->view(), frame_tree_client,
                                     frame_connection.Pass());
 }
 
 }  // namespace web_view