Chromium Code Reviews
Help | Chromium Project | Sign in
(941)

Issue 8590028: Use new getFontFamilyForCharacters() API. (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
2 years, 5 months ago by bashi1
Modified:
2 years, 5 months ago
Reviewers:
tony, darin, brettw, piman
CC:
chromium-reviews_chromium.org, joi+watch-content_chromium.org, darin-cc_chromium.org, jam, dpranke-watch+content_chromium.org
Visibility:
Public.

Description

Use new getFontFamilyForCharacters() API.

Uses new getFontFamilyForCharacters() WebKit Chromium API so that
Chromium can pass bold and italic property. This will fix issue 32109.

No behavior change at this time. The CL will work after the fix of
WebKit side is completed.

The WebKit bug entry is https://bugs.webkit.org/show_bug.cgi?id=38701.

BUG=32109
TEST=compiled.


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=110871

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+69 lines, -41 lines) Lint Patch
M content/browser/renderer_host/render_sandbox_host_linux.cc View 1 chunk +9 lines, -5 lines 0 comments 0 errors Download
M content/common/child_process_sandbox_support_impl_linux.h View 2 chunks +7 lines, -5 lines 0 comments 0 errors Download
M content/common/child_process_sandbox_support_impl_linux.cc View 3 chunks +14 lines, -6 lines 0 comments 0 errors Download
M content/ppapi_plugin/ppapi_webkitplatformsupport_impl.cc View 4 chunks +20 lines, -13 lines 0 comments 1 errors Download
M content/renderer/renderer_webkitplatformsupport_impl.cc View 4 chunks +19 lines, -12 lines 0 comments 0 errors Download
Commit:

Messages

Total messages: 9
bashi1
Hi, Could you take a look?
2 years, 5 months ago #1
darin
LGTM
2 years, 5 months ago #2
tony
LGTM2
2 years, 5 months ago #3
I haz the power (commit-bot)
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/bashi@chromium.org/8590028/1
2 years, 5 months ago #4
I haz the power (commit-bot)
Presubmit check for 8590028-1 failed and returned exit status 1. Running presubmit commit checks ...
2 years, 5 months ago #5
tony
+brettw,piman for content/ppapi_plugin.
2 years, 5 months ago #6
piman
lgtm
2 years, 5 months ago #7
I haz the power (commit-bot)
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/bashi@chromium.org/8590028/1
2 years, 5 months ago #8
I haz the power (commit-bot)
2 years, 5 months ago #9
Change committed as 110871
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld 1280:2d3e6564b7b6