Revert of gpu: Make glTexSubImage2D work with GL_SRGB_ALPHA on OpenGL

Revert of gpu: Make glTexSubImage2D work with GL_SRGB_ALPHA on OpenGL (patchset #2 id:20001 of https://codereview.chromium.org/1426903002/ )

Reason for revert:
Caused Android test failure (GLEXTSRGBTest.TexImageSRGBALPHAFormat).

Original issue's description:
> gpu: Make glTexSubImage2D work with GL_SRGB_ALPHA on OpenGL
>
> Make glTexSubImage2D work when client passes GL_SRGB_ALPHA as the
> format when running on OpenGL.
>
> Considering GL_SRGB_ALPHA texture:
> In OpenGL ES, Tex*Image2D format must be GL_SRGB_ALPHA (same as texture
> internal format).
> In OpenGL, format must not be GL_SRGB_ALPHA.
>
> The format was handled correctly for glTexImage2D, but not for
> glTexSubImage2D. The actual fix is to apply
> TextureManager::AdjustTexFormat to glTexSubImage2D format.
>
> Moves the glTexSubImage2D code from decoder to texture manager, and
> tries to follow the structure of glTexImage2D. This is due to hoping to
> reduce the amount of similar inconsistency bugs. The glTexSubImage3D,
> CopySubTexture* and compressed call variants are still potential
> sources of inconsistencies. The code duplication between glTexImage2D
> and glTexSubImage2D is not addressed, either.
>
> Changes the texture size of
> Service/GLES2DecoderTest.TexSubImage2DBadArgs.  The check for behavior
> "!pixels -> kOutOfBounds" was moved before argument validation. In the
> test, there is a subtest that uses invalid type, GL_UNSIGNED_INT. This
> causes the texture data to be bigger than SHM buffer, the unexpected
> kOutOfBounds would be returned instead of normal GL error for invalid
> type.
>
> BUG=skia:2992
>
> Committed: https://crrev.com/60da545176aed90d91874a456da2bac8b822c67d
> Cr-Commit-Position: refs/heads/master@{#356787}

TBR=bajones@chromium.org,piman@chromium.org,kkinnunen@nvidia.com
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=skia:2992
Committed: https://crrev.com/48b95d23777b0b02d25c37803bcace4cf044cf80
Cr-Commit-Position: refs/heads/master@{#356882}

[wychen, 2015-10-29 17:35:53]

Created Revert of gpu: Make glTexSubImage2D work with GL_SRGB_ALPHA on OpenGL

[commit-bot: I haz the power, 2015-10-29 17:37:55]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1418113009/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1418113009/1

[commit-bot: I haz the power, 2015-10-29 17:37:55]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-10-29 17:37:56]

No L-G-T-M from a valid reviewer yet. Only full committers are accepted.
Even if an L-G-T-M may have been provided, it was from a non-committer,
_not_ a full super star committer.
See http://www.chromium.org/getting-involved/become-a-committer
Note that this has nothing to do with OWNERS files.

[mdjones, 2015-10-29 17:38:44]

mdjones@chromium.org changed reviewers:
+ mdjones@chromium.org

[mdjones, 2015-10-29 17:38:45]

The CQ bit was checked by mdjones@chromium.org

[mdjones, 2015-10-29 17:38:45]

lgtm

[bajones, 2015-10-29 17:39:43]

On 2015/10/29 17:38:45, mdjones wrote:
> lgtm

LGTM.

[commit-bot: I haz the power, 2015-10-29 17:40:26]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1418113009/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1418113009/1

[commit-bot: I haz the power, 2015-10-29 17:42:57]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-10-29 17:44:33]

Patchset 1 (id:??) landed as
https://crrev.com/48b95d23777b0b02d25c37803bcace4cf044cf80
Cr-Commit-Position: refs/heads/master@{#356882}