Remove requirement that mojo::Environment be instantiated.

mojo/public/cpp/environment/environment.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_CPP_ENVIRONMENT_ENVIRONMENT_H_
 #define MOJO_PUBLIC_CPP_ENVIRONMENT_ENVIRONMENT_H_
 
 #include "mojo/public/cpp/system/macros.h"
 
 struct MojoAsyncWaiter;
 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();
   // Setting the logger to null will use the standard implementation.
   static void SetDefaultLogger(const MojoLogger* logger);
 
   // These instantiate and destroy an environment-specific run loop for the
   // current thread, allowing |GetDefaultAsyncWaiter()| to be used. (The run
   // loop itself should be accessible via thread-local storage, using methods
   // specific to the run loop implementation.) Creating and destroying nested
   // run loops is not supported.
   static void InstantiateDefaultRunLoop();
   static void DestroyDefaultRunLoop();
 
  private:
-  MOJO_DISALLOW_COPY_AND_ASSIGN(Environment);
+  Environment() = delete;
+  ~Environment() = delete;
 };
 
 }  // namespace mojo
 
 #endif  // MOJO_PUBLIC_CPP_ENVIRONMENT_ENVIRONMENT_H_