1034070 - APZC not fully supported in overlayed child windows

Status: RESOLVED WORKSFORME

(Firefox OS Graveyard :: Gaia::System::Window Mgmt, defect)

Description

[Ben Francis [:benfrancis]]

STR:

* Email yourself a link to  http://www.cycleworld.com/2014/06/25/moto-guzzi-handbuilt-custom-motorcycle-by-millepercento-scighera-1200-first-look-review-and-photos/?cmpid=enews062614&spPodID=020&spMailingID=21047916&spUserID=NzQ5OTQzOTExODUS1&spJobID=341460242&spReportId=MzQxNDYwMjQyS0
* Follow the link from the email app to open it in an overlayed child windows
* Try to pinch to zoom

Expected:
* Can pinch to zoom

Actual:
* Can't pinch to zoom

Not sure if this is something the window manager isn't doing or whether it's a limitation in Gecko?

Comment 1

[Chris Lord [:cwiiis]]

Zooming is limited to the root frame of a browser element - I don't know how overlayed child windows are handled, but unless they're in an out-of-process browser iframe, they will not be zoomable - this is a Gecko limitation, unless things have changed recently.

cc'ing kats in case I'm spreading misinformation :)

Comment 2

[Jason Smith [:jsmith]]

QA Wanted for branch checks.

Comment 3

[Kartikaya Gupta (email:[e-mail])]

Zooming is limited to the root scrollable frame in a given process. What are the overlayed child windows?

Comment 4

[(inactive after 6/18) Alive Kuo [:alive]]

Is email using window.open? We are placing the child window inside system app,
the hierarchy is:
<iframe of system>
  <div>
    <iframe of email>
    <div>
      <iframe of email's opened window>
    </div>
  </div>
<iframe>

Comment 5

[Kartikaya Gupta (email:[e-mail])]

That sounds wrong. <iframe of email> spawns the email process, so the iframe inside that will either be in the email app or in a new process depending on what attributes are set on the iframe element.

Regardless, the system app also doesn't have APZC enabled at all. Bug 950934 is tracking that issue.

Comment 6

[Pi Wei Cheng [:piwei] (inactive)]

I'm unable to reproduce this bug on Flame 2.1 master, Flame 2.0 Aurora, Flame 1.4, and Buri 2.0 Aurora.

Observed: Pinch zoom can be done correctly in a browser window opened by an email app link.

Device: Flame
Build ID: 20140721055837
Gaia: Unknown
Gecko: 0dc711216018
Version: 33.0a1 (Master)
Firmware Version: v122
User Agent: Mozilla/5.0 (Mobile; rv:33.0) Gecko/33.0 Firefox/33.0

Device: Flame
Build ID: 20140720184018
Gaia: 8cb1a949f2e9650bb2c5598e78a6f24a58bbaf97
Gecko: 4bd4b0ae7bbe
Version: 32.0a2 (2.0)
Firmware Version: v122
User Agent: Mozilla/5.0 (Mobile; rv:32.0) Gecko/32.0 Firefox/32.0

Device: Flame
Build ID: 20140721000201
Gaia: 621d152f89347c79619aa909ad62cc2ac9d3ab5b
Gecko: 83b7be7fb33f
Version: 30.0 (1.4)
Firmware Version: v122
User Agent: Mozilla/5.0 (Mobile; rv:30.0) Gecko/30.0 Firefox/30.0

Device: Buri
Build ID: 20140721082721
Gaia: b9d19011123487009c80d1200937652d58c434a0
Gecko: d69cd84b6824
Version: 32.0a2 (2.0)
Firmware Version: v1.2device.cfg
User Agent: Mozilla/5.0 (Mobile; rv:32.0) Gecko/32.0 Firefox/32.0

Comment 7

[Joshua Mitchell (Inactive)]

Ben - QA-Wanted is unable to repro this issue, would you mind checking in the latest a verify this is now working?

Comment 8

[Ben Francis [:benfrancis]]

I don't think the STR apply any more because the email app now opens a browser window instead of a popup window when following hyperlinks.

I have tested with my hyperlink test app installed http://people.mozilla.org/~bfrancis/hyperlink/ and I can pinch to zoom google.com in a popup window. That may indicate this is fixed although this is a different web site.

Comment 9

[Joshua Mitchell (Inactive)]

So this seems fixed by re-design - which is probably closest to WFM