740721 - pinch and zoom don't work on the Windows with touch input displays

Status: RESOLVED FIXED

(Firefox :: General, defect)

Description

[Jim Mathies [:jimm]]

This could be in widget or something higher up. We added touch input support back when win7 was released, this seems to have broken.

STR:

1) open a web page on the desktop
2) pinch or zoom the page

result: nothing happen.

Comment 2

[Jim Mathies [:jimm]]

Attachment: fix

Comment 3

[Jim Mathies [:jimm]]

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

Reverting changes in bug 613909 for Windows.

Comment 4

[Dão Gottwald [:dao]]

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

Please use ifdef instead of ifndef, swapping these two sections. Can you also add a comment explaining why this varies across OSes?

Comment 5

[Jim Mathies [:jimm]]

Attachment: patch

Updated.

Comment 6

[Jim Mathies [:jimm]]

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

Comment 7

[Ed Morley [:emorley]]

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

Comment 8

[Jim Mathies [:jimm]]

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

This change enables functionality we mistakenly disabled about a year ago for windows touch input displays. Unfortunately once disabled, it stays disabled unless the user manually edits config settings.

Microsoft will have a fresh preview release of win8 due out in the next few weeks, so I would like to get this out to beta in the hope that this functionality will be available for fresh installs on the new os rev.

Comment 9

[Alex Keybl [:akeybl]]

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

[Triage Comment]
This is too late for FF13, but appears to be low risk enough to approve for Aurora 14.

Comment 10

[Jim Mathies [:jimm]]

http://hg.mozilla.org/releases/mozilla-aurora/rev/b61d1ddc8550