Index: mojo/shell/standalone/context.cc |
diff --git a/mojo/shell/standalone/context.cc b/mojo/shell/standalone/context.cc |
index 921ab99a5e120c6d0fd84c3777dac1d38e593ddb..adc285ddfa32dacee5d37a08c0d8f667ce9aa1e0 100644 |
--- a/mojo/shell/standalone/context.cc |
+++ b/mojo/shell/standalone/context.cc |
@@ -33,11 +33,8 @@ |
#include "mojo/services/tracing/public/interfaces/tracing.mojom.h" |
#include "mojo/shell/application_loader.h" |
#include "mojo/shell/connect_to_application_params.h" |
-#include "mojo/shell/package_manager/package_manager_impl.h" |
-#include "mojo/shell/query_util.h" |
#include "mojo/shell/runner/host/in_process_native_runner.h" |
#include "mojo/shell/runner/host/out_of_process_native_runner.h" |
-#include "mojo/shell/standalone/register_local_aliases.h" |
#include "mojo/shell/standalone/switches.h" |
#include "mojo/shell/standalone/tracer.h" |
#include "mojo/shell/switches.h" |
@@ -59,59 +56,6 @@ class Setup { |
DISALLOW_COPY_AND_ASSIGN(Setup); |
}; |
-void InitContentHandlers(PackageManagerImpl* manager, |
- const base::CommandLine& command_line) { |
- // Default content handlers. |
- manager->RegisterContentHandler("application/javascript", |
- GURL("mojo:html_viewer")); |
- manager->RegisterContentHandler("application/pdf", GURL("mojo:pdf_viewer")); |
- manager->RegisterContentHandler("image/gif", GURL("mojo:html_viewer")); |
- manager->RegisterContentHandler("image/jpeg", GURL("mojo:html_viewer")); |
- manager->RegisterContentHandler("image/png", GURL("mojo:html_viewer")); |
- manager->RegisterContentHandler("text/css", GURL("mojo:html_viewer")); |
- manager->RegisterContentHandler("text/html", GURL("mojo:html_viewer")); |
- manager->RegisterContentHandler("text/plain", GURL("mojo:html_viewer")); |
- |
- // Command-line-specified content handlers. |
- std::string handlers_spec = |
- command_line.GetSwitchValueASCII(switches::kContentHandlers); |
- if (handlers_spec.empty()) |
- return; |
- |
-#if defined(OS_ANDROID) |
- // TODO(eseidel): On Android we pass command line arguments is via the |
- // 'parameters' key on the intent, which we specify during 'am shell start' |
- // via --esa, however that expects comma-separated values and says: |
- // am shell --help: |
- // [--esa <EXTRA_KEY> <EXTRA_STRING_VALUE>[,<EXTRA_STRING_VALUE...]] |
- // (to embed a comma into a string escape it using "\,") |
- // Whatever takes 'parameters' and constructs a CommandLine is failing to |
- // un-escape the commas, we need to move this fix to that file. |
- base::ReplaceSubstringsAfterOffset(&handlers_spec, 0, "\\,", ","); |
-#endif |
- |
- std::vector<std::string> parts = base::SplitString( |
- handlers_spec, ",", base::TRIM_WHITESPACE, base::SPLIT_WANT_ALL); |
- if (parts.size() % 2 != 0) { |
- LOG(ERROR) << "Invalid value for switch " << switches::kContentHandlers |
- << ": must be a comma-separated list of mimetype/url pairs." |
- << handlers_spec; |
- return; |
- } |
- |
- for (size_t i = 0; i < parts.size(); i += 2) { |
- GURL url(parts[i + 1]); |
- if (!url.is_valid()) { |
- LOG(ERROR) << "Invalid value for switch " << switches::kContentHandlers |
- << ": '" << parts[i + 1] << "' is not a valid URL."; |
- return; |
- } |
- // TODO(eseidel): We should also validate that the mimetype is valid |
- // net/base/mime_util.h could do this, but we don't want to depend on net. |
- manager->RegisterContentHandler(parts[i], url); |
- } |
-} |
- |
class TracingInterfaceProvider : public shell::mojom::InterfaceProvider { |
public: |
TracingInterfaceProvider(Tracer* tracer, |
@@ -137,8 +81,7 @@ class TracingInterfaceProvider : public shell::mojom::InterfaceProvider { |
} // namespace |
-Context::Context() |
- : package_manager_(nullptr), main_entry_time_(base::Time::Now()) {} |
+Context::Context() : main_entry_time_(base::Time::Now()) {} |
Context::~Context() { |
DCHECK(!base::MessageLoop::current()); |
@@ -170,12 +113,6 @@ void Context::Init(const base::FilePath& shell_file_root) { |
// TODO(vtl): This should be MASTER, not NONE. |
edk::InitIPCSupport(this, task_runners_->io_runner()); |
- package_manager_ = new PackageManagerImpl( |
- shell_file_root, task_runners_->blocking_pool(), nullptr); |
- InitContentHandlers(package_manager_, command_line); |
- |
- RegisterLocalAliases(package_manager_); |
- |
scoped_ptr<NativeRunnerFactory> runner_factory; |
if (command_line.HasSwitch(switches::kSingleProcess)) { |
#if defined(COMPONENT_BUILD) |
@@ -190,8 +127,7 @@ void Context::Init(const base::FilePath& shell_file_root) { |
new OutOfProcessNativeRunnerFactory(task_runners_->blocking_pool())); |
} |
application_manager_.reset(new ApplicationManager( |
- make_scoped_ptr(package_manager_), std::move(runner_factory), |
- task_runners_->blocking_pool(), true)); |
+ std::move(runner_factory), task_runners_->blocking_pool(), true)); |
shell::mojom::InterfaceProviderPtr tracing_remote_interfaces; |
shell::mojom::InterfaceProviderPtr tracing_local_interfaces; |