Bug 1225950 - Remove unnecessary parameter from SetNeedsComposite. r=mchang

This commit is contained in:
Kartikaya Gupta 2015-11-18 18:49:58 -05:00
parent 28ca4b5b3d
commit 509226984e
3 changed files with 6 additions and 7 deletions

View File

@ -342,7 +342,7 @@ CompositorVsyncScheduler::ScheduleComposition()
// Used only for performance testing purposes
PostCompositeTask(TimeStamp::Now());
} else {
SetNeedsComposite(true);
SetNeedsComposite();
}
}
@ -366,13 +366,12 @@ CompositorVsyncScheduler::CancelCurrentSetNeedsCompositeTask()
* How many skipped vsync events until we stop listening to vsync events?
*/
void
CompositorVsyncScheduler::SetNeedsComposite(bool aNeedsComposite)
CompositorVsyncScheduler::SetNeedsComposite()
{
if (!CompositorParent::IsInCompositorThread()) {
MonitorAutoLock lock(mSetNeedsCompositeMonitor);
mSetNeedsCompositeTask = NewRunnableMethod(this,
&CompositorVsyncScheduler::SetNeedsComposite,
aNeedsComposite);
&CompositorVsyncScheduler::SetNeedsComposite);
ScheduleTask(mSetNeedsCompositeTask, 0);
return;
} else {
@ -380,7 +379,7 @@ CompositorVsyncScheduler::SetNeedsComposite(bool aNeedsComposite)
mSetNeedsCompositeTask = nullptr;
}
mNeedsComposite = aNeedsComposite;
mNeedsComposite = true;
if (!mIsObservingVsync && mNeedsComposite) {
ObserveVsync();
}

View File

@ -101,7 +101,7 @@ class CompositorVsyncScheduler
public:
explicit CompositorVsyncScheduler(CompositorParent* aCompositorParent, nsIWidget* aWidget);
bool NotifyVsync(TimeStamp aVsyncTimestamp);
void SetNeedsComposite(bool aSchedule);
void SetNeedsComposite();
void OnForceComposeToTarget();
void ScheduleTask(CancelableTask*, int);

View File

@ -105,7 +105,7 @@ void
GeckoTouchDispatcher::NotifyTouch(MultiTouchInput& aTouch, TimeStamp aEventTime)
{
if (mCompositorVsyncScheduler) {
mCompositorVsyncScheduler->SetNeedsComposite(true);
mCompositorVsyncScheduler->SetNeedsComposite();
}
if (aTouch.mType == MultiTouchInput::MULTITOUCH_MOVE) {