No survey displayed when clicking on 'Leave feedback' button at the end of the first call

VERIFIED FIXED in Firefox 45

Status

Hello (Loop)
Client
P1
normal
VERIFIED FIXED
3 years ago
2 years ago

People

(Reporter: adalucinet, Assigned: mikedeboer)

Tracking

unspecified
mozilla45
Points:
1
Bug Flags:
firefox-backlog +

Firefox Tracking Flags

(firefox45 verified)

Details

Attachments

(1 attachment)

(Reporter)

Description

3 years ago
Reproducible with latest Nightly 45.0a1 (from 2015-11-23)
Affected platforms: Windows 7 64-bit, Ubuntu 14.04 32-bit and Mac OS X 10.10.5

Steps to reproduce:
1. Launch Nightly with a clean profile.
2. Create a conversation. 
3. Copy link and join the conversation from PC2 or different Firefox profile.
3. End conversation on PC1.
4. Click on 'Leave Feedback' button.

Expected results: A new tab with a survey is opened.
Actual results: Conversation windows is closed. No survey displayed.

Additional notes:
1. Browser console output:
'Loop:Loop hawkRequest error: Object { code: 400, errno: 203, error: "Unable to leave a room you have not…" }' thrown by MozLoopService.jsm:708
'Loop:Object { code: 400, errno: 203, error: "Unable to leave a room you have not…" }' thrown by LoopRooms.jsm:826
 
2. Regression range:
Last good revision: eb3016abd37db2e6a6d923265047e84b12c0af61 (2015-11-18)
First bad revision: a523d4c7efe2f43dd6b25a176c07b729918d550f (2015-11-19)
Pushlog:
https://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=eb3016abd37db2e6a6d923265047e84b12c0af61&tochange=a523d4c7efe2f43dd6b25a176c07b729918d550f
(Assignee)

Updated

3 years ago
Assignee: nobody → mdeboer
Status: NEW → ASSIGNED
Iteration: --- → 45.2 - Nov 30
Points: --- → 1
Flags: qe-verify+
Flags: firefox-backlog+
Priority: -- → P1
(Assignee)

Comment 1

3 years ago
Created attachment 8691378 [details] [diff] [review]
Patch v1: Wait for the feedback page to be opened, before closing the window
Attachment #8691378 - Flags: review?(standard8)
(Assignee)

Updated

3 years ago
Attachment #8691378 - Flags: review?(b.mcb)
Comment on attachment 8691378 [details] [diff] [review]
Patch v1: Wait for the feedback page to be opened, before closing the window

Review of attachment 8691378 [details] [diff] [review]:
-----------------------------------------------------------------

I'm a bit puzzled as to why the tests don't need an update? The stubs don't seem to be returning values. openURLStub is a stub, but its not defined to return a promise, so I don't see how the close function is being called.
Attachment #8691378 - Flags: review?(standard8)
Attachment #8691378 - Flags: review?(b.mcb)
Flags: needinfo?(mdeboer)
(Assignee)

Comment 3

3 years ago
(In reply to Mark Banner (:standard8) from comment #2)
> I'm a bit puzzled as to why the tests don't need an update? The stubs don't
> seem to be returning values. openURLStub is a stub, but its not defined to
> return a promise, so I don't see how the close function is being called.

As long as they're stubbed, even though the return value is 'undefined', the fact it's evaluated means success and the synchronous thenable is resolved without error.
Flags: needinfo?(mdeboer)
Comment on attachment 8691378 [details] [diff] [review]
Patch v1: Wait for the feedback page to be opened, before closing the window

Review of attachment 8691378 [details] [diff] [review]:
-----------------------------------------------------------------

Ok, thanks for the explanation. r=Standard8
Attachment #8691378 - Flags: review+
(Assignee)

Comment 5

2 years ago
https://hg.mozilla.org/integration/fx-team/rev/e84368c1f7b7c61f28f60365b64a70fdf3465c08
Bug 1227526: Wait for the feedback page to be opened, before closing the window. r=Standard8

Comment 6

2 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/e84368c1f7b7
Status: ASSIGNED → RESOLVED
Last Resolved: 2 years ago
status-firefox45: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla45
(Reporter)

Comment 7

2 years ago
Verified fixed with latest Nightly 45.0a1 (from 2015-12-03), across platforms [1].

[1] Windows 8.1 64-bit, Mac OS X 10.11 and Ubuntu 14.04 32-bit
Status: RESOLVED → VERIFIED
status-firefox45: fixed → verified
Flags: qe-verify+
You need to log in before you can comment on or make changes to this bug.