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

Issue 10785: Debase our Uniscribe code. This moves FontUtils and all our Uniscribe code fr... (Closed)

Created:
12 years, 1 month ago by brettw
Modified:
9 years, 7 months ago
CC:
chromium-reviews_googlegroups.com
Visibility:
Public.

Description

Debase our Uniscribe code. This moves FontUtils and all our Uniscribe code from base/gfx to webkit/port/platform/graphics. I fixed the indenting and naming of the moved code. Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=5561

Patch Set 1 #

Patch Set 2 : '' #

Patch Set 3 : '' #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1579 lines, -3540 lines) Patch
M base/build/base_gfx.vcproj View 1 2 2 chunks +0 lines, -16 lines 0 comments Download
M base/build/base_unittests.vcproj View 1 2 1 chunk +0 lines, -4 lines 0 comments Download
M base/gfx/base_gfx.scons View 1 2 3 chunks +0 lines, -4 lines 0 comments Download
D base/gfx/font_utils.h View 1 2 1 chunk +0 lines, -87 lines 0 comments Download
D base/gfx/font_utils.cc View 1 2 1 chunk +0 lines, -336 lines 0 comments Download
D base/gfx/uniscribe.h View 1 2 1 chunk +0 lines, -366 lines 0 comments Download
D base/gfx/uniscribe.cc View 1 2 1 chunk +0 lines, -848 lines 0 comments Download
D base/gfx/uniscribe_unittest.cc View 1 2 1 chunk +0 lines, -140 lines 0 comments Download
M webkit/SConscript.port View 1 chunk +3 lines, -0 lines 0 comments Download
M webkit/build/port/port.vcproj View 1 2 4 chunks +28 lines, -12 lines 2 comments Download
D webkit/port/platform/UniscribeStateTextRun.h View 1 2 1 chunk +0 lines, -102 lines 0 comments Download
D webkit/port/platform/UniscribeStateTextRun.cpp View 1 2 1 chunk +0 lines, -128 lines 0 comments Download
A + webkit/port/platform/graphics/FontUtilsWin.h View 1 5 chunks +25 lines, -25 lines 0 comments Download
A + webkit/port/platform/graphics/FontUtilsWin.cpp View 1 6 chunks +236 lines, -233 lines 0 comments Download
M webkit/port/platform/graphics/FontWin.cpp View 1 2 4 chunks +7 lines, -6 lines 0 comments Download
M webkit/port/platform/graphics/GlyphPageTreeNodeWin.cpp View 1 2 2 chunks +6 lines, -6 lines 0 comments Download
A + webkit/port/platform/graphics/UniscribeHelper.h View 1 2 chunks +308 lines, -294 lines 0 comments Download
A + webkit/port/platform/graphics/UniscribeHelper.cpp View 1 3 chunks +718 lines, -698 lines 0 comments Download
A + webkit/port/platform/graphics/UniscribeHelperTextRun.h View 1 chunk +35 lines, -38 lines 0 comments Download
A + webkit/port/platform/graphics/UniscribeHelperTextRun.cpp View 1 chunk +83 lines, -76 lines 0 comments Download
A + webkit/port/platform/graphics/UniscribeHelper_unittest.cpp View 1 1 chunk +105 lines, -103 lines 3 comments Download
M webkit/port/rendering/RenderThemeWin.cpp View 1 2 2 chunks +16 lines, -14 lines 0 comments Download
M webkit/tools/test_shell/SConscript View 1 chunk +1 line, -0 lines 0 comments Download
M webkit/tools/test_shell/test_shell_tests.vcproj View 1 2 3 chunks +8 lines, -4 lines 2 comments Download

Messages

Total messages: 3 (0 generated)
brettw
12 years, 1 month ago (2008-11-17 05:24:36 UTC) #1
Erik does not do reviews
Overall LGTM. You might want to add someone who's a WebKit committer to the reviewer ...
12 years, 1 month ago (2008-11-17 17:01:34 UTC) #2
brettw
12 years, 1 month ago (2008-11-17 17:26:49 UTC) #3
http://codereview.chromium.org/10785/diff/246/259
File webkit/build/port/port.vcproj (right):

http://codereview.chromium.org/10785/diff/246/259#newcode718
Line 718:
RelativePath="..\..\port\platform\chromium\ClipboardUtilitiesChromium.cpp"
On 2008/11/17 17:01:34, Erik Kay wrote:
> it looks like you need to sort the vcproj

Actually, this change made it sorted.

http://codereview.chromium.org/10785/diff/246/252
File webkit/port/platform/graphics/UniscribeHelper_unittest.cpp (right):

http://codereview.chromium.org/10785/diff/246/252#newcode9
Line 9: #include "testing/gtest/include/gtest/gtest.h"
I don't know about gtest. Eric Seidel has tried to get gtest upstream. I think
we'll just see how it goes.

http://codereview.chromium.org/10785/diff/246/260
File webkit/tools/test_shell/test_shell_tests.vcproj (right):

http://codereview.chromium.org/10785/diff/246/260#newcode254
Line 254: RelativePath=".\test_shell_win.cc"
On 2008/11/17 17:01:34, Erik Kay wrote:
> sort the vcproj

Again, this makes it sorted.

Powered by Google App Engine
This is Rietveld 408576698