Revert of Disable tracking of visits to bookmarks in incognito

Revert of Disable tracking of visits to bookmarks in incognito (patchset #1 id:1 of https://codereview.chromium.org/2603213002/ )

Reason for revert:
bookmark_last_visit_updater_unittest.cc does not compile on the M56 branch. 

Signature of one function that I use in the unit-test is different from the one on trunk, I need to fix it.

Original issue's description:
> Disable tracking of visits to bookmarks in incognito
> 
> Date of the last visit to any bookmarked page is stored to
> BookmarkModel. Previously, this happens in off-the-record profiles as
> well. This CL fixes the issue.
> 
> BUG=662382
> 
> Had to manually resolve (trivial) conflicts within drover when merging to M56, so I add TBR.
> 
> TBR=avi@chromium.org,bauerb@chromium.org,battre@chromium.org
> 
> Review-Url: https://codereview.chromium.org/2572433002
> Cr-Commit-Position: refs/heads/master@{#438845}
> (cherry picked from commit 1e385fa1c19d96f343594a0623a7985adb50c483)
> 
> Committed: https://chromium.googlesource.com/chromium/src/+/62f09cadded58c144c804dfacf0ef7305247ad90

TBR=avi@chromium.org,bauerb@chromium.org,battre@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=662382
Review-Url: https://codereview.chromium.org/2606283003

[jkrcal, 2017-01-03 09:19:15]

The CQ bit was checked by jkrcal@chromium.org

[jkrcal, 2017-01-03 09:19:16]

Created Revert of Disable tracking of visits to bookmarks in incognito

[commit-bot: I haz the power, 2017-01-03 09:19:22]

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, 2017-01-03 09:23:46]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1483435155858560, "parent_rev":
"27638ff3b9e7334de87eda3a022a22ae1c6acf5d", "commit_rev":
"0f5493a9b5b12164c4ec120294833efe8f23eddf"}

[commit-bot: I haz the power, 2017-01-03 09:24:12]

Description was changed from

==========
Revert of Disable tracking of visits to bookmarks in incognito (patchset #1 id:1
of https://codereview.chromium.org/2603213002/ )

Reason for revert:
bookmark_last_visit_updater_unittest.cc does not compile on the M56 branch. 

Signature of one function that I use in the unit-test is different from the one
on trunk, I need to fix it.

Original issue's description:
> Disable tracking of visits to bookmarks in incognito
> 
> Date of the last visit to any bookmarked page is stored to
> BookmarkModel. Previously, this happens in off-the-record profiles as
> well. This CL fixes the issue.
> 
> BUG=662382
> 
> Had to manually resolve (trivial) conflicts within drover when merging to M56,
so I add TBR.
> 
> TBR=avi@chromium.org,bauerb@chromium.org,battre@chromium.org
> 
> Review-Url: https://codereview.chromium.org/2572433002
> Cr-Commit-Position: refs/heads/master@{#438845}
> (cherry picked from commit 1e385fa1c19d96f343594a0623a7985adb50c483)
> 
> Committed:
https://chromium.googlesource.com/chromium/src/+/62f09cadded58c144c804dfacf0e...

TBR=avi@chromium.org,bauerb@chromium.org,battre@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=662382
==========

to

==========
Revert of Disable tracking of visits to bookmarks in incognito (patchset #1 id:1
of https://codereview.chromium.org/2603213002/ )

Reason for revert:
bookmark_last_visit_updater_unittest.cc does not compile on the M56 branch. 

Signature of one function that I use in the unit-test is different from the one
on trunk, I need to fix it.

Original issue's description:
> Disable tracking of visits to bookmarks in incognito
> 
> Date of the last visit to any bookmarked page is stored to
> BookmarkModel. Previously, this happens in off-the-record profiles as
> well. This CL fixes the issue.
> 
> BUG=662382
> 
> Had to manually resolve (trivial) conflicts within drover when merging to M56,
so I add TBR.
> 
> TBR=avi@chromium.org,bauerb@chromium.org,battre@chromium.org
> 
> Review-Url: https://codereview.chromium.org/2572433002
> Cr-Commit-Position: refs/heads/master@{#438845}
> (cherry picked from commit 1e385fa1c19d96f343594a0623a7985adb50c483)
> 
> Committed:
https://chromium.googlesource.com/chromium/src/+/62f09cadded58c144c804dfacf0e...

TBR=avi@chromium.org,bauerb@chromium.org,battre@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=662382

Review-Url: https://codereview.chromium.org/2606283003
==========

[commit-bot: I haz the power, 2017-01-03 09:24:13]

Committed patchset #1 (id:1)