[Android] Add IndexedDB to the build
Created attachment 56877 [details]
Comment on attachment 56877 [details]
These should be JSIDBFoo.h
These should be bindings/V8IDBFoo.h
+ ifeq ($(ENABLE_INDEXED_DATABASE), true)
No need for the guards, as the file contents are guarded anyway (the SVG guards are just a legacy thing)
Created attachment 57023 [details]
Comment on attachment 57023 [details]
Change not needed
(In reply to comment #5)
> (From update of attachment 57023 [details])
> + storage/StorageSyncManager.cpp
> Change not needed
Which is preferred? I looked and saw at least a couple cases where there was none at the end, so I assumed that was preferred.
No trailing slash is preferred - just pointing out that this is no
longer required for your patch.
It's not clear from the diff, but that line is in the same block as the one I was adding to. I just noticed it and figured I'd clean it up as I went. It was never strictly required. But I think it makes the world a slightly better place.
Committed r60228: <http://trac.webkit.org/changeset/60228>