Add ApplicationDelegate::Quit()

mojo/public/cpp/application/application_delegate.h

Index: mojo/public/cpp/application/application_delegate.h
diff --git a/mojo/public/cpp/application/application_delegate.h b/mojo/public/cpp/application/application_delegate.h
index f95dca50ba85863ddee0871766deb8f3cfdbb64f..5519f6cd78ff56c921b5519db4b0c0610666401e 100644
--- a/mojo/public/cpp/application/application_delegate.h
+++ b/mojo/public/cpp/application/application_delegate.h
@@ -34,6 +34,10 @@ class ApplicationDelegate {
   // 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);
 };