diff options
author | arv@google.com <arv@google.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-06-24 20:34:11 +0000 |
---|---|---|
committer | arv@google.com <arv@google.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-06-24 20:34:11 +0000 |
commit | d0d8460e70cc8a125e61e1c2c6a21e37daca25ed (patch) | |
tree | e8c4adbc68eb430a891acab2d781c3d76c5fff5b | |
parent | 2ea7df4a15e32dddd7e46da1c0bb108d8b69e7a4 (diff) | |
download | chromium_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.js | 5 |
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'); } |