• Carlos IL's avatar
    Reland "Enable passive mixed content warning by default" · a4abe593
    Carlos IL authored
    This reverts commit 562af1be.
    
    Reason for revert: Failing test was disabled since it should run with autoupgrades disabled, will investigate and fix why InitAndDisableFeatures is not doing that separately
    
    Original change's description:
    > Revert "Enable passive mixed content warning by default"
    >
    > This reverts commit 0a37869a.
    >
    > Reason for revert: Causing failure of DidChangeVisibleSecurityStateTestWithAutoupgradesDisabled.DidChangeVisibleSecurityStateObserver in browser_tests on linux-chromeos-chrome
    > Example failure: https://ci.chromium.org/p/chrome/builders/ci/linux-chromeos-chrome/3194
    >
    > Original change's description:
    > > Enable passive mixed content warning by default
    > >
    > > Sets the feature to enabled by default and removes the config from
    > > fieldtrial_testing_config.json in preparation for launch.
    > >
    > > Bug: 1025276
    > > Change-Id: Iffcdd2153491de25b7038a609776255bbdacf2ac
    > > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2053024
    > > Commit-Queue: Carlos IL <carlosil@chromium.org>
    > > Reviewed-by: Robert Kaplow <rkaplow@chromium.org>
    > > Reviewed-by: Emily Stark <estark@chromium.org>
    > > Auto-Submit: Carlos IL <carlosil@chromium.org>
    > > Cr-Commit-Position: refs/heads/master@{#741506}
    >
    > TBR=rkaplow@chromium.org,estark@chromium.org,carlosil@chromium.org
    >
    > # Not skipping CQ checks because original CL landed > 1 day ago.
    >
    > Bug: 1025276
    > Change-Id: I20e9dc38d83036c7b17d91de24c23ebae3049de9
    > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2058970
    > Reviewed-by: Bret Sepulveda <bsep@chromium.org>
    > Commit-Queue: Bret Sepulveda <bsep@chromium.org>
    > Cr-Commit-Position: refs/heads/master@{#741860}
    
    TBR=rkaplow@chromium.org,estark@chromium.org,bsep@chromium.org,carlosil@chromium.org
    
    # Not skipping CQ checks because original CL landed > 1 day ago.
    
    Bug: 1025276
    Change-Id: I9e2043fc150b9225bc32537b0cee669f1836802a
    Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2062671
    Commit-Queue: Carlos IL <carlosil@chromium.org>
    Reviewed-by: default avatarEmily Stark <estark@chromium.org>
    Reviewed-by: default avatarRobert Kaplow <rkaplow@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#742366}
    a4abe593
fieldtrial_testing_config.json 215 KB