Mobile/Notes/08-Jan-2014: Difference between revisions

 
(23 intermediate revisions by 12 users not shown)
Line 11: Line 11:


== Topics for This Week ==
== Topics for This Week ==
;Rating Dip: We have been investigating a ratings dip that appears to have started around Dec 10th 2013. Currently, we think this could be related to two main issues: 1) a [http://www.androidpolice.com/2013/12/30/multiple-developers-seeing-influx-of-unexplained-1-star-ratings-since-launch-of-play-stores-new-quick-rating-widget/ Google UX change] in the Play Store and 2) Slow performance on mid to low end devices. #1 is not really in our control, but if we can feedback to Google, we probably should. #2 is within our control and we have been working on [https://bugzilla.mozilla.org/show_bug.cgi?id=947390 improving pageload performance]. We need to make sure we include mid and low end devices when profiling/testing. Some of the data we have been gathering shows that regions, like Brazil, where we are growing, are the same regions where we are being rated poorly due to performance. Those regions also seem to be using mid to low end devices.
;Rating Dip: We have been investigating a ratings dip that appears to have started around Dec 10th 2013. Currently, we think this could be related to three main issues:  
;Pageload Performance: As mentioned, we have a lot of [https://bugzilla.mozilla.org/show_bug.cgi?id=947390 work] happening related to improving our pageload performance. Many of the bugs came from profiling, Java and Gecko. Many are small wins, but we have some decent wins related to proxy settings ({{bug|952799}} and {{bug|957037}}) and throbber ({{bug|917896}}). The throbber change is our single biggest pageload win so far.
# a [http://www.androidpolice.com/2013/12/30/multiple-developers-seeing-influx-of-unexplained-1-star-ratings-since-launch-of-play-stores-new-quick-rating-widget/ Google UX change] in the Play Store and  
# Slow performance on mid to low end devices.  
# No Flash support on KitKat
The first  is not really in our control, but if we can feedback to Google, we probably should. #2 is within our control and we have been working on [https://bugzilla.mozilla.org/show_bug.cgi?id=947390 improving pageload performance]. We need to make sure we include mid and low end devices when profiling/testing. Some of the data we have been gathering shows that regions, like Brazil, where we are growing, are the same regions where we are being rated poorly due to performance. Those regions also seem to be using mid to low end devices. The third we may have a solution for, glandium and snorp are investigating.
 
;Pageload Performance: As mentioned, we have a lot of [https://bugzilla.mozilla.org/show_bug.cgi?id=947390 work] happening related to improving our pageload performance. Many of the bugs came from profiling, Java and Gecko. Many are small wins, but we have some decent wins related to proxy settings ({{bug|952799}} and {{bug|957037}}) and throbber ({{bug|917896}}). The throbber change is our single biggest pageload win so far.  
 
http://eideticker.wrla.ch/#/samsung-gn/nytimes-load-poststartup/timetostableframe
 
; PDF.js
 
Checking in to ensure we are on track


== Friends of the Mobile Team ==
== Friends of the Mobile Team ==
Line 54: Line 65:
<!-- Uncomment to provide a readonly update -->
<!-- Uncomment to provide a readonly update -->
<!-- {{readonly}} -->
<!-- {{readonly}} -->
* Last Week(s): Investigating tab streaming. Wrestling with build systems.
* Next Week: Finish tab stream proof of concept. Get Roku NDK.


===Brian Nicholson===
===Brian Nicholson===
<!-- Uncomment to provide a readonly update -->
{{readonly}}
<!-- {{readonly}} -->
* Bug 917896 - Replace progress throbber with a progress bar
* Ongoing requestAutocomplete work


===Sriram===
===Sriram===
Line 65: Line 79:
===WesJ===
===WesJ===
<!-- Uncomment to provide a readonly update -->
<!-- Uncomment to provide a readonly update -->
<!-- {{readonly}} -->
{{readonly}}
* Still fixing actionbar bugs
* Fixing test failures on new async event dispather - Bug 946344 - Replace GeckoEventResponder with an async callback mechanism
* Lots of reviewing
* Going to pick up Bug 943568 - Replace Prompt.PromptListItem with GeckoMenuItems, to move forward quickshare 2


===LucasR===
===LucasR===
<!-- Uncomment to provide a readonly update -->
<!-- Uncomment to provide a readonly update -->
<!-- {{readonly}} -->
{{readonly}}
'''Last week'''
* Bunch of reviews
* HomeConfig backend stuff
* Preparing to land big toolbar refactoring
'''Next week'''
* More HomeConfig stuff
* Prepare talk for Android meetup
* Land toolbar refactoring


===liuche===
===liuche===
<!-- Uncomment to provide a readonly update -->
<!-- Uncomment to provide a readonly update -->
<!-- {{readonly}} -->
{{readonly}}
* Finished:
** {{Bug|910189}} - Remove default search engines
* Current:
** Getting Sola started (new intern!), reviews
** {{Bug|942875}} - Manage Home Lists in Settings


===Margaret===
===Margaret===
Line 120: Line 151:
<!-- Uncomment to provide a readonly update -->
<!-- Uncomment to provide a readonly update -->
<!-- {{readonly}} -->
<!-- {{readonly}} -->
====Stuff I don't have time for update:====
Help wanted for {{Bug|951865}}.  Build system patch has f+, pretty basic review comments.  If somebody could push it over the line, that would help GeckoView development.
If somebody cares about alternate build systems/better Android resource management/integration with IntelliJ, I pushed some rough hacks at building with Gradle (must be 1.9!) to https://github.com/ncalexan/gecko-dev/commits/nalexander/gradle
You need to build and package, and then remove $OBJDIR/mobile/android/base/generated/org/mozilla/gecko/{R,Manifest}.java to prevent build errors.  Easy to add this exclusion to the geckoview/build.gradle file.
There are hard-coded paths, all sorts of nastiness.  It was just proof-of-concept, and indeed, I was able to build a working Fennec APK using gradle.
====Real update:====
Full steam ahead on Firefox Accounts for Android.  I landed {{Bug|955808}} and {{Bug|956816}}.  Prereqs for Firefox Account + Sync login state machine.  rnewman did a heroic first review of said state machine and preliminary UX on Monday night before his PTO.  rnewman landed {{Bug|956442}} and {{Bug|956935}} for Sync migration.  Goal is to push a try build with preliminary UX to design team and QA this Friday.


===BLassey===
===BLassey===
* Been looking at making a stand alone WebRTC client
* Went to Toronto, made it home
<!-- Uncomment to provide a readonly update -->
<!-- Uncomment to provide a readonly update -->
<!-- {{readonly}} -->
<!-- {{readonly}} -->
Line 135: Line 181:


===Ian Barlow===
===Ian Barlow===
<!-- Uncomment to provide a readonly update -->
{{readonly}}
<!-- {{readonly}} -->
(sorry, have to run to another meeting)
* Trying to help out on Sync / Firefox Accounts
* Kicking off some tablet brainstorming tomorrow/friday
* Ongoing 3rd party service work on the Homepage
* Other misc UI refinements, including action bars and page progress indicators.


===Arun===
===Arun===
<!-- Uncomment to provide a readonly update -->
<!-- Uncomment to provide a readonly update -->
<!-- {{readonly}} -->
{{readonly}}
 
* Form auto-fill: Working on security aspects & polishing rest of the designs (slides available end of week)
* Next: Kick-off some explorations on Tab-switching (will sync with lucasr soon)


===Security/MGoodwin===
===Security/MGoodwin===
Line 153: Line 206:
<!-- Uncomment to provide a readonly update -->
<!-- Uncomment to provide a readonly update -->
<!-- {{readonly}} -->
<!-- {{readonly}} -->
===Sola===
* Getting ramped up on the codebase
* Adding "Share" and "Add to Home Page" options for top sites in about home


== Feature Focus==
== Feature Focus==
Line 167: Line 224:
* {{mok}} [[Mobile/Projects/Localization | Added locale: Slovenian (sl)]], {{Bug|522008}}
* {{mok}} [[Mobile/Projects/Localization | Added locale: Slovenian (sl)]], {{Bug|522008}}
;Fx28,March 18, 2014
;Fx28,March 18, 2014
* {{mrisk}} {{Bug|768667}} - Use the action bar for text selection on ICS+
* {{mrisk}} {{Bug|768667}} - Use the action bar for text selection on ICS+ <== need all hands on deck: {{Bug|956782}},{{Bug|956571}},{{Bug|956075}}, {{Bug|955861}}, {{Bug|949613}}. Would like this frozen by 01/24. Thanks for {{Bug|951274}}
* {{mrisk}} [[Mobile/Projects/Localization | Added locale: Belarusian (be)]], {{Bug|521981}}
* {{mrisk}} [[Mobile/Projects/Localization | Added locale: Belarusian (be)]], {{Bug|521981}}
* {{mok}} {{Bug|852608}} - Add support for OpenSearch. Firefox Mobile supports <link rel="search">
* {{mok}} {{Bug|852608}} - Add support for OpenSearch. Firefox Mobile supports <link rel="search">
Line 205: Line 262:


== Round Table ==
== Round Table ==
===QA===
* Crash rate has gone up about .5 per 100 ADU, this represents 2k to 3k more crashes that happen during a release
* [https://bugzilla.mozilla.org/buglist.cgi?keywords=crash%2C%20&keywords_type=allwords&list_id=9121786&resolution=---&resolution=INVALID&resolution=WONTFIX&resolution=DUPLICATE&classification=Client%20Software&classification=Components&op_sys=Android&chfieldto=2014-01-09&chfield=%5BBug%20creation%5D&query_format=advanced&chfieldfrom=2013-12-25 Crashes filed since Dec 25]
* {{bug|900033}} - crash on Samsung 8" and 7" tablets - need someone in Toronto Gfx to look at Aaron's STR
* https://bugzilla.mozilla.org/showdependencytree.cgi?id=752828&hide_resolved=1 - There were some suggestions for DB locked bugs recently
* {{bug|825996}} - needs investigation, kats comment 14 is useful
Confirmed users
69

edits