http://trac.webkit.org/changeset/188663 broke the !ENABLE(CSS_REGIONS) build, because JSDOMNamedFlowCollection class is defined inside guard. ( http://trac.webkit.org/browser/trunk/Source/WebCore/dom/DOMNamedFlowCollection.idl#L1 ) build log: ../../Source/WebCore/bindings/js/JSDOMNamedFlowCollectionCustom.cpp:37:6: error: 'JSDOMNamedFlowCollection' has not been declared ../../Source/WebCore/bindings/js/JSDOMNamedFlowCollectionCustom.cpp: In function 'bool WebCore::nameGetter(JSC::ExecState*, JSC::PropertyName, JSC::JSValue&)': ../../Source/WebCore/bindings/js/JSDOMNamedFlowCollectionCustom.cpp:39:22: error: 'impl' was not declared in this scope ../../Source/WebCore/bindings/js/JSDOMNamedFlowCollectionCustom.cpp:43:37: error: 'globalObject' was not declared in this scope ninja: build stopped: subcommand failed. There is no need for JSDOMNamedFlowCollection::nameGetter() outside the guard, so let's guard the whole Source/WebCore/bindings/js/JSDOMNamedFlowCollectionCustom.cpp
Created attachment 259835 [details] Patch
Comment on attachment 259835 [details] Patch r=me, thanks!
Comment on attachment 259835 [details] Patch Clearing flags on attachment: 259835 Committed r188921: <http://trac.webkit.org/changeset/188921>
All reviewed patches have been landed. Closing bug.