Summary: | Web Inspector: Uncaught Exception: TypeError: null is not an object (evaluating 'issueMessage.sourceCodeLocation.sourceCode') | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Devin Rousso <hi> | ||||||
Component: | Web Inspector | Assignee: | Devin Rousso <hi> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | commit-queue, hi, inspector-bugzilla-changes, joepeck, webkit-bug-importer | ||||||
Priority: | P2 | Keywords: | InRadar | ||||||
Version: | WebKit Nightly Build | ||||||||
Hardware: | All | ||||||||
OS: | All | ||||||||
URL: | https://www.youtube.com/user/NewOnNetflix/videos | ||||||||
Attachments: |
|
Description
Devin Rousso
2019-07-31 00:26:31 PDT
Created attachment 375217 [details]
Patch
Comment on attachment 375217 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=375217&action=review r=me > Source/WebInspectorUI/ChangeLog:10 > + we should pass the `WI.SourceCode` along in that case, and use it when creating a associated Grammar: "a associate" => "an association" > Source/WebInspectorUI/UserInterface/Views/DebuggerSidebarPanel.js:1407 > + if (!sourceCode && !issueMessage.sourceCodeLocation) > + return null; Can this ever happen? Could this just become an assert. Seems the `debuggerObject` case is the only time this could happen. Comment on attachment 375217 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=375217&action=review >> Source/WebInspectorUI/UserInterface/Views/DebuggerSidebarPanel.js:1407 >> + return null; > > Can this ever happen? Could this just become an assert. > > Seems the `debuggerObject` case is the only time this could happen. I don't think it can happen, because the `debuggerObject` path already checks for `debuggerObject.sourceCodeLocation.displaySourceCode`, so it should be valid there too. I'll change it to an assert. Created attachment 375288 [details]
Patch
Comment on attachment 375288 [details] Patch Clearing flags on attachment: 375288 Committed r248094: <https://trac.webkit.org/changeset/248094> All reviewed patches have been landed. Closing bug. |