Remove LazyInstance usage from media/

media/cdm/external_clear_key_test_helper.cc

 // Copyright 2015 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/cdm/external_clear_key_test_helper.h"
 
+#include "base/debug/leak_annotations.h"
 #include "base/files/file_path.h"
 #include "base/files/file_util.h"
 #include "base/native_library.h"
 #include "base/path_service.h"
 #include "media/cdm/api/content_decryption_module.h"
 #include "media/cdm/cdm_paths.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 namespace media {
 
@@ skipping 27 matching lines @@
   // Now load the CDM library.
   base::NativeLibraryLoadError error;
   library_.Reset(base::LoadNativeLibrary(library_path_, &error));
   ASSERT_TRUE(library_.is_valid()) << error.ToString();
 
   // Call INITIALIZE_CDM_MODULE()
   typedef void (*InitializeCdmFunc)();
   InitializeCdmFunc initialize_cdm_func = reinterpret_cast<InitializeCdmFunc>(
       library_.GetFunctionPointer(MAKE_STRING(INITIALIZE_CDM_MODULE)));
   ASSERT_TRUE(initialize_cdm_func) << "No INITIALIZE_CDM_MODULE in library";
+
+  ANNOTATE_SCOPED_MEMORY_LEAK;

[kcc2, 2017/02/10 22:50:37]

This deserves a comment. 
A reader looking at this code has no idea why and what leak we are suppressing. 

[DaleCurtis, 2017/02/10 23:28:23]

Comment added, bug filed to see if this is a real issue.

   initialize_cdm_func();
 }
 
 void ExternalClearKeyTestHelper::UnloadLibrary() {
   // Call DeinitializeCdmModule()
   typedef void (*DeinitializeCdmFunc)();
   DeinitializeCdmFunc deinitialize_cdm_func =
       reinterpret_cast<DeinitializeCdmFunc>(
           library_.GetFunctionPointer("DeinitializeCdmModule"));
   ASSERT_TRUE(deinitialize_cdm_func) << "No DeinitializeCdmModule() in library";
   deinitialize_cdm_func();
 }
 
 }  // namespace media