 Chromium Code Reviews
 Chromium Code Reviews Issue 
            2470743002:
    Revert of Add SafeBrowsingNavigationObserver to listen to navigation events  (Closed)
    
  
    Issue 
            2470743002:
    Revert of Add SafeBrowsingNavigationObserver to listen to navigation events  (Closed) 
  | DescriptionRevert of Add SafeBrowsingNavigationObserver to listen to navigation events (patchset #24 id:680001 of https://codereview.chromium.org/2302913003/ )
Reason for revert:
This is causing flakiness failures in the newly added SBNavigationObserverBrowserTest.SubFrameNewTabDownload test on Linux ChromiumOS Tests (1): see http://test-results.appspot.com/dashboards/flakiness_dashboard.html#testType=browser_tests&tests=SBNavigationObserverBrowserTest.SubFrameNewTabDownload
It starts with the next build after the one where the CL was submitted: https://build.chromium.org/p/chromium.chromiumos/builders/Linux%20ChromiumOS%20Tests%20(1)/builds/28699
Original issue's description:
> Add SafeBrowsingNavigationObserver to listen to navigation events
> happen on all frames.
>
> These navigation events can help safe browsing service identify
> suspicious downloads that intentionally hide their referrers
> and/or landing pages, and report these pieces of info to safe
> browsing backend.
>
> This CL only covers the observing part. More management code
> (e.g. periodically cleanup, actual reporting, etc) will come up
> shortly.
>
> BUG=639467
>
> Committed: https://crrev.com/602009dfabda593406bfd7f5171e8c4183335bf5
> Cr-Commit-Position: refs/heads/master@{#428865}
TBR=creis@chromium.org,nparker@chromium.org,jialiul@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=639467
Committed: https://crrev.com/e95c4a5176841304dabea6d132249c21acbcd05b
Cr-Commit-Position: refs/heads/master@{#428967}
   Patch Set 1 #Messages
    Total messages: 7 (3 generated)
     |