|
Revert of Move content's shell connections to the IO thread (patchset #9 id:220001 of https://codereview.chromium.org/2111353002/ )
Reason for revert:
stupid build failure
Original issue's description:
> Move content's shell connections to the IO thread
>
> Moves all shell connections in the content layer
> (connections owned by instances of MojoShellConnection and
> MojoChildConnection) to the IO thread in all process types.
> Note that instances of MojoShellConnection and
> MojoChildConnection may still be created and used on
> any thread.
>
> Additionally exposes a way for InterfaceRegistry and
> InterfaceProvider to forward to/from other instances
> so we can still have a corresponding instance of each
> on whatever thread owns the Mojo[Shell/Child]Connection
> object.
>
> This simplifies interface registration and acquisition
> on process hosts and ChildThreadImpls, and makes it possible
> for a future CL to introduce IO-thread service registration
> to avoid thread hops when binding incoming interface requests.
>
> BUG= 623398, 612500
>
> Committed: https://crrev.com/439768f0aaaea83ffb7ce6c3ba4438804582c383
> Cr-Commit-Position: refs/heads/master@{#404228}
TBR=ben@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG= 623398, 612500
Committed: https://crrev.com/4a93a86045c9ed89385d13a6f6d154fadd2beaab
Cr-Commit-Position: refs/heads/master@{#404236}
|
Unified diffs |
Side-by-side diffs |
Delta from patch set |
Stats (+559 lines, -1172 lines) |
Patch |
 |
M |
chrome/browser/chrome_content_browser_client.cc
|
View
|
|
2 chunks |
+2 lines, -3 lines |
0 comments
|
Download
|
 |
M |
chrome/browser/chromeos/chrome_interface_factory.h
|
View
|
|
1 chunk |
+46 lines, -9 lines |
0 comments
|
Download
|
 |
M |
chrome/browser/chromeos/chrome_interface_factory.cc
|
View
|
|
3 chunks |
+39 lines, -100 lines |
0 comments
|
Download
|
 |
M |
chrome/test/base/mash_browser_tests_main.cc
|
View
|
|
2 chunks |
+1 line, -4 lines |
0 comments
|
Download
|
 |
M |
content/browser/browser_context.cc
|
View
|
|
2 chunks |
+1 line, -4 lines |
0 comments
|
Download
|
 |
M |
content/browser/browser_main_loop.cc
|
View
|
|
1 chunk |
+2 lines, -2 lines |
0 comments
|
Download
|
 |
M |
content/browser/gpu/gpu_process_host.cc
|
View
|
|
2 chunks |
+6 lines, -5 lines |
0 comments
|
Download
|
 |
M |
content/browser/mojo/mojo_child_connection.h
|
View
|
|
3 chunks |
+5 lines, -26 lines |
0 comments
|
Download
|
 |
M |
content/browser/mojo/mojo_child_connection.cc
|
View
|
|
2 chunks |
+27 lines, -148 lines |
0 comments
|
Download
|
 |
M |
content/browser/mojo/mojo_shell_context.cc
|
View
|
|
4 chunks |
+23 lines, -15 lines |
0 comments
|
Download
|
 |
M |
content/browser/renderer_host/render_process_host_impl.h
|
View
|
|
1 chunk |
+1 line, -0 lines |
0 comments
|
Download
|
 |
M |
content/browser/renderer_host/render_process_host_impl.cc
|
View
|
|
6 chunks |
+16 lines, -12 lines |
0 comments
|
Download
|
 |
M |
content/browser/renderer_host/render_widget_host_view_mus.cc
|
View
|
|
1 chunk |
+1 line, -1 line |
0 comments
|
Download
|
 |
M |
content/browser/utility_process_host_impl.cc
|
View
|
|
2 chunks |
+5 lines, -6 lines |
0 comments
|
Download
|
 |
M |
content/child/background_sync/background_sync_provider.cc
|
View
|
|
1 chunk |
+0 lines, -1 line |
0 comments
|
Download
|
 |
M |
content/child/child_thread_impl.h
|
View
|
|
6 chunks |
+9 lines, -10 lines |
0 comments
|
Download
|
 |
M |
content/child/child_thread_impl.cc
|
View
|
|
8 chunks |
+17 lines, -24 lines |
0 comments
|
Download
|
 |
M |
content/common/mojo/mojo_shell_connection_impl.h
|
View
|
|
1 chunk |
+33 lines, -38 lines |
0 comments
|
Download
|
 |
M |
content/common/mojo/mojo_shell_connection_impl.cc
|
View
|
|
4 chunks |
+108 lines, -357 lines |
0 comments
|
Download
|
 |
M |
content/content_common.gypi
|
View
|
|
1 chunk |
+0 lines, -1 line |
0 comments
|
Download
|
 |
M |
content/gpu/BUILD.gn
|
View
|
|
1 chunk |
+0 lines, -1 line |
0 comments
|
Download
|
 |
M |
content/gpu/gpu_child_thread.h
|
View
|
|
6 chunks |
+18 lines, -1 line |
0 comments
|
Download
|
 |
M |
content/gpu/gpu_child_thread.cc
|
View
|
|
8 chunks |
+43 lines, -55 lines |
0 comments
|
Download
|
 |
M |
content/public/browser/render_process_host.h
|
View
|
|
1 chunk |
+3 lines, -0 lines |
0 comments
|
Download
|
 |
M |
content/public/child/child_thread.h
|
View
|
|
2 chunks |
+1 line, -3 lines |
0 comments
|
Download
|
 |
D |
content/public/common/connection_filter.h
|
View
|
|
1 chunk |
+0 lines, -40 lines |
0 comments
|
Download
|
 |
M |
content/public/common/mojo_shell_connection.h
|
View
|
|
4 chunks |
+11 lines, -43 lines |
0 comments
|
Download
|
 |
M |
content/public/test/mock_render_process_host.h
|
View
|
|
1 chunk |
+1 line, -0 lines |
0 comments
|
Download
|
 |
M |
content/public/test/mock_render_process_host.cc
|
View
|
|
1 chunk |
+4 lines, -0 lines |
0 comments
|
Download
|
 |
M |
content/renderer/mus/render_widget_window_tree_client_factory.h
|
View
|
|
1 chunk |
+1 line, -3 lines |
0 comments
|
Download
|
 |
M |
content/renderer/mus/render_widget_window_tree_client_factory.cc
|
View
|
|
4 chunks |
+11 lines, -27 lines |
0 comments
|
Download
|
 |
M |
content/renderer/render_thread_impl.h
|
View
|
|
1 chunk |
+0 lines, -3 lines |
0 comments
|
Download
|
 |
M |
content/renderer/render_thread_impl.cc
|
View
|
|
3 chunks |
+9 lines, -11 lines |
0 comments
|
Download
|
 |
M |
content/renderer/renderer_blink_platform_impl.cc
|
View
|
|
1 chunk |
+0 lines, -1 line |
0 comments
|
Download
|
 |
M |
content/renderer/service_worker/service_worker_context_client.cc
|
View
|
|
1 chunk |
+0 lines, -2 lines |
0 comments
|
Download
|
 |
M |
services/navigation/BUILD.gn
|
View
|
|
1 chunk |
+2 lines, -7 lines |
0 comments
|
Download
|
 |
M |
services/navigation/content_client/content_browser_client.cc
|
View
|
|
1 chunk |
+2 lines, -2 lines |
0 comments
|
Download
|
 |
M |
services/navigation/navigation.h
|
View
|
|
4 chunks |
+6 lines, -9 lines |
0 comments
|
Download
|
 |
M |
services/navigation/navigation.cc
|
View
|
|
2 chunks |
+10 lines, -29 lines |
0 comments
|
Download
|
 |
M |
services/navigation/view_impl.h
|
View
|
|
3 chunks |
+2 lines, -4 lines |
0 comments
|
Download
|
 |
M |
services/navigation/view_impl.cc
|
View
|
|
4 chunks |
+5 lines, -7 lines |
0 comments
|
Download
|
 |
M |
services/shell/public/cpp/connection.h
|
View
|
|
1 chunk |
+1 line, -1 line |
0 comments
|
Download
|
 |
M |
services/shell/public/cpp/connector.h
|
View
|
|
1 chunk |
+0 lines, -4 lines |
0 comments
|
Download
|
 |
M |
services/shell/public/cpp/interface_provider.h
|
View
|
|
3 chunks |
+0 lines, -15 lines |
0 comments
|
Download
|
 |
M |
services/shell/public/cpp/interface_registry.h
|
View
|
|
4 chunks |
+0 lines, -19 lines |
0 comments
|
Download
|
 |
M |
services/shell/public/cpp/lib/connector_impl.h
|
View
|
|
1 chunk |
+1 line, -3 lines |
0 comments
|
Download
|
 |
M |
services/shell/public/cpp/lib/connector_impl.cc
|
View
|
|
4 chunks |
+20 lines, -42 lines |
0 comments
|
Download
|
 |
M |
services/shell/public/cpp/lib/interface_provider.cc
|
View
|
|
2 chunks |
+1 line, -21 lines |
0 comments
|
Download
|
 |
M |
services/shell/public/cpp/lib/interface_registry.cc
|
View
|
|
2 chunks |
+6 lines, -17 lines |
0 comments
|
Download
|
 |
M |
services/shell/public/cpp/lib/shell_connection.cc
|
View
|
|
5 chunks |
+44 lines, -25 lines |
0 comments
|
Download
|
 |
M |
services/shell/public/cpp/shell_connection.h
|
View
|
|
3 chunks |
+15 lines, -11 lines |
0 comments
|
Download
|
Total messages: 7 (2 generated)
|