[webkit-reviews] review denied: [Bug 171367] Refactor SessionID to support multiple non-ephemeral (persistent) sessions : [Attachment 308352] Patch
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Thu Apr 27 00:40:24 PDT 2017
Alex Christensen <achristensen at apple.com> has denied Brady Eidson
<beidson at apple.com>'s request for review:
Bug 171367: Refactor SessionID to support multiple non-ephemeral (persistent)
sessions
https://bugs.webkit.org/show_bug.cgi?id=171367
Attachment 308352: Patch
https://bugs.webkit.org/attachment.cgi?id=308352&action=review
--- Comment #3 from Alex Christensen <achristensen at apple.com> ---
Comment on attachment 308352
--> https://bugs.webkit.org/attachment.cgi?id=308352
Patch
View in context: https://bugs.webkit.org/attachment.cgi?id=308352&action=review
> Source/WebCore/page/SessionID.cpp:37
> +static uint64_t currentPersistentID = 1;
I don't think this will work. Other processes will also need to be able to
tell whether this session is ephemeral or not, and their static variables won't
be synchronized. We should probably make a SessionID a wrapper around a bool
and a uint64_t.
More information about the webkit-reviews
mailing list