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

Unified Diff: chrome/browser/extensions/chrome_app_sorting.h

Issue 1251323003: Cleanup: give ChromeAppSorting a BrowserContext (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: . Created 5 years, 5 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/chrome_app_sorting.h
diff --git a/chrome/browser/extensions/chrome_app_sorting.h b/chrome/browser/extensions/chrome_app_sorting.h
index d185deeaa743d86f0b2a0b879e1f863e249f785d..2754bc0baaaec978bd273757fbf266e7d40e001f 100644
--- a/chrome/browser/extensions/chrome_app_sorting.h
+++ b/chrome/browser/extensions/chrome_app_sorting.h
@@ -15,7 +15,6 @@
#include "extensions/common/extension.h"
#include "sync/api/string_ordinal.h"
-class ExtensionSyncService;
class PrefService;
namespace extensions {
@@ -24,14 +23,12 @@ class ExtensionScopedPrefs;
class ChromeAppSorting : public AppSorting {
public:
- ChromeAppSorting();
+ explicit ChromeAppSorting(content::BrowserContext* browser_context);
~ChromeAppSorting() override;
// AppSorting implementation:
void SetExtensionScopedPrefs(ExtensionScopedPrefs* prefs) override;
void CheckExtensionScopedPrefs() const override;
- void SetExtensionSyncService(
- ExtensionSyncService* extension_sync_service) override;
void Initialize(const extensions::ExtensionIdList& extension_ids) override;
void FixNTPOrdinalCollisions() override;
void EnsureValidOrdinals(
@@ -156,7 +153,7 @@ class ChromeAppSorting : public AppSorting {
size_t CountItemsVisibleOnNtp(const AppLaunchOrdinalMap& m) const;
ExtensionScopedPrefs* extension_scoped_prefs_; // Weak, owns this instance.
- ExtensionSyncService* extension_sync_service_; // Weak.
+ content::BrowserContext* browser_context_;
// A map of all the StringOrdinal page ordinals mapping to the collections of
// app launch ordinals that exist on that page. This is used for mapping
« no previous file with comments | « no previous file | chrome/browser/extensions/chrome_app_sorting.cc » ('j') | chrome/browser/extensions/chrome_app_sorting.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698