Fix up maximumCSSSampleId, and also add a presubmit check.

The presubmit check scans the whole UseCounter.cpp file upon a change, as it
needs to potentially read lines that were not modified. For example, if the largest
bucket id is commented out, the maximumCSSSampleId still needs to stay the
same.

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

git-svn-id: svn://svn.chromium.org/blink/trunk@173462 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Move SVGElement::finishParsingChildren to SVGSVGElement

Move SVGElement::finishParsingChildren to SVGSVGElement since it only performs actions on <svg>. This means we have to do one less check and SVG elements other than <svg> will have faster execution for finishParsingChildren.

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

git-svn-id: svn://svn.chromium.org/blink/trunk@173463 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Remove dino@apple.com from animation WATCHLIST as per his request.

TBR=dstockwell@chromium.org

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

git-svn-id: svn://svn.chromium.org/blink/trunk@173464 bbb929c8-8fbe-4397-9dbb-9b2b20218538
LeakExpectations: Add more {serviceworker,isolatedWorld} tests.

BUG=364377, 364390
NOTRY=true

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

git-svn-id: svn://svn.chromium.org/blink/trunk@173465 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Remove alancutter@chromium.org from animations watchlist

Following blink-reviews-animation group instead now.

TBR=timloh@chromium.org
NOTRY=TRUE

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

git-svn-id: svn://svn.chromium.org/blink/trunk@173466 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Remove v8_interface.constructor_argument

Refactoring, no changes to generated bindings.

v8_interface.constructor_argument()
is redundant with
v8_methods.generate_argument()
(It's just a simplified version, as pointed out by Jens - thanks!)

There's one v. minor change required to merge this:
constructors have None as their idl_type, so check for that
in addition to 'void'.

R=haraken
BUG=345503

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

git-svn-id: svn://svn.chromium.org/blink/trunk@173467 bbb929c8-8fbe-4397-9dbb-9b2b20218538
NotificationController handles an ownership of NotificaitonClient

NotificationClient is only used by NotificationController. So, NotificationController is able to have ownership of the client by using OwnPtr. The client's life cycle is more clear via clear ownership definition.

BUG=none

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

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