Adding GetTableAction method to Blink to handle color table tags

Adding GetTableAction method to Blink to handle color table tags

Need OTS update, so that this change works correctly and OTS is able to parse Color-Emoji font. 

Related OTS patch: https://codereview.chromium.org/966703002/

For compile and landing, this patch does not depend on above OTS patch.
This patch only depends on OTS patch for proper functioning, 
so that uses defined fonts with color table are parsed correctly.

Checked on Android, after this and OTS patch combined font (with color table) parsing is working fine.

BUG=339857, 97467
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=191612

[h.joshi, 2015-03-03 12:03:00]

h.joshi@samsung.com changed reviewers:
+ behdad@chromium.org, eae@chromium.org

[h.joshi, 2015-03-03 12:03:55]

Pls review.
On Windows bots, every times its failing in different cases.

[behdad_google, 2015-03-03 17:30:11]

behdad@google.com changed reviewers:
+ behdad@google.com

[behdad_google, 2015-03-03 17:30:11]

lgtm

May be better to copy HB_TAG() and rename it.  I actually filed that against OTS
itself:

https://github.com/khaledhosny/ots/issues/50

[eae, 2015-03-03 18:08:01]

Can we add a test for this?

[h.joshi, 2015-03-04 06:17:48]

On 2015/03/03 18:08:01, eae wrote:
> Can we add a test for this?

[h.joshi, 2015-03-04 06:19:32]

@Behdad: Made suggested changes.

@Emil:
Facing following two problems for test case:
1. On Linux, Skia returns NULL Typeface after OTS has successfully passed Color
table font, due to which Emoji is not drawn for Web user font. 
   Seems to be related to FontConfig, as similar bug is filed for FontConfig and
Emoji.
   On Android its working fine and able to see Emoji correctlty drawn from Web
user font.
2. Tried to "pyftsubset.py" on NotoColorEmoji.ttf, seems "pyftsubset.py" does
not work well for Font with Color Table.
   "NotoColorEmoji.ttf" is 3.1MB, so uploading complete font will not be good
idea.

[h.joshi, 2015-03-04 07:21:41]

On my machine, Freetype call "FT_Open_Face" is returning NULL when
NotoColorEmoji is used as 
user define web font.
I have one question, For Chrome on Linux (Ubuntu), Do we use system FreeType?

[behdad_google, 2015-03-04 18:56:16]

On 2015/03/04 07:21:41, h.joshi wrote:
> On my machine, Freetype call "FT_Open_Face" is returning NULL when
> NotoColorEmoji is used as 
> user define web font.
> I have one question, For Chrome on Linux (Ubuntu), Do we use system FreeType?

Sounds like it.  And if it's too old, it won't recognize NotoColorEmoji.

[h.joshi, 2015-03-09 05:15:25]

Pls review.
Layout test is added, for now only on Android correct rendering is shown.
FreeType version 
on Ubuntu looks to be in Old version and not supporting color table.

[eae, 2015-03-09 22:59:57]

LGTM, thank you!

[eae, 2015-03-09 23:00:01]

The CQ bit was checked by eae@chromium.org

[eae, 2015-03-09 23:00:01]

The patchset sent to the CQ was uploaded after l-g-t-m from behdad@google.com
Link to the patchset: https://codereview.chromium.org/954953002/#ps100001
(title: "Updating TestExpectations")

[commit-bot: I haz the power, 2015-03-09 23:00:52]

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

[commit-bot: I haz the power, 2015-03-09 23:05:25]

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

[commit-bot: I haz the power, 2015-03-09 23:05:27]

Try jobs failed on following builders:
  android_blink_compile_dbg on tryserver.blink (JOB_FAILED,
http://build.chromium.org/p/tryserver.blink/builders/android_blink_compile_db...)

[h.joshi, 2015-03-10 04:34:55]

The CQ bit was checked by h.joshi@samsung.com

[h.joshi, 2015-03-10 04:34:56]

The patchset sent to the CQ was uploaded after l-g-t-m from behdad@google.com,
eae@chromium.org
Link to the patchset: https://codereview.chromium.org/954953002/#ps120001
(title: "Rebased Test expecttaion file")

[commit-bot: I haz the power, 2015-03-10 04:35:12]

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

[commit-bot: I haz the power, 2015-03-10 07:56:53]

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

[commit-bot: I haz the power, 2015-03-10 07:56:55]

Try jobs failed on following builders:
  win_blink_rel on tryserver.blink (JOB_FAILED,
http://build.chromium.org/p/tryserver.blink/builders/win_blink_rel/builds/54189)

[h.joshi, 2015-03-10 09:02:29]

The CQ bit was checked by h.joshi@samsung.com

[h.joshi, 2015-03-10 09:02:29]

The patchset sent to the CQ was uploaded after l-g-t-m from behdad@google.com,
eae@chromium.org
Link to the patchset: https://codereview.chromium.org/954953002/#ps140001
(title: "Updating Test Expectation for Virtual test")

[commit-bot: I haz the power, 2015-03-10 09:03:19]

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

[commit-bot: I haz the power, 2015-03-10 10:34:22]

Committed patchset #8 (id:140001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=191612