Revert of DevTools: extract V8DebuggerAgent interface, move it under .../inspector/v8

Revert of DevTools: extract V8DebuggerAgent interface, move it under .../inspector/v8 (patchset #5 id:80001 of https://codereview.chromium.org/1367623006/ )

Reason for revert:
Broke oilpan:

In file included from gen/blink/core/InspectorInstrumentationImpl.cpp:10:
../../third_party/WebKit/Source/core/inspector/AsyncCallTracker.h:58:5: error: only virtual member functions can be marked 'override'
    WILL_BE_USING_GARBAGE_COLLECTED_MIXIN(AsyncCallTracker);
    ^
../../third_party/WebKit/Source/platform/heap/GarbageCollected.h:210:53: note: expanded from macro 'WILL_BE_USING_GARBAGE_COLLECTED_MIXIN'
#define WILL_BE_USING_GARBAGE_COLLECTED_MIXIN(TYPE) USING_GARBAGE_COLLECTED_MIXIN(TYPE)
                                                    ^
../../third_party/WebKit/Source/platform/heap/GarbageCollected.h:199:5: note: expanded from macro 'USING_GARBAGE_COLLECTED_MIXIN'
    DEFINE_GARBAGE_COLLECTED_MIXIN_METHODS(blink::Visitor*, TYPE)       \
    ^
../../third_party/WebKit/Source/platform/heap/GarbageCollected.h:134:47: note: expanded from macro 'DEFINE_GARBAGE_COLLECTED_MIXIN_METHODS'
    void adjustAndMark(VISITOR visitor) const override

Original issue's description:
> DevTools: extract V8DebuggerAgent interface, move it under .../inspector/v8
>
> V8DebuggerAgent was split into interface and implementation. All other agents should depend on the public part only.
>
> Once InjectedScript, InjectedScriptHost and InjectedScriptManager are also moved to .../inspector/v8 some of the methods on V8DebuggerAgent will be removed and called only by the implementation classes.
>
> BUG=439376
>
> Committed: https://crrev.com/80862359ba867d2807e9ba7b366b352ee5cfda2b
> Cr-Commit-Position: refs/heads/master@{#351166}

TBR=dgozman@chromium.org,pfeldman@chromium.org,yurys@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=439376
Committed: https://crrev.com/3ef449bafc04114479f0e6a17ffdbc9fe7419b27
Cr-Commit-Position: refs/heads/master@{#351228}

[Lei Zhang, 2015-09-29 01:42:17]

Created Revert of DevTools: extract V8DebuggerAgent interface, move it under
.../inspector/v8

[commit-bot: I haz the power, 2015-09-29 01:43:34]

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

[commit-bot: I haz the power, 2015-09-29 01:45:45]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-09-29 01:46:33]

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

[yurys, 2015-09-29 02:00:54]

Do we now consider Oilpan breakage as a valid reason for reverting patches? I
thought Oilpan builds were FYI and supported by Oilpan team until Oilpan is
shipped. Did that change recently?

[Lei Zhang, 2015-09-29 02:29:47]

On 2015/09/29 02:00:54, yurys wrote:
> Do we now consider Oilpan breakage as a valid reason for reverting patches? I
> thought Oilpan builds were FYI and supported by Oilpan team until Oilpan is
> shipped. Did that change recently?

Sorry, I don't actually know. Ask Oilpan team. From my perspective:

- oilpan is on the main waterfall
- oilpan bots were green for a long time until earlier today

[Lei Zhang, 2015-09-29 02:31:02]

thestig@chromium.org changed reviewers:
+ haraken@chromium.org

[Lei Zhang, 2015-09-29 02:31:03]

+haraken ^^^

Whoever moved the oilpan bots to the main waterfall should have enabled oilpan
trybots by default. Otherwise it will be very hard work for the sheriffs to keep
oilpan green.

[haraken, 2015-09-29 02:46:57]

On 2015/09/29 02:31:03, Lei Zhang wrote:
> +haraken ^^^
> 
> Whoever moved the oilpan bots to the main waterfall should have enabled oilpan
> trybots by default. Otherwise it will be very hard work for the sheriffs to
keep
> oilpan green.

You don't need to keep Oilpan bots green. At the moment, it is a responsibility
of the oilpan team to keep it green (and we've been trying to address breakages
without reverting CLs).

I'll (re-re-re-re-)request to add oilpan bots to the try bots.

[Lei Zhang, 2015-09-29 03:04:45]

On 2015/09/29 02:46:57, haraken wrote:
> On 2015/09/29 02:31:03, Lei Zhang wrote:
> > +haraken ^^^
> > 
> > Whoever moved the oilpan bots to the main waterfall should have enabled
oilpan
> > trybots by default. Otherwise it will be very hard work for the sheriffs to
> keep
> > oilpan green.
> 
> You don't need to keep Oilpan bots green. At the moment, it is a
responsibility
> of the oilpan team to keep it green (and we've been trying to address
breakages
> without reverting CLs).
> 
> I'll (re-re-re-re-)request to add oilpan bots to the try bots.

Ok, so feel free to un-revert. I just saw main waterfall -> big block of red
bots.

[yurys, 2015-09-29 08:26:19]

A revert of this CL (patchset #1 id:1) has been created in
https://codereview.chromium.org/1374863002/ by yurys@chromium.org.

The reason for reverting is: The patch did not break main bots and should not be
reverted..

[sof, 2015-09-29 11:11:40]

On 2015/09/29 08:26:19, yurys wrote:
> A revert of this CL (patchset #1 id:1) has been created in
> https://codereview.chromium.org/1374863002/ by mailto:yurys@chromium.org.
> 
> The reason for reverting is: The patch did not break main bots and should not
be
> reverted..

Sure would have appreciated a heads-up before this reland, as it needlessly
broke Oilpan again.

[yurys, 2015-09-29 18:56:18]

On 2015/09/29 11:11:40, sof wrote:
> On 2015/09/29 08:26:19, yurys wrote:
> > A revert of this CL (patchset #1 id:1) has been created in
> > https://codereview.chromium.org/1374863002/ by mailto:yurys@chromium.org.
> > 
> > The reason for reverting is: The patch did not break main bots and should
not
> be
> > reverted..
> 
> Sure would have appreciated a heads-up before this reland, as it needlessly
> broke Oilpan again.

Sorry about that. We should probably start moving this code off Oilpan heap to
avoid such breakages.