No need for SubtreeStyleChange for results attribute change.

No need for SubtreeStyleChange for results attribute change.

The results attribute change causes appearance changes when changing
between negative and non-negative values. That is currently handled by a
lazyReattachIfAttached. The following SubtreeStyleChange should not have
an effect in that case.

The reattach condition was changed to not include the change between
different negative values.

R=tkent@chromium.org
BUG=557440
Committed: https://crrev.com/a1f820202cfb28e67a53fa5aa9df318ee85d72b3
Cr-Commit-Position: refs/heads/master@{#362077}

[rune, 2015-11-27 15:17:58]

The CQ bit was checked by rune@opera.com to run a CQ dry run

[commit-bot: I haz the power, 2015-11-27 15:18:06]

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

[commit-bot: I haz the power, 2015-11-27 15:28:38]

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

[commit-bot: I haz the power, 2015-11-27 15:28:39]

Dry run: Try jobs failed on following builders:
  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_...)

[rune, 2015-11-27 15:33:30]

The CQ bit was checked by rune@opera.com to run a CQ dry run

[commit-bot: I haz the power, 2015-11-27 15:34:18]

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

[commit-bot: I haz the power, 2015-11-27 18:00:57]

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

[commit-bot: I haz the power, 2015-11-27 18:00:58]

Dry run: This issue passed the CQ dry run.

[tkent, 2015-11-29 23:44:10]

lgtm.

Though I'd like to remove the non-standard |results| attribute, reducing
SubtreeStyleChange is great.

[rune, 2015-11-30 00:14:34]

The CQ bit was checked by rune@opera.com

[commit-bot: I haz the power, 2015-11-30 00:14:55]

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

[commit-bot: I haz the power, 2015-11-30 01:24:03]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-11-30 01:24:56]

Description was changed from

==========
No need for SubtreeStyleChange for results attribute change.

The results attribute change causes appearance changes when changing
between negative and non-negative values. That is currently handled by a
lazyReattachIfAttached. The following SubtreeStyleChange should not have
an effect in that case.

The reattach condition was changed to not include the change between
different negative values.

R=tkent@chromium.org
BUG=557440
==========

to

==========
No need for SubtreeStyleChange for results attribute change.

The results attribute change causes appearance changes when changing
between negative and non-negative values. That is currently handled by a
lazyReattachIfAttached. The following SubtreeStyleChange should not have
an effect in that case.

The reattach condition was changed to not include the change between
different negative values.

R=tkent@chromium.org
BUG=557440

Committed: https://crrev.com/a1f820202cfb28e67a53fa5aa9df318ee85d72b3
Cr-Commit-Position: refs/heads/master@{#362077}
==========

[commit-bot: I haz the power, 2015-11-30 01:24:56]

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