fix(render_handler): sync with current CEF API

CEF removed GetScreenRect (replaced by CefScreenInfo.rect from
GetScreenInfo) and OnScrollOffsetChanged now passes (browser, x, y).
The pyx/cpp glue had drifted: GetScreenRect was a dead Cython entry
with no C++ caller (but still in the callback allowlist + docs), and
OnScrollOffsetChanged dropped x/y on the floor.

Author: linesight

api/API-index.md

@@ -347,7 +347,6 @@
 * [RenderHandler (interface)](RenderHandler.md#renderhandler-interface)
   * [GetRootScreenRect](RenderHandler.md#getrootscreenrect)
   * [GetViewRect](RenderHandler.md#getviewrect)
-  * [GetScreenRect](RenderHandler.md#getscreenrect)
   * [GetScreenPoint](RenderHandler.md#getscreenpoint)
   * [GetScreenInfo](RenderHandler.md#getscreeninfo)
   * [OnPopupShow](RenderHandler.md#onpopupshow)

api/RenderHandler.md

@@ -15,7 +15,6 @@ Table of contents:
 * [Callbacks](#callbacks)
   * [GetRootScreenRect](#getrootscreenrect)
   * [GetViewRect](#getviewrect)
-  * [GetScreenRect](#getscreenrect)
   * [GetScreenPoint](#getscreenpoint)
   * [GetScreenInfo](#getscreeninfo)
   * [OnPopupShow](#onpopupshow)
@@ -74,18 +73,6 @@ Called to retrieve the view rectangle which is relative to screen
 coordinates. Return true if the rectangle was provided.
 
 
-### GetScreenRect
-
-| Parameter | Type |
-| --- | --- |
-| browser | [Browser](Browser.md) |
-| rect_out | list[x,y,width,height] |
-| __Return__ | bool |
-
-Called to retrieve the simulated screen rectangle. Return true
-if the rectangle was provided.
-
-
 ### GetScreenPoint
 
 | Parameter | Type |
@@ -213,6 +200,8 @@ Called when the browser's cursor has changed. If |type| is CT_CUSTOM then
 | Parameter | Type |
 | --- | --- |
 | browser | [Browser](Browser.md) |
+| x | float |
+| y | float |
 | __Return__ | void |
 
 Called when the scroll offset has changed.

src/browser.pyx

@@ -262,7 +262,6 @@ cdef class PyBrowser:
             # RenderHandler
             self.allowedClientCallbacks += ["GetRootScreenRect",
                     "GetViewRect", "GetScreenPoint", "GetScreenInfo",
-                    "GetScreenRect",
                     "OnPopupShow", "OnPopupSize", "OnPaint", "OnCursorChange",
                     "OnScrollOffsetChanged",
                     "StartDragging", "UpdateDragCursor",

src/client_handler/render_handler.cpp

@@ -74,7 +74,7 @@ void RenderHandler::OnScrollOffsetChanged(CefRefPtr<CefBrowser> browser,
                                           double y)
 {
     REQUIRE_UI_THREAD();
-    RenderHandler_OnScrollOffsetChanged(browser);
+    RenderHandler_OnScrollOffsetChanged(browser, x, y);
 }
 
 

src/handlers/render_handler.pyx

@@ -85,34 +85,6 @@ cdef public cpp_bool RenderHandler_GetViewRect(
         (exc_type, exc_value, exc_trace) = sys.exc_info()
         sys.excepthook(exc_type, exc_value, exc_trace)
 
-cdef public cpp_bool RenderHandler_GetScreenRect(
-        CefRefPtr[CefBrowser] cefBrowser,
-        CefRect& cefRect
-        ) except * with gil:
-    cdef PyBrowser pyBrowser
-    cdef list pyRect = []
-    cdef py_bool ret
-    try:
-        pyBrowser = GetPyBrowser(cefBrowser, "GetScreenRect")
-        callback = pyBrowser.GetClientCallback("GetScreenRect")
-        if callback:
-            ret = callback(browser=pyBrowser, rect_out=pyRect)
-            if ret:
-                assert (pyRect and len(pyRect) == 4), (
-                        "rectangle not provided or invalid")
-                cefRect.x = pyRect[0]
-                cefRect.y = pyRect[1]
-                cefRect.width = pyRect[2]
-                cefRect.height = pyRect[3]
-                return True
-            else:
-                return False
-        else:
-            return False
-    except:
-        (exc_type, exc_value, exc_trace) = sys.exc_info()
-        sys.excepthook(exc_type, exc_value, exc_trace)
-
 cdef public cpp_bool RenderHandler_GetScreenPoint(
         CefRefPtr[CefBrowser] cefBrowser,
         int viewX, int viewY,
@@ -285,14 +257,16 @@ cdef public void RenderHandler_OnPaint(
         sys.excepthook(exc_type, exc_value, exc_trace)
 
 cdef public void RenderHandler_OnScrollOffsetChanged(
-        CefRefPtr[CefBrowser] cefBrowser
+        CefRefPtr[CefBrowser] cefBrowser,
+        double x,
+        double y
         ) noexcept with gil:
     cdef PyBrowser pyBrowser
     try:
         pyBrowser = GetPyBrowser(cefBrowser, "OnScrollOffsetChanged")
         callback = pyBrowser.GetClientCallback("OnScrollOffsetChanged")
         if callback:
-            callback(browser=pyBrowser)
+            callback(browser=pyBrowser, x=x, y=y)
     except:
         (exc_type, exc_value, exc_trace) = sys.exc_info()
         sys.excepthook(exc_type, exc_value, exc_trace)