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

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

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

Powered by Google App Engine
This is Rietveld 408576698