Closed
Bug 1244582
Opened 9 years ago
Closed 9 years ago
[e10s] Ctrl+mouse wheel to change zoom level doesn't work if chrome input fields are focused (crashes tab)
Categories
(Core :: DOM: Core & HTML, defect)
Core
DOM: Core & HTML
Tracking
()
RESOLVED
FIXED
mozilla47
Tracking | Status | |
---|---|---|
firefox47 | --- | fixed |
People
(Reporter: arni2033, Assigned: khuey)
References
()
Details
(Keywords: crash, regression, reproducible)
Attachments
(1 file)
(deleted),
patch
|
smaug
:
review+
|
Details | Diff | Splinter Review |
>>> My Info: Win7_64, Nightly 46, 32bit, ID 20160129030206
STR:
1. Open http://example.org/
2. Press Ctrl+F to open and focus Findbar
3. Place mouse pointer over the page content, hold Ctrl and rotate mouse wheel up and down
AR: Tab crashes
ER: Tab should change zoom level
I think that crash is a recent regression. It doesn't happen on Dev.Edition 45
I was going to file this as simply Ctrl+Wheel not working, but now it also causes the crash...
Comment hidden (obsolete) |
Sorry, info in comment 0 is incorrect. I'm using: Win7_64, Nightly 46, 32bit, ID 20160131030347
This is regression between 47.0a1 (2016-01-30) and 47.0a1 (2016-01-31).
Mozregression GUI can't handle this =(
Keywords: regression,
regressionwindow-wanted
Comment 3•9 years ago
|
||
Could you post a link to a crash report from about:crashes? Seeing the crash stack might allow us to identify the problem faster than a regression window.
Flags: needinfo?(arni2033)
Flags: needinfo?(arni2033)
Comment 5•9 years ago
|
||
Thanks! This looks likely to be a regression from bug 1241764.
Blocks: 1241764
Component: Panning and Zooming → DOM
Flags: needinfo?(khuey)
Keywords: regressionwindow-wanted
Assignee | ||
Comment 6•9 years ago
|
||
Yeah, looks like I effectively removed a null check.
Assignee: nobody → khuey
Flags: needinfo?(khuey)
Assignee | ||
Comment 7•9 years ago
|
||
Attachment #8714189 -
Flags: review?(bugs)
I was going to file separate bug on Ctrl+Wheel not changing the zoom level, unless
you will leave this bug open to track that issue after fixing the crash. So what is the plan here?
Note: zero relevant bugs found for "e10s zoom level"
Comment 9•9 years ago
|
||
(In reply to arni2033 from comment #8)
> I was going to file separate bug on Ctrl+Wheel not changing the zoom level,
> unless
> you will leave this bug open to track that issue after fixing the crash. So
> what is the plan here?
Please file a new bug for that - thanks!
Updated•9 years ago
|
Attachment #8714189 -
Flags: review?(bugs) → review+
Comment 10•9 years ago
|
||
Comment 11•9 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla47
Updated•6 years ago
|
Component: DOM → DOM: Core & HTML
You need to log in
before you can comment on or make changes to this bug.
Description
•