LGTM On Wed, Mar 18, 2009 at 10:21 AM, John LaBanca <[email protected]> wrote:
> Attached is the updated patch with the test case included. > Thanks, > John LaBanca > [email protected] > > > > On Wed, Mar 18, 2009 at 1:13 PM, <[email protected]> wrote: > >> Looks good as soon as there is a parallel change in FileUploadTest >> (which may require creating FileUploadTest). >> >> Even if said test does nothing but instantiate the thing and run its >> enable/disable method, that's more coverage than we have now. >> >> >> http://gwt-code-reviews.appspot.com/12808 >> > > --~--~---------~--~----~------------~-------~--~----~ http://groups.google.com/group/Google-Web-Toolkit-Contributors -~----------~----~----~----~------~----~------~--~---
