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

Unified Diff: chrome/browser/sync/test/integration/sync_app_helper.cc

Issue 9706017: Remove Ordinals Setters and Getters from ExtensionService (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Created 8 years, 9 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/sync/test/integration/sync_app_helper.cc
diff --git a/chrome/browser/sync/test/integration/sync_app_helper.cc b/chrome/browser/sync/test/integration/sync_app_helper.cc
index a2864beef3c6d3efe2e8b8604527414876c126d9..bbe2f501cf24705d1f02b9a94fe90a299f870db3 100644
--- a/chrome/browser/sync/test/integration/sync_app_helper.cc
+++ b/chrome/browser/sync/test/integration/sync_app_helper.cc
@@ -43,8 +43,10 @@ bool AppState::Equals(const AppState& other) const {
void LoadApp(ExtensionService* extension_service,
const std::string& id,
AppState* app_state) {
- app_state->app_launch_ordinal = extension_service->GetAppLaunchOrdinal(id);
- app_state->page_ordinal = extension_service->GetPageOrdinal(id);
+ app_state->app_launch_ordinal = extension_service->extension_prefs()->
+ extension_sorting()->GetAppLaunchOrdinal(id);
+ app_state->page_ordinal = extension_service->extension_prefs()->
+ extension_sorting()->GetPageOrdinal(id);
}
// Returns a map from |profile|'s installed extensions to their state.
@@ -139,36 +141,37 @@ bool SyncAppHelper::AppStatesMatch(Profile* profile1, Profile* profile2) {
StringOrdinal SyncAppHelper::GetPageOrdinalForApp(Profile* profile,
const std::string& name) {
- return profile->GetExtensionService()->GetPageOrdinal(
- SyncExtensionHelper::NameToId(name));
+ return profile->GetExtensionService()->extension_prefs()->
+ extension_sorting()->GetPageOrdinal(SyncExtensionHelper::NameToId(name));
}
void SyncAppHelper::SetPageOrdinalForApp(Profile* profile,
const std::string& name,
const StringOrdinal& page_ordinal) {
- profile->GetExtensionService()->SetPageOrdinal(
- SyncExtensionHelper::NameToId(name), page_ordinal);
+ profile->GetExtensionService()->extension_prefs()->extension_sorting()->
+ SetPageOrdinal(SyncExtensionHelper::NameToId(name), page_ordinal);
}
StringOrdinal SyncAppHelper::GetAppLaunchOrdinalForApp(
Profile* profile,
const std::string& name) {
- return profile->GetExtensionService()->GetAppLaunchOrdinal(
- SyncExtensionHelper::NameToId(name));
+ return profile->GetExtensionService()->extension_prefs()->
+ extension_sorting()->GetAppLaunchOrdinal(
+ SyncExtensionHelper::NameToId(name));
}
void SyncAppHelper::SetAppLaunchOrdinalForApp(
Profile* profile,
const std::string& name,
const StringOrdinal& app_launch_ordinal) {
- profile->GetExtensionService()->SetAppLaunchOrdinal(
- SyncExtensionHelper::NameToId(name),
- app_launch_ordinal);
+ profile->GetExtensionService()->extension_prefs()->extension_sorting()->
+ SetAppLaunchOrdinal(SyncExtensionHelper::NameToId(name),
+ app_launch_ordinal);
}
void SyncAppHelper::FixNTPOrdinalCollisions(Profile* profile) {
- profile->GetExtensionService()->extension_prefs()->
- extension_sorting()->FixNTPOrdinalCollisions();
+ profile->GetExtensionService()->extension_prefs()->extension_sorting()->
+ FixNTPOrdinalCollisions();
}
SyncAppHelper::SyncAppHelper() : setup_completed_(false) {}
« no previous file with comments | « chrome/browser/extensions/test_extension_service.cc ('k') | chrome/browser/sync/test/integration/two_client_apps_sync_test.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698