Docs: Use ETags.

chrome/common/extensions/docs/server2/render_servlet.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.
 
+import hashlib
 import logging
 import posixpath
 import traceback
 
 from branch_utility import BranchUtility
 from environment import IsPreviewServer
 from file_system import FileNotFoundError
 from redirector import Redirector
 from servlet import Servlet, Response
 from special_paths import SITE_VERIFICATION_FILE
 from third_party.handlebar import Handlebar
 
 
-def _MakeHeaders(content_type):
-  return {
+def _MakeHeaders(content_type, etag=None):
+  headers = {
+    # See http://www.w3.org/Protocols/rfc2616/rfc2616-sec14.html#sec14.9.1.
+    'Cache-Control': 'public, max-age=0, no-cache',
+    'Content-Type': content_type,
     'X-Frame-Options': 'sameorigin',
-    'Content-Type': content_type,
-    'Cache-Control': 'max-age=300',
   }
+  if etag is not None:
+    headers['ETag'] = etag
+  return headers
 
 
 class RenderServlet(Servlet):
   '''Servlet which renders templates.
   '''
 
   class Delegate(object):
     def CreateServerInstance(self):
       raise NotImplementedError(self.__class__)
 
@@ skipping 81 matching lines @@
           server_instance.template_renderer.Render(content, self._request))
       # HACK: the site verification file (google2ed...) doesn't have a title.
       content, doc_warnings = server_instance.document_renderer.Render(
           template_content,
           path,
           render_title=path != SITE_VERIFICATION_FILE)
       warnings = template_warnings + doc_warnings
       if warnings:
         sep = '\n - '
         logging.warning('Rendering %s:%s%s' % (path, sep, sep.join(warnings)))
+      # Content was dynamic. The new etag is a hash of the content.
+      etag = None
+    elif content_and_type.version is not None:
+      # Content was static. The new etag is the version of the content.
+      etag = '"%s"' % content_and_type.version
+    else:
+      # Sometimes non-dynamic content does not have a version, for example
+      # .zip files. The new etag is a hash of the content.
+      etag = None
 
     content_type = content_and_type.content_type
     if isinstance(content, unicode):
       content = content.encode('utf-8')
       content_type += '; charset=utf-8'
 
-    return Response.Ok(content, headers=_MakeHeaders(content_type))
+    if etag is None:
+      # Note: we're using md5 as a convenient and fast-enough way to identify
+      # content. It's not intended to be cryptographic in any way, and this
+      # is *not* what etags is for. That's what SSL is for, this is unrelated.
+      etag = '"%s"' % hashlib.md5(content).hexdigest()
+
+    headers = _MakeHeaders(content_type, etag=etag)
+    if etag == self._request.headers.get('If-None-Match'):
+      return Response.NotModified('Not Modified', headers=headers)
+    return Response.Ok(content, headers=headers)