2011-02-20 Ojan Vafai <ojan@chromium.org>

        Reviewed by Adam Barth.

        [codereviewtool] avoid overriding ctrl/cmd + r
        https://bugs.webkit.org/show_bug.cgi?id=54852

        * code-review.js:

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

diff --git a/Websites/bugs.webkit.org/ChangeLog b/Websites/bugs.webkit.org/ChangeLog
index bae1bd017bf682c610e84f568e550ddaa28c8cc4..b6823481d8f35d2d35b68e146f6594039ca51de8 100644 (file)
--- a/Websites/bugs.webkit.org/ChangeLog
+++ b/Websites/bugs.webkit.org/ChangeLog
@@ -1,3 +1,12 @@
+2011-02-20  Ojan Vafai  <ojan@chromium.org>
+
+        Reviewed by Adam Barth.
+
+        [codereviewtool] avoid overriding ctrl/cmd + r
+        https://bugs.webkit.org/show_bug.cgi?id=54852
+
+        * code-review.js:
+
 2011-02-16  Ojan Vafai  <ojan@chromium.org>
 
         Reviewed by Adam Barth.

diff --git a/Websites/bugs.webkit.org/PrettyPatch/PrettyPatch.rb b/Websites/bugs.webkit.org/PrettyPatch/PrettyPatch.rb
index 4c84aa0f509aaecda166f90ac74c59d797826e79..6218ae9ec389475095071b4180ea40908c828776 100644 (file)
--- a/Websites/bugs.webkit.org/PrettyPatch/PrettyPatch.rb
+++ b/Websites/bugs.webkit.org/PrettyPatch/PrettyPatch.rb
@@ -451,7 +451,7 @@ div:focus {
 }
 </style>
 <script src="https://ajax.googleapis.com/ajax/libs/jquery/1.4.2/jquery.min.js"></script> 
-<script src="code-review.js?version=38"></script>
+<script src="code-review.js?version=39"></script>
 EOF
 
     def self.revisionOrDescription(string)

diff --git a/Websites/bugs.webkit.org/code-review.js b/Websites/bugs.webkit.org/code-review.js
index 4bd67fb81fd1f3e022a7e5940b32df3d584464b8..b3917450b783e09cc389aa318d0601f7040202c0 100644 (file)
--- a/Websites/bugs.webkit.org/code-review.js
+++ b/Websites/bugs.webkit.org/code-review.js
@@ -1502,6 +1502,10 @@ var CODE_REVIEW_UNITTEST;
     if (e.target.nodeName == 'TEXTAREA')
       return;
 
+    // Don't want to override browser shortcuts like ctrl+r.
+    if (e.metaKey || e.ctrlKey)
+      return;
+
     if (handleModifyContextKey(e))
       return;