Remove ReferenceResolver dependency in APIDataSource.

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

 #!/usr/bin/env python
 # 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 unittest
 
 from document_renderer import DocumentRenderer
 from server_instance import ServerInstance
 from test_file_system import TestFileSystem
@@ skipping 98 matching lines @@
 
     text, warnings = self._renderer.Render(document, path)
     self.assertEqual(expected_document, text)
     self.assertEqual([], warnings)
 
     text, warnings = self._renderer.Render(document, path, render_title=True)
     self.assertEqual(expected_document, text)
     self.assertEqual(['Expected a title'], warnings)
 
   def testInvalidRef(self):
-    # There needs to be more than 100 characters between the invalid ref
+    # There needs to be more than 110 characters between the invalid ref

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

WDYT about changing this comment to be a bit more generic? It would be easy to
let it get out of date by specifying the character limit.

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

sgtm

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

Do you have any suggestions? I was having trouble thinking of something better.

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

Sure, how about, "DocumentRenderer attempts to detect unclosed $ref() tags by
limiting how far it looks ahead. Lorem Ipsum should be long enough to trigger
that."


http://en.wikipedia.org/wiki/Lorem_ipsum

     # and the next ref
     document = ('An invalid $(ref:foo.foo_t3 a title with some long '
-                'text containing a valid reference pointing to '
+                'text containing a valid reference pointing to a thing called '
                 '$(ref:baz.baz_e1) here')
     expected_document = ('An invalid $(ref:foo.foo_t3 a title with some long '
-                         'text containing a valid reference pointing to <a'
-                         ' href=#type-baz_e1>baz.baz_e1</a> here')
+                         'text containing a valid reference pointing to a '
+                         'thing called <a href=#type-baz_e1>baz.baz_e1</a> '
+                         'here')
     path = 'some/path/to/document_api.html'
 
     text, warnings = self._renderer.Render(document, path)
     self.assertEqual(expected_document, text)
     self.assertEqual([], warnings)
 
     text, warnings = self._renderer.Render(document, path, render_title=True)
     self.assertEqual(expected_document, text)
     self.assertEqual(['Expected a title'], warnings)
 
 
 if __name__ == '__main__':
   unittest.main()