That won't really protect anyone from anything, as they could just call DOM.createElement("bugger") and hand that in. Also, accepting an existing element is another way to say HTMLPanel.wrap(), which we haven't provided so far--you really want to go there? Anyhow, does any browser not accept <bugger> <this><for><a><lark></lark></a></for></this></bugger>
rjrjr On Fri, Jun 12, 2009 at 2:11 PM, <j...@google.com> wrote: > > http://gwt-code-reviews.appspot.com/39802/diff/1/3 > File user/src/com/google/gwt/user/client/ui/HTMLPanel.java (right): > > http://gwt-code-reviews.appspot.com/39802/diff/1/3#newcode61 > Line 61: public HTMLPanel(String tag, String html) { > Could we not force 'tag' to simply be an element, so that we remove one > source of possible errors (i.e. new HTMLPanel('bugger', '...'))? > > > http://gwt-code-reviews.appspot.com/39802 > --~--~---------~--~----~------------~-------~--~----~ http://groups.google.com/group/Google-Web-Toolkit-Contributors -~----------~----~----~----~------~----~------~--~---