Improve support for multiselect list box accessibility on Windows.

base/win/iunknown_impl.cc

+// 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.
+
+#include "base/win/iunknown_impl.h"
+
+namespace base {
+namespace win {
+
+IUnknownImpl::IUnknownImpl()
+    : ref_count_(0) {}

[M-A Ruel, 2011/12/01 00:35:13]

One reason to keep the brackets on two lines, like you did for the destructor,
is that it makes it easier to put a breakpoint with Visual Studio.

[dmazzoni, 2011/12/02 00:44:07]

Done.

+
+IUnknownImpl::~IUnknownImpl() {
+}
+
+ULONG STDMETHODCALLTYPE IUnknownImpl::AddRef() {
+  return base::AtomicRefCountIncAndReturnValue(&ref_count_);
+}
+
+ULONG STDMETHODCALLTYPE IUnknownImpl::Release() {
+  AtomicRefCount new_ref_count =
+      base::AtomicRefCountDecAndReturnValue(&ref_count_);
+  if (new_ref_count == 0)
+    delete this;
+  return new_ref_count;
+}
+
+STDMETHODIMP IUnknownImpl::QueryInterface(REFIID riid, void** ppv) {
+  if (riid == IID_IUnknown) {
+    *ppv = static_cast<IUnknown*>(this);
+    AddRef();
+    return S_OK;
+  }
+
+  *ppv = NULL;
+  return E_NOINTERFACE;
+}
+
+}  // namespace win
+}  // namespace base