Merge pull request immersive-web#300 from cabanier/cac

remove unused chromaticaberration

Author: cabanier

webxrlayers-1.bs

@@ -329,7 +329,6 @@ XRCompositionLayer defines a set of common attributes and behaviors across certa
   readonly attribute XRLayerLayout layout;
 
   attribute boolean blendTextureSourceAlpha;
-  attribute boolean? chromaticAberrationCorrection;
   attribute boolean forceMonoPresentation;
   attribute float opacity;
   readonly attribute unsigned long mipLevels;
@@ -344,11 +343,6 @@ The <dfn attribute for="XRCompositionLayer">layout</dfn> attribute returns the l
 
 The <dfn attribute for="XRCompositionLayer">blendTextureSourceAlpha</dfn> attribute enables the layer’s texture alpha channel.
 
-The <dfn attribute for="XRCompositionLayer">chromaticAberrationCorrection</dfn> attribute is a hint for the [=XR Compositor=] to
-enable optical chromatic aberration correction for the layer. If the user agent or device does not support this attribute, they
-should return <code>null</code> on getting, and setting should be a <code>no-op</code>.
-If the {{XRCompositionLayer/chromaticAberrationCorrection}} was changed, it will take effect at the next {{XRFrame}}.
-
 The <dfn attribute for="XRCompositionLayer">forceMonoPresentation</dfn> attribute forces the right eye to use the same layer configuration
 as the left eye. This MUST send a signal to the [=XR Compositor=] to remove the {{XRLayerLayout/stereo}} effect of this {{XRCompositionLayer}}. Setting this to <code>true</code> on a {{XRLayerLayout/mono}} layer has no effect.
 
@@ -385,13 +379,6 @@ or a {{WebGL2RenderingContext}} |context|, the user agent MUST run the following
   1. Set [=this=] [=XRCompositionLayer/session=] to |session|.
   1. If |context| is defined, set [=this=] [=XRCompositionLayer/context=] to |context|.
   1. Set [=this=] {{XRCompositionLayer/blendTextureSourceAlpha}} to <code>true</code>.
-  1. Initialize [=this=] {{XRCompositionLayer/chromaticAberrationCorrection}} as follows:
-    <dl class="switch">
-      <dt class="switch">If the user agent supports chromatic aberration correction
-      <dd> Set [=this=] {{XRCompositionLayer/chromaticAberrationCorrection}} to <code>true</code> or <code>false</code> depending on the user agent's preference.
-      <dt> Otherwise
-      <dd> Set [=this=] {{XRCompositionLayer/chromaticAberrationCorrection}} to <code>null</code>.
-    </dl>
   1. Set [=this=] {{XRCompositionLayer/opacity}} to <code>1.0</code>.
 
 </div>
@@ -859,7 +846,8 @@ An [=opaque texture=] functions identically to a standard {{WebGLTexture}} with
 - An [=opaque texture=] MUST behave as though it was allocated with [=texStorage2D=] or [=texStorage3D=], as appropriate, even when using a WebGL 1.0 context.
 - A call to {{deleteTexture}} with an [=opaque texture=] MUST generate an {{INVALID_OPERATION}} error.
 
-The buffers attached to an [=opaque texture=] MUST be cleared to the values in the table below during the processing of the first call to {{XRWebGLBinding/getViewSubImage()}} or {{XRWebGLBinding/getSubImage()}} in each [=XR animation frame=].
+The buffers attached to an [=opaque texture=] MUST be cleared to the values in the table below during the processing of the first call to {{XRWebGLBinding/getViewSubImage()}} or {{XRWebGLBinding/getSubImage()}} in each [=XR animation frame=] when  {{XRProjectionLayerInit/clearOnAccess}} is <code>true</code>.
+If {{XRProjectionLayerInit/clearOnAccess}} is <code>false</code>, the buffers attached to an [=opaque texture=] MUST be cleared the first time that they are accessed. Subsequent accesses in later frames MAY NOT clear the buffers.
 
 <table class="tg">
   <thead>
@@ -911,6 +899,7 @@ dictionary XRProjectionLayerInit {
   GLenum colorFormat = 0x1908; // RGBA
   GLenum depthFormat = 0x1902; // DEPTH_COMPONENT
   double scaleFactor = 1.0;
+  boolean clearOnAccess = true;
 };
 </pre>
 
@@ -954,6 +943,8 @@ For {{WebGL2RenderingContext}} contexts these additional formats are supported:
 The <dfn dict-member for="XRProjectionLayerInit">scaleFactor</dfn> attribute defines the value that the |session|'s
 [=recommended WebGL framebuffer resolution=] MUST be multiplied by determining the resolution of the layer's attachments.
 
+The <dfn dict-member for="XRProjectionLayerInit">clearOnAccess</dfn> attribute defines if the texture associated with this layer should be cleared in the initial frame or on every frame.
+
 NOTE: the {{XRProjectionLayerInit}} dictionary does not have support to configure {{XRLayerInit/mipLevels}} like
 {{XRLayerInit}}. If a user agent wants to support mipmapping on projection layers, it is free to allocate the texture with mips.
 In that case the user agent (and not the author) is responsible for generating all the mip levels.
@@ -973,6 +964,7 @@ dictionary XRLayerInit {
   required unsigned long viewPixelHeight;
   XRLayerLayout layout = "mono";
   boolean isStatic = false;
+  boolean clearOnAccess = true;
 };
 </pre>
 
@@ -1026,6 +1018,8 @@ the rectangular dimensions of the {{XRCompositionLayer}}.
 
 The <dfn dict-member for="XRLayerInit">layout</dfn> attribute defines the layout of the layer.
 
+The <dfn dict-member for="XRLayerInit">clearOnAccess</dfn> attribute defines if the texture associated with this layer should be cleared in the initial frame or on every frame.
+
 XRQuadLayerInit {#xrquadlayerinit}
 -------------------