DescriptionRevert of Fix flaky NPE in oobe.js. (patchset #1 id:1 of https://codereview.chromium.org/2803043002/ )
Reason for revert:
Breaking existing tests. (crbug.com/710975)
Original issue's description:
> Fix flaky NPE in oobe.js.
>
> Under some racy condition, telemetry accesses the DOM object before ready.
> This CL observes a new variable which is set to true when DOM is ready.
>
> BUG=chromium:709041
> TEST=Ran try. Ran on ChromeOS device with corresponding CL.
>
> Review-Url: https://codereview.chromium.org/2803043002
> Committed: https://chromium.googlesource.com/external/github.com/catapult-project/catapult/+/9a55abab029cb9ae94f5160ded11b09a4638a955
TBR=achuith@chromium.org,hidehiko@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=chromium:709041
Review-Url: https://codereview.chromium.org/2808763010
Committed: https://chromium.googlesource.com/external/github.com/catapult-project/catapult/+/f49ec2dcbd4701deac2247e4aa59e9cbf281b4dc
Patch Set 1 #
Messages
Total messages: 17 (6 generated)
|