<?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>16692</bug_id>
          
          <creation_ts>2008-01-01 01:26:19 -0800</creation_ts>
          <short_desc>Acid3 expects \u escapes to work in data: urls</short_desc>
          <delta_ts>2008-11-28 12:25:35 -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>Mac</rep_platform>
          <op_sys>OS X 10.4</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="Eric Seidel (no email)">eric</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>kevin</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>65878</commentid>
    <comment_count>0</comment_count>
    <who name="Eric Seidel (no email)">eric</who>
    <bug_when>2008-01-01 01:26:19 -0800</bug_when>
    <thetext>Acid3 expects \u escapes to work in data: urls

We seem to be escaping \&apos;s as /&apos;s before trying \u escapes.

See test case.  This should be an easy fix.

Firefox and opera pass this test, I have not tried WinIE.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>65879</commentid>
    <comment_count>1</comment_count>
      <attachid>18222</attachid>
    <who name="Eric Seidel (no email)">eric</who>
    <bug_when>2008-01-01 01:26:36 -0800</bug_when>
    <thetext>Created attachment 18222
test case</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>65886</commentid>
    <comment_count>2</comment_count>
    <who name="Eric Seidel (no email)">eric</who>
    <bug_when>2008-01-01 02:11:14 -0800</bug_when>
    <thetext>I expect this is the problem right here:

KURL::init()

    // for compatibility with Win IE, we must treat backslashes as if they were slashes, as long as we&apos;re not dealing with the javascript: schema
    DeprecatedString substitutedRelative;
    bool shouldSubstituteBackslashes = relative.contains(&apos;\\&apos;) &amp;&amp; !relative.startsWith(&quot;javascript:&quot;, false);
    if (shouldSubstituteBackslashes) {
        substitutedRelative = substituteBackslashes(relative);
    }

That check is obviously failing (or not getting hit for this code path).</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>65887</commentid>
    <comment_count>3</comment_count>
      <attachid>18223</attachid>
    <who name="Eric Seidel (no email)">eric</who>
    <bug_when>2008-01-01 02:32:00 -0800</bug_when>
    <thetext>Created attachment 18223
Don&apos;t replace \ with / in data: urls

 LayoutTests/ChangeLog                              |   10 ++++++++++
 .../loader/url-data-replace-backslash-expected.txt |    1 +
 .../fast/loader/url-data-replace-backslash.html    |   13 +++++++++++++
 WebCore/ChangeLog                                  |   12 ++++++++++++
 WebCore/platform/KURL.cpp                          |    5 ++---
 5 files changed, 38 insertions(+), 3 deletions(-)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>65888</commentid>
    <comment_count>4</comment_count>
      <attachid>18223</attachid>
    <who name="Alexey Proskuryakov">ap</who>
    <bug_when>2008-01-01 02:37:26 -0800</bug_when>
    <thetext>Comment on attachment 18223
Don&apos;t replace \ with / in data: urls

r=me

\ No newline at end of file

Please add one :)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>65889</commentid>
    <comment_count>5</comment_count>
    <who name="Eric Seidel (no email)">eric</who>
    <bug_when>2008-01-01 02:41:28 -0800</bug_when>
    <thetext>r29061</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>100618</commentid>
    <comment_count>6</comment_count>
    <who name="Alexey Proskuryakov">ap</who>
    <bug_when>2008-11-28 12:25:35 -0800</bug_when>
    <thetext>*** Bug 11464 has been marked as a duplicate of this bug. ***</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="0"
              isprivate="0"
          >
            <attachid>18222</attachid>
            <date>2008-01-01 01:26:36 -0800</date>
            <delta_ts>2008-01-01 01:26:36 -0800</delta_ts>
            <desc>test case</desc>
            <filename>data.html</filename>
            <type>text/html</type>
            <size>281</size>
            <attacher name="Eric Seidel (no email)">eric</attacher>
            
              <data encoding="base64">PHNjcmlwdD4KdmFyIGQ1ID0gImZhaWwiOwo8L3NjcmlwdD4KPHNjcmlwdCB0eXBlPSJ0ZXh0L2ph
dmFzY3JpcHQiIHNyYz0iZGF0YTp0ZXh0L2phdmFzY3JpcHQsZDUlMjAlM0QlMjAnZml2ZVx1MDAy
N3MnJTNCIj48L3NjcmlwdD4KPHNjcmlwdD4KaWYgKGQ1ID09ICJmaXZlJ3MiKSB7CiAgZG9jdW1l
bnQud3JpdGUoIlBBU1MiKTsKfSBlbHNlIHsKICBkb2N1bWVudC53cml0ZSgiRkFJTDogZDUgd2Fz
OiAiICsgZDUgKyAiIHNob3VsZCBoYXZlIGJlZW4gOiBmaXZlJ3MiKTsKfQo8L3NjcmlwdD4=
</data>

          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>18223</attachid>
            <date>2008-01-01 02:32:00 -0800</date>
            <delta_ts>2008-01-01 02:37:26 -0800</delta_ts>
            <desc>Don&apos;t replace \ with / in data: urls</desc>
            <filename>Don-t-replace-with-in-data-urls.patch</filename>
            <type>text/plain</type>
            <size>3099</size>
            <attacher name="Eric Seidel (no email)">eric</attacher>
            
              <data encoding="base64">Y2U2M2M5YmYxZmNjZDJmYTVjZmRmNGRlYmRjZDc1ZDM5MGI0ZGFkZgpkaWZmIC0tZ2l0IGEvTGF5
b3V0VGVzdHMvQ2hhbmdlTG9nIGIvTGF5b3V0VGVzdHMvQ2hhbmdlTG9nCmluZGV4IGM0NDZhYjIu
LjllZjQ5MWIgMTAwNjQ0Ci0tLSBhL0xheW91dFRlc3RzL0NoYW5nZUxvZworKysgYi9MYXlvdXRU
ZXN0cy9DaGFuZ2VMb2cKQEAgLTEsMyArMSwxMyBAQAorMjAwOC0wMS0wMSAgRXJpYyBTZWlkZWwg
IDxlcmljQHdlYmtpdC5vcmc+CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISku
CisgICAgICAgIAorICAgICAgICBEb24ndCByZXBsYWNlIFwgd2l0aCAvIGluIGRhdGE6IHVybHMK
KyAgICAgICAgaHR0cDovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9MTY2OTIKKwor
ICAgICAgICAqIGZhc3QvbG9hZGVyL3VybC1kYXRhLXJlcGxhY2UtYmFja3NsYXNoLWV4cGVjdGVk
LnR4dDogQWRkZWQuCisgICAgICAgICogZmFzdC9sb2FkZXIvdXJsLWRhdGEtcmVwbGFjZS1iYWNr
c2xhc2guaHRtbDogQWRkZWQuCisKIDIwMDctMTItMzEgIFNhbSBXZWluaWcgIDxzYW1Ad2Via2l0
Lm9yZz4KIAogICAgICAgICBSZXZpZXdlZCBieSBEYXJpbi4KZGlmZiAtLWdpdCBhL0xheW91dFRl
c3RzL2Zhc3QvbG9hZGVyL3VybC1kYXRhLXJlcGxhY2UtYmFja3NsYXNoLWV4cGVjdGVkLnR4dCBi
L0xheW91dFRlc3RzL2Zhc3QvbG9hZGVyL3VybC1kYXRhLXJlcGxhY2UtYmFja3NsYXNoLWV4cGVj
dGVkLnR4dApuZXcgZmlsZSBtb2RlIDEwMDY0NAppbmRleCAwMDAwMDAwLi43ZWYyMmU5Ci0tLSAv
ZGV2L251bGwKKysrIGIvTGF5b3V0VGVzdHMvZmFzdC9sb2FkZXIvdXJsLWRhdGEtcmVwbGFjZS1i
YWNrc2xhc2gtZXhwZWN0ZWQudHh0CkBAIC0wLDAgKzEgQEAKK1BBU1MKZGlmZiAtLWdpdCBhL0xh
eW91dFRlc3RzL2Zhc3QvbG9hZGVyL3VybC1kYXRhLXJlcGxhY2UtYmFja3NsYXNoLmh0bWwgYi9M
YXlvdXRUZXN0cy9mYXN0L2xvYWRlci91cmwtZGF0YS1yZXBsYWNlLWJhY2tzbGFzaC5odG1sCm5l
dyBmaWxlIG1vZGUgMTAwNjQ0CmluZGV4IDAwMDAwMDAuLmVlYjMwMGUKLS0tIC9kZXYvbnVsbAor
KysgYi9MYXlvdXRUZXN0cy9mYXN0L2xvYWRlci91cmwtZGF0YS1yZXBsYWNlLWJhY2tzbGFzaC5o
dG1sCkBAIC0wLDAgKzEsMTMgQEAKKzxzY3JpcHQ+Cit2YXIgZDUgPSAiZmFpbCI7Cis8L3Njcmlw
dD4KKzxzY3JpcHQgdHlwZT0idGV4dC9qYXZhc2NyaXB0IiBzcmM9ImRhdGE6dGV4dC9qYXZhc2Ny
aXB0LGQ1JTIwJTNEJTIwJ2ZpdmVcdTAwMjdzJyUzQiI+PC9zY3JpcHQ+Cis8c2NyaXB0PgoraWYg
KHdpbmRvdy5sYXlvdXRUZXN0Q29udHJvbGxlcikKKyAgbGF5b3V0VGVzdENvbnRyb2xsZXIuZHVt
cEFzVGV4dCgpOworaWYgKGQ1ID09ICJmaXZlJ3MiKSB7CisgIGRvY3VtZW50LndyaXRlKCJQQVNT
Iik7Cit9IGVsc2UgeworICBkb2N1bWVudC53cml0ZSgiRkFJTDogZDUgd2FzOiAiICsgZDUgKyAi
IHNob3VsZCBoYXZlIGJlZW4gOiBmaXZlJ3MiKTsKK30KKzwvc2NyaXB0PgpcIE5vIG5ld2xpbmUg
YXQgZW5kIG9mIGZpbGUKZGlmZiAtLWdpdCBhL1dlYkNvcmUvQ2hhbmdlTG9nIGIvV2ViQ29yZS9D
aGFuZ2VMb2cKaW5kZXggZmQ2MDk2Ni4uNThjMGE0NSAxMDA2NDQKLS0tIGEvV2ViQ29yZS9DaGFu
Z2VMb2cKKysrIGIvV2ViQ29yZS9DaGFuZ2VMb2cKQEAgLTEsMyArMSwxNSBAQAorMjAwOC0wMS0w
MSAgRXJpYyBTZWlkZWwgIDxlcmljQHdlYmtpdC5vcmc+CisKKyAgICAgICAgUmV2aWV3ZWQgYnkg
Tk9CT0RZLgorCisgICAgICAgIERvbid0IHJlcGxhY2UgXCB3aXRoIC8gaW4gZGF0YTogdXJscwor
ICAgICAgICBodHRwOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD0xNjY5MgorCisg
ICAgICAgIFRlc3Q6IGZhc3QvbG9hZGVyL3VybC1kYXRhLXJlcGxhY2UtYmFja3NsYXNoLmh0bWwK
KworICAgICAgICAqIHBsYXRmb3JtL0tVUkwuY3BwOgorICAgICAgICAoV2ViQ29yZTo6S1VSTDo6
aW5pdCk6CisKIDIwMDgtMDEtMDEgIEFscCBUb2tlciAgPGFscEBhdG9rZXIuY29tPgogCiAgICAg
ICAgIEdUSysgYXV0b3Rvb2xzIGJ1aWxkIGZpeC4gVHJhY2sgY2hhbmdlcyBpbiByMjkwNTEsIHIy
OTA1OCBhbmQgcGFzcyB0aGUKZGlmZiAtLWdpdCBhL1dlYkNvcmUvcGxhdGZvcm0vS1VSTC5jcHAg
Yi9XZWJDb3JlL3BsYXRmb3JtL0tVUkwuY3BwCmluZGV4IDBhMTNjOTEuLjg0OThiYmEgMTAwNjQ0
Ci0tLSBhL1dlYkNvcmUvcGxhdGZvcm0vS1VSTC5jcHAKKysrIGIvV2ViQ29yZS9wbGF0Zm9ybS9L
VVJMLmNwcApAQCAtMjg1LDEwICsyODUsOSBAQCB2b2lkIEtVUkw6OmluaXQoY29uc3QgS1VSTCAm
YmFzZSwgY29uc3QgRGVwcmVjYXRlZFN0cmluZyAmcmVsYXRpdmUsIGNvbnN0IFRleHRFbgogCiAg
ICAgLy8gZm9yIGNvbXBhdGliaWxpdHkgd2l0aCBXaW4gSUUsIHdlIG11c3QgdHJlYXQgYmFja3Ns
YXNoZXMgYXMgaWYgdGhleSB3ZXJlIHNsYXNoZXMsIGFzIGxvbmcgYXMgd2UncmUgbm90IGRlYWxp
bmcgd2l0aCB0aGUgamF2YXNjcmlwdDogc2NoZW1hCiAgICAgRGVwcmVjYXRlZFN0cmluZyBzdWJz
dGl0dXRlZFJlbGF0aXZlOwotICAgIGJvb2wgc2hvdWxkU3Vic3RpdHV0ZUJhY2tzbGFzaGVzID0g
cmVsYXRpdmUuY29udGFpbnMoJ1xcJykgJiYgIXJlbGF0aXZlLnN0YXJ0c1dpdGgoImphdmFzY3Jp
cHQ6IiwgZmFsc2UpOwotICAgIGlmIChzaG91bGRTdWJzdGl0dXRlQmFja3NsYXNoZXMpIHsKKyAg
ICBib29sIHNob3VsZFN1YnN0aXR1dGVCYWNrc2xhc2hlcyA9IHJlbGF0aXZlLmNvbnRhaW5zKCdc
XCcpICYmICEocmVsYXRpdmUuc3RhcnRzV2l0aCgiamF2YXNjcmlwdDoiLCBmYWxzZSkgfHwgcmVs
YXRpdmUuc3RhcnRzV2l0aCgiZGF0YToiLCBmYWxzZSkpOworICAgIGlmIChzaG91bGRTdWJzdGl0
dXRlQmFja3NsYXNoZXMpCiAgICAgICAgIHN1YnN0aXR1dGVkUmVsYXRpdmUgPSBzdWJzdGl0dXRl
QmFja3NsYXNoZXMocmVsYXRpdmUpOwotICAgIH0KIAogICAgIGNvbnN0IERlcHJlY2F0ZWRTdHJp
bmcgJnJlbCA9IHNob3VsZFN1YnN0aXR1dGVCYWNrc2xhc2hlcyA/IHN1YnN0aXR1dGVkUmVsYXRp
dmUgOiByZWxhdGl2ZTsKICAgICAK
</data>
<flag name="review"
          id="7929"
          type_id="1"
          status="+"
          setter="ap"
    />
          </attachment>
      

    </bug>

</bugzilla>