Revert of Try #3 to fix the android layout tests crash in run-webkit-tests. (patchset #2 id:20001 of https://codereview.chromium.org/591563002/)

Reason for revert:
Reverting, since this broke mac bots:

UnicodeEncodeError raised: 'ascii' codec can't encode characters in position 11332-11333: ordinal not in range(128)

Original issue's description:
> Try #3 to fix the android layout tests crash in run-webkit-tests.
> 
> This patch forcibly reencodes to ASCII at a different layer, matching
> where we're actually seeing the stack trace. This seems like an
> overly broad solution, but may at least get us past the crash so I can
> see where things might really be failing.
> 
> This change also backs out the change in r182282, since it didn't seem
> to help anything.
> 
> TBR=peter@chromium.org
> BUG=386343
> 
> Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=182358

TBR=peter@chromium.org,dpranke@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=386343

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

git-svn-id: svn://svn.chromium.org/blink/trunk@182387 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Auto-rebaseline for r182366

http://src.chromium.org/viewvc/blink?view=revision&revision=182366

TBR=vollick@chromium.org

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

git-svn-id: svn://svn.chromium.org/blink/trunk@182388 bbb929c8-8fbe-4397-9dbb-9b2b20218538
DevTools: Show timeline JS frames even if there no function call lays beneath.

BUG=363976

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

git-svn-id: svn://svn.chromium.org/blink/trunk@182389 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Auto-rebaseline for r182359

http://src.chromium.org/viewvc/blink?view=revision&revision=182359

BUG=415719
TBR=wangxianzhu@chromium.org

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

git-svn-id: svn://svn.chromium.org/blink/trunk@182390 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Fix call stack creation inside Promise Tracker.

BUG=348919

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

git-svn-id: svn://svn.chromium.org/blink/trunk@182391 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Remove ObjectFitPosition Runtime Flag(status=stable)

BUG=402536

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

git-svn-id: svn://svn.chromium.org/blink/trunk@182392 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Take isolatedCopy() when copying SQLErrorData

BUG=415869

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

git-svn-id: svn://svn.chromium.org/blink/trunk@182393 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Assert that frame() derefs a non-null pointer.

The Frame references kept by these objects are assumed to be non-null;
assert that it is so.

R=haraken
BUG=340522

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

git-svn-id: svn://svn.chromium.org/blink/trunk@182394 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Remove ASSERT(renderer()) in HTMLTextAreaElement::updateValue()

This was first removed in r181422 and then brought back due to
revert at r182379 and now causes fast/ime/composition-offsets.html
and fast/forms/autofilled.html to crash. It does not look like
updateValue() needs renderer now (it did back when the ASSERT()
was added), so removing the assert again.

TBR=sudarshan.p@samsung.com,keishi@chromium.org,tkent@chromium.org

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

git-svn-id: svn://svn.chromium.org/blink/trunk@182395 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Mark virtual/threaded/animations/webkit-perspective.html crashing in debug

TBR=hendrikw@chromium.org,pdr,wjmaclean
BUG=416367

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

git-svn-id: svn://svn.chromium.org/blink/trunk@182396 bbb929c8-8fbe-4397-9dbb-9b2b20218538
bindings: Adds a missing associateWithWrapper to AudioBuffer.

Adds a missing associateWithWrapper to AudioBuffer to monitor
array buffer deallocations.

BUG=235436

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

git-svn-id: svn://svn.chromium.org/blink/trunk@182397 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Oilpan: tighten up allowed allocation of some types.

No need to support these objects as being part of collections; adjust
macro usage.

R=ager
BUG=340522

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

git-svn-id: svn://svn.chromium.org/blink/trunk@182398 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Oilpan: Move MediaStreamSource, MediaStreamComponent and MediaStreamDescriptor
to oilpan's heap

- MediaStreamSource, MediaStreamComponent and MediaStreamDescriptor must be moved
to oilpan's heap in one go because their lifetime are tightly coupled.

- Removed m_descriptor->setClient(0) from MediaStream's destructor. This is OK
because MediaStreamDescriptor has a strong Member back to the Client
(i.e., MediaStream).

- Removed m_component->source()->removeObserver(this) from MediaStreamTrack's
destructor. This is OK because this CL made the observers weak
(i.e., MediaStreamSource::m_observers is a hash set of weak members to
MediaStreamTrack objects).

- Removed WebMediaStreamTrack::ExtraData::m_owner because it's unused.

- Introduced MediaStreamComponentDisposer to delay the destruction of
MediaStreamComponent::m_extraData. The ExtraData is exposed to the web
and some Chromium objects inherit from the ExtraData. If we clear the
m_extraData in the MediaStreamComponent's destructor, those Chromium-side
objects are also destructed in the MediaStreamComponent's destructor.
This is problematic because the destructors of the Chromium-side classes
can touch other on-heap objects in the Blink side. To avoid the issue,
we need to delay the destruction of the ExtraData to thread-specific
weak processing. The disposer pattern actually makes the deletion of
the extra data happen earlier and not later. The disposer makes sure
that the extra data is destructed in weak processing which is run before
sweeping and therefore all the objects are still alive and can be touched.


- The same problem arises for MediaStreamDescriptor and MediaStreamSource.
This CL adds MediaStreamDescriptorDisposer and MediaStreamSourceDisposer.

- This CL depends on https://codereview.chromium.org/543603003/ and https://codereview.chromium.org/549153002.

BUG=340522

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=181702

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

git-svn-id: svn://svn.chromium.org/blink/trunk@182399 bbb929c8-8fbe-4397-9dbb-9b2b20218538
DevTools Paint Profiler: display progress banner while profiling snapshot

- clear command log and duration bars while loading new snapshot;
- display "Profiling..." in overview bars window while profiling the snapshot.

BUG=

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

git-svn-id: svn://svn.chromium.org/blink/trunk@182400 bbb929c8-8fbe-4397-9dbb-9b2b20218538
[ServiceWorker] Remove the test for the rejected result of FetchEvent from fetch-event.html

We don't have the way to observe the network error using <iframe>.
So this change removes the rejecting test from fetch-event.html

I will introduce the reject test using XMLHttpRequest later. (step 3)

Step1: this patch
  [ServiceWorker] Remove the test for the rejected result of FetchEvent from fetch-event.html
  https://codereview.chromium.org/588993002/
Step2: [ServiceWorker] Treat the rejected result of onFetch as a Network Error.
  https://codereview.chromium.org/590043002/
Step3: [ServiceWorker] Add test for the rejected result of FetchEvent using XHR.
 https://codereview.chromium.org/588093003/

BUG=411173

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

git-svn-id: svn://svn.chromium.org/blink/trunk@182401 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Web SQL: Remove Database class, and DatabaseBackend should work as Database implementation.

This is a step to merge DatabaseBackend and Database. We need to remove
Database.{cpp,h} before renaming DatabaseBackend.{cpp,h} to Database.{cpp,h} to
keep revision hisotry correctly.

The next CL will rename DatabaseBackend to Database. This CL adds |#define
Database DatabaseBackend| to minimize the size of CLs.

TEST=none; No behavior changes.
BUG=

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

git-svn-id: svn://svn.chromium.org/blink/trunk@182402 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Oilpan: Enable Oilpan by default for core/testing

core/testing is for testing only, so there would be no performance concern in shipping Oilpan.

BUG=340522

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

git-svn-id: svn://svn.chromium.org/blink/trunk@182403 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Clean up passing tests from TestExpectations

TBR=pdr,wjmaclean

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

git-svn-id: svn://svn.chromium.org/blink/trunk@182404 bbb929c8-8fbe-4397-9dbb-9b2b20218538
DevTools: Fix debug_devtools=1 mode "main" extensions loading after r182328

R=dgozman,lushnikov
NOTRY=true

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

git-svn-id: svn://svn.chromium.org/blink/trunk@182405 bbb929c8-8fbe-4397-9dbb-9b2b20218538
WebThreadSupportingGC's destructor should have a safe point scope

WebThreadSupportingGC's destructor can block because WebThread's destructor blocks until all the tasks are processed. Thus we should add a SafePointScope to WebThreadSupportingGC's destructor. This CL does it.

The reason the current WebThreadSupportingGC is working without having the SafePointScope is that the caller side of the WebThreadSupportingGC uses a TaskSynchronizer to avoid the dead lock that the WebThreadSupportingGC may cause.

After landing this CL, we can remove the TaskSynchronizers from the caller side.

BUG=340522

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

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