Add support for ManagedMemoryPolicy (de)serialization

Add support for ManagedMemoryPolicy (de)serialization

As part of serializing cc::LayerTreeHost, we also need to
serialize the cc::LayerTreeSettings and the
cc::ManagedMemoryPolicy.

This CL focuses on the cc::ManagedMemoryPolicy and the
gpu::MemoryAllocation::PriorityCutoff enum.

BUG=561210
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

Committed: https://crrev.com/1d29b060b31f3e4c2d86c818188b8c608e1ac34d
Cr-Commit-Position: refs/heads/master@{#362603}

[nyquist, 2015-11-30 22:40:28]

Description was changed from

==========
Add support for ManagedMemoryPolicy (de)serialization

As part of serializing cc::LayerTreeHost, we also need to
serialize the cc::LayerTreeSettings and the
cc::ManagedMemoryPolicy.

This CL focuses on the cc::ManagedMemoryPolicy and the
gpu::MemoryAllocation::PriorityCutoff enum.

BUG=561210
==========

to

==========
Add support for ManagedMemoryPolicy (de)serialization

As part of serializing cc::LayerTreeHost, we also need to
serialize the cc::LayerTreeSettings and the
cc::ManagedMemoryPolicy.

This CL focuses on the cc::ManagedMemoryPolicy and the
gpu::MemoryAllocation::PriorityCutoff enum.

BUG=561210
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

[nyquist, 2015-11-30 22:49:11]

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

[nyquist, 2015-11-30 22:49:46]

nyquist@chromium.org changed reviewers:
+ vmpstr@chromium.org

[nyquist, 2015-11-30 22:49:46]

vmpstr: PTAL

[commit-bot: I haz the power, 2015-11-30 22:53:11]

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

[nyquist, 2015-11-30 22:58:03]

nyquist@chromium.org changed reviewers:
+ dtrainor@chromium.org

[nyquist, 2015-11-30 22:58:04]

dtrainor: PTAL

[commit-bot: I haz the power, 2015-11-30 23:23:25]

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

[commit-bot: I haz the power, 2015-11-30 23:23:26]

Dry run: Try jobs failed on following builders:
  android_chromium_gn_compile_rel on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/android_chromiu...)
  chromeos_amd64-generic_chromium_compile_only_ng on tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_amd64-...)
  chromeos_daisy_chromium_compile_only_ng on tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_daisy_...)
  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...)
  win_chromium_x64_rel_ng on tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)

[nyquist, 2015-12-01 00:42:06]

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

[commit-bot: I haz the power, 2015-12-01 00:46:09]

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

[commit-bot: I haz the power, 2015-12-01 02:41:16]

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

[commit-bot: I haz the power, 2015-12-01 02:41:17]

Dry run: This issue passed the CQ dry run.

[David Trainor- moved to gerrit, 2015-12-01 07:18:31]

lgtm

[vmpstr, 2015-12-01 18:59:53]

https://codereview.chromium.org/1490513005/diff/40001/cc/cc_tests.gyp
File cc/cc_tests.gyp (right):

https://codereview.chromium.org/1490513005/diff/40001/cc/cc_tests.gyp#newcode77
cc/cc_tests.gyp:77: 'output/managed_memory_policy_unittest',
.cc

https://codereview.chromium.org/1490513005/diff/40001/cc/proto/gpu_conversion...
File cc/proto/gpu_conversions_unittest.cc (right):

https://codereview.chromium.org/1490513005/diff/40001/cc/proto/gpu_conversion...
cc/proto/gpu_conversions_unittest.cc:17: i++) {
++i

https://codereview.chromium.org/1490513005/diff/40001/cc/proto/memory_allocat...
File cc/proto/memory_allocation.proto (right):

https://codereview.chromium.org/1490513005/diff/40001/cc/proto/memory_allocat...
cc/proto/memory_allocation.proto:13: UNKNOWN = 0;
SkRegion has this as ops as Op_Blah, this is BLAH_BLAH. I think we should be
consistent.

I prefer BLAH_BLAH, so could you update SkRegion in a follow-up?

[nyquist, 2015-12-01 22:36:47]

vmpstr: PTAL

https://codereview.chromium.org/1490513005/diff/40001/cc/cc_tests.gyp
File cc/cc_tests.gyp (right):

https://codereview.chromium.org/1490513005/diff/40001/cc/cc_tests.gyp#newcode77
cc/cc_tests.gyp:77: 'output/managed_memory_policy_unittest',
On 2015/12/01 18:59:52, vmpstr wrote:
> .cc

Done. Oh my.

https://codereview.chromium.org/1490513005/diff/40001/cc/proto/gpu_conversion...
File cc/proto/gpu_conversions_unittest.cc (right):

https://codereview.chromium.org/1490513005/diff/40001/cc/proto/gpu_conversion...
cc/proto/gpu_conversions_unittest.cc:17: i++) {
On 2015/12/01 18:59:53, vmpstr wrote:
> ++i

Done.

https://codereview.chromium.org/1490513005/diff/40001/cc/proto/memory_allocat...
File cc/proto/memory_allocation.proto (right):

https://codereview.chromium.org/1490513005/diff/40001/cc/proto/memory_allocat...
cc/proto/memory_allocation.proto:13: UNKNOWN = 0;
On 2015/12/01 18:59:53, vmpstr wrote:
> SkRegion has this as ops as Op_Blah, this is BLAH_BLAH. I think we should be
> consistent.
> 
> I prefer BLAH_BLAH, so could you update SkRegion in a follow-up?

OK. This follow-up is done in:  https://codereview.chromium.org/1492533003/

[nyquist, 2015-12-02 00:23:51]

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

[commit-bot: I haz the power, 2015-12-02 00:24:50]

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

[vmpstr, 2015-12-02 00:25:35]

lgtm

https://codereview.chromium.org/1490513005/diff/60001/cc/proto/gpu_conversion...
File cc/proto/gpu_conversions.cc (right):

https://codereview.chromium.org/1490513005/diff/60001/cc/proto/gpu_conversion...
cc/proto/gpu_conversions.cc:40: DLOG(FATAL) << "Received
MemoryAllocation::PriorityCutoff_UNKNOWN";
You just put NOTREACHED() here

[nyquist, 2015-12-02 00:45:09]

The CQ bit was checked by nyquist@chromium.org

[nyquist, 2015-12-02 00:45:10]

The patchset sent to the CQ was uploaded after l-g-t-m from
dtrainor@chromium.org, vmpstr@chromium.org
Link to the patchset: https://codereview.chromium.org/1490513005/#ps100001
(title: "Rebase on top of origin/master")

[commit-bot: I haz the power, 2015-12-02 00:47:26]

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

[commit-bot: I haz the power, 2015-12-02 00:55:32]

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

[commit-bot: I haz the power, 2015-12-02 00:55:32]

Try jobs failed on following builders:
  mac_chromium_rel_ng on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[nyquist, 2015-12-02 02:03:41]

The CQ bit was checked by nyquist@chromium.org

[commit-bot: I haz the power, 2015-12-02 02:07:59]

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

[commit-bot: I haz the power, 2015-12-02 02:48:27]

Committed patchset #6 (id:100001)

[commit-bot: I haz the power, 2015-12-02 02:49:05]

Description was changed from

==========
Add support for ManagedMemoryPolicy (de)serialization

As part of serializing cc::LayerTreeHost, we also need to
serialize the cc::LayerTreeSettings and the
cc::ManagedMemoryPolicy.

This CL focuses on the cc::ManagedMemoryPolicy and the
gpu::MemoryAllocation::PriorityCutoff enum.

BUG=561210
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

to

==========
Add support for ManagedMemoryPolicy (de)serialization

As part of serializing cc::LayerTreeHost, we also need to
serialize the cc::LayerTreeSettings and the
cc::ManagedMemoryPolicy.

This CL focuses on the cc::ManagedMemoryPolicy and the
gpu::MemoryAllocation::PriorityCutoff enum.

BUG=561210
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

Committed: https://crrev.com/1d29b060b31f3e4c2d86c818188b8c608e1ac34d
Cr-Commit-Position: refs/heads/master@{#362603}
==========

[commit-bot: I haz the power, 2015-12-02 02:49:06]

Patchset 6 (id:??) landed as
https://crrev.com/1d29b060b31f3e4c2d86c818188b8c608e1ac34d
Cr-Commit-Position: refs/heads/master@{#362603}