<?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>102064</bug_id>
          
          <creation_ts>2012-11-13 02:13:56 -0800</creation_ts>
          <short_desc>Meta: add initiator to CachedResourceRequest</short_desc>
          <delta_ts>2012-11-22 02:14:03 -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>Unspecified</rep_platform>
          <op_sys>Unspecified</op_sys>
          <bug_status>REOPENED</bug_status>
          <resolution></resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          <dependson>101935</dependson>
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Marja Hölttä">marja</reporter>
          <assigned_to name="Marja Hölttä">marja</assigned_to>
          <cc>abarth</cc>
    
    <cc>battre</cc>
    
    <cc>jochen</cc>
    
    <cc>simonjam</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>765411</commentid>
    <comment_count>0</comment_count>
    <who name="Marja Hölttä">marja</who>
    <bug_when>2012-11-13 02:13:56 -0800</bug_when>
    <thetext>The initiator information is needed for bug 92761 and for bug 84883.

The current plan is to split this in multiple patches:
- Add initiator to CachedResourceRequest for simple cases (not ThreadableLoaders) (bug 101935).
- Make ThreadableLoaders also set the initiator, assert that the initiator is always set.
- Add tests (make CachedResourceLoader record the requests and their initiators, read the recorded data to check that the initiators are set).</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>765845</commentid>
    <comment_count>1</comment_count>
    <who name="James Simonsen">simonjam</who>
    <bug_when>2012-11-13 11:25:44 -0800</bug_when>
    <thetext>FYI, I have plenty of initiator tests for Resource Timing. I will land them when Resource Timing goes in.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>765869</commentid>
    <comment_count>2</comment_count>
    <who name="Adam Barth">abarth</who>
    <bug_when>2012-11-13 11:46:14 -0800</bug_when>
    <thetext>&gt; FYI, I have plenty of initiator tests for Resource Timing. I will land them when Resource Timing goes in.

I&apos;d rather not block having tests on implementing Resource Timing because I don&apos;t understand how you plan to implement Resource Timing.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>773753</commentid>
    <comment_count>3</comment_count>
    <who name="Marja Hölttä">marja</who>
    <bug_when>2012-11-22 02:13:36 -0800</bug_when>
    <thetext>Then I&apos;ll  close this, and I&apos;ll add tests as part of reviving bug 92761.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>773754</commentid>
    <comment_count>4</comment_count>
    <who name="Marja Hölttä">marja</who>
    <bug_when>2012-11-22 02:14:03 -0800</bug_when>
    <thetext>What, no, not this bug.</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>