Refactoring StyleTagProxyEmbeddedUrlsRewriter to use DefaultConfig instead

Currently StyleTagProxyEmbeddedUrlsRewriter uses the featureConfigFactory to fetch the gadget spec.

However, since accel html web pages don't really have a gadget (In AccelHandler.getProxyUri(), we create a dummy gadget object based on the original HttpRequest object and set its url to the html page being rendered), feature config factory ends up fetching the content of html web page again, rewrites it and falls back to default config since its not able to parse it as gadget xml.

Hence this change allows us to directly use default config for the StyleTagProxyEmbeddedUrlsRewriter (which is an accel specific rewriter).

[anupama.dutta, 2010-10-29 11:21:31]

Code change looks good. Could you update the change description to make the
purpose of this change more clear?

[anupama.dutta, 2010-11-09 03:36:55]

On 2010/10/29 11:21:31, anupama.dutta wrote:
> Code change looks good. Could you update the change description to make the
> purpose of this change more clear?

Suggesting a small rephrasal in the change description so as to give more
context:
- However, since accel html web pages don't really have a gadget (In
AccelHandler.getProxyUri, we create a dummy gadget object based on the original
HttpRequest object and set its url to the html page being rendered), feature
config ... <continue as before>
- The last line can be rewritten as "Hence this change allows us to directly use
default config for the StyleTagProxyEmbeddedUrlsRewriter (which is an accel
specific rewriter)."

Please address these comments, so that this can be sent out to dev@.

[gagan.goku, 2010-11-09 10:07:15]

Sorry for the delay. Changed desc. please take anoher look.

[anupama.dutta, 2010-11-09 11:01:48]

LGTM. Please send out to dev@ for further review.

[Kuntal Loya, 2010-11-11 07:07:29]

lgtm

[gagan.goku, 2010-11-11 11:56:55]

On 2010/11/11 07:07:29, Kuntal Loya wrote:
> lgtm

Build looks good.
Committed as r1033903.
Thanks for the reviews.