Send visit time with HISTORY_URL_VISITED notification.

Send visit time with HISTORY_URL_VISITED notification.

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

[nilesh, 2014-05-29 21:22:07]

Hi Brett, Please take a look.

[haaawk1, 2014-05-29 21:41:56]

On 2014/05/29 21:22:07, nilesh wrote:
> Hi Brett, Please take a look.

[haaawk1, 2014-05-29 21:43:52]

lgtm

[brettw, 2014-05-29 23:21:01]

lgtm

https://codereview.chromium.org/301233004/diff/1/chrome/browser/history/histo...
File chrome/browser/history/history_notifications.h (right):

https://codereview.chromium.org/301233004/diff/1/chrome/browser/history/histo...
chrome/browser/history/history_notifications.h:36: base::Time time;
I think you should call this "visit_time" and delete the comment. Also, separate
from redirects with a blank line.

[nilesh, 2014-05-29 23:29:07]

Thanks Brett.

https://codereview.chromium.org/301233004/diff/1/chrome/browser/history/histo...
File chrome/browser/history/history_notifications.h (right):

https://codereview.chromium.org/301233004/diff/1/chrome/browser/history/histo...
chrome/browser/history/history_notifications.h:36: base::Time time;
On 2014/05/29 23:21:02, brettw wrote:
> I think you should call this "visit_time" and delete the comment. Also,
separate
> from redirects with a blank line.

Done.

[nilesh, 2014-05-29 23:30:15]

The CQ bit was checked by nileshagrawal@chromium.org

[commit-bot: I haz the power, 2014-05-29 23:32:30]

CQ is trying da patch. Follow status at

https://chromium-status.appspot.com/cq/nileshagrawal@chromium.org/301233004/2...

[commit-bot: I haz the power, 2014-05-30 04:17:15]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  win_chromium_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/win_chromium_rel/buil...)

[commit-bot: I haz the power, 2014-05-30 08:07:43]

Change committed as 273760