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

Unified Diff: chrome/browser/safe_browsing/client_side_detection_host_unittest.cc

Issue 99423007: [SafeBrowsing] Reset malware indicator on page nav start. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Update to use NavEntry extra data Created 7 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/safe_browsing/client_side_detection_host_unittest.cc
diff --git a/chrome/browser/safe_browsing/client_side_detection_host_unittest.cc b/chrome/browser/safe_browsing/client_side_detection_host_unittest.cc
index beda4d7afc4fd0bbf99505084c92159f84bbccf7..4169f23c3e5d90d599997c98087af595d6e66a13 100644
--- a/chrome/browser/safe_browsing/client_side_detection_host_unittest.cc
+++ b/chrome/browser/safe_browsing/client_side_detection_host_unittest.cc
@@ -48,9 +48,11 @@ using content::RenderViewHostTester;
using content::WebContents;
namespace {
+
const bool kFalse = false;
const bool kTrue = true;
Greg Billock 2013/12/13 21:10:55 What's going with this?
mattm 2013/12/14 00:09:49 I guess it's so ExpectPreClassificationChecks can
Greg Billock 2013/12/14 18:35:34 ok
-}
+
+} // namespace
namespace safe_browsing {
namespace {

Powered by Google App Engine
This is Rietveld 408576698