Prevent preliminary blacklisted features from being used too early

Prevent preliminary blacklisted features from being used too early

EnableSwiftShaderIfNecessary() was calling GpuAccessAllowed()
before UpdatePreliminaryBlacklistedFeatures() was called, causing
GpuAccessAllowed() to return false because of the uninitialized
preliminary blacklisted features list. This simply adds a check
that prevents using uninitialized data.

BUG=697998
Review-Url: https://codereview.chromium.org/2727273003
Cr-Commit-Position: refs/heads/master@{#454432}
Committed: https://chromium.googlesource.com/chromium/src/+/2df232659f116970f80a6d492ec5945173690e2c

[sugoi1, 2017-03-02 19:55:41]

Description was changed from

==========
Prevent preliminary blacklisted features from being used too early

EnableSwiftShaderIfNecessary() was calling GpuAccessAllowed()
before UpdatePreliminaryBlacklistedFeatures() was called, causing
GpuAccessAllowed() to return false because of the uninitialized
preliminary blacklisted features list. This simply adds a check
that prevents using uninitialized data.

BUG=697998
==========

to

==========
Prevent preliminary blacklisted features from being used too early

EnableSwiftShaderIfNecessary() was calling GpuAccessAllowed()
before UpdatePreliminaryBlacklistedFeatures() was called, causing
GpuAccessAllowed() to return false because of the uninitialized
preliminary blacklisted features list. This simply adds a check
that prevents using uninitialized data.

BUG=697998
==========

[sugoi1, 2017-03-02 19:55:41]

sugoi@chromium.org changed reviewers:
+ jbauman@chromium.org, zmo@chromium.org

[sugoi1, 2017-03-02 19:56:12]

It looks like this fix works. Thanks zmo@!

[sugoi1, 2017-03-02 20:13:58]

The CQ bit was checked by sugoi@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-03-02 20:14:47]

Dry run: 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-03-02 21:34:20]

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

[commit-bot: I haz the power, 2017-03-02 21:34:21]

Dry run: This issue passed the CQ dry run.

[jbauman, 2017-03-02 22:24:00]

lgtm

[sugoi1, 2017-03-02 23:23:39]

The CQ bit was checked by sugoi@chromium.org

[commit-bot: I haz the power, 2017-03-02 23:24:17]

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-03-02 23:31:18]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1488497019491630, "parent_rev":
"5e6a1567a4353cd1b32ed70e7fe887450bacc2d3", "commit_rev":
"2df232659f116970f80a6d492ec5945173690e2c"}

[commit-bot: I haz the power, 2017-03-02 23:33:32]

Description was changed from

==========
Prevent preliminary blacklisted features from being used too early

EnableSwiftShaderIfNecessary() was calling GpuAccessAllowed()
before UpdatePreliminaryBlacklistedFeatures() was called, causing
GpuAccessAllowed() to return false because of the uninitialized
preliminary blacklisted features list. This simply adds a check
that prevents using uninitialized data.

BUG=697998
==========

to

==========
Prevent preliminary blacklisted features from being used too early

EnableSwiftShaderIfNecessary() was calling GpuAccessAllowed()
before UpdatePreliminaryBlacklistedFeatures() was called, causing
GpuAccessAllowed() to return false because of the uninitialized
preliminary blacklisted features list. This simply adds a check
that prevents using uninitialized data.

BUG=697998

Review-Url: https://codereview.chromium.org/2727273003
Cr-Commit-Position: refs/heads/master@{#454432}
Committed:
https://chromium.googlesource.com/chromium/src/+/2df232659f116970f80a6d492ec5...
==========

[commit-bot: I haz the power, 2017-03-02 23:33:33]

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