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

Unified Diff: chrome/installer/util/installer_state_unittest.cc

Issue 606473002: Remove implicit HANDLE conversions from chrome. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 3 months 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/installer/util/installer_state_unittest.cc
diff --git a/chrome/installer/util/installer_state_unittest.cc b/chrome/installer/util/installer_state_unittest.cc
index e0cf3bdf115d1ec16416f0c29b13ca4d2c985a42..974cb86bb3769d15fc942af3ca9cd23a06dbae31 100644
--- a/chrome/installer/util/installer_state_unittest.cc
+++ b/chrome/installer/util/installer_state_unittest.cc
@@ -486,7 +486,7 @@ TEST_F(InstallerStateTest, IsFileInUse) {
SYNCHRONIZE | FILE_EXECUTE,
FILE_SHARE_DELETE | FILE_SHARE_READ,
NULL, OPEN_EXISTING, 0, 0));
- ASSERT_TRUE(temp_handle != NULL);
+ ASSERT_TRUE(temp_handle.IsValid());
// The file should now be in use.
EXPECT_TRUE(MockInstallerState::IsFileInUse(temp_file));

Powered by Google App Engine
This is Rietveld 408576698