Fix for Back navigation keyboard shortcut does not work properly for chrome://settings/cookies.

Fix for Back navigation keyboard shortcut does not work properly for chrome://settings/cookies.

Earlier left and right navigation key's are getting used without check
for modifiers.
Check added for alt,ctrl,meta,shift key, when we have modifer then default
handling will happen that will move it to previous url with alt+left arrow,
and when we have just left or right arrow keys then cookies list specific handling will happen.

BUG=467476
Committed: https://crrev.com/9c2d1afc6031dd54e54f75244438dd4aab8f78b8
Cr-Commit-Position: refs/heads/master@{#320872}

[Deepak, 2015-03-16 13:40:53]

deepak.m1@samsung.com changed reviewers:
+ bauerb@chromium.org, dbeam@chromium.org

[Deepak, 2015-03-16 13:40:54]

PTAL
Thanks.

[Dan Beam, 2015-03-16 21:16:20]

lgtm

https://codereview.chromium.org/1013683002/diff/1/chrome/browser/resources/op...
File chrome/browser/resources/options/cookies_list.js (right):

https://codereview.chromium.org/1013683002/diff/1/chrome/browser/resources/op...
chrome/browser/resources/options/cookies_list.js:735: handleKeyLeftRight_:
function(e) {
nit: do this instead

if (e.altKey || e.ctrlKey || e.metaKey || e.shiftKey)
  return;

[Deepak, 2015-03-17 02:46:16]

On 2015/03/16 21:16:20, Dan Beam wrote:
> lgtm
> 
>
https://codereview.chromium.org/1013683002/diff/1/chrome/browser/resources/op...
> File chrome/browser/resources/options/cookies_list.js (right):
> 
>
https://codereview.chromium.org/1013683002/diff/1/chrome/browser/resources/op...
> chrome/browser/resources/options/cookies_list.js:735: handleKeyLeftRight_:
> function(e) {
> nit: do this instead
> 
> if (e.altKey || e.ctrlKey || e.metaKey || e.shiftKey)
>   return;

Thanks for review.
nit addressed.

[Deepak, 2015-03-17 02:46:27]

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

[Deepak, 2015-03-17 02:46:27]

The patchset sent to the CQ was uploaded after l-g-t-m from dbeam@chromium.org
Link to the patchset: https://codereview.chromium.org/1013683002/#ps20001
(title: "Addressing nit.")

[commit-bot: I haz the power, 2015-03-17 02:46:41]

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

[commit-bot: I haz the power, 2015-03-17 05:47:18]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-03-17 05:48:15]

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