Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix forced repaint on image load #10197

Merged
merged 1 commit into from Mar 25, 2016
Merged

Fix forced repaint on image load #10197

merged 1 commit into from Mar 25, 2016

Conversation

@mbrubeck
Copy link
Contributor

mbrubeck commented Mar 25, 2016

After #10021, LayoutThread::repaint no longer forced a repaint because of the restyle_damage check. This patch adds the correct restyle damage and fixes #10163.

r? @pcwalton


This change is Reviewable

After #10021, `LayoutThread::repaint` no longer forced a repaint because of
the `restyle_damage` check.  This patch adds the correct restyle damage and
fixes #10163.
@highfive
Copy link

highfive commented Mar 25, 2016

warning Warning warning

  • These commits modify layout code, but no tests are modified. Please consider adding a test!
@mbrubeck
Copy link
Contributor Author

mbrubeck commented Mar 25, 2016

I can't easily test this because we disable async image loading when running reftests.

@pcwalton
Copy link
Contributor

pcwalton commented Mar 25, 2016

Huh, I was working on my own implementation of this. I was under the impression that background images have no mechanism of triggering reflows at all when loaded (as opposed to <img> tags). Still, this looks like obviously the right thing to do…

@pcwalton
Copy link
Contributor

pcwalton commented Mar 25, 2016

@bors-servo
Copy link
Contributor

bors-servo commented Mar 25, 2016

📌 Commit afb36ec has been approved by pcwalton

@bors-servo
Copy link
Contributor

bors-servo commented Mar 25, 2016

Testing commit afb36ec with merge 8984d95...

bors-servo added a commit that referenced this pull request Mar 25, 2016
Fix forced repaint on image load

After #10021, `LayoutThread::repaint` no longer forced a repaint because of the `restyle_damage` check.  This patch adds the correct restyle damage and fixes #10163.

r? @pcwalton

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="35" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/10197)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Mar 25, 2016

@bors-servo bors-servo merged commit afb36ec into servo:master Mar 25, 2016
3 checks passed
3 checks passed
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
homu Test successful
Details
@mbrubeck mbrubeck deleted the mbrubeck:force-repaint branch May 11, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

4 participants
You can’t perform that action at this time.