[V8] Remove evaluateInIsolatedWorld() from V8Proxy
https://bugs.webkit.org/show_bug.cgi?id=93679

Reviewed by Adam Barth.

To remove V8Proxy, this patch moves V8Proxy::evaluateInIsolatedWorld()
to ScriptController.

No tests. No change in behavior.

* bindings/v8/ScriptController.cpp:
(WebCore::ScriptController::evaluateInIsolatedWorld):
* bindings/v8/V8Proxy.cpp:
(WebCore::V8Proxy::precompileScript):

git-svn-id: http://svn.webkit.org/repository/webkit/trunk@125475 268f45cc-cd09-0410-ab3c-d52691b4dbfc
Expand list of supported languages for RenderQuote to match WHATWG spec
https://bugs.webkit.org/show_bug.cgi?id=93424

Patch by Elliott Sprehn <esprehn@chromium.org> on 2012-08-13
Reviewed by Eric Seidel.

Source/WebCore:

Expand the table of quotes to include the 148 lang values and associated quotes
from the WHATWG web-apps recommended styling spec.

Test: fast/css-generated-content/quotes-lang-case-insensitive.html

* rendering/RenderQuote.cpp: Added missing copyright.
(WebCore::quotesDataLanguageMap): Expanded the table of quotes.
(WebCore::basicQuotesData): Added a FIXME to switch to using stylized "en" quotes.
* rendering/RenderQuote.h: Added missing copyright.

LayoutTests:

Update quote language test to include the 148 languages in the WHATWG web-apps
recommended styling table.
http://www.whatwg.org/specs/web-apps/current-work/multipage/rendering.html#quotes

Add a new test that ensures language matching for quotes is case insensitive.

* fast/css-generated-content/quotes-lang-case-insensitive-expected.html: Added.
* fast/css-generated-content/quotes-lang-case-insensitive.html: Added.
* fast/css-generated-content/quotes-lang-expected.html:
* fast/css-generated-content/quotes-lang.html:

git-svn-id: http://svn.webkit.org/repository/webkit/trunk@125476 268f45cc-cd09-0410-ab3c-d52691b4dbfc
[V8] Move V8PerIsolateData::visitExternalStrings() to V8PerIsolateData.cpp
https://bugs.webkit.org/show_bug.cgi?id=93803

Reviewed by Adam Barth.

In r125015, I forgot to move V8PerIsolateData::visitExternalStrings()
to V8PerIsolateData.cpp. This patch moves it.

No tests. No change in behavior.

* bindings/v8/V8Binding.cpp:
* bindings/v8/V8PerIsolateData.cpp:
(WebCore):
(WebCore::V8PerIsolateData::visitExternalStrings):

git-svn-id: http://svn.webkit.org/repository/webkit/trunk@125477 268f45cc-cd09-0410-ab3c-d52691b4dbfc
[V8] Remove unused methods and headers from V8Utilities
https://bugs.webkit.org/show_bug.cgi?id=93805

Reviewed by Adam Barth.

This patch removes unused methods, forward declarations and
headers from V8Utilities.{h,cpp}.

No tests. No change in behavior.

* bindings/v8/V8Utilities.cpp:
* bindings/v8/V8Utilities.h:
(WebCore):

git-svn-id: http://svn.webkit.org/repository/webkit/trunk@125478 268f45cc-cd09-0410-ab3c-d52691b4dbfc
[V8] Remove V8BindingHelpers
https://bugs.webkit.org/show_bug.cgi?id=93807

Reviewed by Adam Barth.

V8BindingHelpers contains toV8Context() and toV8Proxy(), which are used
by NPV8Object.cpp only.

(1) Move these methods to NPV8Object.cpp as static methods.
(2) Remove V8BindingHelpers.{h,cpp}.

No tests. No change in behavior.

* UseV8.cmake:
* WebCore.gypi:
* bindings/v8/NPV8Object.cpp:
(WebCore::toV8Context):
(WebCore):
(WebCore::toV8Proxy):
* bindings/v8/V8Binding.h:
* bindings/v8/V8BindingHelpers.cpp: Removed.
* bindings/v8/V8BindingHelpers.h: Removed.

git-svn-id: http://svn.webkit.org/repository/webkit/trunk@125479 268f45cc-cd09-0410-ab3c-d52691b4dbfc
[V8] Move the ConstructorMode class from V8Binding.h to SafeAllocation.h
https://bugs.webkit.org/show_bug.cgi?id=93821

Reviewed by Adam Barth.

This patch factors out the ConstructorMode class from V8Binding.h to SafeAllocation.h.

No tests. No change in behavior.

* bindings/v8/SafeAllocation.h:
(ConstructorMode):
(WebCore::ConstructorMode::ConstructorMode):
(WebCore::ConstructorMode::~ConstructorMode):
(WebCore::ConstructorMode::current):
(WebCore):
* bindings/v8/V8Binding.h:
* bindings/v8/V8RecursionScope.h:

git-svn-id: http://svn.webkit.org/repository/webkit/trunk@125480 268f45cc-cd09-0410-ab3c-d52691b4dbfc
[V8] Remove v8ValueToWebCoreString()
https://bugs.webkit.org/show_bug.cgi?id=93822

Reviewed by Adam Barth.

There are two equivalent ways to convert a V8 value to a WebCore String;
i.e. v8ValueToWebCoreString() and toWebCoreString(). We can remove the former.

In a follow-up patch, I will replace v8ValueToAtomicWebCoreString()
with toWebCoreAtomicString(), for consistency with toWebCoreString().

No tests. No change in behavior.

* bindings/v8/Dictionary.cpp:
(WebCore::Dictionary::get):
(WebCore::Dictionary::getWithUndefinedOrNullCheck):
(WebCore::Dictionary::getOwnPropertiesAsStringHashMap):
* bindings/v8/IDBBindingUtilities.cpp:
(WebCore::createIDBKeyFromValue):
* bindings/v8/ScriptDebugServer.cpp:
(WebCore::ScriptDebugServer::setBreakpoint):
* bindings/v8/V8Binding.cpp:
(WebCore::toWebCoreString):
(WebCore::v8ValueToWebCoreDOMStringList):
* bindings/v8/V8Binding.h:
(WebCore::toWebCoreStringWithNullCheck):
* bindings/v8/custom/V8MessageEventCustom.cpp:
(WebCore::V8MessageEvent::initMessageEventCallback):

git-svn-id: http://svn.webkit.org/repository/webkit/trunk@125481 268f45cc-cd09-0410-ab3c-d52691b4dbfc
[V8] Rename v8ValueToAtomicWebCoreString() to toWebCoreAtomicString()
https://bugs.webkit.org/show_bug.cgi?id=93824

Reviewed by Adam Barth.

For naming consistency with toWebCoreString(), we can rename
v8ValueToAtomicWebCoreString() to toWebCoreAtomicString().
Also we can rename v8ValueToAtomicWebCoreStringWithNullCheck()
to toWebCoreAtomicStringWithNullCheck().

No tests. No change in behavior.

* bindings/scripts/CodeGeneratorV8.pm:
(GenerateNormalAttrSetter):
(GenerateEventListenerCallback):
* bindings/scripts/test/V8/V8TestEventTarget.cpp:
(WebCore::TestEventTargetV8Internal::addEventListenerCallback):
(WebCore::TestEventTargetV8Internal::removeEventListenerCallback):
* bindings/scripts/test/V8/V8TestObj.cpp:
(WebCore::TestObjV8Internal::addEventListenerCallback):
(WebCore::TestObjV8Internal::removeEventListenerCallback):
* bindings/v8/V8Binding.cpp:
(WebCore::toWebCoreAtomicString):
* bindings/v8/V8Binding.h:
(WebCore::toWebCoreStringWithNullOrUndefinedCheck):
(WebCore::toWebCoreAtomicStringWithNullCheck):
* bindings/v8/V8Collection.h:
(WebCore::getNamedPropertyOfCollection):
* bindings/v8/V8DOMWindowShell.cpp:
(WebCore::getter):
* bindings/v8/custom/V8DOMWindowCustom.cpp:
(WebCore::V8DOMWindow::namedPropertyGetter):
* bindings/v8/custom/V8HTMLAllCollectionCustom.cpp:
(WebCore::V8HTMLAllCollection::namedPropertyGetter):
* bindings/v8/custom/V8HTMLCollectionCustom.cpp:
(WebCore::V8HTMLCollection::namedPropertyGetter):
* bindings/v8/custom/V8HTMLFormElementCustom.cpp:
(WebCore::V8HTMLFormElement::namedPropertyGetter):
* bindings/v8/custom/V8HTMLFrameSetElementCustom.cpp:
(WebCore::V8HTMLFrameSetElement::namedPropertyGetter):
* bindings/v8/custom/V8NodeListCustom.cpp:
(WebCore::V8NodeList::namedPropertyGetter):

git-svn-id: http://svn.webkit.org/repository/webkit/trunk@125482 268f45cc-cd09-0410-ab3c-d52691b4dbfc
[V8] Rename v8ValueToWebCoreDOMStringList() to toDOMStringList()
https://bugs.webkit.org/show_bug.cgi?id=93839

Reviewed by Adam Barth.

For naming consistency with toWebCoreString() and toWebCoreAtomicString(),
this patch renames v8ValueToWebCoreDOMStringList() to toDOMStringList().
("WebCore" wouldn't be needed because it's clear from "DOMStringList".)

No tests. No change in behavior.

* bindings/scripts/CodeGeneratorV8.pm:
(JSValueToNative):
* bindings/scripts/test/V8/V8TestObj.cpp:
(WebCore::TestObjV8Internal::overloadedMethod6Callback):
(WebCore::TestObjV8Internal::overloadedMethod7Callback):
(WebCore::TestObjV8Internal::overloadedMethod9Callback):
(WebCore::TestObjV8Internal::stringArrayFunctionCallback):
* bindings/v8/V8Binding.cpp:
(WebCore::toDOMStringList):
* bindings/v8/V8Binding.h:
(WebCore):

git-svn-id: http://svn.webkit.org/repository/webkit/trunk@125483 268f45cc-cd09-0410-ab3c-d52691b4dbfc
[V8] Remove [TreatReturnedNullAs=False]
https://bugs.webkit.org/show_bug.cgi?id=93835

Reviewed by Adam Barth.

[TreatReturnedNullAs=False] is neither defined in the Web IDL spec
nor used in any WebKit IDL file.

No tests. No change in behavior.

* WebCore.order:
* bindings/js/JSDOMBinding.cpp:
* bindings/js/JSDOMBinding.h:
(WebCore):
* bindings/scripts/CodeGeneratorJS.pm:
(NativeToJSValue):
* bindings/scripts/CodeGeneratorV8.pm:
(NativeToJSValue):
* bindings/scripts/IDLAttributes.txt:
* bindings/scripts/test/CPP/WebDOMTestObj.cpp:
* bindings/scripts/test/CPP/WebDOMTestObj.h:
* bindings/scripts/test/GObject/WebKitDOMTestObj.cpp:
* bindings/scripts/test/GObject/WebKitDOMTestObj.h:
* bindings/scripts/test/JS/JSTestObj.cpp:
(WebCore):
* bindings/scripts/test/JS/JSTestObj.h:
(WebCore):
* bindings/scripts/test/ObjC/DOMTestObj.h:
* bindings/scripts/test/ObjC/DOMTestObj.mm:
* bindings/scripts/test/TestObj.idl:
* bindings/scripts/test/V8/V8TestObj.cpp:
(WebCore::ConfigureV8TestObjTemplate):
* bindings/v8/V8Binding.h:

git-svn-id: http://svn.webkit.org/repository/webkit/trunk@125484 268f45cc-cd09-0410-ab3c-d52691b4dbfc
Enable CSS Sticky Position on Windows.
https://bugs.webkit.org/show_bug.cgi?id=93905

Patch by Roger Fong <roger_fong@apple.com> on 2012-08-13
Reviewed by Tim Horton.

As per https://bugs.webkit.org/show_bug.cgi?id=90046, the ENABLE_CSS_STICKY_POSITION needs to be turned on for Windows as well.

* win/tools/vsprops/FeatureDefines.vsprops:
Include ENABLE_CSS_STICKY_POSITION flag.

git-svn-id: http://svn.webkit.org/repository/webkit/trunk@125485 268f45cc-cd09-0410-ab3c-d52691b4dbfc
[chromium] renderSurface in incorrect space if owning layer has empty but non-zero bounds
https://bugs.webkit.org/show_bug.cgi?id=93795

Reviewed by Adrienne Walker.

Source/WebCore:

If a renderSurface is created by a layer that had zero
area (empty) but non-zero bounds (either width or height was
non-zero), then one translation transform was accidentally being
skipped, causing the renderSurface drawTransform to be incorrect.
The fix is simply to move that transform outside of the
if-statement so it is not skipped.

Unit test added:
  CCLayerTreeHostCommonTest.verifyTransformsForDegenerateIntermediateLayer()

* platform/graphics/chromium/cc/CCLayerTreeHostCommon.cpp:
(WebCore::calculateDrawTransformsInternal):

Source/WebKit/chromium:

* tests/CCLayerTreeHostCommonTest.cpp:

git-svn-id: http://svn.webkit.org/repository/webkit/trunk@125486 268f45cc-cd09-0410-ab3c-d52691b4dbfc
Unreviewed update for webcomponents-bugzilla address.

* Scripts/webkitpy/common/config/committers.py:

git-svn-id: http://svn.webkit.org/repository/webkit/trunk@125487 268f45cc-cd09-0410-ab3c-d52691b4dbfc
Add test to demonstrate overpainting seen in bug 92800
https://bugs.webkit.org/show_bug.cgi?id=93901

Reviewed by Levi Weintraub.

Bug 92800 is a rather complicated change to fix this overpainting bug.
Adding the test before the change so that it's easier for me to track as I change
this behavior and for reviewers to see that I'm fixing this overpainting.

* fast/repaint/add-table-overpaint-expected.png: Added.
* fast/repaint/add-table-overpaint-expected.txt: Added.
* fast/repaint/add-table-overpaint.html: Added.


git-svn-id: http://svn.webkit.org/repository/webkit/trunk@125488 268f45cc-cd09-0410-ab3c-d52691b4dbfc
Unreviewed, rolling out r124970.
http://trac.webkit.org/changeset/124970
https://bugs.webkit.org/show_bug.cgi?id=93495

Postponing the merge in a clearer context (Requested by
Michelangelo on #webkit).

Patch by Sheriff Bot <webkit.review.bot@gmail.com> on 2012-08-13

* css/CSSParser.cpp:
(WebCore::CSSParser::parseValue):
(WebCore::CSSParser::parseTransform):
* css/CSSParser.h:

git-svn-id: http://svn.webkit.org/repository/webkit/trunk@125489 268f45cc-cd09-0410-ab3c-d52691b4dbfc
Unreviewed rebaseline after r125193.

* platform/chromium-mac/compositing/masks/layer-mask-placement-expected.png:
* platform/chromium-win/compositing/masks/layer-mask-placement-expected.png:
* platform/chromium/TestExpectations:

git-svn-id: http://svn.webkit.org/repository/webkit/trunk@125490 268f45cc-cd09-0410-ab3c-d52691b4dbfc
[V8] Move hintForGCIfNecessary() from V8Proxy.h to V8GCController.h
https://bugs.webkit.org/show_bug.cgi?id=93834

Reviewed by Adam Barth.

This patch moves hintForGCIfNecessary() from V8Proxy.h to V8GCController.h.

No tests. No change in behavior.

* bindings/v8/V8GCController.cpp:
(WebCore::V8GCController::hintForGCIfNecessary):
* bindings/v8/V8GCController.h:
(V8GCController):
* bindings/v8/V8Proxy.cpp:
(WebCore::V8Proxy::clearForClose):
(WebCore::V8Proxy::clearForNavigation):
* bindings/v8/V8Proxy.h:
(V8Proxy):

git-svn-id: http://svn.webkit.org/repository/webkit/trunk@125491 268f45cc-cd09-0410-ab3c-d52691b4dbfc
[V8] Remove unused methods from V8Binding.h
https://bugs.webkit.org/show_bug.cgi?id=93836

Reviewed by Adam Barth.

enableStringImplCache() and v8UndetectableString() are not used by anybody.

No tests. No change in behavior.

* bindings/v8/V8Binding.h:

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