<html>
    <head>
      <base href="https://bugs.webkit.org/">
    </head>
    <body>
      <p>
        <div>
            <b><a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - Potential privacy issue: DNS prefetching can be re-enabled"
   href="https://bugs.webkit.org/show_bug.cgi?id=182924#c7">Comment # 7</a>
              on <a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - Potential privacy issue: DNS prefetching can be re-enabled"
   href="https://bugs.webkit.org/show_bug.cgi?id=182924">bug 182924</a>
              from <span class="vcard"><a class="email" href="mailto:mcrha@redhat.com" title="Milan Crha <mcrha@redhat.com>"> <span class="fn">Milan Crha</span></a>
</span></b>
        <pre>(In reply to Chris Dumez from <a href="show_bug.cgi?id=182924#c6">comment #6</a>)
<span class="quote">> I'd rather we return early in this method if
> !settings().dnsPrefetchingEnabled(). Otherwise, we end up changing the value
> of m_haveExplicitlyDisabledDNSPrefetch.</span >

I've been thinking of it too, but:
a) I wanted ensure that the prefetch will be disabled
b) that variable is used basically only here (and in initDNSPrefetch(), where
   it's set to 'false' again) and after this change is means "explicitly
   disabled by settings" here, from my point of view.

I added some debug prints around and the initDNSPrefetch() is called plenty of times after each page reload (or load of a new page in case of Evolution), thus the value is reset often.</pre>
        </div>
      </p>


      <hr>
      <span>You are receiving this mail because:</span>

      <ul>
          <li>You are the assignee for the bug.</li>
      </ul>
    </body>
</html>