Revert of [vr] Ensure that textured elements never draw with a dirty texture

Revert of [vr] Ensure that textured elements never draw with a dirty texture (patchset #2 id:20001 of https://codereview.chromium.org/2969023003/ )

Reason for revert:
Breaks some VR browser navigation tests due to failing DCHECK https://build.chromium.org/p/chromium.fyi/builders/Android%20VR%20Tests?numbuilds=200

Original issue's description:
> [vr] Ensure that textured elements never draw with a dirty texture
> 
> This should never be possible, so I've made UpdateTexture a part
> of OnBeginFrame. It will usually early out (because the texture won't
> be dirty), but will ensure that we never again draw with a dirty
> texture. I've also added a DCHECK to ensure that we don't change this.
> 
> NB: this has no impact on the the update throttling done by the URL
> bar since it has its own overridden implementation of OnBeginFrame.
> 
> BUG=None
> 
> Review-Url: https://codereview.chromium.org/2969023003
> Cr-Commit-Position: refs/heads/master@{#484142}
> Committed: https://chromium.googlesource.com/chromium/src/+/286a6b8e56a7bd0b7ac58d8b9ba933e0c3dc2405

TBR=cjgrant@chromium.org,vollick@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=None

Review-Url: https://codereview.chromium.org/2972843002
Cr-Commit-Position: refs/heads/master@{#484300}
Committed: https://chromium.googlesource.com/chromium/src/+/de1d1d17c1eb00d0c618d43a477411d93a42c58e

[bsheedy, 2017-07-05 17:29:42]

The CQ bit was checked by bsheedy@chromium.org

[bsheedy, 2017-07-05 17:29:43]

Created Revert of [vr] Ensure that textured elements never draw with a dirty
texture

[commit-bot: I haz the power, 2017-07-05 17:30:00]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-07-05 17:30:01]

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

[commit-bot: I haz the power, 2017-07-05 17:30:02]

No L-G-T-M from a valid reviewer yet.
CQ run can only be started once the patch has received an L-G-T-M from a full
committer.
Even if an L-G-T-M may have been provided, it was from a non-committer,_not_ a
full super star committer.
Committers are members of the group "project-chromium-committers".
Note that this has nothing to do with OWNERS files.

[mthiesse, 2017-07-05 17:34:31]

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

[mthiesse, 2017-07-05 17:34:31]

lgtm

[bsheedy, 2017-07-05 17:34:55]

+mthiesse since he seems to be available ATM

[bsheedy, 2017-07-05 17:35:16]

The CQ bit was checked by bsheedy@chromium.org

[commit-bot: I haz the power, 2017-07-05 17:35:43]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-07-05 17:36:05]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1499276116565630, "parent_rev":
"d4945f6278ba6d1c93d5fa8afd8b08d0a59cbfd3", "commit_rev":
"de1d1d17c1eb00d0c618d43a477411d93a42c58e"}

[commit-bot: I haz the power, 2017-07-05 17:36:21]

Description was changed from

==========
Revert of [vr] Ensure that textured elements never draw with a dirty texture
(patchset #2 id:20001 of https://codereview.chromium.org/2969023003/ )

Reason for revert:
Breaks some VR browser navigation tests due to failing DCHECK
https://build.chromium.org/p/chromium.fyi/builders/Android%20VR%20Tests?numbu...

Original issue's description:
> [vr] Ensure that textured elements never draw with a dirty texture
> 
> This should never be possible, so I've made UpdateTexture a part
> of OnBeginFrame. It will usually early out (because the texture won't
> be dirty), but will ensure that we never again draw with a dirty
> texture. I've also added a DCHECK to ensure that we don't change this.
> 
> NB: this has no impact on the the update throttling done by the URL
> bar since it has its own overridden implementation of OnBeginFrame.
> 
> BUG=None
> 
> Review-Url: https://codereview.chromium.org/2969023003
> Cr-Commit-Position: refs/heads/master@{#484142}
> Committed:
https://chromium.googlesource.com/chromium/src/+/286a6b8e56a7bd0b7ac58d8b9ba9...

TBR=cjgrant@chromium.org,vollick@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=None
==========

to

==========
Revert of [vr] Ensure that textured elements never draw with a dirty texture
(patchset #2 id:20001 of https://codereview.chromium.org/2969023003/ )

Reason for revert:
Breaks some VR browser navigation tests due to failing DCHECK
https://build.chromium.org/p/chromium.fyi/builders/Android%20VR%20Tests?numbu...

Original issue's description:
> [vr] Ensure that textured elements never draw with a dirty texture
> 
> This should never be possible, so I've made UpdateTexture a part
> of OnBeginFrame. It will usually early out (because the texture won't
> be dirty), but will ensure that we never again draw with a dirty
> texture. I've also added a DCHECK to ensure that we don't change this.
> 
> NB: this has no impact on the the update throttling done by the URL
> bar since it has its own overridden implementation of OnBeginFrame.
> 
> BUG=None
> 
> Review-Url: https://codereview.chromium.org/2969023003
> Cr-Commit-Position: refs/heads/master@{#484142}
> Committed:
https://chromium.googlesource.com/chromium/src/+/286a6b8e56a7bd0b7ac58d8b9ba9...

TBR=cjgrant@chromium.org,vollick@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=None

Review-Url: https://codereview.chromium.org/2972843002
Cr-Commit-Position: refs/heads/master@{#484300}
Committed:
https://chromium.googlesource.com/chromium/src/+/de1d1d17c1eb00d0c618d43a4774...
==========

[commit-bot: I haz the power, 2017-07-05 17:36:22]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/de1d1d17c1eb00d0c618d43a4774...

[Ian Vollick, 2017-07-07 17:25:16]

A revert of this CL (patchset #1 id:1) has been created in
https://codereview.chromium.org/2965413002/ by vollick@chromium.org.

The reason for reverting is: These failures seem to persist on ToT..