Add ApplicationDelegate::Quit()

mojo/public/cpp/application/application_delegate.h

 // 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.
 
 #ifndef MOJO_PUBLIC_APPLICATION_APPLICATION_DELEGATE_H_
 #define MOJO_PUBLIC_APPLICATION_APPLICATION_DELEGATE_H_
 
 #include <string>
 
 #include "mojo/public/cpp/system/macros.h"
@@ skipping 16 matching lines @@
   // Override this method to configure what services a connection supports when
   // being connected to from an app.
   // Return false to reject the connection entirely.
   virtual bool ConfigureIncomingConnection(ApplicationConnection* connection);
 
   // Override this method to configure what services a connection supports when
   // connecting to another app.
   // Return false to reject the connection entirely.
   virtual bool ConfigureOutgoingConnection(ApplicationConnection* connection);
 
+  // Called before ApplicationImpl::Terminate(). After returning from this call
+  // the delegate can no longer rely on the main run loop still running.

[jamesr, 2015/02/23 20:27:31]

"the main run loop" is a feature of the chromium application runner, not the
public ApplicationDelegate type which doesn't know about it.

Can you think of a way to describe this in terms of concepts that exist in this
layer of the code?

[blundell, 2015/02/23 20:35:42]

I was following the terminology used in the comment for
ApplicationImpl::Terminate():

// Quits the main run loop for this application.

Does that terminology similarly need abstracting?

+  virtual void Quit();
+
  private:
   MOJO_DISALLOW_COPY_AND_ASSIGN(ApplicationDelegate);
 };
 
 }  // namespace mojo
 
 #endif  // MOJO_PUBLIC_APPLICATION_APPLICATION_DELEGATE_H_