Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(103)

Issue 2647233002: PaymentApp: Implement simulateInvokingPaymentApp() for LayoutTest.

Created:
3 years, 11 months ago by zino
Modified:
3 years, 11 months ago
Reviewers:
Yeaisme
CC:
chromium-reviews, mlamouri+watch-content_chromium.org, mlamouri+watch-test-runner_chromium.org, viettrungluu+watch_chromium.org, Peter Beverloo, jam, yzshen+watch_chromium.org, abarth-chromium, Aaron Boodman, darin-cc_chromium.org, einbinder+watch-test-runner_chromium.org, darin (slow to review), jochen+watch_chromium.org, qsr+mojo_chromium.org
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

PaymentApp: Implement simulateInvokingPaymentApp() for LayoutTest. BUG=

Patch Set 1 #

Patch Set 2 : PaymentApp: Implement invokePaymentApp() for LayoutTest. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+204 lines, -1 line) Patch
M components/test_runner/test_runner.cc View 1 3 chunks +17 lines, -0 lines 0 comments Download
M components/test_runner/test_runner_for_specific_view.h View 1 1 chunk +5 lines, -0 lines 0 comments Download
M components/test_runner/test_runner_for_specific_view.cc View 1 1 chunk +34 lines, -0 lines 0 comments Download
M components/test_runner/web_test_delegate.h View 1 1 chunk +5 lines, -0 lines 0 comments Download
M content/public/DEPS View 1 chunk +1 line, -0 lines 0 comments Download
M content/public/browser/payment_app_provider.h View 1 chunk +1 line, -0 lines 0 comments Download
M content/shell/BUILD.gn View 1 2 chunks +3 lines, -0 lines 0 comments Download
M content/shell/browser/content_shell_browser_manifest_overlay.json View 1 chunk +2 lines, -1 line 0 comments Download
M content/shell/browser/layout_test/layout_test_content_browser_client.cc View 1 2 chunks +3 lines, -0 lines 0 comments Download
A content/shell/browser/layout_test/layout_test_payment_app_provider_impl.h View 1 1 chunk +32 lines, -0 lines 0 comments Download
A content/shell/browser/layout_test/layout_test_payment_app_provider_impl.cc View 1 1 chunk +64 lines, -0 lines 0 comments Download
A content/shell/common/layout_test/layout_test_payment_app_provider.mojom View 1 1 chunk +10 lines, -0 lines 0 comments Download
M content/shell/renderer/layout_test/blink_test_runner.h View 1 3 chunks +8 lines, -0 lines 0 comments Download
M content/shell/renderer/layout_test/blink_test_runner.cc View 1 3 chunks +19 lines, -0 lines 0 comments Download

Messages

Total messages: 7 (1 generated)
Yeaisme
lgtm
3 years, 11 months ago (2017-01-22 15:33:44 UTC) #2
Yeaisme
lgtm lgtm
3 years, 11 months ago (2017-01-22 15:35:18 UTC) #3
Yeaisme
lgtm lgtm
3 years, 11 months ago (2017-01-22 15:37:18 UTC) #4
Yeaisme
lgtm
3 years, 11 months ago (2017-01-22 15:37:19 UTC) #5
Yeaisme
lgtm lgtm
3 years, 11 months ago (2017-01-22 15:37:19 UTC) #6
zino
3 years, 11 months ago (2017-01-22 16:11:00 UTC) #7
not lgtm myself. I'm still working on this CL.

Powered by Google App Engine
This is Rietveld 408576698