| commit | 7701efcbf44bc3be71b0e0a63618cd408920863d | |
| author | jeremyim <jeremyim@chromium.org> | |
| Wed, 22 Apr 2015 23:26:04 +0000 (22 16:26 -0700) | ||
| committer | Commit bot <commit-bot@chromium.org> | |
| Wed, 22 Apr 2015 23:27:08 +0000 (22 23:27 +0000) | ||
| tree | ad692791c864eb4cd3480d697ff5b0989a69f6d1 | treesnapshot (tar.gz zip) |
| parent | d44c208aa4ab397fc6531eae51cd39fb71c0e8f8 | commitdiff |
Split out DataReductionProxyEventCreator from DataReductionProxyEventStore.
Currently, the storage portion lives on the UI thread (to be read from
net_internals_ui.cc), but all calls for adding events occurs on the IO
thread. In order to maintain thread safety, it used to PostTask to
itself, but that introduces a lifetime problem per bug 472290.
The solution is to split the IO and UI thread portions (in this case
event collection and event storage) and use the existing DRP IO and
DRP Service classes to ensure posted tasks respect the lifetime of
the objects.
BUG=472290
Review URL: https://codereview.chromium.org/1075283004
Cr-Commit-Position: refs/heads/master@{#326396}
Currently, the storage portion lives on the UI thread (to be read from
net_internals_ui.cc), but all calls for adding events occurs on the IO
thread. In order to maintain thread safety, it used to PostTask to
itself, but that introduces a lifetime problem per bug 472290.
The solution is to split the IO and UI thread portions (in this case
event collection and event storage) and use the existing DRP IO and
DRP Service classes to ensure posted tasks respect the lifetime of
the objects.
BUG=472290
Review URL: https://codereview.chromium.org/1075283004
Cr-Commit-Position: refs/heads/master@{#326396}
36 files changed: