Extend Http Cache key to contain other Image request parameter

The existing Http Cache key takes care of the Image request parameters viz resize_h, resize_w, resize_q.
There's another critical parameter 'RewriteMimeType' ("rewriteMime") that's used for converting the image format for the rewritten images.

This change :
- Extends the Http Cache key to take care of the rewrite-MimeType.
- Adds to the existing testcase (AbstractHttpCacheTest.java), the case for validating the cache key for existing Image resize parameters (height, width & quality)
- Adds another testcase for testing the cahce key for this new Image parameter (rewrite-MimeType).

[jtarrio, 2010-09-13 20:31:53]

lgtm

[vikaas.arora, 2010-09-14 03:31:30]

Jacobo -

Thanks for the review.
I am not shindig-committer. So someone has to take this code to
shindig submission.

Thanks,
Vikas

On Tue, Sep 14, 2010 at 2:01 AM,  <jtarrio@gmail.com> wrote:
> lgtm
>
> http://codereview.appspot.com/2119049/
>

[gagan.goku, 2010-09-14 03:37:35]

Hi Vikas

Time to exploit - I mean use my powers :) Lets discuss once in office, il
submit your cl today sometime in the evening.

Thanks
Gagan

-- 
The only thing missing in life is background music.
-- Gagandeep Singh


On Tue, Sep 14, 2010 at 9:01 AM, Vikas Arora <vikaas.arora@gmail.com> wrote:

> Jacobo -
>
> Thanks for the review.
> I am not shindig-committer. So someone has to take this code to
> shindig submission.
>
> Thanks,
> Vikas
>
> On Tue, Sep 14, 2010 at 2:01 AM,  <jtarrio@gmail.com> wrote:
> > lgtm
> >
> > http://codereview.appspot.com/2119049/
> >
>

[vikaas.arora, 2010-09-14 03:41:58]

Gagan -

Thanks for response.
You are already in the reviewer/approver list.

Thanks,
Vikas


On Tue, Sep 14, 2010 at 9:07 AM, Gagandeep Singh <gagan.goku@gmail.com> wrote:
> Hi Vikas
> Time to exploit - I mean use my powers :) Lets discuss once in office, il
> submit your cl today sometime in the evening.
> Thanks
> Gagan
>
> --
> The only thing missing in life is background music.
> -- Gagandeep Singh
>
>
> On Tue, Sep 14, 2010 at 9:01 AM, Vikas Arora <vikaas.arora@gmail.com> wrote:
>>
>> Jacobo -
>>
>> Thanks for the review.
>> I am not shindig-committer. So someone has to take this code to
>> shindig submission.
>>
>> Thanks,
>> Vikas
>>
>> On Tue, Sep 14, 2010 at 2:01 AM,  <jtarrio@gmail.com> wrote:
>> > lgtm
>> >
>> > http://codereview.appspot.com/2119049/
>> >
>
>

[vikaas.arora, 2010-09-15 03:13:22]

Ping ...

On Tue, Sep 14, 2010 at 9:11 AM, Vikas Arora <vikaas.arora@gmail.com> wrote:
> Gagan -
>
> Thanks for response.
> You are already in the reviewer/approver list.
>
> Thanks,
> Vikas
>
>
> On Tue, Sep 14, 2010 at 9:07 AM, Gagandeep Singh <gagan.goku@gmail.com> wrote:
>> Hi Vikas
>> Time to exploit - I mean use my powers :) Lets discuss once in office, il
>> submit your cl today sometime in the evening.
>> Thanks
>> Gagan
>>
>> --
>> The only thing missing in life is background music.
>> -- Gagandeep Singh
>>
>>
>> On Tue, Sep 14, 2010 at 9:01 AM, Vikas Arora <vikaas.arora@gmail.com> wrote:
>>>
>>> Jacobo -
>>>
>>> Thanks for the review.
>>> I am not shindig-committer. So someone has to take this code to
>>> shindig submission.
>>>
>>> Thanks,
>>> Vikas
>>>
>>> On Tue, Sep 14, 2010 at 2:01 AM,  <jtarrio@gmail.com> wrote:
>>> > lgtm
>>> >
>>> > http://codereview.appspot.com/2119049/
>>> >
>>
>>
>

[gagan.goku, 2010-09-16 16:50:37]

Ran "mvn -e", build looks great, committed as r997830.

[vikaas.arora, 2010-09-17 03:20:12]

Thanks Gagan for submitting this.

On Thu, Sep 16, 2010 at 10:20 PM,  <gagan.goku@gmail.com> wrote:
> Ran "mvn -e", build looks great, committed as r997830.
>
>
> http://codereview.appspot.com/2119049/
>