[DevTools] Remove functionDetails from protocol.json

third_party/WebKit/LayoutTests/inspector/console/console-dir-expected.txt

Index: third_party/WebKit/LayoutTests/inspector/console/console-dir-expected.txt
diff --git a/third_party/WebKit/LayoutTests/inspector/console/console-dir-expected.txt b/third_party/WebKit/LayoutTests/inspector/console/console-dir-expected.txt
index 679c8f51b7a6c881e3d0e84e6c7ae8eeb618b470..ac6d12d0fe0cada145cedb38c15ec96d2f201c11 100644
--- a/third_party/WebKit/LayoutTests/inspector/console/console-dir-expected.txt
+++ b/third_party/WebKit/LayoutTests/inspector/console/console-dir-expected.txt
@@ -40,8 +40,10 @@ console-dir.html:19 anonymous()
     name: ""
     prototype: Object
     __proto__: ()
-    <function scope>
-        Global: Window
+    [[FunctionLocation]]: console-dir.html:19

[dgozman, 2016/07/07 19:59:23]

Screenshot of before and after?

[kozy, 2016/07/07 22:59:07]

after:
https://drive.google.com/file/d/0B7RLEApsOKBLY0lnZV9sVnZ3ajQ/view?usp=sharing
before:
https://drive.google.com/file/d/0B7RLEApsOKBLZzlLWWxhSmhuV2s/view?usp=sharing

[dgozman, 2016/07/08 20:37:29]

- Let's omit IsGenerator when it's false - just noise.
- Let's put IsGenerator before Scopes.
- Can we avoid |length| of Scopes?
- What's the difference between name and [[FunctionName]] ?

+    [[FunctionName]]: ""
+    [[Scopes]]: Scopes[1]

[dgozman, 2016/07/07 19:59:23]

Can there be multiple scopes?

[kozy, 2016/07/07 22:59:07]

Yes, it can. It will be Scopes[N] array with each scope.

+    [[IsGenerator]]: false
 console-dir.html:23 inner
     __proto__: Object
 console-dir.html:26 Object
@@ -56,8 +58,9 @@ console-dir.html:29 get __proto__()
     length: 0
     name: "get __proto__"
     __proto__: ()
-    <function scope>
-        No Scopes
+    [[FunctionLocation]]: (program):97

[dgozman, 2016/07/07 19:59:23]

(program)? Is that like a native function or what? Let's not show internal ones,
since we can't show the location anyway.

[kozy, 2016/07/07 22:59:07]

For some reason V8 reports scriptId, line and column for function that looks
like internal one. Added additional check to ObjectPropertySection.js.

+    [[FunctionName]]: "get __proto__"
+    [[IsGenerator]]: false
 console-dir.html:34 Uint8Array[400000000]
     [0 … 99999999]
     [100000000 … 199999999]