remove chrome dependencies from win sandboxing headers. Wrap sandbox code to

make the main routine a little cleaner. Unify the parameters of each of the "main" entry points so we can more easily abstract platform differences in the future. BUG=5323
Review URL: http://codereview.chromium.org/17426

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@7825 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 36 insertions, 0 deletions

diff --git a/chrome/app/scoped_ole_initializer.h b/chrome/app/scoped_ole_initializer.h
new file mode 100644
index 0000000..85e47a5
--- /dev/null
+++ b/chrome/app/scoped_ole_initializer.h
@@ -0,0 +1,36 @@
+// Copyright (c) 2009 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 CHROME_APP_SCOPED_OLE_INITIALIZER_H_
+#define CHROME_APP_SCOPED_OLE_INITIALIZER_H_
+
+// Wraps OLE initialization in a cross-platform class meant to be used on the
+// stack so init/uninit is done with scoping. This class is ok for use by
+// non-windows platforms; it just doesn't do anything.
+
+#if defined(OS_WIN)
+
+class ScopedOleInitializer {
+ public:
+  ScopedOleInitializer() {
+    int ole_result = OleInitialize(NULL);
+    DCHECK(ole_result == S_OK);
+  }
+  ~ScopedOleInitializer() {
+    OleUninitialize();
+  }
+};
+
+#else
+
+class ScopedOleInitializer {
+ public:
+  // Empty, this class does nothing on non-win32 systems. Empty ctor is
+  // necessary to avoid "unused variable" warning on gcc.
+  ScopedOleInitializer() { }
+};
+
+#endif
+
+#endif  // CHROME_APP_SCOPED_OLE_INITIALIZER_H_