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

Unified Diff: chrome/browser/extensions/extension_warning_badge_service_unittest.cc

Issue 921423003: Rename ExtensionWarningBadgeService to WarningBadgeService (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rebase Created 5 years, 10 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/extensions/extension_warning_badge_service_unittest.cc
diff --git a/chrome/browser/extensions/extension_warning_badge_service_unittest.cc b/chrome/browser/extensions/extension_warning_badge_service_unittest.cc
deleted file mode 100644
index 8c533b924999a91c2a8bbe8d1d9ed6353afe0981..0000000000000000000000000000000000000000
--- a/chrome/browser/extensions/extension_warning_badge_service_unittest.cc
+++ /dev/null
@@ -1,94 +0,0 @@
-// 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_warning_badge_service.h"
-
-#include "chrome/app/chrome_command_ids.h"
-#include "chrome/browser/profiles/profile.h"
-#include "chrome/browser/ui/global_error/global_error_service.h"
-#include "chrome/browser/ui/global_error/global_error_service_factory.h"
-#include "chrome/test/base/testing_profile.h"
-#include "extensions/browser/warning_service.h"
-#include "extensions/browser/warning_set.h"
-#include "testing/gtest/include/gtest/gtest.h"
-
-namespace extensions {
-
-namespace {
-
-class TestExtensionWarningSet : public WarningService {
- public:
- explicit TestExtensionWarningSet(Profile* profile) : WarningService(profile) {
- }
- ~TestExtensionWarningSet() override {}
-
- void AddWarning(const Warning& warning) {
- WarningSet warnings;
- warnings.insert(warning);
- AddWarnings(warnings);
- }
-};
-
-class TestExtensionWarningBadgeService : public ExtensionWarningBadgeService {
- public:
- TestExtensionWarningBadgeService(Profile* profile,
- WarningService* warning_service)
- : ExtensionWarningBadgeService(profile),
- warning_service_(warning_service) {}
- ~TestExtensionWarningBadgeService() override {}
-
- const std::set<Warning>& GetCurrentWarnings() const override {
- return warning_service_->warnings();
- }
-
- private:
- WarningService* warning_service_;
-};
-
-bool HasBadge(Profile* profile) {
- GlobalErrorService* service =
- GlobalErrorServiceFactory::GetForProfile(profile);
- return service->GetGlobalErrorByMenuItemCommandID(IDC_EXTENSION_ERRORS) !=
- NULL;
-}
-
-const char ext1_id[] = "extension1";
-const char ext2_id[] = "extension2";
-
-} // namespace
-
-// Check that no badge appears if it has been suppressed for a specific
-// warning.
-TEST(ExtensionWarningBadgeServiceTest, SuppressBadgeForCurrentWarnings) {
- TestingProfile profile;
- TestExtensionWarningSet warnings(&profile);
- TestExtensionWarningBadgeService badge_service(&profile, &warnings);
- warnings.AddObserver(&badge_service);
-
- // Insert first warning.
- warnings.AddWarning(Warning::CreateNetworkDelayWarning(ext1_id));
- EXPECT_TRUE(HasBadge(&profile));
-
- // Suppress first warning.
- badge_service.SuppressCurrentWarnings();
- EXPECT_FALSE(HasBadge(&profile));
-
- // Simulate deinstallation of extension.
- std::set<Warning::WarningType> to_clear =
- warnings.GetWarningTypesAffectingExtension(ext1_id);
- warnings.ClearWarnings(to_clear);
- EXPECT_FALSE(HasBadge(&profile));
-
- // Set first warning again and verify that not badge is shown this time.
- warnings.AddWarning(Warning::CreateNetworkDelayWarning(ext1_id));
- EXPECT_FALSE(HasBadge(&profile));
-
- // Set second warning and verify that it shows a badge.
- warnings.AddWarning(Warning::CreateNetworkConflictWarning(ext2_id));
- EXPECT_TRUE(HasBadge(&profile));
-
- warnings.RemoveObserver(&badge_service);
-}
-
-} // namespace extensions

Powered by Google App Engine
This is Rietveld 408576698