Bug 90700 – Deleting content at the top of prettypatch emails destroys HTML formatting

• Component: WebKit Website
• Status: RESOLVED FIXED

[Alice Cheng]

When replying to a prettypatch email, deleting any text in a <pre></pre> block at the beginning causes the HTML formatting of the patch below to be destroyed.

 STEPS TO REPRODUCE
1. Open attached pretty patch email
2. Hit "Reply" button.
3. Highlight <pre></pre> text (as seen in the attached movie) so that the last blank line above the patch content is included.
4. Hit Delete key to delete it.

 RESULTS
The formatting of prettypatch contents is lost.

[Alice Cheng]

<rdar://problem/7488232>

[Alice Cheng]

Created attachment 151113 [details]
patch

A patch that fix pretty patch for now.
Will be working on an API that moves all style elements from body to head.

[WebKit Review Bot]

Attachment 151113 [details] did not pass style-queue:

Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Websites/bugs.webkit.org/ChangeLog', u'Web..." exit_code: 1
Websites/bugs.webkit.org/ChangeLog:1:  ChangeLog entry has no bug number  [changelog/bugnumber] [5]
Total errors found: 1 in 2 files


If any of these errors are false positives, please file a bug against check-webkit-style.

[Alice Cheng]

Created attachment 151114 [details]
patch

A patch that fix pretty patch for now.
Will be working on an API that moves all style elements from body to head.

Added bug number in ChangeLog

[David Kilzer (:ddkilzer)]

Comment on attachment 151114 [details]
patch

r=me

[Alice Cheng]

Created attachment 151364 [details]
patch

submit for commit queue

[WebKit Review Bot]

Comment on attachment 151114 [details]
patch

Rejecting attachment 151114 [details] from commit-queue.

[e-mail] does not have committer permissions according to http://trac.webkit.org/browser/trunk/Tools/Scripts/webkitpy/common/config/committers.py.

- If you do not have committer rights please read http://webkit.org/coding/contributing.html for instructions on how to use bugzilla flags.

- If you have committer rights please correct the error in Tools/Scripts/webkitpy/common/config/committers.py by adding yourself to the file (no review needed).  The commit-queue restarts itself every 2 hours.  After restart the commit-queue will correctly respect your committer rights.

[WebKit Review Bot]

Comment on attachment 151114 [details]
patch

Clearing flags on attachment: 151114

Committed r122338: <http://trac.webkit.org/changeset/122338>

[WebKit Review Bot]

All reviewed patches have been landed.  Closing bug.