Chromium Code Reviews| OLD | NEW |
|---|---|
| 1 // Copyright 2014 The Chromium Authors. All rights reserved. | 1 // Copyright 2014 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 SYNC_TEST_ENGINE_INJECTABLE_SYNC_CONTEXT_PROXY_H_ | 5 #ifndef SYNC_TEST_ENGINE_INJECTABLE_SYNC_CONTEXT_PROXY_H_ |
| 6 #define SYNC_TEST_ENGINE_INJECTABLE_SYNC_CONTEXT_PROXY_H_ | 6 #define SYNC_TEST_ENGINE_INJECTABLE_SYNC_CONTEXT_PROXY_H_ |
| 7 | 7 |
| 8 #include "sync/internal_api/public/base/model_type.h" | 8 #include "sync/internal_api/public/base/model_type.h" |
| 9 #include "sync/internal_api/public/non_blocking_sync_common.h" | 9 #include "sync/internal_api/public/non_blocking_sync_common.h" |
| 10 #include "sync/internal_api/public/sync_context_proxy.h" | 10 #include "sync/internal_api/public/sync_context_proxy.h" |
| 11 | 11 |
| 12 namespace syncer_v2 { | 12 namespace syncer_v2 { |
| 13 | 13 |
| 14 class ModelTypeSyncProxyImpl; | 14 class ModelTypeProcessorImpl; |
| 15 class ModelTypeSyncWorker; | 15 class CommitQueue; |
| 16 | 16 |
| 17 // A SyncContextProxy implementation that, when a connection request is made, | 17 // A SyncContextProxy implementation that, when a connection request is made, |
| 18 // initalizes a connection to a previously injected ModelTypeSyncProxyImpl. | 18 // initalizes a connection to a previously injected ModelTypeProcessorImpl. |
| 19 class InjectableSyncContextProxy : public SyncContextProxy { | 19 class InjectableSyncContextProxy : public SyncContextProxy { |
| 20 public: | 20 public: |
| 21 explicit InjectableSyncContextProxy(ModelTypeSyncWorker* worker); | 21 explicit InjectableSyncContextProxy(CommitQueue* worker); |
| 22 ~InjectableSyncContextProxy() override; | 22 ~InjectableSyncContextProxy() override; |
| 23 | 23 |
| 24 void ConnectTypeToSync( | 24 void ConnectTypeToSync( |
| 25 syncer::ModelType type, | 25 syncer::ModelType type, |
| 26 const DataTypeState& data_type_state, | 26 const DataTypeState& data_type_state, |
| 27 const UpdateResponseDataList& pending_updates, | 27 const UpdateResponseDataList& pending_updates, |
| 28 const base::WeakPtr<ModelTypeSyncProxyImpl>& type_sync_proxy) override; | 28 const base::WeakPtr<ModelTypeProcessorImpl>& type_sync_proxy) override; |
| 29 void Disconnect(syncer::ModelType type) override; | 29 void Disconnect(syncer::ModelType type) override; |
| 30 scoped_ptr<SyncContextProxy> Clone() const override; | 30 scoped_ptr<SyncContextProxy> Clone() const override; |
| 31 | 31 |
| 32 ModelTypeSyncWorker* GetWorker(); | 32 CommitQueue* GetWorker(); |
|
stanisc
2015/08/31 17:42:19
Probably shouldn't be called GetWorker anymore
Gang Wu
2015/08/31 22:38:05
Done.
| |
| 33 | 33 |
| 34 private: | 34 private: |
| 35 // A flag to ensure ConnectTypeToSync is called at most once. | 35 // A flag to ensure ConnectTypeToSync is called at most once. |
| 36 bool is_worker_connected_; | 36 bool is_worker_connected_; |
| 37 | 37 |
| 38 // The ModelTypeSyncProxy's contract expects that it gets to own this object, | 38 // The ModelTypeProcessor's contract expects that it gets to own this object, |
| 39 // so we can retain only a non-owned pointer to it. | 39 // so we can retain only a non-owned pointer to it. |
| 40 // | 40 // |
| 41 // This is very unsafe, but we can get away with it since these tests are not | 41 // This is very unsafe, but we can get away with it since these tests are not |
| 42 // exercising the proxy <-> worker connection code. | 42 // exercising the proxy <-> worker connection code. |
| 43 ModelTypeSyncWorker* worker_; | 43 CommitQueue* worker_; |
| 44 }; | 44 }; |
| 45 | 45 |
| 46 } // namespace syncer | 46 } // namespace syncer |
| 47 | 47 |
| 48 #endif // SYNC_TEST_ENGINE_INJECTABLE_SYNC_CONTEXT_PROXY_H_ | 48 #endif // SYNC_TEST_ENGINE_INJECTABLE_SYNC_CONTEXT_PROXY_H_ |
| OLD | NEW |