summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorarv@google.com <arv@google.com@0039d316-1c4b-4281-b951-d872f2087c98>2009-06-24 20:34:11 +0000
committerarv@google.com <arv@google.com@0039d316-1c4b-4281-b951-d872f2087c98>2009-06-24 20:34:11 +0000
commitd0d8460e70cc8a125e61e1c2c6a21e37daca25ed (patch)
treee8c4adbc68eb430a891acab2d781c3d76c5fff5b
parent2ea7df4a15e32dddd7e46da1c0bb108d8b69e7a4 (diff)
downloadchromium_src-d0d8460e70cc8a125e61e1c2c6a21e37daca25ed.zip
chromium_src-d0d8460e70cc8a125e61e1c2c6a21e37daca25ed.tar.gz
chromium_src-d0d8460e70cc8a125e61e1c2c6a21e37daca25ed.tar.bz2
Fix issues with the last patch.
BUG=14959 TEST=The option menu on the new new tab page should react to keyboard and mouse events. Review URL: http://codereview.chromium.org/147097 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@19166 0039d316-1c4b-4281-b951-d872f2087c98
-rw-r--r--chrome/browser/resources/new_new_tab.js5
1 files changed, 4 insertions, 1 deletions
diff --git a/chrome/browser/resources/new_new_tab.js b/chrome/browser/resources/new_new_tab.js
index 2064e67..033c7f2 100644
--- a/chrome/browser/resources/new_new_tab.js
+++ b/chrome/browser/resources/new_new_tab.js
@@ -683,6 +683,9 @@ function OptionMenu(button, menu) {
this.button.onkeydown = bind(this.handleKeyDown, this);
this.boundHideMenu_ = bind(this.hideMenu, this);
this.boundMaybeHide_ = bind(this.maybeHide_, this);
+ this.menu.onmouseover = bind(this.handleMouseOver, this);
+ this.menu.onmouseout = bind(this.handleMouseOut, this);
+ this.menu.onmouseup = bind(this.handleMouseUp, this);
}
OptionMenu.prototype = {
@@ -805,7 +808,7 @@ OptionMenu.prototype = {
setSelectedIndex: function(i) {
if (i != this.selectedIndex_) {
var items = this.menu.children;
- var oldItem = this.items[this.selectedIndex_];
+ var oldItem = items[this.selectedIndex_];
if (oldItem) {
oldItem.removeAttribute('selected');
}