WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
59788
Web Inspector: add "open link in new tab" action into the network and resources panels.
https://bugs.webkit.org/show_bug.cgi?id=59788
Summary
Web Inspector: add "open link in new tab" action into the network and resourc...
Pavel Feldman
Reported
2011-04-29 02:14:05 PDT
This change also distinguishes context menu items case between platforms.
Attachments
Patch
(23.87 KB, patch)
2011-04-29 02:25 PDT
,
Pavel Feldman
yurys
: review+
Details
Formatted Diff
Diff
View All
Add attachment
proposed patch, testcase, etc.
Pavel Feldman
Comment 1
2011-04-29 02:25:50 PDT
Created
attachment 91655
[details]
Patch
Yury Semikhatsky
Comment 2
2011-04-29 02:33:43 PDT
Comment on
attachment 91655
[details]
Patch View in context:
https://bugs.webkit.org/attachment.cgi?id=91655&action=review
> Source/WebCore/inspector/front-end/NetworkPanel.js:1018 > + contextMenu.appendItem(WebInspector.UIString(WebInspector.useLowerCaseMenuTitles() ? "Save entry as HAR" : "Save Entry as HAR"), this._exportResource.bind(this, resource));
Can you encapsulate WebInspector.useLowerCaseMenuTitles() logic in UIString to avoid recurring ternary ifs ?
Pavel Feldman
Comment 3
2011-04-29 02:35:15 PDT
Committed
r85312
: <
http://trac.webkit.org/changeset/85312
>
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