Rework arguments of HistoryService::AddPage()

Rework arguments of HistoryService::AddPage()

In preparation for adding a timestamp to NavigationEntry, explicitly
add a time argument to all variants of HistoryService::AddPage().

Make HistoryAddPageArgs a regular struct (i.e., not ref-counted).  Add a default constructor for it.

Make arguments to AddPage() match HistoryAddPageArgs constructor.

Make HistoryBackend::AddPage avoid mutating its parameter.

BUG=128449
TBR=joi@chromium.org

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=158063

[akalin, 2012-09-21 00:58:22]

+sky for review

[sky, 2012-09-21 15:54:46]

LGTM

[commit-bot: I haz the power, 2012-09-21 16:55:42]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/akalin@chromium.org/10963018/1034

[commit-bot: I haz the power, 2012-09-21 16:56:08]

Presubmit check for 10963018-1034 failed and returned exit status 1.


Running presubmit commit checks ...

** Presubmit ERRORS **
Missing LGTM from an OWNER for files in these directories:
    content/public/browser

Presubmit checks took 11.4s to calculate.

[akalin, 2012-09-21 17:05:18]

TBRing joi for content/public/browser trivial changes

[commit-bot: I haz the power, 2012-09-21 17:05:31]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/akalin@chromium.org/10963018/1034