Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(2160)

Unified Diff: chrome/common/extensions/docs/server2/server_instance.py

Issue 14273035: Docserver: refactor the Handler/ServerInstance relationship into a servlet (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: done Created 7 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/common/extensions/docs/server2/server_instance.py
diff --git a/chrome/common/extensions/docs/server2/server_instance.py b/chrome/common/extensions/docs/server2/server_instance.py
index 7f3e6cd01ba1a5a3b4ae27e0020db2fa1d4449f4..078703b51ba45aafd541aa8f987385933144ea54 100644
--- a/chrome/common/extensions/docs/server2/server_instance.py
+++ b/chrome/common/extensions/docs/server2/server_instance.py
@@ -35,10 +35,6 @@ from third_party.json_schema_compiler.memoize import memoize
from third_party.json_schema_compiler.model import UnixName
import url_constants
-def _IsBinaryMimetype(mimetype):
- return any(mimetype.startswith(prefix)
- for prefix in ['audio', 'image', 'video'])
-
def _IsSamplesDisabled():
return IsDevServer()
@@ -213,45 +209,3 @@ class ServerInstance(object):
self.compiled_fs_factory)
self.content_cache = self.compiled_fs_factory.GetOrCreateIdentity()
-
- def _FetchStaticResource(self, path, response):
- """Fetch a resource in the 'static' directory.
- """
- mimetype = mimetypes.guess_type(path)[0] or 'text/plain'
- result = self.content_cache.GetFromFile(
- svn_constants.DOCS_PATH + '/' + path,
- binary=_IsBinaryMimetype(mimetype))
- response.headers['content-type'] = mimetype
- return result
-
- def Get(self, path, request, response):
- templates = self.template_data_source_factory.Create(request, path)
-
- content = None
- try:
- if fnmatch(path, 'extensions/examples/*.zip'):
- content = self.example_zipper.Create(
- path[len('extensions/'):-len('.zip')])
- response.headers['content-type'] = 'application/zip'
- elif path.startswith('extensions/examples/'):
- mimetype = mimetypes.guess_type(path)[0] or 'text/plain'
- content = self.content_cache.GetFromFile(
- '%s/%s' % (svn_constants.DOCS_PATH, path[len('extensions/'):]),
- binary=_IsBinaryMimetype(mimetype))
- response.headers['content-type'] = 'text/plain'
- elif path.startswith('static/'):
- content = self._FetchStaticResource(path, response)
- elif path.endswith('.html'):
- content = templates.Render(path)
- except FileNotFoundError as e:
- logging.warning(traceback.format_exc())
-
- response.headers['x-frame-options'] = 'sameorigin'
- if content is None:
- response.set_status(404);
- response.out.write(templates.Render('404'))
- else:
- if not content:
- logging.error('%s had empty content' % path)
- response.headers['cache-control'] = 'max-age=300'
- response.out.write(content)
« no previous file with comments | « chrome/common/extensions/docs/server2/samples_data_source_test.py ('k') | chrome/common/extensions/docs/server2/servlet.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698