Correct $ref links in extensions documentation server relative location.

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

 # Copyright (c) 2012 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.
 
 from HTMLParser import HTMLParser
 import logging
 import os
 import re
 
 from extensions_paths import INTROS_TEMPLATES, ARTICLES_TEMPLATES
@@ skipping 76 matching lines @@
     self._request = request
     self._cache = server_instance.compiled_fs_factory.Create(
         server_instance.host_file_system_provider.GetTrunk(),
         self._MakeIntro,
         IntroDataSource)
     self._ref_resolver = server_instance.ref_resolver_factory.Create()
 
   def _MakeIntro(self, intro_path, intro):
     # Guess the name of the API from the path to the intro.
     api_name = os.path.splitext(intro_path.split('/')[-1])[0]
+    request_path = ''
+    if self._request:
+      request_path = self._request.path

[not at google - send to devlin, 2013/11/22 20:16:10]

nit: ternary:

request_path = self._request.path if self._request or None

[benwells, 2013/11/25 01:12:37]

I was thinking it might be better to make sure that self._request is never None.
This change was just needed for tests which leave it blank.

I'll take a look into that.

[benwells, 2013/11/27 07:28:31]

Removed the code by making sure self._request was not None in tests.

     intro_with_links = self._ref_resolver.ResolveAllLinks(
-            intro, namespace=api_name)
+            intro, requestPath=request_path, namespace=api_name)

[not at google - send to devlin, 2013/11/22 20:16:10]

python style requires request_path not requestPath

[benwells, 2013/11/27 07:28:31]

Done.

 
     # TODO(kalman): In order to pick up every header tag, and therefore make a
     # complete TOC, the render context of the Handlebar needs to be passed
     # through to here. Even if there were a mechanism to do this it would
     # break caching; we'd need to do the TOC parsing *after* rendering the full
     # template, and that would probably be expensive.
     intro_parser = _IntroParser()
     intro_parser.feed(
         self._template_renderer.Render(Handlebar(intro_with_links),
                                        self._request,
@@ skipping 22 matching lines @@
       except FileNotFoundError:
         continue
     # Not found. Do the first operation again so that we get a stack trace - we
     # know that it'll fail.
     get_from_base_path(base_paths[0])
     raise AssertionError()
 
   def Cron(self):
     # TODO(kalman): Walk through the intros and articles directory.
     return Future(value=())