Implement MusGpuMemoryBufferManager

Implement MusGpuMemoryBufferManager.

Implement the MusGpuMemoryBufferManager which is used by MUS
locally. It supports allocating Native GpuMemoryBuffer(Ozone, etc).
It is needed for ChromeBook/ozone native rendering.

BUG=618316
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel;tryserver.chromium.win:win_optional_gpu_tests_rel

Committed: https://crrev.com/cab01d28306bdec109b957d17d64d68677d9d3ed
Cr-Commit-Position: refs/heads/master@{#399390}

[Peng, 2016-06-08 14:40:20]

Description was changed from

==========
WIP Implement MusGpuMemoryBufferManager

BUG=618316
==========

to

==========
WIP Implement MusGpuMemoryBufferManager

BUG=618316
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel;tryserver.chromium.win:win_optional_gpu_tests_rel
==========

[Peng, 2016-06-08 18:04:22]

Description was changed from

==========
WIP Implement MusGpuMemoryBufferManager

BUG=618316
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel;tryserver.chromium.win:win_optional_gpu_tests_rel
==========

to

==========
Implement MusGpuMemoryBufferManager.

Implement the MusGpuMemoryBufferManager which is used by MUS
locally. It supports allocating Native GpuMemoryBuffer(Ozone, etc). With it, the
MUS can use Chrome IPC command buffer on Pixel.

BUG=618316
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel;tryserver.chromium.win:win_optional_gpu_tests_rel
==========

[Peng, 2016-06-08 18:04:43]

Description was changed from

==========
Implement MusGpuMemoryBufferManager.

Implement the MusGpuMemoryBufferManager which is used by MUS
locally. It supports allocating Native GpuMemoryBuffer(Ozone, etc). With it, the
MUS can use Chrome IPC command buffer on Pixel.

BUG=618316
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel;tryserver.chromium.win:win_optional_gpu_tests_rel
==========

to

==========
Implement MusGpuMemoryBufferManager.

Implement the MusGpuMemoryBufferManager which is used by MUS
locally. It supports allocating Native GpuMemoryBuffer(Ozone, etc).
With it, the MUS can use Chrome IPC command buffer on Pixel.

BUG=618316
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel;tryserver.chromium.win:win_optional_gpu_tests_rel
==========

[Peng, 2016-06-08 18:07:47]

Description was changed from

==========
Implement MusGpuMemoryBufferManager.

Implement the MusGpuMemoryBufferManager which is used by MUS
locally. It supports allocating Native GpuMemoryBuffer(Ozone, etc).
With it, the MUS can use Chrome IPC command buffer on Pixel.

BUG=618316
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel;tryserver.chromium.win:win_optional_gpu_tests_rel
==========

to

==========
Implement MusGpuMemoryBufferManager.

Implement the MusGpuMemoryBufferManager which is used by MUS
locally. It supports allocating Native GpuMemoryBuffer(Ozone, etc). It is needed
for ChromeBook/ozone
native rendering.

BUG=618316
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel;tryserver.chromium.win:win_optional_gpu_tests_rel
==========

[Peng, 2016-06-08 18:08:02]

Description was changed from

==========
Implement MusGpuMemoryBufferManager.

Implement the MusGpuMemoryBufferManager which is used by MUS
locally. It supports allocating Native GpuMemoryBuffer(Ozone, etc). It is needed
for ChromeBook/ozone
native rendering.

BUG=618316
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel;tryserver.chromium.win:win_optional_gpu_tests_rel
==========

to

==========
Implement MusGpuMemoryBufferManager.

Implement the MusGpuMemoryBufferManager which is used by MUS
locally. It supports allocating Native GpuMemoryBuffer(Ozone, etc).
It is needed for ChromeBook/ozone native rendering.

BUG=618316
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel;tryserver.chromium.win:win_optional_gpu_tests_rel
==========

[Peng, 2016-06-08 18:22:29]

Patchset #4 (id:60001) has been deleted

[Peng, 2016-06-08 18:28:34]

Patchset #4 (id:80001) has been deleted

[Peng, 2016-06-08 18:42:18]

penghuang@chromium.org changed reviewers:
+ rjkroege@chromium.org

[Peng, 2016-06-08 18:42:19]

Hi Robert, PTAL. Thanks.

[rjkroege, 2016-06-08 21:16:43]

LGTM with nit.

(I presumed that you tested it on device. :-)

https://codereview.chromium.org/2049083002/diff/100001/components/mus/gpu/gpu...
File components/mus/gpu/gpu_service_mus.cc (right):

https://codereview.chromium.org/2049083002/diff/100001/components/mus/gpu/gpu...
components/mus/gpu/gpu_service_mus.cc:214: #if defined(USE_OZONE)
It should probably be at the top. The CreateNativeType() may someday depend on
ozone already being initialized.

[Peng, 2016-06-08 21:40:36]

https://codereview.chromium.org/2049083002/diff/100001/components/mus/gpu/gpu...
File components/mus/gpu/gpu_service_mus.cc (right):

https://codereview.chromium.org/2049083002/diff/100001/components/mus/gpu/gpu...
components/mus/gpu/gpu_service_mus.cc:214: #if defined(USE_OZONE)
On 2016/06/08 21:16:43, rjkroege wrote:
> It should probably be at the top. The CreateNativeType() may someday depend on
> ozone already being initialized.

Done.

[Peng, 2016-06-08 21:41:16]

On 2016/06/08 21:16:43, rjkroege wrote:
> LGTM with nit.
> 
> (I presumed that you tested it on device. :-)
Yes. Tested on Pixel. It works. :)

> 
>
https://codereview.chromium.org/2049083002/diff/100001/components/mus/gpu/gpu...
> File components/mus/gpu/gpu_service_mus.cc (right):
> 
>
https://codereview.chromium.org/2049083002/diff/100001/components/mus/gpu/gpu...
> components/mus/gpu/gpu_service_mus.cc:214: #if defined(USE_OZONE)
> It should probably be at the top. The CreateNativeType() may someday depend on
> ozone already being initialized.

[Peng, 2016-06-08 21:43:15]

+piman for omponents/mus/gpu/DEPS, welcome to review other files as well.

Hi Antoine, PTAL. Thanks.

[Peng, 2016-06-08 21:43:33]

penghuang@chromium.org changed reviewers:
+ piman@chromium.org

[Peng, 2016-06-08 21:44:04]

On 2016/06/08 21:43:34, Peng wrote:

+piman for omponents/mus/gpu/DEPS, welcome to review other files as well.

Hi Antoine, PTAL. Thanks.

[Peng, 2016-06-09 15:05:15]

penghuang@chromium.org changed reviewers:
+ jam@chromium.org

[Peng, 2016-06-09 15:05:16]

+jam for base/threading/* & components/mus/common/*

Hi John, PTAL. Thanks.

[jam, 2016-06-09 16:01:46]

On 2016/06/09 15:05:16, Peng wrote:
> +jam for base/threading/* & components/mus/common/*

please pick a components/mus owner
> 
> Hi John, PTAL. Thanks.

https://codereview.chromium.org/2049083002/diff/140001/components/mus/common/...
File components/mus/common/gpu_service.cc (right):

https://codereview.chromium.org/2049083002/diff/140001/components/mus/common/...
components/mus/common/gpu_service.cc:78:
base::ThreadRestrictions::ScopedAllowWait allow_wait;
why is this needed? i.e. what's the callstack that triggers the assert in base?

it seems that if this is inside mojo, we may want to put the thread restriction
there.

[Peng, 2016-06-09 17:40:29]

https://codereview.chromium.org/2049083002/diff/140001/components/mus/common/...
File components/mus/common/gpu_service.cc (right):

https://codereview.chromium.org/2049083002/diff/140001/components/mus/common/...
components/mus/common/gpu_service.cc:78:
base::ThreadRestrictions::ScopedAllowWait allow_wait;
On 2016/06/09 16:01:45, jam wrote:
> why is this needed? i.e. what's the callstack that triggers the assert in
base?
> 
> it seems that if this is inside mojo, we may want to put the thread
restriction
> there.

The call stack is listed below. I don't think there is a good place in mojo to
add the ScopedAllowWait. WDYT?

[32510:32510:0609/133601:ERROR:session_manager_client.cc(657)] Failed to call
GetServerBackedStateKeys
[32510:32510:0609/133601:FATAL:thread_restrictions.cc(72)] Waiting is not
allowed to be used on this thread to prevent jank and deadlock.
#0 0x7f16549bd8b6 base::debug::StackTrace::StackTrace()
#1 0x7f16549def6e logging::LogMessage::~LogMessage()
#2 0x7f1654a4f343 base::ThreadRestrictions::AssertWaitAllowed()
#3 0x7f1654a36fd9 base::ConditionVariable::Wait()
#4 0x7f164fdf6148 mojo::edk::Waiter::Wait()
#5 0x7f164fdd772f mojo::edk::Core::WaitManyInternal()
#6 0x7f164fdd7ac3 mojo::edk::Core::Wait()
#7 0x7f164f4b613a mojo::internal::Connector::WaitForIncomingMessage()
#8 0x7f1650cfba5e mus::GpuService::EstablishGpuChannel()
#9 0x7f1650cdfcf7 mus::GLES2Context::Initialize()
#10 0x7f1650ce0b68 mus::GLES2Context::CreateOffscreenContext()
#11 0x7f1650cdef55 mus::ContextProvider::BindToCurrentThread()
#12 0x7f164ee5c3fd cc::OutputSurface::BindToClient()
#13 0x7f164ef11535 cc::LayerTreeHostImpl::InitializeRenderer()
#14 0x7f164ef4e39e cc::SingleThreadProxy::SetOutputSurface()
#15 0x7f164eeff435 cc::LayerTreeHost::SetOutputSurface()
#16 0x7f164edb8f19 ui::Compositor::SetOutputSurface()
#17 0x7f16523f7376 views::SurfaceContextFactory::CreateOutputSurface()
#18 0x7f164edb8cc1 ui::Compositor::RequestNewOutputSurface()
#19 0x7f164ef4de24 cc::SingleThreadProxy::RequestNewOutputSurface()
#20 0x7f164ef4dcc5
_ZN4base8internal7InvokerINS_13IndexSequenceIJLm0EEEENS0_9BindStateINS0_15RunnableAdapterIMN2cc17SingleThreadProxyEFvvEEEFvPS7_EJNS_7WeakPtrIS7_EEEEELb1EFvvEE3RunEPNS0_13BindStateBaseE
#21 0x7f164d6ae435
_ZN4base8internal7InvokerINS_13IndexSequenceIJLm0EEEENS0_9BindStateINS0_15RunnableAdapterIMNS_18CancelableCallbackIFvvEEEKFvvEEEFvPKS8_EJNS_7WeakPtrIS8_EEEEELb1ES7_E3RunEPNS0_13BindStateBaseE
#22 0x7f16549bf3aa base::debug::TaskAnnotator::RunTask()
#23 0x7f16549ebc52 base::MessageLoop::RunTask()
#24 0x7f16549ec0ae base::MessageLoop::DeferOrRunPendingTask()
#25 0x7f16549ede44 base::MessageLoop::DoWork()
#26 0x7f16549f0739 base::MessagePumpLibevent::Run()
#27 0x7f16549eced9 base::MessageLoop::RunHandler()
#28 0x7f1654a22428 base::RunLoop::Run()
#29 0x7f164d3b41f7 ChromeBrowserMainParts::MainMessageLoopRun()
#30 0x7f164f5bada0 content::BrowserMainLoop::RunMainMessageLoopParts()
#31 0x7f164f5bd9c0 content::BrowserMainRunnerImpl::Run()
#32 0x7f164f5b6279 content::BrowserMain()
#33 0x7f16548d8500 content::RunNamedProcessTypeMain()
#34 0x7f16548d8645 content::ContentMainRunnerImpl::Run()
#35 0x7f16548d6299 content::ContentMain()
#36 0x7f164d357d42 ChromeMain
#37 0x7f164ae17fb6 __libc_start_main
#38 0x7f164d357b7d <unknown>

[piman, 2016-06-09 19:31:33]

lgtm

https://codereview.chromium.org/2049083002/diff/140001/components/mus/gpu/mus...
File components/mus/gpu/mus_gpu_memory_buffer_manager.cc (right):

https://codereview.chromium.org/2049083002/diff/140001/components/mus/gpu/mus...
components/mus/gpu/mus_gpu_memory_buffer_manager.cc:64:
GpuServiceMus::GetInstance()->gpu_memory_buffer_factory();
nit: because MusGpuMemoryBufferManager is created by GpuServiceMus, it would be
nice to pass GpuServiceMus explicitly in the constructor, to make the dependency
clear (and avoid global lookups, which can help testing and/or later
refactoring).

https://codereview.chromium.org/2049083002/diff/140001/components/mus/surface...
File components/mus/surfaces/surfaces_context_provider.cc (right):

https://codereview.chromium.org/2049083002/diff/140001/components/mus/surface...
components/mus/surfaces/surfaces_context_provider.cc:191: if (delegate_)
nit: use {} per style.

[Peng, 2016-06-09 20:32:46]

https://codereview.chromium.org/2049083002/diff/140001/components/mus/gpu/mus...
File components/mus/gpu/mus_gpu_memory_buffer_manager.cc (right):

https://codereview.chromium.org/2049083002/diff/140001/components/mus/gpu/mus...
components/mus/gpu/mus_gpu_memory_buffer_manager.cc:64:
GpuServiceMus::GetInstance()->gpu_memory_buffer_factory();
On 2016/06/09 19:31:32, piman OOO back 2016-6-9 wrote:
> nit: because MusGpuMemoryBufferManager is created by GpuServiceMus, it would
be
> nice to pass GpuServiceMus explicitly in the constructor, to make the
dependency
> clear (and avoid global lookups, which can help testing and/or later
> refactoring). 

Done.

https://codereview.chromium.org/2049083002/diff/140001/components/mus/surface...
File components/mus/surfaces/surfaces_context_provider.cc (right):

https://codereview.chromium.org/2049083002/diff/140001/components/mus/surface...
components/mus/surfaces/surfaces_context_provider.cc:191: if (delegate_)
On 2016/06/09 19:31:32, piman OOO back 2016-6-9 wrote:
> nit: use {} per style.

Done.

[piman, 2016-06-09 20:47:55]

https://codereview.chromium.org/2049083002/diff/180001/components/mus/gpu/mus...
File components/mus/gpu/mus_gpu_memory_buffer_manager.cc (right):

https://codereview.chromium.org/2049083002/diff/180001/components/mus/gpu/mus...
components/mus/gpu/mus_gpu_memory_buffer_manager.cc:120:
GpuServiceMus::GetInstance()->gpu_channel_manager();
nit: here too :)

[Peng, 2016-06-09 21:01:37]

https://codereview.chromium.org/2049083002/diff/180001/components/mus/gpu/mus...
File components/mus/gpu/mus_gpu_memory_buffer_manager.cc (right):

https://codereview.chromium.org/2049083002/diff/180001/components/mus/gpu/mus...
components/mus/gpu/mus_gpu_memory_buffer_manager.cc:120:
GpuServiceMus::GetInstance()->gpu_channel_manager();
On 2016/06/09 20:47:55, piman OOO back 2016-6-9 wrote:
> nit: here too :)

Done.

[piman, 2016-06-09 21:33:24]

lgtm

[Fady Samuel, 2016-06-10 15:23:01]

fsamuel@chromium.org changed reviewers:
+ fsamuel@chromium.org

[Fady Samuel, 2016-06-10 15:23:03]

https://codereview.chromium.org/2049083002/diff/200001/components/mus/common/...
File components/mus/common/gpu_service.cc (right):

https://codereview.chromium.org/2049083002/diff/200001/components/mus/common/...
components/mus/common/gpu_service.cc:74:
gpu_service->EstablishGpuChannel(base::Bind(
Can't we just make this sync?

[jam, 2016-06-10 15:31:16]

https://codereview.chromium.org/2049083002/diff/200001/components/mus/common/...
File components/mus/common/gpu_service.cc (right):

https://codereview.chromium.org/2049083002/diff/200001/components/mus/common/...
components/mus/common/gpu_service.cc:74:
gpu_service->EstablishGpuChannel(base::Bind(
On 2016/06/10 15:23:02, Fady Samuel wrote:
> Can't we just make this sync?

+1

[Peng, 2016-06-10 17:39:30]

https://codereview.chromium.org/2049083002/diff/200001/components/mus/common/...
File components/mus/common/gpu_service.cc (right):

https://codereview.chromium.org/2049083002/diff/200001/components/mus/common/...
components/mus/common/gpu_service.cc:74:
gpu_service->EstablishGpuChannel(base::Bind(
Done

[Peng, 2016-06-10 17:41:18]

penghuang@chromium.org changed reviewers:
+ tsepez@chromium.org

[Peng, 2016-06-10 17:41:19]

+tsepez for gpu_service.mojom

Hi Tom. PTAL. Thanks.

[Tom Sepez, 2016-06-10 18:09:10]

mojom lgtm

[Peng, 2016-06-10 18:20:21]

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

[commit-bot: I haz the power, 2016-06-10 18:21:16]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2049083002/240001

[commit-bot: I haz the power, 2016-06-10 19:53:32]

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

[commit-bot: I haz the power, 2016-06-10 19:53:33]

Dry run: This issue passed the CQ dry run.

[Peng, 2016-06-10 20:59:25]

On 2016/06/10 19:53:33, commit-bot: I haz the power wrote:
> Dry run: This issue passed the CQ dry run.

jam, friendly ping.

[jam, 2016-06-10 23:41:45]

On 2016/06/10 20:59:25, Peng wrote:
> On 2016/06/10 19:53:33, commit-bot: I haz the power wrote:
> > Dry run: This issue passed the CQ dry run.
> 
> jam, friendly ping.

thanks. you don't need me to review anymore right? you just need a
components/mus owner

[chromium-reviews, 2016-06-10 23:59:44]

I think piman and Rob are enough, but  I still need lgtm form an owner of
mus/common.

<jam@chromium.org>于2016年6月10日周五 19:41写道：

> On 2016/06/10 20:59:25, Peng wrote:
> > On 2016/06/10 19:53:33, commit-bot: I haz the power wrote:
> > > Dry run: This issue passed the CQ dry run.
> >
> > jam, friendly ping.
>
> thanks. you don't need me to review anymore right? you just need a
> components/mus owner
>
> https://codereview.chromium.org/2049083002/
>

-- 
You received this message because you are subscribed to the Google Groups
"Chromium-reviews" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.

[Peng, 2016-06-11 00:29:59]

penghuang@chromium.org changed reviewers:
+ ben@chromium.org

[Peng, 2016-06-11 00:30:00]

+Ben for mus/common.

Hi Ben, PTAL. Thanks.

[Ben Goodger (Google), 2016-06-12 02:07:13]

lgtm

[Peng, 2016-06-12 02:08:53]

The CQ bit was checked by penghuang@chromium.org

[Peng, 2016-06-12 02:08:54]

The patchset sent to the CQ was uploaded after l-g-t-m from
rjkroege@chromium.org, piman@chromium.org
Link to the patchset: https://codereview.chromium.org/2049083002/#ps240001
(title: "Update the mojom document")

[commit-bot: I haz the power, 2016-06-12 02:08:58]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2049083002/240001

[commit-bot: I haz the power, 2016-06-12 03:21:36]

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

[commit-bot: I haz the power, 2016-06-12 03:21:37]

Try jobs failed on following builders:
  linux_chromium_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[Peng, 2016-06-12 11:04:54]

The CQ bit was checked by penghuang@chromium.org

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

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2049083002/240001

[commit-bot: I haz the power, 2016-06-12 12:09:39]

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

[commit-bot: I haz the power, 2016-06-12 12:09:41]

Try jobs failed on following builders:
  linux_chromium_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[Peng, 2016-06-12 12:14:04]

The CQ bit was checked by penghuang@chromium.org

[commit-bot: I haz the power, 2016-06-12 12:14:15]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2049083002/240001

[commit-bot: I haz the power, 2016-06-12 13:15:33]

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

[commit-bot: I haz the power, 2016-06-12 13:15:35]

Try jobs failed on following builders:
  linux_chromium_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[Peng, 2016-06-12 13:28:54]

The CQ bit was checked by penghuang@chromium.org

[commit-bot: I haz the power, 2016-06-12 13:29:02]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2049083002/240001

[commit-bot: I haz the power, 2016-06-12 14:30:51]

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

[commit-bot: I haz the power, 2016-06-12 14:30:53]

Try jobs failed on following builders:
  linux_chromium_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[Peng, 2016-06-12 14:44:22]

The CQ bit was checked by penghuang@chromium.org

[commit-bot: I haz the power, 2016-06-12 14:44:28]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2049083002/240001

[commit-bot: I haz the power, 2016-06-12 15:17:00]

Description was changed from

==========
Implement MusGpuMemoryBufferManager.

Implement the MusGpuMemoryBufferManager which is used by MUS
locally. It supports allocating Native GpuMemoryBuffer(Ozone, etc).
It is needed for ChromeBook/ozone native rendering.

BUG=618316
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel;tryserver.chromium.win:win_optional_gpu_tests_rel
==========

to

==========
Implement MusGpuMemoryBufferManager.

Implement the MusGpuMemoryBufferManager which is used by MUS
locally. It supports allocating Native GpuMemoryBuffer(Ozone, etc).
It is needed for ChromeBook/ozone native rendering.

BUG=618316
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel;tryserver.chromium.win:win_optional_gpu_tests_rel
==========

[commit-bot: I haz the power, 2016-06-12 15:17:02]

Committed patchset #11 (id:240001)

[commit-bot: I haz the power, 2016-06-12 15:19:52]

Description was changed from

==========
Implement MusGpuMemoryBufferManager.

Implement the MusGpuMemoryBufferManager which is used by MUS
locally. It supports allocating Native GpuMemoryBuffer(Ozone, etc).
It is needed for ChromeBook/ozone native rendering.

BUG=618316
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel;tryserver.chromium.win:win_optional_gpu_tests_rel
==========

to

==========
Implement MusGpuMemoryBufferManager.

Implement the MusGpuMemoryBufferManager which is used by MUS
locally. It supports allocating Native GpuMemoryBuffer(Ozone, etc).
It is needed for ChromeBook/ozone native rendering.

BUG=618316
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel;tryserver.chromium.win:win_optional_gpu_tests_rel

Committed: https://crrev.com/cab01d28306bdec109b957d17d64d68677d9d3ed
Cr-Commit-Position: refs/heads/master@{#399390}
==========

[commit-bot: I haz the power, 2016-06-12 15:19:54]

Patchset 11 (id:??) landed as
https://crrev.com/cab01d28306bdec109b957d17d64d68677d9d3ed
Cr-Commit-Position: refs/heads/master@{#399390}