Bug 1468545 - Remove incorrect comment. r?botond
This comment is wrong because the scrollbar might have scheduled a
post-refresh confirmation, which would not have been sent yet. So if we
sent the preventDefault flag as true, APZ would drop the block improperly,
because the drag block might later (as part of the post-refresh) get
confirmed.
MozReview-Commit-ID: HKzMatUOeiG
--- a/gfx/layers/apz/util/APZEventState.cpp
+++ b/gfx/layers/apz/util/APZEventState.cpp
@@ -413,21 +413,16 @@ APZEventState::ProcessWheelEvent(const W
mContentReceivedInputBlockCallback(aGuid, aInputBlockId, defaultPrevented);
}
void
APZEventState::ProcessMouseEvent(const WidgetMouseEvent& aEvent,
const ScrollableLayerGuid& aGuid,
uint64_t aInputBlockId)
{
- // If we get here and the drag block has not been confirmed by the code in
- // nsSliderFrame, then no scrollbar reacted to the event thus APZC will
- // ignore this drag block. We can send defaultPrevented as either true or
- // false, it doesn't matter, because APZ won't have the scrollbar metrics
- // anyway, and will know to drop the block.
bool defaultPrevented = false;
mContentReceivedInputBlockCallback(aGuid, aInputBlockId, defaultPrevented);
}
void
APZEventState::ProcessAPZStateChange(ViewID aViewId,
APZStateChange aChange,
int aArg)