Summary: | FrameLoaderClient::objectContentType often has copied code | ||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Joe Mason <joenotcharles> | ||||||||||
Component: | Page Loading | Assignee: | Nobody <webkit-unassigned> | ||||||||||
Status: | RESOLVED FIXED | ||||||||||||
Severity: | Normal | CC: | eric, manyoso, staikos | ||||||||||
Priority: | P2 | ||||||||||||
Version: | 528+ (Nightly build) | ||||||||||||
Hardware: | PC | ||||||||||||
OS: | All | ||||||||||||
Attachments: |
|
Description
Joe Mason
2009-10-28 09:39:26 PDT
Created attachment 42037 [details]
Move shared code to FrameLoader::defaultObjectContentType
This patch moves the shared code to FrameLoader::defaultObjectContentType. The haiku and wx ports could now implement objectContentType by calling this function, but I didn't do that because I wanted to make sure I didn't make any functional changes.
Created attachment 42038 [details]
same patch, with build fix for gtk
Please make sure to watch the buildbots like a hawk when this patch lands. Committed revision 50225. Created attachment 42044 [details]
build fix for windows
Created attachment 42045 [details]
build fix for mac
Are these fixes still needed? If so, please attach them as a single patch with one ChangeLog. Otherwise please clear the review flags as this bug is closed. Thank you George! |