There is a regression in this changeset. If you run the html test app, clicking on the 'Load' button would load the html page. With the changeset, this does not happen until after I click the 'Set HTML' button.

Also, in DHTML, I see an error when I click the 'Load' button:

'this._handlers' is null or not an object embed-compressed.js, line 305 character 27


Otherwise, approved.


On Fri, Sep 17, 2010 at 4:51 PM, Max Carlson wrote:

Change 20100917-maxcarlson-1 by maxcarl...@friendly on 2010-09-17 13:36:14 PDT
    in /Users/maxcarlson/openlaszlo/trunk-clean
    for http://svn.openlaszlo.org/openlaszlo/trunk

Summary: Add rpcmethods attribute to <html/>

Bugs Fixed: LPP-9190 - LPP-9190 - Add generic events to html tag

Technical Reviewer: promanik
QA Reviewer: hminsky

Release Notes: <html/> now has an rpcmethods attribute that will contain an array of RPC method names that can be called, provided there are any in the loaded iframe's HTML.

Details: html - Add __getRPCMethods() proxy method for Flash, rpcmethods attribute with default empty array. Move ready event to be a little later. Ask for RPC methods after the page finishes loading. Improve callRPC() documentation. Rename built-in event names to reduce chance of collisions.

iframemanager - Rename built-in event names to reduce chance of collisions.Prevent duplicate __setSendMouseEvents() calls, simplify method. Add __getRPCMethods() API to ask for RPC method names and filter by iframe.

Tests: examples/extensions/html tests work as before.
Files:
M       lps/components/extensions/html.lzx
M       lps/includes/source/iframemanager.js

Changeset: http://svn.openlaszlo.org/openlaszlo/patches/20100917-maxcarlson-1.tar

Reply via email to