Bug 240455

Summary: "box-shadow" does not work on native form controls
Product: WebKit Reporter: Chijin <tlock.chijin>
Component: FormsAssignee: Nobody <webkit-unassigned>
Status: NEW    
Severity: Critical CC: 2843035794, ahmad.saleem792, akeerthi, ap, bfulgham, cdumez, ntim, simon.fraser, webkit-bug-importer, wenson_hsieh, zalan
Priority: P2 Keywords: BrowserCompat, InRadar
Version: Safari 15   
Hardware: All   
OS: All   
See Also: https://bugs.webkit.org/show_bug.cgi?id=240638
https://bugs.webkit.org/show_bug.cgi?id=240706
Attachments:
Description Flags
webkit screenshot
none
chrome screenshot
none
firefox screenshot none

Chijin
Reported 2022-05-16 07:04:25 PDT
Created attachment 459420 [details] webkit screenshot reprodiciable example: https://codepen.io/chijinz/pen/vYdgpdY code: ``` <html> <head> <style> </style> <script> function jsfuzzer() { document.getElementById("htmlvar00004").style["box-shadow"] = "red 1px 73px" } </script> </head> <body onload=jsfuzzer()> <textarea id="htmlvar00004" >aaa</textarea> </body> </html> ``` version: all webkit products (including Safari) description: The textarea element does not have shadow even though the "box-shadow" is set. Chrome and Firefox work fine with it.
Attachments
webkit screenshot (84.62 KB, image/png)
2022-05-16 07:04 PDT, Chijin 		no flags
Details
chrome screenshot (193.46 KB, image/png)
2022-05-16 07:06 PDT, Chijin 		no flags
Details
firefox screenshot (90.65 KB, image/png)
2022-05-16 07:06 PDT, Chijin 		no flags
Details
View All Add attachment proposed patch, testcase, etc.
Chijin
Comment 1 2022-05-16 07:06:41 PDT
Created attachment 459421 [details] chrome screenshot
Chijin
Comment 2 2022-05-16 07:06:55 PDT
Created attachment 459422 [details] firefox screenshot
Radar WebKit Bug Importer
Comment 3 2022-05-16 09:39:38 PDT
<rdar://problem/93351344>
Tim Nguyen (:ntim)
Comment 4 2022-05-16 11:23:34 PDT
Seems intentional: https://webkit-search.igalia.com/webkit/rev/7bebd4483c6744ddf9823572d42d2e34dc9d0bd9/Source/WebCore/rendering/RenderTheme.cpp#174 though I think this behavior should be changed.
Chijin
Comment 5 2022-05-16 18:19:11 PDT
(In reply to Tim Nguyen (:ntim) from comment #4) > Seems intentional: > https://webkit-search.igalia.com/webkit/rev/ > 7bebd4483c6744ddf9823572d42d2e34dc9d0bd9/Source/WebCore/rendering/ > RenderTheme.cpp#174 > > though I think this behavior should be changed. Thanks. Could you backtrace the reason for doing this? Is it for the security concern or just wrong implementation?
Tim Nguyen (:ntim)
Comment 6 2022-05-16 21:25:36 PDT
(In reply to Chijin from comment #5) > (In reply to Tim Nguyen (:ntim) from comment #4) > > Seems intentional: > > https://webkit-search.igalia.com/webkit/rev/ > > 7bebd4483c6744ddf9823572d42d2e34dc9d0bd9/Source/WebCore/rendering/ > > RenderTheme.cpp#174 > > > > though I think this behavior should be changed. > > Thanks. Could you backtrace the reason for doing this? Is it for the > security concern or just wrong implementation? Probably some old UI guideline saying native controls shouldn't get shadows. Anyway, this should be fixed to be in-line with other browsers.
Aditya Keerthi
Comment 7 2022-05-19 15:31:43 PDT
*** Bug 240635 has been marked as a duplicate of this bug. ***
Fujii Hironori
Comment 8 2022-06-29 21:29:13 PDT
*** Bug 242162 has been marked as a duplicate of this bug. ***
Note You need to log in before you can comment on or make changes to this bug.