http issue 231: Remove Beta API deprecated in 1.15 [backwards incompatible]

https://code.google.com/p/google-http-java-client/issues/detail?id=231 https://codereview.appspot.com/9685050/

Author: Yaniv Inbar

clirr-ignored-differences.xml

@@ -1,8 +1,17 @@
 <?xml version="1.0" encoding="UTF-8"?>
 <!-- see http://mojo.codehaus.org/clirr-maven-plugin/examples/ignored-differences.html -->
-
 <differences>
+ <!-- 7002 (method removed): className, method -->
+ <difference>
+ <differenceType>7002</differenceType>
+ <className>com/google/api/client/http/protobuf/ProtoHttpContent</className>
+ <method>com.google.api.client.http.protobuf.ProtoHttpContent setType(java.lang.String)</method>
+ </difference>
  <!-- 8001 (Class removed): className -->
+ <difference>
+ <differenceType>8001</differenceType>
+ <className>com/google/api/client/testing/http/json/*</className>
+ </difference>
  <difference>
  <differenceType>8001</differenceType>
  <className>com/google/api/client/repackaged/**</className>

google-http-client-appengine/src/main/java/com/google/api/client/extensions/appengine/http/UrlFetchTransport.java

@@ -15,12 +15,10 @@
 package com.google.api.client.extensions.appengine.http;
 
 import com.google.api.client.http.HttpMethods;
-import com.google.api.client.http.HttpResponse;
 import com.google.api.client.http.HttpTransport;
 import com.google.api.client.util.Preconditions;
 import com.google.appengine.api.urlfetch.FetchOptions;
 import com.google.appengine.api.urlfetch.HTTPMethod;
-import com.google.appengine.api.urlfetch.HTTPResponse;
 
 import java.io.IOException;
 import java.net.HttpURLConnection;
@@ -46,13 +44,6 @@
  * is the recommended transport to use on App Engine.
  * </p>
  *
- * <p>
- * Upgrade warning: in prior version 1.14 duplicate response headers in
- * {@link HttpResponse#getHeaders()} were combined into a single string value using
- * {@link HTTPResponse#getHeaders()} but starting in version 1.15 such headers are now "uncombined"
- * into a list of strings using {@link HTTPResponse#getHeadersUncombined()}.
- * </p>
- *
  * @since 1.10
  * @author Yaniv Inbar
  */

google-http-client-protobuf/src/main/java/com/google/api/client/http/protobuf/ProtoHttpContent.java

@@ -33,12 +33,10 @@
  * </p>
  *
  * <pre>
- * <code>
  static HttpRequest buildPostRequest(
  HttpRequestFactory requestFactory, GenericUrl url, MessageLite message) throws IOException {
  return requestFactory.buildPostRequest(url, new ProtoHttpContent(message));
  }
- * </code>
  * </pre>
  *
  * <p>
@@ -72,19 +70,6 @@ public void writeTo(OutputStream out) throws IOException {
  out.flush();
  }
 
- /**
- * Sets the content type or {@code null} for none.
- *
- * <p>
- * Default value is {@link ProtocolBuffers#CONTENT_TYPE}.
- * </p>
- * @deprecated (scheduled to be removed in 1.16) Use {@link #setMediaType(HttpMediaType)} instead.
- */
- @Deprecated
- public ProtoHttpContent setType(String type) {
- return setMediaType(type == null ? null : new HttpMediaType(type));
- }
-
  /** Returns the message to serialize. */
  public final MessageLite getMessage() {
  return message;

google-http-client/src/main/java/com/google/api/client/http/BackOffPolicy.java

@@ -24,8 +24,8 @@
  *
  * @since 1.7
  * @author Ravi Mistry
- * @deprecated (scheduled to be removed in the 1.16). Use
- * {@link HttpBackOffUnsuccessfulResponseHandler} instead.
+ * @deprecated (scheduled to be removed in 1.17) Use {@link HttpBackOffUnsuccessfulResponseHandler}
+ * instead.
  */
 @Deprecated
 @Beta

google-http-client/src/main/java/com/google/api/client/http/ExponentialBackOffPolicy.java

@@ -75,9 +75,8 @@
  *
  * @since 1.7
  * @author Ravi Mistry
- * @deprecated (scheduled to be removed in the 1.16). Use
- * {@link HttpBackOffUnsuccessfulResponseHandler} with {@link ExponentialBackOff}
- * instead.
+ * @deprecated (scheduled to be removed in 1.17). Use {@link HttpBackOffUnsuccessfulResponseHandler}
+ * with {@link ExponentialBackOff} instead.
  */
 @Beta
 @Deprecated

google-http-client/src/main/java/com/google/api/client/http/GenericUrl.java

@@ -50,12 +50,6 @@
  * </p>
  *
  * <p>
- * Upgrade warning: in prior version 1.14 the user info part in the URL was dropped, but starting
- * with version 1.15 this URL class contains the user info, and it can be retrieved or set with
- * {@link #getUserInfo} and {@link #setUserInfo}.
- * </p>
- *
- * <p>
  * Implementation is not thread-safe.
  * </p>
  *

google-http-client/src/main/java/com/google/api/client/http/HttpRequest.java

@@ -300,7 +300,7 @@ public HttpRequest setEncoding(HttpEncoding encoding) {
  * Returns the {@link BackOffPolicy} to use between retry attempts or {@code null} for none.
  *
  * @since 1.7
- * @deprecated (scheduled to be removed in the 1.16).
+ * @deprecated (scheduled to be removed in 1.17).
  * {@link #setUnsuccessfulResponseHandler(HttpUnsuccessfulResponseHandler)} with a new
  * {@link HttpBackOffUnsuccessfulResponseHandler} instead.
  */
@@ -315,7 +315,7 @@ public BackOffPolicy getBackOffPolicy() {
  * Sets the {@link BackOffPolicy} to use between retry attempts or {@code null} for none.
  *
  * @since 1.7
- * @deprecated (scheduled to be removed in the 1.16). Use
+ * @deprecated (scheduled to be removed in 1.17). Use
  * {@link #setUnsuccessfulResponseHandler(HttpUnsuccessfulResponseHandler)} with a new
  * {@link HttpBackOffUnsuccessfulResponseHandler} instead.
  */
@@ -735,7 +735,7 @@ public HttpRequest setThrowExceptionOnExecuteError(boolean throwExceptionOnExecu
  * {@link LowLevelHttpRequest#execute()}.
  *
  * @since 1.9
- * @deprecated (scheduled to be removed in 1.16) Use
+ * @deprecated (scheduled to be removed in 1.17) Use
  * {@link #setIOExceptionHandler(HttpIOExceptionHandler)} instead.
  */
  @Deprecated
@@ -754,7 +754,7 @@ public boolean getRetryOnExecuteIOException() {
  * </p>
  *
  * @since 1.9
- * @deprecated (scheduled to be removed in 1.16) Use
+ * @deprecated (scheduled to be removed in 1.17) Use
  * {@link #setIOExceptionHandler(HttpIOExceptionHandler)} instead.
  */
  @Deprecated

google-http-client/src/main/java/com/google/api/client/http/LowLevelHttpResponse.java

@@ -54,14 +54,7 @@ public abstract class LowLevelHttpResponse {
  /** Returns the response status line or {@code null} for none. */
  public abstract String getStatusLine() throws IOException;
 
- /**
- * Returns the response status code or {@code <=0} for none.
- *
- * <p>
- * Upgrade warning: in prior version 1.14 it could not return a negative number, but starting with
- * version 1.15 it may.
- * </p>
- */
+ /** Returns the response status code or {@code <=0} for none. */
  public abstract int getStatusCode() throws IOException;
 
  /** Returns the HTTP reason phrase or {@code null} for none. */

google-http-client/src/main/java/com/google/api/client/http/javanet/NetHttpTransport.java

@@ -53,11 +53,6 @@
  * </p>
  *
  * <p>
- * Upgrade warning: in prior version 1.14 caching was always disabled, but starting with version
- * 1.15 we honor the default global caching behavior.
- * </p>
- *
- * <p>
  * Implementation is thread-safe. For maximum efficiency, applications should use a single
  * globally-shared instance of the HTTP transport.
  * </p>