1518512 - Track Changes - Changes panel misses context menu

Status: RESOLVED FIXED

(DevTools :: Inspector, defect, P3)

Description

[Martin Balfanz [:mbalfanz]]

Until exporting is implemented, the changes panel should offer the normal context menu provided by the operating system which options to copy/paste/select all...

STR:

• track some changes
• highlight the collected changes
• click the right mouse button on the selection

ER:

• the context menu should open, allowing to copy the text

AR:

• context menu doesn't open

Comment 1

[Razvan Caliman [:rcaliman]]

Attachment: Bug 1518512 - (Part 1) Add basic context menu to Changes panel. r=gl

Adds context menu with options to select all and copy text content from
the Changes panel.

Comment 2

[Razvan Caliman [:rcaliman]]

Attachment: Bug 1518512 - (Part 2) Ensure all text in Changes panel can be copied. r=gl

Depends on D17255
Replaces CSS pseudo-elements for added/removed line markers with text
elements that can be copied with the rest of the content.

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/3c03e2282b4f
(Part 1) Add basic context menu to Changes panel. r=gl
https://hg.mozilla.org/integration/autoland/rev/c8f0d19844f6
(Part 2) Ensure all text in Changes panel can be copied. r=gl

Comment 4

[Natalia Csoregi [:nataliaCs]]

Backed out for devtools/client/inspector/changes/test/browser_changes_rule_selector.js failures.

backout: https://hg.mozilla.org/integration/autoland/rev/a7918210d2f16d02f02f7a322375a358fe710e3a

push with failures: https://treeherder.mozilla.org/#/jobs?repo=autoland&revision=c8f0d19844f67cd9a2189f612e4f1a865d697b04&searchStr=devtools&group_state=expanded

failure log: https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=224014389&repo=autoland&lineNumber=4175

[task 2019-01-25T13:07:53.009Z] 13:07:53 INFO - TEST-PASS | devtools/client/inspector/changes/test/browser_changes_rule_selector.js | First rule has only one declaration -
[task 2019-01-25T13:07:53.013Z] 13:07:53 INFO - Buffered messages finished
[task 2019-01-25T13:07:53.015Z] 13:07:53 INFO - TEST-UNEXPECTED-FAIL | devtools/client/inspector/changes/test/browser_changes_rule_selector.js | First rule has correct declaration - Got - color: red;, expected color:red;
[task 2019-01-25T13:07:53.017Z] 13:07:53 INFO - Stack trace:
[task 2019-01-25T13:07:53.019Z] 13:07:53 INFO - chrome://mochikit/content/browser-test.js:test_is:1318
[task 2019-01-25T13:07:53.020Z] 13:07:53 INFO - chrome://mochitests/content/browser/devtools/client/inspector/changes/test/browser_changes_rule_selector.js:null:57
[task 2019-01-25T13:07:53.022Z] 13:07:53 INFO - chrome://mochikit/content/browser-test.js:Tester_execTest/<:1108
[task 2019-01-25T13:07:53.024Z] 13:07:53 INFO - chrome://mochikit/content/browser-test.js:Tester_execTest:1099
[task 2019-01-25T13:07:53.031Z] 13:07:53 INFO - chrome://mochikit/content/browser-test.js:nextTest/<:997
[task 2019-01-25T13:07:53.033Z] 13:07:53 INFO - chrome://mochikit/content/tests/SimpleTest/SimpleTest.js:SimpleTest.waitForFocus/waitForFocusInner/focusedOrLoaded/<:803
[task 2019-01-25T13:07:53.035Z] 13:07:53 INFO - TEST-PASS | devtools/client/inspector/changes/test/browser_changes_rule_selector.js | First rule has declaration tracked as removed -
[task 2019-01-25T13:07:53.036Z] 13:07:53 INFO - TEST-PASS | devtools/client/inspector/changes/test/browser_changes_rule_selector.js | Second rule has only one declaration -
[task 2019-01-25T13:07:53.038Z] 13:07:53 INFO - Not taking screenshot here: see the one that was previously logged
[task 2019-01-25T13:07:53.039Z] 13:07:53 INFO - TEST-UNEXPECTED-FAIL | devtools/client/inspector/changes/test/browser_changes_rule_selector.js | Second rule has correct declaration - Got + color: red;, expected color:red;
[task 2019-01-25T13:07:53.041Z] 13:07:53 INFO - Stack trace:
[task 2019-01-25T13:07:53.043Z] 13:07:53 INFO - chrome://mochikit/content/browser-test.js:test_is:1318
[task 2019-01-25T13:07:53.044Z] 13:07:53 INFO - chrome://mochitests/content/browser/devtools/client/inspector/changes/test/browser_changes_rule_selector.js:null:63
[task 2019-01-25T13:07:53.045Z] 13:07:53 INFO - chrome://mochikit/content/browser-test.js:Tester_execTest/<:1108
[task 2019-01-25T13:07:53.047Z] 13:07:53 INFO - chrome://mochikit/content/browser-test.js:Tester_execTest:1099
[task 2019-01-25T13:07:53.049Z] 13:07:53 INFO - chrome://mochikit/content/browser-test.js:nextTest/<:997
[task 2019-01-25T13:07:53.050Z] 13:07:53 INFO - chrome://mochikit/content/tests/SimpleTest/SimpleTest.js:SimpleTest.waitForFocus/waitForFocusInner/focusedOrLoaded/<:803
[task 2019-01-25T13:07:53.051Z] 13:07:53 INFO - TEST-PASS | devtools/client/inspector/changes/test/browser_changes_rule_selector.js | Second rule has declaration tracked as added -

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/21af538eb319
(Part 1) Add basic context menu to Changes panel. r=gl
https://hg.mozilla.org/integration/autoland/rev/99655666ac5a
(Part 2) Ensure all text in Changes panel can be copied. r=gl

Comment 6

[Razvan Caliman [:rcaliman]]

Fixed failing test and queued again for landing

Comment 7

[Cristina Coroiu [:ccoroiu]]

https://hg.mozilla.org/mozilla-central/rev/21af538eb319
https://hg.mozilla.org/mozilla-central/rev/99655666ac5a