DevTools: Instrument DOMWindowEventQueue for async stacks.

DevTools: Instrument DOMWindowEventQueue for async stacks.

This adds async stacks support for the following DOM events:
- hashchange
- selectionchange
- pointerlockchange and others
- securitypolicyviolation
- storage

BUG=272416
R=yurys, pfeldman@chromium.org

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

[yurys, 2014-06-17 07:34:26]

https://codereview.chromium.org/333333002/diff/1/Source/core/events/DOMWindow...
File Source/core/events/DOMWindowEventQueue.cpp (right):

https://codereview.chromium.org/333333002/diff/1/Source/core/events/DOMWindow...
Source/core/events/DOMWindowEventQueue.cpp:97:
InspectorInstrumentation::didDispatchEvent(event->target(), event);
Don't we need to distinguish cancel operation from regular didDidspatch?

https://codereview.chromium.org/333333002/diff/1/Source/core/events/DOMWindow...
Source/core/events/DOMWindowEventQueue.cpp:112:
InspectorInstrumentation::didDispatchEvent((*it)->target(), it->get());
We shouldn't iterate over the queued events unless instrumentation is on. You
can wrap this code block in if (InspectorInstrumentation::hasFrontend()) check.

[aandrey, 2014-06-17 10:34:18]

Also added same instrumentation for worker event queue.
PTAL.

https://codereview.chromium.org/333333002/diff/1/Source/core/events/DOMWindow...
File Source/core/events/DOMWindowEventQueue.cpp (right):

https://codereview.chromium.org/333333002/diff/1/Source/core/events/DOMWindow...
Source/core/events/DOMWindowEventQueue.cpp:97:
InspectorInstrumentation::didDispatchEvent(event->target(), event);
On 2014/06/17 07:34:26, yurys wrote:
> Don't we need to distinguish cancel operation from regular didDidspatch?

Renamed to didRemoveEvent.

https://codereview.chromium.org/333333002/diff/1/Source/core/events/DOMWindow...
Source/core/events/DOMWindowEventQueue.cpp:112:
InspectorInstrumentation::didDispatchEvent((*it)->target(), it->get());
On 2014/06/17 07:34:26, yurys wrote:
> We shouldn't iterate over the queued events unless instrumentation is on. You
> can wrap this code block in if (InspectorInstrumentation::hasFrontend())
check.

Done.

[yurys, 2014-06-17 11:22:14]

lgtm

https://codereview.chromium.org/333333002/diff/20001/Source/core/workers/Work...
File Source/core/workers/WorkerEventQueue.cpp (right):

https://codereview.chromium.org/333333002/diff/20001/Source/core/workers/Work...
Source/core/workers/WorkerEventQueue.cpp:117:
InspectorInstrumentation::didEnqueueEvent(event->target(), event.get());
Would be nice to have a test for this functionality in case of workers. Can be
addressed in another CL.

[aandrey, 2014-06-17 11:41:18]

The CQ bit was checked by aandrey@chromium.org

[commit-bot: I haz the power, 2014-06-17 11:42:02]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/aandrey@chromium.org/333333002/40001

[commit-bot: I haz the power, 2014-06-17 13:58:30]

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

[commit-bot: I haz the power, 2014-06-17 13:58:30]

Try jobs failed on following builders:
  win_gpu_triggered_tests on tryserver.chromium.gpu
(http://build.chromium.org/p/tryserver.chromium.gpu/builders/win_gpu_triggered...)

[aandrey, 2014-06-17 13:58:44]

The CQ bit was checked by aandrey@chromium.org

[commit-bot: I haz the power, 2014-06-17 13:59:54]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/aandrey@chromium.org/333333002/40001

[commit-bot: I haz the power, 2014-06-17 15:00:31]

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

[commit-bot: I haz the power, 2014-06-17 15:00:32]

Try jobs failed on following builders:
  win_gpu_triggered_tests on tryserver.chromium.gpu
(http://build.chromium.org/p/tryserver.chromium.gpu/builders/win_gpu_triggered...)

[aandrey, 2014-06-17 21:01:51]

The CQ bit was checked by aandrey@chromium.org

[commit-bot: I haz the power, 2014-06-17 21:02:19]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/aandrey@chromium.org/333333002/40001

[commit-bot: I haz the power, 2014-06-17 21:02:50]

Change committed as 176352