From 790397ae0f05a8dfc8694c4d99197d5be2ccbce9 Mon Sep 17 00:00:00 2001 From: Victor Woeltjen Date: Mon, 9 Feb 2015 16:07:05 -0800 Subject: [PATCH] [Plot] Add test for buffer size Add test case for usage of drawing buffer size as reported by WebGL (instead of assuming we always got the full size we requested.) WTD-852. --- platform/features/plot/test/GLChartSpec.js | 13 +++++++++++++ 1 file changed, 13 insertions(+) diff --git a/platform/features/plot/test/GLChartSpec.js b/platform/features/plot/test/GLChartSpec.js index cfd38d0638..67ec83aa73 100644 --- a/platform/features/plot/test/GLChartSpec.js +++ b/platform/features/plot/test/GLChartSpec.js @@ -106,6 +106,19 @@ define( expect(mockGL.drawArrays) .toHaveBeenCalledWith("TRIANGLE_FAN", 0, 4); }); + + it("uses buffer sizes reported by WebGL", function () { + // Make sure that GLChart uses the GL buffer size, which may + // differ from what canvas requested. WTD-852 + mockCanvas.width = 300; + mockCanvas.height = 150; + mockGL.drawingBufferWidth = 200; + mockGL.drawingBufferHeight = 175; + + glChart.clear(); + + expect(mockGL.viewport).toHaveBeenCalledWith(0, 0, 200, 175); + }); }); } ); \ No newline at end of file