[Offline Pages] Convert Download UI Adapter to using OfflinePageAdded

[Offline Pages] Convert Download UI Adapter to using OfflinePageAdded

Recently OfflinePageModelChanged was renamed to OfflinePageAdded to reflect
the reality of when that function was being called.  As a result, the
OfflinePageAdded function now receives as a parameter the OfflinePageItem
that was added.  This allows for a much simpler observer function that
directly operates on the new page in the download cache.

BUG=638276
Committed: https://crrev.com/00c85a032fea6997e8b14e88b288d19618e62b15
Cr-Commit-Position: refs/heads/master@{#437267}

[dewittj, 2016-12-06 22:20:46]

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

[commit-bot: I haz the power, 2016-12-06 22:21:36]

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

[dewittj, 2016-12-06 22:44:27]

Description was changed from

==========
[Offline Pages] Convert Download UI Adapter to using OfflinePageAdded

Recently OfflinePageModelChanged was renamed to OfflinePageAdded to reflect
the reality of when that function was being called.  As a result, the
OfflinePageAdded function now receives as a parameter the OfflinePageItem
that was added.  This allows for a much simpler observer function that
directly operates on the new page in the download cache.

BUG=
==========

to

==========
[Offline Pages] Convert Download UI Adapter to using OfflinePageAdded

Recently OfflinePageModelChanged was renamed to OfflinePageAdded to reflect
the reality of when that function was being called.  As a result, the
OfflinePageAdded function now receives as a parameter the OfflinePageItem
that was added.  This allows for a much simpler observer function that
directly operates on the new page in the download cache.

BUG=638276
==========

[commit-bot: I haz the power, 2016-12-06 22:50:08]

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

[commit-bot: I haz the power, 2016-12-06 22:50:08]

Dry run: Try jobs failed on following builders:
  ios-simulator on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator/bui...)

[dewittj, 2016-12-06 23:16:21]

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

[dewittj, 2016-12-06 23:17:06]

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

[commit-bot: I haz the power, 2016-12-06 23:17:10]

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

[Dmitry Titov, 2016-12-06 23:42:38]

lgtm

[dewittj, 2016-12-06 23:47:37]

The CQ bit was unchecked by dewittj@chromium.org

[dewittj, 2016-12-07 23:41:12]

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

[commit-bot: I haz the power, 2016-12-07 23:41:35]

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-12-08 00:46:57]

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

[commit-bot: I haz the power, 2016-12-08 00:46:58]

Dry run: This issue passed the CQ dry run.

[dewittj, 2016-12-08 17:07:26]

The CQ bit was checked by dewittj@chromium.org

[dewittj, 2016-12-08 17:07:27]

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/2560573003/#ps40001
(title: "rebase")

[commit-bot: I haz the power, 2016-12-08 17:07:54]

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-12-08 17:12:58]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1481216846821900,
"parent_rev": "ed343c0f195c1d484e67b614cbd326181d0cba4d", "commit_rev":
"175357977dab23057e998fdcd76ba5701bf3b7b8"}

[commit-bot: I haz the power, 2016-12-08 17:13:25]

Description was changed from

==========
[Offline Pages] Convert Download UI Adapter to using OfflinePageAdded

Recently OfflinePageModelChanged was renamed to OfflinePageAdded to reflect
the reality of when that function was being called.  As a result, the
OfflinePageAdded function now receives as a parameter the OfflinePageItem
that was added.  This allows for a much simpler observer function that
directly operates on the new page in the download cache.

BUG=638276
==========

to

==========
[Offline Pages] Convert Download UI Adapter to using OfflinePageAdded

Recently OfflinePageModelChanged was renamed to OfflinePageAdded to reflect
the reality of when that function was being called.  As a result, the
OfflinePageAdded function now receives as a parameter the OfflinePageItem
that was added.  This allows for a much simpler observer function that
directly operates on the new page in the download cache.

BUG=638276
==========

[commit-bot: I haz the power, 2016-12-08 17:13:26]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-12-08 17:14:48]

Description was changed from

==========
[Offline Pages] Convert Download UI Adapter to using OfflinePageAdded

Recently OfflinePageModelChanged was renamed to OfflinePageAdded to reflect
the reality of when that function was being called.  As a result, the
OfflinePageAdded function now receives as a parameter the OfflinePageItem
that was added.  This allows for a much simpler observer function that
directly operates on the new page in the download cache.

BUG=638276
==========

to

==========
[Offline Pages] Convert Download UI Adapter to using OfflinePageAdded

Recently OfflinePageModelChanged was renamed to OfflinePageAdded to reflect
the reality of when that function was being called.  As a result, the
OfflinePageAdded function now receives as a parameter the OfflinePageItem
that was added.  This allows for a much simpler observer function that
directly operates on the new page in the download cache.

BUG=638276

Committed: https://crrev.com/00c85a032fea6997e8b14e88b288d19618e62b15
Cr-Commit-Position: refs/heads/master@{#437267}
==========

[commit-bot: I haz the power, 2016-12-08 17:14:49]

Patchset 3 (id:??) landed as
https://crrev.com/00c85a032fea6997e8b14e88b288d19618e62b15
Cr-Commit-Position: refs/heads/master@{#437267}