<html>
<head>
<base href="https://bugs.webkit.org/" />
</head>
<body>
<p>
<div>
<b><a class="bz_bug_link
bz_status_NEW "
title="NEW - [GTK] Fix some video/canvas tests that should be passing"
href="https://bugs.webkit.org/show_bug.cgi?id=160023#c20">Comment # 20</a>
on <a class="bz_bug_link
bz_status_NEW "
title="NEW - [GTK] Fix some video/canvas tests that should be passing"
href="https://bugs.webkit.org/show_bug.cgi?id=160023">bug 160023</a>
from <span class="vcard"><a class="email" href="mailto:cgarcia@igalia.com" title="Carlos Garcia Campos <cgarcia@igalia.com>"> <span class="fn">Carlos Garcia Campos</span></a>
</span></b>
<pre>(In reply to <a href="show_bug.cgi?id=160023#c19">comment #19</a>)
<span class="quote">> Comment on <span class=""><a href="attachment.cgi?id=284333&action=diff" name="attach_284333" title="Patch">attachment 284333</a> <a href="attachment.cgi?id=284333&action=edit" title="Patch">[details]</a></span>
> Patch
>
> View in context:
> <a href="https://bugs.webkit.org/attachment.cgi?id=284333&action=review">https://bugs.webkit.org/attachment.cgi?id=284333&action=review</a>
>
> > LayoutTests/media/video-canvas.js:4
> > + return window.testRunner.videoCanvasPixelComparisonTolerance;
>
> Crazy thought, is it possible to detect WebKitGTK from JavaScript (e.g.
> using user-agent string?). If so, it may be simpler to return 6 if we detect
> WebKitGTK / WebKitEFL and 2 otherwise. It would be pure JS and would not
> require any TestRunner functionality.</span >
I don't think so, because of websites parsing UA string and assuming there are only 3 browsers in the world, we pretend to be safari in our UA. Of course we could do some hacks like checking if there's Linux in the UA, and assume that's EFL/GTK+, but I don't see how that's better than exposing the property in the TestRunner. What we could do, though is modify the user agent from WTR using WKPageSetApplicationNameForUserAgent, appending something like WebKitTestRunnerGTK, WebKitTestRunnerEFL, WebKitTestRunnerFoo and then parse the UA string from tests to detect the port. Good thing of this is that it would be more generic solution, we could add helper functions to the common js files isGtk(), isEfl(), etc.</pre>
</div>
</p>
<hr>
<span>You are receiving this mail because:</span>
<ul>
<li>You are the assignee for the bug.</li>
</ul>
</body>
</html>