Bug 128465

Summary:

Remove outdated workarounds in DocumentLoader::dataReceived

Product:

WebKit

Reporter:

mitz

Component:

Page Loading

Assignee:

mitz

Status:

RESOLVED FIXED

Severity:

Normal

CC:

andersca, beidson, commit-queue, japhet, kling

Priority:

Version:

528+ (Nightly build)

Hardware:

Unspecified

OS:

Unspecified

Attachments:

Description	Flags
Remove outdated code	kling: review+

Description mitz 2014-02-08 12:18:50 PST

DocumentLoader::dataReceived includes workarounds for CFNetwork bugs that are fixed in all platforms we target. We should remove them.

Comment 1 mitz 2014-02-08 14:26:51 PST

Created attachment 223585 [details]
Remove outdated code

Comment 2 Andreas Kling 2014-02-08 17:46:05 PST

Comment on attachment 223585 [details]
Remove outdated code

Jawesome!

Comment 3 mitz 2014-02-08 18:32:54 PST

Committed <http://trac.webkit.org/r163734>.

Comment 4 Andreas Kling 2014-02-08 20:16:42 PST

That assertion started firing on Mavericks/WK2:

http://build.webkit.org/results/Apple%20Mavericks%20Debug%20WK2%20(Tests)/r163734%20(2405)/loader/load-defer-resume-crash-crash-log.txt

Comment 5 mitz 2014-02-08 20:26:48 PST

(In reply to comment #4)
> That assertion started firing on Mavericks/WK2:
> 
> http://build.webkit.org/results/Apple%20Mavericks%20Debug%20WK2%20(Tests)/r163734%20(2405)/loader/load-defer-resume-crash-crash-log.txt

Reverted that piece in r163738 :-(