Bug 16589 - WebCore::cookies, WebCore::setCookies and WebCore::cookiesEnabled need access to the WebCore::Document
Summary: WebCore::cookies, WebCore::setCookies and WebCore::cookiesEnabled need access...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Platform (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC OS X 10.4
: P2 Normal
Assignee: Nobody
URL:
Keywords: Qt
Depends on:
Blocks:
 
Reported: 2007-12-23 15:19 PST by Holger Freyther
Modified: 2008-01-17 03:38 PST (History)
3 users (show)

See Also:


Attachments
add WebCore::Document to the cookie functions (12.37 KB, patch)
2007-12-23 15:22 PST, Holger Freyther
mjs: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Holger Freyther 2007-12-23 15:19:02 PST
Similar to the bug #16588 we need to access the QNetworkAccessManager that the QWebPage is using to return and set the cookies. The information that is always available when calling the function is the WebCore::Document. This allows us to access the WebCore::Frame->FrameLoader->FrameLoaderClient->QWebFrame->QWebPage.
Comment 1 Holger Freyther 2007-12-23 15:22:12 PST
Created attachment 18078 [details]
add WebCore::Document to the cookie functions

This adds WebCore::Document as the first parameter to the WebCore cookie functions. This allows the Qt port to access the QNetworkAccessManager.
Comment 2 Maciej Stachowiak 2007-12-24 02:34:10 PST
Comment on attachment 18078 [details]
add WebCore::Document to the cookie functions

r=me
Comment 3 Alexey Proskuryakov 2007-12-24 07:38:51 PST
Not really disagreeing with Maciej's r+ here, but seeing Document* as a setCookie() parameter is rather confusing to me. Perhaps it would be better to pass FrameLoader or even FrameLoaderClient?
Comment 4 Darin Adler 2008-01-03 00:19:13 PST
(In reply to comment #3)
> Not really disagreeing with Maciej's r+ here, but seeing Document* as a
> setCookie() parameter is rather confusing to me. Perhaps it would be better to
> pass FrameLoader or even FrameLoaderClient?

When it comes to the Frame constellation of objects, we never want to pass around sub-objects like FrameLoader. FrameLoader is really only the loader aspect of the frame.

So you'd want to pass either Frame or Document. I think Document is the right one.
Comment 5 Darin Adler 2008-01-13 11:58:53 PST
Comment on attachment 18078 [details]
add WebCore::Document to the cookie functions

This patch no longer applies cleanly, and since it's a git patch and not Subversion, I can't easily figure out what Subversion version it's based on.

So I'm not landing it today. I would have landed it otherwise.

I know it's easier for the Qt hackers to supply git patches since you are using git, but perhaps you could do the additional work to supply them in a form that identifies the base Subversion revision. I know that's possible because the folks here at Apple using git are posting patches that don't show any sign of their git origins.
Comment 6 Simon Hausmann 2008-01-17 03:38:46 PST
Landed in revision 29566