Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(121)

Issue 1021353003: Merging to M42 (Closed)

Created:
5 years, 9 months ago by ananta
Modified:
5 years, 9 months ago
CC:
chromium-reviews, derat+watch_chromium.org
Base URL:
https://chromium.googlesource.com/chromium/src.git@2311
Target Ref:
refs/pending/branch-heads/2311
Project:
chromium
Visibility:
Public.

Description

Merging to M42 Remove duplicate fonts from the font fallback list. We use std::unique to remove duplicate entries from the fallback list. BUG=467459 TBR=msw Review URL: https://codereview.chromium.org/1019743004 Cr-Commit-Position: refs/heads/master@{#321497} (cherry picked from commit 83de07b1894c61e53469c830c9f7c8986d9ae42c) Committed: https://chromium.googlesource.com/chromium/src/+/1e842dcd121e5dc4a787b612b4bce0610f5b9bf9

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+13 lines, -0 lines) Patch
M ui/gfx/render_text_harfbuzz.cc View 1 chunk +13 lines, -0 lines 0 comments Download

Messages

Total messages: 4 (1 generated)
ananta
5 years, 9 months ago (2015-03-21 01:44:10 UTC) #1
ananta
Committed patchset #1 (id:1) manually as 1e842dcd121e5dc4a787b612b4bce0610f5b9bf9 (presubmit successful).
5 years, 9 months ago (2015-03-21 01:46:58 UTC) #2
msw
5 years, 9 months ago (2015-03-21 02:53:16 UTC) #4
Message was sent while issue was closed.
+relevant folks, lgtm.

Powered by Google App Engine
This is Rietveld 408576698