Closed
Bug 1451314
Opened 7 years ago
Closed 7 years ago
Update Debugger Frontend v32
Categories
(DevTools :: Debugger, enhancement, P3)
DevTools
Debugger
Tracking
(firefox61 fixed)
RESOLVED
FIXED
Firefox 61
Tracking | Status | |
---|---|---|
firefox61 | --- | fixed |
People
(Reporter: jlast, Assigned: jlast)
References
Details
Attachments
(1 file, 1 obsolete file)
(deleted),
patch
|
jlast
:
review+
|
Details | Diff | Splinter Review |
Assignee | ||
Updated•7 years ago
|
Assignee | ||
Comment 1•7 years ago
|
||
Assignee | ||
Comment 2•7 years ago
|
||
Assignee | ||
Comment 3•7 years ago
|
||
second try green is green!
Assignee | ||
Comment 4•7 years ago
|
||
Attachment #8965684 -
Flags: review?(jdescottes)
Comment 5•7 years ago
|
||
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+
Assignee | ||
Comment 6•7 years ago
|
||
blind copy, i'll leave it out
Assignee | ||
Comment 7•7 years ago
|
||
Attachment #8965684 -
Attachment is obsolete: true
Attachment #8965840 -
Flags: review+
Assignee | ||
Comment 8•7 years ago
|
||
Assignee | ||
Comment 9•7 years ago
|
||
Comment 10•7 years ago
|
||
Pushed by jlaster@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/1a7f56d4af25
Update Debugger Frontend v32. r=jdescottes
Assignee | ||
Comment 11•7 years ago
|
||
Looks like v32 some how broke the custom debugger talos test :/
https://treeherder.mozilla.org/#/jobs?repo=mozilla-inbound&revision=1a7f56d4af25927c756f94f333068ba5499b2895&selectedJob=172407439
Comment 12•7 years ago
|
||
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)
Assignee | ||
Comment 13•7 years ago
|
||
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)
Comment 14•7 years ago
|
||
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.
Comment 15•7 years ago
|
||
Pushed by jlaster@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/4e39c75b7189
Update Debugger Frontend v32. r=jdescottes
Assignee | ||
Comment 16•7 years ago
|
||
try 2 sans map-scopes
Comment 17•7 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 7 years ago
status-firefox61:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 61
Comment 18•7 years ago
|
||
Updated•6 years ago
|
Product: Firefox → DevTools
You need to log in
before you can comment on or make changes to this bug.
Description
•