Bug 96978 - Disable adding an AuthorShadowRoot to replaced elements.
Summary: Disable adding an AuthorShadowRoot to replaced elements.
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: DOM (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Shinya Kawanaka
URL:
Keywords:
: 95711 (view as bug list)
Depends on:
Blocks: 82313
  Show dependency treegraph
 
Reported: 2012-09-17 22:04 PDT by Shinya Kawanaka
Modified: 2012-11-29 09:28 PST (History)
4 users (show)

See Also:


Attachments
Patch (32.24 KB, patch)
2012-09-17 23:02 PDT, Shinya Kawanaka
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Shinya Kawanaka 2012-09-17 22:04:02 PDT
Though we have proceeded supporting ShadowDOM for replaced elements, we have concluded that we don't implement the current spec for now, since it has a lot of difficulties to make it understandable.

So we would like to reject adding an AuthroShadowRoot to those elements now. After we reached another conclusion, we will come back to support AuthorShadowDOM for replaced elements.
Comment 1 Shinya Kawanaka 2012-09-17 22:10:29 PDT
img, input, textarea, iframe, canvas, meter, progress, select, 
object, embed, keygen, video, audio, fieldset

I think <button> is OK if AuthorShadowRoot is allowed now.
Comment 2 Shinya Kawanaka 2012-09-17 22:21:59 PDT
Not that supporting AuthorShadowRoot in some of replaced elements is disabled in https://bugs.webkit.org/show_bug.cgi?id=96404
Comment 3 Shinya Kawanaka 2012-09-17 23:02:29 PDT
Created attachment 164495 [details]
Patch
Comment 4 Hajime Morrita 2012-09-17 23:22:56 PDT
Comment on attachment 164495 [details]
Patch

Good catch!
Comment 5 WebKit Review Bot 2012-09-18 00:28:01 PDT
Comment on attachment 164495 [details]
Patch

Clearing flags on attachment: 164495

Committed r128856: <http://trac.webkit.org/changeset/128856>
Comment 6 WebKit Review Bot 2012-09-18 00:28:04 PDT
All reviewed patches have been landed.  Closing bug.
Comment 7 Abhishek Arya 2012-11-29 09:28:49 PST
*** Bug 95711 has been marked as a duplicate of this bug. ***