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

Issue 18176: Linux: stroke font outlines.... (Closed)

Created:
11 years, 11 months ago by Evan Stade
Modified:
9 years, 6 months ago
Reviewers:
agl
CC:
chromium-reviews_googlegroups.com
Visibility:
Public.

Description

Linux: stroke font outlines. TEST=LayoutTests/svg/custom/pointer-events-text.svg Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=8312

Patch Set 1 #

Total comments: 2

Patch Set 2 : '' #

Unified diffs Side-by-side diffs Delta from patch set Stats (+32 lines, -9 lines) Patch
M third_party/WebKit/WebCore/platform/graphics/chromium/FontLinux.cpp View 1 2 chunks +32 lines, -9 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
Evan Stade
11 years, 11 months ago (2009-01-16 22:05:21 UTC) #1
agl
LGTM http://codereview.chromium.org/18176/diff/1/2 File third_party/WebKit/WebCore/platform/graphics/chromium/FontLinux.cpp (right): http://codereview.chromium.org/18176/diff/1/2#newcode78 Line 78: if (textMode & cTextFill) { Are cTextFill ...
11 years, 11 months ago (2009-01-16 22:12:39 UTC) #2
Evan Stade
11 years, 11 months ago (2009-01-16 22:23:11 UTC) #3
Yea, we do draw twice. That's the same thing that FontChromiumWin does. I'll add
a comment.

Powered by Google App Engine
This is Rietveld 408576698