Extensions Docs Server: TDS fix, no local caching

Extensions Docs Server: TDS fix, no local caching

Some minor fixes that were bugging me:
- TemplateDataSource now uses the correct path and does not depend on catching
  an exception
- LocalFileSystem was caching files forever with the new FileSystem interface,
  now it doesn't
- Added some logs for exception handling.

BUG=131095
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=147985

[not at google - send to devlin, 2012-07-23 13:13:27]

https://chromiumcodereview.appspot.com/10808065/diff/1/chrome/common/extensio...
File chrome/common/extensions/docs/server2/local_file_system.py (right):

https://chromiumcodereview.appspot.com/10808065/diff/1/chrome/common/extensio...
chrome/common/extensions/docs/server2/local_file_system.py:48: return
self.StatInfo(self._version)
I don't understand this inc_version stuff. This is the local file system, why
not use os.stat?

https://chromiumcodereview.appspot.com/10808065/diff/1/chrome/common/extensio...
File chrome/common/extensions/docs/server2/template_data_source.py (right):

https://chromiumcodereview.appspot.com/10808065/diff/1/chrome/common/extensio...
chrome/common/extensions/docs/server2/template_data_source.py:112: def get(self,
key, use_path=None):
Have two methods rather than overloading, like

  template = self.GetTemplate(self._public_template_path, template_name)

def get(self, key):
  return self.GetTemplate(self._private_template_path, key)

GetTemplate(self, base_path, template_name):
  ...

[cduvall, 2012-07-23 19:04:41]

http://codereview.chromium.org/10808065/diff/1/chrome/common/extensions/docs/...
File chrome/common/extensions/docs/server2/local_file_system.py (right):

http://codereview.chromium.org/10808065/diff/1/chrome/common/extensions/docs/...
chrome/common/extensions/docs/server2/local_file_system.py:48: return
self.StatInfo(self._version)
On 2012/07/23 13:13:27, kalman wrote:
> I don't understand this inc_version stuff. This is the local file system, why
> not use os.stat?

Done.

http://codereview.chromium.org/10808065/diff/1/chrome/common/extensions/docs/...
File chrome/common/extensions/docs/server2/template_data_source.py (right):

http://codereview.chromium.org/10808065/diff/1/chrome/common/extensions/docs/...
chrome/common/extensions/docs/server2/template_data_source.py:112: def get(self,
key, use_path=None):
On 2012/07/23 13:13:27, kalman wrote:
> Have two methods rather than overloading, like
> 
>   template = self.GetTemplate(self._public_template_path, template_name)
> 
> def get(self, key):
>   return self.GetTemplate(self._private_template_path, key)
> 
> GetTemplate(self, base_path, template_name):
>   ...

Done.

[not at google - send to devlin, 2012-07-23 23:19:58]

lgtm