Summary: | Web Inspector: Debugger: remove ASSERT_NOT_REACHED where it's possible to reach | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Devin Rousso <hi> | ||||||
Component: | Web Inspector | Assignee: | Devin Rousso <hi> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | commit-queue, ews-watchlist, hi, inspector-bugzilla-changes, joepeck, webkit-bug-importer | ||||||
Priority: | P2 | Keywords: | InRadar | ||||||
Version: | WebKit Nightly Build | ||||||||
Hardware: | All | ||||||||
OS: | All | ||||||||
Attachments: |
|
Description
Devin Rousso
2019-04-23 13:55:03 PDT
Created attachment 368060 [details]
Patch
Comment on attachment 368060 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=368060&action=review > Source/WebCore/inspector/InspectorInstrumentation.cpp:1188 > + FAST_RETURN_IF_NO_FRONTENDS(nullptr); Is there ever a case where we would not already have fast returned if there was no frontend? This line seems unnecessary. Comment on attachment 368060 [details]
Patch
r=me
Comment on attachment 368060 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=368060&action=review >> Source/WebCore/inspector/InspectorInstrumentation.cpp:1188 >> + FAST_RETURN_IF_NO_FRONTENDS(nullptr); > > Is there ever a case where we would not already have fast returned if there was no frontend? This line seems unnecessary. Yeah, this one doesn't need it. Created attachment 368061 [details]
Patch
Comment on attachment 368061 [details] Patch Clearing flags on attachment: 368061 Committed r244569: <https://trac.webkit.org/changeset/244569> All reviewed patches have been landed. Closing bug. |