703889 - Remove |ifndef DISABLE_XFORMS_HOOKS| since it's never defined

Status: RESOLVED FIXED

(Firefox Build System :: General, defect)

Description

[Ed Morley [:emorley]]

Bug 337690 added a few ifndef DISABLE_XFORMS_HOOKS conditionals (http://mxr.mozilla.org/mozilla-central/search?string=DISABLE_XFORMS_HOOKS), but it's never defined (believe it was original intended for embedders to be able to disable, but well...we know the story for embedding right now). 

By removing it, it will mean I can remove one of the extra conditionals that bug 696498 is otherwise going to have to add to toolkit-makefiles.sh to keep parity with accessible/src/Makefile.in.

Comment 1

[Ed Morley [:emorley]]

Attachment: Patch v1

Comment 2

[Ed Morley [:emorley]]

Attachment: Patch v2

Slight correction to content/base/public/Makefile.in

Comment 3

[Ed Morley [:emorley]]

https://tbpl.mozilla.org/?tree=Try&rev=51eba91e1c12

Comment 4

[(not currently active) Ted Mielczarek]

Comment on attachment 575685 [details] [diff] [review]
Patch v2

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

Interesting. I wonder where this got removed?

Comment 5

[Ed Morley [:emorley]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/89db63eda545

Comment 6

[Marco Bonardo [:mak] (Away Apr 25 - May 5)]

https://hg.mozilla.org/mozilla-central/rev/89db63eda545