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

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

Issue 2530493002: WebKit: make #if DCHECK_IS_ON consistent in third_party/WebKit/Source/platform/ (Closed)
Patch Set: 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..b02e8b6c8e52b447cd9e6b50c130d73445ec50f8 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_EQ(m_thread, currentThread());
+#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_EQ(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_EQ(m_thread, currentThread());
+#endif
double newTime = now + delay;

Powered by Google App Engine
This is Rietveld 408576698