Closed
Bug 1385034
Opened 7 years ago
Closed 7 years ago
Write reflow tests for the tab strip when it overflows and underflows
Categories
(Firefox :: Tabbed Browser, enhancement, P1)
Firefox
Tabbed Browser
Tracking
()
Tracking | Status | |
---|---|---|
firefox57 | --- | fixed |
People
(Reporter: mconley, Assigned: mconley)
References
(Blocks 1 open bug)
Details
(Whiteboard: [photon-performance])
Attachments
(1 file)
Bug 1356705 is required since this allows us to get a deterministic number of reflows when scrolling the tab strip.
Assignee | ||
Updated•7 years ago
|
Blocks: photon-perf-tabs
Assignee | ||
Updated•7 years ago
|
Whiteboard: [photon-performance]
Assignee | ||
Updated•7 years ago
|
Assignee: nobody → mconley
Comment hidden (mozreview-request) |
Updated•7 years ago
|
Status: NEW → ASSIGNED
Iteration: --- → 56.4 - Aug 1
Flags: qe-verify-
Priority: -- → P1
Comment 2•7 years ago
|
||
mozreview-review |
Comment on attachment 8891007 [details]
Bug 1385034 - Add a reflow test for when the tabstrip over and underflows.
https://reviewboard.mozilla.org/r/162180/#review167970
::: browser/base/content/test/performance/browser_tabstrip_overflow_underflow_reflows.js:103
(Diff revision 1)
> + * This test ensures that there are no unexpected
> + * uninterruptible reflows when opening a new tab that will
> + * cause the existing tabs to overflow and the tab strip to
> + * become scrollable. It also tests that there are no
> + * unexpected uninterruptible reflows when closing that tab,
> + * which causes the tab strip to underflow.
In this the nittiest of nits,
could you rewrap this comment
so it has longer lines and is
not split over quite so many
separate lines making it
unnecessarily hard to scan?
;-)
(the code goes up to 96 chars per line, and 76-80 in many places, and the comment maxes out at 60)
Attachment #8891007 -
Flags: review?(gijskruitbosch+bugs) → review+
Updated•7 years ago
|
Iteration: 56.4 - Aug 1 → 57.1 - Aug 15
Assignee | ||
Comment 3•7 years ago
|
||
mozreview-review-reply |
Comment on attachment 8891007 [details]
Bug 1385034 - Add a reflow test for when the tabstrip over and underflows.
https://reviewboard.mozilla.org/r/162180/#review167970
> In this the nittiest of nits,
> could you rewrap this comment
> so it has longer lines and is
> not split over quite so many
> separate lines making it
> unnecessarily hard to scan?
> ;-)
>
> (the code goes up to 96 chars per line, and 76-80 in many places, and the comment maxes out at 60)
Good catch! Had my editor zoomed in, so I think I got a bit overzealous on the wrapping. :)
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Pushed by mconley@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/0050487d9369
Add a reflow test for when the tabstrip over and underflows. r=Gijs
Comment 7•7 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
status-firefox57:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 57
You need to log in
before you can comment on or make changes to this bug.
Description
•