Further decompose the HTMLMediaElement::IsAllowedToPlay() and AudioChannelAgentCallback::IsAllowedToPlay() so that we can have all the spec. code in the HTMLMediaElement::PlayInternal() and only policy code in the HTMLMediaElement::Play().

Try: https://treeherder.mozilla.org/#/jobs?repo=try&revision=2d86a521a1ba3d2f8ee476dfde947781f8cc87df https://treeherder.mozilla.org/#/jobs?repo=try&revision=ce028aea48e2c118ce7662975c64e320dbe8dc25

Comment on attachment 8815654 [details] Bug 1321196 part 2 - rework the IsAllowedToPlay policy; https://reviewboard.mozilla.org/r/96504/#review96720

Comment on attachment 8815653 [details] Bug 1321196 part 1 - typo correction; https://reviewboard.mozilla.org/r/96502/#review96728

Comment on attachment 8815654 [details] Bug 1321196 part 2 - rework the IsAllowedToPlay policy; https://reviewboard.mozilla.org/r/96504/#review96732

Try looks good, thanks for the reviews!

Pushed by ryanvm@gmail.com: https://hg.mozilla.org/integration/autoland/rev/e972bb041a4e part 1 - typo correction; r=alwu https://hg.mozilla.org/integration/autoland/rev/fcabe39c665f part 2 - rework the IsAllowedToPlay policy; r=alwu,jwwang

https://hg.mozilla.org/mozilla-central/rev/e972bb041a4e https://hg.mozilla.org/mozilla-central/rev/fcabe39c665f