Reviewed by Tim Hatcher.
authortomernic <tomernic@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 12 Jul 2006 08:25:29 +0000 (08:25 +0000)
committertomernic <tomernic@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 12 Jul 2006 08:25:29 +0000 (08:25 +0000)
        <rdar://problem/4624858> AGL isn't 64-bit yet; temporarily remove it from WebKit 64-bit build

        Also, fixed a LOG_ERROR() so that it uses the CGL error instead of the AGL error; Tim H missed this in his build fix
        from earlier.

        * Plugins/WebBaseNetscapePluginView.h:
        * Plugins/WebBaseNetscapePluginView.m:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@15378 268f45cc-cd09-0410-ab3c-d52691b4dbfc

WebKit/ChangeLog
WebKit/Plugins/WebBaseNetscapePluginView.h
WebKit/Plugins/WebBaseNetscapePluginView.m

index c0223d0bb583bafd4ad72b27ea47d71836739ad7..87fdb323d96e0ca00a98b62de9bacea35cefcfd9 100644 (file)
@@ -1,3 +1,15 @@
+2006-07-12  Tim Omernick  <timo@apple.com>
+
+        Reviewed by Tim Hatcher.
+
+        <rdar://problem/4624858> AGL isn't 64-bit yet; temporarily remove it from WebKit 64-bit build
+
+        Also, fixed a LOG_ERROR() so that it uses the CGL error instead of the AGL error; Tim H missed this in his build fix
+        from earlier.
+
+        * Plugins/WebBaseNetscapePluginView.h:
+        * Plugins/WebBaseNetscapePluginView.m:
+
 2006-07-11  John Sullivan  <sullivan@apple.com>
 
         Reviewed by Kevin and Tim O
index a7b646c0117ef4340640050668d7b2a38a4f652e..2f19dd41b1a1360dd758eb3f16220ca1cb6aebf3 100644 (file)
 
 #import <Cocoa/Cocoa.h>
 
+// FIXME: AGL isn't 64-bit (yet).  See <rdar://problem/4624858>.        
+#if !__LP64__
 #import <AGL/agl.h>
+#endif /* __LP64__ */
 #import <WebKit/npfunctions.h>
 #import <WebKit/npapi.h>
 
@@ -44,7 +47,10 @@ typedef union PluginPort {
     NP_Port qdPort;
 #endif        
     NP_CGContext cgPort;
+// FIXME: AGL isn't 64-bit (yet).  See <rdar://problem/4624858>.        
+#if !__LP64__
     NP_GLContext aglPort;
+#endif /* __LP64__ */
 } PluginPort;
 
 @interface WebBaseNetscapePluginView : NSView
@@ -65,9 +71,12 @@ typedef union PluginPort {
     PluginPort lastSetPort;
     NPDrawingModel drawingModel;
     
+// FIXME: AGL isn't 64-bit (yet).  See <rdar://problem/4624858>.        
+#if !__LP64__
     // These are only valid when drawingModel is NPDrawingModelOpenGL
     AGLContext aglContext;
     NSWindow *aglWindow;
+#endif /* __LP64__ */
 
     BOOL isStarted;
     BOOL inSetWindow;
index 300c22a83029be2d74c78507f9202dc672616d2d..a9a73b7c1df500e91fe48ad730dc77d9be84ee1e 100644 (file)
@@ -68,6 +68,8 @@
 @interface WebBaseNetscapePluginView (Internal)
 - (void)_viewHasMoved;
 - (NSBitmapImageRep *)_printedPluginBitmap;
+// FIXME: AGL isn't 64-bit (yet).  See <rdar://problem/4624858>.
+#if !__LP64__
 - (BOOL)_createAGLContextIfNeeded;
 - (BOOL)_createWindowedAGLContext;
 - (BOOL)_createWindowlessAGLContext;
@@ -77,6 +79,7 @@
 - (void)_reshapeAGLWindow;
 - (void)_hideAGLWindow;
 - (NSImage *)_aglOffscreenImageForDrawingInRect:(NSRect)drawingInRect;
+#endif /* __LP64__ */
 @end
 
 static WebBaseNetscapePluginView *currentPluginView = nil;
@@ -102,9 +105,12 @@ typedef struct {
     CGContextRef context;
 } PortState_CG;
 
+// FIXME: AGL isn't 64-bit (yet).  See <rdar://problem/4624858>.        
+#if !__LP64__
 typedef struct {
     AGLContext oldContext;
 } PortState_GL;
+#endif /* __LP64__ */
 
 @interface WebPluginRequest : NSObject
 {
@@ -441,7 +447,9 @@ static OSStatus TSMEventHandler(EventHandlerCallRef inHandlerRef, EventRef inEve
             // FIXME (4544971): Clip to dirty region when updating in "windowless" mode (transparent), like in the QD case
         }
         break;
-        
+
+// FIXME: AGL isn't 64-bit (yet).  See <rdar://problem/4624858>.        
+#if !__LP64__
         case NPDrawingModelOpenGL:
         {
             // An OpenGL plugin's window may only be set while the plugin view is being updated
@@ -488,6 +496,7 @@ static OSStatus TSMEventHandler(EventHandlerCallRef inHandlerRef, EventRef inEve
             }
         }
         break;
+#endif /* __LP64__ */
         
         default:
             ASSERT_NOT_REACHED();
@@ -541,9 +550,12 @@ static OSStatus TSMEventHandler(EventHandlerCallRef inHandlerRef, EventRef inEve
         }
         break;
         
+// FIXME: AGL isn't 64-bit (yet).  See <rdar://problem/4624858>.        
+#if !__LP64__
         case NPDrawingModelOpenGL:
             aglSetCurrentContext(((PortState_GL *)portState)->oldContext);
         break;
+#endif /* __LP64__ */
         
         default:
             ASSERT_NOT_REACHED();
@@ -1015,12 +1027,15 @@ static OSStatus TSMEventHandler(EventHandlerCallRef inHandlerRef, EventRef inEve
                 return NO;
         break;
             
+// FIXME: AGL isn't 64-bit (yet).  See <rdar://problem/4624858>.        
+#if !__LP64__
         case NPDrawingModelOpenGL:
             if (nPort.aglPort.window != lastSetPort.aglPort.window)
                 return NO;
             if (nPort.aglPort.context != lastSetPort.aglPort.context)
                 return NO;
         break;
+#endif /* __LP64__ */
         
         default:
             ASSERT_NOT_REACHED();
@@ -1086,10 +1101,13 @@ static OSStatus TSMEventHandler(EventHandlerCallRef inHandlerRef, EventRef inEve
                 npErr, nPort.cgPort.window, nPort.cgPort.context, (int)window.x, (int)window.y, (int)window.width, (int)window.height);
             break;
 
+// FIXME: AGL isn't 64-bit (yet).  See <rdar://problem/4624858>.        
+#if !__LP64__
             case NPDrawingModelOpenGL:
                 LOG(Plugins, "NPP_SetWindow (CoreGraphics): %d, window=%p, context=%p, window.x:%d window.y:%d window.width:%d window.height:%d",
                 npErr, nPort.aglPort.window, nPort.aglPort.context, (int)window.x, (int)window.y, (int)window.width, (int)window.height);
             break;
+#endif /* __LP64__ */
             
             default:
                 ASSERT_NOT_REACHED();
@@ -1288,8 +1306,11 @@ static OSStatus TSMEventHandler(EventHandlerCallRef inHandlerRef, EventRef inEve
     // may never get called so we can't completely rely on it.
     [self removeKeyEventHandler];
     
+// FIXME: AGL isn't 64-bit (yet).  See <rdar://problem/4624858>.        
+#if !__LP64__
     if (drawingModel == NPDrawingModelOpenGL)
         [self _destroyAGLContext];
+#endif /* __LP64__ */
 }
 
 - (BOOL)isStarted
@@ -1447,8 +1468,11 @@ static OSStatus TSMEventHandler(EventHandlerCallRef inHandlerRef, EventRef inEve
     [pendingFrameLoads release];
     [element release];
     
+// FIXME: AGL isn't 64-bit (yet).  See <rdar://problem/4624858>.        
+#if !__LP64__
     ASSERT(!aglWindow);
     ASSERT(!aglContext);
+#endif /* __LP64__ */
 
     [self freeAttributeKeysAndValues];
 
@@ -1487,6 +1511,8 @@ static OSStatus TSMEventHandler(EventHandlerCallRef inHandlerRef, EventRef inEve
         }
     }
     
+// FIXME: AGL isn't 64-bit (yet).  See <rdar://problem/4624858>.        
+#if !__LP64__
     // If this is a windowless OpenGL plugin, blit its contents back into this view.  The plug-in just drew into the offscreen context.
     if (drawingModel == NPDrawingModelOpenGL && window.type == NPWindowTypeDrawable) {
         NSImage *aglOffscreenImage = [self _aglOffscreenImageForDrawingInRect:rect];
@@ -1505,6 +1531,7 @@ static OSStatus TSMEventHandler(EventHandlerCallRef inHandlerRef, EventRef inEve
             CGContextRestoreGState(cgContext);
         }
     }
+#endif /* __LP64__ */
 }
 
 - (BOOL)isFlipped
@@ -1556,9 +1583,12 @@ static OSStatus TSMEventHandler(EventHandlerCallRef inHandlerRef, EventRef inEve
     [self setHasFocus:NO];
 
     if (!newWindow) {
+// FIXME: AGL isn't 64-bit (yet).  See <rdar://problem/4624858>.        
+#if !__LP64__
         // Hide the AGL child window
         if (drawingModel == NPDrawingModelOpenGL)
             [self _hideAGLWindow];
+#endif /* __LP64__ */
         
         if ([[self webView] hostWindow]) {
             // View will be moved out of the actual window but it still has a host window.
@@ -2150,11 +2180,14 @@ static OSStatus TSMEventHandler(EventHandlerCallRef inHandlerRef, EventRef inEve
             return NPERR_NO_ERROR;
         }
 
+// FIXME: AGL isn't 64-bit (yet).  See <rdar://problem/4624858>.        
+#if !__LP64__
         case NPNVsupportsOpenGLBool:
         {
             *(NPBool *)value = TRUE;
             return NPERR_NO_ERROR;
         }
+#endif /* __LP64__ */
         
         default:
             break;
@@ -2204,7 +2237,10 @@ static OSStatus TSMEventHandler(EventHandlerCallRef inHandlerRef, EventRef inEve
                 case NPDrawingModelQuickDraw:
 #endif
                 case NPDrawingModelCoreGraphics:
+// FIXME: AGL isn't 64-bit (yet).  See <rdar://problem/4624858>.        
+#if !__LP64__
                 case NPDrawingModelOpenGL:
+#endif /* __LP64__ */
                     drawingModel = newDrawingModel;
                     return NPERR_NO_ERROR;
                 
@@ -2278,8 +2314,11 @@ static OSStatus TSMEventHandler(EventHandlerCallRef inHandlerRef, EventRef inEve
     if (![self window])
         return;
     
+// FIXME: AGL isn't 64-bit (yet).  See <rdar://problem/4624858>.        
+#if !__LP64__
     if (drawingModel == NPDrawingModelOpenGL)
         [self _reshapeAGLWindow];
+#endif /* __LP64__ */
 
 #ifndef NP_NO_QUICKDRAW
     if (drawingModel == NPDrawingModelQuickDraw)
@@ -2365,6 +2404,9 @@ static OSStatus TSMEventHandler(EventHandlerCallRef inHandlerRef, EventRef inEve
 #endif
 }
 
+// FIXME: AGL isn't 64-bit (yet).  See <rdar://problem/4624858>.        
+#if !__LP64__
+
 - (BOOL)_createAGLContextIfNeeded
 {
     ASSERT(drawingModel == NPDrawingModelOpenGL);
@@ -2647,7 +2689,7 @@ static OSStatus TSMEventHandler(EventHandlerCallRef inHandlerRef, EventRef inEve
             CGLContextObj cglContext = [self _cglContext];
             CGLError error = CGLSetOffScreen(cglContext, boundsSize.width, boundsSize.height, boundsSize.width * 4, offscreenBuffer);
             if (error) {
-                LOG_ERROR("Could not set offscreen buffer for AGL context: %d", aglGetError());
+                LOG_ERROR("Could not set offscreen buffer for AGL context: %d", error);
                 break;
             }
 
@@ -2744,6 +2786,8 @@ static OSStatus TSMEventHandler(EventHandlerCallRef inHandlerRef, EventRef inEve
     return aglImage;
 }
 
+#endif /* __LP64__ */
+
 @end
 
 @implementation NSData (PluginExtras)