Deleting content at the top of prettypatch emails destroys HTML formatting
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 11 Jul 2012 16:56:18 +0000 (16:56 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 11 Jul 2012 16:56:18 +0000 (16:56 +0000)
https://bugs.webkit.org/show_bug.cgi?id=90700
<rdar://problem/7488232>

Patch by Alice Cheng <alice_cheng@apple.com> on 2012-07-11
Reviewed by David Kilzer.

* PrettyPatch/PrettyPatch.rb:

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

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

index a45d947..db56a93 100644 (file)
@@ -1,3 +1,13 @@
+2012-07-11  Alice Cheng  <alice_cheng@apple.com>
+
+        Deleting content at the top of prettypatch emails destroys HTML formatting
+        https://bugs.webkit.org/show_bug.cgi?id=90700
+        <rdar://problem/7488232>
+
+        Reviewed by David Kilzer.
+
+        * PrettyPatch/PrettyPatch.rb:
+
 2012-07-10  Adam Barth  <abarth@webkit.org>
 
         bugs.webkit.org has mixed content
index 4e43690..60cd762 100644 (file)
@@ -16,7 +16,7 @@ public
         $last_prettify_file_count = -1
         $last_prettify_part_count = { "remove" => 0, "add" => 0, "shared" => 0, "binary" => 0, "extract-error" => 0 }
         string = normalize_line_ending(string)
-        str = HEADER + "\n"
+        str = "#{HEADER}<body>\n"
 
         # Just look at the first line to see if it is an SVN revision number as added
         # by webkit-patch for git checkouts.
@@ -24,7 +24,7 @@ public
         string.each_line do |line|
             match = /^Subversion\ Revision: (\d*)$/.match(line)
             unless match.nil?
-                str += "<span class='revision'>" + match[1] + "</span>\n"
+                str << "<span class='revision'>#{match[1]}</span>\n"
                 $svn_revision = match[1].to_i;
             end
             break
@@ -33,7 +33,8 @@ public
         fileDiffs = FileDiff.parse(string)
 
         $last_prettify_file_count = fileDiffs.length
-        str += fileDiffs.collect{ |diff| diff.to_html }.join
+        str << fileDiffs.collect{ |diff| diff.to_html }.join
+        str << "</body></html>"
     end
 
     def self.filename_from_diff_header(line)
@@ -128,6 +129,8 @@ private
 
 
     HEADER =<<EOF
+<html>
+<head>
 <style>
 :link, :visited {
     text-decoration: none;
@@ -483,6 +486,7 @@ div:focus {
 </style>
 <script src="https://ajax.googleapis.com/ajax/libs/jquery/1.4.2/jquery.min.js"></script> 
 <script src="code-review.js?version=43"></script>
+</head>
 EOF
 
     def self.revisionOrDescription(string)