Meeting Notes Meetings notes from the Mozilla community

Schedule

• 2015-05-11 (The Big Lebowski)

Topics for This Week

Syncing Reading List

We can now sync Reading List items across Android devices and Desktop. Work has been uplifted to Fx38 and should be live for Fx38 Beta 2. Great job! (Known issue: Reading List sync defaults to off — tracked by bug 1149226.)

Tracking Review

Beta

• Next Build:

ID	Summary	Status	Assigned to
1051556	crash in java.lang.IllegalArgumentException: invalid selection notification range at org.mozilla.gecko.GeckoEditable.onSelectionChange(GeckoEditable.java)	ASSIGNED	Eugen Sawin [:esawin] (esawin)
1108527	Severe canvas performance regression compared to Firefox 29	ASSIGNED	James Willcox (:snorp) (jwillcox@mozilla.com) (snorp)
1120511	Autophone – Twitter Throbber stop regression 2015-01-15	REOPENED	Seth Fowler [:seth] (seth)
1123389	Allow Android-side reading list service work to ride the trains	ASSIGNED	Richard Newman [:rnewman] (rnewman)
1126244	Create a maximum reader mode cache size and evict records when necessary	ASSIGNED	Vivek Balakrishnan[:vivek] (vivekb.balakrishnan)
1129074	Content view sometimes blank on load until restart	NEW	Eugen Sawin [:esawin] (esawin)
1129614	Regression: Sometimes thumbnails in the tabs drawer are not updated, they expire	NEW	Richard Newman [:rnewman] (rnewman)
1129840	Tabs are not always restored on Gingerbread devices	NEW	Mark Finkle (:mfinkle) (mark.finkle)
1132508	Last tab is cut off in tab tray after rotation	ASSIGNED	Martyn Haigh (:mhaigh) (mhaigh)
1132747	Fix Android L “share” list item in long press context menu	ASSIGNED	Michael Comella (:mcomella) (michael.l.comella)
1143888	Integrate the Adjust campaign tracking SDK into Firefox	ASSIGNED	Mark Finkle (:mfinkle) (mark.finkle)
1145567	When opening a page from Reader Mode panel, a grey bar is displayed before the reader mode toolbar	NEW	Vivek Balakrishnan[:vivek] (vivekb.balakrishnan)
1149226	Reading List sync check box is deselected (off) after Nightly update	ASSIGNED	Nick Alexander :nalexander (nalexander)

13 Total;
13 Open (100%);
0 Resolved (0%);
0 Verified (0%);

Aurora

• Next Build:

ID	Summary	Status	Assigned to
776030	Allow Fennec to use the AsyncPanZoomController	NEW	Danilo Cesar Lemes de Paula (danilo.eu)
1047127	Panning very stuttery on this page with overflow-x	NEW	Danilo Cesar Lemes de Paula (danilo.eu)
1076692	Tapping on URL bar does not dismiss the tabs tray	REOPENED	Michael Comella (:mcomella) (michael.l.comella)
1084456	Enable MSE for MP4 on Jelly Bean+	NEW
1093815	Use AndroidPlatformDecoder for standalone MP3 on Android	ASSIGNED	Matthew Gregan [:kinetik] (kinetik)
1118818	Flush Gecko preferences when leaving GeckoPreferences	ASSIGNED	Richard Newman [:rnewman] (rnewman)
1132918	Unrecognised words on direct voice input will display two error notifications	NEW	James Hugman [:jhugman] [@jhugman] (jhugman)
1132922	The voice input launches directly with the error ‘Audio problem’	NEW	James Hugman [:jhugman] [@jhugman] (jhugman)
1138433	crash in android.content.ActivityNotFoundException: No Activity found to handle Intent { act=android.speech.action.RECOGNIZE_SPEECH (has extras) } at android.app.Instrumentation.checkStartActivityResult(Instrumentation.java)	NEW	James Hugman [:jhugman] [@jhugman] (jhugman)
1139232	Door hanger on tablet shows up in the tab over view	NEW	Allison Naaktgeboren please NEEDINFO? :ally (ally)

10 Total;
10 Open (100%);
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.

• (iOS) dusek fixed bug 1124828 – “Bold Text” accessibility setting not respected

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)

• Profiling, profiling, and more profiling.
  • Pushed some experiments to Try, filed a bunch of bugs as deps of bug 857359
• Goals, meetings, blah blah blah
• Out next week

JChen

Fixed

• 1149172 – Query IMEStateManager for composition state
• 1149189 – Duplication when setting the same composing text multiple times

Working on

• 1046307 – Intermittent test_ThreadHangStats.js | test failed (with xpcshell return code: 0) | false == true
• 1115480 – [Presentation WebAPI] built-in device discovery and control protocol
• 1137926 – Input Event listener fires twice or incorrectly preventing field Input
• 1148590 – Reproducible crash involving Gecko Selection logic / GeckoEditable

Randall Barker

Last Week:

• Got final r+ need to land standalone WebRTC. Addressing final review comments.
• Waiting for final r+ to land bug 1148149 – Support Android Presentation API.

Next Week:

• Land standalone WebRTC and support for Presentation API.
• Begin investigating bug 659285 which will allow users to prevent websites from using JavaScript to auto play videos.

Eugen Sawin

• MP3 demuxer
  • Last week: integrated new MP3 frame detection with new MP3 demuxer (basic functionality, no seeking yet)
  • This week: added approximate seeking (untested)
  • Next: integrate MP3 demuxer with MP4Reader, write tests and validate functionality

• IME
  • Handing over some bugs to Jim
  • Reviews

• Misc
  • Setting up new development system

Brian Nicholson

• bug 1141429 – Improve ‘about:home’ tabs in the tab drawer
• bug 1109656 – WIP Domain autocompletion

WesJ

• bug 1148578 – Sort top sites by frecency
• bug 1147175 – Use SDWebImage for thumbnail
• bug 1139457 – Status bar style should change depending on the current UI state
• bug 1148835 – Combine urlbar and bottom toolbar in landscape and on tablets
• bug 1134532 – Pages can lock the urlbar from scrolling on screen – https://github.com/wesj/firefox-ios-1/tree/scrollUrlbar2

liuche

Highlights:

• Finishing password doorhanger bugs
• Next: about:passwords improvements and getting it off Nightly

Present:

• 1105997 – Layout in settings differs on tablet rotation
• 1144385 – Add option to edit login to password doorhanger

Past:

• 1088220 – Clean up visuals of password doorhanger notification for saving and updating
• 1143939 – Add single item delete to about:passwords context menu

Margaret

<Read Only>

(Missing the meeting for candidate lunch)

Highlights:

• Trying to wrap up reader view work
• Interviews, meetings, reviews, etc.

Past:

• 1140045 – Fix regressions caused by enabling desktop reader view toolbar button
• 1142183 – ~8MiB AWSY regression due to enabling desktop reader on nightly
• 1146373 – Images in reader view are sometimes too small
• 1147122 – Restore reader view error message if about:reader fails when user clicks reader button

Present:

• 785549 – Possible problems if byline contains <‘s or >’s
• 1147584 – Reader mode strips all the text contained between anchor tags in this document
• 1150174 – Use ReaderMode.isProbablyReaderable instead of full Readability parse to determine whether or not to show reader button

mcomella

• Share overlay visual improvements & regressions
• Android L theme inheritance fallout
• Small bugs to cooldown after aggressively working on FF38 features
• Next: search engines at the bottom of the search screen

Past:

• 1130203 – Implement v2 share overlay mock
• 1132751 – Fix Android L Settings page’s build icon size
• 1147535 – Update failure string in share overlay send tab failure
• 1147661 – Update “Send to devices” icons
• 1148041 – Font in share overlay is unexpectedly bolditalic on 4.4
• 1148197 – Share overlay misaligned on small devices

Present:

• 938845 – Merge StringHelper and org.mozilla.gecko.* Strings
• 939350 – Create python linter wrapper
• 1106935 – Remove old tablet code and resources
• 1132747 – Fix Android L “share” list item in long press context menu
• 1134484 – Implement v1 color palette
• 1134490 – Change tabs panel tab changer text to toolbar grey
• 1137483 – Add a “quick search” bar of search engines
• 1144707 – Tapping in between the three dot menu button and the edge of the screen will enable edit mode
• 1148670 – Correct MenuItemActionView.onLayout calculation
• 1148677 – “Send to other devices” icon blurry in long-press send link context menu

rnewman

• Waiting for RL uplifts to Beta, then it’ll be turned on. Should happen this week.
• Working on Sync state machine. Sync is complicated.
• Partner/marketing/meetings/reviews/etc.

Fixed

• 1123388 – Enable reading list service on Nightly
• 1147715 – Rudimentary Sync state machine
• 1147992 – crash in android.os.BadParcelableException: ClassNotFoundException when unmarshalling: com.facebook.graphql.model.GraphQLPrefetchInfo at android.os.Parcel.readParcelable(Parcel.java)
• 1148432 – Upload local deletions
• 1148464 – Remove redundant addUploadedRecord method
• 1148933 – Remove old storageservice.js and tests
• 1148990 – Don’t ship bagheeraclient, tokenserverclient, etc. unless shipping appropriate desktop features

Working on

• 997663 – Intermittent testPrivateBrowsing, testInputUrlBar | waiting for urlbar text to gain focus – urlbar text gained focus
• 1062537 – Specify –with-android-min-sdk=15 for x86 builds
• 1123389 – Allow Android-side reading list service work to ride the trains
• 1148429 – Track sync state within and across operations

nalexander

• Last week: parachuted into enemy Reading List territory: bug 1147164. We’re pretty much good, save for a known issue with Reading List sync being disabled on upgrade: bug 1149226.

• This week: additional Reading List and Firefox Sync scheduling bugs.

• Fun project: download pre-built Gecko libraries from automation (bug 1128362)
  • Never built C++ again (we hope).
  • Ready for limited beta testing:

    • hg pull -r 97c890d9a7bbd043d9acfac8bbce92464578d478 https://reviewboard-hg.mozilla.org/gecko/

    • Execute

      mach gradle-install

    • Execute

      mach gradle clean app:packageDebug

      and watch Gecko libraries get downloaded (~20megs).

  • Entire process takes about 4 minutes on my machine (pull to APK on device).
  • Blog post to follow!

Martyn Haigh

Past:

• 1145579 – Open tab queue file when fennec starts or resumes
• 1145581 – Show tabs when opening fennec from the Tab Queue notification
• 1146325 – Convert loaded tab queue data to the required format and send to gecko to open
• 1146589 – Ensure that tab opened with “open now” action is selected and visible when fennec loads
• 1148007 – Show Queue Tab toast only if Tab Queue preference is enabled

Present:

• 1132508 – Last tab is cut off in tab tray after rotation
• 1140169 – [meta] Overflow menu UI polish
• 1148380 – Tab Queue: Optimise code in GeckoApp onNewIntent for ACTION_VIEW when Tabs queued
• 1148431 – Create UI to inform users of Tab Queue and allow them to turn on or ignore
• 1149577 – Remove history and recents affecting flags from TabQueueDispatcher entry in AndroidManifest
• 1149645 – Polish UI of Toast to be same as rest of Fennec
• 1150272 – Investigate ThumbnailHelper.THUMBNAIL_ASPECT_RATIO use in TopSitesThumbnailView

James Hugman

A disgusting patch for iOS session restore.

Follow up work on QR codes (with zxing) and Voice Recognition on Android.

Reminded that: Samsung Android is not Google Android (a.k.a. TouchWiz sux).

Referred in an ex-colleague, Emily Toop.

Good Friday/Easter Monday are public holidays in the UK.

BLassey

Fixed

• 1130976 – [e10s] Flash context menu causes CPOW deadlock with AdBlock Plus

Working on

MFinkle

Fixed

Working on

• 1132591 – Shortcut pinned tabs never become unpinned when navigating to a very different page
• 1143888 – Integrate the Adjust campaign tracking SDK into Firefox
• 1148107 – isTooSmall() @ PluginHelper.js causes unnecessary reflow

Antlam

• Past
  • bug 1148460: Initial First Run User testing
  • bug 1112185: Open Later/ Tab Queue UX
  • bug 864958: Private Browsing UI color change
  • bug 1098596: Android L theme UX follow up
  • UX Co-ordination around Tracking Protection, Reading List, Passwords, etc
• Upcoming
  • bug 1137483: Add “quick search” bar **
  • Follow up on Tab Queue w Martyn
  • Follow up on QR and Voice input w James Hugman
  • Follow up on doorhangers polish w Ally and liuche
  • Bugs backlog

Darrin

• Back to working on the UICollectionView port
  • Have now got transitions working properly with the new layout code (gif: http://cl.ly/image/3j3N411q2k1O)
  • Next is cleanup and generating the proper grid layouts for each device size class/orientation
  • Also we are landing the swipe delete gesture, need to make sure that works with new layout
• Next: error pages, a few tweaks from UX feedback on sizing/margins etc.

QA

• QA Release Test-Plans
• Add-on hotfix testing
• Crash stats 37.0 release broken

Feature Focus

• Mobile Roadmap Trello Board

• iOS Roadmap in Aha!
• New iOS Trello Board

---

Details

• Wednesdays – 9:30am Pacific, 12:30pm Eastern, 16:30 UTC
• Dial-in: conference# 99998
  • US/Toll-free: +1 800 707 2533, (pin 369) Conf# 99998
  • US/California/Mountain View: +1 650 903 0800, x92 Conf# 99998
  • US/California/San Francisco: +1 415 762 5700, x92 Conf# 99998
  • US/Oregon/Portland: +1 971 544 8000, x92 Conf# 99998
  • CA/British Columbia/Vancouver: +1 778 785 1540, x92 Conf# 99998
  • CA/Ontario/Toronto: +1 416 848 3114, x92 Conf# 99998
  • UK/London: +44 (0)207 855 3000, x92 Conf# 99998
  • FR/Paris: +33 1 44 79 34 80, x92 Conf# 99998
• irc.mozilla.org #mobile for backchannel
• Mobile Vidyo Room