Changeset 84266 in webkit

Timestamp:

Apr 19, 2011 11:08:12 AM (13 years ago)

Author:

commit-queue@webkit.org

Message:

2011-04-19 Caio Marcelo de Oliveira Filho <​caio.oliveira@openbossa.org>

Reviewed by Adam Roben.

PrettyPatch tries to show images for deleted files
​https://bugs.webkit.org/show_bug.cgi?id=28457

When the image data is empty, instead of generating an <img> tag, generate
a text to indicate the image was deleted.

• PrettyPatch/PrettyPatch.rb:

Location:

trunk/Websites/bugs.webkit.org

Files:

• ChangeLog (modified) (1 diff)
• PrettyPatch/PrettyPatch.rb (modified) (2 diffs)

trunk/Websites/bugs.webkit.org/ChangeLog

@@ +1 @@
+2011-04-19  Caio Marcelo de Oliveira Filho  <caio.oliveira@openbossa.org>
+
+        Reviewed by Adam Roben.
+
+        PrettyPatch tries to show images for deleted files
+        https://bugs.webkit.org/show_bug.cgi?id=28457
+
+        When the image data is empty, instead of generating an <img> tag, generate
+        a text to indicate the image was deleted.
+
+        * PrettyPatch/PrettyPatch.rb:
+
 2011-04-14  Tony Chang  <tony@chromium.org>
 

trunk/Websites/bugs.webkit.org/PrettyPatch/PrettyPatch.rb

@@ -515 +515 @@
             lines_with_contents = lines[startOfSections...lines.length]
             @sections = DiffSection.parse(lines_with_contents) unless @binary
-            if @image
+            if @image and not lines_with_contents.empty?
                 @image_url = "data:image/png;base64," + lines_with_contents.join
                 @image_checksum = FileDiff.read_checksum_from_png(lines_with_contents.join.unpack("m").join)
@@ -545 +545 @@
         end
 
+        def image_to_html
+            if not @image_url then
+                return "<span class='text'>Image file removed</span>"
+            end
+            image_snippet = "<img class='image' src='" + @image_url + "' />"
+            if not @image_checksum then
+                return image_snippet
+            end
+            return "<p>" + @image_checksum + "</p>" + image_snippet
+        end
+
         def to_html
             str = "<div class='FileDiff'>\n"
             str += "<h1>#{PrettyPatch.linkifyFilename(@filename)}</h1>\n"
             if @image then
-                if @image_checksum then
-                    str += "<p>" + @image_checksum + "</p>"
-                end
-                str += "<img class='image' src='" + @image_url + "' />"
+                str += self.image_to_html
             elsif @git_image then
                 if @image_error