[prev in list] [next in list] [prev in thread] [next in thread] 

List:       webkit-dev
Subject:    Re: [webkit-dev] WinCE Font work
From:       George Staikos <staikos () kde ! org>
Date:       2009-08-05 18:18:15
Message-ID: C0FF6689-B076-488D-AA9D-32CC409C0F0B () kde ! org
[Download RAW message or body]


Yes I agree; I was just going to r- this patch in fact.

Some of these problems arose from the fact that we started this work  
nearly two years ago and were working outside of SVN so it's taking  
some time to realign.

On 5-Aug-09, at 2:13 PM, Dan Bernstein wrote:

> Hi,
>
> Today while looking at the patch at <https://bugs.webkit.org/ 
> show_bug.cgi?id=28021> I noticed that the WinCE version of  
> FontCustomPlatformData includes CachedFont.h and uses CachedFont.
>
> It is incorrect for classes in the platform layer to have any  
> knowledge of higher WebCore layers (essentially, anything outside  
> the platform hierarchy). In this case, it also necessitated a  
> platform difference in the createFontCustomPlatformData call site  
> in CachedFont::ensureCustomFontData(), which is how I noticed it.
>
> I think once you should correct this and any other layering  
> violations in the WinCE platform layer, and only then submit  
> patches with changes to cross-platform code, if any such changes  
> are still deemed necessary. I don’t think <https://bugs.webkit.org/ 
> show_bug.cgi?id=27734> and <https://bugs.webkit.org/show_bug.cgi? 
> id=28021> should be reviewed until then.
>
> Thanks,
> —Dan

--
George Staikos
Torch Mobile Inc.
http://www.torchmobile.com/

_______________________________________________
webkit-dev mailing list
webkit-dev@lists.webkit.org
http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev

[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic