Re-order the BookmarkBarView to be behind the ToolbarView

chrome/browser/ui/views/frame/browser_view.cc

 // Copyright 2012 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 "chrome/browser/ui/views/frame/browser_view.h"
 
 #include <stdint.h>
 #include <algorithm>
 #include <utility>
 
@@ skipping 2200 matching lines @@
     needs_layout = true;
 
   return needs_layout;
 }
 
 void BrowserView::SetBookmarkBarParent(views::View* new_parent) {
   if (new_parent == this) {
     // Add it underneath |top_container_| or at the end if top container isn't
     // found.
     int top_container_index = GetIndexOf(top_container_);
+    LOG(ERROR) << " *** Adding |bookmark_bar_view_| to '" << new_parent->GetClassName() << "' top_container_index=" << top_container_index;
     if (top_container_index >= 0)
       AddChildViewAt(bookmark_bar_view_.get(), top_container_index);
     else
       AddChildView(bookmark_bar_view_.get());
   } else if (new_parent) {
     // No special stacking is required for other parents.
+    LOG(ERROR) << " *** Adding |bookmark_bar_view_| to '" << new_parent->GetClassName() << "'";
     new_parent->AddChildView(bookmark_bar_view_.get());
+    // new_parent->AddChildViewAt(bookmark_bar_view_.get(), 0);

[bruthig, 2016/03/30 19:52:43]

This is the actual desired change here.

   } else {
     // Bookmark bar is being detached from all views because it is hidden.
+    LOG(ERROR) << " *** Removing |bookmark_bar_view_| from '" << bookmark_bar_view_->parent()->GetClassName() << "'";
     bookmark_bar_view_->parent()->RemoveChildView(bookmark_bar_view_.get());
   }
+#if !defined(NDEBUG)
+  LOG(ERROR) << " *** " << PrintViewGraph(true);
+#endif
 }
 
 bool BrowserView::MaybeShowInfoBar(WebContents* contents) {
   // TODO(beng): Remove this function once the interface between
   //             InfoBarContainer, DownloadShelfView and WebContents and this
   //             view is sorted out.
   return true;
 }
 
 void BrowserView::UpdateDevToolsForContents(
@@ skipping 403 matching lines @@
 }
 
 extensions::ActiveTabPermissionGranter*
 BrowserView::GetActiveTabPermissionGranter() {
   content::WebContents* web_contents = GetActiveWebContents();
   if (!web_contents)
     return nullptr;
   return extensions::TabHelper::FromWebContents(web_contents)
       ->active_tab_permission_granter();
 }