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

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

Issue 2766053002: [refactor] Fix autofill features for payments when the form is inside an OOPIF (Closed)
Patch Set: Addressing vabr@'s comments Created 3 years, 8 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 1078 matching lines...) Expand 10 before | Expand all | Expand 10 after
1089 void ChromeClientImpl::ShowUnhandledTapUIIfNeeded( 1089 void ChromeClientImpl::ShowUnhandledTapUIIfNeeded(
1090 IntPoint tapped_position_in_viewport, 1090 IntPoint tapped_position_in_viewport,
1091 Node* tapped_node, 1091 Node* tapped_node,
1092 bool page_changed) { 1092 bool page_changed) {
1093 if (web_view_->Client()) 1093 if (web_view_->Client())
1094 web_view_->Client()->ShowUnhandledTapUIIfNeeded( 1094 web_view_->Client()->ShowUnhandledTapUIIfNeeded(
1095 WebPoint(tapped_position_in_viewport), WebNode(tapped_node), 1095 WebPoint(tapped_position_in_viewport), WebNode(tapped_node),
1096 page_changed); 1096 page_changed);
1097 } 1097 }
1098 1098
1099 void ChromeClientImpl::OnMouseDown(Node* mouse_down_node) { 1099 void ChromeClientImpl::OnMouseDown(Node& mouse_down_node) {
1100 if (web_view_->Client()) 1100 if (auto* fill_client =
1101 web_view_->Client()->OnMouseDown(WebNode(mouse_down_node)); 1101 WebLocalFrameImpl::FromFrame(mouse_down_node.GetDocument().GetFrame())
1102 ->AutofillClient()) {
1103 fill_client->DidReceiveLeftMouseDownOrGestureTapInNode(
1104 WebNode(&mouse_down_node));
1105 }
1102 } 1106 }
1103 1107
1104 void ChromeClientImpl::HandleKeyboardEventOnTextField( 1108 void ChromeClientImpl::HandleKeyboardEventOnTextField(
1105 HTMLInputElement& input_element, 1109 HTMLInputElement& input_element,
1106 KeyboardEvent& event) { 1110 KeyboardEvent& event) {
1107 WebLocalFrameImpl* webframe = 1111 WebLocalFrameImpl* webframe =
1108 WebLocalFrameImpl::FromFrame(input_element.GetDocument().GetFrame()); 1112 WebLocalFrameImpl::FromFrame(input_element.GetDocument().GetFrame());
1109 if (webframe->AutofillClient()) 1113 if (webframe->AutofillClient())
1110 webframe->AutofillClient()->TextFieldDidReceiveKeyDown( 1114 webframe->AutofillClient()->TextFieldDidReceiveKeyDown(
1111 WebInputElement(&input_element), WebKeyboardEventBuilder(event)); 1115 WebInputElement(&input_element), WebKeyboardEventBuilder(event));
(...skipping 121 matching lines...) Expand 10 before | Expand all | Expand 10 after
1233 if (RuntimeEnabledFeatures::presentationEnabled()) 1237 if (RuntimeEnabledFeatures::presentationEnabled())
1234 PresentationController::ProvideTo(frame, client->PresentationClient()); 1238 PresentationController::ProvideTo(frame, client->PresentationClient());
1235 if (RuntimeEnabledFeatures::audioOutputDevicesEnabled()) { 1239 if (RuntimeEnabledFeatures::audioOutputDevicesEnabled()) {
1236 ProvideAudioOutputDeviceClientTo(frame, 1240 ProvideAudioOutputDeviceClientTo(frame,
1237 new AudioOutputDeviceClientImpl(frame)); 1241 new AudioOutputDeviceClientImpl(frame));
1238 } 1242 }
1239 InstalledAppController::ProvideTo(frame, client->GetRelatedAppsFetcher()); 1243 InstalledAppController::ProvideTo(frame, client->GetRelatedAppsFetcher());
1240 } 1244 }
1241 1245
1242 } // namespace blink 1246 } // namespace blink
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698