Split the media context from the compositor worker context.

Split the media context from the compositor worker context.

Give the media system its own ContextProviderCommandBuffer, so that
we can move ownership of the compositor raster worker contex to the
compositor thread along with the output surface.

R=piman
BUG=608946
Committed: https://crrev.com/0b4b94e39d65563ab62d634f02e2fb7a207b587f
Cr-Commit-Position: refs/heads/master@{#392750}

[danakj, 2016-05-06 22:10:54]

Description was changed from

==========
Split the media context from the compositor worker context.

Give the media system its own ContextProviderCommandBuffer, so that
we can move ownership of the compositor raster worker contex to the
compositor thread.

R=piman
BUG=608946
==========

to

==========
Split the media context from the compositor worker context.

Give the media system its own ContextProviderCommandBuffer, so that
we can move ownership of the compositor raster worker contex to the
compositor thread along with the output surface.

R=piman
BUG=608946
==========

[danakj, 2016-05-06 22:11:03]

The CQ bit was checked by danakj@chromium.org to run a CQ dry run

[danakj, 2016-05-06 22:11:28]

danakj@chromium.org changed reviewers:
+ holte@chromium.org

[danakj, 2016-05-06 22:11:29]

+holte@chromium.org for histograms

[commit-bot: I haz the power, 2016-05-06 22:11:35]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1960803002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1960803002/1

[commit-bot: I haz the power, 2016-05-06 23:08:57]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-05-06 23:08:58]

Dry run: Try jobs failed on following builders:
  cast_shell_linux on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/cast_shell_linu...)

[piman, 2016-05-06 23:38:57]

https://codereview.chromium.org/1960803002/diff/1/content/renderer/render_thr...
File content/renderer/render_thread_impl.cc (right):

https://codereview.chromium.org/1960803002/diff/1/content/renderer/render_thr...
content/renderer/render_thread_impl.cc:1410: if
(!shared_worker_context_provider_->BindToCurrentThread())
Did you mean shared_context_provider here?

[danakj, 2016-05-09 17:25:39]

PTAL. It's media_context_provider now.

https://codereview.chromium.org/1960803002/diff/1/content/renderer/render_thr...
File content/renderer/render_thread_impl.cc (right):

https://codereview.chromium.org/1960803002/diff/1/content/renderer/render_thr...
content/renderer/render_thread_impl.cc:1410: if
(!shared_worker_context_provider_->BindToCurrentThread())
On 2016/05/06 23:38:57, piman wrote:
> Did you mean shared_context_provider here?

I need to rename this, cuz ya I did.

[danakj, 2016-05-09 17:25:56]

The CQ bit was checked by danakj@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-05-09 17:26:20]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1960803002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1960803002/20001

[piman, 2016-05-09 17:27:47]

lgtm

[commit-bot: I haz the power, 2016-05-09 18:24:09]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-05-09 18:24:10]

Dry run: This issue passed the CQ dry run.

[Steven Holte, 2016-05-09 23:30:15]

histograms lgtm

[danakj, 2016-05-10 20:19:24]

The CQ bit was checked by danakj@chromium.org

[commit-bot: I haz the power, 2016-05-10 20:20:17]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1960803002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1960803002/20001

[commit-bot: I haz the power, 2016-05-10 20:33:08]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-05-10 20:33:09]

Try jobs failed on following builders:
  chromeos_x86-generic_chromium_compile_only_ng on tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_x86-ge...)

[danakj, 2016-05-10 21:05:00]

Streams were added in the meantime. I made the media context get "default"
stream like the main thread context. Please verify this is correct, instead of
sharing stream with the compositor?

[danakj, 2016-05-10 21:06:14]

On 2016/05/10 21:05:00, danakj wrote:
> Streams were added in the meantime. I made the media context get "default"
> stream like the main thread context. Please verify this is correct, instead of
> sharing stream with the compositor?

s/compositor/raster worker/

[danakj, 2016-05-10 21:06:20]

The CQ bit was checked by danakj@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-05-10 21:06:55]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1960803002/60001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1960803002/60001

[piman, 2016-05-10 21:07:27]

LGTM, correct.

[danakj, 2016-05-10 21:46:28]

The CQ bit was unchecked by danakj@chromium.org

[danakj, 2016-05-10 21:46:32]

The CQ bit was checked by danakj@chromium.org

[danakj, 2016-05-10 21:46:33]

The patchset sent to the CQ was uploaded after l-g-t-m from holte@chromium.org
Link to the patchset: https://codereview.chromium.org/1960803002/#ps60001
(title: "workerthread: streams")

[commit-bot: I haz the power, 2016-05-10 21:46:59]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1960803002/60001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1960803002/60001

[commit-bot: I haz the power, 2016-05-10 22:33:19]

Description was changed from

==========
Split the media context from the compositor worker context.

Give the media system its own ContextProviderCommandBuffer, so that
we can move ownership of the compositor raster worker contex to the
compositor thread along with the output surface.

R=piman
BUG=608946
==========

to

==========
Split the media context from the compositor worker context.

Give the media system its own ContextProviderCommandBuffer, so that
we can move ownership of the compositor raster worker contex to the
compositor thread along with the output surface.

R=piman
BUG=608946
==========

[commit-bot: I haz the power, 2016-05-10 22:33:20]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2016-05-10 22:36:08]

Description was changed from

==========
Split the media context from the compositor worker context.

Give the media system its own ContextProviderCommandBuffer, so that
we can move ownership of the compositor raster worker contex to the
compositor thread along with the output surface.

R=piman
BUG=608946
==========

to

==========
Split the media context from the compositor worker context.

Give the media system its own ContextProviderCommandBuffer, so that
we can move ownership of the compositor raster worker contex to the
compositor thread along with the output surface.

R=piman
BUG=608946

Committed: https://crrev.com/0b4b94e39d65563ab62d634f02e2fb7a207b587f
Cr-Commit-Position: refs/heads/master@{#392750}
==========

[commit-bot: I haz the power, 2016-05-10 22:36:10]

Patchset 4 (id:??) landed as
https://crrev.com/0b4b94e39d65563ab62d634f02e2fb7a207b587f
Cr-Commit-Position: refs/heads/master@{#392750}