While we are enabling ad filtering for abusive sites, we also need to update the UI UX Overview: https://docs.google.com/document/d/1qtXgjQq6dis_dgOX_-aTMVciTriLChvFJxU_Q9zpzVs/edit#
Review link: https://chromium-review.googlesource.com/c/chromium/src/+/1232575 Manual test result: https://docs.google.com/document/d/1G3hFfuV9Kt9C_fGhyBn_ueYZ5eG-VuZUCLveKbiozxA/edit (To be updated since the UX doc has recently changed)
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/fa7268bb8be974e51391283a41a12b760196c407 commit fa7268bb8be974e51391283a41a12b760196c407 Author: Yao Xiao <yaoxia@chromium.org> Date: Mon Oct 01 22:03:35 2018 Update UI to prepare for A/B testing & enabling of ad filtering for abusive sites. Bug: 888556 Change-Id: Ia330d9eade0513738ae7b9f5f81574a57489b14f Reviewed-on: https://chromium-review.googlesource.com/1232575 Commit-Queue: Yao Xiao <yaoxia@chromium.org> Reviewed-by: Jochen Eisinger <jochen@chromium.org> Reviewed-by: Charlie Harrison <csharrison@chromium.org> Cr-Commit-Position: refs/heads/master@{#595555} [modify] https://crrev.com/fa7268bb8be974e51391283a41a12b760196c407/chrome/android/java/strings/android_chrome_strings.grd [modify] https://crrev.com/fa7268bb8be974e51391283a41a12b760196c407/chrome/app/generated_resources.grd [modify] https://crrev.com/fa7268bb8be974e51391283a41a12b760196c407/chrome/app/settings_strings.grdp [modify] https://crrev.com/fa7268bb8be974e51391283a41a12b760196c407/chrome/browser/resources/settings/site_settings/site_details_permission.html [modify] https://crrev.com/fa7268bb8be974e51391283a41a12b760196c407/chrome/browser/subresource_filter/subresource_filter_browsertest.cc [modify] https://crrev.com/fa7268bb8be974e51391283a41a12b760196c407/chrome/browser/ui/webui/settings/md_settings_localized_strings_provider.cc [modify] https://crrev.com/fa7268bb8be974e51391283a41a12b760196c407/chrome/test/data/webui/settings/site_details_permission_tests.js [modify] https://crrev.com/fa7268bb8be974e51391283a41a12b760196c407/components/page_info_strings.grdp [modify] https://crrev.com/fa7268bb8be974e51391283a41a12b760196c407/components/subresource_filter/core/browser/subresource_filter_constants.h [modify] https://crrev.com/fa7268bb8be974e51391283a41a12b760196c407/third_party/WebKit/LayoutTests/http/tests/subresource_filter/image-allow-disallow-transitions-expected.txt [modify] https://crrev.com/fa7268bb8be974e51391283a41a12b760196c407/third_party/WebKit/LayoutTests/http/tests/subresource_filter/resource-disallowed-expected.txt [modify] https://crrev.com/fa7268bb8be974e51391283a41a12b760196c407/third_party/blink/renderer/core/loader/subresource_filter.cc
Comment 1 by yaoxia@chromium.org
, Sep 24