Bug 147809

Summary: [EFL] Add disabled CSS_COMPOSITING option with new baseline
Product: WebKit Reporter: Gyuyoung Kim <gyuyoung.kim>
Component: WebKit EFLAssignee: Gyuyoung Kim <gyuyoung.kim>
Status: RESOLVED WONTFIX    
Severity: Normal CC: buildbot, lucas.de.marchi, mcatanzaro, ossy, rniwa
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Archive of layout-test-results from ews104 for mac-mavericks-wk2
none
Patch
none
Patch none

Gyuyoung Kim
Reported 2015-08-08 04:29:23 PDT
CSS_COMPOSITING can be enabled on EFL port. However some tests don't have expectation.
Attachments
Patch (102.07 KB, patch)
2015-08-08 04:37 PDT, Gyuyoung Kim
no flags
Archive of layout-test-results from ews104 for mac-mavericks-wk2 (638.12 KB, application/zip)
2015-08-08 05:17 PDT, Build Bot
no flags
Patch (102.07 KB, patch)
2015-08-08 06:01 PDT, Gyuyoung Kim
no flags
Patch (100.71 KB, patch)
2015-08-10 17:30 PDT, Gyuyoung Kim
no flags
Gyuyoung Kim
Comment 1 2015-08-08 04:37:09 PDT
Build Bot
Comment 2 2015-08-08 05:17:46 PDT
Comment on attachment 258565 [details] Patch Attachment 258565 [details] did not pass mac-wk2-ews (mac-wk2): Output: http://webkit-queues.webkit.org/results/30804 New failing tests: fast/replaced/object-with-embed-url-param.html
Build Bot
Comment 3 2015-08-08 05:17:48 PDT
Created attachment 258566 [details] Archive of layout-test-results from ews104 for mac-mavericks-wk2 The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews. Bot: ews104 Port: mac-mavericks-wk2 Platform: Mac OS X 10.9.5
Gyuyoung Kim
Comment 4 2015-08-08 06:01:27 PDT
Csaba Osztrogonác
Comment 5 2015-08-10 02:09:16 PDT
Have you compared png results with Mac results? Are they same/similar? Additionally it would be great to check-in png results too, having only rendertree dumps aren't so useful for human eyes to determine if the results are correct or not.
Gyuyoung Kim
Comment 6 2015-08-10 02:33:57 PDT
(In reply to comment #5) > Have you compared png results with Mac results? Are they same/similar? > Additionally it would be great to check-in png results too, having only > rendertree dumps aren't so useful for human eyes to determine if the > results are correct or not. EFL layout test hasn't supported image diff officially yet. However let me check it locally. BTW it looks like you are on summer vacation. Have a nice vacation :)
Gyuyoung Kim
Comment 7 2015-08-10 17:30:44 PDT
Gyuyoung Kim
Comment 8 2015-08-10 17:35:35 PDT
(In reply to comment #5) > Have you compared png results with Mac results? Are they same/similar? > Additionally it would be great to check-in png results too, having only > rendertree dumps aren't so useful for human eyes to determine if the > results are correct or not. Actual result of png files have nothing when running css3/blending with CSS3_COMPOSITING. So I need to investigate it. Anyway I would like to land this patch(with PRIVATE OFF) as a first step in order to support CSS3_COMPOSITING. When I fix this problem, I would like to set it to PUBLIC ON. What do you think about it ?
Gyuyoung Kim
Comment 9 2015-08-11 21:42:56 PDT
(In reply to comment #8) > (In reply to comment #5) > > Have you compared png results with Mac results? Are they same/similar? > > Additionally it would be great to check-in png results too, having only > > rendertree dumps aren't so useful for human eyes to determine if the > > results are correct or not. > > Actual result of png files have nothing when running css3/blending with > CSS3_COMPOSITING. So I need to investigate it. Anyway I would like to land > this patch(with PRIVATE OFF) as a first step in order to support > CSS3_COMPOSITING. When I fix this problem, I would like to set it to PUBLIC > ON. What do you think about it ? Ossy ?
Csaba Osztrogonác
Comment 10 2015-08-11 23:12:30 PDT
rs=me to add the option. But why should we add expected files for the tests if the feature is still disabled? I prefer adding them when we enable the feature.
Gyuyoung Kim
Comment 11 2015-08-11 23:40:21 PDT
(In reply to comment #10) > rs=me to add the option. > > But why should we add expected files for > the tests if the feature is still disabled? > I prefer adding them when we enable the feature. Added expected files are similar with mac's one. So render tree expected result seems to be correct to me. If you think it would be good to add both text and png result at a time, it is better to add png result with this patch. Let me try to fix wrong png result soon. Then I will request review again.
Csaba Osztrogonác
Comment 12 2015-08-12 01:38:32 PDT
(In reply to comment #11) > (In reply to comment #10) > > rs=me to add the option. > > > > But why should we add expected files for > > the tests if the feature is still disabled? > > I prefer adding them when we enable the feature. > > Added expected files are similar with mac's one. So render tree expected > result seems to be correct to me. If you think it would be good to add both > text and png result at a time, it is better to add png result with this > patch. Let me try to fix wrong png result soon. Then I will request review > again. How is it possible that we have similar results with disabled CSS compositing as Mac platform with enabled CSS compositing? Something is weird here and should be investigated in details. If these tests are for testing CSS compositing feature, they should fail if the feature is disabled.
Gyuyoung Kim
Comment 13 2015-08-12 01:46:06 PDT
(In reply to comment #12) > (In reply to comment #11) > > (In reply to comment #10) > > > rs=me to add the option. > > > > > > But why should we add expected files for > > > the tests if the feature is still disabled? > > > I prefer adding them when we enable the feature. > > > > Added expected files are similar with mac's one. So render tree expected > > result seems to be correct to me. If you think it would be good to add both > > text and png result at a time, it is better to add png result with this > > patch. Let me try to fix wrong png result soon. Then I will request review > > again. > > How is it possible that we have similar results with disabled CSS > compositing > as Mac platform with enabled CSS compositing? Something is weird here and > should be investigated in details. The similar results were gotten when CSS compositing is enabled. As we said, png results have problem, I just wanted to add the results first.
Csaba Osztrogonác
Comment 14 2015-08-12 02:26:05 PDT
(In reply to comment #13) > The similar results were gotten when CSS compositing is enabled. As we said, > png results have problem, I just wanted to add the results first. This patch doesn't enable CSS compositing, only adds the cmake option. Why is it useful to add expected files for enabled CSS compositing? They would fail on the bot until we enable it. (If they pass now, the tests are completely wrong and don't test CSS compositing at all.) I prefer adding expected files in the patch which enables the feature in the future.
Gyuyoung Kim
Comment 15 2015-08-12 02:36:50 PDT
(In reply to comment #14) > (In reply to comment #13) > > > The similar results were gotten when CSS compositing is enabled. As we said, > > png results have problem, I just wanted to add the results first. > > This patch doesn't enable CSS compositing, only adds the cmake option. As I said, it was gotten when I enabled it. But latest patch disabled it for now. > Why is it useful to add expected files for enabled CSS compositing? > They would fail on the bot until we enable it. (If they pass now, > the tests are completely wrong and don't test CSS compositing at all.) > I prefer adding expected files in the patch which enables the feature > in the future. I already agreed with your suggestion !
Michael Catanzaro
Comment 16 2017-03-11 10:51:23 PST
Closing this bug because the EFL port has been removed from trunk. If you feel this bug applies to a different upstream WebKit port and was closed in error, please either update the title and reopen the bug, or leave a comment to request this.
Note You need to log in before you can comment on or make changes to this bug.