OLD | NEW |
1 // Copyright 2014 The Chromium Authors. All rights reserved. | 1 // Copyright 2014 The Chromium Authors. All rights reserved. |
2 // Use of this source code is governed by a BSD-style license that can be | 2 // Use of this source code is governed by a BSD-style license that can be |
3 // found in the LICENSE file. | 3 // found in the LICENSE file. |
4 | 4 |
5 /** | 5 /** |
6 * @fileoverview | 6 * @fileoverview |
7 * Full-screen implementation for apps v1, using webkitRequestFullscreen. | 7 * Full-screen implementation for apps v1, using webkitRequestFullscreen. |
8 */ | 8 */ |
9 | 9 |
10 'use strict'; | 10 'use strict'; |
(...skipping 61 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
72 }; | 72 }; |
73 | 73 |
74 remoting.FullscreenAppsV1.prototype.removeListener = function(callback) { | 74 remoting.FullscreenAppsV1.prototype.removeListener = function(callback) { |
75 this.eventSource_.removeEventListener(this.kEventName_, callback); | 75 this.eventSource_.removeEventListener(this.kEventName_, callback); |
76 }; | 76 }; |
77 | 77 |
78 /** | 78 /** |
79 * @private | 79 * @private |
80 */ | 80 */ |
81 remoting.FullscreenAppsV1.prototype.onFullscreenChanged_ = function() { | 81 remoting.FullscreenAppsV1.prototype.onFullscreenChanged_ = function() { |
82 this.eventSource_.raiseEvent(this.kEventName_, this.isActive()); | 82 // Querying full-screen immediately after the webkitfullscreenchange |
| 83 // event fires sometimes gives the wrong answer on Mac, perhaps due to |
| 84 // the time taken to animate presentation mode. Since I haven't been able |
| 85 // to isolate the exact repro steps, and we're not planning on using this |
| 86 // API for much longer, this hack will suffice for now. |
| 87 window.setTimeout( |
| 88 /** @this {remoting.FullscreenAppsV1} */ |
| 89 function() { |
| 90 this.eventSource_.raiseEvent(this.kEventName_, this.isActive()); |
| 91 }.bind(this), |
| 92 500); |
83 }; | 93 }; |
OLD | NEW |