SmartPhone Code Transition: Difference between revisions

Jump to navigation Jump to search
Renamed bug links to include the bug #
(added dependencies for chrome meta)
(Renamed bug links to include the bug #)
Line 30: Line 30:
|-
|-
| rowspan="1" | [META] Firefox OS Transition work
| rowspan="1" | [META] Firefox OS Transition work
| [https://bugzilla.mozilla.org/show_bug.cgi?id=1252143 Bug ID] 1252143
| [https://bugzilla.mozilla.org/show_bug.cgi?id=1252143 1252143]  
| META for the Firefox OS Transition work
| META for the Firefox OS Transition work
| Bug dependencies: 1252193, 1252427, 1252570, 1252573, 1252773, 1252776, 1252778, 1252779, 1252789, 1253600, 1254281, 1254282, 1254283, 1254284, 1254285, 1254286, 1254287, 1254292,  
| Bug dependencies: 1252193, 1252427, 1252570, 1252573, 1252773, 1252776, 1252778, 1252779, 1252789, 1253600, 1254281, 1254282, 1254283, 1254284, 1254285, 1254286, 1254287, 1254292,  
|-
|-
| rowspan="1" | [Meta] Get core system apps working as chrome
| rowspan="1" | [Meta] Get core system apps working as chrome
| [https://bugzilla.mozilla.org/show_bug.cgi?id=1261025 Bug ID] 1261025
| [https://bugzilla.mozilla.org/show_bug.cgi?id=1261025 1261025]  
| META for getting core system apps working as chrome
| META for getting core system apps working as chrome
| Bug dependencies:  
| Bug dependencies:  
Line 45: Line 45:
|-
|-
| rowspan="1" |  [transition] Browser: Remove reference and logos relating to Firefox
| rowspan="1" |  [transition] Browser: Remove reference and logos relating to Firefox
| [https://bugzilla.mozilla.org/show_bug.cgi?id=1252193 Bug ID] 1252193
| [https://bugzilla.mozilla.org/show_bug.cgi?id=1252193 1252193]
| 1. Remove all references to Firefox OS
| 1. Remove all references to Firefox OS
2. Change browser icon to not be firefox logo  
2. Change browser icon to not be firefox logo  
Line 52: Line 52:
|-
|-
| rowspan="1" |  Bootstrap the transition
| rowspan="1" |  Bootstrap the transition
| [https://bugzilla.mozilla.org/show_bug.cgi?id=1252371 Bug ID] 1252371
| [https://bugzilla.mozilla.org/show_bug.cgi?id=1252371 1252371]
| Created attachment 8725097 [details] [diff] [review]
| Created attachment 8725097 [details] [diff] [review]
new-apps.patch
new-apps.patch
Line 71: Line 71:
|-
|-
| rowspan="1" |  Remove FTU app
| rowspan="1" |  Remove FTU app
| [https://bugzilla.mozilla.org/show_bug.cgi?id=1252427 Bug ID] 1252427
| [https://bugzilla.mozilla.org/show_bug.cgi?id=1252427 1252427]
| Keeping the FTU app around and maintain it during the transition period isn't a high priority. We should move the code and tests to a separate repository so it doesn't get lost. We might want to leave the 'hooks' to create a custom FTU app in the system app but we won't maintain ours going forward. This also includes the tutorial parts.
| Keeping the FTU app around and maintain it during the transition period isn't a high priority. We should move the code and tests to a separate repository so it doesn't get lost. We might want to leave the 'hooks' to create a custom FTU app in the system app but we won't maintain ours going forward. This also includes the tutorial parts.
My assumption is that users don't use any functionality since they can do everything in the settings app afterwards.
My assumption is that users don't use any functionality since they can do everything in the settings app afterwards.
Line 80: Line 80:
|-
|-
| rowspan="1" |  Remove mozPay
| rowspan="1" |  Remove mozPay
| [https://bugzilla.mozilla.org/show_bug.cgi?id=1252570 Bug ID] 1252570
| [https://bugzilla.mozilla.org/show_bug.cgi?id=1252570 1252570]
| The Marketplace is removing payments support [1], so we can get rid of all mozPay related code.
| The Marketplace is removing payments support [1], so we can get rid of all mozPay related code.


Line 87: Line 87:
|-
|-
| rowspan="1" |  Remove MobileID API  
| rowspan="1" |  Remove MobileID API  
| [https://bugzilla.mozilla.org/show_bug.cgi?id=1252573 Bug ID] 1252573
| [https://bugzilla.mozilla.org/show_bug.cgi?id=1252573 1252573]
| This API was added mostly because of Firefox Hello requirements. Unfortunately, the Firefox Hello client for FxOS didn't last long and I don't think we have any other consumer for this API. So we can probably remove it safely.
| This API was added mostly because of Firefox Hello requirements. Unfortunately, the Firefox Hello client for FxOS didn't last long and I don't think we have any other consumer for this API. So we can probably remove it safely.
| MobileID API removed
| MobileID API removed
|-
|-
| rowspan="1" |  Remove "improve Firefox OS" from settings
| rowspan="1" |  Remove "improve Firefox OS" from settings
| [https://bugzilla.mozilla.org/show_bug.cgi?id=1252773 Bug ID] 1252773
| [https://bugzilla.mozilla.org/show_bug.cgi?id=1252773 1252773]
| Remove "improve Firefox OS" from settings
| Remove "improve Firefox OS" from settings
| No reference to Firefox OS in settings
| No reference to Firefox OS in settings
|-
|-
| rowspan="1" |  Rename Firefox OS to B2G OS
| rowspan="1" |  Rename Firefox OS to B2G OS
| [https://bugzilla.mozilla.org/show_bug.cgi?id=1252776 Bug ID] 1252776
| [https://bugzilla.mozilla.org/show_bug.cgi?id=1252776 1252776]
| Rename  - settings - device information - “about Firefox OS" - to About B2G OS - including all subtext.
| Rename  - settings - device information - “about Firefox OS" - to About B2G OS - including all subtext.
| "about Firefox OS" renamed to "about B2G OS" in device information under settings
| "about Firefox OS" renamed to "about B2G OS" in device information under settings
|-
|-
| rowspan="1" |  Review all legal statements under settings
| rowspan="1" |  Review all legal statements under settings
| [https://bugzilla.mozilla.org/show_bug.cgi?id=1252778 Bug ID] 1252778
| [https://bugzilla.mozilla.org/show_bug.cgi?id=1252778 1252778]
| Review content under:
| Review content under:
  a. settings - device info - your rights
  a. settings - device info - your rights
Line 111: Line 111:
|-
|-
| rowspan="1" |  Remove Firefox OS logos from startup sequence
| rowspan="1" |  Remove Firefox OS logos from startup sequence
| [https://bugzilla.mozilla.org/show_bug.cgi?id=1252779 Bug ID] 1252779
| [https://bugzilla.mozilla.org/show_bug.cgi?id=1252779 1252779]
| Remove all logos from the device startup sequence that shows and refers to Firefox OS.
| Remove all logos from the device startup sequence that shows and refers to Firefox OS.
| No Firefox OS logos in the device startup sequence
| No Firefox OS logos in the device startup sequence
|-
|-
| rowspan="1" |  Remove current location for OTA lookup
| rowspan="1" |  Remove current location for OTA lookup
| [https://bugzilla.mozilla.org/show_bug.cgi?id=1252789 Bug ID] 1252789
| [https://bugzilla.mozilla.org/show_bug.cgi?id=1252789 1252789]
| Remove the current location pointer for OTS search but still leave the OTA option in the settings so future possible OTA hosting services can just put their URL in there for usage.
| Remove the current location pointer for OTS search but still leave the OTA option in the settings so future possible OTA hosting services can just put their URL in there for usage.
| Current location pointer for OTS search removed
| Current location pointer for OTS search removed
|-
|-
| rowspan="1" |  Integrate nginx
| rowspan="1" |  Integrate nginx
| [https://bugzilla.mozilla.org/show_bug.cgi?id=1253600 Bug ID] 1253600
| [https://bugzilla.mozilla.org/show_bug.cgi?id=1253600 1253600]
| Following bug 1252371 we are relying on an http daemon. Let's use one with a bit more features to allow future work (https, multiple domains)
| Following bug 1252371 we are relying on an http daemon. Let's use one with a bit more features to allow future work (https, multiple domains)
| Acceptance criteria to be entered
| Acceptance criteria to be entered
|-
|-
| rowspan="1" |  Build system changes for the transitioned Gaia
| rowspan="1" |  Build system changes for the transitioned Gaia
| [https://bugzilla.mozilla.org/show_bug.cgi?id=1254281 Bug ID] 1254281
| [https://bugzilla.mozilla.org/show_bug.cgi?id=1254281 1254281]
| This is to prepare for a chrome:// served Gaia. Instead of packaging each app independently in an application.zip file, we will package them all together, and put the shared/ alongside the apps.
| This is to prepare for a chrome:// served Gaia. Instead of packaging each app independently in an application.zip file, we will package them all together, and put the shared/ alongside the apps.
The directory tree will look like:
The directory tree will look like:
Line 143: Line 143:
|-
|-
| rowspan="1" |  Remove the dependency on apps status in the audiochannel code
| rowspan="1" |  Remove the dependency on apps status in the audiochannel code
| [https://bugzilla.mozilla.org/show_bug.cgi?id=1254282 Bug ID] 1254282
| [https://bugzilla.mozilla.org/show_bug.cgi?id=1254282 1254282]
| It should be chrome only now. See http://mxr.mozilla.org/mozilla-central/source/dom/system/gonk/AudioChannelManager.cpp#157 for a starting point of things to fix.
| It should be chrome only now. See http://mxr.mozilla.org/mozilla-central/source/dom/system/gonk/AudioChannelManager.cpp#157 for a starting point of things to fix.
| Acceptance criteria
| Acceptance criteria
|-
|-
| rowspan="1" |  Remove the dependency on apps status in the NFC code
| rowspan="1" |  Remove the dependency on apps status in the NFC code
| [https://bugzilla.mozilla.org/show_bug.cgi?id=1254283 Bug ID] 1254283
| [https://bugzilla.mozilla.org/show_bug.cgi?id=1254283 1254283]
| Keeping the permission is ok, but everything that is gated by AvailableIn=CertifiedApps needs to be chrome only instead.
| Keeping the permission is ok, but everything that is gated by AvailableIn=CertifiedApps needs to be chrome only instead.


Line 155: Line 155:
|-
|-
| rowspan="1" |  Remove the dependency on apps status in the Bluetooth code
| rowspan="1" |  Remove the dependency on apps status in the Bluetooth code
| [https://bugzilla.mozilla.org/show_bug.cgi?id=1254284 Bug ID] 1254284
| [https://bugzilla.mozilla.org/show_bug.cgi?id=1254284 1254284]
| Keeping the permission is ok, but everything that is gated by AvailableIn=CertifiedApps needs to be chrome only instead.
| Keeping the permission is ok, but everything that is gated by AvailableIn=CertifiedApps needs to be chrome only instead.


Line 162: Line 162:
|-
|-
| rowspan="1" |  Remove the dependency on apps status in the Downloads api code
| rowspan="1" |  Remove the dependency on apps status in the Downloads api code
| [https://bugzilla.mozilla.org/show_bug.cgi?id=1254285 Bug ID] 1254285
| [https://bugzilla.mozilla.org/show_bug.cgi?id=1254285 1254285]
| Keeping the permission is ok, but everything that is gated by AvailableIn=CertifiedApps needs to be chrome only instead.
| Keeping the permission is ok, but everything that is gated by AvailableIn=CertifiedApps needs to be chrome only instead.


Line 169: Line 169:
|-
|-
| rowspan="1" |  Remove the dependency on apps status in the RequestSync api cod
| rowspan="1" |  Remove the dependency on apps status in the RequestSync api cod
| [https://bugzilla.mozilla.org/show_bug.cgi?id=1254286 Bug ID] 1254286
| [https://bugzilla.mozilla.org/show_bug.cgi?id=1254286 1254286]
| Keeping the permission is ok, but everything that is gated by AvailableIn=CertifiedApps needs to be chrome only instead.
| Keeping the permission is ok, but everything that is gated by AvailableIn=CertifiedApps needs to be chrome only instead.


Line 176: Line 176:
|-
|-
| rowspan="1" |  Remove the dependency on apps status in the EngineeringMode api code
| rowspan="1" |  Remove the dependency on apps status in the EngineeringMode api code
| [https://bugzilla.mozilla.org/show_bug.cgi?id=1254287 Bug ID] 1254287
| [https://bugzilla.mozilla.org/show_bug.cgi?id=1254287 1254287]
| Keeping the permission is ok, but everything that is gated by AvailableIn=CertifiedApps needs to be chrome only instead.
| Keeping the permission is ok, but everything that is gated by AvailableIn=CertifiedApps needs to be chrome only instead.


Line 183: Line 183:
|-
|-
| rowspan="1" |  Remove all uses of APP_STATUS_CERTIFIED and APP_STATUS_PRIVILEGED
| rowspan="1" |  Remove all uses of APP_STATUS_CERTIFIED and APP_STATUS_PRIVILEGED
| [https://bugzilla.mozilla.org/show_bug.cgi?id=1254292 Bug ID] 1254292
| [https://bugzilla.mozilla.org/show_bug.cgi?id=1254292 1254292]
| On top of the webidl gated instances, there are direct use of these constants that we need to update:
| On top of the webidl gated instances, there are direct use of these constants that we need to update:


Line 196: Line 196:
|-
|-
| rowspan="1" |  Remove all uses of AppsService
| rowspan="1" |  Remove all uses of AppsService
| [https://bugzilla.mozilla.org/show_bug.cgi?id=1255046 Bug ID] 1255046
| [https://bugzilla.mozilla.org/show_bug.cgi?id=1255046 1255046]
| AppsService lives in dom/apps and is going to die. We have to kill its use.
| AppsService lives in dom/apps and is going to die. We have to kill its use.
https://dxr.mozilla.org/mozilla-central/search?q=nsIAppsService&redirect=false&case=true
https://dxr.mozilla.org/mozilla-central/search?q=nsIAppsService&redirect=false&case=true
Line 202: Line 202:
|-
|-
| rowspan="1" |  Move gaia to chrome:// urls
| rowspan="1" |  Move gaia to chrome:// urls
| [https://bugzilla.mozilla.org/show_bug.cgi?id=1256506 Bug ID] 1256506
| [https://bugzilla.mozilla.org/show_bug.cgi?id=1256506 1256506]
| Created attachment 8730467 [details] [diff] [review]
| Created attachment 8730467 [details] [diff] [review]
chrome-gaia.patch
chrome-gaia.patch
Line 213: Line 213:
|-
|-
| rowspan="1" |  [meta] b2g build issues
| rowspan="1" |  [meta] b2g build issues
| [https://bugzilla.mozilla.org/show_bug.cgi?id=1245091 Bug ID] 1245091
| [https://bugzilla.mozilla.org/show_bug.cgi?id=1245091 1245091]
| Fix issues preventing builds
| Fix issues preventing builds
| Closing of all dependencies: 1254604 1257793 1258294 1258672 993311 1087161 1244149 1244478 1245016 1245033 1245052 1245057 1245093 1245100 1247570 1248581 1249244 1249245 1251541 1251717 1251948 1252426 1252436 1253217 1253571 1254471 1254515 1256638 1257127 1257335 1258034 1258200 1258638
| Closing of all dependencies: 1254604 1257793 1258294 1258672 993311 1087161 1244149 1244478 1245016 1245033 1245052 1245057 1245093 1245100 1247570 1248581 1249244 1249245 1251541 1251717 1251948 1252426 1252436 1253217 1253571 1254471 1254515 1256638 1257127 1257335 1258034 1258200 1258638
|-
|-
| rowspan="1" |  Testing / Build Related Tasks
| rowspan="1" |  Testing / Build Related Tasks
| [https://bugzilla.mozilla.org/show_bug.cgi?id=1252482 Bug ID] 1252482
| [https://bugzilla.mozilla.org/show_bug.cgi?id=1252482 1252482]
| Meta for tracking build and test related tasks for fxos sunset program
| Meta for tracking build and test related tasks for fxos sunset program
| Bug dependencies: 1252491, 1254501, 1254504, 1254506, 1252484, 1252486, 1252487, 1253002
| Bug dependencies: 1252491, 1254501, 1254504, 1254506, 1252484, 1252486, 1252487, 1253002

Navigation menu