WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
175108
Web Inspector: add button to open Inspector^2
https://bugs.webkit.org/show_bug.cgi?id=175108
Summary
Web Inspector: add button to open Inspector^2
Devin Rousso
Reported
2017-08-02 19:00:36 PDT
Words cannot describe my hatred for the fact that I cannot open Inspector^2 without clicking "Inspect Element" in the context menu of the Inspector page. Furthermore, having it ALWAYS open to Elements is just awful, especially if I am doing work in Debugger/Timelines.
Attachments
Patch
(16.23 KB, patch)
2017-08-02 19:03 PDT
,
Devin Rousso
no flags
Details
Formatted Diff
Diff
[Image] After Patch is applied
(544.35 KB, image/png)
2017-08-02 19:04 PDT
,
Devin Rousso
no flags
Details
View All
Add attachment
proposed patch, testcase, etc.
Devin Rousso
Comment 1
2017-08-02 19:03:54 PDT
Created
attachment 317064
[details]
Patch
Devin Rousso
Comment 2
2017-08-02 19:04:51 PDT
Created
attachment 317065
[details]
[Image] After Patch is applied
Blaze Burg
Comment 3
2017-08-03 10:40:51 PDT
Comment on
attachment 317064
[details]
Patch View in context:
https://bugs.webkit.org/attachment.cgi?id=317064&action=review
r=me
> Source/WebCore/inspector/InspectorFrontendHost.cpp:421 > +void InspectorFrontendHost::inspectInspector()
Did you test this in the remote inspection case? I am pretty sure it should Just Work but that doesn't mean a whole lot.
Devin Rousso
Comment 4
2017-08-03 15:41:17 PDT
Comment on
attachment 317064
[details]
Patch View in context:
https://bugs.webkit.org/attachment.cgi?id=317064&action=review
>> Source/WebCore/inspector/InspectorFrontendHost.cpp:421 >> +void InspectorFrontendHost::inspectInspector() > > Did you test this in the remote inspection case? I am pretty sure it should Just Work but that doesn't mean a whole lot.
I tried inspecting the simulator and it worked fine.
WebKit Commit Bot
Comment 5
2017-08-03 16:11:00 PDT
Comment on
attachment 317064
[details]
Patch Clearing flags on attachment: 317064 Committed
r220235
: <
http://trac.webkit.org/changeset/220235
>
WebKit Commit Bot
Comment 6
2017-08-03 16:11:02 PDT
All reviewed patches have been landed. Closing bug.
Radar WebKit Bug Importer
Comment 7
2017-08-03 16:12:03 PDT
<
rdar://problem/33711711
>
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug