[Extensions Toolbar] Component extensions default to the front of the toolbar

chrome/browser/extensions/extension_toolbar_model_unittest.cc

 // Copyright 2014 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 "base/files/file_util.h"
 #include "base/macros.h"
 #include "base/memory/ref_counted.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/strings/stringprintf.h"
 #include "chrome/browser/extensions/api/extension_action/extension_action_api.h"
@@ skipping 10 matching lines @@
 #include "chrome/browser/sessions/session_tab_helper.h"
 #include "chrome/common/extensions/api/extension_action/action_info.h"
 #include "components/crx_file/id_util.h"
 #include "content/public/test/test_renderer_host.h"
 #include "content/public/test/web_contents_tester.h"
 #include "extensions/browser/extension_prefs.h"
 #include "extensions/browser/extension_registry.h"
 #include "extensions/browser/extension_system.h"
 #include "extensions/browser/test_extension_registry_observer.h"
 #include "extensions/common/extension.h"
+#include "extensions/common/extension_builder.h"
 #include "extensions/common/feature_switch.h"
+#include "extensions/common/value_builder.h"
 
 #if defined(USE_AURA)
 #include "ui/aura/env.h"
 #endif
 
 namespace extensions {
 
 namespace {
 
 // A simple observer that tracks the number of times certain events occur.
@@ skipping 1080 matching lines @@
   base::RunLoop().RunUntilIdle();
 
   // The new order should be reflected in the model.
   EXPECT_EQ(browser_action_b(), GetExtensionAtIndex(0));
   EXPECT_EQ(browser_action_c(), GetExtensionAtIndex(1));
   EXPECT_EQ(browser_action_a(), GetExtensionAtIndex(2));
   EXPECT_EQ(inserted_and_removed_difference,
             observer()->inserted_count() - observer()->removed_count());
 }
 
+TEST_F(ExtensionToolbarModelUnitTest, ComponentExtesionsAddedToEnd) {
+  Init();
+
+  ASSERT_TRUE(AddBrowserActionExtensions());
+
+  EXPECT_EQ(browser_action_a(), GetExtensionAtIndex(0));
+  EXPECT_EQ(browser_action_b(), GetExtensionAtIndex(1));
+  EXPECT_EQ(browser_action_c(), GetExtensionAtIndex(2));
+
+  const char kName[] = "component";
+  DictionaryBuilder manifest;
+  manifest.Set("name", kName)
+          .Set("description", "An extension")
+          .Set("manifest_version", 2)
+          .Set("version", "1.0.0")
+          .Set("browser_action", DictionaryBuilder().Pass());
+  scoped_refptr<const Extension> component_extension =
+      ExtensionBuilder().SetManifest(manifest.Pass())
+                        .SetID(crx_file::id_util::GenerateId(kName))
+                        .SetLocation(Manifest::COMPONENT)
+                        .Build();
+  service()->AddExtension(component_extension.get());
+
+  EXPECT_EQ(component_extension.get(), GetExtensionAtIndex(0));
+  EXPECT_EQ(browser_action_a(), GetExtensionAtIndex(1));
+  EXPECT_EQ(browser_action_b(), GetExtensionAtIndex(2));
+  EXPECT_EQ(browser_action_c(), GetExtensionAtIndex(3));
+}
+
 }  // namespace extensions