Bug 147767 – Web Inspector: Don't include zero-width space into a copied text from the console

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

[Nikita Vasilyev]

Created attachment 258439 [details]
[Animated GIF] Bug

Steps:
1.
    var img = new Image;
    img.src = "http://33.media.tumblr.com/tumblr_m6t5rzAfVU1qc0s10o1_500.gif";
    img;
2. Click on the logged console message to select it
3. Press Command-C to copy it

Expected clipboard content: 
<img src="http://33.media.tumblr.com/tumblr_m6t5rzAfVU1qc0s10o1_500.gif">

Actual clipboard content:
<img src=​"http:​/​/​33.media.tumblr.com/​tumblr_m6t5rzAfVU1qc0s10o1_500.gif">​
 = $1

1. There are a few zero-width spaces (\u200B) in the copied message; they are marked as red dots if you paste this message to the console prompt (see the attached image).

2. " = $1" shouldn't be copied.

[Timothy Hatcher]

WebInspector._copy should already be doing this, I guess it isn't being called?

    // Remove word break characters from the selection before putting it on the pasteboard.
    var selectionString = selection.toString().removeWordBreakCharacters();
    event.clipboardData.setData("text/plain", selectionString);
    event.preventDefault();

[Devin Rousso]

Created attachment 258553 [details]
Patch

[Timothy Hatcher]

Comment on attachment 258553 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=258553&action=review

> Source/WebInspectorUI/UserInterface/Views/ConsoleMessageView.js:190
> +            clipboardString = clipboardString.replace(/(\s*=\s*\$\d+)$/, "");

No need for the () in the regex.

[Devin Rousso]

Should this also remove console information?  For example, entering

console.log("test");

If I copy the resulting line (which should just say "text") and paste it, I get

[Log] test

Should that [Log] be there?  I have also seen source location information at the end for functions.

[Timothy Hatcher]

[Log[ is added on purpose. The location is also added. I think it makes sense to strip out the = $n parts.

[Devin Rousso]

Created attachment 258554 [details]
Patch

[Timothy Hatcher]

Comment on attachment 258554 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=258554&action=review

> Source/WebInspectorUI/UserInterface/Views/ConsoleMessageView.js:190
> +        if (this._message.savedResultIndex)
> +            clipboardString = clipboardString.replace(/\s*=\s*\$\d+$/, "");

Wait, never mind. I think $n should stay. If you use $n in a future line (like $2.foo.bar), you want to be able to look back and see where it came from. So remote this stripping.

[Devin Rousso]

Created attachment 258555 [details]
Patch

[Devin Rousso]

Comment on attachment 258555 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=258555&action=review

> Source/WebInspectorUI/UserInterface/Views/ConsoleMessageView.js:190
> +            clipboardString = clipboardString.replace(/\s*=\s*(\$\d+)$/, " = $1");

By the way, I added this so that, as seen in Nikita's example, the " = $1" doesn't wrap to the next line.

[Timothy Hatcher]

Comment on attachment 258555 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=258555&action=review

>> Source/WebInspectorUI/UserInterface/Views/ConsoleMessageView.js:190
>> +            clipboardString = clipboardString.replace(/\s*=\s*(\$\d+)$/, " = $1");
> 
> By the way, I added this so that, as seen in Nikita's example, the " = $1" doesn't wrap to the next line.

Oh, I see. My bad.

[Timothy Hatcher]

Comment on attachment 258555 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=258555&action=review

>>> Source/WebInspectorUI/UserInterface/Views/ConsoleMessageView.js:190
>>> +            clipboardString = clipboardString.replace(/\s*=\s*(\$\d+)$/, " = $1");
>> 
>> By the way, I added this so that, as seen in Nikita's example, the " = $1" doesn't wrap to the next line.
> 
> Oh, I see. My bad.

Oh, you were stripping before. Now you just strip linebreaks there. Nice!

[WebKit Commit Bot]

Comment on attachment 258555 [details]
Patch

Clearing flags on attachment: 258555

Committed r188184: <http://trac.webkit.org/changeset/188184>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.