Bug 47067

Summary: REGRESSION (r68204-68242): In Mobile Me, replying or forwarding a yahoo email creates a multiple nested blockquote email in message body
Product: WebKit Reporter: Chris Petersen <c.petersen87>
Component: HTML EditingAssignee: Nobody <webkit-unassigned>
Status: NEW    
Severity: Critical CC: darin, harrison, mrowe, rniwa
Priority: P1 Keywords: InRadar, NeedsReduction
Version: 528+ (Nightly build)   
Hardware: All   
OS: OS X 10.6   
Attachments:
Description Flags
Mobile me screen shot none

Chris Petersen
Reported 2010-10-03 20:54:40 PDT
This regression first occurred in Webkit NB r68242. This issue DOESN'T happen in Safari 5.0.2 under stock 10.6.5. 1) Create a rich text message in yahoo mail and send it to your mobile me account 2) Launch Webkit NB r68242 and login into your mobile me account 3) Select this email from yahoo and click either Reply or Forward 4) After the message opens, notice the message contains a multiple blockquotes that are nested. See screen shot.
Attachments
Mobile me screen shot (73.46 KB, image/png)
2010-10-03 20:55 PDT, Chris Petersen 		no flags
Details
View All Add attachment proposed patch, testcase, etc.
Chris Petersen
Comment 1 2010-10-03 20:55:55 PDT
Created attachment 69598 [details] Mobile me screen shot
Darin Adler
Comment 2 2010-10-04 09:32:36 PDT
It’s wrong to say REGRESSION (r68242), because that claims it was that revision. We need the range from the previous nightly to this one.
Darin Adler
Comment 3 2010-10-04 09:38:24 PDT
I can’t find any check-ins in that region that are obviously at fault. We need to make a reduction and then bisect to find which change caused this.
Ryosuke Niwa
Comment 4 2010-10-04 10:13:17 PDT
In WebCore/editing, the only changesets between 68204-68242 are: http://trac.webkit.org/changeset/68240/ http://trac.webkit.org/changeset/68243/ http://trac.webkit.org/changeset/68337/ and http://trac.webkit.org/changeset/68972/ look more suspicious though. In WebCore, we have the following changesets seem to be remotely related to this: http://trac.webkit.org/changeset/68206/ http://trac.webkit.org/changeset/68227/trunk/WebCore and other changesets seem innocent: http://trac.webkit.org/log/trunk/WebCore?action=stop_on_copy&mode=stop_on_copy&rev=68242&stop_rev=68204&limit=100
Chris Petersen
Comment 5 2010-10-04 11:27:17 PDT
Sorry Darin for not adding this to the summary. I did verify the range is between r68202 and 68242.
Chris Petersen
Comment 6 2010-10-04 11:32:04 PDT
(In reply to comment #5) > Sorry Darin for not adding this to the summary. I did verify the range is between r68202 and 68242. I mean between r68204 and 68242.
Alexey Proskuryakov
Comment 7 2011-01-20 16:09:30 PST
<rdar://problem/8895566>
Alexey Proskuryakov
Comment 8 2011-03-16 12:39:27 PDT
I'm wondering if this is related to 56439.
Ryosuke Niwa
Comment 9 2011-03-16 12:43:38 PDT
(In reply to comment #8) > I'm wondering if this is related to 56439. My bug fix only fixes a crash so it's unlikely that it'll fix this bug as well.
Note You need to log in before you can comment on or make changes to this bug.