566610 - [OOPP]Can not start scroll page by mouse wheel when mouse cursor is over a flash object

Status: RESOLVED FIXED

(Core Graveyard :: Plug-ins, defect)

Description

[Alice0775 White]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 6.1; WOW64; en-US; rv:1.9.3a5pre) Gecko/20100518 Minefield/3.7a5pre ID:20100518035958
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 6.1; WOW64; en-US; rv:1.9.3a5pre) Gecko/20100518 Minefield/3.7a5pre ID:20100518035958

If enable OOPP(by default), I can not start scroll page by mouse wheel when
mouse cursor is over a flash video.

If disabled OOPP(set dom.ipc.plugins.enabled to false),then I can start scroll
page by mouse wheel even if mouse cursor is over a flash video.

This problem happens also Mozilla/5.0 (Windows; U; Windows NT 6.1; en-US; rv:1.9.2.5pre) Gecko/20100518 Namoroka/3.6.5pre ID:20100518042225.

Because regression range is different, this is not dupe of Bug 552163  - [OOPP]Can not start scroll page by mouse wheel when mouse cursor is over a flash video.

Reproducible: Always

Steps to Reproduce:
1. Start Minefield/Namoroka with new profile
2. Open URL ( http://www.google.com/finance?q=INDEXDJX:.DJI )
     ( Do not click flash object to focus it.)
3. Hover mouse cursor over a flash video.
4. Scroll by mouse wheel

Actual Results:
 When mouse over flash object, I can not start scroll by mouse wheel.

Expected Results:
 When mouse over flash object, I can start scroll by mouse wheel.



Regression window for 1.9.3

Works:
http://hg.mozilla.org/mozilla-central/rev/0841ec7eec87
Mozilla/5.0 (Windows; U; Windows NT 6.1; WOW64; en-US; rv:1.9.3a5pre) Gecko/20100511 Minefield/3.7a5pre ID:20100511082156

Fails:
http://hg.mozilla.org/mozilla-central/rev/f884c59a5fbf
Mozilla/5.0 (Windows; U; Windows NT 6.1; WOW64; en-US; rv:1.9.3a5pre) Gecko/20100511 Minefield/3.7a5pre ID:20100511090755

Pushlog:
http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=0841ec7eec87&tochange=f884c59a5fbf


Regression window for 1.9.2

Works:
http://hg.mozilla.org/releases/mozilla-1.9.2/rev/e15bd0e4e549
Mozilla/5.0 (Windows; U; Windows NT 6.1; en-US; rv:1.9.2.5pre) Gecko/20100511 Namoroka/3.6.5pre ID:20100511042448

Fails:
http://hg.mozilla.org/releases/mozilla-1.9.2/rev/aface486f2e4
Mozilla/5.0 (Windows; U; Windows NT 6.1; en-US; rv:1.9.2.5pre) Gecko/20100512 Namoroka/3.6.5pre ID:20100512042059

Pushlog:
http://hg.mozilla.org/releases/mozilla-1.9.2/pushloghtml?fromchange=e15bd0e4e549&tochange=aface486f2e4

Candidate bug:
Bug 563377  - Fullscreen video stops responding to keyboard/mouse input, refuses to close itself when switched away

Comment 1

[Benjamin Smedberg]

recent regression then, yes should block :-(

Comment 2

[Alice0775 White]

This issue happens also youtube.
http://www.youtube.com/watch?v=0-XU3vWjDsc

Comment 3

[Jim Mathies [:jimm]]

I guess the hook solution will have to become just a killfocus fix, and the rest of these events will have to go back to the subclass.

Comment 4

[Jim Mathies [:jimm]]

Attachment: tame the nuclear option v.1

We can still use the hook for specific cases like kill frocus, but moving all the "working" sub-class fixes over caused new problems.

Comment 5

[Jim Mathies [:jimm]]

try builds are in the works to confirm.

Comment 6

[LegNeato]

I'm not so sure this is a blocker, as we would ship with the issue if we really needed to. We want it if the fix works and the schedule matches up though.

Comment 7

[Jim Mathies [:jimm]]

confirmed locally, this was easy to reproduce. the patch addresses it.

Comment 8

[Jim Mathies [:jimm]]

http://hg.mozilla.org/mozilla-central/rev/d02a980b0d83

Comment 9

[Jim Mathies [:jimm]]

Attachment: landed patch

- addressed missing wm_mousehwheel build error.

Comment 10

[LegNeato]

Comment on attachment 446002 [details] [diff] [review]
landed patch

a=LegNeato for 1.9.2.4. Please land on both mozilla-1.9.2 default and
GECKO1924_20100413_RELBRANCH

Comment 11

[Jim Mathies [:jimm]]

http://hg.mozilla.org/releases/mozilla-1.9.2/rev/9f5bd81a253b
http://hg.mozilla.org/releases/mozilla-1.9.2/rev/daadd4051a15

Comment 12

[Black_Ps`]

This is not fixed, http://www.youtube.com/watch?v=qrO4YZeyl0I

Comment 13

[Alice0775 White]

(In reply to comment #12)
> This is not fixed, http://www.youtube.com/watch?v=qrO4YZeyl0I
Regression window is different, this maybe different bug.

Comment 14

[Black_Ps`]

And do i need to post another bug report?

Comment 15

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)]

It should be bug 483136.

Comment 16

[Black_Ps`]

Yes sorry you are right.