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

Side by Side Diff: chrome/browser/sync/js/js_mutation_event_observer.h

Issue 7982049: [Sync] Move some code into new class JsMutationEventObserver (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix deps Created 9 years, 3 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2011 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef CHROME_BROWSER_SYNC_JS_JS_TRANSACTION_OBSERVER_H_ 5 #ifndef CHROME_BROWSER_SYNC_JS_JS_MUTATION_EVENT_OBSERVER_H_
6 #define CHROME_BROWSER_SYNC_JS_JS_TRANSACTION_OBSERVER_H_ 6 #define CHROME_BROWSER_SYNC_JS_JS_MUTATION_EVENT_OBSERVER_H_
7 #pragma once 7 #pragma once
8 8
9 #include <string> 9 #include <string>
10 10
11 #include "base/basictypes.h" 11 #include "base/basictypes.h"
12 #include "base/compiler_specific.h" 12 #include "base/compiler_specific.h"
13 #include "base/threading/non_thread_safe.h" 13 #include "base/threading/non_thread_safe.h"
14 #include "chrome/browser/sync/internal_api/sync_manager.h"
14 #include "chrome/browser/sync/syncable/transaction_observer.h" 15 #include "chrome/browser/sync/syncable/transaction_observer.h"
15 #include "chrome/browser/sync/util/weak_handle.h" 16 #include "chrome/browser/sync/util/weak_handle.h"
16 17
17 namespace tracked_objects { 18 namespace tracked_objects {
18 class Location; 19 class Location;
19 } // namespace tracked_objects 20 } // namespace tracked_objects
20 21
21 namespace browser_sync { 22 namespace browser_sync {
22 23
23 class JsEventDetails; 24 class JsEventDetails;
24 class JsEventHandler; 25 class JsEventHandler;
25 26
26 // Routes SyncManager events to a JsEventHandler. 27 // Observes all change- and transaction-related events and routes a
27 class JsTransactionObserver : public syncable::TransactionObserver { 28 // summarized version to a JsEventHandler.
29 class JsMutationEventObserver
30 : public sync_api::SyncManager::ChangeObserver,
31 public syncable::TransactionObserver {
28 public: 32 public:
29 JsTransactionObserver(); 33 JsMutationEventObserver();
30 34
31 virtual ~JsTransactionObserver(); 35 virtual ~JsMutationEventObserver();
32 36
33 void SetJsEventHandler(const WeakHandle<JsEventHandler>& event_handler); 37 void SetJsEventHandler(const WeakHandle<JsEventHandler>& event_handler);
34 38
39 // sync_api::SyncManager::ChangeObserver implementation.
40 virtual void OnChangesApplied(
41 syncable::ModelType model_type,
42 int64 write_transaction_id,
43 const sync_api::ImmutableChangeRecordList& changes) OVERRIDE;
44 virtual void OnChangesComplete(syncable::ModelType model_type) OVERRIDE;
45
35 // syncable::TransactionObserver implementation. 46 // syncable::TransactionObserver implementation.
36 virtual void OnTransactionStart( 47 virtual void OnTransactionStart(
37 const tracked_objects::Location& location, 48 const tracked_objects::Location& location,
38 const syncable::WriterTag& writer) OVERRIDE; 49 const syncable::WriterTag& writer) OVERRIDE;
39 virtual void OnTransactionWrite( 50 virtual void OnTransactionWrite(
40 const syncable::ImmutableWriteTransactionInfo& write_transaction_info, 51 const syncable::ImmutableWriteTransactionInfo& write_transaction_info,
41 const syncable::ModelTypeBitSet& models_with_changes) OVERRIDE; 52 const syncable::ModelTypeBitSet& models_with_changes) OVERRIDE;
42 virtual void OnTransactionEnd( 53 virtual void OnTransactionEnd(
43 const tracked_objects::Location& location, 54 const tracked_objects::Location& location,
44 const syncable::WriterTag& writer) OVERRIDE; 55 const syncable::WriterTag& writer) OVERRIDE;
45 56
46 private: 57 private:
47 base::NonThreadSafe non_thread_safe_; 58 base::NonThreadSafe non_thread_safe_;
48 WeakHandle<JsEventHandler> event_handler_; 59 WeakHandle<JsEventHandler> event_handler_;
49 60
50 void HandleJsEvent( 61 void HandleJsEvent(
51 const tracked_objects::Location& from_here, 62 const tracked_objects::Location& from_here,
52 const std::string& name, const JsEventDetails& details); 63 const std::string& name, const JsEventDetails& details);
53 64
54 DISALLOW_COPY_AND_ASSIGN(JsTransactionObserver); 65 DISALLOW_COPY_AND_ASSIGN(JsMutationEventObserver);
55 }; 66 };
56 67
57 } // namespace browser_sync 68 } // namespace browser_sync
58 69
59 #endif // CHROME_BROWSER_SYNC_JS_JS_TRANSACTION_OBSERVER_H_ 70 #endif // CHROME_BROWSER_SYNC_JS_JS_MUTATION_EVENT_OBSERVER_H_
OLDNEW
« no previous file with comments | « chrome/browser/sync/internal_api/sync_manager.cc ('k') | chrome/browser/sync/js/js_mutation_event_observer.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698