|
Split HistoryClient in two objects
HistoryClient exposed methods accessed by UI or History thread. Split
the object in two only exposing method accessed by a single thread.
The HistoryClient methods can only be invoked from UI thread while the
HistoryBackendClient methods can only be invoked from History thread.
This separation is required before the History code can be refactored
not to create it's own thread.
BUG= 459548, 477975
Committed: https://crrev.com/542b71e372426c2547be0a689c2d7053b1ffdeff
Cr-Commit-Position: refs/heads/master@{#335779}
Total comments: 1
|
Unified diffs |
Side-by-side diffs |
Delta from patch set |
Stats (+492 lines, -350 lines) |
Patch |
 |
M |
chrome/browser/history/android/android_provider_backend.h
|
View
|
|
3 chunks |
+3 lines, -3 lines |
0 comments
|
Download
|
 |
M |
chrome/browser/history/android/android_provider_backend.cc
|
View
|
|
3 chunks |
+5 lines, -5 lines |
0 comments
|
Download
|
 |
M |
chrome/browser/history/android/android_provider_backend_unittest.cc
|
View
|
|
30 chunks |
+36 lines, -33 lines |
0 comments
|
Download
|
 |
A |
chrome/browser/history/chrome_history_backend_client.h
|
View
|
|
1 chunk |
+44 lines, -0 lines |
0 comments
|
Download
|
 |
A |
chrome/browser/history/chrome_history_backend_client.cc
|
View
|
1
|
1 chunk |
+93 lines, -0 lines |
0 comments
|
Download
|
 |
M |
chrome/browser/history/chrome_history_client.h
|
View
|
|
2 chunks |
+4 lines, -18 lines |
0 comments
|
Download
|
 |
M |
chrome/browser/history/chrome_history_client.cc
|
View
|
|
4 chunks |
+4 lines, -77 lines |
0 comments
|
Download
|
 |
M |
chrome/browser/sync/profile_sync_service_typed_url_unittest.cc
|
View
|
|
1 chunk |
+1 line, -0 lines |
0 comments
|
Download
|
 |
M |
chrome/chrome_browser.gypi
|
View
|
1
|
1 chunk |
+2 lines, -0 lines |
0 comments
|
Download
|
 |
M |
components/history.gypi
|
View
|
|
1 chunk |
+1 line, -1 line |
0 comments
|
Download
|
 |
M |
components/history/core/browser/BUILD.gn
|
View
|
|
1 chunk |
+1 line, -1 line |
0 comments
|
Download
|
 |
M |
components/history/core/browser/expire_history_backend.h
|
View
|
|
4 chunks |
+8 lines, -15 lines |
0 comments
|
Download
|
 |
M |
components/history/core/browser/expire_history_backend.cc
|
View
|
|
7 chunks |
+8 lines, -19 lines |
0 comments
|
Download
|
 |
M |
components/history/core/browser/expire_history_backend_unittest.cc
|
View
|
|
3 chunks |
+4 lines, -1 line |
0 comments
|
Download
|
 |
M |
components/history/core/browser/history_backend.h
|
View
|
|
4 chunks |
+4 lines, -10 lines |
0 comments
|
Download
|
 |
M |
components/history/core/browser/history_backend.cc
|
View
|
|
9 chunks |
+16 lines, -20 lines |
0 comments
|
Download
|
 |
A |
components/history/core/browser/history_backend_client.h
|
View
|
|
1 chunk |
+65 lines, -0 lines |
0 comments
|
Download
|
 |
A + |
components/history/core/browser/history_backend_client.cc
|
View
|
|
1 chunk |
+3 lines, -3 lines |
0 comments
|
Download
|
 |
M |
components/history/core/browser/history_backend_unittest.cc
|
View
|
|
7 chunks |
+7 lines, -17 lines |
0 comments
|
Download
|
 |
M |
components/history/core/browser/history_client.h
|
View
|
|
2 chunks |
+11 lines, -47 lines |
0 comments
|
Download
|
 |
D |
components/history/core/browser/history_client.cc
|
View
|
|
1 chunk |
+0 lines, -51 lines |
0 comments
|
Download
|
 |
M |
components/history/core/browser/history_service.cc
|
View
|
|
2 chunks |
+3 lines, -1 line |
0 comments
|
Download
|
 |
M |
components/history/core/browser/thumbnail_database.h
|
View
|
|
3 chunks |
+3 lines, -3 lines |
0 comments
|
Download
|
 |
M |
components/history/core/browser/thumbnail_database.cc
|
View
|
|
4 chunks |
+6 lines, -6 lines |
0 comments
|
Download
|
 |
M |
components/history/core/browser/typed_url_syncable_service_unittest.cc
|
View
|
|
1 chunk |
+1 line, -0 lines |
0 comments
|
Download
|
 |
M |
components/history/core/test/history_backend_db_base_test.cc
|
View
|
|
1 chunk |
+1 line, -0 lines |
0 comments
|
Download
|
 |
M |
components/history/core/test/history_client_fake_bookmarks.h
|
View
|
|
2 chunks |
+14 lines, -6 lines |
0 comments
|
Download
|
 |
M |
components/history/core/test/history_client_fake_bookmarks.cc
|
View
|
|
1 chunk |
+144 lines, -13 lines |
0 comments
|
Download
|
Dependent Patchsets:
Total messages: 14 (8 generated)
|