fix: retain user RPC timeout if set via withTimeout

[noahdietz]

If a Callable has RetrySettings and the calculated RPC Timeout for an attempt is less than the constant, RPC timeout provided by the user via ApiCallContext.withTimeout, the attempt settings will win. This is not caller friendly.

This PR changes AttemptCallable, used for Unary RPCs, only attempts to set the ApiCallContext.withTimeout when it is not already set by the caller.

This was fixed for ServerStreaming RPCs in #1155

Addresses commentary in #1144.

[codecov]

Codecov Report

Merging #1324 (c26cac6) into master (27d92c6) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##             master    #1324   +/-   ##
=========================================
  Coverage     79.65%   79.65%           
- Complexity     1248     1249    +1     
=========================================
  Files           209      209           
  Lines          5461     5461           
  Branches        464      464           
=========================================
  Hits           4350     4350           
  Misses          928      928           
  Partials        183      183           

Impacted Files	Coverage Δ	Complexity Δ
...n/java/com/google/api/gax/rpc/AttemptCallable.java	82.60% <100.00%> (ø)	5.00 <0.00> (+1.00)

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 27d92c6...c26cac6. Read the comment docs.

[noahdietz]

@elharo do you want to take another pass or am I OK to merge?

[elharo]

You don't need to wait for me. If it's approved, you can merge