• Haiyang Pan's avatar
    Revert "Make ContentCapture ready for experiment" · c8026dea
    Haiyang Pan authored
    This reverts commit 564da6d9.
    
    Reason for revert: Very likely the cause of crbug.com/1122472
    See the analysis in crbug.com/1123677#c4 for detail
    
    Original change's description:
    > Make ContentCapture ready for experiment
    > 
    > Currently the triggering of ContentCapture is unpredictable, it is
    > hard for us to get the unbiased result for the UserActivatedDelay
    > experiment.
    > 
    > This patch adds an ExperimentContentCaptureConsumer which triggers
    > the ContentCapture independently and is enabled by default.
    > 
    > The allow list check is moved to consumer if the experiment
    > is enabled.
    > 
    > As ContentCapture is potentially used for all Android versions,
    > we plan to run the experiment for all of them.
    > 
    > This mechanism will also be used for future experiments.
    > 
    > This patch hasn't supported multiple consumers yet.
    > 
    > Test: Added new tests and pass the existing tests
    > 
    > Bug: 1114819, 1119663
    > Change-Id: I72e0b991767329caec37080caae2d5c2a9068eaf
    > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2368104
    > Commit-Queue: Tao Bai <michaelbai@chromium.org>
    > Reviewed-by: David Trainor <dtrainor@chromium.org>
    > Reviewed-by: Changwan Ryu <changwan@chromium.org>
    > Reviewed-by: Xianzhu Wang <wangxianzhu@chromium.org>
    > Cr-Commit-Position: refs/heads/master@{#801672}
    
    TBR=michaelbai@chromium.org,wangxianzhu@chromium.org,dtrainor@chromium.org,changwan@chromium.org
    
    # Not skipping CQ checks because original CL landed > 1 day ago.
    
    Bug: 1114819
    Bug: 1119663
    Bug: 1122472
    Change-Id: Ife98d73f4e77062606f18adb0ba12e9baaf4f6c3
    Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2386578Reviewed-by: default avatarHaiyang Pan <hypan@google.com>
    Commit-Queue: Haiyang Pan <hypan@google.com>
    Cr-Commit-Position: refs/heads/master@{#803349}
    c8026dea
content_capture_features_android.cc 459 Bytes