Revert of MD Settings: ... and we fixed all CHECK()-based URL crashes once and for all.

Revert of MD Settings: ... and we fixed all CHECK()-based URL crashes once and for all. (patchset #3 id:40001 of https://codereview.chromium.org/2076083003/ )

Reason for revert:
New test is failing on DrMemory :(

First failing build:
https://build.chromium.org/p/chromium.memory.fyi/builders/Windows%20Browser%20%28DrMemory%20full%29%20%281%29/builds/3504

Sample test output:
MdSettingsBrowserTest.BackForwardDoesntCrash:
Did not complete.

Original issue's description:
> MD Settings: ... and we fixed all CHECK()-based URL crashes once and for all.
>
> But...
>
> ONCE AND FOR ALL!
>
> BUG=620370
> R=groby@chromium.org
>
> Committed: https://crrev.com/b8e015295703f0f5689688a99c5d232b89db9325
> Cr-Commit-Position: refs/heads/master@{#400528}

TBR=groby@chromium.org,michaelpg@chromium.org,dbeam@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=620370
Committed: https://crrev.com/30f8b46a1fbea34790cd000f4a9871849461f1e4
Cr-Commit-Position: refs/heads/master@{#400654}

[benwells, 2016-06-20 10:59:22]

The CQ bit was checked by benwells@chromium.org

[benwells, 2016-06-20 10:59:23]

Created Revert of MD Settings: ... and we fixed all CHECK()-based URL crashes
once and for all.

[commit-bot: I haz the power, 2016-06-20 10:59:39]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2082563002/1

[commit-bot: I haz the power, 2016-06-20 11:36:10]

Description was changed from

==========
Revert of MD Settings: ... and we fixed all CHECK()-based URL crashes once and
for all. (patchset #3 id:40001 of https://codereview.chromium.org/2076083003/ )

Reason for revert:
New test is failing on DrMemory :(

First failing build:
https://build.chromium.org/p/chromium.memory.fyi/builders/Windows%20Browser%2...

Sample test output:
MdSettingsBrowserTest.BackForwardDoesntCrash:
Did not complete.

Original issue's description:
> MD Settings: ... and we fixed all CHECK()-based URL crashes once and for all.
> 
> But...
> 
> ONCE AND FOR ALL!
> 
> BUG=620370
> R=groby@chromium.org
> 
> Committed: https://crrev.com/b8e015295703f0f5689688a99c5d232b89db9325
> Cr-Commit-Position: refs/heads/master@{#400528}

TBR=groby@chromium.org,michaelpg@chromium.org,dbeam@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=620370
==========

to

==========
Revert of MD Settings: ... and we fixed all CHECK()-based URL crashes once and
for all. (patchset #3 id:40001 of https://codereview.chromium.org/2076083003/ )

Reason for revert:
New test is failing on DrMemory :(

First failing build:
https://build.chromium.org/p/chromium.memory.fyi/builders/Windows%20Browser%2...

Sample test output:
MdSettingsBrowserTest.BackForwardDoesntCrash:
Did not complete.

Original issue's description:
> MD Settings: ... and we fixed all CHECK()-based URL crashes once and for all.
> 
> But...
> 
> ONCE AND FOR ALL!
> 
> BUG=620370
> R=groby@chromium.org
> 
> Committed: https://crrev.com/b8e015295703f0f5689688a99c5d232b89db9325
> Cr-Commit-Position: refs/heads/master@{#400528}

TBR=groby@chromium.org,michaelpg@chromium.org,dbeam@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=620370
==========

[commit-bot: I haz the power, 2016-06-20 11:36:11]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-06-20 11:37:19]

Description was changed from

==========
Revert of MD Settings: ... and we fixed all CHECK()-based URL crashes once and
for all. (patchset #3 id:40001 of https://codereview.chromium.org/2076083003/ )

Reason for revert:
New test is failing on DrMemory :(

First failing build:
https://build.chromium.org/p/chromium.memory.fyi/builders/Windows%20Browser%2...

Sample test output:
MdSettingsBrowserTest.BackForwardDoesntCrash:
Did not complete.

Original issue's description:
> MD Settings: ... and we fixed all CHECK()-based URL crashes once and for all.
> 
> But...
> 
> ONCE AND FOR ALL!
> 
> BUG=620370
> R=groby@chromium.org
> 
> Committed: https://crrev.com/b8e015295703f0f5689688a99c5d232b89db9325
> Cr-Commit-Position: refs/heads/master@{#400528}

TBR=groby@chromium.org,michaelpg@chromium.org,dbeam@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=620370
==========

to

==========
Revert of MD Settings: ... and we fixed all CHECK()-based URL crashes once and
for all. (patchset #3 id:40001 of https://codereview.chromium.org/2076083003/ )

Reason for revert:
New test is failing on DrMemory :(

First failing build:
https://build.chromium.org/p/chromium.memory.fyi/builders/Windows%20Browser%2...

Sample test output:
MdSettingsBrowserTest.BackForwardDoesntCrash:
Did not complete.

Original issue's description:
> MD Settings: ... and we fixed all CHECK()-based URL crashes once and for all.
>
> But...
>
> ONCE AND FOR ALL!
>
> BUG=620370
> R=groby@chromium.org
>
> Committed: https://crrev.com/b8e015295703f0f5689688a99c5d232b89db9325
> Cr-Commit-Position: refs/heads/master@{#400528}

TBR=groby@chromium.org,michaelpg@chromium.org,dbeam@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=620370

Committed: https://crrev.com/30f8b46a1fbea34790cd000f4a9871849461f1e4
Cr-Commit-Position: refs/heads/master@{#400654}
==========

[commit-bot: I haz the power, 2016-06-20 11:37:21]

Patchset 1 (id:??) landed as
https://crrev.com/30f8b46a1fbea34790cd000f4a9871849461f1e4
Cr-Commit-Position: refs/heads/master@{#400654}