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

Side by Side Diff: third_party/WebKit/Source/wtf/AssertionsTest.cpp

Issue 2640163004: Replace ENABLE(ASSERT) with DCHECK_IS_ON(). (Closed)
Patch Set: Created 3 years, 11 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 unified diff | Download patch
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "wtf/Assertions.h" 5 #include "wtf/Assertions.h"
6 6
7 #include "testing/gtest/include/gtest/gtest.h" 7 #include "testing/gtest/include/gtest/gtest.h"
8 #include "wtf/text/StringBuilder.h" 8 #include "wtf/text/StringBuilder.h"
9 #include <stdio.h> 9 #include <stdio.h>
10 10
11 namespace WTF { 11 namespace WTF {
12 12
13 TEST(AssertionsTest, Assertions) { 13 TEST(AssertionsTest, Assertions) {
14 ASSERT(true); 14 ASSERT(true);
15 #if ENABLE(ASSERT) 15 #if DCHECK_IS_ON()
16 EXPECT_DEATH_IF_SUPPORTED(ASSERT(false), ""); 16 EXPECT_DEATH_IF_SUPPORTED(ASSERT(false), "");
17 EXPECT_DEATH_IF_SUPPORTED(ASSERT_NOT_REACHED(), ""); 17 EXPECT_DEATH_IF_SUPPORTED(ASSERT_NOT_REACHED(), "");
18 #endif 18 #endif
19 19
20 RELEASE_ASSERT(true); 20 RELEASE_ASSERT(true);
21 EXPECT_DEATH_IF_SUPPORTED(RELEASE_ASSERT(false), ""); 21 EXPECT_DEATH_IF_SUPPORTED(RELEASE_ASSERT(false), "");
22 22
23 SECURITY_DCHECK(true); 23 SECURITY_DCHECK(true);
24 #if ENABLE(SECURITY_ASSERT) 24 #if ENABLE(SECURITY_ASSERT)
25 EXPECT_DEATH_IF_SUPPORTED(SECURITY_DCHECK(false), ""); 25 EXPECT_DEATH_IF_SUPPORTED(SECURITY_DCHECK(false), "");
(...skipping 37 matching lines...) Expand 10 before | Expand all | Expand 10 after
63 " ( c\n" 63 " ( c\n"
64 " ( d -1 hello )\n" 64 " ( d -1 hello )\n"
65 " )\n" 65 " )\n"
66 " a2 0.5\n" 66 " a2 0.5\n"
67 ")\n", 67 ")\n",
68 gBuilder.toString()); 68 gBuilder.toString());
69 }; 69 };
70 #endif // !LOG_DISABLED 70 #endif // !LOG_DISABLED
71 71
72 } // namespace WTF 72 } // namespace WTF
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698