CefView / QCefView

A Qt Widget encapsulated CEF view based on QWidget
https://cefview.github.io/QCefView/
GNU Lesser General Public License v2.1
547 stars 144 forks source link

js调用C++ #78

Closed Hua-King closed 2 years ago

Hua-King commented 2 years ago

发现js调C++方式不够灵活。 image 直接这样写貌似不行。 必须要传入第二个参数,哪怕是空的都行。 image 是否可以调整?

tishion commented 2 years ago

https://github.com/CefView/CefViewCore/blob/4c5f6da8e8a6e2c3e0e313df0fef217b303b7ff9/src/CefView/CefBrowserApp/CefViewBrowserClient.cpp#L121-L130

bool
CefViewBrowserClient::InvokeMethod(CefRefPtr<CefBrowser> browser,
                                   CefRefPtr<CefFrame> frame,
                                   CefRefPtr<CefProcessMessage> message)
{
  // validate the argument list
  CefRefPtr<CefListValue> arguments = message->GetArgumentList();
  if (!arguments || (arguments->GetSize() < 2))  // 这一行改为 if (!arguments || (arguments->GetSize() < 1))
    return false;

main分支目前依赖了一个较旧的core,所以目前无法切换,你先自己修改你本地的core就行了。

Hua-King commented 2 years ago

哪一个分支比较新?修改了这个问题的呢?

tishion commented 2 years ago

都没有修改

tishion commented 2 years ago

Fixed https://github.com/CefView/QCefView/pull/80