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

Favicon is not loaded on GoBack() and GoForward() #3874

Open
HashidaTKS opened this issue Jan 24, 2025 · 3 comments
Open

Favicon is not loaded on GoBack() and GoForward() #3874

HashidaTKS opened this issue Jan 24, 2025 · 3 comments
Labels
bug Bug report

Comments

@HashidaTKS
Copy link
Contributor

HashidaTKS commented Jan 24, 2025

Describe the bug

A favicon is not loaded after GoBack() and GoForward().
OnFaviconURLChange or OnBeforeResourceLoad for the favicon is not called when this issue occurs.

I have confirmed that this issue occurs on CEF131+, but have not yet confirmed it in earlier versions.

I found that this issue does not occur while DevTools are displayed.

To Reproduce

Steps to reproduce the behavior:

  1. Download the sample program from https://cef-builds.spotifycdn.com/index.html#windows32
  2. Open https://www.google.com/
  3. Open https://www.wikipedia.org/
  4. GoBack and GoFoward several times
  5. The favicon is not loaded (reproduced)

Image

Expected behavior

A favicon is loaded.

Versions (please complete the following information):

  • OS: Windows11
  • CEF Version: 131+ (not yet confirmed it in earlier versions.)

Additional context

Does the problem reproduce with the cefclient or cefsimple sample application at the same version?

  • Yes

Does the problem reproduce with Google Chrome at the same version?

  • No
@HashidaTKS HashidaTKS added the bug Bug report label Jan 24, 2025
@HashidaTKS
Copy link
Contributor Author

HashidaTKS commented Jan 24, 2025

I have tested this on CEF119 and found that GoBack() and GoForward() worked fine, I mean OnFaviconURLChange and OnBeforeResourceLoad for the favicon were called, on the alloy runtime mode, but the same issue occured on the chrome runtime mode.

@amaitland
Copy link
Contributor

How does it behave if you disable the BF cache?

https://peter.sh/experiments/chromium-command-line-switches/#disable-back-forward-cache

@HashidaTKS
Copy link
Contributor Author

How does it behave if you disable the BF cache?

https://peter.sh/experiments/chromium-command-line-switches/#disable-back-forward-cache

Thanks! it works fine if I disable the BF cache.
I will use this approach for my application as a workaround.

I feel it's nice that the favicon to be reloaded even with cache enabled.

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

No branches or pull requests

2 participants