Changeset 41878 in webkit
- Timestamp:
- Mar 20, 2009 4:45:45 PM (15 years ago)
- Location:
- trunk/WebCore
- Files:
-
- 3 edited
Legend:
- Unmodified
- Added
- Removed
-
trunk/WebCore/ChangeLog
r41877 r41878 1 2009-03-20 Mike Belshe <mike@belshe.com> 2 3 Reviewed by Dimitri Glazkov and Dave Hyatt. 4 5 https://bugs.webkit.org/show_bug.cgi?id=24324 6 7 Make the minimum timer configurable for different platforms. 8 9 * page/DOMTimer.cpp: 10 (WebCore::DOMTimer::DOMTimer): 11 (WebCore::DOMTimer::fired): 12 * page/DOMTimer.h: 13 (WebCore::DOMTimer::minTimerInterval): 14 (WebCore::DOMTimer::setMinTimerInterval): 15 1 16 2009-03-20 Dean McNamee <deanm@chromium.org> 2 17 -
trunk/WebCore/page/DOMTimer.cpp
r40534 r41878 39 39 static const int maxTimerNestingLevel = 5; 40 40 static const double oneMillisecond = 0.001; 41 static const doubleminTimerInterval = 0.010; // 10 milliseconds41 double DOMTimer::s_minTimerInterval = 0.010; // 10 milliseconds 42 42 43 43 static int timerNestingLevel = 0; … … 65 65 // nested enough to notice that we're repeating. 66 66 // Faster timers might be "better", but they're incompatible. 67 if (intervalMilliseconds < minTimerInterval && m_nestingLevel >= maxTimerNestingLevel)68 intervalMilliseconds = minTimerInterval;67 if (intervalMilliseconds < s_minTimerInterval && m_nestingLevel >= maxTimerNestingLevel) 68 intervalMilliseconds = s_minTimerInterval; 69 69 if (singleShot) 70 70 startOneShot(intervalMilliseconds); … … 106 106 // Simple case for non-one-shot timers. 107 107 if (isActive()) { 108 if (repeatInterval() && repeatInterval() < minTimerInterval) {108 if (repeatInterval() && repeatInterval() < s_minTimerInterval) { 109 109 m_nestingLevel++; 110 110 if (m_nestingLevel >= maxTimerNestingLevel) 111 augmentRepeatInterval( minTimerInterval - repeatInterval());111 augmentRepeatInterval(s_minTimerInterval - repeatInterval()); 112 112 } 113 113 -
trunk/WebCore/page/DOMTimer.h
r40271 r41878 52 52 virtual void resume(); 53 53 54 // The lowest allowable timer setting (in seconds, 0.001 == 1 ms). 55 // Default is 10ms. 56 // Chromium uses a non-default timeout. 57 static double minTimerInterval() { return s_minTimerInterval; } 58 static void setMinTimerInterval(double value) { s_minTimerInterval = value; } 59 54 60 private: 55 61 DOMTimer(ScriptExecutionContext*, ScheduledAction*, int timeout, bool singleShot); … … 61 67 double m_nextFireInterval; 62 68 double m_repeatInterval; 69 static double s_minTimerInterval; 63 70 }; 64 71
Note: See TracChangeset
for help on using the changeset viewer.