Only SchedulePaint on ShelfWidget if background changes.

Only SchedulePaint on ShelfWidget if background changes.

This is called whenever a toplevel window is moved, and was always doing
a SchedulePaint even if no property changed. The BackgroundAnimator will
do an UpdateBackground if the alpha changes, so the higher-level
SchedulePaint is redundant.

BUG=614193
Committed: https://crrev.com/9a944f96a6dbb2d6b702b2b24329e0bf2e149705
Cr-Commit-Position: refs/heads/master@{#397645}

[jbauman, 2016-05-25 22:51:22]

jbauman@chromium.org changed reviewers:
+ oshima@chromium.org

[oshima, 2016-05-26 19:45:23]

oshima@chromium.org changed reviewers:
+ varkha@chromium.org

[oshima, 2016-05-26 19:45:24]

Is this urgent? If not, I'd like varkha@, who added this, to look at this too.

[jbauman, 2016-05-26 22:46:35]

On 2016/05/26 19:45:24, oshima wrote:
> Is this urgent? If not, I'd like varkha@, who added this, to look at this too.

Sure, I think this can wait for him to get back.

[varkha, 2016-06-02 20:36:53]

This lgtm. I have tried it with various shelf animations and they seem to be
preserved. I am not an owner though so you will still need oshima@ to look at
it.

[oshima, 2016-06-02 21:26:01]

lgtm

[jbauman, 2016-06-02 21:26:15]

The CQ bit was checked by jbauman@chromium.org

[commit-bot: I haz the power, 2016-06-02 21:27:22]

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

[commit-bot: I haz the power, 2016-06-03 00:42:32]

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

[commit-bot: I haz the power, 2016-06-03 00:42: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_...)

[reveman, 2016-06-03 05:00:35]

The CQ bit was checked by reveman@chromium.org

[commit-bot: I haz the power, 2016-06-03 05:00:46]

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

[commit-bot: I haz the power, 2016-06-03 05:57:47]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-06-03 05:58:56]

Description was changed from

==========
Only SchedulePaint on ShelfWidget if background changes.

This is called whenever a toplevel window is moved, and was always doing
a SchedulePaint even if no property changed. The BackgroundAnimator will
do an UpdateBackground if the alpha changes, so the higher-level
SchedulePaint is redundant.

BUG=614193
==========

to

==========
Only SchedulePaint on ShelfWidget if background changes.

This is called whenever a toplevel window is moved, and was always doing
a SchedulePaint even if no property changed. The BackgroundAnimator will
do an UpdateBackground if the alpha changes, so the higher-level
SchedulePaint is redundant.

BUG=614193

Committed: https://crrev.com/9a944f96a6dbb2d6b702b2b24329e0bf2e149705
Cr-Commit-Position: refs/heads/master@{#397645}
==========

[commit-bot: I haz the power, 2016-06-03 05:58:57]

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