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

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

Issue 14241002: Move the parsing of app.launch related keys out of Extension class. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 7 years, 8 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_service_unittest.cc
diff --git a/chrome/browser/extensions/extension_service_unittest.cc b/chrome/browser/extensions/extension_service_unittest.cc
index c171bcc4754e2c293286db026e73eaac1b20d8d5..130fb4d99f691df58ec35b17976dcbe6879e4e39 100644
--- a/chrome/browser/extensions/extension_service_unittest.cc
+++ b/chrome/browser/extensions/extension_service_unittest.cc
@@ -64,6 +64,7 @@
#include "chrome/common/chrome_switches.h"
#include "chrome/common/extensions/api/i18n/default_locale_handler.h"
#include "chrome/common/extensions/api/plugins/plugins_handler.h"
+#include "chrome/common/extensions/app_launcher_info.h"
#include "chrome/common/extensions/background_info.h"
#include "chrome/common/extensions/extension.h"
#include "chrome/common/extensions/extension_l10n_util.h"
@@ -549,6 +550,7 @@ void ExtensionServiceTestBase::SetUpTestCase() {
void ExtensionServiceTestBase::SetUp() {
ExtensionTest::SetUp();
ExtensionErrorReporter::GetInstance()->ClearErrors();
+ (new extensions::AppLaunchManifestHandler)->Register();
(new extensions::BackgroundManifestHandler)->Register();
(new extensions::ContentScriptsHandler)->Register();
(new extensions::DefaultLocaleHandler)->Register();
@@ -2350,8 +2352,9 @@ TEST_F(ExtensionServiceTest, InstallAppsWithUnlimitedStorage) {
EXPECT_TRUE(extension->HasAPIPermission(
APIPermission::kUnlimitedStorage));
EXPECT_TRUE(extension->web_extent().MatchesURL(
- extension->GetFullLaunchURL()));
- const GURL origin1(extension->GetFullLaunchURL().GetOrigin());
+ extensions::AppLauncherInfo::GetFullLaunchURL(extension)));
+ const GURL origin1(
+ extensions::AppLauncherInfo::GetFullLaunchURL(extension).GetOrigin());
EXPECT_TRUE(profile_->GetExtensionSpecialStoragePolicy()->
IsStorageUnlimited(origin1));
@@ -2363,8 +2366,9 @@ TEST_F(ExtensionServiceTest, InstallAppsWithUnlimitedStorage) {
EXPECT_TRUE(extension->HasAPIPermission(
APIPermission::kUnlimitedStorage));
EXPECT_TRUE(extension->web_extent().MatchesURL(
- extension->GetFullLaunchURL()));
- const GURL origin2(extension->GetFullLaunchURL().GetOrigin());
+ extensions::AppLauncherInfo::GetFullLaunchURL(extension)));
+ const GURL origin2(
+ extensions::AppLauncherInfo::GetFullLaunchURL(extension).GetOrigin());
EXPECT_EQ(origin1, origin2);
EXPECT_TRUE(profile_->GetExtensionSpecialStoragePolicy()->
IsStorageUnlimited(origin2));
@@ -2397,7 +2401,8 @@ TEST_F(ExtensionServiceTest, InstallAppsAndCheckStorageProtection) {
ASSERT_EQ(1u, service_->extensions()->size());
EXPECT_TRUE(extension->is_app());
const std::string id1 = extension->id();
- const GURL origin1(extension->GetFullLaunchURL().GetOrigin());
+ const GURL origin1(
+ extensions::AppLauncherInfo::GetFullLaunchURL(extension).GetOrigin());
EXPECT_TRUE(profile_->GetExtensionSpecialStoragePolicy()->
IsStorageProtected(origin1));
@@ -2406,7 +2411,8 @@ TEST_F(ExtensionServiceTest, InstallAppsAndCheckStorageProtection) {
ValidatePrefKeyCount(++pref_count);
ASSERT_EQ(2u, service_->extensions()->size());
const std::string id2 = extension->id();
- const GURL origin2(extension->GetFullLaunchURL().GetOrigin());
+ const GURL origin2(
+ extensions::AppLauncherInfo::GetFullLaunchURL(extension).GetOrigin());
ASSERT_NE(origin1, origin2);
EXPECT_TRUE(profile_->GetExtensionSpecialStoragePolicy()->
IsStorageProtected(origin2));
@@ -3910,7 +3916,8 @@ TEST_F(ExtensionServiceTest, ClearAppData) {
const std::string id1 = extension->id();
EXPECT_TRUE(extension->HasAPIPermission(
APIPermission::kUnlimitedStorage));
- const GURL origin1(extension->GetFullLaunchURL().GetOrigin());
+ const GURL origin1(
+ extensions::AppLauncherInfo::GetFullLaunchURL(extension).GetOrigin());
EXPECT_TRUE(profile_->GetExtensionSpecialStoragePolicy()->
IsStorageUnlimited(origin1));
string16 origin_id =
@@ -3924,8 +3931,9 @@ TEST_F(ExtensionServiceTest, ClearAppData) {
EXPECT_TRUE(extension->HasAPIPermission(
APIPermission::kUnlimitedStorage));
EXPECT_TRUE(extension->web_extent().MatchesURL(
- extension->GetFullLaunchURL()));
- const GURL origin2(extension->GetFullLaunchURL().GetOrigin());
+ extensions::AppLauncherInfo::GetFullLaunchURL(extension)));
+ const GURL origin2(
+ extensions::AppLauncherInfo::GetFullLaunchURL(extension).GetOrigin());
EXPECT_EQ(origin1, origin2);
EXPECT_TRUE(profile_->GetExtensionSpecialStoragePolicy()->
IsStorageUnlimited(origin2));

Powered by Google App Engine
This is Rietveld 408576698