Fix popup window logic - use CEF built-in popup management #303
Conversation
tishion
commented
Aug 26, 2023
tishion
commented
- 重新实现popup browser的逻辑
- 使用CEF内置的popup 生命周期管理逻辑
- onBeforePopup 方法提供允许/禁止弹出窗口的拦截时机，可以重载该函数，返回true/false来控制，以及在该方法内设置一些参数
- onPopupCreated 方法提供自定义已经弹出的popup browser的能力
|
macOS和Linux没有测试过 |
|
@L-Super review一下 |
ASAP，我抽时间测试一下Linux 平台 |
| QCefSettingPrivate::CopyFromCefBrowserSettings(&s, &settings); | ||
| bool cancelPopup = q->onBeforePopup(frameId, url, name, d, rc, s, disableJavascript); | ||
| QCefSettingPrivate::CopyToCefBrowserSettings(&s, &settings); |
There was a problem hiding this comment.
这里不理解为什么还要再调用一次CopyToCefBrowserSettings
There was a problem hiding this comment.
先把CefSettings转为QCefSetting，然后吧QCefSetting传给用户，用户可能需要修改setting，然后再把用户的修改copy回CefSetting，再传回给CEF
| void | ||
| QCefView::onPopupCreated(QWidget* popup, const QString& url, const QString& name) | ||
| { | ||
| // create a top level window container | ||
| QMainWindow* popupWin = new QMainWindow(); | ||
| popupWin->setAttribute(Qt::WA_DeleteOnClose, true); | ||
| popupWin->setWindowTitle(name); | ||
| popupWin->resize(popup->size()); | ||
|
|
||
| // add the popup browser widget into the window | ||
| popupWin->setCentralWidget(popup); | ||
|
|
||
| // show window | ||
| popupWin->show(); | ||
| } |
There was a problem hiding this comment.
这个函数是提供给用户重载用的，内部的默认实现是创建一个顶层窗口来放置popup，用户可能有其他的实现需求，比如放到别的tab里面。
There was a problem hiding this comment.
那实际上应该不需要吧，是否可以通过注释告诉用户可以这样重载使用？
|
在Windows运行demo时，发现通过 |
这个是正常的，因为popup的name没有变化，CEF内部其实保持了一个按照name来映射的popup，如果这个name的popup已经存在了，就不会再走popup流程了。 |
|
这个实现在Linux NCW模式下还是不行，主要是Linux的UI系统问题。 |
OK，明白了 |
通过Qt还是无法隔离Linux UI的问题吗 |
