Move the parsing of 'update_url' & 'options_page' URLs out of Extension.

This CL follows https://codereview.chromium.org/11624036/

Patch from Joe Thomas <mhx348@motorola.com>.

BUG=159265


Review URL: https://chromiumcodereview.appspot.com/11726002

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@176824 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 4 insertions, 3 deletions

diff --git a/chrome/browser/themes/theme_syncable_service_unittest.cc b/chrome/browser/themes/theme_syncable_service_unittest.cc
index e82369a..b72750e 100644
--- a/chrome/browser/themes/theme_syncable_service_unittest.cc
+++ b/chrome/browser/themes/theme_syncable_service_unittest.cc
@@ -15,6 +15,7 @@
 #include "chrome/browser/themes/theme_service_factory.h"
 #include "chrome/common/extensions/extension_manifest_constants.h"
 #include "chrome/common/extensions/extension.h"
+#include "chrome/common/extensions/manifest_url_handler.h"
 #include "chrome/test/base/testing_profile.h"
 #include "content/public/test/test_browser_thread.h"
 #include "sync/api/sync_error_factory_mock.h"
@@ -353,7 +354,7 @@ TEST_F(ThemeSyncableServiceTest, UpdateThemeSpecificsFromCurrentTheme) {
   EXPECT_TRUE(theme_specifics.use_custom_theme());
   EXPECT_EQ(theme_extension_->id(), theme_specifics.custom_theme_id());
   EXPECT_EQ(theme_extension_->name(), theme_specifics.custom_theme_name());
-  EXPECT_EQ(theme_extension_->update_url().spec(),
+  EXPECT_EQ(extensions::ManifestURL::GetUpdateURL(theme_extension_).spec(),
             theme_specifics.custom_theme_update_url());
 }
 
@@ -370,7 +371,7 @@ TEST_F(ThemeSyncableServiceTest, GetAllSyncData) {
   EXPECT_TRUE(theme_specifics.use_custom_theme());
   EXPECT_EQ(theme_extension_->id(), theme_specifics.custom_theme_id());
   EXPECT_EQ(theme_extension_->name(), theme_specifics.custom_theme_name());
-  EXPECT_EQ(theme_extension_->update_url().spec(),
+  EXPECT_EQ(extensions::ManifestURL::GetUpdateURL(theme_extension_).spec(),
             theme_specifics.custom_theme_update_url());
 }
 
@@ -433,7 +434,7 @@ TEST_F(ThemeSyncableServiceTest, OnThemeChangeByUser) {
   EXPECT_TRUE(change_specifics.use_custom_theme());
   EXPECT_EQ(theme_extension_->id(), change_specifics.custom_theme_id());
   EXPECT_EQ(theme_extension_->name(), change_specifics.custom_theme_name());
-  EXPECT_EQ(theme_extension_->update_url().spec(),
+  EXPECT_EQ(extensions::ManifestURL::GetUpdateURL(theme_extension_).spec(),
             change_specifics.custom_theme_update_url());
 }