Simplify WebDataRequestManager::RequestCompletedOnThread()

Simplify WebDataRequestManager::RequestCompletedOnThread()

This CL simplifies to WebDataRequestManager::RequestCompletedOnThread()
to call WebDataRequestManager::CancelRequest() to stop tracking the
request, instead of duplicating the contents of CancelRequest.

BUG=714789
Review-Url: https://codereview.chromium.org/2866813002
Cr-Commit-Position: refs/heads/master@{#470416}
Committed: https://chromium.googlesource.com/chromium/src/+/53b4d73d43f959faf2710241cdae3fd726602c6f

[Roger McFarlane (Chromium), 2017-05-05 20:47:03]

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

[commit-bot: I haz the power, 2017-05-05 20:48:16]

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-05-06 00:01:32]

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

[commit-bot: I haz the power, 2017-05-06 00:01:33]

Dry run: This issue passed the CQ dry run.

[Roger McFarlane (Chromium), 2017-05-08 17:39:02]

Patchset #1 (id:1) has been deleted

[Roger McFarlane (Chromium), 2017-05-08 17:41:32]

rogerm@chromium.org changed reviewers:
+ pkasting@chromium.org

[Roger McFarlane (Chromium), 2017-05-08 17:41:32]

PTAL?

https://codereview.chromium.org/2866813002/diff/20001/components/webdata/comm...
File components/webdata/common/web_data_request_manager.cc (left):

https://codereview.chromium.org/2866813002/diff/20001/components/webdata/comm...
components/webdata/common/web_data_request_manager.cc:131:
tracked_objects::ScopedTracker tracking_profile(
as a side-effect of this change, this ScopedTracker will now do a better job of
(possibly) capturing jank. In previous versions, it was't capturing the time
required to acquire the lock... which would likely be the only source of jank in
this code.

[Peter Kasting, 2017-05-08 23:04:37]

LGTM

[Roger McFarlane (Chromium), 2017-05-09 17:24:34]

The CQ bit was checked by rogerm@chromium.org

[commit-bot: I haz the power, 2017-05-09 17:25:27]

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-05-09 20:31:42]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1494350674967640,
"parent_rev": "d714f13fdb739bb89cf75ec606312d56b0d3d590", "commit_rev":
"53b4d73d43f959faf2710241cdae3fd726602c6f"}

[commit-bot: I haz the power, 2017-05-09 20:31:52]

Description was changed from

==========
Simplify WebDataRequestManager::RequestCompletedOnThread()

This CL simplifies to WebDataRequestManager::RequestCompletedOnThread()
to call WebDataRequestManager::CancelRequest() to stop tracking the
request, instead of duplicating the contents of CancelRequest.

BUG=714789
==========

to

==========
Simplify WebDataRequestManager::RequestCompletedOnThread()

This CL simplifies to WebDataRequestManager::RequestCompletedOnThread()
to call WebDataRequestManager::CancelRequest() to stop tracking the
request, instead of duplicating the contents of CancelRequest.

BUG=714789

Review-Url: https://codereview.chromium.org/2866813002
Cr-Commit-Position: refs/heads/master@{#470416}
Committed:
https://chromium.googlesource.com/chromium/src/+/53b4d73d43f959faf2710241cdae...
==========

[commit-bot: I haz the power, 2017-05-09 20:31:52]

Committed patchset #1 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/53b4d73d43f959faf2710241cdae...