Very Low Activity

Commits : Individual Commit

  Analyzed 4 days ago based on code collected 4 days ago.

Commit ID

Anon80 Contributor: ffeng Files Modified: 1
Date: 31-March-2012 at 06:00 Lines Added: 4
Repository: http://svn.openlaszlo.org/openlaszlo/trunk/ /openlaszlo/trunk Lines Removed: 0
Commit Comment: Change laszlo-20120331-88H by las...@laszlos-macbook-pro-15 on 2012-03-31 14:09:43 CST in /Users/laszlo/Work/fred-office/src/svn/openlaszlo/trunk for http://svn.openlaszlo.org/openlaszlo/trunk
Summary:
DHTML - Chrome - lz.Cursor.setCursorGlobal method will throw error in Debug console

New Features:

Bugs Fixed: LPP-10164

Technical Reviewer: fguo
QA Reviewer: dju
Doc Reviewer: (pending)

Documentation:

Release Notes:

Overview:
lzclick does not contain 'cssRules' style in chrome17, so i create a branch to handle it.
if (rules && rules['length']) {
var rl = rules.length - 1;
for (var j = rl; j >= 0; j--) {
if (rules[j].selectorText == classname) {
rules[j].style[name] = value;
}
}
} else { //Chrome17 may have issue in 'cssRules'.
Debug.debug("__setCSSClassProperty----unexpected css-", classname, rulename, rules);
}

Details:

Tests:

Test case:
----------------------------------------------------------
<canvas>
<view width="200" height="200" bgcolor="red">
<handler name="onmouseover">
lz.Cursor.setCursorGlobal("colResize");
Debug.write("onmouseover----");
</handler>
<handler name="onmousedown">
lz.Cursor.setCursorGlobal("rowResize");
Debug.write("onmousedown----");
</handler>
<handler name="onmouseout">
lz.Cursor.restoreCursor();
Debug.write("onmouseout----");
</handler>
</view>
</canvas>
----------------------------------------------------------

When execute it and mouse over on the view on dhtml of Chrome browser, it will throw the following errors:
ERROR @9.lzx?3: TypeError: Cannot read property 'length' of null
ERROR @9.lzx?7: TypeError: Cannot read property 'length' of null
ERROR @9.lzx?11: TypeError: Cannot read property 'length' of null

COMMENTS:
Build number: 19564
It works on swf10 of chrome and work as expectation on other browsers on dhtml
 

Changes by Language

Language Code Added Code Removed Comments Added Comment Removed Blanks Added Blanks Removed
  JavaScript 4 0 0 0 0 0

Changes by File

Showing page 1 of 1
File Language Code Added Code Removed Comments Added Comment Removed Blanks Added Blanks Removed
WEB-INF/lps/lfc/kernel/dhtml/LzSprite.js JavaScript 4 0 0 0 0 0