Closed Bug 539787 Opened 10 years ago Closed 10 years ago

[regression] center() throws sometimes when switching tabs

Categories

(Firefox for Android Graveyard :: General, defect)

x86
macOS
defect
Not set

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: stechz, Assigned: stechz)

Details

Attachments

(1 file, 1 obsolete file)

Attached patch Fix (obsolete) — Splinter Review
Make sure rect is empty before calculating center of the rectangle.
Attached patch fix v2Splinter Review
Attachment #421703 - Attachment is obsolete: true
tracking-fennec: --- → 1.0+
Pushed http://hg.mozilla.org/mobile-browser/rev/41bd5bdf7f2c
Status: NEW → RESOLVED
tracking-fennec: 1.0+ → ---
Closed: 10 years ago
Resolution: --- → FIXED
Ben, testcase please? It sounds like you've seen this happen in certain situations.
It doesn't always seem to happen, but my steps were:
1) Go to engadget.com and let it load
2) Open a new tab

Result: The awesomebar won't come up.
Expected: It should.
verified FIXED on build:
Mozilla/5.0 (X11; U; Linux armv7l; Nokia N900; en-US; rv:1.9.2pre) Gecko/20100114 Firefox/3.6pre Fennec/1.1a1pre
Status: RESOLVED → VERIFIED
Flags: in-litmus?
Assignee: nobody → ayanshah62
I'm reassigning this to myself for adding litmus testcases for this (aakashd discussed this on irc with blassey and ted on #mobile).
Assignee: ayanshah62 → martijn.martijn
Flags: in-litmus? → in-litmus?(martijn.martijn)
Assignee: martijn.martijn → ayanshah62
Assignee: ayanshah62 → ben
Flags: in-litmus?(martijn.martijn) → in-litmus?(ayanshah62)
https://litmus.mozilla.org/show_test.cgi?id=15196
Flags: in-litmus?(ayanshah62) → in-litmus+
You need to log in before you can comment on or make changes to this bug.