755208 - Create compilation flag for mobile HTML5 forms

Status: RESOLVED WONTFIX

(Core :: DOM: Core & HTML, defect)

Description

[Raphael Catolino (:raphc)]

User Agent: Mozilla/5.0 (X11; Linux x86_64; rv:15.0) Gecko/15.0 Firefox/15.0a1
Build ID: 20120514030523




Expected results:

Implement some HTML5 forms for mobile only

Comment 1

[Raphael Catolino (:raphc)]

Attachment: patch

Comment 2

[Mounir Lamouri (:mounir)]

Glandium, for information this compilation flag will be used to have some new form elements/input types released only on mobile (because we will not have a UI on desktop).
We could change that later for a run-time pref but I think a pref would be dirtier because we have to initialize some static arrays. So, for the moment, I prefer to go with a compilation flag and see if a pref would be useful and cleaner.

Comment 3

[Mike Hommey [:glandium]]

Comment on attachment 624337 [details] [diff] [review]
patch

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

::: configure.in
@@ +7667,5 @@
> +                               on mobile plarforms.],
> +    EXPERIMENTAL_FORMS=1,
> +    EXPERIMENTAL_FORMS= )
> +MOBILE_PLATFORM = `expr "$MOZ_BUILD_APP" : "\(mobile\)"`
> +if test "$MOBILE_PLATFORM" = "mobile" -o "$MOZ_BUILD_APP" = "b2g"; then

That's a very unconventional idiom. Please use something like:
case "$MOZ_BUILD_APP" in
mobile/*|b2g)
    EXPERIMENTAL_FORMS=1
    ;;
esac

instead.

Is there really an interest in having a configure argument for this? It would seem me that no, and that if someone still want to enable it to work on the desktop implementations, they could either broaden the test here, or export EXPERIMENTAL_FORMS=1 in thir mozconfig.

Comment 4

[Mounir Lamouri (:mounir)]

The main idea of this flag is to allow devs to enable this on desktop. If you prefer exporting EXPERIMENTAL_FORMS instead of the compilation flag, I guess I wouldn't mind.

Raphael, how does that sound to you?

Comment 5

[Raphael Catolino (:raphc)]

Attachment: patch 2

Comment 6

[Mounir Lamouri (:mounir)]

The compile flag solution has been abandoned in favour of the pref one.