Project: chromium Issues People Development process History Sign in
New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.
Issue 620927 Mojo communication in Mus/Ozone
Starred by 4 users Project Member Reported by rjkroege@chromium.org, Jun 17 2016 Back to list
Status: Started
Owner:
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug

Blocked on:
issue 649366

Blocking:
issue 581462



Sign in to add a comment
In split-mus, mus-ws needs to communicate with mus-gpu. Existing mus does this via PostTask because these components are in the same process. In split-mus, they are not. Use mojo to communicate between ws and gpu specific components so that they can be easily executed in both single and multiple process modes.

 
Cc: kylec...@chromium.org
Labels: -Pri-3 tadpole mustash mus Pri-2
Owner: rjkroege@chromium.org
Status: Started
Labels: screen
Project Member Comment 3 by bugdroid1@chromium.org, Jun 17 2016
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/e577e7306cf9eb00f5099747630ebd46e0613758

commit e577e7306cf9eb00f5099747630ebd46e0613758
Author: rjkroege <rjkroege@chromium.org>
Date: Fri Jun 17 20:05:13 2016

Add a flexible ozone configuration scheme

Make it possible to deliver additional parameters into an ozone
platform such as the mojo shell::Connector handle on platforms that
wish to use mojo.

BUG=620927

Review-Url: https://codereview.chromium.org/2079693002
Cr-Commit-Position: refs/heads/master@{#400478}

[modify] https://crrev.com/e577e7306cf9eb00f5099747630ebd46e0613758/ui/ozone/public/ozone_platform.cc
[modify] https://crrev.com/e577e7306cf9eb00f5099747630ebd46e0613758/ui/ozone/public/ozone_platform.h

Project Member Comment 4 by bugdroid1@chromium.org, Jun 21 2016
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/160cfd0956839fd92f053cf46db1de3613d831d9

commit 160cfd0956839fd92f053cf46db1de3613d831d9
Author: rjkroege <rjkroege@chromium.org>
Date: Tue Jun 21 20:47:16 2016

Ozone Mojo setup in mus

Configure the ozone platform with the necessary parameters to use mojo
if the platform should desire to do so.

BUG=620927

Review-Url: https://codereview.chromium.org/2079233003
Cr-Commit-Position: refs/heads/master@{#401102}

[modify] https://crrev.com/160cfd0956839fd92f053cf46db1de3613d831d9/components/mus/mus_app.cc

Project Member Comment 5 by bugdroid1@chromium.org, Jun 22 2016
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/6b5aaf6974e27f524ed8e6f87cfe511ff32c5c4e

commit 6b5aaf6974e27f524ed8e6f87cfe511ff32c5c4e
Author: rjkroege <rjkroege@chromium.org>
Date: Wed Jun 22 13:04:38 2016

Refactor ozone drm cursor code for mojo

Adjust code in ozone drm platform to simplify the introduction of mojo IPC for
cursor control.

BUG=620927

Review-Url: https://codereview.chromium.org/2088533002
Cr-Commit-Position: refs/heads/master@{#401265}

[modify] https://crrev.com/6b5aaf6974e27f524ed8e6f87cfe511ff32c5c4e/ui/events/ozone/evdev/cursor_delegate_evdev.h
[modify] https://crrev.com/6b5aaf6974e27f524ed8e6f87cfe511ff32c5c4e/ui/events/ozone/evdev/event_converter_evdev_impl_unittest.cc
[modify] https://crrev.com/6b5aaf6974e27f524ed8e6f87cfe511ff32c5c4e/ui/events/ozone/evdev/event_thread_evdev.cc
[modify] https://crrev.com/6b5aaf6974e27f524ed8e6f87cfe511ff32c5c4e/ui/events/ozone/evdev/input_injector_evdev_unittest.cc
[modify] https://crrev.com/6b5aaf6974e27f524ed8e6f87cfe511ff32c5c4e/ui/events/ozone/evdev/tablet_event_converter_evdev_unittest.cc
[modify] https://crrev.com/6b5aaf6974e27f524ed8e6f87cfe511ff32c5c4e/ui/ozone/platform/drm/gpu/drm_thread.cc
[modify] https://crrev.com/6b5aaf6974e27f524ed8e6f87cfe511ff32c5c4e/ui/ozone/platform/drm/gpu/drm_thread.h
[modify] https://crrev.com/6b5aaf6974e27f524ed8e6f87cfe511ff32c5c4e/ui/ozone/platform/drm/host/drm_cursor.cc
[modify] https://crrev.com/6b5aaf6974e27f524ed8e6f87cfe511ff32c5c4e/ui/ozone/platform/drm/host/drm_cursor.h
[modify] https://crrev.com/6b5aaf6974e27f524ed8e6f87cfe511ff32c5c4e/ui/ozone/platform/drm/host/drm_gpu_platform_support_host.cc
[modify] https://crrev.com/6b5aaf6974e27f524ed8e6f87cfe511ff32c5c4e/ui/ozone/platform/drm/mus_thread_proxy.cc
[modify] https://crrev.com/6b5aaf6974e27f524ed8e6f87cfe511ff32c5c4e/ui/ozone/platform/drm/mus_thread_proxy.h

Cc: j.iso...@samsung.com
Project Member Comment 8 by bugdroid1@chromium.org, Jul 25 2016
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/24836dd05bc3168a3b9833ea9ed0231419d07897

commit 24836dd05bc3168a3b9833ea9ed0231419d07897
Author: rjkroege <rjkroege@chromium.org>
Date: Mon Jul 25 20:33:08 2016

Update thread usage notes in CursorDelegateEvdev

Comments in CursorDelegateEvdev incorrectly specified thread usage
so update appropriately.

BUG=620927

Review-Url: https://codereview.chromium.org/2179123002
Cr-Commit-Position: refs/heads/master@{#407565}

[modify] https://crrev.com/24836dd05bc3168a3b9833ea9ed0231419d07897/ui/events/ozone/evdev/cursor_delegate_evdev.h

Blockedon: 626778
Project Member Comment 10 by bugdroid1@chromium.org, Jul 30 2016
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/19f44a11dcc5866c52e86bee2eab07149d06ddd0

commit 19f44a11dcc5866c52e86bee2eab07149d06ddd0
Author: rjkroege <rjkroege@chromium.org>
Date: Sat Jul 30 01:59:25 2016

Use mojo for cursor control in ozone drm

Use mojo to control the cursor in ozone drm in mus. First in a series of patches to
convert Ozone IPC to mojo.

BUG=620927

Review-Url: https://codereview.chromium.org/2156093004
Cr-Commit-Position: refs/heads/master@{#408840}

[modify] https://crrev.com/19f44a11dcc5866c52e86bee2eab07149d06ddd0/services/ui/common/run_all_shelltests.cc
[modify] https://crrev.com/19f44a11dcc5866c52e86bee2eab07149d06ddd0/services/ui/gpu/gpu_service_mus.cc
[modify] https://crrev.com/19f44a11dcc5866c52e86bee2eab07149d06ddd0/services/ui/manifest.json
[modify] https://crrev.com/19f44a11dcc5866c52e86bee2eab07149d06ddd0/services/ui/service.cc
[modify] https://crrev.com/19f44a11dcc5866c52e86bee2eab07149d06ddd0/ui/gl/test/run_all_unittests.cc
[modify] https://crrev.com/19f44a11dcc5866c52e86bee2eab07149d06ddd0/ui/ozone/BUILD.gn
[modify] https://crrev.com/19f44a11dcc5866c52e86bee2eab07149d06ddd0/ui/ozone/platform/drm/BUILD.gn
[modify] https://crrev.com/19f44a11dcc5866c52e86bee2eab07149d06ddd0/ui/ozone/platform/drm/DEPS
[add] https://crrev.com/19f44a11dcc5866c52e86bee2eab07149d06ddd0/ui/ozone/platform/drm/cursor_proxy_mojo.cc
[add] https://crrev.com/19f44a11dcc5866c52e86bee2eab07149d06ddd0/ui/ozone/platform/drm/cursor_proxy_mojo.h
[modify] https://crrev.com/19f44a11dcc5866c52e86bee2eab07149d06ddd0/ui/ozone/platform/drm/gpu/drm_thread.cc
[modify] https://crrev.com/19f44a11dcc5866c52e86bee2eab07149d06ddd0/ui/ozone/platform/drm/gpu/drm_thread.h
[modify] https://crrev.com/19f44a11dcc5866c52e86bee2eab07149d06ddd0/ui/ozone/platform/drm/gpu/drm_thread_proxy.cc
[modify] https://crrev.com/19f44a11dcc5866c52e86bee2eab07149d06ddd0/ui/ozone/platform/drm/gpu/drm_thread_proxy.h
[modify] https://crrev.com/19f44a11dcc5866c52e86bee2eab07149d06ddd0/ui/ozone/platform/drm/host/drm_cursor.h
[modify] https://crrev.com/19f44a11dcc5866c52e86bee2eab07149d06ddd0/ui/ozone/platform/drm/mus_thread_proxy.cc
[modify] https://crrev.com/19f44a11dcc5866c52e86bee2eab07149d06ddd0/ui/ozone/platform/drm/mus_thread_proxy.h
[modify] https://crrev.com/19f44a11dcc5866c52e86bee2eab07149d06ddd0/ui/ozone/platform/drm/ozone_platform_gbm.cc
[add] https://crrev.com/19f44a11dcc5866c52e86bee2eab07149d06ddd0/ui/ozone/public/interfaces/BUILD.gn
[add] https://crrev.com/19f44a11dcc5866c52e86bee2eab07149d06ddd0/ui/ozone/public/interfaces/OWNERS
[add] https://crrev.com/19f44a11dcc5866c52e86bee2eab07149d06ddd0/ui/ozone/public/interfaces/device_cursor.mojom

Blockedon: -626778
Components: MUS
Blockedon: 649366
Components: Internals>MUS
Labels: Proj-Mustash
Cc: fsam...@chromium.org rjkroege@chromium.org sadrul@chromium.org
Issue 611506 has been merged into this issue.
Blocking: 581462
Components: -MUS -Internals>MUS Internals>GPU>Internals
Labels: -mus -mustash -tadpole Proj-Mustash-Milestone-Tadpole Proj-Ozone-DRM Proj-Mustash-Mus-GPU
Blocking:
Sign in to add a comment