[Text Autosizing] Cleanup change: converted the pointer argument to be a reference since non-null pointer is always expected

[Anton Vayvod]

[Text Autosizing] Cleanup change: converter the pointer argument to be a reference since non-null pointer is always expected

[Anton Vayvod]

Created attachment 186884 [details]
Patch

[Julien Chaffraix]

Comment on attachment 186884 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=186884&action=review

> Source/WebCore/rendering/TextAutosizer.cpp:298
>      TextAutosizingClusterInfo emptyClusterInfo(0);
> -    return isAutosizingContainer(object) && isAutosizingCluster(toRenderBlock(object), &emptyClusterInfo);
> +    return isAutosizingContainer(object) && isAutosizingCluster(toRenderBlock(object), emptyClusterInfo);

Mentioned on bug 109054, I would rather see isAutosizingCluster take a pointer that can be NULL instead of having an empty object. That would remove the need for this overriden method.

[Build Bot]

Comment on attachment 186884 [details]
Patch

Attachment 186884 [details] did not pass win-ews (win):
Output: http://queues.webkit.org/results/16387485

[Anton Vayvod]

Created attachment 187347 [details]
Patch

[WebKit Review Bot]

Comment on attachment 187347 [details]
Patch

Clearing flags on attachment: 187347

Committed r142367: <http://trac.webkit.org/changeset/142367>

[WebKit Review Bot]

All reviewed patches have been landed.  Closing bug.