Remove requirement that mojo::Environment be instantiated.

mojo/public/cpp/environment/environment.h

Index: mojo/public/cpp/environment/environment.h
diff --git a/mojo/public/cpp/environment/environment.h b/mojo/public/cpp/environment/environment.h
index 85d17992c0a865f6f4f5e2b0a5a34ecef5c4e931..d3ab352bf29ed47061fdb6efd61d639e05756281 100644
--- a/mojo/public/cpp/environment/environment.h
+++ b/mojo/public/cpp/environment/environment.h
@@ -12,23 +12,11 @@ struct MojoLogger;
 
 namespace mojo {
 
-// Other parts of the Mojo C++ APIs use the *static* methods of this class.
-//
-// The "standalone" implementation of this class requires that this class (in
-// the lib/ subdirectory) be instantiated (and remain so) while using the Mojo
-// C++ APIs. I.e., the static methods depend on things set up by the constructor
-// and torn down by the destructor.
-//
-// Other implementations may not have this requirement.
+// This class just acts as a "namespace": it only has static methods (whose
+// implementation may be varied). Note that some implementations may require
+// their own explicit initialization/shut down functions to be called.
 class Environment {
  public:
-  Environment();
-  // This constructor allows the standard implementations to be overridden (set
-  // a parameter to null to get the standard implementation).
-  Environment(const MojoAsyncWaiter* default_async_waiter,

[vardhan, 2016/05/20 18:38:06]

if the default async waiter or default logger do need to be initialized for some
reason, I guess we should now expect a GetDefaultLogger() to lazy-initialize?

[viettrungluu, 2016/05/20 21:09:46]

Either that, or some implementations may require explicit initialization (as I
tried to explain in the comment I added).

-              const MojoLogger* default_logger);
-  ~Environment();
-
   static const MojoAsyncWaiter* GetDefaultAsyncWaiter();

[vardhan, 2016/05/20 18:38:06]

should we have a SetDefaultAsyncWaitier() as well?

[viettrungluu, 2016/05/20 21:09:46]

Done.

 
   static const MojoLogger* GetDefaultLogger();
@@ -44,7 +32,8 @@ class Environment {
   static void DestroyDefaultRunLoop();
 
  private:
-  MOJO_DISALLOW_COPY_AND_ASSIGN(Environment);
+  Environment() = delete;
+  ~Environment() = delete;
 };
 
 }  // namespace mojo