{"action":"created","issue":{"url":"https://api.github.com/repos/servo/servo/issues/9081","labels_url":"https://api.github.com/repos/servo/servo/issues/9081/labels{/name}","comments_url":"https://api.github.com/repos/servo/servo/issues/9081/comments","events_url":"https://api.github.com/repos/servo/servo/issues/9081/events","html_url":"https://github.com/servo/servo/issues/9081","id":124215172,"number":9081,"title":"Set the canvas state properly when using pattern arguments for setStrokeStyle/setFillStyle","user":{"login":"jdm","id":27658,"avatar_url":"https://avatars.githubusercontent.com/u/27658?v=3","gravatar_id":"","url":"https://api.github.com/users/jdm","html_url":"https://github.com/jdm","followers_url":"https://api.github.com/users/jdm/followers","following_url":"https://api.github.com/users/jdm/following{/other_user}","gists_url":"https://api.github.com/users/jdm/gists{/gist_id}","starred_url":"https://api.github.com/users/jdm/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jdm/subscriptions","organizations_url":"https://api.github.com/users/jdm/orgs","repos_url":"https://api.github.com/users/jdm/repos","events_url":"https://api.github.com/users/jdm/events{/privacy}","received_events_url":"https://api.github.com/users/jdm/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/servo/servo/labels/A-content/canvas","name":"A-content/canvas","color":"d7e102"},{"url":"https://api.github.com/repos/servo/servo/labels/E-easy","name":"E-easy","color":"02e10c"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2015-12-29T14:01:01Z","updated_at":"2015-12-29T15:21:20Z","closed_at":null,"body":"The test added in #9080 has two cases that don't pass, and they both involve pattern objects. If we compare the branches in `CanvasRenderingContext2d::SetFillStyle`/`SetStrokeStyle`, we see that the strong and gradient paths set the `state` field, while the pattern ones don't, which is why the getters don't retrieve the pattern object as expected.\r\n\r\nCode: `components/script/dom/canvasrenderingcontext2d.rs`\r\nTest: `./mach test-wpt tests/wpt/mozilla/tests/mozilla/canvas/fill_and_stroke_getters_setters.html`"},"comment":{"url":"https://api.github.com/repos/servo/servo/issues/comments/167809214","html_url":"https://github.com/servo/servo/issues/9081#issuecomment-167809214","issue_url":"https://api.github.com/repos/servo/servo/issues/9081","id":167809214,"user":{"login":"jdm","id":27658,"avatar_url":"https://avatars.githubusercontent.com/u/27658?v=3","gravatar_id":"","url":"https://api.github.com/users/jdm","html_url":"https://github.com/jdm","followers_url":"https://api.github.com/users/jdm/followers","following_url":"https://api.github.com/users/jdm/following{/other_user}","gists_url":"https://api.github.com/users/jdm/gists{/gist_id}","starred_url":"https://api.github.com/users/jdm/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jdm/subscriptions","organizations_url":"https://api.github.com/users/jdm/orgs","repos_url":"https://api.github.com/users/jdm/repos","events_url":"https://api.github.com/users/jdm/events{/privacy}","received_events_url":"https://api.github.com/users/jdm/received_events","type":"User","site_admin":false},"created_at":"2015-12-29T15:21:20Z","updated_at":"2015-12-29T15:21:20Z","body":"That is changed in #9080."}}