• Dan Erat's avatar
    Revert "[ozone/drm]: support content and service manager launching" · 062531a9
    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: default avatarDan Erat <derat@chromium.org>
    Commit-Queue: Dan Erat <derat@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#530712}
    062531a9
ozone_platform.cc 3.43 KB