Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(205)

Side by Side Diff: chrome/browser/extensions/activity_log/activity_log_unittest.cc

Issue 503903004: Remove implicit conversions from scoped_refptr to T* in chrome/browser/extensions/activity_log/ (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: timeout insurance Created 6 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "base/command_line.h" 5 #include "base/command_line.h"
6 #include "base/memory/scoped_ptr.h" 6 #include "base/memory/scoped_ptr.h"
7 #include "base/message_loop/message_loop.h" 7 #include "base/message_loop/message_loop.h"
8 #include "base/run_loop.h" 8 #include "base/run_loop.h"
9 #include "base/synchronization/waitable_event.h" 9 #include "base/synchronization/waitable_event.h"
10 #include "chrome/browser/extensions/activity_log/activity_action_constants.h" 10 #include "chrome/browser/extensions/activity_log/activity_action_constants.h"
(...skipping 353 matching lines...) Expand 10 before | Expand all | Expand 10 after
364 Action::ACTION_API_CALL, 364 Action::ACTION_API_CALL,
365 "tabs.testMethod"); 365 "tabs.testMethod");
366 activity_log->LogAction(action); 366 activity_log->LogAction(action);
367 action = new Action(extension->id(), 367 action = new Action(extension->id(),
368 base::Time::Now(), 368 base::Time::Now(),
369 Action::ACTION_DOM_ACCESS, 369 Action::ACTION_DOM_ACCESS,
370 "document.write"); 370 "document.write");
371 action->set_page_url(GURL("http://www.google.com")); 371 action->set_page_url(GURL("http://www.google.com"));
372 372
373 activity_log->OnExtensionUninstalled( 373 activity_log->OnExtensionUninstalled(
374 NULL, extension, extensions::UNINSTALL_REASON_FOR_TESTING); 374 NULL, extension.get(), extensions::UNINSTALL_REASON_FOR_TESTING);
375 activity_log->GetFilteredActions( 375 activity_log->GetFilteredActions(
376 extension->id(), 376 extension->id(),
377 Action::ACTION_ANY, 377 Action::ACTION_ANY,
378 "", 378 "",
379 "", 379 "",
380 "", 380 "",
381 -1, 381 -1,
382 base::Bind(ActivityLogTest::RetrieveActions_LogAndFetchActions0)); 382 base::Bind(ActivityLogTest::RetrieveActions_LogAndFetchActions0));
383 } 383 }
384 384
(...skipping 19 matching lines...) Expand all
404 kExtensionId, 404 kExtensionId,
405 Action::ACTION_ANY, 405 Action::ACTION_ANY,
406 "", 406 "",
407 "", 407 "",
408 "", 408 "",
409 -1, 409 -1,
410 base::Bind(ActivityLogTest::RetrieveActions_ArgUrlApiCalls)); 410 base::Bind(ActivityLogTest::RetrieveActions_ArgUrlApiCalls));
411 } 411 }
412 412
413 } // namespace extensions 413 } // namespace extensions
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698