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>
 
+        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).
 
index 07800d3..90c279d 100755 (executable)
@@ -129,7 +129,7 @@ elsif ($action eq "edit")
 #if WEBKIT_CHANGES
 elsif ($action eq "review")
 {
-    edit("review");
+    prettyPatch();
 }
 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.location.search.match(/action=review/))
+    return;
   var attachment_id = determineAttachmentID();
   if (!attachment_id)
     return;