Make MultiColumnFragmentainerGroup::m_columnSet const.

Make MultiColumnFragmentainerGroup::m_columnSet const.

Ideally, I'd like to get rid of the member altogether, but that would require a
lot of refactoring.

This is a preparatory patch for a fix for bug 604609.

BUG=604609
Committed: https://crrev.com/69b27e85cdf52da2d0be4f011b92b77c461e12ab
Cr-Commit-Position: refs/heads/master@{#388326}

[mstensho (USE GERRIT), 2016-04-19 19:07:38]

mstensho@opera.com changed reviewers:
+ leviw@chromium.org

[mstensho (USE GERRIT), 2016-04-19 19:07:40]

https://codereview.chromium.org/1898293003/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/layout/MultiColumnFragmentainerGroup.h
(right):

https://codereview.chromium.org/1898293003/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/layout/MultiColumnFragmentainerGroup.h:63: bool
recalculateColumnHeight(LayoutMultiColumnSet&);
So, how silly is this? I need a non-const reference here, so that I can call
propagateTallestUnbreakableLogicalHeight().

Any better suggestions?

[leviw_travelin_and_unemployed, 2016-04-19 19:57:51]

https://codereview.chromium.org/1898293003/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/layout/MultiColumnFragmentainerGroup.h
(right):

https://codereview.chromium.org/1898293003/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/layout/MultiColumnFragmentainerGroup.h:63: bool
recalculateColumnHeight(LayoutMultiColumnSet&);
On 2016/04/19 at 19:07:39, mstensho wrote:
> So, how silly is this? I need a non-const reference here, so that I can call
propagateTallestUnbreakableLogicalHeight().
> 
> Any better suggestions?

It's silly, but I don't have better solutions. I'd rather avoid that nasty word
mutable.

LGTM.

[mstensho (USE GERRIT), 2016-04-19 20:04:36]

The CQ bit was checked by mstensho@opera.com

[commit-bot: I haz the power, 2016-04-19 20:05:12]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1898293003/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1898293003/1

[commit-bot: I haz the power, 2016-04-19 22:11:03]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-04-22 19:16:58]

Description was changed from

==========
Make MultiColumnFragmentainerGroup::m_columnSet const.

Ideally, I'd like to get rid of the member altogether, but that would require a
lot of refactoring.

This is a preparatory patch for a fix for bug 604609.

BUG=604609
==========

to

==========
Make MultiColumnFragmentainerGroup::m_columnSet const.

Ideally, I'd like to get rid of the member altogether, but that would require a
lot of refactoring.

This is a preparatory patch for a fix for bug 604609.

BUG=604609

Committed: https://crrev.com/69b27e85cdf52da2d0be4f011b92b77c461e12ab
Cr-Commit-Position: refs/heads/master@{#388326}
==========

[commit-bot: I haz the power, 2016-04-22 19:16:59]

Patchset 1 (id:??) landed as
https://crrev.com/69b27e85cdf52da2d0be4f011b92b77c461e12ab
Cr-Commit-Position: refs/heads/master@{#388326}