-
Joe DeBlasio authored
This CL changes the mixed content download blocking code to return an enum indicating what type of blocking should be used (silent, or two types of blocking UI). Previously, this function returned a boolean that indicated silent blocking. The CL then propagates that result through the DownloadTargetDeterminer into the DownloadItem. That state is needed later to trigger the existing download shelf warning UI. This CL also renames callbacks in/around the DownloadTargetDeterminer from "ShouldBlock"-type names to "GetMixedContentStatus"-type names. This CL causes no functional change to Chrome. Follow-up CLs will add support for, then trigger, user-visible warnings. This is part of go/downloads-as-mixed-content . Bug: 960819 Change-Id: Ic0c3f41caf7e623050d4b75be364f410a46e296d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1982680Reviewed-by:
Avi Drissman <avi@chromium.org> Reviewed-by:
Evan Stade <estade@chromium.org> Reviewed-by:
Sean Topping <seantopping@chromium.org> Reviewed-by:
Min Qin <qinmin@chromium.org> Commit-Queue: Joe DeBlasio <jdeblasio@chromium.org> Cr-Commit-Position: refs/heads/master@{#732954}
c427ca03