2011-02-17 Ojan Vafai <ojan@chromium.org>
authorojan@chromium.org <ojan@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 18 Feb 2011 04:38:03 +0000 (04:38 +0000)
committerojan@chromium.org <ojan@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 18 Feb 2011 04:38:03 +0000 (04:38 +0000)
        Reviewed by Antonio Gomes.

        make event handling work in Gecko
        https://bugs.webkit.org/show_bug.cgi?id=54702

        Gecko doesn't support the global event property. It's an IE-ism.

        * code-review.js:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@78948 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Websites/bugs.webkit.org/ChangeLog
Websites/bugs.webkit.org/code-review.js

index a47031ffcdd825953602a9475d49177f9f2ed2c1..c8f89291c98476b66ad3a6cf426bcd01a0e79ddc 100644 (file)
@@ -1,3 +1,14 @@
+2011-02-17  Ojan Vafai  <ojan@chromium.org>
+
+        Reviewed by Antonio Gomes.
+
+        make event handling work in Gecko
+        https://bugs.webkit.org/show_bug.cgi?id=54702
+
+        Gecko doesn't support the global event property. It's an IE-ism.
+
+        * code-review.js:
+
 2011-02-14  Ojan Vafai  <ojan@chromium.org>
 
         Reviewed by Adam Barth.
index 0967a5ff8ca253ff7e5cc660326279e4e0ee3127..9386493a0c5c8430d8b610c152f1f938644bf026 100644 (file)
@@ -1382,21 +1382,21 @@ var CODE_REVIEW_UNITTEST;
     return node.hasClass('DiffBlock');
   }
 
-  $('textarea').live('keydown', function() {
+  $('textarea').live('keydown', function(e) {
     var unicode_escape = 'U+001B';
-    if (event.keyIdentifier == unicode_escape)
+    if (e.keyIdentifier == unicode_escape)
       handleEscapeKeyDownInTextarea(this);
   });
 
-  $('body').live('keypress', function() {
+  $('body').live('keypress', function(e) {
     // FIXME: There's got to be a better way to avoid seeing these keypress
     // events.
-    if (event.target.nodeName == 'TEXTAREA')
+    if (e.target.nodeName == 'TEXTAREA')
       return;
     
     var handled = false;
 
-    switch (event.charCode) {
+    switch (e.charCode) {
     case kCharCodeForN:
       handled = focusNext(isComment, DIRECTION.FORWARD);
       break;
@@ -1419,7 +1419,7 @@ var CODE_REVIEW_UNITTEST;
     }
     
     if (handled)
-      event.preventDefault();
+      e.preventDefault();
   });
   
   function handleEscapeKeyDownInTextarea(textarea) {
@@ -1509,7 +1509,7 @@ var CODE_REVIEW_UNITTEST;
   }).live('mousedown', function(e) {
     // preventDefault to avoid selecting text when dragging to select comment context lines.
     // FIXME: should we use user-modify CSS instead?
-    event.preventDefault();
+    e.preventDefault();
     if (e.shiftKey)
       return;