Docserver: SidenavDataSource refactor, transition to DataSourceRegistry

chrome/common/extensions/docs/server2/server_instance.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 api_data_source import APIDataSource
 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
 from intro_data_source import IntroDataSource
 from object_store_creator import ObjectStoreCreator
 from path_canonicalizer import PathCanonicalizer
 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
 from test_object_store import TestObjectStore
 
 class ServerInstance(object):
   def __init__(self,
                object_store_creator,
                host_file_system,
                app_samples_file_system,
@@ skipping 53 matching lines @@
         base_path)
 
     self.api_data_source_factory.SetSamplesDataSourceFactory(
         self.samples_data_source_factory)
 
     self.intro_data_source_factory = IntroDataSource.Factory(
         self.compiled_host_fs_factory,
         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,
         '/'.join((svn_constants.API_PATH, '_api_features.json')),
         '/'.join((svn_constants.API_PATH, '_permission_features.json')),
         '/'.join((svn_constants.JSON_PATH, 'permissions.json')))
 
     self.example_zipper = ExampleZipper(
         self.compiled_host_fs_factory,
         self.host_file_system,
         svn_constants.DOCS_PATH)
 
     self.path_canonicalizer = PathCanonicalizer(self.compiled_host_fs_factory)
 
     self.redirector = Redirector(
         self.compiled_host_fs_factory,
         self.host_file_system,
         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(
         (svn_constants.API_PATH, '_manifest_features.json'))
 
     self.template_data_source_factory = TemplateDataSource.Factory(
         self.api_data_source_factory,
         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)
     self.permissions_data_source.SetTemplateDataSource(
         self.template_data_source_factory)
 
   @staticmethod
   def ForTest(file_system):
     object_store_creator = ObjectStoreCreator.ForTest()
     return ServerInstance(object_store_creator,
@@ skipping 14 matching lines @@
         object_store_creator)
     trunk_file_system = host_file_system_creator.Create()
     return ServerInstance(
         object_store_creator,
         trunk_file_system,
         EmptyDirFileSystem(),
         '',
         CompiledFileSystem.Factory(trunk_file_system, object_store_creator),
         TestBranchUtility.CreateWithCannedData(),
         host_file_system_creator)