Add a test for application lifecycle

mojo/shell/application_manager.cc

 // Copyright 2014 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.
 
 #include "mojo/shell/application_manager.h"
 
 #include <stdint.h>
 
 #include <utility>
 
@@ skipping 451 matching lines @@
   DCHECK(it != shell_client_factories_.end());
   shell_client_factories_.erase(it);
 }
 
 void ApplicationManager::OnGotResolvedName(
     scoped_ptr<ConnectParams> params,
     const String& resolved_name,
     const String& resolved_qualifier,
     mojom::CapabilityFilterPtr base_filter,
     const String& file_url) {
+  std::string qualifier = params->target().qualifier();
+  if (qualifier == GetNamePath(params->target().name()))
+    qualifier = resolved_qualifier;
+  Identity target(params->target().name(), qualifier,
+                  params->target().user_id());
+  params->set_target(target);
+
   // It's possible that when this manifest request was issued, another one was
   // already in-progress and completed by the time this one did, and so the
   // requested application may already be running.
   if (ConnectToExistingInstance(&params))
     return;
 
   Identity source = params->source();
   // |base_filter| can be null when there is no manifest, e.g. for URL types
   // not resolvable by the resolver.
   CapabilityFilter filter = GetPermissiveCapabilityFilter();
   if (!base_filter.is_null())
     filter = base_filter->filter.To<CapabilityFilter>();
-  Identity target = params->target();
   target.set_filter(filter);
 
   mojom::ShellClientRequest request;
   Instance* instance = CreateInstance(target, &request);
   instance->ConnectToClient(std::move(params));
 
   if (LoadWithLoader(target, &request))
     return;
 
   CHECK(!file_url.is_null() && !base_filter.is_null());
 
   if (target.name() != resolved_name) {
     // In cases where a package alias is resolved, we have to use the qualifier
     // from the original request rather than for the package itself, which will
     // always be the same.
     CreateShellClient(
-        source, Identity(resolved_name, target.qualifier(), target.user_id()),
+        source, Identity(resolved_name, resolved_qualifier, target.user_id()),
         target.name(), std::move(request));
   } else {
     bool start_sandboxed = false;
     native_runners_.push_back(
         instance->StartWithFileURL(file_url.To<GURL>(), std::move(request),
                                    start_sandboxed,
                                    native_runner_factory_.get()));
   }
 }
 
@@ skipping 17 matching lines @@
   for (auto it = native_runners_.begin(); it != native_runners_.end(); ++it) {
     if (it->get() == runner) {
       native_runners_.erase(it);
       return;
     }
   }
 }
 
 }  // namespace shell
 }  // namespace mojo