Closed Bug 1451314 Opened 7 years ago Closed 7 years ago

Update Debugger Frontend v32

Categories

(DevTools :: Debugger, enhancement, P3)

enhancement

Tracking

(firefox61 fixed)

RESOLVED FIXED
Firefox 61
Tracking Status
firefox61 --- fixed

People

(Reporter: jlast, Assigned: jlast)

References

Details

Attachments

(1 file, 1 obsolete file)

Assignee: nobody → jlaster
Priority: -- → P3
second try green is green!
Attached patch rel-32.patch (obsolete) (deleted) — Splinter Review
Attachment #8965684 - Flags: review?(jdescottes)
Comment on attachment 8965684 [details] [diff] [review] rel-32.patch Review of attachment 8965684 [details] [diff] [review]: ----------------------------------------------------------------- R+ with: - rebased patch - fixed readme Had a comment about the new file, but I guess we don't have anything to exclude files? Maybe it would have been best to leave it in a non synchronized folder in the debugger repo. ::: devtools/client/debugger/new/README.mozilla @@ +1,4 @@ > This is the debugger.html project output. > See https://github.com/devtools-html/debugger.html > > +Version 17.0 Wrong versions ::: devtools/client/debugger/new/debugger.css @@ -2845,5 @@ > flex: 0 1 content; > order: 3; > } > > -:root.theme-light .breakpoint-snippet, As discussed, needs rebase because of https://bugzilla.mozilla.org/show_bug.cgi?id=1378108 ::: devtools/client/debugger/new/test/mochitest/examples/babel/source-maps-semantics.md @@ +1,1 @@ > +# Source Map Semantics Is this file really supposed to be synchronized or is it just because we blindly copy the whole test folder?
Attachment #8965684 - Flags: review?(jdescottes) → review+
blind copy, i'll leave it out
Attached patch rel-32-2.patch (deleted) — Splinter Review
Attachment #8965684 - Attachment is obsolete: true
Attachment #8965840 - Flags: review+
Backed out changeset 1a7f56d4af25 (bug 1451314) for g2 failures in debugger/custom.js: Test timed out on a CLOSED TREE Problematic push: https://treeherder.mozilla.org/#/jobs?repo=mozilla-inbound&revision=1a7f56d4af25927c756f94f333068ba5499b2895&filter-resultStatus=testfailed&filter-resultStatus=busted&filter-resultStatus=exception&filter-classifiedState=unclassified Failure: https://treeherder.mozilla.org/#/jobs?repo=mozilla-inbound&filter-resultStatus=testfailed&filter-resultStatus=busted&filter-resultStatus=exception&filter-classifiedState=unclassified&selectedJob=172405094 Backout: https://treeherder.mozilla.org/#/jobs?repo=mozilla-inbound&revision=c4b19afa06ea11c16ff3983a69e6c3aa1fdad8de&filter-resultStatus=testfailed&filter-resultStatus=busted&filter-resultStatus=exception&filter-classifiedState=unclassified Log: https://treeherder.mozilla.org/logviewer.html#?job_id=172405094&repo=mozilla-inbound&lineNumber=1315 16:29:01 INFO - PID 31046 | Run function in content process: window.hitBreakpoint() 16:33:57 INFO - PID 31046 | TEST-UNEXPECTED-FAIL | damp | debugger/custom.js: Test timed out 16:33:57 INFO - Terminating psutil.Process(pid=31046, name='firefox', started='16:27:55') 16:33:57 INFO - TEST-UNEXPECTED-ERROR | damp | unexpected error 16:33:57 ERROR - Traceback (most recent call last): 16:33:57 INFO - File "/home/cltbld/workspace/build/tests/talos/talos/run_tests.py", line 296, in run_tests 16:33:57 INFO - talos_results.add(mytest.runTest(browser_config, test)) 16:33:57 INFO - File "/home/cltbld/workspace/build/tests/talos/talos/ttest.py", line 63, in runTest 16:33:57 INFO - return self._runTest(browser_config, test_config, setup) 16:33:57 INFO - File "/home/cltbld/workspace/build/tests/talos/talos/ttest.py", line 207, in _runTest 16:33:57 INFO - debugger_args=browser_config['debugger_args'] 16:33:57 INFO - File "/home/cltbld/workspace/build/tests/talos/talos/talos_process.py", line 156, in run_browser 16:33:57 INFO - raise TalosError("unexpected error") 16:33:57 INFO - TalosError: unexpected error 16:33:57 INFO - TEST-INFO took 361711ms 16:33:57 INFO - SUITE-END | took 361s 16:33:58 ERROR - Return code: 2 16:33:58 WARNING - setting return code to 2 16:33:58 ERROR - # TBPL FAILURE #
Flags: needinfo?(jlaster)
Thanks stefan. https://treeherder.mozilla.org/#/jobs?repo=try&revision=f81a24bfc356715d62940f4aec901cc16192ef08 kicking off a new try run for just talos T-e10s to see if it passes without map-scopes. If it does, we can re-land 32 w/o map-scopes and try to isolate the issue.
Flags: needinfo?(jlaster)
For the record, you can run DAMP tests locally. This one-liner will run the custom debugger test. ./mach talos-test --activeTests damp --subtest custom.debugger --cycles 1 --tppagecycles 1 And it does pass when flipping devtools.debugger.features.map-scopes to false.
try 2 sans map-scopes
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 61
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: