Update actions/upload-artifact to v4

Add libjpeg-turbo header files in the distribution

We need src/image/*.h (SkSurface_Raster.h mainly) in the artifacts

Add build instructions, GitHub actions + fix build flags

Include args.gn file (to detect if -stdlib=libc++ was used to compile…

… Skia)

Add build instructions, GitHub actions + fix build flags

Add build instructions & GitHub actions

Fix clang error using implicit tuples ctors with an old stdlibc++ ver…

…sion

Add GitHub actions to build Skia

Fix int16_t for glyphs, const glyphs, and clarify glyph loop.

Several places in the PDF code are using int16_t for glyphs. With
newer NotoSans fonts, all possible glyph ids are being used, so this
can lead to problems.

The PDF glyphs from text code returns the text for the glyphs if the
encoding is for glyphs. However, it returns this using an unsafe const
cast which is hiding possible bugs and preventing correct use of const
in other places.

The way the glyph loop in SkPDFDevice::drawPosText is written uses a
'--i' in the loop, which makes it appear this can loop forever. I don't
believe it can, but it is an unecessary code folding. We should also at
least assert the forward progress correctness here.

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

Cherry-pick: 22edc83
Approval: https://code.google.com/p/chromium/issues/detail?id=418939#c30