Disable NotificationsApiTest.TestBasicUsage for Win debug build

chrome/browser/extensions/api/notifications/notifications_apitest.cc

 // Copyright (c) 2012 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/strings/string_number_conversions.h"
 #include "base/strings/stringprintf.h"
 #include "base/strings/utf_string_conversions.h"
 #include "base/test/scoped_feature_list.h"
+#include "build/build_config.h"
 #include "chrome/browser/apps/app_browsertest_util.h"
 #include "chrome/browser/browser_process.h"
 #include "chrome/browser/extensions/api/notifications/notifications_api.h"
 #include "chrome/browser/extensions/extension_apitest.h"
 #include "chrome/browser/extensions/extension_function_test_utils.h"
 #include "chrome/browser/notifications/notification.h"
 #include "chrome/browser/notifications/notification_ui_manager.h"
 #include "chrome/browser/profiles/profile_manager.h"
 #include "chrome/browser/ui/extensions/app_launch_params.h"
 #include "chrome/browser/ui/extensions/application_launch.h"
@@ skipping 163 matching lines @@
         {extensions::kAllowFullscreenAppNotificationsFeature});
   }
 
  private:
   base::test::ScopedFeatureList feature_list_;
 };
 
 }  // namespace
 
 // http://crbug.com/691913
-#if defined(OS_LINUX) && !defined(NDEBUG)
+#if (defined(OS_LINUX) || defined(OS_WIN)) && !defined(NDEBUG)

[Evan Stade, 2017/02/21 21:11:11]

instead of NDEBUG, should we check DCHECK_IS_ON?

 #define MAYBE_TestBasicUsage DISABLED_TestBasicUsage
 #else
 #define MAYBE_TestBasicUsage TestBasicUsage
 #endif
 IN_PROC_BROWSER_TEST_F(NotificationsApiTest, MAYBE_TestBasicUsage) {
   ASSERT_TRUE(RunExtensionTest("notifications/api/basic_usage")) << message_;
 }
 
 IN_PROC_BROWSER_TEST_F(NotificationsApiTest, TestEvents) {
   ASSERT_TRUE(RunExtensionTest("notifications/api/events")) << message_;
@@ skipping 319 matching lines @@
 
   ASSERT_TRUE(GetFirstAppWindow(extension->id())->IsFullscreen())
       << "Not Fullscreen";
   ASSERT_TRUE(GetFirstAppWindow(extension->id())->GetBaseWindow()->IsActive())
       << "Not Active";
 
   const message_center::NotificationList::PopupNotifications notifications =
       g_browser_process->message_center()->GetPopupNotifications();
   ASSERT_EQ(1u, notifications.size());
 }