diff options
| author | pkasting@chromium.org <pkasting@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-04-07 18:49:09 +0000 |
|---|---|---|
| committer | pkasting@chromium.org <pkasting@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-04-07 18:49:09 +0000 |
| commit | 517617476b7231f63b2f900d9dc825db2ee4161d (patch) | |
| tree | a09d1e81c9796b0859a01dfc467710724046e8ce /media | |
| parent | d91ddfc676314764d240a9ad0da06063bb265d76 (diff) | |
| download | chromium_src-517617476b7231f63b2f900d9dc825db2ee4161d.zip chromium_src-517617476b7231f63b2f900d9dc825db2ee4161d.tar.gz chromium_src-517617476b7231f63b2f900d9dc825db2ee4161d.tar.bz2 | |
Make the windows_version.h functions threadsafe by using a singleton. Add accessors to the singleton for more values that various code wants, then convert almost everyone using OSVERSIONINFO or SYSTEM_INFO structs to calling these accessors. Declare an AtExitManager in the out-of-process test runner since it didn't have one and that breaks singleton-using code in the test executable (as opposed to in chrome.dll).
A few other minor cleanups along the way (binding of "*", shorter code, etc.). Because I ran into problems with it while modifying gcapi.cc, I cleaned up our usage of strsafe.h a bit, so that files that don't need it don't include it and files that do use STRSAFE_NO_DEPRECATE instead of a modified #include order.
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/6713107
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@80819 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'media')
| -rw-r--r-- | media/tools/mfplayer/mf_playback_main.cc | 4 | ||||
| -rw-r--r-- | media/tools/mfplayer/mfplayer.cc | 2 |
2 files changed, 4 insertions, 2 deletions
diff --git a/media/tools/mfplayer/mf_playback_main.cc b/media/tools/mfplayer/mf_playback_main.cc index 3ae8164..3ce897c 100644 --- a/media/tools/mfplayer/mf_playback_main.cc +++ b/media/tools/mfplayer/mf_playback_main.cc @@ -1,4 +1,4 @@ -// Copyright (c) 2010 The Chromium Authors. All rights reserved. +// Copyright (c) 2011 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. @@ -6,7 +6,9 @@ #include "media/tools/mfplayer/mfplayer.h" #include <mfapi.h> +#define STRSAFE_NO_DEPRECATE #include <strsafe.h> + #include <cassert> namespace mfplayer { diff --git a/media/tools/mfplayer/mfplayer.cc b/media/tools/mfplayer/mfplayer.cc index 79894ee..7ef51e1 100644 --- a/media/tools/mfplayer/mfplayer.cc +++ b/media/tools/mfplayer/mfplayer.cc @@ -7,9 +7,9 @@ #include <mfapi.h> #include <mferror.h> #include <shlwapi.h> -#include <strsafe.h> #include <cassert> +#include <cstdio> template <class T> static void SafeRelease(T** pptr) { |