[New Multicolumn] Cleanup: Remove redundant members.

[New Multicolumn] Cleanup: Remove redundant members.

RenderMultiColumnSet doesn't need to store column width and
height. These are readily available from the flow thread.

And the flow thread doesn't need an extra member to store column
width, since it's the same as the logical width. Calculate it on
the fly when we need to determine whether the flow thread will
get a new width, and when actually updating the logical
width. It's not expensive to calculate, and this is not a hot
path. Duplicated storage can only cause confusion and do harm,
anyway.

Also use the CSS 2.1 terms "used" and "actual" when we need to
disambiguate, rather than "computed" and
nothing. Remove "computed" from some members where it didn't add
any value. Those were not CSS 2.1 "computed" values, so I found
it somewhat confusing and misleading.

No need for columnWidth() and columnHeight() in
RenderMultiColumnSet.  Use pageLogicalWidth() and
pageLogicalHeight() instead.

No need for columnWidth() in RenderMultiColumnFlowThread either.

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=174870

[mstensho (USE GERRIT), 2014-05-23 15:58:30]

This patch depends on https://codereview.chromium.org/288263002/

[Julien - ping for review, 2014-05-27 14:23:40]

lgtm after reformatting the description to be <80 characters long (see
https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/rqGhY7PfkYY). I
usually recommend at 72 characters (per the thread as there is a good reason for
this value)

[mstensho (USE GERRIT), 2014-05-27 14:35:05]

The CQ bit was checked by mstensho@opera.com

[commit-bot: I haz the power, 2014-05-27 14:35:18]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/mstensho@opera.com/299213002/1

[commit-bot: I haz the power, 2014-05-27 16:00:11]

Change committed as 174870