Timeline



Jan 5, 2012:

10:41 PM Changeset in webkit [104269] by mitz@apple.com
  • 14 edits in trunk

<rdar://problem/10633760> Update copyright strings

Reviewed by Mark Rowe.

Source/JavaScriptCore:

  • Info.plist:

Source/JavaScriptGlue:

  • Info.plist:

Source/WebCore:

  • Info.plist:

Source/WebKit/mac:

  • Info.plist:

Source/WebKit2:

  • Info.plist:
  • PluginProcess/Info.plist:
  • WebProcess/Info.plist:

WebKitLibraries:

  • win/tools/scripts/COPYRIGHT-END-YEAR:
9:36 PM Changeset in webkit [104268] by alexis.menard@openbossa.org
  • 13 edits
    1 copy
    1 add in trunk/Source/WebCore

Move HTMLFormControlElementWithState class in its own header file.
​https://bugs.webkit.org/show_bug.cgi?id=75482

Reviewed by Kent Tamura.

Move HTMLFormControlElementWithState class which was mixed in HTMLFormControlElement
files into its own header file and its own implementation file.

No new tests : the existing ones should cover the refactoring.

  • CMakeLists.txt:
  • GNUmakefile.list.am:
  • Target.pri:
  • WebCore.gypi:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • html/HTMLElementsAllInOne.cpp:
  • html/HTMLFormControlElement.cpp:
  • html/HTMLFormControlElement.h:
  • html/HTMLKeygenElement.h:
  • html/HTMLSelectElement.h:
  • html/HTMLTextFormControlElement.h:
9:29 PM Changeset in webkit [104267] by barraclough@apple.com
  • 5 edits in trunk

Date constructor handles infinite values incorrectly.
​https://bugs.webkit.org/show_bug.cgi?id=70998

Reviewed by Filip Pizlo.

Source/JavaScriptCore:

  • runtime/DateConstructor.cpp:

(JSC::constructDate):

  • should be checking !finite rather then isnan.

LayoutTests:

  • fast/js/date-constructor-expected.txt:
  • fast/js/script-tests/date-constructor.js:
    • Added test cases.
8:48 PM Changeset in webkit [104266] by barraclough@apple.com
  • 5 edits in trunk

date.toISOString produces incorrect results for dates with ms prior to 1970
​https://bugs.webkit.org/show_bug.cgi?id=75684

Reviewed by Sam Weinig.

Source/JavaScriptCore:

  • runtime/DatePrototype.cpp:

(JSC::dateProtoFuncToISOString):

LayoutTests:

  • fast/js/date-toisostring-expected.txt:
  • fast/js/script-tests/date-toisostring.js:
    • Added test case
8:01 PM Changeset in webkit [104265] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebCore

Optimize with memcpy instead of copying frame by frame in Realtimeanalyser::doFFTAnalysis
​https://bugs.webkit.org/show_bug.cgi?id=74693

Patch by Wei James <​james.wei@intel.com> on 2012-01-05
Reviewed by Kenneth Russell.

  • webaudio/RealtimeAnalyser.cpp:

(WebCore::RealtimeAnalyser::doFFTAnalysis):

7:20 PM Changeset in webkit [104264] by jochen@chromium.org
  • 3 edits in trunk/Tools

Don't hardcode expected file URIs in as they depend on the platform
​https://bugs.webkit.org/show_bug.cgi?id=75666

Reviewed by Dirk Pranke.

  • Scripts/webkitpy/layout_tests/port/win.py:

(WinPort.show_results_html_file):

  • Scripts/webkitpy/layout_tests/run_webkit_tests_integrationtest.py:

(MainTest.test_unexpected_failures):
(MainTest.test_results_directory_default):
(MainTest.test_results_directory_relative):

7:14 PM Changeset in webkit [104263] by rniwa@webkit.org
  • 3 edits in trunk/Source/WebCore

REGRESSION(r104210): Dromaeo DOM test score is lower
​https://bugs.webkit.org/show_bug.cgi?id=75679

Reviewed by Andreas Kling.

The regression was caused by isDomVersionConsistent not being able to obtain the tree version
inside isLengthCacheValid when m_cachedItem is null. Fix the regression by always setting
m_cachedItem to some node when caching the length so that we can obtain the tree version later.

Also address Antti's review comment to fit m_cachedLength, m_isLengthCacheValid, and
m_isItemCacheValid all in 32-bit.

  • dom/DynamicNodeList.cpp:

(WebCore::DynamicSubtreeNodeList::SubtreeCaches::setLengthCache):

7:10 PM Changeset in webkit [104262] by ojan@chromium.org
  • 2 edits
    1 add
    2 deletes in trunk/LayoutTests

More chromium expected results after r104208.

  • platform/chromium-cg-mac-leopard/fast/table/032-expected.png: Added.
  • platform/chromium-mac-leopard/fast/table/032-expected.txt: Removed.
  • platform/chromium-win-xp/fast/table/032-expected.png:
  • platform/chromium-win-xp/fast/table/032-expected.txt: Removed.
6:46 PM Changeset in webkit [104261] by morrita@google.com
  • 2 edits in trunk/Source/WebCore

Unreviewed bad merge fix for r104259 which dropped a line from r104210.

  • dom/TreeScopeAdopter.cpp:

(WebCore::TreeScopeAdopter::moveTreeToNewScope):

6:13 PM Changeset in webkit [104260] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebCore

​https://bugs.webkit.org/show_bug.cgi?id=75593
Patch by Yongjun Zhang <​yongjun_zhang@apple.com> on 2012-01-05
Reviewed by Alexey Proskuryakov.

Null-check 'page' variable before use, to follow the common usage pattern of m_frame->page()
throughout the rest of FrameView.cpp.

  • page/FrameView.cpp:

(WebCore::FrameView::notifyPageThatContentAreaWillPaint):

5:58 PM Changeset in webkit [104259] by morrita@google.com
  • 15 edits
    2 adds in trunk/Source/WebCore

[Refactoring] Moving between TreeScopes should be done by its own class.
​https://bugs.webkit.org/show_bug.cgi?id=75290

Reviewed by Ryosuke Niwa.

This change extracted Node::setTreeScopeRecursively(),
setDocumentRecursively() and a part of setDocument() into a new
class called TreeScopeAdopter. By doing this, the idea of
moving a node from scope to scope, that was originally hidden
behind the forest of Node APIs, has become clearer.

Note that this change is a preparation for Bug 59816.

No new tests. No behavioral change.

  • CMakeLists.txt:
  • GNUmakefile.list.am:
  • Target.pri:
  • WebCore.gypi:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • dom/ContainerNode.cpp: Followed the renaming.

(WebCore::ContainerNode::takeAllChildrenFrom):
(WebCore::ContainerNode::insertBefore):
(WebCore::ContainerNode::replaceChild):
(WebCore::ContainerNode::removeBetween):
(WebCore::ContainerNode::removeChildren):
(WebCore::ContainerNode::appendChild):
(WebCore::ContainerNode::parserAddChild):

  • dom/DOMAllInOne.cpp:
  • dom/Document.cpp: Followed te renaming.

(WebCore::Document::setDocType):
(WebCore::Document::adoptNode):

  • dom/Element.cpp: Followed te renaming.

(WebCore::Element::removeShadowRoot):

  • dom/Node.cpp:

(WebCore::Node::setDocument):
(WebCore::Node::setTreeScope):
(WebCore::Node::didMoveToNewDocument):

  • dom/Node.h:
  • dom/TreeScope.h:
  • dom/TreeScope.cpp:

(WebCore::TreeScope::adoptIfNeeded): moved from setTreeScopeRecursively()

  • dom/TreeScopeAdopter.cpp: Added.

(WebCore::TreeScopeAdopter::TreeScopeAdopter):
(WebCore::TreeScopeAdopter::moveTreeToNewScope):
(WebCore::TreeScopeAdopter::moveTreeToNewDocument):
(WebCore::TreeScopeAdopter::ensureDidMoveToNewDocumentWasCalled):
(WebCore::TreeScopeAdopter::moveNodeToNewDocument):

  • dom/TreeScopeAdopter.h: Added.

(WebCore::TreeScopeAdopter::ensureDidMoveToNewDocumentWasCalled):
(WebCore::TreeScopeAdopter::execute):
(WebCore::TreeScopeAdopter::needsScopeChange()):
(WebCore::TreeScopeAdopter::shadowRootFor):

5:37 PM Changeset in webkit [104258] by barraclough@apple.com
  • 4 edits in trunk

Array.prototype.lastIndexOf ignores undefined fromIndex.
​https://bugs.webkit.org/show_bug.cgi?id=75678

Reviewed by Sam Weinig.

Source/JavaScriptCore:

array.lastIndexOf(x, undefined) is equivalent to array.lastIndexOf(x, 0), not array.lastIndexOf(x)

  • runtime/ArrayPrototype.cpp:

(JSC::arrayProtoFuncLastIndexOf):

  • should check argumnet count, rather than checking agument value for undefined.

LayoutTests:

  • ietestcenter/Javascript/15.4.4.15-5-4-expected.txt:
    • check in passing result
5:32 PM Changeset in webkit [104257] by jochen@chromium.org
  • 8 edits in trunk

Source/WebCore: Disallow access to DOM storage from detached frames.
​https://bugs.webkit.org/show_bug.cgi?id=61326

Reviewed by Adam Barth.

  • storage/StorageAreaImpl.cpp:

(WebCore::StorageAreaImpl::disabledByPrivateBrowsingInFrame):

Source/WebKit/chromium: Check whether a WebView exists before accessing it in StorageAreaProxy. This is not necessarily the case, e.g. for detached iframes.
​https://bugs.webkit.org/show_bug.cgi?id=61326

Reviewed by Adam Barth.

  • src/StorageAreaProxy.cpp:

(WebCore::StorageAreaProxy::canAccessStorage):

LayoutTests: Unskip fast/storage/storage-detached-iframe.html on chromium
​https://bugs.webkit.org/show_bug.cgi?id=61326

Reviewed by Adam Barth.

  • fast/storage/storage-detached-iframe-expected.txt:
  • fast/storage/storage-detached-iframe.html:
  • platform/chromium/test_expectations.txt:
5:32 PM Changeset in webkit [104256] by Lucas Forschler
  • 5 edits in trunk/Source

Versioning.

5:29 PM Changeset in webkit [104255] by noam.rosenthal@nokia.com
  • 2 edits in trunk/Source/WebCore

[Qt][Texmap] Convert shaders in TextureMapperGL to use a macro
​https://bugs.webkit.org/show_bug.cgi?id=75598

Use VERTEX_SHADER() and FRAGMENT_SHADER() macros, instead of quoted string literals when
declaring shaders in TextureMapperGL.

We need two macros to account for the differences between OpenGL and OpenGL ES2.

Reviewed by Martin Robinson.

5:19 PM Changeset in webkit [104254] by rniwa@webkit.org
  • 6 edits in trunk/Source/WebCore

sizeof(CSSRule) is 20 instead of 12 on Windows
​https://bugs.webkit.org/show_bug.cgi?id=75665

Reviewed by Darin Fisher.

Unlike gcc and clang, MSVC pads each consecutive member variables of the same type
in bitfields. e.g. if you have:

sturct AB {
unsigned m_1 : 31;
bool m_2 : 1;
}

then MSVC pads m_1 and allocates sizeof(unsigned) * 2 for AB whereas gcc and clang
only allocate sizeof(unsigned) * 1 for AB.

Fix the bloat by turning all bitfields in CSSRule either signed or unsigned integers.

  • css/CSSRule.cpp:
  • css/CSSRule.h:

(WebCore::CSSRule::sourceLine):
(WebCore::CSSRule::setSourceLine):
(WebCore::CSSRule::hasCachedSelectorText):
(WebCore::CSSRule::setHasCachedSelectorText):

  • css/CSSStyleRule.cpp:

(WebCore::CSSStyleRule::CSSStyleRule):
(WebCore::CSSStyleRule::cleanup):
(WebCore::CSSStyleRule::selectorText):
(WebCore::CSSStyleRule::setSelectorText):

  • css/CSSStyleRule.h:
5:17 PM Changeset in webkit [104253] by Lucas Forschler
  • 1 copy in tags/Safari-535.15

New Tag.

5:15 PM Changeset in webkit [104252] by dgrogan@chromium.org
  • 4 edits
    2 adds in trunk

Source/WebCore: IndexedDB: fix cursor prefetch crash
​http://crbug.com/108071
​https://bugs.webkit.org/show_bug.cgi?id=75596

Reviewed by Tony Chang.

Test: storage/indexeddb/prefetch-bugfix-108071.html
Note: DumpRenderTree doesn't exercise the bug, it only occurs in
multi-process chromium. The layout test will soon be run as a
chromium ui test: ​http://codereview.chromium.org/9108004

  • storage/IDBCursorBackendImpl.cpp:

(WebCore::IDBCursorBackendImpl::IDBCursorBackendImpl):
(WebCore::IDBCursorBackendImpl::~IDBCursorBackendImpl): Destroy
cursors before their objectstores.
(WebCore::IDBCursorBackendImpl::prefetchReset): Don't run continue if
the cursor is closed.
(WebCore::IDBCursorBackendImpl::close): Set a closed flag.

  • storage/IDBCursorBackendImpl.h:

LayoutTests: IndexedDB: fix cursor prefetch crash
​https://bugs.webkit.org/show_bug.cgi?id=75596

Reviewed by Tony Chang.

  • storage/indexeddb/prefetch-bugfix-108071-expected.txt: Added.
  • storage/indexeddb/prefetch-bugfix-108071.html: Added.
5:08 PM Changeset in webkit [104251] by barraclough@apple.com
  • 5 edits in trunk

Date parsing is too restrictive.
​https://bugs.webkit.org/show_bug.cgi?id=75671

Reviewed by Oliver Hunt.

Source/JavaScriptCore:

ES5 date parsing currently requires all fields to be present, which does not match the spec (ES5.1 15.9.1.15).
The spec allow a date to be date only, or date + time.

The date portion on the should match: (pseudocode!:)

[(+)YY]YYYY[-MM[-DD]]

though we are slightly more liberal (permitted by the spec), allowing:

[+]Y+[-MM[-DD]]

The time portion should match:

THH:mm[:ss[.sss]][Z|(+|-)HH:mm]

again we're slightly more liberal, allowing:

THH:mm[:ss[.s+]][Z|(+|-)HH:mm]

  • wtf/DateMath.cpp:

(WTF::parseES5DatePortion):

  • Month/day fields are optional, default to 01.

(WTF::parseES5TimePortion):

  • Hours/Minutes are requires, seconds/timezone are optional.

(WTF::parseES5DateFromNullTerminatedCharacters):

  • Dates may be date only, or date + time.

LayoutTests:

  • fast/js/date-parse-test-expected.txt:
  • fast/js/script-tests/date-parse-test.js:
    • Add more test cases for Date.parse.
5:05 PM Changeset in webkit [104250] by Lucas Forschler
  • 4 edits
    3 copies in branches/safari-534.54-branch

Merged r100203.

5:02 PM Changeset in webkit [104249] by Lucas Forschler
  • 2 edits in branches/safari-534.54-branch/Source/WebCore

Merged r99999.

5:01 PM Changeset in webkit [104248] by jamesr@google.com
  • 13 edits in trunk/Source

[chromium] Route all animate calls through CCLayerTreeHost in composited mode to simplify rate limiting logic
​https://bugs.webkit.org/show_bug.cgi?id=75577

Reviewed by Darin Fisher.

Source/WebCore:

This internalizes the animation rate limiting logic to CCLayerTreeHost and removes the setters/getters for the
m_animating flag. This requires that all animation updates have to go through CCLayerTreeHost to get the right
rate limiting behavior, regardless of which proxy is being used.

  • platform/graphics/chromium/cc/CCLayerTreeHost.cpp:

(WebCore::CCLayerTreeHost::updateAnimations):
(WebCore::CCLayerTreeHost::layout):
(WebCore::CCLayerTreeHost::startRateLimiter):

  • platform/graphics/chromium/cc/CCLayerTreeHost.h:
  • platform/graphics/chromium/cc/CCThreadProxy.cpp:

(WebCore::CCThreadProxy::beginFrameAndCommit):

Source/WebKit/chromium:

Routes WebWidget-initiated animate() calls through the CCLayerTreeHost in composited mode. CCLayerTreeHost's
rate limiting logic needs to be aware of when requestAnimationFrame callbacks are run. In threaded mode, the
animate calls are driven from CCThreadProxy and so the CCLayerTreeHost can set whatever state it needs. This
makes the single-threaded mode where the animate calls are driven through the WebWidget interface outside of the
CCProxy's control.

This is a small step towards inverting the scheduling control from the WebWidget to the compositor.

  • src/WebViewImpl.cpp:

(WebKit::WebViewImpl::animate):
(WebKit::WebViewImpl::updateAnimations):

  • src/WebViewImpl.h:
4:54 PM Changeset in webkit [104247] by eae@chromium.org
  • 1 edit in branches/subpixellayout/Source/WebCore/dom/Element.cpp

Fix rounding for adjustForLocalZoom

4:44 PM Changeset in webkit [104246] by commit-queue@webkit.org
  • 3 edits in trunk/Source/WebCore

[Chromium] NativeImageSkia should mark SkBitmaps as immutable
​https://bugs.webkit.org/show_bug.cgi?id=74962

Removed m_isDataComplete from class NativeImageSkia. Instead, data
completeness will be tracked through SkBitmap::setImmutable/
isImmutable. The immutable state signifies that the pixel data
will no longer change for the lifetime of the bitmap, which corresponds
to the semantic of the old m_isDataComplete member. setImmutable is
also called on the cached resized bitmap, since it too is invariant for
its life time. Temporary resized bitmaps are also marked as immutable
since they technically are.

Patch by Justin Novosad <​junov@chromium.org> on 2012-01-05
Reviewed by Stephen White.

  • platform/graphics/skia/NativeImageSkia.cpp:

(WebCore::NativeImageSkia::NativeImageSkia):
(WebCore::NativeImageSkia::resizedBitmap):
(WebCore::NativeImageSkia::shouldCacheResampling):

  • platform/graphics/skia/NativeImageSkia.h:

(WebCore::NativeImageSkia::setDataComplete):
(WebCore::NativeImageSkia::isDataComplete):

4:42 PM Changeset in webkit [104245] by abarth@webkit.org
  • 2 edits in trunk/LayoutTests

Mark flaky test as flaky. Filed
​https://bugs.webkit.org/show_bug.cgi?id=75647 about making it not
flaky.

  • platform/chromium/test_expectations.txt:
4:39 PM Changeset in webkit [104244] by abarth@webkit.org
  • 5 edits in trunk/Source

Unreviewed, rolling out r104231.
​http://trac.webkit.org/changeset/104231
​https://bugs.webkit.org/show_bug.cgi?id=75668

Breaks the Qt build (Requested by abarth on #webkit).

Patch by Sheriff Bot <​webkit.review.bot@gmail.com> on 2012-01-05

Source/WebCore:

  • platform/SchemeRegistry.cpp:
  • platform/SchemeRegistry.h:

Source/WebKit/qt:

  • Api/qwebpage.cpp:

(QWebPage::acceptNavigationRequest):

4:36 PM Changeset in webkit [104243] by Lucas Forschler
  • 5 edits in branches/safari-534.54-branch

Merged r99756.

4:33 PM Changeset in webkit [104242] by Lucas Forschler
  • 5 edits
    2 copies in branches/safari-534.54-branch

Merged r99731.

4:32 PM Changeset in webkit [104241] by levin@chromium.org
  • 2 edits in trunk/Source/WebKit/chromium

[chromium] Resize requests at the view level should be ignored when in autoresize mode.
​https://bugs.webkit.org/show_bug.cgi?id=75651

Reviewed by Dmitry Titov.

  • src/WebViewImpl.cpp:

(WebKit::WebViewImpl::resize): The assert was here to catch a view that
tried to do a resize after setting autoresize. However, we found that generic
code tries to do resizes as well. Rather than special case each caller,
we'll just make this code ignore the resize call.

4:30 PM Changeset in webkit [104240] by Beth Dakin
  • 3 edits in trunk/Source/WebCore

​https://bugs.webkit.org/show_bug.cgi?id=75654
Text fields should draw using NSTextFieldCell instead of WebKitSystemInterface

Reviewed by John Sullivan.

This change should not have any affect on tests or real web sites. It just changed
the implementation under the hood to the more modern NSCell approach.

  • rendering/RenderThemeMac.h:
  • rendering/RenderThemeMac.mm:

(WebCore::RenderThemeMac::paintTextField):
(WebCore::RenderThemeMac::textField):

4:29 PM Changeset in webkit [104239] by andersca@apple.com
  • 7 edits
    2 adds in trunk

Crash when trying to invalidate the NPRuntimeObjectMap for a plug-in in a subframe
​https://bugs.webkit.org/show_bug.cgi?id=75667
<rdar://problem/10389454>

Reviewed by Kevin Decker.

Source/WebKit2:

NPRuntimeObjectMap::invalidate is called whenever a plug-in view is destroyed. If invalidate is called for an object map
whose plug-in has a null frame, we'd crash.

The plug-in will have a null frame if the plug-in view is destroyed because its containing frame has been removed from the document,
and if the plug-in view is being destroyed asynchronously due to the plug-in itself calling JavaScript that will remove the frame
(see PluginView::unprotectPluginFromDestruction).

The reason NPRuntimeObjectMap::invalidate will crash when the frame is null is because we were trying to access the frame's global
object, causing a null dereference. The reason we were trying to get at the frame's global object was to create a Strong handle to
a JSNPObject so we could stick the object in a vector so we could later iterate over the vector elements and call invalidate() on
each JSNPObject which will end up releasing the underlying NPObject.

However, it turns out that we don't need to stick the JSNPObject in a vector; we can just get the underlying NPObject directly and
stick that in a vector and then iterate over the NPObjects, releasing them.

  • WebProcess/Plugins/Netscape/NPRuntimeObjectMap.cpp:

(WebKit::NPRuntimeObjectMap::invalidate):

Tools:

Add an evaluate method to the plug-in test scriptable object that can be used to evaluate a given JS string.

  • DumpRenderTree/TestNetscapePlugIn/PluginTest.cpp:

(PluginTest::executeScript):

  • DumpRenderTree/TestNetscapePlugIn/PluginTest.h:
  • DumpRenderTree/TestNetscapePlugIn/Tests/NPRuntimeObjectFromDestroyedPlugin.cpp:

(NPRuntimeObjectFromDestroyedPlugin::ScriptableObject::hasMethod):
(NPRuntimeObjectFromDestroyedPlugin::ScriptableObject::invoke):
(NPRuntimeObjectFromDestroyedPlugin::ScriptableObject::hasProperty):

LayoutTests:

  • plugins/npruntime/object-from-destroyed-plugin-in-subframe-expected.txt: Added.
  • plugins/npruntime/object-from-destroyed-plugin-in-subframe.html: Added.
4:15 PM Changeset in webkit [104238] by Lucas Forschler
  • 3 edits
    2 copies in branches/safari-534.54-branch

Merged r99579.

4:12 PM Changeset in webkit [104237] by Lucas Forschler
  • 4 edits
    2 copies in branches/safari-534.54-branch

Merged r99462.

4:01 PM Changeset in webkit [104236] by rniwa@webkit.org
  • 2 edits in trunk/Source/WebCore

Add a compile-time assertion for the size of CSSValue
​https://bugs.webkit.org/show_bug.cgi?id=75635

Reviewed by Tony Chang.

Tightened the compile-time assertion.

  • css/CSSValue.cpp:
3:59 PM Changeset in webkit [104235] by commit-queue@webkit.org
  • 2 edits in trunk/Source/JavaScriptCore

[EFL] Undefined references to ICU_I18N symbols on WTF
​https://bugs.webkit.org/show_bug.cgi?id=75642

Unreviewed build fix.

Add ${ICU_I18N_LIBRARIES} to WTF_LIBRARIES on wtf efl platform cmake.
Some undefined references were ucol_setAttribute_44, ucol_close_44,
ucol_getAttribute_44...

Patch by Bruno Dilly <​bdilly@profusion.mobi> on 2012-01-05

  • wtf/PlatformEfl.cmake:
3:57 PM Changeset in webkit [104234] by eae@chromium.org
  • 6 edits in branches/subpixellayout/LayoutTests/platform/mac/fast/forms

Update mac test expectations for a couple of form tests.

3:57 PM Changeset in webkit [104233] by abarth@webkit.org
  • 10 edits in trunk/Source/WebCore

[V8] CodeGeneration for SerializedScriptValue doesn't play nice with [Constructor]
​https://bugs.webkit.org/show_bug.cgi?id=75641

Reviewed by David Levin.

Rather than generate getters for SerializedScriptValues, we eagerly
deserialize them into JavaScript objects. However, previously, we were
only doing that for DOM wrappers created by taking an existing C++
object and wrapping it. For objects created with Constructors (e.g.,
those with the [Constructor] attribute), we need to do this eager
deserialization during the constructor as well.

This bug isn't observable yet, but it is causing the WebIntent test
being added in Bug 73051 to fail.

Test: TestSerializedScriptValueInterface.idl

webintents/web-intents-api.html (after Bug 73051 lands)

  • bindings/scripts/CodeGeneratorV8.pm:

(GenerateEagerDeserialization):
(GenerateConstructorCallback):
(GenerateNamedConstructorCallback):
(GenerateImplementation):
(GenerateToV8Converters):

  • bindings/scripts/test/JS/JSTestSerializedScriptValueInterface.cpp:

(WebCore::JSTestSerializedScriptValueInterfaceConstructor::constructJSTestSerializedScriptValueInterface):
(WebCore::JSTestSerializedScriptValueInterfaceConstructor::getConstructData):

  • bindings/scripts/test/JS/JSTestSerializedScriptValueInterface.h:
  • bindings/scripts/test/TestSerializedScriptValueInterface.idl:
  • bindings/scripts/test/V8/V8TestInterface.cpp:

(WebCore::V8TestInterface::constructorCallback):

  • bindings/scripts/test/V8/V8TestNamedConstructor.cpp:

(WebCore::V8TestNamedConstructorConstructorCallback):

  • bindings/scripts/test/V8/V8TestObj.cpp:

(WebCore::V8TestObj::constructorCallback):

  • bindings/scripts/test/V8/V8TestSerializedScriptValueInterface.cpp:

(WebCore::V8TestSerializedScriptValueInterface::constructorCallback):
(WebCore::ConfigureV8TestSerializedScriptValueInterfaceTemplate):

  • bindings/scripts/test/V8/V8TestSerializedScriptValueInterface.h:
3:45 PM Changeset in webkit [104232] by ojan@chromium.org
  • 7 edits
    1 move
    4 adds
    2 deletes in trunk/LayoutTests

New chromium expected results after r104208.

  • platform/chromium-linux/fast/table/032-expected.png: Removed.
  • platform/chromium-linux/tables/mozilla/bugs/bug55694-expected.png:
  • platform/chromium-mac-leopard/fast/table/032-expected.png:
  • platform/chromium-mac-leopard/fast/table/032-expected.txt: Added.
  • platform/chromium-mac-leopard/tables/mozilla/bugs/bug55694-expected.png:
  • platform/chromium-mac-snowleopard/tables/mozilla/bugs/bug55694-expected.png: Added.
  • platform/chromium-mac/tables/mozilla/bugs/bug55694-expected.png: Removed.
  • platform/chromium-win-xp/fast/table/032-expected.png: Added.
  • platform/chromium-win-xp/fast/table/032-expected.txt: Renamed from LayoutTests/platform/chromium-win/fast/table/032-expected.txt.
  • platform/chromium-win/fast/table/032-expected.png:
  • platform/chromium-win/tables/mozilla/bugs/bug55694-expected.png:
  • platform/chromium-win/tables/mozilla/bugs/bug55694-expected.txt:
3:11 PM Changeset in webkit [104231] by abarth@webkit.org
  • 5 edits in trunk/Source

Introduce Platform namespace for WebCore/platform
​https://bugs.webkit.org/show_bug.cgi?id=75653

Reviewed by Eric Seidel.

Source/WebCore:

This patch introduces the Platform namespace for WebCore/platform.
Introducing this namespace will help us find and fix layering
violations in preparation for moving WebCore/platform to Platform.

  • platform/SchemeRegistry.cpp:
  • platform/SchemeRegistry.h:

Source/WebKit/qt:

  • Api/qwebpage.cpp:

(QWebPage::acceptNavigationRequest):

3:07 PM Changeset in webkit [104230] by dpranke@chromium.org
  • 7 edits in trunk/Tools

webkitpy: clean up port factory methods
​https://bugs.webkit.org/show_bug.cgi?id=75590

Reviewed by Eric Seidel.

This change consolidates much of the "factory method" logic
of determining which port objects to create for a given set
of configurations by merging the separate factory methods in
chromium_gpu and google_chrome into PortFactory so that at least
all of the logic is in one place.

  • Scripts/webkitpy/layout_tests/port/chromium_gpu.py:
  • Scripts/webkitpy/layout_tests/port/chromium_gpu_unittest.py:
  • Scripts/webkitpy/layout_tests/port/factory.py:

(_port_name_from_arguments_and_options):
(get):

  • Scripts/webkitpy/layout_tests/port/factory_unittest.py:

(test_google_chrome):

  • Scripts/webkitpy/layout_tests/port/google_chrome.py:

(GoogleChromeLinux32Port):
(GoogleChromeLinux32Port.baseline_search_path):
(test_expectations_overrides):
(architecture):
(GoogleChromeLinux64Port):
(GoogleChromeLinux64Port.baseline_search_path):
(GoogleChromeMacPort):
(GoogleChromeMacPort.baseline_search_path):
(GoogleChromeWinPort):
(GoogleChromeWinPort.baseline_search_path):

  • Scripts/webkitpy/layout_tests/port/google_chrome_unittest.py:

(_verify_baseline_path):

3:04 PM Changeset in webkit [104229] by ap@apple.com
  • 2 edits in trunk/Source/WebKit2

REGRESSION (r98912-r99538): Crash in WebKit::WebFrameLoaderClient::didDetectXSS
​https://bugs.webkit.org/show_bug.cgi?id=75578

Reviewed by Daniel Bates.

  • WebProcess/InjectedBundle/InjectedBundlePageLoaderClient.cpp:

(WebKit::InjectedBundlePageLoaderClient::didDetectXSSForFrame):
Check for a correct struct member existence.

2:58 PM Changeset in webkit [104228] by Lucas Forschler
  • 3 edits in branches/safari-534.54-branch/Source/WebKit/win

Merged r103479.

2:55 PM Changeset in webkit [104227] by Lucas Forschler
  • 14 edits in branches/safari-534.54-branch/Source

Merged r99439.

2:42 PM Changeset in webkit [104226] by Lucas Forschler
  • 2 edits in branches/safari-534.54-branch/Source/WebKit/mac

Merged r98970.

2:39 PM Changeset in webkit [104225] by Lucas Forschler
  • 2 edits in branches/safari-534.54-branch/Source/WebCore

Merged r98796.

2:37 PM Changeset in webkit [104224] by Lucas Forschler
  • 2 edits in branches/safari-534.54-branch/Source/WebKit2

Merged r98781.

2:35 PM Changeset in webkit [104223] by Lucas Forschler
  • 3 edits
    2 copies in branches/safari-534.54-branch

Merged r98763.

2:35 PM Changeset in webkit [104222] by eae@chromium.org
  • 2 edits in branches/subpixellayout/Source/WebCore

Remove unused pixelSnapped methods from RenderBox.

2:33 PM Changeset in webkit [104221] by Lucas Forschler
  • 3 edits in branches/safari-534.54-branch

Merged r98561.

2:30 PM Changeset in webkit [104220] by Lucas Forschler
  • 3 edits
    3 copies in branches/safari-534.54-branch

Merged r96294.

2:30 PM Changeset in webkit [104219] by ggaren@apple.com
  • 5 edits in trunk/Source/JavaScriptCore

Refined the fast path for StringImpl::hash()
​https://bugs.webkit.org/show_bug.cgi?id=75178

Reviewed by Darin Adler.

Moved the hash calculation code into an out-of-line function to clean up
the hot path.

No measurable benchmark change, but this knocks some samples off in
Instruments, and I think this is a step toward removing -fomit-frame-pointer.

  • wtf/text/StringImpl.cpp:

(WTF::StringImpl::hashSlowCase):

  • wtf/text/StringImpl.h:

(WTF::StringImpl::hash): The patch.

  • wtf/text/StringStatics.cpp:

(WTF::StringImpl::hashSlowCase): Abide by the cockamamie Windows build
scheme, which requires all out-of-line StringImpl functions used by
WebCore be defined in this file instead of StringImpl.cpp. (See ​http://trac.webkit.org/changeset/59187.)

2:29 PM Changeset in webkit [104218] by dslomov@google.com
  • 2 edits in trunk/LayoutTests

Unreviewed: Updating expecxtations for fast/media/viewport-media-query.
My understanding of the semantics was a bit off.

  • platform/chromium/test_expectations.txt:
2:04 PM Changeset in webkit [104217] by Lucas Forschler
  • 5 edits in branches/safari-534.54-branch/Source/WebKit2

Merged r98513.

2:03 PM Changeset in webkit [104216] by Lucas Forschler
  • 2 edits in branches/safari-534.54-branch/Source/WebCore

Merged r98374.

2:01 PM Changeset in webkit [104215] by Lucas Forschler
  • 2 edits in branches/safari-534.54-branch/Source/WebKit2

Merged r98362.

2:00 PM Changeset in webkit [104214] by Lucas Forschler
  • 3 edits
    2 copies in branches/safari-534.54-branch

Merged r98344.

1:58 PM Changeset in webkit [104213] by Lucas Forschler
  • 2 edits in branches/safari-534.54-branch/Source/WebKit2

Merged r98186.

1:56 PM Changeset in webkit [104212] by barraclough@apple.com
  • 5 edits in trunk

Literal tab in JSONString fails
​https://bugs.webkit.org/show_bug.cgi?id=71772

Reviewed by Oliver Hunt.

rfc4627 does not allow literal tab characters in JSON source.

Source/JavaScriptCore:

  • runtime/LiteralParser.cpp:

(JSC::isSafeStringCharacter):

  • do not allow literal tab in StrictJSON mode.

LayoutTests:

  • fast/js/JSON-parse-expected.txt:
  • fast/js/resources/JSON-parse.js:

(createTests.result):

  • Updated expected results.
1:56 PM Changeset in webkit [104211] by Lucas Forschler
  • 7 edits
    5 copies in branches/safari-534.54-branch

Merge 98033.

1:48 PM Changeset in webkit [104210] by rniwa@webkit.org
  • 11 edits in trunk/Source/WebCore

Inserting nodes is slow due to Node::notifyNodeListsAttributeChanged (20%+)
​https://bugs.webkit.org/show_bug.cgi?id=73853

Reviewed by Antti Koivisto.

Lazily invalidate the node list caches instead of invaliding them at the time of modification. We use
the DOM tree version to detect whether caches need to be invalidated or not. We now invalidate caches more
frequently after this patch (in particular, invalidates caches that are stored on nodes not present in
the ancestry of the modified nodes); however, our study on major Web sites such as Gmail, Facebook, Twitter,
etc... indicate that about 1% of real-world usage benefits from keeping the caches alive across different
DOM tree versions.

In order to invalidate caches lazily, this patch adds replaces the type of m_caches in DynamicSubtreeNodeList
by DynamicSubtreeNodeList::SubtreeCaches which encapsulates member variables in DynamicNodeList::Caches and
invalidates values as needed. Also this change allows m_caches to be allocated as a part of
DynamicSubtreeNodeList instead of a separate ref-counted object.

  • dom/Attr.cpp:

(WebCore::Attr::setValue):
(WebCore::Attr::childrenChanged):

  • dom/DynamicNodeList.cpp:

(WebCore::DynamicSubtreeNodeList::DynamicSubtreeNodeList):
(WebCore::DynamicSubtreeNodeList::length):
(WebCore::DynamicSubtreeNodeList::itemForwardsFromCurrent):
(WebCore::DynamicSubtreeNodeList::itemBackwardsFromCurrent):
(WebCore::DynamicSubtreeNodeList::item):
(WebCore::DynamicSubtreeNodeList::invalidateCache):
(WebCore::DynamicNodeList::Caches::create):
(WebCore::DynamicNodeList::Caches::reset):

  • dom/DynamicNodeList.h:

(WebCore::DynamicSubtreeNodeList::SubtreeCaches::SubtreeCaches): Added.
(WebCore::DynamicSubtreeNodeList::SubtreeCaches::isLengthCacheValid): Added.
(WebCore::DynamicSubtreeNodeList::SubtreeCaches::isItemCacheValid): Added.
(WebCore::DynamicSubtreeNodeList::SubtreeCaches::cachedLength): Added.
(WebCore::DynamicSubtreeNodeList::SubtreeCaches::cachedItem): Added.
(WebCore::DynamicSubtreeNodeList::SubtreeCaches::cachedItemOffset): Added.
(WebCore::DynamicSubtreeNodeList::SubtreeCaches::setLengthCache): Added.
(WebCore::DynamicSubtreeNodeList::SubtreeCaches::setItemCache): Added.
(WebCore::DynamicSubtreeNodeList::SubtreeCaches::reset): Added.
(WebCore::DynamicSubtreeNodeList::SubtreeCaches::domVersionIsConsistent): Added.

  • dom/Element.cpp:

(WebCore::Element::updateAfterAttributeChanged):

  • dom/Node.cpp:

(WebCore::Node::setTreeScopeRecursively): Clear caches when a node moves from one document to another.
(WebCore::Node::invalidateNodeListsCacheAfterAttributeChanged): Only clears child node list of Attr.
(WebCore::Node::invalidateNodeListsCacheAfterChildrenChanged): Only clears child node list.
(WebCore::NodeListsNodeData::invalidateCaches): Merged with invalidateCachesThatDependOnAttributes.

  • dom/Node.h:
  • dom/NodeRareData.h:
  • html/HTMLElement.cpp:

(WebCore::HTMLElement::parseMappedAttribute):

  • html/HTMLLabelElement.cpp:
  • html/HTMLLabelElement.h:
1:41 PM Changeset in webkit [104209] by wjmaclean@chromium.org
  • 3 edits
    1 copy in trunk/Source/WebKit/chromium

[chromium][aura] WebExternalTextureLayerImpl::drawsContent() returns incorrect value, causing accelerated content to not display in Aura desktop
​https://bugs.webkit.org/show_bug.cgi?id=75551

Reviewed by James Robinson.

  • WebKit.gypi:
  • src/WebExternalTextureLayerImpl.cpp:

(WebKit::WebExternalTextureLayerImpl::WebExternalTextureLayerImpl):

  • tests/WebExternalTextureLayerImplTest.cpp: Copied from Source/WebKit/chromium/src/WebExternalTextureLayerImpl.cpp.
1:39 PM Changeset in webkit [104208] by ojan@chromium.org
  • 8 edits
    3 adds in trunk

IE quirk for percentage size on a table element doesn't work with orthogonal writing modes
​https://bugs.webkit.org/show_bug.cgi?id=70195

Reviewed by Eric Seidel.

Source/WebCore:

Remove this quirk entirely. Mozilla and Opera don't implement it and IE
doesn't restrict the quirk to standards mode. As it's unlikely for
webkit-only content to hit this quirk (e.g. use tables for layout),
it should be relatively safe to match Mozilla/Opera here.

Test: fast/writing-mode/table-percent-width-quirk.html

  • rendering/RenderBlock.cpp:

(WebCore::RenderBlock::computeBlockPreferredLogicalWidths):

LayoutTests:

  • fast/table/032.html:
  • fast/writing-mode/table-percent-width-quirk-expected.png: Added.
  • fast/writing-mode/table-percent-width-quirk-expected.txt: Added.
  • fast/writing-mode/table-percent-width-quirk.html: Added.
  • platform/mac/fast/table/032-expected.png:
  • platform/mac/fast/table/032-expected.txt:
  • platform/mac/tables/mozilla/bugs/bug55694-expected.png:
  • platform/mac/tables/mozilla/bugs/bug55694-expected.txt:
1:37 PM Changeset in webkit [104207] by eae@chromium.org
  • 11 edits in branches/subpixellayout/Source

Change IntrinsicSize back to an IntSize and get rid of extraneous rounding.

1:25 PM Changeset in webkit [104206] by jochen@chromium.org
  • 6 edits in trunk/Tools

Replace webkitpy.common.system.filesystem.file_path_as_url with webkitpy.common.system.path.abspath_to_uri
​https://bugs.webkit.org/show_bug.cgi?id=75617

Reviewed by Dirk Pranke.

  • Scripts/webkitpy/common/system/filesystem.py:

(FileSystem.exists):

  • Scripts/webkitpy/common/system/filesystem_mock.py:

(MockFileSystem.exists):

  • Scripts/webkitpy/common/system/filesystem_unittest.py:

(GenericFileSystemTests.teardown_generic_test_dir):

  • Scripts/webkitpy/layout_tests/port/base.py:

(Port.to.show_results_html_file):

  • Scripts/webkitpy/layout_tests/run_webkit_tests_integrationtest.py:

(MainTest.test_results_directory_absolute):

1:22 PM Changeset in webkit [104205] by Lucas Forschler
  • 4 edits
    2 copies in branches/safari-534.54-branch

Merged r98010.

1:19 PM Changeset in webkit [104204] by Lucas Forschler
  • 4 edits
    2 copies in branches/safari-534.54-branch

Merged r97927.

1:11 PM Changeset in webkit [104203] by Lucas Forschler
  • 3 edits
    2 copies in branches/safari-534.54-branch

Merged r97786.

1:04 PM Changeset in webkit [104202] by Lucas Forschler
  • 2 edits in branches/safari-534.54-branch/LayoutTests

Merged r97524.

1:01 PM Changeset in webkit [104201] by Lucas Forschler
  • 4 edits
    2 copies in branches/safari-534.54-branch

Merged r97502.

12:57 PM Changeset in webkit [104200] by Lucas Forschler
  • 10 edits in branches/safari-534.54-branch

Merged r97402.

12:53 PM Changeset in webkit [104199] by nduca@chromium.org
  • 4 edits
    3 adds in trunk/Source

[chromium] Add CCTimer class for the compositor
​https://bugs.webkit.org/show_bug.cgi?id=74769

Patch by Tien Ren Chen <​trchen@chromium.org> on 2012-01-05
Reviewed by James Robinson.

Add a simple timer class for CCThread that the timered task can be
manually cancelled.

Source/WebCore:

  • WebCore.gypi:
  • platform/graphics/chromium/cc/CCTimer.cpp: Added.

(WebCore::CCTimerTask::CCTimerTask):
(WebCore::CCTimerTask::~CCTimerTask):
(WebCore::CCTimerTask::performTask):
(WebCore::CCTimer::CCTimer):
(WebCore::CCTimer::~CCTimer):
(WebCore::CCTimer::startOneShot):
(WebCore::CCTimer::stop):

  • platform/graphics/chromium/cc/CCTimer.h: Added.

(WebCore::CCTimerClient::~CCTimerClient):
(WebCore::CCTimer::isActive):

Source/WebKit/chromium:

  • WebKit.gypi:
  • tests/CCTimerTest.cpp: Added.

(WebKitTests::CCTimerTest::CCTimerTest):
(WebKitTests::CCTimerTest::onTimerFired):
(WebKitTests::TEST_F):

12:48 PM Changeset in webkit [104198] by ap@apple.com
  • 2 edits in trunk/Source/WebKit2

WK2: Safari fails to open a PostScript file in Preview from context menu
​https://bugs.webkit.org/show_bug.cgi?id=75643
<rdar://problem/9823430>

Reviewed by Dan Bernstein.

  • UIProcess/API/mac/PDFViewController.mm: (WebKit::PDFViewController::setPDFDocumentData): Append ".pdf" to file name when converting data, as suggested file name is created for original MIME type.
12:44 PM Changeset in webkit [104197] by eric.carlson@apple.com
  • 10 edits
    24 adds in trunk

Implement temporal dimension portion of Media Fragments URI specification for video/audio
​https://bugs.webkit.org/show_bug.cgi?id=65838

Reviewed by Sam Weinig.

Source/WebCore:

Tests: media/media-fragments/TC0001-TC0009.html

media/media-fragments/TC0010-TC0019.html
media/media-fragments/TC0020-TC0029.html
media/media-fragments/TC0030-TC0039.html
media/media-fragments/TC0040-TC0049.html
media/media-fragments/TC0050-TC0059.html
media/media-fragments/TC0060-TC0069.html
media/media-fragments/TC0070-TC0079.html
media/media-fragments/TC0080-TC0089.html
media/media-fragments/TC0090-TC0099.html

  • CMakeLists.txt: Add fragment parser files.
  • GNUmakefile.list.am: Ditto.
  • Target.pri: Ditto.
  • WebCore.gypi: Ditto.
  • WebCore.vcproj/WebCore.vcproj: Ditto.
  • WebCore.xcodeproj/project.pbxproj: Ditto.
  • html/HTMLMediaElement.cpp:

(WebCore::HTMLMediaElement::HTMLMediaElement): Initialize m_fragmentStartTime and m_fragmentEndTime.
(WebCore::HTMLMediaElement::setReadyState): Check for and parse a media fragment once readyState

reaches HAVE_METADATA, apply it once it reaches HAVE_CURRENT_DATA.

(WebCore::HTMLMediaElement::initialTime): Return the fragment start time if possible.
(WebCore::HTMLMediaElement::playbackProgressTimerFired): Pause if the time is >= the fragment

end time.

(WebCore::HTMLMediaElement::prepareMediaFragmentURI): Look for a temporal fragment.
(WebCore::HTMLMediaElement::applyMediaFragmentURI): Apply the fragment, if any.

  • html/HTMLMediaElement.h:
  • html/MediaFragmentURIParser.cpp: Added.

(WebCore::skipWhiteSpace):
(WebCore::collectDigits):
(WebCore::collectFraction):
(WebCore::MediaFragmentURIParser::invalidTimeValue):
(WebCore::MediaFragmentURIParser::MediaFragmentURIParser):
(WebCore::MediaFragmentURIParser::startTime):
(WebCore::MediaFragmentURIParser::endTime):
(WebCore::MediaFragmentURIParser::parseFragments):
(WebCore::MediaFragmentURIParser::parseTimeFragment):
(WebCore::MediaFragmentURIParser::parseNPTFragment):
(WebCore::MediaFragmentURIParser::parseNPTTime):

  • html/MediaFragmentURIParser.h: Added.

(WebCore::MediaFragmentURIParser::~MediaFragmentURIParser):
(WebCore::MediaFragmentURIParser::create):

LayoutTests:

  • media/media-fragments: Added.
  • media/media-fragments/TC0001-TC0009-expected.txt: Added.
  • media/media-fragments/TC0001-TC0009.html: Added.
  • media/media-fragments/TC0010-TC0019-expected.txt: Added.
  • media/media-fragments/TC0010-TC0019.html: Added.
  • media/media-fragments/TC0020-TC0029-expected.txt: Added.
  • media/media-fragments/TC0020-TC0029.html: Added.
  • media/media-fragments/TC0030-TC0039-expected.txt: Added.
  • media/media-fragments/TC0030-TC0039.html: Added.
  • media/media-fragments/TC0040-TC0049-expected.txt: Added.
  • media/media-fragments/TC0040-TC0049.html: Added.
  • media/media-fragments/TC0050-TC0059-expected.txt: Added.
  • media/media-fragments/TC0050-TC0059.html: Added.
  • media/media-fragments/TC0060-TC0069-expected.txt: Added.
  • media/media-fragments/TC0060-TC0069.html: Added.
  • media/media-fragments/TC0070-TC0079-expected.txt: Added.
  • media/media-fragments/TC0070-TC0079.html: Added.
  • media/media-fragments/TC0080-TC0089-expected.txt: Added.
  • media/media-fragments/TC0080-TC0089.html: Added.
  • media/media-fragments/TC0090-TC0099-expected.txt: Added.
  • media/media-fragments/TC0090-TC0099.html: Added.
  • media/media-fragments/media-fragments.js: Added.

(canplaythrough):
(setupNextTest):
(nextTestName):
(start):

12:44 PM WebKit Team edited by aestes@apple.com
(diff)
12:41 PM Changeset in webkit [104196] by commit-queue@webkit.org
  • 3 edits
    2 adds in trunk

[v8] Null pointer exception if a typed array constructor set to a primitive value.
​https://bugs.webkit.org/show_bug.cgi?id=75532

Make sure that V8ArrayBufferViewCustomScript.js does not throw exception.

Patch by Ulan Degenbaev <​ulan@chromium.org> on 2012-01-05
Reviewed by Kenneth Russell.

  • LayoutTests/fast/canvas/webgl/array-buffer-view-crash-when-reassigned-expected.txt: Added.
  • LayoutTests/fast/canvas/webgl/array-buffer-view-crash-when-reassigned.html: Added.
  • Source/WebCore/bindings/v8/custom/V8ArrayBufferViewCustom.cpp:
  • Source/WebCore/bindings/v8/custom/V8ArrayBufferViewCustomScript.js:
12:28 PM Changeset in webkit [104195] by Adam Roben
  • 2 edits in trunk/Tools

Remove some unnecessary code from old-run-webkit-tests

Thanks to r104187/r104188, we no longer need to set PATH before launching DumpRenderTree or
WebKitTestRunner.

Fixes <​http://webkit.org/b/75640> ORWT unnecessarily sets PATH before launching DRT/WTR on
Windows

Reviewed by Eric Seidel.

  • Scripts/old-run-webkit-tests:

(top level):
(openDumpTool):
Removed calls to setPathForRunningWebKitApp.

12:18 PM Changeset in webkit [104194] by Martin Robinson
  • 5 edits
    2 adds in trunk/Source/WebKit/gtk

[GTK] [AC] Introduce AcceleratedCompositingContext to isolate different accelerated compositing implementations
​https://bugs.webkit.org/show_bug.cgi?id=75519

Reviewed by Alejandro G. Castro.

Add an AcceleratedCompositingContext which will isolate all the different accelerated compositing
implementations from one another. For right now this just includes the Clutter version.

  • GNUmakefile.am: Add the new files to the source list.
  • WebCoreSupport/AcceleratedCompositingContext.h: Added.
  • WebCoreSupport/AcceleratedCompositingContextClutter.cpp: Added. Existing

clutter AC code has moved here.

  • WebCoreSupport/ChromeClientGtk.cpp:

(WebKit::ChromeClient::attachRootGraphicsLayer): Use the new AccleratedCompositingContext now
instead of calling into the WebView.
(WebKit::ChromeClient::setNeedsOneShotDrawingSynchronization):Ditto.
(WebKit::ChromeClient::scheduleCompositingLayerSync): Ditto.

  • webkit/webkitwebview.cpp:

(webkit_web_view_size_allocate): Call in the ACContext.
(webkit_web_view_init): Initialize the ACContext.

  • webkit/webkitwebviewprivate.h: Add the ACContext as a private member.
12:04 PM Changeset in webkit [104193] by zeno.albisser@nokia.com
  • 22 edits
    13 adds in trunk

[Qt][WK2] Implement custom URL schemes defined in QML.
​https://bugs.webkit.org/show_bug.cgi?id=74931

.:

Add new classes to the export map.

Reviewed by Kenneth Rohde Christiansen.

  • Source/qtwebkit-export.map:

Source/WebKit/qt:

Make classes QQuickUrlSchemeDelegate, QQuickNetworkRequest
and QQuickNetworkReply accessible from QML.

Reviewed by Kenneth Rohde Christiansen.

  • declarative/experimental/experimental.pri:
  • declarative/experimental/plugin.cpp:
  • declarative/public.pri:

Source/WebKit2:

Allow creation of custom UrlSchemeDelegates from QML.
A UrlSchemeDelegate contains a handler signal that is triggered
when a url matching the defined scheme is requested.
To allow for this a derivation of QNetworkAccessManager
is necessary.
The request is then handled in QML/JS and a reply
is sent back from the UIProcess to the WebProcess.

Reviewed by Kenneth Rohde Christiansen.

  • Shared/qt/QtNetworkReplyData.cpp: Added.

(WebKit::QtNetworkReplyData::QtNetworkReplyData):
(WebKit::QtNetworkReplyData::encode):
(WebKit::QtNetworkReplyData::decode):

  • Shared/qt/QtNetworkReplyData.h: Added.
  • Shared/qt/QtNetworkRequestData.cpp: Added.

(WebKit::QtNetworkRequestData::QtNetworkRequestData):
(WebKit::QtNetworkRequestData::encode):
(WebKit::QtNetworkRequestData::decode):

  • Shared/qt/QtNetworkRequestData.h: Added.
  • Target.pri:
  • UIProcess/API/qt/qquicknetworkreply.cpp: Added.

(QQuickNetworkReply::QQuickNetworkReply):
(QQuickNetworkReply::contentType):
(QQuickNetworkReply::setContentType):
(QQuickNetworkReply::operation):
(QQuickNetworkReply::setOperation):
(QQuickNetworkReply::contentDisposition):
(QQuickNetworkReply::setContentDisposition):
(QQuickNetworkReply::location):
(QQuickNetworkReply::setLocation):
(QQuickNetworkReply::lastModified):
(QQuickNetworkReply::setLastModified):
(QQuickNetworkReply::cookie):
(QQuickNetworkReply::setCookie):
(QQuickNetworkReply::userAgent):
(QQuickNetworkReply::setUserAgent):
(QQuickNetworkReply::server):
(QQuickNetworkReply::setServer):
(QQuickNetworkReply::data):
(QQuickNetworkReply::setData):
(QQuickNetworkReply::send):
(QQuickNetworkReply::networkRequestData):
(QQuickNetworkReply::setNetworkRequestData):
(QQuickNetworkReply::networkReplyData):

  • UIProcess/API/qt/qquicknetworkreply_p.h: Added.
  • UIProcess/API/qt/qquicknetworkrequest_p.h: Added.
  • UIProcess/API/qt/qquickurlschemedelegate.cpp: Added.

(QQuickUrlSchemeDelegate::QQuickUrlSchemeDelegate):
(QQuickUrlSchemeDelegate::scheme):
(QQuickUrlSchemeDelegate::setScheme):
(QQuickUrlSchemeDelegate::request):
(QQuickUrlSchemeDelegate::reply):

  • UIProcess/API/qt/qquickurlschemedelegate_p.h: Added.
  • UIProcess/API/qt/qquickwebview.cpp:

(QQuickWebViewExperimental::QQuickWebViewExperimental):
(QQuickWebViewExperimental::schemeDelegates_At):
(QQuickWebViewExperimental::schemeDelegates_Append):
(QQuickWebViewExperimental::schemeDelegates_Count):
(QQuickWebViewExperimental::schemeDelegates_Clear):
(QQuickWebViewExperimental::schemeDelegates):
(QQuickWebViewExperimental::invokeApplicationSchemeHandler):
(QQuickWebViewExperimental::sendApplicationSchemeReply):

  • UIProcess/API/qt/qquickwebview_p.h:
  • UIProcess/WebPageProxy.h:
  • UIProcess/WebPageProxy.messages.in:
  • UIProcess/qt/QtPageClient.cpp:

(QtPageClient::handleApplicationSchemeRequest):

  • UIProcess/qt/QtPageClient.h:
  • UIProcess/qt/WebPageProxyQt.cpp:

(WebKit::WebPageProxy::registerApplicationScheme):
(WebKit::WebPageProxy::resolveApplicationSchemeRequest):
(WebKit::WebPageProxy::sendApplicationSchemeReply):

  • WebProcess/WebCoreSupport/WebFrameLoaderClient.cpp:

(WebKit::WebFrameLoaderClient::createNetworkingContext):

  • WebProcess/WebCoreSupport/qt/WebFrameNetworkingContext.cpp:
  • WebProcess/WebCoreSupport/qt/WebFrameNetworkingContext.h:
  • WebProcess/WebPage/WebPage.h:
  • WebProcess/WebPage/WebPage.messages.in:
  • WebProcess/WebPage/qt/WebPageQt.cpp:

(WebKit::WebPage::registerApplicationScheme):
(WebKit::WebPage::receivedApplicationSchemeRequest):
(WebKit::WebPage::applicationSchemeReply):

  • WebProcess/qt/QtNetworkAccessManager.cpp: Added.

(WebKit::QtNetworkAccessManager::QtNetworkAccessManager):
(WebKit::QtNetworkAccessManager::obtainOriginatingWebPage):
(WebKit::QtNetworkAccessManager::createRequest):
(WebKit::QtNetworkAccessManager::registerApplicationScheme):

  • WebProcess/qt/QtNetworkAccessManager.h: Added.
  • WebProcess/qt/QtNetworkReply.cpp: Added.

(WebKit::QtNetworkReply::QtNetworkReply):
(WebKit::QtNetworkReply::setData):
(WebKit::QtNetworkReply::setReplyData):
(WebKit::QtNetworkReply::readData):
(WebKit::QtNetworkReply::bytesAvailable):
(WebKit::QtNetworkReply::setHeader):
(WebKit::QtNetworkReply::abort):
(WebKit::QtNetworkReply::close):
(WebKit::QtNetworkReply::setReadBufferSize):
(WebKit::QtNetworkReply::canReadLine):
(WebKit::QtNetworkReply::finalize):

  • WebProcess/qt/QtNetworkReply.h: Added.
  • WebProcess/qt/WebProcessQt.cpp:

(WebKit::WebProcess::platformInitializeWebProcess):

12:02 PM Changeset in webkit [104192] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebCore

Remove style warning in GraphicsContext3DOpenGL.cpp
​https://bugs.webkit.org/show_bug.cgi?id=75466

Patch by ChangSeok Oh <ChangSeok Oh> on 2012-01-05
Reviewed by Kenneth Russell.

Relocated some headers according to alphabetical order & modified indentation.
And used OwnArrayPtr to deal with character array.

No new tests required.

  • platform/graphics/opengl/GraphicsContext3DOpenGL.cpp:

(WebCore::GraphicsContext3D::getString):
(WebCore::GraphicsContext3D::releaseShaderCompiler):
(WebCore::GraphicsContext3D::getProgramInfoLog):
(WebCore::GraphicsContext3D::getShaderiv):
(WebCore::GraphicsContext3D::getShaderInfoLog):
(WebCore::GraphicsContext3D::getShaderSource):

11:55 AM Changeset in webkit [104191] by Adam Roben
  • 3 edits in trunk/Tools

Teach NRWT to use run-safari to open results.html on Apple's Windows port

NRWT was trying to use User.open_url, which doesn't work on Cygwin. But run-safari is
better, since we always want to open results.html using the user's build of WebKit, not
whatever their default browser is.

Fixes <​http://webkit.org/b/75629> <rdar://problem/10648621> NRWT fails to open results.html
for Apple's Windows port

Reviewed by Eric Seidel.

  • Scripts/webkitpy/layout_tests/port/win.py:

(WinPort.show_results_html_file): Changed to use run_safari, like MacPort does. Also removed
an obsolete comment.

  • Scripts/webkitpy/layout_tests/port/win_unittest.py:

(WinPortTest.test_show_results_html_file): Updated expected results.

11:55 AM Changeset in webkit [104190] by Adam Roben
  • 2 edits in trunk/Tools

Prepend Safari.exe's path to the command-line passed to CreateProcess

Safari expects the first argument in the command-line to be the path to Safari.exe itself.
This is how things get set up when Safari is launched normally (e.g., by Explorer). Safari
thus ignores the first argument when looking for command-line arguments. WebKitLauncherWin
was not including the path to Safari.exe in the command-line, so Safari was ignoring the
first argument that WebKitLauncherWin was trying to pass to it.

Fixes <​http://webkit.org/b/56571> First argument passed to WebKit.exe is ignored by Safari

Reviewed by Steve Falkenburg.

  • WebKitLauncherWin/WebKitLauncherWin.cpp:

(_tWinMain): Construct a command-line for CreateProcess that first contains a double-quoted
path to Safari.exe, then contains any arguments that were passed to WebKitLauncherWin.

11:55 AM Changeset in webkit [104189] by Adam Roben
  • 2 edits in trunk/Tools

Stop using explicit memory management in WebKitLauncherWin

We now use stack-allocated STL objects instead.

Prep work for <​http://webkit.org/b/56571> First argument passed to WebKit.exe is ignored by
Safari

Reviewed by Steve Falkenburg.

  • WebKitLauncherWin/WebKitLauncherWin.cpp:

(getStringValue):
(applePathFromRegistry):
(safariInstallDir):
(safariBrowserExe):
(_tWinMain):
Changed to use stack-allocated STL objects instead of malloc/free.

11:54 AM Changeset in webkit [104188] by Adam Roben
  • 7 edits
    1 copy
    7 adds in trunk

Turn WebKitTestRunner into a stub .exe launcher and a .dll that contains all the real code

This allows us to add the Safari and Apple Application Support directories to PATH before
trying to load WebKit so that SafariTheme.dll, CoreGraphics.dll and friends can be found.

Fixes <​http://webkit.org/b/75486> <rdar://problem/10638124> NRWT fails to launch Apple's
Windows port of DumpRenderTree/WebKitTestRunner

Reviewed by Steve Falkenburg.

Tools:

  • WebKitTestRunner/WebKitTestRunner.sln: Added WebKitTestRunnerLauncher and

ImageDiffLauncher, which I forgot to add to this .sln when that project was created.

  • WebKitTestRunner/Configurations/WebKitTestRunnerCommon.vsprops:
  • WebKitTestRunner/win/WebKitTestRunner.vcproj:

Changed to build WebKitTestRunner.dll instead of WebKitTestRunner.exe.

  • WebKitTestRunner/win/WebKitTestRunnerLauncher.vcproj: Copied from

Tools/WebKitTestRunner/win/WebKitTestRunner.vcproj but changed to build
the stub launcher.

  • WebKitTestRunner/win/WebKitTestRunnerLauncherCommon.vsprops: Added.
  • WebKitTestRunner/win/WebKitTestRunnerLauncherDebug.vsprops: Added.
  • WebKitTestRunner/win/WebKitTestRunnerLauncherDebugAll.vsprops: Added.
  • WebKitTestRunner/win/WebKitTestRunnerLauncherDebugCairoCFLite.vsprops: Added.
  • WebKitTestRunner/win/WebKitTestRunnerLauncherProduction.vsprops: Added.
  • WebKitTestRunner/win/WebKitTestRunnerLauncherRelease.vsprops: Added.
  • WebKitTestRunner/win/WebKitTestRunnerLauncherReleaseCairoCFLite.vsprops: Added.
  • WebKitTestRunner/win/main.cpp:

(dllLauncherEntryPoint): Renamed from main. This is what the stub launcher calls.

Source/WebKit/win:

Add WebKitTestRunnerLauncher to WebKit.sln.

It builds just after WebKitTestRunner.

  • WebKit.vcproj/WebKit.sln:
11:53 AM Changeset in webkit [104187] by Adam Roben
  • 8 edits
    8 adds in trunk

Turn DumpRenderTree into a stub .exe launcher and a .dll that contains all the real code

This allows us to add the Safari and Apple Application Support directories to PATH before
trying to load WebKit so that SafariTheme.dll, CoreGraphics.dll and friends can be found.

Part of <​http://webkit.org/b/75486> <rdar://problem/10638124> NRWT fails to launch Apple's
Windows port of DumpRenderTree/WebKitTestRunner

Reviewed by Steve Falkenburg.

Tools:

  • DumpRenderTree/DumpRenderTree.sln: Added DumpRenderTreeLauncher and ImageDiffLauncher,

which I forgot to add to this .sln when that project was created.

  • DumpRenderTree/win/DumpRenderTree.cpp:

(dllLauncherEntryPoint): Renamed from main. This is what the stub launcher calls.

  • DumpRenderTree/win/DumpRenderTree.vcproj:
  • DumpRenderTree/win/DumpRenderTreeCommon.vsprops:

Changed to build DumpRenderTree.dll instead of DumpRenderTree.exe.

  • DumpRenderTree/win/DumpRenderTreeLauncher.vcproj: Added.
  • DumpRenderTree/win/DumpRenderTreeLauncherCommon.vsprops: Added.
  • DumpRenderTree/win/DumpRenderTreeLauncherDebug.vsprops: Added.
  • DumpRenderTree/win/DumpRenderTreeLauncherDebugAll.vsprops: Added.
  • DumpRenderTree/win/DumpRenderTreeLauncherDebugCairoCFLite.vsprops: Added.
  • DumpRenderTree/win/DumpRenderTreeLauncherProduction.vsprops: Added.
  • DumpRenderTree/win/DumpRenderTreeLauncherRelease.vsprops: Added.
  • DumpRenderTree/win/DumpRenderTreeLauncherReleaseCairoCFLite.vsprops: Added.
  • win/DLLLauncher/DLLLauncherMain.cpp:

(modifyPath): Moved code to modify PATH here from main/wWinMain. Changed to also prepend
Safari's directory to PATH so that SafariTheme.dll can be found.

(main):
(wWinMain):
Changed to use the new modifyPath function.

Source/WebKit/win:

Add DumpRenderTreeLauncher to WebKit.sln.

It builds just after DumpRenderTree.

  • WebKit.vcproj/WebKit.sln:
11:46 AM Changeset in webkit [104186] by Lucas Forschler
  • 2 edits in branches/safari-534.54-branch/Source/JavaScriptCore

Fix debug build.

Reviewed by Oliver Hunt.

  • dfg/DFGOSRExitCompiler64.cpp:

(JSC::DFG::OSRExitCompiler::compileExit):

11:35 AM Changeset in webkit [104185] by jianli@chromium.org
  • 3 edits in trunk/LayoutTests

Unreviewed, skip file-reader-event-listener.html on qt and efl.

  • platform/efl/Skipped:
  • platform/qt/Skipped:
11:32 AM Changeset in webkit [104184] by barraclough@apple.com
  • 2 edits in trunk/Source/JavaScriptCore

push/shift fifo may consume excessive memory
​https://bugs.webkit.org/show_bug.cgi?id=75610

Reviewed by Sam Weinig.

Array object commonly store data in a vector, consisting of a portion that is
in use, a pre-capacity (m_indexBias) and a post-capacity (the delta between
m_length and m_vectorLength). Calls to shift with grow the pre-capacity, and
the current algorithm for increaseVectorLength (used by push, or Put) will
never shrink the pre-capacity, so a push/shift fifo may consume an inordinate
amount of memory, whilst having a relatively small active length.

  • runtime/JSArray.cpp:

(JSC::JSArray::increaseVectorLength):

  • If m_indexBias is non-zero, decay it over time.
11:24 AM Changeset in webkit [104183] by commit-queue@webkit.org
  • 4 edits
    2 adds in trunk

Source/WebCore: Crash due to reparenting of relpositioned object under anonymous block
​https://bugs.webkit.org/show_bug.cgi?id=70848

The associated test case creates a condition where a relative
positioned renderer is a descendant of an anonymous block for a
table column. The anonymous block is the containingBlock() for the
relpositioned renderer. Removal of a div causes the anonymous blocks to
be merged, and the renderer becomes a descendant of a different block.
Since the new containingBlock() has an empty positionedObject list,
the relpositioned renderer does not get layout after being dirtied.

This patch changes containingBlock() so that it returns the container
of an anonymous block for positioned objects, not the anonymous
block itself. It also adds an ASSERT to insertPositionedObject()
to flag any other cases where something is trying to create a
positioned object list on an anonymous block.

Patch by Ken Buchanan <​kenrb@chromium.org> on 2012-01-05
Reviewed by David Hyatt.

  • rendering/RenderBlock.cpp:

(WebCore::RenderBlock::insertPositionedObject):

  • rendering/RenderObject.cpp:

(WebCore::RenderObject::containingBlock):

LayoutTests: Crash due to reparenting of relpositioned object in table
​https://bugs.webkit.org/show_bug.cgi?id=70848

Adding test that exercises the crash condition in bug 70848.

Patch by Ken Buchanan <​kenrb@chromium.org> on 2012-01-05
Reviewed by David Hyatt.

  • fast/css/relative-position-replaced-in-table-display-crash-expected.txt: Added
  • fast/css/relative-position-replaced-in-table-display-crash.html: Added
11:15 AM Changeset in webkit [104182] by dslomov@google.com
  • 2 edits in trunk/LayoutTests

Unreviewed: filed WK75633 and updated expectations

  • platform/chromium/test_expectations.txt:
11:03 AM Changeset in webkit [104181] by psolanki@apple.com
  • 4 edits in trunk/Source/WebKit/mac

WebFrameLoaderClient::userAgent does unnecessary NSString/NSURL conversions
​https://bugs.webkit.org/show_bug.cgi?id=75586

Reviewed by Alexey Proskuryakov.

We do unnecessary KURL->NSURL* and String<->NSString* when calling [WebView
userAgentForURL:] from WebFrameLoaderClient::userAgent. Instead we add a separate function,
_userAgentString that returns a WTF::String to make this call faster.

  • WebCoreSupport/WebFrameLoaderClient.mm:

(WebFrameLoaderClient::userAgent):

  • WebView/WebView.mm:

(-[WebView _userAgentString]):
(-[WebView userAgentForURL:]):

  • WebView/WebViewInternal.h:
10:59 AM Changeset in webkit [104180] by jianli@chromium.org
  • 3 edits
    2 adds in trunk

FileReader needs addEventListener
​https://bugs.webkit.org/show_bug.cgi?id=42723

Reviewed by Adam Barth.

Source/WebCore:

Test: fast/files/file-reader-event-listener.html

  • fileapi/FileReader.idl:

LayoutTests:

  • fast/files/file-reader-event-listener-expected.txt: Added.
  • fast/files/file-reader-event-listener.html: Added.
10:50 AM WebKit Team edited by adele@apple.com
(diff)
10:38 AM Changeset in webkit [104179] by psolanki@apple.com
  • 2 edits in trunk/Source/WebCore

Remove deprecated calls from CookieJarCFNet.cpp
​https://bugs.webkit.org/show_bug.cgi?id=68958

Reviewed by Sam Weinig.

  • platform/network/cf/CookieJarCFNet.cpp:

(WebCore::cookieDomain):
(WebCore::cookieExpirationTime):
(WebCore::cookieName):
(WebCore::cookiePath):
(WebCore::cookieValue):

10:36 AM Changeset in webkit [104178] by Dimitri Glazkov
  • 5 edits
    1 delete in branches/chromium/912

Revert 91125 - 2011-07-15 Rob Buis <​rbuis@rim.com>

Stroking of zero-length paths in SVG should change according to erratum
​https://bugs.webkit.org/show_bug.cgi?id=18356

Reviewed by Nikolas Zimmermann.

Ensure rendering of zero-length paths with stroke-linecap=square works by
creating the square rectangle manually.

Tests: svg/W3C-SVG-1.1-SE/painting-control-04-f.svg

svg/custom/zero-path-square-cap-rendering.svg

  • platform/graphics/GraphicsContext.h: (WebCore::GraphicsContextStateSaver::context):
  • rendering/svg/RenderSVGPath.cpp: (WebCore::RenderSVGPath::strokeContains): (WebCore::RenderSVGPath::shouldStrokeZeroLengthSubpath): (WebCore::RenderSVGPath::zeroLengthSubpathRect): (WebCore::RenderSVGPath::setupSquareCapPath): (WebCore::RenderSVGPath::setupNonScalingStrokePath): (WebCore::RenderSVGPath::fillAndStrokePath): (WebCore::RenderSVGPath::updateCachedBoundaries):
  • rendering/svg/RenderSVGPath.h:

TBR=​rwlbuis@webkit.org
Review URL: ​http://codereview.chromium.org/9116011

10:28 AM Changeset in webkit [104177] by tony@chromium.org
  • 2 edits in trunk/LayoutTests

[chromium] Unreviewed, tighted up the expectations for some perf tests.

  • platform/chromium/test_expectations.txt:
10:26 AM Changeset in webkit [104176] by enne@google.com
  • 2 edits
    1 add in trunk/Source/WebKit/chromium

[chromium] Create unit tests for CCTiledLayerImpl
​https://bugs.webkit.org/show_bug.cgi?id=75557

Reviewed by James Robinson.

Add some quad generation tests for tiled layers.

  • WebKit.gypi:
  • tests/CCTiledLayerImplTest.cpp: Added.

(WebCore::createLayer):
(WebCore::TEST):
(WebCore::completelyContains):
(WebCore::getQuads):
(WebCore::verifyQuadsExactlyCoverRect):
(WebCore::coverageVisibleRectOnTileBoundaries):
(WebCore::coverageVisibleRectIntersectsTiles):
(WebCore::coverageVisibleRectIntersectsBounds):

10:19 AM WebKit Team edited by dgrogan@chromium.org
(diff)
10:18 AM Changeset in webkit [104175] by Dimitri Glazkov
  • 5 edits
    6 copies
    2 deletes in branches/chromium/963

Merge 101903 - REGRESSION (r91125): Polyline tool in google docs is broken
​https://bugs.webkit.org/show_bug.cgi?id=65796

Source/WebCore:

Patch by Stephen Chenney <​schenney@chromium.org> on 2011-12-02
Reviewed by Darin Adler.

Work around a bug in CoreGraphics, that caused incorrect bounds for paths
consisting only of move-to elements. This causes problems in SVG, when the enormous
bounds prevented the drawing of things behind.

Tests: svg/custom/path-moveto-only-rendering.svg

svg/custom/subpaths-moveto-only-rendering.svg

  • platform/graphics/cg/PathCG.cpp:

(WebCore::PathIsEmptyOrSingleMoveTester::PathIsEmptyOrSingleMoveTester): Class to
test for isEmpty accoridng ot the same rules as other platforms.
(WebCore::PathIsEmptyOrSingleMoveTester::isEmpty): Query the result
(WebCore::PathIsEmptyOrSingleMoveTester::testPathElement): Path iterator method
(WebCore::PathHasOnlyMoveToTester::PathHasOnlyMoveToTester): Class to test whether a
path contains only move-to elements, and hence should have null bounds.
(WebCore::PathHasOnlyMoveToTester::hasOnlyMoveTo): Query the result
(WebCore::PathHasOnlyMoveToTester::testPathElement): Path iterator method.
(WebCore::Path::boundingRect): Modified to check for move-to only paths
(WebCore::Path::fastBoundingRect): Modified to check for move-to only paths
(WebCore::Path::isEmpty): Now uses the method that matches other platforms.
(WebCore::Path::hasCurrentPoint): Now uses CGPathIsEmpty directly
(WebCore::Path::transform) : Now uses CGPathIsEmpty directly

LayoutTests:

Work around a bug in CoreGraphics, that caused incorrect bounds for paths
consisting only of move-to elements. This causes problems in SVG, when the enormous
bounds prevented the drawing of things behind.

Will revert expectation file when expectations are stable.

Patch by Stephen Chenney <​schenney@chromium.org> on 2011-12-02
Reviewed by Darin Adler.

  • platform/chromium-mac/svg/custom/zero-path-square-cap-rendering2-expected.txt: Removed.
  • platform/chromium-win/svg/custom/zero-path-square-cap-rendering2-expected.txt: Removed.
  • platform/mac/svg/custom/path-moveto-only-rendering-expected.png: Added.
  • platform/mac/svg/custom/path-moveto-only-rendering-expected.txt: Added.
  • platform/mac/svg/custom/subpaths-moveto-only-rendering-expected.png: Added.
  • platform/mac/svg/custom/subpaths-moveto-only-rendering-expected.txt: Added.
  • svg/custom/path-moveto-only-rendering.svg: Added.
  • svg/custom/subpaths-moveto-only-rendering.svg: Added.
  • svg/custom/zero-path-square-cap-rendering2-expected.txt: Modified text output
  • platform/chromium/test_expectations.txt: Added flakiness for new tests from this patch

TBR=​commit-queue@webkit.org
Review URL: ​http://codereview.chromium.org/9111020

10:16 AM Changeset in webkit [104174] by fsamuel@chromium.org
  • 35 edits in trunk

Move scalePageBy from eventSender to window.internals
​https://bugs.webkit.org/show_bug.cgi?id=64512

Reviewed by Simon Fraser.

.:

  • Source/autotools/symbols.filter:

Source/WebCore:

Added setPageScaleFactor to window.internals.
Renamed window.internals.getPageScaleFactor to window.internals.pageScaleFactor
to match the webkit style.

  • testing/Internals.cpp:

(WebCore::Internals::pageScaleFactor):
(WebCore::Internals::setPageScaleFactor):

  • testing/Internals.h:
  • testing/Internals.idl:

Source/WebKit2:

  • win/WebKit2.def:
  • win/WebKit2CFLite.def:

LayoutTests:

Updated layout tests to use window.internals.setPageScaleFactor.

  • compositing/geometry/fixed-position-composited-page-scale-down.html:
  • compositing/geometry/fixed-position-composited-page-scale.html:
  • compositing/geometry/fixed-position-iframe-composited-page-scale-down.html:
  • compositing/geometry/fixed-position-iframe-composited-page-scale.html:
  • compositing/geometry/fixed-position-transform-composited-page-scale-down.html:
  • compositing/geometry/fixed-position-transform-composited-page-scale.html:
  • compositing/scaling/tiled-layer-recursion.html:
  • fast/dom/Element/scale-page-bounding-client-rect.html:
  • fast/dom/Element/scale-page-client-rects.html:
  • fast/dom/Range/scale-page-bounding-client-rect.html:
  • fast/dom/Range/scale-page-client-rects.html:
  • fast/events/scale-and-scroll-body.html:
  • fast/events/scale-and-scroll-iframe-body.html:
  • fast/events/scale-and-scroll-iframe-window.html:
  • fast/events/scale-and-scroll-window.html:
  • fast/events/script-tests/page-scaled-mouse-click-iframe.js:

(iframeLoaded):

  • fast/events/script-tests/page-scaled-mouse-click.js:
  • fast/events/scroll-in-scaled-page-with-overflow-hidden.html:
  • fast/events/touch/page-scaled-touch-gesture-click.html:
  • fast/frames/frame-set-rotation-hit.html:
  • fast/frames/frame-set-scaling-hit.html:
  • fast/frames/iframe-double-scale-contents.html:
  • fast/repaint/background-scaling.html:
  • fast/repaint/scale-page-shrink.html:
  • fast/transforms/selection-bounds-in-transformed-view.html:
10:08 AM Changeset in webkit [104173] by caryclark@google.com
  • 2 edits in trunk/Source/WebCore

[Skia Mac] Rounded bezel style button needs one more local graphics context to scale
​https://bugs.webkit.org/show_bug.cgi?id=75623
​http://code.google.com/p/chromium/issues/detail?id=108749

In paintButton(), the local context is set up first, and later the context is
scaled if there's a zoomFactor. Skia creates the CoreGraphics context to draw the
button into without that scale factor. To fix this, another local context is added
after the parameter context scale, before the button draw. The first local context
is still required to restore the parameter context state.

Reviewed by Stephen White.

  • platform/chromium/ThemeChromiumMac.mm:

(WebCore::paintButton):

9:52 AM Changeset in webkit [104172] by dslomov@google.com
  • 2 edits in trunk/LayoutTests

Unreviewed: Chromium expectations update.
Filed ​http://crbug.com/109276 and ignored appcache failure on Linux.
Failures seem intermittent in appcache so might need to ignore more tests.

  • platform/chromium/test_expectations.txt:
9:49 AM Changeset in webkit [104171] by mitz@apple.com
  • 4 edits in trunk/Source/ThirdParty

Made Debug/Release builds of gtest build only for the active architecture.

Reviewed by Dave Kilzer.

  • gtest/xcode/Config/DebugProject.xcconfig:
  • gtest/xcode/Config/ProductionProject.xcconfig:
  • gtest/xcode/Config/ReleaseProject.xcconfig:
9:31 AM Changeset in webkit [104170] by commit-queue@webkit.org
  • 2 edits in trunk/LayoutTests

[Chromium] Expectations for svg/carto.net/button.svg can be updated
​https://bugs.webkit.org/show_bug.cgi?id=75621

Unreviewed Chromium test expectations update.

Patch by Stephen Chenney <​schenney@chromium.org> on 2012-01-05

  • platform/chromium-linux-x86/svg/carto.net: Removed.
  • platform/chromium/test_expectations.txt:
9:29 AM Changeset in webkit [104169] by alexis.menard@openbossa.org
  • 3 edits
    2 adds in trunk

getComputedStyle for background is not implemented.
​https://bugs.webkit.org/show_bug.cgi?id=75539

Reviewed by Tony Chang.

Source/WebCore:

Implement getComputedStyle for background.

Test: fast/css/getComputedStyle/getComputedStyle-background-shorthand.html

  • css/CSSComputedStyleDeclaration.cpp:

(WebCore::CSSComputedStyleDeclaration::getPropertyCSSValue):

LayoutTests:

Add new tests to make sure we return the correct values.

  • fast/css/getComputedStyle/getComputedStyle-background-shorthand-expected.txt: Added.
  • fast/css/getComputedStyle/getComputedStyle-background-shorthand.html: Added.
9:02 AM Changeset in webkit [104168] by commit-queue@webkit.org
  • 8 edits in trunk/Source/WebKit2

Pass WebFrame instead of WebCore::Frame to the WebKit2 specific FrameNetworkingContext
​https://bugs.webkit.org/show_bug.cgi?id=75550

Make the WebKit2 FrameNetworkingContext implementation store the
WebFrame instead of the WebCore::Frame as this is needed for the Qt
port to implement a.o. HTTP autentication and SSL support.

As the WebFrameNetworkingContext is partly shared across all WebKit2 ports,
all port specific files have been modified.

For the Qt implementation of FrameNetworkingContext, we also set a
property to the originatingObject for the page ID. We also switch to
using an OwnPtr instead of a raw C++ pointer for the
m_originatingObject member.

Based on original patch by Peter Hartmann.

Patch by Alexander Færøy <​alexander.faeroy@nokia.com> on 2012-01-05
Reviewed by Kenneth Rohde Christiansen.

  • WebProcess/WebCoreSupport/WebFrameLoaderClient.cpp:

(WebKit::WebFrameLoaderClient::createNetworkingContext):

  • WebProcess/WebCoreSupport/efl/WebFrameNetworkingContext.h:

(WebFrameNetworkingContext::create):
(WebFrameNetworkingContext::WebFrameNetworkingContext):

  • WebProcess/WebCoreSupport/gtk/WebFrameNetworkingContext.h:

(WebFrameNetworkingContext::create):
(WebFrameNetworkingContext::WebFrameNetworkingContext):

  • WebProcess/WebCoreSupport/mac/WebFrameNetworkingContext.h:

(WebKit::WebFrameNetworkingContext::create):
(WebKit::WebFrameNetworkingContext::WebFrameNetworkingContext):

  • WebProcess/WebCoreSupport/qt/WebFrameNetworkingContext.cpp:

(WebCore::WebFrameNetworkingContext::WebFrameNetworkingContext):
(WebCore::WebFrameNetworkingContext::~WebFrameNetworkingContext):
(WebCore::WebFrameNetworkingContext::create):

  • WebProcess/WebCoreSupport/qt/WebFrameNetworkingContext.h:
  • WebProcess/WebCoreSupport/win/WebFrameNetworkingContext.h:

(WebFrameNetworkingContext::create):
(WebFrameNetworkingContext::WebFrameNetworkingContext):

8:45 AM Changeset in webkit [104167] by commit-queue@webkit.org
  • 2 edits in trunk/Tools

[Qt] Never enable FontConfig on Mac OS X
​https://bugs.webkit.org/show_bug.cgi?id=75616

Patch by Alexander Færøy <​alexander.faeroy@nokia.com> on 2012-01-05
Reviewed by Tor Arne Vestbø.

  • qmake/mkspecs/features/features.prf:
8:39 AM Changeset in webkit [104166] by hclam@chromium.org
  • 7 edits in trunk/Tools

Unreviewed, rolling out r104159.
​http://trac.webkit.org/changeset/104159
​https://bugs.webkit.org/show_bug.cgi?id=75590

Chromium port GPU tests cannot be executed.

  • Scripts/webkitpy/layout_tests/port/chromium_gpu.py:

(get):
(_default_tests_paths):

  • Scripts/webkitpy/layout_tests/port/chromium_gpu_unittest.py:

(ChromiumGpuTest.assert_port_works):

  • Scripts/webkitpy/layout_tests/port/factory.py:

(PortFactory._port_name_from_arguments_and_options):
(PortFactory):
(PortFactory._get_kwargs):
(PortFactory.get):

  • Scripts/webkitpy/layout_tests/port/factory_unittest.py:

(FactoryTest.test_google_chrome.names):
(FactoryTest.test_google_chrome):

  • Scripts/webkitpy/layout_tests/port/google_chrome.py:

(GetGoogleChromePort):
(GetGoogleChromePort.GoogleChromeLinux32Port):
(GetGoogleChromePort.GoogleChromeLinux32Port.baseline_search_path):
(GetGoogleChromePort.GoogleChromeLinux32Port.test_expectations_overrides):
(GetGoogleChromePort.GoogleChromeLinux32Port.architecture):
(GetGoogleChromePort.GoogleChromeLinux64Port):
(GetGoogleChromePort.GoogleChromeLinux64Port.baseline_search_path):
(GetGoogleChromePort.GoogleChromeLinux64Port.test_expectations_overrides):
(GetGoogleChromePort.GoogleChromeLinux64Port.architecture):
(GetGoogleChromePort.GoogleChromeMacPort):
(GetGoogleChromePort.GoogleChromeMacPort.baseline_search_path):
(GetGoogleChromePort.GoogleChromeMacPort.test_expectations_overrides):
(GetGoogleChromePort.GoogleChromeWinPort):
(GetGoogleChromePort.GoogleChromeWinPort.baseline_search_path):
(GetGoogleChromePort.GoogleChromeWinPort.test_expectations_overrides):

  • Scripts/webkitpy/layout_tests/port/google_chrome_unittest.py:

(GetGoogleChromePortTest._verify_baseline_path):
(GetGoogleChromePortTest._verify_expectations_overrides):

8:00 AM Changeset in webkit [104165] by rniwa@webkit.org
  • 2 edits in trunk/Source/WebCore

DOM Attribute tests on Dromaeo spends 2.7% of time in hasSelectorForAttribute
​https://bugs.webkit.org/show_bug.cgi?id=75569

Reviewed by Andreas Kling.

Check needsStyleRecalc() first to avoid unnecessary hash lookups.

  • dom/Element.cpp:

(WebCore::Element::recalcStyleIfNeededAfterAttributeChanged):

7:59 AM Changeset in webkit [104164] by Philippe Normand
  • 2 edits in trunk/LayoutTests

Unreviewed, GTK test_expectations update.

  • platform/gtk/test_expectations.txt: Give

media/media-blocked-by-beforeload.html another chance on the bot
after its update in r103509.

7:47 AM Changeset in webkit [104163] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebCore

[EFL] Replace alloca to C++ new placement.
​https://bugs.webkit.org/show_bug.cgi?id=72017

Patch by Grzegorz Czajkowski <​g.czajkowski@samsung.com> on 2012-01-05
Reviewed by Andreas Kling.

Replaces alloca to C++ new placement as it is not very portable.
It allows to skip checking of memory allocation as new never returns NULL.
According to Edje's documentation type of val (member of Edje_Message_Float_Set)
is double so I changed it to avoid undefined behaviour.

  • platform/efl/ScrollbarEfl.cpp:

(ScrollbarEfl::updateThumbPositionAndProportion):

7:37 AM Changeset in webkit [104162] by Csaba Osztrogonác
  • 2 edits in trunk/LayoutTests

[Qt][WK2]REGRESSION(r104068): editing/pasteboard/copy-element-with-conflicting-background-color-from-rule.html made the next test fail

Unreviewed gardening. One more attempt to paint the Qt-WK2 bot green.

  • platform/qt-wk2/Skipped:
7:22 AM Changeset in webkit [104161] by hclam@chromium.org
  • 2 edits in trunk/LayoutTests

Not reviewed. Update Chromium port test expectations.

  • platform/chromium/test_expectations.txt:
7:01 AM Changeset in webkit [104160] by commit-queue@webkit.org
  • 5 edits in trunk/Source/WebKit/efl

[EFL] Invalidation request for ewk_view can be discarded without rendering.
​https://bugs.webkit.org/show_bug.cgi?id=71564

Patch by KwangHyuk Kim <​hyuki.kim@samsung.com> on 2012-01-05
Reviewed by Hajime Morita.

As ewk_view discards a queue for invalidation requests
as soon as it proceeds the repaint step, the newly produced invalidation requests
from ewk_view_layout_if_needed_recursive API can be discarded
and unpainted dirty area can occur.
As this issue is just caused by use of flush API in repaint step,
(internal API) ewk_view_repaints_get is replaced with ewk_view_repaints_pop
in order to make sure that old invalidation requests are removed from queue.
And in addition to that, ewk_view_repaint_add is modified
in order to let queue of invalidation requests keep a proper size.
As a result flush API won't be used for the repaint step.

  • ewk/ewk_private.h:
  • ewk/ewk_view.cpp:

(_ewk_view_repaint_add):
(_ewk_view_smart_calculate):
(ewk_view_repaints_pop):

  • ewk/ewk_view_single.cpp:

(_ewk_view_single_smart_repaints_process):

  • ewk/ewk_view_tiled.cpp:

(_ewk_view_tiled_smart_repaints_process):

7:01 AM Changeset in webkit [104159] by dpranke@chromium.org
  • 7 edits in trunk/Tools

webkitpy: clean up port factory methods
​https://bugs.webkit.org/show_bug.cgi?id=75590

Reviewed by Eric Seidel.

This change consolidates much of the "factory method" logic
of determining which port objects to create for a given set
of configurations by merging the separate factory methods in
chromium_gpu and google_chrome into PortFactory so that at least
all of the logic is in one place.

  • Scripts/webkitpy/layout_tests/port/chromium_gpu.py:
  • Scripts/webkitpy/layout_tests/port/chromium_gpu_unittest.py:
  • Scripts/webkitpy/layout_tests/port/factory.py:

(_port_name_from_arguments_and_options):
(get):

  • Scripts/webkitpy/layout_tests/port/factory_unittest.py:

(test_google_chrome):

  • Scripts/webkitpy/layout_tests/port/google_chrome.py:

(GoogleChromeLinux32Port):
(GoogleChromeLinux32Port.baseline_search_path):
(test_expectations_overrides):
(architecture):
(GoogleChromeLinux64Port):
(GoogleChromeLinux64Port.baseline_search_path):
(GoogleChromeMacPort):
(GoogleChromeMacPort.baseline_search_path):
(GoogleChromeWinPort):
(GoogleChromeWinPort.baseline_search_path):

  • Scripts/webkitpy/layout_tests/port/google_chrome_unittest.py:

(_verify_baseline_path):

6:59 AM Changeset in webkit [104158] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebCore

Remove duplicate file references from WebCore.xcodeproj
​https://bugs.webkit.org/show_bug.cgi?id=75581

Patch by Benjamin Poulain <​bpoulain@apple.com> on 2012-01-05
Reviewed by Andreas Kling.

Many generated DOMSVG files had two references.

In platform/graphics, there was two arm directory
with the same files related to ARM Neon.

  • WebCore.xcodeproj/project.pbxproj:
6:59 AM Changeset in webkit [104157] by Adam Roben
  • 6 edits
    1 copy
    8 adds in trunk

Turn ImageDiff into a stub .exe launcher and a .dll that contains all the real code

This allows us to add the Apple Application Support directory to PATH before trying to load
WebKit so that CoreGraphics.dll and friends can be found.

Fixes <​http://webkit.org/b/75572> ImageDiff.exe can't be run without first munging PATH

Reviewed by Anders Carlsson.

Tools:

  • DumpRenderTree/win/ImageDiff.vcproj:
  • DumpRenderTree/win/ImageDiffCommon.vsprops:

Changed to build ImageDiff.dll instead of ImageDiff.exe.

  • DumpRenderTree/win/ImageDiffLauncher.vcproj: Copied from

Tools/DumpRenderTree/win/ImageDiff.vcproj but changed to build the stub launcher.

  • DumpRenderTree/win/ImageDiffLauncherCommon.vsprops: Added.
  • DumpRenderTree/win/ImageDiffLauncherDebug.vsprops: Added.
  • DumpRenderTree/win/ImageDiffLauncherDebugAll.vsprops: Added.
  • DumpRenderTree/win/ImageDiffLauncherDebugCairoCFLite.vsprops: Added.
  • DumpRenderTree/win/ImageDiffLauncherProduction.vsprops: Added.
  • DumpRenderTree/win/ImageDiffLauncherRelease.vsprops: Added.
  • DumpRenderTree/win/ImageDiffLauncherReleaseCairoCFLite.vsprops: Added.
  • DumpRenderTree/win/ImageDiffWin.cpp: Added.

(dllLauncherEntryPoint): This function is called by the stub launcher and just calls through
to main.

  • win/DLLLauncher/DLLLauncherMain.cpp:

(main):
(wWinMain):
Added support for launching console programs. When USE_CONSOLE_ENTRY_POINT is defined, the
stub launcher acts as a console program and calls a console version of
dllLauncherEntryPoint.

Source/WebKit/win:

Add ImageDiffLauncher to WebKit.sln

It builds just after ImageDiff.

  • WebKit.vcproj/WebKit.sln:
6:58 AM Changeset in webkit [104156] by kenneth@webkit.org
  • 3 edits in trunk/Source/WebKit2

[Qt] Implement QQuickWebView::inputMethodQuery

Reviewed by Simon Hausmann.

Implement the current Qt5 queries of inputMethodQuery.

  • UIProcess/API/qt/qquickwebview.cpp:

(QQuickWebView::inputMethodQuery):

  • UIProcess/API/qt/qquickwebview_p.h:
6:57 AM Changeset in webkit [104155] by Adam Roben
  • 7 edits
    1 copy
    7 adds in trunk

Turn MiniBrowser into a stub .exe launcher and a .dll that contains all the real code

This allows us to add the Apple Application Support directory to PATH before trying to load
WebKit so that CoreGraphics.dll and friends can be found.

Fixes <​http://webkit.org/b/68576> <rdar://problem/10163853> MiniBrowser fails to launch when
double-clicked

Reviewed by Anders Carlsson.

Tools:

  • MiniBrowser/MiniBrowser.vcproj:
  • MiniBrowser/Configurations/MiniBrowserCommon.vsprops:

Changed to build MiniBrowser.dll instead of MiniBrowser.exe.

  • MiniBrowser/Configurations/MiniBrowserLauncherCommon.vsprops: Added.
  • MiniBrowser/Configurations/MiniBrowserLauncherDebug.vsprops: Added.
  • MiniBrowser/Configurations/MiniBrowserLauncherDebugAll.vsprops: Added.
  • MiniBrowser/Configurations/MiniBrowserLauncherDebugCairoCFLite.vsprops: Added.
  • MiniBrowser/Configurations/MiniBrowserLauncherProduction.vsprops: Added.
  • MiniBrowser/Configurations/MiniBrowserLauncherRelease.vsprops: Added.
  • MiniBrowser/Configurations/MiniBrowserLauncherReleaseCairoCFLite.vsprops: Added.
  • MiniBrowser/MiniBrowserLauncher.vcproj: Copied from Tools/MiniBrowser/MiniBrowser.vcproj

but changed to build the stub launcher.

  • MiniBrowser/win/main.cpp: Moved the /manifestdependency code to DLLLauncherMain.cpp, since

it needs to be compiled into the .exe instead of the .dll.
(DllMain): Added. We use this to store the DLL's HINSTANCE.
(dllLauncherEntryPoint): Renamed from _tWinMain. This is what gets called by the
DLLLauncherMain code.

  • win/DLLLauncher/DLLLauncherMain.cpp: Moved the /manifestdependency code from MiniBrowser

to here.

Source/WebKit/win:

Add MiniBrowserLauncher to WebKit.sln

It builds just after MiniBrowser.

  • WebKit.vcproj/WebKit.sln: Added MiniBrowserLauncher.
6:56 AM Changeset in webkit [104154] by Adam Roben
  • 2 edits in trunk/Tools

Make WinLauncher's window frame respond to clicks/drags again

Fixes <​http://webkit.org/b/75562> REGRESSION (r84990): Can't drag WinLauncher by its title
bar when Aero is disabled

Reviewed by Brent Fulgham.

  • WinLauncher/WinLauncher.cpp:

(WndProc): Always call up to our parent wndproc when we don't handle WM_NCHITTEST ourselves.
Also removed an unnecessary "break" keyword.

6:52 AM Changeset in webkit [104153] by Adam Roben
  • 6 edits
    1 copy
    12 adds in trunk

Turn WinLauncher into a stub .exe launcher and a .dll that contains all the real code

This allows us to add the Apple Application Support directory to PATH before trying to load
WebKit so that CoreGraphics.dll and friends can be found. Eventually all our WebKit apps
should use this mechanism.

Fixes <​http://webkit.org/b/68578> <rdar://problem/10163861> WinLauncher fails to launch when
double-clicked

Reviewed by Anders Carlsson.

Tools:

  • WinLauncher/WinLauncher.cpp: Removed InitInstance.

(DllMain): Added. Stores the DLL HINSTANCE in the global hInst variable. This used to be
done by the now-removed InitInstance function.
(dllLauncherEntryPoint): Renamed from _tWinMain. This is the function that our stub launcher
calls. Merged code from InitInstance into this function, and change to use the global hInst
variable, which corresponds to WinLauncher.dll, rather than the HINSTANCE passed into this
function, which corresponds to WinLauncher.exe.

  • WinLauncher/WinLauncher.vcproj:
  • WinLauncher/WinLauncherCommon.vsprops:

Changed to build WinLauncher.dll instead of WinLauncher.exe.

  • WinLauncher/WinLauncherLauncher.rc: Added.
  • WinLauncher/WinLauncherLauncherResource.h: Added.

This just references the WinLauncher.ico file to set the application icon (though it looks
like that's a default icon).

  • WinLauncher/WinLauncherLauncher.vcproj: Copied from Tools/WinLauncher/WinLauncher.vcproj

but changed to build WinLauncher.exe using the new DLLLauncherMain.cpp file.

  • WinLauncher/WinLauncherLauncherCommon.vsprops: Added.
  • WinLauncher/WinLauncherLauncherDebug.vsprops: Added.
  • WinLauncher/WinLauncherLauncherDebugAll.vsprops: Added.
  • WinLauncher/WinLauncherLauncherDebugCairoCFLite.vsprops: Added.
  • WinLauncher/WinLauncherLauncherProduction.vsprops: Added.
  • WinLauncher/WinLauncherLauncherRelease.vsprops: Added.
  • WinLauncher/WinLauncherLauncherReleaseCairoCFLite.vsprops: Added.
  • win/DLLLauncher/DLLLauncherMain.cpp: Added.

(enableTerminationOnHeapCorruption): Uses HeapSetInformation to tell Windows to kill the
program if it detects heap corruption.
(getStringValue): Helper function to read a string value from the registry.
(applePathFromRegistry): Helper function to read the InstallDir for a given Apple
application from the registry.
(appleApplicationSupportDirectory): Helper function to return the path to the Apple
Application Support directory.
(copyEnvironmentVariable): Helper function to copy an environment variable's value.
(prependPath): Helper function to prepend a directory to the PATH.
(fatalError): Helper function to display an error to the user.
(wWinMain): Add Apple Application Support to the PATH, then load our DLL and call its
dllLauncherEntryPoint function.

Source/WebKit/win:

Add WinLauncherLauncher to WebKit.sln

It builds just after WinLauncher.

  • WebKit.vcproj/WebKit.sln: Added WinLauncherLauncher.
6:40 AM Changeset in webkit [104152] by Antti Koivisto
  • 2 edits in trunk/Source/WebCore

Improve SelectorChecker::determineSelectorScopes
​https://bugs.webkit.org/show_bug.cgi?id=75619

Reviewed by Andreas Kling.

SelectorChecker::determineSelectorScopes currently searches to the end of the selector chain and
then sees if the last one is suitable to be a scope. A better algorithm will find a scope from
anywhere in the chain.

With this patch we search the whole chain for potential scopes. We now prefer id scopes
over class scopes. Scopes can be found for rules using sibling selectors too.

On both engadget.com and nytimes.com, we can now skip one more full style recalc due to
the improved stylesheet analysis.

  • css/SelectorChecker.cpp:

(WebCore::SelectorChecker::determineSelectorScopes):

6:38 AM Changeset in webkit [104151] by Carlos Garcia Campos
  • 2 edits in trunk/Source/WebCore

[GTK] Use the default screen in PlatformScreenGtk methods when they are called with a NULL widget
​https://bugs.webkit.org/show_bug.cgi?id=75620

Reviewed by Philippe Normand.

We are currently retuning an empty rectangle for
screenRect/screenAvailableRect and 0 for getVisual.

  • platform/gtk/PlatformScreenGtk.cpp:

(WebCore::getVisual):
(WebCore::screenRect):
(WebCore::screenAvailableRect):

6:12 AM Changeset in webkit [104150] by hclam@chromium.org
  • 2 edits
    6 deletes in trunk/LayoutTests

Not reviewed. Update Chromium port test expectations.

  • platform/chromium-win-xp/fast/backgrounds/animated-svg-as-mask-expected.png: Removed.
  • platform/chromium-win-xp/svg/W3C-SVG-1.1/masking-path-02-b-expected.png: Removed.
  • platform/chromium-win-xp/svg/as-background-image/animated-svg-as-background-expected.png: Removed.
  • platform/chromium-win-xp/svg/custom/glyph-selection-lang-attribute-expected.png: Removed.
  • platform/chromium-win-xp/svg/text/text-align-01-b-expected.png: Removed.
  • platform/chromium-win-xp/svg/text/text-path-01-b-expected.png: Removed.
  • platform/chromium/test_expectations.txt:
6:02 AM Changeset in webkit [104149] by commit-queue@webkit.org
  • 4 edits in trunk/Source/WebKit/chromium

[Chromium] Remove WebSettings::setAcceleratedDrawingEnabled from public API.
​https://bugs.webkit.org/show_bug.cgi?id=74125

Patch by David Reveman <​reveman@chromium.org> on 2012-01-05
Reviewed by James Robinson.

Remove setAcceleratedDrawingEnabled method from WebSettings class.

  • public/WebSettings.h:
  • src/WebSettingsImpl.cpp:
  • src/WebSettingsImpl.h:
6:02 AM Changeset in webkit [104148] by commit-queue@webkit.org
  • 2 edits in trunk/LayoutTests

[Chromium] Expected test failures after Skia is rolled to 2918
​https://bugs.webkit.org/show_bug.cgi?id=75019

Fixed now that Skia has rolled past r2936.

Unreviewed Chromium port test expectations update.

Patch by Stephen Chenney <​schenney@chromium.org> on 2012-01-05

  • platform/chromium/test_expectations.txt:
5:38 AM Changeset in webkit [104147] by hclam@chromium.org
  • 2 edits in trunk/Source/WebCore

Unreviewed. Build fix.

Adding missing const_cast<> to fix compilation failures due to r104143.

  • platform/audio/VectorMath.cpp:

(WebCore::VectorMath::zvmul):

5:27 AM Changeset in webkit [104146] by abarth@webkit.org
  • 5 edits
    3 adds in trunk/Source/WebCore

Move Gamepad declarations from Navigator.idl into Modules/gamepad
​https://bugs.webkit.org/show_bug.cgi?id=75559

Reviewed by Eric Seidel.

This patch moves the gamepad-related declarations in Navigator.idl into
Modules/gamepad. The next step is to move the state into
Modules/gamepad as well.

  • Modules/gamepad/NavigatorGamepad.cpp: Added.

(WebCore::NavigatorGamepad::NavigatorGamepad):
(WebCore::NavigatorGamepad::~NavigatorGamepad):
(WebCore::NavigatorGamepad::webkitGamepads):

  • Modules/gamepad/NavigatorGamepad.h: Added.
  • Modules/gamepad/NavigatorGamepad.idl: Added.
  • WebCore.gypi:
  • page/Navigator.cpp:

(WebCore::Navigator::gamepads):

  • page/Navigator.h:
  • page/Navigator.idl:
5:04 AM Changeset in webkit [104145] by Carlos Garcia Campos
  • 5 edits in trunk/Source/WebKit2

[GTK] Add methods to get/set the WebView zoom level to WebKit2 GTK+ API
​https://bugs.webkit.org/show_bug.cgi?id=75248

Reviewed by Gustavo Noronha Silva.

It uses the page zoom factor unconditionally for now.

  • UIProcess/API/gtk/WebKitWebView.cpp:

(webkitWebViewSetProperty):
(webkitWebViewGetProperty):
(webkit_web_view_class_init): Add WebKitWebView:zoom-level
property.
(webkit_web_view_set_zoom_level): Set current zoom level.
(webkit_web_view_get_zoom_level): Get current zoom level.

  • UIProcess/API/gtk/WebKitWebView.h:
  • UIProcess/API/gtk/docs/webkit2gtk-sections.txt: Add new symbols.
  • UIProcess/API/gtk/tests/TestWebKitWebView.cpp:

(testWebViewZoomLevel):
(beforeAll):

4:55 AM Changeset in webkit [104144] by alexis.menard@openbossa.org
  • 5 edits in trunk

getComputedStyle for border-image is not implemented.
​https://bugs.webkit.org/show_bug.cgi?id=75347

Reviewed by Tony Chang.

Source/WebCore:

Implement getComputedStyle for border-color.

fast/css/getComputedStyle/computed-style-border-image.html was extended to cover the
new feature.

  • css/CSSComputedStyleDeclaration.cpp:

(WebCore::CSSComputedStyleDeclaration::getPropertyCSSValue):

LayoutTests:

Extend existing test to cover that we return correct values for border-image.

  • fast/css/getComputedStyle/computed-style-border-image-expected.txt:
  • fast/css/getComputedStyle/computed-style-border-image.html:
4:54 AM Changeset in webkit [104143] by commit-queue@webkit.org
  • 5 edits in trunk/Source/WebCore

Add a SSE2 optimized function zvmul in VectorMatch
​https://bugs.webkit.org/show_bug.cgi?id=74842

Patch by Xingnan Wang <​xingnan.wang@intel.com> on 2012-01-05
Reviewed by Kenneth Russell.

Use zvmul in FFTFrameFFMPEG.cpp::multiply() and FFTFrameMac.cpp::multiply().

  • platform/audio/VectorMath.cpp:

(WebCore::VectorMath::zvmul):

  • platform/audio/VectorMath.h:
  • platform/audio/ffmpeg/FFTFrameFFMPEG.cpp:

(WebCore::FFTFrame::multiply):

  • platform/audio/mac/FFTFrameMac.cpp:

(WebCore::FFTFrame::multiply):

4:51 AM Changeset in webkit [104142] by hclam@chromium.org
  • 4 edits in trunk/Source/WebCore

Unreviewed. Build fix for Chromium Mac Clang build.

Push using namespace WTF to .cpp file from .h file.

  • platform/text/TextCodecASCIIFastPath.h:
  • platform/text/TextCodecLatin1.cpp:
  • platform/text/TextCodecUTF8.cpp:
4:50 AM Changeset in webkit [104141] by Csaba Osztrogonác
  • 2 edits in trunk/LayoutTests

[Qt][WK2]REGRESSION(r104068): It made editing/pasteboard/copy-in-password-field.html fail
​https://bugs.webkit.org/show_bug.cgi?id=75615

Unreviewed gardening.

  • platform/qt-wk2/Skipped: Skip editing/pasteboard/copy-in-password-field.html to paint the bot green.
4:42 AM WebKit Team edited by kubo@profusion.mobi
Add myself to the list of committers (diff)
4:23 AM Changeset in webkit [104140] by commit-queue@webkit.org
  • 3 edits
    3 adds in trunk

WebKit adds vertical paddings and borders to the fixed width of CSS tables
​https://bugs.webkit.org/show_bug.cgi?id=74955

Patch by Max Vujovic <​mvujovic@adobe.com> on 2012-01-05
Reviewed by Julien Chaffraix.

Source/WebCore:

Test: fast/table/css-table-width.html

  • rendering/RenderTable.cpp:

(WebCore::RenderTable::computeLogicalWidth):

Changed the width calculation for CSS tables to take into account horizontal
paddings and borders instead of vertical paddings and borders.

LayoutTests:

Added tests to check that the fixed width style is applied correctly to
CSS tables with respect to table paddings and borders.

  • fast/table/css-table-width-expected.txt: Added.
  • fast/table/css-table-width.html: Added.
  • fast/table/script-tests/css-table-width.js: Added.

(computeCSSTableOffsetWidth):
(computeCSSTableOffsetHeight):
(computeCSSTableProperty):

4:22 AM Changeset in webkit [104139] by peter@chromium.org
  • 7 edits
    4 adds in trunk/Source

[Chromium] Upstream the RenderTheme and ScrollbarTheme for Android
​https://bugs.webkit.org/show_bug.cgi?id=74614

Reviewed by Adam Barth.

Source/WebCore:

Upstream the RenderTheme and ScrollbarTheme for Android. These are
mostly derived from their Linux variants.

RenderThemeChromiumAndroid inherits from RenderThemeChromiumLinux and
only overrides the three methods which have different behavior.
Scrollbars will be drawn in the threaded compositor for Android (which
will be upstreamed later), but for increased layout test parity we'll
match Chromium-Linux.

No new tests. This code will be exercised by existing layout tests, and
by manually verifying that theme parts render as expected.

  • WebCore.gyp/WebCore.gyp:
  • WebCore.gypi:
  • platform/chromium/ScrollbarThemeChromiumAndroid.cpp: Added.

(WebCore::ScrollbarTheme::nativeTheme):
(WebCore::ScrollbarThemeChromiumAndroid::scrollbarThickness):
(WebCore::ScrollbarThemeChromiumAndroid::paintScrollbarBackground):
(WebCore::ScrollbarThemeChromiumAndroid::shouldCenterOnThumb):
(WebCore::ScrollbarThemeChromiumAndroid::buttonSize):
(WebCore::ScrollbarThemeChromiumAndroid::minimumThumbLength):

  • platform/chromium/ScrollbarThemeChromiumAndroid.h: Added.
  • rendering/RenderThemeChromiumAndroid.cpp: Added.

(WebCore::RenderThemeChromiumAndroid::create):
(WebCore::RenderTheme::themeForPage):
(WebCore::RenderThemeChromiumAndroid::~RenderThemeChromiumAndroid):
(WebCore::RenderThemeChromiumAndroid::systemColor):
(WebCore::RenderThemeChromiumAndroid::adjustInnerSpinButtonStyle):

  • rendering/RenderThemeChromiumAndroid.h: Added.
  • rendering/RenderThemeChromiumLinux.cpp:
  • rendering/RenderThemeChromiumLinux.h:

Source/WebKit/chromium:

  • src/WebViewImpl.cpp:

(WebKit::WebViewImpl::setScrollbarColors):
(WebKit::WebViewImpl::setSelectionColors):

4:09 AM Changeset in webkit [104138] by Csaba Osztrogonác
  • 2 edits in trunk/LayoutTests

[Qt] Unreviewed gardening. Skip a new failing test.

  • platform/qt/Skipped:
3:40 AM Changeset in webkit [104137] by fsamuel@chromium.org
  • 4 edits in trunk/Source/WebKit/chromium

[Chromium] Expose Fixed Position Layout to WebViewImpl
​https://bugs.webkit.org/show_bug.cgi?id=75556

Reviewed by Darin Fisher.

  • public/WebView.h:
  • src/WebViewImpl.cpp:

(WebKit::WebViewImpl::shouldLayoutFixedElementsRelativeToFrame):
(WebKit::WebViewImpl::setShouldLayoutFixedElementsRelativeToFrame):

  • src/WebViewImpl.h:
3:33 AM Changeset in webkit [104136] by Csaba Osztrogonác
  • 2 edits in trunk/Source/JavaScriptCore

unshift/pop fifo may consume excessive memory
​https://bugs.webkit.org/show_bug.cgi?id=75588

Reviewed by Zoltan Herczeg.

Buildfix after r104120.

  • runtime/JSArray.cpp: Remove useless asserts, baecause unsigned expression >= 0 is always true

(JSC::JSArray::unshiftCount):

3:24 AM Changeset in webkit [104135] by zherczeg@webkit.org
  • 2 edits in trunk/Source/JavaScriptCore

Unreviewed gardening after r104134.

  • wtf/Assertions.cpp:
3:06 AM Changeset in webkit [104134] by zherczeg@webkit.org
  • 2 edits in trunk/Source/JavaScriptCore

Unreviewed gardening after r75605.

Rubber stamped by NOBODY Csaba Osztrogonác.

  • wtf/Assertions.cpp:
2:57 AM Changeset in webkit [104133] by commit-queue@webkit.org
  • 4 edits
    1 add in trunk

Source/WebCore: Fix potential superlinear runtime of multiple indirect adjenceny combinators.
​https://bugs.webkit.org/show_bug.cgi?id=75083

Patch by Allan Sandfeld Jensen <​allan.jensen@nokia.com> on 2012-01-05
Reviewed by Antti Koivisto.

A sequence of indirect adjencency combinator such as "li ~ li ~ la" could with the
former algorithm potentially do an quadratic number of element matches.
The recursive matching algorithm now detects cases where all siblings have
failed one indirect sibling match and fails the entire selector.

Test: perf/nested-combined-selectors.html

  • css/SelectorChecker.cpp:

(WebCore::SelectorChecker::checkSelector): Return SelectorFailsAllSiblings when
all siblings have failed a selector component.

  • css/SelectorChecker.h: Add SelectorFailsAllSiblings enum value

LayoutTests: Test of styling performance of selectors with multiple indirect combinators

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

Reviewed by Antti Koivisto.

  • perf/nested-combined-selectors.html: Added.

Patch by Allan Sandfeld Jensen <​allan.jensen@nokia.com> on 2012-01-05

2:56 AM Changeset in webkit [104132] by haraken@chromium.org
  • 4 edits in trunk/Tools

prepare-ChangeLog cannot parse python code that nests classes and defs
​https://bugs.webkit.org/show_bug.cgi?id=75524

Reviewed by Ryosuke Niwa.

This patch fixes the python parser of prepare-ChangeLog so that it can parse
python code that nests classes and defs, like this:

class C:

pass
def f():

pass

pass

Also, this patch fixes the python parser so that it inserts a dummy line
at the end of the file to pop out all popped scopes.

  • Scripts/prepare-ChangeLog:

(get_function_line_ranges_for_python):

  • Scripts/webkitperl/prepare-ChangeLog_unittest/resources/python_unittests.py:

Added test cases that nest classes and defs.
(func7):
(func7.func8):
(func7.func8.func9):
(Class2):
(Class2.Class3):
(Class2.Class3.Class4):
(Class5):
(Class5.func10):
(Class5.func11):

  • Scripts/webkitperl/prepare-ChangeLog_unittest/resources/python_unittests-expected.txt:
2:20 AM Changeset in webkit [104131] by kling@webkit.org
  • 2 edits in trunk/Source/WebCore

InspectorStyleSheet: Avoid cloning CSSRuleLists.
<​http://webkit.org/b/75603>

Reviewed by Ryosuke Niwa.

Don't filter out @charset rules from CSSRuleLists. This was forcing us to clone
the stylesheet rule list, and is unnecessary since InspectorStyleSheet disregards
any rule that isn't either a style rule or a rule with an internal rule list.

  • inspector/InspectorStyleSheet.cpp:

(WebCore::asCSSRuleList):
(WebCore::InspectorStyleSheet::buildObjectForStyleSheet):

Don't pass omitCharsetRules=true to the CSSRuleList constructor.

2:18 AM Changeset in webkit [104130] by kling@webkit.org
  • 13 edits in trunk/Source/WebCore

Make elements with attributes smaller by eliminating the ref count in NamedNodeMap.
<​http://webkit.org/b/75068>

Reviewed by Antti Koivisto.

Remove NamedNodeMap's inheritance from RefCounted and forward the ref()/deref()
calls to its owner Element, effectively reducing the size of an element that has
attributes by 4 bytes.

This reduces memory consumption by 311 kB when viewing the full HTML5 spec.

  • dom/NamedNodeMap.cpp:

(WebCore::NamedNodeMap::ref):
(WebCore::NamedNodeMap::deref):

Added. Forwards the operation to m_element. Only ever used by the DOM wrappers.

  • dom/NamedNodeMap.h:

(WebCore::NamedNodeMap::create):

  • dom/DocumentType.h:
  • dom/Element.cpp:

(WebCore::Element::parserSetAttributeMap):

  • dom/Element.h:
  • html/parser/HTMLConstructionSite.cpp:
  • html/parser/HTMLToken.h:

(WebCore::AtomicHTMLToken::AtomicHTMLToken):

  • html/parser/HTMLTreeBuilder.cpp:

(WebCore::HTMLTreeBuilder::processFakeStartTag):
(WebCore::HTMLTreeBuilder::attributesForIsindexInput):

  • html/parser/HTMLTreeBuilder.h:
  • html/parser/TextDocumentParser.cpp:

(WebCore::TextDocumentParser::insertFakePreElement):

  • xml/parser/MarkupTokenBase.h:

(WebCore::AtomicMarkupTokenBase::AtomicMarkupTokenBase):
(WebCore::AtomicMarkupTokenBase::takeAttributes):

  • xml/parser/XMLToken.h:

(WebCore::AtomicXMLToken::AtomicXMLToken):

Store NamedNodeMap in OwnPtr/PassOwnPtr, rather than RefPtr/PassRefPtr.

2:08 AM Changeset in webkit [104129] by Carlos Garcia Campos
  • 10 edits in trunk

[GTK] Rename webkit_web_view_load_alternate_html as webkit_web_view_replace_content in WebKit2 GTK+
​https://bugs.webkit.org/show_bug.cgi?id=75433

Reviewed by Martin Robinson.

Source/WebKit2:

  • UIProcess/API/gtk/WebKitWebView.cpp:

(webkitWebViewLoadFail): Use webkit_web_view_replace_content().
(webkitWebViewLoadChanged): Do not emit load-changed signal when
replacing content.
(webkitWebViewLoadFailed): Do not emit load-failed signal when
replacing content.
(webkitWebViewSetEstimatedLoadProgress): Do not emit
notify::estimated-load-progress signal when replacing content.
(webkit_web_view_replace_content):

  • UIProcess/API/gtk/WebKitWebView.h:
  • UIProcess/API/gtk/docs/webkit2gtk-sections.txt:
  • UIProcess/API/gtk/tests/TestWebKitWebLoaderClient.cpp:

(beforeAll):

  • UIProcess/API/gtk/tests/TestWebKitWebView.cpp:

(replaceContentTitleChangedCallback):
(replaceContentLoadCallback):
(testWebViewReplaceContent):
(beforeAll):

  • UIProcess/API/gtk/tests/WebViewTest.cpp:

(WebViewTest::replaceContent):

  • UIProcess/API/gtk/tests/WebViewTest.h:

Tools:

  • MiniBrowser/gtk/BrowserWindow.c:

(webViewLoadFailed): Reset the progress bar when load fails.
(browserWindowConstructed): Connect to load-failed signal.

2:03 AM Changeset in webkit [104128] by noel.gordon@gmail.com
  • 2 edits
    3 moves
    4 adds in trunk/LayoutTests

[chromium] MAC: update expectations for CSS3 linear gradient tests
​https://bugs.webkit.org/show_bug.cgi?id=55745

Reviewed by Stephen White.

  • platform/chromium-cg-mac-leopard/fast/gradients/css3-color-stop-units-expected.png: Added.
  • platform/chromium-cg-mac-leopard/fast/gradients/css3-color-stops-expected.png: Added.
  • platform/chromium-cg-mac-leopard/fast/gradients/css3-linear-angle-gradients-expected.png: Added.
  • platform/chromium-mac-leopard/fast/gradients/css3-color-stop-units-expected.png: Added.
  • platform/chromium-mac-snowleopard/fast/gradients/css3-color-stop-units-expected.png: Renamed from LayoutTests/platform/chromium-mac/fast/gradients/css3-color-stop-units-expected.png.
  • platform/chromium-mac-snowleopard/fast/gradients/css3-color-stops-expected.png: Renamed from LayoutTests/platform/chromium-mac/fast/gradients/css3-color-stops-expected.png.
  • platform/chromium-mac-snowleopard/fast/gradients/css3-linear-angle-gradients-expected.png: Renamed from LayoutTests/platform/chromium-mac/fast/gradients/css3-linear-angle-gradients-expected.png.
  • platform/chromium/test_expectations.txt:
1:38 AM Changeset in webkit [104127] by benjamin@webkit.org
  • 12 edits
    3 adds in trunk/Source

Improve charactersAreAllASCII() to compare multiple characters at a time
​https://bugs.webkit.org/show_bug.cgi?id=74063

Reviewed by Darin Adler.

Source/JavaScriptCore:

A new header ASCIIFastPath.h contains the functions related to
the detection of ASCII by using machine words. Part of it comes from
WebCore's TextCodecASCIIFastPath.h.

The function charactersAreAllASCII() is moved to TextCodecASCIIFastPath.h
and is implemented with computer word comparison.
The gain over the previous implementation of charactersAreAllASCII() is of
the order of how many comparison are avoided (4x, 8x, 16x depending on the
format and the CPU type).

(WTF::isAlignedToMachineWord):
(WTF::alignToMachineWord):
(WTF::isAllASCII):
(WTF::charactersAreAllASCII):

  • wtf/text/WTFString.h:
  • wtf/wtf.pro:

Source/JavaScriptGlue:

Add forwarding header for ASCIIFastPath.h.

  • ForwardingHeaders/wtf/text/ASCIIFastPath.h: Added.

Source/WebCore:

Move some part of TextCodecASCIIFastPath.h to WTF in ASCIIFastPath.h.
The function isAllASCII() is changed to the template version which now works
with both LChar and UChar.

  • ForwardingHeaders/wtf/text/ASCIIFastPath.h: Added.
  • platform/text/TextCodecASCIIFastPath.h:

(WebCore::copyASCIIMachineWord):

  • platform/text/TextCodecLatin1.cpp:

(WebCore::TextCodecLatin1::decode):

  • platform/text/TextCodecUTF8.cpp:

(WebCore::TextCodecUTF8::decode):

1:29 AM Changeset in webkit [104126] by ericu@chromium.org
  • 19 edits
    1 delete in trunk

[fileapi] WebKitFlags should not be constructable per Directories & System spec
​https://bugs.webkit.org/show_bug.cgi?id=68916

Reviewed by Eric Seidel.

Source/WebCore:

Remove IDL for the object and all DOM references to it.

  • fileapi/WebKitFlags.idl: Removed.
  • page/DOMWindow.idl:
  • workers/WorkerContext.idl: Remove references to the JSC/V8 objects compiled from the IDL.
  • bindings/js/JSDirectoryEntryCustom.cpp: (WebCore::JSDirectoryEntry::getFile): (WebCore::JSDirectoryEntry::getDirectory):
  • bindings/js/JSDirectoryEntrySyncCustom.cpp: (WebCore::getFlags):
  • bindings/v8/custom/V8DirectoryEntryCustom.cpp: (WebCore::V8DirectoryEntry::getDirectoryCallback): (WebCore::V8DirectoryEntry::getFileCallback):
  • bindings/v8/custom/V8DirectoryEntrySyncCustom.cpp: (WebCore::getFlags): Fix up build files.
  • WebCore.gypi:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • WebCore/CMakeLists.txt:
  • WebCore/CodeGenerators.pri:
  • WebCore/DerivedSources.cpp:
  • WebCore/DerivedSources.make:
  • WebCore/GNUmakefile.list.am:

LayoutTests:

Remove a test that constructed the object; add one that tests error-handling for non-object parameters while I'm in there.

  • fast/filesystem/flags-passing-expected.txt:
  • fast/filesystem/script-tests/flags-passing.js:

Remove a special case reference to the global WebKitFlags constructor.

  • fast/dom/script-tests/prototype-inheritance.js:
1:16 AM Changeset in webkit [104125] by mrowe@apple.com
  • 2 edits in trunk/Source/JavaScriptCore

<​http://webkit.org/b/75606> [Mac] WTF logging functions should output to both stderr and ASL

We should always log to both ASL and stderr on platforms where this won't result in launchd
duplicating the messages.

Reviewed by Dan Bernstein.

  • wtf/Assertions.cpp:

(vprintf_stderr_common):

1:16 AM Changeset in webkit [104124] by mrowe@apple.com
  • 2 edits in trunk/Source/JavaScriptCore

<​http://webkit.org/b/75605> WTF logging functions should call vprintf_stderr_common only once per line

Several of the WTF logging functions make multiple calls to vprintf_stderr_common to output a
single line of text. This results in strangely formatted output if vprintf_stderr_common is
retargeted to an output device that is message-oriented (such as ASL) rather than stream-oriented
like stderr.

Reviewed by Dan Bernstein.

  • wtf/Assertions.cpp:

(vprintf_stderr_with_prefix): Helper function to prepend a given prefix on to the given format
string before handing it off to vprintf_stderr_common. This requires disabling warnings about
calling a printf-like function with a non-literal format string for this piece of code. It's
safe in this particular case as vprintf_stderr_with_prefix is only ever given a literal prefix.
(vprintf_stderr_with_trailing_newline): Helper function to append a trailling newline on to the
given format string if one does not already exist. It requires the same treatment with regards
to the non-literal format string warning.
(WTFReportAssertionFailureWithMessage): Switch to using vprintf_stderr_with_prefix.
(WTFReportBacktrace): Switch from calling fprintf directly to using fprintf_stderr_common.
(WTFReportFatalError): Switch to using vprintf_stderr_with_prefix.
(WTFReportError): Ditto.
(WTFLog): Switch to using vprintf_stderr_with_trailing_newline.
(WTFLogVerbose): Ditto.

12:54 AM WebKit Team edited by pfeldman@chromium.org
(diff)
12:53 AM WebKit Team edited by pfeldman@chromium.org
(diff)
12:51 AM Changeset in webkit [104123] by commit-queue@webkit.org
  • 5 edits
    2 adds in trunk

Crash due to first-letter block processing
​https://bugs.webkit.org/show_bug.cgi?id=74009

Source/WebCore:

Fixing the way updateFirstLetter() finds the remaining text fragment
for a given first-letter. Previously this was unreliable in some
circumstances.

This patch provides a reliable mechanism to identify the remaining
text by storing first-letter to remaining text associations in a
hash map, managed by methods in RenderBoxModelObject.

Patch by Ken Buchanan <​kenrb@chromium.org> on 2012-01-05
Reviewed by David Hyatt.

  • rendering/RenderBlock.cpp:

(WebCore::RenderBlock::updateFirstLetter)

  • rendering/RenderBoxModelObject.cpp:

(WebCore::RenderBoxModelObject::willBeDestroyed):
(WebCore::RenderBoxModelObject::setFirstLetterRemainingText): Added
(WebCore::RenderBoxModelObject::firstLetterRemainingText): Added

  • rendering/RenderBoxModelObject.h:

(WebCore::RenderBoxModelObject::setFirstLetterRemainingText): Added
(WebCore::RenderBoxModelObject::firstLetterRemainingText): Added

LayoutTests:

Test for crashing condition in 74009.

Patch by Ken Buchanan <​kenrb@chromium.org> on 2012-01-05
Reviewed by David Hyatt.

  • fast/css/first-letter-inline-flow-split-table-crash-expected.txt: Added
  • fast/css/first-letter-inline-flow-split-table-crash.html: Added
12:51 AM Changeset in webkit [104122] by commit-queue@webkit.org
  • 2 edits
    2 adds in trunk/Source/WebKit2

[WK2][EFL] creating dummy functions for Download class on Efl port.
​https://bugs.webkit.org/show_bug.cgi?id=75246

dummy functions for Download class on Efl port

Patch by Keunsoon Lee <​keunsoon.lee@samsung.com> on 2012-01-05
Reviewed by Hajime Morita.

  • PlatformEfl.cmake: add new file to compile
  • WebProcess/Downloads/efl: Added.
  • WebProcess/Downloads/efl/DownloadEfl.cpp: Added.

(WebKit::Download::start):
(WebKit::Download::startWithHandle):
(WebKit::Download::cancel):
(WebKit::Download::platformInvalidate):
(WebKit::Download::didDecideDestination):
(WebKit::Download::platformDidFinish):
(WebKit::Download::receivedCredential):
(WebKit::Download::receivedRequestToContinueWithoutCredential):
(WebKit::Download::receivedCancellation):
(WebKit::Download::useCredential):
(WebKit::Download::continueWithoutCredential):
(WebKit::Download::cancelAuthenticationChallenge):

12:41 AM WebKit Team edited by ryuan.choi@samsung.com
(diff)
12:34 AM Changeset in webkit [104121] by mihnea@adobe.com
  • 6 edits
    2 adds in trunk

Source/WebCore: Crash in RenderRegion::getRegionRangeForBox.
​https://bugs.webkit.org/show_bug.cgi?id=74781

Reviewed by David Hyatt.

Test: fast/regions/region-range-for-box-crash.html

This patch fixes 2 issues:

  1. When removing a region from a flow thread, we clear the region range information for boxes since

this information is accurate only after the regions are laid out.

  1. While the regions are invalidated (content of flow thread is not yet laid out in regions), the

functions that ask for properties of content in regions should bail out early.

  • rendering/RenderBlock.cpp:

(WebCore::RenderBlock::regionAtBlockOffset):
(WebCore::RenderBlock::logicalWidthChangedInRegions):

  • rendering/RenderBox.cpp:

(WebCore::RenderBox::clearRenderBoxRegionInfo):

  • rendering/RenderFlowThread.cpp:

(WebCore::RenderFlowThread::removeRegionFromThread):
(WebCore::RenderFlowThread::repaintRectangleInRegions):
(WebCore::RenderFlowThread::mapFromFlowToRegion):
(WebCore::RenderFlowThread::contentLogicalWidthOfFirstRegion):
(WebCore::RenderFlowThread::contentLogicalHeightOfFirstRegion):
(WebCore::RenderFlowThread::contentLogicalLeftOfFirstRegion):
(WebCore::RenderFlowThread::firstRegion):
(WebCore::RenderFlowThread::lastRegion):
(WebCore::RenderFlowThread::getRegionRangeForBox):

  • rendering/RenderFlowThread.h:

LayoutTests: Crash in RenderFlowThread::getRegionRangeForBox.
​https://bugs.webkit.org/show_bug.cgi?id=74781

Reviewed by David Hyatt.

  • fast/regions/region-range-for-box-crash-expected.txt: Added.
  • fast/regions/region-range-for-box-crash.html: Added.
12:00 AM Changeset in webkit [104120] by barraclough@apple.com
  • 3 edits in trunk/Source/JavaScriptCore

unshift/pop fifo may consume excessive memory
​https://bugs.webkit.org/show_bug.cgi?id=75588

Reviewed by Sam Weinig.

The Array object commonly store data in a vector, consisting of a portion that
is in use, a pre-capacity (m_indexBias) and a post-capacity (the delta between
m_length and m_vectorLength). Calls to pop with grow the post-capacity, and the
current algorithm for increasePrefixVectorLength (used by unshift) will never
stink the post-capacity, so a unshift/pop fifo may consume an inordinate amount
of memory, whilst having a relatively small active length.

  • runtime/JSArray.cpp:

(JSC::storageSize):

  • sizeof(JSValue) should be sizeof(WriteBarrier<Unknown>)

(JSC::SparseArrayValueMap::put):

  • sizeof(JSValue) should be sizeof(WriteBarrier<Unknown>)

(JSC::JSArray::increaseVectorLength):

  • sizeof(JSValue) should be sizeof(WriteBarrier<Unknown>)

(JSC::JSArray::unshiftCountSlowCase):

  • renamed from increaseVectorPrefixLength (this was a bad name, since it also moved the ArrayStorage header), rewritten.

(JSC::JSArray::shiftCount):

  • sizeof(JSValue) should be sizeof(WriteBarrier<Unknown>), count should be unsigned

(JSC::JSArray::unshiftCount):

  • sizeof(JSValue) should be sizeof(WriteBarrier<Unknown>), count should be unsigned, increaseVectorPrefixLength renamed to unshiftCountSlowCase

(JSC::JSArray::sortNumeric):

  • runtime/JSArray.h:
    • Updated function declarations, m_indexBias should be unsigned.

Jan 4, 2012:

11:39 PM Changeset in webkit [104119] by mrowe@apple.com
  • 3 edits in trunk/Source/JavaScriptCore

<​http://webkit.org/b/75604> All instances of JSC::ArgumentsData appear to be leaked by JSC::Arguments

Since JSC::Arguments has an OwnPtr for a member it needs to override destroy
to ensure that the correct destructor is invoked. This is necessary because
JSCell subclasses all intentionally have non-virtual destructors.

Reviewed by Filip Pizlo.

  • runtime/Arguments.cpp:

(JSC::Arguments::destroy):

  • runtime/Arguments.h:
11:12 PM Changeset in webkit [104118] by ddkilzer@apple.com
  • 2 edits in trunk/Source/ThirdParty

Remove HEADER_SEARCH_PATHS from Production configurations for gtest

Reviewed by Mark Rowe.

  • gtest/xcode/gtest.xcodeproj/project.pbxproj: Remove

HEADER_SEARCH_PATHS from Production configurations to match
changes made for Debug and Release configurations in r104091.

10:49 PM Changeset in webkit [104117] by rniwa@webkit.org
  • 2 edits in trunk/Source/WebKit/chromium

Roll Chromium DEPS from 116031 to 116459.

  • DEPS:
8:59 PM Changeset in webkit [104116] by commit-queue@webkit.org
  • 11 edits in trunk/Source/WebCore

Unreviewed, rolling out r104084.
​http://trac.webkit.org/changeset/104084
​https://bugs.webkit.org/show_bug.cgi?id=75600

Likely kills dom-perf benchmark in chromium
​http://build.chromium.org/f/chromium/perf/linux-
release/dom_perf/report.html?history=150&rev=116444 (Requested
by dslomov on #webkit).

Patch by Sheriff Bot <​webkit.review.bot@gmail.com> on 2012-01-04

  • dom/Attr.cpp:

(WebCore::Attr::setValue):
(WebCore::Attr::childrenChanged):

  • dom/DynamicNodeList.cpp:

(WebCore::DynamicSubtreeNodeList::DynamicSubtreeNodeList):
(WebCore::DynamicSubtreeNodeList::length):
(WebCore::DynamicSubtreeNodeList::itemForwardsFromCurrent):
(WebCore::DynamicSubtreeNodeList::itemBackwardsFromCurrent):
(WebCore::DynamicSubtreeNodeList::item):
(WebCore::DynamicSubtreeNodeList::invalidateCache):
(WebCore::DynamicSubtreeNodeList::Caches::create):
(WebCore::DynamicSubtreeNodeList::Caches::reset):

  • dom/DynamicNodeList.h:
  • dom/Element.cpp:

(WebCore::Element::updateAfterAttributeChanged):

  • dom/Node.cpp:

(WebCore::Node::setTreeScopeRecursively):
(WebCore::Node::invalidateNodeListsCacheAfterAttributeChanged):
(WebCore::Node::invalidateNodeListsCacheAfterChildrenChanged):
(WebCore::Node::notifyLocalNodeListsLabelChanged):
(WebCore::Node::itemTypeAttributeChanged):
(WebCore::NodeListsNodeData::invalidateCaches):
(WebCore::NodeListsNodeData::invalidateCachesThatDependOnAttributes):

  • dom/Node.h:
  • dom/NodeRareData.h:
  • html/HTMLElement.cpp:

(WebCore::HTMLElement::parseMappedAttribute):

  • html/HTMLLabelElement.cpp:

(WebCore::HTMLLabelElement::parseMappedAttribute):

  • html/HTMLLabelElement.h:
8:19 PM Changeset in webkit [104115] by mitz@apple.com
  • 8 edits in trunk

Source/WebKit2: Allow WebProcess to launch a client process and become its first WebProcess.

This is the WebKit2 part of fixing <​http://webkit.org/b/75444> Debugging WebProcess requires running a UI process first and waiting to attach

Reviewed by Anders Carlsson.

  • PluginProcess/mac/PluginProcessMainMac.mm:

(WebKit::PluginProcessMain): Added a newline to stderr output.

  • UIProcess/Launcher/mac/EnvironmentVariables.cpp:

(WebKit::EnvironmentVariables::preexistingProcessServiceNameKey): Added. Returns the name
of the environment variable that optionally tells a UI processs to look for a preexisting
web process instead of launching a new one.
(WebKit::EnvironmentVariables::preexistingProcessTypeKey): Added. Returns the name of the
environment variable that tells a UI process the type of the preexisting web process
indicated by the other variable.

  • UIProcess/Launcher/mac/EnvironmentVariables.h:
  • UIProcess/Launcher/mac/ProcessLauncherMac.mm:

(WebKit::ProcessLauncher::launchProcess): Changed to look for a preexisting web process if
the aforementioned environment variables are set and the preexisting process has not been
used yet.

  • WebProcess/mac/WebProcessMainMac.mm:

(WebKit::WebProcessMain): Changed to look for the -client-executable command-line option,
and if present, launch the specified executable, setting variables in its environment that
tell it to use this preexisting web process, then wait for it to send a send right to its
listening port.

Tools: Add a --target-web-process option to the debug-* scripts. When specified, the scripts will
start WebProcess under gdb and WebProcess will then run the client executable.

This is the Tools part of fixing <​http://webkit.org/b/75444> Debugging WebProcess requires running a UI process first and waiting to attach

Reviewed by Anders Carlsson.

  • Scripts/webkitdirs.pm:

(shouldTargetWebProcess): Added.
(determineShouldTargetWebProcess): Added. Checks for --target-web-process.
(execMacWebKitAppForDebugging): Changed to target gdb at WebProcess and pass the path to the
app using the -client-executable option if targeting the web process.

7:20 PM Changeset in webkit [104114] by levin@chromium.org
  • 2 edits in trunk/Source/WebKit/chromium

Attempt Chromium build fix.

  • src/WorkerFileSystemCallbacksBridge.h:
7:08 PM Changeset in webkit [104113] by levin@chromium.org
  • 2 edits
    1 move
    2 adds
    1 delete in trunk/LayoutTests

[chromium] FileSystem bridge needs threadsafe fixes and other clean-up.
​https://bugs.webkit.org/show_bug.cgi?id=74666

Reviewed by Dmitry Titov.

Tests for all of the fixes done in bugs related to 74666.

  • http/tests/workers/resources/open-database-sync.js: Removed.
  • http/tests/workers/resources/sync-operations.js: Added.

(removeAllInDirectorySync):
(onmessage):

  • http/tests/workers/terminate-during-sync-operation-expected.txt: Added.
  • http/tests/workers/terminate-during-sync-operation.html: Renamed from LayoutTests/http/tests/workers/interrupt-database-sync-open-crash.html-skipped.
  • platform/chromium/test_expectations.txt:
7:08 PM Changeset in webkit [104112] by levin@chromium.org
  • 3 edits in trunk/Source/WebKit/chromium

[chromium] Remove AllowCrossThreadAccess in WorkerFileSystemCallbacksBridge.
​https://bugs.webkit.org/show_bug.cgi?id=75579

Reviewed by Dmitry Titov.

This code doesn't have lifetime issues, but this change is
about getting rid of AllowCrossThreadAccess and getting rid of a leakRef.
This allows the code to follow a more common pattern and be more easily
verified as being correct.

  • src/WorkerFileSystemCallbacksBridge.cpp:

(WebKit::MainThreadFileSystemCallbacks::createLeakedPtr):
(WebKit::MainThreadFileSystemCallbacks::MainThreadFileSystemCallbacks):
(WebKit::WorkerFileSystemCallbacksBridge::postOpenFileSystemToMainThread):
(WebKit::WorkerFileSystemCallbacksBridge::postMoveToMainThread):
(WebKit::WorkerFileSystemCallbacksBridge::postCopyToMainThread):
(WebKit::WorkerFileSystemCallbacksBridge::postRemoveToMainThread):
(WebKit::WorkerFileSystemCallbacksBridge::postRemoveRecursivelyToMainThread):
(WebKit::WorkerFileSystemCallbacksBridge::postReadMetadataToMainThread):
(WebKit::WorkerFileSystemCallbacksBridge::postCreateFileToMainThread):
(WebKit::WorkerFileSystemCallbacksBridge::postCreateDirectoryToMainThread):
(WebKit::WorkerFileSystemCallbacksBridge::postFileExistsToMainThread):
(WebKit::WorkerFileSystemCallbacksBridge::postDirectoryExistsToMainThread):
(WebKit::WorkerFileSystemCallbacksBridge::postReadDirectoryToMainThread):
(WebKit::WorkerFileSystemCallbacksBridge::openFileSystemOnMainThread):
(WebKit::WorkerFileSystemCallbacksBridge::moveOnMainThread):
(WebKit::WorkerFileSystemCallbacksBridge::copyOnMainThread):
(WebKit::WorkerFileSystemCallbacksBridge::removeOnMainThread):
(WebKit::WorkerFileSystemCallbacksBridge::removeRecursivelyOnMainThread):
(WebKit::WorkerFileSystemCallbacksBridge::readMetadataOnMainThread):
(WebKit::WorkerFileSystemCallbacksBridge::createFileOnMainThread):
(WebKit::WorkerFileSystemCallbacksBridge::createDirectoryOnMainThread):
(WebKit::WorkerFileSystemCallbacksBridge::fileExistsOnMainThread):
(WebKit::WorkerFileSystemCallbacksBridge::directoryExistsOnMainThread):
(WebKit::WorkerFileSystemCallbacksBridge::readDirectoryOnMainThread):
(WebKit::WorkerFileSystemCallbacksBridge::didFailOnMainThread):
(WebKit::WorkerFileSystemCallbacksBridge::didOpenFileSystemOnMainThread):
(WebKit::WorkerFileSystemCallbacksBridge::didSucceedOnMainThread):
(WebKit::WorkerFileSystemCallbacksBridge::didReadMetadataOnMainThread):
(WebKit::WorkerFileSystemCallbacksBridge::didReadDirectoryOnMainThread):
(WebKit::WorkerFileSystemCallbacksBridge::didFailOnWorkerThread):
(WebKit::WorkerFileSystemCallbacksBridge::didOpenFileSystemOnWorkerThread):
(WebKit::WorkerFileSystemCallbacksBridge::didSucceedOnWorkerThread):
(WebKit::WorkerFileSystemCallbacksBridge::didReadMetadataOnWorkerThread):
(WebKit::WorkerFileSystemCallbacksBridge::didReadDirectoryOnWorkerThread):
(WebKit::WorkerFileSystemCallbacksBridge::runTaskOnMainThread):
(WebKit::WorkerFileSystemCallbacksBridge::mayPostTaskToWorker):

  • src/WorkerFileSystemCallbacksBridge.h:
7:08 PM Changeset in webkit [104111] by levin@chromium.org
  • 3 edits in trunk/Source/WebKit/chromium

[chromium] Make it safe to delete WorkerFileSystemContextObserver on any thread.
​https://bugs.webkit.org/show_bug.cgi?id=75573

Reviewed by Dmitry Titov.

  • src/WorkerFileSystemCallbacksBridge.cpp:

(WebKit::WorkerFileSystemContextObserver): Move the WorkerContextObserver
out of the WorkerFileSystemCallbacksBridge since an observer should be
destroyed on the WorkerContext thread. (Actually, it could be destroyed on either
thread if you are careful to make a certain method call on it while on the WorkerContext
thread but trying that is a more fragile pattern.)
(WebKit::WorkerFileSystemCallbacksBridge::WorkerFileSystemCallbacksBridge):
(WebKit::WorkerFileSystemCallbacksBridge::stop): Factor out the clean up and make it
clear what the mutex is guarding.
(WebKit::WorkerFileSystemCallbacksBridge::cleanUpAfterCallback): Delete
the observer. Due to where this is called from, it is always called on the WorkerContext thread.
(WebKit::WorkerFileSystemCallbacksBridge::runTaskOnWorkerThread): Replace some code with
the cleanUpAfterCallback call.
(WebKit::WorkerFileSystemCallbacksBridge::mayPostTaskToWorker):

  • src/WorkerFileSystemCallbacksBridge.h: In addition to some comment clean ups and code factoring,

I made the desctructor private since no one should call it directly.

5:50 PM Changeset in webkit [104110] by eae@chromium.org
  • 1 edit
    2 deletes in branches/subpixellayout/Source/WebCore

Manual merge of GNUmakefile.list.am and a couple of files deleted on trunk.

5:50 PM Changeset in webkit [104109] by oliver@apple.com
  • 16 edits in branches/safari-534.54-branch/Source/JavaScriptCore

Rollout r102743

5:36 PM Changeset in webkit [104108] by Lucas Forschler
  • 4 edits
    2 copies in branches/safari-534.54-branch

Merged r97353.

5:22 PM Changeset in webkit [104107] by fpizlo@apple.com
  • 2 edits in trunk/Source/JavaScriptCore

Unreviewed, accidentally turned off the JIT in previous commit. Turning
it back on.

  • wtf/Platform.h:
5:19 PM Changeset in webkit [104106] by jamesr@google.com
  • 12 edits in trunk/Source

[chromium] Remove chromium compositor support for unused zoomAnimatorTransform
​https://bugs.webkit.org/show_bug.cgi?id=75585

Reviewed by Kenneth Russell.

Source/WebCore:

Removes compositor support for zoomAnimatorTransform property. This code is unused and never was used.
Zoom animations are handled by adjusting the page scale factor and scroll offsets.

  • platform/graphics/chromium/LayerChromium.h:
  • platform/graphics/chromium/cc/CCLayerImpl.cpp:
  • platform/graphics/chromium/cc/CCLayerImpl.h:
  • platform/graphics/chromium/cc/CCLayerTreeHost.cpp:

(WebCore::CCLayerTreeHost::finishCommitOnImplThread):

  • platform/graphics/chromium/cc/CCLayerTreeHost.h:
  • platform/graphics/chromium/cc/CCLayerTreeHostCommon.cpp:

(WebCore::calculateDrawTransformsAndVisibilityInternal):

  • platform/graphics/chromium/cc/CCLayerTreeHostImpl.cpp:
  • platform/graphics/chromium/cc/CCLayerTreeHostImpl.h:

Source/WebKit/chromium:

  • src/WebViewImpl.cpp:

(WebKit::WebViewImpl::setZoomLevel):
(WebKit::WebViewImpl::setRootLayerNeedsDisplay):

  • tests/CCLayerImplTest.cpp:

(WebCore::TEST):

5:18 PM Changeset in webkit [104105] by fpizlo@apple.com
  • 3 edits in trunk/Source/JavaScriptCore

Changed "return" to "break" in some macrology I introduced in
​http://trac.webkit.org/changeset/104086. This is a benign change, as
"return" was technically correct for all uses of the macro.

Reviewed by Oliver Hunt.

  • dfg/DFGGraph.cpp:
  • wtf/Platform.h:
5:17 PM Changeset in webkit [104104] by Lucas Forschler
  • 4 edits in branches/safari-534.54-branch/Source/WebCore

Merged r97270.

5:13 PM Changeset in webkit [104103] by Lucas Forschler
  • 1 edit in branches/safari-534.54-branch/LayoutTests/ChangeLog

Merged r97181.

5:12 PM Changeset in webkit [104102] by kbr@google.com
  • 10 edits in trunk

Fix semantics of WEBKIT_WEBGL_lose_context
​https://bugs.webkit.org/show_bug.cgi?id=75053

Reviewed by James Robinson.

Source/WebCore:

Updated WEBGL_lose_context extension to match WebGL spec's
behavior:

  • loseContext() causes the context to be lost immediately, but the webglcontextlost event isn't dispatched until JavaScript returns.
  • restoreContext() may only be called once dispatch of the webglcontextlost event has completed, *and* if that event handler prevented the default behavior of the event.
  • restoreContext() performs its work asynchronously. The context is not restored until just before the webglcontextrestored event is dispatched.
  • html/canvas/WebGLRenderingContext.cpp:

(WebCore::WebGLRenderingContext::WebGLRenderingContext):
(WebCore::WebGLRenderingContext::forceLostContext):
(WebCore::WebGLRenderingContext::forceRestoreContext):
(WebCore::WebGLRenderingContext::dispatchContextLostEvent):
(WebCore::WebGLRenderingContext::maybeRestoreContext):

  • html/canvas/WebGLRenderingContext.h:

LayoutTests:

Updated context-lost and context-lost-restored tests to latest
versions in Khronos repository. Updated WebGLContextEvent.html to
match new semantics.

  • fast/canvas/webgl/WebGLContextEvent-expected.txt:
  • fast/canvas/webgl/WebGLContextEvent.html:
  • fast/canvas/webgl/context-lost-expected.txt:
  • fast/canvas/webgl/context-lost-restored-expected.txt:
  • fast/canvas/webgl/context-lost-restored.html:
  • fast/canvas/webgl/context-lost.html:
5:11 PM Changeset in webkit [104101] by Lucas Forschler
  • 3 edits
    3 copies in branches/safari-534.54-branch

Merged r97180.

5:04 PM Changeset in webkit [104100] by oliver@apple.com
  • 11 edits
    2 adds in branches/safari-534.54-branch

Merge r94457

5:01 PM Changeset in webkit [104099] by Lucas Forschler
  • 3 edits
    3 copies in branches/safari-534.54-branch

Merged r97124.

5:00 PM Changeset in webkit [104098] by eae@chromium.org
  • 1985 edits
    835 copies
    200 deletes in branches/subpixellayout

Merging trunk changes up until 104084 into subpixel branch.

4:54 PM Changeset in webkit [104097] by Lucas Forschler
  • 4 edits
    3 copies in branches/safari-534.54-branch

Merged r97114.

4:40 PM Changeset in webkit [104096] by Lucas Forschler
  • 6 edits
    12 copies in branches/safari-534.54-branch

Merged r97075.

4:39 PM EFLWebKitCodingStyle edited by gyuyoung.kim@samsung.com
Update type casting. (diff)
4:35 PM Changeset in webkit [104095] by Lucas Forschler
  • 5 edits
    3 copies in branches/safari-534.54-branch

Merged r97074.

4:31 PM Changeset in webkit [104094] by msaboff@apple.com
  • 2 edits in trunk/Source/JavaScriptCore

StructureStubInfo not reset when corresponding MethodCallLinkInfo is reset
​https://bugs.webkit.org/show_bug.cgi?id=75583

Reviewed by Filip Pizlo.

  • bytecode/CodeBlock.cpp:

(JSC::CodeBlock::finalizeUnconditionally): Find the corresponding
StructureStubInfo and reset the appropriate JIT and
the StructureStubInfo itself when reseting a MethodCallLinkInfo.

4:29 PM Changeset in webkit [104093] by Lucas Forschler
  • 6 edits
    2 copies in branches/safari-534.54-branch

Merged r96999.

4:19 PM Changeset in webkit [104092] by Lucas Forschler
  • 3 edits
    2 copies in branches/safari-534.54-branch

Merged r96984.

4:15 PM Changeset in webkit [104091] by ddkilzer@apple.com
  • 9 edits in trunk/Source/ThirdParty

TestWebKitAPI has issues due to FastMalloc incompatibility
<​http://webkit.org/b/66521>
<rdar://problem/10607911>

Reviewed by David Levin.

Original patch by Dmitry Lomov <​dslomov@google.com>.

This patch ensures that gtest uses the FastMalloc new and delete
operators that are defined in JavaScriptCore.

  • gtest/README.WebKit: Update with project changes.
  • gtest/include/gtest/internal/gtest-port.h: Import necessary

headers to use FastMalloc in gtest.

  • gtest/msvc/gtest-md.vcproj: Add include directories to find

WTF headers.

  • gtest/xcode/Config/DebugProject.xcconfig: Define

WEBCORE_PRIVATE_HEADERS_DIR for Debug builds.

  • gtest/xcode/Config/General.xcconfig: Define

HEADER_SEARCH_PATHS in terms of WEBCORE_PRIVATE_HEADERS_DIR.

  • gtest/xcode/Config/ProductionProject.xcconfig: Define

WEBCORE_PRIVATE_HEADERS_DIR for Production builds. This
overrides the value in ReleaseProject.xcconfig.

  • gtest/xcode/Config/ReleaseProject.xcconfig: Define

WEBCORE_PRIVATE_HEADERS_DIR for Release builds.

  • gtest/xcode/gtest.xcodeproj/project.pbxproj:
  • Remove local HEADER_SEARCH_PATHS definitions.
  • Link to JavaScriptCore.framework when building gtest.framework.
4:10 PM Changeset in webkit [104090] by msaboff@apple.com
  • 2 edits in trunk/Source/JavaScriptCore

Invalid ASSERT() in DFGRepatch.cpp near line 385
​https://bugs.webkit.org/show_bug.cgi?id=75584

Reviewed by Filip Pizlo.

  • dfg/DFGRepatch.cpp:

(JSC::DFG::tryBuildGetByIDProtoList): Fixed ASSERT to use ==.

3:55 PM Changeset in webkit [104089] by ojan@chromium.org
  • 2 edits in trunk/LayoutTests

Remove a line for a test that no longer fails on the bots.

  • platform/chromium/test_expectations.txt:
3:45 PM Changeset in webkit [104088] by Lucas Forschler
  • 3 edits
    2 copies in branches/safari-534.54-branch

Merged r96868.

3:33 PM Changeset in webkit [104087] by Lucas Forschler
  • 3 edits
    2 copies in branches/safari-534.54-branch

Merged r96834.

3:13 PM Changeset in webkit [104086] by fpizlo@apple.com
  • 6 edits
    3 adds in trunk

Incorrect use of DFG node reference counts when mutating the graph
​https://bugs.webkit.org/show_bug.cgi?id=75580
<rdar://problem/10644607>

Source/JavaScriptCore:

Reviewed by Oliver Hunt.

Made deref(node) follow the pattern of ref(node), which it should have
to begin with.

  • dfg/DFGGraph.cpp:

(JSC::DFG::Graph::refChildren):
(JSC::DFG::Graph::derefChildren):

  • dfg/DFGGraph.h:

(JSC::DFG::Graph::deref):
(JSC::DFG::Graph::clearAndDerefChild1):
(JSC::DFG::Graph::clearAndDerefChild2):
(JSC::DFG::Graph::clearAndDerefChild3):

  • dfg/DFGNode.h:

(JSC::DFG::Node::deref):

  • dfg/DFGPropagator.cpp:

(JSC::DFG::Propagator::fixupNode):

LayoutTests:

Reviewed by Oliver Hunt.

  • fast/js/dfg-array-length-dead-expected.txt: Added.
  • fast/js/dfg-array-length-dead.html: Added.
  • fast/js/script-tests/dfg-array-length-dead.js: Added.

(foo):

3:11 PM Changeset in webkit [104085] by eae@chromium.org
  • 18 edits in branches/subpixellayout/LayoutTests/platform/mac/editing/selection

Update selection expectations for mac to account for pixelSnapped later sizes.

3:08 PM Changeset in webkit [104084] by rniwa@webkit.org
  • 11 edits in trunk/Source/WebCore

Inserting nodes is slow due to Node::notifyNodeListsAttributeChanged (20%+)
​https://bugs.webkit.org/show_bug.cgi?id=73853

Reviewed by Antti Koivisto.

Lazily invalidate the node list caches instead of invaliding them at the time of modification. We use
the DOM tree version to detect whether caches need to be invalidated or not. We now invalidate caches more
frequently after this patch (in particular, invalidates caches that are stored on nodes not present in
the ancestry of the modified nodes); however, our study on major Web sites such as Gmail, Facebook, Twitter,
etc... indicate that about 1% of real-world usage benefits from keeping the caches alive across different
DOM tree versions.

In order to invalidate caches lazily, this patch adds replaces the type of m_caches in DynamicSubtreeNodeList
by DynamicSubtreeNodeList::SubtreeCaches which encapsulates member variables in DynamicNodeList::Caches and
invalidates values as needed. Also this change allows m_caches to be allocated as a part of
DynamicSubtreeNodeList instead of a separate ref-counted object.

  • dom/Attr.cpp:

(WebCore::Attr::setValue):
(WebCore::Attr::childrenChanged):

  • dom/DynamicNodeList.cpp:

(WebCore::DynamicSubtreeNodeList::DynamicSubtreeNodeList):
(WebCore::DynamicSubtreeNodeList::length):
(WebCore::DynamicSubtreeNodeList::itemForwardsFromCurrent):
(WebCore::DynamicSubtreeNodeList::itemBackwardsFromCurrent):
(WebCore::DynamicSubtreeNodeList::item):
(WebCore::DynamicSubtreeNodeList::invalidateCache):
(WebCore::DynamicNodeList::Caches::create):
(WebCore::DynamicNodeList::Caches::reset):

  • dom/DynamicNodeList.h:

(WebCore::DynamicSubtreeNodeList::SubtreeCaches::SubtreeCaches): Added.
(WebCore::DynamicSubtreeNodeList::SubtreeCaches::isLengthCacheValid): Added.
(WebCore::DynamicSubtreeNodeList::SubtreeCaches::isItemCacheValid): Added.
(WebCore::DynamicSubtreeNodeList::SubtreeCaches::cachedLength): Added.
(WebCore::DynamicSubtreeNodeList::SubtreeCaches::cachedItem): Added.
(WebCore::DynamicSubtreeNodeList::SubtreeCaches::cachedItemOffset): Added.
(WebCore::DynamicSubtreeNodeList::SubtreeCaches::setLengthCache): Added.
(WebCore::DynamicSubtreeNodeList::SubtreeCaches::setItemCache): Added.
(WebCore::DynamicSubtreeNodeList::SubtreeCaches::reset): Added.
(WebCore::DynamicSubtreeNodeList::SubtreeCaches::domVersionIsConsistent): Added.

  • dom/Element.cpp:

(WebCore::Element::updateAfterAttributeChanged):

  • dom/Node.cpp:

(WebCore::Node::setTreeScopeRecursively): Clear caches when a node moves from one document to another.
(WebCore::Node::invalidateNodeListsCacheAfterAttributeChanged): Only clears child node list of Attr.
(WebCore::Node::invalidateNodeListsCacheAfterChildrenChanged): Only clears child node list.
(WebCore::NodeListsNodeData::invalidateCaches): Merged with invalidateCachesThatDependOnAttributes.

  • dom/Node.h:
  • dom/NodeRareData.h:
  • html/HTMLElement.cpp:

(WebCore::HTMLElement::parseMappedAttribute):

  • html/HTMLLabelElement.cpp:
  • html/HTMLLabelElement.h:
3:06 PM Changeset in webkit [104083] by Lucas Forschler
  • 3 edits
    2 copies in branches/safari-534.54-branch

Merged r96826.

2:59 PM Changeset in webkit [104082] by Lucas Forschler
  • 3 edits
    2 copies in branches/safari-534.54-branch

Merged r96632.

2:56 PM Changeset in webkit [104081] by rniwa@webkit.org
  • 2 edits in trunk/Source/WebCore

Revert unintentional changes to Element.cpp in r104068.

  • dom/Element.cpp:

(WebCore::Element::recalcStyleIfNeededAfterAttributeChanged):

2:55 PM Changeset in webkit [104080] by Lucas Forschler
  • 4 edits
    3 copies in branches/safari-534.54-branch

Merged r96427.

2:47 PM Changeset in webkit [104079] by Lucas Forschler
  • 5 edits
    3 copies in branches/safari-534.54-branch

Merged r96260.

2:40 PM Changeset in webkit [104078] by Lucas Forschler
  • 2 edits in branches/safari-534.54-branch/Source/WebCore

Merged r95985.

2:37 PM Changeset in webkit [104077] by Lucas Forschler
  • 3 edits
    2 copies in branches/safari-534.54-branch

Merged r95959.

2:34 PM Changeset in webkit [104076] by Lucas Forschler
  • 3 edits
    2 copies in branches/safari-534.54-branch

Merged r95926.

2:30 PM Changeset in webkit [104075] by Lucas Forschler
  • 4 edits
    2 copies in branches/safari-534.54-branch

Merged r95924.

2:26 PM Changeset in webkit [104074] by jamesr@google.com
  • 4 edits in trunk/Source

[chromium] Compile fix, rename libplatform.a to libwebkit_platform.a to avoid collision with nacl

Source/Platform:

  • Platform.gyp/Platform.gyp:

Source/WebKit/chromium:

  • WebKit.gyp:
2:24 PM Changeset in webkit [104073] by enne@google.com
  • 2 edits
    1 delete in trunk/Source/WebKit/chromium

Unreviewed, rolling out r104052.
​http://trac.webkit.org/changeset/104052
​https://bugs.webkit.org/show_bug.cgi?id=75574

Breaks win builder (Requested by enne on #webkit).

Patch by Sheriff Bot <​webkit.review.bot@gmail.com> on 2012-01-04

  • WebKit.gypi:
  • tests/CCTiledLayerImplTest.cpp: Removed.
2:22 PM Changeset in webkit [104072] by eae@chromium.org
  • 53 edits in branches/subpixellayout

Add pixelSnappedIntSize function and change layer size calculation to use it instead of expandedIntSize.

2:16 PM Changeset in webkit [104071] by Lucas Forschler
  • 4 edits
    3 copies in branches/safari-534.54-branch

Merged r95857.

2:13 PM Changeset in webkit [104070] by Lucas Forschler
  • 8 edits in branches/safari-534.54-branch/Source/WebKit2

Merged r95747.

2:09 PM Changeset in webkit [104069] by Lucas Forschler
  • 6 edits
    2 copies in branches/safari-534.54-branch

Merged r95728.

2:05 PM Changeset in webkit [104068] by rniwa@webkit.org
  • 4 edits
    2 adds in trunk

REGRESSION (r92823): Background color not preserved when copying and pasting a table row
​https://bugs.webkit.org/show_bug.cgi?id=75330

Reviewed by Tony Chang.

Source/WebCore:

The bug was caused by the background color of the wrapping style overriding the background color
in a matched rule of a highest element to be serialized. Fixed the bug by removing the conflicting
background color prior to the merge.

Tests: editing/pasteboard/copy-element-with-conflicting-background-color-from-rule-expected.html

editing/pasteboard/copy-element-with-conflicting-background-color-from-rule.html

  • editing/EditingStyle.cpp:

(WebCore::editingStyleFromComputedStyle):
(WebCore::EditingStyle::removeStyleAddedByNode):
(WebCore::EditingStyle::removeStyleConflictingWithStyleOfNode):

LayoutTests:

Add a regression test.

  • editing/pasteboard/copy-element-with-conflicting-background-color-from-rule-expected.html: Added.
  • editing/pasteboard/copy-element-with-conflicting-background-color-from-rule.html: Added.
2:05 PM Changeset in webkit [104067] by ostapenko.viatcheslav@nokia.com
  • 2 edits in trunk/Tools

Adding myself to committers list.

Unreviewed.

2:02 PM Changeset in webkit [104066] by andersca@apple.com
  • 2 edits in trunk/Source/WebKit/mac

Ambiguous if-then-else in Source/WebKit/mac/Misc/WebNSPasteboardExtras.mm
​https://bugs.webkit.org/show_bug.cgi?id=75447

Reviewed by Brian Weinstein.

Resolve the ambiguity.

  • Misc/WebNSPasteboardExtras.mm:

(-[NSPasteboard _web_writeImage:element:URL:title:archive:types:source:]):

2:01 PM Changeset in webkit [104065] by Lucas Forschler
  • 1 edit in branches/safari-534.54-branch/Source/WebCore/svg/SVGDocumentExtensions.h

Remove executable bit from SVGDocumentExtensions.h

1:40 PM Changeset in webkit [104064] by dslomov@google.com
  • 2 edits in trunk/Source/WebKit/chromium

Unreviewed:[Chromium]Disable WebPageNewSerializeTest.CSSResources and WebPageNewSerializeTest.TestMHTMLEncoding.
​https://bugs.webkit.org/show_bug.cgi?id=75567

  • tests/WebPageNewSerializerTest.cpp:

(WebKit::TEST_F):

12:43 PM Changeset in webkit [104063] by Lucas Forschler
  • 16 edits
    3 copies in branches/safari-534.54-branch

Merged r95671.

12:10 PM Changeset in webkit [104062] by Lucas Forschler
  • 2 edits in branches/safari-534.54-branch/Source/WebCore

Merged r95497.

12:06 PM Changeset in webkit [104061] by Lucas Forschler
  • 5 edits
    2 copies in branches/safari-534.54-branch

Merged r95462.

12:03 PM Changeset in webkit [104060] by Antti Koivisto
  • 7 edits in trunk/Source/WebCore

Analyze stylesheet scope to minimize style recalcs
​https://bugs.webkit.org/show_bug.cgi?id=75508

Reviewed by Dave Hyatt.

It is a relatively common pattern to use inline stylesheets in document body where all rules are scoped using descendant selector

<style>
#foo {...}
#foo div {...}
#foo .bar {...}
</style>

When this pattern is used it is also common that the rules only apply to elements that come after the style element.

When the set of active stylesheets changes we invalidate and recompute the entire document style. This is very expensive. We can
detect the case above and avoid the style recalc.

On engadget.com, this patch cuts the time spent in style recalcs to roughly half. There are further savings from reduced
relayouts. In total the engine CPU time used over the page load is reduced by ~10%.

  • css/CSSStyleSelector.cpp:

(WebCore::CSSStyleSelector::CSSStyleSelector):
(WebCore::CSSStyleSelector::collectFeatures):

Refactor feature collection from constructor to a separate function.


(WebCore::CSSStyleSelector::appendAuthorStylesheets):

New function for non-destructively updating the style selector.


(WebCore::CSSStyleSelector::Features::clear):

Clear the features for another collection.


(WebCore::CSSStyleSelector::determineStylesheetSelectorScopes):

Find if all rules on a stylesheetare scoped to some set of ids and classes.


(WebCore::CSSStyleSelector::styleForDocument):

Add optional font selector argument. We updated the correct base style font on style selector construction but that is no longer sufficient
as font selector may be updated without reconstructing the style selector.


(WebCore::RuleSet::addRulesFromSheet):

Invalidate the matched declaration cache in case of new font-face rules.


  • css/CSSStyleSelector.h:
  • css/SelectorChecker.cpp:

(WebCore::SelectorChecker::determineSelectorScopes):

Find if all rules on a selector list are scoped to some set of ids and classes.


  • css/SelectorChecker.h:
  • dom/Document.cpp:

(WebCore::Document::Document):
(WebCore::Document::recalcStyle):

Pass the font selector, if exists, to styleForDocument so we always have the base font up to date.


(WebCore::Document::combineCSSFeatureFlags):
(WebCore::Document::resetCSSFeatureFlags):
(WebCore::Document::createStyleSelector):

Refactor css feature flag resetting to functions.


(WebCore::Document::removePendingSheet):

Use new PendingStylesheetCompleted flag when new stylesheets arrive.

(WebCore::Document::styleSelectorChanged):

Skip style recalc if it is not needed.

(WebCore::Document::collectActiveStylesheets):

Refactor collecting stylesheets to a separate function.


(WebCore::Document::testAddedStylesheetRequiresStyleRecalc):

Determine the scopes and use hasElementWithId/getElementsByClassName to figure out if any scoped elements currently exist in the tree.


(WebCore::Document::analyzeStylesheetChange):

Figure out if we can update the style selector incrementally and if we can skip the style recalc.


(WebCore::Document::updateActiveStylesheets):

Renamed from recalcStyleSelector.
Invoke the new analysis functions.


  • dom/Document.h:
12:02 PM Changeset in webkit [104059] by Lucas Forschler
  • 8 edits
    15 copies in branches/safari-534.54-branch

Merged r95461.

11:58 AM Changeset in webkit [104058] by Lucas Forschler
  • 6 edits
    2 copies in branches/safari-534.54-branch

Merged r95440.

11:51 AM Changeset in webkit [104057] by Lucas Forschler
  • 4 edits in branches/safari-534.54-branch

Merged r95306.

11:47 AM Changeset in webkit [104056] by Lucas Forschler
  • 4 edits
    2 copies in branches/safari-534.54-branch

Merged r95301.

11:44 AM Changeset in webkit [104055] by Lucas Forschler
  • 2 edits in branches/safari-534.54-branch/LayoutTests

Merged r95126.

11:42 AM Changeset in webkit [104054] by Lucas Forschler
  • 3 edits
    2 copies in branches/safari-534.54-branch

Merged r95074.

11:38 AM Changeset in webkit [104053] by Lucas Forschler
  • 3 edits
    2 copies in branches/safari-534.54-branch

Merged r95071.

11:36 AM Changeset in webkit [104052] by enne@google.com
  • 2 edits
    1 add in trunk/Source/WebKit/chromium

[chromium] Create unit tests for CCTiledLayerImpl
​https://bugs.webkit.org/show_bug.cgi?id=75557

Reviewed by James Robinson.

Add some quad generation tests for tiled layers.

  • WebKit.gypi:
  • tests/CCTiledLayerImplTest.cpp: Added.

(WebCore::createLayer):
(WebCore::TEST):
(WebCore::completelyContains):
(WebCore::getQuads):
(WebCore::verifyQuadsExactlyCoverRect):
(WebCore::coverageVisibleRectOnTileBoundaries):
(WebCore::coverageVisibleRectIntersectsTiles):
(WebCore::coverageVisibleRectIntersectsBounds):

11:35 AM Changeset in webkit [104051] by Lucas Forschler
  • 3 edits
    2 copies in branches/safari-534.54-branch

Merged r95059.

11:25 AM Changeset in webkit [104050] by ddkilzer@apple.com
  • 2 edits in trunk/Tools

Revert fix for Bug 75064: TestWebKitAPI: initialize the main thread before running tests

This reverts commit r103546.

  • TestWebKitAPI/TestsController.cpp:

(TestWebKitAPI::TestsController::TestsController):

11:24 AM Changeset in webkit [104049] by ap@apple.com
  • 2 edits in trunk/Source/WebKit2

First sentence is missing or clipped when printing a inline PDF
​https://bugs.webkit.org/show_bug.cgi?id=75514
<rdar://problem/10640680>

Reviewed by Dan Bernstein.

  • UIProcess/WebPageProxy.cpp: (WebKit::WebPageProxy::headerHeight): (WebKit::WebPageProxy::footerHeight): (WebKit::WebPageProxy::drawHeader): (WebKit::WebPageProxy::drawFooter): Do not ask the client about headers and footers when printing a PDF.
11:08 AM Changeset in webkit [104048] by jamesr@google.com
  • 11 edits
    4 copies
    2 moves
    6 adds in trunk

[chromium] Move WebMimeRegistry and dependencies to Source/Platform
​https://bugs.webkit.org/show_bug.cgi?id=74583

Reviewed by Darin Fisher.

.:

Update .gitignore for Source/Platform/Platform.gyp/ generated project files.

  • .gitignore:

Source/Platform:

This creates a skeleton directory structure for the chromium WebKit platform API and moves WebMimeRegistry.h
along with its dependencies (WebString and WebCString) to their final location.

  • Platform.gyp/Platform.gyp: Added.
  • Platform.gypi: Added.
  • chromium/platform/WebCString.h: Copied from Source/WebKit/chromium/public/platform/WebCString.h.

(WebKit::WebCString::~WebCString):
(WebKit::WebCString::WebCString):
(WebKit::WebCString::operator=):
(WebKit::WebCString::isEmpty):
(WebKit::WebCString::isNull):
(WebKit::WebCString::operator std::string):
(WebKit::WebCString::fromUTF16):
(WebKit::operator<):

  • chromium/platform/WebCommon.h: Copied from Source/WebKit/chromium/public/platform/WebCommon.h.
  • chromium/platform/WebMimeRegistry.h: Copied from Source/WebKit/chromium/public/platform/WebMimeRegistry.h.

(WebKit::WebMimeRegistry::~WebMimeRegistry):

  • chromium/platform/WebString.h: Copied from Source/WebKit/chromium/public/platform/WebString.h.

(WebKit::WebString::~WebString):
(WebKit::WebString::WebString):
(WebKit::WebString::operator=):
(WebKit::WebString::isEmpty):
(WebKit::WebString::isNull):
(WebKit::WebString::operator string16):
(WebKit::WebString::operator NullableString16):
(WebKit::WebString::fromUTF8):
(WebKit::operator==):
(WebKit::operator!=):

  • chromium/src/WebCString.cpp: Renamed from Source/WebKit/chromium/src/WebCString.cpp.

(WebKit::WebCString::compare):
(WebKit::WebCString::reset):
(WebKit::WebCString::assign):
(WebKit::WebCString::length):
(WebKit::WebCString::data):
(WebKit::WebCString::utf16):
(WebKit::WebCString::fromUTF16):
(WebKit::WebCString::WebCString):
(WebKit::WebCString::operator=):
(WebKit::WebCString::operator WTF::CString):

  • chromium/src/WebString.cpp: Renamed from Source/WebKit/chromium/src/WebString.cpp.

(WebKit::WebString::reset):
(WebKit::WebString::assign):
(WebKit::WebString::length):
(WebKit::WebString::data):
(WebKit::WebString::utf8):
(WebKit::WebString::fromUTF8):
(WebKit::WebString::equals):
(WebKit::WebString::WebString):
(WebKit::WebString::operator=):
(WebKit::WebString::operator WTF::String):
(WebKit::WebString::operator WTF::AtomicString):

Source/WebKit/chromium:

public/platform/WebMimeRegistry.h is a temporary forwarding header to ease the transition. Once downstream
callers are updated to the new location this header can be removed.

WebString, WebCString, and WebCommon.h will remain in place as forwarding headers for users of the WebKit client
API.

  • WebKit.gyp:
  • public/platform/WebCString.h:
  • public/platform/WebCommon.h:
  • public/platform/WebMimeRegistry.h:
  • public/platform/WebString.h:

Tools:

Teaches the WEBKIT_EXPORT check about the chromium WebKit platform API header location.

  • Scripts/webkitpy/style/checkers/cpp.py:

(check_function_definition):

10:21 AM Changeset in webkit [104047] by igor.oliveira@openbossa.org
  • 2 edits in trunk/Source/WebCore

Share fractional time code between AnimationBase and KeyframeAnimation
​https://bugs.webkit.org/show_bug.cgi?id=75549

Remove left over from changeset 104045.

Unreviewed trivial fix.

  • page/animation/AnimationBase.cpp:

(WebCore::AnimationBase::fractionalTime):

9:51 AM Changeset in webkit [104046] by hclam@chromium.org
  • 1 edit
    1 add in trunk/LayoutTests

Not reviewed. Update Chromium port test expectations.

  • platform/chromium-linux/svg/custom/glyph-selection-lang-attribute-expected.txt: Added.
9:50 AM Changeset in webkit [104045] by igor.oliveira@openbossa.org
  • 4 edits in trunk/Source/WebCore

Share fractional time code between AnimationBase and KeyframeAnimation
​https://bugs.webkit.org/show_bug.cgi?id=75549

Share fractional time calculation between AnimationBase and KeyframeAnimation.

Reviewed by Simon Fraser.

  • page/animation/AnimationBase.cpp:

(WebCore::AnimationBase::fractionalTime):
(WebCore::AnimationBase::progress):

  • page/animation/AnimationBase.h:
  • page/animation/KeyframeAnimation.cpp:

(WebCore::KeyframeAnimation::fetchIntervalEndpointsForProperty):

9:17 AM Changeset in webkit [104044] by vangelis@chromium.org
  • 6 edits in trunk/Source

[chromium] Bypass the shadow texture copy for accelerated
canvas when running the compositor in single threaded mode.
​https://bugs.webkit.org/show_bug.cgi?id=75146

Source/WebCore:

The texture copy fails on Windows as glCopyTexImage2D() doesn't
support BGRA source textures.
This change also modifies Canvas2DLayerChromium::updateCompositorResources
to call glCopyTexSubImage2D() instead of glCopyTexImage2D() so that
the copy can work with texture allocated via the glTexStorage2D
extension.

Reviewed by James Robinson.

Tests: Canvas2DLayerChromiumTest.cpp

  • platform/graphics/chromium/Canvas2DLayerChromium.cpp:

(WebCore::Canvas2DLayerChromium::Canvas2DLayerChromium):
(WebCore::Canvas2DLayerChromium::~Canvas2DLayerChromium):
(WebCore::Canvas2DLayerChromium::paintContentsIfDirty):
(WebCore::Canvas2DLayerChromium::setTextureManager):
(WebCore::Canvas2DLayerChromium::updateCompositorResources):
(WebCore::Canvas2DLayerChromium::pushPropertiesTo):
(WebCore::Canvas2DLayerChromium::unreserveContentsTexture):
(WebCore::Canvas2DLayerChromium::cleanupResources):

  • platform/graphics/chromium/Canvas2DLayerChromium.h:

Source/WebKit/chromium:

Modified Canvas2DLayerChromium unit test to test canvas with
both threaded and non-threaded compositing.

Reviewed by James Robinson.

  • tests/CCSchedulerTestCommon.h:

(WebKitTests::FakeCCThread::threadID):

  • tests/Canvas2DLayerChromiumTest.cpp:

(WebCore::Canvas2DLayerChromiumTest::setTextureManager):
(WebCore::Canvas2DLayerChromiumTest::fullLifecycleTest):
(WebCore::TEST_F):

9:08 AM Changeset in webkit [104043] by hclam@chromium.org
  • 10 edits
    4 adds
    1 delete in trunk/LayoutTests

Not reviewed. Update Chromium port test expectations.

  • platform/chromium-linux/svg/W3C-SVG-1.1/masking-path-02-b-expected.png:
  • platform/chromium-linux/svg/custom/glyph-selection-lang-attribute-expected.png:
  • platform/chromium-mac-leopard/fast/repaint/shadow-multiple-vertical-expected.png:
  • platform/chromium-mac-snowleopard/fast/repaint/shadow-multiple-vertical-expected.png:
  • platform/chromium-mac-snowleopard/svg/W3C-SVG-1.1/masking-path-02-b-expected.png:
  • platform/chromium-mac-snowleopard/svg/custom/glyph-selection-lang-attribute-expected.png: Added.
  • platform/chromium-mac/svg/custom/glyph-selection-lang-attribute-expected.png: Removed.
  • platform/chromium-win-xp/svg/W3C-SVG-1.1/masking-path-02-b-expected.png: Added.
  • platform/chromium-win-xp/svg/custom/glyph-selection-lang-attribute-expected.png: Added.
  • platform/chromium-win/svg/W3C-SVG-1.1/masking-path-02-b-expected.png:
  • platform/chromium-win/svg/W3C-SVG-1.1/masking-path-02-b-expected.txt:
  • platform/chromium-win/svg/custom/glyph-selection-lang-attribute-expected.png:
  • platform/chromium-win/svg/custom/glyph-selection-lang-attribute-expected.txt: Added.
  • platform/chromium/test_expectations.txt:
8:12 AM Changeset in webkit [104042] by vestbo@webkit.org
  • 1 edit in trunk/Tools/qmake/mkspecs/features/wtf.prf

[Qt] Fix windows build. WTF depends on timeEndPeriod from winmm

8:12 AM Changeset in webkit [104041] by hclam@chromium.org
  • 2 edits
    1 delete in trunk/LayoutTests

Not reviewed. Update Chromium port test expectations.

  • platform/chromium-mac-snowleopard/compositing/visibility/visibility-image-layers-expected.png: Removed.
  • platform/chromium/test_expectations.txt:
7:55 AM Changeset in webkit [104040] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebKit2

[Qt] Fix test regressions after r104025
​https://bugs.webkit.org/show_bug.cgi?id=75545

Patch by Alexander Færøy <​alexander.faeroy@nokia.com> on 2012-01-04
Reviewed by Kenneth Rohde Christiansen.

  • WebProcess/WebPage/WebPage.cpp:

(WebKit::WebPage::editorState):

7:41 AM Changeset in webkit [104039] by vestbo@webkit.org
  • 2 edits in trunk/Tools

[Qt] Fix link order of static libraries and unbreak QtWin32

We were keeping the correct static link order between the intermediate
WebKit libraries, but not the internal dependencies for each of those
libraries. The load() will in some instances mess with LIBS, so we need
to ensure that the dependent libraries are added after the library we
are linking against.

Reviewed by Simon Hausmann.

  • qmake/mkspecs/features/default_post.prf:
7:33 AM Changeset in webkit [104038] by Adam Roben
  • 2 edits in trunk/Source/WebCore

Use one big printf command instead of many chained echo commands to generate idl_files.tmp

This works around a bug in Cygwin where sh.exe will crash if given a too-long sequence of
&&-chained commands.

Fixes <​http://webkit.org/b/75546> <rdar://problem/10622193> REGRESSION (r103519): WebCore's
DerivedSources.make crashes sh.exe if path to WebKit source tree is long enough

Reviewed by Antti Koivisto.

  • DerivedSources.make: Change the '(echo foo && echo bar && echo baz && echo -n) >

./idl_files.tmp' command to 'printf "foo\nbar\nbaz\n" > ./idl_files.tmp'. We use the
patsubst function to append "\n" to each IDL filename, then use the subst function to remove
the spaces between the filenames.

7:26 AM Changeset in webkit [104037] by hclam@chromium.org
  • 63 edits
    3 copies
    4 moves
    5 adds
    8 deletes in trunk/LayoutTests

Not reviewed. Update Chromium port test expectations after Skia roll.

  • platform/chromium-cg-mac-snowleopard/svg/custom/fractional-rects-expected.png: Renamed from LayoutTests/platform/chromium-cg-mac/svg/custom/fractional-rects-expected.png.
  • platform/chromium-linux/fast/backgrounds/animated-svg-as-mask-expected.png:
  • platform/chromium-linux/svg/W3C-SVG-1.1/coords-coord-01-t-expected.png:
  • platform/chromium-linux/svg/W3C-SVG-1.1/coords-coord-02-t-expected.png:
  • platform/chromium-linux/svg/W3C-SVG-1.1/paths-data-13-t-expected.png:
  • platform/chromium-linux/svg/W3C-SVG-1.1/render-groups-01-b-expected.png:
  • platform/chromium-linux/svg/W3C-SVG-1.1/render-groups-03-t-expected.png:
  • platform/chromium-linux/svg/W3C-SVG-1.1/text-align-08-b-expected.png:
  • platform/chromium-linux/svg/as-background-image/animated-svg-as-background-expected.png:
  • platform/chromium-linux/svg/carto.net/button-expected.png: Added.
  • platform/chromium-linux/svg/carto.net/button-expected.txt: Removed.
  • platform/chromium-linux/svg/css/group-with-shadow-expected.png:
  • platform/chromium-linux/svg/custom/fractional-rects-expected.png:
  • platform/chromium-linux/svg/custom/scrolling-embedded-svg-file-image-repaint-problem-expected.png:
  • platform/chromium-linux/svg/custom/use-css-no-effect-on-shadow-tree-expected.png:
  • platform/chromium-linux/svg/text/text-align-01-b-expected.png:
  • platform/chromium-linux/svg/text/text-deco-01-b-expected.png:
  • platform/chromium-linux/svg/text/text-path-01-b-expected.png:
  • platform/chromium-linux/svg/text/text-text-03-b-expected.png:
  • platform/chromium-linux/svg/text/text-text-07-t-expected.png:
  • platform/chromium-linux/svg/zoom/page/zoom-svg-through-object-with-percentage-size-expected.png:
  • platform/chromium-mac-leopard/svg/carto.net/button-expected.png:
  • platform/chromium-mac-leopard/svg/css/group-with-shadow-expected.png:
  • platform/chromium-mac-leopard/svg/custom/scrolling-embedded-svg-file-image-repaint-problem-expected.png:
  • platform/chromium-mac-leopard/svg/text/text-align-01-b-expected.png:
  • platform/chromium-mac-leopard/svg/text/text-deco-01-b-expected.png:
  • platform/chromium-mac-leopard/svg/text/text-path-01-b-expected.png:
  • platform/chromium-mac-leopard/svg/text/text-text-03-b-expected.png:
  • platform/chromium-mac-leopard/svg/text/text-text-07-t-expected.png:
  • platform/chromium-mac-leopard/svg/zoom/page/zoom-svg-through-object-with-percentage-size-expected.png:
  • platform/chromium-mac-snowleopard/fast/backgrounds/animated-svg-as-mask-expected.png:
  • platform/chromium-mac-snowleopard/svg/W3C-SVG-1.1/coords-coord-01-t-expected.png:
  • platform/chromium-mac-snowleopard/svg/W3C-SVG-1.1/coords-coord-02-t-expected.png:
  • platform/chromium-mac-snowleopard/svg/W3C-SVG-1.1/paths-data-13-t-expected.png: Added.
  • platform/chromium-mac-snowleopard/svg/W3C-SVG-1.1/render-groups-01-b-expected.png:
  • platform/chromium-mac-snowleopard/svg/W3C-SVG-1.1/render-groups-03-t-expected.png:
  • platform/chromium-mac-snowleopard/svg/W3C-SVG-1.1/text-align-08-b-expected.png:
  • platform/chromium-mac-snowleopard/svg/as-background-image/animated-svg-as-background-expected.png:
  • platform/chromium-mac-snowleopard/svg/carto.net/button-expected.png:
  • platform/chromium-mac-snowleopard/svg/css/group-with-shadow-expected.png:
  • platform/chromium-mac-snowleopard/svg/custom/fractional-rects-expected.png: Added.
  • platform/chromium-mac-snowleopard/svg/custom/scrolling-embedded-svg-file-image-repaint-problem-expected.png:
  • platform/chromium-mac-snowleopard/svg/custom/use-css-no-effect-on-shadow-tree-expected.png: Renamed from LayoutTests/platform/chromium-mac/svg/custom/use-css-no-effect-on-shadow-tree-expected.png.
  • platform/chromium-mac-snowleopard/svg/text/text-align-01-b-expected.png:
  • platform/chromium-mac-snowleopard/svg/text/text-deco-01-b-expected.png:
  • platform/chromium-mac-snowleopard/svg/text/text-path-01-b-expected.png:
  • platform/chromium-mac-snowleopard/svg/text/text-text-03-b-expected.png:
  • platform/chromium-mac-snowleopard/svg/text/text-text-07-t-expected.png:
  • platform/chromium-mac-snowleopard/svg/zoom/page/zoom-svg-through-object-with-percentage-size-expected.png:
  • platform/chromium-mac/svg/W3C-SVG-1.1/paths-data-13-t-expected.png: Removed.
  • platform/chromium-mac/svg/custom/fractional-rects-expected.png: Removed.
  • platform/chromium-win-xp/fast/backgrounds/animated-svg-as-mask-expected.png: Copied from LayoutTests/platform/chromium-win/fast/backgrounds/animated-svg-as-mask-expected.png.
  • platform/chromium-win-xp/svg/as-background-image/animated-svg-as-background-expected.png: Copied from LayoutTests/platform/chromium-win/svg/as-background-image/animated-svg-as-background-expected.png.
  • platform/chromium-win-xp/svg/text/text-align-01-b-expected.png: Added.
  • platform/chromium-win-xp/svg/text/text-path-01-b-expected.png: Copied from LayoutTests/platform/chromium-win/svg/text/text-path-01-b-expected.png.
  • platform/chromium-win/fast/backgrounds/animated-svg-as-mask-expected.png:
  • platform/chromium-win/svg/W3C-SVG-1.1/coords-coord-01-t-expected.png:
  • platform/chromium-win/svg/W3C-SVG-1.1/coords-coord-02-t-expected.png:
  • platform/chromium-win/svg/W3C-SVG-1.1/paths-data-13-t-expected.png:
  • platform/chromium-win/svg/W3C-SVG-1.1/render-groups-01-b-expected.png:
  • platform/chromium-win/svg/W3C-SVG-1.1/render-groups-03-t-expected.png:
  • platform/chromium-win/svg/W3C-SVG-1.1/text-align-08-b-expected.png:
  • platform/chromium-win/svg/as-background-image/animated-svg-as-background-expected.png:
  • platform/chromium-win/svg/carto.net/button-expected.png:
  • platform/chromium-win/svg/css/group-with-shadow-expected.png:
  • platform/chromium-win/svg/custom/fractional-rects-expected.png:
  • platform/chromium-win/svg/custom/scrolling-embedded-svg-file-image-repaint-problem-expected.png:
  • platform/chromium-win/svg/custom/use-css-no-effect-on-shadow-tree-expected.png:
  • platform/chromium-win/svg/text/text-align-01-b-expected.png:
  • platform/chromium-win/svg/text/text-path-01-b-expected.png:
  • platform/chromium-win/svg/text/text-text-03-b-expected.png:
  • platform/chromium-win/svg/text/text-text-07-t-expected.png:
  • platform/chromium-win/svg/zoom/page/zoom-svg-through-object-with-percentage-size-expected.png:
  • platform/chromium/test_expectations.txt:
  • platform/mac/svg/custom/fractional-rects-expected.txt: Removed.
  • platform/mac/svg/custom/use-css-no-effect-on-shadow-tree-expected.txt: Removed.
  • platform/qt/svg/custom/fractional-rects-expected.txt: Removed.
  • platform/qt/svg/custom/use-css-no-effect-on-shadow-tree-expected.txt: Removed.
  • svg/as-background-image/animated-svg-as-background-expected.txt: Removed.
  • svg/custom/fractional-rects-expected.txt: Renamed from LayoutTests/platform/gtk/svg/custom/fractional-rects-expected.txt.
  • svg/custom/use-css-no-effect-on-shadow-tree-expected.txt: Renamed from LayoutTests/platform/gtk/svg/custom/use-css-no-effect-on-shadow-tree-expected.txt.
6:51 AM Changeset in webkit [104036] by mihnea@adobe.com
  • 14 edits in trunk/Source/WebCore

[CSSRegions]Rollout support for background-color region styling
​https://bugs.webkit.org/show_bug.cgi?id=75007

Reviewed by Tony Chang.

Removed functionality, therefore no new tests. Since this is a temporary rollout,
the tests for region-style with background-color are not removed in this patch, just skipped.

  • WebCore.exp.in:
  • css/CSSStyleSelector.cpp:

(WebCore::CSSStyleSelector::CSSStyleSelector):
(WebCore::CSSStyleSelector::addMatchedDeclaration):
(WebCore::CSSStyleSelector::matchRules):
(WebCore::CSSStyleSelector::matchAllRules):
(WebCore::CSSStyleSelector::styleForElement):
(WebCore::CSSStyleSelector::pseudoStyleForElement):
(WebCore::RuleData::RuleData):
(WebCore::RuleSet::RuleSet):
(WebCore::RuleSet::addToRuleSet):
(WebCore::CSSStyleSelector::applyDeclarations):
(WebCore::CSSStyleSelector::applyProperty):

  • css/CSSStyleSelector.h:
  • rendering/RenderFlowThread.cpp:

(WebCore::RenderFlowThread::setRegionRangeForBox):

  • rendering/RenderFlowThread.h:
  • rendering/RenderLayer.cpp:

(WebCore::RenderLayer::paint):
(WebCore::RenderLayer::hitTest):

  • rendering/RenderObject.cpp:
  • rendering/RenderObject.h:
  • rendering/RenderObjectChildList.cpp:

(WebCore::RenderObjectChildList::removeChildNode):

  • rendering/RenderRegion.cpp:
  • rendering/RenderRegion.h:
  • rendering/RenderView.cpp:

(WebCore::RenderView::RenderView):

  • rendering/RenderView.h:
6:34 AM Changeset in webkit [104035] by abecsi@webkit.org
  • 22 edits in trunk/Source/WebKit2

REGRESSION(r104028): Broke editing tests and event coordinates in desktop behaviour
​https://bugs.webkit.org/show_bug.cgi?id=75542

[Qt][WK2] Pinch zoom should affect the page size
​https://bugs.webkit.org/show_bug.cgi?id=74601

Unreviewed roll-out.

Patch by Andras Becsi <​andras.becsi@nokia.com> on 2012-01-04

  • Shared/NativeWebMouseEvent.h:
  • Shared/NativeWebTouchEvent.h:
  • Shared/NativeWebWheelEvent.h:
  • Shared/qt/NativeWebMouseEventQt.cpp:

(WebKit::NativeWebMouseEvent::NativeWebMouseEvent):

  • Shared/qt/NativeWebTouchEventQt.cpp:

(WebKit::NativeWebTouchEvent::NativeWebTouchEvent):

  • Shared/qt/NativeWebWheelEventQt.cpp:

(WebKit::NativeWebWheelEvent::NativeWebWheelEvent):

  • Shared/qt/WebEventFactoryQt.cpp:

(WebKit::WebEventFactory::createWebMouseEvent):
(WebKit::WebEventFactory::createWebWheelEvent):
(WebKit::WebEventFactory::createWebTouchEvent):

  • Shared/qt/WebEventFactoryQt.h:
  • UIProcess/API/qt/qquickwebpage.cpp:

(QQuickWebPage::keyPressEvent):
(QQuickWebPage::keyReleaseEvent):
(QQuickWebPage::inputMethodEvent):
(QQuickWebPage::focusInEvent):
(QQuickWebPage::focusOutEvent):
(QQuickWebPage::mousePressEvent):
(QQuickWebPage::mouseMoveEvent):
(QQuickWebPage::mouseReleaseEvent):
(QQuickWebPage::mouseDoubleClickEvent):
(QQuickWebPage::wheelEvent):
(QQuickWebPage::hoverEnterEvent):
(QQuickWebPage::hoverMoveEvent):
(QQuickWebPage::hoverLeaveEvent):
(QQuickWebPage::dragMoveEvent):
(QQuickWebPage::dragEnterEvent):
(QQuickWebPage::dragLeaveEvent):
(QQuickWebPage::dropEvent):
(QQuickWebPage::geometryChanged):
(QQuickWebPage::event):
(QQuickWebPage::touchEvent):
(QQuickWebPagePrivate::QQuickWebPagePrivate):
(QQuickWebPagePrivate::paintToCurrentGLContext):

  • UIProcess/API/qt/qquickwebpage_p.h:
  • UIProcess/API/qt/qquickwebpage_p_p.h:
  • UIProcess/API/qt/qquickwebview.cpp:

(QQuickWebViewPrivate::enableMouseEvents):
(QQuickWebViewPrivate::disableMouseEvents):
(QQuickWebViewPrivate::loadDidCommit):
(QQuickWebViewPrivate::didFinishFirstNonEmptyLayout):
(QQuickWebViewPrivate::didChangeContentsSize):
(QQuickWebViewPrivate::didChangeViewportProperties):
(QQuickWebViewPrivate::pageDidRequestScroll):
(QQuickWebViewPrivate::updateVisibleContentRectAndScale):
(QQuickWebViewPrivate::PostTransitionState::apply):
(QQuickWebViewPrivate::setUseTraditionalDesktopBehaviour):
(QQuickWebViewExperimental::setUseTraditionalDesktopBehaviour):
(QQuickWebViewExperimental::useTraditionalDesktopBehaviour):
(QQuickWebView::geometryChanged):
(QQuickWebView::focusInEvent):
(QQuickWebView::focusOutEvent):

  • UIProcess/API/qt/qquickwebview_p.h:
  • UIProcess/API/qt/qquickwebview_p_p.h:
  • UIProcess/API/qt/qwebviewportinfo.cpp:

(QWebViewportInfo::contentsSize):

  • UIProcess/API/qt/tests/qquickwebview/tst_qquickwebview.cpp:

(tst_QQuickWebView::scrollRequest):

  • UIProcess/qt/QtPinchGestureRecognizer.cpp:

(WebKit::QtPinchGestureRecognizer::recognize):

  • UIProcess/qt/QtViewportInteractionEngine.cpp:

(WebKit::QtViewportInteractionEngine::QtViewportInteractionEngine):
(WebKit::QtViewportInteractionEngine::setItemRectVisible):
(WebKit::QtViewportInteractionEngine::animateItemRectVisible):
(WebKit::QtViewportInteractionEngine::event):
(WebKit::QtViewportInteractionEngine::wheelEvent):
(WebKit::QtViewportInteractionEngine::pagePositionRequest):
(WebKit::QtViewportInteractionEngine::computePosRangeForItemAtScale):
(WebKit::QtViewportInteractionEngine::ensureContentWithinViewportBoundary):
(WebKit::QtViewportInteractionEngine::applyConstraints):
(WebKit::QtViewportInteractionEngine::currentCSSScale):
(WebKit::QtViewportInteractionEngine::panGestureStarted):
(WebKit::QtViewportInteractionEngine::panGestureRequestUpdate):
(WebKit::QtViewportInteractionEngine::panGestureEnded):
(WebKit::QtViewportInteractionEngine::pinchGestureStarted):
(WebKit::QtViewportInteractionEngine::pinchGestureRequestUpdate):
(WebKit::QtViewportInteractionEngine::scaleContent):

  • UIProcess/qt/QtViewportInteractionEngine.h:
  • UIProcess/qt/QtWebPageEventHandler.cpp:

(QtWebPageEventHandler::handleEvent):
(QtWebPageEventHandler::handleMouseMoveEvent):
(QtWebPageEventHandler::handleMousePressEvent):
(QtWebPageEventHandler::handleMouseReleaseEvent):
(QtWebPageEventHandler::handleWheelEvent):
(QtWebPageEventHandler::handleHoverLeaveEvent):
(QtWebPageEventHandler::handleHoverMoveEvent):
(QtWebPageEventHandler::handleDragEnterEvent):
(QtWebPageEventHandler::handleDragMoveEvent):
(QtWebPageEventHandler::handleDropEvent):
(QtWebPageEventHandler::handleSingleTapEvent):
(QtWebPageEventHandler::handleDoubleTapEvent):
(QtWebPageEventHandler::touchEvent):

  • UIProcess/qt/QtWebPageEventHandler.h:
6:21 AM Changeset in webkit [104034] by vestbo@webkit.org
  • 27 edits in trunk

[Qt] Introduce new qmake variable 'WEBKIT' for signaling dependencies

The custom qmake variable 'WEBKIT' is used for signaling that a
target depends in some way on other subproject of the WebKit
project. For now this is limited to the set of intermediate
libraries: wtf, javascriptcore, webcore, and webkit2.

This replaces the previous convension of using load(foo) for
just include paths, and CONFIG += foo to also link against foo.

Adding a dependency results in additional include paths being
available, and potentially linking to the library. This is
decided by the build system based on conditions such as what
kind of target is being built and the general build config.

An advantage to his approach is that it simplifies the individual
foo.prf files, for example by allowing us to use INCLUDEPATH +=
and LIBS += as normal instead of prepending.

Reviewed by Simon Hausmann.

6:19 AM Changeset in webkit [104033] by peter@chromium.org
  • 2 edits in trunk/Source/WebKit/chromium

[Chromium] Re-use TestShellLinux.cpp for Android
​https://bugs.webkit.org/show_bug.cgi?id=75169

Reviewed by Tony Chang.

TestShellLinux.cpp can be re-used for the test-shell on the WebKit
Chromium port for Android. Most custom inclusions for the DumpRenderTree
target are done in Tools/DumpRenderTree.gypi, but since *Linux.cpp files
are being excluded by a global rule on sources it has to be later in
the project generation process.

  • WebKit.gyp:
6:04 AM Changeset in webkit [104032] by commit-queue@webkit.org
  • 4 edits
    2 copies in trunk/Tools

[Qt] Add UI for JavaScript Prompt dialog in the Qt MiniBrowser
​https://bugs.webkit.org/show_bug.cgi?id=75488

Patch by Alexander Færøy <​ahf@0x90.dk> on 2012-01-04
Reviewed by Tor Arne Vestbø.

  • MiniBrowser/qt/MiniBrowser.qrc:
  • MiniBrowser/qt/qml/BrowserWindow.qml:
  • MiniBrowser/qt/qml/Dialog.qml:
  • MiniBrowser/qt/qml/DialogLineInput.qml: Copied from Tools/MiniBrowser/qt/qml/Dialog.qml.
  • MiniBrowser/qt/qml/PromptDialog.qml: Copied from Tools/MiniBrowser/qt/qml/Dialog.qml.
5:45 AM Changeset in webkit [104031] by hclam@chromium.org
  • 2 edits in trunk/LayoutTests

Not reviewed. Update Chromium port test expectations.

http/tests/appcache/interrupted-update.html is crashing on Win7, mark it so.

  • platform/chromium/test_expectations.txt:
5:36 AM Changeset in webkit [104030] by hclam@chromium.org
  • 2 edits
    10 adds in trunk/LayoutTests

Not reviewed. Update Chromium port test expectations.

  • platform/chromium-cg-mac-leopard/compositing/visibility/visibility-image-layers-expected.png: Added.
  • platform/chromium-cg-mac-snowleopard/compositing/visibility/visibility-image-layers-expected.png: Added.
  • platform/chromium-linux/compositing/visibility/visibility-image-layers-expected.png: Added.
  • platform/chromium-mac-leopard/compositing/visibility/visibility-image-layers-expected.png: Added.
  • platform/chromium-mac-snowleopard/compositing/visibility/visibility-image-layers-expected.png: Added.
  • platform/chromium-win/compositing/visibility/visibility-image-layers-expected.png: Added.
  • platform/chromium/test_expectations.txt:
5:27 AM Changeset in webkit [104029] by Csaba Osztrogonác
  • 2 edits in trunk/LayoutTests

[Qt][WK2] Unreviewed gardening.

  • platform/qt-wk2/Skipped: Skip a failing test.
5:26 AM Changeset in webkit [104028] by abecsi@webkit.org
  • 22 edits in trunk/Source/WebKit2

[Qt][WK2] Pinch zoom should affect the page size
​https://bugs.webkit.org/show_bug.cgi?id=74601

Patch by Andras Becsi <​andras.becsi@nokia.com> on 2012-01-03
Reviewed by Kenneth Rohde Christiansen and Simon Hausmann.

The current implementation uses the scale property of the QQuickWebPage
to scale the page in response to pinch gestures.
However for layout and anchoring to work correctly in QML, pinching needs
to change the page size.
This patch applies the pinch scale to the page size and to the transformation
matrix of the drawing area.
Thus the page item's coordinate system is no longer a direct representation
of the WebCore::Page coordinate system and it is no longer suitable as
an inertial frame of reference for input events. The event propagation had
to be moved to the QQuickWebView and the positions translated to content
coordinates when NativeWebEvents are created.

  • Shared/NativeWebMouseEvent.h:
  • Shared/NativeWebTouchEvent.h:
  • Shared/NativeWebWheelEvent.h:
  • Shared/qt/NativeWebMouseEventQt.cpp:

(WebKit::NativeWebMouseEvent::NativeWebMouseEvent):

  • Shared/qt/NativeWebTouchEventQt.cpp:

(WebKit::NativeWebTouchEvent::NativeWebTouchEvent):

  • Shared/qt/NativeWebWheelEventQt.cpp:

(WebKit::NativeWebWheelEvent::NativeWebWheelEvent):

  • Shared/qt/WebEventFactoryQt.cpp:

(WebKit::WebEventFactory::createWebMouseEvent):
(WebKit::WebEventFactory::createWebWheelEvent):
(WebKit::WebEventFactory::createWebTouchEvent):

  • Shared/qt/WebEventFactoryQt.h:
  • UIProcess/API/qt/qquickwebpage.cpp:

(QQuickWebPage::geometryChanged):
(QQuickWebPagePrivate::QQuickWebPagePrivate):
(QQuickWebPagePrivate::paintToCurrentGLContext):
(QQuickWebPage::usesTraditionalDesktopBehaviour):
(QQuickWebPage::setUsesTraditionalDesktopBehaviour):
(QQuickWebPage::eventHandler):
(QQuickWebPage::setContentSize):
(QQuickWebPage::contentSize):
(QQuickWebPage::setContentScale):
(QQuickWebPage::contentScale):
(QQuickWebPage::transformFromItem):
(QQuickWebPage::transformToItem):
(QQuickWebPagePrivate::updateSize):

  • UIProcess/API/qt/qquickwebpage_p.h:
  • UIProcess/API/qt/qquickwebpage_p_p.h:
  • UIProcess/API/qt/qquickwebview.cpp:

(QQuickWebViewPrivate::enableMouseEvents):
(QQuickWebViewPrivate::disableMouseEvents):
(QQuickWebViewPrivate::loadDidCommit):
(QQuickWebViewPrivate::didFinishFirstNonEmptyLayout):
(QQuickWebViewPrivate::didChangeContentsSize):
(QQuickWebViewPrivate::didChangeViewportProperties):
(QQuickWebViewPrivate::pageDidRequestScroll):
(QQuickWebViewPrivate::updateVisibleContentRectAndScale):
(QQuickWebViewPrivate::PostTransitionState::apply):
(QQuickWebViewPrivate::setUseTraditionalDesktopBehaviour):
(QQuickWebViewExperimental::setUseTraditionalDesktopBehaviour):
(QQuickWebViewExperimental::useTraditionalDesktopBehaviour):
(QQuickWebView::mapToWebContent):
(QQuickWebView::mapRectToWebContent):
(QQuickWebView::mapFromWebContent):
(QQuickWebView::mapRectFromWebContent):
(QQuickWebView::geometryChanged):
(QQuickWebView::keyPressEvent):
(QQuickWebView::keyReleaseEvent):
(QQuickWebView::inputMethodEvent):
(QQuickWebView::focusInEvent):
(QQuickWebView::focusOutEvent):
(QQuickWebView::touchEvent):
(QQuickWebView::mousePressEvent):
(QQuickWebView::mouseMoveEvent):
(QQuickWebView::mouseReleaseEvent):
(QQuickWebView::mouseDoubleClickEvent):
(QQuickWebView::wheelEvent):
(QQuickWebView::hoverEnterEvent):
(QQuickWebView::hoverMoveEvent):
(QQuickWebView::hoverLeaveEvent):
(QQuickWebView::dragMoveEvent):
(QQuickWebView::dragEnterEvent):
(QQuickWebView::dragLeaveEvent):
(QQuickWebView::dropEvent):
(QQuickWebView::event):

  • UIProcess/API/qt/qquickwebview_p.h:
  • UIProcess/API/qt/qquickwebview_p_p.h:
  • UIProcess/API/qt/qwebviewportinfo.cpp:

(QWebViewportInfo::contentsSize):

  • UIProcess/API/qt/tests/qquickwebview/tst_qquickwebview.cpp:

(tst_QQuickWebView::scrollRequest):

  • UIProcess/qt/QtPinchGestureRecognizer.cpp:

(WebKit::QtPinchGestureRecognizer::recognize):

  • UIProcess/qt/QtViewportInteractionEngine.cpp:

(WebKit::QtViewportInteractionEngine::QtViewportInteractionEngine):
(WebKit::QtViewportInteractionEngine::setItemRectVisible):
(WebKit::QtViewportInteractionEngine::animateItemRectVisible):
(WebKit::QtViewportInteractionEngine::event):
(WebKit::QtViewportInteractionEngine::wheelEvent):
(WebKit::QtViewportInteractionEngine::pagePositionRequest):
(WebKit::QtViewportInteractionEngine::computePosRangeForItemAtScale):
(WebKit::QtViewportInteractionEngine::ensureContentWithinViewportBoundary):
(WebKit::QtViewportInteractionEngine::applyConstraints):
(WebKit::QtViewportInteractionEngine::currentCSSScale):
(WebKit::QtViewportInteractionEngine::panGestureStarted):
(WebKit::QtViewportInteractionEngine::panGestureRequestUpdate):
(WebKit::QtViewportInteractionEngine::panGestureEnded):
(WebKit::QtViewportInteractionEngine::pinchGestureStarted):
(WebKit::QtViewportInteractionEngine::pinchGestureRequestUpdate):
(WebKit::QtViewportInteractionEngine::scaleContent):

  • UIProcess/qt/QtViewportInteractionEngine.h:
  • UIProcess/qt/QtWebPageEventHandler.cpp:

(QtWebPageEventHandler::handleEvent):
(QtWebPageEventHandler::handleMouseMoveEvent):
(QtWebPageEventHandler::handleMousePressEvent):
(QtWebPageEventHandler::handleMouseReleaseEvent):
(QtWebPageEventHandler::handleWheelEvent):
(QtWebPageEventHandler::handleHoverLeaveEvent):
(QtWebPageEventHandler::handleHoverMoveEvent):
(QtWebPageEventHandler::handleDragEnterEvent):
(QtWebPageEventHandler::handleDragMoveEvent):
(QtWebPageEventHandler::handleDropEvent):
(QtWebPageEventHandler::handleSingleTapEvent):
(QtWebPageEventHandler::handleDoubleTapEvent):
(QtWebPageEventHandler::touchEvent):

  • UIProcess/qt/QtWebPageEventHandler.h:
5:09 AM EFLWebKit edited by caio.oliveira@openbossa.org
reordering EFL specific packages so that dependencies come first (diff)
4:55 AM Changeset in webkit [104027] by vestbo@webkit.org
  • 2 edits in trunk/Tools

[Qt] Use rpath for WTR's InjectedBundle to pick up the right QtWebKit

It wasn't really a problem at runtime since the right QtWebKit was
already loaded as a dependency of the web process, but the ldd output
would still be busted.

Reviewed by Simon Hausmann.

4:55 AM Changeset in webkit [104026] by vestbo@webkit.org
  • 11 edits in trunk

[Qt] Move use of $$qtLibraryTarget() to a single place

Reviewed by Kenneth Rohde Christiansen.

3:31 AM Changeset in webkit [104025] by kenneth@webkit.org
  • 4 edits in trunk/Source/WebKit2

[Qt] Clean up Qt specific part of editorState()

Reviewed by Simon Hausmann.

Use unsigned instead of int.
No need to clone the range as we don't modify it.
Support selections which include a composition.
If we are not in editable content make sure to use the document element
as the scope for calculating the positions and lengths.
Remove compositionStart/Length as there cannot be a selection and
composition at the same time.

Only tested manually as we don't have everything in place yet
to properly test this.

  • Shared/EditorState.h:
  • WebProcess/WebPage/WebPage.cpp:

(WebKit::WebPage::editorState):

3:24 AM Changeset in webkit [104024] by Csaba Osztrogonác
  • 2 edits in trunk/LayoutTests

[Qt] http/tests/xmlhttprequest/workers/abort-exception-assert.html makes 6 fast/workers test fail
​https://bugs.webkit.org/show_bug.cgi?id=71695

  • platform/qt/Skipped: Unskip http/tests/xmlhttprequest/workers/abort-exception-assert.html,

because the bug was fixed long long time ago by ​http://trac.webkit.org/changeset/102473.

2:23 AM Changeset in webkit [104023] by Csaba Osztrogonác
  • 2 edits in trunk/LayoutTests

[Qt] Unreviewed gardening.

  • platform/qt-mac/Skipped: Skip new failing tests to paint the bot green.
1:36 AM Changeset in webkit [104022] by mihnea@adobe.com
  • 7 edits in trunk/Source

[Mac]Fix build after changeset 103997
​https://bugs.webkit.org/show_bug.cgi?id=75530

Reviewed by Andreas Kling.

Source/WebKit/mac:

  • WebView/WebFrame.mm:
  • WebView/WebHTMLView.mm:
  • WebView/WebPDFView.mm:
  • WebView/WebView.mm:

Source/WebKit2:

  • UIProcess/API/mac/WKView.mm:
1:26 AM Changeset in webkit [104021] by commit-queue@webkit.org
  • 5 edits in trunk/Source/WebKit/efl

[EFL] Refactor the way using cairo in ewk_tiled_backing_store.
​https://bugs.webkit.org/show_bug.cgi?id=74993

Patch by KwangHyuk Kim <​hyuki.kim@samsung.com> on 2012-01-04
Reviewed by Nikolas Zimmermann.

For now there are cairo related code scattered around the ewk_tiled_backing_store.
From the point of code complexity view, it doesn't look good and I couldn't notice any benefit of it.
So, this patch will let cairo related code be used only for painting operation.

  • ewk/ewk_tiled_backing_store.h:
  • ewk/ewk_tiled_matrix.cpp:

(ewk_tile_matrix_tile_new):

  • ewk/ewk_tiled_model.cpp:

(ewk_tile_new):
(ewk_tile_free):

  • ewk/ewk_view_tiled.cpp:

(_ewk_view_tiled_render_cb):

1:17 AM Changeset in webkit [104020] by commit-queue@webkit.org
  • 4 edits in trunk

Enable DOWNLOAD_ATTRIBUTE in BlackBerry porting
​https://bugs.webkit.org/show_bug.cgi?id=75238

Patch by Mary Wu <​mary.wu@torchmobile.com.cn> on 2012-01-04
Reviewed by Antonio Gomes.

.:

  • Source/cmakeconfig.h.cmake: Modified to add a new cmakedefine ENABLE_DOWNLOAD_ATTRIBUTE.

Tools:

Add ENABLE_DOWNLOAD_ATTRIBUTE webkit feature into the build script so that
it could initiate download directly if it has the download attribute in the link
and doesn't need to open new blank page for the target="_blank". Enabling this
feature in the BlackBerry porting also prepares for later blob feature implementation.

  • Scripts/build-webkit: Modified to handle ENABLE_DOWNLOAD_ATTRIBUTE. Make it default on

for the BlackBerry porting.

12:53 AM EFLWebKitCodingStyle edited by t.morawski@samsung.com
(diff)
12:48 AM Changeset in webkit [104019] by commit-queue@webkit.org
  • 7 edits in trunk/LayoutTests

Remove unnecessary [Custom] attribute in CanvasRenderingContext2D.idl
​https://bugs.webkit.org/show_bug.cgi?id=75376

Patch by Raymond Liu <​raymond.liu@intel.com> on 2012-01-04
Reviewed by Adam Barth.

  • canvas/philip/tests/2d.pattern.image.string-expected.txt:
  • canvas/philip/tests/2d.pattern.image.undefined-expected.txt:
  • platform/chromium/test_expectations.txt:
  • platform/gtk/Skipped:
  • platform/mac/canvas/philip/tests/2d.missingargs-expected.txt:
  • platform/qt/Skipped:
Note: See TracTimeline for information about the timeline view.