Try to do less work clearing the font-cache in StyleEngine::didDetach.

Try to do less work clearing the font-cache in StyleEngine::didDetach.

Clearing the whole font-cache without considering css-connected fonts
could be cheaper. This is a speculative fix for issue 677415.

BUG=677415
Review-Url: https://codereview.chromium.org/2622673003
Cr-Commit-Position: refs/heads/master@{#443202}
Committed: https://chromium.googlesource.com/chromium/src/+/1bb2f1d3c48652f9591ce16daa20f7c91ce0f761

[rune, 2017-01-10 14:34:55]

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

[commit-bot: I haz the power, 2017-01-10 14:35:11]

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, 2017-01-10 16:20:35]

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

[commit-bot: I haz the power, 2017-01-10 16:20:35]

Dry run: This issue passed the CQ dry run.

[rune, 2017-01-11 14:38:25]

rune@opera.com changed reviewers:
+ sigbjornf@opera.com

[sof, 2017-01-11 14:51:08]

I can see the value of avoiding removeFontFace().

However, as CSSFontSelector can have multiple clients, I'm wary this will be
clearing too abruptly or too much. Could you try to find someone more familiar
with this code to review?

[rune, 2017-01-11 16:02:36]

rune@opera.com changed reviewers:
+ drott@chromium.org, ksakamoto@chromium.org

[rune, 2017-01-11 16:02:37]

ksakamoto@ or drott@: is this something you could review?

[Kunihiko Sakamoto, 2017-01-12 01:59:14]

lgtm

[sof, 2017-01-12 06:32:27]

Overlaps with https://codereview.chromium.org/2624893003/

[rune, 2017-01-12 09:10:54]

The CQ bit was checked by rune@opera.com

[commit-bot: I haz the power, 2017-01-12 09:11:09]

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, 2017-01-12 10:49:17]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1484212254214980, "parent_rev":
"78dd11ea48a9967a31b7064cbdcea8eb61ca73dc", "commit_rev":
"1bb2f1d3c48652f9591ce16daa20f7c91ce0f761"}

[commit-bot: I haz the power, 2017-01-12 10:49:44]

Description was changed from

==========
Try to do less work clearing the font-cache in StyleEngine::didDetach.

Clearing the whole font-cache without considering css-connected fonts
could be cheaper. This is a speculative fix for issue 677415.

BUG=677415
==========

to

==========
Try to do less work clearing the font-cache in StyleEngine::didDetach.

Clearing the whole font-cache without considering css-connected fonts
could be cheaper. This is a speculative fix for issue 677415.

BUG=677415

Review-Url: https://codereview.chromium.org/2622673003
Cr-Commit-Position: refs/heads/master@{#443202}
Committed:
https://chromium.googlesource.com/chromium/src/+/1bb2f1d3c48652f9591ce16daa20...
==========

[commit-bot: I haz the power, 2017-01-12 10:49:45]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/1bb2f1d3c48652f9591ce16daa20...