Почему browser.FrameLoadEnd перестал работать нормально?

Question

[Дядя Богдан]

Типо раньше было все более менее нормально, срабатывало 2 раза ( при начале и конце загрузки ) и нормально, а теперь срабатывает только при начале, когда html пустой. Че не так?

public Form1()
        {
            InitializeComponent();
            InitializeChromium();

            browser.FrameLoadEnd += WebBrowserFrameLoadEnded;
        }


        private void WebBrowserFrameLoadEnded(object sender, FrameLoadEndEventArgs e)
        {

            Task.Run(async () => html = await browser.GetSourceAsync());


            MessageBox.Show(html); // html = null
        }

Comments

[edward_freedom]

private async void WebBrowserFrameLoadEnded(object sender, FrameLoadEndEventArgs e)
{
	await Task.Run(()=>{
		var html = await browser.GetSourceAsync();
		Debug.WriteLine(html);
	});  
}

[Василий Банников]

edward_freedom, А зачем в Task.Run заворачивать? (я не уверен, как там винформы работают)

[edward_freedom]

Василий Банников, а ты про область видимости в книге проходил главу? Я не особо вижу нужды использовать task run тут

[Василий Банников]

edward_freedom,

а ты про область видимости в книге проходил главу? Я не особо вижу нужды использовать task run тут

Так и я не вижу - вот и убрал его. Подумал, что у тебя в этом есть какой-то сакральный смысл.
Подумал мб, это как-то связано с контекстом синхронизации

[edward_freedom]

Василий Банников,

а ты про область видимости в книге проходил главу? Я не особо вижу нужды использовать task run тут

думал что отвечаю автору, а не тебе

[Василий Банников]

Ответ легко ищется поисковиком по запросу "Cefsharp GetSourceAsync deadlock"

[Дядя Богдан]

Василий Банников, В натуре ничего нету, то что было, не помогло.

Answer 1

[Василий Банников]

Task.Run(async () => html = await browser.GetSourceAsync());
            MessageBox.Show(html); // html = null

Вы не дождались окончания загрузки.
Попробуйте сделать хендлер async void и сделать так:

html = await browser.GetSourceAsync()
MessageBox.Show(html);

Comments

[Василий Банников]

(Хз, мне просто весело давать очевидные ответы вам)

[edward_freedom]

Василий Банников, но пока мы будем ему давать готовые коды, он не соизволит книгу прочитать

[Роман]

edward_freedom, И даже зарплатой не поделится. )))))

[Дядя Богдан]

Василий Банников, Не работает! ( дедлок)

[Василий Банников]

Дядя Богдан, в какой момент?

[Дядя Богдан]

Василий Банников, html = await browser.GetSourceAsync()

Капец вы быстро отвечаете...

[Василий Банников]

Дядя Богдан, попробуй добавить .ConfigureAwait(false)
И покажи полностью код.

[Дядя Богдан]

Уже делал.
Код:

using System;
using CefSharp;
using HtmlAgilityPack;
using CefSharp.OffScreen;
using System.Windows.Forms;
using System.Threading;
using System.Net;
using System.Threading.Tasks;


    public partial class Form1 : Form
    {
        private ChromiumWebBrowser browser;
        private string html;
        HtmlAgilityPack.HtmlDocument doc = new HtmlAgilityPack.HtmlDocument();
        HtmlNodeCollection node;

        public Form1()
        {
            InitializeComponent();
            InitializeChromium();

            browser.FrameLoadEnd += WebBrowserFrameLoadEnded;
        }

        private void button1_Click(object sender, EventArgs e)
        {
            browser.Load($"https://kad.arbitr.ru/Card?number={textBox1.Text}");
            Number.Text = "Номер: " + textBox1.Text;
        }

        private async void WebBrowserFrameLoadEnded(object sender, FrameLoadEndEventArgs e)
        {
            html = await browser.GetSourceAsync().ConfigureAwait(false);
            MessageBox.Show(html);
            if (html != null)
            {

                doc.LoadHtml(html);     

                node = doc.DocumentNode.SelectNodes("//*[@id='b-case-header']/ul[2]/li[4]/a");
                SetTextToControl(Plaintiffs, "Истцы:\n" + node[0].InnerText.Trim());
            }

private void InitializeChromium()
        {

            CefSettings settings = new CefSettings();
            Cef.Initialize(settings);
            browser = new ChromiumWebBrowser("https://kad.arbitr.ru");
           //Controls.Add(browser);
           //browser.Dock = DockStyle.Fill;
        }

        private void SetTextToControl(Control control, string text)
        {
                Invoke(new Action(() => control.Text = text));
        }

        private void Form1_FormClosing(object sender, FormClosingEventArgs e)
        {
            Cef.Shutdown();
        }

[Роман]

Василий Банников, Только по выходу у него не захватит назад контекст синхронизации и будут очередные вопросы. Да скорее всего, что-то навертел в коде, что у него дэдлок вылетает.

[Роман]

Дядя Богдан, у вас browser.Load асинхронный метод, сделать обработчик кнопки async и await browser.Load и убрать ConfigureAwait(false)

[Дядя Богдан]

Роман, Такое... Ожидание void невозможно...

[Василий Банников]

Дядя Богдан, Load синхронный. Его не надо ожидать.

[Дядя Богдан]

Василий Банников, Так Роман написал что он асинхронный...

у вас browser.Load асинхронный метод, сделать обработчик кнопки async и await browser.Load и убрать ConfigureAwait(false)

[Василий Банников]

Дядя Богдан, асинхронный видимо в смысле, что он через евенты результат выдаёт.

[Василий Банников]

Дядя Богдан, посмотри доку и примеры

[Дядя Богдан]

Василий Банников, ВООБЩЕ ВСЕ ПЕРЕПИСАЛ! Но все равно дедлок...

using System;
using CefSharp;
using HtmlAgilityPack;
using CefSharp.WinForms;
using System.Windows.Forms;
using System.Threading;
using System.Net;
using System.Threading.Tasks;

namespace WinFormsApp1
{

    public partial class Form1 : Form
    {
        private ChromiumWebBrowser browser;
     //   HtmlAgilityPack.HtmlDocument doc = new HtmlAgilityPack.HtmlDocument();
     //   HtmlNodeCollection node;

        public Form1()
        {
            InitializeComponent();
            InitializeChromium();

            browser.FrameLoadEnd += Browser_FrameLoadEnd;
        }

        private async void Browser_FrameLoadEnd(object sender, FrameLoadEndEventArgs e)
        {
            string html = await browser.GetSourceAsync(); // тут все умирает...
            MessageBox.Show(html);
        }


        private void button1_Click(object sender, EventArgs e)
        {
            browser.Load($"https://kad.arbitr.ru/Card?number={textBox1.Text}");
        }



        private void InitializeChromium()
        {

            CefSettings settings = new CefSettings();
            Cef.Initialize(settings);
            browser = new ChromiumWebBrowser("https://kad.arbitr.ru");
            Controls.Add(browser);
            browser.Dock = DockStyle.Fill;
        }

        private void Form1_FormClosed(object sender, FormClosedEventArgs e)
        {
            Cef.Shutdown();

        }
    }
}

[Василий Банников]

Дядя Богдан, попробуй что-то отсюда: https://cefsharp.github.io/api/51.0.0/html/T_CefSh...

[Василий Банников]

Дядя Богдан, похоже хрень какая-то с этим методом. Попробуй написать свой StringVisitor и использовать метод GetSource

[Дядя Богдан]

Василий Банников, А типо... А как? Причем тут StringVisitor? Типо создаем переменную ChromiumWebBrowser и там вызываем .GetSourceAsync(), я открыл класс ChromiumWebBrowser и там нету метода GetSourceAsync. Я что-то не понимаю?

[Василий Банников]

Дядя Богдан,

я открыл класс ChromiumWebBrowser

Эмм, а зачем? Скорее всего метод GetSourceAsync лежит в Control. (а вообще я говорил про Frame)

А типо... А как? Причем тут StringVisitor?

StringVisitor необходимо передать в метод GetSource.

Мне кажется, стоит сделать так, тк GetSourceAsync реализован подозрительно и испльзующийся в нём TaskStringVisitor может вызвать дедлок.

[Василий Банников]

Дядя Богдан, ещё можно попробовать ContinueWith, как советуют на гитхабе
Ты вообще пробовал свою проблему гуглить? Это первой строчкой выдаёт по запросу "cefsharp GetSourceAsync deadlock"

И ещё там есть буквально пример в репозитории, как стоит делать, чтобы не было дедлока (но на него жалуются, что иногда таки может зависнуть)

[Дядя Богдан]

Василий Банников, Что смог, то изменил. Надеюсь не бред... Куда это деть?

public interface IStringVisitor : IDisposable
        {
            /// <summary>
            ///  Method that will be executed.
            /// </summary>
            /// <param name="str">string (result of async execution)</param>
            void Visit(string str);
        }


        public class TaskStringVisitor : IStringVisitor
        {
            private string CompletionSource;

            /// <summary>
            ///  Method that will be executed.
            /// </summary>
            /// <param name="str">string</param>
            void IStringVisitor.Visit(string str)
            {
                CompletionSource = str;
            }

            /// <summary>
            /// Task that can be awaited for the result to be retrieved async
            /// </summary>
            public string resultic
            {
                get { return CompletionSource; }
            }

            void IDisposable.Dispose()
            {

            }
        }

[Василий Банников]

Дядя Богдан, выкинуть, тк он бесполезен. У тебя что все сайты кроме этого забанены, что ты даже на гитхаб не зашёл?

[Дядя Богдан]

Василий Банников, Зашел, но мне с ContinueWith не нравится, там странно дебаг работает

[Василий Банников]

Дядя Богдан, нормально с ним дебаг работает. Не выдумывай.

[Дядя Богдан]

Василий Банников, КСТАТИТ ДАЖЕ ОНО НЕ РАБОТАЕТ! Хотя, мне казалось что она раньше работало... хз... А про дебаг.
В натуре плохо работает, сразу все!

Код

private void Browser_FrameLoadEnd(object sender, FrameLoadEndEventArgs e)
        {
            browser.GetSourceAsync().ContinueWith(taskHtml =>
            {
                var html = taskHtml.Result;

                MessageBox.Show(html);
            });
        }

[Дядя Богдан]

Василий Банников, Может, что с проектом не так?

[Василий Банников]

Дядя Богдан, на скрине не вижу, чтобы что-то хоть как-то не работало.
Зато вижу что нугеты кривые

[Дядя Богдан]

Василий Банников, Кривой вообще nuget парсера html который даже в проекте сейчас не использую.Debug: Но типо он не показывает по строчно, а показывает выполнение сразу всей continue. Сейчас новый проект сделал, срабатывает один раз с тем же самым кодом при начале загрузки ( хотя эвент указал что при конце загруки)
Варианты с stack overflow ( меня там забанили почему-то) и github не помогли.

[Василий Банников]

Дядя Богдан, по тому что надо бряку ставить внутрь лямбды, а не снаружи.

[Василий Банников]

Дядя Богдан,

меня там забанили почему-то

Удивляюсь, почему тут не забанили

[Дядя Богдан]

Василий Банников, Бряку?

[Василий Банников]

Дядя Богдан, бряка = break point = точка останова

[Дядя Богдан]

Василий Банников, я думал это по вопросу, но ладно

[edward_freedom]

Василий Банников, выполни js код на странице и получишь исходник
document.documentElement.outerHTML

[Дядя Богдан]

edward_freedom, И как ее в переменную c# занести?
Так не работает:

var html = browser.ExecuteScriptAsync("document.documentElement.outerHTML");

[edward_freedom]

Дядя Богдан, а где await?

[Василий Банников]

edward_freedom, ору

[Дядя Богдан]

edward_freedom, Оно там даже не поставится...

[edward_freedom]

Дядя Богдан, не поленился и проверил, все поставилось

[Василий Банников]

edward_freedom, только зачем делать EvaluateScript, если GetSource сделает ровно то же самое?

[Дядя Богдан]

Василий Банников, У меня нету GetSource, только GetSourceAsync, а там надо FrameLoadEndEventArgs - это слишком сложно для меня

[Василий Банников]

Дядя Богдан, я просто убрал суффикс Async.

это слишком сложно для меня

Может тогда будешь делать что-то попроще?
Попробуй лучше на консольных придожентях учиться

[Дядя Богдан]

Василий Банников, Нужно чтобы было прям круто с winforms или что-нибудь такое

[Дядя Богдан]

edward_freedom, Не работает, но скорее всего из-за того что я использую winforms и там приколы с потоками.

[edward_freedom]

Дядя Богдан, у меня на скрине винформс

[Дядя Богдан]

edward_freedom, Согласен, лохонулся, но вы не знаете, почему это не работает с messagebox и вообще всей этой ui??? Все дело в потоках?

[edward_freedom]

Дядя Богдан, а зачем тебе исходный код в messagebox выводить? Оно же рендериться будет пол часа, в дебаг выведи. И ты потоки не используешь, дело в них не может быть