Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP

Loading…

Fixed bug at https://developers.facebook.com/bugs/168127053284477 #436

Closed
wants to merge 1 commit into from

4 participants

@deyton

It seems to work now

@deyton

These two errors (WebKitDomain 102 and NSURLDomain -999) were both including in FBLoginDialog but not FBDialog.

@allaire

Thank you! Works well!

@champierre

Many thanks!!

@onebit
Collaborator

@deyton thanks for the fix! I have pushed it along with a handful of other with 5077944. Appreciate the contribution!!

@onebit onebit closed this
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
This page is out of date. Refresh to see the latest.
Showing with 3 additions and 1 deletion.
  1. +3 −1 src/FBDialog.m
View
4 src/FBDialog.m
@@ -409,7 +409,9 @@ - (void)webViewDidFinishLoad:(UIWebView *)webView {
- (void)webView:(UIWebView *)webView didFailLoadWithError:(NSError *)error {
// 102 == WebKitErrorFrameLoadInterruptedByPolicyChange
- if (!([error.domain isEqualToString:@"WebKitErrorDomain"] && error.code == 102)) {
+ // -999 == "Operation could not be completed"
+ if (!(([error.domain isEqualToString:@"NSURLErrorDomain"] && error.code == -999) ||
+ ([error.domain isEqualToString:@"WebKitErrorDomain"] && error.code == 102))) {
[self dismissWithError:error animated:YES];
}
}
Something went wrong with that request. Please try again.