[TF:XLA] Change error type in XLA for uninitialized ResourceVariable.

This is to match TensorFlow Classic's implementation and matches the actual error condition more closely.

PiperOrigin-RevId: 231250467

Author: tensorflower-gardener

tensorflow/compiler/tests/variable_ops_test.py

@@ -485,7 +485,7 @@ def testSliceAssign(self):
       checker2[None] = [6]  # new axis
 
   def testUninitialized(self):
-    with self.assertRaisesRegexp(errors.InvalidArgumentError,
+    with self.assertRaisesRegexp(errors.FailedPreconditionError,
                                  "uninitialized variable"):
       with self.test_session() as sess, self.test_scope():
         v = resource_variable_ops.ResourceVariable([1, 2])

tensorflow/compiler/tf2xla/xla_op_kernel.cc

@@ -354,8 +354,8 @@ Status ReadVariableInputTensor(const Tensor& tensor, DataType type,
   TF_RET_CHECK(variable != nullptr);
   TF_RET_CHECK(variable->kind() == XlaResource::kVariable);
   if (!variable->initialized()) {
-    return errors::InvalidArgument("Read of uninitialized variable ",
-                                   variable->name());
+    return errors::FailedPrecondition("Read of uninitialized variable ",
+                                      variable->name());
   }
   if (variable->type() != type) {
     return errors::InvalidArgument(

tensorflow/python/kernel_tests/resource_variable_ops_test.py

@@ -745,7 +745,6 @@ def testAssignSubMethod(self):
 
   @test_util.run_in_graph_and_eager_modes
   @test_util.run_v1_only("b/120545219")
-  @test_util.disable_xla("This test never passed for XLA")
   def testDestroyResource(self):
     v = resource_variable_ops.ResourceVariable(3.0, name="var0")
     self.evaluate(variables.global_variables_initializer())