Summary: | Web Inspector: Support user attributes in DOMNode | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Alexander Pavlov (apavlov) <apavlov> | ||||
Component: | Web Inspector (Deprecated) | Assignee: | Alexander Pavlov (apavlov) <apavlov> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | apavlov, bweinstein, joepeck, keishi, loislo, pfeldman, pmuellr, rik, timothy, yurys | ||||
Priority: | P2 | ||||||
Version: | 528+ (Nightly build) | ||||||
Hardware: | All | ||||||
OS: | All | ||||||
Bug Depends on: | |||||||
Bug Blocks: | 86630 | ||||||
Attachments: |
|
Description
Alexander Pavlov (apavlov)
2012-06-09 04:25:21 PDT
Created attachment 147307 [details]
Patch
Comment on attachment 147307 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=147307&action=review > Source/WebCore/inspector/front-end/DOMAgent.js:707 > + setUserAttribute: function(name, value) setUserProperty ? > Source/WebCore/inspector/front-end/DOMAgent.js:712 > + var didHaveAttribute = this._userAttributes[name] !== null && this._userAttributes[name] !== undefined && this._userAttributes[name] !== false; Lets clear on "null" only > Source/WebCore/inspector/front-end/DOMAgent.js:718 > + if (nowHaveAttribute) This looks weird, I would extract "remove" method > Source/WebCore/inspector/front-end/DOMAgent.js:731 > + return this._descendantUserAttributeCounters && this._descendantUserAttributeCounters[name] ? this._descendantUserAttributeCounters[name] : 0; I think this._descendantUserAttributeCounters can exist at all times. Committed r120208: <http://trac.webkit.org/changeset/120208> |