perf(DeviceService): start/stop updates based on visibility state

argonjs · Apr 21, 2017 · d2f2194 · d2f2194
1 parent b3d7b09
commit d2f2194
Showing 1 changed file with 7 additions and 6 deletions.
diff --git a/src/device.ts b/src/device.ts
@@ -47,6 +47,8 @@ import {
     ViewportMode
 } from './view'
 
+import { VisibilityService } from './visibility'
+
 export class DeviceState {
     viewport?:CanvasViewport;
     subviews?:SerializedSubviewList;
@@ -145,7 +147,8 @@ export class DeviceService {
     constructor(
         protected sessionService:SessionService,
         protected contextService:ContextService,
-        protected viewService:ViewService
+        protected viewService:ViewService,
+        protected visibilityService:VisibilityService
     ) {
         sessionService.manager.on['ar.device.state'] = 
             sessionService.manager.on['ar.device.frameState'] = this._onDeviceState.bind(this);
@@ -217,11 +220,9 @@ export class DeviceService {
         //     });
         // }
 
-        this.startUpdates();
-        this.sessionService.manager.closeEvent.addEventListener(()=>{
-            this.stopUpdates();
-        })
-
+        this.visibilityService.showEvent.addEventListener(() => this.startUpdates());
+        this.visibilityService.hideEvent.addEventListener(() => this.stopUpdates());
+
         this._setupVRPresentChangeHandler();
     }