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

Unified Diff: tracing/tracing/base/extension_registry_basic.html

Issue 2776653002: [ESLint] Fix violations when enabling curly rule in eslint. (Closed)
Patch Set: rebase Created 3 years, 9 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 side-by-side diff with in-line comments
Download patch
Index: tracing/tracing/base/extension_registry_basic.html
diff --git a/tracing/tracing/base/extension_registry_basic.html b/tracing/tracing/base/extension_registry_basic.html
index 3ebb77f5fabf0231dfa0d587eda4af67ed7ea308..0d837174bee230b793a6a85c2225652c96998a40 100644
--- a/tracing/tracing/base/extension_registry_basic.html
+++ b/tracing/tracing/base/extension_registry_basic.html
@@ -20,17 +20,20 @@ tr.exportTo('tr.b', function() {
registry.register = function(constructor,
opt_metadata) {
if (registry.findIndexOfRegisteredConstructor(
- constructor) !== undefined)
+ constructor) !== undefined) {
throw new Error('Handler already registered for ' + constructor);
+ }
extensionRegistryOptions.validateConstructor(constructor);
var metadata = {};
- for (var k in extensionRegistryOptions.defaultMetadata)
+ for (var k in extensionRegistryOptions.defaultMetadata) {
metadata[k] = extensionRegistryOptions.defaultMetadata[k];
+ }
if (opt_metadata) {
- for (var k in opt_metadata)
+ for (var k in opt_metadata) {
metadata[k] = opt_metadata[k];
+ }
}
var typeInfo = new RegisteredTypeInfo(
@@ -63,16 +66,19 @@ tr.exportTo('tr.b', function() {
};
registry.findIndexOfRegisteredConstructor = function(constructor) {
- for (var i = 0; i < registry.registeredTypeInfos_.length; i++)
- if (registry.registeredTypeInfos_[i].constructor === constructor)
+ for (var i = 0; i < registry.registeredTypeInfos_.length; i++) {
+ if (registry.registeredTypeInfos_[i].constructor === constructor) {
return i;
+ }
+ }
return undefined;
};
registry.unregister = function(constructor) {
var foundIndex = registry.findIndexOfRegisteredConstructor(constructor);
- if (foundIndex === undefined)
+ if (foundIndex === undefined) {
throw new Error(constructor + ' not registered');
+ }
registry.registeredTypeInfos_.splice(foundIndex, 1);
var e = new tr.b.Event('registry-changed');
@@ -85,8 +91,9 @@ tr.exportTo('tr.b', function() {
registry.findTypeInfo = function(constructor) {
var foundIndex = this.findIndexOfRegisteredConstructor(constructor);
- if (foundIndex !== undefined)
+ if (foundIndex !== undefined) {
return this.registeredTypeInfos_[foundIndex];
+ }
return undefined;
};
@@ -94,20 +101,21 @@ tr.exportTo('tr.b', function() {
opt_this = opt_this ? opt_this : undefined;
for (var i = 0; i < registry.registeredTypeInfos_.length; ++i) {
var typeInfo = registry.registeredTypeInfos_[i];
- if (predicate.call(opt_this, typeInfo))
+ if (predicate.call(opt_this, typeInfo)) {
return typeInfo;
+ }
}
return extensionRegistryOptions.defaultTypeInfo;
};
registry.findTypeInfoWithName = function(name) {
- if (typeof(name) !== 'string')
+ if (typeof(name) !== 'string') {
throw new Error('Name is not a string.');
+ }
var typeInfo = registry.findTypeInfoMatching(function(ti) {
return ti.constructor.name === name;
});
- if (typeInfo)
- return typeInfo;
+ if (typeInfo) return typeInfo;
return undefined;
};
}
« no previous file with comments | « tracing/tracing/base/extension_registry_base.html ('k') | tracing/tracing/base/extension_registry_type_based.html » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698