ElementShadow should not check confusingAndOftenMisusedAttached() on ShadowRoot in the destructor

The only time we destroy ElementShadow objects is inside ~Element, and if we're running
an Element's destructor all of it's ShadowRoots have already been detached and all
element subtrees as well. There's no reason to check this, the code can never be reached
and we have a RELEASE_ASSERT in ~Node to ensure that renderers have already been
torn down.

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

git-svn-id: svn://svn.chromium.org/blink/trunk@160773 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Avoid unnecessary repaints when scrolling composited layers

If everything that moves in response to a scroll offset update is a composited
layer, then we don't need to invalidate in RenderLayerScrollableArea::setScrollOffset. We also don't have to clip the composited bounds (the compositor will do the clipping for us, and if we do the clipping in blink, we have to repaint every time we update the clipped composited bounds).

TEST=
virtual/softwarecompositing/geometry/limit-layer-bounds-clipping-ancestor.html
compositing/geometry/limit-layer-bounds-clipping-ancestor.html
compositing/overflow/do-not-repaint-if-scrolling-composited-layers.html

BUG=307114

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

git-svn-id: svn://svn.chromium.org/blink/trunk@160774 bbb929c8-8fbe-4397-9dbb-9b2b20218538
InsertionPoint::parseAttribute should not call confusingAndOftenMisusedAttached()

This check doesn't make any sense. Either we're not in the document and
setNeedsStyleRecalc() doesn't do anything, or we are in the document and
so we're attached.

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

git-svn-id: svn://svn.chromium.org/blink/trunk@160775 bbb929c8-8fbe-4397-9dbb-9b2b20218538
IDL compiler: Methods with interface type return type

V. simple one; interface type arguments are more involved, and I'll do
those in a separate CL.

BUG=239771
NOTRY=true

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

git-svn-id: svn://svn.chromium.org/blink/trunk@160776 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Use DOMRequestState in V8PromiseCustom Tasks to capture the world

As commented in that CL[1], use DOMRequestState in V8PromiseCustom.

[1]: https://codereview.chromium.org/25015004/

BUG=

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

git-svn-id: svn://svn.chromium.org/blink/trunk@160777 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Reland 160530 "Need to set preferred logical widths dirty bits w..."

This patch fixes the assertion/ASAN failure caused by
https://codereview.chromium.org/27262002/
The failure was due to accessing detached RenderTableSection through
invalidated section pointers. The pointers should recalc'd when accessed.

BUG=311908
R=jchaffraix@chromium.org

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

git-svn-id: svn://svn.chromium.org/blink/trunk@160778 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Fix remaining warning on mac, turn on -Wall.

BUG=312287

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

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