DevTools: hide page overlay layers in LayerTreeAgent

Filter out layers that are used by page overlays when reporting layer tree to the front-end.

Drive-by: move layer tests to inspector/layers.

BUG=

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

git-svn-id: svn://svn.chromium.org/blink/trunk@167677 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Document::didRemoveAllPendingStylesheet should consider hasNodesWithPlaceholderStyle()
    
While we have some pending stylesheet, StyleResolver::styleForElement returns s_styleNotYetAvailable style.
StyleResolver only checks whether there exist any pending stylesheets or not,
does not check what treeScope owns such pending styles.
    
So after the stylesheet is loaded, it is difficult to determine style recalc scope.
    
BUG=344729
TEST=fast/dom/shadow/import-rule-in-shadow-tree-needs-document-style-recalc.html

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

git-svn-id: svn://svn.chromium.org/blink/trunk@167678 bbb929c8-8fbe-4397-9dbb-9b2b20218538
HashTableConstIterator should use Traits::IteratorConstGetType

HashTableConstIterator should use 'Traits::IteratorConstGetType'
and 'Traits::IteratorConstReferenceType' types.
The rationals:
1) Consistency with HashTableIterator
2) That should help to verify that the mentioned types are defined
   properly in the traits classes.
3) HashTableIterator::get() returns 'const_cast' from
   HashTableConstIterator::get(), that might cause compilation
   errors in case 'Traits::IteratorGetType' is not the same
   as 'ValueType*'.

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

git-svn-id: svn://svn.chromium.org/blink/trunk@167679 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Move popular DOM attributes to prototype chains

Design document: https://docs.google.com/a/google.com/document/d/1yeHTCHhulVIlrKyx9_gCguAhLfcefVOa9uxxfW2LVG0/edit

Intent-to-ship-and-implement in blink-dev: https://groups.google.com/a/chromium.org/forum/#!msg/blink-dev/t0XiZuMey7M/9-5AuhoFyisJ

This CL moves DOM attributes used in Dromaeo to prototype chains. When I landed the CL two months ago (https://codereview.chromium.org/94403002), it caused 20+% regression in benchmarks. However, now that a V8 side fix is landed, there should be no overhead in moving DOM attributes to prototype chains in theory.

Dromaeo/dom-attr 1086 runs/sec => 1077 runs/sec (-0.8%)
Dromaeo/dom-modify 522 runs/sec => 525 runs/sec (+0.5%)
Dromaeo/dom-query 26150 runs/sec => 26200 runs/sec (+0.2%)
Dromaeo/dom-traverse 564 runs/sec => 544 runs/sec (-3.6%)

I don't understand the regression in dom-traverse. The regression is reproducible but I cannot reproduce the regression in a super-micro benchmark that just repeats accessing div.firstChild. Probably the regression might be a noise.

I'm going to land this CL and see how much regression is observed in perf bots (I'll email to perf-sheriff@). If I find the regression is unacceptable, I'll revert the CL.

See the design document and the Intent-to-ship email for justification for the regression.

BUG=43394

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

git-svn-id: svn://svn.chromium.org/blink/trunk@167680 bbb929c8-8fbe-4397-9dbb-9b2b20218538
DevTools: fix test inspector/elements/inline-style-title.html flakiness

This patch fixes flakiness of the
inspector/elements/inline-style-title.html test.

BUG=
TBR=pfeldman

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

git-svn-id: svn://svn.chromium.org/blink/trunk@167681 bbb929c8-8fbe-4397-9dbb-9b2b20218538
DevTools: Prepare SplitView for storing show mode in its setting and unify setting naming.

R=pfeldman

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

git-svn-id: svn://svn.chromium.org/blink/trunk@167682 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Remove handlers for user gesture callbacks

This reverts the CL https://codereview.chromium.org/82693006.

That code was added as a support for password autofill value protection (Chromium part was https://codereview.chromium.org/83023017). That was recently refactored (https://codereview.chromium.org/163843002/) and does not need the Blink support any more.

Precise identification of the revert, as generated by GIT:
Revert "Adding handlers for user gesture callbacks. Whenever a UserGestureToken is created, calls all registered callbacks to alert them of gesture."
This reverts commit b011272eda3e5193a5966522b9f845dd40fae1bd.

BUG=337429,163072

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

git-svn-id: svn://svn.chromium.org/blink/trunk@167683 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Fix null pointer dereference in CompositeEditCommand::moveParagraphs().

Document::documentElement() may return null, and in that case we should not
hand it to PlainTextRange::createRangeForSelection().

BUG=338542

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

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