collectLayerFragments() expects a dirty rect relative to the multicol container.

collectLayerFragments() expects a dirty rect relative to the multicol container.

It was actually relative to the flow thread (see call site at
DeprecatedPaintLayer::collectFragments()), but it's documented as "relative to
the multicol container" in MultiColumnFragmentainerGroup::collectLayerFragments().
So make it relative to the multicol container, then.

There was no layout test to catch this fundamental RTL issue, so I added one.

BUG=511655
R=dsinclair@chromium.org,jchaffraix@chromium.org,leviw@chromium.org

Committed: https://crrev.com/988434c70468340b0be273fb7a0a8f3094bb0852
git-svn-id: svn://svn.chromium.org/blink/trunk@199697 bbb929c8-8fbe-4397-9dbb-9b2b20218538

[mstensho (USE GERRIT), 2015-07-27 11:07:50]

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

[mstensho (USE GERRIT), 2015-07-29 08:09:15]

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

[leviw_travelin_and_unemployed, 2015-07-29 23:55:30]

The CQ bit was checked by leviw@chromium.org

[leviw_travelin_and_unemployed, 2015-07-29 23:55:30]

lgtm

Great layout test :)

Just an FYI: dsinclair mostly doesn't work on the project anymore.

[commit-bot: I haz the power, 2015-07-29 23:55:45]

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

[commit-bot: I haz the power, 2015-07-30 02:25:52]

Committed patchset #1 (id:1) as
https://src.chromium.org/viewvc/blink?view=rev&revision=199697

[mstensho (USE GERRIT), 2015-07-30 05:45:21]

On 2015/07/29 23:55:30, leviw wrote:
> Just an FYI: dsinclair mostly doesn't work on the project anymore.

:(

I knew he was OOO, which is why I added more reviewers, but I didn't know he was
leaving the project. He's been my main multicol reviewer for some time now.

Then I suppose I'll use you and / or jchaffraix in the future, unless you want
to appoint someone else.

[dsinclair, 2015-08-05 15:03:42]

On 2015/07/30 05:45:21, mstensho wrote:
> On 2015/07/29 23:55:30, leviw wrote:
> > Just an FYI: dsinclair mostly doesn't work on the project anymore.
> 
> :(
> 
> I knew he was OOO, which is why I added more reviewers, but I didn't know he
was
> leaving the project. He's been my main multicol reviewer for some time now.
> 
> Then I suppose I'll use you and / or jchaffraix in the future, unless you want
> to appoint someone else.


Yea, sorry about the communication on that one. I was going to send an email
when I got back from vacation to pass the baton over to leviw@.

[mstensho (USE GERRIT), 2015-08-05 16:06:49]

On 2015/08/05 15:03:42, dsinclair wrote:
> On 2015/07/30 05:45:21, mstensho wrote:
> > On 2015/07/29 23:55:30, leviw wrote:
> > > Just an FYI: dsinclair mostly doesn't work on the project anymore.
> > 
> > :(
> > 
> > I knew he was OOO, which is why I added more reviewers, but I didn't know he
> was
> > leaving the project. He's been my main multicol reviewer for some time now.
> > 
> > Then I suppose I'll use you and / or jchaffraix in the future, unless you
want
> > to appoint someone else.
> 
> 
> Yea, sorry about the communication on that one. I was going to send an email
> when I got back from vacation to pass the baton over to leviw@.

No problem. Thanks for all your reviewing, and have fun on your next adventures!
:)

[commit-bot: I haz the power, 2015-09-23 11:54:55]

Patchset 1 (id:??) landed as
https://crrev.com/988434c70468340b0be273fb7a0a8f3094bb0852