[sync] skeleton implementation of TypedURLSyncBridge

components/history/core/browser/history_service.h

Index: components/history/core/browser/history_service.h
diff --git a/components/history/core/browser/history_service.h b/components/history/core/browser/history_service.h
index dd6b19aa490f1898e9413cc7fa355269966a4ef5..c42d4f2950ebbb2f9f24d50889f6e97af4e28a4c 100644
--- a/components/history/core/browser/history_service.h
+++ b/components/history/core/browser/history_service.h
@@ -33,6 +33,7 @@
 #include "components/history/core/browser/delete_directive_handler.h"
 #include "components/history/core/browser/history_types.h"
 #include "components/history/core/browser/keyword_id.h"
+#include "components/history/core/browser/typed_url_sync_bridge.h"

[skym, 2017/03/07 18:08:04]

Is there a reason this file needs either the bridge or syncable service's .h
file to be included? Why not forward declare?

[Gang Wu, 2017/03/31 19:35:58]

chrome_sync_client.cc call history->GetTypedUrlSyncableService()->AsWeakPtr(),
for ->AsWeakPtr() function we need to include .h file here.

 #include "components/history/core/browser/typed_url_syncable_service.h"
 #include "components/keyed_service/core/keyed_service.h"
 #include "components/sync/model/syncable_service.h"
@@ -129,6 +130,11 @@ class HistoryService : public syncer::SyncableService, public KeyedService {
   // They return false if database is not available (e.g. not loaded yet) or the
   // URL does not exist.
 
+  // Returns a pointer to the TypedURLSyncBridge owned by HistoryBackend.
+  // This method should only be called from the history thread, because the
+  // returned bridge is intended to be accessed only via the history thread.
+  TypedURLSyncBridge* GetTypedURLSyncBridge() const;
+
   // Returns a pointer to the TypedUrlSyncableService owned by HistoryBackend.
   // This method should only be called from the history thread, because the
   // returned service is intended to be accessed only via the history thread.