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

Add Reload(bool ignoreCache) method overload #322

Merged
merged 2 commits into from Apr 16, 2014

Conversation

amaitland
Copy link
Member

Add Reload(bool ignoreCache) method overload allowing for force Reload of web pages

auto cefBrowser = _renderClientAdapter->GetCefBrowser();

if (cefBrowser != nullptr)
{
cefBrowser->Reload();
if(ignoreCache)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Here it should be if (ignoreCache) (with space) to match the style of the rest of the file.

@perlun
Copy link
Member

perlun commented Apr 14, 2014

Apart from these suggestions, a nice change! Will happily merge (or @jornh can even do it if you like) as soon as these things have been resolved.

Fix code comment - Update Reload function code comment with suggestion in IWpfWebBrowser
@amaitland
Copy link
Member Author

@perlun Is there anything outstanding on this one?

@perlun
Copy link
Member

perlun commented Apr 16, 2014

No, it's good now. @jornh, please merge and add to changelog. Go for your idea of moving it back to the wiki please. TIA.

Sent from my iPhone

On 16 apr 2014, at 06:27, "amaitland" <notifications@github.commailto:notifications@github.com> wrote:

@perlunhttps://github.com/perlun Is there anything outstanding on this one?

Reply to this email directly or view it on GitHubhttps://github.com//pull/322#issuecomment-40559403.

@jornh jornh added this to the 31.0.0 milestone Apr 16, 2014
@jornh jornh self-assigned this Apr 16, 2014
@jornh jornh merged commit 7ca2dcd into cefsharp:master Apr 16, 2014
@jornh
Copy link
Contributor

jornh commented Apr 16, 2014

Merged now. Thanks! @amaitland :)

@amaitland amaitland deleted the feature/add-reload-overload branch April 16, 2014 12:51
@amaitland
Copy link
Member Author

Sweet! Thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants