Bug 1304457 - Add some missing mCanSend checks. r?dvander
MozReview-Commit-ID: Jctjbgilu5z
--- a/gfx/layers/ipc/RemoteContentController.cpp
+++ b/gfx/layers/ipc/RemoteContentController.cpp
@@ -132,42 +132,48 @@ RemoteContentController::UpdateOverscrol
{
if (MessageLoop::current() != mCompositorThread) {
mCompositorThread->PostTask(NewRunnableMethod<float,
float, bool>(this,
&RemoteContentController::UpdateOverscrollVelocity,
aX, aY, aIsRootContent));
return;
}
- Unused << SendUpdateOverscrollVelocity(aX, aY, aIsRootContent);
+ if (mCanSend) {
+ Unused << SendUpdateOverscrollVelocity(aX, aY, aIsRootContent);
+ }
}
void
RemoteContentController::UpdateOverscrollOffset(float aX, float aY, bool aIsRootContent)
{
if (MessageLoop::current() != mCompositorThread) {
mCompositorThread->PostTask(NewRunnableMethod<float,
float, bool>(this,
&RemoteContentController::UpdateOverscrollOffset,
aX, aY, aIsRootContent));
return;
}
- Unused << SendUpdateOverscrollOffset(aX, aY, aIsRootContent);
+ if (mCanSend) {
+ Unused << SendUpdateOverscrollOffset(aX, aY, aIsRootContent);
+ }
}
void
RemoteContentController::SetScrollingRootContent(bool aIsRootContent)
{
if (MessageLoop::current() != mCompositorThread) {
mCompositorThread->PostTask(NewRunnableMethod<bool>(this,
&RemoteContentController::SetScrollingRootContent,
aIsRootContent));
return;
}
- Unused << SendSetScrollingRootContent(aIsRootContent);
+ if (mCanSend) {
+ Unused << SendSetScrollingRootContent(aIsRootContent);
+ }
}
void
RemoteContentController::NotifyMozMouseScrollEvent(const FrameMetrics::ViewID& aScrollId,
const nsString& aEvent)
{
if (MessageLoop::current() != mCompositorThread) {
// We have to send messages from the compositor thread