Closed
Bug 523373
Opened 15 years ago
Closed 15 years ago
[Regression] _criticalRect is empty and throws in Rect.center()
Categories
(Firefox for Android Graveyard :: General, defect)
Tracking
(Not tracked)
VERIFIED
FIXED
fennec1.0b5
People
(Reporter: mfinkle, Assigned: stechz)
References
Details
Attachments
(1 file, 1 obsolete file)
7.13 KB,
patch
|
froystig
:
review+
|
Details | Diff | Splinter Review |
* Go to the Add-on Manager * Search for an add-on * Select a returned add-on * Press "Go to Page" button "Empty rectangles do not have centers" starts getting thrown from Rect.center()
Assignee | ||
Comment 1•15 years ago
|
||
Since empty rectangles are possible, criticalRect is now never null.
Assignee: nobody → webapps
Attachment #407370 -
Flags: review?(froystig)
Assignee | ||
Comment 2•15 years ago
|
||
Attachment #407384 -
Flags: review?(froystig)
Assignee | ||
Updated•15 years ago
|
Attachment #407370 -
Attachment is obsolete: true
Attachment #407370 -
Flags: review?(froystig)
Comment 3•15 years ago
|
||
Comment on attachment 407384 [details] [diff] [review] Fix missed rectangle checks This seems to correct all the regressions caused by the issue best described as: "Rect operations no longer give null as a sentinel value meaning 'empty', but instead make a Rect whose isEmpty() method returns true", which is the general statement of the bug filed here. It's very difficult to be entirely sure that the patch corrects *all* cases where we used to rely on a null value to mean "empty" but no longer can (some cases might be obscure and hidden), but these are at least big common ones, if not all of them. In particular, it does appear to fix all those associated with the critical rect.
Attachment #407384 -
Flags: review?(froystig) → review+
Assignee | ||
Updated•15 years ago
|
tracking-fennec: --- → ?
Reporter | ||
Comment 6•15 years ago
|
||
pushed: https://hg.mozilla.org/mobile-browser/rev/550540aec3bd
Status: NEW → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Target Milestone: --- → B5
Comment 8•15 years ago
|
||
verified FIXED on builds: Mozilla/5.0 (Windows; U; WindowsCE 5.2; en-US; rv:1.9.2b1pre) Gecko/20091021 Fennec/1.0a4pre and Mozilla/5.0 (X11; U; Linux armv7l; en-US; rv:1.9.2b1pre) Gecko/20091021 Fennec/1.0b5pre and Mozilla/5.0 (X11; U; Linux armv6l; en-US; rv:1.9.3a1pre) Gecko/20091021 Fennec/1.0b5pre
Status: RESOLVED → VERIFIED
Updated•15 years ago
|
tracking-fennec: ? → ---
Flags: in-litmus?
Comment 9•14 years ago
|
||
litmus testcase https://litmus.mozilla.org/show_test.cgi?id=11573 created to regression test this bug.
Flags: in-litmus? → in-litmus+
You need to log in
before you can comment on or make changes to this bug.
Description
•