Chromium Code Reviews
Help | Chromium Project | Gerrit Changes | Sign in

Issue 40095: Enable interstitial pages for SSL certificate errors on Linux. (Closed)

9 years ago by Markus (顧孟勤)
6 years, 10 months ago
jcampan, Evan Martin


Enable interstitial pages for SSL certificate errors on Linux. There are still problems with this patch, as acknowledging the interstitial page often fails to load a new page. Instead, it just displays a blank gray page. This appears to be somewhat timing related. Nonetheless, this changelist is strictly an improvement over what we have right now. So, I would still like to commit it. Ideally, I'd like to remove the special-case Windows code. But I don't have a Windows machine to check whether the new code is actually a full superset of the old Windows code. Committed:

Patch Set 1 #

Patch Set 2 : '' #

Unified diffs Side-by-side diffs Delta from patch set Stats (+18 lines, -15 lines) Patch
M chrome/browser/ssl/ View 1 3 chunks +0 lines, -5 lines 0 comments Download
M chrome/browser/tab_contents/ View 1 5 chunks +18 lines, -10 lines 0 comments Download


Total messages: 4 (0 generated)
Markus (顧孟勤)
9 years ago (2009-03-06 23:31:43 UTC) #1
Evan Martin
This doesn't look bad to me, but I don't know the code too well. Jay, ...
9 years ago (2009-03-06 23:33:40 UTC) #2
Evan Martin
I will say LGTM and Jay can comment if he wants to later. :)
9 years ago (2009-03-06 23:34:39 UTC) #3
9 years ago (2009-03-06 23:48:59 UTC) #4
This probably should work on Windows.
Once you have checked-in I'll look into it.

Powered by Google App Engine
This is Rietveld 408576698