Mobile/Notes/19-Feb-2014: Difference between revisions

From MozillaWiki
< Mobile‎ | Notes
Jump to navigation Jump to search
No edit summary
 
(17 intermediate revisions by 11 users not shown)
Line 12: Line 12:
== Topics for This Week ==
== Topics for This Week ==
;Hub: Refining Home.panels API {{bug|972306}}, {{bug|968573}}, {{bug|968188}}. We'll share the demo add-ons we've been working on (Pocket, Flickr, RSS) once the API stabilizes.
;Hub: Refining Home.panels API {{bug|972306}}, {{bug|968573}}, {{bug|968188}}. We'll share the demo add-ons we've been working on (Pocket, Flickr, RSS) once the API stabilizes.
;talos: What is the point?  If I can read an [https://tbpl.mozilla.org/php/getParsedLog.php?id=34917669&tree=Mozilla-Inbound&full=1 rpr log], we spent 24 minutes of infra time to extract a '''single value''' measured in [http://mxr.mozilla.org/mozilla-central/source/mobile/android/base/tests/testBrowserProviderPerf.java#204 milliseconds].
  09:14:22    INFO -  02-19 09:13:18.414 I/Robocop ( 2607): __start_report509__end_report
  09:14:22    INFO -  02-19 09:13:18.421 I/Robocop ( 2607): __startTimestamp509__endTimestamp
This is incredibly wasteful and not statistically valid!


== Friends of the Mobile Team ==
== Friends of the Mobile Team ==
Line 76: Line 83:


===WesJ===
===WesJ===
{{readonly}}
* {{Bug|946344}} - Replace GeckoEventResponder with an async callback mechanism
* {{Bug|946344}} - Replace GeckoEventResponder with an async callback mechanism
* {{Bug|911904}} - Galaxy S4 - HelperApps PageAction shows on pages that end in .html or .htm
* {{Bug|911904}} - Galaxy S4 - HelperApps PageAction shows on pages that end in .html or .htm
Line 82: Line 88:
* {{Bug|952582}} - Bookmark item is sole item in row of menu of guest-browsing
* {{Bug|952582}} - Bookmark item is sole item in row of menu of guest-browsing
* {{Bug|931900}} - nsIPromptService.prompt's default value parameter acts differently between desktop and mobile
* {{Bug|931900}} - nsIPromptService.prompt's default value parameter acts differently between desktop and mobile
* {{Bug|971939}} - File picker should use a normal intent chooser
* {{Bug|971939}} - File picker should use a normal intent chooser, {{Bug|970506}} - Break up ActivityHandlerHelper
* {{Bug|964994}} - No indication that installing an untrusted extension could be harmful
* {{Bug|964994}} - No indication that installing an untrusted extension could be harmful
* Pulled "{{Bug|942270}} - Add Quickshare buttons to Context menu" apart. Patches in {{Bug|958109}}, {{Bug|959742}}, {{Bug|973013}}, {{Bug|973036}}, and {{Bug|973045}}.
* Pulled "{{Bug|942270}} - Add Quickshare buttons to Context menu" apart. Patches in {{Bug|958109}}, {{Bug|959742}}, {{Bug|973013}}, {{Bug|973036}}, and {{Bug|973045}}.
Line 112: Line 118:
<!-- Uncomment to provide a readonly update -->
<!-- Uncomment to provide a readonly update -->
{{readonly}}
{{readonly}}
'''Last week'''
'''Last week'''
* {{bug|881240}} - Refine title bar reveal behaviour
* {{bug|881240}} - Refine title bar reveal behaviour
Line 132: Line 139:
* More reviews
* More reviews


===Sola===
===sola===
<!-- Uncomment to provide a readonly update -->
{{readonly}}
<!-- {{readonly}} -->
 
'''Last week'''
* {{bug|961238}} - Make an abstract ContentProvider class (patch was breaking tests)
* {{bug|965361}} - Show firefox watermark when you hide all panels in about:home
 
'''This week'''
* {{bug|959290}} - Make ContentProvider for Reading List (writing tests)
* {{bug|857990}} - Save/Restore scroll position for an article in Reader Mode
* {{bug|970524}} - Investigate use of APIv11 check before starting transactions


===liuche===
===liuche===
<!-- Uncomment to provide a readonly update -->
<!-- Uncomment to provide a readonly update -->
<!-- {{readonly}} -->
{{readonly}}
* PTO for the past week+, catching up on bugmail
* next:
** finish up {{bug|950698}} (search tip), {{bug|903535}} (geo strings)
** More panels
** Settings re-rewrite?


===Margaret===
===Margaret===
Line 156: Line 176:
====Past====
====Past====
*{{Bug|938824}} - Remove reflection from testSearchSuggestions
*{{Bug|938824}} - Remove reflection from testSearchSuggestions
====Present====
====Present====
*{{bug|959652}} - BrowserHealthRecorder.ensureEnvironment can be expensive during startup
*{{bug|959652}} - BrowserHealthRecorder.ensureEnvironment can be expensive during startup
Line 179: Line 198:
===RNewman===
===RNewman===
<!-- Uncomment to provide a readonly update -->
<!-- Uncomment to provide a readonly update -->
<!-- {{readonly}} -->
{{readonly}}
 
The usual: 29 blockers for The Syncs, reviews, sticking my oar into the l10n waters.
 
====Past====
{{bugpast|rnewman@mozilla.com}}
====Present====
{{bugpresent|rnewman@mozilla.com}}


===nalexander===
===nalexander===
Line 201: Line 227:
<!-- Uncomment to provide a readonly update -->
<!-- Uncomment to provide a readonly update -->
<!-- {{readonly}} -->
<!-- {{readonly}} -->
* partner stuff
* moving


===MFinkle===
===MFinkle===
Line 211: Line 239:
<!-- Uncomment to provide a readonly update -->
<!-- Uncomment to provide a readonly update -->
<!-- {{readonly}} -->
<!-- {{readonly}} -->
* Hub stuff - panels are looking good, still work to be done on the end-to-end flow
* FxA Sync - filing bugs
* MWC prep


===Arun===
===Arun===
<!-- Uncomment to provide a readonly update -->
<!-- Uncomment to provide a readonly update -->
<!-- {{readonly}} -->
<!-- {{readonly}} -->
* PTO until March 7


===Security/MGoodwin===
===Security/MGoodwin===
Line 226: Line 260:
===A-Team===
===A-Team===
<!-- Uncomment to provide a readonly update -->
<!-- Uncomment to provide a readonly update -->
<!-- {{readonly}} -->
{{readonly}}
* One Android bug in Orange Factor Top 10:
9. 40 oranges: {{bug|959804}}
  Intermittent tcheck2 | application crashed [@ libc.so + 0x11e78]


== Feature Focus==
== Feature Focus==
;Fx28, Ships March 18th
* {{mok}} {{Bug|768667}} - Use the action bar for text selection on ICS+
* {{mok}} {{bug|875750}} - Implement <input type="color">: Android widget/color picker
* {{mok}} {{Bug|852608}} - Add support for OpenSearch. Firefox Mobile supports <link rel="search">
* {{mok}} {{Bug|943475}} - Predictive lookup for awesomebar entries
* {{mok}} [[Mobile/Projects/Dynamic snippets add-on | Dynamic snippets support]] preffed off
* {{mok}} Background Thread Hang Reporting {{Bug|932865}}
* {{mok}} [[Mobile/Projects/Localization | Added locale: Estonian (et)]], {{Bug|946204}}
;Fx29, Ships April 29th
* {{mok}} [[Mobile/Projects/Toolbar Refactor| Toolbar Refactor]]
* {{mok}} [[Mobile/Projects/Localization | Added locale: Hindi(hi)]], {{Bug|960059}}
* {{mok}} Shumway Bundled in Browser but disabled, {{Bug|905668}}
* {{mok}} [[Mobile/Projects/Promo snippet for Sync | Promo snippet for Sync]]
* {{mok}} [[Mobile/Projects/Preloaded bookmark for Marketplace | Preloaded bookmark for Marketplace]]
* {{mok}} [[Mobile/Projects/Quickshare:_Multiple_quickshare_buttons| Multiple Quickshare Buttons]] - 2 Quickshare on main menu
* {{mok}} Ability to "Hide" existing panels in about:home (see [[Mobile/Projects/My List: Manage current lists in settings | the full project page]] for more details)
* {{mrisk}} [[Mobile/Projects/Dynamic snippets add-on | Dynamic snippets support]] preffed on, {{bug|862801}} ''UX Signoff: 03/04'''
* {{mrisk}} Replace throbber with progress bar {{bug|917896}}, {{bug|959776}} '''UX Signoff: 03/04'''
* {{mrisk}} [[Mobile/Projects/Telemetry probes for Fennec UI elements | Telemetry probes for Fennec UI elements]]
* {{mrisk}} [[Mobile/Projects/Localization | Added locale: Indonesian (id)]], {{Bug|521995}}
* {{mrisk}} [[Mobile/Projects/Localization | Added locale: Latvian (lv) ]], {{Bug|958021}}
* {{mrisk}} [[Mobile/Projects/Application management | APK Synthesis on by default]]
* {{mrisk}} [[Mobile/Projects/Promo snippet for Marketplace | Promo snippet for Marketplace]] ''UX Signoff: 03/04'''
* {{mrisk}} [[Mobile/Projects/Firefox Accounts with Sync 1.1 integration | Firefox Accounts with Sync 1.5 integration]] ''UX Signoff: 03/04'''
* {{mprop}} Synced tabs page as entry point into Firefox Accounts & Sync {{Bug|958889}} (nice to have, shift to 30 if needed)
;Fx30, Ships on June 10th
* {{mrisk}} Ability to re-order existing panels in about:home (see [[Mobile/Projects/My List: Manage current lists in settings | My List: Manage current lists in settings]]), {{Bug|942231}}
* {{mok}} Evaluate pngquant for PNG size reduction {{bug|959833}}
* {{mrisk}} [[Mobile/Projects/Third-party service integration MVP | Third-party service integration MVP]]
** {{mrisk}} [[Mobile/Projects/My List: How My List works | My List: How My List works]] (MVP reqs only)
** {{mrisk}} [[Mobile/Projects/My List: Save any web-viewable content | My List: Save any web-viewable content]] (MVP reqs only)
** {{mrisk}} [[Mobile/Projects/My List: Add new My List from Promotional Panel | My List: Add new My List from Promotional Panel]] (MVP reqs only)
** {{mrisk}} [[Mobile/Projects/My List: Subscribe to feed or service as a new list | My List: Subscribe to feed or service as a new list]] (MVP reqs only)
** {{mrisk}} [[Mobile/Projects/My List: Add new and improved list management in Settings | My List: Add new and improved list management in Settings]] (MVP reqs only)
** {{mrisk}} [[Mobile/Projects/Service integration: Downloads | Service integration: Downloads]] (MVP reqs only)
* {{mrisk}} [[Mobile/Projects/Accept language-header | Preferred web content language]]
* {{mrisk}} [[Mobile/Projects/Quickshare:_Contexts| Quickshare Contexts]]
* {{mrisk}} [[Mobile/Projects/Quickshare:_Context_menu | Quickshare on the context menu]]
* {{mprop}} Placeholder for FxA/Sync Backlog
* {{mprop}} [[Mobile/Projects/Tab_streaming | WebRTC tab streaming]]
* {{mprop}} [[Mobile/Projects/API: Contacts | API: Contacts]] (incl permissions bump)
* {{mprop}} [[Mobile/Projects/Geolocation services | Geolocation services]] (incl permissions bump)
* {{mprop}} [[Mobile/Projects/Localization | Added locale: Argentine Spanish (es-AR)]]
* {{mprop}} [[Mobile/Projects/Localization | Added locale: Mexican Spanish (es-MX)]]
* {{mprop}} [[Mobile/Projects/Localization | Added locale: Malay (ms)]]


== Round Table ==
== Round Table ==

Latest revision as of 18:04, 19 February 2014

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: 2014-03-17

Topics for This Week

Hub
Refining Home.panels API bug 972306, bug 968573, bug 968188. We'll share the demo add-ons we've been working on (Pocket, Flickr, RSS) once the API stabilizes.
talos
What is the point? If I can read an rpr log, we spent 24 minutes of infra time to extract a single value measured in milliseconds.
 09:14:22     INFO -  02-19 09:13:18.414 I/Robocop ( 2607): __start_report509__end_report
 09:14:22     INFO -  02-19 09:13:18.421 I/Robocop ( 2607): __startTimestamp509__endTimestamp

This is incredibly wasteful and not statistically valid!

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.

  • p0ff fixed bug 968908 - PromptInputs should return an Object not a String (first patch!)
  • Madushan Nishantha fixed bug 948896 - Move inline scripts and styles into separate file for about:downloads (first patch!)
  • Robin Ricard fixed bug 958111 - "Find in Page" on Android should zoom in on the highlighted result (first patch!)
  • Peiyong Lin fixed bug 968378 - Hide private members in Home.jsm using closures
  • Sebastian Kaspari fixed bug 944142 - Implement NavigationHelper.goForward for phones

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>

  • STILL trying to land bug 939276 - [SkiaGL] [Feature] Support multiple Skia DrawTargets backed by a single GLContext
  • Helped some with the skia rebase bug 910754
  • Working on turning on SkiaGL for Mac.
  • Investigating several other bugs. 788522, 622816, 957276

Chris Lord (cwiiis)

JChen

<Read Only> (appointment :/ )

Past week

  • Continued to work on IME tests
  • BHR improvements
    • Reducing data size
    • Fixing hang times plot
  • Interviews & PTO
Fixed
Working on


GCP

<Read Only>

  • Last week:
    • WebRTC workweek
  • This week:
    • Code reviews. (VP8 HW accel, SafeBrowsing)
    • Load Adaption. Patches sortof finished, starting testing now.

Randall Barker

Brian Nicholson

<Read Only>

  • Band-aid fix for bug 970081 - crash in android.content.res.Resources$NotFoundException
  • bug 973021 - Remove m.foo.com -> foo.com RDS hack
  • Landed bug 970719 - Animate progress bar to end of the screen
  • Lots of reviews

WesJ

jdover

<Read Only>

WIP:

  • bug 776027 - Web Activities, will be coming up with conversions to intents based on Gaia
  • bug 961523 - Refine HomeBanner close button behavior

Finishing:

  • bug 942295 - Filters in hub panels
  • bug 960359 - Going back to hold HomeBanner design, making sure not nagging happens
  • bug 921557 - Shrink HomeBanner text on smaller screens

Investigating:

  • bug 816318 - Hacking into Froyo's internal DownloadService will probably not work (no device to test on?)
  • bug 966491 - Google Phone search has changed in KitKat, now requires "App Indexing"

Done:

  • bug 966489 - Google Phone Search listing displays browser package name
  • bug 972879 - Crash on < ICS in Reader Mode regression

LucasR

<Read Only>

Last week

  • bug 881240 - Refine title bar reveal behaviour
  • bug 968188 - Pass a 'reason' to the Home.panels.add() and Home.panels.remove() APIs
  • bug 966022 - Implement the notion of itemType on existing panel views (list, grid)
  • bug 968573 - Refine interaction of clicks on dynamic panel items
  • bug 972046 - Rename home_empty*page.xml layouts to home_empty*panel.xml
  • bug 970247 - Stub ButtonToast view
  • bug 970700 - Two line row description ellipsizes in the middle
  • bug 973137 - Images in list panel disappear when it is opened in a new tab
  • bug 973666 - Get rid of TwoLineRow view
  • bug 973667 - Get rid of the HomeGridView style
  • bug 973670 - PanelGridView needs better pressed state
  • (In progress) bug 972306 - Rename Home.panels.add/remove to Home.panels.register/unregister
  • (In progress) bug 968172 - Avoid refreshing HomeConfig on every onLocaleReady() call
  • Tons of reviews

Next week

  • More Hub stuff
  • Kick-off layout optimization work
  • More reviews

sola

<Read Only>

Last week

  • bug 961238 - Make an abstract ContentProvider class (patch was breaking tests)
  • bug 965361 - Show firefox watermark when you hide all panels in about:home

This week

  • bug 959290 - Make ContentProvider for Reading List (writing tests)
  • bug 857990 - Save/Restore scroll position for an article in Reader Mode
  • bug 970524 - Investigate use of APIv11 check before starting transactions

liuche

<Read Only>

  • PTO for the past week+, catching up on bugmail
  • next:

Margaret

Highlights

  • Landed home banner tests - bug 935264
  • Re-enabled snippets - bug 962349
  • Working with jdover on home banner refinements

Fixed

Working on


mcomella

<Read Only>

Past

  • bug 938824 - Remove reflection from testSearchSuggestions

Present

  • bug 959652 - BrowserHealthRecorder.ensureEnvironment can be expensive during startup
    • bug 915312 - Ship minimal PBKDF2-SHA256 native library for Android
  • Herding bug 941155 (fix talos on mobile to have pageloader tests working in regular mode and nochrome mode) along
    • bug 969725 - r? - geckoview_example uses incorrect main.xml layout and AndroidManifest
  • bug 942283 - HomeList RSS addon
  • bug 939350 - Create Python linter wrapper

Future

esawin

<Read Only>

Past

Present

Also working on

  • bug 611556 - Zoom levels should persist within a session

RNewman

<Read Only>

The usual: 29 blockers for The Syncs, reviews, sticking my oar into the l10n waters.

Past

Present


nalexander

  • Eclipse support finally landed!
  • Planning to bit rot you all by landing bug 938994 this week.
  • Posted work in progress on a Fennec JUnit 3 test suite bug 903528. Plan of record is to land test suite without TBPL support and encourage uptake by demonstrating how much easier it is to experiment in an instrumentation test that is locally run than in Fennec itself. This is much more convincing when developing in an IDE.
  • Mentoring mcomella with bug 915312, a tricky mix of native code and build system hijinkery.
  • Re-writing Firefox Account status activity to look like GeckoPreferences. I'm using the following small library to ease the PreferenceFragment on >= v11 vs. PreferenceActivity battle on < v11; we'll see how that goes.

BLassey

  • partner stuff
  • moving

MFinkle

  • MWC preparations
  • MWC next week

Ian Barlow

  • Hub stuff - panels are looking good, still work to be done on the end-to-end flow
  • FxA Sync - filing bugs
  • MWC prep

Arun

  • PTO until March 7

Security/MGoodwin

WebRT

A-Team

<Read Only>

  • One Android bug in Orange Factor Top 10:

9. 40 oranges: bug 959804

 Intermittent tcheck2 | application crashed [@ libc.so + 0x11e78]

Feature Focus

Fx28, Ships March 18th
Fx29, Ships April 29th
Fx30, Ships on June 10th

Round Table