<?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>21597</bug_id>
          
          <creation_ts>2008-10-14 13:45:42 -0700</creation_ts>
          <short_desc>Set popup&apos;s location to about:blank while it&apos;s loading</short_desc>
          <delta_ts>2009-04-29 14:31:47 -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>DOM</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="Pam Greene (IRC:pamg)">pam</reporter>
          <assigned_to name="Mike Belshe">mbelshe</assigned_to>
          <cc>ap</cc>
    
    <cc>darin</cc>
    
    <cc>sam</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>95351</commentid>
    <comment_count>0</comment_count>
    <who name="Pam Greene (IRC:pamg)">pam</who>
    <bug_when>2008-10-14 13:45:42 -0700</bug_when>
    <thetext>Different browsers return different things for the properties of a popup window
while the window is opening. In particular, the behavior of window.location,
window.href, and window.pathname vary.  

The Selenium web app test system -- http://selenium.openqa.org -- depends on
this behavior.

When running the fast/dom/Window/window-open-pending-url.html test, popup.location reports as follows:

Safari ToT: location: /
Firefox: location: about:blank
Chrome: location: about:blank
IE7: location: file:///C:/&lt;path to&gt;/fast/dom/Window/resources/destination.html

WebKit should also report &apos;location: about:blank&apos;, for consistency and so Selenium works.  See bug 21419 for earlier discussion.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>111930</commentid>
    <comment_count>1</comment_count>
      <attachid>28203</attachid>
    <who name="Mike Belshe">mbelshe</who>
    <bug_when>2009-03-02 16:31:56 -0800</bug_when>
    <thetext>Created attachment 28203
Location.cpp patch with new expected results.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>112167</commentid>
    <comment_count>2</comment_count>
    <who name="Alexey Proskuryakov">ap</who>
    <bug_when>2009-03-04 01:43:24 -0800</bug_when>
    <thetext>Where does this incorrect data in m_frame-&gt;loader()-&gt;url() come from? Could it be more appropriate to fix it, instead of tweaking results in Location?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>112219</commentid>
    <comment_count>3</comment_count>
    <who name="Mike Belshe">mbelshe</who>
    <bug_when>2009-03-04 10:24:44 -0800</bug_when>
    <thetext>I don&apos;t think there was any incorrect data; the URL was just not yet set, and when you tostring() that, you get &quot;/&quot;.  So I think the fix is correct - if there is no URL (e.g. url.IsValid() == false), then use the about:blank for conformance with other browsers.

We could change the way frame works to initialize to about:blank from the start;  but this seems more like an API compatibility issue with Location rather than a problem with the way the frame initializes its url.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>112603</commentid>
    <comment_count>4</comment_count>
    <who name="Alexey Proskuryakov">ap</who>
    <bug_when>2009-03-06 09:30:23 -0800</bug_when>
    <thetext>A test added in &lt;http://trac.webkit.org/projects/webkit/changeset/41484&gt; will be affected by this patch.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>112646</commentid>
    <comment_count>5</comment_count>
      <attachid>28203</attachid>
    <who name="Darin Fisher (:fishd, Google)">fishd</who>
    <bug_when>2009-03-06 12:50:02 -0800</bug_when>
    <thetext>Comment on attachment 28203
Location.cpp patch with new expected results.

&gt;Index: WebCore/page/Location.cpp
...
&gt;+    const KURL&amp; url = m_frame-&gt;loader()-&gt;url();
&gt;+    if (!url.isValid())
&gt;+        return blankURL();  // Use &quot;about:blank&quot; while the page is still loading (before we have a frame).

isn&apos;t it possible for the URL to be invalid due to bad input from
a web page or from the user?  in those cases, wouldn&apos;t we still want
to report that possibly invalid URL as the location?

e.g.,  12345://foo.com/

hmm... though in my testing, it looks like invalid URLs are always
treated as relative URLs by webkit.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>112666</commentid>
    <comment_count>6</comment_count>
      <attachid>28203</attachid>
    <who name="Darin Adler">darin</who>
    <bug_when>2009-03-06 14:44:30 -0800</bug_when>
    <thetext>Comment on attachment 28203
Location.cpp patch with new expected results.

This looks like a good approach. It would also be possible to change the FrameLoader itself to use the blankURL instead of an empty URL -- riskier and harder to get it right.

You&apos;ll also have to patch the test and results from the test I just added, by modifying fast/dom/resources/location-new-window-no-crash.js and updating fast/dom/location-new-window-no-crash-expected.txt with new results.

The ChangeLog mentions a function WebCore::urlWhileLoading -- this version of the patch doesn&apos;t have that so it shouldn&apos;t be in the change log.

Those change logs are no good -- they don&apos;t say anything about what you&apos;re changing and why, nor do they have the URL of this bug! The change log needs to mention the rationale about why these values need to be returned. I&apos;d like some more data about the behavior of IE here and how you determined this behavior was required.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>113534</commentid>
    <comment_count>7</comment_count>
    <who name="Mike Belshe">mbelshe</who>
    <bug_when>2009-03-12 17:07:17 -0700</bug_when>
    <thetext>Thanks for your comments Darin.

Regarding why I think we should do this change:  there is a selenium test framework which had a dependency on this from Firefox, and since the browsers are inconsistent, we thought this was a better approach.  I suppose we could debate more whether about:blank is the right interim value, but it does seem better than &apos;/&apos;.

Here are the IE8 results with your new location-new-window-no-crash which differ:
location.toString() was [object Object].
location.href was .        (empty string)
location.protocol was .
location.pathname was .

</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>113535</commentid>
    <comment_count>8</comment_count>
      <attachid>28569</attachid>
    <who name="Mike Belshe">mbelshe</who>
    <bug_when>2009-03-12 17:08:18 -0700</bug_when>
    <thetext>Created attachment 28569
Patch w/ Darin&apos;s comments</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>115204</commentid>
    <comment_count>9</comment_count>
    <who name="Jon@Chromium">jon</who>
    <bug_when>2009-03-25 13:08:17 -0700</bug_when>
    <thetext>Darin, could you please look over Mike&apos;s revision?  We are trying to clean up some layout test failures and this patch is needed.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>115360</commentid>
    <comment_count>10</comment_count>
    <who name="Eric Seidel (no email)">eric</who>
    <bug_when>2009-03-26 11:25:22 -0700</bug_when>
    <thetext>Darin was not cc&apos;d on the bug, I&apos;ve done so now.  The patch also looks fine to me, and seems to address Darin and Alexey&apos;s previous comments.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>115361</commentid>
    <comment_count>11</comment_count>
      <attachid>28569</attachid>
    <who name="Eric Seidel (no email)">eric</who>
    <bug_when>2009-03-26 11:27:42 -0700</bug_when>
    <thetext>Comment on attachment 28569
Patch w/ Darin&apos;s comments

I think that this change looks fine, but we should make an effort to better document the behavior of the other browsers in the test or code change for when the next person looks at this. ;)

Specifically, there are comments about what FF does location-new-window-no-crash.js we might as well add the information mike learned in his testing of IE8 to the same comment lines when landing.

I&apos;ve CC&apos;d Darin (and Alexey was already cc&apos;d) so they have the opportunity to point out any subtlety which they had in mind, but didn&apos;t mention in the bug.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>119347</commentid>
    <comment_count>12</comment_count>
    <who name="Eric Seidel (no email)">eric</who>
    <bug_when>2009-04-29 14:31:47 -0700</bug_when>
    <thetext>I fixed up the ChangeLogs a little and landed:

Committing to http://svn.webkit.org/repository/webkit/trunk ...
	M	LayoutTests/ChangeLog
	M	LayoutTests/fast/dom/Window/window-open-pending-url-expected.txt
	M	LayoutTests/fast/dom/location-new-window-no-crash-expected.txt
	M	LayoutTests/fast/dom/resources/location-new-window-no-crash.js
	M	WebCore/ChangeLog
	M	WebCore/page/Location.cpp
Committed r43014
</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>28203</attachid>
            <date>2009-03-02 16:31:56 -0800</date>
            <delta_ts>2009-03-06 14:44:30 -0800</delta_ts>
            <desc>Location.cpp patch with new expected results.</desc>
            <filename>bug21597</filename>
            <type>text/plain</type>
            <size>2014</size>
            <attacher name="Mike Belshe">mbelshe</attacher>
            
              <data encoding="base64">SW5kZXg6IFdlYkNvcmUvQ2hhbmdlTG9nCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFdlYkNvcmUvQ2hhbmdlTG9n
CShyZXZpc2lvbiA0MTM3MSkKKysrIFdlYkNvcmUvQ2hhbmdlTG9nCSh3b3JraW5nIGNvcHkpCkBA
IC0xLDMgKzEsMTEgQEAKKzIwMDktMDMtMDIgIE1pa2UgQmVsc2hlICA8bWlrZUBiZWxzaGUuY29t
PgorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgICogcGFn
ZS9Mb2NhdGlvbi5jcHA6CisgICAgICAgIChXZWJDb3JlOjp1cmxXaGlsZUxvYWRpbmcpOgorICAg
ICAgICAoV2ViQ29yZTo6TG9jYXRpb246OnVybCk6CisKIDIwMDktMDMtMDIgIEtldmluIE9sbGl2
aWVyICA8a2V2aW5vQHRoZW9sbGl2aWVycy5jb20+CiAKICAgICAgICAgQnVpbGQgZml4ZXMgZm9y
IHd4V2lkZ2V0cyBNYWMgdHJ1bmsgYnVpbGQuCkluZGV4OiBXZWJDb3JlL3BhZ2UvTG9jYXRpb24u
Y3BwCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT0KLS0tIFdlYkNvcmUvcGFnZS9Mb2NhdGlvbi5jcHAJKHJldmlzaW9uIDQx
MzU5KQorKysgV2ViQ29yZS9wYWdlL0xvY2F0aW9uLmNwcAkod29ya2luZyBjb3B5KQpAQCAtNDks
NyArNDksMTIgQEAgdm9pZCBMb2NhdGlvbjo6ZGlzY29ubmVjdEZyYW1lKCkKIGlubGluZSBjb25z
dCBLVVJMJiBMb2NhdGlvbjo6dXJsKCkgY29uc3QKIHsKICAgICBBU1NFUlQobV9mcmFtZSk7Ci0g
ICAgcmV0dXJuIG1fZnJhbWUtPmxvYWRlcigpLT51cmwoKTsKKworICAgIGNvbnN0IEtVUkwmIHVy
bCA9IG1fZnJhbWUtPmxvYWRlcigpLT51cmwoKTsKKyAgICBpZiAoIXVybC5pc1ZhbGlkKCkpCisg
ICAgICAgIHJldHVybiBibGFua1VSTCgpOyAgLy8gVXNlICJhYm91dDpibGFuayIgd2hpbGUgdGhl
IHBhZ2UgaXMgc3RpbGwgbG9hZGluZyAoYmVmb3JlIHdlIGhhdmUgYSBmcmFtZSkuCisKKyAgICBy
ZXR1cm4gdXJsOwogfQogCiBTdHJpbmcgTG9jYXRpb246OmhyZWYoKSBjb25zdApJbmRleDogTGF5
b3V0VGVzdHMvQ2hhbmdlTG9nCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIExheW91dFRlc3RzL0NoYW5nZUxvZwko
cmV2aXNpb24gNDEzNzEpCisrKyBMYXlvdXRUZXN0cy9DaGFuZ2VMb2cJKHdvcmtpbmcgY29weSkK
QEAgLTEsMyArMSw5IEBACisyMDA5LTAzLTAyICBNaWtlIEJlbHNoZSAgPG1pa2VAYmVsc2hlLmNv
bT4KKworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICAqIGZh
c3QvZG9tL1dpbmRvdy93aW5kb3ctb3Blbi1wZW5kaW5nLXVybC1leHBlY3RlZC50eHQ6CisKIDIw
MDktMDMtMDEgIERhdmlkIExldmluICA8bGV2aW5AY2hyb21pdW0ub3JnPgogCiAgICAgICAgIFJl
dmlld2VkIGJ5IEFsZXhleSBQcm9za3VyeWFrb3YuCkluZGV4OiBMYXlvdXRUZXN0cy9mYXN0L2Rv
bS9XaW5kb3cvd2luZG93LW9wZW4tcGVuZGluZy11cmwtZXhwZWN0ZWQudHh0Cj09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0K
LS0tIExheW91dFRlc3RzL2Zhc3QvZG9tL1dpbmRvdy93aW5kb3ctb3Blbi1wZW5kaW5nLXVybC1l
eHBlY3RlZC50eHQJKHJldmlzaW9uIDQxMzU5KQorKysgTGF5b3V0VGVzdHMvZmFzdC9kb20vV2lu
ZG93L3dpbmRvdy1vcGVuLXBlbmRpbmctdXJsLWV4cGVjdGVkLnR4dAkod29ya2luZyBjb3B5KQpA
QCAtMSwxMCArMSwxMCBAQAotbG9jYXRpb246IC8KLWhyZWY6IC8KLXBhdGhuYW1lOiAvCitsb2Nh
dGlvbjogYWJvdXQ6YmxhbmsKK2hyZWY6IGFib3V0OmJsYW5rCitwYXRobmFtZTogYmxhbmsKIGhv
c3Q6IAogaG9zdG5hbWU6IAogaGFzaDogCiBwb3J0OiAKLXByb3RvY29sOiA6Citwcm90b2NvbDog
YWJvdXQ6CiBzZWFyY2g6IAogCg==
</data>
<flag name="review"
          id="13784"
          type_id="1"
          status="-"
          setter="darin"
    />
          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>28569</attachid>
            <date>2009-03-12 17:08:18 -0700</date>
            <delta_ts>2009-03-26 11:27:42 -0700</delta_ts>
            <desc>Patch w/ Darin&apos;s comments</desc>
            <filename>bug21597b.patch</filename>
            <type>text/plain</type>
            <size>6502</size>
            <attacher name="Mike Belshe">mbelshe</attacher>
            
              <data encoding="base64">SW5kZXg6IFdlYkNvcmUvQ2hhbmdlTG9nCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFdlYkNvcmUvQ2hhbmdlTG9n
CShyZXZpc2lvbiA0MTY1NykKKysrIFdlYkNvcmUvQ2hhbmdlTG9nCSh3b3JraW5nIGNvcHkpCkBA
IC0xLDMgKzEsMTAgQEAKKzIwMDktMDMtMTIgIE1pa2UgQmVsc2hlICA8bWlrZUBiZWxzaGUuY29t
CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisgICAgICAgIEZpeCBidWcg
MjE1OTc6IHVwZGF0ZSBsb2NhdGlvbiB3aGlsZSBwYWdlIGlzIGxvYWRpbmcuCisKKyAgICAgICAg
KiBwYWdlL0xvY2F0aW9uLmNwcDoKKwogMjAwOS0wMi0yNiAgRXJpYyBTZWlkZWwgIDxlcmljQHdl
YmtpdC5vcmc+CiAKICAgICAgICAgUmV2aWV3ZWQgYnkgSnVzdGluIEdhcmNpYS4KSW5kZXg6IFdl
YkNvcmUvcGFnZS9Mb2NhdGlvbi5jcHAKPT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gV2ViQ29yZS9wYWdlL0xvY2F0
aW9uLmNwcAkocmV2aXNpb24gNDE2MjgpCisrKyBXZWJDb3JlL3BhZ2UvTG9jYXRpb24uY3BwCSh3
b3JraW5nIGNvcHkpCkBAIC00OSw3ICs0OSwxMiBAQCB2b2lkIExvY2F0aW9uOjpkaXNjb25uZWN0
RnJhbWUoKQogaW5saW5lIGNvbnN0IEtVUkwmIExvY2F0aW9uOjp1cmwoKSBjb25zdAogewogICAg
IEFTU0VSVChtX2ZyYW1lKTsKLSAgICByZXR1cm4gbV9mcmFtZS0+bG9hZGVyKCktPnVybCgpOwor
CisgICAgY29uc3QgS1VSTCYgdXJsID0gbV9mcmFtZS0+bG9hZGVyKCktPnVybCgpOworICAgIGlm
ICghdXJsLmlzVmFsaWQoKSkKKyAgICAgICAgcmV0dXJuIGJsYW5rVVJMKCk7ICAvLyBVc2UgImFi
b3V0OmJsYW5rIiB3aGlsZSB0aGUgcGFnZSBpcyBzdGlsbCBsb2FkaW5nIChiZWZvcmUgd2UgaGF2
ZSBhIGZyYW1lKS4KKworICAgIHJldHVybiB1cmw7CiB9CiAKIFN0cmluZyBMb2NhdGlvbjo6aHJl
ZigpIGNvbnN0CkluZGV4OiBMYXlvdXRUZXN0cy9DaGFuZ2VMb2cKPT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gTGF5
b3V0VGVzdHMvQ2hhbmdlTG9nCShyZXZpc2lvbiA0MTY1NykKKysrIExheW91dFRlc3RzL0NoYW5n
ZUxvZwkod29ya2luZyBjb3B5KQpAQCAtMSwzICsxLDEyIEBACisyMDA5LTAzLTEyICBNaWtlIEJl
bHNoZSAgPG1pa2VAYmVsc2hlLmNvbT4KKworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9P
UFMhKS4KKyAgICAgICAgRml4IGJ1ZyAyMTU5NzogdXBkYXRlIGxvY2F0aW9uIHdoaWxlIHBhZ2Ug
aXMgbG9hZGluZy4KKworICAgICAgICAqIGZhc3QvZG9tL1dpbmRvdy93aW5kb3ctb3Blbi1wZW5k
aW5nLXVybC1leHBlY3RlZC50eHQ6CisgICAgICAgICogZmFzdC9kb20vbG9jYXRpb24tbmV3LXdp
bmRvdy1uby1jcmFzaC1leHBlY3RlZC50eHQ6CisgICAgICAgICogZmFzdC9kb20vcmVzb3VyY2Vz
L2xvY2F0aW9uLW5ldy13aW5kb3ctbm8tY3Jhc2guanM6CisKIDIwMDktMDItMjYgIEVyaWMgU2Vp
ZGVsICA8ZXJpY0B3ZWJraXQub3JnPgogCiAgICAgICAgIFJldmlld2VkIGJ5IEp1c3RpbiBHYXJj
aWEuCkluZGV4OiBMYXlvdXRUZXN0cy9mYXN0L2RvbS9sb2NhdGlvbi1uZXctd2luZG93LW5vLWNy
YXNoLWV4cGVjdGVkLnR4dAo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBMYXlvdXRUZXN0cy9mYXN0L2RvbS9sb2Nh
dGlvbi1uZXctd2luZG93LW5vLWNyYXNoLWV4cGVjdGVkLnR4dAkocmV2aXNpb24gNDE2MjgpCisr
KyBMYXlvdXRUZXN0cy9mYXN0L2RvbS9sb2NhdGlvbi1uZXctd2luZG93LW5vLWNyYXNoLWV4cGVj
dGVkLnR4dAkod29ya2luZyBjb3B5KQpAQCAtMywxMyArMywxMyBAQCBUZXN0cyB0aGF0IG1hbmlw
dWxhdGluZyBsb2NhdGlvbiBwcm9wZXJ0CiBPbiBzdWNjZXNzLCB5b3Ugd2lsbCBzZWUgYSBzZXJp
ZXMgb2YgIlBBU1MiIG1lc3NhZ2VzLCBmb2xsb3dlZCBieSAiVEVTVCBDT01QTEVURSIuCiAKIAot
UEFTUyB0ZXN0V2luZG93LmxvY2F0aW9uLnRvU3RyaW5nKCkgaXMgJy8nCi1QQVNTIHRlc3RXaW5k
b3cubG9jYXRpb24uaHJlZiBpcyAnLycKLVBBU1MgdGVzdFdpbmRvdy5sb2NhdGlvbi5wcm90b2Nv
bCBpcyAnOicKK1BBU1MgdGVzdFdpbmRvdy5sb2NhdGlvbi50b1N0cmluZygpIGlzICdhYm91dDpi
bGFuaycKK1BBU1MgdGVzdFdpbmRvdy5sb2NhdGlvbi5ocmVmIGlzICdhYm91dDpibGFuaycKK1BB
U1MgdGVzdFdpbmRvdy5sb2NhdGlvbi5wcm90b2NvbCBpcyAnYWJvdXQ6JwogUEFTUyB0ZXN0V2lu
ZG93LmxvY2F0aW9uLmhvc3QgaXMgJycKIFBBU1MgdGVzdFdpbmRvdy5sb2NhdGlvbi5ob3N0bmFt
ZSBpcyAnJwogUEFTUyB0ZXN0V2luZG93LmxvY2F0aW9uLnBvcnQgaXMgJycKLVBBU1MgdGVzdFdp
bmRvdy5sb2NhdGlvbi5wYXRobmFtZSBpcyAnLycKK1BBU1MgdGVzdFdpbmRvdy5sb2NhdGlvbi5w
YXRobmFtZSBpcyAnYmxhbmsnCiBQQVNTIHRlc3RXaW5kb3cubG9jYXRpb24uc2VhcmNoIGlzICcn
CiBQQVNTIHRlc3RXaW5kb3cubG9jYXRpb24uaGFzaCBpcyAnJwogUEFTUyB0ZXN0V2luZG93Lmxv
Y2F0aW9uLmhyZWYgPSAnZGF0YTp0ZXh0L3BsYWluLGInIGlzICdkYXRhOnRleHQvcGxhaW4sYicK
QEAgLTIzLDEzICsyMywxMyBAQCBQQVNTIHRlc3RXaW5kb3cubG9jYXRpb24uaGFzaCA9ICdoJyBp
cyAnCiBQQVNTIHRlc3RXaW5kb3cubG9jYXRpb24uYXNzaWduKCdkYXRhOnRleHQvcGxhaW4saScp
IGlzIHVuZGVmaW5lZAogUEFTUyB0ZXN0V2luZG93LmxvY2F0aW9uLnJlcGxhY2UoJ2RhdGE6dGV4
dC9wbGFpbixqJykgaXMgdW5kZWZpbmVkCiBQQVNTIHRlc3RXaW5kb3cubG9jYXRpb24ucmVsb2Fk
KCkgaXMgdW5kZWZpbmVkCi1QQVNTIHRlc3RXaW5kb3cubG9jYXRpb24udG9TdHJpbmcoKSBpcyAn
LycKLVBBU1MgdGVzdFdpbmRvdy5sb2NhdGlvbi5ocmVmIGlzICcvJwotUEFTUyB0ZXN0V2luZG93
LmxvY2F0aW9uLnByb3RvY29sIGlzICc6JworUEFTUyB0ZXN0V2luZG93LmxvY2F0aW9uLnRvU3Ry
aW5nKCkgaXMgJ2Fib3V0OmJsYW5rJworUEFTUyB0ZXN0V2luZG93LmxvY2F0aW9uLmhyZWYgaXMg
J2Fib3V0OmJsYW5rJworUEFTUyB0ZXN0V2luZG93LmxvY2F0aW9uLnByb3RvY29sIGlzICdhYm91
dDonCiBQQVNTIHRlc3RXaW5kb3cubG9jYXRpb24uaG9zdCBpcyAnJwogUEFTUyB0ZXN0V2luZG93
LmxvY2F0aW9uLmhvc3RuYW1lIGlzICcnCiBQQVNTIHRlc3RXaW5kb3cubG9jYXRpb24ucG9ydCBp
cyAnJwotUEFTUyB0ZXN0V2luZG93LmxvY2F0aW9uLnBhdGhuYW1lIGlzICcvJworUEFTUyB0ZXN0
V2luZG93LmxvY2F0aW9uLnBhdGhuYW1lIGlzICdibGFuaycKIFBBU1MgdGVzdFdpbmRvdy5sb2Nh
dGlvbi5zZWFyY2ggaXMgJycKIFBBU1MgdGVzdFdpbmRvdy5sb2NhdGlvbi5oYXNoIGlzICcnCiBQ
QVNTIHN1Y2Nlc3NmdWxseVBhcnNlZCBpcyB0cnVlCkluZGV4OiBMYXlvdXRUZXN0cy9mYXN0L2Rv
bS9XaW5kb3cvd2luZG93LW9wZW4tcGVuZGluZy11cmwtZXhwZWN0ZWQudHh0Cj09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0K
LS0tIExheW91dFRlc3RzL2Zhc3QvZG9tL1dpbmRvdy93aW5kb3ctb3Blbi1wZW5kaW5nLXVybC1l
eHBlY3RlZC50eHQJKHJldmlzaW9uIDQxNjI4KQorKysgTGF5b3V0VGVzdHMvZmFzdC9kb20vV2lu
ZG93L3dpbmRvdy1vcGVuLXBlbmRpbmctdXJsLWV4cGVjdGVkLnR4dAkod29ya2luZyBjb3B5KQpA
QCAtMSwxMCArMSwxMCBAQAotbG9jYXRpb246IC8KLWhyZWY6IC8KLXBhdGhuYW1lOiAvCitsb2Nh
dGlvbjogYWJvdXQ6YmxhbmsKK2hyZWY6IGFib3V0OmJsYW5rCitwYXRobmFtZTogYmxhbmsKIGhv
c3Q6IAogaG9zdG5hbWU6IAogaGFzaDogCiBwb3J0OiAKLXByb3RvY29sOiA6Citwcm90b2NvbDog
YWJvdXQ6CiBzZWFyY2g6IAogCkluZGV4OiBMYXlvdXRUZXN0cy9mYXN0L2RvbS9yZXNvdXJjZXMv
bG9jYXRpb24tbmV3LXdpbmRvdy1uby1jcmFzaC5qcwo9PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBMYXlvdXRUZXN0
cy9mYXN0L2RvbS9yZXNvdXJjZXMvbG9jYXRpb24tbmV3LXdpbmRvdy1uby1jcmFzaC5qcwkocmV2
aXNpb24gNDE2MjgpCisrKyBMYXlvdXRUZXN0cy9mYXN0L2RvbS9yZXNvdXJjZXMvbG9jYXRpb24t
bmV3LXdpbmRvdy1uby1jcmFzaC5qcwkod29ya2luZyBjb3B5KQpAQCAtMTEsMTMgKzExLDEzIEBA
IHZhciB0ZXN0V2luZG93ID0gb3BlbigiZGF0YTp0ZXh0L3BsYWluLGEKIC8vIG9mIHRoZSB0ZXN0
LiBXZSdyZSBjaGVja2luZyBiZWhhdmlvciB3aGVuIHRoZSBvYmplY3Qgd2FzIGp1c3QgY3JlYXRl
ZCBhbmQgaXMgbm90IHlldCBhdCBpdHMgbmV3CiAvLyBsb2NhdGlvbi4KIAotc2hvdWxkQmUoInRl
c3RXaW5kb3cubG9jYXRpb24udG9TdHJpbmcoKSIsICInLyciKTsgLy8gRmlyZWZveCByZXR1cm5z
IGFib3V0OmJsYW5rCi1zaG91bGRCZSgidGVzdFdpbmRvdy5sb2NhdGlvbi5ocmVmIiwgIicvJyIp
OyAvLyBGaXJlZm94IHJldHVybnMgYWJvdXQ6YmxhbmsKLXNob3VsZEJlKCJ0ZXN0V2luZG93Lmxv
Y2F0aW9uLnByb3RvY29sIiwgIic6JyIpOyAvLyBGaXJlZm94IHJldHVybnMgYWJvdXQ6CitzaG91
bGRCZSgidGVzdFdpbmRvdy5sb2NhdGlvbi50b1N0cmluZygpIiwgIidhYm91dDpibGFuayciKTsK
K3Nob3VsZEJlKCJ0ZXN0V2luZG93LmxvY2F0aW9uLmhyZWYiLCAiJ2Fib3V0OmJsYW5rJyIpOwor
c2hvdWxkQmUoInRlc3RXaW5kb3cubG9jYXRpb24ucHJvdG9jb2wiLCAiJ2Fib3V0OiciKTsKIHNo
b3VsZEJlKCJ0ZXN0V2luZG93LmxvY2F0aW9uLmhvc3QiLCAiJyciKTsgLy8gRmlyZWZveCB0aHJv
d3MgYW4gZXhjZXB0aW9uCiBzaG91bGRCZSgidGVzdFdpbmRvdy5sb2NhdGlvbi5ob3N0bmFtZSIs
ICInJyIpOyAvLyBGaXJlZm94IHRocm93cyBhbiBleGNlcHRpb24KIHNob3VsZEJlKCJ0ZXN0V2lu
ZG93LmxvY2F0aW9uLnBvcnQiLCAiJyciKTsKLXNob3VsZEJlKCJ0ZXN0V2luZG93LmxvY2F0aW9u
LnBhdGhuYW1lIiwgIicvJyIpOyAvLyBGaXJlZm94IHJldHVybnMgdGhlIGVtcHR5IHN0cmluZwor
c2hvdWxkQmUoInRlc3RXaW5kb3cubG9jYXRpb24ucGF0aG5hbWUiLCAiJ2JsYW5rJyIpOyAvLyBG
aXJlZm94IHJldHVybnMgdGhlIGVtcHR5IHN0cmluZwogc2hvdWxkQmUoInRlc3RXaW5kb3cubG9j
YXRpb24uc2VhcmNoIiwgIicnIik7CiBzaG91bGRCZSgidGVzdFdpbmRvdy5sb2NhdGlvbi5oYXNo
IiwgIicnIik7CiAKQEAgLTM0LDEzICszNCwxMyBAQCBzaG91bGRCZSgidGVzdFdpbmRvdy5sb2Nh
dGlvbi5hc3NpZ24oJ2RhCiBzaG91bGRCZSgidGVzdFdpbmRvdy5sb2NhdGlvbi5yZXBsYWNlKCdk
YXRhOnRleHQvcGxhaW4saicpIiwgInVuZGVmaW5lZCIpOwogc2hvdWxkQmUoInRlc3RXaW5kb3cu
bG9jYXRpb24ucmVsb2FkKCkiLCAidW5kZWZpbmVkIik7CiAKLXNob3VsZEJlKCJ0ZXN0V2luZG93
LmxvY2F0aW9uLnRvU3RyaW5nKCkiLCAiJy8nIik7IC8vIEZpcmVmb3ggcmV0dXJucyBhYm91dDpi
bGFuawotc2hvdWxkQmUoInRlc3RXaW5kb3cubG9jYXRpb24uaHJlZiIsICInLyciKTsgLy8gRmly
ZWZveCByZXR1cm5zIGFib3V0OmJsYW5rCi1zaG91bGRCZSgidGVzdFdpbmRvdy5sb2NhdGlvbi5w
cm90b2NvbCIsICInOiciKTsgLy8gRmlyZWZveCByZXR1cm5zIGFib3V0Ogorc2hvdWxkQmUoInRl
c3RXaW5kb3cubG9jYXRpb24udG9TdHJpbmcoKSIsICInYWJvdXQ6YmxhbmsnIik7CitzaG91bGRC
ZSgidGVzdFdpbmRvdy5sb2NhdGlvbi5ocmVmIiwgIidhYm91dDpibGFuayciKTsKK3Nob3VsZEJl
KCJ0ZXN0V2luZG93LmxvY2F0aW9uLnByb3RvY29sIiwgIidhYm91dDonIik7CiBzaG91bGRCZSgi
dGVzdFdpbmRvdy5sb2NhdGlvbi5ob3N0IiwgIicnIik7IC8vIEZpcmVmb3ggdGhyb3dzIGFuIGV4
Y2VwdGlvbgogc2hvdWxkQmUoInRlc3RXaW5kb3cubG9jYXRpb24uaG9zdG5hbWUiLCAiJyciKTsg
Ly8gRmlyZWZveCB0aHJvd3MgYW4gZXhjZXB0aW9uCiBzaG91bGRCZSgidGVzdFdpbmRvdy5sb2Nh
dGlvbi5wb3J0IiwgIicnIik7Ci1zaG91bGRCZSgidGVzdFdpbmRvdy5sb2NhdGlvbi5wYXRobmFt
ZSIsICInLyciKTsgLy8gRmlyZWZveCByZXR1cm5zIHRoZSBlbXB0eSBzdHJpbmcKK3Nob3VsZEJl
KCJ0ZXN0V2luZG93LmxvY2F0aW9uLnBhdGhuYW1lIiwgIidibGFuayciKTsgLy8gRmlyZWZveCBy
ZXR1cm5zIHRoZSBlbXB0eSBzdHJpbmcKIHNob3VsZEJlKCJ0ZXN0V2luZG93LmxvY2F0aW9uLnNl
YXJjaCIsICInJyIpOwogc2hvdWxkQmUoInRlc3RXaW5kb3cubG9jYXRpb24uaGFzaCIsICInJyIp
OwogCg==
</data>
<flag name="review"
          id="14054"
          type_id="1"
          status="+"
          setter="eric"
    />
          </attachment>
      

    </bug>

</bugzilla>