{"action":"created","issue":{"url":"https://api.github.com/repos/w3c/web-platform-tests/issues/3590","repository_url":"https://api.github.com/repos/w3c/web-platform-tests","labels_url":"https://api.github.com/repos/w3c/web-platform-tests/issues/3590/labels{/name}","comments_url":"https://api.github.com/repos/w3c/web-platform-tests/issues/3590/comments","events_url":"https://api.github.com/repos/w3c/web-platform-tests/issues/3590/events","html_url":"https://github.com/w3c/web-platform-tests/pull/3590","id":173453708,"number":3590,"title":"New presentation start tests that trigger a mixed content error","user":{"login":"tidoust","id":1122927,"avatar_url":"https://avatars.githubusercontent.com/u/1122927?v=3","gravatar_id":"","url":"https://api.github.com/users/tidoust","html_url":"https://github.com/tidoust","followers_url":"https://api.github.com/users/tidoust/followers","following_url":"https://api.github.com/users/tidoust/following{/other_user}","gists_url":"https://api.github.com/users/tidoust/gists{/gist_id}","starred_url":"https://api.github.com/users/tidoust/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tidoust/subscriptions","organizations_url":"https://api.github.com/users/tidoust/orgs","repos_url":"https://api.github.com/users/tidoust/repos","events_url":"https://api.github.com/users/tidoust/events{/privacy}","received_events_url":"https://api.github.com/users/tidoust/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/w3c/web-platform-tests/labels/presentation-api","name":"presentation-api","color":"ededed"},{"url":"https://api.github.com/repos/w3c/web-platform-tests/labels/wg-secondscreen","name":"wg-secondscreen","color":"0052cc"}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":2,"created_at":"2016-08-26T13:31:35Z","updated_at":"2016-08-29T14:33:14Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/w3c/web-platform-tests/pulls/3590","html_url":"https://github.com/w3c/web-platform-tests/pull/3590","diff_url":"https://github.com/w3c/web-platform-tests/pull/3590.diff","patch_url":"https://github.com/w3c/web-platform-tests/pull/3590.patch"},"body":"The tests need to run over HTTPS, hence the `.https.html` extension. They create a `PresentationRequest` with an HTTP URL and ensure that `start` reports a `SecurityError` exception.\r\n\r\nNote the tests will pass if they are run over HTTP by mistake (since there would not be any \"mixed content\" anymore), which could be confusing. A short exchange on IRC suggested that individual test files should not check the protocol themselves, but that the test harness could perhaps be updated to report the error. I prepared a PR on testharness.js accordingly: https://github.com/w3c/testharness.js/pull/207"},"comment":{"url":"https://api.github.com/repos/w3c/web-platform-tests/issues/comments/243141923","html_url":"https://github.com/w3c/web-platform-tests/pull/3590#issuecomment-243141923","issue_url":"https://api.github.com/repos/w3c/web-platform-tests/issues/3590","id":243141923,"user":{"login":"tidoust","id":1122927,"avatar_url":"https://avatars.githubusercontent.com/u/1122927?v=3","gravatar_id":"","url":"https://api.github.com/users/tidoust","html_url":"https://github.com/tidoust","followers_url":"https://api.github.com/users/tidoust/followers","following_url":"https://api.github.com/users/tidoust/following{/other_user}","gists_url":"https://api.github.com/users/tidoust/gists{/gist_id}","starred_url":"https://api.github.com/users/tidoust/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tidoust/subscriptions","organizations_url":"https://api.github.com/users/tidoust/orgs","repos_url":"https://api.github.com/users/tidoust/repos","events_url":"https://api.github.com/users/tidoust/events{/privacy}","received_events_url":"https://api.github.com/users/tidoust/received_events","type":"User","site_admin":false},"created_at":"2016-08-29T14:33:14Z","updated_at":"2016-08-29T14:33:14Z","body":"I extended the scope of this pull request to cover other parts of the Presentation API that also require user agents to check for mixed content and throw a `SecurityError`. The tests should now cover `start`, `reconnect`, and `getAvailability`, the latter two through automated tests."}}
{"actor":{"display_login":"tidoust"}}