Update ui::CompositorObserver instrumentation

ui/compositor/compositor_observer.cc

 // Copyright 2014 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 "ui/compositor/compositor_observer.h"
 
 #include "base/logging.h"
 
 namespace ui {
 
-#if defined(OS_MACOSX)
-// Debugging instrumentation for crbug.com/401630.
-// TODO(ccameron): remove this.
+#if DCHECK_IS_ON
 CompositorObserver::CompositorObserver() : observing_count_(0) {}
 #else
 CompositorObserver::CompositorObserver() {}
 #endif
 
 CompositorObserver::~CompositorObserver() {
+  // TODO(ccameron): Make this check not fire on non-Mac platforms.
+  // http://crbug.com/403011
 #if defined(OS_MACOSX)

[danakj, 2014/08/12 20:57:54]

Did you wanna remove this?

[ccameron, 2014/08/12 21:04:24]

Nah, I wanted to keep the DCHECK for Mac, and eventually delete the
defined(OS_MACOSX) -- there are a bazillion crashes if it's on for non-Mac ATM
(I filed a bug against that and assigned it to me).

-  // Debugging instrumentation for crbug.com/401630.
-  // TODO(ccameron): remove this.
-  CHECK_EQ(observing_count_, 0);
+  DCHECK_EQ(observing_count_, 0);
 #endif
 }
 
 }