Skip to content

GitLab

T
tangled
Commit f2c5241f authored by Anupam Snigdha's avatar Anupam Snigdha Committed by Commit Bot
Browse files
Options

[VirtualKeyboard] Remove bounding rect from geometrychange event. 

Removed the keyboard bounding rectangle from geometrychange event.
The keyboard rectangle is now a property of virtualKeyboard object.
It gets updated when the geometrychange event is fired. The keyboard
rectangle can also be captured from CSS env variables:
https://github.com/MicrosoftEdge/MSEdgeExplainers/blob/main/VirtualKeyboardAPI/explainer.md#virtual-keyboard-visibility-change-css-environment-variables
This was also a feedback from TAG: https://github.com/w3ctag/design-reviews/issues/507#issuecomment-665473068

Bug: 1127749

Change-Id: Id7f19b0e091b820fa45ba4181d482559846d40ba
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2466402
Commit-Queue: Anupam Snigdha <snianu@microsoft.com>
Reviewed-by: default avatarKent Tamura <tkent@chromium.org>
Reviewed-by: default avatarAvi Drissman <avi@chromium.org>
Cr-Commit-Position: refs/heads/master@{#816395}
parent f6c02a26
Hide whitespace changes
Inline Side-by-side
Showing with 9 additions and 63 deletions
content/browser/renderer_host/render_widget_host_view_aura_vk_browsertest.cc
View file @ f2c5241f
......@@ -277,13 +277,11 @@ IN_PROC_BROWSER_TEST_F(RenderWidgetHostViewAuraBrowserMockIMETest,
const char kVirtualKeyboardDataURL[] =
"data:text/html,<!DOCTYPE html>"
"<script>"
" let VKRect, x, y, width, height, numEvents = 0;"
" let VKRect, numEvents = 0;"
" navigator.virtualKeyboard.overlaysContent = true;"
" navigator.virtualKeyboard.addEventListener('geometrychange',"
" evt => {"
" numEvents++;"
" let r = evt.boundingRect;"
" x = r.x; y = r.y; width = r.width; height = r.height;"
" VKRect = navigator.virtualKeyboard.boundingRect"
" }, false);"
"</script>";
......@@ -332,10 +330,6 @@ IN_PROC_BROWSER_TEST_F(RenderWidgetHostViewAuraBrowserMockIMETest,
const int expected_y = kKeyboardY - root_widget_origin.y();
EXPECT_EQ(1, EvalJs(shell(), "numEvents"));
EXPECT_EQ(0, EvalJs(shell(), "x"));
EXPECT_EQ(expected_y, EvalJs(shell(), "y"));
EXPECT_EQ(expected_width, EvalJs(shell(), "width"));
EXPECT_EQ(kKeyboardHeight, EvalJs(shell(), "height"));
EXPECT_EQ(0, EvalJs(shell(), "VKRect.x"));
EXPECT_EQ(expected_y, EvalJs(shell(), "VKRect.y"));
EXPECT_EQ(expected_width, EvalJs(shell(), "VKRect.width"));
......@@ -343,10 +337,6 @@ IN_PROC_BROWSER_TEST_F(RenderWidgetHostViewAuraBrowserMockIMETest,
input_method_->GetMockKeyboardController()->NotifyObserversOnKeyboardHidden();
EXPECT_EQ(2, EvalJs(shell(), "numEvents"));
EXPECT_EQ(0, EvalJs(shell(), "width"));
EXPECT_EQ(0, EvalJs(shell(), "height"));
EXPECT_EQ(0, EvalJs(shell(), "x"));
EXPECT_EQ(0, EvalJs(shell(), "y"));
EXPECT_EQ(0, EvalJs(shell(), "VKRect.x"));
EXPECT_EQ(0, EvalJs(shell(), "VKRect.y"));
EXPECT_EQ(0, EvalJs(shell(), "VKRect.width"));
......
third_party/blink/renderer/bindings/idl_in_modules.gni
View file @ f2c5241f
......@@ -661,7 +661,6 @@ static_idl_files_in_modules = get_path_info(
"//third_party/blink/renderer/modules/virtualkeyboard/navigator_virtual_keyboard.idl",
"//third_party/blink/renderer/modules/virtualkeyboard/virtual_keyboard.idl",
"//third_party/blink/renderer/modules/virtualkeyboard/virtual_keyboard_geometry_change_event.idl",
"//third_party/blink/renderer/modules/virtualkeyboard/virtual_keyboard_geometry_change_event_init.idl",
"//third_party/blink/renderer/modules/wake_lock/navigator_wake_lock.idl",
"//third_party/blink/renderer/modules/wake_lock/wake_lock.idl",
"//third_party/blink/renderer/modules/wake_lock/wake_lock_sentinel.idl",
......
third_party/blink/renderer/modules/virtualkeyboard/idls.gni
View file @ f2c5241f
......@@ -7,7 +7,4 @@ modules_idl_files = [
"virtual_keyboard_geometry_change_event.idl",
]
modules_dictionary_idl_files =
[ "virtual_keyboard_geometry_change_event_init.idl" ]
modules_dependency_idl_files = [ "navigator_virtual_keyboard.idl" ]
third_party/blink/renderer/modules/virtualkeyboard/virtual_keyboard.cc
View file @ f2c5241f
......@@ -85,7 +85,7 @@ void VirtualKeyboard::VirtualKeyboardOverlayChanged(
StyleEnvironmentVariables::FormatPx(keyboard_rect.height()));
}
DispatchEvent(*(MakeGarbageCollected<VirtualKeyboardGeometryChangeEvent>(
event_type_names::kGeometrychange, bounding_rect_)));
event_type_names::kGeometrychange)));
}
void VirtualKeyboard::show() {
......
third_party/blink/renderer/modules/virtualkeyboard/virtual_keyboard_geometry_change_event.cc
View file @ f2c5241f
......@@ -4,31 +4,15 @@
#include "third_party/blink/renderer/modules/virtualkeyboard/virtual_keyboard_geometry_change_event.h"
#include "third_party/blink/renderer/bindings/modules/v8/v8_virtual_keyboard_geometry_change_event_init.h"
#include "third_party/blink/renderer/core/geometry/dom_rect.h"
namespace blink {
VirtualKeyboardGeometryChangeEvent* VirtualKeyboardGeometryChangeEvent::Create(
const AtomicString& type,
const VirtualKeyboardGeometryChangeEventInit* initializer) {
return MakeGarbageCollected<VirtualKeyboardGeometryChangeEvent>(type,
initializer);
const AtomicString& type) {
return MakeGarbageCollected<VirtualKeyboardGeometryChangeEvent>(type);
}
VirtualKeyboardGeometryChangeEvent::VirtualKeyboardGeometryChangeEvent(
const AtomicString& type,
const VirtualKeyboardGeometryChangeEventInit* initializer)
: Event(type, initializer) {}
VirtualKeyboardGeometryChangeEvent::VirtualKeyboardGeometryChangeEvent(
const AtomicString& type,
DOMRect* rect)
: Event(type, Bubbles::kNo, Cancelable::kNo), bounding_rect_(rect) {}
void VirtualKeyboardGeometryChangeEvent::Trace(Visitor* visitor) const {
visitor->Trace(bounding_rect_);
Event::Trace(visitor);
}
const AtomicString& type)
: Event(type, Bubbles::kNo, Cancelable::kNo) {}
} // namespace blink
third_party/blink/renderer/modules/virtualkeyboard/virtual_keyboard_geometry_change_event.h
View file @ f2c5241f
......@@ -10,28 +10,13 @@
namespace blink {
class DOMRect;
class VirtualKeyboardGeometryChangeEventInit;
class VirtualKeyboardGeometryChangeEvent final : public Event {
DEFINE_WRAPPERTYPEINFO();
public:
static VirtualKeyboardGeometryChangeEvent* Create(
const AtomicString& type,
const VirtualKeyboardGeometryChangeEventInit*);
VirtualKeyboardGeometryChangeEvent(
const AtomicString& type,
const VirtualKeyboardGeometryChangeEventInit*);
VirtualKeyboardGeometryChangeEvent(const AtomicString& type, DOMRect*);
DOMRect* boundingRect() const { return bounding_rect_; }
void Trace(Visitor*) const override;
static VirtualKeyboardGeometryChangeEvent* Create(const AtomicString& type);
private:
Member<DOMRect> bounding_rect_;
explicit VirtualKeyboardGeometryChangeEvent(const AtomicString& type);
};
} // namespace blink
......
third_party/blink/renderer/modules/virtualkeyboard/virtual_keyboard_geometry_change_event.idl
View file @ f2c5241f
......@@ -10,7 +10,6 @@
Exposed=Window,
RuntimeEnabled=VirtualKeyboard
] interface VirtualKeyboardGeometryChangeEvent : Event {
constructor(DOMString type, VirtualKeyboardGeometryChangeEventInit eventInitDict);
[SameObject] readonly attribute DOMRect boundingRect;
constructor(DOMString type);
};
third_party/blink/renderer/modules/virtualkeyboard/virtual_keyboard_geometry_change_event_init.idl deleted 100644 → 0
View file @ f6c02a26
// Copyright 2020 The Chromium Authors. All rights reserved.
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
dictionary VirtualKeyboardGeometryChangeEventInit : EventInit {
required DOMRect boundingRect;
};
third_party/blink/web_tests/webexposed/global-interface-listing-expected.txt
View file @ f2c5241f
......@@ -8816,7 +8816,6 @@ interface VirtualKeyboard : EventTarget
setter overlaysContent
interface VirtualKeyboardGeometryChangeEvent : Event
attribute @@toStringTag
getter boundingRect
method constructor
interface VisibilityStateEntry : PerformanceEntry
attribute @@toStringTag
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment