Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(395)

Issue 2063008: (Original patch reviewed at http://codereview.chromium.org/2067003 )... (Closed)

Created:
10 years, 7 months ago by Peter Kasting
Modified:
9 years, 7 months ago
Reviewers:
CC:
chromium-reviews, brettw-cc_chromium.org, amit, ben+cc_chromium.org, Paweł Hajdan Jr.
Visibility:
Public.

Description

(Original patch reviewed at http://codereview.chromium.org/2067003 ) Track "display" and "run" separately for mixed content, and make the latter downgrade the SSL state to "authentication broken". Make the "display" state only affect the current tab (not the entire host). Fix an SSL browser test by supplying the appropriate SiteInstance*. Move a test from "disabled" to "flaky" since it at least passes for me. Make the SSLManager header and .cc files put functions in the same order, and make that order somewhat saner. BUG=15072, 18626, 40932, 42758 TEST=Covered by browser tests Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=47347

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+452 lines, -391 lines) Patch
M chrome/app/generated_resources.grd View 1 chunk +5 lines, -2 lines 0 comments Download
M chrome/browser/external_tab_container.cc View 2 chunks +3 lines, -2 lines 0 comments Download
M chrome/browser/page_info_model.cc View 1 chunk +3 lines, -2 lines 0 comments Download
M chrome/browser/ssl/ssl_browser_tests.cc View 23 chunks +113 lines, -54 lines 0 comments Download
M chrome/browser/ssl/ssl_host_state.h View 3 chunks +6 lines, -9 lines 0 comments Download
M chrome/browser/ssl/ssl_host_state.cc View 2 chunks +8 lines, -9 lines 0 comments Download
M chrome/browser/ssl/ssl_host_state_unittest.cc View 2 chunks +13 lines, -13 lines 0 comments Download
M chrome/browser/ssl/ssl_manager.h View 4 chunks +34 lines, -34 lines 0 comments Download
M chrome/browser/ssl/ssl_manager.cc View 5 chunks +125 lines, -124 lines 0 comments Download
M chrome/browser/ssl/ssl_policy.h View 4 chunks +5 lines, -13 lines 0 comments Download
M chrome/browser/ssl/ssl_policy.cc View 4 chunks +53 lines, -45 lines 0 comments Download
M chrome/browser/ssl/ssl_policy_backend.h View 3 chunks +5 lines, -10 lines 0 comments Download
M chrome/browser/ssl/ssl_policy_backend.cc View 4 chunks +8 lines, -15 lines 0 comments Download
M chrome/browser/tab_contents/navigation_entry.h View 2 chunks +22 lines, -8 lines 0 comments Download
M chrome/browser/tab_contents/navigation_entry_unittest.cc View 1 chunk +9 lines, -5 lines 0 comments Download
M chrome/browser/tab_contents/tab_contents.h View 2 chunks +7 lines, -0 lines 0 comments Download
M chrome/browser/tab_contents/tab_contents.cc View 3 chunks +7 lines, -1 line 0 comments Download
M chrome/browser/toolbar_model.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/common/notification_type.h View 2 chunks +7 lines, -10 lines 0 comments Download
M chrome/test/automation/automation_messages.h View 5 chunks +10 lines, -5 lines 0 comments Download
A + chrome/test/data/ssl/page_displays_mixed_content.html View 1 chunk +1 line, -1 line 0 comments Download
A + chrome/test/data/ssl/page_runs_mixed_content.html View 1 chunk +1 line, -1 line 0 comments Download
D chrome/test/data/ssl/page_with_http_script.html View 1 chunk +0 lines, -8 lines 0 comments Download
D chrome/test/data/ssl/page_with_mixed_contents.html View 1 chunk +0 lines, -15 lines 0 comments Download
M chrome_frame/chrome_active_document.cc View 3 chunks +6 lines, -4 lines 0 comments Download

Powered by Google App Engine
This is Rietveld 408576698