Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(150)

Side by Side Diff: third_party/WebKit/Source/devtools/front_end/workspace/FileSystemMapping.js

Issue 2261933002: DevTools: Use JS symbols instead of strings for eligible events. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 /* 1 /*
2 * Copyright (C) 2012 Google Inc. All rights reserved. 2 * Copyright (C) 2012 Google Inc. All rights reserved.
3 * 3 *
4 * Redistribution and use in source and binary forms, with or without 4 * Redistribution and use in source and binary forms, with or without
5 * modification, are permitted provided that the following conditions are 5 * modification, are permitted provided that the following conditions are
6 * met: 6 * met:
7 * 7 *
8 * * Redistributions of source code must retain the above copyright 8 * * Redistributions of source code must retain the above copyright
9 * notice, this list of conditions and the following disclaimer. 9 * notice, this list of conditions and the following disclaimer.
10 * * Redistributions in binary form must reproduce the above 10 * * Redistributions in binary form must reproduce the above
(...skipping 23 matching lines...) Expand all
34 */ 34 */
35 WebInspector.FileSystemMapping = function() 35 WebInspector.FileSystemMapping = function()
36 { 36 {
37 WebInspector.Object.call(this); 37 WebInspector.Object.call(this);
38 this._fileSystemMappingSetting = WebInspector.settings.createLocalSetting("f ileSystemMapping", {}); 38 this._fileSystemMappingSetting = WebInspector.settings.createLocalSetting("f ileSystemMapping", {});
39 /** @type {!Object.<string, !Array.<!WebInspector.FileSystemMapping.Entry>>} */ 39 /** @type {!Object.<string, !Array.<!WebInspector.FileSystemMapping.Entry>>} */
40 this._fileSystemMappings = {}; 40 this._fileSystemMappings = {};
41 this._loadFromSettings(); 41 this._loadFromSettings();
42 } 42 }
43 43
44 /** @enum {symbol} */
44 WebInspector.FileSystemMapping.Events = { 45 WebInspector.FileSystemMapping.Events = {
45 FileMappingAdded: "FileMappingAdded", 46 FileMappingAdded: Symbol("FileMappingAdded"),
46 FileMappingRemoved: "FileMappingRemoved" 47 FileMappingRemoved: Symbol("FileMappingRemoved")
47 } 48 }
48 49
49 WebInspector.FileSystemMapping.prototype = { 50 WebInspector.FileSystemMapping.prototype = {
50 _loadFromSettings: function() 51 _loadFromSettings: function()
51 { 52 {
52 var savedMapping = this._fileSystemMappingSetting.get(); 53 var savedMapping = this._fileSystemMappingSetting.get();
53 this._fileSystemMappings = {}; 54 this._fileSystemMappings = {};
54 for (var fileSystemPath in savedMapping) { 55 for (var fileSystemPath in savedMapping) {
55 var savedFileSystemMappings = savedMapping[fileSystemPath]; 56 var savedFileSystemMappings = savedMapping[fileSystemPath];
56 fileSystemPath = WebInspector.IsolatedFileSystemManager.normalizePat h(fileSystemPath); 57 fileSystemPath = WebInspector.IsolatedFileSystemManager.normalizePat h(fileSystemPath);
(...skipping 284 matching lines...) Expand 10 before | Expand all | Expand 10 after
341 this.fileSystemPath = fileSystemPath; 342 this.fileSystemPath = fileSystemPath;
342 this.urlPrefix = urlPrefix; 343 this.urlPrefix = urlPrefix;
343 this.pathPrefix = pathPrefix; 344 this.pathPrefix = pathPrefix;
344 this.configurable = configurable; 345 this.configurable = configurable;
345 } 346 }
346 347
347 /** 348 /**
348 * @type {!WebInspector.FileSystemMapping} 349 * @type {!WebInspector.FileSystemMapping}
349 */ 350 */
350 WebInspector.fileSystemMapping; 351 WebInspector.fileSystemMapping;
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698