services/ui: Use the correct offset and stride in MojoGpuMemoryBuffer.

services/ui: Use the correct offset and stride in MojoGpuMemoryBuffer.

BUG=None

Committed: https://crrev.com/9286a11b8ef97363be868c378a6775c20d1e96f3
Cr-Commit-Position: refs/heads/master@{#414494}

[Peng, 2016-08-25 14:53:58]

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

[Peng, 2016-08-25 14:53:59]

Hi Scott, PTAL. Thanks.

[Peng, 2016-08-25 14:54:10]

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

[commit-bot: I haz the power, 2016-08-25 14:54:33]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-25 15:53:43]

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

[commit-bot: I haz the power, 2016-08-25 15:53:44]

Dry run: Try jobs failed on following builders:
  win_chromium_x64_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)

[Peng, 2016-08-25 16:10:48]

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

[commit-bot: I haz the power, 2016-08-25 16:11:17]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[sky, 2016-08-25 17:00:22]

LGTM

https://codereview.chromium.org/2277293002/diff/20001/services/ui/public/cpp/...
File services/ui/public/cpp/mojo_gpu_memory_buffer.h (right):

https://codereview.chromium.org/2277293002/diff/20001/services/ui/public/cpp/...
services/ui/public/cpp/mojo_gpu_memory_buffer.h:46:
MojoGpuMemoryBufferImpl(const gfx::Size& size,
Move definition to match new position.

https://codereview.chromium.org/2277293002/diff/20001/services/ui/public/cpp/...
services/ui/public/cpp/mojo_gpu_memory_buffer.h:53: size_t offset_;
Make const if you can.

[commit-bot: I haz the power, 2016-08-25 17:25:36]

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

[commit-bot: I haz the power, 2016-08-25 17:25:37]

Dry run: Try jobs failed on following builders:
  win_chromium_x64_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)

[Peng, 2016-08-25 17:41:25]

https://codereview.chromium.org/2277293002/diff/20001/services/ui/public/cpp/...
File services/ui/public/cpp/mojo_gpu_memory_buffer.h (right):

https://codereview.chromium.org/2277293002/diff/20001/services/ui/public/cpp/...
services/ui/public/cpp/mojo_gpu_memory_buffer.h:46:
MojoGpuMemoryBufferImpl(const gfx::Size& size,
On 2016/08/25 17:00:22, sky wrote:
> Move definition to match new position.

Done.

https://codereview.chromium.org/2277293002/diff/20001/services/ui/public/cpp/...
services/ui/public/cpp/mojo_gpu_memory_buffer.h:53: size_t offset_;
On 2016/08/25 17:00:22, sky wrote:
> Make const if you can.

Done.

[Peng, 2016-08-25 17:41:28]

The CQ bit was checked by penghuang@chromium.org

[Peng, 2016-08-25 17:41:29]

The patchset sent to the CQ was uploaded after l-g-t-m from sky@chromium.org
Link to the patchset: https://codereview.chromium.org/2277293002/#ps40001
(title: "Update")

[commit-bot: I haz the power, 2016-08-25 17:42:00]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-25 18:51:14]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-08-25 18:54:37]

Description was changed from

==========
services/ui: Use the correct offset and stride in MojoGpuMemoryBuffer.

BUG=None
==========

to

==========
services/ui: Use the correct offset and stride in MojoGpuMemoryBuffer.

BUG=None

Committed: https://crrev.com/9286a11b8ef97363be868c378a6775c20d1e96f3
Cr-Commit-Position: refs/heads/master@{#414494}
==========

[commit-bot: I haz the power, 2016-08-25 18:54:38]

Patchset 3 (id:??) landed as
https://crrev.com/9286a11b8ef97363be868c378a6775c20d1e96f3
Cr-Commit-Position: refs/heads/master@{#414494}