DevTools: Avoid private member access in non-profiler code

R=caseq, pfeldman, sergeyv

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

git-svn-id: svn://svn.chromium.org/blink/trunk@177029 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Promisify Service Worker test unregister_and_register helper function

Service Worker API is promise-based, so this helper should be too.

Also take care of a FIXME now that testharness has been updated.

BUG=
R=horo@chromium.org

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

git-svn-id: svn://svn.chromium.org/blink/trunk@177030 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Share more code between boundingBoxForCompositing and physicalBoundingBoxIncludingReflectionAndStackingChildren

This also makes it easier to see what the code is computing.

BUG=

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

git-svn-id: svn://svn.chromium.org/blink/trunk@177031 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Remove an unneccessary special case for the main resource from InspectorPageClient

Miniature cleanup, the special case is not needed as the default: case handles
main resources well: it returns early with false as resourceBuffer() is null.

R=pfeldman@chromium.org

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

git-svn-id: svn://svn.chromium.org/blink/trunk@177032 bbb929c8-8fbe-4397-9dbb-9b2b20218538
DevTools: Fix breakpoints in source mapped files after reload.

Note: Test expectations change for the testBreakpointInCollectedReload test
are not caused by the change, but by the change in test harness.
These changes reveal an error in BreakpointManager implementation that will be fixed in the following patch.

BUG=388131
R=sergeyv

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

git-svn-id: svn://svn.chromium.org/blink/trunk@177033 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Move animation properties back into LowPriorityProperties

Previously we've attempted to have animations and transitions apply as
part of the cascade instead of always as the last step, which required
applying animation-related properties before high priority properties.
This hasn't ended up working and we have no plans to fix the cascade,
so we no longer need the separate animation properties pass.

BUG=232273

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

git-svn-id: svn://svn.chromium.org/blink/trunk@177034 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Move PositionWithAffinity class to editing

This patch moves PositionWithAffinity class to editing/ from dom/Position.h
since it is "affinity" is editing concept rather than DOM concept.

BUG=n/a
TEST=n/a; no behavior changes

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

git-svn-id: svn://svn.chromium.org/blink/trunk@177035 bbb929c8-8fbe-4397-9dbb-9b2b20218538
DevTools: Use in JavaScriptSourceFrame current target's debuggerModel instead of WebInspector.debuggerModel

BUG=

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

git-svn-id: svn://svn.chromium.org/blink/trunk@177036 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Remove StyleResolver::ensureScopedStyleResolver

This makes the callers less awkward, and also removes this API from the
public interface of StyleResolver which is out of hand.

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

git-svn-id: svn://svn.chromium.org/blink/trunk@177037 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Rework MediaQueryMatcher to batch up listener notification

MediaQueryMatcher currently runs script synchronously inside layout and
recalc style which is bad. Instead we should collect all the listeners
that have changed and notify them asynchronously. Incidentally this is
what the spec says to do too, and also what Firefox and the polyfills
do.

This is the first step where we collect all the listeners that changed
in a Vector and then notify them as a group. This also fixes our behavior
where adding a listener in the middle of notification would get that one
called too, which is not what the spec says.

This also changes the notification order such that listeners are notified
in groups based on the creation time of their MediaQueryList which also
matches Firefox (and the polyfill) instead of in the order of global
calls to addListener.

Based on https://codereview.chromium.org/214383008/ by esprehn@

BUG=356487

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

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