Note: There are a few cases of duplicates in user autocompletion which are being worked on.

nsResizerFrame may deanchor panels

RESOLVED FIXED in mozilla8

Status

()

Core
Widget
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: mak, Assigned: mak)

Tracking

(Blocks: 1 bug)

Trunk
mozilla8
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

1.80 KB, patch
Neil Deakin (not available until Aug 9)
: review+
Details | Diff | Splinter Review
(Assignee)

Description

6 years ago
Since it invokes nsMenuPopupFrame::moveTo that overrides mScreenXPos and mScreenYPos even if they are -1 (thus anchored).
(Assignee)

Comment 1

6 years ago
Created attachment 549059 [details] [diff] [review]
patch v1.0
Attachment #549059 - Flags: review?(enndeakin)
(Assignee)

Updated

6 years ago
Blocks: 652115
(Assignee)

Comment 2

6 years ago
Created attachment 549060 [details] [diff] [review]
patch v1.1

the comment is now useless as well
Attachment #549059 - Attachment is obsolete: true
Attachment #549059 - Flags: review?(enndeakin)
Attachment #549060 - Flags: review?(enndeakin)
Attachment #549060 - Flags: review?(enndeakin) → review+
(Assignee)

Comment 3

6 years ago
http://hg.mozilla.org/integration/mozilla-inbound/rev/f839abc8f150
Whiteboard: [inbound]
http://hg.mozilla.org/mozilla-central/rev/f839abc8f150
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Whiteboard: [inbound]
Target Milestone: --- → mozilla8
is it possible that this is causing ForecastFox to crash the browser when hovering over the icons when ForecastFox is placed in the application bar ? 

Crashing did not occur until I started using hourly build:
http://hg.mozilla.org/mozilla-central/rev/8fb752f5e1fa

Prior hourly build based on cset: 
http://hg.mozilla.org/mozilla-central/rev/f5f1e3822540 does not crash. 

Being as its an hourly build the crash-reports will be useless.
Forgot to mention in comment #5 above, I'm using m-c win32 builds and ForecastFox and Flashblock are the only addons I use.
(Assignee)

Comment 7

6 years ago
(In reply to comment #5)
> is it possible that this is causing ForecastFox to crash the browser when
> hovering over the icons when ForecastFox is placed in the application bar ? 

Don't think so, the changes don't seem to be able to cause any new crash that could not happen before. If you can get a stack trace with next nightly may be useful.
(In reply to comment #7)
> (In reply to comment #5)
> > is it possible that this is causing ForecastFox to crash the browser when
> > hovering over the icons when ForecastFox is placed in the application bar ? 
> 
> Don't think so, the changes don't seem to be able to cause any new crash
> that could not happen before. If you can get a stack trace with next nightly
> may be useful.

Marco, thanks for the reply.  The issue with ForecastFox and some sites was caused by https://bugzilla.mozilla.org/show_bug.cgi?id=675437 which has a fix ready for review.

Sorry for barking up the wrong tree.
You need to log in before you can comment on or make changes to this bug.