Bug 47067 - REGRESSION (r68204-68242): In Mobile Me, replying or forwarding a yahoo email creates a multiple nested blockquote email in message body
Summary: REGRESSION (r68204-68242): In Mobile Me, replying or forwarding a yahoo email...
Status: NEW
Alias: None
Product: WebKit
Classification: Unclassified
Component: HTML Editing (show other bugs)
Version: 528+ (Nightly build)
Hardware: All OS X 10.6
: P1 Critical
Assignee: Nobody
URL:
Keywords: InRadar, NeedsReduction
Depends on:
Blocks:
 
Reported: 2010-10-03 20:54 PDT by Chris Petersen
Modified: 2011-03-16 12:43 PDT (History)
4 users (show)

See Also:


Attachments
Mobile me screen shot (73.46 KB, image/png)
2010-10-03 20:55 PDT, Chris Petersen
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Chris Petersen 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.
Comment 1 Chris Petersen 2010-10-03 20:55:55 PDT
Created attachment 69598 [details]
Mobile me screen shot
Comment 2 Darin Adler 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.
Comment 3 Darin Adler 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.
Comment 4 Ryosuke Niwa 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
Comment 5 Chris Petersen 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.
Comment 6 Chris Petersen 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.
Comment 7 Alexey Proskuryakov 2011-01-20 16:09:30 PST
<rdar://problem/8895566>
Comment 8 Alexey Proskuryakov 2011-03-16 12:39:27 PDT
I'm wondering if this is related to 56439.
Comment 9 Ryosuke Niwa 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.