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

Side by Side Diff: third_party/WebKit/Source/web/ChromeClientImpl.cpp

Issue 2378503002: Observe visibility of password inputs, for HTTP-bad phase 1 (Closed)
Patch Set: fix components_unittests build failure Created 4 years, 2 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 /* 1 /*
2 * Copyright (C) 2009 Google Inc. All rights reserved. 2 * Copyright (C) 2009 Google Inc. All rights reserved.
3 * Copyright (C) 2010 Nokia Corporation and/or its subsidiary(-ies). 3 * Copyright (C) 2010 Nokia Corporation and/or its subsidiary(-ies).
4 * 4 *
5 * Redistribution and use in source and binary forms, with or without 5 * Redistribution and use in source and binary forms, with or without
6 * modification, are permitted provided that the following conditions are 6 * modification, are permitted provided that the following conditions are
7 * met: 7 * met:
8 * 8 *
9 * * Redistributions of source code must retain the above copyright 9 * * Redistributions of source code must retain the above copyright
10 * notice, this list of conditions and the following disclaimer. 10 * notice, this list of conditions and the following disclaimer.
(...skipping 1052 matching lines...) Expand 10 before | Expand all | Expand 10 after
1063 webframe->autofillClient()->dataListOptionsChanged(WebInputElement(&inpu t)); 1063 webframe->autofillClient()->dataListOptionsChanged(WebInputElement(&inpu t));
1064 } 1064 }
1065 1065
1066 void ChromeClientImpl::ajaxSucceeded(LocalFrame* frame) 1066 void ChromeClientImpl::ajaxSucceeded(LocalFrame* frame)
1067 { 1067 {
1068 WebLocalFrameImpl* webframe = WebLocalFrameImpl::fromFrame(frame); 1068 WebLocalFrameImpl* webframe = WebLocalFrameImpl::fromFrame(frame);
1069 if (webframe->autofillClient()) 1069 if (webframe->autofillClient())
1070 webframe->autofillClient()->ajaxSucceeded(); 1070 webframe->autofillClient()->ajaxSucceeded();
1071 } 1071 }
1072 1072
1073 void ChromeClientImpl::passwordFieldBecameVisible(HTMLInputElement& input)
1074 {
1075 WebLocalFrameImpl* webframe = WebLocalFrameImpl::fromFrame(input.document(). frame());
1076 if (webframe->autofillClient())
1077 webframe->autofillClient()->passwordFieldBecameVisible();
esprehn 2016/09/27 23:30:19 Please use mojo directly instead, I don't want to
estark 2016/09/27 23:35:42 Can you point to an example of what you mean by "u
estark 2016/09/27 23:39:42 Oh, actually, I just found and am reading https://
1078 }
1079
1073 void ChromeClientImpl::registerViewportLayers() const 1080 void ChromeClientImpl::registerViewportLayers() const
1074 { 1081 {
1075 if (m_webView->rootGraphicsLayer() && m_webView->layerTreeView()) 1082 if (m_webView->rootGraphicsLayer() && m_webView->layerTreeView())
1076 m_webView->registerViewportLayersWithCompositor(); 1083 m_webView->registerViewportLayersWithCompositor();
1077 } 1084 }
1078 1085
1079 void ChromeClientImpl::didUpdateTopControls() const 1086 void ChromeClientImpl::didUpdateTopControls() const
1080 { 1087 {
1081 m_webView->didUpdateTopControls(); 1088 m_webView->didUpdateTopControls();
1082 } 1089 }
(...skipping 68 matching lines...) Expand 10 before | Expand all | Expand 10 after
1151 ScreenOrientationController::provideTo(frame, client ? client->webScreenOrie ntationClient() : nullptr); 1158 ScreenOrientationController::provideTo(frame, client ? client->webScreenOrie ntationClient() : nullptr);
1152 if (RuntimeEnabledFeatures::presentationEnabled()) 1159 if (RuntimeEnabledFeatures::presentationEnabled())
1153 PresentationController::provideTo(frame, client ? client->presentationCl ient() : nullptr); 1160 PresentationController::provideTo(frame, client ? client->presentationCl ient() : nullptr);
1154 if (RuntimeEnabledFeatures::audioOutputDevicesEnabled()) 1161 if (RuntimeEnabledFeatures::audioOutputDevicesEnabled())
1155 provideAudioOutputDeviceClientTo(frame, AudioOutputDeviceClientImpl::cre ate()); 1162 provideAudioOutputDeviceClientTo(frame, AudioOutputDeviceClientImpl::cre ate());
1156 if (RuntimeEnabledFeatures::installedAppEnabled()) 1163 if (RuntimeEnabledFeatures::installedAppEnabled())
1157 InstalledAppController::provideTo(frame, client ? client->installedAppCl ient() : nullptr); 1164 InstalledAppController::provideTo(frame, client ? client->installedAppCl ient() : nullptr);
1158 } 1165 }
1159 1166
1160 } // namespace blink 1167 } // namespace blink
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698