Support markdown template for html editor

chrome/common/extensions/docs/server2/content_provider.py

 # Copyright 2013 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
-

[not at google - send to devlin, 2014/01/03 03:18:01]

leave this here

[hukun, 2014/01/06 08:25:19]

Done

 import mimetypes
 import os
 
 from compiled_file_system import SingleFile
 from directory_zipper import DirectoryZipper
 from docs_server_utils import ToUnicode
+from file_system import FileNotFoundError
 from future import Gettable, Future
 from third_party.handlebar import Handlebar
+from third_party.markdown import markdown
 
 
 class ContentAndType(object):
   '''Return value from ContentProvider.GetContentAndType.
   '''
 
   def __init__(self, content, content_type):
     self.content = content
     self.content_type = content_type
 
@@ skipping 25 matching lines @@
     self._supports_templates = supports_templates
     if supports_zip:
       self._directory_zipper = DirectoryZipper(compiled_fs_factory, file_system)
     else:
       self._directory_zipper = None
 
   @SingleFile
   def _CompileContent(self, path, text):
     assert text is not None, path
     mimetype = mimetypes.guess_type(path)[0]
-    if mimetype is None:
+    if os.path.splitext(path)[1] == '.md':
+      # markdown extensions' introduction in the following url
+      # http://pythonhosted.org/Markdown/extensions/index.html

[not at google - send to devlin, 2014/01/03 03:18:01]

I can't really parse the comment. How about:

# See http://pythonhosted.org/Markdown/extensions for details on "extensions=".

[hukun, 2014/01/06 08:25:19]

Done

+      content = markdown(ToUnicode(text), extensions=('extra', 'headerid',
+          'sane_lists'))

[not at google - send to devlin, 2014/01/03 03:18:01]

indentation:

content = markdown(ToUnicode(text),
                   extensions=('extra', 'headerid', 'sane_lists'))

[hukun, 2014/01/06 08:25:19]

Done

+      if self._supports_templates:
+        content = Handlebar(content, name=path)
+      mimetype = 'text/html'
+    elif mimetype is None:
       content = text
       mimetype = 'text/plain'
     elif mimetype == 'text/html':
       content = ToUnicode(text)
       if self._supports_templates:
         content = Handlebar(content, name=path)
     elif (mimetype.startswith('text/') or
           mimetype in ('application/javascript', 'application/json')):
       content = ToUnicode(text)
     else:
       content = text
     return ContentAndType(content, mimetype)
 
+  def _MaybeMarkdown(self, path):
+    if os.path.splitext(path)[1] != '.html':

[not at google - send to devlin, 2014/01/03 03:21:41]

use posixpath.splitext

[hukun, 2014/01/06 08:25:19]

Done

+      return path
+

[not at google - send to devlin, 2014/01/03 03:18:01]

Use posixpath not os.path since the latter will break on windows. I discovered
this pattern recently:

dirname, basename = posixpath.split(path)

which is neat, and will let you write like

dirname, file_name = posixpath.split(path)
file_list = self.file_system.ReadSingle(dirname + '/').Get()
...

[hukun, 2014/01/06 08:25:19]

Done

+    file_list = self.file_system.ReadSingle(os.path.dirname(path) + '/').Get()
+    file_name = os.path.basename(path)
+    if file_name not in file_list:

[not at google - send to devlin, 2014/01/03 03:18:01]

prefer early return like "if file_name in file_list: return path"

[hukun, 2014/01/06 08:25:19]

Done

+      md_file_name = file_name.replace('html', 'md')

[not at google - send to devlin, 2014/01/03 03:18:01]

replace .html -> .md not html -> md in case the filename happens to have "html"
in it. I would also inline it below rather than assigning to an md_file_name
variable.

[not at google - send to devlin, 2014/01/03 03:21:41]

Actually even better may be to write more like

basepath, ext = posixpath.splitext(path)
...
if posixpath.splitext(file_name)[0] + '.md' in file_list:
  return basepath + '.md'

in case .html happens to appear somewhere in the filename too.

[hukun, 2014/01/06 08:25:19]

Done

[hukun, 2014/01/06 08:25:19]

Done

+      if md_file_name in file_list:
+        return path.replace('html', 'md')

[not at google - send to devlin, 2014/01/03 03:18:01]

ditto.

[hukun, 2014/01/06 08:25:19]

Done

+    return path
+
   def GetContentAndType(self, path):
     path = path.lstrip('/')
     base, ext = os.path.splitext(path)
 
     # Check for a zip file first, if zip is enabled.
     if self._directory_zipper and ext == '.zip':
       zip_future = self._directory_zipper.Zip(base)
       return Future(delegate=Gettable(
           lambda: ContentAndType(zip_future.Get(), 'application/zip')))
 
-    return self._content_cache.GetFromFile(path)
+    return self._content_cache.GetFromFile(self._MaybeMarkdown(path))
 
   def Cron(self):
     # Running Refresh() on the file system is enough to pull GitHub content,
     # which is all we need for now while the full render-every-page cron step
     # is in effect.
     # TODO(kalman): Walk over the whole filesystem and compile the content.
     return self.file_system.Refresh()
 
   def __repr__(self):
     return 'ContentProvider of <%s>' % repr(self.file_system)