Fix EmbeddedWorker code so that ServiceWorker test code works

* Changed to call WorkerReportingProxy::workerGlobalScopeStarted() after
  didStartWorkerRunLoop() is notified so that chromium side can get the
  correct worker thread's ID within workerGlobalScopeStarted()
* Added missing WebFrameClient::didFinishDocumentLoad() override in
  WebEmbeddedWorkerImpl so that it can correctly wait for document
  loading before loading the worker script
* Separated out the WebEmbeddedWorkerImpl's Loader::load() method
  from its ctor so that m_mainScriptLoader is always non-null when
  notifyFinished() is called.
* Tentatively changed to use ResourceRequest::TargetIsSharedWorker instead of
  ResourceRequest::TargetIsServiceWorker until more chrome side plumbing is done

BUG=313530
TEST=to be added

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

git-svn-id: svn://svn.chromium.org/blink/trunk@164611 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Cleaning up EventHandler scrolling methods.

Merged scrollNode into scrollOverflow and renamed to make it clear
it scrolls the DOM tree. Renamed WebViewImpl::propagateScroll and
EventHandler::recursiveScroll to bubblingScroll and made the WebViewImpl
method a simple plumbing call into EventHandler.

Note that this has a subtle behavior change when scrolling with the
space bar. Previously, space bar would scroll the DOM tree at the focused
node, but once it bubbled through the frame, it would bubble up only
through frames. Now that it uses the EventHandler method it will bubble
up to the containing DOM node. I believe this was a bug and the change
is consistent with how we usually bubble scrolls.

BUG=325554

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

git-svn-id: svn://svn.chromium.org/blink/trunk@164612 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Add a toDifferentDocument parameter to WebViewClient::didStartLoading

RenderViewImpl has a new version that includes it. This bit will be used to
differentiate between same document (navigations to anchors, pushState, etc.)
and different document navigations.

BUG=50298

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

git-svn-id: svn://svn.chromium.org/blink/trunk@164613 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Clean up virtual methods in MarkupAccumulator and subclasses

Clean up virtual methods in MarkupAccumulator and subclasses:
- Use OVERRIDE for overriden virtual methods in StyledMarkupAccumulator and
  SerializerMarkupAccumulator
- Mark StyledMarkupAccumulator and SerializerMarkupAccumulator and FINAL as
  these are never subclassed and they override several virtual methods.
- MarkupAccumulator::appendString() does not need to be virtual as it does not
  need to be overriden

R=adamk

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

git-svn-id: svn://svn.chromium.org/blink/trunk@164614 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Cleanup - Get rid of unnecessary local variables.

Trivial cleanup patch to get rid of not needed local variables.

TESTS=No tests as there is no behavior change.

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

git-svn-id: svn://svn.chromium.org/blink/trunk@164615 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Clean up FrameLoader/DocumentLoader commit logic

This is primarily moving stuff that is currently in FrameLoader::clear() and DocumentLoader::createWriterFor(), reducing the number of paramters on these functions and moving parts of them to more conceptually accurate locations.

BUG=none

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

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