|
(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
|
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
|
|