Platform/2013-09-10

From MozillaWiki
Jump to: navigation, search


« previous week | index | next week »

Engineering Meeting Details

  • Tuesday 2013-09-10 - 11:00 am Pacific
  • Dial-in: Audio-only conference# 98411
    • People with Mozilla phones or softphones please dial x4000 Conf# 98411
    • US/Toll-free: +1 800 707 2533, (pin 4000) Conf# 98411
    • US/California/Mountain View: +1 650 903 0800, x4000 Conf# 98411
    • US/California/San Francisco: +1 415 762 5700, x4000 Conf# 98411
    • US/Oregon/Portland: +1 971 544 8000, x4000 Conf# 98411
    • CA/British Columbia/Vancouver: +1 778 785 1540, x4000 Conf# 98411
    • CA/Ontario/Toronto: +1 416 848 3114, x4000 Conf# 98411
    • UK/London: +44 (0)207 855 3000, x4000 Conf# 98411
    • FR/Paris: +33 1 84 88 37 37, x4000 Conf# 98411
    • 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
  • Engineering Vidyo Room / Air Mozilla / MTV Warp Core / TOR Finch / SFO Warfield / PDX Widmer
  • join irc.mozilla.org #planning for back channel

Actions

Hot Bugs

(Important bugs for which we need to find owners or additional help. If known, please include suggested team or knowledge needed to advance the bug.)

Orange Factor
  • bug 886999 - Crashes in OSX OpenGL library.
    • Previously brought up in this meeting and assigned to an intern who is no longer with Mozilla.
    • milan will find an owner
  • bug 910521 - Intermittent test_printpreview_bug396024.xul | application crashed [@ nsPrincipal::GetAppId(unsigned int*)] after "Assertion failure: false, at ../../../caps/src/nsPrincipal.cpp:446"
    • Some discussion in bug but no owner.
    • lmandel will find owner
  • Various reftest failures from the fig landing (bug 907899, bug 907903, bug 907905, bug 907908)
    • Some sort of zoom issue?
    • mfinkle will find owners
Stability
  • bug 800347 - bsmedberg needs help sorting through a regression range from 20-July.
    • Changes which affect how clients use or shut down RPC channels.
    • Changes which may change the timing or nature of nested event loops, especially thread-pool shutdown.
    • We've ruled out OMTC (not enabled on Windows) and content processes (was not enabled at the time of the regression).
Other

The Need To Know

(Release and system issues that may impact engineering this week.)

Notices/Schedule (akeybl/lsblakk/bajaj)

Next Merge: October 21, 2019 Next Release: October 22, 2019
Trains
Central: 71 Aurora: 54 Beta: 70 Release:
  1. REDIRECT Template:Version/Gecko/release/current
  • FF24.0b10 for Desktop and Mobile have gone to build
    • Backed out bug 881634 - crash in mozilla::gfx::DrawTargetSkia::DrawSurface for Fx24.b10 to resolve bug 911502
57 bugs (46 bugs last week)
18 bugs (30 bugs last week)
Unresolved Aurora 54 Trackers (non-security, not tracked for Beta) Unresolved Beta 70 Trackers (non-security)
  • RelEng doing branch setup this week for FF24, FF24esr, TB24, TB24esr and b2g v1.2.

Build Changes (gps)

(Build changes of which engineers should be aware.)

<Read Only>

  • Builds in automation (and presumably developer machines by extension) have been getting faster!: http://brasstacks.mozilla.com/gofaster/#/executiontime/build?range=60&include_outliers=0
    • Combination of C++ header dependency elimination and core build system changes.
  • Build tiers have been inverted. Expect some fully concurrent building soon \o/
    • This clearly shows how much C++ compilation dominates build times. Anything you can do to reduce C++ compilation time will make the build faster. This work is largely outside the scope of the core build config group.
  • |mach test| has landed. You can now run test suites by their TBPL abbreviation e.g. |mach test M1|. Not all tests supported. File bugs against Testing :: General to plug gaps.

Upcoming Outages/Upgrades

(System outages/upgrades and tree closures that impact engineering.)

  • treeclosure from last night SCL1 outage as part of rollout DNS change: details in bug 914570. Gap in nagios coverage, so IT setting up nagios alert in bug 914735

Key Issues

(Non team specific issues that impact engineering.)

Team Stand-ups

(In <2 mins, what did your team accomplish last week, on what is your team working on this week, and on what, if anything, is your team blocked? No questions during the stand-ups. All questions should be asked during the roundtable.)

Accessibility (dbolter)

<Read Only>

Our kickass intern Max Li blogged:

Add-on SDK (dcamp)

App Tools (dcamp)

B2G Services (dougt)

Developer Tools (dcamp)

  • Pseudo element inspection added.
  • A simple highlighter was added for b2g/android.
  • UI for break on uncaught exceptions landed.

DOM (khuey)

  • Switching event listeners to using the WebIDL callback code (instead of XPCWrappedJS) is getting close. We're fixing tests at this point; speedy reviews appreciated. Bug 862627
  • WebIDL-only event generator is ready to land, so no more need for nsIDOMFooEvent interfaces for events. Bug 900904
  • We have unified exception handling on workers and main thread now. Bug 911258
  • Thread-agnostic event dispatching is the last major worker/main-thread parity blocker Bug 853893
  • Also thinking about what event loops should look like. Bug 914762
  • janv's temporary storage work is about to land Bug 785884
  • Unions now work in WebIDL dictionaries Bug 767926

Firefox Desktop (gavin/dolske)

Firefox Metro (bbondy/jmathies/mbrubeck)

<Read Only>

  • APZ now enabled by default. Issues tracked by bug 886321.
  • bug 907410 - Windows 8.1 issues resolved. Follow up work to completely split gecko/winrt event threads booted to v2 in bug 911133.
  • Misc. polish work: IT14
  • Final decision on Fx26 vs. Fx27 uplift to Aurora will happen this week. Looks like we may hold off for Fx27.

Firefox Mobile (mfinkle/blassey)

  • For Fx24, we are re-spinning a beta in order to pick up an upstream fix for bug 904784 - Local shared active media does not stop recording on tab close
  • For Fx25, broken webapps fix has landed bug 901426
  • For Fx26, Fixups and refining new about:home continues with landings bug 909274, bug 911830 bug 904172, bug 872528

Firefox OS Browser (blassey)

Firefox OS Communications (scravag)

Firefox OS Devices/Porting (mchen)

Firefox OS Media Apps (hema)

<Read Only>

https://www.pivotaltracker.com/s/projects/886840

  • This Week
    • FxOS Team Workweek in Oslo with focus on wrapping up user stories for 1.2.

Firefox OS Media Recording (cjku)

<Read Only>

  • bug 912960 - [B2G] The video device name in MediaEngineWebRTCVideoSource is not a valid UTF8 string
  • bug 881840 - [MediaEncoder] Implement VP8 video track encoder
  • bug 891705 - [MediaEncoder] Implement WebM 1.0 container writer
  • bug 911883 - Recording audio during phone call
  • bug 907048 - Skip colorlayer if other opacity layer just covers the colorlayer region
  • bug 907303 - AudioNodeEngineNEON.cpp doesn't build on GCC 4.4.3

Firefox OS Performance (mlee)

Firefox OS Productivity (doliver)

Last week:

  • The first cut at email notifications has landed! https://bugzil.la/892519
    • These should be pretty functional, so go ahead and give it a shot. Visit your account settings to turn on notifications for existing accounts.
    • New accounts will have the option to turn them on during setup
    • Note: as currently implemented, you may find notifications overly enthusiastic -- every new mail will wake up your screen and play a sound. We are working with the systems team this week to see what we can do to add a less exuberant class of notification to use for less urgent cases like this.

This week:

  • Work week - Oslo
    • Objective is to complete all 1.2 blocking features -- the remaining pieces of notifications and streaming for attachments (https://bugzil.la/871897)

Firefox OS Radio/Bluetooth/GPS (kenhkchang)

<Read Only>

  • Bugs list, http://goo.gl/0qGqtN
  • Last week
    • Fix Leo+ and hd+ issues.
    • CDMA, to implement call waiting/switching, CDMA subscription information, DTMF and information record.
    • WAP Push, to implmenet the MSISDN fileter function.
    • NFC, to implement NFCD, WebAPI, NFC manager and NFC session manager.
    • WPA-EAP, to implement the importing key function.
    • WIFI-Direct, to implement wifi-direct state machine.
  • This week
    • Fix Leo+ and hd+ issues.
    • CDMA, to implement call waiting/switching and DTMF function.
    • WAP Push, to implmenet the MSISDN fileter function.
    • NFC, to implement NFCD, WebAPI, NFC manager and NFC session manager.
    • WPA-EAP, to implement the importing key function.
    • EAP-SIM, to discuss the architecture.
    • WIFI-Direct, to implement wifi-direct state machine.

Firefox OS Systems - Front End (gwagner)

Firefox OS Systems - Platform (timdream)

GFX (milan)

  • SkiaGL on by default on B2G canvas (above a threshold size)
  • Australis/TART performance gap - on standby once bisecting is done
  • Metro team identified train 27 as the one where they may need Gfx team help if problems are found
  • Tracking B2G 1.2 graphics bugs in 902643

Identity (jedp)

JS (naveed)

<Read Only>

  • Last Week
    • Compiler (JIT)
      • Perf improvements for odin compilation time on ARM
    • Garbage Collection
      • rooting analysis: mozharness script and buildbot builder have landed
    • Front End and Other
      • wrap up Intl relanding this week, before uplift
      • bug 907744: Implement iterator result boxing in ES6 generators
      • bug 909672: Make cross-compartment cloning possible with JS_StructuredClone
      • bug 913415: Tracelogger: support multiple threads and log background thread compilation.
      • bug 912863: Improved how Talos measures RSS.
  • Next
    • Compiler (JIT)
      • bug 910960: IonMonkey: Improve codegen of TypeBarrier
      • bug 914132: Land bug (greatly improve Ion |typeof object| performance)
    • Garbage Collection
      • get analysis job pushed to production (need branch merge + reconfig)
      • bug 877658: Gecko profiler has lots of undetected JS rooting hazards
  • Blockers
    • None

Layout (jet/dbaron)

Media (mreavy)

<Read Only>

  • Daala - coding party in MV Sept 30th to Oct 3rd -- if interested, contact Maire (mreavy@mozilla.com)
  • Webrtc:
    • New team member started: Byron Campen
    • Deadlock Fix for FF 24 Android beta (bug 904784)
    • Uplifted major audio latency improvements to Aurora 25 (bug 884365)

Necko (dougt/jduell)

  • We now have the ability (thanks to Bob Clary) to run Webpagetest on same URI with different try builds, so you can see how patches affect speedindex performance. Ask me or :mcmanus for recipe if you need this--not an official service yet.
  • new HTTP cache: now under review, then we'll land it preffed off.
  • resource timing, network prediction (seer) getting close to landing
  • lots of race condition chasing, PTO on team (PTO is more fun)

Performance (vladan)

  • bug 902587: We will soon be collecting data on all main thread I/O with Telemetry
  • The new Telemetry dash is quickly evolving. It now has medians, hovering over a data point shows its values, etc. Next: adding percentiles, security review scheduled for late September. Mark Reid's architecture sketch: pipeline2.png
  • bug 911820, bug 911621, bug 911014: Handling issues from add-on manager refactor landing
  • Glandium's build improvements
  • The Gecko Profiler does not display pseudo-stacks properly on Windows XP unless you install a newer version of dbghelp.dll. You can get a more recent version from the Visual Studio install directories. Copy it over to System32 or next to firefox.exe

Plugins (bsmedberg)

  • Non-Flash plugins are now click-to-play by default (Nightly/Firefox 26), bug 899080
  • Hoping to mark Java as permanently unsafe in Firefox 24, see bug 914690

Seceng (tanvi)

WebAPI (overholt)

Quality Programs

(An opportunity to hear about status with the various quality programs that do not have a formal team structure.)

CritSmash (dbolter)

MemShrink (njn)

OrangeFactor (ryanvm)

  • Past week's OrangeFactor: 3.41 (Previous Week: 10.81).
  • 20 intermittent failures fixed in the last week - List - Thanks!.

Stability (kairo/bsmedberg)

  • Overall numbers look good, except startup crash percentages on mobile.
  • We did not have updates on Android Nightly for a few days, so people are stuck without a few fixes we did. They should catch up with today's build, though.

Roundtable

(Comments and questions that arise during the course of the meeting or otherwise do not have a section.)

  • ASan builds and tests are now enabled on m-c and m-i. [dveditz]

<Read only beyond this point>

Friends of the Tree

There were lots of great changes by volunteers in Core and Toolkit during the past week. Thanks to all of these contributors for their passion and hard work! Here are some highlights:

Mailing List Threads

(Threads that are likely to be of interest to engineering from various mailing lists.)

Good Reads

(Links to blog posts, books, videos, etc. that you think will be of interest to others.)

irc #planning Log From Today's Meeting

[1:59pm] lmandel: https://wiki.mozilla.org/Platform/2013-09-10
[1:59pm] cpeterso_ joined the chat room.
[2:00pm] dholbert_ joined the chat room.
[2:00pm] kats-lunch joined the chat room.
[2:00pm] dholbert_ left the chat room. (Client exited)
[2:00pm] kats-lunch: woah the airmo feed for this meeting is weird
[2:00pm] kats-lunch is now known as kats.
[2:00pm] RyanVM|sheriffduty joined the chat room.
[2:00pm] davidb: how so?
[2:00pm] cpeterson left the chat room. (Ping timeout)
[2:00pm] cpeterso_ is now known as cpeterson.
[2:01pm] joduinn-coffee is now known as joduinn-mtg.
[2:01pm] davidb: the two screen split/overlap?
[2:02pm] kats: yeah and how it keeps switching who's where
[2:02pm] kats: it's very disconcerting
[2:02pm] smaug joined the chat room.
[2:03pm] KaiRo: it's just based on where the loudest noises come from
[2:03pm] paolo joined the chat room.
[2:03pm] kats: yeah but sometimes it puts the loudest noisemaker in the top box and sometimes it's in the bottom box
[2:03pm] kats: it seems to have settled down now though
[2:04pm] dcamp: RyanVM|sheriffduty: I dig the shirt
[2:04pm] smooney left the chat room. (Quit: smooney)
[2:04pm] Waldo joined the chat room.
[2:04pm] ehsan left the chat room. (Input/output error)
[2:05pm] ehsan joined the chat room.
[2:05pm] margaret: RyanVM|sheriffduty: reftests?
[2:05pm] margaret: i thought it was just talos tests
[2:06pm] RyanVM|sheriffduty: dcamp: 
[2:06pm] RyanVM|sheriffduty: margaret: nope
[2:06pm] RyanVM|sheriffduty: margaret: you're even CCed to them 
[2:06pm] margaret: RyanVM|sheriffduty:  i see i'm already cc'd, sorry i haven't been following more closely
[2:07pm] jhopkins|mtg is now known as jhopkins.
[2:07pm] lmandel: RyanVM|sheriffduty: Any audible for your orange factor update?
[2:07pm] ehsan left the chat room. (Ping timeout)
[2:07pm] RyanVM|sheriffduty: lmandel: no
[2:07pm] dcamp: gps is my hero
[2:08pm] kats: dcamp++
[2:08pm] lforrest left the chat room. (Quit: Leaving.)
[2:09pm] sfink: how easy is heck?
[2:09pm] dcamp: easier than hell
[2:09pm] joduinn-mtg: lmandel: read-only, but noting details of last night's tree closure from colo outage
[2:09pm] lmandel: gavin, dolske: Desktiop update?
[2:09pm] lmandel: joduinn-mtg: Thanks
[2:09pm] joduinn-mtg: hit refresh to see in "outages/upgrades" section
[2:09pm] mfinkle: dcamp, any docs for this new stuff?
[2:09pm] margaret: lmandel: gavin is still in indonesia
[2:09pm] dolske: nope
[2:09pm] mfinkle: blogs?
[2:09pm] marcia left the chat room. (Ping timeout)
[2:09pm] dcamp: mfinkle: yeah, one sec
[2:09pm] mfinkle: like, what's the highlighter thing?
[2:10pm] lmandel: margaret, dolske: Thanks
[2:10pm] AsaDotzler joined the chat room.
[2:10pm] Asa left the chat room. (Ping timeout)
[2:10pm] joduinn-mtg: mfinkle: mobile-only respin on that beta?
[2:10pm] mcote is now known as mcote|lunch.
[2:11pm] coop|mtg is now known as coop|buildduty.
[2:11pm] dcamp: mfinkle: highlighter thing is simple;  it just draws an outline around the currently-selected element.
[2:11pm] dcamp: mfinkle: and lets you tap an element to focus it in the devtools
[2:12pm] mfinkle: joduinn-mtg, i assume, but...
[2:12pm] dcamp: mfinkle: we're putting together a big chunk of documentation/tutorials to announce alongside 26.  Who's the best person to ping to make sure the fennec stuff is correct?
[2:12pm] joduinn-mtg: bajaj: ^^ ?
[2:12pm] dcamp: https://bugzilla.mozilla.org/show_bug.cgi?id=912915
[2:13pm] firebot: Bug 912915 nor, --, Firefox 26, paul, RESO FIXED, Implement a simple generic highlighter
[2:13pm] dveditz: hope it's not his house... I think he's bandwidth capped and going over satellite
[2:14pm] mfinkle: dcamp, lucasr, me, margaret... anyone in #mobile really 
[2:14pm] lmandel: airmo has a 20 second delay. This is a call for roundtable items.
[2:14pm] dcamp: alrighty, thanks
[2:14pm] damons joined the chat room.
[2:14pm] bajaj: joduinn-mtg: what's the question ? If aorund around mobile beta I will send updates to release-drivers once i have the needed info https://bugzilla.mozilla.org/show_bug.cgi?id=904784#c58
[2:14pm] firebot: Bug 904784 maj, --, Firefox 26, rjesup, VERI FIXED, Local shared active media does not stop recording on tab close
[2:14pm] mixedpuppy left the chat room. (Quit: mixedpuppy)
[2:15pm] bajaj: everyone including releng is updated to the most recent events in the channel meeting
[2:15pm] davidb: bsmedberg: say that without smiling 
[2:15pm] bsmedberg: davidb: I can't!
[2:15pm] Waldo: haha
[2:15pm] mfinkle: lmandel, you can always pick up some bugs for android
[2:15pm] mfinkle: get back into java
[2:15pm] lmandel: mfinkle: Thought about that 2 years ago when I started.
[2:15pm] vladan joined the chat room.
[2:16pm] lmandel: mfinkle: Never found the time.
[2:16pm] lmandel: roundtable items?
[2:16pm] jduell: "asan" sounds like "ehsan"
[2:16pm] davidb: indeed
[2:16pm] KaiRo: yay for ASan
[2:17pm] myk left the chat room. (Ping timeout)
[2:17pm] blassey: dveditz: what  platforms do we have ASan on for?
[2:17pm] bsmedberg: you have to say it whiny so that there's no confusion
[2:17pm] margaret: mfinkle: you need to work on your serious smiling face
[2:17pm] nhirata joined the chat room.
[2:17pm] Waldo left the chat room.
[2:18pm] cpeterson left the chat room. (Input/output error)
[2:18pm] dveditz: blassey: just linux64 for now
[2:18pm] cpeterson joined the chat room.
[2:18pm] mfinkle: oh, i thought we were doing the "get off my lawn" face
[2:19pm] dveditz: it also builds on Mac64, but we don't have that on tbpl
[2:19pm] dveditz: it's available on Try though
[2:19pm] milan_ left the chat room. (Input/output error)
[2:19pm] lforrest joined the chat room.
[2:19pm] milan joined the chat room.
[2:19pm] blassey: dveditz: do you know if ASan works on android?
[2:19pm] dveditz: we're experimenting with syzyASan for windows, but it's a different kind of thing
[2:19pm] mixedpuppy joined the chat room.
[2:19pm] blassey: I ask because valgrind is very heavy on these little phone
[2:20pm] blassey: s
[2:20pm] dveditz: it's clang based... can you compile with clang?
[2:20pm] dveditz: ASan is also memory hungry
[2:21pm] dria left the chat room. (Quit: dria)
[2:21pm] khuey: bsmedberg: http://hg.mozilla.org/mozilla-central/rev/868ce514bba7 seems like something that could change thread timings
[2:21pm] khuey: and it's in your range