Converting private fields to protected for ease of extending

Converting private final requestPipeline, responseRewriterRegistry and uriManager to protected so that it is easy to override them when extending HtmlAccelServlet.

[zhoresh, 2010-06-21 20:51:03]

http://codereview.appspot.com/1705043/diff/1/2
File
java/gadgets/src/main/java/org/apache/shindig/gadgets/servlet/AccelHandler.java
(right):

http://codereview.appspot.com/1705043/diff/1/2#newcode59
java/gadgets/src/main/java/org/apache/shindig/gadgets/servlet/AccelHandler.java:59:
protected AccelUriManager uriManager;
Maybe keep them final, and override them using injection in derived class

[gagan.goku, 2010-06-21 20:57:00]

On 2010/06/21 20:51:03, zhoresh wrote:
> http://codereview.appspot.com/1705043/diff/1/2
> File
>
java/gadgets/src/main/java/org/apache/shindig/gadgets/servlet/AccelHandler.java
> (right):
> 
> http://codereview.appspot.com/1705043/diff/1/2#newcode59
>
java/gadgets/src/main/java/org/apache/shindig/gadgets/servlet/AccelHandler.java:59:
> protected AccelUriManager uriManager;
> Maybe keep them final, and override them using injection in derived class

Done.

[gagan.goku, 2010-06-21 20:57:09]

http://codereview.appspot.com/1705043/diff/1/2
File
java/gadgets/src/main/java/org/apache/shindig/gadgets/servlet/AccelHandler.java
(right):

http://codereview.appspot.com/1705043/diff/1/2#newcode59
java/gadgets/src/main/java/org/apache/shindig/gadgets/servlet/AccelHandler.java:59:
protected AccelUriManager uriManager;
On 2010/06/21 20:51:04, zhoresh wrote:
> Maybe keep them final, and override them using injection in derived class

Done.

[zhoresh, 2010-06-21 21:09:27]

lgtm, committed (r956695)

[gagan.goku, 2010-06-21 21:10:45]

On 2010/06/21 21:09:27, zhoresh wrote:
> lgtm, committed (r956695)

Thanks