Fischer [:Fischer] Assignee
	Description • 9 years ago

+++ This bug was initially created as a clone of Bug #1221859 +++
In the simulator, the document.registerElement method is not available so the smart-button's function will not work.
We should handle the focus on our own.

	Fischer [:Fischer] Assignee
	Comment 1 • 9 years ago

The document.registerElement method is unavailable inside iframe in the simulator.
The bug 1228929 is to separate the offline content from the iframe.
However, the online content has to be loaded inside a iframe so for the online content we have to handle the button focus by our own script.

	Fischer [:Fischer] Assignee
	Comment 2 • 9 years ago

The reason that the document.registerElement method is unavailable is because the online content iframe has no permission to it under the production mode.

	Fischer [:Fischer] Assignee
	Comment 3 • 9 years ago

@Rex,

This patch does:

- In the production build iframe content has no permission to document.registerElement method so remove the gaia-components/smart-button which needs document.registerElement

- Reimplement the UI styles and behaviours of the gaia-components/smart-button without using document.registerElement

p.s:
If need to see this update from the real site, for convenience, please just change the mozIframe.src in fling-tutorial/js/index.js to 'http://fischer-l.github.io/casting-tutorial/'.
The test github page above has been updated with the latest changes.

Thank you

	Fischer [:Fischer] Assignee
	Comment 4 • 9 years ago

Update the issue description:

In the production build because the app is 'app://' protocol and the iframe is 'http://' protocol, even the app claims the web-component permission but the iframe still cannot get the web-component permission.

As a result, the document.registerElement method is unavailable to  the iframe content and the gaia-components/smart-button which relies on the document.registerElement method wouldn't function.

p.s:
If it was in the engineering build, the iframe would be allowed to get the web-component permission in this case.

	KM Lee [:rexboy] (inactivate after Sep. 22)
	Comment 5 • 9 years ago

Comment on attachment 8710829 [details]
PR# 7

Looks good to me but please add some comments to describe the situation of web components. See my comments on Github.

	Fischer [:Fischer] Assignee
	Comment 6 • 9 years ago

Land to master: https://github.com/mozilla-b2g/casting-tutorial/commit/ec578ad528239952d3db204584df8b6fc43bfe7c