Removing primaryFontHasGlyphForCharacter method

Removing primaryFontHasGlyphForCharacter method

Method primaryFontHasGlyphForCharacter was using duplicate code already
 present in SimpleFontData and has single caller.

Done as per comments in https://codereview.chromium.org/870523003/#msg15

BUG=

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

[h.joshi, 2015-05-19 18:53:21]

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

[h.joshi, 2015-05-19 18:53:22]

Pls review

[behdad_google, 2015-05-19 19:14:35]

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

[behdad_google, 2015-05-19 19:14:35]

lgtm

[h.joshi, 2015-05-20 01:43:42]

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

[commit-bot: I haz the power, 2015-05-20 01:44:34]

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

[commit-bot: I haz the power, 2015-05-20 01:44:37]

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

[commit-bot: I haz the power, 2015-05-20 01:44:41]

No LGTM from a valid reviewer yet. Only full committers are accepted.
Even if an LGTM may have been provided, it was from a non-committer,
_not_ a full super star committer.
See http://www.chromium.org/getting-involved/become-a-committer
Note that this has nothing to do with OWNERS files.

[h.joshi, 2015-05-20 02:01:50]

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

[h.joshi, 2015-05-20 02:01:51]

@Emil: Pls review

[pdr., 2015-05-20 03:00:06]

On 2015/05/20 at 02:01:51, h.joshi wrote:
> @Emil: Pls review

LGTM

It looks like there is only a single caller? We may want to consider removing
this function entirely in favor of just using
primaryFont()->glyphForCharacter(character).

[h.joshi, 2015-05-20 05:24:34]

@pdr: Thank you,
Removed method primaryFontHasGlyphForCharacter entirely as it is used in one
place only.

Pls review.

[pdr., 2015-05-20 05:27:43]

pdr@chromium.org changed reviewers:
+ pdr@chromium.org

[pdr., 2015-05-20 05:27:43]

The CQ bit was checked by pdr@chromium.org

[pdr., 2015-05-20 05:27:44]

lgtm

[pdr., 2015-05-20 05:27:44]

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/1130913007/#ps20001
(title: "Comment fixes")

[commit-bot: I haz the power, 2015-05-20 05:27:51]

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

[commit-bot: I haz the power, 2015-05-20 05:30:53]

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

[commit-bot: I haz the power, 2015-05-20 05:30:54]

Try jobs failed on following builders:
  mac_blink_compile_dbg on tryserver.blink (JOB_FAILED,
http://build.chromium.org/p/tryserver.blink/builders/mac_blink_compile_dbg/bu...)
  mac_blink_rel on tryserver.blink (JOB_FAILED,
http://build.chromium.org/p/tryserver.blink/builders/mac_blink_rel/builds/55763)

[h.joshi, 2015-05-20 05:34:49]

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

[h.joshi, 2015-05-20 05:34:49]

The patchset sent to the CQ was uploaded after l-g-t-m from behdad@google.com,
pdr@chromium.org
Link to the patchset: https://codereview.chromium.org/1130913007/#ps40001
(title: "Rebase patch")

[commit-bot: I haz the power, 2015-05-20 05:35:15]

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

[commit-bot: I haz the power, 2015-05-20 05:49:20]

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

[commit-bot: I haz the power, 2015-05-20 05:49:21]

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

[h.joshi, 2015-05-20 05:50:35]

Patchset #3 (id:40001) has been deleted

[h.joshi, 2015-05-20 05:50:52]

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

[h.joshi, 2015-05-20 05:50:53]

The patchset sent to the CQ was uploaded after l-g-t-m from behdad@google.com,
pdr@chromium.org
Link to the patchset: https://codereview.chromium.org/1130913007/#ps60001
(title: "Rebase patch")

[commit-bot: I haz the power, 2015-05-20 05:51:01]

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

[commit-bot: I haz the power, 2015-05-20 08:01:07]

Committed patchset #3 (id:60001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=195598