Use AnalyzedStyleUpdate for non-blocking sheets.

Use AnalyzedStyleUpdate for non-blocking sheets.

For no apparent reason we were always doing a full document recalc when
finishing loading a non-blocking stylesheet.

From webkit: https://bugs.webkit.org/show_bug.cgi?id=119236

BUG=284142
Committed: https://crrev.com/5b71fe7f8ba2c184108d9fe41d80c2b9668f89c4
Cr-Commit-Position: refs/heads/master@{#414682}

[rune, 2016-08-25 00:21:03]

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

[commit-bot: I haz the power, 2016-08-25 00:22: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-08-25 01:55:07]

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

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

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

[sashab, 2016-08-25 02:53:45]

sashab@chromium.org changed reviewers:
+ sashab@chromium.org

[sashab, 2016-08-25 02:53:45]

Haha, nice find. It looks like fast/css/link-alternate-style-recalc is actaully
testing exactly what you're fixing, so you'll want to update that test. :)

[rune, 2016-08-25 11:14:08]

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

[commit-bot: I haz the power, 2016-08-25 11:14:23]

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

[rune, 2016-08-25 11:15:06]

On 2016/08/25 02:53:45, sashab wrote:
> Haha, nice find. It looks like fast/css/link-alternate-style-recalc is
actaully
> testing exactly what you're fixing, so you'll want to update that test. :)

Since modifying the old test would essentially test the same as the new test for
alternate does, I just removed the old one.

[commit-bot: I haz the power, 2016-08-25 11:20:39]

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

[commit-bot: I haz the power, 2016-08-25 11:20:39]

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...)
  linux_chromium_asan_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_chromeos_compile_dbg_ng on master.tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_chromeos_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[rune, 2016-08-25 11:55:45]

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

[commit-bot: I haz the power, 2016-08-25 11:56:10]

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-08-25 13:23:35]

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

[commit-bot: I haz the power, 2016-08-25 13:23:36]

Dry run: This issue passed the CQ dry run.

[sashab, 2016-08-25 23:25:13]

lgtm

https://codereview.chromium.org/2278803002/diff/40001/third_party/WebKit/Layo...
File third_party/WebKit/LayoutTests/fast/css/add-non-blocking-sheet-recalc.html
(right):

https://codereview.chromium.org/2278803002/diff/40001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/fast/css/add-non-blocking-sheet-recalc.html:10: 
Remove blank line

https://codereview.chromium.org/2278803002/diff/40001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/fast/css/add-non-blocking-sheet-recalc.html:10: 
Remove blank line

[rune, 2016-08-26 07:15:34]

https://codereview.chromium.org/2278803002/diff/40001/third_party/WebKit/Layo...
File third_party/WebKit/LayoutTests/fast/css/add-non-blocking-sheet-recalc.html
(right):

https://codereview.chromium.org/2278803002/diff/40001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/fast/css/add-non-blocking-sheet-recalc.html:10: 
On 2016/08/25 23:25:12, sashab wrote:
> Remove blank line

Done.

[rune, 2016-08-26 07:15:49]

The CQ bit was checked by rune@opera.com

[rune, 2016-08-26 07:15:50]

The patchset sent to the CQ was uploaded after l-g-t-m from sashab@chromium.org
Link to the patchset: https://codereview.chromium.org/2278803002/#ps60001
(title: "Removed blank line.")

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

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-08-26 10:49:57]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2016-08-26 10:52:06]

Description was changed from

==========
Use AnalyzedStyleUpdate for non-blocking sheets.

For no apparent reason we were always doing a full document recalc when
finishing loading a non-blocking stylesheet.

From webkit: https://bugs.webkit.org/show_bug.cgi?id=119236

BUG=284142
==========

to

==========
Use AnalyzedStyleUpdate for non-blocking sheets.

For no apparent reason we were always doing a full document recalc when
finishing loading a non-blocking stylesheet.

From webkit: https://bugs.webkit.org/show_bug.cgi?id=119236

BUG=284142

Committed: https://crrev.com/5b71fe7f8ba2c184108d9fe41d80c2b9668f89c4
Cr-Commit-Position: refs/heads/master@{#414682}
==========

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

Patchset 4 (id:??) landed as
https://crrev.com/5b71fe7f8ba2c184108d9fe41d80c2b9668f89c4
Cr-Commit-Position: refs/heads/master@{#414682}