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

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

Issue 13470005: Refactor the devserver to make it easier to control caching (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: cduvall, rebase Created 7 years, 8 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/branch_utility.py
diff --git a/chrome/common/extensions/docs/server2/branch_utility.py b/chrome/common/extensions/docs/server2/branch_utility.py
index adfb38eae66d456dc4f062e83355aec49de9b72d..282c6a4e7bf33d86705dd8e5dccd73fdc6b19bf7 100644
--- a/chrome/common/extensions/docs/server2/branch_utility.py
+++ b/chrome/common/extensions/docs/server2/branch_utility.py
@@ -4,14 +4,16 @@
import json
import logging
-
-import object_store
import operator
+from object_store_creator import ObjectStoreCreator
+
class BranchUtility(object):
- def __init__(self, base_path, fetcher, object_store):
- self._base_path = base_path
+ def __init__(self, fetch_url, fetcher, object_store=None):
+ self._fetch_url = fetch_url
self._fetcher = fetcher
+ if object_store is None:
+ object_store = ObjectStoreCreator(BranchUtility).Create()
self._object_store = object_store
@staticmethod
@@ -22,7 +24,8 @@ class BranchUtility(object):
return [(branch, self.GetBranchNumberForChannelName(branch))
for branch in BranchUtility.GetAllBranchNames()]
- def SplitChannelNameFromPath(self, path):
+ @staticmethod
+ def SplitChannelNameFromPath(path):
"""Splits the channel name out of |path|, returning the tuple
(channel_name, real_path). If the channel cannot be determined then returns
(None, path).
@@ -41,13 +44,12 @@ class BranchUtility(object):
if channel_name == 'trunk':
return 'trunk'
- branch_number = self._object_store.Get(channel_name + '.' + self._base_path,
- object_store.BRANCH_UTILITY).Get()
+ branch_number = self._object_store.Get(channel_name).Get()
if branch_number is not None:
return branch_number
try:
- version_json = json.loads(self._fetcher.Fetch(self._base_path).content)
+ version_json = json.loads(self._fetcher.Fetch(self._fetch_url).content)
except Exception as e:
# This can happen if omahaproxy is misbehaving, which we've seen before.
# Quick hack fix: just serve from trunk until it's fixed.
@@ -72,8 +74,6 @@ class BranchUtility(object):
None,
operator.itemgetter(1),
True)
- self._object_store.Set(channel_name + '.' + self._base_path,
- sorted_branches[0][0],
- object_store.BRANCH_UTILITY)
+ self._object_store.Set(channel_name, sorted_branches[0][0])
return sorted_branches[0][0]

Powered by Google App Engine
This is Rietveld 408576698