Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(700)

Issue 2168143002: Adding keyboard shortcut BROWSE+SHIFT+N to launch 'Connectivity diag.' extension.

Created:
4 years, 5 months ago by yyefet
Modified:
4 years, 5 months ago
Reviewers:
sky
CC:
chromium-reviews, kalyank, sadrul, yusukes+watch_chromium.org, derat+watch_chromium.org
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Adding keyboard shortcut BROWSE+SHIFT+N to launch 'Connectivity diag.' extension. Chrome has a built in 'Connectivity Diagnostics' extension (shipped with all versions) that performs network tests such as captive portal, dns, firewall, google services check, local network check. As of now, the only way to access this tool is a link in the browser when a site fails to load. It would be useful to have a keyboard shortcut BROWSE+SHIFT+N that launches this extension directly when end users encounter other issues such as sync issues, enterprise issues, cloud print issues, etc. BUG=630310 R=sky@chromium.org TEST=build chromium and enter BROWSE+SHIFT+N; extension launches in browser.

Patch Set 1 #

Total comments: 1

Patch Set 2 : Fixed comments #

Patch Set 3 : Changed key combo to browse shift N to avoid accidents. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+35 lines, -0 lines) Patch
M ash/accelerators/accelerator_controller.cc View 3 chunks +9 lines, -0 lines 0 comments Download
M ash/accelerators/accelerator_table.h View 1 chunk +1 line, -0 lines 0 comments Download
M ash/accelerators/accelerator_table.cc View 1 2 1 chunk +2 lines, -0 lines 0 comments Download
M ash/new_window_delegate.h View 1 1 chunk +3 lines, -0 lines 0 comments Download
M ash/shell/shell_delegate_impl.cc View 1 chunk +1 line, -0 lines 0 comments Download
M ash/test/test_shell_delegate.cc View 1 chunk +1 line, -0 lines 0 comments Download
M chrome/browser/ui/ash/chrome_new_window_delegate.h View 1 chunk +1 line, -0 lines 0 comments Download
M chrome/browser/ui/ash/chrome_new_window_delegate.cc View 1 1 chunk +17 lines, -0 lines 0 comments Download

Messages

Total messages: 5 (1 generated)
yyefet
Submitting keyboard shortcut CL for review, thanks!
4 years, 5 months ago (2016-07-21 17:02:30 UTC) #1
sky
Did you run this by ux? You'll need to merge with tip of tree as ...
4 years, 5 months ago (2016-07-21 19:30:54 UTC) #2
yyefet
On 2016/07/21 19:30:54, sky wrote: > Did you run this by ux? You'll need to ...
4 years, 5 months ago (2016-07-21 20:14:24 UTC) #3
sky
4 years, 5 months ago (2016-07-21 21:43:29 UTC) #4
Resync.

On Thu, Jul 21, 2016 at 1:14 PM,  <yyefet@chromium.org> wrote:
> On 2016/07/21 19:30:54, sky wrote:
>> Did you run this by ux? You'll need to merge with tip of tree as I just
>> moved
>> all these files (sorry). Also, run git cl format.
>>
>>
>> https://codereview.chromium.org/2168143002/diff/1/ash/new_window_delegate.h
>> File ash/new_window_delegate.h (right):
>>
>>
>
https://codereview.chromium.org/2168143002/diff/1/ash/new_window_delegate.h#n...
>> ash/new_window_delegate.h:43: // Invoked when the user opens Network Diag.
>> diag? Dialog.
>
> Waiting for comments from UX team on crbug 630310.
>
> Is there a specific git command to merge w/ tip of tree or should I just
> re-sync?
>
> Ran 'git cl format' with no output.
>
> Thanks!
>
>
> https://codereview.chromium.org/2168143002/

-- 
You received this message because you are subscribed to the Google Groups
"Chromium-reviews" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.

Powered by Google App Engine
This is Rietveld 408576698