1138870 - [Stingray] Move content of tv_shared folder to shared folder

Status: RESOLVED FIXED

(Firefox OS Graveyard :: Gaia::TV, defect)

Description

[Tzu-Lin Huang [:dwi2][:tzhuang]]

This is the required step before we could enable automatic test. Because currently we rely on build script to copy files from tv_shared to each apps. And build script would only do that with flag GAIA_DEVICE_TYPE=tv is set. And currently this flag is not set on TPBL.

By moving content of tv_shared to shared, we could utilize Gu and run unit test of tv_apps/

Comment 1

[Tzu-Lin Huang [:dwi2][:tzhuang]]

There are six folders under tv_shared:
1. components
2. font
3. js
4. resources
5. style
6. test

I'll move components/ to shared/elements, because they are all web-components, and put 3,4,5,6 into corresponding directory under shared/. I think font/ should be also under shared/style/

Comment 2

[Tim Guan-tin Chien [:timdream] (please needinfo)]

(In reply to Tzu-Lin Huang [:dwi2][:tzhuang] from comment #1)
> I'll move components/ to shared/elements, because they are all
> web-components, and put 3,4,5,6 into corresponding directory under shared/.
> I think font/ should be also under shared/style/

Ni Wilson on component -- I think the intention was having things inside that folder managed by bower.

Comment 3

[Tzu-Lin Huang [:dwi2][:tzhuang]]

That would be a good idea. Here is a bug for tracking smart-screen components and make them managed by bower. https://bugzilla.mozilla.org/show_bug.cgi?id=1136509

We should fix that bug before fixing this.

Comment 4

[GitHub Autolander Bot]

Attachment: [gaia] dwi2:bug1138870 > mozilla-b2g:master

Comment 5

[Tzu-Lin Huang [:dwi2][:tzhuang]]

Comment on attachment 8575223 [details] [review]
[gaia] dwi2:bug1138870 > mozilla-b2g:master

Hi Ricky, John,

Please help to review the patch. Thanks.

This patch did the following things:
1. Move everything under tv_apps/tv_shared/js/ to shared/js/
2. Move everything under tv_apps/tv_shared/style/ to shared/style/smart-screen/ because there is file name collision
3. Move everything under tv_apps/tv_shared/test/unit/mocks/ to shared/test/unit/mocks/
4. Move everything under tv_apps/tv_shared/resources/ to shared/resources/
5. Remove tv_apps/tv_shared/font/ and replace it with bower component https://github.com/smart-components/smart-icons. This will later merge with gaia-icons (see bug 1141425)
6. Remove tv_shared_helper.js
7. Remove almost every build/ directory of each app under tv_apps/, except smart-system
8. Adjust all affected paths in source code.

Comment 6

[Tzu-Lin Huang [:dwi2][:tzhuang]]

There is one more thing:

  tv_shared/ folder is history. (in the patch, for now :) )

Comment 7

[John Hu [:johnhu][:johu][:醬糊小弟]]

Should we move the `tv_shared/` folder to `shared/tv/`??

We had put about 16+ files in `tv_shared/`. It would be good to have it owned folder.

Comment 8

[John Hu [:johnhu][:johu][:醬糊小弟]]

Sorry, there is a typo at comment 7.

Should we move the `tv_shared/js/` folder to `shared/js/tv/`??

Comment 9

[Ricky Chien [:rickychien] (inactive)]

Comment on attachment 8575223 [details] [review]
[gaia] dwi2:bug1138870 > mozilla-b2g:master

r=rickychien, +1 with john's comment.

Comment 10

[Tzu-Lin Huang [:dwi2][:tzhuang]]

Thanks, I'll move js files originally under tv_shared/js/ to shared/smart-screen/js/ to make it more consistent with shared/style/smart-screen/

Comment 11

[John Hu [:johnhu][:johu][:醬糊小弟]]

Comment on attachment 8575223 [details] [review]
[gaia] dwi2:bug1138870 > mozilla-b2g:master

Looks good to me.

Comment 12

[GitHub Autolander Bot]

Pull request has landed in master: https://github.com/mozilla-b2g/gaia/commit/7459e53450079fa9a6142e2b62477b993942dad1