{"action":"created","issue":{"url":"https://api.github.com/repos/html5lib/html5lib-tests/issues/78","repository_url":"https://api.github.com/repos/html5lib/html5lib-tests","labels_url":"https://api.github.com/repos/html5lib/html5lib-tests/issues/78/labels{/name}","comments_url":"https://api.github.com/repos/html5lib/html5lib-tests/issues/78/comments","events_url":"https://api.github.com/repos/html5lib/html5lib-tests/issues/78/events","html_url":"https://github.com/html5lib/html5lib-tests/issues/78","id":154919437,"number":78,"title":"Incorrect expectations in tree-construction webkit02 tests","user":{"login":"iabudiab","id":5612057,"avatar_url":"https://avatars.githubusercontent.com/u/5612057?v=3","gravatar_id":"","url":"https://api.github.com/users/iabudiab","html_url":"https://github.com/iabudiab","followers_url":"https://api.github.com/users/iabudiab/followers","following_url":"https://api.github.com/users/iabudiab/following{/other_user}","gists_url":"https://api.github.com/users/iabudiab/gists{/gist_id}","starred_url":"https://api.github.com/users/iabudiab/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/iabudiab/subscriptions","organizations_url":"https://api.github.com/users/iabudiab/orgs","repos_url":"https://api.github.com/users/iabudiab/repos","events_url":"https://api.github.com/users/iabudiab/events{/privacy}","received_events_url":"https://api.github.com/users/iabudiab/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":10,"created_at":"2016-05-15T17:22:26Z","updated_at":"2016-05-17T11:53:44Z","closed_at":null,"body":"Correct if I'm wrong. I could prepare a PR, but wanted to verify this first.\r\n\r\nI believe the expectations for the following tests (15, 16, 17 in webkit02) to be incorrect:\r\n`<b><em><foo><foo><foo><aside></b></em>`\r\n`<b><em><foo><foo><foo><foo><foo><foo><foo><foo><foo><foo><aside></b></em>`\r\n`<b><em><foo><foob><foob><foob><foob><fooc><fooc><fooc><fooc><food><aside></b></em>`\r\n\r\nThis `<b><em><foo><foo><foo><aside></b></em>` expects the following tree:\r\n\r\n```text\r\n| <html>\r\n|   <head>\r\n|   <body>\r\n|     <b>\r\n|       <em>\r\n|         <foo>\r\n|           <foo>\r\n|             <foo>\r\n|     <aside>\r\n|       <b>\r\n```\r\n\r\nbut should expect:\r\n\r\n```text\r\n| <html>\r\n|   <head>\r\n|   <body>\r\n|     <b>\r\n|       <em>\r\n|         <foo>\r\n|           <foo>\r\n|             <foo>\r\n|     <aside>\r\n|       <b>\r\n|         <em>\r\n```\r\n\r\nThe same is true for the other two tests, i.e. the `<b>` element, child of `<aside>`, should have an `<em>` after running the adoption agency algorithm.\r\n\r\n"},"comment":{"url":"https://api.github.com/repos/html5lib/html5lib-tests/issues/comments/219695762","html_url":"https://github.com/html5lib/html5lib-tests/issues/78#issuecomment-219695762","issue_url":"https://api.github.com/repos/html5lib/html5lib-tests/issues/78","id":219695762,"user":{"login":"nox","id":123095,"avatar_url":"https://avatars.githubusercontent.com/u/123095?v=3","gravatar_id":"","url":"https://api.github.com/users/nox","html_url":"https://github.com/nox","followers_url":"https://api.github.com/users/nox/followers","following_url":"https://api.github.com/users/nox/following{/other_user}","gists_url":"https://api.github.com/users/nox/gists{/gist_id}","starred_url":"https://api.github.com/users/nox/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/nox/subscriptions","organizations_url":"https://api.github.com/users/nox/orgs","repos_url":"https://api.github.com/users/nox/repos","events_url":"https://api.github.com/users/nox/events{/privacy}","received_events_url":"https://api.github.com/users/nox/received_events","type":"User","site_admin":false},"created_at":"2016-05-17T11:53:44Z","updated_at":"2016-05-17T11:53:44Z","body":"I don't see what in @iabudiab's explanation disproves what @gsnedders stated in https://github.com/html5lib/html5lib-tests/issues/78#issuecomment-219328831.\r\n\r\nTo me by the time the algorithm reaches `<em>`, the inner loop counter already reached 3 and thus `<em>` gets removed.\r\n\r\nFWIW, html5ever passes this test."}}