Docserver: Modify DocumentRenderer to generate absolute links

chrome/common/extensions/docs/server2/document_renderer.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 logging
 import os
 from document_parser import ParseDocument
 from platform_util import ExtractPlatformFromURL
 from third_party.json_schema_compiler.model import UnixName
 
@@ skipping 61 matching lines @@
         if platform is None:
           logging.error('Cannot resolve reference without a platform.')
           continue
         ref_dict = self._platform_bundle.GetReferenceResolver(
             platform).SafeGetLink(ref_parts[0],
                                   namespace=api_name,
                                   title=title,
                                   path=path)
 
         new_document.append(document[cursor_index:start_ref_index])
-        new_document.append('<a href=%s>%s</a>' % (ref_dict['href'],
-                                                   ref_dict['text']))
+        new_document.append('<a href=%s/%s>%s</a>' % (
+            self._platform_bundle._base_path + platform,

[not at google - send to devlin, 2014/08/07 23:59:31]

nit - but perhaps use %s%s and split up base_path/platform rather than %s and
adding them together? the mix just looks a little odd.

+            ref_dict['href'],
+            ref_dict['text']))
 
       cursor_index = end_ref_index + 1
       start_ref_index = document.find(START_REF, cursor_index)
 
     new_document.append(document[cursor_index:])
 
     return ''.join(new_document)
 
   def Render(self, document, path, render_title=False):
+    ''' |document|: document to be rendered.
+            |path|: request path to the document.
+    |render_title|: boolean representing whether or not to render a title.
+    '''
     # Render links first so that parsing and later replacements aren't
     # affected by $(ref...) substitutions
     document = self._RenderLinks(document, path)
 
     parsed_document = ParseDocument(document, expect_title=render_title)
     toc_text, toc_warnings = self._table_of_contents_renderer.Render(
         parsed_document.sections)
 
     # Only 1 title and 1 table of contents substitution allowed; in the common
     # case, save necessarily running over the entire file.
     if parsed_document.title:
       document = document.replace('$(title)', parsed_document.title, 1)
     return (document.replace('$(table_of_contents)', toc_text, 1),
             parsed_document.warnings + toc_warnings)