Implement MarkRequestCompleted.

Implement MarkRequestCompleted.

When the foreground tab finishes a request, it will let us know, and we
need to send out the observer notification.  This code catches the
foreground request completing, and calls the observer with SUCCESS
after removing the request, and starting the next available request.

BUG=652889
Committed: https://crrev.com/b1c4cc221dd01af09ceaf1eba05735046bd46d0b
Cr-Commit-Position: refs/heads/master@{#424346}

[Pete Williamson, 2016-10-10 22:18:49]

petewil@chromium.org changed reviewers:
+ dimich@chromium.org

[Dmitry Titov, 2016-10-11 00:05:29]

Looks ok, thanks!
lgtm

[Pete Williamson, 2016-10-11 00:09:30]

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

[commit-bot: I haz the power, 2016-10-11 00:10:05]

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, 2016-10-11 00:13:26]

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

[commit-bot: I haz the power, 2016-10-11 00:13:28]

Dry run: Try jobs failed on following builders:
  mac_chromium_compile_dbg_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[Pete Williamson, 2016-10-11 01:00:40]

The CQ bit was checked by petewil@chromium.org

[Pete Williamson, 2016-10-11 01:00:41]

The patchset sent to the CQ was uploaded after l-g-t-m from dimich@chromium.org
Link to the patchset: https://codereview.chromium.org/2409823002/#ps20001
(title: "rebase-update")

[commit-bot: I haz the power, 2016-10-11 01:01:37]

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, 2016-10-11 02:38:21]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-10-11 02:42:08]

Description was changed from

==========
Implement MarkRequestCompleted.

When the foreground tab finishes a request, it will let us know, and we
need to send out the observer notification.  This code catches the
foreground request completing, and calls the observer with SUCCESS
after removing the request, and starting the next available request.

BUG=652889
==========

to

==========
Implement MarkRequestCompleted.

When the foreground tab finishes a request, it will let us know, and we
need to send out the observer notification.  This code catches the
foreground request completing, and calls the observer with SUCCESS
after removing the request, and starting the next available request.

BUG=652889

Committed: https://crrev.com/b1c4cc221dd01af09ceaf1eba05735046bd46d0b
Cr-Commit-Position: refs/heads/master@{#424346}
==========

[commit-bot: I haz the power, 2016-10-11 02:42:09]

Patchset 2 (id:??) landed as
https://crrev.com/b1c4cc221dd01af09ceaf1eba05735046bd46d0b
Cr-Commit-Position: refs/heads/master@{#424346}