It is very difficult to resize Picture-in-Picture window by mouse dragging

RESOLVED FIXED in Firefox 68

Status

()

defect
P2
normal
RESOLVED FIXED
2 months ago
3 days ago

People

(Reporter: alice0775, Assigned: mconley, NeedInfo)

Tracking

(Blocks 1 bug, Regression, {regression})

68 Branch
mozilla68
x86_64
Windows 10
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox-esr60 unaffected, firefox66 unaffected, firefox67 unaffected, firefox68 fixed)

Details

Attachments

(1 attachment)

Reporter

Description

2 months ago

Sometimes, the mouse pointer will change to resizer cursor, and I can resize the PiP window by mouse pointer.
But, sometime, it does not. The cursor would not change.

Almost impossible.

Steps To Reproduce:

  1. Enable Picture in Picture media.videocontrols.picture-in-picture.enabled=true if not
  2. Play and Video
  3. Enter Picture in Picture mode
  4. Try to resize the Pip window by mouse

Actual Results:
The mouse pointer would not be changed to resizer cursor on the PiP window border.
Sometimes, the mouse pointer is changed to resizer cursor. But the pointer will be immediately changed to normal cursor.
So, I cannot resize the window.

Expected Results:
At least, A mouse pointer should react like main window.

Regression window:
https://hg.mozilla.org/integration/autoland/pushloghtml?fromchange=32552726554321d35a8c18224343fe73ece324c2&tochange=0930049c0ca984c909d11ba2853ef6d96e43e242

Regressed by: 0930049c0ca984c909d11ba2853ef6d96e43e242 Mike Conley — Bug 1534389 - Send normal mouse events when cursor is over a draggable region on Windows. r=jmathies

:mconley,
Your patch seems to cause the regression. Can you please look into this?

Flags: needinfo?(mconley)
Assignee

Comment 1

2 months ago

Yes, I've added it to my queue - thank you for filing.

Blocks: 1527926
Flags: needinfo?(mconley)
Priority: -- → P2

This is made easier by making the window-dragging region inset by 5px on each side.

Depends on D30100

Comment 3

Last month
Pushed by mconley@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/93e59a14b907
Make it easier to resize the Picture-in-Picture player window. r=dthayer

Comment 4

Last month
bugherder
Status: NEW → RESOLVED
Closed: Last month
Resolution: --- → FIXED
Target Milestone: --- → mozilla68

Comment 5

Last month

I have the same problem with resizing the whole browser window.
Is this related and fixed as well?

Reporter

Updated

Last month
Blocks: 1549972
Reporter

Comment 6

Last month
Flags: qe-verify+

Removing the "qe-verify+ tag, considering the newly added bugs.

Also, Considering that the fix did not fix the reporter's issue, should it not be backed out?
This also applies to the clone of this bug: bug 1549972.

Flags: qe-verify+ → needinfo?(mconley)
You need to log in before you can comment on or make changes to this bug.