[New Multicolumn] Remove unnecessary calls to setRegionBasedColumnsEnabled.

[New Multicolumn] Remove unnecessary calls to setRegionBasedColumnsEnabled.

Tests which enable region based multicolumn behavior using Javascript
don't need to do that anymore, because they are run via
virtual/regionbasedmulticol, which enables the new behavior using a
command line flag.

Note that tests in fast/multicol/newmulticol/compare-with-old-impl still
need to call internals.settings.setRegionBasedColumnsEnabled(true). They
are now the only tests which need to do so. (And it should remain like
that until they are removed during Issue 350853).

BUG=334335
R=rune@opera.com

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

[rune, 2014-06-30 20:41:11]

lgtm

[Julien - ping for review, 2014-06-30 21:30:08]

lgtm

Ideally we should remove the Settings code and use a pure runtime flag. This can
be done in a separate patch though.

[andersr, 2014-07-04 07:40:27]

On 2014/06/30 21:30:08, Julien Chaffraix - PST wrote:
> lgtm
> 
> Ideally we should remove the Settings code and use a pure runtime flag. This
can
> be done in a separate patch though.

Yes, that's the plan ( http://crbug.com/350853 ).

[andersr, 2014-07-04 07:40:38]

The CQ bit was checked by andersr@opera.com

[commit-bot: I haz the power, 2014-07-04 07:41:44]

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

[commit-bot: I haz the power, 2014-07-04 08:48:44]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  linux_chromium_gn_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/linux_chromium_gn_rel/bu...)

[commit-bot: I haz the power, 2014-07-04 09:25:43]

Change committed as 177528