Bug 199588 – Web Inspector: REGRESSION: Elements: pasting in the Styles sidebar adds a text node to the DOM tree

• Status: RESOLVED
• Resolution: FIXED
• Component: Web Inspector

[Devin Rousso]

# STEPS TO REPRODUCE:
1. copy "foo"
2. inspect any page
3. select the inline style of any node in the Elements tab
4. start adding a new property in the inline style
5. paste
 => "foo" added as a sibling of the currently selected node in the DOM tree

[Devin Rousso]

(In reply to Devin Rousso from comment #0)
> 1. copy "foo"
Actually, you need to copy something that would instead be rendered as a CSS property (or multiple): "foo:".

[Devin Rousso]

Created attachment 373664 [details]
Patch

[Joseph Pecoraro]

Comment on attachment 373664 [details]
Patch

rs=me

Should this be the case for other events then? WI._copy?

[Devin Rousso]

Comment on attachment 373664 [details]
Patch

(In reply to Joseph Pecoraro from comment #3)
> Should this be the case for other events then? WI._copy?
I'd say yes, but in the interest of avoiding regressions, I'd rather not.  Also, the default functionality of a "copy" won't modify the DOM like "paste" would, so there's less "risk" there of multiple things acting on the same "copy".

[WebKit Commit Bot]

Comment on attachment 373664 [details]
Patch

Clearing flags on attachment: 373664

Committed r247241: <https://trac.webkit.org/changeset/247241>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.

[Radar WebKit Bug Importer]

<rdar://problem/52806667>