[Laszlo-dev] For Review: Change 20070628-Philip-6. Summary: Fix getContextMenu()

[Philip Romanik]

Change 20070628-Philip-6 by Philip at Philip-DC on 2007-06-28 11:24:58 EST
     in /cygdrive/f/laszlo/svn/src/svn/openlaszlo/branches/legals
     for http://svn.openlaszlo.org/openlaszlo/branches/legals

Summary: Fix getContextMenu()

New Features:

Bugs Fixed: LPP-4155

Technical Reviewer: max
QA Reviewer: (pending)
Doc Reviewer: (pending)

Documentation:

Release Notes:

Details:
LaszloCanvas.lzs
   - Added a call to setContextMenu() in setDefaultContextMenu() to match 
the behavior of 3.4.
LaszloView.lzs
   - Removed this.contextMenu (not needed)
   - getContextMenu() calls the sprite to get the value
LzSprite.as
   - Store current context menu in this.__contextmenu.
   - Added getContextMenu()
LzSprite.js
   - Added getContextMenu()

I found a small dhtml/swf issue that I submitted as LPP-4213.

Tests:
http://localhost:8080/legals/test/contextmenu/api.lzx?lzr=dhtml
smokecheck passes

This app runs in dhtml and swf:

<canvas debug="true">
   <button text="getContextMenu">
     <handler name="onclick">
       var cm = canvas.getContextMenu();
       var del = new LzDelegate (this, 'myHandler');
       cm.addItem(cm.makeMenuItem('My Function', del));
       Debug.write(cm);
     </handler>

     <method name="myHandler">
       Debug.write("myHandler");
     </method>
   </button>
</canvas>


Files:
M      WEB-INF/lps/lfc/kernel/swf/LzSprite.as
M      WEB-INF/lps/lfc/kernel/dhtml/LzSprite.js
M      WEB-INF/lps/lfc/views/LaszloView.lzs
M      WEB-INF/lps/lfc/views/LaszloCanvas.lzs

Changeset: http://svn.openlaszlo.org/openlaszlo/patches/20070628-Philip-6.tar