You still need to traverse the tree if you are doubling a node. Drop the else
so that you always traverse. Also, I've removed the assignments because you are not changing the tree structure:
if(root != null) {
if(root.data > 0) {
root.data = 2 * root.data;
}
doublePositives(root.left);
doublePositives(root.right);
}