Please report any other irregularities here.
This bug should implement FTE for casting video to TV. Refer to UX: https://drive.google.com/a/mozilla.com/folderview?id=0B5RN80W56Ga9fjlncXJuZW5YbW0zX2tDX0sta3FsMXU3cmRYM0hTOEJXZFZ2ck5wSW1PZlU&usp=sharing&tid=0B5RN80W56Ga9WkNBRlZmWUtiaU0 Visual: https://drive.google.com/a/mozilla.com/folderview?id=0B4K8q1qWmtAvdUJFYUx5NFBEaVk&usp=sharing&tid=0B2-G3kew1WpXLURNc29vTWl5eUk
Summary: [TV] FTE for Casting video to TV → [TV] FTE for Casting video and website to TV
Not sure if we need to split casting website into another bug. We can open another one if needed.
Assignee: nobody → fliu
Whiteboard: [ft:conndevices] → [ft:conndevices][partner-blocker][partner-cherry-pick]
Created attachment 8685980 [details] [review] [gaia] Fischer-L:bug_1220584-send-video-to-tv-app > mozilla-b2g:master
Created attachment 8686414 [details] [review] PR #33135
Comment on attachment 8686414 [details] [review] PR #33135 Can you put your offline contents together for reviewing?
Comment on attachment 8686414 [details] [review] PR #33135 You need to figure out bower_components and change directory structure in some way. And please resolve the focus problem. I guess the cause is you didn't focus on the Iframe.
Comment on attachment 8686414 [details] [review] PR #33135 r=me with 1 comment. Please make corresponding changes before merging.
Attachment #8686414 - Flags: review?(rexboy) → review+
Land to Master: https://github.com/mozilla-b2g/gaia/pull/33135/commits Test: https://treeherder.mozilla.org/#/jobs?repo=gaia&revision=8d73f8d781d5f6deecb017723b2a85d39a3b1f8c
Status: ASSIGNED → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED
I've just noticed that the icons that were introduced with this commit have not been compressed. They take over 4 MB of space whereas they could take ~1.5 MiB. Please run the compression script in the gaia tools and commit the compressed images instead (you'll need the optipng and advancecomp/advpng packages installed). From the gaia root directory execute the following command: ./tools/png_recompress -v \ tv_apps/fling-tutorial/style/icons/fling-tutorial_336.png \ tv_apps/fling-tutorial/style/index.css \ tv_apps/fling-tutorial/style/offline_content/images/background.png \ tv_apps/fling-tutorial/style/offline_content/images/cast_link_icon.png \ tv_apps/fling-tutorial/style/offline_content/images/cast_video.png \ tv_apps/fling-tutorial/style/offline_content/images/cast_video_icon.png \ tv_apps/fling-tutorial/style/offline_content/images/cast_website.png \ tv_apps/fling-tutorial/style/offline_content/images/phone.png Be patient as this may take a while but the results are well-worth it. Besides the space savings we get faster load times, less memory use when decoding them and better caching.
@Josh, This patch should be uplifted for TV 2.5. Thank you
Comment on attachment 8685980 [details] [review] [gaia] Fischer-L:bug_1220584-send-video-to-tv-app > mozilla-b2g:master [Approval Request Comment] [Bug caused by] (feature/regressing bug #): 1220584 [User impact] if declined: The use will not see the Send-to-TV tutorial [Testing completed]: Yes [Risk to taking this patch] (and alternatives if risky): Low [String changes made]: No
Attachment #8685980 - Flags: approval-gaia-v2.5?
[Blocking Requested - why for this release]:
blocking-b2g: --- → 2.5?
blocking-b2g: 2.5? → 2.5+
status-b2g-v2.5: --- → affected
status-b2g-master: --- → fixed
Comment on attachment 8685980 [details] [review] [gaia] Fischer-L:bug_1220584-send-video-to-tv-app > mozilla-b2g:master Approve for TV 2.5
Attachment #8685980 - Flags: approval-gaia-v2.5? → approval-gaia-v2.5+
status-b2g-v2.5: affected → fixed
You need to log in before you can comment on or make changes to this bug.