nagai/fune
3
0
Fork 0
forked from mirrors/gecko-dev
Code Pull requests Activity

Bug 1789811 - Move ReentrantMonitorAutoEnter close to the monitor usage in SynchronousTask. r=bradwerth

Browse source 
Differential Revision: https://phabricator.services.mozilla.com/D179807
This commit is contained in:
Jens Stutte 2023-06-05 08:03:14 +00:00
parent 6d7d951bde
commit 669b302e69
1 changed files with 5 additions and 6 deletions
repo.diff.show_diff_stats Download patch file Download diff file Expand all files Collapse all files

11
gfx/layers/ipc/SynchronousTask.h
View file

@ -17,10 +17,11 @@ class MOZ_STACK_CLASS SynchronousTask {
friend class AutoCompleteTask;
public:
explicit SynchronousTask(const char* name)
: mMonitor(name), mAutoEnter(mMonitor), mDone(false) {}
explicit SynchronousTask(const char* name) : mMonitor(name), mDone(false) {}
nsresult Wait(PRIntervalTime aInterval = PR_INTERVAL_NO_TIMEOUT) {
ReentrantMonitorAutoEnter lock(mMonitor);
// For indefinite timeouts, wait in a while loop to handle spurious
// wakeups.
while (aInterval == PR_INTERVAL_NO_TIMEOUT && !mDone) {
@ -46,25 +47,23 @@ class MOZ_STACK_CLASS SynchronousTask {
private:
void Complete() {
ReentrantMonitorAutoEnter lock(mMonitor);
mDone = true;
mMonitor.NotifyAll();
}
private:
ReentrantMonitor mMonitor MOZ_UNANNOTATED;
ReentrantMonitorAutoEnter mAutoEnter;
bool mDone;
};
class MOZ_STACK_CLASS AutoCompleteTask final {
public:
explicit AutoCompleteTask(SynchronousTask* aTask)
: mTask(aTask), mAutoEnter(aTask->mMonitor) {}
explicit AutoCompleteTask(SynchronousTask* aTask) : mTask(aTask) {}
~AutoCompleteTask() { mTask->Complete(); }
private:
SynchronousTask* mTask;
ReentrantMonitorAutoEnter mAutoEnter;
};
} // namespace layers