Keyboard shortcut for 'Back','Forward' and 'Save page' are not present.

Keyboard shortcut for 'Back','Forward' and 'Save page' are not present.

keyboard shortcuts are not mentioned with 'Back','Forward' and 'Save page'
in context menu. Keyboard shortcut's should be shown, so that user can
do these operation via keyboard also.

Changes done for adding info for keyboard shortcuts.

BUG=435036
Committed: https://crrev.com/e5eb027f40c10856870fb024320060b7f6a42267
Cr-Commit-Position: refs/heads/master@{#305180}

[Deepak, 2014-11-20 10:31:15]

deepak.m1@samsung.com changed reviewers:
+ msw@chromium.org

[Deepak, 2014-11-20 10:31:15]

Followup for previous issue for providing Accelerator key's for menu items.
AFAIK with this patch All possible Accelerator key's for menu items will be
done.
PTAL.

[msw, 2014-11-20 17:36:46]

https://codereview.chromium.org/727533004/diff/1/chrome/browser/ui/views/rend...
File
chrome/browser/ui/views/renderer_context_menu/render_view_context_menu_views.cc
(right):

https://codereview.chromium.org/727533004/diff/1/chrome/browser/ui/views/rend...
chrome/browser/ui/views/renderer_context_menu/render_view_context_menu_views.cc:114:
case IDC_SAVE_PAGE:
Put this in alphabetical order (after IDC_RELOAD).

[Deepak, 2014-11-21 02:42:36]

On 2014/11/20 17:36:46, msw wrote:
>
https://codereview.chromium.org/727533004/diff/1/chrome/browser/ui/views/rend...
> File
>
chrome/browser/ui/views/renderer_context_menu/render_view_context_menu_views.cc
> (right):
> 
>
https://codereview.chromium.org/727533004/diff/1/chrome/browser/ui/views/rend...
>
chrome/browser/ui/views/renderer_context_menu/render_view_context_menu_views.cc:114:
> case IDC_SAVE_PAGE:
> Put this in alphabetical order (after IDC_RELOAD).

Thanks for review, Changes done.
PTAL.

[msw, 2014-11-21 05:25:04]

lgtm

[Deepak, 2014-11-21 05:29:44]

On 2014/11/21 05:25:04, msw wrote:
> lgtm

Thanks.

[Deepak, 2014-11-21 05:29:47]

The CQ bit was checked by deepak.m1@samsung.com

[commit-bot: I haz the power, 2014-11-21 05:30:59]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/727533004/20001

[commit-bot: I haz the power, 2014-11-21 06:16:57]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2014-11-21 06:17:37]

Patchset 2 (id:??) landed as
https://crrev.com/e5eb027f40c10856870fb024320060b7f6a42267
Cr-Commit-Position: refs/heads/master@{#305180}