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

Side by Side Diff: chrome/common/extensions/manifest_tests/extension_manifests_scriptbadge_unittest.cc

Issue 10806058: Move icon fallbacks into ExtensionAction. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix kalman's comments. Created 8 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 (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/manifest_tests/extension_manifest_test.h" 5 #include "chrome/common/extensions/manifest_tests/extension_manifest_test.h"
6 6
7 #include "chrome/common/extensions/extension_action.h" 7 #include "chrome/common/extensions/extension_action.h"
8 #include "chrome/common/extensions/extension_manifest_constants.h" 8 #include "chrome/common/extensions/extension_manifest_constants.h"
9 #include "chrome/common/extensions/extension_switch_utils.h" 9 #include "chrome/common/extensions/extension_switch_utils.h"
10 #include "grit/theme_resources.h"
11 #include "testing/gmock/include/gmock/gmock.h"
10 #include "testing/gtest/include/gtest/gtest.h" 12 #include "testing/gtest/include/gtest/gtest.h"
11 #include "testing/gmock/include/gmock/gmock.h" 13 #include "ui/base/resource/resource_bundle.h"
12 #include "third_party/skia/include/core/SkBitmap.h" 14 #include "ui/gfx/image/image.h"
15 #include "ui/gfx/skia_util.h"
13 16
14 namespace errors = extension_manifest_errors; 17 namespace errors = extension_manifest_errors;
15 namespace switch_utils = extensions::switch_utils; 18 namespace switch_utils = extensions::switch_utils;
16 using extensions::Extension; 19 using extensions::Extension;
17 20
21 namespace {
22
23 bool ImagesAreEqual(const gfx::Image& i1, const gfx::Image& i2) {
24 return gfx::BitmapsAreEqual(*i1.ToSkBitmap(), *i2.ToSkBitmap());
25 }
26
18 std::vector<Extension::InstallWarning> StripMissingFlagWarning( 27 std::vector<Extension::InstallWarning> StripMissingFlagWarning(
19 const std::vector<Extension::InstallWarning>& install_warnings) { 28 const std::vector<Extension::InstallWarning>& install_warnings) {
20 std::vector<Extension::InstallWarning> result; 29 std::vector<Extension::InstallWarning> result;
21 for (size_t i = 0; i < install_warnings.size(); ++i) { 30 for (size_t i = 0; i < install_warnings.size(); ++i) {
22 if (install_warnings[i].message != errors::kScriptBadgeRequiresFlag) 31 if (install_warnings[i].message != errors::kScriptBadgeRequiresFlag)
23 result.push_back(install_warnings[i]); 32 result.push_back(install_warnings[i]);
24 } 33 }
25 return result; 34 return result;
26 } 35 }
27 36
28 TEST_F(ExtensionManifestTest, ScriptBadgeBasic) { 37 TEST_F(ExtensionManifestTest, ScriptBadgeBasic) {
29 scoped_refptr<Extension> extension( 38 scoped_refptr<Extension> extension(
30 LoadAndExpectSuccess("script_badge_basic.json")); 39 LoadAndExpectSuccess("script_badge_basic.json"));
31 ASSERT_TRUE(extension.get()); 40 ASSERT_TRUE(extension.get());
32 ASSERT_TRUE(extension->script_badge()); 41 ASSERT_TRUE(extension->script_badge());
33 EXPECT_THAT(StripMissingFlagWarning(extension->install_warnings()), 42 EXPECT_THAT(StripMissingFlagWarning(extension->install_warnings()),
34 testing::ElementsAre(/*empty*/)); 43 testing::ElementsAre(/*empty*/));
35 44
36 EXPECT_EQ("my extension", extension->script_badge()->GetTitle( 45 EXPECT_EQ("my extension", extension->script_badge()->GetTitle(
37 ExtensionAction::kDefaultTabId)); 46 ExtensionAction::kDefaultTabId));
38 EXPECT_TRUE(extension->script_badge()->HasPopup( 47 EXPECT_TRUE(extension->script_badge()->HasPopup(
39 ExtensionAction::kDefaultTabId)); 48 ExtensionAction::kDefaultTabId));
40 EXPECT_TRUE(extension->script_badge()->GetIcon( 49 ExtensionAction::PathToIconCache empty_cache;
41 ExtensionAction::kDefaultTabId).isNull()); 50 EXPECT_TRUE(ImagesAreEqual(
51 ui::ResourceBundle::GetSharedInstance().GetImageNamed(
52 IDR_EXTENSIONS_FAVICON),
53 extension->script_badge()->GetIcon(
54 ExtensionAction::kDefaultTabId, empty_cache)));
42 EXPECT_EQ("icon16.png", extension->script_badge()->default_icon_path()); 55 EXPECT_EQ("icon16.png", extension->script_badge()->default_icon_path());
43 } 56 }
44 57
45 TEST_F(ExtensionManifestTest, ScriptBadgeExplicitTitleAndIconsIgnored) { 58 TEST_F(ExtensionManifestTest, ScriptBadgeExplicitTitleAndIconsIgnored) {
46 scoped_refptr<Extension> extension( 59 scoped_refptr<Extension> extension(
47 LoadAndExpectSuccess("script_badge_title_icons_ignored.json")); 60 LoadAndExpectSuccess("script_badge_title_icons_ignored.json"));
48 ASSERT_TRUE(extension.get()); 61 ASSERT_TRUE(extension.get());
49 ASSERT_TRUE(extension->script_badge()); 62 ASSERT_TRUE(extension->script_badge());
50 63
51 EXPECT_THAT(StripMissingFlagWarning(extension->install_warnings()), 64 EXPECT_THAT(StripMissingFlagWarning(extension->install_warnings()),
52 testing::ElementsAre( 65 testing::ElementsAre(
53 Extension::InstallWarning( 66 Extension::InstallWarning(
54 Extension::InstallWarning::FORMAT_TEXT, 67 Extension::InstallWarning::FORMAT_TEXT,
55 errors::kScriptBadgeTitleIgnored), 68 errors::kScriptBadgeTitleIgnored),
56 Extension::InstallWarning( 69 Extension::InstallWarning(
57 Extension::InstallWarning::FORMAT_TEXT, 70 Extension::InstallWarning::FORMAT_TEXT,
58 errors::kScriptBadgeIconIgnored))); 71 errors::kScriptBadgeIconIgnored)));
59 EXPECT_EQ("my extension", extension->script_badge()->GetTitle( 72 EXPECT_EQ("my extension", extension->script_badge()->GetTitle(
60 ExtensionAction::kDefaultTabId)); 73 ExtensionAction::kDefaultTabId));
61 EXPECT_TRUE(extension->script_badge()->GetIcon( 74 ExtensionAction::PathToIconCache empty_cache;
62 ExtensionAction::kDefaultTabId).isNull()); 75 EXPECT_TRUE(ImagesAreEqual(
76 ui::ResourceBundle::GetSharedInstance().GetImageNamed(
77 IDR_EXTENSIONS_FAVICON),
78 extension->script_badge()->GetIcon(
79 ExtensionAction::kDefaultTabId, empty_cache)));
63 EXPECT_EQ("icon16.png", extension->script_badge()->default_icon_path()); 80 EXPECT_EQ("icon16.png", extension->script_badge()->default_icon_path());
64 } 81 }
65 82
66 TEST_F(ExtensionManifestTest, ScriptBadgeIconFallsBackToPuzzlePiece) { 83 TEST_F(ExtensionManifestTest, ScriptBadgeIconFallsBackToPuzzlePiece) {
67 scoped_refptr<Extension> extension( 84 scoped_refptr<Extension> extension(
68 LoadAndExpectSuccess("script_badge_only_use_icon16.json")); 85 LoadAndExpectSuccess("script_badge_only_use_icon16.json"));
69 ASSERT_TRUE(extension.get()); 86 ASSERT_TRUE(extension.get());
70 ASSERT_TRUE(extension->script_badge()); 87 ASSERT_TRUE(extension->script_badge());
71 EXPECT_THAT(extension->install_warnings(), 88 EXPECT_THAT(extension->install_warnings(),
72 testing::ElementsAre(/*empty*/)); 89 testing::ElementsAre(/*empty*/));
73 90
74 EXPECT_EQ("", extension->script_badge()->default_icon_path()) 91 EXPECT_EQ("", extension->script_badge()->default_icon_path())
75 << "Should not fall back to the 64px icon."; 92 << "Should not fall back to the 64px icon.";
93 ExtensionAction::PathToIconCache empty_cache;
76 EXPECT_FALSE(extension->script_badge()->GetIcon( 94 EXPECT_FALSE(extension->script_badge()->GetIcon(
77 ExtensionAction::kDefaultTabId).isNull()) 95 ExtensionAction::kDefaultTabId, empty_cache).IsEmpty())
78 << "Should set the puzzle piece as the default, but there's no way " 96 << "Should set the puzzle piece as the default, but there's no way "
79 << "to assert in a unittest what the image looks like."; 97 << "to assert in a unittest what the image looks like.";
80 } 98 }
99
100 } // namespace
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698