Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(426)

Issues Search
    My Issues | Starred     Open | Closed | All

Issue 2771553003: Allow mojo media interfaces to be obtained from other threads. (Closed)

Created:
3 years, 9 months ago by servolk
Modified:
3 years, 8 months ago
Reviewers:
CC:
chromium-reviews, mlamouri+watch-content_chromium.org, posciak+watch_chromium.org, chfremer+watch_chromium.org, jam, feature-media-reviews_chromium.org, darin-cc_chromium.org
Target Ref:
refs/heads/master
Project:
chromium
Visibility:
Public.

Description

Allow mojo media interfaces to be obtained from other threads. Currently mojo media objects are always created on the main thread, but this change allows them to be created from arbitrary thread, for example the media thread.

Patch Set 1 #

Patch Set 2 : buildfix #

Patch Set 3 : buildfix #

Patch Set 4 : fix #

Patch Set 5 : Use base::Passed #

Unified diffs Side-by-side diffs Delta from patch set Stats (+15 lines, -1 line) Patch
M content/renderer/media/media_interface_provider.h View 1 2 chunks +5 lines, -0 lines 0 comments Download
M content/renderer/media/media_interface_provider.cc View 1 2 3 4 2 chunks +10 lines, -1 line 0 comments Download

Messages

Total messages: 15 (15 generated)
servolk
The CQ bit was checked by servolk@chromium.org to run a CQ dry run
3 years, 9 months ago (2017-03-22 23:29:08 UTC) #1
commit-bot: I haz the power
Dry run: CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2771553003/1
3 years, 9 months ago (2017-03-22 23:29:49 UTC) #2
servolk
The CQ bit was checked by servolk@chromium.org to run a CQ dry run
3 years, 9 months ago (2017-03-22 23:45:46 UTC) #3
commit-bot: I haz the power
Dry run: CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2771553003/20001
3 years, 9 months ago (2017-03-22 23:46:32 UTC) #4
servolk
The CQ bit was checked by servolk@chromium.org to run a CQ dry run
3 years, 9 months ago (2017-03-23 00:00:15 UTC) #5
commit-bot: I haz the power
Dry run: CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2771553003/40001
3 years, 9 months ago (2017-03-23 00:00:28 UTC) #6
servolk
The CQ bit was checked by servolk@chromium.org to run a CQ dry run
3 years, 9 months ago (2017-03-23 00:15:56 UTC) #7
commit-bot: I haz the power
Dry run: CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2771553003/60001
3 years, 9 months ago (2017-03-23 00:16:37 UTC) #8
commit-bot: I haz the power
The CQ bit was unchecked by commit-bot@chromium.org
3 years, 9 months ago (2017-03-23 00:35:39 UTC) #9
commit-bot: I haz the power
Dry run: Try jobs failed on following builders: cast_shell_linux on master.tryserver.chromium.linux (JOB_FAILED, http://build.chromium.org/p/tryserver.chromium.linux/builders/cast_shell_linux/builds/332221)
3 years, 9 months ago (2017-03-23 00:35:40 UTC) #10
servolk
The CQ bit was checked by servolk@chromium.org to run a CQ dry run
3 years, 9 months ago (2017-03-23 01:20:00 UTC) #11
commit-bot: I haz the power
Dry run: CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2771553003/80001
3 years, 9 months ago (2017-03-23 01:20:25 UTC) #12
servolk
Description was changed from ========== Allow mojo media interfaces to be obtained from other threads. ...
3 years, 9 months ago (2017-03-23 01:25:12 UTC) #13
commit-bot: I haz the power
The CQ bit was unchecked by commit-bot@chromium.org
3 years, 9 months ago (2017-03-23 02:26:00 UTC) #14
commit-bot: I haz the power
3 years, 9 months ago (2017-03-23 02:26:00 UTC) #15 
Dry run: This issue passed the CQ dry run.

Powered by Google App Engine
This is Rietveld 408576698