Bug 23700 - Upstream local changes to GlyphPageTreeNodeChromiumWin.cpp
Summary: Upstream local changes to GlyphPageTreeNodeChromiumWin.cpp
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Platform (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Dimitri Glazkov (Google)
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-02-02 20:38 PST by Dimitri Glazkov (Google)
Modified: 2009-02-04 11:07 PST (History)
1 user (show)

See Also:


Attachments
Upstream local changes to due to unforking, bug fixes. (2.71 KB, patch)
2009-02-02 20:40 PST, Dimitri Glazkov (Google)
darin: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Dimitri Glazkov (Google) 2009-02-02 20:38:03 PST
The change in fillBMPGlyphs is due to unforking of SimpleFontData and the change in fillNonBMPGlyphs is to address http://code.google.com/p/chromium/issues/detail?id=6303
Comment 1 Dimitri Glazkov (Google) 2009-02-02 20:40:35 PST
Created attachment 27274 [details]
Upstream local changes to due to unforking, bug fixes.

 WebCore/ChangeLog                                  |   11 +++++++++++
 .../chromium/GlyphPageTreeNodeChromiumWin.cpp      |   15 +++------------
 2 files changed, 14 insertions(+), 12 deletions(-)
Comment 2 Eric Seidel (no email) 2009-02-02 23:35:30 PST
Comment on attachment 27274 [details]
Upstream local changes to due to unforking, bug fixes.

It would be useful to know what these changes are for?  Otherwise the change looks fine... if you update the ChangeLog to point to the chromium bugs fixed (or at least the commits) that would be helpful.
Comment 3 Darin Adler 2009-02-03 09:28:32 PST
Comment on attachment 27274 [details]
Upstream local changes to due to unforking, bug fixes.

rs=me
Comment 4 Jungshik Shin 2009-02-03 11:17:01 PST
Dimitiri, this patch overlaps with what I put up for bug 23602. setDisableFontFallback bit relies on that patch. Can you remove that part from the patch here or merge two patches (for bug 23602 and this bug)? 

Eric, can you review the patch for bug 23602? 
Comment 5 Dimitri Glazkov (Google) 2009-02-03 11:38:13 PST
Oops -- I will remove your modification, Jungshik. Didn't mean to step on your toes.
Comment 6 Dimitri Glazkov (Google) 2009-02-04 11:07:55 PST
Landed as http://trac.webkit.org/changeset/40619