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

Side by Side Diff: components/autofill/core/browser/autofill_manager.cc

Issue 2672623005: Record Autofill form events specially for nonsecure pages (Closed)
Patch Set: rebase Created 3 years, 10 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 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 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 "components/autofill/core/browser/autofill_manager.h" 5 #include "components/autofill/core/browser/autofill_manager.h"
6 6
7 #include <stddef.h> 7 #include <stddef.h>
8 #include <stdint.h> 8 #include <stdint.h>
9 9
10 #include <algorithm> 10 #include <algorithm>
(...skipping 1464 matching lines...) Expand 10 before | Expand all | Expand 10 after
1475 is_local_data_available = true; 1475 is_local_data_available = true;
1476 else if (profile->record_type() == AutofillProfile::SERVER_PROFILE) 1476 else if (profile->record_type() == AutofillProfile::SERVER_PROFILE)
1477 is_server_data_available = true; 1477 is_server_data_available = true;
1478 } 1478 }
1479 address_form_event_logger_->set_is_server_data_available( 1479 address_form_event_logger_->set_is_server_data_available(
1480 is_server_data_available); 1480 is_server_data_available);
1481 address_form_event_logger_->set_is_local_data_available( 1481 address_form_event_logger_->set_is_local_data_available(
1482 is_local_data_available); 1482 is_local_data_available);
1483 } 1483 }
1484 1484
1485 credit_card_form_event_logger_->set_is_main_url_secure(
sebsg 2017/02/03 19:28:34 Can you move this with the other credit_card_form_
estark 2017/02/04 02:16:13 Done.
1486 client_->IsMainUrlSecure());
1487
1485 if (profiles.empty() && credit_cards.empty()) 1488 if (profiles.empty() && credit_cards.empty())
1486 return false; 1489 return false;
1487 1490
1488 return true; 1491 return true;
1489 } 1492 }
1490 1493
1491 bool AutofillManager::GetProfile(int unique_id, 1494 bool AutofillManager::GetProfile(int unique_id,
1492 const AutofillProfile** profile) { 1495 const AutofillProfile** profile) {
1493 // Unpack the |unique_id| into component parts. 1496 // Unpack the |unique_id| into component parts.
1494 std::string credit_card_id; 1497 std::string credit_card_id;
(...skipping 675 matching lines...) Expand 10 before | Expand all | Expand 10 after
2170 if (i > 0) 2173 if (i > 0)
2171 fputs("Next oldest form:\n", file); 2174 fputs("Next oldest form:\n", file);
2172 } 2175 }
2173 fputs("\n", file); 2176 fputs("\n", file);
2174 2177
2175 fclose(file); 2178 fclose(file);
2176 } 2179 }
2177 #endif // ENABLE_FORM_DEBUG_DUMP 2180 #endif // ENABLE_FORM_DEBUG_DUMP
2178 2181
2179 } // namespace autofill 2182 } // namespace autofill
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698