From 8df0cfb9d95821e53f6801776c71ff72901d2187 Mon Sep 17 00:00:00 2001
From: Tom Payne <twpayne@gmail.com>
Date: Mon, 8 Oct 2012 12:55:20 +0200
Subject: [PATCH 01/26] Abstract out key conditions for interactions, fixes
 #51, see also #54

---
 src/ol/interaction/condition.js               | 56 +++++++++++++++++++
 src/ol/interaction/dragpan.js                 | 14 ++++-
 .../{altdragrotate.js => dragrotate.js}       | 20 +++++--
 ...grotateandzoom.js => dragrotateandzoom.js} | 20 +++++--
 .../{shiftdragzoom.js => dragzoom.js}         | 20 +++++--
 src/ol/map.js                                 | 14 +++--
 6 files changed, 119 insertions(+), 25 deletions(-)
 create mode 100644 src/ol/interaction/condition.js
 rename src/ol/interaction/{altdragrotate.js => dragrotate.js} (63%)
 rename src/ol/interaction/{shiftdragrotateandzoom.js => dragrotateandzoom.js} (72%)
 rename src/ol/interaction/{shiftdragzoom.js => dragzoom.js} (70%)

diff --git a/src/ol/interaction/condition.js b/src/ol/interaction/condition.js
new file mode 100644
index 0000000000..81a34c4f63
--- /dev/null
+++ b/src/ol/interaction/condition.js
@@ -0,0 +1,56 @@
+goog.provide('ol.interaction.ConditionType');
+goog.provide('ol.interaction.condition');
+
+
+/**
+ * @typedef {function(goog.events.BrowserEvent): boolean}
+ */
+ol.interaction.ConditionType;
+
+
+/**
+ * @param {goog.events.BrowserEvent} browserEvent Browser event.
+ * @return {boolean} True if only the alt key is pressed.
+ */
+ol.interaction.condition.altKeyOnly = function(browserEvent) {
+  return (
+      browserEvent.altKey &&
+      !browserEvent.platformModifierKey &&
+      !browserEvent.shiftKey);
+};
+
+
+/**
+ * @param {goog.events.BrowserEvent} browserEvent Browser event.
+ * @return {boolean} True if only the no modifier keys are pressed.
+ */
+ol.interaction.condition.noModifierKeys = function(browserEvent) {
+  return (
+      !browserEvent.altKey &&
+      !browserEvent.platformModifierKey &&
+      !browserEvent.shiftKey);
+};
+
+
+/**
+ * @param {goog.events.BrowserEvent} browserEvent Browser event.
+ * @return {boolean} True if only the platform modifier key is pressed.
+ */
+ol.interaction.condition.platformModifierKeyOnly = function(browserEvent) {
+  return (
+      !browserEvent.altKey &&
+      browserEvent.platformModifierKey &&
+      !browserEvent.shiftKey);
+};
+
+
+/**
+ * @param {goog.events.BrowserEvent} browserEvent Browser event.
+ * @return {boolean} True if only the shift key is pressed.
+ */
+ol.interaction.condition.shiftKeyOnly = function(browserEvent) {
+  return (
+      !browserEvent.altKey &&
+      !browserEvent.platformModifierKey &&
+      browserEvent.shiftKey);
+};
diff --git a/src/ol/interaction/dragpan.js b/src/ol/interaction/dragpan.js
index 07e857be40..8a49b6846a 100644
--- a/src/ol/interaction/dragpan.js
+++ b/src/ol/interaction/dragpan.js
@@ -2,6 +2,7 @@ goog.provide('ol.interaction.DragPan');
 
 goog.require('ol.Coordinate');
 goog.require('ol.MapBrowserEvent');
+goog.require('ol.interaction.ConditionType');
 goog.require('ol.interaction.Drag');
 
 
@@ -9,9 +10,18 @@ goog.require('ol.interaction.Drag');
 /**
  * @constructor
  * @extends {ol.interaction.Drag}
+ * @param {ol.interaction.ConditionType} condition Condition.
  */
-ol.interaction.DragPan = function() {
+ol.interaction.DragPan = function(condition) {
+
   goog.base(this);
+
+  /**
+   * @private
+   * @type {ol.interaction.ConditionType}
+   */
+  this.condition_ = condition;
+
 };
 goog.inherits(ol.interaction.DragPan, ol.interaction.Drag);
 
@@ -39,7 +49,7 @@ ol.interaction.DragPan.prototype.handleDrag = function(mapBrowserEvent) {
  */
 ol.interaction.DragPan.prototype.handleDragStart = function(mapBrowserEvent) {
   var browserEvent = mapBrowserEvent.browserEvent;
-  if (!browserEvent.shiftKey) {
+  if (this.condition_(browserEvent)) {
     return true;
   } else {
     return false;
diff --git a/src/ol/interaction/altdragrotate.js b/src/ol/interaction/dragrotate.js
similarity index 63%
rename from src/ol/interaction/altdragrotate.js
rename to src/ol/interaction/dragrotate.js
index cb65554588..b40ab74690 100644
--- a/src/ol/interaction/altdragrotate.js
+++ b/src/ol/interaction/dragrotate.js
@@ -1,6 +1,7 @@
-goog.provide('ol.interaction.AltDragRotate');
+goog.provide('ol.interaction.DragRotate');
 
 goog.require('ol.MapBrowserEvent');
+goog.require('ol.interaction.ConditionType');
 goog.require('ol.interaction.Drag');
 
 
@@ -8,11 +9,18 @@ goog.require('ol.interaction.Drag');
 /**
  * @constructor
  * @extends {ol.interaction.Drag}
+ * @param {ol.interaction.ConditionType} condition Condition.
  */
-ol.interaction.AltDragRotate = function() {
+ol.interaction.DragRotate = function(condition) {
 
   goog.base(this);
 
+  /**
+   * @private
+   * @type {ol.interaction.ConditionType}
+   */
+  this.condition_ = condition;
+
   /**
    * @private
    * @type {number}
@@ -20,13 +28,13 @@ ol.interaction.AltDragRotate = function() {
   this.startRotation_ = 0;
 
 };
-goog.inherits(ol.interaction.AltDragRotate, ol.interaction.Drag);
+goog.inherits(ol.interaction.DragRotate, ol.interaction.Drag);
 
 
 /**
  * @inheritDoc
  */
-ol.interaction.AltDragRotate.prototype.handleDrag = function(mapBrowserEvent) {
+ol.interaction.DragRotate.prototype.handleDrag = function(mapBrowserEvent) {
   var browserEvent = mapBrowserEvent.browserEvent;
   var map = mapBrowserEvent.map;
   var size = map.getSize();
@@ -41,11 +49,11 @@ ol.interaction.AltDragRotate.prototype.handleDrag = function(mapBrowserEvent) {
 /**
  * @inheritDoc
  */
-ol.interaction.AltDragRotate.prototype.handleDragStart =
+ol.interaction.DragRotate.prototype.handleDragStart =
     function(mapBrowserEvent) {
   var browserEvent = mapBrowserEvent.browserEvent;
   var map = mapBrowserEvent.map;
-  if (browserEvent.isMouseActionButton() && browserEvent.altKey &&
+  if (browserEvent.isMouseActionButton() && this.condition_(browserEvent) &&
       map.canRotate()) {
     var size = map.getSize();
     var offset = mapBrowserEvent.getPixel();
diff --git a/src/ol/interaction/shiftdragrotateandzoom.js b/src/ol/interaction/dragrotateandzoom.js
similarity index 72%
rename from src/ol/interaction/shiftdragrotateandzoom.js
rename to src/ol/interaction/dragrotateandzoom.js
index 175a2a46b9..730b4def6a 100644
--- a/src/ol/interaction/shiftdragrotateandzoom.js
+++ b/src/ol/interaction/dragrotateandzoom.js
@@ -1,7 +1,8 @@
-goog.provide('ol.interaction.ShiftDragRotateAndZoom');
+goog.provide('ol.interaction.DragRotateAndZoom');
 
 goog.require('goog.math.Vec2');
 goog.require('ol.MapBrowserEvent');
+goog.require('ol.interaction.ConditionType');
 goog.require('ol.interaction.Drag');
 
 
@@ -9,11 +10,18 @@ goog.require('ol.interaction.Drag');
 /**
  * @constructor
  * @extends {ol.interaction.Drag}
+ * @param {ol.interaction.ConditionType} condition Condition.
  */
-ol.interaction.ShiftDragRotateAndZoom = function() {
+ol.interaction.DragRotateAndZoom = function(condition) {
 
   goog.base(this);
 
+  /**
+   * @private
+   * @type {ol.interaction.ConditionType}
+   */
+  this.condition_ = condition;
+
   /**
    * @private
    * @type {number}
@@ -27,13 +35,13 @@ ol.interaction.ShiftDragRotateAndZoom = function() {
   this.startRotation_ = 0;
 
 };
-goog.inherits(ol.interaction.ShiftDragRotateAndZoom, ol.interaction.Drag);
+goog.inherits(ol.interaction.DragRotateAndZoom, ol.interaction.Drag);
 
 
 /**
  * @inheritDoc
  */
-ol.interaction.ShiftDragRotateAndZoom.prototype.handleDrag =
+ol.interaction.DragRotateAndZoom.prototype.handleDrag =
     function(mapBrowserEvent) {
   var browserEvent = mapBrowserEvent.browserEvent;
   var map = mapBrowserEvent.map;
@@ -52,11 +60,11 @@ ol.interaction.ShiftDragRotateAndZoom.prototype.handleDrag =
 /**
  * @inheritDoc
  */
-ol.interaction.ShiftDragRotateAndZoom.prototype.handleDragStart =
+ol.interaction.DragRotateAndZoom.prototype.handleDragStart =
     function(mapBrowserEvent) {
   var browserEvent = mapBrowserEvent.browserEvent;
   var map = mapBrowserEvent.map;
-  if (map.canRotate() && browserEvent.shiftKey) {
+  if (map.canRotate() && this.condition_(browserEvent)) {
     var resolution = map.getResolution();
     var size = map.getSize();
     var delta = new goog.math.Vec2(
diff --git a/src/ol/interaction/shiftdragzoom.js b/src/ol/interaction/dragzoom.js
similarity index 70%
rename from src/ol/interaction/shiftdragzoom.js
rename to src/ol/interaction/dragzoom.js
index 7086ea0dc3..367b80000f 100644
--- a/src/ol/interaction/shiftdragzoom.js
+++ b/src/ol/interaction/dragzoom.js
@@ -1,10 +1,11 @@
 // FIXME draw drag box
 
-goog.provide('ol.interaction.ShiftDragZoom');
+goog.provide('ol.interaction.DragZoom');
 
 goog.require('ol.Extent');
 goog.require('ol.MapBrowserEvent');
 goog.require('ol.control.DragBox');
+goog.require('ol.interaction.ConditionType');
 goog.require('ol.interaction.Drag');
 
 
@@ -26,11 +27,18 @@ ol.SHIFT_DRAG_ZOOM_HYSTERESIS_PIXELS_SQUARED =
 /**
  * @constructor
  * @extends {ol.interaction.Drag}
+ * @param {ol.interaction.ConditionType} condition Condition.
  */
-ol.interaction.ShiftDragZoom = function() {
+ol.interaction.DragZoom = function(condition) {
 
   goog.base(this);
 
+  /**
+   * @private
+   * @type {ol.interaction.ConditionType}
+   */
+  this.condition_ = condition;
+
   /**
    * @type {ol.control.DragBox}
    * @private
@@ -39,13 +47,13 @@ ol.interaction.ShiftDragZoom = function() {
 
 
 };
-goog.inherits(ol.interaction.ShiftDragZoom, ol.interaction.Drag);
+goog.inherits(ol.interaction.DragZoom, ol.interaction.Drag);
 
 
 /**
  * @inheritDoc
  */
-ol.interaction.ShiftDragZoom.prototype.handleDragEnd =
+ol.interaction.DragZoom.prototype.handleDragEnd =
     function(mapBrowserEvent) {
   this.dragBox_.setMap(null);
   this.dragBox_ = null;
@@ -63,10 +71,10 @@ ol.interaction.ShiftDragZoom.prototype.handleDragEnd =
 /**
  * @inheritDoc
  */
-ol.interaction.ShiftDragZoom.prototype.handleDragStart =
+ol.interaction.DragZoom.prototype.handleDragStart =
     function(mapBrowserEvent) {
   var browserEvent = mapBrowserEvent.browserEvent;
-  if (browserEvent.isMouseActionButton() && browserEvent.shiftKey) {
+  if (browserEvent.isMouseActionButton() && this.condition_(browserEvent)) {
     this.dragBox_ = new ol.control.DragBox({
       map: mapBrowserEvent.map,
       startCoordinate: this.startCoordinate
diff --git a/src/ol/map.js b/src/ol/map.js
index fe147e1541..5ce5c60163 100644
--- a/src/ol/map.js
+++ b/src/ol/map.js
@@ -41,14 +41,15 @@ goog.require('ol.Size');
 goog.require('ol.TransformFunction');
 goog.require('ol.control.Attribution');
 goog.require('ol.control.Zoom');
-goog.require('ol.interaction.AltDragRotate');
 goog.require('ol.interaction.DblClickZoom');
 goog.require('ol.interaction.DragPan');
+goog.require('ol.interaction.DragRotate');
+goog.require('ol.interaction.DragZoom');
 goog.require('ol.interaction.Interaction');
 goog.require('ol.interaction.KeyboardPan');
 goog.require('ol.interaction.KeyboardZoom');
 goog.require('ol.interaction.MouseWheelZoom');
-goog.require('ol.interaction.ShiftDragZoom');
+goog.require('ol.interaction.condition');
 goog.require('ol.renderer.Layer');
 goog.require('ol.renderer.Map');
 goog.require('ol.renderer.dom');
@@ -1129,7 +1130,8 @@ ol.Map.createInteractions_ = function(mapOptions) {
   var rotate = goog.isDef(mapOptions.rotate) ?
       mapOptions.rotate : true;
   if (rotate) {
-    interactions.push(new ol.interaction.AltDragRotate());
+    interactions.push(
+        new ol.interaction.DragRotate(ol.interaction.condition.altKeyOnly));
   }
 
   var doubleClickZoom = goog.isDef(mapOptions.doubleClickZoom) ?
@@ -1143,7 +1145,8 @@ ol.Map.createInteractions_ = function(mapOptions) {
   var dragPan = goog.isDef(mapOptions.dragPan) ?
       mapOptions.dragPan : true;
   if (dragPan) {
-    interactions.push(new ol.interaction.DragPan());
+    interactions.push(
+        new ol.interaction.DragPan(ol.interaction.condition.noModifierKeys));
   }
 
   var keyboard = goog.isDef(mapOptions.keyboard) ?
@@ -1167,7 +1170,8 @@ ol.Map.createInteractions_ = function(mapOptions) {
   var shiftDragZoom = goog.isDef(mapOptions.shiftDragZoom) ?
       mapOptions.shiftDragZoom : true;
   if (shiftDragZoom) {
-    interactions.push(new ol.interaction.ShiftDragZoom());
+    interactions.push(
+        new ol.interaction.DragZoom(ol.interaction.condition.shiftKeyOnly));
   }
 
   return interactions;

From dc491f9114f9da1b4888844e7a2a633f20d378ec Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=C3=89ric=20Lemoine?= <eric.lemoine@gmail.com>
Date: Wed, 10 Oct 2012 20:24:18 +0200
Subject: [PATCH 02/26] disposeInternal implementation for ol.Map

---
 src/ol/map.js | 10 ++++++++++
 1 file changed, 10 insertions(+)

diff --git a/src/ol/map.js b/src/ol/map.js
index fe147e1541..42dd95b574 100644
--- a/src/ol/map.js
+++ b/src/ol/map.js
@@ -290,6 +290,16 @@ ol.Map.prototype.canRotate = function() {
 };
 
 
+/**
+ *
+ * @inheritDoc
+ */
+ol.Map.prototype.disposeInternal = function() {
+  goog.base(this, 'disposeInternal');
+  goog.dom.removeNode(this.viewport_);
+};
+
+
 /**
  * @param {ol.Extent} extent Extent.
  */

From 655e04368bb9cb2309359b02fbd50a40a1ca7167 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=C3=89ric=20Lemoine?= <eric.lemoine@gmail.com>
Date: Wed, 10 Oct 2012 20:24:32 +0200
Subject: [PATCH 03/26] disposeInternal implementation for ol.control.Control

---
 src/ol/control/control.js | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/src/ol/control/control.js b/src/ol/control/control.js
index a640b2beca..5dda618613 100644
--- a/src/ol/control/control.js
+++ b/src/ol/control/control.js
@@ -49,6 +49,14 @@ ol.control.Control = function(controlOptions) {
 goog.inherits(ol.control.Control, goog.Disposable);
 
 
+/**
+ * @inheritDoc
+ */
+ol.control.Control.prototype.disposeInternal = function() {
+  goog.dom.removeNode(this.element);
+};
+
+
 /**
  * @return {ol.Map} Map.
  */

From fbc810264bcec5d6ca2b42b5981c26b6ba0265c6 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=C3=89ric=20Lemoine?= <eric.lemoine@gmail.com>
Date: Wed, 10 Oct 2012 20:25:53 +0200
Subject: [PATCH 04/26] goog.array.forEach does not accept null values

---
 src/ol/mapbrowserevent.js | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/src/ol/mapbrowserevent.js b/src/ol/mapbrowserevent.js
index b5b8d49481..a2d241eda9 100644
--- a/src/ol/mapbrowserevent.js
+++ b/src/ol/mapbrowserevent.js
@@ -285,9 +285,10 @@ ol.MapBrowserEventHandler.prototype.disposeInternal = function() {
       this.handleUp_, false, this);
   goog.events.unlisten(element,
       goog.events.EventType.CLICK, this.click_, false, this);
-  goog.asserts.assert(goog.isDef(this.dragListenerKeys_));
-  goog.array.forEach(this.dragListenerKeys_, goog.events.unlistenByKey);
-  this.dragListenerKeys_ = null;
+  if (!goog.isNull(this.dragListenerKeys_)) {
+    goog.array.forEach(this.dragListenerKeys_, goog.events.unlistenByKey);
+    this.dragListenerKeys_ = null;
+  }
 };
 
 

From 0b411ad69c5b630a68fd70199cd8022c332ec5b9 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=C3=89ric=20Lemoine?= <eric.lemoine@gmail.com>
Date: Thu, 11 Oct 2012 17:46:22 +0200
Subject: [PATCH 05/26] Have disposeInternal call parent

---
 src/ol/control/control.js | 1 +
 src/ol/map.js             | 2 +-
 src/ol/mapbrowserevent.js | 1 +
 3 files changed, 3 insertions(+), 1 deletion(-)

diff --git a/src/ol/control/control.js b/src/ol/control/control.js
index 5dda618613..ad7ab15f7e 100644
--- a/src/ol/control/control.js
+++ b/src/ol/control/control.js
@@ -54,6 +54,7 @@ goog.inherits(ol.control.Control, goog.Disposable);
  */
 ol.control.Control.prototype.disposeInternal = function() {
   goog.dom.removeNode(this.element);
+  goog.base(this, 'disposeInternal');
 };
 
 
diff --git a/src/ol/map.js b/src/ol/map.js
index 42dd95b574..9ed3f64a5f 100644
--- a/src/ol/map.js
+++ b/src/ol/map.js
@@ -295,8 +295,8 @@ ol.Map.prototype.canRotate = function() {
  * @inheritDoc
  */
 ol.Map.prototype.disposeInternal = function() {
-  goog.base(this, 'disposeInternal');
   goog.dom.removeNode(this.viewport_);
+  goog.base(this, 'disposeInternal');
 };
 
 
diff --git a/src/ol/mapbrowserevent.js b/src/ol/mapbrowserevent.js
index a2d241eda9..63b3c956cd 100644
--- a/src/ol/mapbrowserevent.js
+++ b/src/ol/mapbrowserevent.js
@@ -289,6 +289,7 @@ ol.MapBrowserEventHandler.prototype.disposeInternal = function() {
     goog.array.forEach(this.dragListenerKeys_, goog.events.unlistenByKey);
     this.dragListenerKeys_ = null;
   }
+  goog.base(this, 'disposeInternal');
 };
 
 

From 2a76a23b6f244af016038ad4c0abe4733fd4655f Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=C3=89ric=20Lemoine?= <eric.lemoine@gmail.com>
Date: Thu, 11 Oct 2012 17:46:47 +0200
Subject: [PATCH 06/26] Add tests for ol.control.Control

---
 test/ol.html                         |  2 ++
 test/spec/ol/control/control.test.js | 27 +++++++++++++++++++++++++++
 2 files changed, 29 insertions(+)
 create mode 100644 test/spec/ol/control/control.test.js

diff --git a/test/ol.html b/test/ol.html
index d504a2059c..1d438b53cd 100644
--- a/test/ol.html
+++ b/test/ol.html
@@ -83,6 +83,7 @@
   <script type="text/javascript" src="spec/ol/tilegrid.test.js"></script>
   <script type="text/javascript" src="spec/ol/tilerange.test.js"></script>
   <script type="text/javascript" src="spec/ol/tileurlfunction.test.js"></script>
+  <script type="text/javascript" src="spec/ol/control/control.test.js"></script>
 
   <script type="text/javascript">
 
@@ -124,5 +125,6 @@
 </head>
 
 <body>
+  <div id="map"></div>
 </body>
 </html>
diff --git a/test/spec/ol/control/control.test.js b/test/spec/ol/control/control.test.js
new file mode 100644
index 0000000000..3e515c9003
--- /dev/null
+++ b/test/spec/ol/control/control.test.js
@@ -0,0 +1,27 @@
+goog.require('ol.Map');
+goog.require('ol.control.Control');
+goog.require('goog.dom');
+
+describe('ol.control.Control', function() {
+  var map, control;
+
+  beforeEach(function() {
+    map = new ol.Map({
+      target: document.getElementById('map')
+    });
+    var element = goog.dom.createDom(goog.dom.TagName.DIV);
+    control = new ol.control.Control({element:element});
+    control.setMap(map);
+  });
+
+  afterEach(function() {
+    map.dispose();
+  });
+
+  describe('dispose', function() {
+    it('removes the control element from its parent', function() {
+      control.dispose();
+      expect(goog.dom.getParentElement(control.element)).toBeNull();
+    });
+  });
+});

From 9c48dff2869298cba6963d33e8b626500f5ff52d Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=C3=89ric=20Lemoine?= <eric.lemoine@gmail.com>
Date: Thu, 11 Oct 2012 17:53:35 +0200
Subject: [PATCH 07/26] Add tests for ol.Map.prototype.disposeInternal

---
 test/spec/ol/map.test.js | 16 ++++++++++++++++
 1 file changed, 16 insertions(+)

diff --git a/test/spec/ol/map.test.js b/test/spec/ol/map.test.js
index 80fe996677..889c91edad 100644
--- a/test/spec/ol/map.test.js
+++ b/test/spec/ol/map.test.js
@@ -1,7 +1,23 @@
 goog.require('ol.Map');
+goog.require('goog.dom');
 
 describe('ol.Map', function() {
 
+  describe('dispose', function() {
+    var map;
+
+    beforeEach(function() {
+      map = new ol.Map({
+        target: document.getElementById('map')
+      });
+    });
+
+    it('removes the viewport from its parent', function() {
+      map.dispose();
+      expect(goog.dom.getParentElement(map.getViewport())).toBeNull();
+    });
+  });
+
   describe('create constraints', function() {
 
     describe('create resolution constraint', function() {

From f49f3708579119f47a6370ddad8e2f4cc09186cd Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=C3=89ric=20Lemoine?= <eric.lemoine@gmail.com>
Date: Thu, 11 Oct 2012 18:09:18 +0200
Subject: [PATCH 08/26] Make linter happy

---
 test/spec/ol/control/control.test.js | 4 ++--
 test/spec/ol/map.test.js             | 2 +-
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/test/spec/ol/control/control.test.js b/test/spec/ol/control/control.test.js
index 3e515c9003..f1245ae66a 100644
--- a/test/spec/ol/control/control.test.js
+++ b/test/spec/ol/control/control.test.js
@@ -1,6 +1,6 @@
+goog.require('goog.dom');
 goog.require('ol.Map');
 goog.require('ol.control.Control');
-goog.require('goog.dom');
 
 describe('ol.control.Control', function() {
   var map, control;
@@ -10,7 +10,7 @@ describe('ol.control.Control', function() {
       target: document.getElementById('map')
     });
     var element = goog.dom.createDom(goog.dom.TagName.DIV);
-    control = new ol.control.Control({element:element});
+    control = new ol.control.Control({element: element});
     control.setMap(map);
   });
 
diff --git a/test/spec/ol/map.test.js b/test/spec/ol/map.test.js
index 889c91edad..a936388a7d 100644
--- a/test/spec/ol/map.test.js
+++ b/test/spec/ol/map.test.js
@@ -1,5 +1,5 @@
-goog.require('ol.Map');
 goog.require('goog.dom');
+goog.require('ol.Map');
 
 describe('ol.Map', function() {
 

From 98340c55ce506445c31f30b6ea77110d24b238a5 Mon Sep 17 00:00:00 2001
From: Tom Payne <twpayne@gmail.com>
Date: Mon, 15 Oct 2012 19:11:02 +0200
Subject: [PATCH 09/26] Don't use goog.* functions in examples

---
 examples/side-by-side.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/examples/side-by-side.js b/examples/side-by-side.js
index 1a8f82fbae..61e9fdb3a5 100644
--- a/examples/side-by-side.js
+++ b/examples/side-by-side.js
@@ -39,7 +39,7 @@ var webglMap = new ol.Map({
   renderer: ol.RendererHint.WEBGL,
   target: 'webglMap'
 });
-if (!goog.isNull(webglMap)) {
+if (webglMap !== null) {
   webglMap.bindTo('center', domMap);
   webglMap.bindTo('layers', domMap);
   webglMap.bindTo('resolution', domMap);

From b45c2ca4bc6d8ace796896f684423d107fa64476 Mon Sep 17 00:00:00 2001
From: Tom Payne <twpayne@gmail.com>
Date: Mon, 15 Oct 2012 19:11:51 +0200
Subject: [PATCH 10/26] Add optional argument to @exportSymbol

---
 bin/generate-exports | 13 ++++++++-----
 1 file changed, 8 insertions(+), 5 deletions(-)

diff --git a/bin/generate-exports b/bin/generate-exports
index 4239c8eed7..49360b47d7 100755
--- a/bin/generate-exports
+++ b/bin/generate-exports
@@ -114,9 +114,10 @@ class ObjectLiteral(Exportable):
 
 class Symbol(Exportable):
 
-    def __init__(self, name, export_symbol):
+    def __init__(self, name, export_symbol, export_as=None):
         Exportable.__init__(self, name)
         self.export_symbol = export_symbol
+        self.export_as = export_as or self.name
         self.props = set()
 
     __repr__ = simplerepr
@@ -124,7 +125,7 @@ class Symbol(Exportable):
     def export(self):
         lines = []
         if self.export_symbol:
-            lines.append('\n\ngoog.exportSymbol(\n    \'%s\',\n    %s);\n' % (self.name, self.name))
+            lines.append('\n\ngoog.exportSymbol(\n    \'%s\',\n    %s);\n' % (self.name, self.export_as))
         lines.extend('goog.exportProperty(\n    %s,\n    \'%s\',\n    %s.%s);\n' % (self.name, prop, self.name, prop) for prop in sorted(self.props))
         return ''.join(lines)
 
@@ -193,14 +194,16 @@ def main(argv):
                     objects[name] = symbol
                 symbol.props.add(prop)
                 continue
-            m = re.match(r'@exportSymbol\s+(?P<name>\S+)\Z', line)
+            m = re.match(r'@exportSymbol\s+(?P<name>\S+)(?:\s+(?P<export_as>\S+))?\Z', line)
             if m:
                 name = m.group('name')
                 if name in objects:
                     raise RuntimeError(line)  # Name already defined
-                symbol = Symbol(name, True)
+                export_as = m.group('export_as')
+                symbol = Symbol(name, True, export_as)
                 objects[name] = symbol
-                requires.add(name)
+                if not export_as:
+                    requires.add(name)
                 continue
             raise RuntimeError(line)
 

From f9302ab0301a74ccc2f0505534eabfbf768d289d Mon Sep 17 00:00:00 2001
From: Tom Payne <twpayne@gmail.com>
Date: Mon, 15 Oct 2012 19:12:06 +0200
Subject: [PATCH 11/26] Export goog.require as goog.nullFunction

---
 src/ol/exports.txt | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/src/ol/exports.txt b/src/ol/exports.txt
index cea5105990..6b6d09418c 100644
--- a/src/ol/exports.txt
+++ b/src/ol/exports.txt
@@ -1,3 +1,5 @@
+@exportSymbol goog.require goog.nullFunction
+
 @exportProperty ol.MapBrowserEvent.prototype.getCoordinate
 
 @exportSymbol ol.Collection

From f7adc1ae753b1e05f026d54691becb1727443b45 Mon Sep 17 00:00:00 2001
From: Tom Payne <twpayne@gmail.com>
Date: Mon, 15 Oct 2012 19:12:55 +0200
Subject: [PATCH 12/26] Use identical code in compiled and standalone examples

---
 .../standalone/full-screen-standalone.html    | 14 +--
 .../overlay-and-popup-standalone.html         | 36 +-------
 .../standalone/side-by-side-standalone.html   | 88 +------------------
 3 files changed, 3 insertions(+), 135 deletions(-)

diff --git a/examples/standalone/full-screen-standalone.html b/examples/standalone/full-screen-standalone.html
index b44f01358a..afb0f97f38 100644
--- a/examples/standalone/full-screen-standalone.html
+++ b/examples/standalone/full-screen-standalone.html
@@ -18,18 +18,6 @@
   </head>
   <body>
     <div id="map"></div>
-    <script type="text/javascript">
-
-        var layer = new ol.layer.TileLayer({
-          source: new ol.source.MapQuestOpenAerial()
-        });
-        var map = new ol.Map({
-          center: new ol.Coordinate(0, 0),
-          layers: new ol.Collection([layer]),
-          target: 'map',
-          zoom: 2
-        });
-
-    </script>
+    <script src="../full-screen.js" type="text/javascript"></script>
   </body>
 </html>
diff --git a/examples/standalone/overlay-and-popup-standalone.html b/examples/standalone/overlay-and-popup-standalone.html
index efd99db820..e1293b1c54 100644
--- a/examples/standalone/overlay-and-popup-standalone.html
+++ b/examples/standalone/overlay-and-popup-standalone.html
@@ -77,40 +77,6 @@
       <!-- Popup -->
       <div class="overlay arrow_box" id="popup"></div>
     </div>
-    <script type="text/javascript">
-
-      var layer = new ol.layer.TileLayer({
-        source: new ol.source.MapQuestOpenAerial()
-      });
-      var map = new ol.Map({
-        center: new ol.Coordinate(0, 0),
-        layers: new ol.Collection([layer]),
-        target: 'map',
-        zoom: 2
-      });
-
-      // Vienna label
-      var vienna = new ol.overlay.Overlay({
-        map: map,
-        coordinate: ol.Projection.transformWithCodes(
-            new ol.Coordinate(16.3725, 48.208889), 'EPSG:4326', 'EPSG:3857'),
-        element: document.getElementById('vienna')
-      });
-
-      // Popup showing the position the user clicked
-      var popup = new ol.overlay.Overlay({
-        element: document.getElementById('popup')
-      });
-      map.addEventListener('click', function(evt) {
-        var coordinate = evt.getCoordinate();
-        popup.getElement().innerHTML =
-            'Welcome to ol3. The location you clicked was<br>' +
-            ol.CoordinateFormat.hdms(ol.Projection.transformWithCodes(
-                coordinate, 'EPSG:3857', 'EPSG:4326'));
-        popup.setMap(evt.map);
-        popup.setCoordinate(evt.getCoordinate());
-      });
-
-    </script>
+    <script src="../overlay-and-popup.js" type="text/javascript"></script>
   </body>
 </html>
diff --git a/examples/standalone/side-by-side-standalone.html b/examples/standalone/side-by-side-standalone.html
index 425dfb6409..e28c0425a7 100644
--- a/examples/standalone/side-by-side-standalone.html
+++ b/examples/standalone/side-by-side-standalone.html
@@ -66,92 +66,6 @@
       </tr>
     </table>
     <p><b>Notes:</b> The two maps share the same center, resolution, rotation and layers.</p>
-    <script type="text/javascript">
-
-        var layer = new ol.layer.TileLayer({
-          source: new ol.source.MapQuestOpenAerial()
-        });
-
-        var domMap = new ol.Map({
-          center: new ol.Coordinate(0, 0),
-          layers: new ol.Collection([layer]),
-          renderer: ol.RendererHint.DOM,
-          target: 'domMap',
-          zoom: 1
-        });
-
-        domMap.getControls().push(new ol.control.MousePosition({
-          coordinateFormat: ol.CoordinateFormat.hdms,
-          projection: ol.Projection.getFromCode('EPSG:4326'),
-          target: document.getElementById('domMousePosition'),
-          undefinedHtml: '&nbsp;'
-        }));
-
-        var webglMap = new ol.Map({
-          renderer: ol.RendererHint.WEBGL,
-          target: 'webglMap'
-        });
-        if (webglMap) {
-          webglMap.bindTo('center', domMap);
-          webglMap.bindTo('layers', domMap);
-          webglMap.bindTo('resolution', domMap);
-          webglMap.bindTo('rotation', domMap);
-        }
-
-        webglMap.getControls().push(new ol.control.MousePosition({
-          coordinateFormat: ol.CoordinateFormat.hdms,
-          projection: ol.Projection.getFromCode('EPSG:4326'),
-          target: document.getElementById('webglMousePosition'),
-          undefinedHtml: '&nbsp;'
-        }));
-
-        var keyboardInteraction = new ol.interaction.Keyboard();
-        keyboardInteraction.addCallback('0', function() {
-          layer.setBrightness(0);
-          layer.setContrast(0);
-          layer.setHue(0);
-          layer.setSaturation(0);
-          layer.setOpacity(1);
-          layer.setVisible(true);
-        });
-        keyboardInteraction.addCallback('b', function() {
-          layer.setBrightness(layer.getBrightness() - 0.1);
-        });
-        keyboardInteraction.addCallback('B', function() {
-          layer.setBrightness(layer.getBrightness() + 0.1);
-        });
-        keyboardInteraction.addCallback('c', function() {
-          layer.setContrast(layer.getContrast() - 0.1);
-        });
-        keyboardInteraction.addCallback('C', function() {
-          layer.setContrast(layer.getContrast() + 0.1);
-        });
-        keyboardInteraction.addCallback('h', function() {
-          layer.setHue(layer.getHue() - 0.1);
-        });
-        keyboardInteraction.addCallback('H', function() {
-          layer.setHue(layer.getHue() + 0.1);
-        });
-        keyboardInteraction.addCallback('o', function() {
-          layer.setOpacity(layer.getOpacity() - 0.1);
-        });
-        keyboardInteraction.addCallback('O', function() {
-          layer.setOpacity(layer.getOpacity() + 0.1);
-        });
-        keyboardInteraction.addCallback('r', function() {
-          webglMap.setRotation(0);
-        });
-        keyboardInteraction.addCallback('s', function() {
-          layer.setSaturation(layer.getSaturation() - 0.1);
-        });
-        keyboardInteraction.addCallback('S', function() {
-          layer.setSaturation(layer.getSaturation() + 0.1);
-        });
-        keyboardInteraction.addCallback('vV', function() {
-          layer.setVisible(!layer.getVisible());
-        });
-        domMap.getInteractions().push(keyboardInteraction);
-
-    </script>
+    <script src="../side-by-side.js" type="text/javascript"></script>
   </body>
 </html>

From ed197f3cfe9b1962e8586e0f71b91a51615dfb25 Mon Sep 17 00:00:00 2001
From: Tom Payne <twpayne@gmail.com>
Date: Tue, 16 Oct 2012 10:59:53 +0200
Subject: [PATCH 13/26] Revert "Two lessons learned"

This reverts commit f8462460bf82f46484149d52822d62a6ab660cfc.
---
 src/ol/renderer/dom/tilelayer.js | 19 +++++++++----------
 1 file changed, 9 insertions(+), 10 deletions(-)

diff --git a/src/ol/renderer/dom/tilelayer.js b/src/ol/renderer/dom/tilelayer.js
index 67f2199460..24de2b5eac 100644
--- a/src/ol/renderer/dom/tilelayer.js
+++ b/src/ol/renderer/dom/tilelayer.js
@@ -169,10 +169,10 @@ ol.renderer.dom.TileLayer.prototype.render = function() {
   }
 
   /**
-   * @type {Object.<string, Object.<string, ol.Tile>>}
+   * @type {Object.<number, Object.<string, ol.Tile>>}
    */
   var tilesToDrawByZ = {};
-  tilesToDrawByZ[String(z)] = {};
+  tilesToDrawByZ[z] = {};
 
   var tileRange =
       tileGrid.getTileRangeForExtentAndResolution(mapExtent, mapResolution);
@@ -188,7 +188,7 @@ ol.renderer.dom.TileLayer.prototype.render = function() {
 
     var key = tile.tileCoord.toString();
     if (tile.getState() == ol.TileState.LOADED) {
-      tilesToDrawByZ[String(z)][key] = tile;
+      tilesToDrawByZ[z][key] = tile;
       return;
     } else {
       if (!(key in this.loadingTiles_)) {
@@ -198,7 +198,7 @@ ol.renderer.dom.TileLayer.prototype.render = function() {
         tile.load();
       }
       // TODO: only append after load?
-      tilesToDrawByZ[String(z)][key] = tile;
+      tilesToDrawByZ[z][key] = tile;
     }
 
     /**
@@ -210,12 +210,10 @@ ol.renderer.dom.TileLayer.prototype.render = function() {
     tileGrid.forEachTileCoordParentTileRange(
         tileCoord,
         function(altZ, altTileRange) {
-          altZ = String(altZ);
           var fullyCovered = true;
           altTileRange.forEachTileCoord(altZ, function(altTileCoord) {
             var tileKey = altTileCoord.toString();
-            if (tilesToDrawByZ[altZ] &&
-                tilesToDrawByZ[altZ][tileKey]) {
+            if (tilesToDrawByZ[altZ] && tilesToDrawByZ[altZ][tileKey]) {
               return;
             }
             var altTile = tileSource.getTile(altTileCoord);
@@ -234,16 +232,17 @@ ol.renderer.dom.TileLayer.prototype.render = function() {
 
   }, this);
 
+  /** @type {Array.<number>} */
   var zs = goog.object.getKeys(tilesToDrawByZ);
-  zs.sort(function(a, b) {return a - b});
+  goog.array.sort(zs);
 
   var fragment = document.createDocumentFragment();
   var altFragment = document.createDocumentFragment();
   var newTiles = false;
   var newAltTiles = false;
   for (var i = 0, ii = zs.length; i < ii; ++i) {
-    var tileZ = +zs[i];
-    var tilesToDraw = tilesToDrawByZ[String(tileZ)];
+    var tileZ = zs[i];
+    var tilesToDraw = tilesToDrawByZ[tileZ];
     var tileOffset = this.getTileOffset_(tileZ, mapResolution);
     for (var key in tilesToDraw) {
       var tile = tilesToDraw[key];

From 3e6d3748592ddcd2d09f170b3cb2940de7e54f9f Mon Sep 17 00:00:00 2001
From: Tom Payne <twpayne@gmail.com>
Date: Tue, 16 Oct 2012 11:04:56 +0200
Subject: [PATCH 14/26] Ensure z as object keys are sorted as numbers

---
 src/ol/renderer/dom/tilelayer.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/ol/renderer/dom/tilelayer.js b/src/ol/renderer/dom/tilelayer.js
index 24de2b5eac..477eae9f3d 100644
--- a/src/ol/renderer/dom/tilelayer.js
+++ b/src/ol/renderer/dom/tilelayer.js
@@ -233,7 +233,7 @@ ol.renderer.dom.TileLayer.prototype.render = function() {
   }, this);
 
   /** @type {Array.<number>} */
-  var zs = goog.object.getKeys(tilesToDrawByZ);
+  var zs = goog.array.map(goog.object.getKeys(tilesToDrawByZ), Number);
   goog.array.sort(zs);
 
   var fragment = document.createDocumentFragment();

From 5c6b5cb4955b4ec6580e68fecad01fce9a176aa3 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=C3=89ric=20Lemoine?= <eric.lemoine@gmail.com>
Date: Tue, 16 Oct 2012 21:45:52 +0200
Subject: [PATCH 15/26] Add BingMaps exports

---
 src/ol/exports.txt        | 10 ++++++++++
 src/ol/source/bingmaps.js | 10 +---------
 2 files changed, 11 insertions(+), 9 deletions(-)

diff --git a/src/ol/exports.txt b/src/ol/exports.txt
index 6b6d09418c..f59f548c69 100644
--- a/src/ol/exports.txt
+++ b/src/ol/exports.txt
@@ -124,6 +124,16 @@
 @exportProperty ol.overlay.Overlay.prototype.setMap
 
 @exportSymbol ol.source.BingMaps
+@exportSymbol ol.BingMapsStyle
+@exportProperty ol.BingMapsStyle.AERIAL
+@exportProperty ol.BingMapsStyle.AERIAL_WITH_LABELS
+@exportProperty ol.BingMapsStyle.ROAD
+@exportProperty ol.BingMapsStyle.ORDNANCE_SURVEY
+@exportProperty ol.BingMapsStyle.COLLINS_BART
+@exportObjectLiteral ol.source.BingMapsOptions
+@exportObjectLiteralProperty ol.source.BingMapsOptions.culture string|undefined
+@exportObjectLiteralProperty ol.source.BingMapsOptions.key string
+@exportObjectLiteralProperty ol.source.BingMapsOptions.style ol.BingMapsStyle
 
 @exportSymbol ol.source.MapQuestOSM
 
diff --git a/src/ol/source/bingmaps.js b/src/ol/source/bingmaps.js
index 339abd5811..d1b6bfd729 100644
--- a/src/ol/source/bingmaps.js
+++ b/src/ol/source/bingmaps.js
@@ -1,4 +1,5 @@
 goog.provide('ol.source.BingMaps');
+goog.provide('ol.BingMapsStyle');
 
 goog.require('goog.Uri');
 goog.require('goog.events');
@@ -21,15 +22,6 @@ ol.BingMapsStyle = {
 };
 
 
-/**
- * @typedef {{culture: (string|undefined),
- *            key: string,
- *            style: ol.BingMapsStyle}}
- */
-ol.source.BingMapsOptions;
-
-
-
 /**
  * @constructor
  * @extends {ol.source.TileSource}

From 1f7ecd2529da3a43b4d4f43f5d751e9604d466a2 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=C3=89ric=20Lemoine?= <eric.lemoine@gmail.com>
Date: Tue, 16 Oct 2012 21:50:20 +0200
Subject: [PATCH 16/26] Make linter happy

---
 src/ol/source/bingmaps.js | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/ol/source/bingmaps.js b/src/ol/source/bingmaps.js
index d1b6bfd729..1b805faf93 100644
--- a/src/ol/source/bingmaps.js
+++ b/src/ol/source/bingmaps.js
@@ -1,5 +1,5 @@
-goog.provide('ol.source.BingMaps');
 goog.provide('ol.BingMapsStyle');
+goog.provide('ol.source.BingMaps');
 
 goog.require('goog.Uri');
 goog.require('goog.events');
@@ -22,6 +22,7 @@ ol.BingMapsStyle = {
 };
 
 
+
 /**
  * @constructor
  * @extends {ol.source.TileSource}

From f4b20175cc9a9de7dfbebedb5476749f603060c9 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=C3=89ric=20Lemoine?= <eric.lemoine@gmail.com>
Date: Tue, 16 Oct 2012 21:51:48 +0200
Subject: [PATCH 17/26] Add a specific loader for hosted examples

This loader is responsible for adding ol.js, ol.css and the example-specific js script to the document. This is the loader used in examples when hosted.
---
 build/loader_hosted_examples.js | 46 +++++++++++++++++++++++++++++++++
 1 file changed, 46 insertions(+)
 create mode 100644 build/loader_hosted_examples.js

diff --git a/build/loader_hosted_examples.js b/build/loader_hosted_examples.js
new file mode 100644
index 0000000000..3ec22aa5e2
--- /dev/null
+++ b/build/loader_hosted_examples.js
@@ -0,0 +1,46 @@
+/**
+ *
+ * Loader to add ol.css, ol.js and the example-specific js file to the
+ * documents.
+ *
+ * This loader is used for the hosted examples. It is used in place of the
+ * development loader (examples/loader.js).
+ *
+ * ol.css and ol.js are built with Plovr/Closure, based build/ol.json.
+ * (`make build` should build them). They are located in the ../build/
+ * directory, relatively to this script.
+ *
+ * The script should be named loader.js. So it needs to be renamed to
+ * loader.js from loader_hosted_examples.js.
+ *
+ * Usage:
+ *
+ *     <script src="../loader.js?id=my-demo"></script>
+ */
+
+(function() {
+  var scripts = document.getElementsByTagName('script');
+
+  var i, src, index, search, chunks, pair, params = {};
+  for (i = scripts.length - 1; i >= 0; --i) {
+    src = scripts[i].getAttribute('src');
+    if (~(index = src.indexOf('loader.js?'))) {
+      search = src.substr(index + 10);
+      chunks = search ? search.split('&') : [];
+      for (i = chunks.length - 1; i >= 0; --i) {
+        pair = chunks[i].split('=');
+        params[decodeURIComponent(pair[0])] = decodeURIComponent(pair[1]);
+      }
+      break;
+    }
+  }
+
+  document.write('<link rel="stylesheet" href="../build/ol.css" '+
+                 'type="text/css">');
+  document.write('<scr' + 'ipt type="text/javascript" ' +
+                 'src="../build/ol.js">' +
+                 '</scr' + 'ipt>');
+  document.write('<scr' + 'ipt type="text/javascript" ' +
+                 'src="' + encodeURIComponent(params.id) + '.js">' +
+                 '</scr' + 'ipt>');
+}());

From 190e21f79631dfc732152fc90563ad6eeb50810a Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=C3=89ric=20Lemoine?= <eric.lemoine@gmail.com>
Date: Tue, 16 Oct 2012 21:55:12 +0200
Subject: [PATCH 18/26] The loader adds link tag for ol.css

---
 examples/full-screen.html       | 1 -
 examples/loader.js              | 2 ++
 examples/overlay-and-popup.html | 1 -
 examples/side-by-side.html      | 1 -
 examples/two-layers.html        | 1 -
 5 files changed, 2 insertions(+), 4 deletions(-)

diff --git a/examples/full-screen.html b/examples/full-screen.html
index f61977613c..a8ab852868 100644
--- a/examples/full-screen.html
+++ b/examples/full-screen.html
@@ -12,7 +12,6 @@
         height: 100%;
       }
     </style>
-    <link rel="stylesheet" href="../css/ol.css" type="text/css">
     <title>ol3 full-screen demo</title>
   </head>
   <body>
diff --git a/examples/loader.js b/examples/loader.js
index 3ea91f7536..52feee5a4e 100644
--- a/examples/loader.js
+++ b/examples/loader.js
@@ -54,6 +54,8 @@
     pairs.push(encodeURIComponent(key) + '=' + encodeURIComponent(params[key]));
   }
 
+  document.write('<link rel="stylesheet" href="../css/ol.css" ' +
+                 'type="text/css">');
   var url = 'http://' + host + '/compile?' + pairs.join('&');
   document.write('<script type="text/javascript" src="' + url + '"></script>');
 }());
diff --git a/examples/overlay-and-popup.html b/examples/overlay-and-popup.html
index 67bba0aa9b..fd0d44eaee 100644
--- a/examples/overlay-and-popup.html
+++ b/examples/overlay-and-popup.html
@@ -66,7 +66,6 @@
         margin-left: -13px;
       }
     </style>
-    <link rel="stylesheet" href="../css/ol.css" type="text/css">
     <title>ol3 overlay-and-popup demo</title>
   </head>
   <body>
diff --git a/examples/side-by-side.html b/examples/side-by-side.html
index c1c16f3220..da4a0472e3 100644
--- a/examples/side-by-side.html
+++ b/examples/side-by-side.html
@@ -2,7 +2,6 @@
 <html>
   <head>
     <meta http-equiv="Content-Type" content="text/html; charset=utf-8">
-    <link rel="stylesheet" href="../css/ol.css" type="text/css">
     <link rel="stylesheet" href="style.css" type="text/css">
     <style type="text/css">
       .map {
diff --git a/examples/two-layers.html b/examples/two-layers.html
index 75d4f9c301..17be65f288 100644
--- a/examples/two-layers.html
+++ b/examples/two-layers.html
@@ -2,7 +2,6 @@
 <html>
   <head>
     <meta http-equiv="Content-Type" content="text/html; charset=utf-8">
-    <link rel="stylesheet" href="../css/ol.css" type="text/css">
     <link rel="stylesheet" href="style.css" type="text/css">
     <style type="text/css">
       .map {

From 4b6ef3c0467bea99f4b8f06571bcaad12ea07643 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=C3=89ric=20Lemoine?= <eric.lemoine@gmail.com>
Date: Tue, 16 Oct 2012 21:56:35 +0200
Subject: [PATCH 19/26] Better comments in loader.js

---
 examples/loader.js | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

diff --git a/examples/loader.js b/examples/loader.js
index 52feee5a4e..7c56ba7ff6 100644
--- a/examples/loader.js
+++ b/examples/loader.js
@@ -1,10 +1,12 @@
 
 
 /**
- * Loader to add the plovr generated script to the document.  The following
- * default values may be overridden with query string parameters:
+ * Loader to add the plovr generated script and ol.css to the document.
  *
- *     * hostname - the current hostname
+ * The following default values may be overridden with query string
+ * parameters:
+ *
+ *     * hostname - the current hostname (window.location.hostname)
  *     * port - 9810
  *     * mode - ADVANCED
  *     * id - id param in loader.js query string; defaults to 'ol' if not set

From 1ac188c90619b4619c7333c57795f4a6fe5d0536 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=C3=89ric=20Lemoine?= <eric.lemoine@gmail.com>
Date: Tue, 16 Oct 2012 21:56:45 +0200
Subject: [PATCH 20/26] Add a hostexamples target to Makefile

This target copies the ol built files, examples, and loader script to the build/gh-pages directory, making it possible to host our examples on gh-pages.
---
 Makefile | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/Makefile b/Makefile
index f56d48150c..bcccf0b198 100644
--- a/Makefile
+++ b/Makefile
@@ -109,6 +109,14 @@ build/jsdoc-$(BRANCH)-timestamp: $(SRC) $(shell find doc/template -type f)
 	$(JSDOC) -t doc/template -r src -d build/gh-pages/$(BRANCH)/apidoc
 	touch $@
 
+.PHONY: hostexamples
+hostexamples: build examples
+	mkdir -p build/gh-pages/$(BRANCH)/examples
+	mkdir -p build/gh-pages/$(BRANCH)/build
+	cp $(EXAMPLES) $(subst .html,.js,$(EXAMPLES)) examples/style.css build/gh-pages/$(BRANCH)/examples/
+	cp build/loader_hosted_examples.js build/gh-pages/$(BRANCH)/examples/loader.js
+	cp build/ol.js build/ol.css build/gh-pages/$(BRANCH)/build/
+
 .PHONY: test
 test: $(INTERNAL_SRC)
 	$(PHANTOMJS) test/phantom-jasmine/run_jasmine_test.coffee test/ol.html

From 1d40b64db123893ec490a42473a55dc27ef1fff7 Mon Sep 17 00:00:00 2001
From: Tom Payne <twpayne@gmail.com>
Date: Wed, 17 Oct 2012 17:23:30 +0200
Subject: [PATCH 21/26] Move WebGL context creation into ol.webgl

---
 src/ol/renderer/webgl/map.js |  3 ++-
 src/ol/webgl/webgl.js        | 11 +++++++++++
 2 files changed, 13 insertions(+), 1 deletion(-)

diff --git a/src/ol/renderer/webgl/map.js b/src/ol/renderer/webgl/map.js
index 3b628321ef..bb0d1c6b9f 100644
--- a/src/ol/renderer/webgl/map.js
+++ b/src/ol/renderer/webgl/map.js
@@ -22,6 +22,7 @@ goog.require('ol.layer.TileLayer');
 goog.require('ol.renderer.webgl.FragmentShader');
 goog.require('ol.renderer.webgl.TileLayer');
 goog.require('ol.renderer.webgl.VertexShader');
+goog.require('ol.webgl');
 goog.require('ol.webgl.WebGLContextEventType');
 
 
@@ -163,7 +164,7 @@ ol.renderer.webgl.Map = function(container, map) {
    * @private
    * @type {WebGLRenderingContext}
    */
-  this.gl_ = this.canvas_.getContext('experimental-webgl', {
+  this.gl_ = ol.webgl.getContext(this.canvas_, {
     alpha: false,
     antialias: true,
     depth: false,
diff --git a/src/ol/webgl/webgl.js b/src/ol/webgl/webgl.js
index 05cc4a8a79..1711a64742 100644
--- a/src/ol/webgl/webgl.js
+++ b/src/ol/webgl/webgl.js
@@ -1,3 +1,4 @@
+goog.provide('ol.webgl');
 goog.provide('ol.webgl.WebGLContextEventType');
 
 
@@ -8,3 +9,13 @@ ol.webgl.WebGLContextEventType = {
   LOST: 'webglcontextlost',
   RESTORED: 'webglcontextrestored'
 };
+
+
+/**
+ * @param {Element} canvas Canvas.
+ * @param {Object=} opt_attributes Attributes.
+ * @return {WebGLRenderingContext} WebGL rendering context.
+ */
+ol.webgl.getContext = function(canvas, opt_attributes) {
+  return canvas.getContext('experimental-webgl', opt_attributes);
+};

From 8cb28a721756309ffa3b7f2111573dcf1d6b6bd7 Mon Sep 17 00:00:00 2001
From: Tom Payne <twpayne@gmail.com>
Date: Wed, 17 Oct 2012 17:25:32 +0200
Subject: [PATCH 22/26] Move WebGL support test into ol.webgl

---
 src/ol/renderer/webgl/webgl.js | 14 +++-----------
 src/ol/webgl/webgl.js          | 16 ++++++++++++++++
 2 files changed, 19 insertions(+), 11 deletions(-)

diff --git a/src/ol/renderer/webgl/webgl.js b/src/ol/renderer/webgl/webgl.js
index d7a0629ac2..c8eae9c377 100644
--- a/src/ol/renderer/webgl/webgl.js
+++ b/src/ol/renderer/webgl/webgl.js
@@ -1,5 +1,7 @@
 goog.provide('ol.renderer.webgl');
 
+goog.require('ol.webgl');
+
 
 /**
  * @define {boolean} Free resources immediately.
@@ -10,14 +12,4 @@ ol.renderer.webgl.FREE_RESOURCES_IMMEDIATELY = false;
 /**
  * @return {boolean} Is supported.
  */
-ol.renderer.webgl.isSupported = function() {
-  if (!('WebGLRenderingContext' in goog.global)) {
-    return false;
-  }
-  try {
-    var canvas = goog.dom.createElement(goog.dom.TagName.CANVAS);
-    return !goog.isNull(canvas.getContext('experimental-webgl'));
-  } catch (e) {
-    return false;
-  }
-};
+ol.renderer.webgl.isSupported = ol.webgl.isSupported;
diff --git a/src/ol/webgl/webgl.js b/src/ol/webgl/webgl.js
index 1711a64742..c7084a6299 100644
--- a/src/ol/webgl/webgl.js
+++ b/src/ol/webgl/webgl.js
@@ -19,3 +19,19 @@ ol.webgl.WebGLContextEventType = {
 ol.webgl.getContext = function(canvas, opt_attributes) {
   return canvas.getContext('experimental-webgl', opt_attributes);
 };
+
+
+/**
+ * @return {boolean} Is supported.
+ */
+ol.webgl.isSupported = function() {
+  if (!('WebGLRenderingContext' in goog.global)) {
+    return false;
+  }
+  try {
+    var canvas = goog.dom.createElement(goog.dom.TagName.CANVAS);
+    return !goog.isNull(canvas.getContext('experimental-webgl'));
+  } catch (e) {
+    return false;
+  }
+};

From 50111232281c03640664d41df78aaf6d4faf54bb Mon Sep 17 00:00:00 2001
From: Tom Payne <twpayne@gmail.com>
Date: Wed, 17 Oct 2012 17:27:11 +0200
Subject: [PATCH 23/26] Use ol.webgl.getContext is WebGL support test

---
 src/ol/webgl/webgl.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/ol/webgl/webgl.js b/src/ol/webgl/webgl.js
index c7084a6299..204c6ecb29 100644
--- a/src/ol/webgl/webgl.js
+++ b/src/ol/webgl/webgl.js
@@ -30,7 +30,7 @@ ol.webgl.isSupported = function() {
   }
   try {
     var canvas = goog.dom.createElement(goog.dom.TagName.CANVAS);
-    return !goog.isNull(canvas.getContext('experimental-webgl'));
+    return !goog.isNull(ol.webgl.getContext(canvas));
   } catch (e) {
     return false;
   }

From a82e523f6bfdf7cab2a3e6605b50f18cd134341b Mon Sep 17 00:00:00 2001
From: Tom Payne <twpayne@gmail.com>
Date: Wed, 17 Oct 2012 17:36:07 +0200
Subject: [PATCH 24/26] Try multiple context ids to get a WebGL context

---
 src/ol/webgl/webgl.js | 25 ++++++++++++++++++++++++-
 1 file changed, 24 insertions(+), 1 deletion(-)

diff --git a/src/ol/webgl/webgl.js b/src/ol/webgl/webgl.js
index 204c6ecb29..a3fae01859 100644
--- a/src/ol/webgl/webgl.js
+++ b/src/ol/webgl/webgl.js
@@ -2,6 +2,19 @@ goog.provide('ol.webgl');
 goog.provide('ol.webgl.WebGLContextEventType');
 
 
+/**
+ * @const
+ * @private
+ * @type {Array.<string>}
+ */
+ol.webgl.CONTEXT_IDS_ = [
+  'webgl',
+  'webgl-experimental',
+  'webkit-3d',
+  'moz-webgl'
+];
+
+
 /**
  * @enum {string}
  */
@@ -17,7 +30,17 @@ ol.webgl.WebGLContextEventType = {
  * @return {WebGLRenderingContext} WebGL rendering context.
  */
 ol.webgl.getContext = function(canvas, opt_attributes) {
-  return canvas.getContext('experimental-webgl', opt_attributes);
+  var context, i, ii = ol.webgl.CONTEXT_IDS_.length;
+  for (i = 0; i < ii; ++i) {
+    try {
+      context = canvas.getContext(ol.webgl.CONTEXT_IDS_[i], opt_attributes);
+      if (!goog.isNull(context)) {
+        return context;
+      }
+    } catch (e) {
+    }
+  }
+  return null;
 };
 
 

From 68a71c4a53691712f255f55f3633f27717179052 Mon Sep 17 00:00:00 2001
From: Tom Payne <twpayne@gmail.com>
Date: Wed, 17 Oct 2012 17:43:27 +0200
Subject: [PATCH 25/26] Add Tom Payne to authors.txt

---
 authors.txt | 1 +
 1 file changed, 1 insertion(+)

diff --git a/authors.txt b/authors.txt
index dfcd9ff2fd..057978f513 100644
--- a/authors.txt
+++ b/authors.txt
@@ -27,6 +27,7 @@ Eric Lemoine
 Philip Lindsay
 Martijn van Oosterhout
 David Overstrom
+Tom Payne
 Corey Puffault
 Peter William Robins
 Gregers Rygg

From e21c18264101bc6354fdb638d8635e96d0a11a5a Mon Sep 17 00:00:00 2001
From: Tom Payne <twpayne@gmail.com>
Date: Wed, 17 Oct 2012 18:20:25 +0200
Subject: [PATCH 26/26] Use Chrome Frame if available

---
 examples/full-screen.html       | 1 +
 examples/overlay-and-popup.html | 1 +
 examples/side-by-side.html      | 1 +
 examples/two-layers.html        | 1 +
 4 files changed, 4 insertions(+)

diff --git a/examples/full-screen.html b/examples/full-screen.html
index a8ab852868..f4fb9d5b62 100644
--- a/examples/full-screen.html
+++ b/examples/full-screen.html
@@ -2,6 +2,7 @@
 <html>
   <head>
     <meta http-equiv="Content-Type" content="text/html; charset=utf-8">
+    <meta http-equiv="X-UA-Compatible" content="chrome=1">
     <meta name="viewport" content="initial-scale=1.0, user-scalable=no, width=device-width">
     <link rel="stylesheet" href="style.css" type="text/css">
     <style type="text/css">
diff --git a/examples/overlay-and-popup.html b/examples/overlay-and-popup.html
index fd0d44eaee..37b6aa871b 100644
--- a/examples/overlay-and-popup.html
+++ b/examples/overlay-and-popup.html
@@ -2,6 +2,7 @@
 <html>
   <head>
     <meta http-equiv="Content-Type" content="text/html; charset=utf-8">
+    <meta http-equiv="X-UA-Compatible" content="chrome=1">
     <meta name="viewport" content="initial-scale=1.0, user-scalable=no, width=device-width">
     <link rel="stylesheet" href="style.css" type="text/css">
     <style type="text/css">
diff --git a/examples/side-by-side.html b/examples/side-by-side.html
index da4a0472e3..3e5ee89250 100644
--- a/examples/side-by-side.html
+++ b/examples/side-by-side.html
@@ -2,6 +2,7 @@
 <html>
   <head>
     <meta http-equiv="Content-Type" content="text/html; charset=utf-8">
+    <meta http-equiv="X-UA-Compatible" content="chrome=1">
     <link rel="stylesheet" href="style.css" type="text/css">
     <style type="text/css">
       .map {
diff --git a/examples/two-layers.html b/examples/two-layers.html
index 17be65f288..75bd4cc918 100644
--- a/examples/two-layers.html
+++ b/examples/two-layers.html
@@ -2,6 +2,7 @@
 <html>
   <head>
     <meta http-equiv="Content-Type" content="text/html; charset=utf-8">
+    <meta http-equiv="X-UA-Compatible" content="chrome=1">
     <link rel="stylesheet" href="style.css" type="text/css">
     <style type="text/css">
       .map {