Test that simulates touch exploration and checks speech on tabs and also the empty bookmark bar.

chrome/browser/ui/views/bookmarks/bookmark_bar_view.cc

 // Copyright (c) 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/bookmarks/bookmark_bar_view.h"
 
 #include <algorithm>
 #include <limits>
 #include <string>
 #include <vector>
@@ skipping 608 matching lines @@
 }
 
 views::MenuItemView* BookmarkBarView::GetContextMenu() {
   return bookmark_menu_ ? bookmark_menu_->context_menu() : NULL;
 }
 
 views::MenuItemView* BookmarkBarView::GetDropMenu() {
   return bookmark_drop_menu_ ? bookmark_drop_menu_->menu() : NULL;
 }
 
+views::View* BookmarkBarView::GetBookmarkBarInstructionsView() {
+  if (instructions_ && instructions_->visible())

[sky, 2014/08/08 23:30:03]

Is there a reason for the NULL check? instructions_ is never set to NULL.

[evy, 2014/08/09 02:26:52]

Is there a chance that code could be added in the future to set it to NULL? Or
should that never happen (it should only be set to not visible, and the code
assumes that/tests ensure that)

[sky, 2014/08/11 16:26:58]

If we start allowing it to be NULL we would have to revisit any existing code.

[evy, 2014/08/11 17:19:07]

Done.

+    return instructions_;
+  return NULL;
+}
+
 void BookmarkBarView::StopThrobbing(bool immediate) {
   if (!throbbing_view_)
     return;
 
   // If not immediate, cycle through 2 more complete cycles.
   throbbing_view_->StartThrobbing(immediate ? 0 : 4);
   throbbing_view_ = NULL;
 }
 
 // static
@@ skipping 1318 matching lines @@
   DCHECK(apps_page_shortcut_);
   // Only perform layout if required.
   bool visible = chrome::ShouldShowAppsShortcutInBookmarkBar(
       browser_->profile(), browser_->host_desktop_type());
   if (apps_page_shortcut_->visible() == visible)
     return;
   apps_page_shortcut_->SetVisible(visible);
   UpdateBookmarksSeparatorVisibility();
   Layout();
 }