1301318 - Don't copy invisible portion of logs in Browser/Web Console when selecting 'copy' from context menu

Status: NEW

(DevTools :: Console, defect, P3)

Description

[magicp]

Attachment: dont-copy-invisible-logs-in-console.png

User Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:51.0) Gecko/20100101 Firefox/51.0
Build ID: 20160907030427

Steps to reproduce:

1. Start Nightly
2. Open Browser Console (Ctrl+Shift+J)
3. Enable JS Warning
4. Open DevTools > Inspector   ..... Two warnings will be displayed in Console

CASE 1 (Copy an error and paste)
- Right-click on the first row (* Bug 1158474: First right-click is not enabling the Copy-command)
- Select “Copy” in context menus
- Paste to notepad

CASE 2 (Copy all errors and paste)
- Right-click on any rows
- Select “Select All” in context menus
- Select “Copy” in context menus
- Paste to notepad


Actual results:

[CASE 1] ... There are unnecessary invisible logs
15:32:38.211 Use of nsIFile in content process is deprecated. 1 NetUtil.jsm:350:12
^^^^^^^^                                                                 ^^

[CASE 2] ... Good: Expected results
Use of nsIFile in content process is deprecated.  NetUtil.jsm:350:12
Use of nsIFile in content process is deprecated.  filesystem.js:40:9



Expected results:

In CASE 1, Remove unnecessary invisible logs. Same with CASE 2.

Comment 1

[Brian Grinstead [:bgrins]]

Good catch, it seems like the 'copy' context menu is doing some extra stuff when there's no selected text (which is causing extra info like timestamp to be included).  I'm not sure if this was intentional originally but we should just copy what is visible in the UI.