Remove implicit conversions from scoped_refptr to T* in chrome/browser/extensions/

chrome/browser/extensions/extension_context_menu_model_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/extensions/extension_context_menu_model.h"
 
 #include "chrome/browser/extensions/api/extension_action/extension_action_api.h"
 #include "chrome/browser/extensions/extension_service.h"
 #include "chrome/browser/extensions/extension_service_test_base.h"
 #include "chrome/browser/extensions/menu_manager.h"
@@ skipping 154 matching lines @@
   MenuManager* manager = static_cast<MenuManager*>(
       (MenuManagerFactory::GetInstance()->SetTestingFactoryAndUse(
           profile(),
           &MenuManagerFactory::BuildServiceInstanceForTesting)));
   ASSERT_TRUE(manager);
 
   scoped_refptr<ExtensionContextMenuModel> menu(
       new ExtensionContextMenuModel(extension.get(), browser.get()));
 
   // There should be no extension items yet.
-  EXPECT_EQ(0, CountExtensionItems(menu));
+  EXPECT_EQ(0, CountExtensionItems(menu.get()));
 
   // Add a browser action menu item for |extension| to |manager|.
   AddContextItemAndRefreshModel(
-      manager, extension.get(), MenuItem::BROWSER_ACTION, menu);
+      manager, extension.get(), MenuItem::BROWSER_ACTION, menu.get());
 
   // Since |extension| has a page action, the browser action menu item should
   // not be present.
-  EXPECT_EQ(0, CountExtensionItems(menu));
+  EXPECT_EQ(0, CountExtensionItems(menu.get()));
 
   // Add a page action menu item and reset the context menu.
   AddContextItemAndRefreshModel(
-      manager, extension.get(), MenuItem::PAGE_ACTION, menu);
+      manager, extension.get(), MenuItem::PAGE_ACTION, menu.get());
 
   // The page action item should be present because |extension| has a page
   // action.
-  EXPECT_EQ(1, CountExtensionItems(menu));
+  EXPECT_EQ(1, CountExtensionItems(menu.get()));
 
   // Create more page action items to test top level menu item limitations.
   for (int i = 0; i < api::context_menus::ACTION_MENU_TOP_LEVEL_LIMIT; ++i)
     AddContextItemAndRefreshModel(
-        manager, extension.get(), MenuItem::PAGE_ACTION, menu);
+        manager, extension.get(), MenuItem::PAGE_ACTION, menu.get());
 
   // The menu should only have a limited number of extension items, since they
   // are all top level items, and we limit the number of top level extension
   // items.
   EXPECT_EQ(api::context_menus::ACTION_MENU_TOP_LEVEL_LIMIT,
-            CountExtensionItems(menu));
+            CountExtensionItems(menu.get()));
 
   AddContextItemAndRefreshModel(
-      manager, extension.get(), MenuItem::PAGE_ACTION, menu);
+      manager, extension.get(), MenuItem::PAGE_ACTION, menu.get());
 
   // Adding another top level item should not increase the count.
   EXPECT_EQ(api::context_menus::ACTION_MENU_TOP_LEVEL_LIMIT,
-            CountExtensionItems(menu));
+            CountExtensionItems(menu.get()));
 }
 
 // Test that the "show" and "hide" menu items appear correctly in the extension
 // context menu.
 TEST_F(ExtensionContextMenuModelTest, ExtensionContextMenuShowAndHide) {
   InitializeEmptyExtensionService();
   scoped_refptr<const Extension> page_action =
       BuildExtension("page_action_extension", manifest_keys::kPageAction);
   ASSERT_TRUE(page_action.get());
   scoped_refptr<const Extension> browser_action =
@@ skipping 46 matching lines @@
   // Hide the browser action. This should mean the string is "show".
   ExtensionActionAPI::SetBrowserActionVisibility(
       prefs, browser_action->id(), false);
   menu = new ExtensionContextMenuModel(browser_action.get(), browser.get());
   index = GetCommandIndex(menu, visibility_command);
   EXPECT_NE(-1, index);
   EXPECT_EQ(show_string, menu->GetLabelAt(index));
 }
 
 }  // namespace extensions