Bug 35628 - add a flag to WebURLResponse so we can identify multipart content
Summary: add a flag to WebURLResponse so we can identify multipart content
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 528+ (Nightly build)
Hardware: Other OS X 10.5
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-03-02 20:45 PST by Tony Chang
Modified: 2010-03-02 23:22 PST (History)
1 user (show)

See Also:


Attachments
Patch (3.69 KB, patch)
2010-03-02 20:45 PST, Tony Chang
fishd: review+
fishd: commit-queue-
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Tony Chang 2010-03-02 20:45:28 PST
add a flag to WebURLResponse so we can identify multipart content
Comment 1 Tony Chang 2010-03-02 20:45:51 PST
Created attachment 49875 [details]
Patch
Comment 2 Tony Chang 2010-03-02 20:46:51 PST
Comment on attachment 49875 [details]
Patch

This is the webkit side of http://codereview.chromium.org/661445.  This is to fix http://code.google.com/p/chromium/issues/detail?id=34350
Comment 3 Darin Fisher (:fishd, Google) 2010-03-02 22:11:47 PST
Comment on attachment 49875 [details]
Patch

> +++ b/WebCore/ChangeLog
> @@ -1,3 +1,17 @@
> +2010-03-02  Tony Chang  <tony@chromium.org>
> +
> +        Reviewed by NOBODY (OOPS!).
> +
> +        add a flag to WebURLResponse so we can identify multipart content
> +        https://bugs.webkit.org/show_bug.cgi?id=35628
> +
> +        No new tests. (OOPS!)

^^^ This oops is going to cause the commit-queue to barf.

Otherwise, r=me
Comment 4 Tony Chang 2010-03-02 22:14:03 PST
(In reply to comment #3)
> (From update of attachment 49875 [details])
> > +++ b/WebCore/ChangeLog
> > @@ -1,3 +1,17 @@
> > +2010-03-02  Tony Chang  <tony@chromium.org>
> > +
> > +        Reviewed by NOBODY (OOPS!).
> > +
> > +        add a flag to WebURLResponse so we can identify multipart content
> > +        https://bugs.webkit.org/show_bug.cgi?id=35628
> > +
> > +        No new tests. (OOPS!)
> 
> ^^^ This oops is going to cause the commit-queue to barf.
> 
> Otherwise, r=me

D'oh.  Will land manually.  Thanks for the fast review!
Comment 5 Tony Chang 2010-03-02 23:22:35 PST
Committed r55451: <http://trac.webkit.org/changeset/55451>