A pre-render request is processed in order of priority. Hence as a pre-render concept, it is always low priority. Because tiles in viewport should be displayed first. It is not necessary to add a request in the head of the queue. And additionally to keep pace with upstream code, I try to remove _Ewk_Tiled_Backing_Store_Pre_Render_Priority flags and functions which uses the flags as a parameter.
Created attachment 102611 [details] remove pre_render priority flags
I think this patch needs Rafael review. So, CC'ing Rafael.
LGTM from my side except for the comment below. > Source/WebKit/efl/ChangeLog:8 > + Remove _Ewk_Tiled_Backing_Store_Pre_Render_Priority to keep pace with upstream. Huh? This is upstream.
Was upstream not a term of the latest code? Then I'll change the word. I meant 'to keep pace with the latest code'
Created attachment 103598 [details] second_patch I've changed the comment.
> Source/WebKit/efl/ChangeLog:6 > + High pre-render priority flag is not using anywhere. Only low priority is using. How about just "The high priority pre-render flag was not used anywhere, so we can get rid of the whole concept and always consider the requests have low priority"?
Created attachment 104634 [details] third patch No problem, I have changed the description.
Comment on attachment 104634 [details] third patch LGTM.
Created attachment 114513 [details] a new rebase patch this patch has been left for a couple of month. So I rebase and file it again.
Looks OK.
Comment on attachment 114513 [details] a new rebase patch Clearing flags on attachment: 114513 Committed r100025: <http://trac.webkit.org/changeset/100025>
All reviewed patches have been landed. Closing bug.