Looks good, with suggestions.  Attached is what I meant when I was talking
about subclassing HostedModeBase.BrowserWidgetHostImpl.  This should remove
some duplicate code.  This also brings HostedMode in OOPHM up to the
present.  Apply my patch on top of yours.

On Tue, Mar 17, 2009 at 6:14 PM, John Tamplin <[email protected]> wrote:

> Here is a revised patch after the other pieces were split out.  I also
> added an oophm target to generated build.xml files and Showcase-OOPHM.launch
> (after you ok it I will replicate it for all the other samples).
>
>
> --
> John A. Tamplin
> Software Engineer (GWT), Google
>

--~--~---------~--~----~------------~-------~--~----~
http://groups.google.com/group/Google-Web-Toolkit-Contributors
-~----------~----~----~----~------~----~------~--~---

Attachment: oophm-on-oophm2.patch
Description: Binary data

Reply via email to