<?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>45790</bug_id>
          
          <creation_ts>2010-09-14 16:17:21 -0700</creation_ts>
          <short_desc>ASSERTION FAILED: m_loadEventDelayCount</short_desc>
          <delta_ts>2010-09-17 11:04:37 -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>Other</rep_platform>
          <op_sys>Other</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="Tony Gentilcore">tonyg</reporter>
          <assigned_to name="Tony Gentilcore">tonyg</assigned_to>
          <cc>abarth</cc>
    
    <cc>alex</cc>
    
    <cc>ap</cc>
    
    <cc>commit-queue</cc>
    
    <cc>eric.carlson</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>279243</commentid>
    <comment_count>0</comment_count>
    <who name="Tony Gentilcore">tonyg</who>
    <bug_when>2010-09-14 16:17:21 -0700</bug_when>
    <thetext>On the GTK linux 32 bot, we are hitting this ASSERT occasionally.

ASSERTION FAILED: m_loadEventDelayCount
(../../WebCore/dom/Document.cpp:4785 void WebCore::Document::decrementLoadEventDelayCount())

An example of a test that has triggered it is:
fast/dom/HTMLScriptElement/script-decoding-error-after-setting-src.html

It is possible this isn&apos;t really port-specific, but just happens to have triggered there. This was almost certainly introduced by https://bugs.webkit.org/show_bug.cgi?id=45310

Stack:
#0  0x55aa3d98 in WebCore::Document::decrementLoadEventDelayCount (
    this=0xc62b788) at ../../WebCore/dom/Document.cpp:4750
#1  0x55a75b5c in WebCore::AsyncScriptRunner::timerFired (this=0x956d260, 
    timer=0x956d270) at ../../WebCore/dom/AsyncScriptRunner.cpp:87
#2  0x55a7652a in WebCore::Timer&lt;WebCore::AsyncScriptRunner&gt;::fired (
    this=0x956d270) at ../../WebCore/platform/Timer.h:98
#3  0x55ea42e8 in WebCore::ThreadTimers::sharedTimerFiredInternal (
    this=0x8148730) at ../../WebCore/platform/ThreadTimers.cpp:112
#4  0x55ea422f in WebCore::ThreadTimers::sharedTimerFired ()
    at ../../WebCore/platform/ThreadTimers.cpp:90
#5  0x561cd0ad in timeout_cb ()
    at ../../WebCore/platform/gtk/SharedTimerGtk.cpp:49
#6  0x5923da6c in g_timeout_dispatch (source=0x954c3b0, callback=0xbbadbeef, 
    user_data=0x0)
    at /build/buildd-glib2.0_2.24.1-1-i386-84Pp4V/glib2.0-2.24.1/glib/gmain.c:3396
#7  0x5923d2f5 in g_main_dispatch (context=0x8094b68)
    at /build/buildd-glib2.0_2.24.1-1-i386-84Pp4V/glib2.0-2.24.1/glib/gmain.c:1960
#8  IA__g_main_context_dispatch (context=0x8094b68)
    at /build/buildd-glib2.0_2.24.1-1-i386-84Pp4V/glib2.0-2.24.1/glib/gmain.c:2513
#9  0x59240fd8 in g_main_context_iterate (context=0x8094b68, 
    block=&lt;value optimized out&gt;, dispatch=1, self=0x8073268)
    at /build/buildd-glib2.0_2.24.1-1-i386-84Pp4V/glib2.0-2.24.1/glib/gmain.c:2591
#10 0x59241517 in IA__g_main_loop_run (loop=0xc636eb0)
    at /build/buildd-glib2.0_2.24.1-1-i386-84Pp4V/glib2.0-2.24.1/glib/gmain.c:2799
#11 0x58cebdc9 in IA__gtk_main ()
    at /build/buildd-gtk+2.0_2.20.1-1-i386-Ixfflh/gtk+2.0-2.20.1/gtk/gtkmain.c:1219
#12 0x0805c451 in runTest (testPathOrURL=...)
    at ../../WebKitTools/DumpRenderTree/gtk/DumpRenderTree.cpp:573
#13 0x0805bba7 in runTestingServerLoop ()
    at ../../WebKitTools/DumpRenderTree/gtk/DumpRenderTree.cpp:383
#14 0x0805d879 in main (argc=2, argv=0xffffc524)
    at ../../WebKitTools/DumpRenderTree/gtk/DumpRenderTree.cpp:1013</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>280228</commentid>
    <comment_count>1</comment_count>
      <attachid>67823</attachid>
    <who name="Tony Gentilcore">tonyg</who>
    <bug_when>2010-09-16 11:34:27 -0700</bug_when>
    <thetext>Created attachment 67823
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>280313</commentid>
    <comment_count>2</comment_count>
      <attachid>67823</attachid>
    <who name="Adam Barth">abarth</who>
    <bug_when>2010-09-16 13:34:17 -0700</bug_when>
    <thetext>Comment on attachment 67823
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=67823&amp;action=prettypatch

&gt; WebCore/WebCore.xcodeproj/project.pbxproj:2388
&gt; -		893C47DF123EF4A9002B3D86 /* JSDirectoryEntryCustom.cpp in Sources */ = {isa = PBXBuildFile; fileRef = 893C47DE123EF4A9002B3D86 /* JSDirectoryEntryCustom.cpp */; };
&gt;  		893C47CC123EEBA2002B3D86 /* JSEntryCustom.cpp in Sources */ = {isa = PBXBuildFile; fileRef = 893C47CA123EEBA2002B3D86 /* JSEntryCustom.cpp */; };
&gt; +		893C47DF123EF4A9002B3D86 /* JSDirectoryEntryCustom.cpp in Sources */ = {isa = PBXBuildFile; fileRef = 893C47DE123EF4A9002B3D86 /* JSDirectoryEntryCustom.cpp */; };

These changes seems spurious.  Please land without them.  If the project file is wrong, we can fix it in a separate commit.

&gt; WebCore/dom/AsyncScriptRunner.cpp:81
&gt; +    RefPtr&lt;Document&gt; document(m_document);

Usually we&apos;d call this variable protector and then just m_document through the function.

&gt; WebCore/dom/AsyncScriptRunner.cpp:87
&gt;          scripts[i].first-&gt;execute(scripts[i].second.get());

Does this execute script when the document is detached from the frame?  It&apos;s an invariant that we never execute scripts on behalf of detached documents.  (Not really an issue for this patch, just something to think about.)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>280327</commentid>
    <comment_count>3</comment_count>
    <who name="Tony Gentilcore">tonyg</who>
    <bug_when>2010-09-16 13:44:16 -0700</bug_when>
    <thetext>&gt; These changes seems spurious.  Please land without them.  If the project file is wrong, we can fix it in a separate commit.

OK

&gt; Usually we&apos;d call this variable protector and then just m_document through the function.

Ah, right.

But I went with protect over protector.
$ grep -e &quot;protect[(]&quot; * -r | wc -l
     108
$ grep -e &quot;protector[(]&quot; * -r | wc -l
      39

&gt; Does this execute script when the document is detached from the frame?  It&apos;s an invariant that we never execute scripts on behalf of detached documents.  (Not really an issue for this patch, just something to think about.)

I&apos;ll try to add a test and look into it separately.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>280329</commentid>
    <comment_count>4</comment_count>
      <attachid>67837</attachid>
    <who name="Tony Gentilcore">tonyg</who>
    <bug_when>2010-09-16 13:45:29 -0700</bug_when>
    <thetext>Created attachment 67837
Patch for landing</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>280336</commentid>
    <comment_count>5</comment_count>
    <who name="Alexey Proskuryakov">ap</who>
    <bug_when>2010-09-16 13:50:47 -0700</bug_when>
    <thetext>Yes, protect(something) reads better than protector(something), as long as you&apos;re never using the protector object.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>280408</commentid>
    <comment_count>6</comment_count>
    <who name="Adam Barth">abarth</who>
    <bug_when>2010-09-16 15:40:45 -0700</bug_when>
    <thetext>(In reply to comment #5)
&gt; Yes, protect(something) reads better than protector(something), as long as you&apos;re never using the protector object.

/me look around to see who wants to fix the 39 stray occurrences.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>280745</commentid>
    <comment_count>7</comment_count>
      <attachid>67837</attachid>
    <who name="Adam Barth">abarth</who>
    <bug_when>2010-09-17 11:04:32 -0700</bug_when>
    <thetext>Comment on attachment 67837
Patch for landing

Clearing flags on attachment: 67837

Committed r67728: &lt;http://trac.webkit.org/changeset/67728&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>280746</commentid>
    <comment_count>8</comment_count>
    <who name="Adam Barth">abarth</who>
    <bug_when>2010-09-17 11:04:37 -0700</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>67823</attachid>
            <date>2010-09-16 11:34:27 -0700</date>
            <delta_ts>2010-09-16 13:45:23 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-45790-20100916113426.patch</filename>
            <type>text/plain</type>
            <size>5310</size>
            <attacher name="Tony Gentilcore">tonyg</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1dlYkNvcmUvQ2hhbmdlTG9nIGIvV2ViQ29yZS9DaGFuZ2VMb2cKaW5kZXgg
ZDIwZmJhMDE2M2I4ZWE4ZGUzNjM2MTc3YzA0MGMyNjdiY2M4Y2RlYi4uZWRmYzJjYzYwNjZhZjFj
MjExNWQ4Y2EwZGNjNWY0NGRlZTY0ZDhlMyAxMDA2NDQKLS0tIGEvV2ViQ29yZS9DaGFuZ2VMb2cK
KysrIGIvV2ViQ29yZS9DaGFuZ2VMb2cKQEAgLTEsMyArMSwxNCBAQAorMjAxMC0wOS0xNiAgVG9u
eSBHZW50aWxjb3JlICA8dG9ueWdAY2hyb21pdW0ub3JnPgorCisgICAgICAgIFJldmlld2VkIGJ5
IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgIEFTU0VSVElPTiBGQUlMRUQ6IG1fbG9hZEV2ZW50
RGVsYXlDb3VudAorICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/
aWQ9NDU3OTAKKworICAgICAgICAqIFdlYkNvcmUueGNvZGVwcm9qL3Byb2plY3QucGJ4cHJvajog
WENvZGUgYXV0b21hdGljYWxseSBjaGFuZ2VkIHRoaXMuIEkgZ3Vlc3MgaXQgaGFzIHRvIGJlIGZs
aXBwZWQgc29tZXRpbWUgc28gaXQgaXMgZ2V0dGluZyBhIHJpZGUgb24gdGhpcyBwYXRjaC4KKyAg
ICAgICAgKiBkb20vQXN5bmNTY3JpcHRSdW5uZXIuY3BwOiAKKyAgICAgICAgKFdlYkNvcmU6OkFz
eW5jU2NyaXB0UnVubmVyOjp0aW1lckZpcmVkKTogUHJvdGVjdCBkb2N1bWVudCBiZWZvcmUgd2Ug
Z28gcnVubmluZyBzY3JpcHRzLgorCiAyMDEwLTA5LTE2ICBEYW5pZWwgQ2hlbmcgIDxkY2hlbmdA
Y2hyb21pdW0ub3JnPgogCiAgICAgICAgIFJldmlld2VkIGJ5IFRvbnkgQ2hhbmcuCmRpZmYgLS1n
aXQgYS9XZWJDb3JlL1dlYkNvcmUueGNvZGVwcm9qL3Byb2plY3QucGJ4cHJvaiBiL1dlYkNvcmUv
V2ViQ29yZS54Y29kZXByb2ovcHJvamVjdC5wYnhwcm9qCmluZGV4IDViMGFlODU2MTlhMjRjNjBi
MTFiYjUyNjJhYTU3YzlkNjQ3NWQxZTAuLjBlZGE0NjI3NzU2M2Y3YmI2YzU3Mzg0MDc5NGEwYzQ1
MWRhNmJlYzcgMTAwNjQ0Ci0tLSBhL1dlYkNvcmUvV2ViQ29yZS54Y29kZXByb2ovcHJvamVjdC5w
Ynhwcm9qCisrKyBiL1dlYkNvcmUvV2ViQ29yZS54Y29kZXByb2ovcHJvamVjdC5wYnhwcm9qCkBA
IC0yMzg0LDggKzIzODQsOCBAQAogCQk4OTNDNDdCODEyMzhBMDk5MDAyQjNEODYgLyogSlNGaWxl
Q2FsbGJhY2suaCBpbiBIZWFkZXJzICovID0ge2lzYSA9IFBCWEJ1aWxkRmlsZTsgZmlsZVJlZiA9
IDg5M0M0N0I2MTIzOEEwOTkwMDJCM0Q4NiAvKiBKU0ZpbGVDYWxsYmFjay5oICovOyB9OwogCQk4
OTNDNDdCQjEyMzhBMEE5MDAyQjNEODYgLyogSlNGaWxlV3JpdGVyQ2FsbGJhY2suY3BwIGluIFNv
dXJjZXMgKi8gPSB7aXNhID0gUEJYQnVpbGRGaWxlOyBmaWxlUmVmID0gODkzQzQ3QjkxMjM4QTBB
OTAwMkIzRDg2IC8qIEpTRmlsZVdyaXRlckNhbGxiYWNrLmNwcCAqLzsgfTsKIAkJODkzQzQ3QkMx
MjM4QTBBOTAwMkIzRDg2IC8qIEpTRmlsZVdyaXRlckNhbGxiYWNrLmggaW4gSGVhZGVycyAqLyA9
IHtpc2EgPSBQQlhCdWlsZEZpbGU7IGZpbGVSZWYgPSA4OTNDNDdCQTEyMzhBMEE5MDAyQjNEODYg
LyogSlNGaWxlV3JpdGVyQ2FsbGJhY2suaCAqLzsgfTsKLQkJODkzQzQ3REYxMjNFRjRBOTAwMkIz
RDg2IC8qIEpTRGlyZWN0b3J5RW50cnlDdXN0b20uY3BwIGluIFNvdXJjZXMgKi8gPSB7aXNhID0g
UEJYQnVpbGRGaWxlOyBmaWxlUmVmID0gODkzQzQ3REUxMjNFRjRBOTAwMkIzRDg2IC8qIEpTRGly
ZWN0b3J5RW50cnlDdXN0b20uY3BwICovOyB9OwogCQk4OTNDNDdDQzEyM0VFQkEyMDAyQjNEODYg
LyogSlNFbnRyeUN1c3RvbS5jcHAgaW4gU291cmNlcyAqLyA9IHtpc2EgPSBQQlhCdWlsZEZpbGU7
IGZpbGVSZWYgPSA4OTNDNDdDQTEyM0VFQkEyMDAyQjNEODYgLyogSlNFbnRyeUN1c3RvbS5jcHAg
Ki87IH07CisJCTg5M0M0N0RGMTIzRUY0QTkwMDJCM0Q4NiAvKiBKU0RpcmVjdG9yeUVudHJ5Q3Vz
dG9tLmNwcCBpbiBTb3VyY2VzICovID0ge2lzYSA9IFBCWEJ1aWxkRmlsZTsgZmlsZVJlZiA9IDg5
M0M0N0RFMTIzRUY0QTkwMDJCM0Q4NiAvKiBKU0RpcmVjdG9yeUVudHJ5Q3VzdG9tLmNwcCAqLzsg
fTsKIAkJODk4Nzg1NTIxMjJDQTA2NDAwM0FBQkRBIC8qIERpcmVjdG9yeUVudHJ5LmNwcCBpbiBT
b3VyY2VzICovID0ge2lzYSA9IFBCWEJ1aWxkRmlsZTsgZmlsZVJlZiA9IDg5ODc4NTM5MTIyQ0Ew
NjQwMDNBQUJEQSAvKiBEaXJlY3RvcnlFbnRyeS5jcHAgKi87IH07CiAJCTg5ODc4NTUzMTIyQ0Ew
NjQwMDNBQUJEQSAvKiBEaXJlY3RvcnlFbnRyeS5oIGluIEhlYWRlcnMgKi8gPSB7aXNhID0gUEJY
QnVpbGRGaWxlOyBmaWxlUmVmID0gODk4Nzg1M0ExMjJDQTA2NDAwM0FBQkRBIC8qIERpcmVjdG9y
eUVudHJ5LmggKi87IH07CiAJCTg5ODc4NTU0MTIyQ0EwNjQwMDNBQUJEQSAvKiBEaXJlY3RvcnlS
ZWFkZXIuY3BwIGluIFNvdXJjZXMgKi8gPSB7aXNhID0gUEJYQnVpbGRGaWxlOyBmaWxlUmVmID0g
ODk4Nzg1M0IxMjJDQTA2NDAwM0FBQkRBIC8qIERpcmVjdG9yeVJlYWRlci5jcHAgKi87IH07CkBA
IC04MjkwLDggKzgyOTAsOCBAQAogCQk4OTNDNDdCNjEyMzhBMDk5MDAyQjNEODYgLyogSlNGaWxl
Q2FsbGJhY2suaCAqLyA9IHtpc2EgPSBQQlhGaWxlUmVmZXJlbmNlOyBmaWxlRW5jb2RpbmcgPSA0
OyBsYXN0S25vd25GaWxlVHlwZSA9IHNvdXJjZWNvZGUuYy5oOyBwYXRoID0gSlNGaWxlQ2FsbGJh
Y2suaDsgc291cmNlVHJlZSA9ICI8Z3JvdXA+IjsgfTsKIAkJODkzQzQ3QjkxMjM4QTBBOTAwMkIz
RDg2IC8qIEpTRmlsZVdyaXRlckNhbGxiYWNrLmNwcCAqLyA9IHtpc2EgPSBQQlhGaWxlUmVmZXJl
bmNlOyBmaWxlRW5jb2RpbmcgPSA0OyBsYXN0S25vd25GaWxlVHlwZSA9IHNvdXJjZWNvZGUuY3Bw
LmNwcDsgcGF0aCA9IEpTRmlsZVdyaXRlckNhbGxiYWNrLmNwcDsgc291cmNlVHJlZSA9ICI8Z3Jv
dXA+IjsgfTsKIAkJODkzQzQ3QkExMjM4QTBBOTAwMkIzRDg2IC8qIEpTRmlsZVdyaXRlckNhbGxi
YWNrLmggKi8gPSB7aXNhID0gUEJYRmlsZVJlZmVyZW5jZTsgZmlsZUVuY29kaW5nID0gNDsgbGFz
dEtub3duRmlsZVR5cGUgPSBzb3VyY2Vjb2RlLmMuaDsgcGF0aCA9IEpTRmlsZVdyaXRlckNhbGxi
YWNrLmg7IHNvdXJjZVRyZWUgPSAiPGdyb3VwPiI7IH07Ci0JCTg5M0M0N0RFMTIzRUY0QTkwMDJC
M0Q4NiAvKiBKU0RpcmVjdG9yeUVudHJ5Q3VzdG9tLmNwcCAqLyA9IHtpc2EgPSBQQlhGaWxlUmVm
ZXJlbmNlOyBmaWxlRW5jb2RpbmcgPSA0OyBsYXN0S25vd25GaWxlVHlwZSA9IHNvdXJjZWNvZGUu
Y3BwLmNwcDsgcGF0aCA9IEpTRGlyZWN0b3J5RW50cnlDdXN0b20uY3BwOyBzb3VyY2VUcmVlID0g
Ijxncm91cD4iOyB9OwogCQk4OTNDNDdDQTEyM0VFQkEyMDAyQjNEODYgLyogSlNFbnRyeUN1c3Rv
bS5jcHAgKi8gPSB7aXNhID0gUEJYRmlsZVJlZmVyZW5jZTsgZmlsZUVuY29kaW5nID0gNDsgbGFz
dEtub3duRmlsZVR5cGUgPSBzb3VyY2Vjb2RlLmNwcC5jcHA7IHBhdGggPSBKU0VudHJ5Q3VzdG9t
LmNwcDsgc291cmNlVHJlZSA9ICI8Z3JvdXA+IjsgfTsKKwkJODkzQzQ3REUxMjNFRjRBOTAwMkIz
RDg2IC8qIEpTRGlyZWN0b3J5RW50cnlDdXN0b20uY3BwICovID0ge2lzYSA9IFBCWEZpbGVSZWZl
cmVuY2U7IGZpbGVFbmNvZGluZyA9IDQ7IGxhc3RLbm93bkZpbGVUeXBlID0gc291cmNlY29kZS5j
cHAuY3BwOyBwYXRoID0gSlNEaXJlY3RvcnlFbnRyeUN1c3RvbS5jcHA7IHNvdXJjZVRyZWUgPSAi
PGdyb3VwPiI7IH07CiAJCTg5ODc4NTM5MTIyQ0EwNjQwMDNBQUJEQSAvKiBEaXJlY3RvcnlFbnRy
eS5jcHAgKi8gPSB7aXNhID0gUEJYRmlsZVJlZmVyZW5jZTsgZmlsZUVuY29kaW5nID0gNDsgbGFz
dEtub3duRmlsZVR5cGUgPSBzb3VyY2Vjb2RlLmNwcC5jcHA7IG5hbWUgPSBEaXJlY3RvcnlFbnRy
eS5jcHA7IHBhdGggPSBmaWxlYXBpL0RpcmVjdG9yeUVudHJ5LmNwcDsgc291cmNlVHJlZSA9ICI8
Z3JvdXA+IjsgfTsKIAkJODk4Nzg1M0ExMjJDQTA2NDAwM0FBQkRBIC8qIERpcmVjdG9yeUVudHJ5
LmggKi8gPSB7aXNhID0gUEJYRmlsZVJlZmVyZW5jZTsgZmlsZUVuY29kaW5nID0gNDsgbGFzdEtu
b3duRmlsZVR5cGUgPSBzb3VyY2Vjb2RlLmMuaDsgbmFtZSA9IERpcmVjdG9yeUVudHJ5Lmg7IHBh
dGggPSBmaWxlYXBpL0RpcmVjdG9yeUVudHJ5Lmg7IHNvdXJjZVRyZWUgPSAiPGdyb3VwPiI7IH07
CiAJCTg5ODc4NTNCMTIyQ0EwNjQwMDNBQUJEQSAvKiBEaXJlY3RvcnlSZWFkZXIuY3BwICovID0g
e2lzYSA9IFBCWEZpbGVSZWZlcmVuY2U7IGZpbGVFbmNvZGluZyA9IDQ7IGxhc3RLbm93bkZpbGVU
eXBlID0gc291cmNlY29kZS5jcHAuY3BwOyBuYW1lID0gRGlyZWN0b3J5UmVhZGVyLmNwcDsgcGF0
aCA9IGZpbGVhcGkvRGlyZWN0b3J5UmVhZGVyLmNwcDsgc291cmNlVHJlZSA9ICI8Z3JvdXA+Ijsg
fTsKZGlmZiAtLWdpdCBhL1dlYkNvcmUvZG9tL0FzeW5jU2NyaXB0UnVubmVyLmNwcCBiL1dlYkNv
cmUvZG9tL0FzeW5jU2NyaXB0UnVubmVyLmNwcAppbmRleCA4NjI1MWI4ZjNhOTZmOTI0YmE0NDU1
NmE4NGYzYmI2OTgxMjg5NzBlLi4zM2Y2MjRjYzIzODE1MmViZTY4YjJhNzgwZGY3YzAzZTVjODUz
NDI1IDEwMDY0NAotLS0gYS9XZWJDb3JlL2RvbS9Bc3luY1NjcmlwdFJ1bm5lci5jcHAKKysrIGIv
V2ViQ29yZS9kb20vQXN5bmNTY3JpcHRSdW5uZXIuY3BwCkBAIC03OCwxMyArNzgsMTUgQEAgdm9p
ZCBBc3luY1NjcmlwdFJ1bm5lcjo6dGltZXJGaXJlZChUaW1lcjxBc3luY1NjcmlwdFJ1bm5lcj4q
IHRpbWVyKQogewogICAgIEFTU0VSVF9VTlVTRUQodGltZXIsIHRpbWVyID09ICZtX3RpbWVyKTsK
IAorICAgIFJlZlB0cjxEb2N1bWVudD4gZG9jdW1lbnQobV9kb2N1bWVudCk7CisgICAgCiAgICAg
VmVjdG9yPHBhaXI8U2NyaXB0RWxlbWVudERhdGEqLCBDYWNoZWRSZXNvdXJjZUhhbmRsZTxDYWNo
ZWRTY3JpcHQ+ID4gPiBzY3JpcHRzOwogICAgIHNjcmlwdHMuc3dhcChtX3NjcmlwdHNUb0V4ZWN1
dGVTb29uKTsKICAgICBzaXplX3Qgc2l6ZSA9IHNjcmlwdHMuc2l6ZSgpOwogICAgIGZvciAoc2l6
ZV90IGkgPSAwOyBpIDwgc2l6ZTsgKytpKSB7CiAgICAgICAgIHNjcmlwdHNbaV0uZmlyc3QtPmV4
ZWN1dGUoc2NyaXB0c1tpXS5zZWNvbmQuZ2V0KCkpOwogICAgICAgICBzY3JpcHRzW2ldLmZpcnN0
LT5lbGVtZW50KCktPmRlcmVmKCk7IC8vIEJhbGFuY2VzIHJlZigpIGluIGV4ZWN1dGVTY3JpcHRT
b29uKCkuCi0gICAgICAgIG1fZG9jdW1lbnQtPmRlY3JlbWVudExvYWRFdmVudERlbGF5Q291bnQo
KTsKKyAgICAgICAgZG9jdW1lbnQtPmRlY3JlbWVudExvYWRFdmVudERlbGF5Q291bnQoKTsKICAg
ICB9CiB9CiAK
</data>

          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>67837</attachid>
            <date>2010-09-16 13:45:29 -0700</date>
            <delta_ts>2010-09-17 11:04:32 -0700</delta_ts>
            <desc>Patch for landing</desc>
            <filename>bug-45790-20100916134528.patch</filename>
            <type>text/plain</type>
            <size>1233</size>
            <attacher name="Tony Gentilcore">tonyg</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1dlYkNvcmUvQ2hhbmdlTG9nIGIvV2ViQ29yZS9DaGFuZ2VMb2cKaW5kZXgg
MDJlODYxOTY3YThlYmRkNmZmODVlZjE0Y2YxYWQ2YmZjYjljY2Y2MS4uN2Q3MzUwMzBlNjk4MGY0
ZGExMDg4NWMyMTgwZWJkOTFhNmQ2YTA4ZiAxMDA2NDQKLS0tIGEvV2ViQ29yZS9DaGFuZ2VMb2cK
KysrIGIvV2ViQ29yZS9DaGFuZ2VMb2cKQEAgLTEsMyArMSwxMyBAQAorMjAxMC0wOS0xNiAgVG9u
eSBHZW50aWxjb3JlICA8dG9ueWdAY2hyb21pdW0ub3JnPgorCisgICAgICAgIFJldmlld2VkIGJ5
IEFkYW0gQmFydGguCisKKyAgICAgICAgQVNTRVJUSU9OIEZBSUxFRDogbV9sb2FkRXZlbnREZWxh
eUNvdW50CisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD00
NTc5MAorCisgICAgICAgICogZG9tL0FzeW5jU2NyaXB0UnVubmVyLmNwcDogCisgICAgICAgIChX
ZWJDb3JlOjpBc3luY1NjcmlwdFJ1bm5lcjo6dGltZXJGaXJlZCk6IFByb3RlY3QgZG9jdW1lbnQg
YmVmb3JlIHdlIGdvIHJ1bm5pbmcgc2NyaXB0cy4KKwogMjAxMC0wOS0xNSAgRGF2aWQgSHlhdHQg
IDxoeWF0dEBhcHBsZS5jb20+CiAKICAgICAgICAgUmV2aWV3ZWQgYnkgRGFyaW4gQWRsZXIuCmRp
ZmYgLS1naXQgYS9XZWJDb3JlL2RvbS9Bc3luY1NjcmlwdFJ1bm5lci5jcHAgYi9XZWJDb3JlL2Rv
bS9Bc3luY1NjcmlwdFJ1bm5lci5jcHAKaW5kZXggODYyNTFiOGYzYTk2ZjkyNGJhNDQ1NTZhODRm
M2JiNjk4MTI4OTcwZS4uYTcyZGY3YzY1ODNkNGRhM2MzMDU1ZjE4NjdjNGQwYTU0MzA2NTFmNyAx
MDA2NDQKLS0tIGEvV2ViQ29yZS9kb20vQXN5bmNTY3JpcHRSdW5uZXIuY3BwCisrKyBiL1dlYkNv
cmUvZG9tL0FzeW5jU2NyaXB0UnVubmVyLmNwcApAQCAtNzgsNiArNzgsOCBAQCB2b2lkIEFzeW5j
U2NyaXB0UnVubmVyOjp0aW1lckZpcmVkKFRpbWVyPEFzeW5jU2NyaXB0UnVubmVyPiogdGltZXIp
CiB7CiAgICAgQVNTRVJUX1VOVVNFRCh0aW1lciwgdGltZXIgPT0gJm1fdGltZXIpOwogCisgICAg
UmVmUHRyPERvY3VtZW50PiBwcm90ZWN0KG1fZG9jdW1lbnQpOworICAgIAogICAgIFZlY3Rvcjxw
YWlyPFNjcmlwdEVsZW1lbnREYXRhKiwgQ2FjaGVkUmVzb3VyY2VIYW5kbGU8Q2FjaGVkU2NyaXB0
PiA+ID4gc2NyaXB0czsKICAgICBzY3JpcHRzLnN3YXAobV9zY3JpcHRzVG9FeGVjdXRlU29vbik7
CiAgICAgc2l6ZV90IHNpemUgPSBzY3JpcHRzLnNpemUoKTsK
</data>

          </attachment>
      

    </bug>

</bugzilla>