Issue ID: 645123003 Title: Do not AppCache responses with SSL cert errors Status: Closed Created: 6 years, 2 months ago Modified: 6 years, 2 months ago Reviewers: palmer, Ilya Sherman, Ryan Sleevi CC: chromium-reviews, darin-cc_chromium.org, jam Base URL: https://chromium.googlesource.com/chromium/src.git@master Project: chromium Visibility: Public Description Summary: Do not AppCache responses with SSL cert errors. BUG: 414026 Committed URL: https://crrev.com/3ec982d993b769efc893d4f3a0fa28eea95b69d0 Commit Position: refs/heads/master@{#299999} Patch Sets Set 1: Created 6 years, 2 months ago - Files Modified: - content/browser/appcache/appcache_update_job.h - content/browser/appcache/appcache_update_job.cc - tools/metrics/histograms/histograms.xml - Stats: +37 lines, -33 lines Messages (Key Points) Palmer: Thanked for the solution, noted typo in commit message. Ryan Sleevi: Approved the changes. Michaeln: Asked for review and followed up on failed layout tests. Ilya Sherman: Approved histograms part. Michaeln: Discussed broken layout tests and certificate errors.