Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 7a112151 authored by Steve Block's avatar Steve Block
Browse files

Remove some superfluous null-checks when constructing a CacheResult 

Change-Id: I57f0283872b788f84fef1667f79c37f4d92a5868
parent 67ba204a

core/java/android/webkit/CacheManager.java

+6 −11
Original line number Diff line number Diff line
@@ -734,8 +734,7 @@ public final class CacheManager { 
        CacheResult ret = new CacheResult();

        ret.httpStatusCode = statusCode;



        String location = headers.getLocation();

        if (location != null) ret.location = location;

        ret.location = headers.getLocation();



        ret.expires = -1;

        ret.expiresString = headers.getExpires();
@@ -754,15 +753,9 @@ public final class CacheManager { 
            }

        }



        String contentDisposition = headers.getContentDisposition();

        if (contentDisposition != null) {

            ret.contentdisposition = contentDisposition;

        }

        ret.contentdisposition = headers.getContentDisposition();



        String crossDomain = headers.getXPermittedCrossDomainPolicies();

        if (crossDomain != null) {

            ret.crossDomain = crossDomain;

        }

        ret.crossDomain = headers.getXPermittedCrossDomainPolicies();



        // lastModified and etag may be set back to http header. So they can't

        // be empty string.
@@ -772,7 +765,9 @@ public final class CacheManager { 
        }



        String etag = headers.getEtag();

        if (etag != null && etag.length() > 0) ret.etag = etag;

        if (etag != null && etag.length() > 0) {

            ret.etag = etag;

        }



        String cacheControl = headers.getCacheControl();

        if (cacheControl != null) {