Skip to content

Commit

Permalink
CR comments
Browse files Browse the repository at this point in the history
  • Loading branch information
noamr committed Jun 27, 2021
1 parent 550cd15 commit da9f571
Showing 1 changed file with 21 additions and 21 deletions.
42 changes: 21 additions & 21 deletions source
Original file line number Diff line number Diff line change
Expand Up @@ -28056,11 +28056,11 @@ was an English &lt;a href="/wiki/Music_hall">music hall&lt;/a> singer, ...</code
null.</p>

<p>Each <span>image request</span> has a <dfn>current pixel density</dfn>, which must initially be
undefined.</p>
1.</p>

<p>Each <span>image request</span> has <dfn>preferred density-corrected dimensions</dfn>,
undefined or set to a struct consisting of a <code data-x="attr-dim-width">width</code> and a
<code data-x="attr-dim-height">height</code>, which must initially be undefined.</p>
Null or set to a struct consisting of a width and a height, which must initially be
null.</p>

<p>To determine the <dfn>density-corrected intrinsic width and height</dfn> of an
<code>img</code> element <var>img</var>:</p>
Expand All @@ -28069,13 +28069,13 @@ was an English &lt;a href="/wiki/Music_hall">music hall&lt;/a> singer, ...</code
<li><p>Let <var>dim</var> be <var>img</var>'s <span>current request</span>'s <span>preferred
density-corrected dimensions</span>.</p></li>

<li><p>If <var>dim</var> is undefined, set <var>dim</var> to <var>img</var>'s <span>intrinsic
<li><p>If <var>dim</var> is null, set <var>dim</var> to <var>img</var>'s <span>intrinsic
dimensions</span>.</p></li>

<li><p>Set <var>dim</var>'s width to <var>dim</var>'s width divided by <var>img</var>'s
<li><p>Set <var>dim</var>'s width to <var>dim</var>'s width divided by <span>current request</span>'s
<span>current pixel density</span>.</p></li>

<li><p>Set <var>dim</var>'s height to <var>dim</var>'s height divided by <var>img</var>'s
<li><p>Set <var>dim</var>'s height to <var>dim</var>'s height divided by <span>current request</span>'s
<span>current pixel density</span>.</p></li>

<li><p>Return <var>dim</var>.
Expand Down Expand Up @@ -28364,7 +28364,7 @@ was an English &lt;a href="/wiki/Music_hall">music hall&lt;/a> singer, ...</code
available</span>.</p></li>

<li><p><span data-x="prepare an image for presentation">Prepare <var>current request</var>
for presentation</span></p></li>
for presentation</span> given <var>img</var>.</p></li>

<li><p>Set <span>current request</span>'s <span>current pixel density</span> to <var>selected
pixel density</var>.</p></li>
Expand Down Expand Up @@ -28605,7 +28605,7 @@ was an English &lt;a href="/wiki/Music_hall">music hall&lt;/a> singer, ...</code
fetched must update the presentation of the image, but as each new body part comes in, if the
user agent is able to determine the image's width and height, it must <span data-x="prepare
an image for presentation">prepare the <code>img</code> element's <span>current request</span>
for presentation </span> and replace the previous image. Once one body part has been
for presentation</span> and replace the previous image. Once one body part has been
completely decoded, the user agent must set the <code>img</code> element's <span>current
request</span>'s <span data-x="img-req-state">state</span> to <span
data-x="img-all">completely available</span> and <span>queue an element task</span> on the
Expand Down Expand Up @@ -28633,7 +28633,7 @@ was an English &lt;a href="/wiki/Music_hall">music hall&lt;/a> singer, ...</code
<li><p>Otherwise, if the user agent is able to determine <var>image request</var>'s image's
width and height, and <var>image request</var> is <span>current request</span>, <span
data-x="prepare an image for presentation">prepare <var>image request</var> for
presentation</span> and set <var>image request</var>'s <span
presentation</span> given <var>img</var> and set <var>image request</var>'s <span
data-x="img-req-state">state</span> to <span data-x="img-inc">partially
available</span>.</p></li>

Expand Down Expand Up @@ -28677,7 +28677,7 @@ was an English &lt;a href="/wiki/Music_hall">music hall&lt;/a> singer, ...</code
<span>abort the image request</span> for the <span>current request</span>,
<span>upgrade the pending request to the current request</span> and
<span data-x="prepare an image for presentation">prepare <var>image request</var> for
presentation</span>.</p></li>
presentation</span> given <var>img</var>.</p></li>

<li><p>Set <var>image request</var> to the <span data-x="img-all">completely
available</span> state.</p></li>
Expand Down Expand Up @@ -28711,7 +28711,6 @@ was an English &lt;a href="/wiki/Music_hall">music hall&lt;/a> singer, ...</code

<ol>
<li><p>Forget <var>image request</var>'s <span data-x="img-req-data">image data</span>, if any.</p></li>

<li><p>Abort any instance of the <span data-x="concept-fetch">fetching</span> algorithm for
<var>image request</var>, discarding any pending tasks generated by that algorithm.</p></li>
</ol>
Expand All @@ -28728,13 +28727,15 @@ was an English &lt;a href="/wiki/Music_hall">music hall&lt;/a> singer, ...</code
<h6>Preparing an image for presentation</h6>

<p>When asked to <dfn>prepare an image for presentation</dfn> for an <span>image request</span>
<var>req</var>, user agents must do the following:</p>
<var>req</var> given image element <var>img</var>, user agents must do the following:</p>

<ol>
<li><p>Let <var>img</var> be <var>req</var>'s <code>img</code>.</p></li>
<li><p>Let <var>exifTagMap</var> be the EXIF tags obtained from <var>req</var>'s
<span data-x="img-req-data">image data</span>, as defined by the relevant codec.<ref spec=EXIF></p></li>

<li><p>Let <var>exifTagMap</var> be the EXIF tags obtained from <var>req</var>'s image data, as
defined by the relevant codec. <ref spec=EXIF></p></li>
<li><p>Let <var>physicalWidth</var> and <var>physicalHeight</var> be the width and height
obtained from <var>req</var>'s <span data-x="img-req-data">image data</span>, as defined by the
relevant codec.</p></li>

<li><p>Let <var>dimX</var> be the value of <var>exifTagMap</var>'s tag <code
data-x="">0xA002</code> (<code data-x="">PixelXDimension</code>).</p></li>
Expand All @@ -28760,10 +28761,10 @@ was an English &lt;a href="/wiki/Music_hall">music hall&lt;/a> singer, ...</code
<li><p>If <var>resUnit</var> is not equal to <code data-x="">2</code> (<code
data-x="">Inch</code>), then return.</p></li>

<li><p>Let <var>widthFromDensity</var> be the value of <var>img</var>'s width, multiplied by 72
<li><p>Let <var>widthFromDensity</var> be the value of <var>physicalWidth</var>, multiplied by 72
and divided by <var>resX</var>.</p></li>

<li><p>Let <var>heightFromDensity</var> be the value of <var>img</var>'s height, multiplied by
<li><p>Let <var>heightFromDensity</var> be the value of <var>physicalHeight</var>, multiplied by
72 and divided by <var>resY</var>.</p></li>

<li><p>If <var>widthFromDensity</var> is not equal to <var>dimX</var> or
Expand All @@ -28777,8 +28778,7 @@ was an English &lt;a href="/wiki/Music_hall">music hall&lt;/a> singer, ...</code
</li>

<li><p>Set <var>req</var>'s <span>preferred density-corrected dimensions</span> to
a struct with its <code data-x="attr-dim-width">width</code> set to <var>dimX</var> and its
<code data-x="attr-dim-width">height</code> set to <var>dimY</var>.</p></li>
a struct with its width set to <var>dimX</var> and its height set to <var>dimY</var>.</p></li>

<li><p>Update <var>req</var>'s <code>img</code> element's presentation appropriately.</p></li>
</ol>
Expand Down Expand Up @@ -29381,8 +29381,8 @@ was an English &lt;a href="/wiki/Music_hall">music hall&lt;/a> singer, ...</code

<li><p><span>Upgrade the pending request to the current request</span>.</p></li>

<li><p><span data-x="prepare an image for presentation">prepare <var>image request</var> for
presentation</span></p></li>
<li><p><span data-x="prepare an image for presentation">Prepare <var>image request</var> for
presentation</span> given <var>img</var>.</p></li>

<li><p><span data-x="concept-event-fire">Fire an event</span> named <code
data-x="event-load">load</code> at the <code>img</code> element.</p></li>
Expand Down

0 comments on commit da9f571

Please sign in to comment.