========================================================================
http://mondrian.corp.google.com/file/8744075///depot/googleclient/gears/opensource/gears/test/testcases/config.js?a=1
File //depot/googleclient/gears/opensource/gears/test/testcases/config.js
(snapshot 1)
------------------------------------
Line 105: desktopSuite.addFile('../testcases/desktop_tests.js',
On Mon Oct 27 20:19:14 2008 PDT, aa wrote:
> trailing whitespace
Done.
------------------------------------
Line 106: {useWorker: false, useIFrame: true});
On Mon Oct 27 20:19:49 2008 PDT, aa wrote:
> Might be useful to separately test that you cannot use the API in workers.
I've added another test to that effect (in desktop_tests.js).
========================================================================
http://mondrian.corp.google.com/file/8744075///depot/googleclient/gears/opensource/gears/test/testcases/desktop_tests.js?a=1
File
//depot/googleclient/gears/opensource/gears/test/testcases/desktop_tests.js
(snapshot 1)
------------------------------------
Line 23: // OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN
IF
On Mon Oct 27 20:20:05 2008 PDT, aa wrote:
> Lots of trailing whitespace.
Done.
------------------------------------
Line 31: assertEqual(null, document.getElementById('dropTargetId'));
On Mon Oct 27 20:20:20 2008 PDT, aa wrote:
> There may be an assertNull()
Done.
------------------------------------
Line 39: assertNotEqual(null, document.getElementById('dropTargetId'));
On Mon Oct 27 21:49:04 2008 PDT, aa wrote:
> Are these testing something to do with the drag and drop API? Just looks like
> testing the DOM to me.
Commentary added.
========================================================================
--
To respond, reply to this email or visit http://mondrian.corp.google.com/8744075
