1379474 - Update various ReflowInput variable names to match the new type name

Status: RESOLVED FIXED

(Core :: Layout, enhancement, P3)

Description

[Jonathan Watt [:jwatt]]

We have a bunch of ReflowInput variable names that are still named with "state" or "size" or similar and were never updated when the argument type was changed to ReflowInput. This patch cleans that up.

Comment 1

[Jonathan Watt [:jwatt]]

Attachment: patch

Comment 2

[Jonathan Watt [:jwatt]]

Attachment: patch

Looks like Mats is away until August, so redirecting to Xidorn.

Comment 3

[Xidorn Quan [:xidorn] UTC+10]

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

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

::: layout/generic/ReflowInput.cpp
@@ +781,1 @@
>        

As far as you are here, could you also strip the tailing spaces on line 777 and 781, and wrap the "break;" into brace according to the coding style?

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/31c43a9ffa8b
Update various ReflowInput variable names to match the new type name. r=xidorn

Comment 5

[Pulsebot]

Backout by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/d0e7d5dcc8ad
Backed out changeset 31c43a9ffa8b for bustage

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/dd1e362339c5
Update various ReflowInput variable names to match the new type name. r=xidorn

Comment 7

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/dd1e362339c5