Make "all user" services work when packaged.

services/catalog/entry.cc

Index: services/catalog/entry.cc
diff --git a/services/catalog/entry.cc b/services/catalog/entry.cc
index 7a090156a5b6795c8b028dbfc1900b4cf42fd8fc..35be1ca30f2410461c708f47d19e306ebc2e0a5c 100644
--- a/services/catalog/entry.cc
+++ b/services/catalog/entry.cc
@@ -259,6 +259,12 @@ TypeConverter<service_manager::mojom::ResolveResultPtr,
   result->resolved_name = package.name();
   result->qualifier = input.qualifier();
   result->interface_provider_specs = input.interface_provider_specs();
+  if (input.package()) {
+    auto it = package.interface_provider_specs().find(
+        service_manager::mojom::kServiceManager_ConnectorSpec);

[Ken Rockot(use gerrit already), 2016/10/27 17:01:29]

note to self: this is weird, we should probably put service manager capability
declarations (e.g. all_users, instance_per_child) somewhere that isn't
interface_provider_specs. let's someday soon introduce a service_options key
into the manifest toplevel, and we can do more reasonable things like

  "service_options": {
    "all_users": true,
    "instance_per_child": true
  }

etc

[Jay Civelli, 2016/10/27 17:43:28]

Acknowledged.

+    if (it != package.interface_provider_specs().end())
+      result->package_spec = it->second;
+  }
   result->package_path = package.path();
   return result;
 }