Remove debugger id, use local frame id instead.

Remove debugger id, use local frame id instead.

BUG=none

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=198395

[dgozman, 2015-07-06 10:47:09]

dgozman@chromium.org changed reviewers:
+ yurys@chromium.org

[dgozman, 2015-07-06 10:47:10]

Take a look please.

[pfeldman, 2015-07-06 12:35:49]

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

[pfeldman, 2015-07-06 12:35:50]

https://codereview.chromium.org/1225763002/diff/1/Source/bindings/core/v8/Win...
File Source/bindings/core/v8/WindowProxy.cpp (right):

https://codereview.chromium.org/1225763002/diff/1/Source/bindings/core/v8/Win...
Source/bindings/core/v8/WindowProxy.cpp:258:
MainThreadDebugger::setContextDebugData(context, m_world->worldId() ==
MainWorldId ? "page" : "injected", frame);
Why do we have 3 instrumentation callbacks in a row? What is
MainThreadDebugger's contract? I don't like the fact that it is visible in
core/.

https://codereview.chromium.org/1225763002/diff/1/Source/core/inspector/PageR...
File Source/core/inspector/PageRuntimeAgent.cpp (right):

https://codereview.chromium.org/1225763002/diff/1/Source/core/inspector/PageR...
Source/core/inspector/PageRuntimeAgent.cpp:101: if (!m_enabled)
This agent no longer needs to be enabled at all times.

[pfeldman, 2015-07-06 12:44:25]

lgtm

[dgozman, 2015-07-07 11:33:05]

The CQ bit was checked by dgozman@chromium.org

[dgozman, 2015-07-07 11:33:06]

The patchset sent to the CQ was uploaded after l-g-t-m from
pfeldman@chromium.org
Link to the patchset: https://codereview.chromium.org/1225763002/#ps40001
(title: "rebased")

[commit-bot: I haz the power, 2015-07-07 11:33:17]

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

[commit-bot: I haz the power, 2015-07-07 12:29:25]

Committed patchset #3 (id:40001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=198395