Bug 128410 – Add API for accessing session ephemerality from a page bundle

• Component: WebKit2
• Status: RESOLVED FIXED

[Martin Hock]

Add API for accessing session from a page or bundle.

[Martin Hock]

Created attachment 223510 [details]
patch

[Alexey Proskuryakov]

Comment on attachment 223510 [details]
patch

View in context: https://bugs.webkit.org/attachment.cgi?id=223510&action=review

I think that it is tricky to expose a session API when we don't really track sessions internally yet.

I would suggest not doing anything in UIProcess API, and exposing WKBundlePageIsUsingEphemeralSession in bundle API.

> Source/WebKit2/UIProcess/API/C/WKPage.cpp:1496
> +    return toAPI(session.release().leakRef());

An API function that returns something that should be released should have "create" or "copy" in the name, not "get".

This one would be "WKPageCopySession" (but note the above, let's not add it yet).

> Source/WebKit2/UIProcess/APISession.h:39
> +    static PassRefPtr<Session> fromID(uint64_t sessionID);

This looks like something SessionTracker would be responsible for, not Session itself.

> Source/WebKit2/WebProcess/InjectedBundle/API/c/WKBundlePage.cpp:548
> +    RefPtr<API::Session> session = API::Session::fromID(toImpl(pageRef)->sessionID());

UI process API and bundle API are different. APISession.h file is in UIProcess directory, so it cannot be used here.

You have this header indirectly included via WKSharedAPICast.h. But it shouldn't be there, as WKSession.h itself is in UIProcess. The include should be in WKAPICast.h instead.

[Martin Hock]

Created attachment 223523 [details]
expose just the bool from bundle

[Alexey Proskuryakov]

Comment on attachment 223523 [details]
expose just the bool from bundle

View in context: https://bugs.webkit.org/attachment.cgi?id=223523&action=review

> Source/WebKit2/WebProcess/WebPage/WebPage.cpp:2110
> +bool WebPage::isEphemeral() const

Why not use the same name, isUsingEphemeralSession()? "Page is ephemeral" is not a concept we have.

[Martin Hock]

Created attachment 223722 [details]
rename method

[WebKit Commit Bot]

Comment on attachment 223722 [details]
rename method

Clearing flags on attachment: 223722

Committed r163800: <http://trac.webkit.org/changeset/163800>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.