<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugs.webkit.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.4.1"
          urlbase="https://bugs.webkit.org/"
          
          maintainer="admin@webkit.org"
>

    <bug>
          <bug_id>52077</bug_id>
          
          <creation_ts>2011-01-07 13:17:49 -0800</creation_ts>
          <short_desc>[Qt] QtWebKit always supports cookies</short_desc>
          <delta_ts>2011-01-07 13:49:06 -0800</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>New Bugs</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>All</rep_platform>
          <op_sys>All</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>INVALID</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords>Qt, QtTriaged</keywords>
          <priority>P5</priority>
          <bug_severity>Trivial</bug_severity>
          <target_milestone>---</target_milestone>
          
          <blocked>52081</blocked>
          <everconfirmed>1</everconfirmed>
          <reporter name="Benjamin Poulain">benjamin</reporter>
          <assigned_to name="Benjamin Poulain">benjamin</assigned_to>
          
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>330955</commentid>
    <comment_count>0</comment_count>
    <who name="Benjamin Poulain">benjamin</who>
    <bug_when>2011-01-07 13:17:49 -0800</bug_when>
    <thetext>The function bool cookiesEnabled(const Document* document) does not actually need to check anything, it is not possible to disable cookies with Qt.

Checking for the cookie jar was useless because one is created on the fly is none is set:

QNetworkCookieJar *QNetworkAccessManager::cookieJar() const
{
    Q_D(const QNetworkAccessManager);
    if (!d-&gt;cookieJar)
        d-&gt;createCookieJar();
    return d-&gt;cookieJar;
}

Checking for the network access manager is not useful for the same reason.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>330968</commentid>
    <comment_count>1</comment_count>
    <who name="Benjamin Poulain">benjamin</who>
    <bug_when>2011-01-07 13:26:46 -0800</bug_when>
    <thetext>Invalid. I realised my mistake while doing the autotest.

QNetworkAccessManagerPrivate::createCookieJar() creates the jar only once. You can set a null cookie jar and the current implementation of cookiesEnabled() is incorrect.

Follow up bug report soon.</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>