Bail out and assert if the glyph to character map is missing

Change HarfBuzzShaper::shapeHarfBuzzRuns to fail early and assert
(instead of crashing) if the runs m_glyphToCharacterIndexes is
unpopulated.
This is a stop-gap measure until we can track down the root cause,
see bug for details.

BUG=337886
R=dglazkov@chromium.org

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

git-svn-id: svn://svn.chromium.org/blink/trunk@166096 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Break updateSquashingLayerGeometry out of updateGraphicsLayerGeometry

A big block of code which probably works better as a helper function.

BUG=338896

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

git-svn-id: svn://svn.chromium.org/blink/trunk@166097 bbb929c8-8fbe-4397-9dbb-9b2b20218538
[import] Handle 40x response as an error.

Resource invokes its client in slightly different ways for
file:// and http:// URLs: It deson't call responseReceived
for non-existent local URLs. This difference caused crbug.com/336609.

This change adds a check against status code in responseReceived() so that
the loader doesn't create document for error response.
The client of RawResource needs to handle these low level details.

TEST=import-onerror.html,import-onload.html
BUG=336609
R=dglazkov@chromium.org, dominicc@chromium.org

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

git-svn-id: svn://svn.chromium.org/blink/trunk@166098 bbb929c8-8fbe-4397-9dbb-9b2b20218538
[XHR] Move the Cache-Control test from /inspector to /xmlhttprequest.

Move a test that tests that XHR respects Cache-Control request header
from LayoutTests/http/tests/inspector to LayoutTests/http/tests/xmlhttprequest.

BUG=333804
R=tyoshino

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

git-svn-id: svn://svn.chromium.org/blink/trunk@166099 bbb929c8-8fbe-4397-9dbb-9b2b20218538
IDL compiler: [Custom] special operations

* A few minor Perl changes to regularize include handling
　(so we don't need to special-case [Custom] in Python)
　We did this before for [Custom] attributes; this results in changing 3 bindings: adding V8Node.h to 3, and GetPtr.h/RefPtr.h to 1.
* Reusing existing test case, with 2 minor features commented out;
　will uncomment when implement in follow-up.

BUG=239771
R=haraken

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

git-svn-id: svn://svn.chromium.org/blink/trunk@166100 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Pass around CSSSelector by reference instead of pointer

Pass around CSSSelector by reference instead of pointer as it is expected
to be non-null. This makes the code look safer and gets rid of unnecessary
null-checks.

R=adamk

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

git-svn-id: svn://svn.chromium.org/blink/trunk@166101 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Remove unused keyNeeded() interface

Original (https://codereview.chromium.org/129793008/) broke the Android build,
but that has now been updated (https://codereview.chromium.org/146693003/).
Test from the original CL is being updated seperately.

BUG=224786
TEST=EME content tests pass

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

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