Removed Document::addedStyleSheet.

Removed Document::addedStyleSheet.

Replace it with resolverChanged(). It was only a call from StyleEngine
into Document and back into StyleEngine. Removed a couple of comments
referring to the removed method which were confusing/out-of-date.

Moved call to resolverChanged() immediately following
modifiedStyleSheetCandidateNode() into the latter method.

Made StyleEngine::markDocumentDirty() private.

No functional changes.

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

[rune, 2016-02-22 12:59:05]

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

[commit-bot: I haz the power, 2016-02-22 12:59:45]

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

[commit-bot: I haz the power, 2016-02-22 14:22:38]

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

[commit-bot: I haz the power, 2016-02-22 14:22:38]

Dry run: This issue passed the CQ dry run.

[rune, 2016-02-23 01:02:43]

Description was changed from

==========
Removed Document::addedStyleSheet.

Replace it with resolverChanged(). It was only a call from StyleEngine
into Document and back into StyleEngine. Removed a couple of comments
referring to the removed method which were confusing/out-of-date.

Moved call to resolverChanged() immediately following
modifiedStyleSheetCandidateNode() into the latter method.

Made StyleEngine::markDocumentDirty() private.

No functional changes.
==========

to

==========
Removed Document::addedStyleSheet.

Replace it with resolverChanged(). It was only a call from StyleEngine
into Document and back into StyleEngine. Removed a couple of comments
referring to the removed method which were confusing/out-of-date.

Moved call to resolverChanged() immediately following
modifiedStyleSheetCandidateNode() into the latter method.

Made StyleEngine::markDocumentDirty() private.

No functional changes.

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

[rune, 2016-02-23 01:02:43]

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

[rune, 2016-02-23 01:05:39]

I'm in the process of moving resolverChanged() calls, which synchronously
updates the active stylesheets, into StyleEngine and make it asynchronous
afterwards.

[esprehn, 2016-02-23 01:18:04]

lgtm, hopefully we'll get to the place where the idea behind those added/removed
methods is handled (adding/removing a sheet should be cheaper than a recalc the
whole world, we should instead use invalidation sets and delete
StyleInvalidationAnalysis).

[rune, 2016-02-23 01:22:11]

The CQ bit was checked by rune@opera.com

[rune, 2016-02-23 01:23:56]

On 2016/02/23 01:18:04, esprehn wrote:
> lgtm, hopefully we'll get to the place where the idea behind those
added/removed
> methods is handled (adding/removing a sheet should be cheaper than a recalc
the
> whole world, we should instead use invalidation sets and delete
> StyleInvalidationAnalysis).

Yes. I'm approaching that, inch by inch (or bit by bit).

[commit-bot: I haz the power, 2016-02-23 01:23:57]

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

[commit-bot: I haz the power, 2016-02-23 01:31:03]

Description was changed from

==========
Removed Document::addedStyleSheet.

Replace it with resolverChanged(). It was only a call from StyleEngine
into Document and back into StyleEngine. Removed a couple of comments
referring to the removed method which were confusing/out-of-date.

Moved call to resolverChanged() immediately following
modifiedStyleSheetCandidateNode() into the latter method.

Made StyleEngine::markDocumentDirty() private.

No functional changes.

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

to

==========
Removed Document::addedStyleSheet.

Replace it with resolverChanged(). It was only a call from StyleEngine
into Document and back into StyleEngine. Removed a couple of comments
referring to the removed method which were confusing/out-of-date.

Moved call to resolverChanged() immediately following
modifiedStyleSheetCandidateNode() into the latter method.

Made StyleEngine::markDocumentDirty() private.

No functional changes.

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

[commit-bot: I haz the power, 2016-02-23 01:31:03]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-02-23 01:31:59]

Description was changed from

==========
Removed Document::addedStyleSheet.

Replace it with resolverChanged(). It was only a call from StyleEngine
into Document and back into StyleEngine. Removed a couple of comments
referring to the removed method which were confusing/out-of-date.

Moved call to resolverChanged() immediately following
modifiedStyleSheetCandidateNode() into the latter method.

Made StyleEngine::markDocumentDirty() private.

No functional changes.

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

to

==========
Removed Document::addedStyleSheet.

Replace it with resolverChanged(). It was only a call from StyleEngine
into Document and back into StyleEngine. Removed a couple of comments
referring to the removed method which were confusing/out-of-date.

Moved call to resolverChanged() immediately following
modifiedStyleSheetCandidateNode() into the latter method.

Made StyleEngine::markDocumentDirty() private.

No functional changes.

BUG=567021

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

[commit-bot: I haz the power, 2016-02-23 01:32:00]

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