Bug 79595 - Setting style="" should destroy the element's inline style.
Summary: Setting style="" should destroy the element's inline style.
Status: RESOLVED CONFIGURATION CHANGED
Alias: None
Product: WebKit
Classification: Unclassified
Component: DOM (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Andreas Kling
URL:
Keywords: InRadar
Depends on: 79597
Blocks:
  Show dependency tree / graph
 
Reported: 2012-02-25 20:58 PST by Andreas Kling
Modified: 2022-07-26 10:04 PDT (History)
6 users (show)

See Also:

Attachments
Pschtat (1.46 KB, patch)
2012-02-25 21:00 PST, Andreas Kling 		no flags Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Andreas Kling 2012-02-25 20:58:17 PST 
No need to hang an inline style object off of an element with an empty style attribute.
Comment 1 Andreas Kling 2012-02-25 21:00:47 PST 
Created attachment 128901 [details]
Pschtat
Comment 2 Andreas Kling 2012-02-25 21:08:34 PST 
Comment on attachment 128901 [details]
Pschtat

Clearing flags on attachment: 128901

Committed r108924: <http://trac.webkit.org/changeset/108924>
Comment 3 Andreas Kling 2012-02-25 21:08:42 PST 
All reviewed patches have been landed.  Closing bug.
Comment 4 Csaba Osztrogonác 2012-02-25 22:23:31 PST 
Reopen, because it broke 4 inspector tests: (at least on Chromium and Qt)
http://build.webkit.org/results/Chromium%20Win%20Release%20%28Tests%29/r108924%20%2823955%29/results.html
Comment 5 Ahmad Saleem 2022-07-26 09:58:29 PDT 
I think the code is changed significantly and I only see following reference:

https://github.com/WebKit/WebKit/blob/aff87f6cbcf56b5fe448770a00f51814fa626b7c/Source/WebCore/dom/Element.cpp#L2164

https://github.com/WebKit/WebKit/blob/aff87f6cbcf56b5fe448770a00f51814fa626b7c/Source/WebCore/dom/Element.cpp#L1964

While searching for "classAttributeChanged". I think this can be closed now as "RESOLVED CONFIGURATION CHANGED", ignore my comment if I am wrong. There was also no test in the attached patch, which I can test. Thanks!
Comment 6 Ryosuke Niwa 2022-07-26 10:03:15 PDT 
Indeed, this code has changed substantially. No need to keep this bug open at this point.
Comment 7 Radar WebKit Bug Importer 2022-07-26 10:04:18 PDT 
<rdar://problem/97612300>