CODEWITHSUNDEEP
c#winformsclassbrowserwebbrowser-control
Mathlight
Mathlight

Reputation: 6653

winform webbrowser not replaced

From my Form1 I initialize a class scraper. In the scraper class is an function login. The idea is that that class log's the user in on an website, and returned the web browser so that an logged in webbrowser control is available in Form1.

I've got this code so far: Form1

private void button1_Click(object sender, EventArgs e)
{
    Scraper scraper = new Scraper(this);
    scraper.login(conf._webLogin);  
}

public void updateLoginWeb(WebBrowser web)
{
    webBrowser1 = web;
    MessageBox.Show("DONE");
}

The conf class:

public WebBrowser _webLogin = new WebBrowser();

The scraper class:

private Form1 parent;

private WebBrowser _web_Login = new WebBrowser();


public Scraper()
{

}

public Scraper(Form1 parent)
    : this()
{
    this.parent = parent;
}

public void login(WebBrowser web)
{
    _web_Login = web;
    _web_Login.DocumentCompleted += new WebBrowserDocumentCompletedEventHandler(login_DocumentCompleted);
    _web_Login.Navigate("http://www.google.com/");
}

private void login_DocumentCompleted(object sender, WebBrowserDocumentCompletedEventArgs e)
{
    //This line is so you only do the event once   
    if (e.Url != _web_Login.Url)
        return;
    parent.updateLoginWeb(_web_Login);
}

I use google as test, but nothing works (not even an another site).

The problem is that the webbrowser in the Form isn't updated. It's still an white screen.

What do you guys think of this? Do you know what the problem is or do you guys know an better way to handle this?

Upvotes: 0

Views: 151

Answers (1)

fcuesta
fcuesta

Reputation: 4520

I think your problem is that you can not simply assign the webBrowser variable:

webBrowser1 = web;

You are changing the Form1.webBrower1 variable, but the Forms.Controls collection is still pointing to the original webBrowser control.

Can't you just pass Form1.webBrower1 to scraper.login function?:

private void button1_Click(object sender, EventArgs e)
{
    Scraper scraper = new Scraper(this);
    scraper.login(webBrowser1);  
}

public void updateLoginWeb(WebBrowser web)
{
    //webBrowser1 = web;  // you don't need this anymore
    MessageBox.Show("DONE");
}

If you really need to replace your control you can do something like:

public void updateLoginWeb(WebBrowser web)
{
    Controls.Remove(webBrowser1);
    Controls.Add(web);
    webBrowser1 = web;  // you don't need this anymore
    MessageBox.Show("DONE");
}

But you will probably to set the new webbrowser layout properties manually.

Upvotes: 1

Related Questions