diff options
author | hclam@chromium.org <hclam@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-06-25 07:25:52 +0000 |
---|---|---|
committer | hclam@chromium.org <hclam@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-06-25 07:25:52 +0000 |
commit | 052c92700b2c71e60ab89e765467b9b0926d0f7c (patch) | |
tree | cc5181425481f5728bdcdda9933449007bd64b46 /chrome/browser/extensions/extensions_service_unittest.cc | |
parent | 16f17f5edd89d6b6a41709a9946a0183f5754c35 (diff) | |
download | chromium_src-052c92700b2c71e60ab89e765467b9b0926d0f7c.zip chromium_src-052c92700b2c71e60ab89e765467b9b0926d0f7c.tar.gz chromium_src-052c92700b2c71e60ab89e765467b9b0926d0f7c.tar.bz2 |
Revert "Revert "Show extension icons next to their top-level context menu items.""
This reverts r50828 as the error is already suppressed r50793.
TBR=asargent
Review URL: http://codereview.chromium.org/2844028
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@50831 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/extensions/extensions_service_unittest.cc')
-rw-r--r-- | chrome/browser/extensions/extensions_service_unittest.cc | 11 |
1 files changed, 7 insertions, 4 deletions
diff --git a/chrome/browser/extensions/extensions_service_unittest.cc b/chrome/browser/extensions/extensions_service_unittest.cc index a6bbb3c..6ce579c 100644 --- a/chrome/browser/extensions/extensions_service_unittest.cc +++ b/chrome/browser/extensions/extensions_service_unittest.cc @@ -257,7 +257,7 @@ void ExtensionsServiceTestBase::InitializeExtensionsService( new JsonPrefStore( /* user defined preference values */ pref_file, ChromeThread::GetMessageLoopProxyForThread(ChromeThread::FILE)), - NULL /* No suggested preference values */ ))); + NULL /* No suggested preference values */))); Profile::RegisterUserPrefs(prefs_.get()); browser::RegisterUserPrefs(prefs_.get()); @@ -663,12 +663,14 @@ TEST_F(ExtensionsServiceTest, LoadAllExtensionsFromDirectorySuccess) { EXPECT_EQ("https://*.google.com/*", scripts[0].url_patterns()[2].GetAsString()); EXPECT_EQ(2u, scripts[0].js_scripts().size()); - ExtensionResource resource00(scripts[0].js_scripts()[0].extension_root(), + ExtensionResource resource00(extension->id(), + scripts[0].js_scripts()[0].extension_root(), scripts[0].js_scripts()[0].relative_path()); FilePath expected_path(extension->path().AppendASCII("script1.js")); ASSERT_TRUE(file_util::AbsolutePath(&expected_path)); EXPECT_TRUE(resource00.ComparePathWithDefault(expected_path)); - ExtensionResource resource01(scripts[0].js_scripts()[1].extension_root(), + ExtensionResource resource01(extension->id(), + scripts[0].js_scripts()[1].extension_root(), scripts[0].js_scripts()[1].relative_path()); expected_path = extension->path().AppendASCII("script2.js"); ASSERT_TRUE(file_util::AbsolutePath(&expected_path)); @@ -676,7 +678,8 @@ TEST_F(ExtensionsServiceTest, LoadAllExtensionsFromDirectorySuccess) { EXPECT_TRUE(extension->plugins().empty()); EXPECT_EQ(1u, scripts[1].url_patterns().size()); EXPECT_EQ("http://*.news.com/*", scripts[1].url_patterns()[0].GetAsString()); - ExtensionResource resource10(scripts[1].js_scripts()[0].extension_root(), + ExtensionResource resource10(extension->id(), + scripts[1].js_scripts()[0].extension_root(), scripts[1].js_scripts()[0].relative_path()); expected_path = extension->path().AppendASCII("js_files").AppendASCII("script3.js"); |