• calamity's avatar
    Revert "NetworkService: Fix WebViewTest.ClearDataCache" · d0a1ab99
    calamity authored
    This reverts commit b94164ad.
    
    Reason for revert: Suspected of causing
    ExtensionWebRequestApiTest.WebRequestApiDoesNotCrashOnErrorAfterProfileDestroyed
    failure in
    https://ci.chromium.org/p/chromium/builders/luci.chromium.ci/Win7%20Tests%20%28dbg%29%281%29/71221
    
    Original change's description:
    > NetworkService: Fix WebViewTest.ClearDataCache
    > 
    > This patch does 3 things to fix the test:
    > 1. Register listeners before loading the actual guest page. See bug for
    >    more background.
    > 2. Add |was_fetched_via_cache| to |ResourceResponseInfo|.
    > 3. Call |NetworkContext::ClearHttpCache()| in
    >    |WebViewGuest::ClearData()| when Network Service was enabled. We
    >    cannot use |BrowsingDataRemover| because it doesn't support
    >    non-default |StoragePartition|.
    > 
    > Bug: 878060
    > Cq-Include-Trybots: luci.chromium.try:linux_mojo
    > Change-Id: I5f0ea8e62ee8f44944428cfb041f9d4e427bd67c
    > Reviewed-on: https://chromium-review.googlesource.com/1195596
    > Reviewed-by: Tom Sepez <tsepez@chromium.org>
    > Reviewed-by: Ken Rockot <rockot@chromium.org>
    > Reviewed-by: Clark DuVall <cduvall@chromium.org>
    > Commit-Queue: Chong Zhang <chongz@chromium.org>
    > Cr-Commit-Position: refs/heads/master@{#587753}
    
    TBR=rockot@chromium.org,tsepez@chromium.org,chongz@chromium.org,cduvall@chromium.org
    
    Change-Id: I0a4f300e25d7fd9160d9a8e9ac02c8c733654818
    No-Presubmit: true
    No-Tree-Checks: true
    No-Try: true
    Bug: 878060
    Cq-Include-Trybots: luci.chromium.try:linux_mojo
    Reviewed-on: https://chromium-review.googlesource.com/1198642Reviewed-by: default avatarcalamity <calamity@chromium.org>
    Commit-Queue: calamity <calamity@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#587892}
    d0a1ab99
web_request_info.cc 14.2 KB