Moved applyRuleSetChanges functions to StyleEngine.

Moved applyRuleSetChanges functions to StyleEngine.

A lot of the side effects were calls to StyleEngine, so moved the
method there instead. Also fixed the TODO for adding the call to make
CSSGlobalRuleSet dirty.

BUG=567021
Committed: https://crrev.com/c2e0d164125e177ce62925ac66f9fcdb4f457e1c
Cr-Commit-Position: refs/heads/master@{#431227}

[rune, 2016-11-08 12:21:38]

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

[commit-bot: I haz the power, 2016-11-08 12:21:55]

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

[rune, 2016-11-08 14:02:20]

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

[rune, 2016-11-08 14:02:21]

ptal

[commit-bot: I haz the power, 2016-11-08 15:19:14]

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

[commit-bot: I haz the power, 2016-11-08 15:19:15]

Dry run: This issue passed the CQ dry run.

[sashab, 2016-11-09 23:45:29]

lgtm

https://codereview.chromium.org/2487653002/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/dom/StyleEngine.cpp (right):

https://codereview.chromium.org/2487653002/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/dom/StyleEngine.cpp:1067:
m_globalRuleSet.markDirty();
Maybe add a comment explaining what this line is for.

[rune, 2016-11-10 08:10:45]

https://codereview.chromium.org/2487653002/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/dom/StyleEngine.cpp (right):

https://codereview.chromium.org/2487653002/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/dom/StyleEngine.cpp:1067:
m_globalRuleSet.markDirty();
On 2016/11/09 23:45:29, sashab wrote:
> Maybe add a comment explaining what this line is for.

Done.

[rune, 2016-11-10 08:10:54]

The CQ bit was checked by rune@opera.com

[rune, 2016-11-10 08:10:55]

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/2487653002/#ps20001
(title: "Added comment")

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

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-11-10 09:49:43]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-11-10 09:54:05]

Description was changed from

==========
Moved applyRuleSetChanges functions to StyleEngine.

A lot of the side effects were calls to StyleEngine, so moved the
method there instead. Also fixed the TODO for adding the call to make
CSSGlobalRuleSet dirty.

BUG=567021
==========

to

==========
Moved applyRuleSetChanges functions to StyleEngine.

A lot of the side effects were calls to StyleEngine, so moved the
method there instead. Also fixed the TODO for adding the call to make
CSSGlobalRuleSet dirty.

BUG=567021

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

[commit-bot: I haz the power, 2016-11-10 09:54:06]

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