[prev in list] [next in list] [prev in thread] [next in thread]
List: webkit-dev
Subject: [webkit-dev] WinCE Font work
From: Dan Bernstein <mitz () apple ! com>
Date: 2009-08-05 18:13:58
Message-ID: BA092EA2-3EF4-4E02-8D83-ADB428EFD689 () apple ! com
[Download RAW message or body]
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
_______________________________________________
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