http://codereview.appspot.com/4148044/diff/14001/java/gadgets/src/main/java/org/apache/shindig/gadgets/rewrite/image/BaseOptimizer.java File java/gadgets/src/main/java/org/apache/shindig/gadgets/rewrite/image/BaseOptimizer.java (right):
http://codereview.appspot.com/4148044/diff/14001/java/gadgets/src/main/java/org/apache/shindig/gadgets/rewrite/image/BaseOptimizer.java#newcode146 java/gadgets/src/main/java/org/apache/shindig/gadgets/rewrite/image/BaseOptimizer.java:146: .setResponse(minBytes); let's remove the ETAG header within HttpResponse(Builder) itself on write ie. in the setResponse() method, since that is the actual operation that changes it. http://codereview.appspot.com/4148044/
