Bug 1055747 - Fix Loop UI showcase for Chrome to avoid visual regressions, r=andrei.br92@gmail.com

This commit is contained in:
Dan Mosedale 2014-08-19 12:40:16 -07:00
parent dfae414b64
commit 6d4a32dbc5
3 changed files with 33 additions and 31 deletions

View File

@ -10,18 +10,20 @@ loop.Client = (function($) {
"use strict";
// The expected properties to be returned from the POST /call-url/ request.
const expectedCallUrlProperties = ["callUrl", "expiresAt"];
var expectedCallUrlProperties = ["callUrl", "expiresAt"];
// The expected properties to be returned from the GET /calls request.
const expectedCallProperties = ["calls"];
var expectedCallProperties = ["calls"];
/**
* Loop server client.
*
* @param {Object} settings Settings object.
*/
function Client(settings = {}) {
function Client(settings) {
if (!settings) {
settings = {};
}
// allowing an |in| test rather than a more type || allows us to dependency
// inject a non-existent mozLoop
if ("mozLoop" in settings) {
@ -108,7 +110,7 @@ loop.Client = (function($) {
*/
_requestCallUrlInternal: function(nickname, cb) {
this.mozLoop.hawkRequest("/call-url/", "POST", {callerId: nickname},
(error, responseText) => {
function (error, responseText) {
if (error) {
this._failureHandler(cb, error);
return;
@ -124,7 +126,7 @@ loop.Client = (function($) {
console.log("Error requesting call info", err);
cb(err);
}
});
}.bind(this));
},
/**
@ -149,7 +151,7 @@ loop.Client = (function($) {
_deleteCallUrlInternal: function(token, cb) {
this.mozLoop.hawkRequest("/call-url/" + token, "DELETE", null,
(error, responseText) => {
function (error, responseText) {
if (error) {
this._failureHandler(cb, error);
return;
@ -163,7 +165,7 @@ loop.Client = (function($) {
console.log("Error deleting call info", err);
cb(err);
}
});
}.bind(this));
},
/**
@ -207,7 +209,7 @@ loop.Client = (function($) {
}
this.mozLoop.hawkRequest("/calls?version=" + version, "GET", null,
(error, responseText) => {
function (error, responseText) {
if (error) {
this._failureHandler(cb, error);
return;
@ -221,8 +223,8 @@ loop.Client = (function($) {
console.log("Error requesting calls info", err);
cb(err);
}
});
},
}.bind(this));
}
};
return Client;

View File

@ -49,10 +49,10 @@ loop.conversation = (function(OT, mozL10n) {
},
_handleAccept: function(callType) {
return () => {
return function() {
this.props.model.set("selectedCallType", callType);
this.props.model.trigger("accept");
};
}.bind(this);
},
_handleDecline: function() {
@ -175,17 +175,17 @@ loop.conversation = (function(OT, mozL10n) {
incoming: function(loopVersion) {
navigator.mozLoop.startAlerting();
this._conversation.set({loopVersion: loopVersion});
this._conversation.once("accept", () => {
this._conversation.once("accept", function() {
this.navigate("call/accept", {trigger: true});
});
this._conversation.once("decline", () => {
}.bind(this));
this._conversation.once("decline", function() {
this.navigate("call/decline", {trigger: true});
});
this._conversation.once("declineAndBlock", () => {
}.bind(this));
this._conversation.once("declineAndBlock", function() {
this.navigate("call/declineAndBlock", {trigger: true});
});
}.bind(this));
this._conversation.once("call:incoming", this.startCall, this);
this._client.requestCallsInfo(loopVersion, (err, sessionData) => {
this._client.requestCallsInfo(loopVersion, function(err, sessionData) {
if (err) {
console.error("Failed to get the sessionData", err);
// XXX Not the ideal response, but bug 1047410 will be replacing
@ -202,7 +202,7 @@ loop.conversation = (function(OT, mozL10n) {
this._conversation.setIncomingSessionData(sessionData[0]);
this._setupWebSocketAndCallView();
});
}.bind(this));
},
/**

View File

@ -49,10 +49,10 @@ loop.conversation = (function(OT, mozL10n) {
},
_handleAccept: function(callType) {
return () => {
return function() {
this.props.model.set("selectedCallType", callType);
this.props.model.trigger("accept");
};
}.bind(this);
},
_handleDecline: function() {
@ -175,17 +175,17 @@ loop.conversation = (function(OT, mozL10n) {
incoming: function(loopVersion) {
navigator.mozLoop.startAlerting();
this._conversation.set({loopVersion: loopVersion});
this._conversation.once("accept", () => {
this._conversation.once("accept", function() {
this.navigate("call/accept", {trigger: true});
});
this._conversation.once("decline", () => {
}.bind(this));
this._conversation.once("decline", function() {
this.navigate("call/decline", {trigger: true});
});
this._conversation.once("declineAndBlock", () => {
}.bind(this));
this._conversation.once("declineAndBlock", function() {
this.navigate("call/declineAndBlock", {trigger: true});
});
}.bind(this));
this._conversation.once("call:incoming", this.startCall, this);
this._client.requestCallsInfo(loopVersion, (err, sessionData) => {
this._client.requestCallsInfo(loopVersion, function(err, sessionData) {
if (err) {
console.error("Failed to get the sessionData", err);
// XXX Not the ideal response, but bug 1047410 will be replacing
@ -202,7 +202,7 @@ loop.conversation = (function(OT, mozL10n) {
this._conversation.setIncomingSessionData(sessionData[0]);
this._setupWebSocketAndCallView();
});
}.bind(this));
},
/**