Auto merge of #5957 - ecoal95:glcontext-mac-support, r=jdm

Now we have mac support, and since android build is broken, we can
drop glutin from WebGL code.

Went back to upstream repo, see:
https://github.com/servo/rust-offscreen-rendering-context/pull/1#issuecomment-99234534

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/5957)
<!-- Reviewable:end -->

components/canvas/Cargo.toml

@@ -22,13 +22,8 @@ path = "../util"
 [dependencies.gfx]
 path = "../gfx"
 
-[dependencies.glutin]
-git = "https://github.com/servo/glutin"
-branch = "servo"
-features = ["headless"]
-
 [dependencies.offscreen_gl_context]
-git = "https://github.com/servo/rust-offscreen-rendering-context"
+git = "https://github.com/ecoal95/rust-offscreen-rendering-context"
 
 [dependencies]
 cssparser = "0.3.1"

components/canvas/lib.rs

@@ -14,7 +14,6 @@ extern crate util;
 extern crate gleam;
 extern crate num;
 extern crate offscreen_gl_context;
-extern crate glutin;
 
 #[macro_use]
 extern crate log;

components/canvas/webgl_paint_task.rs

@@ -16,40 +16,10 @@ use std::sync::mpsc::{channel, Sender};
 use util::vec::byte_swap;
 use offscreen_gl_context::{GLContext, GLContextAttributes};
 
-use glutin::{HeadlessRendererBuilder, HeadlessContext};
-
-// FIXME(ecoal95): We use glutin as a fallback until GLContext support improves.
-enum PlatformIndependentContext {
-    GLContext(GLContext),
-    GlutinContext(HeadlessContext),
-}
-
-impl PlatformIndependentContext {
-    fn make_current(&self) {
-        match *self {
-            PlatformIndependentContext::GLContext(ref ctx) => ctx.make_current().unwrap(),
-            PlatformIndependentContext::GlutinContext(ref ctx) => unsafe { ctx.make_current() }
-        }
-    }
-}
-
-fn create_offscreen_context(size: Size2D<i32>, attrs: GLContextAttributes) -> Result<PlatformIndependentContext, &'static str> {
-    match GLContext::create_offscreen(size, attrs) {
-        Ok(ctx) => Ok(PlatformIndependentContext::GLContext(ctx)),
-        Err(msg) => {
-            debug!("GLContext creation error: {}", msg);
-            match HeadlessRendererBuilder::new(size.width as u32, size.height as u32).build() {
-                Ok(ctx) => Ok(PlatformIndependentContext::GlutinContext(ctx)),
-                Err(_) => Err("Glutin headless context creation failed")
-            }
-        }
-    }
-}
-
 pub struct WebGLPaintTask {
     size: Size2D<i32>,
     original_context_size: Size2D<i32>,
-    gl_context: PlatformIndependentContext,
+    gl_context: GLContext,
 }
 
 // This allows trying to create the PaintTask
@@ -58,7 +28,8 @@ unsafe impl Send for WebGLPaintTask {}
 
 impl WebGLPaintTask {
     fn new(size: Size2D<i32>) -> Result<WebGLPaintTask, &'static str> {
-        let context = try!(create_offscreen_context(size, GLContextAttributes::default()));
+        // TODO(ecoal95): Get the GLContextAttributes from the `GetContext` call
+        let context = try!(GLContext::create_offscreen(size, GLContextAttributes::default()));
         Ok(WebGLPaintTask {
             size: size,
             original_context_size: size,
@@ -104,7 +75,8 @@ impl WebGLPaintTask {
                         match message {
                             CanvasCommonMsg::Close => break,
                             CanvasCommonMsg::SendPixelContents(chan) => painter.send_pixel_contents(chan),
-                            CanvasCommonMsg::Recreate(size) => painter.recreate(size),
+                            // TODO(ecoal95): handle error nicely
+                            CanvasCommonMsg::Recreate(size) => painter.recreate(size).unwrap(),
                         }
                     },
                     CanvasMsg::Canvas2d(_) => panic!("Wrong message sent to WebGLTask"),
@@ -232,20 +204,19 @@ impl WebGLPaintTask {
         gl::viewport(x, y, width, height);
     }
 
-    fn recreate(&mut self, size: Size2D<i32>) {
-        // TODO(ecoal95): GLContext should support a resize() method
+    fn recreate(&mut self, size: Size2D<i32>) -> Result<(), &'static str> {
         if size.width > self.original_context_size.width ||
            size.height > self.original_context_size.height {
-            panic!("Can't grow a GLContext (yet)");
-        } else {
-            // Right now we just crop the viewport, it will do the job
+            try!(self.gl_context.resize(size));
+            self.size = size;
+        } else {
             self.size = size;
-            gl::viewport(0, 0, size.width, size.height);
             unsafe { gl::Scissor(0, 0, size.width, size.height); }
         }
+        Ok(())
     }
 
     fn init(&mut self) {
-        self.gl_context.make_current();
+        self.gl_context.make_current().unwrap();
     }
 }

components/servo/Cargo.lock

@@ -73,16 +73,15 @@ dependencies = [
  "geom 0.1.0 (git+https://github.com/servo/rust-geom)",
  "gfx 0.0.1",
  "gleam 0.0.1 (git+https://github.com/servo/gleam)",
- "glutin 0.0.26 (git+https://github.com/servo/glutin?branch=servo)",
  "num 0.1.24 (registry+https://github.com/rust-lang/crates.io-index)",
- "offscreen_gl_context 0.0.1 (git+https://github.com/servo/rust-offscreen-rendering-context)",
+ "offscreen_gl_context 0.0.1 (git+https://github.com/ecoal95/rust-offscreen-rendering-context)",
  "util 0.0.1",
 ]
 
 [[package]]
 name = "cgl"
 version = "0.0.1"
-source = "git+https://github.com/servo/rust-cgl#16144321dc18d8ab538ee2acc0d2dad155a17899"
+source = "git+https://github.com/servo/rust-cgl#851ca1b90081d221c3c38d33548d3e22a19db79f"
 dependencies = [
  "gleam 0.0.1 (git+https://github.com/servo/gleam)",
  "libc 0.1.6 (registry+https://github.com/rust-lang/crates.io-index)",
@@ -833,13 +832,15 @@ dependencies = [
 [[package]]
 name = "offscreen_gl_context"
 version = "0.0.1"
-source = "git+https://github.com/servo/rust-offscreen-rendering-context#9ef802439467c287178afe46c5dd7adec13993eb"
+source = "git+https://github.com/ecoal95/rust-offscreen-rendering-context#c2b3dfd7fe344384e4206672b99c296141f5b4d6"
 dependencies = [
  "cgl 0.0.1 (git+https://github.com/servo/rust-cgl)",
+ "core_foundation 0.1.0 (git+https://github.com/servo/rust-core-foundation)",
  "geom 0.1.0 (git+https://github.com/servo/rust-geom)",
  "gleam 0.0.1 (git+https://github.com/servo/gleam)",
  "glx 0.0.1 (git+https://github.com/servo/rust-glx)",
  "libc 0.1.6 (registry+https://github.com/rust-lang/crates.io-index)",
+ "log 0.3.1 (registry+https://github.com/rust-lang/crates.io-index)",
  "xlib 0.1.0 (git+https://github.com/servo/rust-xlib)",
 ]