Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(5312)

Unified Diff: chrome/common/extensions/docs/server2/subversion_fetcher.py

Issue 10545043: Extensions docs server: Design changes, partial template support. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/common/extensions/docs/server2/subversion_fetcher.py
diff --git a/chrome/common/extensions/docs/server2/subversion_fetcher.py b/chrome/common/extensions/docs/server2/subversion_fetcher.py
index 719864aa3f3fe1cb899e1b30a9bf5669cfaaecec..c37b4a77ffb18d62365d4642c42ff7772ab5f6aa 100644
--- a/chrome/common/extensions/docs/server2/subversion_fetcher.py
+++ b/chrome/common/extensions/docs/server2/subversion_fetcher.py
@@ -9,16 +9,16 @@ BRANCH_URL = SUBVERSION_URL + 'branches/'
class SubversionFetcher(object):
"""Class to fetch resources from src.chromium.org.
"""
- def __init__(self, base_path, urlfetch):
- self.base_path = base_path
- self.urlfetch = urlfetch
+ def __init__(self, branch, base_path, urlfetch):
+ self._base_path = self._GetURLFromBranch(branch) + base_path
+ self._urlfetch = urlfetch
not at google - send to devlin 2012/06/07 04:43:57 url_fetcher ?
cduvall 2012/06/08 00:39:23 Done.
def _GetURLFromBranch(self, branch):
if branch == 'trunk':
return TRUNK_URL + 'src/'
return BRANCH_URL + branch + '/src/'
- def FetchResource(self, branch, path):
- url = self._GetURLFromBranch(branch) + self.base_path + path
- result = self.urlfetch.fetch(url)
+ def FetchResource(self, path):
+ url = self._base_path + path
+ result = self._urlfetch.fetch(url)
return result
not at google - send to devlin 2012/06/07 04:43:57 nit: inline url and result
cduvall 2012/06/08 00:39:23 Done.

Powered by Google App Engine
This is Rietveld 408576698