GitLab

InspectorInstrumentation::didReceiveResourceResponse should  be called whenever it comes clear that request won't be finished normally.

BUG=409739

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

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

RenderFlexibleBox has several methods that take pointers to RenderBox instances
but those methods assumes that the objects will never be NULL. We should replace
them by references. This makes the caller code safer.

This is in continuation to the CL https://codereview.chromium.org/555213002/

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

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

ForceHorriblySlowRectMapping was needed in RenderInline::
clippedOverflowRectForPaintInvalidation() to avoid errors when calling
containingBlock->mapRectToPaintInvalidationBacking() with the provided
paintInvalidationState.

Now call RenderInline's own mapRectToPaintInvalidationBacking() instead
of containingBlock's to make paintInvalidationState applicable and
reduce duplicate code.

BUG=402994
TEST=exsiting tests

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

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

This was fixed in V8 in r23858.

BUG=411196
TBR=loislo, yangguo

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

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

The async scripts execute in non-deterministic order so I removed their
debug output.  The only thing we really care about testing in the
script-priorities layout test is the request priority which is
determined and logged deterministically at parse time (and still
captured).

BUG=413604

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

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

    
The ensure-valid-python script seems to be unused,
and additionally in webkitpy there is already another
version checker implementation.

BUG=none

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

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

Blink should make use of blink_resources.grd for the inline resources of user agent stylesheets.
This removes the dependency upon using make-file-arrays.py which embeds these resources as strings. 

Also the .rodata section of libblink_web (in component build mode) is reduced by ~33kb.

Part 1: https://codereview.chromium.org/436843004/
Part 2: https://codereview.chromium.org/422023008
Part 2 Unit Test support: https://codereview.chromium.org/456413002/

BUG=312586

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

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

This method is called a lot during browser launch. Instead of
reconstructing the same AtomicString every time (without even
ConstructFromLiteral) pass in a local static AtomicString.

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

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

An empty list of grid line names (represented by "()") does not add anything to
the list of parsed values. That's why trying to concatenate an adjacent list of
grid line names was failing, because we were trying to concatenate a list with
the last parsed CSSValue which was not the expected grid line names list.

BUG=402004

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

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

imported/web-platform-tests/shadow-dom/elements-and-dom-objects/shadowroot-object/shadowroot-methods/test-004.html
imported/web-platform-tests/shadow-dom/user-interaction/ranges-and-selections/test-001.html
imported/web-platform-tests/shadow-dom/user-interaction/ranges-and-selections/test-002.html

NOTRY=true
BUG=413731
TBR= dpranke@chromium.org, jianli@chromium.org

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

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

Remove all the embedded resources from blink and migrate to using grd generated
pak files

BUG=312586

R=haraken@chromium.org, jochen@chromium.org

Part 1: https://codereview.chromium.org/558933002
Part 2: https://codereview.chromium.org/561653002

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

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

The spec says that 'vertical-align' has no effect on a grid item.
Currently there are no tests for this, so added a new test case
for verifying the same.

BUG=395788

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

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

Revert of HTMLSelectElement does not include selected index/indices while saving state (patchset #8 id:160001 of https://codereview.chromium.org/541693003/)

Reason for revert:
Looks like this test crashes consistently on Android Tests (dbg) starting at this build:
http://build.chromium.org/p/chromium.webkit/builders/Android%20Tests%20%28dbg%29/builds/21459


Original issue's description:
> HTMLSelectElement does not include selected index/indices while saving state
> 
> Since index is not part of the FormControl state for select, there is no
> way to restore it to proper index if there are duplicate values. This
> patch includes the index in the FormControl state for select.
> 
> BUG=401506
> 
> Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=181883

TBR=tkent@chromium.org,keishi@chromium.org,sudarshan.p@samsung.com
NOTREECHECKS=true
NOTRY=true
BUG=401506

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

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

We don't have an ExecutionContext when dealing with 'frame-ancestors'
violations. This patch uses the LocalFrame which we pass into
allowAncestors in order to provide hooks for both PingLoader and
ConsoleMessages.

It also fiddles a bit with the reporting data, as we'd otherwise suggest
that 'about:blank' failed to load (as that's the initial contents of a
frame before we navigate to a document).

BUG=412725

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

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

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

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

BUG=391005

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

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

Factor the fill/stroke code out of the various paint-server
RenderSVGResource subclasses and move it to SVGRenderSupport.

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

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

This method is large, and the gain that propagating the constant mode from
its caller is relatively small. Make {fill,stroke}PaintingResource default
inline instead, and un-inline requestPaintingResource and move it into the
class.

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

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

On windows we receive longer script because \r\n

BUG=30536
TBR= vsevik@chromium.org

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

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

The SVGInlineTextBox member variable |m_paintingResource| is only set
temporarily within paintTextWithShadows and paintDecorationWithStyle, and
is only accessed from the methods that do setup and teardown of it. This
makes it possible to remove it from SVGInlineTextBox and instead manage it
in through a variable on the stack.
The lifetime management is kept manual/the same as previously. The
interaction with TextRun::RenderingContext is changed to be passed as a
parameter to the added "scope" object in order to simplify the interface
a bit.

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

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

BUG=391593

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

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

BUG=413067
R=ojan@chromium.org

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

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

BUG=391593

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

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

Gardening: mark large-inlined-script.html as Skip because it is really long, may fail, pass and timeout.

TBR= eustas@chromium.org, jianli@chromium.org
BUG=413249
NOTRY=true

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

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

Add setDeviceColorProfile() to allow a render widget to set the
screen color profile of its blink::Page.

Also add a resetDeviceColorProfile() method used to restore the
blink::Page to default color profile state.

Implementation of each in Page are FIXME.

BUG=368663

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

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

A trivial conversion is defined as one that cannot throw an exception,
and thus need no v8::TryCatch or ExceptionState, and no "did it succeed?"
check after the conversion.

For such conversions, generate the code

  Type variable = convertedValue()

instead of

  TONATIVE_VOID(Type, variable, convertedValue())

Also include all such conversions in the set of argument conversions that
don't require a v8::TryCatch, since they then no longer do.

BUG=413257

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

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

String concatenation is used a lot to build error and exception
messages and it was larger than intended because of over eager
inlining.

Outlining template code saves 90 KB with clang and 65 KB with
gcc (lin64 content_shell). After this there is roughly 60 KB of
concatenation code left (clang).

clang:
Total change: -91866 bytes
==========================
  47 added, totalling +6264 bytes across 3 sources
  38 removed, totalling -18354 bytes across 2 sources
  79 grown, for a net change of +4604 bytes (98534 bytes before, 103138 bytes after) across 50 sources
  405 shrunk, for a net change of -84380 bytes (370130 bytes before, 285750 bytes after) across 134 sources

gcc:
Total change: -66849 bytes
==========================
  333 added, totalling +81561 bytes across 69 sources
  203 removed, totalling -65785 bytes across 76 sources
  211 grown, for a net change of +18666 bytes (228381 bytes before, 247047 bytes after) across 85 sources
  406 shrunk, for a net change of -101291 bytes (568083 bytes before, 466792 bytes after) across 160 sources

BUG=

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

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

Resources which were already loaded and then revalidated were skewing the data:
load time was coming from the already loaded resource, compile time was the new
compilation time.

R=jochen@chromium.org
BUG=

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

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

After removing the temporary SCM directory change back to the
original cwd. This will avoid errors on future os.cwd() calls.

BUG=412774
TEST=test-webkitpy --all

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

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

Remove Host() call from RebaselineServer. The host object is passed
to the RebaselineServer from outside.
This could help with the Mocking as the FIXME mentioned.

BUG=None
TEST=test-webkitpy

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

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

BUG=413604
TBR= pmeenan@chromium.org, jianli@chromium.org
NOTRY=true

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

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

To follow the spec[1]. We should throw an type error when
a V8 value for IDL dictionary is neither an object, null, nor
undefined. The current implementation returns an
impl instance with default values.

[1] http://heycam.github.io/webidl/#es-dictionary

BUG=321462

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

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

Replacing generic Dictionary. No changes in behavior.
Tests under fast/encoding cover this change.

BUG=403150

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

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

Document for no reason.

This is important for Oilpan because the Ranges do not die until the
next GC. This gives a 10% improvement on DOM/textarea-edit in the
oilpan build.

R=yosin@chromium.org
BUG=388681

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

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

delete LayoutTests/platform/win-xp/svg/filters/feMerge-wrong-input-expected.txt

BUG=
TBR= jianli@chromium.org

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

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

Month buttons should have role=gridcell, and YearListCell should not. So this CL
moves role=gridcell from ListCell to DayCell.
Month buttons should have their own IDs, and readable month label.

BUG=123896

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

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

Code border became light gray; indents appeared.

BUG=391593

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

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

This patch completes the transition away from MixedContentChecker as an
object with state, into a world where it just provides static methods.
This should clear up the interface, and make migration to RemoteFrame
a bit simpler.

After this patch, we no longer hang a MixedContentChecker object off
of FrameLoader, nor does MixedContentChecker need to hold a reference
to a frame. Everything we need to do the checks can be done by passing
values into the checker methods.

Note also that this patch deletes a number of expectation files: these
are testharness.js tests, which shouldn't have expectations
committed into the repository. Rather than modifying them, this patch
removes them.

As a drive-by, this also corrects the FormsSubmitted counter.
It ought to count all form submissions as "FormsSubmitted", and mixed
submissions as a subset thereof (e.g. 5 form submissions, 
with 2 of them mixed would result in FormsSubmitted: 5, 
MixedContentFormsSubmitted: 2, as opposed to the current 3 and 2).

BUG=411184

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

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

- bindings_core_dictionary_impl_generated target should have
  core_dictionary_idl_files as inputs, otherwise IDL dictionaries
  impl classes won't get re-generated when they are edited.
- impl_includes_for_type() should check idl_type.base_type
  (not idl_type.name), otherwise the CG won't include necessary
  header files for nullable interface members.

BUG=321462

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

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

Since index is not part of the FormControl state for select, there is no
way to restore it to proper index if there are duplicate values. This
patch includes the index in the FormControl state for select.

BUG=401506

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

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