predictors: Refactor UpdateData() function.

predictors: Refactor UpdateData() function.

This CL splits UpdateData() function into UpdateResourceData() and
UpdateRedirectData() with less number of arguments.
ResourcePrefetchPredictor uses UpdateData() to update only one of the tables 
at the same time so there is no need to pass extra arguments each time.

BUG=631966
Review-Url: https://codereview.chromium.org/2804283002
Cr-Commit-Position: refs/heads/master@{#464323}
Committed: https://chromium.googlesource.com/chromium/src/+/6090964fb7c737742493816401a93ef120535875

[alexilin, 2017-04-07 13:59:29]

alexilin@chromium.org changed reviewers:
+ lizeb@chromium.org

[alexilin, 2017-04-07 13:59:30]

Hi lizeb,

I wanted to do this refactoring for a long time. 
#(deleted lines) > #(added lines) so yay!

P.S. This CL depends on http://crrev.com/2800783002/

[alexilin, 2017-04-07 14:00:38]

Description was changed from

==========
predictors: Refactor UpdateData() function.

This CL splits UpdateData() function into UpdateResourceData() and
UpdateRedirectData() with less number of arguments.
ResourcePrefetchPredictor uses UpdateData() to update only one of tables at the
same time so there is no need to pass extra arguments each time.

BUG=631966
==========

to

==========
predictors: Refactor UpdateData() function.

This CL splits UpdateData() function into UpdateResourceData() and
UpdateRedirectData() with less number of arguments.
ResourcePrefetchPredictor uses UpdateData() to update only one of the tables 
at the same time so there is no need to pass extra arguments each time.

BUG=631966
==========

[Benoit L, 2017-04-11 09:20:36]

That's cleaner this way, indeed, thanks.
lgtm.

[alexilin, 2017-04-12 17:17:41]

The CQ bit was checked by alexilin@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-04-12 17:18:18]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-04-12 18:07:50]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-04-12 18:07:51]

Dry run: This issue passed the CQ dry run.

[alexilin, 2017-04-13 06:40:18]

The CQ bit was checked by alexilin@chromium.org

[alexilin, 2017-04-13 06:40:18]

The patchset sent to the CQ was uploaded after l-g-t-m from lizeb@chromium.org
Link to the patchset: https://codereview.chromium.org/2804283002/#ps60001
(title: "Oops fix.")

[commit-bot: I haz the power, 2017-04-13 06:40:36]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-04-13 06:46:01]

CQ is committing da patch.

Bot data: {"patchset_id": 60001, "attempt_start_ts": 1492065618074000,
"parent_rev": "245e999d5e19bd5f4bd44feed0336b2156581df1", "commit_rev":
"6090964fb7c737742493816401a93ef120535875"}

[commit-bot: I haz the power, 2017-04-13 06:47:11]

Description was changed from

==========
predictors: Refactor UpdateData() function.

This CL splits UpdateData() function into UpdateResourceData() and
UpdateRedirectData() with less number of arguments.
ResourcePrefetchPredictor uses UpdateData() to update only one of the tables 
at the same time so there is no need to pass extra arguments each time.

BUG=631966
==========

to

==========
predictors: Refactor UpdateData() function.

This CL splits UpdateData() function into UpdateResourceData() and
UpdateRedirectData() with less number of arguments.
ResourcePrefetchPredictor uses UpdateData() to update only one of the tables 
at the same time so there is no need to pass extra arguments each time.

BUG=631966

Review-Url: https://codereview.chromium.org/2804283002
Cr-Commit-Position: refs/heads/master@{#464323}
Committed:
https://chromium.googlesource.com/chromium/src/+/6090964fb7c737742493816401a9...
==========

[commit-bot: I haz the power, 2017-04-13 06:47:12]

Committed patchset #4 (id:60001) as
https://chromium.googlesource.com/chromium/src/+/6090964fb7c737742493816401a9...