WebGL: Return both unprefixed and prefixed supported names from getSupportedExtensions

WebGL: Return both unprefixed and prefixed supported names from getSupportedExtensions

WebGL spec requires that getExtension "Returns an object if, and only if,
name is an ASCII case-insensitive match [HTML] for one of the names
returned from getSupportedExtensions; otherwise, returns null." Respect
this requirement by always returning both unprefixed and prefixed names
from getSupportedExtensions in case they are supported by getExtension.

Functionality that is common to both getExtension and
getSupportedExtensions is refactored to separate helper functions so that
maintaining this becomes easier.

BUG=354454
TEST=WebGL conformance tests
R=bajones@chromium.org, kbr@chromium.org

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=170071

[bajones, 2014-03-21 21:35:11]

Non-owner LGTM.

This patch feels more complete than the version I submitted
(https://crrev.com/200033004/). I'd prefer to see it land. (Ken is OOO, so it
may take a few days to get the appropriate approval)

[oetuaho-nv, 2014-03-24 10:09:55]

On 2014/03/21 21:35:11, bajones wrote:
> Non-owner LGTM.
> 
> This patch feels more complete than the version I submitted
> (https://crrev.com/200033004/). I'd prefer to see it land. (Ken is OOO, so it
> may take a few days to get the appropriate approval)

Thanks, looks like both of us rushed to patch this. I will wait for owner
approval.

[Ken Russell (switch to Gerrit), 2014-03-25 18:21:11]

LGTM with one comment adjusted.

Let's move toward community approval of the draft extensions currently exposed
under prefixes, and toward removing all of Chrome's use of vendor prefixes for
previously-approved extensions.

https://codereview.chromium.org/205903004/diff/1/Source/core/html/canvas/WebG...
File Source/core/html/canvas/WebGLRenderingContextBase.h (right):

https://codereview.chromium.org/205903004/diff/1/Source/core/html/canvas/WebG...
Source/core/html/canvas/WebGLRenderingContextBase.h:538: EnabledDraftExtension  
        = 0x04, // Extension that is still in draft state, but has been
selectively enabled by default.
Please update this comment to say something like

"...selectively enabled by default under a prefix. Do not use this for enabling
new draft extensions; use the DraftExtension flag instead, and do not use vendor
prefixes."

[oetuaho-nv, 2014-03-26 10:55:38]

The CQ bit was checked by oetuaho@nvidia.com

[commit-bot: I haz the power, 2014-03-26 10:55:40]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/oetuaho@nvidia.com/205903004/40001

[commit-bot: I haz the power, 2014-03-26 12:05:12]

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

[commit-bot: I haz the power, 2014-03-26 12:05:12]

Try jobs failed on following builders:
  tryserver.blink on linux_blink_dbg

[bajones, 2014-03-26 16:08:22]

On 2014/03/26 12:05:12, I haz the power (commit-bot) wrote:
> Try jobs failed on following builders:
>   tryserver.blink on linux_blink_dbg

I'll land this manually. CQ has been hard to do a clean commit with recently.

[bajones, 2014-03-26 17:52:40]

Committed patchset #2 manually as r170071 (presubmit successful).

[Ken Russell (switch to Gerrit), 2014-03-27 20:30:23]

On 2014/03/26 16:08:22, bajones wrote:
> On 2014/03/26 12:05:12, I haz the power (commit-bot) wrote:
> > Try jobs failed on following builders:
> >   tryserver.blink on linux_blink_dbg
> 
> I'll land this manually. CQ has been hard to do a clean commit with recently.

Please notify the Chrome troopers about failures like this.