cc: Move solid color scrollbar layer data in a seperate struct.

cc: Move solid color scrollbar layer data in a seperate struct.

Just a clean up as a part of Layers cleanup.

BUG=625284
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel

Committed: https://crrev.com/11c20a91030988c6938c831ebe2c128d275a57c6
Cr-Commit-Position: refs/heads/master@{#408813}

[Khushal, 2016-07-28 22:21:35]

Description was changed from

==========
cc: Move solid color scrollbar layer data in a seperate struct.

Just a clean up as a part of Layers cleanup.

BUG=625284
==========

to

==========
cc: Move solid color scrollbar layer data in a seperate struct.

Just a clean up as a part of Layers cleanup.

BUG=625284
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_blink_rel
==========

[Khushal, 2016-07-28 22:22:18]

khushalsagar@chromium.org changed reviewers:
+ ajuma@chromium.org

[Khushal, 2016-07-28 22:22:19]

Trivial change since we are moving data in all these classes into structs.

[ajuma, 2016-07-28 23:34:16]

lgtm

[Khushal, 2016-07-29 00:26:58]

The CQ bit was checked by khushalsagar@chromium.org

[commit-bot: I haz the power, 2016-07-29 00:27:19]

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-07-29 00:27:19]

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

[commit-bot: I haz the power, 2016-07-29 00:27:20]

Your CL was about to rely on recently removed CQ feature(s):
 * master.tryserver.blink builder linux_blink_rel was renamed to
linux_precise_blink_rel (http://crbug.com/590036)

[Khushal, 2016-07-29 00:37:26]

Description was changed from

==========
cc: Move solid color scrollbar layer data in a seperate struct.

Just a clean up as a part of Layers cleanup.

BUG=625284
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_blink_rel
==========

to

==========
cc: Move solid color scrollbar layer data in a seperate struct.

Just a clean up as a part of Layers cleanup.

BUG=625284
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
==========

[Khushal, 2016-07-29 00:37:30]

The CQ bit was checked by khushalsagar@chromium.org

[commit-bot: I haz the power, 2016-07-29 00:38: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-07-29 00:49:26]

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

[commit-bot: I haz the power, 2016-07-29 00:49:27]

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

[Khushal, 2016-07-29 01:26:07]

On 2016/07/29 00:49:27, commit-bot: I haz the power wrote:
> Try jobs failed on following builders:
>   linux_chromium_chromeos_rel_ng on master.tryserver.chromium.linux
(JOB_FAILED,
>
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

Looks like in places where we friend layers and subclasses, it becomes unclear
whether you want to access Layer Inputs or the subclass. So I've renamed them in
the subclasses. Does this look better?

[ajuma, 2016-07-29 22:52:25]

On 2016/07/29 01:26:07, Khushal wrote:
> On 2016/07/29 00:49:27, commit-bot: I haz the power wrote:
> > Try jobs failed on following builders:
> >   linux_chromium_chromeos_rel_ng on master.tryserver.chromium.linux
> (JOB_FAILED,
> >
>
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
> 
> Looks like in places where we friend layers and subclasses, it becomes unclear
> whether you want to access Layer Inputs or the subclass. So I've renamed them
in
> the subclasses. Does this look better?

Ah, makes sense. Still lgtm.

[Khushal, 2016-07-29 23:07:41]

The CQ bit was checked by khushalsagar@chromium.org

[commit-bot: I haz the power, 2016-07-29 23:08:09]

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-07-30 00:07:04]

Description was changed from

==========
cc: Move solid color scrollbar layer data in a seperate struct.

Just a clean up as a part of Layers cleanup.

BUG=625284
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
==========

to

==========
cc: Move solid color scrollbar layer data in a seperate struct.

Just a clean up as a part of Layers cleanup.

BUG=625284
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
==========

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

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-07-30 00:09:05]

Description was changed from

==========
cc: Move solid color scrollbar layer data in a seperate struct.

Just a clean up as a part of Layers cleanup.

BUG=625284
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
==========

to

==========
cc: Move solid color scrollbar layer data in a seperate struct.

Just a clean up as a part of Layers cleanup.

BUG=625284
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel

Committed: https://crrev.com/11c20a91030988c6938c831ebe2c128d275a57c6
Cr-Commit-Position: refs/heads/master@{#408813}
==========

[commit-bot: I haz the power, 2016-07-30 00:09:06]

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