[Payment Request] window.PaymentRequest must be undefined if PR is disabled

[Payment Request] window.PaymentRequest must be undefined if PR is disabled

BUG=734586, 602666
Review-Url: https://codereview.chromium.org/2949583002
Cr-Commit-Position: refs/heads/master@{#480578}
Committed: https://chromium.googlesource.com/chromium/src/+/3c7d91dbad5abcb55c58b37637522aa4e2a56bc1

[Moe, 2017-06-19 14:22:14]

The CQ bit was checked by mahmadi@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-06-19 14:22:21]

Dry run: CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Moe, 2017-06-19 14:23:02]

mahmadi@chromium.org changed reviewers:
+ mathp@chromium.org

[Moe, 2017-06-19 14:23:02]

Hi Mathieu, Please take a look.

[commit-bot: I haz the power, 2017-06-19 14:32:44]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-06-19 14:32:45]

Dry run: Try jobs failed on following builders:
  ios-simulator-xcode-clang on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator-xco...)

[Mathieu, 2017-06-19 14:38:55]

Thanks, this lgtm as a short term fix to be merged back.

[Moe, 2017-06-19 14:47:54]

The CQ bit was checked by mahmadi@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-06-19 14:48:09]

Dry run: CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Moe, 2017-06-19 14:50:30]

mahmadi@chromium.org changed reviewers:
+ eugenebut@chromium.org
- mathp@chromium.org

[Moe, 2017-06-19 14:50:30]

Hi Eugene, Please take a look. This needs to be merged back into M60.

[Eugene But (OOO till 7-30), 2017-06-19 14:53:59]

Thanks for the test! lgtm

https://codereview.chromium.org/2949583002/diff/20001/ios/chrome/browser/web/...
File ios/chrome/browser/web/chrome_web_client.mm (right):

https://codereview.chromium.org/2949583002/diff/20001/ios/chrome/browser/web/...
ios/chrome/browser/web/chrome_web_client.mm:166: NSString* chromePageScript =
GetPageScript(@"chrome_bundle");
s/chromePageScript/chrome_page_script

[Moe, 2017-06-19 15:11:09]

mahmadi@chromium.org changed reviewers:
+ mathp@chromium.org

[Moe, 2017-06-19 15:11:09]

The CQ bit was checked by mahmadi@chromium.org

[Moe, 2017-06-19 15:11:10]

Thank you.

https://codereview.chromium.org/2949583002/diff/20001/ios/chrome/browser/web/...
File ios/chrome/browser/web/chrome_web_client.mm (right):

https://codereview.chromium.org/2949583002/diff/20001/ios/chrome/browser/web/...
ios/chrome/browser/web/chrome_web_client.mm:166: NSString* chromePageScript =
GetPageScript(@"chrome_bundle");
On 2017/06/19 at 14:53:58, Eugene But wrote:
> s/chromePageScript/chrome_page_script

done.

[Moe, 2017-06-19 15:11:11]

The patchset sent to the CQ was uploaded after l-g-t-m from mathp@chromium.org,
eugenebut@chromium.org
Link to the patchset: https://codereview.chromium.org/2949583002/#ps40001
(title: "Addressed comment")

[commit-bot: I haz the power, 2017-06-19 15:11:27]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Moe, 2017-06-19 15:17:21]

Description was changed from

==========
[Payment Request] window.PaymentRequest must be undefined if PR is disabled

BUG=602666
==========

to

==========
[Payment Request] window.PaymentRequest must be undefined if PR is disabled

BUG=734586,602666
==========

[commit-bot: I haz the power, 2017-06-19 16:23:31]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-06-19 16:23:32]

Try jobs failed on following builders:
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[Moe, 2017-06-19 21:01:00]

The CQ bit was checked by mahmadi@chromium.org

[commit-bot: I haz the power, 2017-06-19 21:01:28]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-06-19 21:07:33]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1497906060123550,
"parent_rev": "e3dfa3cb5052c95d6d6825a305ce8f40c686960f", "commit_rev":
"3c7d91dbad5abcb55c58b37637522aa4e2a56bc1"}

[commit-bot: I haz the power, 2017-06-19 21:07:45]

Description was changed from

==========
[Payment Request] window.PaymentRequest must be undefined if PR is disabled

BUG=734586,602666
==========

to

==========
[Payment Request] window.PaymentRequest must be undefined if PR is disabled

BUG=734586,602666

Review-Url: https://codereview.chromium.org/2949583002
Cr-Commit-Position: refs/heads/master@{#480578}
Committed:
https://chromium.googlesource.com/chromium/src/+/3c7d91dbad5abcb55c58b3763752...
==========

[commit-bot: I haz the power, 2017-06-19 21:07:46]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/chromium/src/+/3c7d91dbad5abcb55c58b3763752...