Remove ReferenceResolver dependency in APIDataSource.

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 third_party.json_schema_compiler.model import UnixName
 
 
@@ skipping 12 matching lines @@
 
   def __init__(self, table_of_contents_renderer, ref_resolver):
     self._table_of_contents_renderer = table_of_contents_renderer
     self._ref_resolver = ref_resolver
 
   def _RenderLinks(self, document, path):
     ''' Replaces all $(ref:...) references in |document| with html links
     '''
     START_REF = '$(ref:'
     END_REF = ')'
-    MAX_REF_LENGTH = 100
+    MAX_REF_LENGTH = 110

[ahernandez, 2014/03/28 19:04:52]

It turns out a couple refs are actually longer than 100 characters, so I bumped
the limit a tiny bit.

[not at google - send to devlin, 2014/03/28 21:48:22]

yikes, which ones?

[ahernandez, 2014/03/28 22:03:25]

fileBrowserHandler.FileHandlerExecuteEventDetails is one example. It only breaks
the character limit when you have
$(ref:fileBrowserHandler.FileHandlerExecuteEventDetails
fileBrowserHandler.FileHandlerExecuteEventDetails)

 
     new_document = []
 
     # Keeps track of position within |document|
     cursor_index = 0
     start_ref_index = document.find(START_REF)
 
     while start_ref_index != -1:
       end_ref_index = document.find(END_REF, start_ref_index)
 
@@ skipping 38 matching lines @@
     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)