Reland "gpu: Fix Vivante's "hisilicon" GPUs"

Reland "gpu: Fix Vivante's "hisilicon" GPUs"

This fixes "Hisilicon" GPUs, which is an instance of a
Vivante's GPU design. This involves enabling virtual
contexts, since they don't support share-groups, and further
adds a work-around for switching surfaces. Without the work-
around the view surface "inherits" the size of the last
bound surface (which for Chrome tends to be a 1x1 pbuffer)
resulting in a black screen. The following steps "repair"
the view surface every time it is made current:

- Make it current.
- Bind the default frame-buffer.
- Make it not current.
- Destroy/recreate it from the native handle.
- Make it current again.

NOTE: The "Recreate" function is added to surface because
the Destroy/Initialize may be intercepted or modified by
wrapper surface classes. Recreate is similar to resize,
which after being forwarded by a wrapper can call
Destroy/Initialize on the 'real' surface.

BUG=179250
NOTRY=true

No-try, as all the relevant bots are already green.

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=195349

[epenner, 2013-04-19 03:54:24]

Ptal. New Patch

[epenner, 2013-04-19 04:08:39]

On 2013/04/19 03:54:24, epenner wrote:
> Ptal. New Patch

apatrick@: I refactored it partially the way you suggested, which is a lot
cleaner I think. But the code needs to stay in the context, as this shouldn't
occur for virtual context switches.

kareng@:  I've done it this time such that only new code/functions are added and
practically no code is changed. Then, just in case I've added #ifdefs around
where the new functions are actually called.

[epenner, 2013-04-19 04:13:14]

https://codereview.chromium.org/13852023/diff/1/gpu/command_buffer/service/fe...
File gpu/command_buffer/service/feature_info.cc (right):

https://codereview.chromium.org/13852023/diff/1/gpu/command_buffer/service/fe...
gpu/command_buffer/service/feature_info.cc:194: bool is_hisilicon = false;
The three lines in this file is the only new code that will execute on other
platforms. The rest of the changes is new code which is never called (the
call-sites are #ifdef'd)

[apatrick_chromium, 2013-04-19 18:38:44]

lgtm

[commit-bot: I haz the power, 2013-04-19 21:09:06]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/epenner@chromium.org/13852023/1

[commit-bot: I haz the power, 2013-04-20 00:25:56]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/epenner@chromium.org/13852023/1

[commit-bot: I haz the power, 2013-04-20 00:54:28]

Change committed as 195349