|
[New Multicolumn] Make offsetLeft, offsetTop and getClientRects() behave.
Get rid of mapFromFlowToRegion() and mapLocalToContainer() in
RenderFlowThread, and reimplement columnOffset() in
RenderMultiColumnFlowThread instead. This method is already used a lot
in the engine, but it has up until now only worked with the old multicol
implementation. Need to add a call from
RenderBox::offsetFromContainer(), since the one that's already there
is for old multicol only and does special stuff.
This incidentally also gets rid of assertion failures in
RenderGeometryMap when compositing is turned on (what used to be in
RenderFlowThread::mapLocalToContainer() did the wrong thing if the
target was that flow thread itself). Re-enable tests that used to
fail on that assertion.
BUG= 367688, 364960
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=172961
|
Unified diffs |
Side-by-side diffs |
Delta from patch set |
Stats (+211 lines, -53 lines) |
Patch |
|
M |
LayoutTests/TestExpectations
|
View
|
1
|
1 chunk |
+0 lines, -5 lines |
0 comments
|
Download
|
|
A |
LayoutTests/fast/multicol/cssom-view.html
|
View
|
|
1 chunk |
+50 lines, -0 lines |
0 comments
|
Download
|
|
A |
LayoutTests/fast/multicol/cssom-view-expected.txt
|
View
|
|
1 chunk |
+37 lines, -0 lines |
0 comments
|
Download
|
|
A |
LayoutTests/fast/multicol/inline-getclientrects.html
|
View
|
|
1 chunk |
+36 lines, -0 lines |
0 comments
|
Download
|
|
A |
LayoutTests/fast/multicol/inline-getclientrects-expected.txt
|
View
|
|
1 chunk |
+19 lines, -0 lines |
0 comments
|
Download
|
|
A + |
LayoutTests/fast/multicol/newmulticol/compare-with-old-impl/client-rects.html
|
View
|
|
1 chunk |
+5 lines, -0 lines |
0 comments
|
Download
|
|
A + |
LayoutTests/fast/multicol/newmulticol/compare-with-old-impl/client-rects-expected.html
|
View
|
|
1 chunk |
+1 line, -0 lines |
0 comments
|
Download
|
|
M |
LayoutTests/resources/js-test.js
|
View
|
|
1 chunk |
+21 lines, -0 lines |
0 comments
|
Download
|
|
M |
Source/core/rendering/RenderBlock.h
|
View
|
|
1 chunk |
+1 line, -1 line |
0 comments
|
Download
|
|
M |
Source/core/rendering/RenderBox.cpp
|
View
|
|
2 chunks |
+11 lines, -4 lines |
0 comments
|
Download
|
|
M |
Source/core/rendering/RenderFlowThread.h
|
View
|
|
2 chunks |
+0 lines, -4 lines |
0 comments
|
Download
|
|
M |
Source/core/rendering/RenderFlowThread.cpp
|
View
|
|
2 chunks |
+0 lines, -39 lines |
0 comments
|
Download
|
|
M |
Source/core/rendering/RenderMultiColumnFlowThread.h
|
View
|
|
1 chunk |
+2 lines, -0 lines |
0 comments
|
Download
|
|
M |
Source/core/rendering/RenderMultiColumnFlowThread.cpp
|
View
|
|
1 chunk |
+14 lines, -0 lines |
0 comments
|
Download
|
|
M |
Source/core/rendering/RenderMultiColumnSet.h
|
View
|
|
1 chunk |
+4 lines, -0 lines |
0 comments
|
Download
|
|
M |
Source/core/rendering/RenderMultiColumnSet.cpp
|
View
|
|
1 chunk |
+10 lines, -0 lines |
0 comments
|
Download
|
Total messages: 9 (0 generated)
|