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

Side by Side Diff: chrome/browser/extensions/extension_service_unittest.cc

Issue 17038002: Separate the NTP app ordering from the app list app ordering (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: change from notification to observer Created 7 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) 2013 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2013 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_service_unittest.h" 5 #include "chrome/browser/extensions/extension_service_unittest.h"
6 6
7 #include <algorithm> 7 #include <algorithm>
8 #include <set> 8 #include <set>
9 #include <vector> 9 #include <vector>
10 10
(...skipping 1618 matching lines...) Expand 10 before | Expand all | Expand 10 after
1629 virtual void OnExtensionLoaded(const Extension* extension) OVERRIDE { 1629 virtual void OnExtensionLoaded(const Extension* extension) OVERRIDE {
1630 } 1630 }
1631 1631
1632 virtual void OnExtensionUnloaded(const Extension* extension) OVERRIDE { 1632 virtual void OnExtensionUnloaded(const Extension* extension) OVERRIDE {
1633 } 1633 }
1634 1634
1635 virtual void OnExtensionUninstalled(const Extension* extension) OVERRIDE { 1635 virtual void OnExtensionUninstalled(const Extension* extension) OVERRIDE {
1636 last_extension_uninstalled = extension->id(); 1636 last_extension_uninstalled = extension->id();
1637 } 1637 }
1638 1638
1639 virtual void OnAppsReordered() OVERRIDE { 1639 virtual void OnAppListReordered() OVERRIDE {
1640 } 1640 }
1641 1641
1642 virtual void OnAppInstalledToAppList( 1642 virtual void OnAppInstalledToAppList(
1643 const std::string& extension_id) OVERRIDE { 1643 const std::string& extension_id) OVERRIDE {
1644 } 1644 }
1645 1645
1646 virtual void OnShutdown() OVERRIDE { 1646 virtual void OnShutdown() OVERRIDE {
1647 } 1647 }
1648 1648
1649 std::string last_extension_installed; 1649 std::string last_extension_installed;
(...skipping 4721 matching lines...) Expand 10 before | Expand all | Expand 10 after
6371 EXPECT_TRUE(notifications.CheckNotifications( 6371 EXPECT_TRUE(notifications.CheckNotifications(
6372 chrome::NOTIFICATION_EXTENSION_INSTALLED)); 6372 chrome::NOTIFICATION_EXTENSION_INSTALLED));
6373 6373
6374 EXPECT_TRUE(service_->GetInstalledExtension(id)); 6374 EXPECT_TRUE(service_->GetInstalledExtension(id));
6375 EXPECT_FALSE(service_->extensions()->Contains(id)); 6375 EXPECT_FALSE(service_->extensions()->Contains(id));
6376 EXPECT_TRUE(service_->blacklisted_extensions()->Contains(id)); 6376 EXPECT_TRUE(service_->blacklisted_extensions()->Contains(id));
6377 EXPECT_TRUE(service_->extension_prefs()->IsExtensionBlacklisted(id)); 6377 EXPECT_TRUE(service_->extension_prefs()->IsExtensionBlacklisted(id));
6378 EXPECT_TRUE( 6378 EXPECT_TRUE(
6379 service_->extension_prefs()->IsBlacklistedExtensionAcknowledged(id)); 6379 service_->extension_prefs()->IsBlacklistedExtensionAcknowledged(id));
6380 } 6380 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698