history: fix tappable blank space at the end of history entries.

chrome/browser/resources/history/history.js

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 <include src="../uber/uber_utils.js">
 <include src="history_focus_manager.js">
 
 ///////////////////////////////////////////////////////////////////////////////
 // Globals:
 /** @const */ var RESULTS_PER_PAGE = 150;
@@ skipping 187 matching lines @@
     checkbox.id = 'checkbox-' + this.id_;
     checkbox.time = this.date.getTime();
     checkbox.addEventListener('click', checkboxClicked);
     time.setAttribute('for', checkbox.id);
     entryBox.appendChild(checkbox);
 
     if (focusless)
       checkbox.tabIndex = -1;
 
     // Clicking anywhere in the entryBox will check/uncheck the checkbox.
     entryBox.setAttribute('for', checkbox.id);

[Evan Stade, 2014/09/25 01:53:27]

NOOOOOOOOOOO

kill for!!

[Dan Beam, 2014/09/25 03:59:45]

Acknowledged.

     entryBox.addEventListener('mousedown', entryBoxMousedown);

[Evan Stade, 2014/09/25 01:53:26]

i feel like all of these should go in the if block (it's actually possible to
have a keyboard on an android device, in case you're wondering)

[Dan Beam, 2014/09/25 03:59:45]

Done.

-    entryBox.addEventListener('click', entryBoxClick);
     entryBox.addEventListener('keydown', this.handleKeydown_.bind(this));
+
+    if (!isMobileVersion())
+      entryBox.addEventListener('click', entryBoxClick);
   }
 
   // Keep track of the drop down that triggered the menu, so we know
   // which element to apply the command to.
   // TODO(dubroy): Ideally we'd use 'activate', but MenuButton swallows it.
   var setActiveVisit = function(e) {
     activeVisit = self;
     var menu = $('action-menu');
     menu.dataset.devicename = self.deviceName;
     menu.dataset.devicetype = self.deviceType;
@@ skipping 2044 matching lines @@
     historyView.reload();
 }
 
 // Add handlers to HTML elements.
 document.addEventListener('DOMContentLoaded', load);
 
 // This event lets us enable and disable menu items before the menu is shown.
 document.addEventListener('canExecute', function(e) {
   e.canExecute = true;
 });