1237813 - refactor nsLayoutUtils::FindScrollableFrameFor to use GetScrollFrameFromContent

Status: RESOLVED FIXED

(Core :: Layout, defect)

Description

[Timothy Nikkel (:tnikkel)]

From bug 1194851, comment 49.

Comment 1

[Timothy Nikkel (:tnikkel)]

Attachment: patch

Comment 2

[Botond Ballo [:botond]]

Comment on attachment 8705382 [details] [diff] [review]
patch

Review of attachment 8705382 [details] [diff] [review]:
-----------------------------------------------------------------

::: layout/base/nsLayoutUtils.cpp
@@ +774,5 @@
>    if (!content) {
>      return nullptr;
>    }
>  
> +  nsIFrame* scrolledFrame = GetScrollFrameFromContent(content);

This should probably be called "scrollFrame" rather than "scrolledFrame"?

Comment 3

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/6529572ba56e

Comment 4

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/6529572ba56e