Bug 164577

Summary: Updating class name doesn't update the slotted content's style
Product: WebKit Reporter: Ryosuke Niwa <rniwa>
Component: CSSAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: cdumez, commit-queue, dbates, esprehn+autocc, jan, kangil.han, koivisto, webkit-bug-importer, webkit
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 148695    
Attachments:
Description Flags
Demo
none
patch
none
patch
none
patch
rniwa: review+
patch for landing none

Description Ryosuke Niwa 2016-11-09 18:35:11 PST
Created attachment 294319 [details]
Demo

::slotted(.selected) wouldn't apply when the class name is applied dynamically.
Comment 1 Radar WebKit Bug Importer 2016-11-10 13:27:19 PST
<rdar://problem/29205873>
Comment 2 Ryosuke Niwa 2016-11-10 15:42:40 PST
*** Bug 160865 has been marked as a duplicate of this bug. ***
Comment 3 Ryosuke Niwa 2016-11-10 15:43:26 PST
A few more test cases listed on https://polymerlabs.github.io/wc-demos/
Comment 4 Antti Koivisto 2016-11-11 03:42:57 PST
Created attachment 294494 [details]
patch
Comment 5 Antti Koivisto 2016-11-11 09:38:10 PST
Created attachment 294501 [details]
patch
Comment 6 Antti Koivisto 2016-11-11 09:43:34 PST
Created attachment 294502 [details]
patch
Comment 7 Ryosuke Niwa 2016-11-11 13:06:53 PST
Comment on attachment 294502 [details]
patch

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

> LayoutTests/fast/shadow-dom/css-scoping-slotted-invalidation.html:86
> +

Maybe we should add a test which involves multiple slots in a single shadow tree?
Comment 8 Antti Koivisto 2016-11-11 14:35:44 PST
Created attachment 294539 [details]
patch for landing
Comment 9 WebKit Commit Bot 2016-11-11 15:11:40 PST
Comment on attachment 294539 [details]
patch for landing

Clearing flags on attachment: 294539

Committed r208616: <http://trac.webkit.org/changeset/208616>
Comment 10 WebKit Commit Bot 2016-11-11 15:11:46 PST
All reviewed patches have been landed.  Closing bug.