media: Quit MojoMediaApplication when no service is bound.

media/mojo/services/mojo_media_application.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 "media/mojo/services/mojo_media_application.h"
 
+#include "base/command_line.h"
 #include "base/logging.h"
 #include "media/mojo/services/mojo_cdm_service.h"
 #include "media/mojo/services/mojo_renderer_service.h"
 #include "mojo/application/public/cpp/application_connection.h"
 #include "mojo/application/public/cpp/application_impl.h"
 
 namespace media {
 
 const char kMojoMediaAppUrl[] = "mojo:media";
+const int kIdleTimeoutInSeconds = 30;
+
+// Switch to enable fast application destruction (for testing).
+const char kFastAppDestruction[] = "fast-app-destruction";

[sandersd (OOO until July 31), 2015/06/30 21:32:47]

Perhaps too generic a name for a media-specific flag. Some options:
    --fast-media-app-destruction
    --media-app-idle-timeout=0
    --disable-media-app-idle

They're all bad, because the default app behavior ("no idle") is to always run,
so every choice is confusing.

[xhwang, 2015/07/07 20:29:24]

Picked fast-media-app-destruction to make it more media specific. And I agree
naming is hard ;)

 
 // static
 GURL MojoMediaApplication::AppUrl() {
   return GURL(kMojoMediaAppUrl);
 }
 
 // static
 scoped_ptr<mojo::ApplicationDelegate> MojoMediaApplication::CreateApp() {
   return scoped_ptr<mojo::ApplicationDelegate>(new MojoMediaApplication());
 }
 
-MojoMediaApplication::MojoMediaApplication() {
+MojoMediaApplication::MojoMediaApplication()
+    : app_impl_(nullptr),
+      release_delay_(base::TimeDelta::FromSeconds(kIdleTimeoutInSeconds)) {
 }
 
 MojoMediaApplication::~MojoMediaApplication() {
 }
 
 void MojoMediaApplication::Initialize(mojo::ApplicationImpl* app) {
+  app_impl_ = app;
+
   logging::LoggingSettings settings;
   settings.logging_dest = logging::LOG_TO_SYSTEM_DEBUG_LOG;
   logging::InitLogging(settings);
   // Display process ID, thread ID and timestamp in logs.
   logging::SetLogItems(true, true, true, false);
+
+  if (base::CommandLine::ForCurrentProcess()->HasSwitch(kFastAppDestruction))

[sandersd (OOO until July 31), 2015/06/30 21:32:47]

It's a little strange to have this in Initialize(). Is it not safe to put in the
constructor? (Or option B: move it to a const or even static method.)

[xhwang, 2015/07/07 20:29:24]

No. Added comment to explain that.

+    release_delay_ = base::TimeDelta();
 }
 
 bool MojoMediaApplication::ConfigureIncomingConnection(
     mojo::ApplicationConnection* connection) {
   connection->AddService<mojo::ContentDecryptionModule>(this);
   connection->AddService<mojo::MediaRenderer>(this);
   return true;
 }
 
 void MojoMediaApplication::Create(
     mojo::ApplicationConnection* connection,
     mojo::InterfaceRequest<mojo::ContentDecryptionModule> request) {
   // The created object is owned by the pipe.
-  new MojoCdmService(&cdm_service_context_, request.Pass());
+  new MojoCdmService(
+      request.Pass(), &cdm_service_context_,
+      app_impl_->app_lifetime_helper()->CreateAppRefCount(release_delay_));
 }
 
 void MojoMediaApplication::Create(
     mojo::ApplicationConnection* connection,
     mojo::InterfaceRequest<mojo::MediaRenderer> request) {
   // The created object is owned by the pipe.
-  new MojoRendererService(&cdm_service_context_, request.Pass());
+  new MojoRendererService(
+      request.Pass(), &cdm_service_context_,
+      app_impl_->app_lifetime_helper()->CreateAppRefCount(release_delay_));
 }
 
 }  // namespace media