Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

tab previews: stick to changing properties that can be handled by the compositor alone #10291

Closed
cezaraugusto opened this issue Aug 4, 2017 · 1 comment · Fixed by #10293
Closed

tab previews: stick to changing properties that can be handled by the compositor alone #10291

cezaraugusto opened this issue Aug 4, 2017 · 1 comment · Fixed by #10293
Assignees
@cezaraugusto
Labels
feature/tabsbar QA/checked-Linux QA/checked-macOS QA/checked-Win64 QA/test-plan-specified release-notes/include
Milestone
0.18.23 Hotfix

Comments

@cezaraugusto
Copy link
Contributor

cezaraugusto commented Aug 4, 2017 •
edited by luixxiul
Loading

Test plan

#10293 (comment)

this avoids layout and paint steps and let the task for the compositor
@cezaraugusto cezaraugusto added this to the 0.18.x Hotfix milestone Aug 4, 2017
@cezaraugusto cezaraugusto self-assigned this Aug 4, 2017
cezaraugusto added a commit that referenced this issue Aug 4, 2017
@cezaraugusto
rely on compositor properties for tab previews
c07be24 
Auditors: @bsclifton, @luixxiul
close #10291
@cezaraugusto cezaraugusto mentioned this issue Aug 4, 2017
Merged
@luixxiul
Copy link
Contributor

luixxiul commented Aug 5, 2017

I'm changing the label to include this issue as every change which improves the performance should be noted.

This was referenced Aug 7, 2017
Closed
Closed
Closed
Closed
Closed
This was referenced Aug 9, 2017
Closed
Closed
This was referenced Aug 10, 2017
Closed
Closed
@srirambv srirambv mentioned this issue Aug 16, 2017
Closed
dfperry5 pushed a commit to dfperry5/browser-laptop that referenced this issue Aug 18, 2017
@cezaraugusto @dfperry5
rely on compositor properties for tab previews
57115cc 
Auditors: @bsclifton, @luixxiul
close brave#10291
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@cezaraugusto cezaraugusto

Labels
feature/tabsbar QA/checked-Linux QA/checked-macOS QA/checked-Win64 QA/test-plan-specified release-notes/include
Projects
None yet
Milestone
0.18.23 Hotfix
Development

Successfully merging a pull request may close this issue.

rely on compositor properties for tab previews brave/browser-laptop
4 participants
@luixxiul @cezaraugusto @srirambv @LaurenWags