GitLab

If we fire internalAbort() after handleRequestError(), a new request
initiated by the event handler reusing the same XHR instance will be
aborted.

BUG=402855

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

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

The function CSSParserValue::createCSSValue is only actually used by
calc parsing, although we can more easily do the same thing there in
much less code.

BUG=404023

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

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

This is patch 1 of 6 in the roll.

TBR=phoglund@chromium.org
NOTRY=1
BUG=404022

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

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

The indirection of parseWebkitTransformOrigin only adds complexity.

BUG=404023

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

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

When we resolve style for an element with an explicit font size, we build a new
FontDescription but we inherit the text autosizing multiplier from the parent's
RenderStyle (see http://crbug.com/380903).

We need to respect this multiplier when we initialize the computed size in the
FontDescription, or else they are out of sync with each other.  They remain out
of sync even after we layout the element, because TextAutosizer sees that the
element already has the correct multiplier and doesn't touch it.

BUG=403132

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

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

This patch fixes a bug that frame's load event does not fire when
FontLoader::loadPendingFonts does not start any load because all the
pending fonts are already started loading. This happens when a webfont
is used in multiple iframes; only one of them actually starts the load.

After this patch, if a font is used in multiple frames, subsequent
frames do not schedule a load (hence do not block the load event).

BUG=401846
TEST=fast/css/font-face-iframe-onload.html

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

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

BUG=402536,352134

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

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

Navigation transitions: Hide transition elements by setting opacity: 0 rather than display: none, to not shift other elements around.

Also removed a redundant parameter from Internals::serializeNavigationMarkup.

R=abarth
BUG=370696

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

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

This enables ServiceWorkerRegistration interface and updates layout tests
to work with it.

NOTE: ServiceWorker feature is still behind a flag, so this doesn't break
any real web applications.

[Spec] https://slightlyoff.github.io/ServiceWorker/spec/service_worker/index.html#service-worker-registration-obj


BUG=396400
TEST=run_webkit_tests.py --debug http/tests/serviceworker

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

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

We need to add "platform" to public/web/DEPS in order to include
platform/text/TextDirection.h in WebTextDirection.h.
This change should have no problems because we already have "core"
dependency, and "core" depends on "platform."

TEST=none; no behavior changes.
BUG=

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

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

This reverts commit 51fe8a513278ca949b5848bce3495f69a67072e2.

Reason for revert:
Broke iframe repainting: https://code.google.com/p/chromium/issues/detail?id=403734.

Original issue's description:
> Re-land of: Allow paint invalidation containers to cross frame boundaries.
>
> The previous version had an infinite loop bug in RenderObject when there are two nested frames.
>
> Previously, the *actual* paint invalidation container, meaning the enclosing
> compositing layer / root RenderView, could already have been across a frame boundary. The logic to do this correctly was done via special code in RenderView.
>
> Instead, generalize the existing mechanisms to find a paint invalidation container
> and map rects to repaint container coordinate space to cross frame boundaries.
> This simplifies the code, and also causes paint invalidation rects to always be stored
> in the coordinate space of their graphics layer backing. The latter is important if we want
> to use these rects for determining which parts of a graphics layer need to be painted.
>
> BUG=401156
>
> Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=179858

TBR=chrishtr@chromium.org
NOTREECHECKS=true
BUG=401156

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

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

Adds a category to ct-failure-group and enables ct-failure-stream to filter
to a single category.

Ports the previously broken ct-failure-stream-tests to mocha in order to
add new tests.

NOTRY=true
BUG=400397,399967, 400387

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

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

http://src.chromium.org/viewvc/blink?view=revision&revision=180305

TBR=eae@chromium.org

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

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

Use label="" instead of the textContent

R=ojan@chromium.org
NOTRY=true

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

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

The file handler was getting confused when we would upload a full_results.json
that contained a "results" key that pointed to a trie of tests.

Also fix some file permissions issues that the presubmit check complained about.

R=ojan@chromium.org
BUG=403663
NOTRY=true

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

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

This CL fixes a bug that non-composited scrolling div forgets to invalidate
because the div believed all its descandants are composited, when actually
some descendants paints to ancestor backing although composited.

BUG=401047

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

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

BUG=403987
TBR=kouhei@chromium.org

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

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

R=jamesr@chromium.org
BUG=354261

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

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

Remove in-class platform ifdefs from platform/fonts/FontPlatformData.h
as the entire file is wrapped in a OS(MACOSX) macro.

R=leviw@chromium.org, pdr@chromium.org

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

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

Implement support for the CSS font-stretch property on platforms using
SkFontMgr. This allows the width/stretch of a given font to be specified
and matches the IE and Firefox implementations.

Currently SkFontMgr is only used on Windows however as other platforms
migrate to SkFontMgr they'll get this support for free.

Entry in Chromium Dashboard:
  http://www.chromestatus.com/admin/features/edit/4598830058176512

Intent to Implement and Ship:
  https://groups.google.com/a/chromium.org/forum/#!msg/blink-dev/N24I2zJemvI/ZClPu585uAsJ
  
TEST=fast/text/font-stretch.html
BUG=331119
R=dglazkov@chromium.org

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

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

R=abarth
BUG=

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

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

Now also timing out on Mac/Debug, and it's consistently timing out,
not flakily.

BUG=356957
TBR=mkwst@chromium.org

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

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

Fix some leftovers after the WebCore -> blink namespace changes.

NOTRY=true

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

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

Use new match Functor ElementTraversal API in  DOMSiblingTraversalStrategy to
simplify the code a bit.

R=adamk@chromium.org, esprehn@chromium.org

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

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

Representing array and sequence types this way, rather than via auxiliary
flags on the member type, means exotic types like array-of-arrays,
array-of-sequences, arrays-of-nullables and similar can be supported.

More importantly, this also means that an array or sequence type can't be
mistaken for its member type if you forget to check the "is array" and
"is sequence" flags, and thus lets us remove a bunch of "is this an array
or sequence type" checks from code that has nothing to do with supporting
arrays or sequences.

This patch doesn't change code generation.

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

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

This CL adds GC memory profiling using the about:tracing infrastructure. The
profiling can be time consuming, so the GC_PROFILE_* macros are used to
selectively enable various profiling features for the time being.

The GC_TRACING macro has been renamed to GC_PROFILE_MARKING to avoid confusion with about:tracing. The tracing event group has been renamed from blink to blink_gc to allow collection of GC related events independent of other blink events.

BUG=

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

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

Set Battery Status API status to stable in blink. We have
implementations for all platforms supported by blink now,
i.e. on Android, Windows, Linux, ChromeOS and Mac.

BUG=122593
TEST=http://jsbin.com/battery-status-test

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

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

Shut down the scheduler before terminating the heap (i.e.,
PartitionAlloc). This is needed because the scheduler makes heap
allocations that need to be freed at destruction time.

BUG=391005

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

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

-A bunch of minor UI tweaks to get things to line up better.
-Align the examine/snooze buttons vertically so they take up less space.
-Make the tabs in ct-results-by-builder black and more tab-looking.
-Change the name to Sheriff-o-matic now that it's clear we're sticking with it.
-Add a useful links link to the right of the revisions.

BUG=400411,400398
NOTRY=true

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

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

The files CSSGrammar.y, CSSParserMode.{h,cpp}, CSSTokenizer{.h,-in.cpp},
CSSParserValues{.h,.cpp,Test.cpp} are all primarily for the parser so
should live in core/css/parser (making a lexing vs parsing distinction
here doesn't seem very useful).

BUG=330389

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

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

This layout test demonstrates a case where they do not.

BUG=403819

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

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

The interface was given a less generic name in a recent spec update:
https://github.com/slightlyoff/ServiceWorker/issues/350

BUG=403536
R=falken@chromium.org,haraken@chromium.org

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

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

http://src.chromium.org/viewvc/blink?view=revision&revision=180279

BUG=394692
TBR=eae@chromium.org

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

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

Some duplication had sneaked in as part of the IDL dictionary work, with
IdlTypeBase.impl_should_use_nullable_container meaning almost the same
thing as IdlTypeBase.is_explicit_nullable, except the latter also implies
that the type is nullable.

Common up the actual logic in the new IdlTypeBase.cpp_type_has_null_value
property, and use it for both impl_should_use_nullable_container and
is_{implicit,explicit}_nullable.

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

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

http://src.chromium.org/viewvc/blink?view=revision&revision=180269

BUG=382809
TBR=yoichio@chromium.org

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

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

BUG=402749

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

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

BUG=403818
TBR=tkent@chromium.org

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

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

Oilpan: Ensure that left-most vtable is the first thing to be initialized for polymorphic classes with trace methods.
    
This is done by ensuring one of two properties for any polymorphic class with a trace method.
1. If the trace method is virtual, then the left-most base class must define a virtual trace method too.
2. If the trace method is non-virtual, then the left-most base class must define some virtual method.
    
The Blink GC plugin statically checks these properties.

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

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

This newly-added test always crashes when run on Android.

BUG=403804
TBR=abarth@chromium.org

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

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

Implement support for numeric font-weights (i.e. 100..900) on platforms
using SkFontMgr by switching to the new matchFamilyStyle API instead of
the old legacyCreateTypeface one.

Currently SkFontMgr is only used on Windows.

R=dglazkov@chromium.org
BUG=394692
TEST=fast/text/font-weight.html

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

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