Unreviewed, rolling out r94554.

http://trac.webkit.org/changeset/94554
https://bugs.webkit.org/show_bug.cgi?id=67631

This patch did not quite fix the problem (Requested by
benwells on #webkit).

Patch by Sheriff Bot <webkit.review.bot@gmail.com> on 2011-09-06

* PrettyPatch/PrettyPatch.rb:
* PrettyPatch/PrettyPatch_test.rb:

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

diff --git a/Websites/bugs.webkit.org/ChangeLog b/Websites/bugs.webkit.org/ChangeLog
index 261dba8cebdb90f35a1874a254acb45d5312e553..33007b09321963e03b8710cdd4ca0aeda406a744 100644 (file)
--- a/Websites/bugs.webkit.org/ChangeLog
+++ b/Websites/bugs.webkit.org/ChangeLog
@@ -1,3 +1,15 @@
+2011-09-06  Sheriff Bot  <webkit.review.bot@gmail.com>
+
+        Unreviewed, rolling out r94554.
+        http://trac.webkit.org/changeset/94554
+        https://bugs.webkit.org/show_bug.cgi?id=67631
+
+        This patch did not quite fix the problem (Requested by
+        benwells on #webkit).
+
+        * PrettyPatch/PrettyPatch.rb:
+        * PrettyPatch/PrettyPatch_test.rb:
+
 2011-09-06  Ben Wells  <benwells@chromium.org>
 
         PrettyPatch should handle "delta" patch mechanism in git binary patches

diff --git a/Websites/bugs.webkit.org/PrettyPatch/PrettyPatch.rb b/Websites/bugs.webkit.org/PrettyPatch/PrettyPatch.rb
index 1ad120d3387812370d6b6359f0cd37eaf8f6b3d9..4652551ca41d3ac76f909e726e10022ab7311e68 100644 (file)
--- a/Websites/bugs.webkit.org/PrettyPatch/PrettyPatch.rb
+++ b/Websites/bugs.webkit.org/PrettyPatch/PrettyPatch.rb
@@ -13,7 +13,7 @@ public
 
     def self.prettify(string)
         $last_prettify_file_count = -1
-        $last_prettify_part_count = { "remove" => 0, "add" => 0, "shared" => 0, "binary" => 0 }
+        $last_prettify_part_count = { "remove" => 0, "add" => 0, "shared" => 0 }
         string = normalize_line_ending(string)
         fileDiffs = FileDiff.parse(string)
 
@@ -65,7 +65,7 @@ private
 
     GIT_BINARY_FILE_MARKER_FORMAT = /^GIT binary patch$/
 
-    GIT_BINARY_PATCH_FORMAT = /^(literal|delta) \d+$/
+    GIT_LITERAL_FORMAT = /^literal \d+$/
 
     START_OF_BINARY_DATA_FORMAT = /^[0-9a-zA-Z\+\/=]{20,}/ # Assume 20 chars without a space is base64 binary data.
 
@@ -508,7 +508,7 @@ EOF
                     @git_indexes = [$1, $2]
                 when GIT_BINARY_FILE_MARKER_FORMAT
                     @binary = true
-                    if (GIT_BINARY_PATCH_FORMAT.match(lines[i + 1]) and PrettyPatch.has_image_suffix(@filename)) then
+                    if (GIT_LITERAL_FORMAT.match(lines[i + 1]) and PrettyPatch.has_image_suffix(@filename)) then
                         @git_image = true
                         startOfSections = i + 1
                     end
@@ -585,7 +585,6 @@ EOF
                     end
                 end
             elsif @binary then
-                $last_prettify_part_count["binary"] += 1
                 str += "<span class='text'>Binary file, nothing to see here</span>"
             else
                 str += @sections.collect{ |section| section.to_html }.join("<br>\n") unless @sections.nil?

diff --git a/Websites/bugs.webkit.org/PrettyPatch/PrettyPatch_test.rb b/Websites/bugs.webkit.org/PrettyPatch/PrettyPatch_test.rb
index d3387cf4ac6583dc7ad733f94b8844aa6870aa06..8b714cb818eed95b89d89891aff0659b01c13a02 100755 (executable)
--- a/Websites/bugs.webkit.org/PrettyPatch/PrettyPatch_test.rb
+++ b/Websites/bugs.webkit.org/PrettyPatch/PrettyPatch_test.rb
@@ -26,7 +26,6 @@ class PrettyPatch_test < Test::Unit::TestCase
         80852 => ["Changes one line plus ChangeLog", 2, 2, 1, 4],
         83127 => ["Only add stuff", 2, 2, 0, 3],
         85071 => ["Adds and removes from a file plus git signature", 2, 5, 3, 9],
-        104633 => ["Delta mechanism for binary patch in git diff", 12, 3, 5, 3],
     }
 
     def get_patch_uri(id)
@@ -58,7 +57,6 @@ class PrettyPatch_test < Test::Unit::TestCase
         assert_equal(info[Info::ADD], $last_prettify_part_count["add"], "Wrong number of 'add' parts in " + description)
         assert_equal(info[Info::REMOVE], $last_prettify_part_count["remove"], "Wrong number of 'remove' parts in " + description)
         assert_equal(info[Info::SHARED], $last_prettify_part_count["shared"], "Wrong number of 'shared' parts in " + description)
-        assert_equal(0, $last_prettify_part_count["binary"], "Wrong number of 'binary' parts in " + description)
     end
 
     def test_patches