GitLab

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

BUG=505411
TBR=fmalita@chromium.org

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

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

Report accurate Overscroll on handleWheel. Currently 
invalid ScrollResult is returned on overscroll.

compositor changes @
https://codereview.chromium.org/1203693003/

Bug=442859

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

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

TBR=fmalita@chromium.org
NOTRY=true

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

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

Move one of them that still fails to a new bug.

BUG=382491

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

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

TBR=aboxhall
NOTRY=true

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

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

"Copy as cURL" command in Network panel produces an output that
is platform-dependent: escapings are different for cmd and posix

However many Windows user use unixy shell (for instance msys / git bash)
and hence it makes sense to expose them a unixy curl command.

BUG=488371

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

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

The StylesSidebarPane should be force-refreshed on undo/redo command
happens.

BUG=501310
R=pfeldman

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

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

After https://codereview.chromium.org/1180903006, the GPU results for
fill-stroke-clip-reset-path.html are slightly different for path vs.
rect fills.  Unfortunately this means a ref test is no longer feasible.

Change fill-stroke-clip-reset-path.html to be a simple pixel test, and
mark for rebaseline.

BUG=505411
TBR=bsalomon@google.com
NOTRY=true

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

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

They convert from LayoutRect to FloatRect and back again, which is far from free.

BUG=321237

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

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

BUG=438023

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

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

BUG=503852
R=wangxianzhu

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

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

BUG=490047
TBR=pfeldman

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

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

TBR=dgozman
NOTRY=true

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

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

BUG=504412
TBR=pfeldman

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

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

BUG=503887
R=pfeldman@chromium.org

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

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

BUG=488371
TBR=pfeldman

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

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

BUG=505395

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

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

R=alph
BUG=502186

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

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

While transitioning Blink to Oilpan, there will be LifecycleNotifier<>s
not on the Oilpan heap, but with some of their LifecycleObserver<>s being
on the heap.

Should a GC strike while such a notifier iterates over its observer set,
dead observers may remove themselves (eagerly) while being finalized
during that GC. This will mutate the notifier set, which isn't currently
assumed nor supported.

There's no need to support this when fully transitioned, Oilpan will
handle GCs across such iterations gracefully, but in the meantime we will
have to support this by snapshot'ing the observer set and explicitly
checking for liveness. Like already done over ContextLifecycleNotifier,
where such dynamic unregistrations are supported.

Notice that this issue isn't tied to lazy sweeping; indeed, lazy sweeping
without eager finalization of lifecycle observers will potentially make this
bug rarer.

R=haraken
BUG=480837

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

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

The width and height should be valid, but add a size check while
we figure out the root cause of issue 504690.

BUG=504690

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

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

R=dgozman
BUG=495202

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

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

Currently, when dumping blink_gc (Oilpan) memory, both the heap pages
and the freelists are scanned. Both of them report free bytes/objects.
At current state, this causes a dump of the following form:
blink_gc/thread_X/heaps/HeapName
blink_gc/thread_X/heaps/HeapName/pages    free_size=1M
blink_gc/thread_X/heaps/HeapName/buckets  free_size=1M

which causes the UI to infer the /HeapName has free_size=2M, which is
incorrect. This change forces the /HeapName dump to override the
free_{size,count} attributes in order to match the HeapName/pages dump.

BUG=490087

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

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

TBR=aboxhall
NOTRY=true

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

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

BUG=497167

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

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

Considering V8DOMWrapper::createWrapper invokes v8::FunctionTemplate::GetFunction
and v8::Function::NewInstance, it may run an arbitrary script and reenter to
ScriptWrappable::wrap with the same object.

This CL supports such a case and consistently uses the wrapper that is created
and associated first.

BUG=477425,389445

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

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

XMLHttpRequestProgressEventThrottle::dispatchReadyStateChangeEvent may dispatch
multiple events: a deferred progress event and the given ready state change.
Multiple dispatching may lead to a use-after-free bug.

This CL adds a protection for the event target. In addition to that,
XMLHttpRequestProgressEventThrottle stops dispatching the latter event when
dispatching the former event changes the readyState.

BUG=505362

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

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

Some WebDataConsumerHandle subclasses use on-heap objects and Oilpan requires
the followings:

- Non-oilpan threads must not touch on-heap objects.
- Objects must not outlive the thread that created the objects. For example, it
is invalid to keep holding a Persistent handle to an object that was created by
a thread already detached.

As a result, such WebDataConsumerHandle subclasses cannot be used
in non-oilpan threads.
We hope we will get rid of this restriction in the future.

BUG=480746

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

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

As this is not a part of the specification https://w3c.github.io/webappsec/specs/credentialmanagement/#credentialcontainer

This patch is a part of a two-sided patchset:
Blink: https://codereview.chromium.org/1217463002/ <-- this patch
Chromium: https://codereview.chromium.org/1215653003/

BUG=494880

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

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

BUG=505411
TBR=aboxhall,fmalita
NOTRY=true

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

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

TBR=dgozman
NOTRY=true

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

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

Implement getComputedStyle(e).quotes

BUG=472963

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

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

This replaces the Document it previously took. It allows Internals to
contain objects which depend on ScriptState, like ScriptPromise or (in
the future) ReadableStream.

BUG=
R=haraken@chromium.org

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

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

As explained here
http://lists.w3.org/Archives/Public/www-style/2014Mar/0512.html we sometimes
need to consider that some tracks are infinitely growable even when they are
really not, in order to produce more "natural" results.

For example the following case:
    grid-template-columns: auto auto;

    item 1 in column 1 with min-content = max-content = 10px;
    item 2 in columns 1-2 with min-content = 30, max-content = 100px;

will outcome (45px, 55px) without this patch. But considering the second
column as infinitely growable the result is (10px, 90px), a more "natural"
result because column 1 just needs to be 10px to accommodate item 1.

From now on we can flag GridTracks so that they can infinitely grow even
when the growth limit is finite. In order to completely support the feature
some changes were required:
  * m_plannedIncrease became m_plannedSize because we need to store both the
old and new values of the track size in order to decide if a track should be
considered infinitely growable. Consequently m_increaseDuringDistribution
became m_sizeDuringDistribution.
  * distributeSpaceToTracks() is now unconditionally called even though the
extra space is 0. That's because it computes the plannedSize value.
  * a new templatized function called markAsInfinitelyGrowableForTrackSizeComputationPhase()
marks/unmarks tracks as infinitely growable only during the max functions resolution.

Apart from that all the GridTrack's public attributes were made private so
that accessors are now provided for getting/setting their values. Also
AccumulatorSetterFunction was removed as it is no longer needed.

BUG=455724

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

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

Mark virtual/prefer_compositing_to_lcd_text/scrollbars/rtl/overflow-scroll-rtl.html as failing on win

BUG=505387
TBR=wangxianzhu,aboxhall
NOTRY=true

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

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

This's the first CL to make Chrome side CL build.

BUG=498596

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

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

BUG=358074

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

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

Followup CLs will add additional items to the sidebar.

BUG=503167

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

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

In LayoutTests/platform/android, as a follow-up to https://codereview.chromium.org/1160283002.

BUG=494880

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

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

NOTRY=true
TBR=fmalita@chromium.org,reed@chromium.org

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

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

The "promise" keeps the "reason" alive anyway

BUG=495801
R=haraken@chromium.org,domenic@chromium.org

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

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