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

Added overload of WebView::LoadHtml with missing parameter String^ url, as provided by CEF. #37

Merged
merged 1 commit into from
Mar 1, 2013

Conversation

krauskopf
Copy link

Added overload of method LoadHtml with additional parameter 'string^ url' to to Winforms and Wpf control.

This is necessary, because the current implementation of LoadHtml doesn't give access to the paremter as exposed by _clientAdapter->GetCefBrowser()->GetMainFrame()->LoadString.
Instead, the parameter is hardcoded to "about:blank". This has 2 disadvanteges with no workaround:

  1. html code with relative paths to resources are not possible.
  2. html code with (even absolute )paths to local files(images) are not possible, due to different security zone.

…' to Winforms and Wpf control.

This is necessary, because the current implementation of LoadHtml doesn't give access to the paremter as exposed by _clientAdapter->GetCefBrowser()->GetMainFrame()->LoadString.
Instead, the parameter is hardcoded to "about:blank". This has 2 disadvanteges with no workaround:
1. html code with relative paths to resources are not possible.
2. html code with (even absolute )paths to local files(images) are not possible, due to different security zone.
@ataranto
Copy link
Owner

Thanks. I haven't had much time for CefSharp lately but when I do I'll merge this.

@krauskopf
Copy link
Author

Thank you very much. I really appreciate your effort.

Von: anthony [mailto:notifications@github.com]
Gesendet: Dienstag, 23. Oktober 2012 22:41
An: ataranto/CefSharp
Cc: flowcoder
Betreff: Re: [CefSharp] Added overload of WebView::LoadHtml with missing parameter String^ url, as provided by CEF. (#37)

Thanks. I haven't had much time for CefSharp lately but when I do I'll merge this.


Reply to this email directly or view it on GitHub #37 (comment) .

Das Bild wurde vom Absender entfernt.

@ataranto ataranto merged commit d80dc3d into ataranto:master Mar 1, 2013
@ataranto
Copy link
Owner

ataranto commented Mar 1, 2013

merged, thanks!

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

Successfully merging this pull request may close these issues.

None yet

2 participants