LGTM

On 2011/07/26 19:40:41, unnurg wrote:
sure

On Tue, Jul 26, 2011 at 12:39 PM, Ray Ryan <mailto:rj...@google.com>
wrote:

> Unnur, would you be comfortable taking this?
>
>
> On Tue, Jul 26, 2011 at 12:12 PM, <mailto:b...@google.com> wrote:
>
>> Reviewers: rjrjr,
>>
>> Message:
>> This is a re-spin of a patch that was written back in October but
that
>> got dropped.  The original patch is at
>>

http://gwt-code-reviews.**appspot.com/943802%3Chttp://gwt-code-reviews.appspot.com/943802>
>> .
>>
>> Description:
>> Add font-face support to CssResource.
>> Patch by: bobv
>> Review by: rjrjr
>>
>>
>> Please review this at

http://gwt-code-reviews.**appspot.com/1502806/%3Chttp://gwt-code-reviews.appspot.com/1502806/>
>>
>> Affected files:
>>  M
user/src/com/google/gwt/**resources/css/**CssGenerationVisitor.java
>>  M user/src/com/google/gwt/**resources/css/GenerateCssAst.**java
>>  A user/src/com/google/gwt/**resources/css/ast/CssFontFace.**java
>>  M user/src/com/google/gwt/**resources/css/ast/**CssNodeCloner.java
>>  M user/src/com/google/gwt/**resources/css/ast/CssVisitor.**java
>>  M
user/test/com/google/gwt/**resources/client/**CSSResourceTest.java
>>  M user/test/com/google/gwt/**resources/client/test.css
>>
>>
>>
>


--
DO NOT FORWARD



http://gwt-code-reviews.appspot.com/1502806/

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

Reply via email to