Remove stray Changelog message.

Patch by Jon Lee <jonlee@apple.com> on 2012-09-10

git-svn-id: http://svn.webkit.org/repository/webkit/trunk@128120 268f45cc-cd09-0410-ab3c-d52691b4dbfc
Mark gradient-with-scaled-ancestor.html as flaky

* platform/chromium/TestExpectations:

git-svn-id: http://svn.webkit.org/repository/webkit/trunk@128121 268f45cc-cd09-0410-ab3c-d52691b4dbfc
Try to fix the Windows (32-bit) build.

* jit/JITOpcodes.cpp:
(JSC::JIT::emit_op_tear_off_arguments):
* jit/JITOpcodes32_64.cpp:
(JSC::JIT::emit_op_tear_off_arguments): Get operands 1 and 2, not 1 and 1. :(

Also took this opportunity to rename to indicate that these values are
not destinations anymore.



git-svn-id: http://svn.webkit.org/repository/webkit/trunk@128122 268f45cc-cd09-0410-ab3c-d52691b4dbfc
[V8] Inline firstFrame and currentFrame into their unique callers
https://bugs.webkit.org/show_bug.cgi?id=96309

Patch by Adam Barth <abarth@chromium.org> on 2012-09-10
Reviewed by Eric Seidel.

Source/WebCore: 

These functions have only one caller. We'd prefer not to have more
callers (because there's usually no reason to indirect through the
Frame), so we should inline them into their one caller (which is the
Chromium WebKit API).

* bindings/js/BindingState.h:
(WebCore):
* bindings/v8/BindingState.cpp:
* bindings/v8/BindingState.h:
(WebCore):
* bindings/v8/ScriptController.cpp:
* bindings/v8/ScriptController.h:
(ScriptController):

Source/WebKit/chromium: 

This patch just lines these WebCore functions because we only call them
from here.

* src/WebFrameImpl.cpp:
(WebKit::WebFrame::frameForEnteredContext):
(WebKit::WebFrame::frameForCurrentContext):


git-svn-id: http://svn.webkit.org/repository/webkit/trunk@128123 268f45cc-cd09-0410-ab3c-d52691b4dbfc
[V8] currentDocument doesn't need to handle the case where there is no v8::Context
https://bugs.webkit.org/show_bug.cgi?id=96312

Patch by Adam Barth <abarth@chromium.org> on 2012-09-10
Reviewed by Eric Seidel.

This function is always called while V8 is on the stack. It doesn't
need to handle this impossible error condition.

* bindings/v8/BindingState.cpp:
(WebCore::currentDocument):


git-svn-id: http://svn.webkit.org/repository/webkit/trunk@128124 268f45cc-cd09-0410-ab3c-d52691b4dbfc
[V8] V8AuxiliaryContext used by IDB leaks memory
https://bugs.webkit.org/show_bug.cgi?id=96317

Patch by Adam Barth <abarth@chromium.org> on 2012-09-10
Reviewed by Tony Chang.

Source/WebCore: 

Before this patch, we leaked m_auxiliaryContext on V8PerIsolateData
because no one ever called Dispose to balance the call to
V8::Context::New. This patch uses ScopedPersistent to call Dispose
automatically.

Also, I've deleted the V8AuxiliaryContext because it just reinvents
V8::Context::Scope.

* bindings/v8/IDBBindingUtilities.cpp:
(WebCore::createIDBKeyFromSerializedValueAndKeyPath):
(WebCore::injectIDBKeyIntoSerializedValue):
* bindings/v8/V8PerIsolateData.cpp:
(WebCore::V8PerIsolateData::ensureAuxiliaryContext):
(WebCore):
* bindings/v8/V8PerIsolateData.h:
(V8PerIsolateData):
* bindings/v8/V8Utilities.cpp:
* bindings/v8/V8Utilities.h:

Source/WebKit/chromium: 

Call the V8 APIs directly instead of using a helper class.

* tests/IDBBindingUtilitiesTest.cpp:
(WebCore::TEST):


git-svn-id: http://svn.webkit.org/repository/webkit/trunk@128125 268f45cc-cd09-0410-ab3c-d52691b4dbfc
[V8] V8LazyEventListener::prepareListenerObject uses inefficient string operations
https://bugs.webkit.org/show_bug.cgi?id=96324

Patch by Adam Barth <abarth@chromium.org> on 2012-09-10
Reviewed by Kentaro Hara.

Previously, this code called String::append several times, which
mallocs a new string buffer each time. This patch switches this code to
use operator+, which uses StringAppend to optimize these operations
down to a single malloc.

* bindings/v8/V8LazyEventListener.cpp:
(WebCore::V8LazyEventListener::prepareListenerObject):


git-svn-id: http://svn.webkit.org/repository/webkit/trunk@128126 268f45cc-cd09-0410-ab3c-d52691b4dbfc
[V8] V8PerIsolateData::m_toStringTemplate leaks memory
https://bugs.webkit.org/show_bug.cgi?id=96322

Patch by Adam Barth <abarth@chromium.org> on 2012-09-10
Reviewed by Kentaro Hara.

We never call Dispose to balance the call to v8::Persistent::New. This
patch uses ScopedPersistent to call Dispose automatically.

* bindings/v8/V8PerIsolateData.cpp:
(WebCore::V8PerIsolateData::toStringTemplate):
    - I've changed this function to return a new local handle in case
      anyone is trying to hold on to these handles during shutdown.
* bindings/v8/V8PerIsolateData.h:
(V8PerIsolateData):


git-svn-id: http://svn.webkit.org/repository/webkit/trunk@128127 268f45cc-cd09-0410-ab3c-d52691b4dbfc
Reduce the number of treeScope queries when matching rules.
https://bugs.webkit.org/show_bug.cgi?id=96315

Reviewed by Eric Seidel.

In the worst case, treeScope() requires a HashMap lookup, which could be slow. Since the value is always the same for the element,
we can just ask for it once.

No new tests, no change in behavior.

* css/StyleResolver.cpp:
(WebCore::StyleResolver::collectMatchingRulesForList): Moved the call to treeScope out of the loop.


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