Revert of Separate out resource mailbox creation and fix synchronization issue.

Revert of Separate out resource mailbox creation and fix synchronization issue. (patchset #3 id:40001 of https://codereview.chromium.org/1666203003/ )

Reason for revert:
Causing renderer synchronization issues

Original issue's description:
> Separate out resource mailbox creation and fix synchronization issue.
>
> The resource transfer function has been split up so it does the lazy
> resource mailbox creation in a separate pass. This allows us to setup
> the sync token directly on the resource itself before the transfer.
>
> The texture binding code is also done on the first pass, this fixes
> a subtle synchronization issue where previously the bound textures
> could potentially not have a sync token attached to it if no mailboxes
> were lazily created.
>
> BUG=584381
> CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
>
> Committed: https://crrev.com/d5650cab68401bcd2bb89e1a98c65378d493f785
> Cr-Commit-Position: refs/heads/master@{#373847}

TBR=reveman@chromium.org,piman@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=584381, 584780
Committed: https://crrev.com/a14747b0409da6ca3700c03a0a4e5cb28de9f3b0
Cr-Commit-Position: refs/heads/master@{#373974}

[David Yen, 2016-02-06 00:41:10]

Created Revert of Separate out resource mailbox creation and fix synchronization
issue.

[David Yen, 2016-02-06 00:41:18]

Description was changed from

==========
Revert of Separate out resource mailbox creation and fix synchronization issue.
(patchset #3 id:40001 of https://codereview.chromium.org/1666203003/ )

Reason for revert:
Causing renderer synchronization issues

Original issue's description:
> Separate out resource mailbox creation and fix synchronization issue.
> 
> The resource transfer function has been split up so it does the lazy
> resource mailbox creation in a separate pass. This allows us to setup
> the sync token directly on the resource itself before the transfer.
> 
> The texture binding code is also done on the first pass, this fixes
> a subtle synchronization issue where previously the bound textures
> could potentially not have a sync token attached to it if no mailboxes
> were lazily created.
> 
> BUG=584381
> CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
> 
> Committed: https://crrev.com/d5650cab68401bcd2bb89e1a98c65378d493f785
> Cr-Commit-Position: refs/heads/master@{#373847}

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

to

==========
Revert of Separate out resource mailbox creation and fix synchronization issue.
(patchset #3 id:40001 of https://codereview.chromium.org/1666203003/ )

Reason for revert:
Causing renderer synchronization issues

Original issue's description:
> Separate out resource mailbox creation and fix synchronization issue.
> 
> The resource transfer function has been split up so it does the lazy
> resource mailbox creation in a separate pass. This allows us to setup
> the sync token directly on the resource itself before the transfer.
> 
> The texture binding code is also done on the first pass, this fixes
> a subtle synchronization issue where previously the bound textures
> could potentially not have a sync token attached to it if no mailboxes
> were lazily created.
> 
> BUG=584381
> CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
> 
> Committed: https://crrev.com/d5650cab68401bcd2bb89e1a98c65378d493f785
> Cr-Commit-Position: refs/heads/master@{#373847}

TBR=reveman@chromium.org,piman@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=584381, 584780
==========

[commit-bot: I haz the power, 2016-02-06 00:41:28]

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

[commit-bot: I haz the power, 2016-02-06 00:44:10]

Description was changed from

==========
Revert of Separate out resource mailbox creation and fix synchronization issue.
(patchset #3 id:40001 of https://codereview.chromium.org/1666203003/ )

Reason for revert:
Causing renderer synchronization issues

Original issue's description:
> Separate out resource mailbox creation and fix synchronization issue.
> 
> The resource transfer function has been split up so it does the lazy
> resource mailbox creation in a separate pass. This allows us to setup
> the sync token directly on the resource itself before the transfer.
> 
> The texture binding code is also done on the first pass, this fixes
> a subtle synchronization issue where previously the bound textures
> could potentially not have a sync token attached to it if no mailboxes
> were lazily created.
> 
> BUG=584381
> CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
> 
> Committed: https://crrev.com/d5650cab68401bcd2bb89e1a98c65378d493f785
> Cr-Commit-Position: refs/heads/master@{#373847}

TBR=reveman@chromium.org,piman@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=584381, 584780
==========

to

==========
Revert of Separate out resource mailbox creation and fix synchronization issue.
(patchset #3 id:40001 of https://codereview.chromium.org/1666203003/ )

Reason for revert:
Causing renderer synchronization issues

Original issue's description:
> Separate out resource mailbox creation and fix synchronization issue.
> 
> The resource transfer function has been split up so it does the lazy
> resource mailbox creation in a separate pass. This allows us to setup
> the sync token directly on the resource itself before the transfer.
> 
> The texture binding code is also done on the first pass, this fixes
> a subtle synchronization issue where previously the bound textures
> could potentially not have a sync token attached to it if no mailboxes
> were lazily created.
> 
> BUG=584381
> CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
> 
> Committed: https://crrev.com/d5650cab68401bcd2bb89e1a98c65378d493f785
> Cr-Commit-Position: refs/heads/master@{#373847}

TBR=reveman@chromium.org,piman@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=584381, 584780
==========

[commit-bot: I haz the power, 2016-02-06 00:44:11]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-02-06 00:45:29]

Description was changed from

==========
Revert of Separate out resource mailbox creation and fix synchronization issue.
(patchset #3 id:40001 of https://codereview.chromium.org/1666203003/ )

Reason for revert:
Causing renderer synchronization issues

Original issue's description:
> Separate out resource mailbox creation and fix synchronization issue.
> 
> The resource transfer function has been split up so it does the lazy
> resource mailbox creation in a separate pass. This allows us to setup
> the sync token directly on the resource itself before the transfer.
> 
> The texture binding code is also done on the first pass, this fixes
> a subtle synchronization issue where previously the bound textures
> could potentially not have a sync token attached to it if no mailboxes
> were lazily created.
> 
> BUG=584381
> CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
> 
> Committed: https://crrev.com/d5650cab68401bcd2bb89e1a98c65378d493f785
> Cr-Commit-Position: refs/heads/master@{#373847}

TBR=reveman@chromium.org,piman@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=584381, 584780
==========

to

==========
Revert of Separate out resource mailbox creation and fix synchronization issue.
(patchset #3 id:40001 of https://codereview.chromium.org/1666203003/ )

Reason for revert:
Causing renderer synchronization issues

Original issue's description:
> Separate out resource mailbox creation and fix synchronization issue.
>
> The resource transfer function has been split up so it does the lazy
> resource mailbox creation in a separate pass. This allows us to setup
> the sync token directly on the resource itself before the transfer.
>
> The texture binding code is also done on the first pass, this fixes
> a subtle synchronization issue where previously the bound textures
> could potentially not have a sync token attached to it if no mailboxes
> were lazily created.
>
> BUG=584381
> CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
>
> Committed: https://crrev.com/d5650cab68401bcd2bb89e1a98c65378d493f785
> Cr-Commit-Position: refs/heads/master@{#373847}

TBR=reveman@chromium.org,piman@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=584381, 584780

Committed: https://crrev.com/a14747b0409da6ca3700c03a0a4e5cb28de9f3b0
Cr-Commit-Position: refs/heads/master@{#373974}
==========

[commit-bot: I haz the power, 2016-02-06 00:45:30]

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

[David Yen, 2016-02-08 18:28:30]

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

The reason for reverting is: Trying to reland patch with fixes..