<?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>60017</bug_id>
          
          <creation_ts>2011-05-03 03:14:13 -0700</creation_ts>
          <short_desc>PreloadScanner should fetch iframes</short_desc>
          <delta_ts>2012-07-09 09:00:12 -0700</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>WebCore Misc.</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>All</rep_platform>
          <op_sys>All</op_sys>
          <bug_status>NEW</bug_status>
          <resolution></resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords>Performance</keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          <dependson>49246</dependson>
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Tony Gentilcore">tonyg</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>ap</cc>
    
    <cc>ddkilzer</cc>
    
    <cc>koivisto</cc>
    
    <cc>simonjam</cc>
    
    <cc>skyul</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>397027</commentid>
    <comment_count>0</comment_count>
    <who name="Tony Gentilcore">tonyg</who>
    <bug_when>2011-05-03 03:14:13 -0700</bug_when>
    <thetext>The PreloadScanner should fetch subdocuments (iframes). To do this, I believe we need to allow main resources in the memory cache, then the fix should be trivial.

We might also explore preload scanning within the subframe.

FWIW, this will also let us pass browserscope&apos;s &quot;|| Script Iframe&quot; test here: http://www.browserscope.org/?category=network</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>397145</commentid>
    <comment_count>1</comment_count>
    <who name="Alexey Proskuryakov">ap</who>
    <bug_when>2011-05-03 10:01:27 -0700</bug_when>
    <thetext>Is it common that iframes contain anything worthwhile? It seems that delaying loading them might be best.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>397208</commentid>
    <comment_count>2</comment_count>
    <who name="David Kilzer (:ddkilzer)">ddkilzer</who>
    <bug_when>2011-05-03 11:22:00 -0700</bug_when>
    <thetext>(In reply to comment #1)
&gt; Is it common that iframes contain anything worthwhile? It seems that delaying loading them might be best.

Or maybe only preload them if they&apos;re from a similar domain, or set their priority higher if they&apos;re from a similar domain?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>397359</commentid>
    <comment_count>3</comment_count>
    <who name="Antti Koivisto">koivisto</who>
    <bug_when>2011-05-03 13:56:03 -0700</bug_when>
    <thetext>(In reply to comment #1)
&gt; Is it common that iframes contain anything worthwhile? It seems that delaying loading them might be best.

Iframe content is part of the page content and we want the full content to be loaded as quickly as possible. &quot;Worthwhile&quot; or not, it doesn&apos;t help anyone if it loads slowly.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>397373</commentid>
    <comment_count>4</comment_count>
    <who name="Alexey Proskuryakov">ap</who>
    <bug_when>2011-05-03 14:07:57 -0700</bug_when>
    <thetext>Total load time is not the only metric to improve.

What I&apos;m saying is that prefetching iframes (and even more so, their subresources) has a lot of potential to significantly increase the time until the page is usable.</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>