This is has been, by far, my most productive summer. Had lots of fun working with the Mozilla community. Piqued myself with intriguing details of browser engineering. Mozilla Blog and Mozilla Hacks never failed to keep me entertained :D
Now it’s all coming to an end. Not my efforts to stay engaged with the Mozilla community, but Google summer of code. So this post is a summary of what I’ve been upto in the last couple of months.
These months have been an immense learning experience. I learnt how difficult it is to complete even the most trivial tasks when the code base is huge. Adhering to software engineering principles is necessary and fun but also painful.
Produced usable code. Implemented the basic Download spam protection mechanism. Spent time studying the code base and finding suitable plugpoints to insert the spam protection mechanism.
What has been done?
- A web application that can serve as a test case for initiating multiple automatic and user inititated downloads.
- XPCOM interface for the backend to invoke the front end download count mecahnism.
- Download count mechanism in the front end to count the number of downloads associated with its origin
- Associate a download with its invoking browser element and display a prompt in that browser element.
- Mechanism to queue up downloads triggered while the prompt is being shown so that they can be allowed or blocked in the future once the user takes an action
Test Web Application : GitHub Repo
The bug that I worked on : Bug 1463527
I’ve ‘documented’ some of my thought process as comments on this bug. Although most of it happened over IRC and private IRC messages with Nihanth.
Mozilla has begun using Phabricator for code reviews recently. I submitted all my patches to Phabricator in an incremental fashion.
Links to my pathces :
Mozilla has a try server to run tests and make sure the new code is not meddling with previous work flows. The test results are reported using Treeherder.
Treeherder results for my patch :
(Note : Failures are intermittent and unrelated to my patch)
- UI/UX tweaks and tests
- Localisation of Permission prompt
- Differentiating between user initiated and automatic downloads
Currently, the download spam protection feature is hidden behind a pref. After elaborate testing and reviews, the pref will be flipped by default and shipped with nightly.