[WebGL-chromium]Test to validate glinfo when webglcontext fails.

[Webgl-chromium] Test  to validate glinfo when  webglcontext fails.

Now we read the GPU(Vendor,Renderer,Driver info) from
chromium side and send it as part of status message when
EventTypeNames::webglcontextcreationerror is being sent.
This patch introduces the test for the above functionality.

forceNextWebGLContextCreationToFail(bool) is introduced
to fail the webgl context forcefully from JS-test runner
utility.

The below cl uses the test-runner code in this patch
https://codereview.chromium.org/706193003/

BUG=412400
Committed: https://crrev.com/3ccb614464d1ae6af89cbf02a028ef9f492df300
Cr-Commit-Position: refs/heads/master@{#304143}

[sivag, 2014-11-10 13:43:33]

siva.gunturi@samsung.com changed reviewers:
+ kbr@chromium.org

[sivag, 2014-11-10 13:45:31]

ptal..

[Ken Russell (switch to Gerrit), 2014-11-10 19:52:15]

Aside from this comment the change generally looks good. Please make this update
and re-upload both CLs.

Note I'm not an OWNER in this directory; you'll have to find an OWNER to review
this.

https://codereview.chromium.org/706193003/diff/20001/content/shell/renderer/t...
File content/shell/renderer/test_runner/test_runner.h (right):

https://codereview.chromium.org/706193003/diff/20001/content/shell/renderer/t...
content/shell/renderer/test_runner/test_runner.h:595: void
forceNextWebGLContextCreationToFail(bool);
The boolean argument isn't needed. The flag should auto-reset itself on the
Blink side. It's not desirable (or, really, feasible) for the test to have to
reset this flag to true.

[sivag, 2014-11-11 12:47:44]

siva.gunturi@samsung.com changed reviewers:
+ tkent@chromium.org

[sivag, 2014-11-11 12:50:57]

@tkent, ptal.

https://codereview.chromium.org/706193003/diff/20001/content/shell/renderer/t...
File content/shell/renderer/test_runner/test_runner.h (right):

https://codereview.chromium.org/706193003/diff/20001/content/shell/renderer/t...
content/shell/renderer/test_runner/test_runner.h:595: void
forceNextWebGLContextCreationToFail(bool);
On 2014/11/10 19:52:14, Ken Russell wrote:
> The boolean argument isn't needed. The flag should auto-reset itself on the
> Blink side. It's not desirable (or, really, feasible) for the test to have to
> reset this flag to true.

Done.

[tkent, 2014-11-12 00:17:44]

On 2014/11/11 12:50:57, Sikugu wrote:
> @tkent, ptal.

This and 703903002 depend on each other.  We can't land them safely.  You should
do either of the following steps:

1. Bink: Land a CL to implement WebView::forceNextWebGLContextCreationToFail
2. Chromium: Land this
3. Blink: Land the layout test

or

1. Blink: Land 703903002 with a failure expectation of the layout test
2. Chromium: Land this
3. Blink: Remove the failure expectation.

[tkent, 2014-11-12 00:19:18]

https://codereview.chromium.org/706193003/diff/40001/content/shell/renderer/t...
File content/shell/renderer/test_runner/test_runner.cc (right):

https://codereview.chromium.org/706193003/diff/40001/content/shell/renderer/t...
content/shell/renderer/test_runner/test_runner.cc:311: void
forceNextWebGLContextCreationToFail();
force -> Force

https://codereview.chromium.org/706193003/diff/40001/content/shell/renderer/t...
content/shell/renderer/test_runner/test_runner.cc:550:
&TestRunnerBindings::forceNextWebGLContextCreationToFail)
force -> Force

https://codereview.chromium.org/706193003/diff/40001/content/shell/renderer/t...
content/shell/renderer/test_runner/test_runner.cc:1455: void
TestRunnerBindings::forceNextWebGLContextCreationToFail() {
force -> Force

https://codereview.chromium.org/706193003/diff/40001/content/shell/renderer/t...
content/shell/renderer/test_runner/test_runner.cc:1457:
runner_->forceNextWebGLContextCreationToFail();
force -> Force

https://codereview.chromium.org/706193003/diff/40001/content/shell/renderer/t...
content/shell/renderer/test_runner/test_runner.cc:2857: void
TestRunner::forceNextWebGLContextCreationToFail() {
force -> Force

https://codereview.chromium.org/706193003/diff/40001/content/shell/renderer/t...
File content/shell/renderer/test_runner/test_runner.h (right):

https://codereview.chromium.org/706193003/diff/40001/content/shell/renderer/t...
content/shell/renderer/test_runner/test_runner.h:595: void
forceNextWebGLContextCreationToFail();
force -> Force

[sivag, 2014-11-12 18:47:21]

ptal..

https://codereview.chromium.org/706193003/diff/40001/content/shell/renderer/t...
File content/shell/renderer/test_runner/test_runner.cc (right):

https://codereview.chromium.org/706193003/diff/40001/content/shell/renderer/t...
content/shell/renderer/test_runner/test_runner.cc:311: void
forceNextWebGLContextCreationToFail();
On 2014/11/12 00:19:17, tkent wrote:
> force -> Force

Done.

https://codereview.chromium.org/706193003/diff/40001/content/shell/renderer/t...
content/shell/renderer/test_runner/test_runner.cc:550:
&TestRunnerBindings::forceNextWebGLContextCreationToFail)
On 2014/11/12 00:19:17, tkent wrote:
> force -> Force

Done.

https://codereview.chromium.org/706193003/diff/40001/content/shell/renderer/t...
content/shell/renderer/test_runner/test_runner.cc:1455: void
TestRunnerBindings::forceNextWebGLContextCreationToFail() {
On 2014/11/12 00:19:17, tkent wrote:
> force -> Force

Done.

https://codereview.chromium.org/706193003/diff/40001/content/shell/renderer/t...
content/shell/renderer/test_runner/test_runner.cc:1457:
runner_->forceNextWebGLContextCreationToFail();
On 2014/11/12 00:19:17, tkent wrote:
> force -> Force

Done.

https://codereview.chromium.org/706193003/diff/40001/content/shell/renderer/t...
content/shell/renderer/test_runner/test_runner.cc:2857: void
TestRunner::forceNextWebGLContextCreationToFail() {
On 2014/11/12 00:19:17, tkent wrote:
> force -> Force

Done.

https://codereview.chromium.org/706193003/diff/40001/content/shell/renderer/t...
File content/shell/renderer/test_runner/test_runner.h (right):

https://codereview.chromium.org/706193003/diff/40001/content/shell/renderer/t...
content/shell/renderer/test_runner/test_runner.h:595: void
forceNextWebGLContextCreationToFail();
On 2014/11/12 00:19:18, tkent wrote:
> force -> Force

Done.

[Ken Russell (switch to Gerrit), 2014-11-12 22:09:05]

LGTM - but note I'm not an OWNER.

[tkent, 2014-11-13 23:34:56]

The CQ bit was checked by tkent@chromium.org

[tkent, 2014-11-13 23:34:57]

lgtm

[commit-bot: I haz the power, 2014-11-13 23:39:07]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/706193003/60001

[commit-bot: I haz the power, 2014-11-14 01:55:26]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2014-11-14 01:57:34]

Patchset 4 (id:??) landed as
https://crrev.com/3ccb614464d1ae6af89cbf02a028ef9f492df300
Cr-Commit-Position: refs/heads/master@{#304143}