Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(763)

Unified Diff: ui/base/win/atl_module.h

Issue 10399084: Fix crash in the accessibility code in the shared build. The problem is that the ATL module pointer… (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Update license header in atl_module.h Created 8 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: ui/base/win/atl_module.h
diff --git a/ui/base/win/atl_module.h b/ui/base/win/atl_module.h
deleted file mode 100644
index 3fe3be9e9520f393c92abc4ceaf733147fdbb03e..0000000000000000000000000000000000000000
--- a/ui/base/win/atl_module.h
+++ /dev/null
@@ -1,38 +0,0 @@
-// 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.
-
-#ifndef UI_BASE_WIN_ATL_MODULE_H_
-#define UI_BASE_WIN_ATL_MODULE_H_
-#pragma once
-
-#include <atlbase.h>
-#include <atlcom.h>
-
-#include "base/basictypes.h"
-
-namespace ui {
-namespace win {
-
-// Ensure that we have exactly one ATL module registered. It's safe to
-// call this more than once. ATL functions will crash if there's no
-// ATL module registered, or if you try to register two of them, so
-// dynamically registering one if needed makes it much easier for us
-// to support different build configurations like multi-dll without
-// worrying about which side of a module boundary each ATL module object
-// belongs on.
-//
-// This function must be implemented in this header file rather than a
-// source file so that it's inlined into the module where it's included,
-// rather than in the "ui" module.
-static void CreateATLModuleIfNeeded() {
- if (_pAtlModule == NULL) {
- // This creates the module and automatically updates _pAtlModule.
- CR_DEFINE_STATIC_LOCAL(CComModule, module, ());
- }
-}
-
-} // namespace win
-} // namespace ui
-
-#endif // UI_BASE_WIN_ATL_MODULE_H_

Powered by Google App Engine
This is Rietveld 408576698