OLD | NEW |
---|---|
1 // Copyright (c) 2006-2008 The Chromium Authors. All rights reserved. | 1 // Copyright (c) 2006-2008 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 "base/message_loop/message_pump_default.h" | 5 #include "base/message_loop/message_pump_default.h" |
6 | 6 |
7 #include "base/logging.h" | 7 #include "base/logging.h" |
8 #include "base/threading/thread_restrictions.h" | 8 #include "base/threading/thread_restrictions.h" |
9 | 9 |
10 #if defined(OS_MACOSX) | 10 #if defined(OS_MACOSX) |
(...skipping 34 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
45 break; | 45 break; |
46 | 46 |
47 if (did_work) | 47 if (did_work) |
48 continue; | 48 continue; |
49 | 49 |
50 ThreadRestrictions::ScopedAllowWait allow_wait; | 50 ThreadRestrictions::ScopedAllowWait allow_wait; |
51 if (delayed_work_time_.is_null()) { | 51 if (delayed_work_time_.is_null()) { |
52 event_.Wait(); | 52 event_.Wait(); |
53 } else { | 53 } else { |
54 TimeDelta delay = delayed_work_time_ - TimeTicks::Now(); | 54 TimeDelta delay = delayed_work_time_ - TimeTicks::Now(); |
55 // If the delay is under 1 ms we need to execute the task right away. | 55 #if defined(OS_WIN) |
56 if (delay.InMilliseconds() >= 1) { | 56 // If the delay is greater than zero and under 1 ms we need to round up to |
57 // 1 ms or else we will end up spinning until it counts down to zero | |
58 // because sub-ms waits aren't supported on Windows. | |
59 if (delay > TimeDelta()) { | |
cpu_(ooo_6.6-7.5)
2015/05/22 22:57:54
don't use the braces in single if() statement
lgt
| |
60 delay = std::max(delay, TimeDelta::FromMilliseconds(1)); | |
61 } | |
62 #endif | |
63 if (delay > TimeDelta()) { | |
57 event_.TimedWait(delay); | 64 event_.TimedWait(delay); |
58 } else { | 65 } else { |
59 // It looks like delayed_work_time_ indicates a time in the past, so we | 66 // It looks like delayed_work_time_ indicates a time in the past, so we |
60 // need to call DoDelayedWork now. | 67 // need to call DoDelayedWork now. |
61 delayed_work_time_ = TimeTicks(); | 68 delayed_work_time_ = TimeTicks(); |
62 } | 69 } |
63 } | 70 } |
64 // Since event_ is auto-reset, we don't need to do anything special here | 71 // Since event_ is auto-reset, we don't need to do anything special here |
65 // other than service each delegate method. | 72 // other than service each delegate method. |
66 } | 73 } |
(...skipping 13 matching lines...) Expand all Loading... | |
80 | 87 |
81 void MessagePumpDefault::ScheduleDelayedWork( | 88 void MessagePumpDefault::ScheduleDelayedWork( |
82 const TimeTicks& delayed_work_time) { | 89 const TimeTicks& delayed_work_time) { |
83 // We know that we can't be blocked on Wait right now since this method can | 90 // We know that we can't be blocked on Wait right now since this method can |
84 // only be called on the same thread as Run, so we only need to update our | 91 // only be called on the same thread as Run, so we only need to update our |
85 // record of how long to sleep when we do sleep. | 92 // record of how long to sleep when we do sleep. |
86 delayed_work_time_ = delayed_work_time; | 93 delayed_work_time_ = delayed_work_time; |
87 } | 94 } |
88 | 95 |
89 } // namespace base | 96 } // namespace base |
OLD | NEW |