Initialize debugger together with isolate.

Initialize debugger together with isolate.

This will enable following improvements:
- keep track of contexts in debugger, remove V8DebuggerClient::contextsToReport;
- move console object to v8_inspector;
- instantiate agents in debugger;
- removed withInspector from workerGlobalScope.

We should also simplify contextCreated/contextDestroyed notifications by
moving them to ThreadDebugger, but this needs additional investigation.

BUG=580337
Committed: https://crrev.com/1d37785ee530b5f8325238101895a86a4b0c85e8
Cr-Commit-Position: refs/heads/master@{#383618}

[dgozman, 2016-03-26 00:25:57]

dgozman@chromium.org changed reviewers:
+ kozyatinskiy@chromium.org, pfeldman@chromium.org

[dgozman, 2016-03-26 00:25:58]

Take a look please.

[pfeldman, 2016-03-28 17:51:16]

lgtm

[dgozman, 2016-03-28 18:09:18]

The CQ bit was checked by dgozman@chromium.org

[commit-bot: I haz the power, 2016-03-28 18:09:28]

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

[commit-bot: I haz the power, 2016-03-28 18:28:41]

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

[commit-bot: I haz the power, 2016-03-28 18:28:42]

Try jobs failed on following builders:
  cast_shell_linux on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/cast_shell_linu...)

[dgozman, 2016-03-28 20:24:41]

The CQ bit was checked by dgozman@chromium.org

[dgozman, 2016-03-28 20:24:41]

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/1838523004/#ps20001
(title: "removed unnecessary assert hitting in unit tests")

[commit-bot: I haz the power, 2016-03-28 20:25:04]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1838523004/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1838523004/20001

[commit-bot: I haz the power, 2016-03-28 23:59:16]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-03-29 00:00:46]

Description was changed from

==========
Initialize debugger together with isolate.

This will enable following improvements:
- keep track of contexts in debugger, remove V8DebuggerClient::contextsToReport;
- move console object to v8_inspector;
- instantiate agents in debugger;
- removed withInspector from workerGlobalScope.

We should also simplify contextCreated/contextDestroyed notifications by
moving them to ThreadDebugger, but this needs additional investigation.

BUG=580337
==========

to

==========
Initialize debugger together with isolate.

This will enable following improvements:
- keep track of contexts in debugger, remove V8DebuggerClient::contextsToReport;
- move console object to v8_inspector;
- instantiate agents in debugger;
- removed withInspector from workerGlobalScope.

We should also simplify contextCreated/contextDestroyed notifications by
moving them to ThreadDebugger, but this needs additional investigation.

BUG=580337

Committed: https://crrev.com/1d37785ee530b5f8325238101895a86a4b0c85e8
Cr-Commit-Position: refs/heads/master@{#383618}
==========

[commit-bot: I haz the power, 2016-03-29 00:00:48]

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