webkit: IntersectionObserver broken or unimplemented

[nephele]

Webkit test/demo: ​https://webkit.org/demos/intersection-observer/simple/simple-observer.html

hrev55618+5
haikuwebkit 1.8.3

[pulkomandy]

It is a runtime enabled feature. See for example setIntersectionObserverEnabled in WebKitLegacy/mac/WebView/WebPreferences.mm.

So it needs to be enabled somehow.

in WTF/Scripts/Preferences/WebPreferencesExperimental.yaml, it is enabled by default for WebKit but not WebKitLegacy.

Do you need it for anything specific?

[nephele]

It's abscence is a possible cause for these two tickets:
https://dev.haiku-os.org/ticket/17325
https://dev.haiku-os.org/ticket/17404

[pulkomandy]

You can see how I changed the setting for image scaling here: ​https://github.com/haiku/haikuwebkit/commit/8198852ff648facbaf5c2dc0ff71a1086ce694bd

IF you want to try enabling this other setting you can do it in the ame way. Let me know if it helps with these bugs. Unfortunately it requires recompiling about 700 files, which takes a while.

[nephele]

Enabeling IntersectionObserver fixes the github branch overview combobox for me, don't know about the merge button. It also fixed that other tickets site.

[nephele]

My git server is beeing particularily picky about not wanting anything pushed today, so here is the patch inline :/

From f62bedc9890475bac30ecd822fd1dd2ff09b6148 Mon Sep 17 00:00:00 2001
From: Pascal Abresch <nep@packageloss.eu>
Date: Sat, 20 Nov 2021 11:36:21 +0100
Subject: [PATCH] Haiku: enable IntersectionObserverEnabled

This fixes rendering of github branch view
---
 .../WTF/Scripts/Preferences/WebPreferencesExperimental.yaml  | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/Source/WTF/Scripts/Preferences/WebPreferencesExperimental.yaml b/Source/WTF/Scripts/Preferences/WebPreferencesExperimental.yaml
index 6567e8d94e..6d950e6899 100644
--- a/Source/WTF/Scripts/Preferences/WebPreferencesExperimental.yaml
+++ b/Source/WTF/Scripts/Preferences/WebPreferencesExperimental.yaml
@@ -499,7 +499,7 @@ FTPEnabled:
       default: false
     WebKitLegacy:
       default: false
-      
+
 # FIXME: This seems to be accidentally enabled for WebKit1 right now due to the old code using the
 # wrong preference key in some places. We should identify whether it really makes sense to keep this
 # enabled.
@@ -670,6 +670,7 @@ IntersectionObserverEnabled:
   humanReadableDescription: "Enable Intersection Observer support"
   defaultValue:
     WebKitLegacy:
+      PLATFORM(HAIKU): true
       default: false
     WebKit:
       default: true
@@ -1203,7 +1204,7 @@ ScreenCaptureEnabled:
       default: false
     WebCore:
       default: false
-      
+
 ScrollToTextFragmentEnabled:
   type: bool
   humanReadableName: "Scroll To Text Fragment"
-- 
2.30.2

[nephele]

enabled in ​https://github.com/haiku/haikuwebkit/commit/fddc2944fa2cda4281ec801f69b9fdd11f0e7d60