nagai/fune
3
0
Fork 0
forked from mirrors/gecko-dev
Code Pull requests Activity

Bug 1202458 - part2: fix eslint errors in string actor;r=pbro

Browse source 
MozReview-Commit-ID: 1nguzarA57L

--HG--
extra : rebase_source : 4861bf47e31ea6d1d0193663a8c92f16eef12214
extra : histedit_source : 28d396641f0c8f12d9f28bc321a72e0666dd2db8
This commit is contained in:
Julian Descottes 2016-06-02 10:35:03 +02:00
parent fc46d5012b
commit cd86158629
2 changed files with 19 additions and 12 deletions
repo.diff.show_diff_stats Download patch file Download diff file Expand all files Collapse all files

1
.eslintignore
View file

@ -110,6 +110,7 @@ devtools/client/webide/**
devtools/server/**
!devtools/server/actors/webbrowser.js
!devtools/server/actors/styles.js
!devtools/server/actors/string.js
devtools/shared/*.js
!devtools/shared/css-lexer.js
!devtools/shared/defer.js

30
devtools/server/actors/string.js
View file

@ -4,14 +4,13 @@
"use strict";
var {Cu} = require("chrome");
var {DebuggerServer} = require("devtools/server/main");
var promise = require("promise");
var {Class} = require("sdk/core/heritage");
var protocol = require("devtools/shared/protocol");
var {method, Arg, Option, RetVal} = protocol;
var {method, Arg, RetVal} = protocol;
exports.LongStringActor = protocol.ActorClass({
typeName: "longstractor",
@ -64,9 +63,17 @@ exports.ShortLongString = Class({
this.str = str;
},
get length() { return this.str.length; },
get initial() { return this.str; },
string: function () { return promise.resolve(this.str); },
get length() {
return this.str.length;
},
get initial() {
return this.str;
},
string: function () {
return promise.resolve(this.str);
},
substring: function (start, end) {
return promise.resolve(this.str.substring(start, end));
@ -99,13 +106,12 @@ exports.LongStringFront = protocol.FrontClass(exports.LongStringActor, {
string: function () {
if (!this.strPromise) {
let promiseRest = (thusFar) => {
if (thusFar.length === this.length)
if (thusFar.length === this.length) {
return promise.resolve(thusFar);
else {
return this.substring(thusFar.length,
thusFar.length + DebuggerServer.LONG_STRING_READ_LENGTH)
.then((next) => promiseRest(thusFar + next));
}
return this.substring(thusFar.length,
thusFar.length + DebuggerServer.LONG_STRING_READ_LENGTH)
.then((next) => promiseRest(thusFar + next));
};
this.strPromise = promiseRest(this.initial);
@ -124,11 +130,11 @@ protocol.types.addType("longstring", {
if (!(context instanceof protocol.Actor)) {
throw Error("Passing a longstring as an argument isn't supported.");
}
if (value.short) {
return value.str;
} else {
return stringActorType.write(value, context, detail);
}
return stringActorType.write(value, context, detail);
},
read: (value, context, detail) => {
if (context instanceof protocol.Actor) {