<?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>26884</bug_id>
          
          <creation_ts>2009-07-01 08:17:09 -0700</creation_ts>
          <short_desc>[WML] WMLTestCase.js is cluttered &amp; depends on several hacks</short_desc>
          <delta_ts>2009-07-01 09:47:56 -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>Mac</rep_platform>
          <op_sys>OS X 10.5</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>
          
          <blocked>20393</blocked>
    
    <blocked>26885</blocked>
          <everconfirmed>1</everconfirmed>
          <reporter name="Nikolas Zimmermann">zimmermann</reporter>
          <assigned_to name="Nikolas Zimmermann">zimmermann</assigned_to>
          <cc>manyoso</cc>
    
    <cc>staikos</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>129291</commentid>
    <comment_count>0</comment_count>
    <who name="Nikolas Zimmermann">zimmermann</who>
    <bug_when>2009-07-01 08:17:09 -0700</bug_when>
    <thetext>As the summary says, WMLTestCase.js needs some love.
It depends on timing hacks at the moment, which can be easily resolved. Attaching a patch soon.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>129293</commentid>
    <comment_count>1</comment_count>
      <attachid>32123</attachid>
    <who name="Nikolas Zimmermann">zimmermann</who>
    <bug_when>2009-07-01 08:24:53 -0700</bug_when>
    <thetext>Created attachment 32123
Initial patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>129306</commentid>
    <comment_count>2</comment_count>
    <who name="Holger Freyther">zecke</who>
    <bug_when>2009-07-01 09:19:05 -0700</bug_when>
    <thetext>I agree with george, the cleanup looks fine...</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>129307</commentid>
    <comment_count>3</comment_count>
    <who name="Nikolas Zimmermann">zimmermann</who>
    <bug_when>2009-07-01 09:47:56 -0700</bug_when>
    <thetext>Landed in r45439.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>32123</attachid>
            <date>2009-07-01 08:24:53 -0700</date>
            <delta_ts>2009-07-01 09:16:09 -0700</delta_ts>
            <desc>Initial patch</desc>
            <filename>bugzilla_requires_a_filename.patch</filename>
            <type>text/plain</type>
            <size>4925</size>
            <attacher name="Nikolas Zimmermann">zimmermann</attacher>
            
              <data encoding="base64">SW5kZXg6IENoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBDaGFuZ2VMb2cJKHJldmlzaW9uIDQ1NDM2
KQorKysgQ2hhbmdlTG9nCSh3b3JraW5nIGNvcHkpCkBAIC0xLDMgKzEsMTcgQEAKKzIwMDktMDct
MDEgIE5pa29sYXMgWmltbWVybWFubiAgPG5pa29sYXMuemltbWVybWFubkB0b3JjaG1vYmlsZS5j
b20+CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgRml4
ZXM6IGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD0yNjg4NAorCisgICAg
ICAgIFdNTFRlc3RDYXNlLmpzIHJlbGllcyBvbiB0aW1pbmctZGVwZW5kYW50IGhhY2tzLCBtYWtp
bmcgaXQgZGlmZmljdWx0IHRvIHByZWRpY3QgcmVzdWx0cyBvbiBzbG93ZXIgbWFjaGluZXMuCisg
ICAgICAgIEZpeCBieSByZXdyaXRpbmcgYW5kIHNpbXBsaWZ5aW5nIHRoZSB0ZXN0aW5nIHByb2Nl
ZHVyZSwgd2hpY2ggY29udGFpbmVkIHVubmVjZXNzYXJ5IHF1aXJrcyB3b3JraW5nIGFyb3VuZCBv
bGRlciBXTUwgYnVncy4KKworICAgICAgICAqIHdtbC9yZXNvdXJjZXMvV01MVGVzdENhc2UuanM6
CisgICAgICAgIChjcmVhdGVXTUxUZXN0Q2FzZS5pZnJhbWVFbGVtZW50Lm9ubG9hZCk6CisgICAg
ICAgIChjcmVhdGVXTUxUZXN0Q2FzZSk6CisgICAgICAgIChzdGFydFRlc3QpOgorCiAyMDA5LTA3
LTAxICBOaWtvbGFzIFppbW1lcm1hbm4gIDxuaWtvbGFzLnppbW1lcm1hbm5AdG9yY2htb2JpbGUu
Y29tPgogCiAgICAgICAgIFJldmlld2VkIGJ5IEhvbGdlciBGcmV5dGhlci4KSW5kZXg6IHdtbC9y
ZXNvdXJjZXMvV01MVGVzdENhc2UuanMKPT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gd21sL3Jlc291cmNlcy9XTUxU
ZXN0Q2FzZS5qcwkocmV2aXNpb24gNDU0MzYpCisrKyB3bWwvcmVzb3VyY2VzL1dNTFRlc3RDYXNl
LmpzCSh3b3JraW5nIGNvcHkpCkBAIC05LDU5ICs5LDYyIEBAIGZ1bmN0aW9uIGNyZWF0ZVdNTEVs
ZW1lbnQobmFtZSkgewogICAgIHJldHVybiB0ZXN0RG9jdW1lbnQuY3JlYXRlRWxlbWVudE5TKHdt
bE5TLCAid21sOiIgKyBuYW1lKTsKIH0KIAotZnVuY3Rpb24gY3JlYXRlV01MVGVzdENhc2UoZGVz
Yywgc3Vic3RpdHV0ZVZhcmlhYmxlcywgdGVzdE5hbWUpIHsKLSAgICBpZiAoc3Vic3RpdHV0ZVZh
cmlhYmxlcyA9PSBudWxsKQotICAgICAgICBzdWJzdGl0dXRlVmFyaWFibGVzID0gdHJ1ZTsKK2Z1
bmN0aW9uIGNyZWF0ZVdNTFRlc3RDYXNlKHRlc3REZXNjcmlwdGlvbiwgc3Vic3RpdHV0ZXNWYXJp
YWJsZXMsIHRlc3ROYW1lKSB7CisgICAgLy8gU2V0dXAgZGVmYXVsdCB0ZXN0IG9wdGlvbnMKKyAg
ICBpZiAoc3Vic3RpdHV0ZXNWYXJpYWJsZXMgPT0gbnVsbCkgeworICAgICAgICBzdWJzdGl0dXRl
c1ZhcmlhYmxlcyA9IHRydWU7CisgICAgfQogCi0gICAgdmFyIGRlZmF1bHRUZXN0ID0gdHJ1ZTsK
KyAgICAvLyBTZXR1cCBkZWZhdWx0IHRlc3QgbmFtZQorICAgIHZhciB1c2VzRGVmYXVsdFRlc3RE
b2N1bWVudCA9IGZhbHNlOwogICAgIGlmICh0ZXN0TmFtZSA9PSBudWxsKSB7Ci0gICAgICAgIGRl
ZmF1bHRUZXN0ID0gZmFsc2U7CisgICAgICAgIHVzZXNEZWZhdWx0VGVzdERvY3VtZW50ID0gdHJ1
ZTsKICAgICAgICAgdGVzdE5hbWUgPSByZWxhdGl2ZVBhdGhUb0xheW91dFRlc3RzICsgIi93bWwv
cmVzb3VyY2VzL3Rlc3QtZG9jdW1lbnQud21sIjsKICAgICB9CiAKLSAgICBkZXNjcmlwdGlvbihk
ZXNjKTsKKyAgICAvLyBJbml0aWFsaXplIEpTIHRlc3QKKyAgICBkZXNjcmlwdGlvbih0ZXN0RGVz
Y3JpcHRpb24pOwogICAgIGJvZHlFbGVtZW50ID0gZG9jdW1lbnQuZ2V0RWxlbWVudHNCeVRhZ05h
bWUoImJvZHkiKVswXTsKIAogICAgIC8vIENsZWFyIHZhcmlhYmxlIHN0YXRlICYgaGlzdG9yeQog
ICAgIGRvY3VtZW50LnJlc2V0V01MUGFnZVN0YXRlKCk7CiAKKyAgICAvLyBTZXR1cCBEUlQgc3Bl
Y2lmaWMgc2V0dGluZ3MKICAgICBpZiAod2luZG93LmxheW91dFRlc3RDb250cm9sbGVyKSB7CiAg
ICAgICAgIGxheW91dFRlc3RDb250cm9sbGVyLmR1bXBDaGlsZEZyYW1lc0FzVGV4dCgpOwogICAg
ICAgICBsYXlvdXRUZXN0Q29udHJvbGxlci53YWl0VW50aWxEb25lKCk7CiAgICAgfQogCisgICAg
Ly8gQ3JlYXRlIGNvbnRhaW5lciBlbGVtZW50IHRvIGxvYWQgdGhlIFdNTCBkb2N1bWVudAogICAg
IGlmcmFtZUVsZW1lbnQgPSBkb2N1bWVudC5jcmVhdGVFbGVtZW50TlMoeGh0bWxOUywgImlmcmFt
ZSIpOwogICAgIGlmcmFtZUVsZW1lbnQuc3JjID0gdGVzdE5hbWU7CiAKLSAgICB2YXIgbG9hZGVk
ID0gZmFsc2U7Ci0gICAgdmFyIGV4ZWN1dGVkID0gZmFsc2U7Ci0KKyAgICAvLyBQcm9jZXNzIGxv
YWQgZXZlbnRzLCB0YWtpbmcgY2FyZSBvZiBzZXR0aW5nIHVwIHRoZSBuYXRpdmUgV01MIHRlc3Rj
YXNlCiAgICAgaWZyYW1lRWxlbWVudC5vbmxvYWQgPSBmdW5jdGlvbigpIHsKLSAgICAgICAgaWYg
KGV4ZWN1dGVkICYmICFkZWZhdWx0VGVzdCkKLSAgICAgICAgICAgIHJldHVybjsKKyAgICAgICAg
aWYgKHRlc3REb2N1bWVudCkgeworICAgICAgICAgICAgLy8gVmFyaWFibGUgc3Vic3RpdGl0aW9u
IG1vZGU6IHJlc2V0dXAgdGVzdCBkb2N1bWVudCwgc3Vic3RpdHV0aW5nIGFsbCB2YXJpYWJsZXMg
c3RvcmVkIGluIHRoZSBjdXJyZW50IFdNTCBwYWdlIHN0YXRlLgorICAgICAgICAgICAgLy8gVmFy
aWFibGUgc3Vic3RpdHV0aW9uIG9ubHkgaGFwcGVucyBhZnRlciB0aGUgaW5pdGlhbCBkb2N1bWVu
dCBoYXMgYmVlbiBsb2FkZWQgYW5kIGFmdGVyIGEgcmVsb2FkIHdhcyB0cmlnZ2VyZWQuICAgICAg
ICAKKyAgICAgICAgICAgIGlmIChzdWJzdGl0dXRlc1ZhcmlhYmxlcykgeworICAgICAgICAgICAg
ICAgIHRlc3REb2N1bWVudCA9IGlmcmFtZUVsZW1lbnQuY29udGVudERvY3VtZW50OworICAgICAg
ICAgICAgICAgIHNldHVwVGVzdERvY3VtZW50KCk7CisgICAgICAgICAgICB9CiAKLSAgICAgICAg
Ly8gRXh0ZXJuYWwgZGVjayBqdW1wcwotICAgICAgICBpZiAodGVzdERvY3VtZW50ICE9IG51bGwg
JiYgIXN1YnN0aXR1dGVWYXJpYWJsZXMpIHsKLSAgICAgICAgICAgIGRlbGF5RXhlY3V0ZVRlc3Qo
KTsKKyAgICAgICAgICAgIGV4ZWN1dGVUZXN0KCk7CiAgICAgICAgICAgICByZXR1cm47CiAgICAg
ICAgIH0KIAogICAgICAgICB0ZXN0RG9jdW1lbnQgPSBpZnJhbWVFbGVtZW50LmNvbnRlbnREb2N1
bWVudDsKICAgICAgICAgc2V0dXBUZXN0RG9jdW1lbnQoKTsKLQotICAgICAgICAvLyBWYXJpYWJs
ZSByZWZyZXNoCi0gICAgICAgIGlmIChsb2FkZWQgJiYgc3Vic3RpdHV0ZVZhcmlhYmxlcykgewot
ICAgICAgICAgICAgZGVsYXlFeGVjdXRlVGVzdCgpOwotICAgICAgICAgICAgcmV0dXJuOwotICAg
ICAgICB9Ci0KLSAgICAgICAgbG9hZGVkID0gdHJ1ZTsKICAgICAgICAgcHJlcGFyZVRlc3QoKTsK
IAotICAgICAgICAvLyBJbnRlcm5hbCBkZWNrIGp1bXBzCi0gICAgICAgIGlmICghc3Vic3RpdHV0
ZVZhcmlhYmxlcykgewotICAgICAgICAgICAgZXhlY3V0ZWQgPSB0cnVlOwotICAgICAgICAgICAg
ZGVsYXlFeGVjdXRlVGVzdCgpOworICAgICAgICAvLyBJbiBhIHJlZ3VsYXIgV01MIGRvY3VtZW50
LCB0aGlzIHdvdWxkIGhhcHBlbiBhZnRlciB0aGUgcGFyc2luZyBmaW5pc2hlZC4KKyAgICAgICAg
Ly8gVGhvdWdoIGFzIHdlIGR5bmFtaWNhbGx5IGNyZWF0ZSB0ZXN0Y2FzZXMsIHdlIGhhdmUgdG8g
dGFrZSBjYXJlIG9mIGluaXRpYWxpemluZyBXTUwgdmFyaWFibGUgc3RhdGUgbWFudWFsbHkuCisg
ICAgICAgIHRlc3REb2N1bWVudC5pbml0aWFsaXplV01MUGFnZVN0YXRlKCk7CisKKyAgICAgICAg
Ly8gSGFuZGxlIHRoZSBjYXNlIG9mIGEgc3BlY2lhbCBXTUwgdGVzdCBkb2N1bWVudC4gVGhhdCdz
IGFsd2F5cyB0aGUgY2FzZSBmb3Igc3RhdGljIFdNTCB0ZXN0Y2FzZXMKKyAgICAgICAgLy8od2hp
Y2ggYXJlIE5PVCBjcmVhdGVkIG1hbnVhbGx5IHVzaW5nIEpTIGluIHByZXBhcmVUZXN0KS4gRmly
ZSBvZmYgdGVzdCBpbW1lZGlhdGVseSBmb3IgdGhvc2UuCisgICAgICAgIGlmICghdXNlc0RlZmF1
bHRUZXN0RG9jdW1lbnQpIHsKKyAgICAgICAgICAgIGV4ZWN1dGVUZXN0KCk7CiAgICAgICAgIH0K
ICAgICB9CiAKQEAgLTgxLDIwICs4NCwxMCBAQCBmdW5jdGlvbiB0cmlnZ2VyVXBkYXRlKHgsIHkp
IHsKIH0KIAogZnVuY3Rpb24gc3RhcnRUZXN0KHgsIHkpIHsKLSAgICAvLyBJbml0aWFsaXplIHZh
cmlhYmxlIHN0YXRlCi0gICAgLy8gSW4gYSByZWd1bGFyIFdNTCBkb2N1bWVudCwgdGhpcyB3b3Vs
ZCBoYXBwZW4gYWZ0ZXIgdGhlIHBhcnNpbmcgZmluaXNoZWQuCi0gICAgLy8gVGhvdWdoIGFzIHdl
IGR5bmFtaWNhbGx5IGNyZWF0ZSB0ZXN0Y2FzZXMsIHdlIGhhdmUgdG8gdGFrZSBjYXJlIG9mIGlu
aXRpYWxpemluZyBXTUwgdmFyaWFibGUgc3RhdGUgbWFudWFsbHkuCi0gICAgdGVzdERvY3VtZW50
LmluaXRpYWxpemVXTUxQYWdlU3RhdGUoKTsKLQogICAgIC8vIEFzc3VyZSBmaXJzdCBsYXlvdXQg
ZmluaXNoZWQKICAgICB3aW5kb3cuc2V0VGltZW91dCgidHJpZ2dlclVwZGF0ZSgiICsgeCArICIs
ICIgKyB5ICsgIikiLCAwKTsKIH0KIAotZnVuY3Rpb24gZGVsYXlFeGVjdXRlVGVzdCgpIHsKLSAg
ICAvLyBBc3N1cmUgZmlyc3QgbGF5b3V0IGZpbmlzaGVkLCBhZnRlciBtYWtpbmcgY2hhbmdlcwot
ICAgIHdpbmRvdy5zZXRUaW1lb3V0KCJleGVjdXRlVGVzdCgpIiwgMCk7Ci19Ci0KIGZ1bmN0aW9u
IGNvbXBsZXRlVGVzdCgpIHsKICAgICB2YXIgc2NyaXB0ID0gZG9jdW1lbnQuY3JlYXRlRWxlbWVu
dE5TKHhodG1sTlMsICJzY3JpcHQiKTs=
</data>
<flag name="review"
          id="16629"
          type_id="1"
          status="+"
          setter="staikos"
    />
          </attachment>
      

    </bug>

</bugzilla>