Changeset 56162 in webkit


Ignore:
Timestamp:
Mar 18, 2010 8:41:51 AM (14 years ago)
Author:
eric@webkit.org
Message:

2010-03-18 Adam Langley <agl@chromium.org>

Reviewed by David Levin.

[chromium] Fix inverted logic in per-strike renderer patch.

I screwed up in r55089 and got one of the conditions backwards. Thanks
to Roman Tsisyk for pointing it out.

https://bugs.webkit.org/show_bug.cgi?id=35495

  • platform/chromium/fast/text/chromium-linux-fontconfig-renderstyle.html: Added.

2010-03-18 Adam Langley <agl@chromium.org>

Reviewed by David Levin.

[chromium] Fix inverted logic in per-strike renderer patch.

I screwed up in r55089 and got one of the conditions backwards. Thanks
to Roman Tsisyk for pointing it out.

https://bugs.webkit.org/show_bug.cgi?id=35495

Test: platform/chromium/fast/text/chromium-linux-fontconfig-renderstyle.html

  • platform/graphics/chromium/FontPlatformDataLinux.cpp: (WebCore::FontPlatformData::querySystemForRenderStyle):
Location:
trunk
Files:
2 added
3 edited

Legend:

Unmodified
Added
Removed
  • trunk/LayoutTests/ChangeLog

    r56157 r56162  
     12010-03-18  Adam Langley  <agl@chromium.org>
     2
     3        Reviewed by David Levin.
     4
     5        [chromium] Fix inverted logic in per-strike renderer patch.
     6
     7        I screwed up in r55089 and got one of the conditions backwards. Thanks
     8        to Roman Tsisyk for pointing it out.
     9
     10        https://bugs.webkit.org/show_bug.cgi?id=35495
     11
     12        * platform/chromium/fast/text/chromium-linux-fontconfig-renderstyle.html: Added.
     13
    1142010-03-18  Sergio Villar Senin  <svillar@igalia.com>
    215
  • trunk/WebCore/ChangeLog

    r56161 r56162  
     12010-03-18  Adam Langley  <agl@chromium.org>
     2
     3        Reviewed by David Levin.
     4
     5        [chromium] Fix inverted logic in per-strike renderer patch.
     6
     7        I screwed up in r55089 and got one of the conditions backwards. Thanks
     8        to Roman Tsisyk for pointing it out.
     9
     10        https://bugs.webkit.org/show_bug.cgi?id=35495
     11
     12        Test: platform/chromium/fast/text/chromium-linux-fontconfig-renderstyle.html
     13
     14        * platform/graphics/chromium/FontPlatformDataLinux.cpp:
     15        (WebCore::FontPlatformData::querySystemForRenderStyle):
     16
    1172010-03-18  Pavel Feldman  <pfeldman@chromium.org>
    218
  • trunk/WebCore/platform/graphics/chromium/FontPlatformDataLinux.cpp

    r55089 r56162  
    198198void FontPlatformData::querySystemForRenderStyle()
    199199{
    200     if (m_family.length()) {
     200    if (!m_family.length()) {
    201201        // We don't have a family for this. Probably because it's a webfont. We
    202202        // set all the values to 'no preference' and take the defaults passed
Note: See TracChangeset for help on using the changeset viewer.