Mobile/Notes/06-Nov-2013
From MozillaWiki
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
- 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!
James W. (snorp)
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
Chris Lord (cwiiis)
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
- bug 910859: Bootstrap new UI testing framework
- 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)
- FHR
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
Security/MGoodwin
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
Performance
A-Team
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)