[Webkit-unassigned] [Bug 42432] Enable Web Timing for GTK

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=42432


Martin Robinson <mrobinson at webkit.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #93584|review?                     |review-
               Flag|                            |




--- Comment #12 from Martin Robinson <mrobinson at webkit.org>  2011-05-16 12:35:31 PST ---
(From update of attachment 93584)
View in context: https://bugs.webkit.org/attachment.cgi?id=93584&action=review

Nice work. I think the new code in Source/WebCore/platform/network/soup/ResourceHandleSoup.cpp should only be compiled in if web timing is enabled though.

> LayoutTests/platform/gtk/Skipped:-302
> +# Navigation Timing http tests that are still failing
>  http/tests/misc/webtiming-origins.html
> -http/tests/misc/webtiming-one-redirect.php
>  http/tests/misc/webtiming-slow-load.php
>  http/tests/misc/webtiming-ssl.php
> -http/tests/misc/webtiming-two-redirects.php

It'd be nice to have some understanding of why these are still failing.

> Source/WebCore/platform/network/soup/ResourceHandleSoup.cpp:35
>  #include "ChromeClient.h"
>  #include "CookieJarSoup.h"
>  #include "CachedResourceLoader.h"
> +#include "CurrentTime.h"

Shouldn't all the new code in this file be guarded by #if ENABLE(WEB_TIMING)?

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.