Added a policy to control the visibility of the apps shortcut.

chrome/browser/ui/bookmarks/bookmark_context_menu_controller_unittest.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/bookmarks/bookmark_context_menu_controller.h"
 
 #include <string>
 
 #include "base/memory/scoped_ptr.h"
 #include "base/message_loop/message_loop.h"
 #include "base/strings/utf_string_conversions.h"
 #include "chrome/app/chrome_command_ids.h"
 #include "chrome/browser/bookmarks/bookmark_model_factory.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/ui/bookmarks/bookmark_utils.h"
+#include "chrome/common/pref_names.h"
+#include "chrome/test/base/testing_pref_service_syncable.h"
 #include "chrome/test/base/testing_profile.h"
 #include "components/bookmarks/browser/bookmark_model.h"
 #include "components/bookmarks/test/bookmark_test_helpers.h"
 #include "content/public/browser/page_navigator.h"
 #include "content/public/test/test_browser_thread.h"
 #include "grit/generated_resources.h"
 #include "testing/gtest/include/gtest/gtest.h"
 #include "ui/base/clipboard/clipboard.h"
 
 using base::ASCIIToUTF16;
@@ skipping 300 matching lines @@
   ASSERT_EQ(bb_node->GetChild(0)->url(), bb_node->GetChild(1)->url());
 
   controller.reset(new BookmarkContextMenuController(
       NULL, NULL, NULL, profile_.get(), NULL, nodes[0]->parent(), nodes));
   // Cut the URL.
   controller->ExecuteCommand(IDC_CUT, 0);
   ASSERT_TRUE(bb_node->GetChild(0)->is_url());
   ASSERT_TRUE(bb_node->GetChild(1)->is_folder());
   ASSERT_EQ(old_count, bb_node->child_count());
 }
+
+TEST_F(BookmarkContextMenuControllerTest,
+       ManagedShowAppsShortcutInBookmarksBar) {
+  BookmarkContextMenuController controller(
+      NULL, NULL, NULL, profile_.get(), NULL, model_->bookmark_bar_node(),
+      std::vector<const BookmarkNode*>());

[bartfab (slow), 2014/06/04 10:55:20]

Nit: #include "components/bookmarks/browser/bookmark_node.h"

[Joao da Silva, 2014/06/04 11:13:42]

Done.

+
+  // By default the pref is not managed and the command is enabled.

[bartfab (slow), 2014/06/04 10:55:20]

Nit: s/default/default,/

[Joao da Silva, 2014/06/04 11:13:42]

Done.

+  TestingPrefServiceSyncable* prefs = profile_->GetTestingPrefService();
+  EXPECT_FALSE(
+      prefs->IsManagedPreference(prefs::kShowAppsShortcutInBookmarkBar));
+  EXPECT_TRUE(
+      controller.IsCommandIdEnabled(IDC_BOOKMARK_BAR_SHOW_APPS_SHORTCUT));
+
+  // Disabling the shorcut by policy disables the command.
+  prefs->SetManagedPref(prefs::kShowAppsShortcutInBookmarkBar,
+                        new base::FundamentalValue(false));

[bartfab (slow), 2014/06/04 10:55:20]

Nit: #include "base/values.h"

[Joao da Silva, 2014/06/04 11:13:41]

Done.

+  EXPECT_FALSE(
+      controller.IsCommandIdEnabled(IDC_BOOKMARK_BAR_SHOW_APPS_SHORTCUT));
+
+  // And enabling the shortcut by policy disables the command too.
+  prefs->SetManagedPref(prefs::kShowAppsShortcutInBookmarkBar,
+                        new base::FundamentalValue(true));
+  EXPECT_FALSE(
+      controller.IsCommandIdEnabled(IDC_BOOKMARK_BAR_SHOW_APPS_SHORTCUT));
+}