[SCM] WebKit Debian packaging branch, debian/experimental, updated. upstream/1.3.3-9427-gc2be6fc

[abarth at webkit.org]

The following commit has been merged in the debian/experimental branch:
commit 6fc572a614e532b3e9a69b144d58eac97f463133
Author: abarth at webkit.org <abarth at webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Date:   Thu Sep 23 04:19:47 2010 +0000

    2010-09-22  Adam Barth  <abarth at webkit.org>
    
            Reviewed by John Sullivan.
    
            New review-page design doesn't include name of reviewer
            https://bugs.webkit.org/show_bug.cgi?id=46271
    
            * PrettyPatch/PrettyPatch.rb:
            * code-review.js:
    
    
    git-svn-id: http://svn.webkit.org/repository/webkit/trunk@68120 268f45cc-cd09-0410-ab3c-d52691b4dbfc

diff --git a/BugsSite/ChangeLog b/BugsSite/ChangeLog
index 44cea2e..cfca68c 100644
--- a/BugsSite/ChangeLog
+++ b/BugsSite/ChangeLog
@@ -1,5 +1,15 @@
 2010-09-22  Adam Barth  <abarth at webkit.org>
 
+        Reviewed by John Sullivan.
+
+        New review-page design doesn't include name of reviewer
+        https://bugs.webkit.org/show_bug.cgi?id=46271
+
+        * PrettyPatch/PrettyPatch.rb:
+        * code-review.js:
+
+2010-09-22  Adam Barth  <abarth at webkit.org>
+
         Fix typo in CSS.
 
         * PrettyPatch/PrettyPatch.rb:
diff --git a/BugsSite/PrettyPatch/PrettyPatch.rb b/BugsSite/PrettyPatch/PrettyPatch.rb
index 67daa00..0e7c3ae 100644
--- a/BugsSite/PrettyPatch/PrettyPatch.rb
+++ b/BugsSite/PrettyPatch/PrettyPatch.rb
@@ -312,7 +312,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=12"></script> 
+<script src="code-review.js?version=13"></script> 
 EOF
 
     def self.revisionOrDescription(string)
diff --git a/BugsSite/code-review.js b/BugsSite/code-review.js
index 8339d61..4064c89 100644
--- a/BugsSite/code-review.js
+++ b/BugsSite/code-review.js
@@ -218,7 +218,15 @@
       $('<span class="commitQueue"> cq: ' + flag_control + '</span>'));
 
     details.find('#flags select').each(function() {
-      findControlForFlag(this).attr('selectedIndex', $(this).attr('selectedIndex'));
+      var requestee = $(this).parent().siblings('td:first-child').text().trim();
+      if (requestee.length) {
+        // Remove trailing ':'.
+        requestee = requestee.substr(0, requestee.length - 1);
+        requestee = ' (' + requestee + ')';
+      }
+      var control = findControlForFlag(this)
+      control.attr('selectedIndex', $(this).attr('selectedIndex'));
+      control.parent().prepend(requestee);
     });
   }
 

-- 
WebKit Debian packaging