New issue
Advanced search Search tips

Issue 921029 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Closed: Jan 14
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug



Sign in to add a comment

Font Cache can be primed to reject a family match if postscript name and family name do not match

Project Member Reported by drott@chromium.org, Jan 11

Issue description

When performing a local() lookup for a font name first, the search is stricter, as it can only be uniquely matched, for example, a local() lookup for "Times" does not find the Times font, as its Postscript / Full Font name is "Times-Roman" and "Times Roman" respectively. 

However, because whether it is supposed to be a unique match or a family match is not part of the font lookup key, the font cache now stores, that for this query, nothing was found. 

Later, when a family match for "Times" comes in, this is rejected, as the font cache cannot distinguish whether the stored result was from an unique match, or from a family match.

 
Project Member

Comment 1 by bugdroid1@chromium.org, Jan 11

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/e356e802917cea2049ad5f3aacc8ba40601c5639

commit e356e802917cea2049ad5f3aacc8ba40601c5639
Author: Dominik Röttsches <drott@chromium.org>
Date: Fri Jan 11 21:44:42 2019

Add match type to FontCacheKey information

Fixes situations in which a locally unique font name match masks a
result for a family name match.

This cannot be tested before src: local() matching is enabled by
switching the flag. A layout test covering this situation will be added
when the flag is enabled.

Bug:  921029 
Change-Id: I4123cb70aa65cb59e4a00c3a58e312ef6fa24b64
Reviewed-on: https://chromium-review.googlesource.com/c/1407069
Commit-Queue: Dominik Röttsches <drott@chromium.org>
Reviewed-by: Koji Ishii <kojii@chromium.org>
Reviewed-by: Emil A Eklund <eae@chromium.org>
Cr-Commit-Position: refs/heads/master@{#622167}
[modify] https://crrev.com/e356e802917cea2049ad5f3aacc8ba40601c5639/third_party/blink/renderer/core/css/css_font_face_source.cc
[modify] https://crrev.com/e356e802917cea2049ad5f3aacc8ba40601c5639/third_party/blink/renderer/core/css/css_font_face_source_test.cc
[modify] https://crrev.com/e356e802917cea2049ad5f3aacc8ba40601c5639/third_party/blink/renderer/core/css/css_segmented_font_face.cc
[modify] https://crrev.com/e356e802917cea2049ad5f3aacc8ba40601c5639/third_party/blink/renderer/platform/fonts/font_cache.cc
[modify] https://crrev.com/e356e802917cea2049ad5f3aacc8ba40601c5639/third_party/blink/renderer/platform/fonts/font_cache_key.h
[modify] https://crrev.com/e356e802917cea2049ad5f3aacc8ba40601c5639/third_party/blink/renderer/platform/fonts/font_description.cc
[modify] https://crrev.com/e356e802917cea2049ad5f3aacc8ba40601c5639/third_party/blink/renderer/platform/fonts/font_description.h
[modify] https://crrev.com/e356e802917cea2049ad5f3aacc8ba40601c5639/third_party/blink/renderer/platform/fonts/font_fallback_list.cc

Status: Fixed (was: Started)
Project Member

Comment 3 by bugdroid1@chromium.org, Jan 14

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/2e8e7c130cbb432732e973fd8c266a0608727b5f

commit 2e8e7c130cbb432732e973fd8c266a0608727b5f
Author: Dominik Röttsches <drott@chromium.org>
Date: Mon Jan 14 15:06:36 2019

Enable unique font matching matching by default

Now that all implementations for Android, Windows, Linux/CrOS and Mac
are in place, enable the feature globally. Chrome will match fonts
references in @font-face { src: local(<font-name>) } correctly by
postscript name or full font name. This addresses a long standing,
highly starred font matching issue in Chrome.

Bug: 828317, 627143,  921029 
Change-Id: I348855fdca817a4cd279f4d938e6d5ffe91ee004
Reviewed-on: https://chromium-review.googlesource.com/c/1400695
Commit-Queue: Dominik Röttsches <drott@chromium.org>
Reviewed-by: Avi Drissman <avi@chromium.org>
Reviewed-by: Emil A Eklund <eae@chromium.org>
Cr-Commit-Position: refs/heads/master@{#622456}
[modify] https://crrev.com/2e8e7c130cbb432732e973fd8c266a0608727b5f/content/public/common/content_features.cc
[modify] https://crrev.com/2e8e7c130cbb432732e973fd8c266a0608727b5f/third_party/blink/web_tests/NeverFixTests
[modify] https://crrev.com/2e8e7c130cbb432732e973fd8c266a0608727b5f/third_party/blink/web_tests/fast/css/font-face-character-fallback.html
[modify] https://crrev.com/2e8e7c130cbb432732e973fd8c266a0608727b5f/third_party/blink/web_tests/fast/css/font-face-descending-unicode-range.html
[modify] https://crrev.com/2e8e7c130cbb432732e973fd8c266a0608727b5f/third_party/blink/web_tests/fast/css/font-face-download-error.html
[modify] https://crrev.com/2e8e7c130cbb432732e973fd8c266a0608727b5f/third_party/blink/web_tests/fast/css/font-face-in-media-rule.html
[modify] https://crrev.com/2e8e7c130cbb432732e973fd8c266a0608727b5f/third_party/blink/web_tests/fast/css/font-face-multiple-ranges-for-unicode-range.html
[modify] https://crrev.com/2e8e7c130cbb432732e973fd8c266a0608727b5f/third_party/blink/web_tests/fast/css/font-face-unicode-range-monospace.html
[modify] https://crrev.com/2e8e7c130cbb432732e973fd8c266a0608727b5f/third_party/blink/web_tests/fast/css/fontfaceset-add-remove.html
[modify] https://crrev.com/2e8e7c130cbb432732e973fd8c266a0608727b5f/third_party/blink/web_tests/fast/css/fontfaceset-download-error.html
[modify] https://crrev.com/2e8e7c130cbb432732e973fd8c266a0608727b5f/third_party/blink/web_tests/fast/css/fontfaceset-multiple-faces.html
[modify] https://crrev.com/2e8e7c130cbb432732e973fd8c266a0608727b5f/third_party/blink/web_tests/fast/forms/select-popup/popup-menu-font-face.html
[modify] https://crrev.com/2e8e7c130cbb432732e973fd8c266a0608727b5f/third_party/blink/web_tests/fast/inline/vertical-align-with-fallback-fonts.html
[modify] https://crrev.com/2e8e7c130cbb432732e973fd8c266a0608727b5f/third_party/blink/web_tests/fast/text/capitalize-boundaries.html
[add] https://crrev.com/2e8e7c130cbb432732e973fd8c266a0608727b5f/third_party/blink/web_tests/fast/text/unique-vs-family-match.html
[modify] https://crrev.com/2e8e7c130cbb432732e973fd8c266a0608727b5f/third_party/blink/web_tests/inspector-protocol/layout-fonts/unicode-range-order.js
[add] https://crrev.com/2e8e7c130cbb432732e973fd8c266a0608727b5f/third_party/blink/web_tests/platform/mac-mac10.10/fast/text/unique-vs-family-match-expected.png
[add] https://crrev.com/2e8e7c130cbb432732e973fd8c266a0608727b5f/third_party/blink/web_tests/platform/mac-mac10.12/fast/text/unique-vs-family-match-expected.png
[add] https://crrev.com/2e8e7c130cbb432732e973fd8c266a0608727b5f/third_party/blink/web_tests/platform/mac/fast/text/unique-vs-family-match-expected.png

Project Member

Comment 4 by bugdroid1@chromium.org, Jan 16

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/f6c717eb8b60f311b6357b3ddace4c79b1a2e64e

commit f6c717eb8b60f311b6357b3ddace4c79b1a2e64e
Author: Dominik Röttsches <drott@chromium.org>
Date: Wed Jan 16 00:08:23 2019

Revert "Enable unique font matching matching by default"

This reverts commit 2e8e7c130cbb432732e973fd8c266a0608727b5f.

Reason for revert: Crashes observed in Windows Canary

Bug: 922183

Original change's description:
> Enable unique font matching matching by default
> 
> Now that all implementations for Android, Windows, Linux/CrOS and Mac
> are in place, enable the feature globally. Chrome will match fonts
> references in @font-face { src: local(<font-name>) } correctly by
> postscript name or full font name. This addresses a long standing,
> highly starred font matching issue in Chrome.
> 
> Bug: 828317, 627143,  921029 
> Change-Id: I348855fdca817a4cd279f4d938e6d5ffe91ee004
> Reviewed-on: https://chromium-review.googlesource.com/c/1400695
> Commit-Queue: Dominik Röttsches <drott@chromium.org>
> Reviewed-by: Avi Drissman <avi@chromium.org>
> Reviewed-by: Emil A Eklund <eae@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#622456}

TBR=avi@chromium.org,eae@chromium.org,drott@chromium.org

# Not skipping CQ checks because original CL landed > 1 day ago.

Bug: 828317, 627143,  921029 
Change-Id: I6a89f36923abb7d87de712c9ac0852da06a66d37
Reviewed-on: https://chromium-review.googlesource.com/c/1412824
Reviewed-by: Dominik Röttsches <drott@chromium.org>
Commit-Queue: Dominik Röttsches <drott@chromium.org>
Cr-Commit-Position: refs/heads/master@{#622898}
[modify] https://crrev.com/f6c717eb8b60f311b6357b3ddace4c79b1a2e64e/content/public/common/content_features.cc
[modify] https://crrev.com/f6c717eb8b60f311b6357b3ddace4c79b1a2e64e/third_party/blink/web_tests/NeverFixTests
[modify] https://crrev.com/f6c717eb8b60f311b6357b3ddace4c79b1a2e64e/third_party/blink/web_tests/fast/css/font-face-character-fallback.html
[modify] https://crrev.com/f6c717eb8b60f311b6357b3ddace4c79b1a2e64e/third_party/blink/web_tests/fast/css/font-face-descending-unicode-range.html
[modify] https://crrev.com/f6c717eb8b60f311b6357b3ddace4c79b1a2e64e/third_party/blink/web_tests/fast/css/font-face-download-error.html
[modify] https://crrev.com/f6c717eb8b60f311b6357b3ddace4c79b1a2e64e/third_party/blink/web_tests/fast/css/font-face-in-media-rule.html
[modify] https://crrev.com/f6c717eb8b60f311b6357b3ddace4c79b1a2e64e/third_party/blink/web_tests/fast/css/font-face-multiple-ranges-for-unicode-range.html
[modify] https://crrev.com/f6c717eb8b60f311b6357b3ddace4c79b1a2e64e/third_party/blink/web_tests/fast/css/font-face-unicode-range-monospace.html
[modify] https://crrev.com/f6c717eb8b60f311b6357b3ddace4c79b1a2e64e/third_party/blink/web_tests/fast/css/fontfaceset-add-remove.html
[modify] https://crrev.com/f6c717eb8b60f311b6357b3ddace4c79b1a2e64e/third_party/blink/web_tests/fast/css/fontfaceset-download-error.html
[modify] https://crrev.com/f6c717eb8b60f311b6357b3ddace4c79b1a2e64e/third_party/blink/web_tests/fast/css/fontfaceset-multiple-faces.html
[modify] https://crrev.com/f6c717eb8b60f311b6357b3ddace4c79b1a2e64e/third_party/blink/web_tests/fast/forms/select-popup/popup-menu-font-face.html
[modify] https://crrev.com/f6c717eb8b60f311b6357b3ddace4c79b1a2e64e/third_party/blink/web_tests/fast/inline/vertical-align-with-fallback-fonts.html
[modify] https://crrev.com/f6c717eb8b60f311b6357b3ddace4c79b1a2e64e/third_party/blink/web_tests/fast/text/capitalize-boundaries.html
[delete] https://crrev.com/286a6a456f744bc4e41a4e9f7037aefcdb18ff61/third_party/blink/web_tests/fast/text/unique-vs-family-match.html
[modify] https://crrev.com/f6c717eb8b60f311b6357b3ddace4c79b1a2e64e/third_party/blink/web_tests/inspector-protocol/layout-fonts/unicode-range-order.js
[delete] https://crrev.com/286a6a456f744bc4e41a4e9f7037aefcdb18ff61/third_party/blink/web_tests/platform/mac-mac10.10/fast/text/unique-vs-family-match-expected.png
[delete] https://crrev.com/286a6a456f744bc4e41a4e9f7037aefcdb18ff61/third_party/blink/web_tests/platform/mac-mac10.12/fast/text/unique-vs-family-match-expected.png
[delete] https://crrev.com/286a6a456f744bc4e41a4e9f7037aefcdb18ff61/third_party/blink/web_tests/platform/mac/fast/text/unique-vs-family-match-expected.png

Project Member

Comment 5 by bugdroid1@chromium.org, Jan 16 (6 days ago)

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/1d366d59cf85b3013d0e3f4a9ee9edb98b99a49f

commit 1d366d59cf85b3013d0e3f4a9ee9edb98b99a49f
Author: Dominik Röttsches <drott@chromium.org>
Date: Wed Jan 16 11:36:33 2019

Reland "Enable unique font matching matching by default"

This reverts commit f6c717eb8b60f311b6357b3ddace4c79b1a2e64e.

Reason for revert: Crash has been addressed in 
https://chromium-review.googlesource.com/c/chromium/src/+/1414852

Original change's description:
> Revert "Enable unique font matching matching by default"
> 
> This reverts commit 2e8e7c130cbb432732e973fd8c266a0608727b5f.
> 
> Reason for revert: Crashes observed in Windows Canary
> 
> Bug: 922183
> 
> Original change's description:
> > Enable unique font matching matching by default
> > 
> > Now that all implementations for Android, Windows, Linux/CrOS and Mac
> > are in place, enable the feature globally. Chrome will match fonts
> > references in @font-face { src: local(<font-name>) } correctly by
> > postscript name or full font name. This addresses a long standing,
> > highly starred font matching issue in Chrome.
> > 
> > Bug: 828317, 627143,  921029 
> > Change-Id: I348855fdca817a4cd279f4d938e6d5ffe91ee004
> > Reviewed-on: https://chromium-review.googlesource.com/c/1400695
> > Commit-Queue: Dominik Röttsches <drott@chromium.org>
> > Reviewed-by: Avi Drissman <avi@chromium.org>
> > Reviewed-by: Emil A Eklund <eae@chromium.org>
> > Cr-Commit-Position: refs/heads/master@{#622456}
> 
> TBR=avi@chromium.org,eae@chromium.org,drott@chromium.org
> 
> # Not skipping CQ checks because original CL landed > 1 day ago.
> 
> Bug: 828317, 627143,  921029 
> Change-Id: I6a89f36923abb7d87de712c9ac0852da06a66d37
> Reviewed-on: https://chromium-review.googlesource.com/c/1412824
> Reviewed-by: Dominik Röttsches <drott@chromium.org>
> Commit-Queue: Dominik Röttsches <drott@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#622898}

TBR=avi@chromium.org,eae@chromium.org,drott@chromium.org

Change-Id: I915e1bb20ee51325befa5ef7eab07f78eaa8c974
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 922183, 828317, 627143,  921029 , 922183
Reviewed-on: https://chromium-review.googlesource.com/c/1413603
Reviewed-by: Dominik Röttsches <drott@chromium.org>
Commit-Queue: Dominik Röttsches <drott@chromium.org>
Cr-Commit-Position: refs/heads/master@{#623199}
[modify] https://crrev.com/1d366d59cf85b3013d0e3f4a9ee9edb98b99a49f/content/public/common/content_features.cc
[modify] https://crrev.com/1d366d59cf85b3013d0e3f4a9ee9edb98b99a49f/third_party/blink/web_tests/NeverFixTests
[modify] https://crrev.com/1d366d59cf85b3013d0e3f4a9ee9edb98b99a49f/third_party/blink/web_tests/fast/css/font-face-character-fallback.html
[modify] https://crrev.com/1d366d59cf85b3013d0e3f4a9ee9edb98b99a49f/third_party/blink/web_tests/fast/css/font-face-descending-unicode-range.html
[modify] https://crrev.com/1d366d59cf85b3013d0e3f4a9ee9edb98b99a49f/third_party/blink/web_tests/fast/css/font-face-download-error.html
[modify] https://crrev.com/1d366d59cf85b3013d0e3f4a9ee9edb98b99a49f/third_party/blink/web_tests/fast/css/font-face-in-media-rule.html
[modify] https://crrev.com/1d366d59cf85b3013d0e3f4a9ee9edb98b99a49f/third_party/blink/web_tests/fast/css/font-face-multiple-ranges-for-unicode-range.html
[modify] https://crrev.com/1d366d59cf85b3013d0e3f4a9ee9edb98b99a49f/third_party/blink/web_tests/fast/css/font-face-unicode-range-monospace.html
[modify] https://crrev.com/1d366d59cf85b3013d0e3f4a9ee9edb98b99a49f/third_party/blink/web_tests/fast/css/fontfaceset-add-remove.html
[modify] https://crrev.com/1d366d59cf85b3013d0e3f4a9ee9edb98b99a49f/third_party/blink/web_tests/fast/css/fontfaceset-download-error.html
[modify] https://crrev.com/1d366d59cf85b3013d0e3f4a9ee9edb98b99a49f/third_party/blink/web_tests/fast/css/fontfaceset-multiple-faces.html
[modify] https://crrev.com/1d366d59cf85b3013d0e3f4a9ee9edb98b99a49f/third_party/blink/web_tests/fast/forms/select-popup/popup-menu-font-face.html
[modify] https://crrev.com/1d366d59cf85b3013d0e3f4a9ee9edb98b99a49f/third_party/blink/web_tests/fast/inline/vertical-align-with-fallback-fonts.html
[modify] https://crrev.com/1d366d59cf85b3013d0e3f4a9ee9edb98b99a49f/third_party/blink/web_tests/fast/text/capitalize-boundaries.html
[add] https://crrev.com/1d366d59cf85b3013d0e3f4a9ee9edb98b99a49f/third_party/blink/web_tests/fast/text/unique-vs-family-match.html
[modify] https://crrev.com/1d366d59cf85b3013d0e3f4a9ee9edb98b99a49f/third_party/blink/web_tests/inspector-protocol/layout-fonts/unicode-range-order.js
[add] https://crrev.com/1d366d59cf85b3013d0e3f4a9ee9edb98b99a49f/third_party/blink/web_tests/platform/mac-mac10.10/fast/text/unique-vs-family-match-expected.png
[add] https://crrev.com/1d366d59cf85b3013d0e3f4a9ee9edb98b99a49f/third_party/blink/web_tests/platform/mac-mac10.12/fast/text/unique-vs-family-match-expected.png
[add] https://crrev.com/1d366d59cf85b3013d0e3f4a9ee9edb98b99a49f/third_party/blink/web_tests/platform/mac/fast/text/unique-vs-family-match-expected.png

Project Member

Comment 6 by bugdroid1@chromium.org, Jan 18 (4 days ago)

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/09e747504f7caf471e94acc4916864790517e40c

commit 09e747504f7caf471e94acc4916864790517e40c
Author: Dominik Röttsches <drott@chromium.org>
Date: Fri Jan 18 14:37:47 2019

Revert "Reland "Enable unique font matching matching by default""

This reverts commit 1d366d59cf85b3013d0e3f4a9ee9edb98b99a49f.

Reason for revert: 
* Preliminary Windows font table lookup build times look too high
* Time to first paint affected on some lower end Android tests 

Original change's description:
> Reland "Enable unique font matching matching by default"
> 
> This reverts commit f6c717eb8b60f311b6357b3ddace4c79b1a2e64e.
> 
> Reason for revert: Crash has been addressed in 
> https://chromium-review.googlesource.com/c/chromium/src/+/1414852
> 
> Original change's description:
> > Revert "Enable unique font matching matching by default"
> > 
> > This reverts commit 2e8e7c130cbb432732e973fd8c266a0608727b5f.
> > 
> > Reason for revert: Crashes observed in Windows Canary
> > 
> > Bug: 922183
> > 
> > Original change's description:
> > > Enable unique font matching matching by default
> > > 
> > > Now that all implementations for Android, Windows, Linux/CrOS and Mac
> > > are in place, enable the feature globally. Chrome will match fonts
> > > references in @font-face { src: local(<font-name>) } correctly by
> > > postscript name or full font name. This addresses a long standing,
> > > highly starred font matching issue in Chrome.
> > > 
> > > Bug: 828317, 627143,  921029 
> > > Change-Id: I348855fdca817a4cd279f4d938e6d5ffe91ee004
> > > Reviewed-on: https://chromium-review.googlesource.com/c/1400695
> > > Commit-Queue: Dominik Röttsches <drott@chromium.org>
> > > Reviewed-by: Avi Drissman <avi@chromium.org>
> > > Reviewed-by: Emil A Eklund <eae@chromium.org>
> > > Cr-Commit-Position: refs/heads/master@{#622456}
> > 
> > TBR=avi@chromium.org,eae@chromium.org,drott@chromium.org
> > 
> > # Not skipping CQ checks because original CL landed > 1 day ago.
> > 
> > Bug: 828317, 627143,  921029 
> > Change-Id: I6a89f36923abb7d87de712c9ac0852da06a66d37
> > Reviewed-on: https://chromium-review.googlesource.com/c/1412824
> > Reviewed-by: Dominik Röttsches <drott@chromium.org>
> > Commit-Queue: Dominik Röttsches <drott@chromium.org>
> > Cr-Commit-Position: refs/heads/master@{#622898}
> 
> TBR=avi@chromium.org,eae@chromium.org,drott@chromium.org
> 
> Change-Id: I915e1bb20ee51325befa5ef7eab07f78eaa8c974
> No-Presubmit: true
> No-Tree-Checks: true
> No-Try: true
> Bug: 922183, 828317, 627143,  921029 , 922183
> Reviewed-on: https://chromium-review.googlesource.com/c/1413603
> Reviewed-by: Dominik Röttsches <drott@chromium.org>
> Commit-Queue: Dominik Röttsches <drott@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#623199}

TBR=avi@chromium.org,eae@chromium.org,drott@chromium.org

# Not skipping CQ checks because original CL landed > 1 day ago.

Bug: 922183, 828317, 627143,  921029 , 923245,  922495 , 922941
Change-Id: I27efa4f06148c2fc58d7feb5f15f48623713a570
Reviewed-on: https://chromium-review.googlesource.com/c/1421099
Reviewed-by: Dominik Röttsches <drott@chromium.org>
Commit-Queue: Dominik Röttsches <drott@chromium.org>
Cr-Commit-Position: refs/heads/master@{#624117}
[modify] https://crrev.com/09e747504f7caf471e94acc4916864790517e40c/content/public/common/content_features.cc
[modify] https://crrev.com/09e747504f7caf471e94acc4916864790517e40c/third_party/blink/web_tests/NeverFixTests
[modify] https://crrev.com/09e747504f7caf471e94acc4916864790517e40c/third_party/blink/web_tests/fast/css/font-face-character-fallback.html
[modify] https://crrev.com/09e747504f7caf471e94acc4916864790517e40c/third_party/blink/web_tests/fast/css/font-face-descending-unicode-range.html
[modify] https://crrev.com/09e747504f7caf471e94acc4916864790517e40c/third_party/blink/web_tests/fast/css/font-face-download-error.html
[modify] https://crrev.com/09e747504f7caf471e94acc4916864790517e40c/third_party/blink/web_tests/fast/css/font-face-in-media-rule.html
[modify] https://crrev.com/09e747504f7caf471e94acc4916864790517e40c/third_party/blink/web_tests/fast/css/font-face-multiple-ranges-for-unicode-range.html
[modify] https://crrev.com/09e747504f7caf471e94acc4916864790517e40c/third_party/blink/web_tests/fast/css/font-face-unicode-range-monospace.html
[modify] https://crrev.com/09e747504f7caf471e94acc4916864790517e40c/third_party/blink/web_tests/fast/css/fontfaceset-add-remove.html
[modify] https://crrev.com/09e747504f7caf471e94acc4916864790517e40c/third_party/blink/web_tests/fast/css/fontfaceset-download-error.html
[modify] https://crrev.com/09e747504f7caf471e94acc4916864790517e40c/third_party/blink/web_tests/fast/css/fontfaceset-multiple-faces.html
[modify] https://crrev.com/09e747504f7caf471e94acc4916864790517e40c/third_party/blink/web_tests/fast/forms/select-popup/popup-menu-font-face.html
[modify] https://crrev.com/09e747504f7caf471e94acc4916864790517e40c/third_party/blink/web_tests/fast/inline/vertical-align-with-fallback-fonts.html
[modify] https://crrev.com/09e747504f7caf471e94acc4916864790517e40c/third_party/blink/web_tests/fast/text/capitalize-boundaries.html
[delete] https://crrev.com/16c9e690685282e3e5d61b393347352b9507e6cd/third_party/blink/web_tests/fast/text/unique-vs-family-match.html
[modify] https://crrev.com/09e747504f7caf471e94acc4916864790517e40c/third_party/blink/web_tests/inspector-protocol/layout-fonts/unicode-range-order.js
[delete] https://crrev.com/16c9e690685282e3e5d61b393347352b9507e6cd/third_party/blink/web_tests/platform/mac-mac10.10/fast/text/unique-vs-family-match-expected.png
[delete] https://crrev.com/16c9e690685282e3e5d61b393347352b9507e6cd/third_party/blink/web_tests/platform/mac-mac10.12/fast/text/unique-vs-family-match-expected.png
[delete] https://crrev.com/16c9e690685282e3e5d61b393347352b9507e6cd/third_party/blink/web_tests/platform/mac/fast/text/unique-vs-family-match-expected.png

Sign in to add a comment