[Webkit-unassigned] [Bug 88866] New: ASSERT_NOT_REACHED in StylePropertySet::fontValue when accessing font style property through JS after setting style font size.

[bugzilla-daemon at webkit.org]

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

           Summary: ASSERT_NOT_REACHED in StylePropertySet::fontValue when
                    accessing font style property through JS after setting
                    style font size.
           Product: WebKit
           Version: 528+ (Nightly build)
          Platform: Unspecified
        OS/Version: Unspecified
            Status: NEW
          Severity: Normal
          Priority: P2
         Component: CSS
        AssignedTo: webkit-unassigned at lists.webkit.org
        ReportedBy: michael.bruning at nokia.com
                CC: apavlov at chromium.org, michael.bruning at nokia.com


Created an attachment (id=147071)
 --> (https://bugs.webkit.org/attachment.cgi?id=147071&action=review)
Test case html file that triggers the assert.

Noticed with a Qt 5 WK2 trunk debug build (r119588).

To trigger, load the attached test.html file in a debug build (I used MiniBrowser).

ASSERTS with the following stack trace: 
/home/mibrunin/swork/webkit/Source/WebCore/css/StylePropertySet.cpp(285) : WTF::String WebCore::StylePropertySet::fontValue() const
1   0x7f97978f055f [...]/libQtWebKit.so.5(_ZNK7WebCore16StylePropertySet9fontValueEv+0x305) [0x7f97978f055f]
2   0x7f97978efb21 [...]/libQtWebKit.so.5(_ZNK7WebCore16StylePropertySet16getPropertyValueENS_13CSSPropertyIDE+0x3e7) [0x7f97978efb21]
3   0x7f979789eeb1 [...]/libQtWebKit.so.5(_ZN7WebCore30PropertySetCSSStyleDeclaration24getPropertyValueInternalENS_13CSSPropertyIDE+0x2d) [0x7f979789eeb1]
4   0x7f97976d822b [...]/libQtWebKit.so.5(+0x23d322b) [0x7f97976d822b]
5   0x7f97976d8512 [...]/libQtWebKit.so.5(+0x23d3512) [0x7f97976d8512]
6   0x7f97976d85ac [...]/libQtWebKit.so.5(+0x23d35ac) [0x7f97976d85ac]
7   0x7f9797579800 [...]/libQtWebKit.so.5(_ZNK3JSC12PropertySlot8getValueEPNS_9ExecStateENS_12PropertyNameE+0x76) [0x7f9797579800]
8   0x7f9797735927 [...]/libQtWebKit.so.5(_ZNK3JSC7JSValue3getEPNS_9ExecStateENS_12PropertyNameERNS_12PropertySlotE+0xd3) [0x7f9797735927]
9   0x7f9797735851 [...]/libQtWebKit.so.5(_ZNK3JSC7JSValue3getEPNS_9ExecStateENS_12PropertyNameE+0x4f) [0x7f9797735851]
10  0x7f9798c28646 [...]/libQtWebKit.so.5(cti_op_get_by_val+0x435) [0x7f9798c28646]
11  0x7f9798c22a0c [...]/libQtWebKit.so.5(+0x391da0c) [0x7f9798c22a0c]

I debugged this a bit and found out that after setting the font size through JS, the font size property is set, but not all the other longhand font properties that are expected to be present at least implicitly (e.g. CSSPropertyFontStyle) are not set and therefore, the assert is reached.

Alexander, if you can provide me with some guidance on where to look into fixing this, I'll be happy to do so.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.