Prototype CL: https://crrev.com/c/1131177
The following bridge implementation do still require WeakPtr support:
ConsentSyncBridgeImpl
ReadingListStore
SessionSyncBridge
DeviceInfoSyncBridge
UserEventSyncBridge
Overall, I'm not entirely sure this is a worthwhile change:
Many (though not all) do still require WeakPtrs, but we could use WeakPtrFactory instead of SupportsWeakPtr and thus make it private...
Huh, I did not expect so many bridges to need it themselves :|
I'd lean towards WeakPtrFactories where it makes sense. Feel free to also drop the bug by removing the TODO...
Comment 1 by mastiz@chromium.org
, Jul 4