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

Unified Diff: chrome/browser/ui/android/usb_chooser_dialog_android.cc

Issue 2363623002: Remove SecurityStateModel memoization (Closed)
Patch Set: Created 4 years, 3 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/android/usb_chooser_dialog_android.cc
diff --git a/chrome/browser/ui/android/usb_chooser_dialog_android.cc b/chrome/browser/ui/android/usb_chooser_dialog_android.cc
index c779222b2aea9c572a17207fea046f854cd3d8be..82d5336fe644c0bd9ac854803fe64ce03102c7e8 100644
--- a/chrome/browser/ui/android/usb_chooser_dialog_android.cc
+++ b/chrome/browser/ui/android/usb_chooser_dialog_android.cc
@@ -80,9 +80,10 @@ UsbChooserDialogAndroid::UsbChooserDialogAndroid(
ChromeSecurityStateModelClient* security_model_client =
ChromeSecurityStateModelClient::FromWebContents(web_contents);
DCHECK(security_model_client);
+ security_state::SecurityStateModel::SecurityInfo security_info;
+ security_model_client->GetSecurityInfo(&security_info);
java_dialog_.Reset(Java_UsbChooserDialog_create(
- env, window_android, origin_string,
- security_model_client->GetSecurityInfo().security_level,
+ env, window_android, origin_string, security_info.security_level,
reinterpret_cast<intptr_t>(this)));
if (!java_dialog_.is_null()) {

Powered by Google App Engine
This is Rietveld 408576698