Bug 139283

Summary: Fix style sharing with the "type" and "readonly" attributes
Product: WebKit Reporter: Benjamin Poulain <benjamin>
Component: New BugsAssignee: Benjamin Poulain <benjamin>
Status: RESOLVED FIXED    
Severity: Normal CC: jeffrey+webkit, kling, koivisto, mikey.m, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch koivisto: review+

Description Benjamin Poulain 2014-12-04 19:43:46 PST
Fix style sharing with the "type" and "readonly" attributes
Comment 1 Benjamin Poulain 2014-12-04 20:04:04 PST
Created attachment 242612 [details]
Patch
Comment 2 Benjamin Poulain 2014-12-04 20:14:16 PST
Comment on attachment 242612 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=242612&action=review

> Source/WebCore/css/RuleSet.cpp:105
> +static bool containsUncommonAttributeSelector(const CSSSelector& rootSelector, bool startsOnRightmostElement)

Before landing I should rename this boolean.

startsOnRightmostElement->matchesRightmostElement
Comment 3 Antti Koivisto 2014-12-05 12:50:43 PST
Comment on attachment 242612 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=242612&action=review

> Source/WebCore/css/RuleSet.cpp:111
> +            // It is a very aggressive solution which disable a lot of style sharing on pages that rely on attributes for styling.

seems more 'primitive' than 'aggressive'
Comment 4 Benjamin Poulain 2014-12-05 12:57:35 PST
Committed r176864: <http://trac.webkit.org/changeset/176864>
Comment 5 Benjamin Poulain 2015-02-01 17:58:38 PST
*** Bug 123239 has been marked as a duplicate of this bug. ***
Comment 6 Benjamin Poulain 2015-02-01 18:03:11 PST
*** Bug 119000 has been marked as a duplicate of this bug. ***
Comment 7 Tim Nguyen (:ntim) 2023-04-28 21:11:19 PDT
rdar://14643850