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

Side by Side Diff: chrome/browser/ui/website_settings/website_settings.cc

Issue 1964383002: Fullscreen / mouselock: Always auto-accept (even if flag disabled). (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Also remove setting from OIB. Created 4 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/ui/website_settings/website_settings.h" 5 #include "chrome/browser/ui/website_settings/website_settings.h"
6 6
7 #include <stddef.h> 7 #include <stddef.h>
8 #include <stdint.h> 8 #include <stdint.h>
9 9
10 #include <string> 10 #include <string>
(...skipping 101 matching lines...) Expand 10 before | Expand all | Expand 10 after
112 #if defined(OS_ANDROID) 112 #if defined(OS_ANDROID)
113 CONTENT_SETTINGS_TYPE_PUSH_MESSAGING, 113 CONTENT_SETTINGS_TYPE_PUSH_MESSAGING,
114 #endif 114 #endif
115 CONTENT_SETTINGS_TYPE_KEYGEN, 115 CONTENT_SETTINGS_TYPE_KEYGEN,
116 CONTENT_SETTINGS_TYPE_BACKGROUND_SYNC, 116 CONTENT_SETTINGS_TYPE_BACKGROUND_SYNC,
117 }; 117 };
118 118
119 // Determines whether to show permission |type| in the Website Settings UI. Only 119 // Determines whether to show permission |type| in the Website Settings UI. Only
120 // applies to permissions listed in |kPermissionType|. 120 // applies to permissions listed in |kPermissionType|.
121 bool ShouldShowPermission(ContentSettingsType type) { 121 bool ShouldShowPermission(ContentSettingsType type) {
122 // TODO(mgiuca): When simplified-fullscreen-ui is enabled on all platforms, 122 // TODO(mgiuca): When simplified-fullscreen-ui is enabled permanently on
123 // remove these from kPermissionType, rather than having this check 123 // Android, remove these from kPermissionType, rather than having this check
124 // (http://crbug.com/577396). 124 // (http://crbug.com/577396).
125 #if !defined(OS_ANDROID) 125 #if !defined(OS_ANDROID)
126 // Fullscreen and mouselock settings are not shown in simplified fullscreen 126 // Fullscreen and mouselock settings are no longer shown (always allow).
127 // mode (always allow).
128 if (type == CONTENT_SETTINGS_TYPE_FULLSCREEN || 127 if (type == CONTENT_SETTINGS_TYPE_FULLSCREEN ||
129 type == CONTENT_SETTINGS_TYPE_MOUSELOCK) { 128 type == CONTENT_SETTINGS_TYPE_MOUSELOCK) {
130 return !ExclusiveAccessManager::IsSimplifiedFullscreenUIEnabled(); 129 return false;
131 } 130 }
132 #endif 131 #endif
133 132
134 return true; 133 return true;
135 } 134 }
136 135
137 // Returns true if any of the given statuses match |status|. 136 // Returns true if any of the given statuses match |status|.
138 bool CertificateTransparencyStatusMatchAny( 137 bool CertificateTransparencyStatusMatchAny(
139 const std::vector<net::ct::SCTVerifyStatus>& sct_verify_statuses, 138 const std::vector<net::ct::SCTVerifyStatus>& sct_verify_statuses,
140 net::ct::SCTVerifyStatus status) { 139 net::ct::SCTVerifyStatus status) {
(...skipping 620 matching lines...) Expand 10 before | Expand all | Expand 10 after
761 info.connection_status = site_connection_status_; 760 info.connection_status = site_connection_status_;
762 info.connection_status_description = 761 info.connection_status_description =
763 UTF16ToUTF8(site_connection_details_); 762 UTF16ToUTF8(site_connection_details_);
764 info.identity_status = site_identity_status_; 763 info.identity_status = site_identity_status_;
765 info.identity_status_description = 764 info.identity_status_description =
766 UTF16ToUTF8(site_identity_details_); 765 UTF16ToUTF8(site_identity_details_);
767 info.cert_id = cert_id_; 766 info.cert_id = cert_id_;
768 info.show_ssl_decision_revoke_button = show_ssl_decision_revoke_button_; 767 info.show_ssl_decision_revoke_button = show_ssl_decision_revoke_button_;
769 ui_->SetIdentityInfo(info); 768 ui_->SetIdentityInfo(info);
770 } 769 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698