Bug 32559 – [Android] Android needs implementation of setCookies, cookies and cookiesEnabled functions.

• Component: WebCore Misc.
• Status: RESOLVED FIXED

[Andrei Popescu]

Android needs implementation of  setCookies, cookies and cookiesEnabled functions. The cookie functionality is on the Java side so we need to proxy to it via the PlatformBridge iface.

[Andrei Popescu]

Created attachment 44889 [details]
Add implementation of cookie functions, AuthenticationChallenge and fix a small bug.

[WebKit Review Bot]

Attachment 44889 [details] did not pass style-queue:

Failed to run "WebKitTools/Scripts/check-webkit-style" exit_code: 1
WebCore/platform/network/android/Cookie.cpp:28:  Found other header before a header this file implements. Should be: config.h, primary header, blank line, and then alphabetically sorted.  [build/include_order] [4]
Total errors found: 1

[Darin Fisher (:fishd, Google)]

Comment on attachment 44889 [details]
Add implementation of cookie functions, AuthenticationChallenge and fix a small bug.

> Index: WebCore/platform/network/android/AuthenticationChallenge.h
...
> +class ResourceHandle;

^^^ This forward declaration is unnecessary.


> Index: WebCore/platform/network/android/Cookie.cpp

^^^ This file should be named CookieJarAndroid.cpp to stick with conventions.


> +#include "config.h"
> +

^^^ please include CookieJar.h here.


> +#include "PlatformBridge.h"
> +
> +namespace WebCore {
> +
> +class Document;

^^^ no need for Document forward declare if you include CookieJar.h

[Andrei Popescu]

Created attachment 44894 [details]
Add implementation of cookie functions, AuthenticationChallenge and fix a small bug.

[Andrei Popescu]

(In reply to comment #3)
> (From update of attachment 44889 [details])
> > Index: WebCore/platform/network/android/AuthenticationChallenge.h
> ...
> > +class ResourceHandle;
> 
> ^^^ This forward declaration is unnecessary.
> 

Removed.

> 
> > Index: WebCore/platform/network/android/Cookie.cpp
> 
> ^^^ This file should be named CookieJarAndroid.cpp to stick with conventions.
>

Done.
 
> 
> > +#include "config.h"
> > +
> 
> ^^^ please include CookieJar.h here.
>

Included.
 
> 
> > +#include "PlatformBridge.h"
> > +
> > +namespace WebCore {
> > +
> > +class Document;
> 
> ^^^ no need for Document forward declare if you include CookieJar.h

Removed.

Thanks,
Andrei

[Andrei Popescu]

But forgot the PlatformBridge.h changed in the new patch. Uploading new patch...

[WebKit Review Bot]

style-queue ran check-webkit-style on attachment 44894 [details] without any errors.

[Andrei Popescu]

Created attachment 44895 [details]
Add implementation of cookie functions, AuthenticationChallenge and fix a small bug.

[WebKit Review Bot]

style-queue ran check-webkit-style on attachment 44895 [details] without any errors.

[Andrei Popescu]

ping?

[Adam Barth]

Comment on attachment 44895 [details]
Add implementation of cookie functions, AuthenticationChallenge and fix a small bug.

I guess you're not implementing third-party cookie blocking?  If you decide to do that later, you'll need to revise this code.

[WebKit Commit Bot]

Comment on attachment 44895 [details]
Add implementation of cookie functions, AuthenticationChallenge and fix a small bug.

Clearing flags on attachment: 44895

Committed r52406: <http://trac.webkit.org/changeset/52406>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.