49663 2010-11-17 07:13:01 -0800 Web Inspector: name-only property appears in style declaration but not in the Styles pane 2010-12-01 06:14:57 -0800 1 1 1 Unclassified WebKit Web Inspector (Deprecated) 528+ (Nightly build) All All RESOLVED FIXED P2 Normal --- 1 apavlov apavlov apavlov bweinstein joepeck keishi loislo pfeldman pmuellr rik timothy yurys oldest_to_newest 310265 0 apavlov 2010-11-17 07:13:01 -0800 This is observed with the latest migration of the CSS editing framework to the source-based approach. To reproduce: 1. In an "element.style" style, add an invalid name-only property, e.g. "baz". 2. The "baz;" property is appended to the "style" property value but does not appear in the "element.style" style declaration. 315610 1 75268 apavlov 2010-12-01 05:55:12 -0800 Created attachment 75268 [PATCH] Suggested fix 315614 2 apavlov 2010-12-01 06:14:57 -0800 Committing to http://svn.webkit.org/repository/webkit/trunk ... M LayoutTests/ChangeLog M LayoutTests/inspector/styles-new-API.html M WebCore/ChangeLog M WebCore/inspector/InspectorStyleSheet.cpp Committed r73018 75268 2010-12-01 05:55:12 -0800 2010-12-01 06:01:25 -0800 [PATCH] Suggested fix checkprop.patch text/plain 3086 apavlov Y29tbWl0IDFhODEwZGY3NjJkODE5NGE0MzhjZGYwY2VhYjQ4Njg5M2Y4NjY1MjAKQXV0aG9yOiBB bGV4YW5kZXIgUGF2bG92IDxhcGF2bG92QGNocm9taXVtLm9yZz4KRGF0ZTogICBXZWQgRGVjIDEg MTY6MzM6MzkgMjAxMCArMDMwMAoKICAgIEZpeEJhZFByb3BzCgpkaWZmIC0tZ2l0IGEvTGF5b3V0 VGVzdHMvQ2hhbmdlTG9nIGIvTGF5b3V0VGVzdHMvQ2hhbmdlTG9nCmluZGV4IGQ4N2ZjNGIuLmNi MDBmY2IgMTAwNjQ0Ci0tLSBhL0xheW91dFRlc3RzL0NoYW5nZUxvZworKysgYi9MYXlvdXRUZXN0 cy9DaGFuZ2VMb2cKQEAgLTEsMyArMSwxMiBAQAorMjAxMC0xMi0wMSAgQWxleGFuZGVyIFBhdmxv diAgPGFwYXZsb3ZAY2hyb21pdW0ub3JnPgorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAo T09QUyEpLgorCisgICAgICAgIFdlYiBJbnNwZWN0b3I6IG5hbWUtb25seSBwcm9wZXJ0eSBhcHBl YXJzIGluIHN0eWxlIGRlY2xhcmF0aW9uIGJ1dCBub3QgaW4gdGhlIFN0eWxlcyBwYW5lCisgICAg ICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD00OTY2MworCisgICAg ICAgICogaW5zcGVjdG9yL3N0eWxlcy1uZXctQVBJLmh0bWw6CisKIDIwMTAtMTItMDEgIENzYWJh IE9zenRyb2dvbsOhYyAgPG9zc3lAd2Via2l0Lm9yZz4KIAogICAgICAgICBVbnJldmlld2VkIHRy aXZpYWwgZml4LgpkaWZmIC0tZ2l0IGEvTGF5b3V0VGVzdHMvaW5zcGVjdG9yL3N0eWxlcy1uZXct QVBJLmh0bWwgYi9MYXlvdXRUZXN0cy9pbnNwZWN0b3Ivc3R5bGVzLW5ldy1BUEkuaHRtbAppbmRl eCBiZjVkMGIzLi5mZGExNmE4IDEwMDY0NAotLS0gYS9MYXlvdXRUZXN0cy9pbnNwZWN0b3Ivc3R5 bGVzLW5ldy1BUEkuaHRtbAorKysgYi9MYXlvdXRUZXN0cy9pbnNwZWN0b3Ivc3R5bGVzLW5ldy1B UEkuaHRtbApAQCAtMTI5LDYgKzEyOSw5IEBAIGZ1bmN0aW9uIGZyb250ZW5kX3NldFN0eWxlVGV4 dCh0ZXN0Q29udHJvbGxlciwgc3R5bGVTaGVldCkKIAogICAgIEluc3BlY3RvckJhY2tlbmQuc2V0 UHJvcGVydHlUZXh0MihzdHlsZVNoZWV0LnJ1bGVzWzBdLnN0eWxlLnN0eWxlSWQsIDAsICIiLCB0 cnVlKTsKICAgICBJbnNwZWN0b3JCYWNrZW5kLnNldFByb3BlcnR5VGV4dDIoc3R5bGVTaGVldC5y dWxlc1swXS5zdHlsZS5zdHlsZUlkLCAwLCAiIiwgdHJ1ZSk7CisKKyAgICAvLyBUaGlzIG9wZXJh dGlvbiBzaG91bGQgbm90IHVwZGF0ZSB0aGUgc3R5bGUgYXMgdGhlIG5ldyBwcm9wZXJ0eSB0ZXh0 IGlzIG5vdCBwYXJzYWJsZS4KKyAgICBJbnNwZWN0b3JCYWNrZW5kLnNldFByb3BlcnR5VGV4dDIo c3R5bGVTaGVldC5ydWxlc1swXS5zdHlsZS5zdHlsZUlkLCAwLCAienp6OyIsIGZhbHNlKTsKICAg ICBJbnNwZWN0b3JCYWNrZW5kLnNldFByb3BlcnR5VGV4dDIoc3R5bGVTaGVldC5ydWxlc1swXS5z dHlsZS5zdHlsZUlkLCAwLCAiY29sb3I6IHdoaXRlOyBiYWNrZ3JvdW5kOiBibGFjazsiLCBmYWxz ZSwgZGlkU2V0U3R5bGVUZXh0LmJpbmQobnVsbCwgdGVzdENvbnRyb2xsZXIpKTsKIH0KIApkaWZm IC0tZ2l0IGEvV2ViQ29yZS9DaGFuZ2VMb2cgYi9XZWJDb3JlL0NoYW5nZUxvZwppbmRleCBhNmI1 OGU5Li45OTc1ZmFiIDEwMDY0NAotLS0gYS9XZWJDb3JlL0NoYW5nZUxvZworKysgYi9XZWJDb3Jl L0NoYW5nZUxvZwpAQCAtMSwzICsxLDE1IEBACisyMDEwLTEyLTAxICBBbGV4YW5kZXIgUGF2bG92 ICA8YXBhdmxvdkBjaHJvbWl1bS5vcmc+CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChP T1BTISkuCisKKyAgICAgICAgV2ViIEluc3BlY3RvcjogbmFtZS1vbmx5IHByb3BlcnR5IGFwcGVh cnMgaW4gc3R5bGUgZGVjbGFyYXRpb24gYnV0IG5vdCBpbiB0aGUgU3R5bGVzIHBhbmUKKyAgICAg ICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTQ5NjYzCisKKyAgICAg ICAgQ2hlY2sgdGhhdCBhdCBsZWFzdCBvbmUgcHJvcGVydHkgaGFzIGJlZW4gcGFyc2VkIGZyb20g dGhlIHVzZXIgaW5wdXQuCisKKyAgICAgICAgKiBpbnNwZWN0b3IvSW5zcGVjdG9yU3R5bGVTaGVl dC5jcHA6CisgICAgICAgIChXZWJDb3JlOjpJbnNwZWN0b3JTdHlsZTo6c2V0UHJvcGVydHlUZXh0 KToKKwogMjAxMC0xMi0wMSAgTmlrb2xhcyBaaW1tZXJtYW5uICA8bnppbW1lcm1hbm5AcmltLmNv bT4KIAogICAgICAgICBSZXZpZXdlZCBieSBEaXJrIFNjaHVsemUuCmRpZmYgLS1naXQgYS9XZWJD b3JlL2luc3BlY3Rvci9JbnNwZWN0b3JTdHlsZVNoZWV0LmNwcCBiL1dlYkNvcmUvaW5zcGVjdG9y L0luc3BlY3RvclN0eWxlU2hlZXQuY3BwCmluZGV4IDViNjdhMjYuLmViMTUzOGYgMTAwNjQ0Ci0t LSBhL1dlYkNvcmUvaW5zcGVjdG9yL0luc3BlY3RvclN0eWxlU2hlZXQuY3BwCisrKyBiL1dlYkNv cmUvaW5zcGVjdG9yL0luc3BlY3RvclN0eWxlU2hlZXQuY3BwCkBAIC0xNzgsNyArMTc4LDkgQEAg Ym9vbCBJbnNwZWN0b3JTdHlsZTo6c2V0UHJvcGVydHlUZXh0KHVuc2lnbmVkIGluZGV4LCBjb25z dCBTdHJpbmcmIHByb3BlcnR5VGV4dCwKICAgICAgICAgcC5wYXJzZURlY2xhcmF0aW9uKHRlbXBN dXRhYmxlU3R5bGUuZ2V0KCksIHByb3BlcnR5VGV4dCArICIgLXdlYmtpdC1ib2d1ei1wcm9wZXJ0 ZWU6IG5vbmUiLCAmc291cmNlRGF0YSk7CiAgICAgICAgIFZlY3RvcjxDU1NQcm9wZXJ0eVNvdXJj ZURhdGE+JiBwcm9wZXJ0eURhdGEgPSBzb3VyY2VEYXRhLT5wcm9wZXJ0eURhdGE7CiAgICAgICAg IHVuc2lnbmVkIHByb3BlcnR5Q291bnQgPSBwcm9wZXJ0eURhdGEuc2l6ZSgpOwotICAgICAgICBp ZiAoIXByb3BlcnR5Q291bnQpCisKKyAgICAgICAgLy8gQXQgbGVhc3Qgb25lIHByb3BlcnR5ICsg dGhlIGJvZ3VzIHByb3BlcnR5IGFkZGVkIGp1c3QgYWJvdmUgc2hvdWxkIGJlIHByZXNlbnQuCisg ICAgICAgIGlmIChwcm9wZXJ0eUNvdW50IDwgMikKICAgICAgICAgICAgIHJldHVybiBmYWxzZTsK IAogICAgICAgICAvLyBDaGVjayBmb3IgYSBwcm9wZXIgcHJvcGVydHlUZXh0IHRlcm1pbmF0aW9u ICh0aGUgcGFyc2VyIGNvdWxkIGF0IGxlYXN0IHJlc3RvcmUgdG8gdGhlIFBST1BFUlRZX05BTUUg c3RhdGUpLgo=