Eric Rahm [:erahm] Reporter
	Description • 9 years ago

+++ This bug was initially created as a clone of Bug #525100 +++

> 1054 [NNNNN] WARNING: have unconstrained inline-size; this should only result from very large sizes, not attempts at intrinsic inline-size calculation: 'aContainingBlockISize != NS_UNCONSTRAINEDSIZE', file layout/base/nsLayoutUtils.cpp, line 4727

This warning [1], introduced in bug 525100, shows up in the following test suites:

> mozilla-central_ubuntu64_vm-debug_test-mochitest-e10s-5-bm52-tests1-linux64-build2.txt:432
> mozilla-central_ubuntu64_vm-debug_test-mochitest-5-bm115-tests1-linux64-build29.txt:432
> mozilla-central_ubuntu64_vm-debug_test-reftest-e10s-2-bm123-tests1-linux64-build2.txt:48
> mozilla-central_ubuntu64_vm-debug_test-reftest-4-bm121-tests1-linux64-build1.txt:48
> mozilla-central_ubuntu64_vm-debug_test-reftest-e10s-1-bm68-tests1-linux64-build0.txt:40
> mozilla-central_ubuntu64_vm-debug_test-reftest-2-bm53-tests1-linux64-build27.txt:40
> mozilla-central_ubuntu64_vm-debug_test-crashtest-bm53-tests1-linux64-build28.txt:14

It shows up in 12 tests. A few of the most prevalent:

> 864 - layout/style/test/test_logical_properties.html
> 40 - file:///builds/slave/test/build/tests/reftest/tests/layout/reftests/floats/orthogonal-floats-1b.html
> 40 - file:///builds/slave/test/build/tests/reftest/tests/layout/reftests/floats/orthogonal-floats-1a.html
> 16 - file:///builds/slave/test/build/tests/reftest/tests/layout/reftests/writing-mode/tables/fixed-table-layout-009-vlr.html
> 16 - file:///builds/slave/test/build/tests/reftest/tests/layout/reftests/writing-mode/1172774-percent-padding-4.html
> 16 - file:///builds/slave/test/build/tests/reftest/tests/layout/reftests/writing-mode/1172774-percent-padding-3.html
> 16 - file:///builds/slave/test/build/tests/reftest/tests/layout/reftests/writing-mode/1172774-percent-margin-4.html
> 16 - file:///builds/slave/test/build/tests/reftest/tests/layout/reftests/writing-mode/1172774-percent-margin-3.html
> 9 - file:///builds/slave/test/build/tests/reftest/tests/layout/base/crashtests/265986-1.html
> 8 - file:///builds/slave/test/build/tests/reftest/tests/layout/reftests/writing-mode/tables/fixed-table-layout-009-vrl.html

[1] https://hg.mozilla.org/mozilla-central/annotate/49683d4e9ebd/layout/base/nsLayoutUtils.cpp#l4727

	Eric Rahm [:erahm] Reporter
	Comment 1 • 9 years ago

We're seeing this in b2g tests as well.

	Eric Rahm [:erahm] Reporter
	Comment 2 • 9 years ago

Considering this is only showing up in 12 tests I'm inclined to think it's indicating a real issue.

Cameron, we're seeing a lot of:

> WARNING: have unconstrained inline-size; this should only result from very large sizes, not attempts at intrinsic inline-size calculation: 'aContainingBlockISize != NS_UNCONSTRAINEDSIZE'

generated by |layout/style/test/test_logical_properties.html| [1], does this seem legit or is it indicating an error we need to fix (I didn't see any obviously large sizes)?

[1] https://hg.mozilla.org/mozilla-central/annotate/e7434cafdf2f/layout/style/test/test_logical_properties.html

	David Baron :dbaron:
	Comment 3 • 9 years ago

Might be more useful to ask Jonathan.

	Jonathan Kew [:jfkthame] Assignee
	Comment 4 • 9 years ago

The warnings here occur because the container of the test <div> has unconstrained height when we're reflowing, which translates to unconstrained containing-block isize when the writing mode is vertical. We can suppress them by ensuring the <div> has a container with specified dimensions; this shouldn't affect what the tests here are actually checking.

	Cameron McCormack (:heycam)
	Comment 5 • 9 years ago

Comment on attachment 8638956 [details] [diff] [review]
Ensure the containing block has a constrained size during logical-properties tests, to suppress unconstrained-isize warnings

Review of attachment 8638956 [details] [diff] [review]:
-----------------------------------------------------------------

Thanks Jonathan.

	Jonathan Kew [:jfkthame] Assignee
	Comment 6 • 9 years ago

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/3741aa98b8918c7d18116cd42f9f5c82d6e13048
changeset:  3741aa98b8918c7d18116cd42f9f5c82d6e13048
user:       Jonathan Kew <jkew@mozilla.com>
date:       Mon Jul 27 08:06:46 2015 +0100
description:
Bug 1184295 - Ensure the containing block has a constrained size during logical-properties tests, to suppress unconstrained-isize warnings. r=heycam

	Ryan VanderMeulen [:RyanVM]
	Comment 7 • 9 years ago

https://hg.mozilla.org/mozilla-central/rev/3741aa98b891