Bug 85622 - [EFL] Update Web Timing tests expectations
Summary: [EFL] Update Web Timing tests expectations
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit EFL (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Chris Dumez
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-05-04 07:32 PDT by Thiago Marcos P. Santos
Modified: 2012-05-08 12:03 PDT (History)
5 users (show)

See Also:


Attachments
Patch (1.42 KB, patch)
2012-05-08 11:52 PDT, Chris Dumez
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Thiago Marcos P. Santos 2012-05-04 07:32:35 PDT
This feature is enable on EFL port, but the fallowing tests are failing.

http/tests/misc/webtiming-origins.html
http/tests/misc/webtiming-ssl

Important to note that they are either marked as failure or skipped for most of the other platforms.
Comment 1 Chris Dumez 2012-05-04 09:37:51 PDT
This is odd, they were passing for me otherwise I wouldn't unskipped them when I enabled Web Timing. Can I investigate those or do you want to Thiago?
Comment 2 Thiago Marcos P. Santos 2012-05-04 10:14:31 PDT
(In reply to comment #1)
> This is odd, they were passing for me otherwise I wouldn't unskipped them when I enabled Web Timing. Can I investigate those or do you want to Thiago?

It's all yours. They are failing on the bot and also on my machine (64-bits and 32-bits Ubuntu 12.04)
Comment 3 Thiago Marcos P. Santos 2012-05-04 14:19:01 PDT
Changed the bug description since they are passing after bug 85072. Nice catch btw.
Comment 4 Chris Dumez 2012-05-08 11:52:42 PDT
Created attachment 140757 [details]
Patch

Could someone please cq+ ?
Comment 5 Raphael Kubo da Costa (:rakuco) 2012-05-08 12:03:49 PDT
Comment on attachment 140757 [details]
Patch

Clearing flags on attachment: 140757

Committed r116443: <http://trac.webkit.org/changeset/116443>
Comment 6 Raphael Kubo da Costa (:rakuco) 2012-05-08 12:03:56 PDT
All reviewed patches have been landed.  Closing bug.