DescriptionRevert of "[telemetry] Wire TsProxy through telemetry's network stack." (patchset #1 id:1 of https://codereview.chromium.org/2335403003/ )
Reason for revert:
Did not actually break WPR. Fine to reland and enable TsProxy
Original issue's description:
> Reland of "[telemetry] Wire TsProxy through telemetry's network stack." (patchset #1 id:1 of https://codereview.chromium.org/2342503002/ )
>
> Reason for revert:
> Broke WPR replay on WebView bots
>
> Original issue's description:
> > Revert of Revert "[telemetry] Wire TsProxy through telemetry's network stack." (patchset #1 id:1 of https://codereview.chromium.org/2267613002/ )
> >
> > Reason for revert:
> > Webview now support --proxy-server flag (see attached bug)
> >
> > Original issue's description:
> > > Revert "[telemetry] Wire TsProxy through telemetry's network stack."
> > >
> > > This reverts commit 9fcccf0b65703994c9541273f00a993806a95356.
> > >
> > > Revert "[Telemetry] Explictly initialize & clean up network_controller"
> > >
> > > This reverts commit 30ce231266c6fb6147bb6df7c56a7eb75c23fccf.
> > >
> > > Seems like webview currently does not support proxy server?
> > > (See attached chromium bug)
> > >
> > > BUG=chromium:639632
> > > BUG=catapult:#2584
> > >
> > > TBR=pmeenan@chromium.org
> > >
> > > Committed: https://chromium.googlesource.com/external/github.com/catapult-project/catapult/+/947506bacfb5047857a52033256635faebf38af2
> >
> > TBR=nednguyen@chromium.org
> > # Not skipping CQ checks because original CL landed more than 1 days ago.
> > BUG=chromium:639632
> >
> > Committed: https://chromium.googlesource.com/external/github.com/catapult-project/catapult/+/b86cfef4b9da4749ff51e0fc4432869f8209dc42
>
> TBR=nednguyen@chromium.org,nednguyen@google.com
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=chromium:639632
>
> Committed: https://chromium.googlesource.com/external/github.com/catapult-project/catapult/+/32dd31095913725705e9cf5f5bc1671f32821a35
TBR=nednguyen@chromium.org,nednguyen@google.com
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=chromium:639632
Committed: https://chromium.googlesource.com/external/github.com/catapult-project/catapult/+/3cc40b37372927435ed69f91aec2117488b0825c
Patch Set 1 #Messages
Total messages: 5 (2 generated)
|