cc: Make OutputSurface::Reshape abstract

cc: Make OutputSurface::Reshape abstract

DirectRenderer will cache the values given to Reshape and
only call it when things change.

TBR=dtrainor
R=enne@chromium.org
BUG=606056
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel

Committed: https://crrev.com/70ffc43ceda0ffba0edf7ea136567af5bed5c819
Cr-Commit-Position: refs/heads/master@{#424255}

[danakj, 2016-10-07 01:37:18]

Description was changed from

==========
cc: Make OutputSurface::Reshape abstract

R=enne@chromium.org
BUG=606056
==========

to

==========
cc: Make OutputSurface::Reshape abstract

R=enne@chromium.org
BUG=606056
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
==========

[danakj, 2016-10-07 01:37:20]

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

[commit-bot: I haz the power, 2016-10-07 01:37:33]

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

[danakj, 2016-10-07 01:37:34]

Description was changed from

==========
cc: Make OutputSurface::Reshape abstract

R=enne@chromium.org
BUG=606056
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
==========

to

==========
cc: Make OutputSurface::Reshape abstract

DirectRenderer will cache the values given to Reshape and
only call it when things change.

R=enne@chromium.org
BUG=606056
==========

[danakj, 2016-10-07 01:37:49]

danakj@chromium.org changed reviewers:
+ boliu@chromium.org, fsamuel@google.com

[danakj, 2016-10-07 01:38:22]

enne: PTAL at everything
boliu: webview and content owners
fsamuel: services owners

[commit-bot: I haz the power, 2016-10-07 01:49:36]

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

[commit-bot: I haz the power, 2016-10-07 01:49:37]

Dry run: Try jobs failed on following builders:
  cast_shell_android on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/cast_shell_a...)
  chromeos_amd64-generic_chromium_compile_only_ng on
master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_amd64-...)
  linux_chromium_chromeos_ozone_rel_ng on master.tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[boliu, 2016-10-07 02:39:13]

lgtm

[danakj, 2016-10-07 18:21:06]

Description was changed from

==========
cc: Make OutputSurface::Reshape abstract

DirectRenderer will cache the values given to Reshape and
only call it when things change.

R=enne@chromium.org
BUG=606056
==========

to

==========
cc: Make OutputSurface::Reshape abstract

DirectRenderer will cache the values given to Reshape and
only call it when things change.

R=enne@chromium.org
BUG=606056
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
==========

[danakj, 2016-10-07 18:21:08]

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

[commit-bot: I haz the power, 2016-10-07 18:21:43]

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

[danakj, 2016-10-07 18:22:39]

Description was changed from

==========
cc: Make OutputSurface::Reshape abstract

DirectRenderer will cache the values given to Reshape and
only call it when things change.

R=enne@chromium.org
BUG=606056
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
==========

to

==========
cc: Make OutputSurface::Reshape abstract

DirectRenderer will cache the values given to Reshape and
only call it when things change.

R=enne@chromium.org
BUG=606056
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
==========

[danakj, 2016-10-07 18:22:39]

danakj@chromium.org changed reviewers:
+ fsamuel@chromium.org
- fsamuel@google.com

[commit-bot: I haz the power, 2016-10-07 18:38:35]

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

[commit-bot: I haz the power, 2016-10-07 18:38:36]

Dry run: Try jobs failed on following builders:
  chromeos_amd64-generic_chromium_compile_only_ng on
master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_amd64-...)

[danakj, 2016-10-07 18:43:19]

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

[commit-bot: I haz the power, 2016-10-07 18:43:37]

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-10-07 19:02:36]

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

[commit-bot: I haz the power, 2016-10-07 19:02:37]

Dry run: Try jobs failed on following builders:
  cast_shell_android on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/cast_shell_a...)
  chromeos_amd64-generic_chromium_compile_only_ng on
master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_amd64-...)

[danakj, 2016-10-07 21:30:43]

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

[commit-bot: I haz the power, 2016-10-07 21:31:09]

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-10-07 21:44:18]

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

[commit-bot: I haz the power, 2016-10-07 21:44:19]

Dry run: Try jobs failed on following builders:
  android_compile_dbg on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_comp...)

[danakj, 2016-10-07 21:46:09]

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

[commit-bot: I haz the power, 2016-10-07 21:46:41]

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

[danakj, 2016-10-07 21:54:31]

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

[commit-bot: I haz the power, 2016-10-07 21:55:17]

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-10-07 22:05:48]

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

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

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_linu...)

[danakj, 2016-10-07 22:52:11]

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

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

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-10-07 23:04:04]

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

[commit-bot: I haz the power, 2016-10-07 23:04:05]

Dry run: Try jobs failed on following builders:
  cast_shell_android on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/cast_shell_a...)
  chromeos_daisy_chromium_compile_only_ng on master.tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_daisy_...)
  chromeos_x86-generic_chromium_compile_only_ng on
master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_x86-ge...)
  linux_chromium_chromeos_ozone_rel_ng on master.tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[danakj, 2016-10-07 23:10:15]

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

[commit-bot: I haz the power, 2016-10-07 23:10:34]

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-10-07 23:23:50]

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

[commit-bot: I haz the power, 2016-10-07 23:23:51]

Dry run: Try jobs failed on following builders:
  chromeos_amd64-generic_chromium_compile_only_ng on
master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_amd64-...)

[danakj, 2016-10-07 23:29:34]

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

[commit-bot: I haz the power, 2016-10-07 23:30:05]

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

[enne (OOO), 2016-10-08 00:20:55]

lgtm

[Fady Samuel, 2016-10-08 00:21:31]

lgtm

[commit-bot: I haz the power, 2016-10-08 01:01:07]

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

[commit-bot: I haz the power, 2016-10-08 01:01:08]

Dry run: This issue passed the CQ dry run.

[danakj, 2016-10-08 01:02:18]

The CQ bit was checked by danakj@chromium.org

[danakj, 2016-10-08 01:02:19]

The patchset sent to the CQ was uploaded after l-g-t-m from boliu@chromium.org
Link to the patchset: https://codereview.chromium.org/2399983003/#ps180001
(title: "reshapeabstract: .")

[commit-bot: I haz the power, 2016-10-08 01:02:40]

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

[danakj, 2016-10-08 01:02:50]

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

[danakj, 2016-10-08 01:02:52]

Oh +nyquist@chromium.org for blimp/

[danakj, 2016-10-08 01:03:17]

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

[danakj, 2016-10-08 01:03:31]

On 2016/10/08 01:02:52, danakj wrote:
> Oh mailto:+nyquist@chromium.org for blimp/

Who is OOO. So +dtrainor.

[commit-bot: I haz the power, 2016-10-08 01:13:44]

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

[commit-bot: I haz the power, 2016-10-08 01:13:46]

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

[danakj, 2016-10-10 20:53:14]

Description was changed from

==========
cc: Make OutputSurface::Reshape abstract

DirectRenderer will cache the values given to Reshape and
only call it when things change.

R=enne@chromium.org
BUG=606056
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
==========

to

==========
cc: Make OutputSurface::Reshape abstract

DirectRenderer will cache the values given to Reshape and
only call it when things change.

TBR=dtrainor
R=enne@chromium.org
BUG=606056
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
==========

[danakj, 2016-10-10 20:53:21]

The CQ bit was checked by danakj@chromium.org

[commit-bot: I haz the power, 2016-10-10 20:53:43]

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

[David Trainor- moved to gerrit, 2016-10-10 20:57:57]

blimp/ lgtm sorry for the delay :(

[commit-bot: I haz the power, 2016-10-10 21:47:54]

Description was changed from

==========
cc: Make OutputSurface::Reshape abstract

DirectRenderer will cache the values given to Reshape and
only call it when things change.

TBR=dtrainor
R=enne@chromium.org
BUG=606056
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
==========

to

==========
cc: Make OutputSurface::Reshape abstract

DirectRenderer will cache the values given to Reshape and
only call it when things change.

TBR=dtrainor
R=enne@chromium.org
BUG=606056
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
==========

[commit-bot: I haz the power, 2016-10-10 21:47:56]

Committed patchset #10 (id:180001)

[commit-bot: I haz the power, 2016-10-10 21:49:20]

Description was changed from

==========
cc: Make OutputSurface::Reshape abstract

DirectRenderer will cache the values given to Reshape and
only call it when things change.

TBR=dtrainor
R=enne@chromium.org
BUG=606056
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
==========

to

==========
cc: Make OutputSurface::Reshape abstract

DirectRenderer will cache the values given to Reshape and
only call it when things change.

TBR=dtrainor
R=enne@chromium.org
BUG=606056
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel

Committed: https://crrev.com/70ffc43ceda0ffba0edf7ea136567af5bed5c819
Cr-Commit-Position: refs/heads/master@{#424255}
==========

[commit-bot: I haz the power, 2016-10-10 21:49:22]

Patchset 10 (id:??) landed as
https://crrev.com/70ffc43ceda0ffba0edf7ea136567af5bed5c819
Cr-Commit-Position: refs/heads/master@{#424255}