613731 - nsEventStates.h uses undefined NS_ERROR

Status: RESOLVED FIXED

(Core :: General, defect)

Description

[Philipp Wagner [:imphil]]

Attachment: patch v1

When building xforms in debug mode I get 

c++ -o nsXFormsRangeElement.o -c -I../../dist/stl_wrappers -I../../dist/system_wrappers -include /home/philipp/src/mozilla-central/config/gcc_hidden.h -DXPI_NAME=xforms -DOSTYPE=\"Linux2.6.34.7-0\" -DOSARCH=Linux  -I/home/philipp/src/mozilla-central/extensions/xforms -I. -I../../dist/include -I../../dist/include/nsprpub  -I/home/philipp/src/mozilla-central/obj-debug/dist/include/nspr -I/home/philipp/src/mozilla-central/obj-debug/dist/include/nss       -fPIC  -fno-rtti -fno-exceptions -Wall -Wpointer-arith -Woverloaded-virtual -Wsynth -Wno-ctor-dtor-privacy -Wno-non-virtual-dtor -Wcast-align -Wno-invalid-offsetof -Wno-variadic-macros -Werror=return-type -pedantic -Wno-long-long -fno-strict-aliasing -fshort-wchar -pthread -pipe  -DDEBUG -D_DEBUG -DTRACING -g   -DMOZILLA_CLIENT -include ../../mozilla-config.h -MD -MF .deps/nsXFormsRangeElement.pp /home/philipp/src/mozilla-central/extensions/xforms/nsXFormsRangeElement.cpp
In file included from /home/philipp/src/mozilla-central/extensions/xforms/nsXFormsNodeState.h:44:0,
                 from /home/philipp/src/mozilla-central/extensions/xforms/nsXFormsNodeState.cpp:40:
../../dist/include/nsEventStates.h: In member function ‘bool nsEventStates::HasState(nsEventStates) const’:
../../dist/include/nsEventStates.h:142:69: error: ‘NS_ERROR’ was not declared in this scope



It looks like nsEventStates.h is missing a #include nsDebug.h, a patch is attached.

Comment 1

[Mounir Lamouri (:mounir)]

Comment on attachment 492059 [details] [diff] [review]
patch v1

r=me

I guess you can cancel Boris' review and push with mine. That's just adding one include.
You can probably push that with a=bustage.

Comment 2

[Philipp Wagner [:imphil]]

Attachment: for checkin

volkmar, thanks for the review. could you check that in for me, I don't have commit access to m-c? Patch for checkin is attached.

Comment 3

[Mounir Lamouri (:mounir)]

I will do that tomorrow or on Monday. Thank you for this fix :)

Comment 4

[Mounir Lamouri (:mounir)]

Pushed:
http://hg.mozilla.org/mozilla-central/rev/4335fe1a4f95