Check isActive in onRestore to work-around crbug.com/394819.

remoting/webapp/fullscreen_v2.js

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 /**
  * @fileoverview
  * Full-screen implementation for apps v2, using chrome.AppWindow.
  */
 
 'use strict';
@@ skipping 94 matching lines @@
   document.body.classList.add('fullscreen');
 };
 
 remoting.FullscreenAppsV2.prototype.onMaximized_ = function() {
   if (this.hookingWindowEvents_) {
     this.activate(true);
   }
 };
 
 remoting.FullscreenAppsV2.prototype.onRestored_ = function() {
+  // TODO(jamiewalch): ChromeOS generates a spurious onRestore event if
+  // fullscreen() is called from an onMaximized handler (crbug.com/394819),
+  // so test the full-screen status to make sure that this callback is

[Wez, 2014/07/17 18:49:42]

nit: Clarify what you mean by this; i.e. what you mean is "don't do anything if
we receive onRestored_ but Chrome indicates that full-screen is actually
active." I think?

[Jamie, 2014/07/17 21:36:44]

Done.

+  // genuine.
+  if (this.isActive()) {
+    return;
+  }
+
   document.body.classList.remove('fullscreen');
   if (this.hookingWindowEvents_) {
     this.activate(false);
   }
   if (this.notifyCallbacksOnRestore_) {
     this.notifyCallbacksOnRestore_ = false;
     this.eventSource_.raiseEvent(this.kEventName_, false);
   }
 };