On linux when no VDMX table is present the leading is applied twice causing the line gap to be doubled.

When isVDMXValid is false, the following effectively happens:

descent = metrics.fDescent + metrics.fLeading;
lineGap = SkScalarToFloat(metrics.fLeading);
m_fontMetrics.setLineGap(lineGap);
m_fontMetrics.setLineSpacing(ascent + descent + lineGap);

Note that since descent already incorporates the fLeading, the fLeading will be applied twice in the total height. When isVDMXValid is true, the descent does not incorporate the fLeading.

BUG=265268

Review URL: https://chromiumcodereview.appspot.com/21259002

git-svn-id: svn://svn.chromium.org/blink/trunk@156936 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Rebaseline Skia suppressions.

Missed a couple in r156926.

TBR=robertphillips@chromium.org
BUG=263331,280292

Review URL: https://codereview.chromium.org/23786002

git-svn-id: svn://svn.chromium.org/blink/trunk@156937 bbb929c8-8fbe-4397-9dbb-9b2b20218538