<?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>16032</bug_id>
          
          <creation_ts>2007-11-17 14:07:47 -0800</creation_ts>
          <short_desc>JS minidom is not portable</short_desc>
          <delta_ts>2007-11-17 15:20: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>JavaScriptCore</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>
          <dependson>16029</dependson>
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Alp Toker">alp</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>darin</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>61479</commentid>
    <comment_count>0</comment_count>
    <who name="Alp Toker">alp</who>
    <bug_when>2007-11-17 14:07:47 -0800</bug_when>
    <thetext>JavaScriptCore/API/minidom.c is not portable. It has a trivial dependency on CFString and is missing a couple of includes.

JavaScriptCore.h also needs to be fixed as described in #16029 for this to actually build on non-CF platforms.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>61482</commentid>
    <comment_count>1</comment_count>
      <attachid>17330</attachid>
    <who name="Alp Toker">alp</who>
    <bug_when>2007-11-17 14:33:32 -0800</bug_when>
    <thetext>Created attachment 17330
Make minidom.c portable

Note that it will not actually compile until JavaScriptCore.h is also fixed (see #16029)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>61483</commentid>
    <comment_count>2</comment_count>
      <attachid>17331</attachid>
    <who name="Alp Toker">alp</who>
    <bug_when>2007-11-17 15:02:24 -0800</bug_when>
    <thetext>Created attachment 17331
Make minidom.c portable

This also fixes exception output which was being sent to stderr instead of stdout.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>61484</commentid>
    <comment_count>3</comment_count>
      <attachid>17331</attachid>
    <who name="Eric Seidel (no email)">eric</who>
    <bug_when>2007-11-17 15:05:29 -0800</bug_when>
    <thetext>Comment on attachment 17331
Make minidom.c portable

Looks fine to me.  r=me.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>61485</commentid>
    <comment_count>4</comment_count>
    <who name="Alp Toker">alp</who>
    <bug_when>2007-11-17 15:15:21 -0800</bug_when>
    <thetext>Landed in r27879.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>61487</commentid>
    <comment_count>5</comment_count>
      <attachid>17330</attachid>
    <who name="Darin Adler">darin</who>
    <bug_when>2007-11-17 15:20:35 -0800</bug_when>
    <thetext>Comment on attachment 17330
Make minidom.c portable

Looks fine. It&apos;s going to be tricky to make this truly platform-independent though, because Mac OS X uses a special include style for frameworks (#include &lt;JavaScriptCore/JavaScriptCore.h&gt;), and I would not expect the library to work this way on other platforms.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>17330</attachid>
            <date>2007-11-17 14:33:32 -0800</date>
            <delta_ts>2007-11-17 15:20:35 -0800</delta_ts>
            <desc>Make minidom.c portable</desc>
            <filename>minidom-fix.patch</filename>
            <type>text/plain</type>
            <size>2072</size>
            <attacher name="Alp Toker">alp</attacher>
            
              <data encoding="base64">SW5kZXg6IEphdmFTY3JpcHRDb3JlL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBKYXZhU2NyaXB0
Q29yZS9DaGFuZ2VMb2cJKHJldmlzaW9uIDI3ODc4KQorKysgSmF2YVNjcmlwdENvcmUvQ2hhbmdl
TG9nCSh3b3JraW5nIGNvcHkpCkBAIC0xLDMgKzEsMTcgQEAKKzIwMDctMTEtMTcgIEFscCBUb2tl
ciAgPGFscEBhdG9rZXIuY29tPgorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEp
LgorCisgICAgICAgIGh0dHA6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTE2MDMy
CisgICAgICAgIEpTIG1pbmlkb20gaXMgbm90IHBvcnRhYmxlCisKKyAgICAgICAgVXNlIGEgcGxh
aW4gVVRGLTggc3RyaW5nIGluc3RlYWQgb2YgYSBDRlN0cmluZy4KKworICAgICAgICBQcmludCB0
byBzdGRlcnIgdG8gbWF0Y2ggdGhlIGZ1bmN0aW9uYWxpdHkgb2YgQ0ZTaG93KCkuCisKKyAgICAg
ICAgKiBBUEkvbWluaWRvbS5jOgorICAgICAgICAobWFpbik6CisKIDIwMDctMTEtMTcgIFN0ZXZl
IEZhbGtlbmJ1cmcgIDxzZmFsa2VuQGFwcGxlLmNvbT4KIAogICAgICAgICBXaW5kb3dzIGJ1aWxk
IGZpeC4KSW5kZXg6IEphdmFTY3JpcHRDb3JlL0FQSS9taW5pZG9tLmMKPT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0g
SmF2YVNjcmlwdENvcmUvQVBJL21pbmlkb20uYwkocmV2aXNpb24gMjc4NzgpCisrKyBKYXZhU2Ny
aXB0Q29yZS9BUEkvbWluaWRvbS5jCSh3b3JraW5nIGNvcHkpCkBAIC0xLDYgKzEsNyBAQAogLy8g
LSotIG1vZGU6IGMrKzsgYy1iYXNpYy1vZmZzZXQ6IDQgLSotCiAvKgogICogQ29weXJpZ2h0IChD
KSAyMDA2IEFwcGxlIENvbXB1dGVyLCBJbmMuICBBbGwgcmlnaHRzIHJlc2VydmVkLgorICogQ29w
eXJpZ2h0IChDKSAyMDA3IEFscCBUb2tlciA8YWxwQGF0b2tlci5jb20+CiAgKgogICogUmVkaXN0
cmlidXRpb24gYW5kIHVzZSBpbiBzb3VyY2UgYW5kIGJpbmFyeSBmb3Jtcywgd2l0aCBvciB3aXRo
b3V0CiAgKiBtb2RpZmljYXRpb24sIGFyZSBwZXJtaXR0ZWQgcHJvdmlkZWQgdGhhdCB0aGUgZm9s
bG93aW5nIGNvbmRpdGlvbnMKQEAgLTI1LDYgKzI2LDggQEAKICAqLwogCiAjaW5jbHVkZSAiSlNO
b2RlLmgiCisjaW5jbHVkZSA8c3RkaW8uaD4KKyNpbmNsdWRlIDxzdGRsaWIuaD4KICNpbmNsdWRl
IDx3dGYvQXNzZXJ0aW9ucy5oPgogI2luY2x1ZGUgPHd0Zi9VbnVzZWRQYXJhbS5oPgogCkBAIC01
Niw5ICs1OSwxMSBAQCBpbnQgbWFpbihpbnQgYXJnYywgY2hhciogYXJndltdKQogICAgIGVsc2Ug
ewogICAgICAgICBwcmludGYoIkZBSUw6IFRlc3Qgc2NyaXB0IHRocmV3IGV4Y2VwdGlvbjpcbiIp
OwogICAgICAgICBKU1N0cmluZ1JlZiBleGNlcHRpb25JU3RyaW5nID0gSlNWYWx1ZVRvU3RyaW5n
Q29weShjb250ZXh0LCBleGNlcHRpb24sIE5VTEwpOwotICAgICAgICBDRlN0cmluZ1JlZiBleGNl
cHRpb25DRiA9IEpTU3RyaW5nQ29weUNGU3RyaW5nKGtDRkFsbG9jYXRvckRlZmF1bHQsIGV4Y2Vw
dGlvbklTdHJpbmcpOwotICAgICAgICBDRlNob3coZXhjZXB0aW9uQ0YpOwotICAgICAgICBDRlJl
bGVhc2UoZXhjZXB0aW9uQ0YpOworICAgICAgICBzaXplX3QgZXhjZXB0aW9uVVRGOFNpemUgPSBK
U1N0cmluZ0dldE1heGltdW1VVEY4Q1N0cmluZ1NpemUoZXhjZXB0aW9uSVN0cmluZyk7CisgICAg
ICAgIGNoYXIqIGV4Y2VwdGlvblVURjggPSAoY2hhciopbWFsbG9jKGV4Y2VwdGlvblVURjhTaXpl
KTsKKyAgICAgICAgSlNTdHJpbmdHZXRVVEY4Q1N0cmluZyhleGNlcHRpb25JU3RyaW5nLCBleGNl
cHRpb25VVEY4LCBleGNlcHRpb25VVEY4U2l6ZSk7CisgICAgICAgIGZwcmludGYoc3RkZXJyLCAi
JXNcbiIsIGV4Y2VwdGlvblVURjgpOworICAgICAgICBmcmVlKGV4Y2VwdGlvblVURjgpOwogICAg
ICAgICBKU1N0cmluZ1JlbGVhc2UoZXhjZXB0aW9uSVN0cmluZyk7CiAgICAgfQogICAgIEpTU3Ry
aW5nUmVsZWFzZShzY3JpcHQpOwo=
</data>
<flag name="review"
          id="7397"
          type_id="1"
          status="+"
          setter="darin"
    />
          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>17331</attachid>
            <date>2007-11-17 15:02:24 -0800</date>
            <delta_ts>2007-11-17 15:05:29 -0800</delta_ts>
            <desc>Make minidom.c portable</desc>
            <filename>minidom-fix-2.patch</filename>
            <type>text/plain</type>
            <size>2117</size>
            <attacher name="Alp Toker">alp</attacher>
            
              <data encoding="base64">SW5kZXg6IEphdmFTY3JpcHRDb3JlL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBKYXZhU2NyaXB0
Q29yZS9DaGFuZ2VMb2cJKHJldmlzaW9uIDI3ODc4KQorKysgSmF2YVNjcmlwdENvcmUvQ2hhbmdl
TG9nCSh3b3JraW5nIGNvcHkpCkBAIC0xLDMgKzEsMTggQEAKKzIwMDctMTEtMTcgIEFscCBUb2tl
ciAgPGFscEBhdG9rZXIuY29tPgorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEp
LgorCisgICAgICAgIGh0dHA6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTE2MDMy
CisgICAgICAgIEpTIG1pbmlkb20gaXMgbm90IHBvcnRhYmxlCisKKyAgICAgICAgVXNlIGEgcGxh
aW4gVVRGLTggc3RyaW5nIGluc3RlYWQgb2YgYSBDRlN0cmluZy4KKworICAgICAgICBQcmludCB0
byBzdGRvdXQsIG5vdCBzdGRlcnIgbGlrZSBDRlNob3coKSB3b3VsZCBoYXZlIGRvbmUsIHNpbmNl
IHRoYXQKKyAgICAgICAgYmVoYXZpb3VyIHNlZW1zIHVuaW50ZW50aW9uYWwuCisKKyAgICAgICAg
KiBBUEkvbWluaWRvbS5jOgorICAgICAgICAobWFpbik6CisKIDIwMDctMTEtMTcgIFN0ZXZlIEZh
bGtlbmJ1cmcgIDxzZmFsa2VuQGFwcGxlLmNvbT4KIAogICAgICAgICBXaW5kb3dzIGJ1aWxkIGZp
eC4KSW5kZXg6IEphdmFTY3JpcHRDb3JlL0FQSS9taW5pZG9tLmMKPT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gSmF2
YVNjcmlwdENvcmUvQVBJL21pbmlkb20uYwkocmV2aXNpb24gMjc4NzgpCisrKyBKYXZhU2NyaXB0
Q29yZS9BUEkvbWluaWRvbS5jCSh3b3JraW5nIGNvcHkpCkBAIC0xLDYgKzEsNyBAQAogLy8gLSot
IG1vZGU6IGMrKzsgYy1iYXNpYy1vZmZzZXQ6IDQgLSotCiAvKgogICogQ29weXJpZ2h0IChDKSAy
MDA2IEFwcGxlIENvbXB1dGVyLCBJbmMuICBBbGwgcmlnaHRzIHJlc2VydmVkLgorICogQ29weXJp
Z2h0IChDKSAyMDA3IEFscCBUb2tlciA8YWxwQGF0b2tlci5jb20+CiAgKgogICogUmVkaXN0cmli
dXRpb24gYW5kIHVzZSBpbiBzb3VyY2UgYW5kIGJpbmFyeSBmb3Jtcywgd2l0aCBvciB3aXRob3V0
CiAgKiBtb2RpZmljYXRpb24sIGFyZSBwZXJtaXR0ZWQgcHJvdmlkZWQgdGhhdCB0aGUgZm9sbG93
aW5nIGNvbmRpdGlvbnMKQEAgLTI1LDYgKzI2LDggQEAKICAqLwogCiAjaW5jbHVkZSAiSlNOb2Rl
LmgiCisjaW5jbHVkZSA8c3RkaW8uaD4KKyNpbmNsdWRlIDxzdGRsaWIuaD4KICNpbmNsdWRlIDx3
dGYvQXNzZXJ0aW9ucy5oPgogI2luY2x1ZGUgPHd0Zi9VbnVzZWRQYXJhbS5oPgogCkBAIC01Niw5
ICs1OSwxMSBAQCBpbnQgbWFpbihpbnQgYXJnYywgY2hhciogYXJndltdKQogICAgIGVsc2Ugewog
ICAgICAgICBwcmludGYoIkZBSUw6IFRlc3Qgc2NyaXB0IHRocmV3IGV4Y2VwdGlvbjpcbiIpOwog
ICAgICAgICBKU1N0cmluZ1JlZiBleGNlcHRpb25JU3RyaW5nID0gSlNWYWx1ZVRvU3RyaW5nQ29w
eShjb250ZXh0LCBleGNlcHRpb24sIE5VTEwpOwotICAgICAgICBDRlN0cmluZ1JlZiBleGNlcHRp
b25DRiA9IEpTU3RyaW5nQ29weUNGU3RyaW5nKGtDRkFsbG9jYXRvckRlZmF1bHQsIGV4Y2VwdGlv
bklTdHJpbmcpOwotICAgICAgICBDRlNob3coZXhjZXB0aW9uQ0YpOwotICAgICAgICBDRlJlbGVh
c2UoZXhjZXB0aW9uQ0YpOworICAgICAgICBzaXplX3QgZXhjZXB0aW9uVVRGOFNpemUgPSBKU1N0
cmluZ0dldE1heGltdW1VVEY4Q1N0cmluZ1NpemUoZXhjZXB0aW9uSVN0cmluZyk7CisgICAgICAg
IGNoYXIqIGV4Y2VwdGlvblVURjggPSAoY2hhciopbWFsbG9jKGV4Y2VwdGlvblVURjhTaXplKTsK
KyAgICAgICAgSlNTdHJpbmdHZXRVVEY4Q1N0cmluZyhleGNlcHRpb25JU3RyaW5nLCBleGNlcHRp
b25VVEY4LCBleGNlcHRpb25VVEY4U2l6ZSk7CisgICAgICAgIHByaW50ZigiJXNcbiIsIGV4Y2Vw
dGlvblVURjgpOworICAgICAgICBmcmVlKGV4Y2VwdGlvblVURjgpOwogICAgICAgICBKU1N0cmlu
Z1JlbGVhc2UoZXhjZXB0aW9uSVN0cmluZyk7CiAgICAgfQogICAgIEpTU3RyaW5nUmVsZWFzZShz
Y3JpcHQpOwo=
</data>
<flag name="review"
          id="7396"
          type_id="1"
          status="+"
          setter="eric"
    />
          </attachment>
      

    </bug>

</bugzilla>