Revert of [debugger] reapply break points after clearing one shots.

Revert of [debugger] reapply break points after clearing one shots. (patchset #1 id:1 of https://codereview.chromium.org/2221333002/ )

Reason for revert:
Found better solution.

Original issue's description:
> [debugger] reapply break points after clearing one shots.
>
> On the debug info, we have a list of existing break points.  When we
> step in the debugger, we flood the function with one-shot break points.
> Afterwards, we clear these one-shots by clearing all break locations.
>
> Previously, while clearing break locations, we would skip ones that have
> actual break points. Now we clear all break locations, and then reapply
> break points. This is necessary for the next step, when we encode break
> point info by source position, and not code offset. Encoding by code
> offset would mean that break points are dependent on the code kind we
> use.
>
> R=jgruber@chromium.org
> BUG=v8:5265
>
> Committed: https://crrev.com/808981a22ee19461bbe981ab33b58792d991533e
> Cr-Commit-Position: refs/heads/master@{#38492}

TBR=jgruber@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=v8:5265
Committed: https://crrev.com/56b7a5fa68cfc8d4a54185e1c74dfaf42c30e4a4
Cr-Commit-Position: refs/heads/master@{#38521}

[Yang, 2016-08-10 08:22:10]

The CQ bit was checked by yangguo@chromium.org

[Yang, 2016-08-10 08:22:12]

Created Revert of [debugger] reapply break points after clearing one shots.

[commit-bot: I haz the power, 2016-08-10 08:22:23]

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, 2016-08-10 08:22:36]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-08-10 08:23:02]

Description was changed from

==========
Revert of [debugger] reapply break points after clearing one shots. (patchset #1
id:1 of https://codereview.chromium.org/2221333002/ )

Reason for revert:
Found better solution.

Original issue's description:
> [debugger] reapply break points after clearing one shots.
> 
> On the debug info, we have a list of existing break points.  When we
> step in the debugger, we flood the function with one-shot break points.
> Afterwards, we clear these one-shots by clearing all break locations.
> 
> Previously, while clearing break locations, we would skip ones that have
> actual break points. Now we clear all break locations, and then reapply
> break points. This is necessary for the next step, when we encode break
> point info by source position, and not code offset. Encoding by code
> offset would mean that break points are dependent on the code kind we
> use.
> 
> R=jgruber@chromium.org
> BUG=v8:5265
> 
> Committed: https://crrev.com/808981a22ee19461bbe981ab33b58792d991533e
> Cr-Commit-Position: refs/heads/master@{#38492}

TBR=jgruber@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=v8:5265
==========

to

==========
Revert of [debugger] reapply break points after clearing one shots. (patchset #1
id:1 of https://codereview.chromium.org/2221333002/ )

Reason for revert:
Found better solution.

Original issue's description:
> [debugger] reapply break points after clearing one shots.
>
> On the debug info, we have a list of existing break points.  When we
> step in the debugger, we flood the function with one-shot break points.
> Afterwards, we clear these one-shots by clearing all break locations.
>
> Previously, while clearing break locations, we would skip ones that have
> actual break points. Now we clear all break locations, and then reapply
> break points. This is necessary for the next step, when we encode break
> point info by source position, and not code offset. Encoding by code
> offset would mean that break points are dependent on the code kind we
> use.
>
> R=jgruber@chromium.org
> BUG=v8:5265
>
> Committed: https://crrev.com/808981a22ee19461bbe981ab33b58792d991533e
> Cr-Commit-Position: refs/heads/master@{#38492}

TBR=jgruber@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=v8:5265

Committed: https://crrev.com/56b7a5fa68cfc8d4a54185e1c74dfaf42c30e4a4
Cr-Commit-Position: refs/heads/master@{#38521}
==========

[commit-bot: I haz the power, 2016-08-10 08:23:03]

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