[LanguageModel] Clear the model when clearing full history

[LanguageModel] Clear the model when clearing full history

Before this CL, the LanguageModel did not react in any way to user
clearing browsing data.

This CL adds wiping the model whenever the user clears the complete
browsing history.

TBR=groby
NOTRY=true
NOPRESUBMIT=true
BUG=683875
Review-Url: https://codereview.chromium.org/2649163003
Cr-Commit-Position: refs/heads/master@{#446350}
Committed: https://chromium.googlesource.com/chromium/src/+/afb5fba70d41e1cc8971942d1d78cffa491933a0 
(cherry picked from commit afb5fba70d41e1cc8971942d1d78cffa491933a0)

Review-Url: https://codereview.chromium.org/2662783003
Cr-Commit-Position: refs/branch-heads/2987@{#189}
Cr-Branched-From: ad51088c0e8776e8dcd963dbe752c4035ba6dab6-refs/heads/master@{#444943}
Committed: https://chromium.googlesource.com/chromium/src/+/1e297e01a5092bc072d897cba9309af7c35a9910

[jkrcal, 2017-01-30 14:55:17]

Patchset #2 (id:20001) has been deleted

[jkrcal, 2017-01-30 15:01:18]

jkrcal@chromium.org changed reviewers:
+ groby@chromium.org, msramek@chromium.org

[jkrcal, 2017-01-30 15:01:18]

Merging the CL 2649163003 to M57. No drover, as there are non-trivial changes in
chrome_browsing_data_remover_delegate_unittest.cc /
browsing_data_remover_unittest.cc.

Martin, PTAL at the browsing_data removal changes.
Rachel, PTAL at the language_model changes.

[msramek, 2017-01-30 16:30:45]

I confirm that browsing_data/ code is equivalent to
https://codereview.chromium.org/2649163003/.

LGTM, thanks for tackling this!

[jkrcal, 2017-01-30 21:27:51]

Description was changed from

==========
[LanguageModel] Clear the model when clearing full history

Before this CL, the LanguageModel did not react in any way to user
clearing browsing data.

This CL adds wiping the model whenever the user clears the complete
browsing history.

BUG=683875
Review-Url: https://codereview.chromium.org/2649163003
Cr-Commit-Position: refs/heads/master@{#446350}
Committed:
https://chromium.googlesource.com/chromium/src/+/afb5fba70d41e1cc8971942d1d78...
(cherry picked from commit afb5fba70d41e1cc8971942d1d78cffa491933a0)
==========

to

==========
[LanguageModel] Clear the model when clearing full history

Before this CL, the LanguageModel did not react in any way to user
clearing browsing data.

This CL adds wiping the model whenever the user clears the complete
browsing history.

BUG=683875
Review-Url: https://codereview.chromium.org/2649163003
Cr-Commit-Position: refs/heads/master@{#446350}
Committed:
https://chromium.googlesource.com/chromium/src/+/afb5fba70d41e1cc8971942d1d78...
TBR=groby
(cherry picked from commit afb5fba70d41e1cc8971942d1d78cffa491933a0)
==========

[jkrcal, 2017-01-30 21:27:51]

jkrcal@chromium.org changed reviewers:
- groby@chromium.org

[jkrcal, 2017-01-30 21:28:29]

The CQ bit was checked by jkrcal@chromium.org

[commit-bot: I haz the power, 2017-01-30 21:29:00]

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-30 21:29:01]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-01-30 21:29:02]

CLs for remote refs other than refs/heads/master must contain NOTRY=true and
NOPRESUBMIT=true in order for the CQ to process them

[jkrcal, 2017-01-30 21:39:32]

Description was changed from

==========
[LanguageModel] Clear the model when clearing full history

Before this CL, the LanguageModel did not react in any way to user
clearing browsing data.

This CL adds wiping the model whenever the user clears the complete
browsing history.

BUG=683875
Review-Url: https://codereview.chromium.org/2649163003
Cr-Commit-Position: refs/heads/master@{#446350}
Committed:
https://chromium.googlesource.com/chromium/src/+/afb5fba70d41e1cc8971942d1d78...
TBR=groby
(cherry picked from commit afb5fba70d41e1cc8971942d1d78cffa491933a0)
==========

to

==========
[LanguageModel] Clear the model when clearing full history

Before this CL, the LanguageModel did not react in any way to user
clearing browsing data.

This CL adds wiping the model whenever the user clears the complete
browsing history.

TBR=groby
NOTRY=true
NOPRESUBMIT=true
BUG=683875
Review-Url: https://codereview.chromium.org/2649163003
Cr-Commit-Position: refs/heads/master@{#446350}
Committed:
https://chromium.googlesource.com/chromium/src/+/afb5fba70d41e1cc8971942d1d78...

(cherry picked from commit afb5fba70d41e1cc8971942d1d78cffa491933a0)
==========

[jkrcal, 2017-01-30 21:39:38]

The CQ bit was checked by jkrcal@chromium.org

[commit-bot: I haz the power, 2017-01-30 21:40:21]

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-30 21:41:10]

CQ is committing da patch.

Bot data: {"patchset_id": 60001, "attempt_start_ts": 1485812378571650,
"parent_rev": "85d16735d1bdb59df1a1c229e8d381aa1a1a92f5", "commit_rev":
"1e297e01a5092bc072d897cba9309af7c35a9910"}

[commit-bot: I haz the power, 2017-01-30 21:41:43]

Description was changed from

==========
[LanguageModel] Clear the model when clearing full history

Before this CL, the LanguageModel did not react in any way to user
clearing browsing data.

This CL adds wiping the model whenever the user clears the complete
browsing history.

TBR=groby
NOTRY=true
NOPRESUBMIT=true
BUG=683875
Review-Url: https://codereview.chromium.org/2649163003
Cr-Commit-Position: refs/heads/master@{#446350}
Committed:
https://chromium.googlesource.com/chromium/src/+/afb5fba70d41e1cc8971942d1d78...

(cherry picked from commit afb5fba70d41e1cc8971942d1d78cffa491933a0)
==========

to

==========
[LanguageModel] Clear the model when clearing full history

Before this CL, the LanguageModel did not react in any way to user
clearing browsing data.

This CL adds wiping the model whenever the user clears the complete
browsing history.

TBR=groby
NOTRY=true
NOPRESUBMIT=true
BUG=683875
Review-Url: https://codereview.chromium.org/2649163003
Cr-Commit-Position: refs/heads/master@{#446350}
Committed:
https://chromium.googlesource.com/chromium/src/+/afb5fba70d41e1cc8971942d1d78...

(cherry picked from commit afb5fba70d41e1cc8971942d1d78cffa491933a0)

Review-Url: https://codereview.chromium.org/2662783003
Cr-Commit-Position: refs/branch-heads/2987@{#189}
Cr-Branched-From:
ad51088c0e8776e8dcd963dbe752c4035ba6dab6-refs/heads/master@{#444943}
Committed:
https://chromium.googlesource.com/chromium/src/+/1e297e01a5092bc072d897cba930...
==========

[commit-bot: I haz the power, 2017-01-30 21:41:44]

Committed patchset #3 (id:60001) as
https://chromium.googlesource.com/chromium/src/+/1e297e01a5092bc072d897cba930...

[groby-ooo-7-16, 2017-01-30 21:59:52]

groby@chromium.org changed reviewers:
+ groby@chromium.org

[groby-ooo-7-16, 2017-01-30 21:59:52]

lgtm