{"push_id":1223008383,"size":11,"distinct_size":11,"ref":"refs/heads/auto","head":"8d1ddba47eb3c81b30a6676371a8a683143504fd","before":"3afa150cd8ef9f1d370ed425399aa0e1f42b339e","commits":[{"sha":"a3020419d9eace898727f5b778e661b61a1bb8df","author":{"name":"Emilio Cobos Álvarez","email":"ac7d742765dd00d21c3ba30feda77cb44cb363f2@gmail.com"},"message":"stylo: Don't traverse the whole dom every restyle, propagate the dirty flag down the DOM.\n\nThis commit adds hooks to the Servo style traversal to avoid traversing all the\nDOM for every restyle. Additionally it changes the behavior of the dirty flag to\nbe propagated top down, to prevent extra overhead when an element is dirtied.\n\nThis commit doesn't aim to change the behavior on Servo just yet, since Servo\nmight rely on a full bottom up reconstruction of the flows. I'll need to double\ncheck and implement that separately.","distinct":true,"url":"https://api.github.com/repos/servo/servo/commits/a3020419d9eace898727f5b778e661b61a1bb8df"},{"sha":"96ea1a335cab7a55e7d9bdbc988c6d636e2ac001","author":{"name":"Emilio Cobos Álvarez","email":"ac7d742765dd00d21c3ba30feda77cb44cb363f2@gmail.com"},"message":"stylo: don't double check that the root node is dirty in restyle_tree, since now we ensure it in the caller.","distinct":true,"url":"https://api.github.com/repos/servo/servo/commits/96ea1a335cab7a55e7d9bdbc988c6d636e2ac001"},{"sha":"d81fe27b114c12a9fdb292b66ad52f9c1def3f1f","author":{"name":"Emilio Cobos Álvarez","email":"ac7d742765dd00d21c3ba30feda77cb44cb363f2@gmail.com"},"message":"style: Avoid propagating the restyle flag all through the dom when a node gets dirty.\n\nThis puts us in pair with stylo.","distinct":true,"url":"https://api.github.com/repos/servo/servo/commits/d81fe27b114c12a9fdb292b66ad52f9c1def3f1f"},{"sha":"e6958d3947adedda7f3f52f26fae321f0b4d0ad4","author":{"name":"Emilio Cobos Álvarez","email":"ac7d742765dd00d21c3ba30feda77cb44cb363f2@gmail.com"},"message":"script: Fix a few load related bugs.\n\nThis is what was making me hit the new test failures. So turns out that when the\nDOMContentLoaded event is fired we fired no messages to the constellation, but\nwe fired the DOMLoad message from the DocumentProgressHandler, effectively after\nhaving dispatched the Load message from script thread.\n\nThis also fixes the possibility of a subframe navigation not blocking the load\nevent of the parent document, for example.","distinct":true,"url":"https://api.github.com/repos/servo/servo/commits/e6958d3947adedda7f3f52f26fae321f0b4d0ad4"},{"sha":"702445a3ec4d406472a618750f5dab64c38459bb","author":{"name":"Emilio Cobos Álvarez","email":"ac7d742765dd00d21c3ba30feda77cb44cb363f2@gmail.com"},"message":"script: Don't hold the stderr lock while doing sync operations with the constellation.\n\nOtherwise if you enable debug logging, you deadlock.","distinct":true,"url":"https://api.github.com/repos/servo/servo/commits/702445a3ec4d406472a618750f5dab64c38459bb"},{"sha":"354dc660292b798bb0beafcc39a98e2519518e53","author":{"name":"Emilio Cobos Álvarez","email":"ac7d742765dd00d21c3ba30feda77cb44cb363f2@gmail.com"},"message":"Some debugging improvements and code style nits across gfx and constellation.\n\n * Propagate the RUST_LOG env var to the child process\n * Add debug information when a recv() fails in paint thread's select!.","distinct":true,"url":"https://api.github.com/repos/servo/servo/commits/354dc660292b798bb0beafcc39a98e2519518e53"},{"sha":"c3a727ebda5bc9a0d58c8895c7d4626790a9e45d","author":{"name":"Emilio Cobos Álvarez","email":"ac7d742765dd00d21c3ba30feda77cb44cb363f2@gmail.com"},"message":"style: Remove a few more unuseful traversals now we can.","distinct":true,"url":"https://api.github.com/repos/servo/servo/commits/c3a727ebda5bc9a0d58c8895c7d4626790a9e45d"},{"sha":"572b3c31bd7df82bd47b10381c3bff80e75c42b0","author":{"name":"Emilio Cobos Álvarez","email":"ac7d742765dd00d21c3ba30feda77cb44cb363f2@gmail.com"},"message":"style: Add a comment about pre_process_children_hook.","distinct":true,"url":"https://api.github.com/repos/servo/servo/commits/572b3c31bd7df82bd47b10381c3bff80e75c42b0"},{"sha":"36376461f4e1a2f4b3db7f940029df7dcf24be20","author":{"name":"Emilio Cobos Álvarez","email":"ac7d742765dd00d21c3ba30feda77cb44cb363f2@gmail.com"},"message":"script: Unify LoadComplete and DOMLoad messages.\n\nSee the PR in which this commit landed and also\nhttps://github.com/servo/servo/pull/6415#issuecomment-122294169","distinct":true,"url":"https://api.github.com/repos/servo/servo/commits/36376461f4e1a2f4b3db7f940029df7dcf24be20"},{"sha":"f6043a23942816989c51798c069c11e8e8726882","author":{"name":"Emilio Cobos Álvarez","email":"ac7d742765dd00d21c3ba30feda77cb44cb363f2@gmail.com"},"message":"layout: Remove now unused dirty_all_nodes function in layout thread.","distinct":true,"url":"https://api.github.com/repos/servo/servo/commits/f6043a23942816989c51798c069c11e8e8726882"},{"sha":"8d1ddba47eb3c81b30a6676371a8a683143504fd","author":{"name":"bors-servo","email":"b09d4f4a3e19bc290c035b624e2133ac70195a90@mozilla.com"},"message":"Auto merge of #12563 - emilio:stylo, r=bholley,jdm,pcwalton\n\nstylo: Improve restyling performance\n\nThis commit adds hooks to the Servo style traversal to avoid traversing all the\nDOM for every restyle. Additionally it changes the behavior of the dirty flag to\nbe propagated top down, to prevent extra overhead when an element is dirtied.\n\nThis commit doesn't aim to change the behavior on Servo just yet, since Servo does extra job when dirtying the node related with DOM revision counters that might be necessary.\n\nCC @asajeffrey for the DOM revision counters stuff. When a node is dirty, do all its descendants really need to increment the revision counter, or is this an unintended effect? My intuition is that this is hurting performance quite a lot for servo.\n\nr? @bholley\n\n<!-- Please describe your changes on the following line: -->\n\n---\n<!-- Thank you for contributing to Servo! Please replace each `[ ]` by `[X]` when the step is complete, and replace `__` with appropriate data: -->\n- [x] `./mach build -d` does not report any errors\n- [x] `./mach test-tidy` does not report any errors\n\n<!-- Either: -->\n- [x] These changes do not require tests because no geckolib tests yet.\n\n<!-- Pull requests that do not address these steps are welcome, but they will require additional verification as part of the review process. -->\n\n<!-- Reviewable:start -->\n---\nThis change is [<img src=\"https://reviewable.io/review_button.svg\" height=\"34\" align=\"absmiddle\" alt=\"Reviewable\"/>](https://reviewable.io/reviews/servo/servo/12563)\n<!-- Reviewable:end -->","distinct":true,"url":"https://api.github.com/repos/servo/servo/commits/8d1ddba47eb3c81b30a6676371a8a683143504fd"}]}
{"actor":{"display_login":"bors-servo"}}