Commit 7b6d6034 authored by Henrik Boström's avatar Henrik Boström Committed by Commit Bot

Disable flaking ReportRejectedByTriggerManager test on Windows.

See https://crbug.com/1009917

TBR=archanasimha@google.com, vakh@chromium.org
NOTRY=True # Yolo

Bug: 1009917
Change-Id: Id0def6288f9b1386f8dbfc0df0647882be0529a5
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1831876
Commit-Queue: Henrik Boström <hbos@chromium.org>
Reviewed-by: default avatarHenrik Boström <hbos@chromium.org>
Cr-Commit-Position: refs/heads/master@{#701516}
parent 8b846c39
......@@ -8,6 +8,7 @@
#include "base/test/metrics/histogram_tester.h"
#include "base/test/scoped_feature_list.h"
#include "base/test/test_simple_task_runner.h"
#include "build/build_config.h"
#include "components/prefs/testing_pref_service.h"
#include "components/safe_browsing/features.h"
#include "components/safe_browsing/triggers/mock_trigger_manager.h"
......@@ -138,7 +139,14 @@ TEST_F(AdPopupTriggerTest, PopupWithAds) {
1);
}
TEST_F(AdPopupTriggerTest, ReportRejectedByTriggerManager) {
// TODO(https://crbug.com/1009917): Fix flakes on Windows bots.
#if defined(OS_WIN)
#define MAYBE_ReportRejectedByTriggerManager \
DISABLED_ReportRejectedByTriggerManager
#else
#define MAYBE_ReportRejectedByTriggerManager ReportRejectedByTriggerManager
#endif
TEST_F(AdPopupTriggerTest, MAYBE_ReportRejectedByTriggerManager) {
// If the trigger manager rejects the report, we don't try to finish/send the
// report.
CreateTrigger();
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment