<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugs.webkit.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.4.1"
          urlbase="https://bugs.webkit.org/"
          
          maintainer="admin@webkit.org"
>

    <bug>
          <bug_id>275805</bug_id>
          
          <creation_ts>2024-06-24 02:39:00 -0700</creation_ts>
          <short_desc>[WPE] WPE Platform: add WPEToplevel</short_desc>
          <delta_ts>2024-09-26 22:15:11 -0700</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>WPE WebKit</component>
          <version>WebKit Nightly Build</version>
          <rep_platform>Unspecified</rep_platform>
          <op_sys>Unspecified</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          <blocked>265636</blocked>
          <everconfirmed>1</everconfirmed>
          <reporter name="Carlos Garcia Campos">cgarcia</reporter>
          <assigned_to name="Carlos Garcia Campos">cgarcia</assigned_to>
          <cc>bugs-noreply</cc>
    
    <cc>Qing-wu.Li</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>2042733</commentid>
    <comment_count>0</comment_count>
    <who name="Carlos Garcia Campos">cgarcia</who>
    <bug_when>2024-06-24 02:39:00 -0700</bug_when>
    <thetext>Add a new class to handle platform toplevels. This will allow us to support multiple views in the same toplevel, but for now only one is supported.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2042735</commentid>
    <comment_count>1</comment_count>
    <who name="Carlos Garcia Campos">cgarcia</who>
    <bug_when>2024-06-24 02:45:51 -0700</bug_when>
    <thetext>Pull request: https://github.com/WebKit/WebKit/pull/30105</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2043379</commentid>
    <comment_count>2</comment_count>
    <who name="EWS">ews-feeder</who>
    <bug_when>2024-06-27 03:24:04 -0700</bug_when>
    <thetext>Committed 280412@main (af3cd7645c2f): &lt;https://commits.webkit.org/280412@main&gt;

Reviewed commits have been landed. Closing PR #30105 and removing active labels.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2063631</commentid>
    <comment_count>3</comment_count>
    <who name="Qing-wu.Li@leica-geosystems.com.cn">Qing-wu.Li</who>
    <bug_when>2024-09-26 22:07:13 -0700</bug_when>
    <thetext>Can I comment here to get help?

I got 2 errors for Qt6 build:

## &apos;wpe_view_resize&apos; was not declared

```
Source/WebKit/UIProcess/API/wpe/qt6/WPEQtView.cpp:87:5: error: &apos;wpe_view_resize&apos; was not declared in this scope; did you mean &apos;wpe_view_resized&apos;?
|    87 |     wpe_view_resize(wpeView, d-&gt;m_size.width(), d-&gt;m_size.height());
|       |     ^~~~~~~~~~~~~~~
|       |     wpe_view_resized
```

## &apos;wpe_toplevel_qtquick&apos; has not been declared

```
| Source/WebKit/UIProcess/API/wpe/qt6/WPEToplevelQtQuick.cpp:35:46: error: &apos;wpe_toplevel_qtquick&apos; has not been declared
|    35 | WEBKIT_DEFINE_FINAL_TYPE(WPEToplevelQtQuick, wpe_toplevel_qtquick, WPE_TYPE_TOPLEVEL, WPEToplevel)
|       |                                              ^~~~~~~~~~~~~~~~~~~~
| In file included from Source/WebKit/WPEPlatform/wpe/WPEView.h:36,
|                  from Source/WebKit/WPEPlatform/wpe/WPEEvent.h:35,
|                  from Source/WebKit/WPEPlatform/wpe/wpe-platform.h:31,
|                  from Source/WebKit/UIProcess/API/wpe/qt6/WPEDisplayQtQuick.h:30,
|                  from Source/WebKit/UIProcess/API/wpe/qt6/WPEToplevelQtQuick.h:29,
|                  from Source/WebKit/UIProcess/API/wpe/qt6/WPEToplevelQtQuick.cpp:27:
| Source/WebKit/WPEPlatform/wpe/WPEToplevel.h:38:27: error: expected identifier before &apos;(&apos; token
|    38 | #define WPE_TYPE_TOPLEVEL (wpe_toplevel_get_type())
|       |                           ^
| Source/WebKit/UIProcess/API/wpe/qt6/WPEToplevelQtQuick.cpp:35:68: note: in expansion of macro &apos;WPE_TYPE_TOPLEVEL&apos;
|    35 | WEBKIT_DEFINE_FINAL_TYPE(WPEToplevelQtQuick, wpe_toplevel_qtquick, WPE_TYPE_TOPLEVEL, WPEToplevel)
|       |                                                                    ^~~~~~~~~~~~~~~~~
| Source/WebKit/UIProcess/API/wpe/qt6/WPEToplevelQtQuick.cpp:37:1: error: expected constructor, destructor, or type conversion before &apos;static&apos;
|    37 | static gboolean wpeToplevelQtQuickResize(WPEToplevel* toplevel, int width, int height)
|       | ^~~~~~
| Source/WebKit/UIProcess/API/wpe/qt6/WPEToplevelQtQuick.cpp: In function &apos;void wpe_toplevel_qtquick_class_init(WPEToplevelQtQuickClass*)&apos;:
| Source/WebKit/UIProcess/API/wpe/qt6/WPEToplevelQtQuick.cpp:52:29: error: &apos;wpeToplevelQtQuickResize&apos; was not declared in this scope; did you mean &apos;WPEToplevelQtQuickClass&apos;?
|    52 |     toplevelClass-&gt;resize = wpeToplevelQtQuickResize;
|       |                             ^~~~~~~~~~~~~~~~~~~~~~~~
|       |                             WPEToplevelQtQuickClass
| Source/WebKit/UIProcess/API/wpe/qt6/WPEToplevelQtQuick.cpp: At global scope:
| Source/WebKit/UIProcess/API/wpe/qt6/WPEToplevelQtQuick.cpp:49:13: warning: &apos;void wpe_toplevel_qtquick_class_init(WPEToplevelQtQuickClass*)&apos; defined but not used [-Wunused-function]
|    49 | static void wpe_toplevel_qtquick_class_init(WPEToplevelQtQuickClass* toplevelQtQuickClass)
|       |             ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2063633</commentid>
    <comment_count>4</comment_count>
    <who name="Qing-wu.Li@leica-geosystems.com.cn">Qing-wu.Li</who>
    <bug_when>2024-09-26 22:15:11 -0700</bug_when>
    <thetext>For &apos;wpe_view_resize&apos; was not declared, change to wpe_view_resized fixed

but for &apos;wpe_toplevel_qtquick&apos; has not been declared, how to fix?</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>