Split BrowsingHistoryHandler out of history_ui.cc

Split BrowsingHistoryHandler out of history_ui.cc

This cleans up history_ui.cc in preparation for adding a new WebUI for Material
design history.

BUG=425625
Committed: https://crrev.com/31dca18fdd6871a32e2a6c119b52020347be5941
Cr-Commit-Position: refs/heads/master@{#361060}

[tsergeant, 2015-11-20 04:06:07]

tsergeant@chromium.org changed reviewers:
+ dbeam@chromium.org

[tsergeant, 2015-11-20 04:06:07]

As discussed!

[Dan Beam, 2015-11-20 20:13:00]

kinda rubbery lgtm (I assume you're mainly moving + shuffling includes
correctly)

https://codereview.chromium.org/1462813003/diff/1/chrome/browser/ui/webui/bro...
File chrome/browser/ui/webui/browsing_history_handler.cc (right):

https://codereview.chromium.org/1462813003/diff/1/chrome/browser/ui/webui/bro...
chrome/browser/ui/webui/browsing_history_handler.cc:88: base::string16
getRelativeDateLocalized(const base::Time& visit_time) {
GetRelativeDateLocalized

https://codereview.chromium.org/1462813003/diff/1/chrome/browser/ui/webui/bro...
chrome/browser/ui/webui/browsing_history_handler.cc:103: void
normalizeMonths(base::Time::Exploded* exploded) {
NormalizeMonths

https://codereview.chromium.org/1462813003/diff/1/chrome/browser/ui/webui/bro...
File chrome/browser/ui/webui/browsing_history_handler.h (right):

https://codereview.chromium.org/1462813003/diff/1/chrome/browser/ui/webui/bro...
chrome/browser/ui/webui/browsing_history_handler.h:34: struct QueryOptions;
nit: alphabetize by type name

[tsergeant, 2015-11-22 23:45:18]

The CQ bit was checked by tsergeant@chromium.org

[tsergeant, 2015-11-22 23:45:18]

Thanks!

https://codereview.chromium.org/1462813003/diff/1/chrome/browser/ui/webui/bro...
File chrome/browser/ui/webui/browsing_history_handler.cc (right):

https://codereview.chromium.org/1462813003/diff/1/chrome/browser/ui/webui/bro...
chrome/browser/ui/webui/browsing_history_handler.cc:88: base::string16
getRelativeDateLocalized(const base::Time& visit_time) {
On 2015/11/20 at 20:13:00, Dan Beam wrote:
> GetRelativeDateLocalized

Done.

https://codereview.chromium.org/1462813003/diff/1/chrome/browser/ui/webui/bro...
chrome/browser/ui/webui/browsing_history_handler.cc:103: void
normalizeMonths(base::Time::Exploded* exploded) {
On 2015/11/20 at 20:13:00, Dan Beam wrote:
> NormalizeMonths

Done.

https://codereview.chromium.org/1462813003/diff/1/chrome/browser/ui/webui/bro...
File chrome/browser/ui/webui/browsing_history_handler.h (right):

https://codereview.chromium.org/1462813003/diff/1/chrome/browser/ui/webui/bro...
chrome/browser/ui/webui/browsing_history_handler.h:34: struct QueryOptions;
On 2015/11/20 at 20:13:00, Dan Beam wrote:
> nit: alphabetize by type name

Done.

[tsergeant, 2015-11-22 23:45:18]

The patchset sent to the CQ was uploaded after l-g-t-m from dbeam@chromium.org
Link to the patchset: https://codereview.chromium.org/1462813003/#ps20001
(title: "Review comments")

[commit-bot: I haz the power, 2015-11-22 23:45:29]

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

[commit-bot: I haz the power, 2015-11-23 00:26:40]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-11-23 00:27:30]

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