Closed Bug 1317485 Opened 7 years ago Closed 7 years ago
Can't evaluate scripts in console if loading debugger frontend on a page that has no sources
58 bytes, text/x-review-board-request
STR: Open a page with no JS: `data:text/html,`. Or one that has a non-parseable script tag: https://bug1313921.bmoattachments.org/attachment.cgi?id=8805840. Open debugger Open console Evaluate anything in the console Expected: Evaluation happens Actual: No response. Error from debugger forntend: TypeError: frames is null panel.js:53:9. Filing under debugger because if I don't open the debugger UI the evaluation happens successfully.
James, looks like we might need an extra null check here: https://dxr.mozilla.org/mozilla-central/source/devtools/client/debugger/new/panel.js#53. Is it expected that this._selectors().getFrames will return null in certain cases?
I just checked and I think this is also breaking input in the Browser Content Toolbox entirely. Developer -> Browser Content Toolbox Open console and run some script Get the same 'TypeError: frames is null' error
I was going to review this today but then Jason mentioned that Julian also filed this bug: bug 1317819 Brian, you can only make changes to the tests on github: https://github.com/devtools-html/debugger.html/tree/master/src/test/mochitest. When we deploy the debugger, we copy over those tests as well. Since you have changes to the test, are you OK if we go with Julian's patch? What do the changes to the tests do? If you don't mind, can you make a PR instead?
OK, looks that that's already landed so going to dupe this to that bug (In reply to James Long (:jlongster) from comment #5) > I was going to review this today but then Jason mentioned that Julian also > filed this bug: bug 1317819 > > Brian, you can only make changes to the tests on github: > https://github.com/devtools-html/debugger.html/tree/master/src/test/ > mochitest. When we deploy the debugger, we copy over those tests as well. > Since you have changes to the test, are you OK if we go with Julian's patch? > > What do the changes to the tests do? If you don't mind, can you make a PR > instead? It's just a regression test that hits this particular case. I'll copy it over
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → DUPLICATE
You need to log in before you can comment on or make changes to this bug.