I am upgrading EO.WebBrowser from 2017(17.1.65.0) to 2018(latest 18.2.53.0) however the latest version is failing my original program logic.

The LoadCompleted event is not firing on sub-sequence requests after first "LoadUrl".

I see a similar issue on older builds:
https://www.essentialobjects.com/forum/postst9751_EO-Webbrowser-form-submit-and-wait.aspx
https://www.essentialobjects.com/Forum/postsm36936_LoadCompleted-event-problem.aspx

Steps to reproduce:

1. Use 18.2.53.0 to make a simple webbrowser.
2. Register WebView.LoadCompleted handler to write log when triggered
3. WebView.LoadRequest("http://www.example.com")
4. LoadCompleted triggered when finished loading the page
5. Click any link on the page to make it navigate to another page
6. LoadCompleted not triggering any more.


EDIT:
just did some testing, the feature in 18.0.70 is working, and broken in 18.0.98

Hi,

We have looked into both issues. Here are our findings:

1. LoadCompleted event is in fact fired correctly. However the Url property of the event argument is not properly populated (is always null). This may have led to the wrong conclusion that the event has not been fired at all. The real issue is e.Url should not be empty. We will fix this in our next build;

2. Calling GetHtml from inside BeforeNavigate is not supported. It might have worked "accidentally" in early builds but it's not supposed to work. The root of the problem is BeforeNavigate is triggered by Chromium's render process's main thread and it's a blocking call. This means the render's main thread will wait until it hears an answer from your code about whether to proceed or not. While it is waiting for you, it can not do anything else that are also done in the render's main thread. These would include but not limiting to tasks such as evaluate JavaScript code, making changes to the DOM tree, or retrieving the HTML/source of the document. This is why GetHtml would hang in this case. We have some safe guard mechanism in place for such scenarios. For example, if you were to call WebView.EvalScript inside BeforeNavigate, you would get an exception stating this is not allowed. GetHtml should have perform the same check as well. As such you must change your design not to rely on anything that you will need at this last minutes to decide whether to proceed or not;

Hope this helps. Please feel free to let us know if you still have any questions.

Thanks!

Yes. We will fix it ASAP and reply here again once the new build is out.



Yes. You should be able to do that.

Tested and confirmed working.