<?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>27394</bug_id>
          
          <creation_ts>2009-07-17 15:53:22 -0700</creation_ts>
          <short_desc>V8: Selenium test failures</short_desc>
          <delta_ts>2009-07-17 17:24:12 -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 JavaScript</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>PC</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>
          
          
          <everconfirmed>0</everconfirmed>
          <reporter name="Mads Ager">ager</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>dglazkov</cc>
    
    <cc>eroman</cc>
    
    <cc>sam</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>132743</commentid>
    <comment_count>0</comment_count>
    <who name="Mads Ager">ager</who>
    <bug_when>2009-07-17 15:53:22 -0700</bug_when>
    <thetext>After 

http://trac.webkit.org/changeset/45797

there are new selenium test failures using WebCore/V8.

If javascript code holds a reference to the global object of a frame we need to eagerly initialize the JavaScript context when a new document is loaded in that frame to allow access to properties of that global object.  This kind of pattern is used in selenium tests to detect page loads.

Patch coming up.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>132745</commentid>
    <comment_count>1</comment_count>
      <attachid>32986</attachid>
    <who name="Mads Ager">ager</who>
    <bug_when>2009-07-17 15:55:47 -0700</bug_when>
    <thetext>Created attachment 32986
Fix global access after navigation</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>132752</commentid>
    <comment_count>2</comment_count>
      <attachid>32986</attachid>
    <who name="Dimitri Glazkov (Google)">dglazkov</who>
    <bug_when>2009-07-17 16:01:42 -0700</bug_when>
    <thetext>Comment on attachment 32986
Fix global access after navigation

ok</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>132771</commentid>
    <comment_count>3</comment_count>
    <who name="Sam Weinig">sam</who>
    <bug_when>2009-07-17 16:39:54 -0700</bug_when>
    <thetext>Is there at test that can be added to the test suite for this bug fix?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>132777</commentid>
    <comment_count>4</comment_count>
    <who name="Mads Ager">ager</who>
    <bug_when>2009-07-17 16:50:41 -0700</bug_when>
    <thetext>Yes, you are right.  I should add a layout test with a reduced test case.  Let me put something together and upload that as a separate patch.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>132786</commentid>
    <comment_count>5</comment_count>
    <who name="Eric Roman">eroman</who>
    <bug_when>2009-07-17 17:12:58 -0700</bug_when>
    <thetext>Landed r46073.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>32986</attachid>
            <date>2009-07-17 15:55:47 -0700</date>
            <delta_ts>2009-07-17 16:01:41 -0700</delta_ts>
            <desc>Fix global access after navigation</desc>
            <filename>global-access-after-navigation.txt</filename>
            <type>text/plain</type>
            <size>1486</size>
            <attacher name="Mads Ager">ager</attacher>
            
              <data encoding="base64">SW5kZXg6IFdlYkNvcmUvQ2hhbmdlTG9nCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFdlYkNvcmUvQ2hhbmdlTG9n
CShyZXZpc2lvbiA0NjA2MCkKKysrIFdlYkNvcmUvQ2hhbmdlTG9nCSh3b3JraW5nIGNvcHkpCkBA
IC0xLDMgKzEsMTQgQEAKKzIwMDktMDctMTcgIE1hZHMgQWdlciAgPGFnZXJAY2hyb21pdW0ub3Jn
PgorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgIGh0dHBz
Oi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD0yNzM5NAorICAgICAgICBGaXggYWNj
ZXNzIHRvIGdsb2JhbCBvYmplY3Qgd3JhcHBlcnMgYWZ0ZXIgbmF2aWdhdGlvbiBvZiB0aGVpcgor
ICAgICAgICBmcmFtZSBpbiB0aGUgVjggYmluZGluZ3MuICBUaGlzIGZpeGVzIHNlbGVuaXVtIHRl
c3QgZmFpbHVyZXMuCisKKyAgICAgICAgKiBiaW5kaW5ncy92OC9WOFByb3h5LmNwcDoKKyAgICAg
ICAgKFdlYkNvcmU6OlY4UHJveHk6OnVwZGF0ZURvY3VtZW50KToKKwogMjAwOS0wNy0xNyAgUGV0
ZXIgS2FzdGluZyAgPHBrYXN0aW5nQGdvb2dsZS5jb20+CiAKICAgICAgICAgUmV2aWV3ZWQgYnkg
U3RldmUgRmFsa2VuYnVyZy4KSW5kZXg6IFdlYkNvcmUvYmluZGluZ3MvdjgvVjhQcm94eS5jcHAK
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PQotLS0gV2ViQ29yZS9iaW5kaW5ncy92OC9WOFByb3h5LmNwcAkocmV2aXNpb24g
NDYwNTgpCisrKyBXZWJDb3JlL2JpbmRpbmdzL3Y4L1Y4UHJveHkuY3BwCSh3b3JraW5nIGNvcHkp
CkBAIC0xMDAwLDkgKzEwMDAsMTYgQEAgdm9pZCBWOFByb3h5Ojp1cGRhdGVEb2N1bWVudCgpCiAg
ICAgaWYgKCFtX2ZyYW1lLT5kb2N1bWVudCgpKQogICAgICAgICByZXR1cm47CiAKLSAgICBpZiAo
bV9jb250ZXh0LklzRW1wdHkoKSkKKyAgICBpZiAobV9nbG9iYWwuSXNFbXB0eSgpKQogICAgICAg
ICByZXR1cm47CiAKKyAgICAvLyBUaGVyZSBpcyBhbiBleGlzdGluZyBKYXZhU2NyaXB0IHdyYXBw
ZXIgZm9yIHRoZSBnbG9iYWwgb2JqZWN0CisgICAgLy8gb2YgdGhpcyBmcmFtZS4gSmF2YVNjcmlw
dCBjb2RlIGluIG90aGVyIGZyYW1lcyBtaWdodCBob2xkIGEKKyAgICAvLyByZWZlcmVuY2UgdG8g
dGhpcyB3cmFwcGVyLiBXZSBlYWdlcmx5IGluaXRpYWxpemUgdGhlIEphdmFTY3JpcHQKKyAgICAv
LyBjb250ZXh0IGZvciB0aGUgbmV3IGRvY3VtZW50IHRvIG1ha2UgcHJvcGVydHkgYWNjZXNzIG9u
IHRoZQorICAgIC8vIGdsb2JhbCBvYmplY3Qgd3JhcHBlciBzdWNjZWVkLgorICAgIGluaXRDb250
ZXh0SWZOZWVkZWQoKTsKKwogICAgIC8vIFdlIGhhdmUgYSBuZXcgZG9jdW1lbnQgYW5kIHdlIG5l
ZWQgdG8gdXBkYXRlIHRoZSBjYWNoZS4KICAgICB1cGRhdGVEb2N1bWVudFdyYXBwZXJDYWNoZSgp
OwogCg==
</data>
<flag name="review"
          id="17328"
          type_id="1"
          status="+"
          setter="dglazkov"
    />
          </attachment>
      

    </bug>

</bugzilla>