• Lan Wei's avatar
    Reland "Fix orientation change test full-screen-orientation-change.html" · ed8c12d4
    Lan Wei authored
    This is a reland of a4cdec69
    I checked it does not change the FullscreenControllerTests tests.
    
    Original change's description:
    > Fix orientation change test full-screen-orientation-change.html
    >
    > There are two ways to trigger fullscreen, either by user activation or by
    > a user generated orientation change. When we enter the fullscreen mode,
    > we should not consume the user activation in the browser side when the
    > fullscreen is triggered by a user generated orientation change, so we
    > need to set the orientation change flag in the browser side when we are
    > calling the mock function setMockScreenOrientation through a mojo
    > message.
    >
    > The CL that consumes the user activation on both the browser and
    > renderer sides when entering fullscreen:
    > https://chromium-review.googlesource.com/c/chromium/src/+/1903675
    >
    >
    > Bug: 1046933
    > Change-Id: I1eb86205bc82ac237bc12d63bbdb0c8985c06090
    > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2026170
    > Reviewed-by: Ken Buchanan <kenrb@chromium.org>
    > Reviewed-by: Alex Moshchuk <alexmos@chromium.org>
    > Commit-Queue: Lan Wei <lanwei@chromium.org>
    > Cr-Commit-Position: refs/heads/master@{#740812}
    
    TBR=kenrb@chromium.org,alexmos@chromium.org,lanwei@chromium.org
    
    Bug: 1046933
    Change-Id: Ieaa57d457edbfe8b432b02df65eed409f40e63dd
    Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2054844
    Commit-Queue: Lan Wei <lanwei@chromium.org>
    Reviewed-by: default avatarLan Wei <lanwei@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#741126}
    ed8c12d4
test_runner.cc 88 KB