Avoid extra DB lookups in BuildRequest

When returning the list of pending build requests,
BuilderStatus.getPendingBuildRequests loads on brdicts
just to grab the brid to pass into the BuildRequest
constructor.  This brdict is then reloaded from the DB
if virtually any of the BuildRequest methods are called.
To avoid this extra lookup, this diff adds an extra
constructor parameter to thread the brdict into the
BuildRequest object.
Merge andrewjcg/buildbot:brdict (PR #1124)
Merge branch 'master' into nine