Rewrite only CSS links; Do no proxy empty resource.

Only the LINK tags which have rel="stylesheet" and type="text/css" should be rewritten. Others should be bypassed.

There can be resources, which have the resource url as empty string; those should be bypassed as well.

[anupama.dutta, 2010-08-09 11:02:16]

Good tests!
LGTM.

http://codereview.appspot.com/1891051/diff/1/2
File
java/gadgets/src/test/java/org/apache/shindig/gadgets/rewrite/ProxyingVisitorTest.java
(left):

http://codereview.appspot.com/1891051/diff/1/2#oldcode33
java/gadgets/src/test/java/org/apache/shindig/gadgets/rewrite/ProxyingVisitorTest.java:33:
import static org.easymock.EasyMock.*;
Retain these instead of the expanded forms?

[gagan.goku, 2010-08-09 12:42:28]

On 2010/08/09 11:02:16, anupama.dutta wrote:
> Good tests!
> LGTM.
> 
> http://codereview.appspot.com/1891051/diff/1/2
> File
>
java/gadgets/src/test/java/org/apache/shindig/gadgets/rewrite/ProxyingVisitorTest.java
> (left):
> 
> http://codereview.appspot.com/1891051/diff/1/2#oldcode33
>
java/gadgets/src/test/java/org/apache/shindig/gadgets/rewrite/ProxyingVisitorTest.java:33:
> import static org.easymock.EasyMock.*;
> Retain these instead of the expanded forms?

Nice change. Lgtm

[Kuntal Loya, 2010-08-10 06:44:20]

Description : 

Only the LINK tags which have rel="stylesheet" and type="text/css" should be
rewritten. Others should be bypassed.

There can be resources, which have the resource url as empty string; those
should be bypassed as well.

[johnfargo, 2010-08-11 18:47:57]

On 2010/08/10 06:44:20, Kuntal Loya wrote:
> Description : 
> 
> Only the LINK tags which have rel="stylesheet" and type="text/css" should be
> rewritten. Others should be bypassed.
> 
> There can be resources, which have the resource url as empty string; those
> should be bypassed as well.

LGTM, committed as r984526.