nsISHEntry.index is readonly, but if you pass `true` as getEntryAtIndex()'s second argument, nsISHEntry.index will be modified. This is pretty gross. This patch changes `index` so it's not readonly (because it's not!) and removes getEntryAtIndex()'s second argument.