Disable NotificationsApiTest.TestBasicUsage for Win debug build

Disable NotificationsApiTest.TestBasicUsage for Win debug build

BUG=691913
TBR=dewittj@chromium.org

Review-Url: https://codereview.chromium.org/2708093003
Cr-Commit-Position: refs/heads/master@{#452056}
Committed: https://chromium.googlesource.com/chromium/src/+/59bef6e2cbf1b3058d14de15b4614a88fefc01ea

[fdoray, 2017-02-21 21:00:18]

The CQ bit was checked by fdoray@chromium.org

[commit-bot: I haz the power, 2017-02-21 21:03:38]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Evan Stade, 2017-02-21 21:11:10]

estade@chromium.org changed reviewers:
+ estade@chromium.org

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

https://codereview.chromium.org/2708093003/diff/1/chrome/browser/extensions/a...
File chrome/browser/extensions/api/notifications/notifications_apitest.cc
(right):

https://codereview.chromium.org/2708093003/diff/1/chrome/browser/extensions/a...
chrome/browser/extensions/api/notifications/notifications_apitest.cc:193: #if
(defined(OS_LINUX) || defined(OS_WIN)) && !defined(NDEBUG)
instead of NDEBUG, should we check DCHECK_IS_ON?

[commit-bot: I haz the power, 2017-02-21 23:07:25]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-02-21 23:07:26]

Try jobs failed on following builders:
  cast_shell_linux on master.tryserver.chromium.linux (JOB_TIMED_OUT, no build
URL)
  chromeos_amd64-generic_chromium_compile_only_ng on
master.tryserver.chromium.linux (JOB_TIMED_OUT, no build URL)
  chromeos_daisy_chromium_compile_only_ng on master.tryserver.chromium.linux
(JOB_TIMED_OUT, no build URL)
  linux_chromium_asan_rel_ng on master.tryserver.chromium.linux (JOB_TIMED_OUT,
no build URL)
  linux_chromium_chromeos_rel_ng on master.tryserver.chromium.linux
(JOB_TIMED_OUT, no build URL)
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_TIMED_OUT, no
build URL)

[fdoray, 2017-02-22 15:02:48]

The CQ bit was checked by fdoray@chromium.org

[commit-bot: I haz the power, 2017-02-22 15:03:13]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-02-22 15:21:24]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1487775768886440, "parent_rev":
"99e51965aae5623c4d16975578802caf7b3c8ac9", "commit_rev":
"59bef6e2cbf1b3058d14de15b4614a88fefc01ea"}

[commit-bot: I haz the power, 2017-02-22 15:22:05]

Description was changed from

==========
Disable NotificationsApiTest.TestBasicUsage for Win debug build

BUG=691913
TBR=dewittj@chromium.org
==========

to

==========
Disable NotificationsApiTest.TestBasicUsage for Win debug build

BUG=691913
TBR=dewittj@chromium.org

Review-Url: https://codereview.chromium.org/2708093003
Cr-Commit-Position: refs/heads/master@{#452056}
Committed:
https://chromium.googlesource.com/chromium/src/+/59bef6e2cbf1b3058d14de15b461...
==========

[commit-bot: I haz the power, 2017-02-22 15:22:06]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/59bef6e2cbf1b3058d14de15b461...