Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(2516)

Unified Diff: chrome/browser/resources/ntp4/recently_closed.js

Issue 12754009: Fix recently closed multiple tabs regression. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/resources/ntp4/recently_closed.js
diff --git a/chrome/browser/resources/ntp4/recently_closed.js b/chrome/browser/resources/ntp4/recently_closed.js
index b0d26b052c810cd5b58897bb8affb7c3c2eee187..92e4f95e8d2b8a514a6bb0c3bfe5bf24970da8c8 100644
--- a/chrome/browser/resources/ntp4/recently_closed.js
+++ b/chrome/browser/resources/ntp4/recently_closed.js
@@ -101,12 +101,9 @@ cr.define('ntp', function() {
orig.metaKey,
orig.shiftKey];
chrome.send('reopenTab', params);
-
- // We are likely deleted by this point!
- e.stopPropagation();
- e.preventDefault();
}
a.addEventListener('activate', onActivated);
+ a.addEventListener('click', function(e) { e.preventDefault(); });
this.menu.appendChild(a);
cr.ui.decorate(a, MenuItem);
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698