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

Unified Diff: chrome/browser/sync/glue/app_change_processor.h

Issue 7564037: Apps/Extensions Sync refactoring -- delete most of the old glue, implement new sync API. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix Release build warning :-/ Created 9 years, 4 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/glue/app_change_processor.h
diff --git a/chrome/browser/sync/glue/app_change_processor.h b/chrome/browser/sync/glue/app_change_processor.h
deleted file mode 100644
index 0a692a8d8ad60995263b3ac35f9d5d446d2c1197..0000000000000000000000000000000000000000
--- a/chrome/browser/sync/glue/app_change_processor.h
+++ /dev/null
@@ -1,68 +0,0 @@
-// Copyright (c) 2011 The Chromium Authors. All rights reserved.
-// Use of this source code is governed by a BSD-style license that can be
-// found in the LICENSE file.
-
-#ifndef CHROME_BROWSER_SYNC_GLUE_APP_CHANGE_PROCESSOR_H_
-#define CHROME_BROWSER_SYNC_GLUE_APP_CHANGE_PROCESSOR_H_
-#pragma once
-
-#include "base/basictypes.h"
-#include "chrome/browser/sync/glue/change_processor.h"
-#include "chrome/browser/sync/glue/extension_sync_traits.h"
-#include "chrome/browser/sync/internal_api/sync_manager.h"
-#include "content/common/content_notification_types.h"
-#include "content/common/notification_observer.h"
-#include "content/common/notification_registrar.h"
-
-class ExtensionServiceInterface;
-
-namespace browser_sync {
-
-class UnrecoverableErrorHandler;
-
-// This class is responsible for taking changes from the
-// ExtensionService and applying them to the sync_api 'syncable'
-// model, and vice versa. All operations and use of this class are
-// from the UI thread.
-class AppChangeProcessor : public ChangeProcessor,
- public NotificationObserver {
- public:
- // Does not take ownership of |error_handler|.
- explicit AppChangeProcessor(UnrecoverableErrorHandler* error_handler);
- virtual ~AppChangeProcessor();
-
- // NotificationObserver implementation.
- // BrowserExtensionProvider -> sync_api model change application.
- virtual void Observe(int type,
- const NotificationSource& source,
- const NotificationDetails& details);
-
- // ChangeProcessor implementation.
- // sync_api model -> BrowserExtensionProvider change application.
- virtual void ApplyChangesFromSyncModel(
- const sync_api::BaseTransaction* trans,
- const sync_api::SyncManager::ChangeRecord* changes,
- int change_count);
-
- protected:
- // ChangeProcessor implementation.
- virtual void StartImpl(Profile* profile);
- virtual void StopImpl();
-
- private:
- void StartObserving();
- void StopObserving();
-
- NotificationRegistrar notification_registrar_;
- const ExtensionSyncTraits traits_;
-
- // Non-NULL iff |running()| is true.
- Profile* profile_;
- ExtensionServiceInterface* extension_service_;
-
- DISALLOW_COPY_AND_ASSIGN(AppChangeProcessor);
-};
-
-} // namespace browser_sync
-
-#endif // CHROME_BROWSER_SYNC_GLUE_APP_CHANGE_PROCESSOR_H_
« no previous file with comments | « chrome/browser/extensions/test_extension_service.cc ('k') | chrome/browser/sync/glue/app_change_processor.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698