Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

新版本为何没有onBeforeBrowse重载了, 想要阻断redirect需要怎么操作呢 #415

Open
liaowubinf opened this issue Jun 27, 2024 · 10 comments

Comments

@liaowubinf
Copy link

如题
新版本为何没有onBeforeBrowse重载了, 想要阻断redirect需要怎么操作呢

@liaowubinf
Copy link
Author

@L-Super
Copy link
Member

L-Super commented Jul 3, 2024

看了一下相关提交,没有onBeforeBrowser接口。

@liaowubinf
Copy link
Author

看了一下相关提交,没有onBeforeBrowser接口。

是的, 我也没找到, 但最近接手的一个老项目, 里面的qcefview.h里面有这个
///


/// Gets called before the main browser redirect url
///

///
///
///
///
///
virtual bool onBeforeBrowse(qint64 frameId,
const std::string& url,
const bool user_gesture,
const bool is_redirect);

我在将老项目升级为64位时发现新版没有这个方法了

@liaowubinf
Copy link
Author

看了一下相关提交,没有onBeforeBrowser接口。

想问下没有这个onBeforeBrowse方法, 还有别的方式能拦截window.location.href这种redirect吗

@liaowubinf
Copy link
Author

先确认一下,你是使用的哪个仓库? https://github.com/CefView/CefViewCore/blob/fe335787e8ce5d9bf719617a248c5a9bf7e8e35c/include/CefViewBrowserClient.h#L367

https://github.com/winsoft666/QCefWidget/blob/3237d0165b51c60f8deb5e2919420d0a33ce50a1/src/QCefWidget/CefBrowserApp/QCefBrowserHandler.h#L256

仓库没错, 头文件就是跟这个仓库一样的,里面的用法也是QCefview, 但是之前有onBeforeBrowse, 升级dll后编译报错了

@L-Super
Copy link
Member

L-Super commented Jul 4, 2024

想问下没有这个onBeforeBrowse方法, 还有别的方式能拦截window.location.href这种redirect吗

可以参考文档,看哪个函数能够实现您的目的。感觉可以通过addressChanged进行判断。

@tishion
Copy link
Member

tishion commented Jul 4, 2024

QCefView没有暴露过这个接口给上层,需要的话你自己实现就行了。

@liaowubinf
Copy link
Author

想问下没有这个onBeforeBrowse方法, 还有别的方式能拦截window.location.href这种redirect吗

可以参考文档,看哪个函数能够实现您的目的。感觉可以通过addressChanged进行判断。

不行, adressChanged是 ed ,代表是改变之后的, 不能阻断redirect

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants