Add support for Page.getResourceTree

Add support for Page.getResourceTree

Unfortunately we don't yet have a way to walk into
imports, so this only is the first-level of the
source tree, but this is a huge step forward.

You can now click to set breakpoints, however
setting breakpoints crashes sky.

I had to fix Console.log to flush stdout
so that I could actually debug my timing-out test.

Also had to fix Chai to dump differences for deepEqual.

R=esprehn@chromium.org, abarth@chromium.org, esprhen@chromium.org

Committed: https://chromium.googlesource.com/external/mojo/+/0820a82bc3796271adaa3b06ea88c3a13c3e1593

[abarth-chromium, 2014-11-18 23:08:43]

https://codereview.chromium.org/732413004/diff/1/sky/framework/inspector/page...
File sky/framework/inspector/page-agent.sky (right):

https://codereview.chromium.org/732413004/diff/1/sky/framework/inspector/page...
sky/framework/inspector/page-agent.sky:40: // WTF? Why is !![] == true in this
forsaken language?
Not sure we want this comment...

[eseidel, 2014-11-18 23:11:39]

muzzled.  ptal.

[esprehn, 2014-11-18 23:23:57]

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

[esprehn, 2014-11-18 23:23:59]

This doesn't handle Shadow DOM, you need to look for a .shadowRoot on every
element and recurse into those subtrees.

https://codereview.chromium.org/732413004/diff/20001/sky/framework/inspector/...
File sky/framework/inspector/page-agent.sky (right):

https://codereview.chromium.org/732413004/diff/20001/sky/framework/inspector/...
sky/framework/inspector/page-agent.sky:36: Page.prototype._treeOrderWalk =
function(root, visit) {
You can actually just do querySelectorAll("*") and then walk that array if you
want a tree order traversal of only elements.

function forEachElement(node, visit) {
  var elements = node.querySelectorAll("*");
  for (var i = 0; i < elements.length; ++i) {
    visit(elements[i]);
  }
}

https://codereview.chromium.org/732413004/diff/20001/sky/framework/inspector/...
sky/framework/inspector/page-agent.sky:39: var parent_stack = [];
JS vars should really be camelCase.

https://codereview.chromium.org/732413004/diff/20001/sky/framework/inspector/...
sky/framework/inspector/page-agent.sky:51: Page.prototype._getElementsByTagName
= function(root, tag_name) {
Just use querySelectorAll()

https://codereview.chromium.org/732413004/diff/20001/sky/framework/inspector/...
sky/framework/inspector/page-agent.sky:63: for (var e of iterator)
You could also do Array.prototype.slice.call(iterator) but maybe that's too
crazy for you.

https://codereview.chromium.org/732413004/diff/20001/sky/framework/inspector/...
sky/framework/inspector/page-agent.sky:70: function _addResource(url, type,
mimeType) {
This requires allocating a new function each time _resourcesMapForSubtree is
called, is this called a lot like in a loop?

https://codereview.chromium.org/732413004/diff/20001/sky/framework/inspector/...
sky/framework/inspector/page-agent.sky:72: if (resource_map.has(absolute_url))
String first, new object means this is always false.

https://codereview.chromium.org/732413004/diff/20001/sky/framework/inspector/...
sky/framework/inspector/page-agent.sky:75: 'url': String(absolute_url),
no quotes needed on any of these

https://codereview.chromium.org/732413004/diff/20001/sky/framework/inspector/...
sky/framework/inspector/page-agent.sky:84: var imports =
this._getElementsByTagName(doc, 'import');
querySelectorAll("import,img") will only walk the document once then branch on
the type below instead.

[eseidel, 2014-11-18 23:42:53]

ptal

[esprehn, 2014-11-18 23:50:52]

lgtm

[eseidel, 2014-11-18 23:53:16]

Committed patchset #4 (id:60001) manually as
0820a82bc3796271adaa3b06ea88c3a13c3e1593.