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

Unified Diff: third_party/WebKit/Source/platform/Timer.cpp

Issue 2513893002: Make ifdefs consistent in WebKit/Source/core/ (Closed)
Patch Set: Added #if DCHECK_IS_ON where needed. Created 4 years, 1 month 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: third_party/WebKit/Source/platform/Timer.cpp
diff --git a/third_party/WebKit/Source/platform/Timer.cpp b/third_party/WebKit/Source/platform/Timer.cpp
index 18202fe630183598c3b5d0f50fda86782a8d6ca0..768ef2304fad53c233ceb34d8e71796346383660 100644
--- a/third_party/WebKit/Source/platform/Timer.cpp
+++ b/third_party/WebKit/Source/platform/Timer.cpp
@@ -58,7 +58,9 @@ TimerBase::~TimerBase() {
void TimerBase::start(double nextFireInterval,
double repeatInterval,
const WebTraceLocation& caller) {
- ASSERT(m_thread == currentThread());
+#if DCHECK_IS_ON()
+ DCHECK(m_thread == currentThread());
Yuta Kitamura 2016/11/21 09:26:12 Ditto for this file.
Alexander Alekseev 2016/11/23 09:20:30 Done.
+#endif
m_location = caller;
m_repeatInterval = repeatInterval;
@@ -66,7 +68,9 @@ void TimerBase::start(double nextFireInterval,
}
void TimerBase::stop() {
- ASSERT(m_thread == currentThread());
+#if DCHECK_IS_ON()
+ DCHECK(m_thread == currentThread());
+#endif
m_repeatInterval = 0;
m_nextFireTime = 0;
@@ -96,7 +100,9 @@ WebTaskRunner* TimerBase::timerTaskRunner() const {
}
void TimerBase::setNextFireTime(double now, double delay) {
- ASSERT(m_thread == currentThread());
+#if DCHECK_IS_ON()
+ DCHECK(m_thread == currentThread());
+#endif
double newTime = now + delay;

Powered by Google App Engine
This is Rietveld 408576698