Simplify Physical Web UrlManager logic

Simplify Physical Web UrlManager logic

UrlManager has several confusing methods that take parameters for
the number of previously displayable URLs and the number of
currently displayable URLs.  This is no longer necessary given that
we now store these values in member variables.  This change simplifies
these methods and their calling code.

BUG=596668
Committed: https://crrev.com/a416f1b1baaf280fd829abd98e8016d2560c97a2
Cr-Commit-Position: refs/heads/master@{#393605}

[cco3, 2016-05-12 22:41:44]

cco3@chromium.org changed reviewers:
+ mattreynolds@chromium.org

[mattreynolds, 2016-05-12 23:09:06]

lgtm

[cco3, 2016-05-13 00:07:49]

cco3@chromium.org changed reviewers:
+ nyquist@chromium.org

[nyquist, 2016-05-13 17:47:35]

lgtm

[cco3, 2016-05-13 17:49:50]

The CQ bit was checked by cco3@chromium.org

[cco3, 2016-05-13 17:49:50]

The patchset sent to the CQ was uploaded after l-g-t-m from
mattreynolds@chromium.org
Link to the patchset: https://codereview.chromium.org/1977633002/#ps40001
(title: "Rebase")

[cco3, 2016-05-13 17:49:57]

On 2016/05/13 17:47:35, nyquist wrote:
> lgtm

Thank you!

[commit-bot: I haz the power, 2016-05-13 17:50:28]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1977633002/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1977633002/40001

[cco3, 2016-05-13 18:49:13]

The CQ bit was checked by cco3@chromium.org

[cco3, 2016-05-13 18:49:14]

The patchset sent to the CQ was uploaded after l-g-t-m from
nyquist@chromium.org, mattreynolds@chromium.org
Link to the patchset: https://codereview.chromium.org/1977633002/#ps60001
(title: "Add VisibleForTesting")

[commit-bot: I haz the power, 2016-05-13 18:49:59]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1977633002/60001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1977633002/60001

[commit-bot: I haz the power, 2016-05-13 19:37:04]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2016-05-13 19:39:02]

Description was changed from

==========
Simplify Physical Web UrlManager logic

UrlManager has several confusing methods that take parameters for
the number of previously displayable URLs and the number of
currently displayable URLs.  This is no longer necessary given that
we now store these values in member variables.  This change simplifies
these methods and their calling code.

BUG=596668
==========

to

==========
Simplify Physical Web UrlManager logic

UrlManager has several confusing methods that take parameters for
the number of previously displayable URLs and the number of
currently displayable URLs.  This is no longer necessary given that
we now store these values in member variables.  This change simplifies
these methods and their calling code.

BUG=596668

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

[commit-bot: I haz the power, 2016-05-13 19:39:03]

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