Remove the backspace/shift+backspace navigating the page.

Remove the backspace/shift+backspace navigating the page.

This is part two of
https://crrev.com/a9fc5462ba084936191d51a9391ec5255e79a930.
Previously, EventHandler.cpp was doing the navigation
on Win/Mac and the code in chrome/browser was handling
Linux. So the previous UseCounter was correct for Win/Mac.

Committed: https://crrev.com/6bb48eb86c7ecd5e735e945166075df48a16e828
Cr-Commit-Position: refs/heads/master@{#390283}

[ojan, 2016-04-26 18:27:58]

The CQ bit was checked by ojan@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-04-26 18:28:22]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1922993002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1922993002/1

[commit-bot: I haz the power, 2016-04-26 19:09:37]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-04-26 19:09:38]

Dry run: Try jobs failed on following builders:
  linux_chromium_asan_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  win_chromium_x64_rel_ng on tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)

[ojan, 2016-04-26 20:17:22]

The CQ bit was checked by ojan@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-04-26 20:17:39]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1922993002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1922993002/20001

[commit-bot: I haz the power, 2016-04-26 21:16:51]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-04-26 21:16:52]

Dry run: This issue passed the CQ dry run.

[ojan, 2016-04-26 22:52:21]

ojan@chromium.org changed reviewers:
+ rsesek@chromium.org

[Robert Sesek, 2016-04-27 00:10:31]

LGTM

[Robert Sesek, 2016-04-27 00:13:35]

I should note that there's a test that references this accelerator, but in the
negative:
https://code.google.com/p/chromium/codesearch#chromium/src/chrome/browser/glo...

Can remove, or not.

[ojan, 2016-04-27 00:45:52]

On 2016/04/27 at 00:13:35, rsesek wrote:
> I should note that there's a test that references this accelerator, but in the
negative:
https://code.google.com/p/chromium/codesearch#chromium/src/chrome/browser/glo...
> 
> Can remove, or not.

Good catch. Was hard to grep the code for all the cases here. :)

[ojan, 2016-04-27 00:46:03]

The CQ bit was checked by ojan@chromium.org

[ojan, 2016-04-27 00:46:03]

The patchset sent to the CQ was uploaded after l-g-t-m from rsesek@chromium.org
Link to the patchset: https://codereview.chromium.org/1922993002/#ps40001
(title: "address review")

[commit-bot: I haz the power, 2016-04-27 00:46:28]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1922993002/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1922993002/40001

[commit-bot: I haz the power, 2016-04-27 00:54:18]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-04-27 00:54:20]

Try jobs failed on following builders:
  chromium_presubmit on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[ojan, 2016-04-27 17:03:11]

ojan@chromium.org changed reviewers:
+ sky@chromium.org

[ojan, 2016-04-27 17:03:12]

Hmmm...git cl lied. Apparently I need review from sky as well. sky, see your
email for context if the justification for this is unclear.

[sky, 2016-04-27 17:44:28]

LGTM

[ojan, 2016-04-28 02:18:45]

The CQ bit was checked by ojan@chromium.org

[commit-bot: I haz the power, 2016-04-28 02:19:07]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1922993002/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1922993002/40001

[commit-bot: I haz the power, 2016-04-28 03:06:41]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-04-30 17:15:47]

Patchset 3 (id:??) landed as
https://crrev.com/6bb48eb86c7ecd5e735e945166075df48a16e828
Cr-Commit-Position: refs/heads/master@{#390283}

[commit-bot: I haz the power, 2016-04-30 17:53:54]

Description was changed from

==========
Remove the backspace/shift+backspace navigating the page.

This is part two of
https://crrev.com/a9fc5462ba084936191d51a9391ec5255e79a930.
Previously, EventHandler.cpp was doing the navigation
on Win/Mac and the code in chrome/browser was handling
Linux. So the previous UseCounter was correct for Win/Mac.
==========

to

==========
Remove the backspace/shift+backspace navigating the page.

This is part two of
https://crrev.com/a9fc5462ba084936191d51a9391ec5255e79a930.
Previously, EventHandler.cpp was doing the navigation
on Win/Mac and the code in chrome/browser was handling
Linux. So the previous UseCounter was correct for Win/Mac.

Committed: https://crrev.com/6bb48eb86c7ecd5e735e945166075df48a16e828
Cr-Commit-Position: refs/heads/master@{#390283}
==========