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

CefFrame LoadString doesn't trigger OnAddressChange and OnLoadingStateChange #797

Closed
magreenblatt opened this issue Nov 22, 2012 · 9 comments
Labels
bug Bug report

Comments

@magreenblatt
Copy link
Collaborator

Original report by Anonymous.


Original issue 797 created by tumapathy on 2012-11-22T09:49:25.000Z:

What steps will reproduce the problem?

  1. Create Browser
  2. Call LoadString with valid html string and url name
  3. Observe OnAddressChange and OnLoadingStateChange is not triggered

What is the expected output? What do you see instead?
It should trigger these events.

What version of the product are you using? On what operating system?
CEF3 1271 Windows 7

@magreenblatt
Copy link
Collaborator Author

Original comment by Anonymous.


Comment 1. originally posted by tumapathy on 2012-11-22T13:35:36.000Z:

Please wait. It works fine CefClient. I will investigate further and add information here.

@magreenblatt
Copy link
Collaborator Author

Original comment by Anonymous.


Comment 2. originally posted by tumapathy on 2012-11-22T15:25:31.000Z:

It is reproducible in CefClient. If you don't load start page and call LoadString directly, the page itself is not getting loaded. Attached cefclient.patch file.

It has been created in version CEF3 1271.

@magreenblatt
Copy link
Collaborator Author

Original changes by Anonymous.


  • set attachment to "cefclient.patch"

@magreenblatt
Copy link
Collaborator Author

Original comment by Anonymous.


Comment 3. originally posted by tumapathy on 2012-11-22T15:27:59.000Z:

It is same as issue #579
http://code.google.com/p/chromiumembedded/issues/detail?id=579&q=LoadString

@magreenblatt
Copy link
Collaborator Author

Comment 4. originally posted by magreenblatt on 2012-11-26T18:06:22.000Z:

@magreenblatt
Copy link
Collaborator Author

  • set state to "duplicate"

@magreenblatt
Copy link
Collaborator Author

Original comment by Anonymous.


Comment 5. originally posted by vlhm@splitmedialabs.com on 2013-10-16T14:28:15.000Z:

We have spend some time trying to port from CEF 1 to CEF 3 . This has been a good amount of work due to the multi-process architecture requiring to implement custom marshalling...we are almost finishing but now just found out that some fundamental functions like LoadString (this issue and 579) has not worked properly since 2012.

Initially we used following build: cef_binary_3.1547.1412_windows32
LoadString worked fine with supplied http URL. But I can not make it to work when supply some file URL. I tried following:
file:///dummy.html (this works with CEF1)
file:///C:/dummy.html
file:///C|/dummy.html

Then we tried to update to following build: cef_binary_3.1621.1450_windows32
Now we can not make LoadString to work at all even with http URL.

We trying to call LoadString in OnAfterCreated and in OnBeforeBrowse callbacks in following way:

CefString Content = L"test page";
CefString Url = L"http://localhost";
browser->GetMainFrame()->LoadString(Content, Url);

Any ETA on a fix for LoadString ?

@magreenblatt
Copy link
Collaborator Author

Comment 6. originally posted by magreenblatt on 2013-10-16T14:35:51.000Z:

@ commentcomment 5.: LoadString is inherently flawed due to the requirement of creating render processes -- you're better off using standard network loading (browse to a URL and handle via a custom scheme handler or GetResourceHandler). See https://code.google.com/p/chromiumembedded/wiki/GeneralUsage.

@magreenblatt
Copy link
Collaborator Author

Original comment by Anonymous.


Comment 7. originally posted by vlhm@splitmedialabs.com on 2013-10-20T13:30:52.000Z:

@ commentcomment 6.: Thank you very much for the explanation and advise. I already finished implementing using this method and getting ready to test it.

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

1 participant