Diff
Modified: trunk/Source/WebKit/ChangeLog (242395 => 242396)
--- trunk/Source/WebKit/ChangeLog 2019-03-04 22:57:12 UTC (rev 242395)
+++ trunk/Source/WebKit/ChangeLog 2019-03-04 23:11:58 UTC (rev 242396)
@@ -1,3 +1,16 @@
+2019-03-04 Truitt Savell <tsav...@apple.com>
+
+ Unreviewed, rolling out r242222.
+
+ Broke internal debug testing
+
+ Reverted changeset:
+
+ "[iOS] Move calls to [UIKeyboard isInHardwareKeyboardMode] to
+ the UI process."
+ https://bugs.webkit.org/show_bug.cgi?id=193683
+ https://trac.webkit.org/changeset/242222
+
2019-03-04 Brent Fulgham <bfulg...@apple.com>
Unreviewed build fix after r242378
Modified: trunk/Source/WebKit/UIProcess/API/Cocoa/WKWebView.mm (242395 => 242396)
--- trunk/Source/WebKit/UIProcess/API/Cocoa/WKWebView.mm 2019-03-04 22:57:12 UTC (rev 242395)
+++ trunk/Source/WebKit/UIProcess/API/Cocoa/WKWebView.mm 2019-03-04 23:11:58 UTC (rev 242396)
@@ -3303,9 +3303,7 @@
{
ASSERT(observer);
WKWebView *webView = (__bridge WKWebView *)observer;
- auto keyboardIsAttached = GSEventIsHardwareKeyboardAttached();
- webView._page->process().setKeyboardIsAttached(keyboardIsAttached);
- webView._page->hardwareKeyboardAvailabilityChanged(keyboardIsAttached);
+ webView._page->hardwareKeyboardAvailabilityChanged();
}
- (void)_windowDidRotate:(NSNotification *)notification
Modified: trunk/Source/WebKit/UIProcess/WebPageProxy.h (242395 => 242396)
--- trunk/Source/WebKit/UIProcess/WebPageProxy.h 2019-03-04 22:57:12 UTC (rev 242395)
+++ trunk/Source/WebKit/UIProcess/WebPageProxy.h 2019-03-04 23:11:58 UTC (rev 242396)
@@ -688,7 +688,7 @@
void storeSelectionForAccessibility(bool);
void startAutoscrollAtPosition(const WebCore::FloatPoint& positionInWindow);
void cancelAutoscroll();
- void hardwareKeyboardAvailabilityChanged(bool keyboardIsAttached);
+ void hardwareKeyboardAvailabilityChanged();
bool isScrollingOrZooming() const { return m_isScrollingOrZooming; }
void requestEvasionRectsAboveSelection(CompletionHandler<void(const Vector<WebCore::FloatRect>&)>&&);
#if ENABLE(DATA_INTERACTION)
Modified: trunk/Source/WebKit/UIProcess/WebProcessProxy.h (242395 => 242396)
--- trunk/Source/WebKit/UIProcess/WebProcessProxy.h 2019-03-04 22:57:12 UTC (rev 242395)
+++ trunk/Source/WebKit/UIProcess/WebProcessProxy.h 2019-03-04 23:11:58 UTC (rev 242396)
@@ -269,11 +269,6 @@
void sendProcessDidResume() override;
void didSetAssertionState(AssertionState) override;
-#if PLATFORM(IOS_FAMILY)
- void setKeyboardIsAttached(bool keyboardIsAttached) { m_keyboardIsAttached = keyboardIsAttached; }
- bool keyboardIsAttached() const { return m_keyboardIsAttached; }
-#endif
-
#if PLATFORM(COCOA)
enum SandboxExtensionType : uint32_t {
None = 0,
@@ -463,10 +458,6 @@
ProcessThrottler::BackgroundActivityToken m_backgroundActivityTokenForFullscreenFormControls;
#endif
-#if PLATFORM(IOS_FAMILY)
- bool m_keyboardIsAttached { false };
-#endif
-
#if PLATFORM(COCOA)
MediaCaptureSandboxExtensions m_mediaCaptureSandboxExtensions { SandboxExtensionType::None };
#endif
Modified: trunk/Source/WebKit/UIProcess/ios/WKContentViewInteraction.mm (242395 => 242396)
--- trunk/Source/WebKit/UIProcess/ios/WKContentViewInteraction.mm 2019-03-04 22:57:12 UTC (rev 242395)
+++ trunk/Source/WebKit/UIProcess/ios/WKContentViewInteraction.mm 2019-03-04 23:11:58 UTC (rev 242396)
@@ -4850,7 +4850,7 @@
if (_isChangingFocus)
return YES;
- if (_page->process().keyboardIsAttached())
+ if (UIKeyboard.isInHardwareKeyboardMode)
return YES;
#endif
}
Modified: trunk/Source/WebKit/UIProcess/ios/WebPageProxyIOS.mm (242395 => 242396)
--- trunk/Source/WebKit/UIProcess/ios/WebPageProxyIOS.mm 2019-03-04 22:57:12 UTC (rev 242395)
+++ trunk/Source/WebKit/UIProcess/ios/WebPageProxyIOS.mm 2019-03-04 23:11:58 UTC (rev 242396)
@@ -671,8 +671,6 @@
{
bool isSuspendedUnderLock = [UIApp isSuspendedUnderLock];
m_process->send(Messages::WebPage::ApplicationWillEnterForeground(isSuspendedUnderLock), m_pageID);
- m_process->setKeyboardIsAttached([UIKeyboard isInHardwareKeyboardMode]);
- m_process->send(Messages::WebPage::HardwareKeyboardAvailabilityChanged(m_process->keyboardIsAttached()), m_pageID);
}
void WebPageProxy::applicationWillResignActive()
@@ -1110,10 +1108,10 @@
m_validationBubble->show();
}
-void WebPageProxy::hardwareKeyboardAvailabilityChanged(bool keyboardIsAttached)
+void WebPageProxy::hardwareKeyboardAvailabilityChanged()
{
updateCurrentModifierState();
- m_process->send(Messages::WebPage::HardwareKeyboardAvailabilityChanged(keyboardIsAttached), m_pageID);
+ m_process->send(Messages::WebPage::HardwareKeyboardAvailabilityChanged(), m_pageID);
}
void WebPageProxy::requestEvasionRectsAboveSelection(CompletionHandler<void(const Vector<WebCore::FloatRect>&)>&& callback)
Modified: trunk/Source/WebKit/WebProcess/WebPage/WebPage.h (242395 => 242396)
--- trunk/Source/WebKit/WebProcess/WebPage/WebPage.h 2019-03-04 22:57:12 UTC (rev 242395)
+++ trunk/Source/WebKit/WebProcess/WebPage/WebPage.h 2019-03-04 23:11:58 UTC (rev 242396)
@@ -937,7 +937,7 @@
bool platformPrefersTextLegibilityBasedZoomScaling() const;
const WebCore::ViewportConfiguration& viewportConfiguration() const { return m_viewportConfiguration; }
- void hardwareKeyboardAvailabilityChanged(bool keyboardIsAttached);
+ void hardwareKeyboardAvailabilityChanged();
void updateStringForFind(const String&);
#endif
@@ -1843,9 +1843,6 @@
OptionSet<LayerTreeFreezeReason> m_LayerTreeFreezeReasons;
bool m_isSuspended { false };
bool m_needsFontAttributes { false };
-#if PLATFORM(IOS_FAMILY)
- bool m_keyboardIsAttached { false };
-#endif
};
} // namespace WebKit
Modified: trunk/Source/WebKit/WebProcess/WebPage/WebPage.messages.in (242395 => 242396)
--- trunk/Source/WebKit/WebProcess/WebPage/WebPage.messages.in 2019-03-04 22:57:12 UTC (rev 242395)
+++ trunk/Source/WebKit/WebProcess/WebPage/WebPage.messages.in 2019-03-04 23:11:58 UTC (rev 242396)
@@ -108,7 +108,7 @@
StartAutoscrollAtPosition(WebCore::FloatPoint positionInWindow)
CancelAutoscroll()
RequestFocusedElementInformation(WebKit::CallbackID callbackID)
- HardwareKeyboardAvailabilityChanged(bool keyboardIsAttached)
+ HardwareKeyboardAvailabilityChanged()
#endif
SetControlledByAutomation(bool controlled)
Modified: trunk/Source/WebKit/WebProcess/WebPage/ios/WebPageIOS.mm (242395 => 242396)
--- trunk/Source/WebKit/WebProcess/WebPage/ios/WebPageIOS.mm 2019-03-04 22:57:12 UTC (rev 242395)
+++ trunk/Source/WebKit/WebProcess/WebPage/ios/WebPageIOS.mm 2019-03-04 23:11:58 UTC (rev 242396)
@@ -203,7 +203,7 @@
bool needsLayout = !frame.view() || frame.view()->needsLayout();
bool requiresPostLayoutData = frame.editor().hasComposition();
#if !PLATFORM(IOSMAC)
- requiresPostLayoutData |= m_keyboardIsAttached;
+ requiresPostLayoutData |= [UIKeyboard isInHardwareKeyboardMode];
#endif
if (shouldIncludePostLayoutData == IncludePostLayoutDataHint::No && needsLayout && !requiresPostLayoutData) {
result.isMissingPostLayoutData = true;
@@ -3233,10 +3233,8 @@
return String();
}
-void WebPage::hardwareKeyboardAvailabilityChanged(bool keyboardIsAttached)
+void WebPage::hardwareKeyboardAvailabilityChanged()
{
- m_keyboardIsAttached = keyboardIsAttached;
-
if (auto* focusedFrame = m_page->focusController().focusedFrame())
focusedFrame->eventHandler().capsLockStateMayHaveChanged();
}