One instead of three resolverChanged replacing source in inspector.

One instead of three resolverChanged replacing source in inspector.

resolverChanged() was called three times when replacing the stylesheet
text from the inspector. Two mutation scopes and an explicit call at
the end.

Kept one of the mutation scopes. Two shouldn't be necessary, and I have
confirmed that the crash tests for which this was justified earlier
don't crash when removing one of the scopes.

Moved the stylesheet modifications into CSSStyleSheet::setText().

Also moved clearing of the CSSOM wrappers before the mutation scope
declaration as the mutation scope constructor would unnecessarily
re-attach wrappers which would then be cleared right after.

R=esprehn@chromium.org,pfeldman@chromium.org
BUG=591599
Committed: https://crrev.com/5958ca8683dd9478a780d516205355a36c48c5aa
Cr-Commit-Position: refs/heads/master@{#378978}

[rune, 2016-03-03 02:26:36]

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

[commit-bot: I haz the power, 2016-03-03 02:26:50]

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

[rune, 2016-03-03 02:41:53]

Description was changed from

==========
One instead of three resolverChanged replacing source in inspector.

resolverChanged() was called three times when replacing the stylesheet
text from the inspector. Two mutation scopes and an explicit call at
the end.

Kept one of the mutation scopes. Two shouldn't be necessary, and I have
confirmed that the crash tests don't crash when removing one of the
scopes.

Moved the stylesheet specifics into CSSStyleSheet.

Also moved clearing of the CSSOM wrappers before the mutation scope
declaration as the mutation scope constructor would unnecessarily
re-attach wrappers which would then be cleared right after.

R=esprehn@chromium.org,pfeldman@chromium.org
BUG=591599
==========

to

==========
One instead of three resolverChanged replacing source in inspector.

resolverChanged() was called three times when replacing the stylesheet
text from the inspector. Two mutation scopes and an explicit call at
the end.

Kept one of the mutation scopes. Two shouldn't be necessary, and I have
confirmed that the crash tests for which this was justified earlier
don't crash when removing one of the scopes.

Moved the stylesheet modifications into CSSStyleSheet::setText().

Also moved clearing of the CSSOM wrappers before the mutation scope
declaration as the mutation scope constructor would unnecessarily
re-attach wrappers which would then be cleared right after.

R=esprehn@chromium.org,pfeldman@chromium.org
BUG=591599
==========

[rune, 2016-03-03 02:42:10]

ptal

[pfeldman, 2016-03-03 02:49:55]

lgtm

[commit-bot: I haz the power, 2016-03-03 03:37:40]

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

[commit-bot: I haz the power, 2016-03-03 03:37:40]

Dry run: This issue passed the CQ dry run.

[esprehn, 2016-03-03 04:08:27]

lgtm

[rune, 2016-03-03 07:35:35]

The CQ bit was checked by rune@opera.com

[commit-bot: I haz the power, 2016-03-03 07:35:50]

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

[commit-bot: I haz the power, 2016-03-03 07:41:26]

Description was changed from

==========
One instead of three resolverChanged replacing source in inspector.

resolverChanged() was called three times when replacing the stylesheet
text from the inspector. Two mutation scopes and an explicit call at
the end.

Kept one of the mutation scopes. Two shouldn't be necessary, and I have
confirmed that the crash tests for which this was justified earlier
don't crash when removing one of the scopes.

Moved the stylesheet modifications into CSSStyleSheet::setText().

Also moved clearing of the CSSOM wrappers before the mutation scope
declaration as the mutation scope constructor would unnecessarily
re-attach wrappers which would then be cleared right after.

R=esprehn@chromium.org,pfeldman@chromium.org
BUG=591599
==========

to

==========
One instead of three resolverChanged replacing source in inspector.

resolverChanged() was called three times when replacing the stylesheet
text from the inspector. Two mutation scopes and an explicit call at
the end.

Kept one of the mutation scopes. Two shouldn't be necessary, and I have
confirmed that the crash tests for which this was justified earlier
don't crash when removing one of the scopes.

Moved the stylesheet modifications into CSSStyleSheet::setText().

Also moved clearing of the CSSOM wrappers before the mutation scope
declaration as the mutation scope constructor would unnecessarily
re-attach wrappers which would then be cleared right after.

R=esprehn@chromium.org,pfeldman@chromium.org
BUG=591599
==========

[commit-bot: I haz the power, 2016-03-03 07:41:27]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-03-03 07:42:54]

Description was changed from

==========
One instead of three resolverChanged replacing source in inspector.

resolverChanged() was called three times when replacing the stylesheet
text from the inspector. Two mutation scopes and an explicit call at
the end.

Kept one of the mutation scopes. Two shouldn't be necessary, and I have
confirmed that the crash tests for which this was justified earlier
don't crash when removing one of the scopes.

Moved the stylesheet modifications into CSSStyleSheet::setText().

Also moved clearing of the CSSOM wrappers before the mutation scope
declaration as the mutation scope constructor would unnecessarily
re-attach wrappers which would then be cleared right after.

R=esprehn@chromium.org,pfeldman@chromium.org
BUG=591599
==========

to

==========
One instead of three resolverChanged replacing source in inspector.

resolverChanged() was called three times when replacing the stylesheet
text from the inspector. Two mutation scopes and an explicit call at
the end.

Kept one of the mutation scopes. Two shouldn't be necessary, and I have
confirmed that the crash tests for which this was justified earlier
don't crash when removing one of the scopes.

Moved the stylesheet modifications into CSSStyleSheet::setText().

Also moved clearing of the CSSOM wrappers before the mutation scope
declaration as the mutation scope constructor would unnecessarily
re-attach wrappers which would then be cleared right after.

R=esprehn@chromium.org,pfeldman@chromium.org
BUG=591599

Committed: https://crrev.com/5958ca8683dd9478a780d516205355a36c48c5aa
Cr-Commit-Position: refs/heads/master@{#378978}
==========

[commit-bot: I haz the power, 2016-03-03 07:42:56]

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