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

Side by Side Diff: chrome/common/extensions/api/extension_action/script_badge_manifest_unittest.cc

Issue 22470007: Add a "key" entry to InstallWarnings, remove InstallWarning::Format (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@dc_ec_infrastructure
Patch Set: 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) 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 "chrome/common/extensions/api/extension_action/action_info.h" 5 #include "chrome/common/extensions/api/extension_action/action_info.h"
6 #include "chrome/common/extensions/extension_builder.h" 6 #include "chrome/common/extensions/extension_builder.h"
7 #include "chrome/common/extensions/extension_constants.h" 7 #include "chrome/common/extensions/extension_constants.h"
8 #include "chrome/common/extensions/extension_icon_set.h" 8 #include "chrome/common/extensions/extension_icon_set.h"
9 #include "chrome/common/extensions/extension_manifest_constants.h" 9 #include "chrome/common/extensions/extension_manifest_constants.h"
10 #include "chrome/common/extensions/feature_switch.h" 10 #include "chrome/common/extensions/feature_switch.h"
11 #include "chrome/common/extensions/manifest_tests/extension_manifest_test.h" 11 #include "chrome/common/extensions/manifest_tests/extension_manifest_test.h"
12 #include "extensions/common/install_warning.h" 12 #include "extensions/common/install_warning.h"
13 #include "extensions/common/manifest_constants.h"
13 #include "testing/gmock/include/gmock/gmock.h" 14 #include "testing/gmock/include/gmock/gmock.h"
14 #include "testing/gtest/include/gtest/gtest.h" 15 #include "testing/gtest/include/gtest/gtest.h"
15 16
16 namespace errors = extension_manifest_errors; 17 namespace errors = extension_manifest_errors;
17 18
18 namespace extensions { 19 namespace extensions {
19 20
20 namespace { 21 namespace {
21 22
22 std::vector<InstallWarning> StripMissingFlagWarning( 23 std::vector<InstallWarning> StripMissingFlagWarning(
(...skipping 68 matching lines...) Expand 10 before | Expand all | Expand 10 after
91 .Set("default_title", "Other Extension") 92 .Set("default_title", "Other Extension")
92 .Set("default_icon", "malicious.png"))) 93 .Set("default_icon", "malicious.png")))
93 .Build()); 94 .Build());
94 ASSERT_TRUE(extension.get()); 95 ASSERT_TRUE(extension.get());
95 const ActionInfo* script_badge_info = 96 const ActionInfo* script_badge_info =
96 ActionInfo::GetScriptBadgeInfo(extension.get()); 97 ActionInfo::GetScriptBadgeInfo(extension.get());
97 ASSERT_TRUE(script_badge_info); 98 ASSERT_TRUE(script_badge_info);
98 99
99 EXPECT_THAT(StripMissingFlagWarning(extension->install_warnings()), 100 EXPECT_THAT(StripMissingFlagWarning(extension->install_warnings()),
100 testing::ElementsAre( 101 testing::ElementsAre(
101 InstallWarning(InstallWarning::FORMAT_TEXT, 102 InstallWarning(errors::kScriptBadgeTitleIgnored,
102 errors::kScriptBadgeTitleIgnored), 103 manifest_keys::kScriptBadge),
103 InstallWarning(InstallWarning::FORMAT_TEXT, 104 InstallWarning(errors::kScriptBadgeIconIgnored,
104 errors::kScriptBadgeIconIgnored))); 105 manifest_keys::kScriptBadge)));
105 106
106 const ExtensionIconSet& default_icon = 107 const ExtensionIconSet& default_icon =
107 script_badge_info->default_icon; 108 script_badge_info->default_icon;
108 ASSERT_FALSE(default_icon.empty()); 109 ASSERT_FALSE(default_icon.empty());
109 110
110 EXPECT_EQ(1u, default_icon.map().size()); 111 EXPECT_EQ(1u, default_icon.map().size());
111 EXPECT_EQ("icon16.png", 112 EXPECT_EQ("icon16.png",
112 default_icon.Get(extension_misc::EXTENSION_ICON_BITTY, 113 default_icon.Get(extension_misc::EXTENSION_ICON_BITTY,
113 ExtensionIconSet::MATCH_EXACTLY)); 114 ExtensionIconSet::MATCH_EXACTLY));
114 115
(...skipping 25 matching lines...) Expand all
140 EXPECT_EQ(2u, default_icon.map().size()); 141 EXPECT_EQ(2u, default_icon.map().size());
141 EXPECT_EQ("icon128.png", 142 EXPECT_EQ("icon128.png",
142 default_icon.Get(extension_misc::EXTENSION_ICON_BITTY, 143 default_icon.Get(extension_misc::EXTENSION_ICON_BITTY,
143 ExtensionIconSet::MATCH_EXACTLY)); 144 ExtensionIconSet::MATCH_EXACTLY));
144 EXPECT_EQ("icon128.png", 145 EXPECT_EQ("icon128.png",
145 default_icon.Get(2 * extension_misc::EXTENSION_ICON_BITTY, 146 default_icon.Get(2 * extension_misc::EXTENSION_ICON_BITTY,
146 ExtensionIconSet::MATCH_EXACTLY)); 147 ExtensionIconSet::MATCH_EXACTLY));
147 } 148 }
148 149
149 } // namespace extensions 150 } // namespace extensions
OLDNEW
« no previous file with comments | « chrome/common/extensions/api/extension_action/script_badge_handler.cc ('k') | chrome/common/extensions/extension_file_util.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698