{"action":"created","issue":{"url":"https://api.github.com/repos/servo/webrender/issues/581","repository_url":"https://api.github.com/repos/servo/webrender","labels_url":"https://api.github.com/repos/servo/webrender/issues/581/labels{/name}","comments_url":"https://api.github.com/repos/servo/webrender/issues/581/comments","events_url":"https://api.github.com/repos/servo/webrender/issues/581/events","html_url":"https://github.com/servo/webrender/pull/581","id":190785825,"number":581,"title":"Clip stack support","user":{"login":"kvark","id":107301,"avatar_url":"https://avatars.githubusercontent.com/u/107301?v=3","gravatar_id":"","url":"https://api.github.com/users/kvark","html_url":"https://github.com/kvark","followers_url":"https://api.github.com/users/kvark/followers","following_url":"https://api.github.com/users/kvark/following{/other_user}","gists_url":"https://api.github.com/users/kvark/gists{/gist_id}","starred_url":"https://api.github.com/users/kvark/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/kvark/subscriptions","organizations_url":"https://api.github.com/users/kvark/orgs","repos_url":"https://api.github.com/users/kvark/repos","events_url":"https://api.github.com/users/kvark/events{/privacy}","received_events_url":"https://api.github.com/users/kvark/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":11,"created_at":"2016-11-21T17:33:04Z","updated_at":"2016-11-24T23:30:44Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/servo/webrender/pulls/581","html_url":"https://github.com/servo/webrender/pull/581","diff_url":"https://github.com/servo/webrender/pull/581.diff","patch_url":"https://github.com/servo/webrender/pull/581.patch"},"body":"Closes #498 \r\n\r\nRemoves `StackingContext::overflow` in favour of `DisplayItem::clip`.\r\nGenerates clip tasks for the layers that the primitives may depend on when generating their masks, copies over the affected sub-regions instead of clearing (new `cs_clip_copy` shader).\r\n\r\nCurrent status: base tests are passing, new logic test cases are pending\r\n\r\n<!-- Reviewable:start -->\r\n---\r\nThis change is [<img src=\"https://reviewable.io/review_button.svg\" height=\"34\" align=\"absmiddle\" alt=\"Reviewable\"/>](https://reviewable.io/reviews/servo/webrender/581)\r\n<!-- Reviewable:end -->\r\n"},"comment":{"url":"https://api.github.com/repos/servo/webrender/issues/comments/262855776","html_url":"https://github.com/servo/webrender/pull/581#issuecomment-262855776","issue_url":"https://api.github.com/repos/servo/webrender/issues/581","id":262855776,"user":{"login":"glennw","id":2350882,"avatar_url":"https://avatars.githubusercontent.com/u/2350882?v=3","gravatar_id":"","url":"https://api.github.com/users/glennw","html_url":"https://github.com/glennw","followers_url":"https://api.github.com/users/glennw/followers","following_url":"https://api.github.com/users/glennw/following{/other_user}","gists_url":"https://api.github.com/users/glennw/gists{/gist_id}","starred_url":"https://api.github.com/users/glennw/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/glennw/subscriptions","organizations_url":"https://api.github.com/users/glennw/orgs","repos_url":"https://api.github.com/users/glennw/repos","events_url":"https://api.github.com/users/glennw/events{/privacy}","received_events_url":"https://api.github.com/users/glennw/received_events","type":"User","site_admin":false},"created_at":"2016-11-24T23:30:44Z","updated_at":"2016-11-24T23:30:44Z","body":"@kvark I did some initial testing - the tests so far that I've run pass - but there seems to be a timing issue, which will expose those Servo intermittents again. Could you see if you can reproduce the results below locally? Hopefully I was just testing something incorrectly...\r\n\r\nIf you run:\r\n\r\n```\r\ntime ./mach test-wpt --release --processes=8 --log-raw=wpt.log tests/wpt/mozilla/tests/css/\r\n```\r\n\r\nThe results I get *without* this patch:\r\n\r\n```\r\nreal\t1m25.866s\r\nuser\t8m48.928s\r\nsys\t0m58.704s\r\n```\r\n\r\nThe results *with* this patch:\r\n\r\n```\r\nreal\t2m33.484s\r\nuser\t17m6.348s\r\nsys\t0m57.728s\r\n```\r\n"}}
{"actor":{"display_login":"glennw"}}