Re-enable ActivityLog AdNetworkDatabase unittest

Re-enable ActivityLog AdNetworkDatabase unittest

Re-enabling the test required a bit of a refactor. The implementation of
AdNetworkDatabase is now public as HashedAdNetworkDatabase (name change from
AdNetworkDatabaseImpl at Kalman's request).

BUG=369656
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=268714

[felt, 2014-05-06 17:04:01]

On 2014/05/02 21:39:59, D Cronin wrote:

lgtm, looks like a straightforward refactor

I don't see any trybot runs here?

also: do you want to prep a CL to re-disable the unit tests in case they are
flaky once this lands?

[Devlin, 2014-05-06 17:07:42]

On 2014/05/06 17:04:01, felt wrote:
> On 2014/05/02 21:39:59, D Cronin wrote:
> 
> lgtm, looks like a straightforward refactor
> 
> I don't see any trybot runs here?
> 
> also: do you want to prep a CL to re-disable the unit tests in case they are
> flaky once this lands?

It is a straight-forward refactor, mostly to allow for easier testing (without
having to mock up the ResourceBundle).  Other than that, should be almost
identical.  And yeah, I'll be ready to disable it again just in case.

[Devlin, 2014-05-06 22:07:41]

The CQ bit was checked by rdevlin.cronin@chromium.org

[commit-bot: I haz the power, 2014-05-06 22:10:30]

CQ is trying da patch. Follow status at

https://chromium-status.appspot.com/cq/rdevlin.cronin@chromium.org/263953003/...

[commit-bot: I haz the power, 2014-05-07 04:16:17]

FYI, CQ is re-trying this CL (attempt #1).
Please consider checking whether the failures are real,
and report flakes to chrome-troopers@google.com.
The failing builders are:
  mac_chromium_rel on tryserver.chromium

[commit-bot: I haz the power, 2014-05-07 11:25:44]

Change committed as 268714