Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(62)

Issue 716013004: Disable two more Phishing* tests (Closed)

Created:
6 years, 1 month ago by vabr (Chromium)
Modified:
6 years, 1 month ago
Reviewers:
mattm
CC:
chromium-reviews, grt+watch_chromium.org, benwells
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Project:
chromium
Visibility:
Public.

Description

Disable two more Phishing* tests PhishingClassifierDelegateTest.IgnorePreliminaryCapture failed, e.g., on http://build.chromium.org/p/chromium.memory/builders/Linux%20Chromium%20OS%20ASan%20LSan%20Tests%20%281%29/builds/4359 PhishingDOMFeatureExtractorTest.Continuation failed, e.g., on http://build.chromium.org/p/chromium.memory/builders/Linux%20Chromium%20OS%20ASan%20LSan%20Tests%20%281%29/builds/4369 More info in http://crbug.com/373155#c8 BUG=373155 TBR=mattm@chromium.org Committed: https://crrev.com/066c860a26d1062f9aa700d3d652f0f6b0a0c737 Cr-Commit-Position: refs/heads/master@{#304402}

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+14 lines, -2 lines) Patch
M chrome/renderer/safe_browsing/phishing_classifier_delegate_browsertest.cc View 1 chunk +7 lines, -1 line 0 comments Download
M chrome/renderer/safe_browsing/phishing_dom_feature_extractor_browsertest.cc View 1 chunk +7 lines, -1 line 0 comments Download

Messages

Total messages: 4 (1 generated)
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/716013004/1
6 years, 1 month ago (2014-11-17 09:49:27 UTC) #2
commit-bot: I haz the power
Committed patchset #1 (id:1)
6 years, 1 month ago (2014-11-17 10:29:26 UTC) #3
commit-bot: I haz the power
6 years, 1 month ago (2014-11-17 10:30:09 UTC) #4
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/066c860a26d1062f9aa700d3d652f0f6b0a0c737
Cr-Commit-Position: refs/heads/master@{#304402}

Powered by Google App Engine
This is Rietveld 408576698