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

Unified Diff: components/password_manager/content/browser/content_password_manager_driver.h

Issue 2378503002: Observe visibility of password inputs, for HTTP-bad phase 1 (Closed)
Patch Set: fix browser-side binding 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: components/password_manager/content/browser/content_password_manager_driver.h
diff --git a/components/password_manager/content/browser/content_password_manager_driver.h b/components/password_manager/content/browser/content_password_manager_driver.h
index e4ed94cd1210fa1935b1dbcaf839ac4e528e4ea5..20c2b97336d6622c5f609d1c5c1dfad29ed166aa 100644
--- a/components/password_manager/content/browser/content_password_manager_driver.h
+++ b/components/password_manager/content/browser/content_password_manager_driver.h
@@ -19,6 +19,7 @@
#include "components/password_manager/core/browser/password_manager.h"
#include "components/password_manager/core/browser/password_manager_driver.h"
#include "mojo/public/cpp/bindings/binding.h"
+#include "third_party/WebKit/public/platform/modules/sensitive_input_visibility/sensitive_input_visibility_service.mojom.h"
namespace autofill {
class AutofillManager;
@@ -43,7 +44,8 @@ enum class BadMessageReason;
// The lifetime is managed by the ContentPasswordManagerDriverFactory.
class ContentPasswordManagerDriver
: public PasswordManagerDriver,
- public autofill::mojom::PasswordManagerDriver {
+ public autofill::mojom::PasswordManagerDriver,
+ public blink::mojom::SensitiveInputVisibilityService {
public:
ContentPasswordManagerDriver(content::RenderFrameHost* render_frame_host,
PasswordManagerClient* client,
@@ -55,6 +57,8 @@ class ContentPasswordManagerDriver
content::RenderFrameHost* render_frame_host);
void BindRequest(autofill::mojom::PasswordManagerDriverRequest request);
+ void BindSensitiveInputVisibilityRequest(
+ blink::mojom::SensitiveInputVisibilityServiceRequest request);
// PasswordManagerDriver implementation.
void FillPasswordForm(
@@ -111,6 +115,9 @@ class ContentPasswordManagerDriver
const autofill::PasswordForm& password_form,
const base::string16& generation_field) override;
+ // blink::mojom::SensitiveInputVisibility:
+ void PasswordFieldVisible() override;
+
void OnPasswordFormsParsedNoRenderCheck(
const std::vector<autofill::PasswordForm>& forms);
void OnFocusedPasswordFormFound(const autofill::PasswordForm& password_form);
@@ -143,6 +150,9 @@ class ContentPasswordManagerDriver
mojo::Binding<autofill::mojom::PasswordManagerDriver> binding_;
+ mojo::Binding<blink::mojom::SensitiveInputVisibilityService>
+ sensitive_input_visibility_binding_;
+
base::WeakPtrFactory<ContentPasswordManagerDriver> weak_factory_;
DISALLOW_COPY_AND_ASSIGN(ContentPasswordManagerDriver);

Powered by Google App Engine
This is Rietveld 408576698