This fixes bug with trying to get properties of non-object (symbol).

Source/core/inspector/InjectedScriptSource.js

 /*
  * Copyright (C) 2007 Apple Inc.  All rights reserved.
  * Copyright (C) 2013 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  *
  * 1.  Redistributions of source code must retain the above copyright
  *     notice, this list of conditions and the following disclaimer.
@@ skipping 161 matching lines @@
     return a > b ? a : b;
 }
 
 /**
  * @constructor
  */
 var InjectedScript = function()
 {
     /** @type {number} */
     this._lastBoundObjectId = 1;
     /** @type {!Object.<number, Object>} */

[aandrey, 2014/06/04 15:39:08]

@type {!Object.<number, (!Object|symbol)>}

[Alexandra Mikhaylova, 2014/06/05 13:27:43]

Done.

     this._idToWrappedObject = { __proto__: null };
     /** @type {!Object.<number, string>} */
     this._idToObjectGroupName = { __proto__: null };
     /** @type {!Object.<string, Array.<number>>} */
     this._objectGroups = { __proto__: null };
     /** @type {!Object.<string, Object>} */
     this._modules = { __proto__: null };
 }
 
 /**
@@ skipping 112 matching lines @@
             try {
                 var description = injectedScript._describe(e);
             } catch (ex) {
                 var description = "<failed to convert exception to string>";
             }
             return new InjectedScript.RemoteObject(description);
         }
     },
 
     /**
      * @param {Object} object

[aandrey, 2014/06/04 15:39:08]

{!Object|symbol}

[Alexandra Mikhaylova, 2014/06/05 13:27:43]

Done.

      * @param {string=} objectGroupName
      * @return {string}
      */
     _bind: function(object, objectGroupName)
     {
         var id = this._lastBoundObjectId++;
         this._idToWrappedObject[id] = object;
         var objectId = "{\"injectedScriptId\":" + injectedScriptId + ",\"id\":" + id + "}";
         if (objectGroupName) {
             var group = this._objectGroups[objectGroupName];
@@ skipping 50 matching lines @@
      * @param {boolean} ownProperties
      * @param {boolean} accessorPropertiesOnly
      * @return {Array.<RuntimeAgent.PropertyDescriptor>|boolean}
      */
     getProperties: function(objectId, ownProperties, accessorPropertiesOnly)
     {
         var parsedObjectId = this._parseObjectId(objectId);
         var object = this._objectForId(parsedObjectId);
         var objectGroupName = this._idToObjectGroupName[parsedObjectId.id];
 
-        if (!this._isDefined(object))
+        if (!this._isDefined(object) || typeof object !== "object")

[aandrey, 2014/06/04 15:39:08]

I think "console.dir(function(){})" will no longer work.
Could you please check it, and add a test for it?

Let's create isSymbol() function and check for it explicitly, and then cast to
/** @type {!Object} */

[Alexandra Mikhaylova, 2014/06/05 13:27:43]

Done.

             return false;
         var descriptors = this._propertyDescriptors(object, ownProperties, accessorPropertiesOnly);
 
         // Go over properties, wrap object values.
         for (var i = 0; i < descriptors.length; ++i) {
             var descriptor = descriptors[i];
             if ("get" in descriptor)
                 descriptor.get = this._wrapObject(descriptor.get, objectGroupName);
             if ("set" in descriptor)
                 descriptor.set = this._wrapObject(descriptor.set, objectGroupName);
@@ skipping 11 matching lines @@
 
     /**
      * @param {string} objectId
      * @return {Array.<Object>|boolean}
      */
     getInternalProperties: function(objectId, ownProperties)
     {
         var parsedObjectId = this._parseObjectId(objectId);
         var object = this._objectForId(parsedObjectId);
         var objectGroupName = this._idToObjectGroupName[parsedObjectId.id];
-        if (!this._isDefined(object))
+        if (!this._isDefined(object) || typeof object !== "object")
             return false;
         var descriptors = [];
         var internalProperties = InjectedScriptHost.getInternalProperties(object);
         if (internalProperties) {
             for (var i = 0; i < internalProperties.length; i++) {
                 var property = internalProperties[i];
                 var descriptor = {
                     name: property.name,
                     value: this._wrapObject(property.value, objectGroupName),
                     __proto__: null
@@ skipping 438 matching lines @@
             topCallFrame = asyncCallStacks[asyncOrdinal - 1];
         var ordinal = parsedCallFrameId["ordinal"];
         var callFrame = topCallFrame;
         while (--ordinal >= 0 && callFrame)
             callFrame = callFrame.caller;
         return callFrame;
     },
 
     /**
      * @param {Object} objectId
-     * @return {Object}
+     * @return {Object|symbol}
      */
     _objectForId: function(objectId)
     {
         return this._idToWrappedObject[objectId.id];
     },
 
     /**
      * @param {string} objectId
-     * @return {Object}
+     * @return {Object|symbol}
      */
     findObjectById: function(objectId)
     {
         var parsedObjectId = this._parseObjectId(objectId);
         return this._objectForId(parsedObjectId);
     },
 
     /**
      * @param {string} objectId
      * @return {Node}
@@ skipping 745 matching lines @@
      */
     _logEvent: function(event)
     {
         inspectedWindow.console.log(event.type, event);
     }
 }
 
 injectedScript._commandLineAPIImpl = new CommandLineAPIImpl();
 return injectedScript;
 })