<?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>114888</bug_id>
          
          <creation_ts>2013-04-19 13:37:01 -0700</creation_ts>
          <short_desc>Web Inspector: Support the SourceMap header, X-SourceMap was deprecated</short_desc>
          <delta_ts>2013-04-19 16:00:51 -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>Web Inspector (Deprecated)</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>All</rep_platform>
          <op_sys>All</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</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>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Joseph Pecoraro">joepeck</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>graouts</cc>
    
    <cc>joepeck</cc>
    
    <cc>timothy</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>878632</commentid>
    <comment_count>0</comment_count>
    <who name="Joseph Pecoraro">joepeck</who>
    <bug_when>2013-04-19 13:37:01 -0700</bug_when>
    <thetext>The SourceMap spec mentions that X-SourceMap was deprecated, we should support the expected header:
&lt;https://docs.google.com/document/d/1U1RGAehQwRypUTovF1KRlpiOFze0b-_2gc6fAH0KY0k/edit&gt;

&gt;   The HTTP header should supply the source map URL reference as:
&gt;    
&gt;       SourceMap: &lt;url&gt;
&gt;   
&gt;   Note: previous revisions of this document recommended a header name of “X-SourceMap”. This is now deprecated; “SourceMap” is now expected.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>878637</commentid>
    <comment_count>1</comment_count>
      <attachid>198914</attachid>
    <who name="Joseph Pecoraro">joepeck</who>
    <bug_when>2013-04-19 13:53:04 -0700</bug_when>
    <thetext>Created attachment 198914
[PATCH] Proposed Fix</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>878665</commentid>
    <comment_count>2</comment_count>
    <who name="Joseph Pecoraro">joepeck</who>
    <bug_when>2013-04-19 16:00:51 -0700</bug_when>
    <thetext>Committed r148769:
&lt;http://trac.webkit.org/changeset/148769&gt;</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>198914</attachid>
            <date>2013-04-19 13:53:04 -0700</date>
            <delta_ts>2013-04-19 13:57:09 -0700</delta_ts>
            <desc>[PATCH] Proposed Fix</desc>
            <filename>header-1.patch</filename>
            <type>text/plain</type>
            <size>3692</size>
            <attacher name="Joseph Pecoraro">joepeck</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZyBiL1NvdXJjZS9XZWJDb3JlL0No
YW5nZUxvZwppbmRleCBjOTFkZTFmLi5kODRhNzhjIDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViQ29y
ZS9DaGFuZ2VMb2cKKysrIGIvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCkBAIC0xLDUgKzEsMjEg
QEAKIDIwMTMtMDQtMTkgIEpvc2VwaCBQZWNvcmFybyAgPHBlY29yYXJvQGFwcGxlLmNvbT4KIAor
ICAgICAgICBXZWIgSW5zcGVjdG9yOiBTdXBwb3J0IHRoZSBTb3VyY2VNYXAgaGVhZGVyLCBYLVNv
dXJjZU1hcCB3YXMgZGVwcmVjYXRlZAorICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9z
aG93X2J1Zy5jZ2k/aWQ9MTE0ODg4CisKKyAgICAgICAgQ2hlY2sgZmlyc3QgZm9yIFNvdXJjZU1h
cCwgdGhlbiBmYWxsYmFjayB0byBYLVNvdXJjZU1hcC4gTGVhdmluZyBpbgorICAgICAgICBzdXBw
b3J0IGZvciB0aGUgZGVwcmVjYXRlZCBoZWFkZXIgYmVjYXVzZSBtb3N0IHRvb2xzIGFuZCBhcnRp
Y2xlcworICAgICAgICBvbmxpbmUgbWVudGlvbiB0aGF0IHZlcnNpb24gYW5kIGhhdmUgbm90IHll
dCB1cGRhdGVkLgorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAg
ICAgICogaW5zcGVjdG9yL0luc3BlY3RvckRlYnVnZ2VyQWdlbnQuY3BwOgorICAgICAgICAoV2Vi
Q29yZTo6SW5zcGVjdG9yRGVidWdnZXJBZ2VudDo6c291cmNlTWFwVVJMRm9yU2NyaXB0KToKKyAg
ICAgICAgKiBpbnNwZWN0b3IvSW5zcGVjdG9yUGFnZUFnZW50LmNwcDoKKyAgICAgICAgKFdlYkNv
cmU6Okluc3BlY3RvclBhZ2VBZ2VudDo6c291cmNlTWFwVVJMRm9yUmVzb3VyY2UpOgorCisyMDEz
LTA0LTE5ICBKb3NlcGggUGVjb3Jhcm8gIDxwZWNvcmFyb0BhcHBsZS5jb20+CisKICAgICAgICAg
V2ViIEluc3BlY3RvcjogQmFja2VuZCBzaG91bGQgZGV0ZWN0IHNvdXJjZU1hcHBpbmdVUkxzIGlu
IENTUyBSZXNvdXJjZXMKICAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcu
Y2dpP2lkPTExNDg1NAogCmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9pbnNwZWN0b3IvSW5z
cGVjdG9yRGVidWdnZXJBZ2VudC5jcHAgYi9Tb3VyY2UvV2ViQ29yZS9pbnNwZWN0b3IvSW5zcGVj
dG9yRGVidWdnZXJBZ2VudC5jcHAKaW5kZXggYWUwODMwNy4uNThlZWY3NSAxMDA2NDQKLS0tIGEv
U291cmNlL1dlYkNvcmUvaW5zcGVjdG9yL0luc3BlY3RvckRlYnVnZ2VyQWdlbnQuY3BwCisrKyBi
L1NvdXJjZS9XZWJDb3JlL2luc3BlY3Rvci9JbnNwZWN0b3JEZWJ1Z2dlckFnZW50LmNwcApAQCAt
NjMyLDcgKzYzMiw4IEBAIFBhc3NSZWZQdHI8QXJyYXk8VHlwZUJ1aWxkZXI6OkRlYnVnZ2VyOjpD
YWxsRnJhbWU+ID4gSW5zcGVjdG9yRGVidWdnZXJBZ2VudDo6Y3VyCiAKIFN0cmluZyBJbnNwZWN0
b3JEZWJ1Z2dlckFnZW50Ojpzb3VyY2VNYXBVUkxGb3JTY3JpcHQoY29uc3QgU2NyaXB0JiBzY3Jp
cHQpCiB7Ci0gICAgREVGSU5FX1NUQVRJQ19MT0NBTChTdHJpbmcsIHNvdXJjZU1hcEhUVFBIZWFk
ZXIsIChBU0NJSUxpdGVyYWwoIlgtU291cmNlTWFwIikpKTsKKyAgICBERUZJTkVfU1RBVElDX0xP
Q0FMKFN0cmluZywgc291cmNlTWFwSFRUUEhlYWRlciwgKEFTQ0lJTGl0ZXJhbCgiU291cmNlTWFw
IikpKTsKKyAgICBERUZJTkVfU1RBVElDX0xPQ0FMKFN0cmluZywgc291cmNlTWFwSFRUUEhlYWRl
ckRlcHJlY2F0ZWQsIChBU0NJSUxpdGVyYWwoIlgtU291cmNlTWFwIikpKTsKIAogICAgIGlmICgh
c2NyaXB0LnVybC5pc0VtcHR5KCkpIHsKICAgICAgICAgaWYgKEluc3BlY3RvclBhZ2VBZ2VudCog
cGFnZUFnZW50ID0gbV9pbnN0cnVtZW50aW5nQWdlbnRzLT5pbnNwZWN0b3JQYWdlQWdlbnQoKSkg
ewpAQCAtNjQxLDYgKzY0MiwxMCBAQCBTdHJpbmcgSW5zcGVjdG9yRGVidWdnZXJBZ2VudDo6c291
cmNlTWFwVVJMRm9yU2NyaXB0KGNvbnN0IFNjcmlwdCYgc2NyaXB0KQogICAgICAgICAgICAgICAg
IFN0cmluZyBzb3VyY2VNYXBIZWFkZXIgPSByZXNvdXJjZS0+cmVzcG9uc2UoKS5odHRwSGVhZGVy
RmllbGQoc291cmNlTWFwSFRUUEhlYWRlcik7CiAgICAgICAgICAgICAgICAgaWYgKCFzb3VyY2VN
YXBIZWFkZXIuaXNFbXB0eSgpKQogICAgICAgICAgICAgICAgICAgICByZXR1cm4gc291cmNlTWFw
SGVhZGVyOworCisgICAgICAgICAgICAgICAgc291cmNlTWFwSGVhZGVyID0gcmVzb3VyY2UtPnJl
c3BvbnNlKCkuaHR0cEhlYWRlckZpZWxkKHNvdXJjZU1hcEhUVFBIZWFkZXJEZXByZWNhdGVkKTsK
KyAgICAgICAgICAgICAgICBpZiAoIXNvdXJjZU1hcEhlYWRlci5pc0VtcHR5KCkpCisgICAgICAg
ICAgICAgICAgICAgIHJldHVybiBzb3VyY2VNYXBIZWFkZXI7ICAgICAgICAgICAgICAgIAogICAg
ICAgICAgICAgfQogICAgICAgICB9CiAgICAgfQpkaWZmIC0tZ2l0IGEvU291cmNlL1dlYkNvcmUv
aW5zcGVjdG9yL0luc3BlY3RvclBhZ2VBZ2VudC5jcHAgYi9Tb3VyY2UvV2ViQ29yZS9pbnNwZWN0
b3IvSW5zcGVjdG9yUGFnZUFnZW50LmNwcAppbmRleCAxYzdhOTgwLi4wZDgwNTkwIDEwMDY0NAot
LS0gYS9Tb3VyY2UvV2ViQ29yZS9pbnNwZWN0b3IvSW5zcGVjdG9yUGFnZUFnZW50LmNwcAorKysg
Yi9Tb3VyY2UvV2ViQ29yZS9pbnNwZWN0b3IvSW5zcGVjdG9yUGFnZUFnZW50LmNwcApAQCAtMjcw
LDcgKzI3MCw4IEBAIHZvaWQgSW5zcGVjdG9yUGFnZUFnZW50OjpyZXNvdXJjZUNvbnRlbnQoRXJy
b3JTdHJpbmcqIGVycm9yU3RyaW5nLCBGcmFtZSogZnJhbWUsCiAvL3N0YXRpYwogU3RyaW5nIElu
c3BlY3RvclBhZ2VBZ2VudDo6c291cmNlTWFwVVJMRm9yUmVzb3VyY2UoQ2FjaGVkUmVzb3VyY2Uq
IGNhY2hlZFJlc291cmNlKQogewotICAgIERFRklORV9TVEFUSUNfTE9DQUwoU3RyaW5nLCBzb3Vy
Y2VNYXBIVFRQSGVhZGVyLCAoQVNDSUlMaXRlcmFsKCJYLVNvdXJjZU1hcCIpKSk7CisgICAgREVG
SU5FX1NUQVRJQ19MT0NBTChTdHJpbmcsIHNvdXJjZU1hcEhUVFBIZWFkZXIsIChBU0NJSUxpdGVy
YWwoIlNvdXJjZU1hcCIpKSk7CisgICAgREVGSU5FX1NUQVRJQ19MT0NBTChTdHJpbmcsIHNvdXJj
ZU1hcEhUVFBIZWFkZXJEZXByZWNhdGVkLCAoQVNDSUlMaXRlcmFsKCJYLVNvdXJjZU1hcCIpKSk7
CiAKICAgICBpZiAoIWNhY2hlZFJlc291cmNlKQogICAgICAgICByZXR1cm4gU3RyaW5nKCk7CkBA
IC0yODMsNiArMjg0LDEwIEBAIFN0cmluZyBJbnNwZWN0b3JQYWdlQWdlbnQ6OnNvdXJjZU1hcFVS
TEZvclJlc291cmNlKENhY2hlZFJlc291cmNlKiBjYWNoZWRSZXNvdXJjCiAgICAgaWYgKCFzb3Vy
Y2VNYXBIZWFkZXIuaXNFbXB0eSgpKQogICAgICAgICByZXR1cm4gc291cmNlTWFwSGVhZGVyOwog
CisgICAgc291cmNlTWFwSGVhZGVyID0gY2FjaGVkUmVzb3VyY2UtPnJlc3BvbnNlKCkuaHR0cEhl
YWRlckZpZWxkKHNvdXJjZU1hcEhUVFBIZWFkZXJEZXByZWNhdGVkKTsKKyAgICBpZiAoIXNvdXJj
ZU1hcEhlYWRlci5pc0VtcHR5KCkpCisgICAgICAgIHJldHVybiBzb3VyY2VNYXBIZWFkZXI7CisK
ICAgICBTdHJpbmcgY29udGVudDsKICAgICBib29sIGJhc2U2NEVuY29kZWQ7CiAgICAgaWYgKElu
c3BlY3RvclBhZ2VBZ2VudDo6Y2FjaGVkUmVzb3VyY2VDb250ZW50KGNhY2hlZFJlc291cmNlLCAm
Y29udGVudCwgJmJhc2U2NEVuY29kZWQpICYmICFiYXNlNjRFbmNvZGVkKQo=
</data>
<flag name="review"
          id="220592"
          type_id="1"
          status="+"
          setter="timothy"
    />
    <flag name="commit-queue"
          id="220593"
          type_id="3"
          status="-"
          setter="joepeck"
    />
          </attachment>
      

    </bug>

</bugzilla>