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

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

Issue 12316077: 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: Rebase + review comments addressed Created 7 years, 10 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 14f169b39cb84e88b845b1fddc6c61a1ceb11eef..ac2a7ff629b0f85f652994f77b9b71c726de675e 100644
--- a/chrome/browser/extensions/extension_service_unittest.cc
+++ b/chrome/browser/extensions/extension_service_unittest.cc
@@ -65,6 +65,7 @@
#include "chrome/common/chrome_paths.h"
#include "chrome/common/chrome_switches.h"
#include "chrome/common/extensions/api/i18n/default_locale_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"
@@ -546,6 +547,7 @@ void ExtensionServiceTestBase::SetUpTestCase() {
void ExtensionServiceTestBase::SetUp() {
testing::Test::SetUp();
ExtensionErrorReporter::GetInstance()->ClearErrors();
+ (new extensions::AppLaunchManifestHandler)->Register();
(new extensions::BackgroundManifestHandler)->Register();
(new extensions::DefaultLocaleHandler)->Register();
}
@@ -2344,8 +2346,9 @@ TEST_F(ExtensionServiceTest, MAYBE_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));
@@ -2357,8 +2360,9 @@ TEST_F(ExtensionServiceTest, MAYBE_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));
@@ -2391,7 +2395,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));
@@ -2400,7 +2405,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));
@@ -3867,7 +3873,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 =
@@ -3881,8 +3888,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