vaev-loader: Added comment about default object size.

Author: sleepy-monax

src/main.cpp

@@ -172,7 +172,7 @@ static Async::Task<> renderAsync(
     auto scene = window->render();
 
     // NOTE: Override the background of HTML document, since no
-    //       one really expect a html document to transparent
+    //       one really expect a html document to be transparent
     if (window->document()->documentElement()->namespaceUri() == Vaev::Html::NAMESPACE) {
         scene = makeRc<Scene::Clear>(scene, Gfx::WHITE);
     }

src/vaev-engine/loader/loader-impl.cpp

@@ -37,8 +37,13 @@ Async::Task<Rc<Scene::Node>> _fetchImageContentAsync(Http::Client& client, Ref::
         window->computeStyle();
         auto root = window->document()->documentElement();
         Layout::Resolver resolver;
+
+        // NOSPEC: The spec references a “default object size” but does not define explicit values.
+        //         Historically, browsers (including Chrome) default to 300x150, as mentioned in older drafts:
+        //         https://www.w3.org/TR/2011/WD-css3-images-20110908/#default-object-size
         auto width = resolver.resolve(root->specifiedValues()->sizing->width.unwrapOr<CalcValue<PercentOr<Length>>>(Length{300_au}), 300_au);
         auto height = resolver.resolve(root->specifiedValues()->sizing->height.unwrapOr<CalcValue<PercentOr<Length>>>(Length{300_au}), 300_au);
+
         window->changeMedia(Style::Media::forRender({width, height}, Resolution::fromDppx(1)));
         co_return Ok(window->render());
     } else {

tests/html/tag-img.xhtml

@@ -31,6 +31,8 @@
             src="data:image/svg+xml;base64,PHN2ZyB3aWR0aD0iMjAwIiBoZWlnaHQ9IjIwMCIgeG1sbnM9Imh0dHA6Ly93d3cudzMub3JnLzIwMDAvc3ZnIj4KICAgIDxyZWN0IHdpZHRoPSIyMDAiIGhlaWdodD0iMjAwIiBmaWxsPSJncmVlbiIgLz4KPC9zdmc+Cg==" />
     </rendering>
 
+    <!-- TODO: Add test for loading images from actual file when this is supported by reftest -->
+
     <error>
     </error>
 </test>