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

        Reviewed by Oliver Hunt.

        Could review tool include style failure info?
        https://bugs.webkit.org/show_bug.cgi?id=49049

        This does not handle patches that have old paths (e.g. without the Source prefix).
        Although that would be fairly straightforward to workaround.

        * code-review.js:

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

diff --git a/Websites/bugs.webkit.org/ChangeLog b/Websites/bugs.webkit.org/ChangeLog
index 3e16960db251338758c07e27767d01fc0418799f..ee88f2842222c3145a34b90f8fca1db60c5a1f4b 100644 (file)
--- a/Websites/bugs.webkit.org/ChangeLog
+++ b/Websites/bugs.webkit.org/ChangeLog
@@ -1,3 +1,15 @@
+2011-01-20  Ojan Vafai  <ojan@chromium.org>
+
+        Reviewed by Oliver Hunt.
+
+        Could review tool include style failure info?
+        https://bugs.webkit.org/show_bug.cgi?id=49049
+
+        This does not handle patches that have old paths (e.g. without the Source prefix).
+        Although that would be fairly straightforward to workaround.
+
+        * code-review.js:
+
 2011-01-20  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 203f2131e08c6ae49d18e91441eaa119c975fe51..b103819594b949fb652d30910273621e2f2412d8 100644 (file)
--- a/Websites/bugs.webkit.org/PrettyPatch/PrettyPatch.rb
+++ b/Websites/bugs.webkit.org/PrettyPatch/PrettyPatch.rb
@@ -417,7 +417,7 @@ body {
 }
 </style>
 <script src="https://ajax.googleapis.com/ajax/libs/jquery/1.4.2/jquery.min.js"></script> 
-<script src="code-review.js?version=27"></script>
+<script src="code-review.js?version=28"></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 ffa251df72c6e19a78c9952283b05cb1363f1dfa..45b4bc8b9fff914011f80a4694388dd34bee2fe3 100644 (file)
--- a/Websites/bugs.webkit.org/code-review.js
+++ b/Websites/bugs.webkit.org/code-review.js
@@ -201,6 +201,33 @@
     $('#message .commentStatus').text('This patch has ' + descriptor + '.  Scroll through them with the "n" and "p" keys. ' + help_text);
   }
 
+  function scanForStyleQueueComments(text) {
+    var comments = []
+    var lines = text.split('\n');
+    for (var i = 0; i < lines.length; ++i) {
+      var parts = lines[i].match(/^([^:]+):(-?\d+):(.*)$/);
+      if (!parts)
+        continue;
+
+      var file_name = parts[1];
+      var line_number = parts[2];
+      var comment_text = parts[3].trim();
+
+      if (!file_name in files) {
+        console.log('Filename in style queue output is not in the patch: ' + file_name);
+        continue;
+      }
+
+      comments.push({
+        'author': 'StyleQueue',
+        'file_name': file_name,
+        'line_number': line_number,
+        'comment_text': comment_text
+      });
+    }
+    return comments;
+  }
+
   function scanForComments(author, text) {
     var comments = []
     var lines = text.split('\n');
@@ -290,9 +317,14 @@
         $(data).find('.bz_comment').each(function() {
           var author = $(this).find('.email').text();
           var text = $(this).find('.bz_comment_text').text();
+
           var comment_marker = '(From update of attachment ' + attachment_id + ' .details.)';
           if (text.match(comment_marker))
             $.merge(comments, scanForComments(author, text));
+
+          var style_queue_comment_marker = 'Attachment ' + attachment_id + ' .details. did not pass style-queue.'
+          if (text.match(style_queue_comment_marker))
+            $.merge(comments, scanForStyleQueueComments(text));
         });
         displayPreviousComments(comments);
       });