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

chrome/browser/extensions/activity_log/fullstream_ui_policy_unittest.cc

 // Copyright 2013 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/cancelable_callback.h"
 #include "base/command_line.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/run_loop.h"
 #include "base/strings/stringprintf.h"
 #include "base/synchronization/waitable_event.h"
@@ skipping 136 matching lines @@
     ASSERT_EQ(2, static_cast<int>(i->size()));
   }
 
   static void RetrieveActions_FetchFilteredActions300(
       scoped_ptr<std::vector<scoped_refptr<Action> > > i) {
     ASSERT_EQ(300, static_cast<int>(i->size()));
   }
 
   static void Arguments_Present(scoped_ptr<Action::ActionVector> i) {
     scoped_refptr<Action> last = i->front();
-    CheckAction(*last, "odlameecjipmbmbejkplpemijjgpljce",
-                Action::ACTION_API_CALL, "extension.connect",
-                "[\"hello\",\"world\"]", "", "", "");
+    CheckAction(*last.get(),
+                "odlameecjipmbmbejkplpemijjgpljce",
+                Action::ACTION_API_CALL,
+                "extension.connect",
+                "[\"hello\",\"world\"]",
+                "",
+                "",
+                "");
   }
 
   static void Arguments_GetTodaysActions(
       scoped_ptr<Action::ActionVector> actions) {
     ASSERT_EQ(2, static_cast<int>(actions->size()));
-    CheckAction(*actions->at(0), "punky", Action::ACTION_DOM_ACCESS, "lets",
-                "[\"vamoose\"]", "http://www.google.com/", "Page Title",
+    CheckAction(*actions->at(0).get(),
+                "punky",
+                Action::ACTION_DOM_ACCESS,
+                "lets",
+                "[\"vamoose\"]",
+                "http://www.google.com/",
+                "Page Title",
                 "http://www.arg-url.com/");
-    CheckAction(*actions->at(1), "punky", Action::ACTION_API_CALL, "brewster",
-                "[\"woof\"]", "", "Page Title", "http://www.arg-url.com/");
+    CheckAction(*actions->at(1).get(),
+                "punky",
+                Action::ACTION_API_CALL,
+                "brewster",
+                "[\"woof\"]",
+                "",
+                "Page Title",
+                "http://www.arg-url.com/");
   }
 
   static void Arguments_GetOlderActions(
       scoped_ptr<Action::ActionVector> actions) {
     ASSERT_EQ(2, static_cast<int>(actions->size()));
-    CheckAction(*actions->at(0), "punky", Action::ACTION_DOM_ACCESS, "lets",
-                "[\"vamoose\"]", "http://www.google.com/", "", "");
-    CheckAction(*actions->at(1), "punky", Action::ACTION_API_CALL, "brewster",
-                "[\"woof\"]", "", "", "");
+    CheckAction(*actions->at(0).get(),
+                "punky",
+                Action::ACTION_DOM_ACCESS,
+                "lets",
+                "[\"vamoose\"]",
+                "http://www.google.com/",
+                "",
+                "");
+    CheckAction(*actions->at(1).get(),
+                "punky",
+                Action::ACTION_API_CALL,
+                "brewster",
+                "[\"woof\"]",
+                "",
+                "",
+                "");
   }
 
   static void AllURLsRemoved(scoped_ptr<Action::ActionVector> actions) {
     ASSERT_EQ(2, static_cast<int>(actions->size()));
-    CheckAction(*actions->at(0), "punky", Action::ACTION_API_CALL, "lets",
-                "[\"vamoose\"]", "", "", "");
-    CheckAction(*actions->at(1), "punky", Action::ACTION_DOM_ACCESS, "lets",
-                "[\"vamoose\"]", "", "", "");
+    CheckAction(*actions->at(0).get(),
+                "punky",
+                Action::ACTION_API_CALL,
+                "lets",
+                "[\"vamoose\"]",
+                "",
+                "",
+                "");
+    CheckAction(*actions->at(1).get(),
+                "punky",
+                Action::ACTION_DOM_ACCESS,
+                "lets",
+                "[\"vamoose\"]",
+                "",
+                "",
+                "");
   }
 
   static void SomeURLsRemoved(scoped_ptr<Action::ActionVector> actions) {
     // These will be in the vector in reverse time order.
     ASSERT_EQ(5, static_cast<int>(actions->size()));
-    CheckAction(*actions->at(0), "punky", Action::ACTION_DOM_ACCESS, "lets",
-                "[\"vamoose\"]", "http://www.google.com/", "Google",
+    CheckAction(*actions->at(0).get(),
+                "punky",
+                Action::ACTION_DOM_ACCESS,
+                "lets",
+                "[\"vamoose\"]",
+                "http://www.google.com/",
+                "Google",
                 "http://www.args-url.com/");
-    CheckAction(*actions->at(1), "punky", Action::ACTION_DOM_ACCESS, "lets",
-                "[\"vamoose\"]", "http://www.google.com/", "Google", "");
-    CheckAction(*actions->at(2), "punky", Action::ACTION_DOM_ACCESS, "lets",
-                "[\"vamoose\"]", "", "", "");
-    CheckAction(*actions->at(3), "punky", Action::ACTION_DOM_ACCESS, "lets",
-                "[\"vamoose\"]", "", "", "http://www.google.com/");
-    CheckAction(*actions->at(4), "punky", Action::ACTION_DOM_ACCESS, "lets",
-                "[\"vamoose\"]", "", "", "");
+    CheckAction(*actions->at(1).get(),
+                "punky",
+                Action::ACTION_DOM_ACCESS,
+                "lets",
+                "[\"vamoose\"]",
+                "http://www.google.com/",
+                "Google",
+                "");
+    CheckAction(*actions->at(2).get(),
+                "punky",
+                Action::ACTION_DOM_ACCESS,
+                "lets",
+                "[\"vamoose\"]",
+                "",
+                "",
+                "");
+    CheckAction(*actions->at(3).get(),
+                "punky",
+                Action::ACTION_DOM_ACCESS,
+                "lets",
+                "[\"vamoose\"]",
+                "",
+                "",
+                "http://www.google.com/");
+    CheckAction(*actions->at(4).get(),
+                "punky",
+                Action::ACTION_DOM_ACCESS,
+                "lets",
+                "[\"vamoose\"]",
+                "",
+                "",
+                "");
   }
 
   static void CheckAction(const Action& action,
                           const std::string& expected_id,
                           const Action::ActionType& expected_type,
                           const std::string& expected_api_name,
                           const std::string& expected_args_str,
                           const std::string& expected_page_url,
                           const std::string& expected_page_title,
                           const std::string& expected_arg_url) {
@@ skipping 62 matching lines @@
     policy->DeleteDatabase();
   }
 
   static void AllActionsDeleted(scoped_ptr<Action::ActionVector> actions) {
     ASSERT_EQ(0, static_cast<int>(actions->size()));
   }
 
   static void NoActionsDeleted(scoped_ptr<Action::ActionVector> actions) {
     // These will be in the vector in reverse time order.
     ASSERT_EQ(4, static_cast<int>(actions->size()));
-    CheckAction(*actions->at(0),
+    CheckAction(*actions->at(0).get(),
                 "punky2",
                 Action::ACTION_API_CALL,
                 "lets2",
                 "[\"vamoose2\"]",
                 "http://www.google2.com/",
                 "Google2",
                 "http://www.args-url2.com/");
     ASSERT_EQ(3, actions->at(0)->action_id());
-    CheckAction(*actions->at(1),
+    CheckAction(*actions->at(1).get(),
                 "punky2",
                 Action::ACTION_API_CALL,
                 "lets2",
                 "[\"vamoose2\"]",
                 "http://www.google2.com/",
                 "Google2",
                 "http://www.args-url2.com/");
     ASSERT_EQ(4, actions->at(1)->action_id());
-    CheckAction(*actions->at(2),
+    CheckAction(*actions->at(2).get(),
                 "punky1",
                 Action::ACTION_DOM_ACCESS,
                 "lets1",
                 "[\"vamoose1\"]",
                 "http://www.google1.com/",
                 "Google1",
                 "http://www.args-url1.com/");
     ASSERT_EQ(1, actions->at(2)->action_id());
-    CheckAction(*actions->at(3),
+    CheckAction(*actions->at(3).get(),
                 "punky1",
                 Action::ACTION_DOM_ACCESS,
                 "lets1",
                 "[\"vamoose1\"]",
                 "http://www.google1.com/",
                 "Google1",
                 "http://www.args-url1.com/");
     ASSERT_EQ(2, actions->at(3)->action_id());
   }
 
   static void Action1Deleted(scoped_ptr<Action::ActionVector> actions) {
     // These will be in the vector in reverse time order.
     ASSERT_EQ(2, static_cast<int>(actions->size()));
-    CheckAction(*actions->at(0),
+    CheckAction(*actions->at(0).get(),
                 "punky2",
                 Action::ACTION_API_CALL,
                 "lets2",
                 "[\"vamoose2\"]",
                 "http://www.google2.com/",
                 "Google2",
                 "http://www.args-url2.com/");
     ASSERT_EQ(3, actions->at(0)->action_id());
-    CheckAction(*actions->at(1),
+    CheckAction(*actions->at(1).get(),
                 "punky2",
                 Action::ACTION_API_CALL,
                 "lets2",
                 "[\"vamoose2\"]",
                 "http://www.google2.com/",
                 "Google2",
                 "http://www.args-url2.com/");
     ASSERT_EQ(4, actions->at(1)->action_id());
   }
 
   static void Action2Deleted(scoped_ptr<Action::ActionVector> actions) {
     // These will be in the vector in reverse time order.
     ASSERT_EQ(2, static_cast<int>(actions->size()));
-    CheckAction(*actions->at(0),
+    CheckAction(*actions->at(0).get(),
                 "punky1",
                 Action::ACTION_DOM_ACCESS,
                 "lets1",
                 "[\"vamoose1\"]",
                 "http://www.google1.com/",
                 "Google1",
                 "http://www.args-url1.com/");
     ASSERT_EQ(1, actions->at(0)->action_id());
-    CheckAction(*actions->at(1),
+    CheckAction(*actions->at(1).get(),
                 "punky1",
                 Action::ACTION_DOM_ACCESS,
                 "lets1",
                 "[\"vamoose1\"]",
                 "http://www.google1.com/",
                 "Google1",
                 "http://www.args-url1.com/");
     ASSERT_EQ(2, actions->at(1)->action_id());
   }
 
@@ skipping 614 matching lines @@
   action_ids.push_back(3);
   action_ids.push_back(4);
   CheckRemoveActions(
       policy, action_ids, base::Bind(&FullStreamUIPolicyTest::Action2Deleted));
   action_ids.clear();
 
   policy->Close();
 }
 
 }  // namespace extensions