window.open() creates window that passes zooming to parent window

UNCONFIRMED
Unassigned

Status

Firefox OS
Gaia::System::Window Mgmt
UNCONFIRMED
3 years ago
2 years ago

People

(Reporter: hobinjk, Unassigned)

Tracking

unspecified
ARM
Gonk (Firefox OS)

Firefox Tracking Flags

(b2g-v2.2 unaffected, b2g-master unaffected)

Details

Attachments

(1 attachment)

(Reporter)

Description

3 years ago
Created attachment 8634412 [details]
Minimal reproducing app (tap text to open window)

User Agent: Mozilla/5.0 (X11; Linux x86_64; rv:42.0) Gecko/20100101 Firefox/42.0
Build ID: 20150713030204

Steps to reproduce:

1. Open popup window using window.open in an app with `<meta  name="viewport" content="width=device-width, initial-scale=1">`
2. Attempt to zoom in on new window


Actual results:

Zooming has no effect or animation. Instead the original window is zoomed in on.


Expected results:

The opened window should zoom.
Can we see this behavior somewhere in our gaia code?
Flags: needinfo?(hobinjk)
(Reporter)

Comment 2

3 years ago
(In reply to Gregor Wagner [:gwagner] from comment #1)
> Can we see this behavior somewhere in our gaia code?

I don't know. I put this in the window management component because it is related to the behavior of two windows. If there is a better componenent for this please move it there.
Flags: needinfo?(hobinjk)
Window management component is good enough. I was wondering if this bug shows up somewhere in gaia so QA can find out if this is a regression or not.
(Reporter)

Comment 4

3 years ago
I found that it also occurs if you download a PDF and open it from the notification that pops up (for a while I just thought that there was no pinch-to-zoom in the PDF viewer). Is this good enough to get QA involved?
Flags: needinfo?(anygregor)
(Reporter)

Comment 5

3 years ago
(In reply to James Hobin [:hobinjk] from comment #4)
> I found that it also occurs if you download a PDF and open it from the
> notification that pops up (for a while I just thought that there was no
> pinch-to-zoom in the PDF viewer). Is this good enough to get QA involved?

Correction: Download a PDF in Email
(Reporter)

Comment 6

3 years ago
(In reply to James Hobin [:hobinjk] from comment #5)
> (In reply to James Hobin [:hobinjk] from comment #4)
> > I found that it also occurs if you download a PDF and open it from the
> > notification that pops up (for a while I just thought that there was no
> > pinch-to-zoom in the PDF viewer). Is this good enough to get QA involved?
> 
> Correction: Download a PDF in Email

Correction to the correction: I can only reproduce the zooming not happening part of the bug, Email prohibits scrolling. The other part (that the parent zooms in), requires a zoomable parent, which Email is not.
Can we get a branch check?
Flags: needinfo?(anygregor)
Keywords: qawanted
I don't think STR at comment 4 makes sense. The built-in PDF viewer can't pinch-zoom, period. It doesn't matter how you open it. That's why there's zoom in/out buttons on the PDF viewer. Tested with 1040 form on IRS.

I installed the attached app at comment 0 via webIDE and tested it out. I was able to pinch zoom on the popup window and zoom state was not passed to parent window.

Issue does NOT reproduce on:
Device: Aries 2.5
BuildID: 20150924111215
Gaia: 4bb17b24620818cbda0ba0c0d69e0ce3f914e1b7
Gecko: 001942e4617b2324bfa6cdfb1155581cbc3f0cc4
Gonk: 2916e2368074b5383c80bf5a0fba3fc83ba310bd
Version: 44.0a1 (2.5) 
Firmware Version: D5803_23.1.A.1.28_NCB.ftf
User Agent: Mozilla/5.0 (Mobile; rv:44.0) Gecko/44.0 Firefox/44.0

Device: Flame 2.5
BuildID: 20150924030228
Gaia: 4bb17b24620818cbda0ba0c0d69e0ce3f914e1b7
Gecko: bf2bc1aa78c0b72d9b6b13f7a8c6ae61c60a51dc
Gonk: c4779d6da0f85894b1f78f0351b43f2949e8decd
Version: 44.0a1 (2.5) 
Firmware Version: v18D
User Agent: Mozilla/5.0 (Mobile; rv:44.0) Gecko/44.0 Firefox/44.0

Device: Flame 2.2
BuildID: 20150924032502
Gaia: 5dd95cfb9f1d6501ce0e34414596ef3dd9c2f583
Gecko: 65ddad73ad6b
Gonk: bd9cb3af2a0354577a6903917bc826489050b40d
Version: 37.0 (2.2) 
Firmware Version: v18D
User Agent: Mozilla/5.0 (Mobile; rv:37.0) Gecko/37.0 Firefox/37.0
QA Whiteboard: [QAnalyst-Triage?]
status-b2g-v2.2: --- → unaffected
status-b2g-master: --- → unaffected
Flags: needinfo?(jmercado)
Keywords: qawanted
QA Whiteboard: [QAnalyst-Triage?] → [QAnalyst-Triage+]
Flags: needinfo?(jmercado)
You need to log in before you can comment on or make changes to this bug.