Implement PaymentResponse.prototype.onpayerdetailchange

RESOLVED FIXED in Firefox 64

Status

()

enhancement
P1
normal
RESOLVED FIXED
11 months ago
6 months ago

People

(Reporter: marcosc, Assigned: marcosc)

Tracking

(Blocks 1 bug, {dev-doc-complete})

unspecified
mozilla64
Points:
---
Dependency tree / graph
Bug Flags:
qe-verify -

Firefox Tracking Flags

(firefox64 fixed)

Details

(Whiteboard: [webpayments-reserve], )

Attachments

(2 attachments, 6 obsolete attachments)

Assignee

Description

11 months ago
As part of .retry(), PaymentResponse gains a "payerdetailchange" event.

Updated

11 months ago
Priority: -- → P2
Assignee

Updated

11 months ago
Blocks: 1473081
Assignee

Updated

9 months ago
Summary: Implement PaymentResponse.prototype.onuserdetailchange → Implement PaymentResponse.prototype.onpayerdetailchange
Assignee

Comment 1

9 months ago
Started work on this.
Assignee

Comment 2

9 months ago
Posted patch WIP (obsolete) — Splinter Review
Just making a start on this - seeking feedback that it's on the right track :)
Attachment #9001562 - Flags: feedback?(echuang)
Comment on attachment 9001562 [details] [diff] [review]
WIP

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

There is no big deal with implementation. Please send me a review again after comments are applied.

I know the spec says the payerdetailschange event targets on PaymentResponse object.
But why not on PaymentRequest? Does it mean payer information would not change before the merchant call retry()?

::: dom/payments/PaymentResponse.cpp
@@ +26,5 @@
> +NS_IMPL_CYCLE_COLLECTION_TRAVERSE_BEGIN_INHERITED(PaymentResponse,
> +                                                  DOMEventTargetHelper)
> +  NS_IMPL_CYCLE_COLLECTION_TRAVERSE(mOwner)
> +  NS_IMPL_CYCLE_COLLECTION_TRAVERSE(mShippingAddress)
> +  NS_IMPL_CYCLE_COLLECTION_TRAVERSE(mPromise)

NS_IMPL_CYCLE_COLLECTION_TRAVERSE(mTimer)

@@ +33,5 @@
> +NS_IMPL_CYCLE_COLLECTION_UNLINK_BEGIN_INHERITED(PaymentResponse,
> +                                                DOMEventTargetHelper)
> +  NS_IMPL_CYCLE_COLLECTION_UNLINK(mOwner)
> +  NS_IMPL_CYCLE_COLLECTION_UNLINK(mShippingAddress)
> +  NS_IMPL_CYCLE_COLLECTION_UNLINK(mPromise)

NS_IMPL_CYCLE_COLLECTION_UNLINK(mTimer)

@@ +39,2 @@
>  
>  NS_INTERFACE_MAP_BEGIN_CYCLE_COLLECTION(PaymentResponse)

NS_INTERFACE_MAP_ENTRY(nsITimerCallback)

@@ +224,5 @@
> +                                    nsAString& aPayerName,
> +                                    nsAString& aPayerPhone)
> +{
> +  // TODO: if (options.RequestPayerEmail) {
> +  // need to get options from the request first :) 

nit: tailing space

@@ +228,5 @@
> +  // need to get options from the request first :) 
> +  mPayerEmail = aPayerEmail;
> +  mPayerPhone = aPayerPhone;
> +  mPayerName = aPayerName;
> +  return this->DispatchPayerDetailChangeEvent();

No need to use "this" pointer.

return DispatchPayerDetailChangeEvent();

@@ +236,5 @@
> +PaymentResponse::DispatchPayerDetailChangeEvent()
> +{
> +  MOZ_ASSERT(mRequest->ReadyForUpdate());
> +  auto& type = NS_LITERAL_STRING("payerdetailchange");
> +  PaymentRequestUpdateEventInit init;

We probably need to set the event as non-bubbles and non-cancelable as we did in PaymentRequest.

https://searchfox.org/mozilla-central/source/dom/payments/PaymentRequest.cpp#943-944
Attachment #9001562 - Flags: feedback+
I think the attached patch is the Part 1 implementation, implementing PaymentResponse as the EventTarget on payerdetailschange.
I assume there is a Part 2 implementation, dispatching payerdetailschange while the user changes the payer information.
Attachment #9001562 - Flags: feedback?(echuang) → feedback+
Assignee

Comment 5

9 months ago
(In reply to Eden Chuang[:edenchuang] from comment #3)
> Comment on attachment 9001562 [details] [diff] [review]
> I know the spec says the payerdetailschange event targets on PaymentResponse
> object.
> But why not on PaymentRequest? Does it mean payer information would not
> change before the merchant call retry()?

Good question! so, today, use user info doesn't exist on PaymentRequest, only on PaymentResponse (i.e., you only get this data when the user hits "pay"). Moving either data to PaymentRequest would be a breaking change with what we have today. Moving the event to PaymentRequest might be a bit odd, because the the point where the data is changing is PaymentResponse. 

<snip> 
Great feedback, thanks! Will integrate the rest! (In reply to Eden Chuang[:edenchuang] from comment #4)


> I think the attached patch is the Part 1 implementation, implementing
> PaymentResponse as the EventTarget on payerdetailschange.
> I assume there is a Part 2 implementation, dispatching payerdetailschange
> while the user changes the payer information.

Correct. Just didn't want to get ahead of myself :)
Assignee

Comment 6

9 months ago
I've tried to add the IPC stuff and the ability to do the update from script via `nsIPaymentActionRequest.changePayerDetail(name, email, phone)`. 

Eden, is this looking ok? It compiles, but I've not actually tested this yet... will do that tomorrow.
Attachment #9001562 - Attachment is obsolete: true
Attachment #9002681 - Flags: feedback?(echuang)
Status: NEW → ASSIGNED
Priority: P2 → P1
Whiteboard: [webpayments]
Comment on attachment 9002681 [details] [diff] [review]
IPC, PaymentManager integration

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

::: dom/payments/PaymentRequest.cpp
@@ +604,4 @@
>      aRv.Throw(NS_ERROR_DOM_TYPE_ERR);
>      return nullptr;
>    }
> +  request->mOptions = aOptions;

PaymentRequest::mOptions should be a private member of PaymentRequest. That means request->mOptions would get compile error. 

There are two ways to set up PaymentOptions in PaymentRequest
One is passing PaymentOptions into PaymentRequest constructor.

That means you will have a constructor like
PaymentRequest::PaymentRequest(..., const PaymentOptions& aOptions)
In this way, you can only setup PaymentOptions when constructing.

The other way is using a set method for PaymentRequest::mOptions. That would like
void PaymentRequest::SetOptions(const PaymentOptions& aOptions);

@@ +934,5 @@
>    mUpdating = aUpdating;
>  }
>  
> +void
> +PaymentRequest::GetResponse(RefPtr<PaymentResponse> aRetValue)

Don't use smart pointer as a method parameter.

already_AddRefed<PaymentResponse> GetResponse() {
  RefPtr<PaymentResponse> response = mResponse;
  return response.forget();
}

and use the method 

RefPtr<PaymentResponse> response = request.GetResponse();

Additional, for the case you have a return value for the method already, and you still need another return parameter. You can do it like

bool GetResponse2(PaymentRequest** aRetVal) {
  RefPtr<PaymentResponse> response = mResponse;
  *aRetVal = response.forget();
  return true;
}

and using the method

RefPtr<PaymentRespones> response;
bool result = GetResponse2(getter_AddRefs(response));

::: dom/payments/PaymentRequest.h
@@ +119,4 @@
>    bool IsUpdating() const { return mUpdating; }
>    void SetUpdating(bool aUpdating);
>  
> +  void GetResponse(RefPtr<PaymentResponse> aRetValue);

Usually, we would not pass RefPtr as a method parameter. It is related to some copy overhead on smart pointer.

already_AddRefed<PaymentResponse> GetResponse();
Attachment #9002681 - Flags: feedback?(echuang) → feedback-
Assignee

Comment 8

9 months ago
Posted patch Feedback based update (obsolete) — Splinter Review
Posting this for more feedback while trying to figure out the nsIPayment* things and UI service...
Attachment #9002681 - Attachment is obsolete: true
Attachment #9004480 - Flags: feedback?(echuang)
Assignee

Comment 10

9 months ago
Fat fingered that one by accident. Sending again.
Assignee

Comment 11

9 months ago
This represents as far as we can get without .retry(). By calling .retry(), restarts the state machinery.
Attachment #9004480 - Attachment is obsolete: true
Attachment #9004489 - Attachment is obsolete: true
Attachment #9004480 - Flags: feedback?(echuang)
Attachment #9004491 - Flags: review?(echuang)
Flags: qe-verify-
Comment on attachment 9004491 [details] [diff] [review]
As far as we can get without retry()

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

r+ for the updated patch with comments.

::: dom/interfaces/payments/nsIPaymentUIService.idl
@@ +69,5 @@
> +   *  @param requestId - the request identify of the payment request.
> +   *                     Notice that this requestId is an internal request Id
> +   *                     generated by Gecko
> +   */
> +  void updatePayerDetail(in AString requestId);

No need this interface. When merchant calls PaymentRequestUpdateEvent.updateWith() in the handler of onpayerdetailchange. PaymentUIService::updatePayment() will be called to inform UI to update the payment information.

::: dom/payments/PaymentRequest.cpp
@@ +604,4 @@
>      aRv.Throw(NS_ERROR_DOM_TYPE_ERR);
>      return nullptr;
>    }
> +  request->mOptions = aOptions;

Here I would like to suggest to move this assignment to PaymentRequestManager::CreatePayment()
And using a setter for options. Because it is the place we really create/construct a PaymentRequest.

That means

PaymentRequestManager::CreatePayment(...) {
  paymentRequest->SetOptions(aOptions);
}
Attachment #9004491 - Flags: review?(echuang) → review+
Assignee

Comment 13

9 months ago
Posted patch Review feedback. (obsolete) — Splinter Review
Eden, could you kindly double check that I addressed your feedback correctly? 

PaymentRequest didn't have a public setter for options, so I added it. 

Q: It feels like that SetOptions() should really be private function and PaymentRequestManager should be access it as a "friend"... or am I over thinking this?
Attachment #9004491 - Attachment is obsolete: true
Attachment #9006812 - Flags: feedback?(echuang)
Comment on attachment 9006812 [details] [diff] [review]
Review feedback.

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

All generally good for me. Since nsIPaymentActionRequest and nsIPaymentActionCallback are removed in retry() implementation, we need some improvement when we rebase.

::: dom/interfaces/payments/nsIPaymentActionRequest.idl
@@ +17,4 @@
>    void respondPayment(in nsIPaymentActionResponse aResponse);
>    void changeShippingAddress(in AString aRequestId, in nsIPaymentAddress aAddress);
>    void changeShippingOption(in AString aRequestId, in AString aOption);
> +  void changePayerDetail(in AString aRequestId, in AString aPayerName, in AString aPayerEmail, in AString aPayerPhone);

I remove the nsIPaymentActionRequest and nsIPaymentActionCallback interfaces in retry() implementation. Since these two interfaces are not exposed to front-end, they needn't be XPCOM. Instead of using XPCOM interface, methods of nsIPaymentActionCallback are declared/defined in PaymentRequestParent.

I think we can review this again after retry() be checked in.

::: dom/payments/PaymentRequestManager.cpp
@@ +637,5 @@
> +                                         const nsAString& aPayerEmail,
> +                                         const nsAString& aPayerPhone)
> +{
> +  RefPtr<PaymentResponse> response = aRequest->GetResponse();
> +  MOZ_ASSERT(response);

I am not sure if response must exist.

PaymentRequestService::ChangePayerDetails can be called during show(). I think UI code would call it once any payer details change no matter during show() or retry(), because PaymentRequestService doesn't support UI to distinguish the show() and retry(). 
Therefore, for the case, we probably should ignore the update when "response" does not exist.

So I would suggest remove

  MOZ_ASSERT(reposne);

and insert followings code

// response does not exist, that means during show()
if (!response) {
  return NS_OK;
}

::: dom/payments/PaymentRequestService.cpp
@@ +527,5 @@
> +                                          const nsAString& aPayerName,
> +                                          const nsAString& aPayerEmail,
> +                                          const nsAString& aPayerPhone)
> +{
> +  nsCOMPtr<nsIPaymentActionCallback> callback;

Since the nsIPaymentActionCallback will be removed after retry(), we also need some improvement here.

Let's rebase and review it again after retry checked in.
Attachment #9006812 - Flags: feedback?(echuang) → feedback+
rebase the patch for retry() implementation.
Attachment #9006812 - Attachment is obsolete: true
Assignee

Comment 16

9 months ago
Thanks again for the rebase! Resuming work on this.
Assignee

Comment 17

9 months ago
Implement PaymentResponse.prototype.onpayerdetailchange, per spec.
Comment on attachment 9009020 [details]
Bug 1472026 - Implement PaymentResponse.prototype.onpayerdetailchange. r=edenchuang,baku

Andrea Marchesini [:baku] has approved the revision.
Attachment #9009020 - Flags: review+
Whiteboard: [webpayments] → [webpayments-reserve]
Assignee

Updated

8 months ago
Comment on attachment 9009020 [details]
Bug 1472026 - Implement PaymentResponse.prototype.onpayerdetailchange. r=edenchuang,baku

Eden Chuang[:edenchuang] has approved the revision.
Attachment #9009020 - Flags: review+

Comment 20

8 months ago
Pushed by mcaceres@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/376c2bf5bcbf
Implement PaymentResponse.prototype.onpayerdetailchange. r=edenchuang,baku
https://hg.mozilla.org/integration/autoland/rev/376c2bf5bcbf0ddfe13fa4a3c1c76b7923e6ae21

https://treeherder.mozilla.org/#/jobs?repo=autoland&resultStatus=testfailed,busted,exception&classifiedState=unclassified&fromchange=b902b6f391e3a6b16f8815fa7140bb92d8dc57ff&selectedJob=200129951

https://treeherder.mozilla.org/logviewer.html#?job_id=200129951&repo=autoland&lineNumber=2635


task 2018-09-19T09:22:25.305Z] 09:22:25     INFO - TEST-UNEXPECTED-FAIL | dom/payments/test/test_block_none10s.html | Unexpected, new PaymentRequest() can not be used in non-e10s. 
[task 2018-09-19T09:22:25.305Z] 09:22:25     INFO - testInNone10s/<@https://example.com/tests/dom/payments/test/test_block_none10s.html:27:11
[task 2018-09-19T09:22:25.306Z] 09:22:25     INFO - testInNone10s@https://example.com/tests/dom/payments/test/test_block_none10s.html:14:14
[task 2018-09-19T09:22:25.308Z] 09:22:25     INFO - runTests@https://example.com/tests/dom/payments/test/test_block_none10s.html:38:7
[task 2018-09-19T09:22:25.311Z] 09:22:25     INFO - GECKO(1682) | MEMORY STAT | vsize 405MB | residentFast 121MB | heapAllocated 11MB
[task 2018-09-19T09:22:25.311Z] 09:22:25     INFO - TEST-OK | dom/payments/test/test_block_none10s.html | took 311ms
[task 2018-09-19T09:22:25.312Z] 09:22:25     INFO - GECKO(1682) | ++DOMWINDOW == 13 (0xe9b3fc00) [pid = 1767] [serial = 13] [outer = 0xe9b72160]
[task 2018-09-19T09:22:25.312Z] 09:22:25     INFO - TEST-START | dom/payments/test/test_bug1490698.html
[task 2018-09-19T09:22:25.313Z] 09:22:25     INFO - GECKO(1682) | ++DOMWINDOW == 14 (0xe9b83800) [pid = 1767] [serial = 14] [outer = 0xe9b72160]
[task 2018-09-19T09:22:25.489Z] 09:22:25     INFO - GECKO(1682) | MEMORY STAT | vsize 405MB | residentFast 122MB | heapAllocated 10MB
[task 2018-09-19T09:22:25.525Z] 09:22:25     INFO - TEST-OK | dom/payments/test/test_bug1490698.html | took 804ms
[task 2018-09-19T09:22:25.585Z] 09:22:25     INFO - GECKO(1682) | ++DOMWINDOW == 15 (0xf7093400) [pid = 1767] [serial = 15] [outer = 0xe9b72160]
[task 2018-09-19T09:22:25.621Z] 09:22:25     INFO - TEST-START | dom/payments/test/test_canMakePayment.html
[task 2018-09-19T09:22:25.658Z] 09:22:25     INFO - GECKO(1682) | --DOMWINDOW == 14 (0xe9b75800) [pid = 1767] [serial = 6] [outer = (nil)] [url = about:blank]
[task 2018-09-19T09:22:25.658Z] 09:22:25     INFO - GECKO(1682) | --DOMWINDOW == 13 (0xe9b37400) [pid = 1767] [serial = 2] [outer = (nil)] [url = about:blank]
[task 2018-09-19T09:22:25.778Z] 09:22:25     INFO - GECKO(1682) | ++DOMWINDOW == 14 (0xe9b7f000) [pid = 1767] [serial = 16] [outer = 0xe9b72160]
[task 2018-09-19T09:22:25.897Z] 09:22:25     INFO - GECKO(1682) | ++DOCSHELL 0xdf445000 == 6 [pid = 1682] [id = {d42d2c13-db99-4989-ab31-f536c9db5fb1}]
[task 2018-09-19T09:22:25.898Z] 09:22:25     INFO - GECKO(1682) | ++DOMWINDOW == 19 (0xde19c4c0) [pid = 1682] [serial = 19] [outer = (nil)]
[task 2018-09-19T09:22:25.898Z] 09:22:25     INFO - GECKO(1682) | ++DOMWINDOW == 20 (0xdf446000) [pid = 1682] [serial = 20] [outer = 0xde19c4c0]
[task 2018-09-19T09:22:25.919Z] 09:22:25     INFO - GECKO(1682) | JavaScript error: , line 0: AbortError: The operation was aborted.
[task 2018-09-19T09:22:25.920Z] 09:22:25     INFO - GECKO(1682) | ++DOCSHELL 0xdf447000 == 7 [pid = 1682] [id = {fd48e8c9-4a29-49d5-b38e-28c2d87cb9b9}]
[task 2018-09-19T09:22:25.921Z] 09:22:25     INFO - GECKO(1682) | ++DOMWINDOW == 21 (0xde19c700) [pid = 1682] [serial = 21] [outer = (nil)]
[task 2018-09-19T09:22:25.922Z] 09:22:25     INFO - GECKO(1682) | ++DOMWINDOW == 22 (0xdf448800) [pid = 1682] [serial = 22] [outer = 0xde19c700]
[task 2018-09-19T09:22:25.947Z] 09:22:25     INFO - GECKO(1682) | JavaScript error: , line 0: AbortError: The operation was aborted.
[task 2018-09-19T09:22:27.681Z] 09:22:27     INFO - GECKO(1682) | MEMORY STAT | vsize 405MB | residentFast 123MB | heapAllocated 11MB
[task 2018-09-19T09:22:27.697Z] 09:22:27     INFO - TEST-OK | dom/payments/test/test_canMakePayment.html | took 2070ms
[task 2018-09-19T09:22:27.814Z] 09:22:27     INFO - GECKO(1682) | ++DOMWINDOW == 15 (0xe8bbbc00) [pid = 1767] [serial = 17] [outer = 0xe9b72160]
[task 2018-09-19T09:22:27.883Z] 09:22:27     INFO - TEST-START | dom/payments/test/test_closePayment.html
[task 2018-09-19T09:22:28.222Z] 09:22:28     INFO - GECKO(1682) | --DOMWINDOW == 14 (0xe9b7fc00) [pid = 1767] [serial = 11] [outer = (nil)] [url = https://example.com/tests/SimpleTest/iframe-between-tests.html]
[task 2018-09-19T09:22:28.448Z] 09:22:28     INFO - GECKO(1682) | ++DOMWINDOW == 15 (0xe8bb3400) [pid = 1767] [serial = 18] [outer = 0xe9b72160]
[task 2018-09-19T09:22:28.700Z] 09:22:28     INFO - GECKO(1682) | ++DOCSHELL 0xe8bb7400 == 3 [pid = 1767] [id = {2d860045-108a-460c-a893-7c0ed53030d6}]
[task 2018-09-19T09:22:28.701Z] 09:22:28     INFO - GECKO(1682) | ++DOMWINDOW == 16 (0xe8b37040) [pid = 1767] [serial = 19] [outer = (nil)]
[task 2018-09-19T09:22:28.702Z] 09:22:28     INFO - GECKO(1682) | ++DOMWINDOW == 17 (0xe8bb7c00) [pid = 1767] [serial = 20] [outer = 0xe8b37040]
[task 2018-09-19T09:22:28.741Z] 09:22:28     INFO - GECKO(1682) | --DOMWINDOW == 16 (0xe9b7c800) [pid = 1767] [serial = 8] [outer = (nil)] [url = https://example.com/tests/dom/payments/test/test_abortPayment.html]
[task 2018-09-19T09:22:28.744Z] 09:22:28     INFO - GECKO(1682) | --DOMWINDOW == 15 (0xe9b7c000) [pid = 1767] [serial = 7] [outer = (nil)] [url = about:blank]
[task 2018-09-19T09:22:28.746Z] 09:22:28     INFO - GECKO(1682) | --DOMWINDOW == 14 (0xe9b82400) [pid = 1767] [serial = 9] [outer = (nil)] [url = https://example.com/tests/SimpleTest/iframe-between-tests.html]
[task 2018-09-19T09:22:28.747Z] 09:22:28     INFO - GECKO(1682) | --DOMWINDOW == 13 (0xe8fedc00) [pid = 1767] [serial = 10] [outer = (nil)] [url = https://example.com/tests/dom/payments/test/test_basiccard.html]
[task 2018-09-19T09:22:28.748Z] 09:22:28     INFO - GECKO(1682) | --DOMWINDOW == 12 (0xe9b3f400) [pid = 1767] [serial = 12] [outer = (nil)] [url = https://example.com/tests/dom/payments/test/test_block_none10s.html]
[task 2018-09-19T09:22:28.750Z] 09:22:28     INFO - GECKO(1682) | --DOMWINDOW == 11 (0xe8fee400) [pid = 1767] [serial = 3] [outer = (nil)] [url = about:blank]
[task 2018-09-19T09:22:29.085Z] 09:22:29     INFO - GECKO(1682) | ++DOMWINDOW == 12 (0xe8bb9c00) [pid = 1767] [serial = 21] [outer = 0xe8b37040]
[task 2018-09-19T09:22:29.121Z] 09:22:29     INFO - GECKO(1682) | [Child 1767, Main Thread] WARNING: NS_ENSURE_SUCCESS(rv, false) failed with result 0x80004005: file /builds/worker/workspace/build/src/docshell/shistory/nsSHistory.cpp, line 1291
[task 2018-09-19T09:22:29.123Z] 09:22:29     INFO - GECKO(1682) | [Child 1767, Main Thread] WARNING: NS_ENSURE_SUCCESS(rv, false) failed with result 0x80004005: file /builds/worker/workspace/build/src/docshell/shistory/nsSHistory.cpp, line 1291
[task 2018-09-19T09:22:29.141Z] 09:22:29     INFO - GECKO(1682) | ++DOCSHELL 0xe8bba000 == 4 [pid = 1767] [id = {f94c65cb-d956-4691-a356-23d09d2d8c76}]
[task 2018-09-19T09:22:29.143Z] 09:22:29     INFO - GECKO(1682) | ++DOMWINDOW == 13 (0xe8b37160) [pid = 1767] [serial = 22] [outer = (nil)]
[task 2018-09-19T09:22:29.144Z] 09:22:29     INFO - GECKO(1682) | ++DOMWINDOW == 14 (0xe8bba800) [pid = 1767] [serial = 23] [outer = 0xe8b37160]
[task 2018-09-19T09:22:29.446Z] 09:22:29     INFO - GECKO(1682) | ++DOMWINDOW == 15 (0xe8bbc400) [pid = 1767] [serial = 24] [outer = 0xe8b37160]
[task 2018-09-19T09:22:29.507Z] 09:22:29     INFO - GECKO(1682) | [Child 1767, Main Thread] WARNING: NS_ENSURE_SUCCESS(rv, false) failed with result 0x80004005: file /builds/worker/workspace/build/src/docshell/shistory/nsSHistory.cpp, line 1291
[task 2018-09-19T09:22:29.508Z] 09:22:29     INFO - GECKO(1682) | [Child 1767, Main Thread] WARNING: NS_ENSURE_SUCCESS(rv, false) failed with result 0x80004005: file /builds/worker/workspace/build/src/docshell/shistory/nsSHistory.cpp, line 1291
[task 2018-09-19T09:22:29.525Z] 09:22:29     INFO - GECKO(1682) | ++DOCSHELL 0xe8bbfc00 == 5 [pid = 1767] [id = {05c9c5dd-09fa-48ae-8267-22f054f85f76}]
[task 2018-09-19T09:22:29.527Z] 09:22:29     INFO - GECKO(1682) | ++DOMWINDOW == 16 (0xe8b37280) [pid = 1767] [serial = 25] [outer = (nil)]
[task 2018-09-19T09:22:29.528Z] 09:22:29     INFO - GECKO(1682) | ++DOMWINDOW == 17 (0xe8bc0400) [pid = 1767] [serial = 26] [outer = 0xe8b37280]
[task 2018-09-19T09:22:29.727Z] 09:22:29     INFO - GECKO(1682) | ++DOMWINDOW == 18 (0xe8bc0800) [pid = 1767] [serial = 27] [outer = 0xe8b37280]
[task 2018-09-19T09:22:29.836Z] 09:22:29     INFO - GECKO(1682) | [Child 1767, Main Thread] WARNING: NS_ENSURE_SUCCESS(rv, false) failed with result 0x80004005: file /builds/worker/workspace/build/src/docshell/shistory/nsSHistory.cpp, line 1291
[task 2018-09-19T09:22:29.838Z] 09:22:29     INFO - GECKO(1682) | [Child 1767, Main Thread] WARNING: NS_ENSURE_SUCCESS(rv, false) failed with result 0x80004005: file /builds/worker/workspace/build/src/docshell/shistory/nsSHistory.cpp, line 1291
[task 2018-09-19T09:22:29.854Z] 09:22:29     INFO - GECKO(1682) | ++DOCSHELL 0xe8bc1c00 == 6 [pid = 1767] [id = {febdcd83-deca-48da-8817-16b6ebfbfff1}]
[task 2018-09-19T09:22:29.855Z] 09:22:29     INFO - GECKO(1682) | ++DOMWINDOW == 19 (0xe8b373a0) [pid = 1767] [serial = 28] [outer = (nil)]
[task 2018-09-19T09:22:29.856Z] 09:22:29     INFO - GECKO(1682) | ++DOMWINDOW == 20 (0xe8fecc00) [pid = 1767] [serial = 29] [outer = 0xe8b373a0]
[task 2018-09-19T09:22:30.036Z] 09:22:30     INFO - GECKO(1682) | [Child 1767, Main Thread] WARNING: NS_ENSURE_SUCCESS(rv, false) failed with result 0x80004005: file /builds/worker/workspace/build/src/docshell/shistory/nsSHistory.cpp, line 1291
[task 2018-09-19T09:22:30.036Z] 09:22:30     INFO - GECKO(1682) | [Child 1767, Main Thread] WARNING: NS_ENSURE_SUCCESS(rv, false) failed with result 0x80004005: file /builds/worker/workspace/build/src/docshell/shistory/nsSHistory.cpp, line 1291
[task 2018-09-19T09:22:30.085Z] 09:22:30     INFO - GECKO(1682) | ++DOCSHELL 0xe8fed000 == 7 [pid = 1767] [id = {4289b564-12f3-4ea0-943f-0e525909c98b}]
[task 2018-09-19T09:22:30.085Z] 09:22:30     INFO - GECKO(1682) | ++DOMWINDOW == 21 (0xe8b374c0) [pid = 1767] [serial = 30] [outer = (nil)]
[task 2018-09-19T09:22:30.086Z] 09:22:30     INFO - GECKO(1682) | ++DOMWINDOW == 22 (0xe8fee000) [pid = 1767] [serial = 31] [outer = 0xe8b374c0]
[task 2018-09-19T09:22:30.243Z] 09:22:30     INFO - GECKO(1682) | [Child 1767, Main Thread] WARNING: NS_ENSURE_SUCCESS(rv, false) failed with result 0x80004005: file /builds/worker/workspace/build/src/docshell/shistory/nsSHistory.cpp, line 1291
[task 2018-09-19T09:22:30.244Z] 09:22:30     INFO - GECKO(1682) | [Child 1767, Main Thread] WARNING: NS_ENSURE_SUCCESS(rv, false) failed with result 0x80004005: file /builds/worker/workspace/build/src/docshell/shistory/nsSHistory.cpp, line 1291
[task 2018-09-19T09:22:30.259Z] 09:22:30     INFO - GECKO(1682) | MEMORY STAT | vsize 406MB | residentFast 123MB | heapAllocated 11MB
[task 2018-09-19T09:22:30.282Z] 09:22:30     INFO - TEST-OK | dom/payments/test/test_closePayment.html | took 2398ms
[task 2018-09-19T09:22:30.310Z] 09:22:30     INFO - GECKO(1682) | ++DOMWINDOW == 23 (0xe9b37400) [pid = 1767] [serial = 32] [outer = 0xe9b72160]
[task 2018-09-19T09:22:30.368Z] 09:22:30     INFO - TEST-START | dom/payments/test/test_constructor.html
[task 2018-09-19T09:22:30.558Z] 09:22:30     INFO - GECKO(1682) | ++DOMWINDOW == 24 (0xe8bb8000) [pid = 1767] [serial = 33] [outer = 0xe9b72160]
[task 2018-09-19T09:22:30.876Z] 09:22:30     INFO - GECKO(1682) | --DOMWINDOW == 21 (0xe8553000) [pid = 1682] [serial = 2] [outer = (nil)] [url = about:blank]
[task 2018-09-19T09:22:30.878Z] 09:22:30     INFO - GECKO(1682) | --DOMWINDOW == 20 (0xe443e000) [pid = 1682] [serial = 8] [outer = (nil)] [url = about:blank]
[task 2018-09-19T09:22:30.878Z] 09:22:30     INFO - GECKO(1682) | --DOMWINDOW == 19 (0xe2c2c000) [pid = 1682] [serial = 14] [outer = (nil)] [url = about:blank]
[task 2018-09-19T09:22:30.879Z] 09:22:30     INFO - GECKO(1682) | --DOMWINDOW == 18 (0xe827f400) [pid = 1682] [serial = 15] [outer = (nil)] [url = about:blank]
[task 2018-09-19T09:22:30.895Z] 09:22:30     INFO - GECKO(1682) | ++DOCSHELL 0xe8bbc000 == 8 [pid = 1767] [id = {9b3d7053-b36e-4495-9386-119ab6972ca6}]
[task 2018-09-19T09:22:30.896Z] 09:22:30     INFO - GECKO(1682) | ++DOMWINDOW == 25 (0xe8b37700) [pid = 1767] [serial = 34] [outer = (nil)]
[task 2018-09-19T09:22:30.904Z] 09:22:30     INFO - GECKO(1682) | ++DOMWINDOW == 26 (0xe8e93800) [pid = 1767] [serial = 35] [outer = 0xe8b37700]
[task 2018-09-19T09:22:31.188Z] 09:22:31     INFO - GECKO(1682) | --DOCSHELL 0xdf445000 == 6 [pid = 1682] [id = {d42d2c13-db99-4989-ab31-f536c9db5fb1}]
[task 2018-09-19T09:22:31.189Z] 09:22:31     INFO - GECKO(1682) | --DOCSHELL 0xdf447000 == 5 [pid = 1682] [id = {fd48e8c9-4a29-49d5-b38e-28c2d87cb9b9}]
[task 2018-09-19T09:22:31.190Z] 09:22:31     INFO - GECKO(1682) | --DOMWINDOW == 17 (0xe3052940) [pid = 1682] [serial = 10] [outer = (nil)] [url = about:blank]
[task 2018-09-19T09:22:31.191Z] 09:22:31     INFO - GECKO(1682) | --DOMWINDOW == 16 (0xde19c3a0) [pid = 1682] [serial = 17] [outer = (nil)] [url = about:blank]
[task 2018-09-19T09:22:31.288Z] 09:22:31     INFO - GECKO(1682) | ++DOMWINDOW == 27 (0xe9b34400) [pid = 1767] [serial = 36] [outer = 0xe8b37700]
[task 2018-09-19T09:22:31.352Z] 09:22:31     INFO - GECKO(1682) | MEMORY STAT | vsize 406MB | residentFast 124MB | heapAllocated 11MB
[task 2018-09-19T09:22:31.429Z] 09:22:31     INFO - GECKO(1682) | --DOMWINDOW == 5 (0xe8f8fc00) [pid = 1742] [serial = 5] [outer = (nil)] [url = about:blank]
[task 2018-09-19T09:22:31.431Z] 09:22:31     INFO - GECKO(1682) | --DOMWINDOW == 4 (0xe9489400) [pid = 1742] [serial = 2] [outer = (nil)] [url = about:blank]
[task 2018-09-19T09:22:31.431Z] 09:22:31     INFO - TEST-OK | dom/payments/test/test_constructor.html | took 1067ms
[task 2018-09-19T09:22:31.467Z] 09:22:31     INFO - GECKO(1682) | [Child 1767, Main Thread] WARNING: NS_ENSURE_SUCCESS(rv, false) failed with result 0x80004005: file /builds/worker/workspace/build/src/docshell/shistory/nsSHistory.cpp, line 1291
[task 2018-09-19T09:22:31.469Z] 09:22:31     INFO - GECKO(1682) | [Child 1767, Main Thread] WARNING: NS_ENSURE_SUCCESS(rv, false) failed with result 0x80004005: file /builds/worker/workspace/build/src/docshell/shistory/nsSHistory.cpp, line 1291
[task 2018-09-19T09:22:31.470Z] 09:22:31     INFO - GECKO(1682) | ++DOMWINDOW == 28 (0xe9b75800) [pid = 1767] [serial = 37] [outer = 0xe9b72160]
[task 2018-09-19T09:22:31.526Z] 09:22:31     INFO - TEST-START | dom/payments/test/test_currency_amount_validation.html
[task 2018-09-19T09:22:31.725Z] 09:22:31     INFO - GECKO(1682) | ++DOMWINDOW == 29 (0xe8bbac00) [pid = 1767] [serial = 38] [outer = 0xe9b72160]
[task 2018-09-19T09:22:31.970Z] 09:22:31     INFO - GECKO(1682) | MEMORY STAT | vsize 406MB | residentFast 125MB | heapAllocated 12MB
[task 2018-09-19T09:22:31.986Z] 09:22:31     INFO - TEST-OK | dom/payments/test/test_currency_amount_validation.html | took 466ms
[task 2018-09-19T09:22:32.034Z] 09:22:32     INFO - GECKO(1682) | ++DOMWINDOW == 30 (0xe8fec800) [pid = 1767] [serial = 39] [outer = 0xe9b72160]
[task 2018-09-19T09:22:32.092Z] 09:22:32     INFO - TEST-START | dom/payments/test/test_payment-request-in-iframe.html
[task 2018-09-19T09:22:32.271Z] 09:22:32     INFO - GECKO(1682) | ++DOMWINDOW == 31 (0xe8bc0000) [pid = 1767] [serial = 40] [outer = 0xe9b72160]
[task 2018-09-19T09:22:32.349Z] 09:22:32     INFO - GECKO(1682) | ++DOCSHELL 0xe8ff2400 == 9 [pid = 1767] [id = {9556da84-faf3-49c6-a83a-a709b56d95bb}]
[task 2018-09-19T09:22:32.350Z] 09:22:32     INFO - GECKO(1682) | ++DOMWINDOW == 32 (0xe8b37820) [pid = 1767] [serial = 41] [outer = (nil)]
[task 2018-09-19T09:22:32.350Z] 09:22:32     INFO - GECKO(1682) | ++DOMWINDOW == 33 (0xe9b3f400) [pid = 1767] [serial = 42] [outer = 0xe8b37820]
[task 2018-09-19T09:22:32.411Z] 09:22:32     INFO - GECKO(1682) | ++DOCSHELL 0xe9b79800 == 10 [pid = 1767] [id = {392f7f96-1101-4247-915c-799873a81afd}]
[task 2018-09-19T09:22:32.413Z] 09:22:32     INFO - GECKO(1682) | ++DOMWINDOW == 34 (0xe8b38ee0) [pid = 1767] [serial = 43] [outer = (nil)]
[task 2018-09-19T09:22:32.415Z] 09:22:32     INFO - GECKO(1682) | ++DOMWINDOW == 35 (0xe9b7c800) [pid = 1767] [serial = 44] [outer = 0xe8b38ee0]
[task 2018-09-19T09:22:32.534Z] 09:22:32     INFO - GECKO(1682) | ++DOMWINDOW == 36 (0xe9b7e400) [pid = 1767] [serial = 45] [outer = 0xe8b38ee0]
[task 2018-09-19T09:22:32.563Z] 09:22:32     INFO - GECKO(1682) | ++DOCSHELL 0xe9b7d400 == 11 [pid = 1767] [id = {b9fb60f6-43cd-46e4-ab3e-d3cc1647dc57}]
[task 2018-09-19T09:22:32.565Z] 09:22:32     INFO - GECKO(1682) | ++DOMWINDOW == 37 (0xe7c4c280) [pid = 1767] [serial = 46] [outer = (nil)]
[task 2018-09-19T09:22:32.566Z] 09:22:32     INFO - GECKO(1682) | ++DOMWINDOW == 38 (0xe9b7ec00) [pid = 1767] [serial = 47] [outer = 0xe7c4c280]
[task 2018-09-19T09:22:32.611Z] 09:22:32     INFO - GECKO(1682) | ++DOMWINDOW == 39 (0xe9b82800) [pid = 1767] [serial = 48] [outer = 0xe7c4c280]
[task 2018-09-19T09:22:32.652Z] 09:22:32     INFO - GECKO(1682) | ++DOCSHELL 0xe9b7f800 == 12 [pid = 1767] [id = {625b2bd6-0eae-430b-9e34-3a0758ac79b0}]
[task 2018-09-19T09:22:32.655Z] 09:22:32     INFO - GECKO(1682) | ++DOMWINDOW == 40 (0xe7c79ee0) [pid = 1767] [serial = 49] [outer = (nil)]
[task 2018-09-19T09:22:32.657Z] 09:22:32     INFO - GECKO(1682) | ++DOMWINDOW == 41 (0xe9b83400) [pid = 1767] [serial = 50] [outer = 0xe7c79ee0]
[task 2018-09-19T09:22:32.908Z] 09:22:32     INFO - GECKO(1682) | ++DOMWINDOW == 42 (0xe9b75400) [pid = 1767] [serial = 51] [outer = 0xe7c79ee0]
[task 2018-09-19T09:22:33.025Z] 09:22:33     INFO - GECKO(1682) | ++DOMWINDOW == 43 (0xe9b7f400) [pid = 1767] [serial = 52] [outer = 0xe7c79ee0]
[task 2018-09-19T09:22:33.074Z] 09:22:33     INFO - GECKO(1682) | ++DOCSHELL 0xe8bb7800 == 13 [pid = 1767] [id = {cefd3dae-fc4e-4511-a8a4-6db3d947538f}]
[task 2018-09-19T09:22:33.075Z] 09:22:33     INFO - GECKO(1682) | ++DOMWINDOW == 44 (0xe7cb8040) [pid = 1767] [serial = 53] [outer = (nil)]
[task 2018-09-19T09:22:33.076Z] 09:22:33     INFO - GECKO(1682) | ++DOMWINDOW == 45 (0xe9b84000) [pid = 1767] [serial = 54] [outer = 0xe7cb8040]
[task 2018-09-19T09:22:33.195Z] 09:22:33     INFO - GECKO(1682) | ++DOMWINDOW == 46 (0xf708f400) [pid = 1767] [serial = 55] [outer = 0xe7cb8040]
[task 2018-09-19T09:22:33.252Z] 09:22:33     INFO - GECKO(1682) | ++DOCSHELL 0xe8bb8400 == 14 [pid = 1767] [id = {a7d34f04-171e-42e8-a815-b43b4b8ccac2}]
[task 2018-09-19T09:22:33.253Z] 09:22:33     INFO - GECKO(1682) | ++DOMWINDOW == 47 (0xe6e473a0) [pid = 1767] [serial = 56] [outer = (nil)]
[task 2018-09-19T09:22:33.253Z] 09:22:33     INFO - GECKO(1682) | ++DOMWINDOW == 48 (0xf7094000) [pid = 1767] [serial = 57] [outer = 0xe6e473a0]
[task 2018-09-19T09:22:33.312Z] 09:22:33     INFO - GECKO(1682) | --DOMWINDOW == 47 (0xe8b37040) [pid = 1767] [serial = 19] [outer = (nil)] [url = https://example.com/tests/dom/payments/test/simple_payment_request.html]
[task 2018-09-19T09:22:33.314Z] 09:22:33     INFO - GECKO(1682) | --DOMWINDOW == 46 (0xe8b37160) [pid = 1767] [serial = 22] [outer = (nil)] [url = https://example.com/tests/dom/payments/test/blank_page.html]
[task 2018-09-19T09:22:33.314Z] 09:22:33     INFO - GECKO(1682) | --DOMWINDOW == 45 (0xe8b37280) [pid = 1767] [serial = 25] [outer = (nil)] [url = https://example.com/tests/dom/payments/test/blank_page.html]
[task 2018-09-19T09:22:33.314Z] 09:22:33     INFO - GECKO(1682) | --DOMWINDOW == 44 (0xe8b37700) [pid = 1767] [serial = 34] [outer = (nil)] [url = https://test1.example.com/tests/dom/payments/test/simple_payment_request.html]
[task 2018-09-19T09:22:33.315Z] 09:22:33     INFO - GECKO(1682) | --DOMWINDOW == 43 (0xe8b374c0) [pid = 1767] [serial = 30] [outer = (nil)] [url = https://example.com/tests/dom/payments/test/simple_payment_request.html]
[task 2018-09-19T09:22:33.317Z] 09:22:33     INFO - GECKO(1682) | --DOMWINDOW == 42 (0xe8b373a0) [pid = 1767] [serial = 28] [outer = (nil)] [url = https://example.com/tests/dom/payments/test/simple_payment_request.html]
[task 2018-09-19T09:22:33.369Z] 09:22:33     INFO - GECKO(1682) | ++DOMWINDOW == 43 (0xf7096400) [pid = 1767] [serial = 58] [outer = 0xe6e473a0]
[task 2018-09-19T09:22:33.459Z] 09:22:33     INFO - GECKO(1682) | [Child 1767, Main Thread] WARNING: NS_ENSURE_SUCCESS(rv, false) failed with result 0x80004005: file /builds/worker/workspace/build/src/docshell/shistory/nsSHistory.cpp, line 1291
[task 2018-09-19T09:22:33.461Z] 09:22:33     INFO - GECKO(1682) | [Child 1767, Main Thread] WARNING: NS_ENSURE_SUCCESS(rv, false) failed with result 0x80004005: file /builds/worker/workspace/build/src/docshell/shistory/nsSHistory.cpp, line 1291
[task 2018-09-19T09:22:33.463Z] 09:22:33     INFO - GECKO(1682) | ++DOMWINDOW == 44 (0xe6e02000) [pid = 1767] [serial = 59] [outer = 0xe7cb8040]
[task 2018-09-19T09:22:33.541Z] 09:22:33     INFO - GECKO(1682) | ++DOCSHELL 0xe6e02400 == 15 [pid = 1767] [id = {20d75f7e-f4c2-4b67-8b95-4c5a5fb68cb1}]
[task 2018-09-19T09:22:33.543Z] 09:22:33     INFO - GECKO(1682) | ++DOMWINDOW == 45 (0xe6e47040) [pid = 1767] [serial = 60] [outer = (nil)]
[task 2018-09-19T09:22:33.545Z] 09:22:33     INFO - GECKO(1682) | ++DOMWINDOW == 46 (0xe6e03000) [pid = 1767] [serial = 61] [outer = 0xe6e47040]
[task 2018-09-19T09:22:33.623Z] 09:22:33     INFO - GECKO(1682) | ++DOMWINDOW == 47 (0xe6e04c00) [pid = 1767] [serial = 62] [outer = 0xe6e47040]
[task 2018-09-19T09:22:33.700Z] 09:22:33     INFO - GECKO(1682) | MEMORY STAT | vsize 409MB | residentFast 127MB | heapAllocated 13MB
[task 2018-09-19T09:22:33.803Z] 09:22:33     INFO - TEST-OK | dom/payments/test/test_payment-request-in-iframe.html | took 1714ms
[task 2018-09-19T09:22:33.844Z] 09:22:33     INFO - GECKO(1682) | [Child 1767, Main Thread] WARNING: NS_ENSURE_SUCCESS(rv, false) failed with result 0x80004005: file /builds/worker/workspace/build/src/docshell/shistory/nsSHistory.cpp, line 1291
[task 2018-09-19T09:22:33.845Z] 09:22:33     INFO - GECKO(1682) | [Child 1767, Main Thread] WARNING: NS_ENSURE_SUCCESS(rv, false) failed with result 0x80004005: file /builds/worker/workspace/build/src/docshell/shistory/nsSHistory.cpp, line 1291
[task 2018-09-19T09:22:33.847Z] 09:22:33     INFO - GECKO(1682) | ++DOMWINDOW == 48 (0xe6e06000) [pid = 1767] [serial = 63] [outer = 0xe9b72160]
[task 2018-09-19T09:22:33.904Z] 09:22:33     INFO - TEST-START | dom/payments/test/test_pmi_validation.html
[task 2018-09-19T09:22:34.127Z] 09:22:34     INFO - GECKO(1682) | ++DOMWINDOW == 49 (0xe6e06400) [pid = 1767] [serial = 64] [outer = 0xe9b72160]
[task 2018-09-19T09:22:34.671Z] 09:22:34     INFO - GECKO(1682) | --DOMWINDOW == 48 (0xe9b34400) [pid = 1767] [serial = 36] [outer = (nil)] [url = https://test1.example.com/tests/dom/payments/test/simple_payment_request.html]
[task 2018-09-19T09:22:34.671Z] 09:22:34     INFO - GECKO(1682) | --DOMWINDOW == 47 (0xe8fecc00) [pid = 1767] [serial = 29] [outer = (nil)] [url = about:blank]
[task 2018-09-19T09:22:34.672Z] 09:22:34     INFO - GECKO(1682) | --DOMWINDOW == 46 (0xe9b75800) [pid = 1767] [serial = 37] [outer = (nil)] [url = https://example.com/tests/SimpleTest/iframe-between-tests.html]
[task 2018-09-19T09:22:34.672Z] 09:22:34     INFO - GECKO(1682) | --DOMWINDOW == 45 (0xe8bc0800) [pid = 1767] [serial = 27] [outer = (nil)] [url = https://example.com/tests/dom/payments/test/blank_page.html]
[task 2018-09-19T09:22:34.672Z] 09:22:34     INFO - GECKO(1682) | --DOMWINDOW == 44 (0xe8fee000) [pid = 1767] [serial = 31] [outer = (nil)] [url = about:blank]
[task 2018-09-19T09:22:34.672Z] 09:22:34     INFO - GECKO(1682) | --DOMWINDOW == 43 (0xe8e93800) [pid = 1767] [serial = 35] [outer = (nil)] [url = about:blank]
[task 2018-09-19T09:22:34.673Z] 09:22:34     INFO - GECKO(1682) | --DOMWINDOW == 42 (0xe8bbbc00) [pid = 1767] [serial = 17] [outer = (nil)] [url = https://example.com/tests/SimpleTest/iframe-between-tests.html]
[task 2018-09-19T09:22:34.675Z] 09:22:34     INFO - GECKO(1682) | --DOMWINDOW == 41 (0xe8bc0400) [pid = 1767] [serial = 26] [outer = (nil)] [url = about:blank]
[task 2018-09-19T09:22:34.677Z] 09:22:34     INFO - GECKO(1682) | --DOMWINDOW == 40 (0xe8bbc400) [pid = 1767] [serial = 24] [outer = (nil)] [url = https://example.com/tests/dom/payments/test/blank_page.html]
[task 2018-09-19T09:22:34.678Z] 09:22:34     INFO - GECKO(1682) | --DOMWINDOW == 39 (0xe8bb7c00) [pid = 1767] [serial = 20] [outer = (nil)] [url = about:blank]
[task 2018-09-19T09:22:34.679Z] 09:22:34     INFO - GECKO(1682) | --DOMWINDOW == 38 (0xe9b37400) [pid = 1767] [serial = 32] [outer = (nil)] [url = https://example.com/tests/SimpleTest/iframe-between-tests.html]
[task 2018-09-19T09:22:34.685Z] 09:22:34     INFO - GECKO(1682) | --DOMWINDOW == 37 (0xe8bba800) [pid = 1767] [serial = 23] [outer = (nil)] [url = about:blank]
[task 2018-09-19T09:22:34.687Z] 09:22:34     INFO - GECKO(1682) | --DOMWINDOW == 36 (0xe9b83800) [pid = 1767] [serial = 14] [outer = (nil)] [url = https://example.com/tests/dom/payments/test/test_bug1490698.html]
[task 2018-09-19T09:22:34.688Z] 09:22:34     INFO - GECKO(1682) | --DOMWINDOW == 35 (0xe9b3fc00) [pid = 1767] [serial = 13] [outer = (nil)] [url = https://example.com/tests/SimpleTest/iframe-between-tests.html]
[task 2018-09-19T09:22:34.689Z] 09:22:34     INFO - GECKO(1682) | --DOMWINDOW == 34 (0xf7093400) [pid = 1767] [serial = 15] [outer = (nil)] [url = https://example.com/tests/SimpleTest/iframe-between-tests.html]
[task 2018-09-19T09:22:34.690Z] 09:22:34     INFO - GECKO(1682) | --DOMWINDOW == 33 (0xe8bb9c00) [pid = 1767] [serial = 21] [outer = (nil)] [url = https://example.com/tests/dom/payments/test/simple_payment_request.html]
[task 2018-09-19T09:22:34.739Z] 09:22:34     INFO - GECKO(1682) | MEMORY STAT | vsize 409MB | residentFast 127MB | heapAllocated 11MB
[task 2018-09-19T09:22:34.755Z] 09:22:34     INFO - TEST-OK | dom/payments/test/test_pmi_validation.html | took 846ms
[task 2018-09-19T09:22:34.796Z] 09:22:34     INFO - GECKO(1682) | ++DOMWINDOW == 34 (0xe6e07800) [pid = 1767] [serial = 65] [outer = 0xe9b72160]
[task 2018-09-19T09:22:34.832Z] 09:22:34     INFO - TEST-START | dom/payments/test/test_requestShipping.html
[task 2018-09-19T09:22:35.055Z] 09:22:35     INFO - GECKO(1682) | ++DOMWINDOW == 35 (0xe6e06800) [pid = 1767] [serial = 66] [outer = 0xe9b72160]
[task 2018-09-19T09:22:35.286Z] 09:22:35     INFO - GECKO(1682) | MEMORY STAT | vsize 411MB | residentFast 128MB | heapAllocated 13MB
[task 2018-09-19T09:22:35.302Z] 09:22:35     INFO - TEST-OK | dom/payments/test/test_requestShipping.html | took 470ms
[task 2018-09-19T09:22:35.326Z] 09:22:35     INFO - GECKO(1682) | --DOCSHELL 0xe9b7f800 == 14 [pid = 1767] [id = {625b2bd6-0eae-430b-9e34-3a0758ac79b0}]
[task 2018-09-19T09:22:35.327Z] 09:22:35     INFO - GECKO(1682) | --DOCSHELL 0xe9b7d400 == 13 [pid = 1767] [id = {b9fb60f6-43cd-46e4-ab3e-d3cc1647dc57}]
[task 2018-09-19T09:22:35.328Z] 09:22:35     INFO - GECKO(1682) | --DOCSHELL 0xe9b79800 == 12 [pid = 1767] [id = {392f7f96-1101-4247-915c-799873a81afd}]
[task 2018-09-19T09:22:35.331Z] 09:22:35     INFO - GECKO(1682) | --DOCSHELL 0xe8ff2400 == 11 [pid = 1767] [id = {9556da84-faf3-49c6-a83a-a709b56d95bb}]
[task 2018-09-19T09:22:35.332Z] 09:22:35     INFO - GECKO(1682) | --DOCSHELL 0xe8bbc000 == 10 [pid = 1767] [id = {9b3d7053-b36e-4495-9386-119ab6972ca6}]
[task 2018-09-19T09:22:35.334Z] 09:22:35     INFO - GECKO(1682) | --DOCSHELL 0xe8fed000 == 9 [pid = 1767] [id = {4289b564-12f3-4ea0-943f-0e525909c98b}]
[task 2018-09-19T09:22:35.335Z] 09:22:35     INFO - GECKO(1682) | --DOCSHELL 0xe8bc1c00 == 8 [pid = 1767] [id = {febdcd83-deca-48da-8817-16b6ebfbfff1}]
[task 2018-09-19T09:22:35.337Z] 09:22:35     INFO - GECKO(1682) | --DOCSHELL 0xe8bbfc00 == 7 [pid = 1767] [id = {05c9c5dd-09fa-48ae-8267-22f054f85f76}]
[task 2018-09-19T09:22:35.338Z] 09:22:35     INFO - GECKO(1682) | --DOCSHELL 0xe8bba000 == 6 [pid = 1767] [id = {f94c65cb-d956-4691-a356-23d09d2d8c76}]
[task 2018-09-19T09:22:35.339Z] 09:22:35     INFO - GECKO(1682) | --DOCSHELL 0xe8bb7400 == 5 [pid = 1767] [id = {2d860045-108a-460c-a893-7c0ed53030d6}]
[task 2018-09-19T09:22:35.340Z] 09:22:35     INFO - GECKO(1682) | --DOCSHELL 0xe8bb8400 == 4 [pid = 1767] [id = {a7d34f04-171e-42e8-a815-b43b4b8ccac2}]
[task 2018-09-19T09:22:35.382Z] 09:22:35     INFO - GECKO(1682) | ++DOMWINDOW == 36 (0xe6e0c000) [pid = 1767] [serial = 67] [outer = 0xe9b72160]
[task 2018-09-19T09:22:35.419Z] 09:22:35     INFO - TEST-START | dom/payments/test/test_retryPayment.html
[task 2018-09-19T09:22:35.642Z] 09:22:35     INFO - GECKO(1682) | ++DOMWINDOW == 37 (0xe6e0b400) [pid = 1767] [serial = 68] [outer = 0xe9b72160]
[task 2018-09-19T09:22:35.909Z] 09:22:35     INFO - GECKO(1682) | [Child 1767, Main Thread] WARNING: 'NS_FAILED(rv)', file /builds/worker/workspace/build/src/dom/payments/PaymentResponse.cpp, line 246
[task 2018-09-19T09:22:35.967Z] 09:22:35     INFO - GECKO(1682) | --DOMWINDOW == 36 (0xe8bb8000) [pid = 1767] [serial = 33] [outer = (nil)] [url = https://example.com/tests/dom/payments/test/test_constructor.html]
[task 2018-09-19T09:22:35.969Z] 09:22:35     INFO - GECKO(1682) | --DOMWINDOW == 35 (0xe8bb3400) [pid = 1767] [serial = 18] [outer = (nil)] [url = https://example.com/tests/dom/payments/test/test_closePayment.html]
[task 2018-09-19T09:22:35.970Z] 09:22:35     INFO - GECKO(1682) | --DOMWINDOW == 34 (0xe9b7f000) [pid = 1767] [serial = 16] [outer = (nil)] [url = https://example.com/tests/dom/payments/test/test_canMakePayment.html]
[task 2018-09-19T09:22:35.971Z] 09:22:35     INFO - GECKO(1682) | --DOMWINDOW == 33 (0xf7094000) [pid = 1767] [serial = 57] [outer = (nil)] [url = about:blank]
[task 2018-09-19T09:22:35.972Z] 09:22:35     INFO - GECKO(1682) | --DOMWINDOW == 32 (0xe9b84000) [pid = 1767] [serial = 54] [outer = (nil)] [url = about:blank]
[task 2018-09-19T09:22:35.973Z] 09:22:35     INFO - GECKO(1682) | --DOMWINDOW == 31 (0xe9b83400) [pid = 1767] [serial = 50] [outer = (nil)] [url = about:blank]
[task 2018-09-19T09:22:35.974Z] 09:22:35     INFO - GECKO(1682) | --DOMWINDOW == 30 (0xe9b7ec00) [pid = 1767] [serial = 47] [outer = (nil)] [url = about:blank]
[task 2018-09-19T09:22:35.975Z] 09:22:35     INFO - GECKO(1682) | --DOMWINDOW == 29 (0xe9b7c800) [pid = 1767] [serial = 44] [outer = (nil)] [url = about:blank]
[task 2018-09-19T09:22:35.976Z] 09:22:35     INFO - GECKO(1682) | --DOMWINDOW == 28 (0xe8bbac00) [pid = 1767] [serial = 38] [outer = (nil)] [url = https://example.com/tests/dom/payments/test/test_currency_amount_validation.html]
[task 2018-09-19T09:22:35.977Z] 09:22:35     INFO - GECKO(1682) | --DOMWINDOW == 27 (0xe8fec800) [pid = 1767] [serial = 39] [outer = (nil)] [url = https://example.com/tests/SimpleTest/iframe-between-tests.html]
[task 2018-09-19T09:22:36.002Z] 09:22:36     INFO - GECKO(1682) | MEMORY STAT | vsize 411MB | residentFast 126MB | heapAllocated 11MB
[task 2018-09-19T09:22:36.018Z] 09:22:36     INFO - TEST-OK | dom/payments/test/test_retryPayment.html | took 599ms
[task 2018-09-19T09:22:36.115Z] 09:22:36     INFO - GECKO(1682) | ++DOMWINDOW == 28 (0xe6e0f400) [pid = 1767] [serial = 69] [outer = 0xe9b72160]
[task 2018-09-19T09:22:36.163Z] 09:22:36     INFO - TEST-START | dom/payments/test/test_shippingOptions.html
[task 2018-09-19T09:22:36.365Z] 09:22:36     INFO - GECKO(1682) | --DOCSHELL 0xe6e02400 == 3 [pid = 1767] [id = {20d75f7e-f4c2-4b67-8b95-4c5a5fb68cb1}]
[task 2018-09-19T09:22:36.367Z] 09:22:36     INFO - GECKO(1682) | --DOCSHELL 0xe8bb7800 == 2 [pid = 1767] [id = {cefd3dae-fc4e-4511-a8a4-6db3d947538f}]
[task 2018-09-19T09:22:36.368Z] 09:22:36     INFO - GECKO(1682) | --DOMWINDOW == 27 (0xe6e47040) [pid = 1767] [serial = 60] [outer = (nil)] [url = https://example.com/tests/dom/payments/test/simple_payment_request.html]
[task 2018-09-19T09:22:36.369Z] 09:22:36     INFO - GECKO(1682) | --DOMWINDOW == 26 (0xe7c4c280) [pid = 1767] [serial = 46] [outer = (nil)] [url = https://test1.example.com/tests/dom/payments/test/simple_payment_request.html]
[task 2018-09-19T09:22:36.370Z] 09:22:36     INFO - GECKO(1682) | --DOMWINDOW == 25 (0xe8b38ee0) [pid = 1767] [serial = 43] [outer = (nil)] [url = https://test1.example.com/tests/dom/payments/test/simple_payment_request.html]
[task 2018-09-19T09:22:36.371Z] 09:22:36     INFO - GECKO(1682) | --DOMWINDOW == 24 (0xe6e473a0) [pid = 1767] [serial = 56] [outer = (nil)] [url = https://example.com/tests/dom/payments/test/simple_payment_request.html]
[task 2018-09-19T09:22:36.372Z] 09:22:36     INFO - GECKO(1682) | --DOMWINDOW == 23 (0xe7c79ee0) [pid = 1767] [serial = 49] [outer = (nil)] [url = https://test1.example.com/tests/dom/payments/test/echo_payment_request.html]
[task 2018-09-19T09:22:36.373Z] 09:22:36     INFO - GECKO(1682) | --DOMWINDOW == 22 (0xe7cb8040) [pid = 1767] [serial = 53] [outer = (nil)] [url = https://test1.example.com/tests/dom/payments/test/echo_payment_request.html]
[task 2018-09-19T09:22:36.374Z] 09:22:36     INFO - GECKO(1682) | --DOMWINDOW == 21 (0xe8b37820) [pid = 1767] [serial = 41] [outer = (nil)] [url = https://example.com/tests/dom/payments/test/simple_payment_request.html]
[task 2018-09-19T09:22:36.500Z] 09:22:36     INFO - GECKO(1682) | ++DOMWINDOW == 22 (0xe6e05c00) [pid = 1767] [serial = 70] [outer = 0xe9b72160]
[task 2018-09-19T09:22:36.521Z] 09:22:36     INFO - GECKO(1682) | [Parent 1682, StreamTrans #29] WARNING: 'NS_FAILED(rv)', file /builds/worker/workspace/build/src/modules/libjar/nsJARChannel.cpp, line 417
[task 2018-09-19T09:22:36.523Z] 09:22:36     INFO - GECKO(1682) | [Parent 1682, Main Thread] WARNING: NS_ENSURE_SUCCESS(rv, rv) failed with result 0x80520012: file /builds/worker/workspace/build/src/modules/libjar/nsJARChannel.cpp, line 1125
[task 2018-09-19T09:22:36.826Z] 09:22:36     INFO - GECKO(1682) | MEMORY STAT | vsize 411MB | residentFast 127MB | heapAllocated 10MB
[task 2018-09-19T09:22:36.844Z] 09:22:36     INFO - TEST-OK | dom/payments/test/test_shippingOptions.html | took 680ms
[task 2018-09-19T09:22:36.866Z] 09:22:36     INFO - GECKO(1682) | ++DOMWINDOW == 23 (0xe6e0d800) [pid = 1767] [serial = 71] [outer = 0xe9b72160]
[task 2018-09-19T09:22:36.914Z] 09:22:36     INFO - TEST-START | dom/payments/test/test_showPayment.html
[task 2018-09-19T09:22:36.955Z] 09:22:36     INFO - GECKO(1682) | --DOMWINDOW == 22 (0xe6e04c00) [pid = 1767] [serial = 62] [outer = (nil)] [url = https://example.com/tests/dom/payments/test/simple_payment_request.html]
[task 2018-09-19T09:22:36.955Z] 09:22:36     INFO - GECKO(1682) | --DOMWINDOW == 21 (0xe6e02000) [pid = 1767] [serial = 59] [outer = (nil)] [url = https://test1.example.com/tests/dom/payments/test/echo_payment_request.html]
[task 2018-09-19T09:22:36.955Z] 09:22:36     INFO - GECKO(1682) | --DOMWINDOW == 20 (0xe6e03000) [pid = 1767] [serial = 61] [outer = (nil)] [url = about:blank]
[task 2018-09-19T09:22:36.956Z] 09:22:36     INFO - GECKO(1682) | --DOMWINDOW == 19 (0xe9b75400) [pid = 1767] [serial = 51] [outer = (nil)] [url = https://test1.example.com/tests/dom/payments/test/echo_payment_request.html]
[task 2018-09-19T09:22:36.956Z] 09:22:36     INFO - GECKO(1682) | --DOMWINDOW == 18 (0xe9b82800) [pid = 1767] [serial = 48] [outer = (nil)] [url = https://test1.example.com/tests/dom/payments/test/simple_payment_request.html]
[task 2018-09-19T09:22:36.957Z] 09:22:36     INFO - GECKO(1682) | --DOMWINDOW == 17 (0xe9b7e400) [pid = 1767] [serial = 45] [outer = (nil)] [url = https://test1.example.com/tests/dom/payments/test/simple_payment_request.html]
[task 2018-09-19T09:22:36.958Z] 09:22:36     INFO - GECKO(1682) | --DOMWINDOW == 16 (0xf7096400) [pid = 1767] [serial = 58] [outer = (nil)] [url = https://example.com/tests/dom/payments/test/simple_payment_request.html]
[task 2018-09-19T09:22:36.959Z] 09:22:36     INFO - GECKO(1682) | --DOMWINDOW == 15 (0xe9b7f400) [pid = 1767] [serial = 52] [outer = (nil)] [url = https://test1.example.com/tests/dom/payments/test/echo_payment_request.html]
[task 2018-09-19T09:22:36.960Z] 09:22:36     INFO - GECKO(1682) | --DOMWINDOW == 14 (0xf708f400) [pid = 1767] [serial = 55] [outer = (nil)] [url = https://test1.example.com/tests/dom/payments/test/echo_payment_request.html]
[task 2018-09-19T09:22:36.961Z] 09:22:36     INFO - GECKO(1682) | --DOMWINDOW == 13 (0xe6e06800) [pid = 1767] [serial = 66] [outer = (nil)] [url = https://example.com/tests/dom/payments/test/test_requestShipping.html]
[task 2018-09-19T09:22:36.963Z] 09:22:36     INFO - GECKO(1682) | --DOMWINDOW == 12 (0xe6e06000) [pid = 1767] [serial = 63] [outer = (nil)] [url = https://example.com/tests/SimpleTest/iframe-between-tests.html]
[task 2018-09-19T09:22:36.964Z] 09:22:36     INFO - GECKO(1682) | --DOMWINDOW == 11 (0xe6e06400) [pid = 1767] [serial = 64] [outer = (nil)] [url = https://example.com/tests/dom/payments/test/test_pmi_validation.html]
[task 2018-09-19T09:22:36.965Z] 09:22:36     INFO - GECKO(1682) | --DOMWINDOW == 10 (0xe9b3f400) [pid = 1767] [serial = 42] [outer = (nil)] [url = about:blank]
[task 2018-09-19T09:22:36.966Z] 09:22:36     INFO - GECKO(1682) | --DOMWINDOW == 9 (0xe6e07800) [pid = 1767] [serial = 65] [outer = (nil)] [url = https://example.com/tests/SimpleTest/iframe-between-tests.html]
[task 2018-09-19T09:22:36.967Z] 09:22:36     INFO - GECKO(1682) | --DOMWINDOW == 8 (0xe8bc0000) [pid = 1767] [serial = 40] [outer = (nil)] [url = https://example.com/tests/dom/payments/test/test_payment-request-in-iframe.html]
[task 2018-09-19T09:22:37.174Z] 09:22:37     INFO - GECKO(1682) | ++DOMWINDOW == 9 (0xe6e03800) [pid = 1767] [serial = 72] [outer = 0xe9b72160]
[task 2018-09-19T09:22:37.176Z] 09:22:37     INFO - GECKO(1682) | --DOMWINDOW == 15 (0xe3073800) [pid = 1682] [serial = 11] [outer = (nil)] [url = about:blank]
[task 2018-09-19T09:22:37.179Z] 09:22:37     INFO - GECKO(1682) | --DOMWINDOW == 14 (0xde684400) [pid = 1682] [serial = 18] [outer = (nil)] [url = about:blank]
[task 2018-09-19T09:22:37.519Z] 09:22:37     INFO - GECKO(1682) | --DOMWINDOW == 8 (0xe6e0b400) [pid = 1767] [serial = 68] [outer = (nil)] [url = https://example.com/tests/dom/payments/test/test_retryPayment.html]
[task 2018-09-19T09:22:37.520Z] 09:22:37     INFO - GECKO(1682) | --DOMWINDOW == 7 (0xe6e0c000) [pid = 1767] [serial = 67] [outer = (nil)] [url = https://example.com/tests/SimpleTest/iframe-between-tests.html]
[task 2018-09-19T09:22:37.521Z] 09:22:37     INFO - GECKO(1682) | --DOMWINDOW == 6 (0xe6e0f400) [pid = 1767] [serial = 69] [outer = (nil)] [url = https://example.com/tests/SimpleTest/iframe-between-tests.html]
[task 2018-09-19T09:22:37.760Z] 09:22:37     INFO - GECKO(1682) | --DOMWINDOW == 13 (0xde19c4c0) [pid = 1682] [serial = 19] [outer = (nil)] [url = about:blank]
[task 2018-09-19T09:22:37.760Z] 09:22:37     INFO - GECKO(1682) | --DOMWINDOW == 12 (0xde19c700) [pid = 1682] [serial = 21] [outer = (nil)] [url = about:blank]
[task 2018-09-19T09:22:37.796Z] 09:22:37     INFO - GECKO(1682) | [Parent 1682, Main Thread] WARNING: NS_ENSURE_TRUE(aData) failed: file /builds/worker/workspace/build/src/dom/payments/PaymentActionResponse.cpp, line 242
[task 2018-09-19T09:22:37.817Z] 09:22:37     INFO - GECKO(1682) | MEMORY STAT | vsize 411MB | residentFast 126MB | heapAllocated 10MB
[task 2018-09-19T09:22:37.833Z] 09:22:37     INFO - TEST-OK | dom/payments/test/test_showPayment.html | took 917ms
[task 2018-09-19T09:22:37.882Z] 09:22:37     INFO - GECKO(1682) | ++DOMWINDOW == 7 (0xe6e06400) [pid = 1767] [serial = 73] [outer = 0xe9b72160]
[task 2018-09-19T09:22:37.924Z] 09:22:37     INFO - TEST-START | dom/payments/test/test_update_errors.html
[task 2018-09-19T09:22:38.104Z] 09:22:38     INFO - GECKO(1682) | ++DOMWINDOW == 8 (0xe6e03c00) [pid = 1767] [serial = 74] [outer = 0xe9b72160]
[task 2018-09-19T09:22:38.326Z] 09:22:38     INFO - GECKO(1682) | MEMORY STAT | vsize 411MB | residentFast 125MB | heapAllocated 10MB
[task 2018-09-19T09:22:38.347Z] 09:22:38     INFO - TEST-OK | dom/payments/test/test_update_errors.html | took 421ms
[task 2018-09-19T09:22:38.405Z] 09:22:38     INFO - GECKO(1682) | ++DOMWINDOW == 9 (0xe6e08000) [pid = 1767] [serial = 75] [outer = 0xe9b72160]
[task 2018-09-19T09:22:38.450Z] 09:22:38     INFO - TEST-START | Shutdown
Flags: needinfo?(mcaceres)
^^ Backed out changeset 376c2bf5bcbf (Bug 1472026) for PaymentRequest failures in dom/payments/test/test_block_none10s.html ^^
Assignee

Comment 23

8 months ago
Thanks! Will take a look.
Flags: needinfo?(mcaceres)

Comment 24

8 months ago
Backout by shindli@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/c22b1347ccf2
Backed out changeset 376c2bf5bcbf for PaymentRequest failures in  dom/payments/test/test_block_none10s.html

Comment 25

8 months ago
Pushed by mcaceres@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/2a9df8249cc8
Implement PaymentResponse.prototype.onpayerdetailchange. r=edenchuang,baku

Comment 26

8 months ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/2a9df8249cc8
Status: ASSIGNED → RESOLVED
Last Resolved: 8 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla64
You need to log in before you can comment on or make changes to this bug.