LGTM. However, please follow the below mentioned information now that we've switched infra :)
This Patch Submission will be closed as part of the migration to the Phabricator Differential application. It will need to be resubmitted using arcanist as described in Submitting The Package.
For existing repositories:
You will need to re-clone the common repository from its new location:
After that, you can automatically migrate to using the new Diffusion repositories on Phabricator by running:
make -jNN pull
where NN is a number of parallel tasks typically between 1-15.