Fix the coverity issues

This PR is to resolve issues raised by coverity.

BUG=XWALK-2928
I don't see why one or the other should be the default. It probably makes more sense to make `manifest_type` a mandatory argument to the constructor and then change the child classes. @pozdnyakov, wdyt?