Setting preferred stylesheet simplified.

Setting preferred stylesheet simplified.

With active stylesheets being applied asynchronously, we no longer need
to avoid the synchronous stylesheet update during link processing. We
can just mark the treeScope dirty to trigger the preferred set to be
updated as part of updateActiveStyleSheets().

R=meade@chromium.org
BUG=567021
Committed: https://crrev.com/626379c7a2bed66978eb739ee3afcc3df47ac4b2
Cr-Commit-Position: refs/heads/master@{#439115}

[rune, 2016-12-13 08:58:39]

rune@opera.com changed reviewers:
+ esprehn@chromium.org, sashab@chromium.org

[rune, 2016-12-13 08:58:40]

ptal

[esprehn, 2016-12-13 09:09:34]

lgtm

[rune, 2016-12-13 09:13:20]

I think I'll hold off landing these cleanup CLs until [1] has been a round
through perfbots to make it simpler to revert.

[1] https://codereview.chromium.org/2557533005/

[rune, 2016-12-13 18:59:58]

On 2016/12/13 09:13:20, rune wrote:
> I think I'll hold off landing these cleanup CLs until [1] has been a round
> through perfbots to make it simpler to revert.
> 
> [1] https://codereview.chromium.org/2557533005/

It's been through the linux-release profile now. Browsing what I'd expect to be
affected negatively didn't show badness. I'll take the chance of landing this
and the following ones now.

[rune, 2016-12-13 19:00:08]

The CQ bit was checked by rune@opera.com

[commit-bot: I haz the power, 2016-12-13 19:00:47]

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-12-13 19:03:19]

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

[commit-bot: I haz the power, 2016-12-13 19:03:20]

Try jobs failed on following builders:
  ios-device on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device/builds...)
  ios-device-xcode-clang on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device-xcode-...)
  ios-simulator on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator/bui...)
  ios-simulator-xcode-clang on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator-xco...)

[rune, 2016-12-16 12:34:37]

The CQ bit was checked by rune@opera.com

[rune, 2016-12-16 12:34:38]

The patchset sent to the CQ was uploaded after l-g-t-m from esprehn@chromium.org
Link to the patchset: https://codereview.chromium.org/2552353003/#ps20001
(title: "Rebased.")

[commit-bot: I haz the power, 2016-12-16 12:34:45]

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-12-16 15:53:08]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1481891677612290,
"parent_rev": "944c0aa43a91e3b6cf52776de428239af05e624a", "commit_rev":
"b601865b0e9bdd976a482fad584e3ba52c603cd4"}

[commit-bot: I haz the power, 2016-12-16 15:53:36]

Description was changed from

==========
Setting preferred stylesheet simplified.

With active stylesheets being applied asynchronously, we no longer need
to avoid the synchronous stylesheet update during link processing. We
can just mark the treeScope dirty to trigger the preferred set to be
updated as part of updateActiveStyleSheets().

R=meade@chromium.org
BUG=567021
==========

to

==========
Setting preferred stylesheet simplified.

With active stylesheets being applied asynchronously, we no longer need
to avoid the synchronous stylesheet update during link processing. We
can just mark the treeScope dirty to trigger the preferred set to be
updated as part of updateActiveStyleSheets().

R=meade@chromium.org
BUG=567021

Review-Url: https://codereview.chromium.org/2552353003
==========

[commit-bot: I haz the power, 2016-12-16 15:53:37]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-12-16 15:55:53]

Description was changed from

==========
Setting preferred stylesheet simplified.

With active stylesheets being applied asynchronously, we no longer need
to avoid the synchronous stylesheet update during link processing. We
can just mark the treeScope dirty to trigger the preferred set to be
updated as part of updateActiveStyleSheets().

R=meade@chromium.org
BUG=567021

Review-Url: https://codereview.chromium.org/2552353003
==========

to

==========
Setting preferred stylesheet simplified.

With active stylesheets being applied asynchronously, we no longer need
to avoid the synchronous stylesheet update during link processing. We
can just mark the treeScope dirty to trigger the preferred set to be
updated as part of updateActiveStyleSheets().

R=meade@chromium.org
BUG=567021

Committed: https://crrev.com/626379c7a2bed66978eb739ee3afcc3df47ac4b2
Cr-Commit-Position: refs/heads/master@{#439115}
==========

[commit-bot: I haz the power, 2016-12-16 15:55:54]

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