-
Dan Erat authored
This reverts commit 7faa3b0c. Reason for revert: This appears to cause Chrome crashes in the desktopui_MashLogin Autotest, resulting in failures in the informational Chrome PFQ builders: https://crbug.com/804052 Original change's description: > [ozone/drm]: support content and service manager launching > > The existing in-progress implementation of ozone/drm over mojo IPC > assumed that the ozone host would use a mojo ServiceManager provided > at initialization that it could use to launch and connect to the DRM > and cursor services. However, when used in a src/content context (such > as would be the case in Chrome), there is no service manager available > at ozone initialization time. Instead, content::GpuProcessHost > notifies the ozone host later from the browser I/O thread when the GPU > process containing the DRM and cursor services is running. > > This CL refactors the ozone/drm/mojo implementation to isolate the > acquisition of mojo pipes into a separate class that supports both > styles of pipe acquistion: either by the use of a ServiceManager or > via an asynchronously injected dependency on content. > > This CL also restructures the launch of the DRM thread itself so that > it can be deferred until after the GPU sandbox is initialized in a way > that does not depend on the use of legacy IPC message filters. > > BUG=620927 > > Cq-Include-Trybots: master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel > Change-Id: Icfb2261326cd1d5d6e2769d9801c02a744e1102a > Reviewed-on: https://chromium-review.googlesource.com/861749 > Commit-Queue: Robert Kroeger <rjkroege@chromium.org> > Reviewed-by: Daniel Nicoara <dnicoara@chromium.org> > Reviewed-by: Yuzhu Shen <yzshen@chromium.org> > Reviewed-by: Antoine Labour <piman@chromium.org> > Cr-Commit-Position: refs/heads/master@{#530562} TBR=rjkroege@chromium.org,yzshen@chromium.org,dnicoara@chromium.org,piman@chromium.org Change-Id: Ieb533a2c3db1634953437fcd4c34200f38e51601 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 620927 Cq-Include-Trybots: master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel Reviewed-on: https://chromium-review.googlesource.com/876936Reviewed-by:
Dan Erat <derat@chromium.org> Commit-Queue: Dan Erat <derat@chromium.org> Cr-Commit-Position: refs/heads/master@{#530712}
062531a9