Devtools: [WIP] Implement enhanced devtools extension language APIs

Source/devtools/front_end/resources/ResourcesPanel.js

 /*
  * Copyright (C) 2007, 2008, 2010 Apple Inc.  All rights reserved.
  * Copyright (C) 2009 Joseph Pecoraro
  * Copyright (C) 2013 Samsung Electronics. 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
@@ skipping 2317 matching lines @@
 }
 
 WebInspector.FileSystemTreeElement.prototype = {
     get fileSystemName()
     {
         return this._fileSystem.name;
     },
 
     get itemURL()
     {
-        return "filesystem://" + this._fileSystem.name;
+        return "filesystem:" + this._fileSystem.name;

[pfeldman, 2015/08/13 21:15:46]

This is only used to restore last used item.

[wes, 2015/08/14 00:55:05]

This may be pedantic, but it's the result of going through (literally grep) and
unifying every 
single url returning function which dealt with files (file:// or filesystem: or
filesystem://) 
to all uniformly return `filesystem:` URLs rather than the menagerie of URL
prefixes each API 
returned before. The lack of consistency made things very confusing. Though this
one wasn't directly
used, it still got caught up in the mass change. One of my other comments asks
which should be
considered the 'canonical' URL for a file.

[pfeldman, 2015/08/17 21:15:51]

Right, except for these are used for different purposes and are different on
purpose.

[wes, 2015/08/25 18:13:18]

Okay - but why is there also both filesystem: and filesystem:// URLS? (And the
filesystem: URLs used for workspace files aren't even parsed as valid URLs by
the URL class, BTW, since it lacks the // to delimit the protocol).

     },
 
     /**
      * @override
      * @return {boolean}
      */
     onselect: function(selectedByUser)
     {
         WebInspector.BaseStorageTreeElement.prototype.onselect.call(this, selectedByUser);
         this._fileSystemView = new WebInspector.FileSystemView(this._fileSystem);
@@ skipping 66 matching lines @@
 WebInspector.ResourcesPanelFactory.prototype = {
     /**
      * @override
      * @return {!WebInspector.Panel}
      */
     createPanel: function()
     {
         return WebInspector.ResourcesPanel._instance();
     }
 }