2010-09-19 Adam Barth <abarth@webkit.org>
authorabarth@webkit.org <abarth@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 20 Sep 2010 04:26:46 +0000 (04:26 +0000)
committerabarth@webkit.org <abarth@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 20 Sep 2010 04:26:46 +0000 (04:26 +0000)
        Reviewed by Eric Seidel.

        Move reviewtool to action=review
        https://bugs.webkit.org/show_bug.cgi?id=46071

        This patch removes the old Review Patch screen and moves the new
        reviewtool from Pretty Diff to Review Patch.

        * attachment.cgi:
        * code-review.js:

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

BugsSite/ChangeLog
BugsSite/attachment.cgi
BugsSite/code-review.js

index 5ae4f41..a7b8079 100644 (file)
@@ -1,5 +1,18 @@
 2010-09-19  Adam Barth  <abarth@webkit.org>
 
 2010-09-19  Adam Barth  <abarth@webkit.org>
 
+        Reviewed by Eric Seidel.
+
+        Move reviewtool to action=review
+        https://bugs.webkit.org/show_bug.cgi?id=46071
+
+        This patch removes the old Review Patch screen and moves the new
+        reviewtool from Pretty Diff to Review Patch.
+
+        * attachment.cgi:
+        * code-review.js:
+
+2010-09-19  Adam Barth  <abarth@webkit.org>
+
         Review tool UI tweak.  This patch lengthens the status bubbles so all
         the bubble fit (even when they have numbers inside).
 
         Review tool UI tweak.  This patch lengthens the status bubbles so all
         the bubble fit (even when they have numbers inside).
 
index 07800d3..90c279d 100755 (executable)
@@ -129,7 +129,7 @@ elsif ($action eq "edit")
 #if WEBKIT_CHANGES
 elsif ($action eq "review")
 {
 #if WEBKIT_CHANGES
 elsif ($action eq "review")
 {
-    edit("review");
+    prettyPatch();
 }
 elsif ($action eq "reviewform")
 {
 }
 elsif ($action eq "reviewform")
 {
index c9042a5..2e4b3ef 100644 (file)
     }
   }
 
     }
   }
 
-  // Attempt to activate only in the "Formatted Diff" context.
+  // Attempt to activate only in the "Review Patch" context.
   if (window.top != window)
     return;
   if (window.top != window)
     return;
+  if (!window.location.search.match(/action=review/))
+    return;
   var attachment_id = determineAttachmentID();
   if (!attachment_id)
     return;
   var attachment_id = determineAttachmentID();
   if (!attachment_id)
     return;