Closed
Bug 1406120
Opened 7 years ago
Closed 7 years ago
Remove fade delay for about:home
Categories
(Firefox :: New Tab Page, defect)
Firefox
New Tab Page
Tracking
()
RESOLVED
FIXED
Firefox 58
People
(Reporter: Mardak, Assigned: Mardak)
References
Details
User Story
Attachments
(1 file)
59 bytes,
text/x-review-board-request
|
k88hudson
:
review+
ritu
:
approval-mozilla-beta+
|
Details |
A delay on the fade was added in bug 1401682 to avoid a tart regression. But that talos test is using about:newtab and not about:home, so we can apply the delay only there.
Assignee | ||
Updated•7 years ago
|
User Story: (updated)
Assignee | ||
Updated•7 years ago
|
Assignee: nobody → edilee
Comment hidden (mozreview-request) |
Comment 2•7 years ago
|
||
mozreview-review |
Comment on attachment 8915692 [details]
Bug 1406120 - Remove fade delay for about:home.
https://reviewboard.mozilla.org/r/186902/#review191954
Attachment #8915692 -
Flags: review?(khudson) → review+
Comment 3•7 years ago
|
||
We're sorry, Autoland could not rebase your commits for you automatically. Please manually rebase your commits and try again.
hg error in cmd: hg rebase -s 62a443b0d56b -d eaa0d99df029: rebasing 424371:62a443b0d56b "Bug 1406120 - Remove fade delay for about:home. r=k88hudson" (tip)
merging browser/extensions/activity-stream/install.rdf.in
warning: conflicts while merging browser/extensions/activity-stream/install.rdf.in! (edit, then use 'hg resolve --mark')
unresolved conflicts (see hg resolve, then hg rebase --continue)
Pushed by edilee@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/eb855465eb9c
Remove fade delay for about:home. r=k88hudson
![]() |
||
Comment 5•7 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 58
Assignee | ||
Comment 6•7 years ago
|
||
Comment on attachment 8915692 [details]
Bug 1406120 - Remove fade delay for about:home.
Approval Request Comment
[Feature/Bug causing the regression]: Talos tart regression fix from bug
1401682. Blocks tracking-firefox57 blocking bug 1399961.
[User impact if declined]: Users forced to wait before seeing most of about:home content making Firefox feel slow. See https://ed.agadak.net/as/57%20uplift.webmhd.webm
[Is this code covered by automated tests?]: No
[Has the fix been verified in Nightly?]: Yes, 20171007100142
[Needs manual test from QE? If yes, steps to reproduce]: No
[List of other uplifts needed for the feature/fix]: This bug after bug 1406096
[Is the change risky?]: No
[Why is the change risky/not risky?]: The change is css removing the transition-delay
[String changes made/needed]: Nope
[Perf win]: From video above, before the content starts showing on average 4.25 frames (30fps) = 141ms after the localized text appears, and after the content starts showing on average 0.2 frames = 6ms. A 23.5x (2350%) improvement.
Attachment #8915692 -
Flags: approval-mozilla-beta?
status-firefox57:
--- → affected
Comment on attachment 8915692 [details]
Bug 1406120 - Remove fade delay for about:home.
Fix targeting perf improvement, Beta57+
Attachment #8915692 -
Flags: approval-mozilla-beta? → approval-mozilla-beta+
Comment 8•7 years ago
|
||
bugherder uplift |
Assignee | ||
Updated•7 years ago
|
User Story: (updated)
Updated•6 years ago
|
Component: Activity Streams: Newtab → New Tab Page
You need to log in
before you can comment on or make changes to this bug.
Description
•