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

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

Issue 22824042: Docserver: SidenavDataSource refactor, transition to DataSourceRegistry (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 7 years, 4 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/server_instance.py
diff --git a/chrome/common/extensions/docs/server2/server_instance.py b/chrome/common/extensions/docs/server2/server_instance.py
index 1f9321d85c63e85fbb1cfdfca184a7744a86c9ec..cb53d3c553704e3f73d020b5f1b91fc1e7a85527 100644
--- a/chrome/common/extensions/docs/server2/server_instance.py
+++ b/chrome/common/extensions/docs/server2/server_instance.py
@@ -7,7 +7,6 @@ from api_list_data_source import APIListDataSource
from appengine_wrappers import IsDevServer
from availability_finder import AvailabilityFinder
from compiled_file_system import CompiledFileSystem
-from data_source_registry import CreateDataSources
from empty_dir_file_system import EmptyDirFileSystem
from example_zipper import ExampleZipper
from host_file_system_creator import HostFileSystemCreator
@@ -18,7 +17,6 @@ from permissions_data_source import PermissionsDataSource
from redirector import Redirector
from reference_resolver import ReferenceResolver
from samples_data_source import SamplesDataSource
-from sidenav_data_source import SidenavDataSource
import svn_constants
from template_data_source import TemplateDataSource
from test_branch_utility import TestBranchUtility
@@ -92,10 +90,6 @@ class ServerInstance(object):
self.ref_resolver_factory,
[svn_constants.INTRO_PATH, svn_constants.ARTICLE_PATH])
- self.sidenav_data_source_factory = SidenavDataSource.Factory(
- self.compiled_host_fs_factory,
- svn_constants.JSON_PATH)
-
self.permissions_data_source = PermissionsDataSource(
self.compiled_host_fs_factory,
self.host_file_system,
@@ -116,6 +110,7 @@ class ServerInstance(object):
svn_constants.PUBLIC_TEMPLATE_PATH)
self.strings_json_path = '/'.join((svn_constants.JSON_PATH, 'strings.json'))
+ self.sidenav_json_base_path = svn_constants.JSON_PATH
self.manifest_json_path = '/'.join(
(svn_constants.JSON_PATH, 'manifest.json'))
self.manifest_features_path = '/'.join(
@@ -126,16 +121,12 @@ class ServerInstance(object):
self.api_list_data_source_factory,
self.intro_data_source_factory,
self.samples_data_source_factory,
- self.sidenav_data_source_factory,
self.compiled_host_fs_factory,
self.ref_resolver_factory,
self.permissions_data_source,
svn_constants.PUBLIC_TEMPLATE_PATH,
svn_constants.PRIVATE_TEMPLATE_PATH,
- base_path,
- # TODO(jshumway): Remove this hack after data source registry
- # transition, ServerInstance should not know about DataSourceRegistry.
- CreateDataSources(self))
+ base_path)
self.api_data_source_factory.SetTemplateDataSource(
self.template_data_source_factory)

Powered by Google App Engine
This is Rietveld 408576698