Remove ASSERT in ~IDBRequest since it is firing during legitimate uses in Web Inspector.

Adding the ASSERT back is tracked by https://webkit.org/b/129593.

https://bugs.webkit.org/show_bug.cgi?id=129328

Reviewed by Sam Weinig.

Source/WebCore:

* Modules/indexeddb/IDBRequest.cpp:
(WebCore::IDBRequest::~IDBRequest):

LayoutTests:

* TestExpectations: Remove skipped tests.
* inspector-protocol/model/probe-manager-add-remove-actions-expected.txt: Rebaselined.


git-svn-id: http://svn.webkit.org/repository/webkit/trunk@164967 268f45cc-cd09-0410-ab3c-d52691b4dbfc
JSDOMWindow::commonVM() should return a reference.
<https://webkit.org/b/129293>

Source/JavaScriptCore:

Added a DropAllLocks constructor that takes VM& without null checks.

Reviewed by Geoff Garen.

Source/WebCore:

Since this function always returns a VM and never null, make it
return a reference instead. There are more VM getters that never
return null, but I tried to keep the patch from snowballing.

Reviewed by Geoff Garen.

git-svn-id: http://svn.webkit.org/repository/webkit/trunk@164968 268f45cc-cd09-0410-ab3c-d52691b4dbfc
Blind Windows build fix.

* WebKit.vcxproj/WebKitExportGenerator/WebKitExports.def.in:

    Tweak JSDOMWindowBase::commonVM() return type to be a reference.

git-svn-id: http://svn.webkit.org/repository/webkit/trunk@164969 268f45cc-cd09-0410-ab3c-d52691b4dbfc
Debugging improvements from my gbemu investigation session
https://bugs.webkit.org/show_bug.cgi?id=129599

Reviewed by Mark Lam.
        
Various improvements from when I was investigating bug 129411.

* bytecode/CodeBlock.cpp:
(JSC::CodeBlock::optimizationThresholdScalingFactor): Make the dataLog() statement print the actual multiplier.
* jsc.cpp:
(GlobalObject::finishCreation):
(functionDescribe): Make describe() return a string rather than printing the string.
(functionDescribeArray): Like describe(), but prints details about arrays.



git-svn-id: http://svn.webkit.org/repository/webkit/trunk@164970 268f45cc-cd09-0410-ab3c-d52691b4dbfc
PolymorphicPutByIdList should have a simpler construction API with basically a single entrypoint
https://bugs.webkit.org/show_bug.cgi?id=129591

Reviewed by Michael Saboff.

* bytecode/PolymorphicPutByIdList.cpp:
(JSC::PutByIdAccess::fromStructureStubInfo): This function can figure out the slow path target for itself.
(JSC::PolymorphicPutByIdList::PolymorphicPutByIdList): This constuctor should be private, only from() should call it.
(JSC::PolymorphicPutByIdList::from):
* bytecode/PolymorphicPutByIdList.h:
(JSC::PutByIdAccess::stubRoutine):
* jit/Repatch.cpp:
(JSC::tryBuildPutByIdList): Don't pass the slow path target since it can be derived from the stubInfo.



git-svn-id: http://svn.webkit.org/repository/webkit/trunk@164971 268f45cc-cd09-0410-ab3c-d52691b4dbfc