If you get an error while refreshing you cannot allow an essential upgrade dialog...

[cydia.git] / CyteKit / WebViewController.mm

diff --git a/CyteKit/WebViewController.mm b/CyteKit/WebViewController.mm
index 61786da995512f1d97c1f2c47c144c65f0a7f91c..76379fb61f9ac1f2ea17289196f87a04d78d4710 100644 (file)
--- a/CyteKit/WebViewController.mm
+++ b/CyteKit/WebViewController.mm
@@ -147,6 +147,10 @@ float CYScrollViewDecelerationRateNormal;
     [super dealloc];
 }
 
     [super dealloc];
 }
 

+- (CyteWebView *) webView {
+    return (CyteWebView *) [self view];
+}
+

 - (NSURL *) URLWithURL:(NSURL *)url {
     return url;
 }
 - (NSURL *) URLWithURL:(NSURL *)url {
     return url;
 }


@@ -181,7 +185,7 @@ float CYScrollViewDecelerationRateNormal;
     ready_ = true;
 
     WebThreadLocked lock;
     ready_ = true;
 
     WebThreadLocked lock;

-    [webview_ loadRequest:request];
+    [[self webView] loadRequest:request];

 }
 
 - (void) reloadURLWithCache:(BOOL)cache {
 }
 
 - (void) reloadURLWithCache:(BOOL)cache {


@@ -257,7 +261,7 @@ float CYScrollViewDecelerationRateNormal;
 }
 
 - (void) _setViewportWidth {
 }
 
 - (void) _setViewportWidth {

-    [[webview_ _documentView] setViewportSize:CGSizeMake(width_, UIWebViewGrowsAndShrinksToFitHeight) forDocumentTypes:0x10];
+    [[[self webView] _documentView] setViewportSize:CGSizeMake(width_, UIWebViewGrowsAndShrinksToFitHeight) forDocumentTypes:0x10];

 }
 
 - (void) setViewportWidth:(float)width {
 }
 
 - (void) setViewportWidth:(float)width {


@@ -380,8 +384,6 @@ float CYScrollViewDecelerationRateNormal;
             action:@selector(close)
         ] autorelease]];
 
             action:@selector(close)
         ] autorelease]];
 

-        [delegate_ unloadData];
-

         [[self navigationController] presentModalViewController:navigation animated:YES];
     }
 }
         [[self navigationController] presentModalViewController:navigation animated:YES];
     }
 }


@@ -621,7 +623,7 @@ float CYScrollViewDecelerationRateNormal;
         } else if (button == [alert firstOtherButtonIndex]) {
             if (request_ != nil) {
                 WebThreadLocked lock;
         } else if (button == [alert firstOtherButtonIndex]) {
             if (request_ != nil) {
                 WebThreadLocked lock;

-                [webview_ loadRequest:request_];
+                [[self webView] loadRequest:request_];

             }
         }
 
             }
         }
 


@@ -703,7 +705,7 @@ float CYScrollViewDecelerationRateNormal;
         else if (button == (UIBarButtonItem *) [NSNull null])
             button = nil;
 
         else if (button == (UIBarButtonItem *) [NSNull null])
             button = nil;
 

-        [[self navigationItem] setRightBarButtonItem:button];
+        [[self navigationItem] setRightBarButtonItem:button animated:YES];

     }
 }
 
     }
 }
 


@@ -742,122 +744,141 @@ float CYScrollViewDecelerationRateNormal;
 
 - (id) initWithWidth:(float)width ofClass:(Class)_class {
     if ((self = [super init]) != nil) {
 
 - (id) initWithWidth:(float)width ofClass:(Class)_class {
     if ((self = [super init]) != nil) {

+        width_ = width;
+        class_ = _class;
+

         allowsNavigationAction_ = true;
 
         allowsNavigationAction_ = true;
 

-        class_ = _class;

         loading_ = [NSMutableSet setWithCapacity:5];
         loading_ = [NSMutableSet setWithCapacity:5];

-

         indirect_ = [[[IndirectDelegate alloc] initWithDelegate:self] autorelease];
 
         indirect_ = [[[IndirectDelegate alloc] initWithDelegate:self] autorelease];
 

-        CGRect bounds([[self view] bounds]);
+        reloaditem_ = [[[UIBarButtonItem alloc]
+            initWithTitle:UCLocalize("RELOAD")
+            style:[self rightButtonStyle]
+            target:self
+            action:@selector(reloadButtonClicked)
+        ] autorelease];

 
 

-        webview_ = [[[CyteWebView alloc] initWithFrame:bounds] autorelease];
-        [webview_ setDelegate:self];
-        [self setView:webview_];
+        loadingitem_ = [[[UIBarButtonItem alloc]
+            initWithTitle:@" "
+            style:UIBarButtonItemStylePlain
+            target:self
+            action:@selector(reloadButtonClicked)
+        ] autorelease];

 
 

-        if ([webview_ respondsToSelector:@selector(setDataDetectorTypes:)])
-            [webview_ setDataDetectorTypes:UIDataDetectorTypeAutomatic];
-        else
-            [webview_ setDetectsPhoneNumbers:NO];
+        indicator_ = [[[UIActivityIndicatorView alloc] initWithActivityIndicatorStyle:UIActivityIndicatorViewStyleWhite] autorelease];
+        [indicator_ setFrame:CGRectMake(15, 5, [indicator_ frame].size.width, [indicator_ frame].size.height)];
+        [indicator_ setAutoresizingMask:UIViewAutoresizingFlexibleLeftMargin];
+
+        [self applyLeftButton];
+        [self applyRightButton];
+    } return self;
+}
+
+- (NSString *) applicationNameForUserAgent {
+    return nil;
+}
+
+- (void) loadView {
+    CGRect bounds([[UIScreen mainScreen] applicationFrame]);
+
+    webview_ = [[[CyteWebView alloc] initWithFrame:bounds] autorelease];
+    [webview_ setDelegate:self];
+    [self setView:webview_];

 
 

-        [webview_ setScalesPageToFit:YES];
+    if ([webview_ respondsToSelector:@selector(setDataDetectorTypes:)])
+        [webview_ setDataDetectorTypes:UIDataDetectorTypeAutomatic];
+    else
+        [webview_ setDetectsPhoneNumbers:NO];
+
+    [webview_ setScalesPageToFit:YES];

 
 

-        UIWebDocumentView *document([webview_ _documentView]);
+    UIWebDocumentView *document([webview_ _documentView]);

 
 

-        // XXX: I think this improves scrolling; the hardcoded-ness sucks
-        [document setTileSize:CGSizeMake(320, 500)];
+    // XXX: I think this improves scrolling; the hardcoded-ness sucks
+    [document setTileSize:CGSizeMake(320, 500)];

 
 

-        [document setBackgroundColor:[UIColor clearColor]];
+    [document setBackgroundColor:[UIColor clearColor]];

 
 

-        // XXX: this is terribly (too?) expensive
-        [document setDrawsBackground:NO];
+    // XXX: this is terribly (too?) expensive
+    [document setDrawsBackground:NO];

 
 

-        WebView *webview([document webView]);
-        WebPreferences *preferences([webview preferences]);
+    WebView *webview([document webView]);
+    WebPreferences *preferences([webview preferences]);

 
 

-        // XXX: I have no clue if I actually /want/ this modification
-        if ([webview respondsToSelector:@selector(_setLayoutInterval:)])
-            [webview _setLayoutInterval:0];
-        else if ([preferences respondsToSelector:@selector(_setLayoutInterval:)])
-            [preferences _setLayoutInterval:0];
+    // XXX: I have no clue if I actually /want/ this modification
+    if ([webview respondsToSelector:@selector(_setLayoutInterval:)])
+        [webview _setLayoutInterval:0];
+    else if ([preferences respondsToSelector:@selector(_setLayoutInterval:)])
+        [preferences _setLayoutInterval:0];

 
 

-        [preferences setCacheModel:WebCacheModelDocumentBrowser];
-        [preferences setJavaScriptCanOpenWindowsAutomatically:YES];
-        [preferences setOfflineWebApplicationCacheEnabled:YES];
+    [preferences setCacheModel:WebCacheModelDocumentBrowser];
+    [preferences setJavaScriptCanOpenWindowsAutomatically:YES];
+    [preferences setOfflineWebApplicationCacheEnabled:YES];

 
 

-        if ([webview respondsToSelector:@selector(setShouldUpdateWhileOffscreen:)])
-            [webview setShouldUpdateWhileOffscreen:NO];
+    if (NSString *agent = [self applicationNameForUserAgent])
+        [webview setApplicationNameForUserAgent:agent];
+
+    if ([webview respondsToSelector:@selector(setShouldUpdateWhileOffscreen:)])
+        [webview setShouldUpdateWhileOffscreen:NO];

 
 #if LogMessages
 
 #if LogMessages

-        if ([document respondsToSelector:@selector(setAllowsMessaging:)])
-            [document setAllowsMessaging:YES];
-        if ([webview respondsToSelector:@selector(_setAllowsMessaging:)])
-            [webview _setAllowsMessaging:YES];
+    if ([document respondsToSelector:@selector(setAllowsMessaging:)])
+        [document setAllowsMessaging:YES];
+    if ([webview respondsToSelector:@selector(_setAllowsMessaging:)])
+        [webview _setAllowsMessaging:YES];

 #endif
 
 #endif
 

-        if ([webview_ respondsToSelector:@selector(_scrollView)]) {
-            scroller_ = [webview_ _scrollView];
+    if ([webview_ respondsToSelector:@selector(_scrollView)]) {
+        scroller_ = [webview_ _scrollView];

 
 

-            [scroller_ setDirectionalLockEnabled:YES];
-            [scroller_ setDecelerationRate:CYScrollViewDecelerationRateNormal];
-            [scroller_ setDelaysContentTouches:NO];
+        [scroller_ setDirectionalLockEnabled:YES];
+        [scroller_ setDecelerationRate:CYScrollViewDecelerationRateNormal];
+        [scroller_ setDelaysContentTouches:NO];

 
 

-            [scroller_ setCanCancelContentTouches:YES];
-        } else if ([webview_ respondsToSelector:@selector(_scroller)]) {
-            UIScroller *scroller([webview_ _scroller]);
-            scroller_ = (UIScrollView *) scroller;
+        [scroller_ setCanCancelContentTouches:YES];
+    } else if ([webview_ respondsToSelector:@selector(_scroller)]) {
+        UIScroller *scroller([webview_ _scroller]);
+        scroller_ = (UIScrollView *) scroller;

 
 

-            [scroller setDirectionalScrolling:YES];
-            // XXX: we might be better off /not/ setting this on older systems
-            [scroller setScrollDecelerationFactor:CYScrollViewDecelerationRateNormal]; /* 0.989324 */
-            [scroller setScrollHysteresis:0]; /* 8 */
+        [scroller setDirectionalScrolling:YES];
+        // XXX: we might be better off /not/ setting this on older systems
+        [scroller setScrollDecelerationFactor:CYScrollViewDecelerationRateNormal]; /* 0.989324 */
+        [scroller setScrollHysteresis:0]; /* 8 */

 
 

-            [scroller setThumbDetectionEnabled:NO];
+        [scroller setThumbDetectionEnabled:NO];

 
 

-            // use NO with UIApplicationUseLegacyEvents(YES)
-            [scroller setEventMode:YES];
+        // use NO with UIApplicationUseLegacyEvents(YES)
+        [scroller setEventMode:YES];

 
 

-            // XXX: this is handled by setBounces, right?
-            //[scroller setAllowsRubberBanding:YES];
-        }
+        // XXX: this is handled by setBounces, right?
+        //[scroller setAllowsRubberBanding:YES];
+    }

 
 

-        [scroller_ setFixedBackgroundPattern:YES];
-        [scroller_ setBackgroundColor:[UIColor clearColor]];
-        [scroller_ setClipsSubviews:YES];
+    [scroller_ setFixedBackgroundPattern:YES];
+    [scroller_ setBackgroundColor:[UIColor clearColor]];
+    [scroller_ setClipsSubviews:YES];

 
 

-        [scroller_ setBounces:YES];
-        [scroller_ setScrollingEnabled:YES];
-        [scroller_ setShowBackgroundShadow:NO];
+    [scroller_ setBounces:YES];
+    [scroller_ setScrollingEnabled:YES];
+    [scroller_ setShowBackgroundShadow:NO];

 
 

-        [self setViewportWidth:width];
+    [self setViewportWidth:width_];

 
 

-        reloaditem_ = [[[UIBarButtonItem alloc]
-            initWithTitle:UCLocalize("RELOAD")
-            style:[self rightButtonStyle]
-            target:self
-            action:@selector(reloadButtonClicked)
-        ] autorelease];
+    UITableView *table([[[UITableView alloc] initWithFrame:[webview_ bounds] style:UITableViewStyleGrouped] autorelease]);
+    [webview_ insertSubview:table atIndex:0];

 
 

-        loadingitem_ = [[[UIBarButtonItem alloc]
-            initWithTitle:@" "
-            style:UIBarButtonItemStylePlain
-            target:self
-            action:@selector(reloadButtonClicked)
-        ] autorelease];
-
-        indicator_ = [[[UIActivityIndicatorView alloc] initWithActivityIndicatorStyle:UIActivityIndicatorViewStyleWhite] autorelease];
-        [indicator_ setFrame:CGRectMake(15, 5, [indicator_ frame].size.width, [indicator_ frame].size.height)];
+    [table setAutoresizingMask:(UIViewAutoresizingFlexibleWidth | UIViewAutoresizingFlexibleHeight)];
+    [webview_ setAutoresizingMask:(UIViewAutoresizingFlexibleWidth | UIViewAutoresizingFlexibleHeight)];

 
 

-        UITableView *table([[[UITableView alloc] initWithFrame:bounds style:UITableViewStyleGrouped] autorelease]);
-        [webview_ insertSubview:table atIndex:0];
+    ready_ = false;
+}

 
 

-        [self applyLeftButton];
-        [self applyRightButton];
+- (void) releaseSubviews {
+    webview_ = nil;
+    scroller_ = nil;

 
 

-        [table setAutoresizingMask:(UIViewAutoresizingFlexibleWidth | UIViewAutoresizingFlexibleHeight)];
-        [webview_ setAutoresizingMask:(UIViewAutoresizingFlexibleWidth | UIViewAutoresizingFlexibleHeight)];
-        [indicator_ setAutoresizingMask:UIViewAutoresizingFlexibleLeftMargin];
-    } return self;
+    [super releaseSubviews];

 }
 
 - (id) initWithWidth:(float)width {
 }
 
 - (id) initWithWidth:(float)width {


@@ -877,7 +898,7 @@ float CYScrollViewDecelerationRateNormal;
 - (void) callFunction:(WebScriptObject *)function {
     WebThreadLocked lock;
 
 - (void) callFunction:(WebScriptObject *)function {
     WebThreadLocked lock;
 

-    WebView *webview([[webview_ _documentView] webView]);
+    WebView *webview([[[self webView] _documentView] webView]);

     WebFrame *frame([webview mainFrame]);
 
     JSGlobalContextRef context([frame globalContext]);
     WebFrame *frame([webview mainFrame]);
 
     JSGlobalContextRef context([frame globalContext]);


@@ -934,7 +955,7 @@ float CYScrollViewDecelerationRateNormal;
 }
 
 - (void) dispatchEvent:(NSString *)event {
 }
 
 - (void) dispatchEvent:(NSString *)event {

-    [(CyteWebView *) webview_ dispatchEvent:event];
+    [[self webView] dispatchEvent:event];

 }
 
 - (bool) hidesNavigationBar {
 }
 
 - (bool) hidesNavigationBar {