Mobile/Notes/04-Feb-2015: Difference between revisions
(→Stefan) |
(→Robin) |
||
(25 intermediate revisions by 17 users not shown) | |||
Line 8: | Line 8: | ||
== Schedule == | == Schedule == | ||
* Next merge: 2015-02-23 | |||
== Topics for This Week == | == Topics for This Week == | ||
Line 38: | Line 39: | ||
== Friends of the Mobile Team == | == Friends of the Mobile Team == | ||
''Give a shoutout/thanks to people for helping fix and test bugs. Make sure friends also get awarded a [https://badges.mozilla.org/en-US/badges/badge/Friends-of-the-Mobile-Team badge]. '''New contributors''' are highlighted in '''bold'''.'' | ''Give a shoutout/thanks to people for helping fix and test bugs. Make sure friends also get awarded a [https://badges.mozilla.org/en-US/badges/badge/Friends-of-the-Mobile-Team badge]. '''New contributors''' are highlighted in '''bold'''.'' | ||
* '''Andy Pusch''' fixed {{bug|1125529}} - Remove unused string in StringHelper class | |||
* Robin Ricard fixed {{bug|1015395}} - Make testFindInPage more robust | |||
* Dominique fixed {{bug|1127909}} - Nits in ZoomedView code | |||
* Federico fixed {{bug|1114499}} - NPE when opening the popup menu when tabs are exposed | |||
* (iOS) '''codestergit''' fixed {{Bug|1109666}} - Customize behavior when long-pressing a link | |||
* vivek!!! | |||
** landed "magic year" COPPA changes in native Firefox Accounts sign up flow {{bug|1058806}} | |||
** also landed split plane Remote Tabs home panel {{bug|1063742}} | |||
** these are highly non-trivial functional changes -- bravo! | |||
* psd contributed a fix for |mach bootstrap| on Linux | |||
==Stand ups== | ==Stand ups== | ||
Line 48: | Line 59: | ||
<!-- Uncomment to provide a readonly update --> | <!-- Uncomment to provide a readonly update --> | ||
{{readonly}} | {{readonly}} | ||
{{bugpast|snorp@snorp.net}} | |||
<!-- ==== Past week ==== --> | <!-- ==== Past week ==== --> | ||
<!-- * <worked on> --> | <!-- * <worked on> --> | ||
Line 67: | Line 80: | ||
===GCP=== | ===GCP=== | ||
<!-- Uncomment to provide a readonly update --> | <!-- Uncomment to provide a readonly update --> | ||
* Last week | |||
** Finished Sandboxing support for Window/Application/Screen sharing | |||
** {{Bug|1104616}} Sandboxing support for Video camera access | |||
** SafeBrowsing discussions | |||
* Next week | |||
** {{Bug|659188}} - Camera support for Android (Clean up) | |||
** {{Bug|1129365}} - getUserMedia Video/Camera broken on Samsung Galaxy S4 | |||
** {{Bug|1125048}} Slow fling speed with thumb flings (high DPI) | |||
** Sandboxing support for Audio | |||
===Randall Barker=== | ===Randall Barker=== | ||
<!-- Uncomment to provide a readonly update --> | <!-- Uncomment to provide a readonly update --> | ||
{{readonly}} | |||
Last Week | |||
* Fixed texture unpack GL bug in desktop and Roku WebRTC Player | |||
* Slayed try demons. | |||
* Need r+ from two people on patch part 2 of {{bug|1101651}}. All other patches have been r+ for standalone WebRTC. | |||
** {{bug|1093934}} Create a XPCOM library that can be used to support standalone WebRTC | |||
** {{bug|1097804}} Create a library containing nsISocketTransportService and nsIDNS that can be used to support standalone WebRTC | |||
** {{bug|1101651}} Enable WebRTC unit tests to be built using standalone WebRTC library | |||
Next Week | |||
* Investigate {{bug|1121679}} Enable receiving audio in standalone WebRTC | |||
* Land standalone WebRTC if we decided to go forward with the project. | |||
===Eugen Sawin=== | ===Eugen Sawin=== | ||
{{readonly}} | {{readonly}} | ||
''' | '''Finished''' | ||
{{ | * {{Bug|792992}} - Can't override query URL for update service | ||
'''Working on''' | '''Working on''' | ||
{{ | * {{Bug|1093815}} - Use AndroidPlatformDecoder for standalone MP3 on Android | ||
-- | * Looking into regressions | ||
** {{Bug|792992}} - Can't override query URL for update service | |||
** {{Bug|1129074}} - Content view sometimes blank on load until restart | |||
===Brian Nicholson=== | ===Brian Nicholson=== | ||
{{readonly}} | {{readonly}} | ||
* {{Bug|1128774}} - Reader Mode fails hard on initial Gmail loading | |||
* {{Bug|1127989}} - Fixes to BackForwardListViewController | |||
* {{Bug|1127547}} - Follow-up fixes for custom context menu | |||
* Spent some time looking into slow Gmail redirects ({{bug|1129087}}). No dice. | |||
* Investigating UI automation frameworks | |||
===WesJ=== | ===WesJ=== | ||
<!-- Uncomment to provide a readonly update --> | <!-- Uncomment to provide a readonly update --> | ||
* {{Bug|1123809}} - Record history as visits | |||
* {{Bug|1128363}} - Table versioning should be per-table | |||
* {{Bug|1128595}} - Sqlite db for favicons | |||
* {{Bug|1129306}} - Passwords | |||
* Working on theming tabs on iOS | |||
===liuche=== | ===liuche=== | ||
Line 96: | Line 139: | ||
Highlights: | Highlights: | ||
* Passwords work day | |||
** Triaged Mobile password bugs into Q1-metabug | |||
** Discussed multi-realm logins, recipes (see [https://mail.mozilla.org/pipermail/passwords-dev/2015-February/thread.html passwords-dev] mailing list for more detail) | |||
** Telemetry | |||
Present: | Present: | ||
Line 107: | Line 154: | ||
Highlights: | Highlights: | ||
* All things reader mode/reading list | |||
* Gave a talk at FOSDEM: http://leibovic.github.io/privacy-fxandroid/ | |||
Past: | Past: | ||
{{bugpast|margaret.leibovic@gmail.com}} | {{bugpast|margaret.leibovic@gmail.com}} | ||
Line 116: | Line 164: | ||
{{readonly}} | {{readonly}} | ||
====Past==== | ====Past==== | ||
{{ | *New tablet | ||
**{{Bug|1127154}} - Enabling LWT changes the pressed/focused/etc. color of the navigation buttons | |||
*investigate {{Bug|1127599}} - Crash when pressing TabsPanel menu button | |||
*investigate {{Bug|1014603}} - crash in java.lang.NullPointerException: at org.mozilla.gecko.db.TopSitesCursorWrapper.moveToPosition(TopSitesCursorWrapper.java) | |||
*{{bug|1096958}} - crash in java.lang.NullPointerException: at org.mozilla.gecko.home.TopSitesPanel.access$N(TopSitesPanel.java) | |||
* {{bug|1090287}} - java.lang.NullPointerException: at org.mozilla.gecko.toolbar.BrowserToolbar.updateProgressVisibility(BrowserToolbar.java:626) | |||
====Present==== | ====Present==== | ||
{{ | *{{Bug|939350}} - Create a python linter wrapper | ||
*{{bug|1106935}} - Remove old tablet code and resources | |||
**{{bug|1072646}} - Use resource alias to remove branch in Favicons.initializeWithContext | |||
*Implement share overlay visual design ({{bug|1059554}}) | |||
====Future==== | ====Future==== | ||
*(Q1 goals) Toolbar cleanup & awesomescreen refresh | |||
===rnewman=== | ===rnewman=== | ||
Line 181: | Line 239: | ||
Also: | Also: | ||
* iOS builds from https://xcode.sateh.com seem to not work anymore on iOS 8.1.3. SImply refuses to install. Not sure how to fix, but we will have dogfooding builds Real Soon Now. | |||
* We setup a repository to host our shared Readability.js code - Going to merge iOS changes back in there. Will be a central point, also for contributors to fix Readability specific bugs. | * We setup a repository to host our shared Readability.js code - Going to merge iOS changes back in there. Will be a central point, also for contributors to fix Readability specific bugs. | ||
Line 203: | Line 262: | ||
===James Hugman=== | ===James Hugman=== | ||
<!-- Uncomment to provide a readonly update --> | <!-- Uncomment to provide a readonly update --> | ||
{{readonly}} | |||
Moved to the Firefox for iOS team! | |||
Ramping up on Swift & the codebase. | |||
[https://bugzilla.mozilla.org/show_bug.cgi?id=778588 Voice integration on Fennec]: nit reviews | |||
=== | ===Allison=== | ||
<!-- Uncomment to provide a readonly update --> | <!-- Uncomment to provide a readonly update --> | ||
<!-- {{readonly}} --> | <!-- {{readonly}} --> | ||
* Android Bugs | |||
** Bug 1101746 - Display message when no passwords are present | |||
*** unblocked | |||
** Bug 1124711 - Site identity popup overlaps the URL Bar on phone | |||
*** on round 2 | |||
** Bug 1127237 - Text from history panel is not entirely displayed | |||
*** in review | |||
*** mfinkle: "layout issues are fixed with experience + trial & error. | |||
in lieu of experience, add more trial & error" | |||
* Passwords | |||
* Bug 1025703 - Ignore autocomplete="off" for filling login forms | |||
** passwords team had multiple lonoong discussions about this one | |||
** finally settled on what the way forward looks like | |||
** new variant looks ok on try (minus the tests) | |||
* Date Safety & Privacy | |||
** International Privacy Day | |||
** finished blog series on applying privacy | |||
*** extension/application of my talk last year | |||
** Data Collection Module | |||
*** (still) reviewing Desktop FHR/telemetry v4+AWS plan | |||
*** (still) reviewing Query Modes for FHR+Telemetry v4 | |||
*** module will meet later this month | |||
===BLassey=== | ===BLassey=== | ||
Line 243: | Line 333: | ||
<!-- Uncomment to provide a readonly update --> | <!-- Uncomment to provide a readonly update --> | ||
<!-- {{readonly}} --> | <!-- {{readonly}} --> | ||
iOS | |||
* Reading List Prototype 1 http://people.mozilla.org/~randersen/readinglist-prototype/ | |||
* Reading List Prototype 2 http://people.mozilla.org/~randersen/readinglist-prototype-b/ (where we're going) | |||
—use Chrome (Framer + Fx don't play well together), emulate iPhone 6 | |||
* Tab Exploration http://cl.ly/Zcpb - about to prototype | |||
* Settings + Profile flows | |||
* Null states for all (including signed-out) | |||
===Darrin=== | ===Darrin=== | ||
<!-- Uncomment to provide a readonly update --> | <!-- Uncomment to provide a readonly update --> | ||
{{readonly}} | |||
* More iOS progress, v1 UX overview here http://invis.io/HA254M642 | |||
* feel free to comment, question, etc. within InVision | |||
* if you are blocked by UX, please let me know and we'll re-juggle to get you what you need! | |||
===Security/MGoodwin=== | ===Security/MGoodwin=== | ||
Line 258: | Line 359: | ||
===A-Team=== | ===A-Team=== | ||
<!-- Uncomment to provide a readonly update --> | <!-- Uncomment to provide a readonly update --> | ||
{{readonly}} | |||
* Be aware of {{bug|1127916}} -- all Android x86 tests are now hidden. | |||
===QA=== | ===QA=== | ||
Line 270: | Line 372: | ||
== Feature Focus== | == Feature Focus== | ||
* [https://trello.com/b/LOT5QEMp/fennec-tracking-master-list Mobile Roadmap Trello Board] | * [https://trello.com/b/LOT5QEMp/fennec-tracking-master-list Mobile Roadmap Trello Board] | ||
** Funnel review moved to next week to avoid conflicts and holidays | |||
* [https://mozilla.aha.io/products/IOS/feature_cards iOS Roadmap in Aha!] | |||
* [https://trello.com/b/ytKprCHH/firefox-for-ios-aha-linked New iOS Trello Board] | |||
** Trello board is linked to Aha! Roadmap Jenn to explain how this works :-) | |||
== Round Table == | == Round Table == |
Latest revision as of 17:50, 4 February 2015
Details
- Wednesdays - 9:30am Pacific, 12:30pm Eastern, 16:30 UTC
- Dial-in: Audio-only conference# 99998
- People with Mozilla phones or softphones please dial x4000 Conf# 99998
- US/Toll-free: +1 800 707 2533, (pin 4000) Conf# 99998
- US/California/Mountain View: +1 650 903 0800, x4000 Conf# 99998
- US/California/San Francisco: +1 415 762 5700, x4000 Conf# 99998
- US/Oregon/Portland: +1 971 544 8000, x4000 Conf# 99998
- CA/British Columbia/Vancouver: +1 778 785 1540, x4000 Conf# 99998
- CA/Ontario/Toronto: +1 416 848 3114, x4000 Conf# 99998
- UK/London: +44 (0)207 855 3000, x4000 Conf# 99998
- FR/Paris: +33 1 84 88 37 37, x4000 Conf# 99998
- Gmail Chat (requires Flash and the Google Talk plugin): paste +1 650 903 0800 into the Gmail Chat box that doesn't look like it accepts phone numbers
- SkypeOut is free if you use the 800 number
- irc.mozilla.org #mobile for backchannel
- Mobile Vidyo Room
Schedule
- Next merge: 2015-02-23
Topics for This Week
Tracking Review
Beta
- Next Build:
No results.
0 Total; 0 Open (0%); 0 Resolved (0%); 0 Verified (0%);
Aurora
- Next Build:
No results.
0 Total; 0 Open (0%); 0 Resolved (0%); 0 Verified (0%);
Friends of the Mobile Team
Give a shoutout/thanks to people for helping fix and test bugs. Make sure friends also get awarded a badge. New contributors are highlighted in bold.
- Andy Pusch fixed bug 1125529 - Remove unused string in StringHelper class
- Robin Ricard fixed bug 1015395 - Make testFindInPage more robust
- Dominique fixed bug 1127909 - Nits in ZoomedView code
- Federico fixed bug 1114499 - NPE when opening the popup menu when tabs are exposed
- (iOS) codestergit fixed bug 1109666 - Customize behavior when long-pressing a link
- vivek!!!
- landed "magic year" COPPA changes in native Firefox Accounts sign up flow bug 1058806
- also landed split plane Remote Tabs home panel bug 1063742
- these are highly non-trivial functional changes -- bravo!
- psd contributed a fix for |mach bootstrap| on Linux
Stand ups
Suggested format:
- What did you do last week?
- What are working on this week?
- Anything blocking you?
Please keep your update to under 2 minutes!
James W. (snorp)
<Read Only>
- 1062758 - crash in nsObserverService::RemoveObserver(nsIObserver*, char const*)
- 1108709 - Android crash in libOMX_Core.so@0x1a40 on Mali-400 MP and some PowerVR SGX 540 adapters
- 1123084 - crash in mozilla::layers::SharedSurfaceTextureHost::Lock()
- 1126454 - java.lang.RuntimeException: Unable to get provider org.mozilla.gecko.db.PasswordsProvider: java.lang.NullPointerException
- 1116469 - Video trailers are playing upside down on imdb.com on the Sony Xperia Z2
- 1121647 - Screen turns off by timeout while playing flash video
JChen
<Read Only> (on leave)
GCP
- Last week
- Finished Sandboxing support for Window/Application/Screen sharing
- bug 1104616 Sandboxing support for Video camera access
- SafeBrowsing discussions
- Next week
- bug 659188 - Camera support for Android (Clean up)
- bug 1129365 - getUserMedia Video/Camera broken on Samsung Galaxy S4
- bug 1125048 Slow fling speed with thumb flings (high DPI)
- Sandboxing support for Audio
Randall Barker
<Read Only>
Last Week
- Fixed texture unpack GL bug in desktop and Roku WebRTC Player
- Slayed try demons.
- Need r+ from two people on patch part 2 of bug 1101651. All other patches have been r+ for standalone WebRTC.
- bug 1093934 Create a XPCOM library that can be used to support standalone WebRTC
- bug 1097804 Create a library containing nsISocketTransportService and nsIDNS that can be used to support standalone WebRTC
- bug 1101651 Enable WebRTC unit tests to be built using standalone WebRTC library
Next Week
- Investigate bug 1121679 Enable receiving audio in standalone WebRTC
- Land standalone WebRTC if we decided to go forward with the project.
Eugen Sawin
<Read Only>
Finished
- bug 792992 - Can't override query URL for update service
Working on
- bug 1093815 - Use AndroidPlatformDecoder for standalone MP3 on Android
- Looking into regressions
- bug 792992 - Can't override query URL for update service
- bug 1129074 - Content view sometimes blank on load until restart
Brian Nicholson
<Read Only>
- bug 1128774 - Reader Mode fails hard on initial Gmail loading
- bug 1127989 - Fixes to BackForwardListViewController
- bug 1127547 - Follow-up fixes for custom context menu
- Spent some time looking into slow Gmail redirects (bug 1129087). No dice.
- Investigating UI automation frameworks
WesJ
- bug 1123809 - Record history as visits
- bug 1128363 - Table versioning should be per-table
- bug 1128595 - Sqlite db for favicons
- bug 1129306 - Passwords
- Working on theming tabs on iOS
liuche
<Read Only>
Highlights:
- Passwords work day
- Triaged Mobile password bugs into Q1-metabug
- Discussed multi-realm logins, recipes (see passwords-dev mailing list for more detail)
- Telemetry
Present:
Past:
Margaret
<Read Only>
Highlights:
- All things reader mode/reading list
- Gave a talk at FOSDEM: http://leibovic.github.io/privacy-fxandroid/
Past:
- 1089653 - crash in java.lang.NullPointerException: at org.mozilla.search.SearchActivity.updateSettingsButtonVisibility(SearchActivity.java)
- 1117258 - Implement CSS styling for desktop about:reader content
- 1123471 - Create basic browser-chrome test for reader mode toolbar button
- 1125364 - TypeError: content is null in AboutReaderListener
- 1125396 - TypeError: message.target.messageManager is undefined in ReaderParent
Present:
mcomella
<Read Only>
Past
- New tablet
- bug 1127154 - Enabling LWT changes the pressed/focused/etc. color of the navigation buttons
- investigate bug 1127599 - Crash when pressing TabsPanel menu button
- investigate bug 1014603 - crash in java.lang.NullPointerException: at org.mozilla.gecko.db.TopSitesCursorWrapper.moveToPosition(TopSitesCursorWrapper.java)
- bug 1096958 - crash in java.lang.NullPointerException: at org.mozilla.gecko.home.TopSitesPanel.access$N(TopSitesPanel.java)
- bug 1090287 - java.lang.NullPointerException: at org.mozilla.gecko.toolbar.BrowserToolbar.updateProgressVisibility(BrowserToolbar.java:626)
Present
- bug 939350 - Create a python linter wrapper
- bug 1106935 - Remove old tablet code and resources
- bug 1072646 - Use resource alias to remove branch in Favicons.initializeWithContext
- Implement share overlay visual design (bug 1059554)
Future
- (Q1 goals) Toolbar cleanup & awesomescreen refresh
rnewman
<Read Only>
(Might be missing the meeting for an urgent errand. Apologies.)
Reading List
- Met with desktop team re Reading List storage and synchronization.
- Multiple RL meetings, legal discussion, protocol discussion.
- Estimated load for pre-release and release Android RL clients.
- Part-way through building Android RL client, filing bugs as I go. Here's my work.
Misc
- OTA distributions uplifted to Beta. Should be in partner hands this week?
- Several phone screens. Intern candidates are on their way.
- Met with Winston and Liz re Fennec Android marketing measurement. Follow-up research and discussion.
- Miscellaneous fixes and reviews, as always.
Fixed
Working on
nalexander
Projects for this cycle:
- Pushed migrating legacy Sync 1.1 devices across the finish line
- Testing starts next week!
- Make FxA provide OAuth implicit grants for Fennec consumers, tracked by bug 1117829
- Implement Firefox Accounts for iOS
- Tracked by bug iosfxa
- Landed embedded accounts.firefox.com bug 1126584
- Working on persisting and exposing Firefox Account status bug 1129158, bug 1129161
- |mach bootstrap| for mobile/android should be good for OS X and Linux now
- Building with Gradle
- Green build, green try run with debug APK in automation
- I am building locally without compiling Gecko, but I'm a good ways off landing something like this
Past
- 1125330 - Expose |mach| output when internal Gradle mach invocations fail
- 1123013 - IntelliJ needs Project SDK set due to dynamic code in android.gradle
- 1122829 - Tighten vertical spacing of Android Firefox Accounts sign up/sign flow
- 1112725 - [android] "Next" button on Sync setup has blue line
- 1123824 - |mach bootstrap| suggests a mozconfig with bad --with-android-sdk on Linux
- 1126584 - Support signing in to Firefox Account via embedding accounts.firefox.com
Present
Martyn Haigh
Past:
Present:
Stefan
Current focus:
- 1124489 Get tests going on AppThwack
- 1129483 Setup build configurations for Release, Dogfooding, Development
Also:
- iOS builds from https://xcode.sateh.com seem to not work anymore on iOS 8.1.3. SImply refuses to install. Not sure how to fix, but we will have dogfooding builds Real Soon Now.
- We setup a repository to host our shared Readability.js code - Going to merge iOS changes back in there. Will be a central point, also for contributors to fix Readability specific bugs.
Past reviews:
- 1128774 Reader Mode fails hard on initial Gmail loading
- 1128595 Sqlite db for favicons
- 1127989 Fixes to BackForwardListViewController
- 1127505 Linking against dylib not safe for use in application extensions: Storage.framework
- 1127547 Follow-up fixes for custom context menu
- 1123809 Record history as visits
Past bugs:
- 1124500 Render/readerizing issues (webkit incompatibilty / gecko specific issues)
- 1125849 Reader Mode button is not right aligned after device orientation change
- 1126484 Reader Mode should use the right fonts
- 1127331 Include a local web server to serve local assets
- 1126488 Missing byline
- 1125960 Reader Mode webview width is not correct after device rotation
James Hugman
<Read Only>
Moved to the Firefox for iOS team!
Ramping up on Swift & the codebase.
Voice integration on Fennec: nit reviews
Allison
- Android Bugs
- Bug 1101746 - Display message when no passwords are present
- unblocked
- Bug 1124711 - Site identity popup overlaps the URL Bar on phone
- on round 2
- Bug 1127237 - Text from history panel is not entirely displayed
- in review
- mfinkle: "layout issues are fixed with experience + trial & error.
- Bug 1101746 - Display message when no passwords are present
in lieu of experience, add more trial & error"
- Passwords
- Bug 1025703 - Ignore autocomplete="off" for filling login forms
- passwords team had multiple lonoong discussions about this one
- finally settled on what the way forward looks like
- new variant looks ok on try (minus the tests)
- Date Safety & Privacy
- International Privacy Day
- finished blog series on applying privacy
- extension/application of my talk last year
- Data Collection Module
- (still) reviewing Desktop FHR/telemetry v4+AWS plan
- (still) reviewing Query Modes for FHR+Telemetry v4
- module will meet later this month
BLassey
Fixed
Working on
MFinkle
Fixed
Working on
Antlam
<Read Only>
- Past
- Passwords work week
- Tablet UI blog post! https://medium.com/@antlam/firefox-for-android-on-tablets-f67edc83dd46
- bug 1122302: Show synced devices earlier in share menu
- bug 1058818: Clean up doorhangers UI (mocks, finally!)
- Upcoming
- bug 1114708: Reader mode transition mocks
- bug 1059554: Share overlay UI with Mcomella
- Open-in-background work with Martyn
- MCEConf
Robin
iOS
- Reading List Prototype 1 http://people.mozilla.org/~randersen/readinglist-prototype/
- Reading List Prototype 2 http://people.mozilla.org/~randersen/readinglist-prototype-b/ (where we're going)
—use Chrome (Framer + Fx don't play well together), emulate iPhone 6
- Tab Exploration http://cl.ly/Zcpb - about to prototype
- Settings + Profile flows
- Null states for all (including signed-out)
Darrin
<Read Only>
- More iOS progress, v1 UX overview here http://invis.io/HA254M642
- feel free to comment, question, etc. within InVision
- if you are blocked by UX, please let me know and we'll re-juggle to get you what you need!
Security/MGoodwin
WebRT
A-Team
<Read Only>
- Be aware of bug 1127916 -- all Android x86 tests are now hidden.
QA
Garvan
Feature Focus
- Mobile Roadmap Trello Board
- Funnel review moved to next week to avoid conflicts and holidays
- iOS Roadmap in Aha!
- New iOS Trello Board
- Trello board is linked to Aha! Roadmap Jenn to explain how this works :-)