Revert "Video Tutorials: Added video player UI"
This reverts commit 278cb45f. Reason for revert: Findit (https://goo.gl/kROfz5) identified CL at revision 811094 as the culprit for failures in the build cycles as shown on: https://analysis.chromium.org/waterfall/culprit?key=ag9zfmZpbmRpdC1mb3ItbWVyRAsSDVdmU3VzcGVjdGVkQ0wiMWNocm9taXVtLzI3OGNiNDVmZDcxNzBjMGYzYWZhMGQ4ZGExYzRjYzRmOWU0YzkzZTgM Sample Failed Build: https://ci.chromium.org/b/8867949354369276768 Sample Failed Step: compile Original change's description: > Video Tutorials: Added video player UI > > This CL adds the video player UI. > 1 - Added VideoPlayerActivity at chrome layer, which creates > VideoPlayerCoordinator and passes a WebContent supplier in order to > create ThinWebView. > 2 - Added video player MVC, the mediator handles display logic between > loading screen, controls, thinwebview, and language picker. > It also handles callbacks for control buttons, and observes media > session through a PlaybackStateObserver. > > TODO : > Add screenshot. Add UI polish. Define share/watchnext behavior. > Polish media session logic. > > Bug: 1117172 > Binary-Size: Unavoidable method count increase (includes methods from previous CL). > Change-Id: Ide7c0d7bf4fa20a9bfbe0c10717e6983d0da88dc > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2415672 > Commit-Queue: Shakti Sahu <shaktisahu@chromium.org> > Reviewed-by: David Trainor <dtrainor@chromium.org> > Cr-Commit-Position: refs/heads/master@{#811094} Change-Id: I8de44b83196ee7ac90090f2c6d765a9aa3429430 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 1117172 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2434557 Cr-Commit-Position: refs/heads/master@{#811099}
Showing
Please register or sign in to comment