Bug 123981 – [GTK] Move baselines after r158742 into proper place

• Status: RESOLVED
• Resolution: FIXED
• Component: WebKitGTK

[Krzysztof Czech]

r158742 added files to platform/gtk-wk1/accessibility/ which was wrong.
Correcting this by moving them to platform/gtk/accessibility.

[Krzysztof Czech]

Created attachment 216282 [details]
patch

[Mario Sanchez Prada]

Comment on attachment 216282 [details]
patch

(In reply to comment #0)
> r158742 added files to platform/gtk-wk1/accessibility/ which was wrong.
> Correcting this by moving them to platform/gtk/accessibility.

Now problem, after all I haven't realized of it either.

Thanks for taking care of getting it back to the original state

[Mario Sanchez Prada]

Comment on attachment 216282 [details]
patch

Hmm.. it seems the quilty commit was r158810, as a result of running "webkit-patch optimize-baselines accessibility". Setting cq- for not, need to double check this better...

[Mario Sanchez Prada]

Comment on attachment 216282 [details]
patch

(In reply to comment #3)
> (From update of attachment 216282 [details])
> Hmm.. it seems the quilty commit was r158810, as a result of running "webkit-?
> patch optimize-baselines accessibility". Setting cq- for not, need to double 
> check this better...

It was a bit confusing but after some investigation I understand what happened: the "optimize baselines" command just move expectations around to make their storage more efficient but, unluckily enough, those two tests related two tables also got at almost the same time in the state of "needing rebaseline".

So that's why they were failing in the bots: not because of their wrong location but because neither the expectations in gtk-wk1/ nor the ones in gtk-wk2/ were correct at this point. Fortunately, they are correct anyway and, even better, exactly the same among the two versions of the port, so the right solution is to generate new expectations, place them together under gtk/ again and remove the specific ones we used to have in gtk-wk2.

So, clearing flags for now

[Mario Sanchez Prada]

(In reply to comment #4)
> [...] so the right solution is to generate new expectations, place them
> together under gtk/ again and remove the specific ones we used to have
> in gtk-wk2.
> 
Done in SVN r158847:
http://trac.webkit.org/changeset/158847