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

Side by Side Diff: trunk/src/chrome/browser/extensions/extension_gcm_app_handler_unittest.cc

Issue 369863002: Revert 281130 "Replace "external_install" boolean parameter with..." (Closed) Base URL: svn://svn.chromium.org/chrome/
Patch Set: Created 6 years, 5 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 | Annotate | Revision Log
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 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 "chrome/browser/extensions/extension_gcm_app_handler.h" 5 #include "chrome/browser/extensions/extension_gcm_app_handler.h"
6 6
7 #include <vector> 7 #include <vector>
8 8
9 #include "base/bind.h" 9 #include "base/bind.h"
10 #include "base/bind_helpers.h" 10 #include "base/bind_helpers.h"
(...skipping 315 matching lines...) Expand 10 before | Expand all | Expand 10 after
326 void DisableExtension(const Extension* extension) { 326 void DisableExtension(const Extension* extension) {
327 extension_service_->DisableExtension( 327 extension_service_->DisableExtension(
328 extension->id(), Extension::DISABLE_USER_ACTION); 328 extension->id(), Extension::DISABLE_USER_ACTION);
329 } 329 }
330 330
331 void EnableExtension(const Extension* extension) { 331 void EnableExtension(const Extension* extension) {
332 extension_service_->EnableExtension(extension->id()); 332 extension_service_->EnableExtension(extension->id());
333 } 333 }
334 334
335 void UninstallExtension(const Extension* extension) { 335 void UninstallExtension(const Extension* extension) {
336 extension_service_->UninstallExtension( 336 extension_service_->UninstallExtension(extension->id(), false, NULL);
337 extension->id(), ExtensionService::UNINSTALL_REASON_FOR_TESTING, NULL);
338 } 337 }
339 338
340 void SignIn(const std::string& username) { 339 void SignIn(const std::string& username) {
341 signin_manager_->SignIn(username); 340 signin_manager_->SignIn(username);
342 waiter_.PumpIOLoop(); 341 waiter_.PumpIOLoop();
343 } 342 }
344 343
345 void SignOut() { 344 void SignOut() {
346 signin_manager_->SignOut(signin_metrics::SIGNOUT_TEST); 345 signin_manager_->SignOut(signin_metrics::SIGNOUT_TEST);
347 waiter_.PumpIOLoop(); 346 waiter_.PumpIOLoop();
(...skipping 136 matching lines...) Expand 10 before | Expand all | Expand 10 after
484 483
485 // App handler is removed when the extension is updated to the version that 484 // App handler is removed when the extension is updated to the version that
486 // has GCM permission removed. 485 // has GCM permission removed.
487 UpdateExtension(extension, "good2.crx"); 486 UpdateExtension(extension, "good2.crx");
488 waiter()->PumpUILoop(); 487 waiter()->PumpUILoop();
489 EXPECT_TRUE(gcm_app_handler()->app_handler_count_drop_to_zero()); 488 EXPECT_TRUE(gcm_app_handler()->app_handler_count_drop_to_zero());
490 EXPECT_FALSE(HasAppHandlers(extension->id())); 489 EXPECT_FALSE(HasAppHandlers(extension->id()));
491 } 490 }
492 491
493 } // namespace extensions 492 } // namespace extensions
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698