commit 074e1ef34f66bb24f6821f3b7e28e30433ede67c
author Hiroki Nakagawa <nhiroki@chromium.org> Fri Dec 06 09:38:46 2024
committer Chromium LUCI CQ <chromium-scoped@luci-project-accounts.iam.gserviceaccount.com> Fri Dec 06 09:38:46 2024
tree dffb17370b554f2877a3ba5e03c1308a934213c0
parent d547e68c2be460a5f165e8c6b64ba02fad9fbac5

Reland "WebView: Implement request monitor in EmbeddedTestServerAndroid"

This reverts commit 369c5093d8d05a674aa4d68c73dd15df5354fb56.

Reason for revert (reland):
The original CL was reverted due to compile failures on the Cronet bot.
This is because the code generated by AIDL used a language feature that
requires SDK version 24 for Map<String, String>, but the Cronet requires
SDK version 23 as the minimum version (https://crrev.com/c/5913439). To
avoid the restriction, this CL encodes the headers into String[] that
should be available even in SDK version 23 instead of Map.

Original change's description:
> Revert "WebView: Implement request monitor in EmbeddedTestServerAndroid"
>
> This reverts commit 3d9eb93fb149da6bb363b877326fe4e115729737.
>
> Reason for revert:
> This CL breaks the android-cronet-arm-dbg build:
> https://ci.chromium.org/b/8729401193435602897
>
> Original change's description:
> > WebView: Implement request monitor in EmbeddedTestServerAndroid
> >
> > This CL implements the request monitor to capture request headers in
> > EmbeddedTestServerAndroid. The headers are exposed to Java, so that
> > WebView tests can check if requests have appropriate headers.
> >
> > As an example usage, this CL uses the mechanism in a prerendering test.
> > The test checks if prerendering navigation has the Sec-Purpose header.
> >
> > Change-Id: Iec9843917fc4ac3cafa15b71304ba92352502635
> > Bug: 41490450
> > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/6072982
> > Commit-Queue: Hiroki Nakagawa <nhiroki@chromium.org>
> > Reviewed-by: Takashi Toyoshima <toyoshim@chromium.org>
> > Reviewed-by: Kenichi Ishibashi <bashi@chromium.org>
> > Cr-Commit-Position: refs/heads/main@{#1392138}
>
> Bug: 41490450
> Change-Id: Id50f9bc047209bd593a015373b93e7bbceaf3b0c
> No-Presubmit: true
> No-Tree-Checks: true
> No-Try: true
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/6072588
> Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com>
> Auto-Submit: Hiroki Nakagawa <nhiroki@chromium.org>
> Owners-Override: Yifan Luo <lyf@chromium.org>
> Commit-Queue: Yifan Luo <lyf@chromium.org>
> Reviewed-by: Kenichi Ishibashi <bashi@chromium.org>
> Cr-Commit-Position: refs/heads/main@{#1392197}

Bug: 41490450
Change-Id: Ib189894a9df2e5923564fe81fc95129db0ce2b94
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/6072608
Reviewed-by: Takashi Toyoshima <toyoshim@chromium.org>
Commit-Queue: Hiroki Nakagawa <nhiroki@chromium.org>
Reviewed-by: Kenichi Ishibashi <bashi@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1392770}