791037 - Build issue from "IsInlineOutside" renaming, when MOZ_FLEXBOX is #defined

Status: RESOLVED FIXED

(Core :: Layout, defect)

Description

[Daniel Holbert [:dholbert]]

A month ago, nsStyleDisplay::IsInlineOutside changed to be called "IsInlineOutsideStyle":
  https://hg.mozilla.org/mozilla-central/rev/d7749fadb594#l13.41
...and a new IsInlineOutside method was created that requires an argument:
  https://hg.mozilla.org/mozilla-central/rev/d7749fadb594#l13.67

This broke some ifdeffed-off flexbox code (which is only noticeable when you #define MOZ_FLEXBOX).

I've had a fix for this in my tree for a while, but hadn't posted it on a bug yet.

The error is:
{
../../../mozilla/layout/base/nsCSSFrameConstructor.cpp: In member function ‘void nsCSSFrameConstructor::CreateNeededAnonFlexItems(nsFrameConstructorState&, nsCSSFrameConstructor::FrameConstructionItemList&, nsIFrame*)’:
../../../mozilla/layout/base/nsCSSFrameConstructor.cpp:9562:66: error: no matching function for call to ‘nsStyleDisplay::IsInlineOutside() const’
../../../mozilla/layout/base/nsCSSFrameConstructor.cpp:9562:66: note: candidate is:
In file included from ../../../mozilla/layout/base/../generic/nsIFrame.h:3204:0,
                 from ../../../mozilla/layout/base/nsGenConList.h:11,
                 from ../../../mozilla/layout/base/nsQuoteList.h:11,
                 from ../../../mozilla/layout/base/nsCSSFrameConstructor.h:18,
                 from ../../../mozilla/layout/base/nsCSSFrameConstructor.cpp:14:
../../../mozilla/layout/base/../style/nsStyleStructInlines.h:92:1: note: bool nsStyleDisplay::IsInlineOutside(const nsIFrame*) const
../../../mozilla/layout/base/../style/nsStyleStructInlines.h:92:1: note:   candidate expects 1 argument, 0 provided
}

For reference, the cset that broke this was:
  https://hg.mozilla.org/mozilla-central/rev/d7749fadb594#l13.41

Fix is just s/IsInlineOutside/IsInlineOutsideStyle/ -- attaching it in a minute.

Comment 1

[Daniel Holbert [:dholbert]]

Attachment: fix

Comment 2

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Comment on attachment 660909 [details] [diff] [review]
fix

r=dbaron

Comment 3

[Daniel Holbert [:dholbert]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/b704d314748d

Comment 4

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/b704d314748d