Accelerator Key's are not visible for Print and Inspect Element in pdf context menu.

Accelerator Key's are not visible for Print and Inspect Element in pdf context menu.

Accelerator key's should be visible in context menu,so that user will learn and can use
these Accelerator keys for Print and Inspect Element without opening context menu everytime.
Changes done to show Accelerator label for Print and Inspect Element.

BUG=432004
Committed: https://crrev.com/36ae1ba62b3b4b8dd3b622a76fdd927a08545705
Cr-Commit-Position: refs/heads/master@{#303609}

[Deepak, 2014-11-11 04:52:51]

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

[Deepak, 2014-11-11 04:52:51]

This is follow up for 431313 issue for adding Accelerator key's for "Print" and
"Inspect Element" as Suggested.
PTAL.

[msw, 2014-11-11 07:27:09]

lgtm with a nit

https://codereview.chromium.org/718583002/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/718583002/diff/1/chrome/browser/ui/views/rend...
chrome/browser/ui/views/renderer_context_menu/render_view_context_menu_views.cc:82:
ui::Accelerator(ui::VKEY_I, ui::EF_SHIFT_DOWN | ui::EF_CONTROL_DOWN);
nit: follow the surrounding line breaking convention.

[Deepak, 2014-11-11 07:35:47]

https://codereview.chromium.org/718583002/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/718583002/diff/1/chrome/browser/ui/views/rend...
chrome/browser/ui/views/renderer_context_menu/render_view_context_menu_views.cc:82:
ui::Accelerator(ui::VKEY_I, ui::EF_SHIFT_DOWN | ui::EF_CONTROL_DOWN);
On 2014/11/11 07:27:09, msw wrote:
> nit: follow the surrounding line breaking convention.

Done.

[Deepak, 2014-11-11 07:36:14]

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

[commit-bot: I haz the power, 2014-11-11 07:36:40]

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

[commit-bot: I haz the power, 2014-11-11 08:24:10]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2014-11-11 08:24:47]

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