wtf/PlatformString.h was removed. So, IconEfl.cpp file can't find PlatformString.h. This patch fixes this build error.
Created attachment 132745 [details] Patch
Comment on attachment 132745 [details] Patch Clearing flags on attachment: 132745 Committed r111348: <http://trac.webkit.org/changeset/111348>
All reviewed patches have been landed. Closing bug.
I'm sorry I broke your build. This fix is wrong however. < > is used for system headers. Headers which are sought for beyond the current directory. <text/PlatformString> is suggesting that somehow "text" is some separate module, which is not true. This header should just be "PlatformString.h", I'm sorry for wrongly changing it to <wtf/PlatformString.h> earlier. <wtf/text/WTFString.h> is basically equivalent to PlatformString.h and some day we'll remove PlatformString.h I think.
(In reply to comment #4) > I'm sorry I broke your build. This fix is wrong however. > > < > is used for system headers. Headers which are sought for beyond the current directory. > > <text/PlatformString> is suggesting that somehow "text" is some separate module, which is not true. > > This header should just be "PlatformString.h", I'm sorry for wrongly changing it to <wtf/PlatformString.h> earlier. <wtf/text/WTFString.h> is basically equivalent to PlatformString.h and some day we'll remove PlatformString.h I think. Ah, thank you for your fix again.