79595 2012-02-25 20:58:17 -0800 Setting style="" should destroy the element's inline style. 2022-07-26 10:04:18 -0700 1 1 1 Unclassified WebKit DOM 528+ (Nightly build) Unspecified Unspecified RESOLVED CONFIGURATION CHANGED InRadar P2 Normal --- 79597 1 kling kling ahmad.saleem792 ap bfulgham ossy rniwa webkit-bug-importer oldest_to_newest 565090 0 kling 2012-02-25 20:58:17 -0800 No need to hang an inline style object off of an element with an empty style attribute. 565091 1 128901 kling 2012-02-25 21:00:47 -0800 Created attachment 128901 Pschtat 565092 2 128901 kling 2012-02-25 21:08:34 -0800 Comment on attachment 128901 Pschtat Clearing flags on attachment: 128901 Committed r108924: <http://trac.webkit.org/changeset/108924> 565093 3 kling 2012-02-25 21:08:42 -0800 All reviewed patches have been landed. Closing bug. 565106 4 ossy 2012-02-25 22:23:31 -0800 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 1886800 5 ahmad.saleem792 2022-07-26 09:58:29 -0700 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! 1886801 6 rniwa 2022-07-26 10:03:15 -0700 Indeed, this code has changed substantially. No need to keep this bug open at this point. 1886803 7 webkit-bug-importer 2022-07-26 10:04:18 -0700 <rdar://problem/97612300> 128901 2012-02-25 21:00:47 -0800 2012-02-25 21:08:34 -0800 Pschtat bug-79595.diff text/plain 1496 kling ZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZyBiL1NvdXJjZS9XZWJDb3JlL0No YW5nZUxvZwppbmRleCAyZTMyYThiLi41MDNlOTRjIDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViQ29y ZS9DaGFuZ2VMb2cKKysrIGIvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCkBAIC0xLDUgKzEsMTkg QEAKIDIwMTItMDItMjUgIEFuZHJlYXMgS2xpbmcgIDxhd2Vzb21la2xpbmdAYXBwbGUuY29tPgog CisgICAgICAgIFNldHRpbmcgc3R5bGU9IiIgc2hvdWxkIGRlc3Ryb3kgdGhlIGVsZW1lbnQncyBp bmxpbmUgc3R5bGUuCisgICAgICAgIDxodHRwOi8vd2Via2l0Lm9yZy9iLzc5NTk1PgorCisgICAg ICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgIFRoZXJlJ3Mgbm8gcmVh c29uIGZvciBhbiBlbGVtZW50IHdpdGggc3R5bGU9IiIgdG8gaGF2ZSBhbiBpbmxpbmUgc3R5bGUg b2JqZWN0LgorICAgICAgICBSZW1vdmUgdGhlIGlubGluZSBzdHlsZSBpbiB0aGF0IGNhc2UsIGp1 c3QgbGlrZSB3ZSBkbyB3aGVuIHJlbW92aW5nIHRoZSBzdHlsZQorICAgICAgICBhdHRyaWJ1dGUg YWx0b2dldGhlci4KKworICAgICAgICAqIGRvbS9TdHlsZWRFbGVtZW50LmNwcDoKKyAgICAgICAg KFdlYkNvcmU6OlN0eWxlZEVsZW1lbnQ6OnBhcnNlQXR0cmlidXRlKToKKworMjAxMi0wMi0yNSAg QW5kcmVhcyBLbGluZyAgPGF3ZXNvbWVrbGluZ0BhcHBsZS5jb20+CisKICAgICAgICAgUmVtb3Zl IEhUTUxFbWJlZEVsZW1lbnQ6Omluc2VydGVkSW50b0RvY3VtZW50KCkuCiAgICAgICAgIDxodHRw Oi8vd2Via2l0Lm9yZy9iLzc5NTc2PgogCmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9kb20v U3R5bGVkRWxlbWVudC5jcHAgYi9Tb3VyY2UvV2ViQ29yZS9kb20vU3R5bGVkRWxlbWVudC5jcHAK aW5kZXggOTllN2YzNS4uYTg1MmE2MSAxMDA2NDQKLS0tIGEvU291cmNlL1dlYkNvcmUvZG9tL1N0 eWxlZEVsZW1lbnQuY3BwCisrKyBiL1NvdXJjZS9XZWJDb3JlL2RvbS9TdHlsZWRFbGVtZW50LmNw cApAQCAtOTksNyArOTksNyBAQCB2b2lkIFN0eWxlZEVsZW1lbnQ6OnBhcnNlQXR0cmlidXRlKEF0 dHJpYnV0ZSogYXR0cikKICAgICBpZiAoYXR0ci0+bmFtZSgpID09IGNsYXNzQXR0cikKICAgICAg ICAgY2xhc3NBdHRyaWJ1dGVDaGFuZ2VkKGF0dHItPnZhbHVlKCkpOwogICAgIGVsc2UgaWYgKGF0 dHItPm5hbWUoKSA9PSBzdHlsZUF0dHIpIHsKLSAgICAgICAgaWYgKGF0dHItPmlzTnVsbCgpKQor ICAgICAgICBpZiAoYXR0ci0+aXNFbXB0eSgpKQogICAgICAgICAgICAgZGVzdHJveUlubGluZVN0 eWxlRGVjbCgpOwogICAgICAgICBlbHNlIGlmIChkb2N1bWVudCgpLT5jb250ZW50U2VjdXJpdHlQ b2xpY3koKS0+YWxsb3dJbmxpbmVTdHlsZSgpKQogICAgICAgICAgICAgZW5zdXJlSW5saW5lU3R5 bGVEZWNsKCktPnBhcnNlRGVjbGFyYXRpb24oYXR0ci0+dmFsdWUoKSwgZG9jdW1lbnQoKS0+ZWxl bWVudFNoZWV0KCkpOwo=