Implement PaymentResponse.prototype.onpayerdetailchange

RESOLVED FIXED in Firefox 64

Status

()

P1
normal
RESOLVED FIXED
9 months ago
4 months ago

People

(Reporter: marcosc, Assigned: marcosc)

Tracking

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

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

Firefox Tracking Flags

(firefox64 fixed)

Details

(Whiteboard: [webpayments-reserve], URL)

Attachments

(2 attachments, 6 obsolete attachments)

(Assignee)

Description

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

Updated

9 months ago
Priority: -- → P2
(Assignee)

Updated

9 months ago
Blocks: 1473081
(Assignee)

Updated

7 months ago
Summary: Implement PaymentResponse.prototype.onuserdetailchange → Implement PaymentResponse.prototype.onpayerdetailchange
(Assignee)

Comment 1

7 months ago
Started work on this.
(Assignee)

Comment 2

7 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

7 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

7 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

7 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

7 months ago
Fat fingered that one by accident. Sending again.
(Assignee)

Comment 11

7 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

7 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

6 months ago
Thanks again for the rebase! Resuming work on this.
(Assignee)

Comment 17

6 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

6 months ago
Blocks: 1318984
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

6 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

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

Comment 24

6 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

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

Comment 26

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