[webkit-reviews] review denied: [Bug 86469] Web Inspector: merge show navigator with pin navigator buttons. : [Attachment 141929] Patch

[bugzilla-daemon at webkit.org]

Vsevolod Vlasov <vsevik at chromium.org> has denied Pavel Feldman
<pfeldman at chromium.org>'s request for review:
Bug 86469: Web Inspector: merge show navigator with pin navigator buttons.
https://bugs.webkit.org/show_bug.cgi?id=86469

Attachment 141929: Patch
https://bugs.webkit.org/attachment.cgi?id=141929&action=review

------- Additional Comments from Vsevolod Vlasov <vsevik at chromium.org>
View in context: https://bugs.webkit.org/attachment.cgi?id=141929&action=review


> Source/WebCore/inspector/front-end/NavigatorOverlayController.js:102
>      _pinNavigator: function()

You sohuld unregister shortcut and remove mouse down listener here.