Created attachment 255499[details]
Archive of layout-test-results from ews102 for mac-mavericks
The attached test failures were seen while running run-webkit-tests on the mac-ews.
Bot: ews102 Port: mac-mavericks Platform: Mac OS X 10.9.5
Created attachment 255500[details]
Archive of layout-test-results from ews105 for mac-mavericks-wk2
The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: ews105 Port: mac-mavericks-wk2 Platform: Mac OS X 10.9.5
Created attachment 255639[details]
Patch
This new version ensures that the new policy is not applied to RenderFlexibleBox derived classes. Also includes the removal of some hacks added to properly shrink some elements
Created attachment 255642[details]
Archive of layout-test-results from ews100 for mac-mavericks
The attached test failures were seen while running run-webkit-tests on the mac-ews.
Bot: ews100 Port: mac-mavericks Platform: Mac OS X 10.9.5
Created attachment 255644[details]
Archive of layout-test-results from ews106 for mac-mavericks-wk2
The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: ews106 Port: mac-mavericks-wk2 Platform: Mac OS X 10.9.5
Created attachment 255921[details]
Archive of layout-test-results from ews102 for mac-mavericks
The attached test failures were seen while running run-webkit-tests on the mac-ews.
Bot: ews102 Port: mac-mavericks Platform: Mac OS X 10.9.5
Created attachment 255922[details]
Archive of layout-test-results from ews104 for mac-mavericks-wk2
The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: ews104 Port: mac-mavericks-wk2 Platform: Mac OS X 10.9.5
This broke various internal tools at Apple, and other companies, which now expand a flex box vertically instead of showing overflow-scroll. Were there any follow-up fixes in Blink?
(In reply to comment #28)
> This broke various internal tools at Apple, and other companies, which now
> expand a flex box vertically instead of showing overflow-scroll. Were there
> any follow-up fixes in Blink?
Flexbox code in Blink has been subject to tons of changes lately. I can try to bisect the fix for that issue but I'd need a reduced test case. Could you try to cook one?
(In reply to comment #28)
> This broke various internal tools at Apple, and other companies, which now
> expand a flex box vertically instead of showing overflow-scroll. Were there
> any follow-up fixes in Blink?
BTW just in case, as the ChangeLog says, many use cases behave the same just by adding min-width|height: 0px which was the previous default value.
Of the other bugs I attached, it looks like we definitely behave differently from Firefox and Chrome with 156786 and 157678. The others might be evangelism or behaves correctly.
It seems that under the following conditions:
- the flex container has flex-direction:column and overflow:auto/scroll;
- the flex item is a flex container itself with flex-direction:row
the default min-height:auto still doesn't work correctly (works as 0 instead of the content height). Here is a small self-explaining testcase demonstrating the issue: https://jsfiddle.net/9xzhb6m4/2/
(In reply to Ilya Streltsyn from comment #34)
> It seems that under the following conditions:
>
> - the flex container has flex-direction:column and overflow:auto/scroll;
> - the flex item is a flex container itself with flex-direction:row
>
> the default min-height:auto still doesn't work correctly (works as 0 instead
> of the content height). Here is a small self-explaining testcase
> demonstrating the issue: https://jsfiddle.net/9xzhb6m4/2/
Would you be willing to file a new bug report? Or is there some reason that a comment on this existing report is a better way to handle this?
(In reply to Ilya Streltsyn from comment #34)
> It seems that under the following conditions:
>
> - the flex container has flex-direction:column and overflow:auto/scroll;
> - the flex item is a flex container itself with flex-direction:row
>
> the default min-height:auto still doesn't work correctly (works as 0 instead
> of the content height). Here is a small self-explaining testcase
> demonstrating the issue: https://jsfiddle.net/9xzhb6m4/2/
Based on the jsfiddle, it looks like we currently agree with Chrome but disagree with Firefox.
Ilya, please file a new bug citing this example.
2015-06-24 11:23 PDT, Sergio Villar Senin
2015-06-24 11:57 PDT, Build Bot
2015-06-24 12:06 PDT, Build Bot
2015-06-26 09:16 PDT, Sergio Villar Senin
2015-06-26 09:55 PDT, Build Bot
2015-06-26 10:02 PDT, Build Bot
2015-07-01 07:42 PDT, Sergio Villar Senin
2015-07-01 08:21 PDT, Build Bot
2015-07-01 08:26 PDT, Build Bot
2015-07-02 00:17 PDT, Sergio Villar Senin
2015-09-10 03:55 PDT, Sergio Villar Senin
2015-09-10 04:02 PDT, Sergio Villar Senin