EFL debug build is broken after r163234 on EFL.
Created attachment 222902 [details]
Comment on attachment 222902 [details]
View in context: https://bugs.webkit.org/attachment.cgi?id=222902&action=review
> + Reviewed by NOBODY (OOPS!).
Because this is build fix, you can make the patch as "Unreviewed build fix"
> + Fixed the broken build on EFL after r163234.
> + * Modules/indexeddb/IDBKeyData.cpp:
> + (WebCore::IDBKeyData::loggingString): Added the default: case.
I am not sure whether it is EFL specific.
Maybe "Debug build is broken with INDEXED_DATABASE
Like other statements,
Can we add it out of switch() with "ASSERT_NOT_REACHED();"
Created attachment 222906 [details]
Comment on attachment 222906 [details]
View in context: https://bugs.webkit.org/attachment.cgi?id=222906&action=review
You don't need to request review in unreviewed case.
> + [EFL] REGRESSION(r163234) EFL debug build is broken
Remove [EFL] prefix.
> + Unriewed. Debug build is broken with INDEXED_DATABASE.
Created attachment 222909 [details]
Comment on attachment 222909 [details]
Clearing flags on attachment: 222909
Committed r163261: <http://trac.webkit.org/changeset/163261>
All reviewed patches have been landed. Closing bug.