Suppress browser shortcuts.

chrome/browser/resources/active_downloads.html

Index: chrome/browser/resources/active_downloads.html
===================================================================
--- chrome/browser/resources/active_downloads.html	(revision 86295)
+++ chrome/browser/resources/active_downloads.html	(working copy)
@@ -220,6 +220,11 @@
   localStrings = new LocalStrings();
   initTestHarness();
 
+  // Suppress browser shortcuts. 
+  window.onkeydown = function(e) {

[rginda, 2011/05/25 04:13:38]

LGTM as long as you're sure you want to suppress *all* shortcuts rather than
blacklisting known trouble combos.

[achuithb, 2011/05/25 06:58:25]

I don't think we want any shortcuts for the pop-up. Can you think of any that
would be useful?

On 2011/05/25 04:13:38, rginda wrote:

[xiyuan, 2011/05/25 16:55:51]

Ctrl+W is useful, I think.

Yeah, I saw the TEST says it works. But I wonder if it still works after you
click inside the download panel. The click should put focus on the
RenderWidgetHostView (or something like that) and makes the window here get key
events first.

On 2011/05/25 06:58:25, achuith.bhandarkar wrote:

+    e.preventDefault();
+  };
+
   $('showalldownloadstext').textContent =
       localStrings.getString('showalldownloads');
 
@@ -264,7 +269,7 @@
 /**
  * Create a results array with test data and call downloadsList.
  */
-var id = 1;
+var id = 0;

[rginda, 2011/05/25 04:13:38]

Is this here on purpose?

[achuithb, 2011/05/25 06:58:25]

Ya, test code. 0 is a better place to start.

 var results = [];
 function sendTestResults() {
   results.push({