Lighthouse has a new layout. Prefer the old one? Return to the old layout, and switch back any time from the link at the top of each page.

Execution of scripts added after page load

#82

A script tag is added to the head of a document after the page has loaded but the script does not appear to be executing.

More details here:

http://groups.google.com/group/envjs/browse_thread/thread/23cc869b3...

I'll have to double check tomorrow that I'm using 1.1.rcl and not 1.0.x but I'm running env-js.jar

Reported by Adrees · December 15th, 2009 @ 11:37 AM

State: resolved
Milestone: 1.2.x
Assigned to: Thatcher Thatcher

Activity

  1. Thatcher
    Thatcher
    • State changed from new to open
    • Milestone set to 1.2.x

    This is highest priority fix. 1.2 will not be release until we get this fixed. Look for the test in src/html/document.js when the ticket is closed. Basically we plan to resolve it along with a stateful switch which is triggered by mutation events. There are many cases we hope to codify in the specs. Ideally we can use minimal events. Our choices are:

    case "DOMSubtreeModified":
    case "DOMNodeInserted":
    case "DOMNodeRemoved":
    case "DOMNodeRemovedFromDocument":
    case "DOMNodeInsertedIntoDocument":

    I personally believe we can handle all cases with

    case "DOMNodeRemovedFromDocument":
    case "DOMNodeInsertedIntoDocument":

    and that will be minimal work. The reason is that reasources arent re-loaded(re-executed) unless the node is removed from the document entirely. Plus script nodes after document load arent executed (I think) unless the are appended as children of the head element.

    There are additional cases that complicate the issue because the script elements (again I think) shouldnt be executed if they are added via set innerHTML since that is essentially a parser operation/ I believe we can solve this by inspecting a state of the ducument during initial parse vrs later parse.

    Thanks for you patience, this shouldn't have gone unnoticed. Hopefully a series of isolated, clear, simple tests will prevent this from breaking in the future.

    Thatcher

    February 3rd, 2010 @ 04:43 PM

  2. Thatcher
    Thatcher

    this is fixed an unit tested in 1.2.0.1

    March 4th, 2010 @ 12:38 AM

  3. Thatcher
    Thatcher
    • State changed from open to resolved

    March 4th, 2010 @ 10:58 AM

Please Sign in or create a free account to add a new ticket.

With your very own profile, you can contribute to projects, track your activity, watch tickets, receive and update tickets through your email and much more.