GitLab

BUG=None

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

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

This CL add power profiler in DevTools frontend, and draw power data in PowerOverview of Timeline panel.

BUG=337138

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

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

Only EventTarget.h needs this include

This is a follow up patch for this CL by Adam Barth
https://codereview.chromium.org/26890003

BUG=

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

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

BUG=341032

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

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

The last extractedStyle parameter of removeInlineStyleFromElement() is not mandatory and it's set
to default 0. This way we have to check its existence before the usage.

Backported from my WebKit patch: http://trac.webkit.org/changeset/154439

R=hayato@chromium.org
BUG=350414

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

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

For async stacks we will collect scopes up to the Local scope (i.e. no inner With or Catch scopes).
This is way more faster (about 2x), since inner scopes require script parsing and scope analyzing.

In the DevTools UI this is a small reduction, as any With object or exception can be evaluated on
async call frame in the console while paused, or looked via a popup on mouseover.

Speed gain: more than 2x.

BUG=340285
R=yurys, pfeldman

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

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

BUG=341032

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

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

R=tkent@chromium.org
BUG=

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

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

R=aandrey@chromium.org, apavlov@chromium.org

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

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

BUG=341032

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

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

Now there is no test failing when I change RefCounted<Event> to RefCountedGarbageCollected<Event>. Thus this CL does the change.

BUG=341032

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

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

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

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

TBR=philipj@opera.com
BUG=356148

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

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

R=vsevik@chromium.org

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

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

TBR=pfeldman

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

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

We weren't checking if the input element is a range control inside paintSliderTicks and so we were hitting an assert.

TEST=Automated. Added slider-appearance-with-ticks-crash.html.
BUG=350905

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

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

Since InterpolationRecord has a RefPtr, it should be GarbageCollectedFinalized.

BUG=341032
TBR=ager

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

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

buttons and <select> buttons were darker on Android than on Ash.

BUG=351275

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

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

It also returns 'portrait-primary' when the frame is detached
instead of empty string (current behaviour).

BUG=162827

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

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

Revert of DevTools: Implement Styles search and Computed style filter (https://codereview.chromium.org/129793013/)

Reason for revert:
Broke the Styles pane

Original issue's description:
> DevTools: Implement Styles search and Computed style filter
> 
> The user can search styles by property name, value, or rule selector.
> Computed style properties are filtered, so that only matching ones are visible.
> 
> R=eustas, pfeldman, vsevik
> BUG=278852
> 
> Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=169860

TBR=eustas@chromium.org,pfeldman@chromium.org,vsevik@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=278852

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

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

Note: This CL changes the argument type of SQLiteDatabase::setAuthorizer from
PassRefPtr<DatabaseAuthorizer> to DatabaseAuthorizer*. We don't need to use
PassRefPtr<> here.

BUG=347902

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

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

Without this CL, a SQLResultSet object in SQLStatementBackend can be constructed
in a database thread, moved to main/worker thread, and referred by JavaScript
object in the main/worker thread.

We can't move object ownership over threads in Oilpan. So, SQLStatementBackend
should construct a SQLResultSet object in the SQLStatementBackend constructor,
which runs in main/worker thread. SQLResultSet should have a validity flag to
represent nullness in the code without this CL.

Note:
* Change the return type of AbstractSQLStatementBackend::sqlResultSet from
  PassRefPtr<SQLResultSet> to SQLResultSet* because the function doesn't release
  the ownership of the SQLResultSet object.

* Change the order of data members of SQLResultSet to pack better.

BUG=347902

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

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

BUG=

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

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

Traverse the selectorList() for :host pseudo, extract features, and create
class invalidation sets like we do for features and classes on the top level.

R=esprehn,ojan
BUG=335247,339729

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

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

BUG=353921

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

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

RESOLUTION: to change draw SystemFocusRing to drawFocusIfNeeded.

public discussion:
  http://www.w3.org/Bugs/Public/show_bug.cgi?id=23980
  http://www.w3.org/WAI/PF/HTML/wiki/Summary_of_2014-01-13_Meeting
  http://www.w3.org/2014/01/13-html-a11y-minutes.html#item02

other browsers:
  It has already been agreed or changed.
  http://bugzilla.mozilla.org/show_bug.cgi?id=959820 (Firefox)

BUG=354960

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

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

Intent to Deprecate:
https://groups.google.com/a/chromium.org/d/msg/blink-dev/nNsZDn_utUc/dUtD3ulsFMkJ

BUG=354921

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

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

EventPath::parent() is only used in adjusting the result of HitTest in some places.
For example, if a text node is clicked, we have to adjust the result to the *reasonable* parent element of the text node.

In adjusting, we should use NodeRenderingTraversal::parent() instead of EventPath::parent().
That means the adjusted result is no longer shadow roots or insertion points.
The result would be the parent element of the text node in the composed tree.
That should make sense because neither shadow roots nor insertion points are rendered.

BUG=354366

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

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

RawPtr is a transition type that will be removed and replace with a
normal pointer. Normal pointers are not default initialized and therefore
RawPtr shouldn't be either. Instead, we introduce a zap value in debug
mode to make it easier to find uninitialized RawPtrs during the transition.

R=erik.corry@gmail.com, oilpan-reviews@chromium.org, tkent@chromium.org, zerny@chromium.org

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

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

Bump up the limit from 10 StepIns to 20.
Add a test on framework code.

BUG=267592
R=pfeldman, yurys

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

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

We can use add() to avoid doing two hash lookups in the common case where
there's no callback queue yet (ex. setting a new attribute) and we can
also return a reference.

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

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

This CL moves Interpolation's hierarchy and InterpolationRecord to oilpan's heap. I'll move other Interpolation objects in a follow-up CL.

BUG=341032
NOTRY=true

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

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

This is a prerequisite for deprecating the prefixed function, because
otherwise the forms tests get a deprecation message. This internal use
of webkitRequestAnimationFrame may also have inflated the use counter
data for it.

BUG=354921
R=abarth@chromium.org, keishi@chromium.org, tkent@chromium.org

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

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

BUG=

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

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

Promises are now implemented tested in V8.
I delete Promises tests except for tests testing ScriptPromise or
ScriptPromiseResolver.

BUG=352597

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

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

NOTRY=true

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

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

This is as per the the coding guidelines for blink, which disallows
importing names from the STL library through the "using" declarations.

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

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

The function draws a focus ring around the current default path or the given path if the given element is focused.
Currently, the focus ring is drawn incorrectly because it is larger than the given path region.
To solve this problem, we should add focus ring width to dirty rect.

BUG=352976

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

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

Runtime ShadowDOM flag was already set to true at r169201. This change shouldn't have any effects.

CSSSelector still uses this flag. I'll clean up CSSSelector in another patch.

BUG=336121

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

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

If you delete the documentElement then that also means all <style> and
<link> elements must have been removed from the document so the
StyleResolver should already be reset or cleared and we don't need to
call clearStyleResolver() from inside Document::childrenChanged.

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

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