Mobile/Notes/06-Nov-2013

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 is 2013-12-09

Topics for This Week

Tips, Tricks and Best Practices

We had several posts to the mailing list last week covering some good information about Android, Java and code maintenance. Sriram has a post on setImageBitmap(null) being evil. Richard walks through his investigation of a memory issue he fixed. He also started a discussion about some best practices.

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.

• Noppanit fixed bug 774696 - Default icon for about:apps (first patch!)
• Errietta fixed bug 910768 - Add a link to the marketplace in the "No apps installed for this url" toast
• WG9s fixed bug 931469 - Tapping an .apk file download link results in 2 downloads starting, one of which never completes
• Federico finished bug 909932 - A new Notifications.jsm API for add-ons
  • https://developer.mozilla.org/en-US/docs/Extensions/Mobile/API/Notifications.jsm
• darkowlzz fixed bug 920935 - Reading list items opened using keywords are not opened in Reading List
  • This bug was later marked a WONTFIX and the patch was backed out, but we still like to recognize good effort! :)
• capella fixed bug 906402 - security exception when checking signature of favicon

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!

Kats

<Read Only>

• No Fennec work recently
• Spent a lot of time discussing coordinate systems with Botond, things are starting to make sense. This will help both Metro and eventually Fennec

JChen

• Past week
  • Patch reviewed for bug 909974 - Background thread hang monitoring
  • Been working on bug 932865 - Background thread hang reporting
    • and associated bugs:
    • bug 935089 - Array util methods should support mozilla::Array
    • bug 935092 - Make last-seen peudo stack available
    • bug 896100 - Implement the C++11 rvalue reference helpers in MFBT
• Next
  • More hang reporting

GCP

• Last week:
  • WebRTC 3.43 uplift patches. Ready to land but found some x86 issues.
  • bug 932816 Unable to close the app if webrtc is being used
  • bug 933284 crash in @0x0 | audiotrack_get_max_channel_count
  • bug 932692 Crash [@ libdvm.so@0x50e96 ]
  • bug 933705 Android WebRTC deletes a shared GlobalRef
• Next week:
  • More of the same

Randall Barker

• Last week:
  • Chased crashes in IPC
  • Chased crashes when using shared memory
• This week
  • Figure out safe way to share compositor frame metrics with out crashing.

Brian N

• bug 933501 - Update SUTAgent to look for main activity rather than first one in manifest
• bug 933614 - Add support for creating a GeckoView library project using mach
• bug 911364 - New GeckoView instances aren't connected to Gecko when Activity is recreated
• Trying to figure out a good way to handle GeckoView lifecycle
• Reviews

Sriram

• bug 932967 - LightweightTheme cleanup
• bug 934001 - ImageView's setImageBitmap is evil
• bug 933459 - Reduce unnecessary drawables created by FaviconView
• bug 934658 - Investigated and fixed non-rectangular clipping in UrlBar
• bug 935169 - Minor cleanup on TabMenuStrip
• Working on example application for DroidCon slides

WesJ

• Lots of reviews
• Bug 889620 - [FIG] Transition for creating a new tab from the tab tray
• Bug 932690 - [GeckoView]Can't disable the zoom in/out in html
• Debugging Bug 934345 - Downloading an APK crashes fennec, with fedepaol
• Starting to dig into second screen stuff
• Documenting https://developer.mozilla.org/en-US/docs/Extensions/Mobile/API/HelperApps.jsm and https://developer.mozilla.org/en-US/docs/Extensions/Mobile/API/Notifications.jsm

LucasR

This week

• Tons of patch reviews
• bug 935543 - Remove unused members of BrowserToolbar
• bug 935523 - Remove setShadowVisibility() from toolbar
• bug 935530 - Fix subtle misalignment of stop button
• bug 935534 - Misc tweaks/cleanups in toolbar-related layout
• bug 935527 - Make all BrowserToolbar members private
• bug 935526 - Remove redudant uses of 'this' in PageActionLayout
• bug 935473 - Exiting editing mode while forward button is visible might leave it visible and disabled
• bug 934900 - [TABLET] Back/Forward and Bookmark buttons are enabled in editing mode if tap on URL bar before loading page
• (Investigated) bug 883500 - java.lang.IllegalArgumentException: the bind value at index <n> is null at android.database.sqlite.SQLiteProgram.bindString(SQLiteProgram.java)
• Playing a bit with new KitKat APIs
• Australis bug filing sprint

Next week

• More toolbar stuff, new batch of patches coming soon
• More Fx26 stuff, tracking bugs

liuche

• bug 852608 - Add OpenSearch support [r+]
• Follow-ups/future work:
  • bug 934591 - better handling of re-adding engines
  • bug 935259 - split DOMLinkAdded up
  • bug 934596 - OpenSearch in search shortcuts
• bug 932092 - UI Telemetry probes
  • talking to bwinton (Australis)
  • looking at Google Analytics
• (new laptop setup)
• bug 884590 - prefsObserver bug [mentoring!]

Margaret

• Done
  • bug 932694 - about:apps is empty after closing Firefox
  • bug 932830 - Add home banner API to let add-on know when message is shown to user
  • bug 934678 - Write test for Home.banner API
  • bug 925082 - Open PinSiteDialog with current pinned term
  • bug 930589 - Remove "Edit" context menu item from reading list
  • bug 933428 - Remove "Share" and "Add to Home Screen" from about:home context menus

• In Progress/Next
  • Investigating bug 846569 - Tab list should keep scroll position when closing a tab, currently jumps back to active tab
  • Finishing up Adrian's patch for bug 899187 - Robocop tests for bookmarks page
  • Working on dynamic snippets in bug 922792
  • More about:home context menu work - dependencies in bug 931021

mcomella

• Past
  • bug 928598 - (WFM) Some favicons missing in new-new-about-home for tablet
  • bug 870925 - FHR Provider: submissions
  • bug 933445 - Incorrect paths for android-services.mozbuild with fennec-code-copy.sh on Linux
• Present
  • bug 910859: Bootstrap new UI testing framework
    • First revision is up (Bugzilla); feel free to drive-by, but I'll f? a wider audience for the next revision
    • Second revision is on the way (github diff to first revision)
  • FHR architecture docs
• Future
  • FHR
    • Minor bugs and regressions
    • Create bugs/roadmap for potential next steps
  • Researching the use of a linter (JSHint?) in browser.js
  • bug 898613: [fig] Editing mode tests (followup to bug 895828)

RNewman

• Landing and uplifting thumbnail stuff.
• Assorted other bugs.
• Benchmarking for elan!
• Soon: investigating locale switching.

nalexander

Spent all week working on |mach projectify|. See the small blog post and video at http://www.ncalexander.net/blog/2013/11/05/building-fennec-with-eclipse-ide-demo/.

BLassey

• working on tab streaming for second screeen
• cleaning up and landing GeckoView patches
• need to update web compat addon

MFinkle

• GeckoView
  • Cleaning up patches for phase v0.2 (should land today)
  • Lot's of external interest around GeckoView
  • Android WebView now implemented with Chromium, but still lacks some WebAPI support
• Second-screen
  • Unbitrotted and fixed some bugs in the patches
  • Started playing with Brad's tab streaming code
  • Started planning some details around a "Push video to second screen" project
• Mostly doing manager stuff though

Ian Barlow

• Out for the afternoon, wiki update only
• Working on a deck outlining our plans to make Firefox a "Hub" experience for online life. Includes parts about Home (adding feeds and services) and Quick Share (support for multiple quick share spots)
  • Draft here: http://cl.ly/3q2L3n2y3O3g
  • More to come, but feedback is welcome!

Arun

• PDF.js design draft for core interactions
  • https://www.dropbox.com/s/you7kjg9d73u031/PDF-js%20%28draft%20for%20front-end%29.pdf

WebRT

• synthetic APK implementation continues apace
• we've started using meta bug 888391 to track our work
• we develop in branches, primarily this "apks" branch
• compare branches to see our work in progress

Feature Focus

Fx26 (Ships 12/10)

• [ON TRACK] AMO discoverability
• [ON TRACK] Addition of search engines - Yahoo and Bing for en-US, FR and es-ES
• [AT RISK] Addition of search engines - Yahoo and Bing for en-GB and de
• [ON TRACK] About:home redesign <= top issue is getting good traction, bug 933323
• [ON TRACK] search provider management in settings menu
• [ON TRACK] Add-ons can add content to the "promotional banner" tile
• [ON TRACK] WebRTC platform support, continued

Fx27 (Ships 01/21)

• [AT RISK] Overhaul of the favicon system bug 914296 <==can this be 'on track'?
• [ON TRACK] Marketplace discoverability - simple UI polish
• [ON TRACK] WebRTC platform support, continued
• [ON TRACK] Remove overscroll and show highlight instead bug 705246 and bug 791475
• [ON TRACK] Developer improvements: Geckoview support for 2.3 -> 4.3 and IDE support bug 924961
• [ON TRACK] Added locale: South African English (za-EN)
• [ON TRACK] Added locale: Thai (th)
• [ON TRACK] Added locale: Lithuanian (lt)
• [ON TRACK] Added locale: Slovenian (sl)

Fx28 (Ships 03/04)

• What are we coining the toolbar work? as "split action"?
• [PROPOSED] Dynamic snippets add-on
• [PROPOSED] Added locale: Greek (el)
• [PROPOSED] Added locale: Punjabi (pa-IN)
• [PROPOSED] New Sync (PiCL MVP)