Timeline



Jan 2, 2013:

11:11 PM Changeset in webkit [138702] by rniwa@webkit.org
  • 2 edits in trunk/LayoutTests

Add failing test expectations for two canvas tests on Mac Lion
since they have been failing as far as we could tell on the flakiness dashboard.

The failure is tracked by the bug 105999.

  • platform/mac/TestExpectations:
11:09 PM Changeset in webkit [138701] by commit-queue@webkit.org
  • 8 edits in trunk/Source/WebCore

Remove superfluous 'IntPoint toPoint(const IntSize&)'
https://bugs.webkit.org/show_bug.cgi?id=105993

Patch by Steve Block <steveblock@chromium.org> on 2013-01-02
Reviewed by Eric Seidel.

Refactoring only, no change in behavior.

  • accessibility/AccessibilityObject.cpp:

(WebCore::AccessibilityObject::elementAccessibilityHitTest):

  • accessibility/AccessibilityRenderObject.cpp:

(WebCore::AccessibilityRenderObject::remoteSVGElementHitTest):

  • page/EventHandler.cpp:

(WebCore::EventHandler::handleDrag):

  • page/scrolling/ScrollingCoordinator.cpp:

(WebCore::ScrollingCoordinator::updateMainFrameScrollPosition):

  • page/scrolling/mac/ScrollingTreeScrollingNodeMac.mm:

(WebCore::ScrollingTreeScrollingNodeMac::setScrollLayerPosition):

  • platform/graphics/IntPoint.h:
  • rendering/RenderLayer.cpp:

(WebCore::RenderLayer::scrollToOffset):
(WebCore::RenderLayer::offsetFromResizeCorner):
(WebCore::RenderLayer::updateScrollInfoAfterLayout):

11:05 PM Changeset in webkit [138700] by rniwa@webkit.org
  • 2 edits in trunk/LayoutTests

Add a Mac test expectation for the bug 105998.

  • platform/mac-wk2/TestExpectations:
10:11 PM Changeset in webkit [138699] by zhajiang@rim.com
  • 3 edits in trunk/Source/WebKit/blackberry

[BlackBerry] file:/// pages don't get correct scales after several rotations and history navigations
https://bugs.webkit.org/show_bug.cgi?id=105996

Patch by Jacky Jiang <zhajiang@rim.com>.
Reviewed by George Staikos.

PR: 273541
When navigating back to the previous page on portrait mode, we were
trying to set virtual viewport after recalculation. However, we didn't
change the fixed layout size so that we still layouted the contents at
the old virtual viewport size. In this case, the old virtual viewport
was 570 * 293 which was set in landscape mode unfortunately. There
would be no chance for such kind of contents size to fit the portrait
mode screen size nicely.
This was a regression of webkit/cc31d9bad where setViewMode() was
removed as a noop. So just get setViewMode() back to apply the fixed
layout size with a little bit change as SVG document path was proved
invalid in that patch. Thanks to George for the float comparison change.
Put a FIXME as we don't have a way to save the scale precisely before
the rotation and history navigation currently.

  • Api/WebPage.cpp:

(BlackBerry::WebKit::WebPagePrivate::setLoadState):

  • WebCoreSupport/FrameLoaderClientBlackBerry.cpp:

(WebCore::FrameLoaderClientBlackBerry::saveViewStateToItem):

9:36 PM Changeset in webkit [138698] by rniwa@webkit.org
  • 3 edits in trunk/LayoutTests

More Mac test expectation updates per bugs 105988 and 105987.

  • platform/mac-wk2/TestExpectations:
  • platform/mac/TestExpectations:
8:47 PM Changeset in webkit [138697] by commit-queue@webkit.org
  • 2 edits in trunk/Tools

[EFL][WK2] Fix tooltip bugs in MiniBrowser.
https://bugs.webkit.org/show_bug.cgi?id=102209

Patch by Byungwoo Lee <bw80.lee@samsung.com> on 2013-01-02
Reviewed by Gyuyoung Kim.

Set tooltip on the elm_window instead of ewk_view, and add delay to
show the tooltip window.

Setting tooltip on the ewk_view makes elementary error message because
ewk_view is not an elementary widget. To remove the error message,
tooltip should be set on the elm_window.

Most browsers show the tooltip after a mouse pointer stays on an
element object that has tooltip text.
But elm_tooltip widget itself doesn't have delay or handle mouse
events to show/hide.

So additional codes are added to show/hide tooltip widget along with
mouse events.

1) When a mouse pointer stays for a second on an element which has

tooltip text, then tooltip widget is displayed.

2) When a mouse pointer is moved or out of ewk_view, hide the tooltip

widget.

  • MiniBrowser/efl/main.c:

(_Tooltip_Information):
(_Browser_Window):
(on_tooltip_show):
(window_tooltip_hide):
(window_tooltip_update):
(on_mouse_in):
(on_mouse_move):
(on_mouse_out):
(window_free):
(on_tooltip_text_set):
(on_tooltip_text_unset):
(window_create):

8:44 PM Changeset in webkit [138696] by commit-queue@webkit.org
  • 3 edits in trunk/Source/WebKit/blackberry

[BlackBerry] Do not determine db quota limit using db file size
https://bugs.webkit.org/show_bug.cgi?id=105991

PR 267191

Patch by Otto Derek Cheung <otcheung@rim.com> on 2013-01-02
Reviewed by George Staikos.

Passing the current quota to WebPageClientImpl::databaseQuota so we can determine a better
quota limit without using the db file size.

  • Api/WebPageClient.h:
  • WebCoreSupport/ChromeClientBlackBerry.cpp:

(WebCore::ChromeClientBlackBerry::exceededDatabaseQuota):

8:39 PM Changeset in webkit [138695] by commit-queue@webkit.org
  • 3 edits in trunk/Source/WebCore

[TexMap] Remove m_size in TextureMapperLayer.
https://bugs.webkit.org/show_bug.cgi?id=105781

Patch by Huang Dongsung <luxtella@company100.net> on 2013-01-02
Reviewed by Noam Rosenthal.

m_size is duplicated by State::size, so this patch removes it.

No new tests. Refactoring only.

  • platform/graphics/texmap/TextureMapperLayer.cpp:

(WebCore::TextureMapperLayer::computeTransformsRecursive):
(WebCore::TextureMapperLayer::paintSelf):
(WebCore::TextureMapperLayer::isVisible):
(WebCore::TextureMapperLayer::flushCompositingStateForThisLayerOnly):

  • platform/graphics/texmap/TextureMapperLayer.h:

(WebCore::TextureMapperLayer::layerRect):
(TextureMapperLayer):

7:24 PM Changeset in webkit [138694] by rniwa@webkit.org
  • 2 edits in trunk/LayoutTests

Add flaky crash expectations for multiple tests on Mac WebKit2.

  • platform/mac-wk2/TestExpectations:
7:22 PM Changeset in webkit [138693] by kbr@google.com
  • 15 edits
    4 adds in trunk

[chromium] Creation of dedicated workers (isolates) leaks reference to HTMLDocument
https://bugs.webkit.org/show_bug.cgi?id=105367

Reviewed by Dmitry Titov.

Source/WebCore:

Eliminated a Chromium-specific object wrapping WorkerMessagingProxy in order to fix a
lifetime management bug, which leaked every Document which started a dedicated worker.

Test: fast/workers/worker-document-leak.html

  • workers/WorkerLoaderProxy.h:

(WorkerLoaderProxy):

Added Chromium-specific casting method to bridge two now-distinct class hierarchies.

  • workers/WorkerMessagingProxy.h:

(WorkerMessagingProxy):

Made destructor protected instead of private to allow subclassing.

Source/WebKit/chromium:

Made WebWorkerClientImpl a subclass of WorkerMessagingProxy rather than an object wrapping
WorkerMessagingProxy. WorkerMessagingProxy manages its own lifetime and it is impossible to
properly synchronize the lifetime of WebWorkerClientImpl separately.

This allowed most of WebWorkerClientImpl to be deleted, but forced a divergence in the class
hierarchies of WebWorkerClientImpl and WebSharedWorkerImpl. Conversion methods were added to
WorkerLoaderProxy and WebWorkerBase to bridge the hierarchies of in-process and
out-of-process workers.

  • src/DatabaseObserver.cpp:

(WebCore::DatabaseObserver::canEstablishDatabase):

Adjusted how WebWorkerBase is obtained from WorkerLoaderProxy.

  • src/IDBFactoryBackendProxy.cpp:

(WebKit::AllowIndexedDBMainThreadBridge::signalCompleted):

Adjusted how WorkerLoaderProxy's methods are called.

(WebKit::IDBFactoryBackendProxy::allowIndexedDB):

Adjusted how WebWorkerBase is obtained from WorkerLoaderProxy.

  • src/LocalFileSystemChromium.cpp:

(WebCore::openFileSystemHelper):

Adjusted how WebWorkerBase is obtained from WorkerLoaderProxy.

  • src/WebSharedWorkerImpl.cpp:

(WebKit::WebSharedWorkerImpl::toWebWorkerBase):

Implemented new conversion method.

  • src/WebSharedWorkerImpl.h:

(WebSharedWorkerImpl):

Explicitly derive from WorkerLoaderProxy now that WebWorkerBase no longer does.

(WebKit::WebSharedWorkerImpl::workerLoaderProxy):

Added new conversion method.

  • src/WebWorkerBase.h:

(WebWorkerBase):

Removed derivation from WorkerLoaderProxy. Added method to convert to WorkerLoaderProxy.

  • src/WebWorkerClientImpl.cpp:

(WebKit):

Adjusted comment.

(WebKit::WebWorkerClientImpl::createWorkerContextProxy):

Adjusted whitespace.

(WebKit::WebWorkerClientImpl::terminateWorkerContext):

Eliminated delegation to separate object.

(WebKit::WebWorkerClientImpl::toWebWorkerBase):

Implemented new conversion method.

(WebKit::WebWorkerClientImpl::view):
(WebKit::WebWorkerClientImpl::allowDatabase):
(WebKit::WebWorkerClientImpl::allowFileSystem):
(WebKit::WebWorkerClientImpl::openFileSystem):
(WebKit::WebWorkerClientImpl::allowIndexedDB):

Eliminated delegation to separate object.

(WebKit::WebWorkerClientImpl::WebWorkerClientImpl):

  • src/WebWorkerClientImpl.h:

(WebKit):

Changed to inherit from WorkerMessagingProxy directly.

(WebWorkerClientImpl):

Deleted most methods previously overridden from WorkerContextProxy, etc.

  • src/WorkerAsyncFileSystemChromium.cpp:

(WebCore::WorkerAsyncFileSystemChromium::WorkerAsyncFileSystemChromium):
(WebCore::WorkerAsyncFileSystemChromium::createWorkerFileSystemCallbacksBridge):

Hold on to, and use, WorkerLoaderProxy rather than WebWorkerBase.

  • src/WorkerAsyncFileSystemChromium.h:

(WebKit):
(WebCore):
(WorkerAsyncFileSystemChromium):

Hold on to WorkerLoaderProxy rather than WebWorkerBase.

LayoutTests:

  • fast/workers/resources/empty-worker.js: Added.
  • fast/workers/resources/worker-document-leak-iframe.html: Added.
  • fast/workers/worker-document-leak-expected.txt: Added.
  • fast/workers/worker-document-leak.html: Added.
7:09 PM Changeset in webkit [138692] by rniwa@webkit.org
  • 3 edits
    2 deletes in trunk/LayoutTests

One more Mac rebaseline attempt for r138654. Also remove an entry for the bug 105514
which has been dup'ed with the bug 103663 for which we have more specific expectations.

  • platform/mac-lion/editing/deleting: Removed.
  • platform/mac-lion/editing/deleting/delete-to-select-table-expected.txt: Removed.
  • platform/mac-wk2/editing/deleting: Removed.
  • platform/mac-wk2/editing/deleting/delete-to-select-table-expected.txt: Removed.
  • platform/mac/TestExpectations:
  • platform/mac/editing/deleting/delete-to-select-table-expected.txt:
6:59 PM Changeset in webkit [138691] by esprehn@chromium.org
  • 6 edits in trunk/Source/WebCore

Make ClassList::reset's purpose obvious and don't keep quirks string when not needed
https://bugs.webkit.org/show_bug.cgi?id=105963

Reviewed by Ojan Vafai.

ClassList::reset only exists to handle updating the special SpaceSplitString
for quirks mode documents. This adds a new method that makes this obvious and
instead of updating the string immediately clear the value and lazily
update it. We also clear the value whenever we're inserted into the
document so that when moving from a quirks mode document to a non-quirks
mode document we don't keep the SpaceSplitString around if it's not needed.

No new tests, I'm not sure how to write a test that we don't keep the
SpaceSplitString when moving between quirks and non-quirks mode documents.

  • dom/Element.cpp:

(WebCore::Element::classAttributeChanged):
(WebCore::Element::insertedInto):

  • dom/Element.h:

(Element):

  • dom/ElementRareData.h:

(WebCore::ElementRareData::clearClassListValueForQuirksMode):

  • html/ClassList.cpp:
  • html/ClassList.h:

(WebCore::ClassList::clearValueForQuirksMode):

6:05 PM Changeset in webkit [138690] by commit-queue@webkit.org
  • 6 edits in trunk/Source/WebKit/chromium

[chromium] Remove unused transitional #defines from WebKit Client API
https://bugs.webkit.org/show_bug.cgi?id=105974

Patch by James Robinson <jamesr@chromium.org> on 2013-01-02
Reviewed by Adam Barth.

The transitions these #defines helped with have completed.

  • public/WebFileSystemCallbacks.h:

(WebKit):

  • public/WebKit.h:
  • public/WebRuntimeFeatures.h:
  • public/WebSettings.h:
  • public/WebWidget.h:
6:03 PM Changeset in webkit [138689] by barraclough@apple.com
  • 14 edits in trunk/Source/JavaScriptCore

Objective-C API for JavaScriptCore
https://bugs.webkit.org/show_bug.cgi?id=105889

Reviewed by Geoff Garen.

Fixes for more issues raised by Darin.

  • API/JSBlockAdaptor.mm:

(BlockArgument):
(BlockArgumentStruct::BlockArgumentStruct):
(BlockArgumentTypeDelegate::typeStruct):
(BlockResult):
(BlockResultStruct::BlockResultStruct):
(buildBlockSignature):
(-[JSBlockAdaptor initWithBlockSignatureFromProtocol:]):
(-[JSBlockAdaptor blockFromValue:inContext:withException:]):

  • fix * position for Objective-C types
  • API/JSContext.h:
    • fix * position for Objective-C types
  • API/JSContext.mm:

(-[JSContext initWithVirtualMachine:]):
(-[JSContext virtualMachine]):
(contextInternalContext):

  • fix * position for Objective-C types

(-[JSContext dealloc]):
(-[JSContext protect:]):
(-[JSContext unprotect:]):

  • HashMap<JSValueRef, size_t> -> HashCountedSet<JSValueRef>
  • API/JSContextInternal.h:

(WeakContextRef):

  • fix * position for Objective-C types
  • API/JSValue.mm:

(valueToString):

  • fix * position for Objective-C types

(isNSBoolean):

  • Added helper to check for booleans.

(objectToValueWithoutCopy):

  • Added contextRef
  • fix * position for Objective-C types
  • Remove @YES, @NO literal usage, use isNSBoolean instead

(objectToValue):

  • Added contextRef

(+[JSValue valueWithValue:inContext:]):
(-[JSValue initWithValue:inContext:]):

  • fix * position for Objective-C types

(createStructHandlerMap):
(handerForStructTag):

  • getStructTagHandler -> handerForStructTag
  • Split out createStructHandlerMap
  • strncmp -> memcmp
  • String(type).impl() -> StringImpl::create(type)

(+[JSValue selectorForStructToValue:]):
(+[JSValue selectorForValueToStruct:]):

  • getStructTagHandler -> handerForStructTag

(typeToValueInvocationFor):
(valueToTypeInvocationFor):

  • fix * position for Objective-C types
  • API/JSValueInternal.h:
    • fix * position for Objective-C types
  • API/JSVirtualMachineInternal.h:
    • fix * position for Objective-C types
  • API/JSWrapperMap.h:
    • fix * position for Objective-C types
  • API/JSWrapperMap.mm:

(selectorToPropertyName):
(createObjectWithCustomBrand):
(createRenameMap):
(putNonEnumerable):
(copyMethodsToObject):
(copyPrototypeProperties):
(-[JSObjCClassInfo initWithContext:forClass:superClassInfo:]):
(-[JSWrapperMap initWithContext:]):
(-[JSWrapperMap wrapperForObject:]):
(getJSExportProtocol):

  • fix * position for Objective-C types
  • API/ObjCCallbackFunction.h:
    • fix * position for Objective-C types
  • API/ObjCCallbackFunction.mm:

(CallbackArgument):
(CallbackArgumentStruct::CallbackArgumentStruct):

  • fix * position for Objective-C types

(CallbackArgumentBlockCallback::createAdoptingJSBlockAdaptor):

  • Added to make adopt explicit

(CallbackArgumentBlockCallback):
(CallbackArgumentBlockCallback::CallbackArgumentBlockCallback):
(ArgumentTypeDelegate::typeBlock):

  • Call createAdoptingJSBlockAdaptor

(ArgumentTypeDelegate::typeStruct):
(CallbackResult):
(CallbackResultStruct::CallbackResultStruct):
(ResultTypeDelegate::typeStruct):
(ObjCCallbackFunction::ObjCCallbackFunction):
(ObjCCallbackFunction::context):
(objCCallbackFunctionForInvocation):
(objCCallbackFunctionForMethod):
(objCCallbackFunctionForBlock):

  • fix * position for Objective-C types
  • API/ObjcRuntimeExtras.h:

(protocolImplementsProtocol):
(forEachProtocolImplementingProtocol):
(forEachMethodInProtocol):
(forEachPropertyInProtocol):

  • fix * position for Objective-C types
  • API/tests/testapi.m:

(-[TestObject testArgumentTypesWithInt:double:boolean:string:number:array:dictionary:]):
(testObjectiveCAPI):

  • fix * position for Objective-C types
5:29 PM Changeset in webkit [138688] by Lucas Forschler
  • 48 edits
    20 adds in branches/safari-536.28-branch/LayoutTests

Rebaseline Chopin SVG results.
<rdar://problem/12947584> SVG test failures caused by missing merge

Reviewed by Tim Horton.

  • platform/mac/svg/W3C-SVG-1.1/animate-elem-04-t-expected.txt:
  • platform/mac/svg/W3C-SVG-1.1/animate-elem-05-t-expected.txt:
  • platform/mac/svg/W3C-SVG-1.1/animate-elem-06-t-expected.txt:
  • platform/mac/svg/W3C-SVG-1.1/animate-elem-07-t-expected.txt:
  • platform/mac/svg/carto.net/tabgroup-expected.txt:
  • platform/mac/svg/css/shadow-changes-expected.txt:
  • platform/mac/svg/custom/SVGPoint-matrixTransform-expected.txt:
  • platform/mac/svg/custom/animate-path-morphing-expected.txt:
  • platform/mac/svg/custom/circle-move-invalidation-expected.txt:
  • platform/mac/svg/custom/clip-path-child-changes-expected.txt: Added.
  • platform/mac/svg/custom/clip-path-href-changes-expected.txt: Added.
  • platform/mac/svg/custom/clip-path-id-changes-expected.txt: Added.
  • platform/mac/svg/custom/clip-path-units-changes-expected.txt: Added.
  • platform/mac/svg/custom/getscreenctm-in-scrollable-div-area-expected.txt:
  • platform/mac/svg/custom/getscreenctm-in-scrollable-div-area-nested-expected.txt:
  • platform/mac/svg/custom/getscreenctm-in-scrollable-svg-area-expected.txt:
  • platform/mac/svg/custom/js-late-clipPath-and-object-creation-expected.txt:
  • platform/mac/svg/custom/js-late-clipPath-creation-expected.txt:
  • platform/mac/svg/custom/js-late-gradient-creation-expected.txt:
  • platform/mac/svg/custom/js-late-pattern-creation-expected.txt:
  • platform/mac/svg/custom/js-repaint-rect-on-path-with-stroke-expected.txt: Added.
  • platform/mac/svg/custom/js-update-bounce-expected.txt: Added.
  • platform/mac/svg/custom/js-update-container-expected.txt: Added.
  • platform/mac/svg/custom/js-update-polygon-changes-expected.txt: Added.
  • platform/mac/svg/custom/js-update-polygon-removal-expected.txt: Added.
  • platform/mac/svg/custom/js-update-style-expected.txt: Added.
  • platform/mac/svg/custom/js-update-transform-addition-expected.txt: Added.
  • platform/mac/svg/custom/js-update-transform-changes-expected.txt: Added.
  • platform/mac/svg/custom/marker-viewBox-changes-expected.txt: Added.
  • platform/mac/svg/custom/mask-invalidation-expected.txt: Added.
  • platform/mac/svg/custom/mouse-move-on-svg-container-expected.txt:
  • platform/mac/svg/custom/mouse-move-on-svg-container-standalone-expected.txt:
  • platform/mac/svg/custom/mouse-move-on-svg-root-expected.txt:
  • platform/mac/svg/custom/mouse-move-on-svg-root-standalone-expected.txt:
  • platform/mac/svg/custom/preserve-aspect-ratio-syntax-expected.txt:
  • platform/mac/svg/custom/relative-sized-content-with-resources-expected.txt:
  • platform/mac/svg/custom/relative-sized-deep-shadow-tree-content-expected.txt:
  • platform/mac/svg/custom/relative-sized-inner-svg-expected.txt:
  • platform/mac/svg/custom/relative-sized-use-on-symbol-expected.txt:
  • platform/mac/svg/custom/relative-sized-use-without-attributes-on-symbol-expected.txt:
  • platform/mac/svg/custom/repaint-stroke-width-changes-expected.txt: Added.
  • platform/mac/svg/custom/resource-invalidate-on-target-update-expected.txt: Added.
  • platform/mac/svg/custom/text-hit-test-expected.txt:
  • platform/mac/svg/custom/text-repaint-including-stroke-expected.txt:
  • platform/mac/svg/custom/text-xy-updates-SVGList-expected.txt:
  • platform/mac/svg/custom/use-modify-container-in-target-expected.txt:
  • platform/mac/svg/custom/use-move-to-offset-expected.txt: Added.
  • platform/mac/svg/custom/use-referencing-nonexisting-symbol-expected.txt:
  • platform/mac/svg/custom/viewbox-syntax-expected.txt:
  • platform/mac/svg/filters/invalidate-on-child-layout-expected.txt: Added.
  • platform/mac/svg/hixie/text/003b-expected.txt:
  • platform/mac/svg/repaint/repaint-webkit-svg-shadow-expected.txt: Added.
  • platform/mac/svg/repaint/repainting-after-animation-element-removal-expected.txt:
  • platform/mac/svg/text/remove-text-node-from-tspan-expected.txt:
  • platform/mac/svg/text/select-x-list-1-expected.txt:
  • platform/mac/svg/text/select-x-list-2-expected.txt:
  • platform/mac/svg/text/select-x-list-3-expected.txt:
  • platform/mac/svg/text/select-x-list-4-expected.txt:
  • platform/mac/svg/text/select-x-list-with-tspans-1-expected.txt:
  • platform/mac/svg/text/select-x-list-with-tspans-2-expected.txt:
  • platform/mac/svg/text/select-x-list-with-tspans-3-expected.txt:
  • platform/mac/svg/text/select-x-list-with-tspans-4-expected.txt:
  • platform/mac/svg/text/text-rescale-expected.txt:
  • platform/mac/svg/text/text-text-01-b-expected.txt:
  • platform/mac/svg/text/tspan-dynamic-positioning-expected.txt:
  • platform/mac/svg/transforms/animated-path-inside-transformed-html-expected.txt:
  • platform/mac/svg/zoom/page/relative-sized-document-scrollbars-expected.txt: Added.
5:21 PM Changeset in webkit [138687] by commit-queue@webkit.org
  • 5 edits in trunk/Source/Platform

[chromium] Remove unused transitional #defines from Platform API
https://bugs.webkit.org/show_bug.cgi?id=105971

Patch by James Robinson <jamesr@chromium.org> on 2013-01-02
Reviewed by Adrienne Walker.

These are leftovers from multi-sided patches that landed months ago.

  • chromium/public/WebContentLayerClient.h:

(WebContentLayerClient):

  • chromium/public/WebLayer.h:
  • chromium/public/WebLayerTreeView.h:
  • chromium/public/WebThread.h:
5:20 PM Changeset in webkit [138686] by rniwa@webkit.org
  • 1 edit
    5 adds
    3 deletes in trunk/LayoutTests

Mac and Chromium rebaselines after r138654.

  • editing/deleting/delete-to-select-table-expected.txt: Added.
  • platform/chromium-mac/editing/deleting/delete-to-select-table-expected.txt: Added.
  • platform/chromium/editing/deleting/delete-to-select-table-expected.txt: Removed.
  • platform/efl/editing/deleting/delete-to-select-table-expected.txt: Removed.
  • platform/gtk/editing/deleting/delete-to-select-table-expected.txt: Removed.
  • platform/mac-lion/editing/deleting/delete-to-select-table-expected.txt: Added.
  • platform/mac-wk2/editing/deleting: Added.
  • platform/mac-wk2/editing/deleting/delete-to-select-table-expected.txt: Added.
5:11 PM Changeset in webkit [138685] by macpherson@chromium.org
  • 1 edit
    1 add in trunk/LayoutTests

Unreviewed Chromium gardening.

Rebaseline win-xp image for svg/dynamic-updates/SVGFEComponentTransferElement-dom-tableValues-attr-expected.html.

  • platform/chromium-win-xp/svg/dynamic-updates/SVGFEComponentTransferElement-dom-tableValues-attr-expected.png: Added.
5:10 PM Changeset in webkit [138684] by cdn@chromium.org
  • 2 edits in trunk/Source/WebKit/chromium

Initialize WebInputEvent to prevent sending uninitialized bytes with IPC messages.
https://bugs.webkit.org/show_bug.cgi?id=105934

Reviewed by James Robinson.

  • public/WebInputEvent.h:

(WebKit::WebInputEvent::WebInputEvent):

5:06 PM Changeset in webkit [138683] by weinig@apple.com
  • 14 edits in trunk/Source/WebKit2

Stop pretending its possible to run the WebProcess in threaded mode
https://bugs.webkit.org/show_bug.cgi?id=105969

Reviewed by Alexey Proskuryakov.

  • Removes WebProcess::isSeparateProcess()
  • Stop storing a RunLoop on the WebProcess and just use RunLoop::main().
  • WebProcess/Network/NetworkProcessConnection.cpp:

(WebKit::NetworkProcessConnection::NetworkProcessConnection):

  • WebProcess/Plugins/Netscape/NPRuntimeObjectMap.cpp:

(WebKit::NPRuntimeObjectMap::NPRuntimeObjectMap):

  • WebProcess/Plugins/PluginProcessConnection.cpp:

(WebKit::PluginProcessConnection::PluginProcessConnection):

  • WebProcess/WebPage/DrawingAreaImpl.cpp:

(WebKit::DrawingAreaImpl::DrawingAreaImpl):

  • WebProcess/WebPage/PageOverlay.cpp:

(WebKit::PageOverlay::PageOverlay):

  • WebProcess/WebPage/WebPage.cpp:

(WebKit::WebPage::WebPage):
(WebKit::WebPage::close):

  • WebProcess/WebProcess.cpp:

(WebKit::WebProcess::initializeConnection):
(WebKit::WebProcess::shouldTerminate):
(WebKit::WebProcess::terminate):
(WebKit::WebProcess::didClose):

  • WebProcess/WebProcess.h:

(WebProcess):

  • WebProcess/efl/WebProcessMainEfl.cpp:

(WebKit::WebProcessMainEfl):

  • WebProcess/gtk/WebProcessMainGtk.cpp:

(WebKit::WebProcessMainGtk):

  • WebProcess/mac/WebProcessInitialization.mm:

(WebKit::initializeWebProcess):

  • WebProcess/qt/WebProcessMainQt.cpp:

(WebKit::WebProcessMainQt):

  • WebProcess/win/WebProcessMainWin.cpp:

(WebKit::WebProcessMain):

5:04 PM Changeset in webkit [138682] by abarth@webkit.org
  • 2 edits in trunk/Source/WebCore

[V8] V8DOMWrapper.cpp has many more includes than necessary
https://bugs.webkit.org/show_bug.cgi?id=105968

Reviewed by Kentaro Hara.

Now that this file isn't in charge of doing everything under the sun,
it can have a more focused set of include directives.

  • bindings/v8/V8DOMWrapper.cpp:
5:00 PM Changeset in webkit [138681] by gyuyoung.kim@samsung.com
  • 4 edits in trunk

[EFL][CMAKE] Remove duplicated conditionals
https://bugs.webkit.org/show_bug.cgi?id=105905

Reviewed by Laszlo Gombos.

.:

  • Source/cmake/OptionsEfl.cmake: Remove set(WTF_USE_GLX 1).

Source/WebCore:

Remove conditionals from CMake build system, where the files being built are already wrapped with
the conditionals. It just messes cmake file up.

  • PlatformEfl.cmake:
4:55 PM Changeset in webkit [138680] by commit-queue@webkit.org
  • 4 edits in trunk/Source

AX: Chromium needs platform localized strings for media controls.
https://bugs.webkit.org/show_bug.cgi?id=105940

Patch by David Tseng <dtseng@google.com> on 2013-01-02
Reviewed by Tony Chang.

Source/Platform:

  • chromium/public/WebLocalizedString.h:

Source/WebKit/chromium:

  • src/LocalizedStrings.cpp:

(WebCore::localizedMediaControlElementString):
(WebCore):
(WebCore::localizedMediaControlElementHelpText):

4:53 PM Changeset in webkit [138679] by ap@apple.com
  • 2 edits in trunk/Source/WebCore

Release build fix.

  • platform/network/cf/FormDataStreamCFNet.cpp: (WebCore::formFinalize): Use ASSERT_UNUSED for an otherwise unused variable.
4:43 PM Changeset in webkit [138678] by tony@chromium.org
  • 23 edits in trunk

Unreviewed, rolling out r138661.
http://trac.webkit.org/changeset/138661
https://bugs.webkit.org/show_bug.cgi?id=104740

Compile problems on EFL

.:

  • Source/cmake/WebKitMacros.cmake:

Source/WebCore:

  • CMakeLists.txt:
  • DerivedSources.make:
  • DerivedSources.pri:
  • GNUmakefile.am:
  • UseJSC.cmake:
  • UseV8.cmake:
  • WebCore.gyp/WebCore.gyp:
  • WebCore.gypi:
  • WebCore.vcproj/WebCoreTestSupport.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • bindings/scripts/CodeGenerator.pm:

(IDLFileForInterface):

  • page/make_settings.pl:

(generateCode):
(generateHeader):
(printGetterAndSetter):
(printInitializer):

  • testing/InternalSettings.cpp:

(WebCore::InternalSettings::Backup::Backup):
(WebCore::InternalSettings::Backup::restoreTo):
(WebCore::InternalSettings::from):
(WebCore::InternalSettings::InternalSettings):
(WebCore::InternalSettings::reset):
(WebCore::InternalSettings::setPasswordEchoEnabled):
(WebCore):
(WebCore::InternalSettings::setPasswordEchoDurationInSeconds):

  • testing/InternalSettings.h:

(Backup):
(InternalSettings):

  • testing/InternalSettings.idl:
  • testing/js/WebCoreTestSupport.cpp:

(WebCoreTestSupport::resetInternalsObject):

  • testing/v8/WebCoreTestSupport.cpp:

(WebCoreTestSupport::resetInternalsObject):

Tools:

  • GNUmakefile.am:
  • qmake/mkspecs/features/default_post.prf:
4:30 PM Changeset in webkit [138677] by abarth@webkit.org
  • 23 edits in trunk/Source/WebCore

[V8] There are some extra blank lines after removing INC_STATS
https://bugs.webkit.org/show_bug.cgi?id=105964

Reviewed by Ojan Vafai.

This patch is a followup to http://trac.webkit.org/changeset/138665
that removes some extra blank lines that sed wasn't smart enough to
delete for us.

  • bindings/v8/custom/V8CSSStyleDeclarationCustom.cpp:

(WebCore::V8CSSStyleDeclaration::namedPropertyQuery):

  • bindings/v8/custom/V8DOMFormDataCustom.cpp:

(WebCore::V8DOMFormData::appendCallback):

  • bindings/v8/custom/V8DOMWindowCustom.cpp:

(WebCore::V8DOMWindow::addEventListenerCallback):
(WebCore::V8DOMWindow::removeEventListenerCallback):
(WebCore::V8DOMWindow::postMessageCallback):

  • bindings/v8/custom/V8DedicatedWorkerContextCustom.cpp:

(WebCore::V8DedicatedWorkerContext::postMessageCallback):

  • bindings/v8/custom/V8DocumentCustom.cpp:

(WebCore::V8Document::evaluateCallback):

  • bindings/v8/custom/V8GeolocationCustom.cpp:

(WebCore::V8Geolocation::getCurrentPositionCallback):
(WebCore::V8Geolocation::watchPositionCallback):

  • bindings/v8/custom/V8HTMLAllCollectionCustom.cpp:

(WebCore::V8HTMLAllCollection::namedPropertyGetter):

  • bindings/v8/custom/V8HTMLCollectionCustom.cpp:

(WebCore::V8HTMLCollection::namedPropertyGetter):

  • bindings/v8/custom/V8HTMLFormControlsCollectionCustom.cpp:

(WebCore::V8HTMLFormControlsCollection::namedPropertyGetter):

  • bindings/v8/custom/V8HTMLImageElementConstructor.cpp:

(WebCore::v8HTMLImageElementConstructorCallback):

  • bindings/v8/custom/V8HTMLOptionsCollectionCustom.cpp:

(WebCore::V8HTMLOptionsCollection::namedPropertyGetter):

  • bindings/v8/custom/V8MessagePortCustom.cpp:

(WebCore::V8MessagePort::postMessageCallback):

  • bindings/v8/custom/V8NamedNodeMapCustom.cpp:

(WebCore::V8NamedNodeMap::namedPropertyGetter):

  • bindings/v8/custom/V8NotificationCustom.cpp:

(WebCore::V8Notification::requestPermissionCallback):

  • bindings/v8/custom/V8PopStateEventCustom.cpp:

(WebCore::V8PopStateEvent::stateAccessorGetter):

  • bindings/v8/custom/V8SQLResultSetRowListCustom.cpp:

(WebCore::V8SQLResultSetRowList::itemCallback):

  • bindings/v8/custom/V8SQLTransactionCustom.cpp:

(WebCore::V8SQLTransaction::executeSqlCallback):

  • bindings/v8/custom/V8SQLTransactionSyncCustom.cpp:

(WebCore::V8SQLTransactionSync::executeSqlCallback):

  • bindings/v8/custom/V8StorageCustom.cpp:

(WebCore::V8Storage::namedPropertyQuery):

  • bindings/v8/custom/V8StyleSheetListCustom.cpp:

(WebCore::V8StyleSheetList::namedPropertyGetter):

  • bindings/v8/custom/V8WebGLRenderingContextCustom.cpp:

(WebCore::V8WebGLRenderingContext::getAttachedShadersCallback):
(WebCore::V8WebGLRenderingContext::getFramebufferAttachmentParameterCallback):
(WebCore::V8WebGLRenderingContext::getParameterCallback):
(WebCore::V8WebGLRenderingContext::getProgramParameterCallback):
(WebCore::V8WebGLRenderingContext::getShaderParameterCallback):
(WebCore::V8WebGLRenderingContext::getUniformCallback):

  • bindings/v8/custom/V8WorkerCustom.cpp:

(WebCore::V8Worker::postMessageCallback):

4:26 PM Changeset in webkit [138676] by ap@apple.com
  • 2 edits in trunk/Source/WebCore

[WK2 NetworkProcess] Uploading fails if a client modified the request
https://bugs.webkit.org/show_bug.cgi?id=105965

Reviewed by Brady Eidson.

Associating streams with additional information via a side HashMap does not work,
because the stream we can see is an internal one, with address and other information
not matching a stream that we created.

It's surprising that this design issue was not causing major trouble without NetworkProcess.

  • platform/network/cf/FormDataStreamCFNet.cpp: (WebCore): Renamed FormContext to FormCreationContext, because this type only used for creation. Later, we use FormStreamFields as context. (WebCore::closeCurrentStream): Moved a star. (WebCore::formCreate): We no longer have a side map to keep updated. (WebCore::formFinalize): Ditto. (WebCore::formCopyProperty): Added an accessor to get FormData pointer from a stream. (WebCore::setHTTPBody): Ditto. (WebCore::httpBodyFromStream): Use the new property to get FormData pointer.
4:25 PM Changeset in webkit [138675] by ggaren@apple.com
  • 3 edits in trunk/Source/JavaScriptCore

Some renaming in the CodeCache
https://bugs.webkit.org/show_bug.cgi?id=105966

Reviewed by Gavin Barraclough.

CodeBlockKey => SourceCodeKey because the key is not a CodeBlock.

m_recentlyUsedFunctionCode => m_recentlyUsedFunctions to match other names.

GlobalFunctionKey => FunctionKey because the key is not unique to globalness.

m_cachedGlobalFunctions => m_globalFunctions because "cached" is redundant
for data members in an object called "CodeCache".

kMaxRootCodeBlockEntries => kMaxRootEntries because there are no non-CodeBlock
entries in a CodeBlock cache.

kMaxFunctionCodeBlocks => kMaxChildFunctionEntries to clarify that this
number models a parent-child relationship.

Also removed the initial "k" from enum constants. That's an interesting
style for calling out constants, but it's not the WebKit style.

Finally, a behavior change: Use MaxRootEntries for the limit on global
functions, and not MaxChildFunctionEntries. Previously, there was an
unused constant that seemed to have been intended for this purpose.

  • runtime/CodeCache.cpp:

(JSC::CodeCache::makeSourceCodeKey):
(JSC::CodeCache::getCodeBlock):
(JSC::CodeCache::generateFunctionCodeBlock):
(JSC::CodeCache::makeFunctionKey):
(JSC::CodeCache::getFunctionExecutableFromGlobalCode):
(JSC::CodeCache::usedFunctionCode):

  • runtime/CodeCache.h:

(JSC::CodeCache::clear):

4:21 PM Changeset in webkit [138674] by esprehn@chromium.org
  • 17 edits in trunk/Source/WebCore

Clean up dispatchEvent overrides and overloads
https://bugs.webkit.org/show_bug.cgi?id=105959

Reviewed by Adam Barth.

Add OVERRIDE to all dispatchEvent overrides and use "using" instead of
reimplementing EventTarget::dispatchEvent in subclasses where having the
virtual override of dispatchEvent hides the overloads in
the superclass.

No new tests, just refactoring.

  • Modules/indexeddb/IDBDatabase.h:

(IDBDatabase):

  • Modules/indexeddb/IDBOpenDBRequest.h:

(IDBOpenDBRequest):

  • Modules/indexeddb/IDBRequest.h:

(IDBRequest):

  • Modules/indexeddb/IDBTransaction.h:

(IDBTransaction):

  • dom/EventDispatchMediator.h:

(FocusEventDispatchMediator):
(BlurEventDispatchMediator):

  • dom/GestureEvent.h:

(GestureEventDispatchMediator):

  • dom/KeyboardEvent.h:

(KeyboardEventDispatchMediator):

  • dom/MouseEvent.h:
  • dom/Node.h:

(Node):

  • dom/UIEvent.h:

(FocusInEventDispatchMediator):
(FocusOutEventDispatchMediator):

  • dom/WheelEvent.h:

(WheelEventDispatchMediator):

  • html/HTMLMediaElement.h:
  • html/track/TextTrackCue.cpp:
  • html/track/TextTrackCue.h:

(TextTrackCue):

  • page/DOMWindow.h:

(DOMWindow):

  • svg/SVGElementInstance.h:

(SVGElementInstance):

4:08 PM Changeset in webkit [138673] by pilgrim@chromium.org
  • 2 edits in trunk/Source/WebKit/chromium

[Chromium] Add Platform/chromium to include_dirs
https://bugs.webkit.org/show_bug.cgi?id=105955

Reviewed by James Robinson.

Currently the headers in WebKit/chromium/public/Web*.h can not
include headers from the new Platform/chromium/ directory with the
"#include <public/WebFoo.h>" syntax, because the include_dirs for
dependent builds (like Chromium-within-WebKit) do not include the
new Platform/chromium/ directory. This patch adds that capability
without managing to break anything else.

  • WebKit.gyp:
3:56 PM Changeset in webkit [138672] by simonjam@chromium.org
  • 2 edits
    1 delete in branches/chromium/1364

Revert 129070

[Chromium] Disable resource load scheduling
https://bugs.webkit.org/show_bug.cgi?id=97131

Reviewed by Nate Chapin.

Source/WebCore:

We'll use Chrome's network stack for scheduling instead.

No new tests.

  • loader/ResourceLoadScheduler.cpp:
  • loader/cache/CachedResourceLoader.cpp:

(WebCore::CachedResourceLoader::preload):

LayoutTests:

The preload scanner loads resources sooner and these tests are affected.

  • platform/chromium/http/tests/css/link-css-disabled-value-with-slow-loading-sheet-in-error-expected.txt: Added.
  • platform/chromium/http/tests/inspector/network/network-initiator-expected.txt: Added.

TBR=simonjam@chromium.org
Review URL: https://codereview.chromium.org/11759004

3:56 PM Changeset in webkit [138671] by kbr@google.com
  • 4 edits in branches/chromium/1364

Merge 138393

Expose ArrayBufferView constructor on DOMWindow
https://bugs.webkit.org/show_bug.cgi?id=105605

Reviewed by Sam Weinig.

Source/WebCore:

Update IDL to track recent spec change exposing ArrayBufferView
constructor on DOMWindow for instanceof checks. There are no
constructors exposed in the Web IDL, however, so calling it via
operator new throws TypeError.

Test (updated): fast/canvas/webgl/array-unit-tests.html

  • html/canvas/ArrayBufferView.idl:

Removed OmitConstructor attribute.

  • page/DOMWindow.idl:

Exposed ArrayBufferView constructor function attribute.

LayoutTests:

Updated test from Khronos repository.

  • fast/canvas/webgl/array-unit-tests-expected.txt:
  • fast/canvas/webgl/array-unit-tests.html:

TBR=kbr@google.com
Review URL: https://codereview.chromium.org/11747006

3:55 PM Changeset in webkit [138670] by dgrogan@chromium.org
  • 5 edits in trunk/Source

IndexedDB: Surface a few more leveldb errors.
https://bugs.webkit.org/show_bug.cgi?id=105670

Reviewed by Tony Chang.

Two calls to old LevelDBTransaction::get slipped through the cracks.

  • Modules/indexeddb/IDBBackingStore.cpp:

(WebCore::IndexKeyCursorImpl::loadCurrentRow):
(WebCore::IndexCursorImpl::loadCurrentRow):

Make these two functions use safeGet instead of get.

  • platform/leveldb/LevelDBTransaction.cpp:
  • platform/leveldb/LevelDBTransaction.h: Remove get now that it is unused.
3:54 PM Changeset in webkit [138669] by fpizlo@apple.com
  • 11 edits in trunk/Source/JavaScriptCore

DFG inlining machinery should be robust against the inline callee varying while the executable stays the same
https://bugs.webkit.org/show_bug.cgi?id=105953

Reviewed by Mark Hahnenberg.

This institutes the policy that if InlineCallFrame::callee is null, then the callee and scope have already
been stored into the true call frame (i.e. the place where the call frame of the inlined call would have
been) and so any attempt to access the callee or scope should do a load instead of assuming that the value
is constant. This wires the changes through the bytecode parser, the stack scanning logic, and the compiler
optimization phases and backends.

  • bytecode/CodeOrigin.cpp:

(JSC::InlineCallFrame::dump):

  • bytecode/CodeOrigin.h:

(CodeOrigin):
(InlineCallFrame):
(JSC::InlineCallFrame::isClosureCall):
(JSC::CodeOrigin::stackOffset):
(JSC):

  • dfg/DFGAssemblyHelpers.h:
  • dfg/DFGByteCodeParser.cpp:

(JSC::DFG::ByteCodeParser::get):
(InlineStackEntry):
(JSC::DFG::ByteCodeParser::getScope):
(JSC::DFG::ByteCodeParser::InlineStackEntry::InlineStackEntry):

  • dfg/DFGCSEPhase.cpp:

(CSEPhase):
(JSC::DFG::CSEPhase::genericPureCSE):
(JSC::DFG::CSEPhase::pureCSE):
(JSC::DFG::CSEPhase::pureCSERequiringSameInlineCallFrame):
(JSC::DFG::CSEPhase::getMyScopeLoadElimination):
(JSC::DFG::CSEPhase::performNodeCSE):

  • dfg/DFGOSRExitCompiler32_64.cpp:

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

  • dfg/DFGOSRExitCompiler64.cpp:

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

  • dfg/DFGSpeculativeJIT32_64.cpp:

(JSC::DFG::SpeculativeJIT::compile):

  • dfg/DFGSpeculativeJIT64.cpp:

(JSC::DFG::SpeculativeJIT::compile):

  • interpreter/CallFrame.cpp:

(JSC::CallFrame::trueCallFrame):

3:54 PM Changeset in webkit [138668] by commit-queue@webkit.org
  • 4 edits
    2 adds in trunk

When a block's height is determined by min-height/max-height, children with percentage heights are sized incorrectly
https://bugs.webkit.org/show_bug.cgi?id=26559

Patch by Bem Jones-Bey <Bem Jones-Bey> on 2013-01-02
Reviewed by Tony Chang.

Source/WebCore:

Percentage logical height computation now takes into account the min and max height of the container.

Test: fast/block/min-max-height-percent-height-child.html

  • rendering/RenderBox.cpp:

(WebCore::RenderBox::constrainContentBoxLogicalHeightByMinMax): Added. Like constrainLogicalHeightByMinMax,

but constrains the content box instead of the border box.

(WebCore):
(WebCore::RenderBox::computeContentLogicalHeight): Make const so it can be called from the const method

constrainContentBoxLogicalHeightByMinMax.

(WebCore::RenderBox::computePercentageLogicalHeight): Use constrainContentBoxLogicalHeightByMinMax to properly

respect min and max height on the containing box when computing the percentage height.

  • rendering/RenderBox.h:

(RenderBox):

LayoutTests:

Add tests to ensure percent heights are correctly determined.

  • fast/block/min-max-height-percent-height-child-expected.txt: Added.
  • fast/block/min-max-height-percent-height-child.html: Added.
3:51 PM Changeset in webkit [138667] by tony@chromium.org
  • 2 edits in trunk/Source/WebCore

Unreviewed, force InternalSettings.cpp to rebuild to try and fix the Apple Win build.

  • testing/InternalSettings.cpp:
3:48 PM Changeset in webkit [138666] by alecflett@chromium.org
  • 11 edits in trunk/Source

IndexedDB: Use non-const buffers in put() to avoid copies
https://bugs.webkit.org/show_bug.cgi?id=105572

Reviewed by Adam Barth.

Source/WebCore:

Change the new put() method to allow the implementation to consume
or adopt the vector, to avoid copying.

No new tests as this is an interface change that will be implemented later.

  • Modules/indexeddb/IDBDatabaseBackendImpl.cpp:

(WebCore::IDBDatabaseBackendImpl::put):

  • Modules/indexeddb/IDBDatabaseBackendImpl.h:

(IDBDatabaseBackendImpl):

  • Modules/indexeddb/IDBDatabaseBackendInterface.h:

Source/WebKit/chromium:

Pass on non-const Vectors to allow buffer adoption rather than copies.

  • public/WebIDBDatabase.h:

(WebIDBDatabase):
(WebKit::WebIDBDatabase::put):

  • src/IDBDatabaseBackendProxy.cpp:

(WebKit::IDBDatabaseBackendProxy::put):

  • src/IDBDatabaseBackendProxy.h:

(IDBDatabaseBackendProxy):

  • src/WebIDBDatabaseImpl.cpp:

(WebKit::WebIDBDatabaseImpl::put):

  • src/WebIDBDatabaseImpl.h:

(WebIDBDatabaseImpl):

  • tests/IDBDatabaseBackendTest.cpp:
3:41 PM Changeset in webkit [138665] by abarth@webkit.org
  • 73 edits in trunk/Source/WebCore

[V8] Remove INC_STATS because it is unused
https://bugs.webkit.org/show_bug.cgi?id=105957

Reviewed by Ojan Vafai.

We haven't been able to compile with INC_STATS turned on in a long
time. This patch just removes this ancient cruft. I generated this
patch using

find . -type f | xargs sed -i '/INC_STATS/d'

and then fixing up V8BindingMacros.h.

3:34 PM Changeset in webkit [138664] by barraclough@apple.com
  • 15 edits in trunk/Source/JavaScriptCore

Objective-C API for JavaScriptCore
https://bugs.webkit.org/show_bug.cgi?id=105889

Reviewed by Geoff Garen.

Fixes for a number of issues raised by Darin.

  • API/APIJSValue.h:
    • Fix typos in comment
    • Add newline before NS_CLASS_AVAILABLE(10_9, NA)
    • cls -> expectedClass
    • key type for -setObject:forKeyedSubscript: is now NSObject <NSCopying> *
  • API/JSBase.h:
    • JS_OBJC_API_ENABLED no longer implies OBJC
  • API/JSBlockAdaptor.mm:

(BlockArgumentStruct::BlockArgumentStruct):
(BlockArgumentStruct):

  • mark virtual functions as virtual, override, and private
  • refactor out buffer allocation for struct types

(BlockArgumentTypeDelegate::typeVoid):
(BlockArgumentTypeDelegate::typeBlock):
(BlockArgumentTypeDelegate::typeStruct):

  • return nil -> return 0

(BlockResultStruct::BlockResultStruct):
(BlockResultStruct):

  • mark virtual functions as virtual, override, and private
  • refactor out buffer allocation for struct types

(buildBlockSignature):

  • %lu is not an appropriate format specifier for NSInteger

(-[JSBlockAdaptor initWithBlockSignatureFromProtocol:]):

  • nil check [super init]

(-[JSBlockAdaptor blockMatchesSignature:]):
(-[JSBlockAdaptor blockFromValue:inContext:withException:]):

  • ctx -> contextRef
  • API/JSContext.h:
    • Fix typos in comment
    • Add newline before NS_CLASS_AVAILABLE(10_9, NA)
    • key type for -setObject:forKeyedSubscript: is now NSObject <NSCopying> *
  • API/JSContext.mm:

(-[JSContext initWithVirtualMachine:]):

  • nil check [super init]

(+[JSContext currentArguments]):

  • args -> argumentArray

(-[JSContext setObject:forKeyedSubscript:]):

  • key type for -setObject:forKeyedSubscript: is now NSObject <NSCopying> *

(-[JSContext dealloc]):
(-[JSContext protect:]):
(-[JSContext unprotect:]):

  • m_protected -> m_protectCounts
  • API/JSValue.mm:

(-[JSValue toObjectOfClass:]):

  • cls -> expectedClass

(-[JSValue toBool]):
(-[JSValue deleteProperty:]):
(-[JSValue hasProperty:]):
(-[JSValue isUndefined]):
(-[JSValue isNull]):
(-[JSValue isBoolean]):
(-[JSValue isNumber]):
(-[JSValue isString]):
(-[JSValue isObject]):
(-[JSValue isEqualToObject:]):
(-[JSValue isEqualWithTypeCoercionToObject:]):
(-[JSValue isInstanceOf:]):

  • removed ? YES : NO

(-[JSValue callWithArguments:]):
(-[JSValue constructWithArguments:]):
(-[JSValue invokeMethod:withArguments:]):

  • args -> argumentArray

(+[JSValue valueWithPoint:inContext:]):
(+[JSValue valueWithRange:inContext:]):
(+[JSValue valueWithRect:inContext:]):
(+[JSValue valueWithSize:inContext:]):

  • [NSNumber numberWithFloat:] -> @()

(-[JSValue objectForKeyedSubscript:]):
(-[JSValue setObject:forKeyedSubscript:]):

  • key type for -setObject:forKeyedSubscript: is now NSObject <NSCopying> *

(JSContainerConvertor):
(JSContainerConvertor::isWorkListEmpty):
(JSContainerConvertor::convert):
(ObjcContainerConvertor):
(ObjcContainerConvertor::isWorkListEmpty):

  • remove WTF::
  • isWorkListEmpty is const

(objectToValue):

  • use fast enumeration

(-[JSValue initWithValue:inContext:]):

  • nil check [super init]

(getStructTagHandler):

  • m_structHandlers -> structHandlers
  • API/JSVirtualMachine.h:
    • Add newline before NS_CLASS_AVAILABLE(10_9, NA)
  • API/JSVirtualMachine.mm:

(-[JSVirtualMachine init]):

  • nil check [super init]
  • API/JSWrapperMap.mm:

(selectorToPropertyName):
(copyPrototypeProperties):

  • remove WTF::
  • use static_cast

(-[JSObjCClassInfo initWithContext:forClass:superClassInfo:]):
(-[JSWrapperMap initWithContext:]):

  • nil check [super init]

(-[JSWrapperMap wrapperForObject:]):
(tryUnwrapObjcObject):

  • enable ASSERT

(getJSExportProtocol):
(getNSBlockClass):

  • remove if check on initializing static
  • API/JavaScriptCore.h:
    • JS_OBJC_API_ENABLED no longer implies OBJC
  • API/ObjCCallbackFunction.mm:

(CallbackArgumentOfClass):
(CallbackArgumentOfClass::~CallbackArgumentOfClass):
(CallbackArgumentStruct::CallbackArgumentStruct):
(CallbackArgumentStruct):
(CallbackArgumentBlockCallback):

  • mark virtual functions as virtual, override, and private
  • refactor out buffer allocation for struct types

(ArgumentTypeDelegate::typeVoid):
(ArgumentTypeDelegate::typeOfClass):
(ArgumentTypeDelegate::typeStruct):

  • return nil -> return 0

(CallbackResultStruct::CallbackResultStruct):
(CallbackResultStruct):

  • mark virtual functions as virtual, override, and private
  • refactor out buffer allocation for struct types

(ResultTypeDelegate::typeStruct):

  • return nil -> return 0

(ObjCCallbackFunction):

  • remove WTF::

(objCCallbackFunctionFinalize):

  • use static_cast

(objCCallbackFunctionCallAsFunction):

  • Fix typos in comment

(createObjCCallbackFunctionClass):
(objCCallbackFunctionClass):

  • Split out createObjCCallbackFunctionClass from objCCallbackFunctionClass

(ObjCCallbackFunction::call):

  • ctx -> contextRef

(blockSignatureContainsClass):

  • Remove tri-state enum.

(skipNumber):

  • isdigit -> isASCIIDigit

(objCCallbackFunctionForInvocation):

  • clean up & comment blockSignatureContainsClass() usage

(tryUnwrapBlock):

  • use static_cast
  • API/ObjcRuntimeExtras.h:

(forEachProtocolImplementingProtocol):
(forEachMethodInClass):
(forEachMethodInProtocol):
(forEachPropertyInProtocol):

  • Remove WTF::
  • Remove if (count) checks

(skipPair):

  • NSUInteger -> size_t

(StringRange):
(StringRange::operator const char*):
(StringRange::get):
(StructBuffer):
(StructBuffer::StructBuffer):
(StructBuffer::~StructBuffer):
(StructBuffer::operator void*):

  • Added helper for creating an aligned buffer, used by struct conversion invocations.

(parseObjCType):

  • *(position++) -> *position++
  • API/tests/testapi.c:
    • PLATFORM(MAC) -> JS_OBJC_API_ENABLED
  • API/tests/testapi.m:

(blockSignatureContainsClass):

  • Remove tri-state enum.

(testObjectiveCAPI):

  • Added more result type checks.
3:32 PM Changeset in webkit [138663] by rniwa@webkit.org
  • 2 edits in trunk/LayoutTests

Add a failing test expectation to a test added by r136697.
The failure is tracked by the bug 105960.

  • platform/mac-wk2/TestExpectations:
3:23 PM Changeset in webkit [138662] by rniwa@webkit.org
  • 2 edits in trunk/LayoutTests

Add a failing test expectations to a test added by r132411.
The patch updated DumpRenderTree but didn't update WebKitTestRunner.
The failure is tracked by the bug 105958.

  • platform/mac-wk2/TestExpectations:
3:15 PM Changeset in webkit [138661] by tony@chromium.org
  • 23 edits in trunk

Generate internal.settings from Settings.in
https://bugs.webkit.org/show_bug.cgi?id=104740

Reviewed by Adam Barth.

.:

  • Source/cmake/WebKitMacros.cmake: Specify additional output files from make_settings.pl.

Source/WebCore:

If the setting is declared in Settings.in and is a basic type, generate a
setter function at internals.settings.

InternalSettingsGenerated.idl is created by make_settings.pl. InternalSettings.idl
inherits from InternalSettingsGenerated.idl so we can override generated functions if
needed or add additional custom setters.

This patch removes 2 password echo methods from InternalSettings.idl since they are
auto generated. There are other methods we could remove, but I'll do that in a followup.

No new tests, existing tests should pass. Specifically, editing/input/password* tests should pass.

  • CMakeLists.txt: Add generated cpp file to the testing lib.
  • DerivedSources.make: Add generated idl file and specify all the outputs of make_settings.pl
  • DerivedSources.pri: Generate idl file before generating bindings and include generated idl

for bindings generation.

  • GNUmakefile.am: Specify all the generated files of make_settings.pl. Add generated idl to

list of files to be processed and resulting output to be compiled.

  • UseJSC.cmake: Add generated idl file to list of idl files to process.
  • UseV8.cmake: Same as UseJSC.cmake.
  • WebCore.gyp/WebCore.gyp: Run make_settings.pl earlier so we can run the output through the

bindings generator.

  • WebCore.gypi: Add new files.
  • WebCore.vcproj/WebCoreTestSupport.vcproj: Add new files.
  • WebCore.xcodeproj/project.pbxproj: Add new files.
  • bindings/scripts/CodeGenerator.pm:

(IDLFileForInterface): Add current directory to the include path so DerivedSources.make can
find the generated idl file.

  • page/make_settings.pl:

(generateCode): Generate InternalSettingsGenerated.{idl,h,cpp}.
(generateSettingsMacrosHeader): Rename since we're generating 2 headers now.
(setterFunctionName): Helper function for naming a setter.
(printGetterAndSetter): Use setterFunctionName.
(enumerateParsedItems): Helper function that visits each parsed item.
(generateInternalSettingsIdlFile): Generate idl file.
(generateInternalSettingsHeaderFile): Generate header file.
(generateInternalSettingsCppFile): Generate C++ file.

  • testing/InternalSettings.cpp:

(WebCore::InternalSettings::Backup::Backup): Remove password echo members.
(WebCore::InternalSettings::Backup::restoreTo): Remove password echo members.
(InternalSettingsWrapper): We can't use RefCountedSupplemental because we're we're already
RefCounted. Create a wrapper class to wrap the RefCounted InternalSettings for Supplement<>::provideTo,
which uses an OwnPtr.
(WebCore::InternalSettingsWrapper::InternalSettingsWrapper): Wraps an InternalSettings that the Page can own.
(WebCore::InternalSettingsWrapper::~InternalSettingsWrapper): Clear's m_page when the Page goes away.
(WebCore::InternalSettingsWrapper::internalSettings): Unwrap InternalSettings.
(WebCore::InternalSettings::from): Update to use InternalSettingsWrapper.
(WebCore::InternalSettings::InternalSettings): Call base class constructor.
(WebCore::InternalSettings::resetToConsistentState): Renamed from reset() and pass through to base class.
Also remove unnecessary pageScaleFactor reset (done in Internals::resetToConsistentState).

  • testing/InternalSettings.h:

(Backup): Remove password echo members.
(WebCore::InternalSettings::create): Create an instance instead of using new.
(WebCore::InternalSettings::hostDestroyed): Clear the page. This is similar to the code that
RefCountedSupplemental has.

  • testing/InternalSettings.idl: Extends InternalSettingsGenerated.
  • testing/js/WebCoreTestSupport.cpp:

(WebCoreTestSupport::resetInternalsObject): Rename to resetToConsistentState to be like other code.

  • testing/v8/WebCoreTestSupport.cpp:

(WebCoreTestSupport::resetInternalsObject): Rename to resetToConsistentState to be like other code.
Remove an unused header.

Tools:

  • GNUmakefile.am: Compile new generated files on GTK+.
  • qmake/mkspecs/features/default_post.prf: Qt: Make it so an action

with both extra_sources and add_output_to_sources=false will still
compile the files in extra_sources. This makes it possible to compile
InternalSettingsGenerated.cpp and not be confused by
InternalSettingsGenerated.idl.

3:15 PM Changeset in webkit [138660] by rniwa@webkit.org
  • 2 edits in trunk/LayoutTests

Add a Mac WebKit2 test expectation for the bug 105954.

  • platform/mac-wk2/TestExpectations:
3:06 PM Changeset in webkit [138659] by rniwa@webkit.org
  • 2 edits in trunk/LayoutTests

Add a failing test expectation for the bug 105954.

  • platform/mac-wk2/TestExpectations:
3:06 PM Changeset in webkit [138658] by abarth@webkit.org
  • 7 edits in trunk/Source/WebCore

HTML preload scanner doesn't need to remember whether we're scanning the body
https://bugs.webkit.org/show_bug.cgi?id=105945

Reviewed by Eric Seidel.

As far as I can tell, no one uses this state.

  • html/parser/CSSPreloadScanner.cpp:

(WebCore::CSSPreloadScanner::CSSPreloadScanner):
(WebCore::CSSPreloadScanner::scan):
(WebCore::CSSPreloadScanner::emitRule):

  • html/parser/CSSPreloadScanner.h:

(CSSPreloadScanner):

  • html/parser/HTMLPreloadScanner.cpp:

(WebCore::PreloadTask::preload):
(WebCore::HTMLPreloadScanner::HTMLPreloadScanner):
(WebCore::HTMLPreloadScanner::processToken):

  • html/parser/HTMLPreloadScanner.h:

(HTMLPreloadScanner):

  • loader/cache/CachedResourceLoader.cpp:

(WebCore::CachedResourceLoader::preload):

  • loader/cache/CachedResourceLoader.h:

(CachedResourceLoader):

3:03 PM Changeset in webkit [138657] by inferno@chromium.org
  • 3 edits in trunk/Source/WebCore

Crash in WebCore::Element::cloneElementWithoutChildren.
https://bugs.webkit.org/show_bug.cgi?id=105949

Reviewed by Ryosuke Niwa.

RefPtr |ancestors| vector since its elements can be destroyed from mutation events
fired in CompositeEditCommand::appendNode.

No new tests. The testcase relies on recursive DOM mutations and does not minimize.

  • editing/InsertParagraphSeparatorCommand.cpp:

(WebCore::InsertParagraphSeparatorCommand::getAncestorsInsideBlock):
(WebCore::InsertParagraphSeparatorCommand::cloneHierarchyUnderNewBlock):
(WebCore::InsertParagraphSeparatorCommand::doApply):

  • editing/InsertParagraphSeparatorCommand.h:

(InsertParagraphSeparatorCommand):

3:00 PM Changeset in webkit [138656] by roger_fong@apple.com
  • 1 edit
    2 adds in trunk/Source/WebKit/win

Unreviewed. Build fix after reintroducing r138331.

Attempted to remove WebKit2 from Apple Windows port in r138331 but two vsprops files were missing.

  • WebKit.vcproj/WebKitApple.vsprops: Added.
  • WebKit.vcproj/WebKitDirectX.vsprops: Added.
3:00 PM Changeset in webkit [138655] by weinig@apple.com
  • 5 edits
    2 adds in trunk/Source/WebKit2

Factor out NetworkProcess initialization into its own function in preparation of adding a NetworkProcess service
https://bugs.webkit.org/show_bug.cgi?id=105946

Reviewed by Brady Eidson.

  • NetworkProcess/NetworkProcess.h:
  • NetworkProcess/mac/NetworkProcessInitialization.h: Added.

(NetworkProcessInitializationParameters):

  • NetworkProcess/mac/NetworkProcessInitialization.mm: Added.

(WebKit::initializeNetworkProcess):

  • NetworkProcess/mac/NetworkProcessMac.mm:

(WebKit::NetworkProcess::initializeSandbox):
(WebKit::NetworkProcess::platformInitialize):

  • NetworkProcess/mac/NetworkProcessMainMac.mm:

(WebKit::NetworkProcessMain):

  • WebKit2.xcodeproj/project.pbxproj:
2:52 PM Changeset in webkit [138654] by commit-queue@webkit.org
  • 8 edits
    1 move
    14 adds
    3 deletes in trunk

Crash in WebCore::InlineBox::deleteLine
https://bugs.webkit.org/show_bug.cgi?id=93448

Patch by Douglas Stockwell <dstockwell@chromium.org> on 2013-01-02
Reviewed by Eric Seidel.

Source/WebCore:

When we ran off the end of the line while looking for line breaks in an
inline with white-space:nowrap nested in a block with white-space:pre
it was possible for the line break to be set at or before the current
position -- this could result in duplications in the render tree or
infinite looping.

This patch changes the "fixup" logic that runs after we have finished
iterating through elements and text and have potentially found a break
point. In the case of a block setting white-space:pre we would back up
a character in some cases. Not doing so could leave whitespace that
should have been collapsed at the end of an inline.

For example in '<span style="white-space:nowrap">x_</span>_y' if a
break was inserted before 'y' the space after 'x' would still be
rendered (rather than be collapsed with the break).

To avoid this problem we will not take the opportunity to break until
we have finished collapsing whitespace.

Tests: fast/text/whitespace/inline-whitespace-wrapping-1.html

fast/text/whitespace/inline-whitespace-wrapping-2.html
fast/text/whitespace/inline-whitespace-wrapping-3.html
fast/text/whitespace/inline-whitespace-wrapping-4.html
fast/text/whitespace/nowrap-white-space-collapse.html
fast/text/whitespace/pre-block-normal-inline-crash-1.html
fast/text/whitespace/pre-block-normal-inline-crash-2.html

  • rendering/RenderBlockLineLayout.cpp:

(WebCore::RenderBlock::LineBreaker::nextLineBreak): Collapse
whitespace before breaking. Avoid setting the break before the current
position.

LayoutTests:

  • editing/deleting/delete-block-table-expected.txt:
  • editing/execCommand/selectAll-expected.txt: Renamed from LayoutTests/platform/chromium/editing/execCommand/selectAll-expected.txt.
  • fast/text/whitespace/inline-whitespace-wrapping-1-expected.html: Added.
  • fast/text/whitespace/inline-whitespace-wrapping-1.html: Added.
  • fast/text/whitespace/inline-whitespace-wrapping-2-expected.html: Added.
  • fast/text/whitespace/inline-whitespace-wrapping-2.html: Added.
  • fast/text/whitespace/inline-whitespace-wrapping-3-expected.html: Added.
  • fast/text/whitespace/inline-whitespace-wrapping-3.html: Added.
  • fast/text/whitespace/inline-whitespace-wrapping-4-expected.html: Added.
  • fast/text/whitespace/inline-whitespace-wrapping-4.html: Added.
  • fast/text/whitespace/nowrap-white-space-collapse-expected.html: Added.
  • fast/text/whitespace/nowrap-white-space-collapse.html: Added.
  • fast/text/whitespace/pre-block-normal-inline-crash-1-expected.txt: Added.
  • fast/text/whitespace/pre-block-normal-inline-crash-1.html: Added.
  • fast/text/whitespace/pre-block-normal-inline-crash-2-expected.txt: Added.
  • fast/text/whitespace/pre-block-normal-inline-crash-2.html: Added.
  • platform/chromium-win/editing/deleting/delete-to-select-table-expected.txt:
  • platform/chromium-win/editing/execCommand/print-expected.txt:
  • platform/chromium-win/editing/execCommand/selectAll-expected.txt:
  • platform/chromium-win/editing/inserting/editable-html-element-expected.txt:
  • platform/efl/editing/execCommand/selectAll-expected.txt: Removed.
  • platform/gtk/editing/execCommand/selectAll-expected.txt: Removed.
  • platform/mac/editing/execCommand/selectAll-expected.txt: Removed.
2:45 PM Changeset in webkit [138653] by ojan@chromium.org
  • 4 edits in trunk/Websites/bugs.webkit.org

REGRESSION: Review tool sometimes doesn't include some comments in preview & posts
https://bugs.webkit.org/show_bug.cgi?id=105252

Reviewed by Tony Chang.

When adding context, the LineContainer for the context line can get removed.
In that case, forEachLine needs to know to keep looping past that line number.

Also, make it so that you can't leave comments on context lines.

  • code-review-test.html:
  • code-review.js:
2:41 PM Changeset in webkit [138652] by yoli@rim.com
  • 2 edits in trunk/Source/WebKit/blackberry

[BlackBerry] Cached frame shouldn't access the page when being destroyed
https://bugs.webkit.org/show_bug.cgi?id=105942

Reviewed by George Staikos.

In order to prevent our FrameLoaderClient from dangling WebPagePrivate object in any case,
this patches clear the m_webPagePrivate pointer when the frame is detached or cached.

RIM PR# 272137.

  • WebCoreSupport/FrameLoaderClientBlackBerry.cpp:

(WebCore::FrameLoaderClientBlackBerry::isMainFrame):
(WebCore::FrameLoaderClientBlackBerry::detachedFromParent2):
(WebCore::FrameLoaderClientBlackBerry::didSaveToPageCache):
(WebCore::FrameLoaderClientBlackBerry::didRestoreFromPageCache):

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

DFG should not use the InlineCallFrame's callee when it could have used the executable istead
https://bugs.webkit.org/show_bug.cgi?id=105947

Reviewed by Mark Hahnenberg.

We shouldn't use the callee to get the executable when we have the executable already. Not only
does this make the logic more clear, but it also allows for a world where the executable is known
but the callee isn't.

  • dfg/DFGAssemblyHelpers.h:

(JSC::DFG::AssemblyHelpers::strictModeFor):

2:34 PM Changeset in webkit [138650] by rniwa@webkit.org
  • 2 edits in trunk/LayoutTests

Add a test expectation for the bug 105952.

  • platform/wk2/TestExpectations:
2:27 PM Changeset in webkit [138649] by commit-queue@webkit.org
  • 10 edits in trunk/Source

Unreviewed, rolling out r138403.
http://trac.webkit.org/changeset/138403
https://bugs.webkit.org/show_bug.cgi?id=105950

Fixing e138331 which was rolled out in r138403 (Requested by
rfong on #webkit).

Patch by Sheriff Bot <webkit.review.bot@gmail.com> on 2013-01-02

Source/WebKit/win:

  • WebKit.vcproj/WebKit.sln:
  • WebKit.vcproj/WebKit.vcproj:
  • WebKit.vcproj/WebKitLibCommon.vsprops:
  • WebKit.vcproj/WebKitLibDebug.vsprops:
  • WebKit.vcproj/WebKitLibDebugAll.vsprops:
  • WebKit.vcproj/WebKitLibProduction.vsprops:
  • WebKit.vcproj/WebKitLibRelease.vsprops:

Source/WebKit2:

  • win/WebKit2.def.in:
2:23 PM Changeset in webkit [138648] by rniwa@webkit.org
  • 2 edits in trunk/LayoutTests

Add a failing test expectation to a test added by r138380 on Mac WK2.
The failure is tracked by the bug 105948.

  • platform/mac-wk2/TestExpectations:
2:14 PM Changeset in webkit [138647] by scherkus@chromium.org
  • 1 edit in branches/chromium/1364/LayoutTests/platform/chromium/TestExpectations

Merge 138289

[Chromium] Restore WebM EME canPlayType() expectations.
https://bugs.webkit.org/show_bug.cgi?id=105070

Unreviewed TestExpectations revert of r137225.

Patch by David Dorwin <ddorwin@chromium.org> on 2012-12-20

  • platform/chromium/TestExpectations:

TBR=commit-queue@webkit.org
Review URL: https://codereview.chromium.org/11748009

2:14 PM Changeset in webkit [138646] by abarth@webkit.org
  • 13 edits in trunk/Source/WebCore

Remove webkitPostMessage
https://bugs.webkit.org/show_bug.cgi?id=105910

Reviewed by Kentaro Hara.

The vendor-prefixed version of postMessage was never widely used.
We've been compiling it out of the Chromium port for a while now and
haven't received any reports of compatibility problems. I've done a
number of web searches for uses of the API and personally contacted a
handful of authors I found using the API, all of whom have now moved to
the identical unprefixed API. We've publicied the prefix removal in a
blog post:

http://blog.chromium.org/2012/11/a-web-developers-guide-to-latest-chrome.html

and discussed it on webkit-dev:

http://lists.macosforge.org/pipermail/webkit-dev/2012-September/022189.html

We've done our diligence, and now we should remove the API from the
remainder of the ports.

  • bindings/js/JSDOMWindowCustom.cpp:
  • bindings/js/JSDedicatedWorkerContextCustom.cpp:
  • bindings/js/JSMessagePortCustom.cpp:
  • bindings/js/JSWorkerCustom.cpp:
  • bindings/v8/custom/V8DOMWindowCustom.cpp:

(WebCore::V8DOMWindow::postMessageCallback):

  • bindings/v8/custom/V8DedicatedWorkerContextCustom.cpp:

(WebCore::V8DedicatedWorkerContext::postMessageCallback):

  • bindings/v8/custom/V8MessagePortCustom.cpp:

(WebCore::V8MessagePort::postMessageCallback):

  • bindings/v8/custom/V8WorkerCustom.cpp:

(WebCore::V8Worker::postMessageCallback):

  • dom/MessagePort.idl:
  • page/DOMWindow.idl:
  • workers/DedicatedWorkerContext.idl:
  • workers/Worker.idl:
2:11 PM Changeset in webkit [138645] by tsepez@chromium.org
  • 2 edits
    2 adds in trunk/LayoutTests

X-XSS-Protection report-uri doesn't work with mode=block
https://bugs.webkit.org/show_bug.cgi?id=105907

Reviewed by Adam Barth.

Appears to work as-is, but the test was missing for this case. Checking
for completion is complicated by the frame navigating to about:blank, rather
than the echo-report page, so check it from the parent frame.

  • http/tests/security/xssAuditor/report-script-tag-full-block-expected.txt: Added.
  • http/tests/security/xssAuditor/report-script-tag-full-block.html: Added.
  • http/tests/security/xssAuditor/resources/echo-intertag.pl:
2:06 PM Changeset in webkit [138644] by tdanderson@chromium.org
  • 2 edits in trunk/LayoutTests

[chromium] Mark webaudio/automatic-pull-node.html as Timeout for XP
https://bugs.webkit.org/show_bug.cgi?id=105941

Unreviewed gardening.

  • platform/chromium/TestExpectations:
2:03 PM Changeset in webkit [138643] by rniwa@webkit.org
  • 4 edits in trunk/LayoutTests

Rebaseline tests after r137798 and update test expectations per bug 105943.

  • platform/mac-wk2/TestExpectations:
  • platform/mac-wk2/compositing/rtl/rtl-fixed-expected.txt:
  • platform/mac-wk2/compositing/rtl/rtl-fixed-overflow-expected.txt:
1:33 PM Changeset in webkit [138642] by adamk@chromium.org
  • 1 edit in branches/chromium/1364/Source/WebKit/chromium/features.gypi

Turn off ENABLE_TEMPLATE_ELEMENT
Review URL: https://codereview.chromium.org/11635022

1:28 PM Changeset in webkit [138641] by fpizlo@apple.com
  • 2 edits in trunk/Source/JavaScriptCore

DFG inliner should not use the callee's bytecode variable for resolving references to the callee in inlined code
https://bugs.webkit.org/show_bug.cgi?id=105938

Reviewed by Mark Hahnenberg.

This simplifies a bunch of code for referring to the callee. It also ought to simplify how we do
closure call inlining: for inlined closure call frames we will simply require that the callee is
already stashed on the stack in the Callee slot in the inline call frame header.

  • dfg/DFGByteCodeParser.cpp:

(ByteCodeParser):
(JSC::DFG::ByteCodeParser::getDirect):
(JSC::DFG::ByteCodeParser::get):
(InlineStackEntry):
(JSC::DFG::ByteCodeParser::InlineStackEntry::remapOperand):
(JSC::DFG::ByteCodeParser::handleCall):
(JSC::DFG::ByteCodeParser::handleInlining):
(JSC::DFG::ByteCodeParser::InlineStackEntry::InlineStackEntry):
(JSC::DFG::ByteCodeParser::parse):

1:19 PM Changeset in webkit [138640] by benjamin@webkit.org
  • 3 edits in trunk/Source/WebCore

Optimize TransformationMatrix::multiply() for x86_64
https://bugs.webkit.org/show_bug.cgi?id=105719

Reviewed by Sam Weinig.

On x86_64, we have access to 16 XMM registers. This can hold 32 double values.
We can use that in two ways to optimize matrix multiplications:
-Keep the source matrix completely in registers. Write the result directly in

the source matrix's memory. This avoids the memcpy at the end of the multiplication
and various memory operations.

-Use SIMD with SSE to perform 2 operations at a time.

The parameter from the second matrix are loaded one by one in XMM registers.
Loading them with SSE then shuffling the values perform worse than loading
one by one.

This is only enabled on 64bits as x86 only has access to 8 XMM registers and
the function should be written differently.

On a i5, TransformationMatrix::multiply() perform about 3 times faster with the change.

  • platform/graphics/transforms/TransformationMatrix.cpp:

(WebCore::TransformationMatrix::multiply):

  • platform/graphics/transforms/TransformationMatrix.h:

(TransformationMatrix): Fix an incorrect comment. Unify the comment with the cpp file.

1:13 PM Changeset in webkit [138639] by mkwst@chromium.org
  • 2 edits in trunk/Source/WebCore

Clean up the loadXXXStyle() idiom in StyleResolver.
https://bugs.webkit.org/show_bug.cgi?id=105903

Reviewed by Darin Adler.

This patch refactors 'loadViewSourceStyle' in order to avoid the
necessity of each call site checking whether the relevant data has
been loaded. The new 'viewSourceStyle' will load the data if required,
and return a pointer which can be used directly.

No new tests, as this refactoring should exhibit no visible change in
behavior.

  • css/StyleResolver.cpp:

(WebCore::viewSourceStyle):

Lazily populate the static 'defaultViewSourceStyle' with data. I
haven't removed the outer variable, as it's used directly in
StyleResolver::reportMemoryUsage.

(WebCore::StyleResolver::matchUARules):
(WebCore::StyleResolver::collectFeatures):

Use the new hotness rather than the old, lame method.

12:49 PM Changeset in webkit [138638] by rniwa@webkit.org
  • 2 edits in trunk/Source/WebKit2

Yet another Windows build fix attempt. Try exporting more symbols.

  • win/WebKit2.def.in:
12:48 PM Changeset in webkit [138637] by esprehn@chromium.org
  • 1 edit
    2 adds in trunk/LayoutTests

View source mode is missing the processing instruction line.
https://bugs.webkit.org/show_bug.cgi?id=15961

Reviewed by Adam Barth.

Add a test for XML documents shown in viewsource mode. The bug
with the missing processing instruction line was fixed long
ago with the HTML5 parser rewrite so this just adds a test.

  • fast/frames/viewsource-xml-expected.txt: Added.
  • fast/frames/viewsource-xml.html: Added.
12:40 PM Changeset in webkit [138636] by rniwa@webkit.org
  • 2 edits in trunk/Source/JavaScriptCore

Another Windows port build fix attempt. Try not exporting this symbol from JSC
since it's also compiled in WebCore.

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

REGRESSION (r138464): Safari hangs when opening Privacy preferences
https://bugs.webkit.org/show_bug.cgi?id=105933

Reviewed by Sam Weinig.

Quick and dirty fix - added a special case for main thread. But I'm not sure if
dispatch_sync on main queue is really the best idiom here.

  • Shared/Plugins/Netscape/mac/NetscapePluginModuleMac.mm:

(WebKit::getPluginInfoFromCarbonResources):

12:37 PM Changeset in webkit [138634] by abarth@webkit.org
  • 8 edits in trunk/Tools

Remove old-run-webkit-tests support from webkitpy
https://bugs.webkit.org/show_bug.cgi?id=105619

Reviewed by Eric Seidel.

This patch removes support for webkitpy parsing the
old-run-webkit-tests results format. There doesn't seem to be any
reason to keep this code around anymore now that most bots have
switched to new-run-webkit-tests.

  • Scripts/webkitpy/common/net/buildbot/buildbot.py:

(Builder.fetch_layout_test_results):

  • Scripts/webkitpy/common/net/layouttestresults.py:

(path_for_layout_test):
(LayoutTestResults.results_from_string):
(LayoutTestResults):

  • Scripts/webkitpy/common/net/layouttestresults_unittest.py:

(LayoutTestResultsTest.test_results_from_string):

  • Scripts/webkitpy/tool/bot/commitqueuetask_unittest.py:

(test_flaky_test_failure):

  • Scripts/webkitpy/tool/bot/layouttestresultsreader_unittest.py:

(LayoutTestResultsReaderTest.test_missing_layout_test_results):

  • Scripts/webkitpy/tool/bot/patchanalysistask.py:

(PatchAnalysisTask._test_patch):

  • Scripts/webkitpy/tool/commands/queuestest.py:

(QueuesTest.assert_queue_outputs):

12:33 PM Changeset in webkit [138633] by rniwa@webkit.org
  • 2 edits in trunk/Source/WebCore

Unreviewed, rolling out r138627.
http://trac.webkit.org/changeset/138627
https://bugs.webkit.org/show_bug.cgi?id=105935

This patch doesn't fix Windows port build failure (Requested
by rniwa on #webkit).

Patch by Sheriff Bot <webkit.review.bot@gmail.com> on 2013-01-02

  • WebCore.vcproj/WebCore.vcproj:
12:31 PM Changeset in webkit [138632] by esprehn@chromium.org
  • 9 edits
    4 adds in trunk

Transitions and animations do not apply to CSS ::before and ::after pseudo-elements
https://bugs.webkit.org/show_bug.cgi?id=92591

Reviewed by Eric Seidel.

.:

Expose Element::pseudoElement for Internals.

  • Source/autotools/symbols.filter:

Source/WebCore:

The new DOM based :before and :after pseudo elements support animations
and transitions already, but I had disabled support so we could turn them
on in a separate step. This patch just removes the checks and adds tests.

This also adds two methods to internals to allow pausing animations and
transitions on pseudo elements at predictable locations so the tests
are not flaky.

Tests: fast/css-generated-content/pseudo-animation.html

fast/css-generated-content/pseudo-transition.html

  • WebCore.exp.in:
  • page/animation/AnimationController.cpp:

(WebCore::AnimationController::updateAnimations):

  • testing/Internals.cpp:

(WebCore::Internals::pauseAnimationAtTimeOnPseudoElement):
(WebCore::Internals::pauseTransitionAtTimeOnPseudoElement):

  • testing/Internals.h:

(Internals):

  • testing/Internals.idl:

LayoutTests:

Add tests that check if animations and transitions play for pseudo
:before and :after pseudo elements.

  • fast/css-generated-content/pseudo-animation-expected.txt: Added.
  • fast/css-generated-content/pseudo-animation.html: Added.
  • fast/css-generated-content/pseudo-transition-expected.txt: Added.
  • fast/css-generated-content/pseudo-transition.html: Added.
12:27 PM Changeset in webkit [138631] by crogers@google.com
  • 14 edits
    2 copies in trunk

Implement WebIDL-style string constants in WebAudio
https://bugs.webkit.org/show_bug.cgi?id=105058

Reviewed by Kentaro Hara.

Source/WebCore:

See Deprecation Notes for more detail:
https://dvcs.w3.org/hg/audio/raw-file/tip/webaudio/specification.html#DeprecationNotes

PannerNode, BiquadFilterNode, OscillatorNode constants must support WebIDL-style string constants.
Legacy support in the setters for the old integer values should be supported.

This patch handles the changes for OscillatorNode.

  • GNUmakefile.list.am:
  • Modules/webaudio/OscillatorNode.cpp:

(WebCore::OscillatorNode::OscillatorNode):
(WebCore::OscillatorNode::type):
(WebCore):
(WebCore::OscillatorNode::setType):

  • Modules/webaudio/OscillatorNode.h:
  • Modules/webaudio/OscillatorNode.idl:
  • Target.pri:
  • UseJSC.cmake:
  • UseV8.cmake:
  • WebCore.gypi:
  • WebCore.xcodeproj/project.pbxproj:
  • bindings/js/JSOscillatorNodeCustom.cpp: Copied from Source/WebCore/Modules/webaudio/OscillatorNode.idl.

(WebCore):
(WebCore::JSOscillatorNode::setType):

  • bindings/v8/custom/V8OscillatorNodeCustom.cpp: Copied from Source/WebCore/Modules/webaudio/OscillatorNode.idl.

(WebCore):
(WebCore::V8OscillatorNode::typeAccessorSetter):

LayoutTests:

  • webaudio/oscillator-basic-expected.txt:
  • webaudio/oscillator-basic.html:
  • webaudio/resources/audio-testing.js:
12:18 PM Changeset in webkit [138630] by robert@webkit.org
  • 3 edits
    2 adds in trunk

Regression (r132935): WebKit breaks between <nobr> tags
https://bugs.webkit.org/show_bug.cgi?id=11857

Reviewed by David Hyatt.

Source/WebCore:

The fix for bug 29648 in http://trac.webkit.org/changeset/132935 was too broad - it caused us
to find linebreaks in autowrap containers between non-wrapping elements where there was no
whitespace to allow the break. This reverts the code change in r132935 and ensures we attempt
to move below floats when inside an autowrap container but leaves the treatment of non-wrapping
elements in an autowrap container unaffected.

Test: fast/inline/break-between-nobr.html

  • rendering/RenderBlockLineLayout.cpp:

(WebCore::RenderBlock::LineBreaker::nextSegmentBreak):

LayoutTests:

  • fast/inline/break-between-nobr-expected.html: Added.
  • fast/inline/break-between-nobr.html: Added.
12:16 PM Changeset in webkit [138629] by rjkroege@chromium.org
  • 4 edits in trunk/Source/WebKit/chromium

[chromium] Use top level field for gesture event source
https://bugs.webkit.org/show_bug.cgi?id=105664

Reviewed by Adam Barth.

Previously, only GestureFlingStart subtypes of WebGestureEvent
had a deviceSource flag. A previous CL added a top-level
deviceSource field for all WebGestureEvent. Now that this field
is populated in Chromium, use it as necessary in the Chromium
WebKit layer.

No new tests: covered by existing tests.

  • src/WebCompositorInputHandlerImpl.cpp:

(WebKit::WebCompositorInputHandlerImpl::handleGestureFling):

  • src/WebViewImpl.cpp:

(WebKit::WebViewImpl::handleGestureEvent):

  • tests/WebCompositorInputHandlerImplTest.cpp:

(WebKit::TEST_F):

12:00 PM Changeset in webkit [138628] by abarth@webkit.org
  • 2 edits in trunk/Source/WTF

WTF.gypi lists Platform.h twice
https://bugs.webkit.org/show_bug.cgi?id=105731

Reviewed by Tony Chang.

Once ought to be enough for everybody.

  • WTF.gypi:
11:48 AM Changeset in webkit [138627] by rniwa@webkit.org
  • 2 edits in trunk/Source/WebCore

Another Windows port build fix attempt. Don't try to build WTF files in WebCore.

Rubber-stamped by Sam Weinig.

  • WebCore.vcproj/WebCore.vcproj:
11:28 AM Changeset in webkit [138626] by rniwa@webkit.org
  • 2 edits in trunk/Source/WebKit2

Don't include a header that has been removed in r138413.

  • WebProcess/win/WebProcessWin.cpp:
11:27 AM Changeset in webkit [138625] by mkwst@chromium.org
  • 4 edits
    29 adds
    2 deletes in trunk/LayoutTests

sandboxed iframe attribute parsing layout tests are flaky
https://bugs.webkit.org/show_bug.cgi?id=104848

Reviewed by Adam Barth.

This patch breaks the monolithic sandboxed-iframe-attribute-parsing.html
test into separate tests for each case. The flaky failures were due to
one of the 14 frames loading out of order, which shifted console message
positions around. Breaking them into separate tests should both speed up
the tests, as more can run in parallel, and solve the flake.

  • fast/frames/resources/sandboxed-iframe-attribute-test.js: Added.

(fail):
(setStatus):
(allowedCallFromSandbox):
(disallowedCallFromSandbox):
(disallowedFormSubmitted):
(window.onload):

  • fast/frames/sandboxed-iframe-attribute-parsing-01-expected.txt: Added.
  • fast/frames/sandboxed-iframe-attribute-parsing-01.html: Added.
  • fast/frames/sandboxed-iframe-attribute-parsing-02-expected.txt: Added.
  • fast/frames/sandboxed-iframe-attribute-parsing-02.html: Added.
  • fast/frames/sandboxed-iframe-attribute-parsing-03-expected.txt: Added.
  • fast/frames/sandboxed-iframe-attribute-parsing-03.html: Added.
  • fast/frames/sandboxed-iframe-attribute-parsing-04-expected.txt: Added.
  • fast/frames/sandboxed-iframe-attribute-parsing-04.html: Added.
  • fast/frames/sandboxed-iframe-attribute-parsing-05-expected.txt: Added.
  • fast/frames/sandboxed-iframe-attribute-parsing-05.html: Added.
  • fast/frames/sandboxed-iframe-attribute-parsing-06-expected.txt: Added.
  • fast/frames/sandboxed-iframe-attribute-parsing-06.html: Added.
  • fast/frames/sandboxed-iframe-attribute-parsing-07-expected.txt: Added.
  • fast/frames/sandboxed-iframe-attribute-parsing-07.html: Added.
  • fast/frames/sandboxed-iframe-attribute-parsing-08-expected.txt: Added.
  • fast/frames/sandboxed-iframe-attribute-parsing-08.html: Added.
  • fast/frames/sandboxed-iframe-attribute-parsing-09-expected.txt: Added.
  • fast/frames/sandboxed-iframe-attribute-parsing-09.html: Added.
  • fast/frames/sandboxed-iframe-attribute-parsing-10-expected.txt: Added.
  • fast/frames/sandboxed-iframe-attribute-parsing-10.html: Added.
  • fast/frames/sandboxed-iframe-attribute-parsing-11-expected.txt: Added.
  • fast/frames/sandboxed-iframe-attribute-parsing-11.html: Added.
  • fast/frames/sandboxed-iframe-attribute-parsing-12-expected.txt: Added.
  • fast/frames/sandboxed-iframe-attribute-parsing-12.html: Added.
  • fast/frames/sandboxed-iframe-attribute-parsing-13-expected.txt: Added.
  • fast/frames/sandboxed-iframe-attribute-parsing-13.html: Added.
  • fast/frames/sandboxed-iframe-attribute-parsing-14-expected.txt: Added.
  • fast/frames/sandboxed-iframe-attribute-parsing-14.html: Added.
  • fast/frames/sandboxed-iframe-attribute-parsing-expected.txt: Removed.
  • fast/frames/sandboxed-iframe-attribute-parsing.html: Removed.
  • platform/chromium/TestExpectations:
  • platform/mac/TestExpectations:
  • platform/qt-arm/TestExpectations:
11:15 AM Changeset in webkit [138624] by mkwst@chromium.org
  • 1 edit
    3 adds in trunk/LayoutTests

Sandboxed iframes should not be granted notification permission of the parent frame unless allow-same-origin is specified
https://bugs.webkit.org/show_bug.cgi?id=36732

Reviewed by Adam Barth.

The behavior was fixed in wkbug.com/79704, but there aren't currently
any tests ensuring that it doesn't regress for sandboxed frames. This
patch adds one.

  • fast/notifications/notifications-sandbox-permission-expected.txt: Added.
  • fast/notifications/notifications-sandbox-permission.html: Added.
  • fast/notifications/resources/notifications-iframe.html: Added.
11:10 AM Changeset in webkit [138623] by rniwa@webkit.org
  • 2 edits in trunk/LayoutTests

Update Mac test expectation per bug 105932.

  • platform/mac/TestExpectations:
9:42 AM Changeset in webkit [138622] by Carlos Garcia Campos
  • 9 edits in trunk/Source/WebKit2

[WK2] Rename SetFrameInViewSourceMode to SetMainFrameInViewSourceMode
https://bugs.webkit.org/show_bug.cgi?id=105885

Reviewed by Martin Robinson.

This way we don't depend on a WebFrameProxy and
webkit_web_view_set_view_mode() works even if the main frame proxy
hasn't been created yet.

  • UIProcess/API/gtk/WebKitWebView.cpp:

(webkit_web_view_set_view_mode): Call
WebPageProxy::setMainFrameInViewSourceMode() directly.

  • UIProcess/WebFrameProxy.cpp:

(WebKit::WebFrameProxy::WebFrameProxy): Remove
setInViewSourceMode().

  • UIProcess/WebFrameProxy.h:

(WebFrameProxy):

  • UIProcess/WebPageProxy.cpp:

(WebKit::WebPageProxy::setMainFrameInViewSourceMode):

  • UIProcess/WebPageProxy.h:

(WebPageProxy):

  • WebProcess/WebPage/WebPage.cpp:

(WebKit::WebPage::setMainFrameInViewSourceMode):

  • WebProcess/WebPage/WebPage.h:

(WebPage):

  • WebProcess/WebPage/WebPage.messages.in:
9:10 AM Changeset in webkit [138621] by caio.oliveira@openbossa.org
  • 6 edits in trunk/Source/WebKit2

[Qt] Use WebPageGroup to create user scripts instead of Qt-specific function in WebPage
https://bugs.webkit.org/show_bug.cgi?id=105921

Reviewed by Simon Hausmann.

After r131281 the WebPageGroup.h (in UIProcess) exposes a way to add user scripts, so we can
use this directly to implement the user script functionality in Qt. The previous solution was
a Qt-port specific function in WebPage that can be removed now.

  • UIProcess/API/qt/qquickwebview.cpp:

(QQuickWebViewPrivate::updateUserScripts): Use WebPageGroup functions to manipulate user scripts directly.

  • UIProcess/WebPageProxy.h:

(WebPageProxy): Remove now unused function.

  • UIProcess/qt/WebPageProxyQt.cpp: Ditto.
  • WebProcess/WebPage/WebPage.messages.in: Ditto.
  • WebProcess/WebPage/qt/WebPageQt.cpp: Remove now unused function and headers.
8:46 AM Changeset in webkit [138620] by commit-queue@webkit.org
  • 5 edits in trunk/Source/WebCore

[EFL] [WebGL] Minor cleanup in PlatformContext.
https://bugs.webkit.org/show_bug.cgi?id=105872

Patch by Kondapally Kalyan <kalyan.kondapally@intel.com> on 2013-01-02
Reviewed by Kenneth Rohde Christiansen.

The logic to check for supported extensions is scattered in different classes i.e PlatformContext, eglcontext etc.
This patch cleans up any duplicate code and adds new api in GLPlatformContext to query for supported GL, EGL and GLX extensions.

  • PlatformEfl.cmake:
  • platform/graphics/opengl/GLPlatformContext.cpp:

(WebCore):
(WebCore::GLPlatformContext::createContext):
(WebCore::GLPlatformContext::supportsGLExtension):
(WebCore::GLPlatformContext::supportsEGLExtension):
(WebCore::GLPlatformContext::supportsGLXExtension):
(WebCore::GLPlatformContext::makeCurrent):
(WebCore::GLPlatformContext::initializeResetStatusExtension):

  • platform/graphics/opengl/GLPlatformContext.h:
  • platform/graphics/surfaces/egl/EGLContext.cpp:

(WebCore::isRobustnessExtSupported):
(WebCore::EGLOffScreenContext::initialize):

  • platform/graphics/surfaces/glx/GLXContext.cpp:

(WebCore::initializeARBExtensions):
(WebCore::GLXOffScreenContext::initialize):

8:42 AM Changeset in webkit [138619] by nghanavatian@rim.com
  • 2 edits in trunk/Source/WebKit/blackberry

[BlackBerry] Use fatfingers result when checking for misspelled words
https://bugs.webkit.org/show_bug.cgi?id=105923

Reviewed by Yong Li.

PR270866
Currently we are using the raw touch point when checking to see if spell check
is required on a word. When tapping close to a boundary, the raw point can be
placed off but fatfingers moves the point onto the word. This causes the caret
to be placed on a word but no spell check request triggered.

Internally reviewed by Mike Fenton and Tiancheng Jiang

  • WebKitSupport/TouchEventHandler.cpp:

(BlackBerry::WebKit::TouchEventHandler::handleTouchPoint):

7:05 AM Changeset in webkit [138618] by commit-queue@webkit.org
  • 11 edits
    4 adds in trunk

Source/WebKit2: [Qt][WK2] Add experimental API to find text from page
https://bugs.webkit.org/show_bug.cgi?id=96481

Patch by Heikki Paajanen <heikki.paajanen@palm.com> on 2013-01-02
Reviewed by Jocelyn Turcotte.

Expose findString and PageFindClient provided functionality to QtQuick
applications.

  • Target.pri:
  • UIProcess/API/qt/qquickwebview.cpp:

(QQuickWebViewPrivate::initialize):
(QQuickWebViewExperimental::findText):
(QQuickWebViewPrivate::didFindString):

  • UIProcess/API/qt/qquickwebview_p.h:

(WebKit):

  • UIProcess/API/qt/qquickwebview_p_p.h:

(QQuickWebViewPrivate):

  • UIProcess/API/qt/tests/qmltests/WebView/tst_findText.qml: Added.
  • UIProcess/qt/QtWebPageFindClient.cpp: Added.

(WebKit):
(WebKit::QtWebPageFindClient::QtWebPageFindClient):
(WebKit::QtWebPageFindClient::didFindString):
(WebKit::toQtWebPageFindClient):
(WebKit::QtWebPageFindClient::didFailToFindString):

  • UIProcess/qt/QtWebPageFindClient.h: Added.

(WebKit):
(QtWebPageFindClient):

Tools: [Qt][WK2] Add simple UI to find text from page
https://bugs.webkit.org/show_bug.cgi?id=96481

Patch by Heikki Paajanen <heikki.paajanen@palm.com> on 2013-01-02
Reviewed by Jocelyn Turcotte.

Find bar that slides between navigation bar and WebView.

  • MiniBrowser/qt/BrowserWindow.cpp:

(BrowserWindow::toggleFind):

  • MiniBrowser/qt/BrowserWindow.h:

(BrowserWindow):

  • MiniBrowser/qt/MiniBrowser.qrc:
  • MiniBrowser/qt/MiniBrowserApplication.cpp:

(MiniBrowserApplication::notify):

  • MiniBrowser/qt/icons/find.png: Added.
  • MiniBrowser/qt/qml/BrowserWindow.qml:
6:16 AM Changeset in webkit [138617] by Christophe Dumez
  • 3 edits in trunk/Tools

Regression(r138603): Caused 2 webkitpy tests to fail
https://bugs.webkit.org/show_bug.cgi?id=105913

Reviewed by Kentaro Hara.

Fix 2 webkitpy tests that started to fail after
r138603.

  • Scripts/webkitpy/tool/bot/irc_command_unittest.py:

(IRCCommandTest._sheriff_test_data_url): Make sure the
returned test file URL is valid no matter what the
current working directory is.

  • Scripts/webkitpy/tool/bot/ircbot_unittest.py:

(IRCBotTest.test_help): Add "sheriffs" command to the
list of available commands.

4:19 AM Changeset in webkit [138616] by commit-queue@webkit.org
  • 4 edits
    3 deletes in trunk

Unreviewed, rolling out r138601.
http://trac.webkit.org/changeset/138601
https://bugs.webkit.org/show_bug.cgi?id=105917

Also not an optimal solution, needs more discussion.
(Requested by mkwst_ on #webkit).

Patch by Sheriff Bot <webkit.review.bot@gmail.com> on 2013-01-02

Source/WebCore:

  • dom/Document.cpp:

(WebCore::Document::finishedParsing):

LayoutTests:

  • fast/frames/seamless/resources/span.html: Removed.
  • fast/frames/seamless/seamless-contenteditable-not-inherited-expected.txt:
  • fast/frames/seamless/seamless-srcdoc-expected.txt: Removed.
  • fast/frames/seamless/seamless-srcdoc.html: Removed.
4:07 AM Changeset in webkit [138615] by commit-queue@webkit.org
  • 14 edits
    3 deletes in trunk

Unreviewed, rolling out r138611.
http://trac.webkit.org/changeset/138611
https://bugs.webkit.org/show_bug.cgi?id=105916

Not an optimal solution, needs more discussion. (Requested by
mkwst_ on #webkit).

Patch by Sheriff Bot <webkit.review.bot@gmail.com> on 2013-01-02

Source/WebCore:

  • CMakeLists.txt:
  • DerivedSources.make:
  • DerivedSources.pri:
  • GNUmakefile.am:
  • WebCore.gyp/WebCore.gyp:
  • WebCore.vcproj/WebCore.vcproj:
  • css/StyleResolver.cpp:

(WebCore):
(WebCore::StyleResolver::matchUARules):
(WebCore::StyleResolver::collectFeatures):
(WebCore::StyleResolver::reportMemoryUsage):

  • css/seamless.css: Removed.

LayoutTests:

  • fast/frames/seamless/resources/nested-seamless.html:
  • fast/frames/seamless/resources/quirks-square.html:
  • fast/frames/seamless/resources/two-inline-blocks.html:
  • fast/frames/seamless/seamless-body-margin-expected.txt: Removed.
  • fast/frames/seamless/seamless-body-margin.html: Removed.
  • fast/frames/seamless/seamless-inherited-origin-expected.txt:
  • fast/frames/seamless/seamless-inherited-origin.html:
3:24 AM Changeset in webkit [138614] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebCore

[AC] Memory leak in GraphicsSurfacePrivate::findFBConfigWithAlpha()
https://bugs.webkit.org/show_bug.cgi?id=105770

Patch by Sudarsana Nagineni <sudarsana.nagineni@intel.com> on 2013-01-02
Reviewed by Noam Rosenthal.

Free the memory returned by glXGetVisualFromFBConfig() using
XFree to avoid a memory leak.

No new tests. No change in behavior.

  • platform/graphics/surfaces/glx/GraphicsSurfaceGLX.cpp:

(WebCore::GraphicsSurfacePrivate::findFBConfigWithAlpha):

2:09 AM Changeset in webkit [138613] by Christophe Dumez
  • 2 edits in trunk/Source/WebKit2

Regression(r138607): Causes crashes related to WebIconDatabase
https://bugs.webkit.org/show_bug.cgi?id=105909

Reviewed by Gyuyoung Kim.

Add back WebIconDatabase initialization statement
in WebContext constructor. It was removed in
r138607 and causing a lot of crashes on WebKit2.

  • UIProcess/WebContext.cpp:

(WebKit::WebContext::WebContext):

1:43 AM Changeset in webkit [138612] by Csaba Osztrogonác
  • 2 edits in trunk/Source/JavaScriptCore

One more unreviewed buildfix after r138609.

  • jit/JITCall.cpp: Add a missing include.
1:17 AM Changeset in webkit [138611] by mkwst@chromium.org
  • 14 edits
    3 adds in trunk

The 'body' of seamless IFrames should default to 'margin: 0'
https://bugs.webkit.org/show_bug.cgi?id=90834

Reviewed by Eric Seidel.

Source/WebCore:

Documents displayed in seamless iframes should override the 'body'
element's normal 8px default margins with 0px, in order to ensure that
contents are displayed flush with the boundries of the seamless box.[1]

This patch does so by adding a new user agent stylesheet for seamless
documents, and teaching the StyleResolver how and when to apply it.

[1]: http://lists.w3.org/Archives/Public/public-whatwg-archive/2012Jul/0039.html

Test: fast/frames/seamless/seamless-body-margin.html

  • DerivedSources.make:
  • DerivedSources.pri:
  • GNUmakefile.am:
  • WebCore.gyp/WebCore.gyp:
  • WebCore.vcproj/WebCore.vcproj:
  • CMakeLists.txt:

Added a new CSS file! Let's tell everyone!

  • css/StyleResolver.cpp:

(WebCore):
(WebCore::loadSeamlessStyle):
(WebCore::StyleResolver::matchUARules):
(WebCore::StyleResolver::collectFeatures):
(WebCore::StyleResolver::reportMemoryUsage):

Load the new stylesheet, and apply it to the document in
matchUARules iff the document should be displayed seamlessly.

  • css/seamless.css: Added.

(body):

Stylesheet containing defaults for seamless documents.

LayoutTests:

  • fast/frames/seamless/seamless-body-margin-expected.txt: Added.
  • fast/frames/seamless/seamless-body-margin.html: Added.

New test to explicitly check 'body' margins.

  • fast/frames/seamless/resources/nested-seamless.html:
  • fast/frames/seamless/resources/quirks-square.html:
  • fast/frames/seamless/resources/two-inline-blocks.html:
  • fast/frames/seamless/seamless-inherited-origin-expected.txt:
  • fast/frames/seamless/seamless-inherited-origin.html:

Updated these tests to remove the assumption that the body had an
8px margin.

1:07 AM Changeset in webkit [138610] by Csaba Osztrogonác
  • 2 edits in trunk/Source/JavaScriptCore

Unreviewed buildfix after r138609.

  • jit/JITCall32_64.cpp: Add a missing include.
12:48 AM Changeset in webkit [138609] by fpizlo@apple.com
  • 12 edits in trunk/Source/JavaScriptCore

Baseline JIT should have closure call caching
https://bugs.webkit.org/show_bug.cgi?id=105900

Reviewed by Gavin Barraclough.

This is not a speed-up by itself, but is meant to allow the DFG inliner to
accurately discern between closure calls and non-closure calls, so that it can
do closure call inlining in the future.

  • bytecode/CallLinkStatus.cpp:

(JSC::CallLinkStatus::computeFromLLInt):
(JSC::CallLinkStatus::computeFor):

  • bytecode/CallLinkStatus.h:

(JSC::CallLinkStatus::CallLinkStatus):
(JSC::CallLinkStatus::isClosureCall):
(CallLinkStatus):

  • dfg/DFGByteCodeParser.cpp:

(JSC::DFG::ByteCodeParser::handleCall):

  • jit/JIT.cpp:

(JSC::JIT::linkFor):
(JSC::JIT::linkSlowCall):

  • jit/JIT.h:

(JSC::JIT::compileClosureCall):

  • jit/JITCall.cpp:

(JSC::JIT::privateCompileClosureCall):

  • jit/JITCall32_64.cpp:

(JSC::JIT::privateCompileClosureCall):

  • jit/JITStubs.cpp:

(JSC::DEFINE_STUB_FUNCTION):

  • jit/JITStubs.h:
  • jit/ThunkGenerators.cpp:

(JSC::linkClosureCallGenerator):

  • jit/ThunkGenerators.h:

Jan 1, 2013:

10:53 PM Changeset in webkit [138608] by Christophe Dumez
  • 6 edits in trunk

[WK2] plugins/clicking-missing-plugin-fires-delegate.html is failing
https://bugs.webkit.org/show_bug.cgi?id=105897

Reviewed by Alexey Proskuryakov.

Tools:

Dump in WebKitTestRunner the message that is expected by layout tests
when the missing plugin button is clicked.

  • WebKitTestRunner/TestController.cpp:

(WTR::TestController::createWebViewWithOptions):
(WTR::TestController::unavailablePluginButtonClicked):
(WTR):

  • WebKitTestRunner/TestController.h:

(TestController):

LayoutTests:

Unskip plugins/clicking-missing-plugin-fires-delegate.html for
WK2 now that WebKitTestRunner is dumping the expected text when
the missing plugin button is clicked.

  • platform/efl-wk2/TestExpectations:
  • platform/mac-wk2/TestExpectations:
8:19 PM Changeset in webkit [138607] by weinig@apple.com
  • 28 edits in trunk/Source/WebKit2

Convert more classes to be WebContextSupplements
https://bugs.webkit.org/show_bug.cgi?id=105890

Reviewed by Dan Bernstein.

  • UIProcess/API/C/WKContext.cpp:

(WKContextGetCookieManager):
(WKContextGetDatabaseManager):
(WKContextGetGeolocationManager):
(WKContextGetKeyValueStorageManager):
(WKContextGetMediaCacheManager):
(WKContextGetNotificationManager):
(WKContextGetResourceCacheManager):

  • UIProcess/API/efl/ewk_context.cpp:

(EwkContext::cookieManager):
(EwkContext::clearResourceCache):

  • UIProcess/API/efl/ewk_database_manager_private.h:

(EwkDatabaseManager::create):

  • UIProcess/API/efl/ewk_storage_manager_private.h:

(EwkStorageManager::create):

  • UIProcess/API/gtk/WebKitWebContext.cpp:

(createDefaultWebContext):
(webkit_web_context_clear_cache):
(webkit_web_context_get_cookie_manager):

  • UIProcess/Notifications/WebNotificationManagerProxy.cpp:

(WebKit::WebNotificationManagerProxy::supplementName):
(WebKit):
(WebKit::WebNotificationManagerProxy::WebNotificationManagerProxy):
(WebKit::WebNotificationManagerProxy::initializeProvider):
(WebKit::WebNotificationManagerProxy::contextDestroyed):
(WebKit::WebNotificationManagerProxy::processDidClose):
(WebKit::WebNotificationManagerProxy::refWebContextSupplement):
(WebKit::WebNotificationManagerProxy::derefWebContextSupplement):
(WebKit::WebNotificationManagerProxy::didReceiveMessage):
(WebKit::WebNotificationManagerProxy::providerDidShowNotification):
(WebKit::WebNotificationManagerProxy::providerDidClickNotification):
(WebKit::WebNotificationManagerProxy::providerDidCloseNotifications):
(WebKit::WebNotificationManagerProxy::providerDidUpdateNotificationPolicy):
(WebKit::WebNotificationManagerProxy::providerDidRemoveNotificationPolicies):

  • UIProcess/Notifications/WebNotificationManagerProxy.h:

(WebNotificationManagerProxy):

  • UIProcess/WebContext.cpp:

(WebKit::WebContext::WebContext):
(WebKit::WebContext::~WebContext):
(WebKit::WebContext::removeNetworkProcessProxy):
(WebKit::WebContext::createNewWebProcess):
(WebKit::WebContext::shouldTerminate):
(WebKit::WebContext::disconnectProcess):

  • UIProcess/WebContext.h:

(WebKit):
(WebContext):
(WebKit::WebContext::iconDatabase):

  • UIProcess/WebContextSupplement.h:

(WebKit):
(WebKit::WebContextSupplement::contextDestroyed):
(WebKit::WebContextSupplement::processDidClose):
(WebContextSupplement):
(WebKit::WebContextSupplement::shouldTerminate):
(WebKit::WebContextSupplement::ref):
(WebKit::WebContextSupplement::deref):

  • UIProcess/WebCookieManagerProxy.cpp:

(WebKit::WebCookieManagerProxy::supplementName):
(WebKit):
(WebKit::WebCookieManagerProxy::WebCookieManagerProxy):
(WebKit::WebCookieManagerProxy::initializeClient):
(WebKit::WebCookieManagerProxy::contextDestroyed):
(WebKit::WebCookieManagerProxy::processDidClose):
(WebKit::WebCookieManagerProxy::shouldTerminate):
(WebKit::WebCookieManagerProxy::refWebContextSupplement):
(WebKit::WebCookieManagerProxy::derefWebContextSupplement):
(WebKit::WebCookieManagerProxy::getHostnamesWithCookies):
(WebKit::WebCookieManagerProxy::deleteCookiesForHostname):
(WebKit::WebCookieManagerProxy::deleteAllCookies):
(WebKit::WebCookieManagerProxy::startObservingCookieChanges):
(WebKit::WebCookieManagerProxy::stopObservingCookieChanges):
(WebKit::WebCookieManagerProxy::setHTTPCookieAcceptPolicy):
(WebKit::WebCookieManagerProxy::getHTTPCookieAcceptPolicy):

  • UIProcess/WebCookieManagerProxy.h:

(WebCookieManagerProxy):

  • UIProcess/WebDatabaseManagerProxy.cpp:

(WebKit::WebDatabaseManagerProxy::supplementName):
(WebKit):
(WebKit::WebDatabaseManagerProxy::WebDatabaseManagerProxy):
(WebKit::WebDatabaseManagerProxy::initializeClient):
(WebKit::WebDatabaseManagerProxy::contextDestroyed):
(WebKit::WebDatabaseManagerProxy::processDidClose):
(WebKit::WebDatabaseManagerProxy::refWebContextSupplement):
(WebKit::WebDatabaseManagerProxy::derefWebContextSupplement):
(WebKit::WebDatabaseManagerProxy::didReceiveMessage):
(WebKit::WebDatabaseManagerProxy::getDatabasesByOrigin):
(WebKit::WebDatabaseManagerProxy::getDatabaseOrigins):
(WebKit::WebDatabaseManagerProxy::deleteDatabaseWithNameForOrigin):
(WebKit::WebDatabaseManagerProxy::deleteDatabasesForOrigin):
(WebKit::WebDatabaseManagerProxy::deleteAllDatabases):
(WebKit::WebDatabaseManagerProxy::setQuotaForOrigin):

  • UIProcess/WebDatabaseManagerProxy.h:

(WebDatabaseManagerProxy):

  • UIProcess/WebGeolocationManagerProxy.cpp:

(WebKit::WebGeolocationManagerProxy::supplementName):
(WebKit):
(WebKit::WebGeolocationManagerProxy::WebGeolocationManagerProxy):
(WebKit::WebGeolocationManagerProxy::initializeProvider):
(WebKit::WebGeolocationManagerProxy::contextDestroyed):
(WebKit::WebGeolocationManagerProxy::processDidClose):
(WebKit::WebGeolocationManagerProxy::refWebContextSupplement):
(WebKit::WebGeolocationManagerProxy::derefWebContextSupplement):
(WebKit::WebGeolocationManagerProxy::didReceiveMessage):
(WebKit::WebGeolocationManagerProxy::providerDidChangePosition):
(WebKit::WebGeolocationManagerProxy::providerDidFailToDeterminePosition):

  • UIProcess/WebGeolocationManagerProxy.h:

(WebGeolocationManagerProxy):

  • UIProcess/WebKeyValueStorageManagerProxy.cpp:

(WebKit::WebKeyValueStorageManagerProxy::supplementName):
(WebKit):
(WebKit::WebKeyValueStorageManagerProxy::WebKeyValueStorageManagerProxy):
(WebKit::WebKeyValueStorageManagerProxy::contextDestroyed):
(WebKit::WebKeyValueStorageManagerProxy::processDidClose):
(WebKit::WebKeyValueStorageManagerProxy::refWebContextSupplement):
(WebKit::WebKeyValueStorageManagerProxy::derefWebContextSupplement):
(WebKit::WebKeyValueStorageManagerProxy::getKeyValueStorageOrigins):
(WebKit::WebKeyValueStorageManagerProxy::deleteEntriesForOrigin):
(WebKit::WebKeyValueStorageManagerProxy::deleteAllEntries):

  • UIProcess/WebKeyValueStorageManagerProxy.h:

(WebKeyValueStorageManagerProxy):

  • UIProcess/WebMediaCacheManagerProxy.cpp:

(WebKit::WebMediaCacheManagerProxy::supplementName):
(WebKit):
(WebKit::WebMediaCacheManagerProxy::WebMediaCacheManagerProxy):
(WebKit::WebMediaCacheManagerProxy::contextDestroyed):
(WebKit::WebMediaCacheManagerProxy::processDidClose):
(WebKit::WebMediaCacheManagerProxy::refWebContextSupplement):
(WebKit::WebMediaCacheManagerProxy::derefWebContextSupplement):
(WebKit::WebMediaCacheManagerProxy::getHostnamesWithMediaCache):
(WebKit::WebMediaCacheManagerProxy::clearCacheForHostname):
(WebKit::WebMediaCacheManagerProxy::clearCacheForAllHostnames):

  • UIProcess/WebMediaCacheManagerProxy.h:

(WebMediaCacheManagerProxy):

  • UIProcess/WebPageProxy.cpp:

(WebKit::WebPageProxy::showNotification):

  • UIProcess/WebResourceCacheManagerProxy.cpp:

(WebKit::WebResourceCacheManagerProxy::supplementName):
(WebKit):
(WebKit::WebResourceCacheManagerProxy::WebResourceCacheManagerProxy):
(WebKit::WebResourceCacheManagerProxy::contextDestroyed):
(WebKit::WebResourceCacheManagerProxy::processDidClose):
(WebKit::WebResourceCacheManagerProxy::refWebContextSupplement):
(WebKit::WebResourceCacheManagerProxy::derefWebContextSupplement):
(WebKit::WebResourceCacheManagerProxy::didReceiveMessage):
(WebKit::WebResourceCacheManagerProxy::getCacheOrigins):
(WebKit::WebResourceCacheManagerProxy::clearCacheForOrigin):
(WebKit::WebResourceCacheManagerProxy::clearCacheForAllOrigins):

  • UIProcess/WebResourceCacheManagerProxy.h:

(WebResourceCacheManagerProxy):

  • UIProcess/efl/WebContextEfl.cpp:

(WebKit::WebContext::platformInitializeWebProcess):

  • UIProcess/gtk/WebContextGtk.cpp:

(WebKit::WebContext::platformInitializeWebProcess):

  • UIProcess/qt/WebContextQt.cpp:

(WebKit::WebContext::platformInitializeWebProcess):

8:18 PM Changeset in webkit [138606] by mitz@apple.com
  • 16 edits in trunk

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

Reviewed by Sam Weinig.

Source/JavaScriptCore:

  • Info.plist:

Source/WebCore:

  • Info.plist:

Source/WebKit/mac:

  • Info.plist:

Source/WebKit2:

  • Info.plist:
  • NetworkProcess/Info.plist:
  • PluginProcess/Info.plist:
  • SharedWorkerProcess/Info.plist:
  • WebProcess/Info.plist:
  • WebProcessService/Info.plist:
  • WebProcessServiceForWebKitDevelopment/Info.plist:

WebKitLibraries:

  • win/tools/scripts/COPYRIGHT-END-YEAR:
8:14 PM Changeset in webkit [138605] by commit-queue@webkit.org
  • 9 edits in trunk

[EFL] Enable MHTML feature
https://bugs.webkit.org/show_bug.cgi?id=105815

Patch by KwangYong Choi <ky0.choi@samsung.com> on 2013-01-01
Reviewed by Laszlo Gombos.

.:

ENABLE_MHTML is now ON for EFL.

  • Source/cmake/OptionsEfl.cmake:

Source/WebCore:

Add application/x-mimearchive type to archive mime types.
Add mht and mhtml to extension map.

No new tests. Existing layout tests pass.

  • loader/archive/ArchiveFactory.cpp:

(WebCore::archiveMIMETypes):

  • platform/efl/MIMETypeRegistryEfl.cpp:

(WebCore):

Source/WebKit2:

Add missing include directory for ENABLE_MHTML.

  • CMakeLists.txt:

Tools:

Enable MHTML feature for EFL port by default.

  • Scripts/webkitperl/FeatureList.pm:
7:51 PM Changeset in webkit [138604] by barraclough@apple.com
  • 13 edits
    18 adds in trunk/Source

Objective-C API for JavaScriptCore
https://bugs.webkit.org/show_bug.cgi?id=105889

Reviewed by Filip Pizlo.

../JavaScriptCore:

For a detailed description of the API implemented here, see:

JSContext.h
APIJSValue.h
JSVirtualMachine.h
JSExport.h

Still to do -

(1) Shoud rename APIJSValue.h -> JSValue.h (but we'll have to rename JSValue.h first).
(2) Numerous FIXMEs, all with separate bugs filed.

  • API/APIJSValue.h: Added.
    • this Objective-C class is used to reference a JavaScript object.
  • API/JSBase.h:
    • added JS_OBJC_API_ENABLED macro to control ObjC API support.
  • API/JSBlockAdaptor.h: Added.
    • this Objective-C class is used in creating a special NSBlock proxying a JavaScript function.
  • API/JSBlockAdaptor.mm: Added.

(BlockArgument):
(BlockArgument::~BlockArgument):
(BlockArgumentBoolean):
(BlockArgumentBoolean::get):
(BlockArgumentNumeric):
(BlockArgumentNumeric::get):
(BlockArgumentId):
(BlockArgumentId::get):
(BlockArgumentStruct):
(BlockArgumentStruct::BlockArgumentStruct):
(BlockArgumentStruct::~BlockArgumentStruct):
(BlockArgumentStruct::get):

  • decoded arguent type information of a JSBlockAdaptor.

(BlockArgumentTypeDelegate):
(BlockArgumentTypeDelegate::typeInteger):
(BlockArgumentTypeDelegate::typeDouble):
(BlockArgumentTypeDelegate::typeBool):
(BlockArgumentTypeDelegate::typeVoid):
(BlockArgumentTypeDelegate::typeId):
(BlockArgumentTypeDelegate::typeOfClass):
(BlockArgumentTypeDelegate::typeBlock):
(BlockArgumentTypeDelegate::typeStruct):

  • delegate for use in conjunction with parseObjCType.

(BlockResult):
(BlockResult::~BlockResult):
(BlockResultVoid):
(BlockResultVoid::set):
(BlockResultInteger):
(BlockResultInteger::set):
(BlockResultDouble):
(BlockResultDouble::set):
(BlockResultBoolean):
(BlockResultBoolean::set):
(BlockResultStruct):
(BlockResultStruct::BlockResultStruct):
(BlockResultStruct::~BlockResultStruct):
(BlockResultStruct::set):

  • decoded result type information of a JSBlockAdaptor.

(buildBlockSignature):

  • partial step in constructing a signature with stack offset information from one without.

(-[JSBlockAdaptor initWithBlockSignatureFromProtocol:]):

  • constructor.

(-[JSBlockAdaptor blockMatchesSignature:]):

  • check whether signature strings match, where only one contains stack frame offsets.

(-[JSBlockAdaptor blockFromValue:inContext:withException:]):

  • use the adaptor to create a special forwarding block.
  • API/JSCallbackObjectFunctions.h:

(JSC::::inherits):

  • add missing braces to multiline for statement.
  • API/JSContext.h: Added.
    • this Objective-C class is used to reference a JavaScript context.
  • API/JSContext.mm: Added.

(-[JSContext init]):

  • constructor.

(-[JSContext initWithVirtualMachine:]):

  • construct in a given VM (JSGlobalData).

(-[JSContext evaluateScript:]):
(-[JSContext globalObject]):

  • evaluate a script, global object accessor.

(+[JSContext currentContext]):
(+[JSContext currentThis]):
(+[JSContext currentArguments]):

  • These methods obtain context, this, arguments from within a callback.

(-[JSContext virtualMachine]):

  • implementation for .virtualMachine property.

(-[JSContext objectForKeyedSubscript:]):
(-[JSContext setObject:forKeyedSubscript:]):

  • support for subscript property access.

(contextInternalContext):

  • internal accessor to m_context.

(-[JSContext dealloc]):

  • desctructor.

(-[JSContext notifyException:]):
(-[JSContext valueFromNotifyException:]):
(-[JSContext boolFromNotifyException:]):

  • internal method to record an exception was thrown.

(-[JSContext beginCallbackWithData:thisValue:argumentCount:arguments:]):
(-[JSContext endCallbackWithData:]):

  • internal methods to push/pop a callback record.

(-[JSContext protect:]):
(-[JSContext unprotect:]):

  • internal methods to add a value to a protect set (used to protect the internal property of JSValue).

(-[JSContext wrapperForObject:]):

  • internal method to create a wrapper object.

(WeakContextRef::WeakContextRef):
(WeakContextRef::~WeakContextRef):
(WeakContextRef::get):
(WeakContextRef::set):

  • Helper class to implement a weak reference to a JSContext.
  • API/JSContextInternal.h: Added.

(CallbackData):
(WeakContextRef):

  • see API/JSContext.mm for description of internal methods.
  • API/JSExport.h: Added.
    • Provides JSExport protocol & JSExportAs macro.
  • API/JSValue.mm: Added.

(+[JSValue valueWithObject:inContext:]):
(+[JSValue valueWithBool:inContext:]):
(+[JSValue valueWithDouble:inContext:]):
(+[JSValue valueWithInt32:inContext:]):
(+[JSValue valueWithUInt32:inContext:]):
(+[JSValue valueWithNewObjectInContext:]):
(+[JSValue valueWithNewArrayInContext:]):
(+[JSValue valueWithNewRegularExpressionFromPattern:flags:inContext:]):
(+[JSValue valueWithNewErrorFromMessage:inContext:]):
(+[JSValue valueWithNullInContext:]):
(+[JSValue valueWithUndefinedInContext:]):

  • Constructors.

(-[JSValue toObject]):
(-[JSValue toObjectOfClass:]):
(-[JSValue toBool]):
(-[JSValue toDouble]):
(-[JSValue toInt32]):
(-[JSValue toUInt32]):
(-[JSValue toNumber]):
(-[JSValue toString]):
(-[JSValue toDate]):
(-[JSValue toArray]):
(-[JSValue toDictionary]):

  • Conversion to Objective-C types.

(-[JSValue valueForProperty:]):
(-[JSValue setValue:forProperty:]):
(-[JSValue deleteProperty:]):
(-[JSValue hasProperty:]):
(-[JSValue defineProperty:descriptor:]):

  • Property access by property name.

(-[JSValue valueAtIndex:]):
(-[JSValue setValue:atIndex:]):

  • Property access by index.

(-[JSValue isUndefined]):
(-[JSValue isNull]):
(-[JSValue isBoolean]):
(-[JSValue isNumber]):
(-[JSValue isString]):
(-[JSValue isObject]):

  • Test JavaScript type.

(-[JSValue isEqualToObject:]):
(-[JSValue isEqualWithTypeCoercionToObject:]):
(-[JSValue isInstanceOf:]):

  • ===, ==, instanceof operators.

(-[JSValue callWithArguments:]):
(-[JSValue constructWithArguments:]):
(-[JSValue invokeMethod:withArguments:]):

  • Call & construct.

(-[JSValue context]):

  • implementation for .context property.

(-[JSValue toPoint]):
(-[JSValue toRange]):
(-[JSValue toRect]):
(-[JSValue toSize]):
(+[JSValue valueWithPoint:inContext:]):
(+[JSValue valueWithRange:inContext:]):
(+[JSValue valueWithRect:inContext:]):
(+[JSValue valueWithSize:inContext:]):

  • Support for NS struct types.

(-[JSValue objectForKeyedSubscript:]):
(-[JSValue objectAtIndexedSubscript:]):
(-[JSValue setObject:forKeyedSubscript:]):
(-[JSValue setObject:atIndexedSubscript:]):

  • support for subscript property access.

(isDate):
(isArray):

  • internal helper functions to check for instances of JS Date, Array types.

(JSContainerConvertor):
(Task):
(JSContainerConvertor::JSContainerConvertor):
(JSContainerConvertor::isWorkListEmpty):
(JSContainerConvertor::convert):
(JSContainerConvertor::add):
(JSContainerConvertor::take):

  • helper class for tracking state while converting to Array/Dictionary objects.

(valueToObjectWithoutCopy):
(containerValueToObject):
(valueToObject):
(valueToNumber):
(valueToString):
(valueToDate):
(valueToArray):
(valueToDictionary):

  • function for converting JavaScript values to Objective-C objects.

(ObjcContainerConvertor):
(ObjcContainerConvertor::ObjcContainerConvertor):
(ObjcContainerConvertor::isWorkListEmpty):
(ObjcContainerConvertor::convert):
(ObjcContainerConvertor::add):
(ObjcContainerConvertor::take):

  • helper class for tracking state while converting to Array/Dictionary values.

(objectToValueWithoutCopy):
(objectToValue):
(valueInternalValue):

  • function for converting Objective-C objects to JavaScript values.

(+[JSValue valueWithValue:inContext:]):
(-[JSValue initWithValue:inContext:]):

  • internal constructors.

(StructTagHandler):
(getStructTagHandler):
(+[JSValue selectorForStructToValue:]):
(+[JSValue selectorForValueToStruct:]):

  • methods to tracking struct types that support conversion to/from JSValue.

(-[JSValue dealloc]):

  • destructor.

(-[JSValue description]):

  • Objective-C to-NSString conversion.

(typeToValueInvocationFor):
(valueToTypeInvocationFor):

  • create invocation objects for conversion to/from JSValue.
  • API/JSValueInternal.h: Added.
    • see API/JSValue.mm for description of internal methods.
  • API/JSVirtualMachine.h: Added.
    • this Objective-C class is used to reference a JavaScript virtual machine (JSGlobalData).
  • API/JSVirtualMachine.mm: Added.

(-[JSVirtualMachine init]):
(-[JSVirtualMachine dealloc]):

  • constructor & destructor.

(getGroupFromVirtualMachine):

  • internal accessor for m_group property.
  • API/JSVirtualMachineInternal.h: Added.
    • see API/JSVirtualMachine.mm for description of internal methods.
  • API/JSWrapperMap.h: Added.
  • API/JSWrapperMap.mm: Added.

(wrapperClass):

  • singleton root for detction (& unwrapping) of wrapper objects.

(selectorToPropertyName):

  • default selector to property name conversion.

(createObjectWithCustomBrand):

  • creates a JSObject with a custom NativeBrand (class name).

(createRenameMap):

  • parse @optional properties of a JSExport protocol.

(putNonEnumerable):

  • property put with enumerable=false.

(copyMethodsToObject):

  • iterate methods in a protocol; add functions to a JSObject.

(parsePropertyAttributes):

  • examine protocol property metadata.

(makeSetterName):

  • "foo" -> "setFoo"

(copyPrototypeProperties):

  • create properties on a Protocol object reflecting the instance methods & properties of a protocol.

(-[JSObjCClassInfo initWithContext:forClass:superClassInfo:]):
(-[JSObjCClassInfo dealloc]):
(-[JSObjCClassInfo wrapperForObject:]):
(-[JSObjCClassInfo constructor]):

  • cache the Protocol/Constructor objects for an Objective-C type.

(-[JSWrapperMap initWithContext:]):
(-[JSWrapperMap dealloc]):

  • constructor & desctructor.

(-[JSWrapperMap classInfoForClass:]):

  • maps Class -> JSObjCClassInfo.

(-[JSWrapperMap wrapperForObject:]):

  • cretae or retrieve a cached wrapper value for an object.

(tryUnwrapObjcObject):

  • check whether a value is a wrapper object; unwrap if so.
  • API/JavaScriptCore.h:
    • Added includes for new API headers.
  • API/ObjCCallbackFunction.h: Added.
    • this class is used to wrap Objective-C instance methods, class methods & blocks as JSFunction objects.
  • API/ObjCCallbackFunction.mm: Added.

(CallbackArgument):
(CallbackArgument::~CallbackArgument):
(CallbackArgumentBoolean):
(CallbackArgumentBoolean::set):
(CallbackArgumentInteger):
(CallbackArgumentInteger::set):
(CallbackArgumentDouble):
(CallbackArgumentDouble::set):
(CallbackArgumentJSValue):
(CallbackArgumentJSValue::set):
(CallbackArgumentId):
(CallbackArgumentId::set):
(CallbackArgumentOfClass):
(CallbackArgumentOfClass::CallbackArgumentOfClass):
(CallbackArgumentOfClass::~CallbackArgumentOfClass):
(CallbackArgumentOfClass::set):
(CallbackArgumentNSNumber):
(CallbackArgumentNSNumber::set):
(CallbackArgumentNSString):
(CallbackArgumentNSString::set):
(CallbackArgumentNSDate):
(CallbackArgumentNSDate::set):
(CallbackArgumentNSArray):
(CallbackArgumentNSArray::set):
(CallbackArgumentNSDictionary):
(CallbackArgumentNSDictionary::set):
(CallbackArgumentStruct):
(CallbackArgumentStruct::CallbackArgumentStruct):
(CallbackArgumentStruct::~CallbackArgumentStruct):
(CallbackArgumentStruct::set):
(CallbackArgumentBlockCallback):
(CallbackArgumentBlockCallback::CallbackArgumentBlockCallback):
(CallbackArgumentBlockCallback::~CallbackArgumentBlockCallback):
(CallbackArgumentBlockCallback::set):

  • decoded arguent type information of a ObjCCallbackFunction.

(ArgumentTypeDelegate):
(ArgumentTypeDelegate::typeInteger):
(ArgumentTypeDelegate::typeDouble):
(ArgumentTypeDelegate::typeBool):
(ArgumentTypeDelegate::typeVoid):
(ArgumentTypeDelegate::typeId):
(ArgumentTypeDelegate::typeOfClass):
(ArgumentTypeDelegate::typeBlock):
(ArgumentTypeDelegate::typeStruct):

  • delegate for use in conjunction with parseObjCType.

(CallbackResult):
(CallbackResult::~CallbackResult):
(CallbackResultVoid):
(CallbackResultVoid::get):
(CallbackResultId):
(CallbackResultId::get):
(CallbackResultNumeric):
(CallbackResultNumeric::get):
(CallbackResultBoolean):
(CallbackResultBoolean::get):
(CallbackResultStruct):
(CallbackResultStruct::CallbackResultStruct):
(CallbackResultStruct::~CallbackResultStruct):
(CallbackResultStruct::get):

  • decoded result type information of a ObjCCallbackFunction.

(ResultTypeDelegate):
(ResultTypeDelegate::typeInteger):
(ResultTypeDelegate::typeDouble):
(ResultTypeDelegate::typeBool):
(ResultTypeDelegate::typeVoid):
(ResultTypeDelegate::typeId):
(ResultTypeDelegate::typeOfClass):
(ResultTypeDelegate::typeBlock):
(ResultTypeDelegate::typeStruct):

  • delegate for use in conjunction with parseObjCType.

(ObjCCallbackFunction):
(ObjCCallbackFunction::ObjCCallbackFunction):
(ObjCCallbackFunction::~ObjCCallbackFunction):

  • constructor & destructor.

(ObjCCallbackFunction::context):

  • accessor.

(ObjCCallbackFunction::wrappedBlock):

  • attemmpt to unwrap a block object.

(objCCallbackFunctionFinalize):
(objCCallbackFunctionCallAsFunction):
(objCCallbackFunctionClass):

  • JSClassRef used to represent ObjCCallbackFunction objects.

(ObjCCallbackFunction::call):
(blockSignatureContainsClass):

  • helper function to determine if we're running on a recent Clang.

(skipNumber):

  • helper used in parsing signature strings.

(objCCallbackFunctionForInvocation):
(objCCallbackFunctionForMethod):
(objCCallbackFunctionForBlock):

  • functions to try to create ObjCCallbackFunction instances for methods/blocks.

(tryUnwrapBlock):

  • attemmpt to unwrap a block object.
  • API/ObjcRuntimeExtras.h: Added.

(protocolImplementsProtocol):
(forEachProtocolImplementingProtocol):
(forEachMethodInClass):
(forEachMethodInProtocol):
(forEachPropertyInProtocol):

  • functions used in reflecting on Objective-C types.

(skipPair):

  • parsing helper used by parseObjCType, scans for matching parentheses.

(StringRange):
(StringRange::StringRange):
(StringRange::~StringRange):
(StringRange::operator const char*):
(StringRange::get):

  • Helper class - create a c string copy of a range of an existing string.

(parseObjCType):

  • function to parse Objective-C type strings, makes callbacks to a deleagte.
  • API/tests/testapi.c:

(main):

  • added call to testObjectiveCAPI (in testapi.m).
  • API/tests/testapi.m: Added.

(+[ParentObject parentTest]):
(+[TestObject testObject]):
(+[TestObject classTest]):
(-[TestObject getString]):
(-[TestObject testArgumentTypesWithInt:double:boolean:string:number:array:dictionary:]):
(-[TestObject callback:]):
(-[TextXYZ test:]):

  • test object, used in various test vases.

(checkResult):

  • helper function.

(blockSignatureContainsClass):

  • helper function to determine if we're running on a recent Clang.

(testObjectiveCAPI):

  • new test cases.
  • JavaScriptCore.xcodeproj/project.pbxproj:
    • added new files.
  • runtime/JSGlobalData.cpp:

(JSC::JSGlobalData::JSGlobalData):

  • runtime/JSGlobalData.h:

(JSGlobalData):

  • added m_apiData - provide convenient storage for use by the API.
  • runtime/JSGlobalObject.cpp:

(JSC::JSGlobalObject::JSGlobalObject):

  • runtime/JSGlobalObject.h:

(JSGlobalObject):

  • added m_apiData - provide convenient storage for use by the API.

../WTF:

  • wtf/WTFThreadData.cpp:

(WTF::WTFThreadData::WTFThreadData):

  • wtf/WTFThreadData.h:

(WTFThreadData):

  • Added m_apiData - provide convenient storage for use by the API.
6:54 PM Changeset in webkit [138603] by commit-queue@webkit.org
  • 4 edits
    6 adds in trunk/Tools

sheriff-bot should know who the gardeners/sheriffs are
https://bugs.webkit.org/show_bug.cgi?id=105698

Patch by Alan Cutter <alancutter@chromium.org> on 2013-01-01
Reviewed by Eric Seidel.

Added a "sheriffs" command to sheriffbot to fetch the current Chromium Webkit sheriffs.

  • Scripts/webkitpy/tool/bot/irc_command.py:

(Current):
(Current._retrieve_webkit_sheriffs):
(Current.execute):

  • Scripts/webkitpy/tool/bot/irc_command_unittest.py:

(IRCCommandTest.test_current):

  • Scripts/webkitpy/tool/bot/testdata/webkit_sheriff_0.js: Added.
  • Scripts/webkitpy/tool/bot/testdata/webkit_sheriff_1.js: Added.
  • Scripts/webkitpy/tool/bot/testdata/webkit_sheriff_2.js: Added.
  • Scripts/webkitpy/tool/bot/testdata/webkit_sheriff_malformed.js: Added.
  • Scripts/webkitpy/tool/bot/testdata/webkit_sheriff_zero.js: Added.
4:23 PM Changeset in webkit [138602] by commit-queue@webkit.org
  • 2 edits in trunk/LayoutTests

[EFL][WK1] Skip some webaudio test cases due to lack of implementation
https://bugs.webkit.org/show_bug.cgi?id=105882

Unreviewed, EFL gardening.

Bunch of webaudio test cases need setAudioData implementation and DRT has not it.
So, skips those cases.

Patch by Kangil Han <kangil.han@samsung.com> on 2013-01-01

  • platform/efl-wk1/TestExpectations:
9:20 AM Changeset in webkit [138601] by mkwst@chromium.org
  • 4 edits
    3 adds in trunk

seamless iframes don't inherit styles when srcdoc is used
https://bugs.webkit.org/show_bug.cgi?id=103539

Reviewed by Eric Seidel.

Source/WebCore:

Seamless iframes populated via a 'srcdoc' attribute should always
inherit styles from their parent documents. At the moment, this is
only the case when they contain a stylesheet or some other markup
that forces a style recalculation on the document. Simple 'srcdoc'
attributes (e.g. "srcdoc='<p>This is a comment.</p>'") bail out of
recalculating style early, resulting in unstyled appearance.

This patch instructs WebCore to treat seamless documents as having an
updated StyleResolver regardless of what actions the parser takes,
which in turn ensures that the document's style is recalculated
correctly.

Test: fast/frames/seamless/seamless-srcdoc.html

  • dom/Document.cpp:

(WebCore::Document::finishedParsing):

If it's a seamless document, notify it that its StyleResolver isn't
what it might have expected.

LayoutTests:

  • fast/frames/seamless/resources/span.html: Added.
  • fast/frames/seamless/seamless-contenteditable-not-inherited-expected.txt:

Updating the previously FAILing expectation.

  • fast/frames/seamless/seamless-srcdoc-expected.txt: Added.
  • fast/frames/seamless/seamless-srcdoc.html: Added.

Exciting new test to ensure that this doesn't regress, neither
for totally simple srcdoc attriubtes, nor for slightly more
complex variations.

3:47 AM WebKitGTK/WebKit2Roadmap edited by Carlos Garcia Campos
Added web extensions, and updated target versions (diff)

Dec 31, 2012:

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

[BlackBerry] Fullscreen Media Time Indicators Background Incorrect
https://bugs.webkit.org/show_bug.cgi?id=105888

Patch by Jeff Rogers <jrogers@rim.com> on 2012-12-31
Reviewed by George Staikos.

Internal PR 272699.
Update fullscreen media controls CSS to fix time container background
color. Also fix transparency to be 80% opaque to and the time
remaining text color to match the system player.

No new tests as there is no functionality change.

  • css/mediaControlsBlackBerryFullscreen.css:

(video:-webkit-full-screen::-webkit-media-controls-panel):
(video:-webkit-full-screen::-webkit-media-controls-fullscreen-time-display-container):
(video:-webkit-full-screen::-webkit-media-controls-fullscreen-timeline-container):
(video:-webkit-full-screen::-webkit-media-controls-fullscreen-current-time-display):
(video:-webkit-full-screen::-webkit-media-controls-fullscreen-time-remaining-display):
(video:-webkit-full-screen::-webkit-media-controls-fullscreen-button-container, video:-webkit-full-screen::-webkit-media-controls-play-button-container):
(video:-webkit-full-screen::-webkit-media-controls-placeholder):

11:53 AM Changeset in webkit [138599] by weinig@apple.com
  • 7 edits
    1 add in trunk/Source/WebKit2

Add supplementability for WebContext
https://bugs.webkit.org/show_bug.cgi?id=105878

Reviewed by Dan Bernstein.

Add WebContextSupplement and convert WebApplicationCacheManagerProxy to it.

  • UIProcess/API/C/WKContext.cpp:

(WKContextGetApplicationCacheManager):

  • UIProcess/WebApplicationCacheManagerProxy.cpp:

(WebKit::WebApplicationCacheManagerProxy::supplementName):
(WebKit::WebApplicationCacheManagerProxy::WebApplicationCacheManagerProxy):
(WebKit::WebApplicationCacheManagerProxy::contextDestroyed):
(WebKit::WebApplicationCacheManagerProxy::processDidClose):
(WebKit::WebApplicationCacheManagerProxy::refWebContextSupplement):
(WebKit::WebApplicationCacheManagerProxy::derefWebContextSupplement):
(WebKit::WebApplicationCacheManagerProxy::getApplicationCacheOrigins):
(WebKit::WebApplicationCacheManagerProxy::deleteEntriesForOrigin):
(WebKit::WebApplicationCacheManagerProxy::deleteAllEntries):

  • UIProcess/WebApplicationCacheManagerProxy.h:

(WebApplicationCacheManagerProxy):

  • UIProcess/WebContext.cpp:

(WebKit::WebContext::WebContext):
(WebKit::WebContext::~WebContext):
(WebKit::WebContext::shouldTerminate):
(WebKit::WebContext::disconnectProcess):

  • UIProcess/WebContext.h:

(WebKit::WebContext::supplement):
(WebKit::WebContext::addSupplement):

  • UIProcess/WebContextSupplement.h: Added.

(WebKit::WebContextSupplement::WebContextSupplement):
(WebKit::WebContextSupplement::~WebContextSupplement):
(WebKit::WebContextSupplement::ref):
(WebKit::WebContextSupplement::deref):
(WebKit::WebContextSupplement::context):
(WebKit::WebContextSupplement::clearContext):

  • WebKit2.xcodeproj/project.pbxproj:
8:56 AM Changeset in webkit [138598] by commit-queue@webkit.org
  • 4 edits in trunk

[EFL][WebGL] Add compile time support for GLES2.
https://bugs.webkit.org/show_bug.cgi?id=105816

Patch by Kondapally Kalyan <kalyan.kondapally@intel.com> on 2012-12-31
Reviewed by Kenneth Rohde Christiansen.

.:

This patch adds build support for GLES2.
GLES2 support can be enabled during compile time by passing -DENABLE_GLES2=ON as
cmake config parameter. GLES2 options is not enabled by default. This patch doesn't
address all the build issues related to GLES2 but only adds the needed support in
OptionsEfl.cmake.

  • Source/cmake/OptionsEfl.cmake:

Source/WebCore:

This patch adds build support for GLES2.
GLES2 support can be enabled during compile time by passing -DENABLE_GLES2=ON as
cmake config parameter. GLES2 options is not enabled by default. This patch doesn't
address all the build issues related to GLES2 but only adds the needed support in
PlatformEfl.cmake.

  • PlatformEfl.cmake:
8:31 AM Changeset in webkit [138597] by Christophe Dumez
  • 2 edits in trunk/Source/WebKit2

[EFL[WK2] Regression(r138574): Should not allocate memory in the fork
https://bugs.webkit.org/show_bug.cgi?id=105868

Reviewed by Kenneth Rohde Christiansen.

Store prefixed executable path in a CString before the fork to
avoid allocating memory in the fork by calling String::utf8().
Allocating memory in the fork may lead to deadlocks due to
FastMalloc and the fork not inheriting the threads.

  • UIProcess/Launcher/efl/ProcessLauncherEfl.cpp:

(WebKit::ProcessLauncher::launchProcess):

8:05 AM Changeset in webkit [138596] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebCore

[BlackBerry] The HTML5 video fullscreen time indicator on the right cuts clipped when the video is longer than 1 hour
https://bugs.webkit.org/show_bug.cgi?id=105887

Patch by Andy Chen <andchen@rim.com> on 2012-12-31
Reviewed by George Staikos.

The width of the time part was fixed. Remove it.

No new test is required because no functionality change.

  • platform/blackberry/RenderThemeBlackBerry.cpp:

(WebCore::RenderThemeBlackBerry::adjustMediaControlStyle):

5:54 AM Changeset in webkit [138595] by commit-queue@webkit.org
  • 4 edits
    1 move
    1 add
    1 delete in trunk

[GTK] Move ImageDiff to the Tools directory root
https://bugs.webkit.org/show_bug.cgi?id=105421

Patch by Carlos Garcia Campos <cgarcia@igalia.com> on 2012-12-31
Reviewed by Kenneth Rohde Christiansen.

.:

  • GNUmakefile.am: Remove the old ImageDiff makefile include.

Tools:

  • DumpRenderTree/gtk/GNUmakefile.ImageDiff.am: Removed.
  • GNUmakefile.am:
  • ImageDiff/gtk/ImageDiff.cpp: Renamed from Tools/DumpRenderTree/gtk/ImageDiff.cpp.
5:25 AM Changeset in webkit [138594] by commit-queue@webkit.org
  • 3 edits in trunk/LayoutTests

[EFL] css3/masking/clip-path-circle-relative-overflow.html needs expected results update after 97217
https://bugs.webkit.org/show_bug.cgi?id=105883

Unreviewed, EFL gardening.

Rebaseline css3/masking/clip-path-circle-relative-overflow.html because BUG 97217 has been resolved.

Patch by Kangil Han <kangil.han@samsung.com> on 2012-12-31

  • platform/efl/TestExpectations:
  • platform/efl/css3/masking/clip-path-circle-relative-overflow-expected.png:
4:50 AM Changeset in webkit [138593] by Carlos Garcia Campos
  • 12 edits in trunk/Source/WebKit2

[GTK] Add API to set a WebKitWebView in view source mode to WebKit2
https://bugs.webkit.org/show_bug.cgi?id=99315

Reviewed by Martin Robinson.

Add a generic API to set/get a view mode. For now there are only
two modes, Web and Source, but the API can be extended if we
eventually support other view modes like Printing for example.

  • UIProcess/API/gtk/WebKitWebView.cpp:

(webkitWebViewSetProperty): Implement setter for view-mode
property.
(webkitWebViewGetProperty): Implement getter for view-mode
property.
(webkit_web_view_class_init): Add view-source property.
(webkit_web_view_set_view_mode): Public method to set the view
mode.
(webkit_web_view_get_view_mode): Public method to get the view
mode.

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

(testWebViewMode):
(beforeAll):

  • UIProcess/WebFrameProxy.cpp:

(WebKit::WebFrameProxy::WebFrameProxy): Initialize
m_inViewSourceMode.
(WebKit::WebFrameProxy::setInViewSourceMode): Set
m_inViewSourceMode if it has changed and call
WebPageProxy::setFrameInViewSourceMode() to notify the WebProcess.

  • UIProcess/WebFrameProxy.h:

(WebKit::WebFrameProxy::inViewSourceMode): Return whether the
frame is currently in view source mode.

  • UIProcess/WebPageProxy.cpp:

(WebKit::WebPageProxy::setFrameInViewSourceMode): Send a message
to the WebProcess to set the given frame in view source mode or not.

  • UIProcess/WebPageProxy.h:

(WebPageProxy): Add setFrameInViewSourceMode().

  • WebProcess/WebPage/WebPage.cpp:

(WebKit::WebPage::setFrameInViewSourceMode): Call
setInViewSourceMode for the main frame.

  • WebProcess/WebPage/WebPage.h:

(WebPage): Add setFrameInViewSourceMode().

  • WebProcess/WebPage/WebPage.messages.in: Add new message

SetFrameInViewSourceMode.

12:57 AM Changeset in webkit [138592] by Christophe Dumez
  • 2 edits in trunk/LayoutTests

Unreviewed EFL gardening.

Mark editing/spelling/spelling-insert-html.html as flaky
since it sometimes hits an assertion in RenderObject
destructor.

  • platform/efl/TestExpectations:

Dec 30, 2012:

11:57 PM Changeset in webkit [138591] by Christophe Dumez
  • 2 edits in trunk/LayoutTests

Unreviewed EFL gardening.

Mark fast/dom/Range/range-extract-contents.html as flaky
since it sometimes hits an assertion in
WebCore::endOfContinuations().

  • platform/efl/TestExpectations:
11:49 PM Changeset in webkit [138590] by Christophe Dumez
  • 2 edits in trunk/LayoutTests

Unreviewed EFL gardening.

Mark editing/pasteboard/4806874.html as flaky since
it sometimes hits an assertion in RenderObject
destructor.

  • platform/efl/TestExpectations:
11:41 PM Changeset in webkit [138589] by Christophe Dumez
  • 2 edits in trunk/LayoutTests

Unreviewed EFL gardening.

Rebaseline fast/js/global-constructors.html after
TEMPLATE_ELEMENT was enabled for EFL port in r138586.

  • platform/efl/fast/js/global-constructors-expected.txt:
11:33 PM Changeset in webkit [138588] by commit-queue@webkit.org
  • 5 edits
    2 moves in trunk

[EFL] [WebGL] Rename EGLConfigHelper as EGLConfigSelector.
https://bugs.webkit.org/show_bug.cgi?id=105876

Patch by Kondapally Kalyan <kalyan.kondapally@intel.com> on 2012-12-30
Reviewed by Kenneth Rohde Christiansen.

This is to sync the naming conventions of our classes in both EGL and GLX implementations.
In our GLX implementation, class responsible for the same functionality is named as GLXConfigSelector.
This patch renames EGLConfigHelper as EGLConfigSelector.

.:

  • Source/cmake/OptionsEfl.cmake:

Source/WebCore:

  • PlatformEfl.cmake:
  • platform/graphics/surfaces/egl/EGLConfigSelector.cpp: Renamed from Source/WebCore/platform/graphics/surfaces/egl/EGLConfigHelper.cpp.

(WebCore):
(WebCore::SharedEGLDisplay::deref):
(WebCore::SharedEGLDisplay::sharedEGLDisplay):
(WebCore::SharedEGLDisplay::SharedEGLDisplay):
(WebCore::SharedEGLDisplay::cleanup):
(WebCore::SharedEGLDisplay::~SharedEGLDisplay):
(WebCore::EGLConfigSelector::EGLConfigSelector):
(WebCore::EGLConfigSelector::~EGLConfigSelector):
(WebCore::EGLConfigSelector::display):
(WebCore::EGLConfigSelector::pBufferContextConfig):
(WebCore::EGLConfigSelector::surfaceContextConfig):
(WebCore::EGLConfigSelector::nativeVisualId):
(WebCore::EGLConfigSelector::reset):
(WebCore::EGLConfigSelector::createConfig):

  • platform/graphics/surfaces/egl/EGLConfigSelector.h: Renamed from Source/WebCore/platform/graphics/surfaces/egl/EGLConfigHelper.h.

(WebCore):
(SharedEGLDisplay):
(WebCore::SharedEGLDisplay::create):
(EGLConfigSelector):

  • platform/graphics/surfaces/egl/EGLSurface.cpp:

(WebCore::EGLWindowTransportSurface::EGLWindowTransportSurface):
(WebCore::EGLWindowTransportSurface::configuration):
(WebCore::EGLWindowTransportSurface::destroy):

  • platform/graphics/surfaces/egl/EGLSurface.h:

(EGLWindowTransportSurface):

11:12 PM Changeset in webkit [138587] by mkwst@chromium.org
  • 3 edits in trunk/Tools

Add myself to watchlists for CSP and console logs.
https://bugs.webkit.org/show_bug.cgi?id=105874

Reviewed by Dirk Pranke.

Because I really, really, really need more email.

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

Add mkwst+watchlist@chromium.org as a contributor.

  • Scripts/webkitpy/common/config/watchlist:

Create watchlist categories for 1) the various mechanisms used to
generate console messages, and 2) CSP usage and files.

8:44 PM Changeset in webkit [138586] by gyuyoung.kim@samsung.com
  • 6 edits in trunk

.: [EFL] Enable TEMPLATE_ELEMENT feature
https://bugs.webkit.org/show_bug.cgi?id=105865

Reviewed by Laszlo Gombos.

  • Source/cmake/OptionsEfl.cmake: Add ENABLE_TEMPLATE_ELEMENT macro.

Tools: [EFL] Enable TEMPLATE_ELEMENT feature
https://bugs.webkit.org/show_bug.cgi?id=105865

Reviewed by Laszlo Gombos.

  • Scripts/webkitperl/FeatureList.pm: Enable template element feature for EFL port.

LayoutTests: [EFL] Enable TEMPLATE_ELEMENT feature
https://bugs.webkit.org/show_bug.cgi?id=105865

Reviewed by Laszlo Gombos.

  • platform/efl/TestExpectations: Unskip test cases for template element.
1:22 PM Changeset in webkit [138585] by Martin Robinson
  • 2 edits in trunk/Websites/bugs.webkit.org

PrettyDiff.rb fails to render image diffs with Ruby 1.9.3p194
https://bugs.webkit.org/show_bug.cgi?id=104932

Reviewed by Eric Seidel.

Ruby 1.9 added the concept of string encodings, so the use of `match' with binary data
treats it as UTF-8. Forcing the png_bytes string to have the "binary" encoding avoids
this error.

  • PrettyPatch/PrettyPatch.rb: Use force_encoding.
9:51 AM Changeset in webkit [138584] by Christophe Dumez
  • 2 edits in trunk/LayoutTests

Unreviewed EFL gardening.

Add several webaudio tests to TestExpectations since
they sometimes hit !m_finishedNodes.size() assertion
in AudioContext destructor.

  • platform/efl/TestExpectations:
9:37 AM Changeset in webkit [138583] by Christophe Dumez
  • 2 edits in trunk/LayoutTests

Unreviewed EFL gardening.

Unskip XSS Auditor test case that has been consistently
passing on EFL port for a while.

  • platform/efl/TestExpectations:
9:08 AM Changeset in webkit [138582] by Christophe Dumez
  • 2 edits in trunk/LayoutTests

Unreviewed EFL gardening.

Unskip fast/dom/shadow/host-wrapper-reclaimed.html
since it is passing after r137978.

  • platform/efl/TestExpectations:
9:05 AM Changeset in webkit [138581] by Christophe Dumez
  • 2 edits in trunk/LayoutTests

Unreviewed EFL gardening.

Update expectations for 2 fast/dom test cases so
that they are not reported as unexpected passes.

Those tests are passing because their generated
expectations are wrong.

  • platform/efl/TestExpectations:
8:57 AM Changeset in webkit [138580] by Philippe Normand
  • 4 edits in trunk

[GStreamer] webaudio/decode-audio-data-basic.html fails
https://bugs.webkit.org/show_bug.cgi?id=105298

Reviewed by Martin Robinson.

Source/WebCore:

  • platform/audio/gstreamer/AudioFileReaderGStreamer.cpp:

(WebCore::AudioFileReader::createBus): Return a null pointer if an
error happened while processing a media file.

LayoutTests:

  • platform/efl/TestExpectations: Unflag fixed test.
8:55 AM Changeset in webkit [138579] by Christophe Dumez
  • 2 edits in trunk/LayoutTests

Unreviewed EFL gardening.

Unskip text-decoration-style tests passing after
r126054 and r132076.

  • platform/efl/TestExpectations:
7:55 AM Changeset in webkit [138578] by Christophe Dumez
  • 2 edits in trunk/LayoutTests

Unreviewed EFL gardening.

Unskip cairo-related tests that are now consistently
passing on both WK1 and WK2 EFL.

  • platform/efl/TestExpectations:
7:45 AM Changeset in webkit [138577] by Christophe Dumez
  • 3 edits in trunk/LayoutTests

Unreviewed EFL gardening.

Unskip several compositing test cases that are passing
consistently on EFL WK2.

  • platform/efl-wk2/TestExpectations:
  • platform/efl/TestExpectations:
6:57 AM Changeset in webkit [138576] by Christophe Dumez
  • 3 edits in trunk/Source/WebCore

Regression(r138545): Makes WebAudio tests crash
https://bugs.webkit.org/show_bug.cgi?id=105869

Reviewed by Philippe Normand.

Stop using smart pointers and adoptGRef() for the
GstBus object and call gst_object_unref() manually.
This fixes assertions hits when calling adoptGRef()
since the GstBus object has a floating reference.

Alternatively, we could keep using smart pointers
and stop calling adoptGRef() so that
gst_object_ref_sink() is called and the floating
flag is cleared but I think it is clearer to do
it manually here.

No new tests, already covered by existing tests.

  • platform/audio/gstreamer/AudioDestinationGStreamer.cpp:

(WebCore::messageCallback):
(WebCore::AudioDestinationGStreamer::AudioDestinationGStreamer):
(WebCore::AudioDestinationGStreamer::~AudioDestinationGStreamer):

  • platform/audio/gstreamer/AudioFileReaderGStreamer.cpp:

(WebCore::AudioFileReader::~AudioFileReader):
(WebCore::AudioFileReader::decodeAudioForBusCreation):

6:52 AM Changeset in webkit [138575] by Csaba Osztrogonác
  • 2 edits in trunk/LayoutTests

Unreviewed gardening, unskip a now passing test.

  • platform/qt/TestExpectations:
3:56 AM Changeset in webkit [138574] by Christophe Dumez
  • 2 edits in trunk/Source/WebKit2

[EFL][WK2] Invalid use of temporary object in ProcessLauncher::launchProcess()
https://bugs.webkit.org/show_bug.cgi?id=105867

Reviewed by Kenneth Rohde Christiansen.

Fix invalid use of temporary object in ProcessLauncher::launchProcess()
for prefixedExecutablePath.

  • UIProcess/Launcher/efl/ProcessLauncherEfl.cpp:

(WebKit::ProcessLauncher::launchProcess):

1:51 AM Changeset in webkit [138573] by zandobersek@gmail.com
  • 2 edits in trunk/Tools

Unreviewed GTK gardening of the unit tests.

Skip the TestInspectorServer unit test that's timing out completely.

  • Scripts/run-gtk-tests:

(TestRunner):

Dec 29, 2012:

5:22 PM Changeset in webkit [138572] by Dimitri Glazkov
  • 3 edits in trunk/Source/WebCore

Rename StyleResolver's member variable that holds SelectorChecker to a proper name.
https://bugs.webkit.org/show_bug.cgi?id=105864

In https://bugs.webkit.org/show_bug.cgi?id=105660#c11, Antti suggested renaming m_checker to
m_selectorChecker. I've forgotten about that, but now I remembered it.

Reviewed by Eric Seidel.

No change in functionality, covered by existing tests.

  • css/StyleResolver.cpp/h: Renamed all instances of m_checker to m_selectorChecker.
4:29 PM Changeset in webkit [138571] by Dimitri Glazkov
  • 8 edits in trunk/Source/WebCore

Move pointer to Document up from SelectorChecker to StyleResolver.
https://bugs.webkit.org/show_bug.cgi?id=105863

Now that SelectorChecker is mostly stateless, it no longer needs to keep a pointer to Document, which
makes all the code in StyleResolver that reaches for it sort of weird.

Reviewed by Eric Seidel.

No change in functionality, covered by existing tests.

  • css/SelectorChecker.cpp:

(WebCore::SelectorChecker::SelectorChecker): Changed to take document as argument (only used to set bit fields).
(WebCore::SelectorChecker::checkOneSelector): Changed to use element document.
(WebCore::SelectorChecker::checkScrollbarPseudoClass): Ditto.

  • css/SelectorChecker.h:

(WebCore): Removed unnecessary Document forward declaration.
(SelectorChecker): Changed constructor declaration, and removed m_document member.

  • css/StyleResolver.cpp:

(WebCore::StyleResolver::StyleResolver): Adjusted SelectorChecker initializer accordingly.
(WebCore::StyleResolver::initElement): Changed to use own document accessor.
(WebCore::StyleResolver::initForStyleResolve): Ditto.
(WebCore::StyleResolver::matchUARules): Ditto.
(WebCore::StyleResolver::styleForPage): Ditto.
(WebCore::StyleResolver::applyProperty): Ditto.
(WebCore::StyleResolver::checkForGenericFamilyChange): Ditto.
(WebCore::StyleResolver::initializeFontStyle): Ditto.
(WebCore::StyleResolver::setFontSize): Ditto.

  • css/StyleResolver.h:

(WebCore::StyleResolver::document): Changed to use own member.
(WebCore::StyleResolver::documentSettings): Ditto.
(StyleResolver): Added m_document member.

  • dom/SelectorQuery.cpp:

(WebCore::SelectorQuery::matches): Changed callsite to reflect new constructor signature.
(WebCore::SelectorQuery::queryAll): Ditto.
(WebCore::SelectorQuery::queryFirst): Ditto.

  • html/shadow/ContentSelectorQuery.cpp:

(WebCore::ContentSelectorChecker::ContentSelectorChecker): Ditto.
(WebCore::ContentSelectorQuery::ContentSelectorQuery): Ditto.

  • html/shadow/ContentSelectorQuery.h:

(ContentSelectorChecker): Removed unnecessary argument.

2:17 PM Changeset in webkit [138570] by weinig@apple.com
  • 2 edits in trunk/Source/WebKit2

Windows build fix.

  • win/WebKit2Common.vsprops:
1:58 PM Changeset in webkit [138569] by weinig@apple.com
  • 21 edits
    1 add in trunk/Source/WebKit2

Add supplementability to the NetworkProcess
https://bugs.webkit.org/show_bug.cgi?id=105838

Reviewed by Dan Bernstein.

This patch adds a NetworkProcessSupplement, which is almost identical to
WebProcessSupplement, but has a different initialization override. It also
stops WebProcessSupplement from inheriting from MessageReceiver (and makes
everything that is a WebProcessSupplement also a MessageReceiver) as keeping
that would cause classes that need to be both WebProcessSupplements and
NetworkProcessSupplements inherit from MessageReceiver twice.

  • GNUmakefile.list.am:
  • NetworkProcess/NetworkProcess.cpp:

(WebKit::NetworkProcess::NetworkProcess):
(WebKit::NetworkProcess::downloadsAuthenticationManager):
(WebKit::NetworkProcess::initializeNetworkProcess):

  • NetworkProcess/NetworkProcess.h:

(WebKit):
(NetworkProcess):
(WebKit::NetworkProcess::supplement):
(WebKit::NetworkProcess::addSupplement):

  • Shared/Network/CustomProtocols/CustomProtocolManager.h:

(CustomProtocolManager):

  • Shared/Network/NetworkProcessSupplement.h: Added.

(WebKit):
(NetworkProcessSupplement):
(WebKit::NetworkProcessSupplement::~NetworkProcessSupplement):
(WebKit::NetworkProcessSupplement::initialize):

  • Target.pri:
  • WebKit2.xcodeproj/project.pbxproj:
  • WebProcess/ApplicationCache/WebApplicationCacheManager.h:
  • WebProcess/Authentication/AuthenticationManager.h:
  • WebProcess/Cookies/WebCookieManager.h:
  • WebProcess/Geolocation/WebGeolocationManager.h:
  • WebProcess/KeyValueStorage/WebKeyValueStorageManager.h:
  • WebProcess/MediaCache/WebMediaCacheManager.h:
  • WebProcess/Notifications/WebNotificationManager.h:
  • WebProcess/ResourceCache/WebResourceCacheManager.h:
  • WebProcess/WebCoreSupport/WebDatabaseManager.h:
  • WebProcess/WebProcessSupplement.h:
  • win/WebKit2.vcproj:
12:11 PM BuildingGtk edited by Philippe Normand
(diff)
11:10 AM Changeset in webkit [138568] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebCore

[BlackBerry] Cookies with an IP domain are not being loaded properly into memory
https://bugs.webkit.org/show_bug.cgi?id=105859

PR 271597

Patch by Otto Derek Cheung <otcheung@rim.com> on 2012-12-29
Reviewed by George Staikos.

The isIP flag isn't persistented in the database. We need to check it
again when we load it into memory.

  • platform/blackberry/CookieManager.cpp:

(WebCore::CookieManager::getBackingStoreCookies):

10:11 AM Changeset in webkit [138567] by commit-queue@webkit.org
  • 7 edits
    1 move in trunk/Source/WebCore

[EFL][WebGL] Refactor GLXImplementation.
https://bugs.webkit.org/show_bug.cgi?id=105825

Patch by Kondapally Kalyan <kalyan.kondapally@intel.com> on 2012-12-29
Reviewed by Kenneth Rohde Christiansen.

This patch decouples native window management and logic to find surface configuration in GLX implementation.

  • platform/graphics/opengl/GLDefs.h:
  • platform/graphics/surfaces/glx/GLXConfigSelector.h: Renamed from Source/WebCore/platform/graphics/surfaces/glx/GLXWindowResources.h.

(WebCore):
(GLXConfigSelector):
(WebCore::GLXConfigSelector::GLXConfigSelector):
(WebCore::GLXConfigSelector::~GLXConfigSelector):
(WebCore::GLXConfigSelector::visualInfo):
(WebCore::GLXConfigSelector::pBufferContextConfig):
(WebCore::GLXConfigSelector::surfaceContextConfig):
(WebCore::GLXConfigSelector::reset):
(WebCore::GLXConfigSelector::createConfig):

Helper class to find surface configuration.

  • platform/graphics/surfaces/glx/GLXContext.h:

(GLXOffScreenContext):

  • platform/graphics/surfaces/glx/GLXSurface.cpp:

(WebCore::GLXTransportSurface::GLXTransportSurface):
(WebCore::GLXTransportSurface::configuration):
(WebCore::GLXTransportSurface::setGeometry):
(WebCore::GLXTransportSurface::destroy):
(WebCore::GLXPBuffer::GLXPBuffer):
(WebCore::GLXPBuffer::initialize):
(WebCore::GLXPBuffer::configuration):

  • platform/graphics/surfaces/glx/GLXSurface.h:

(GLXTransportSurface):
(GLXPBuffer):

Inheritance changed from X11OffScreenWindow to GLPlatformSurface.

  • platform/graphics/surfaces/glx/X11WindowResources.cpp:

(WebCore):
(WebCore::X11OffScreenWindow::X11OffScreenWindow):
(WebCore::X11OffScreenWindow::reSizeWindow):
(WebCore::X11OffScreenWindow::createOffscreenWindow):
(WebCore::X11OffScreenWindow::destroyWindow):
(WebCore::X11OffScreenWindow::nativeSharedDisplay):
(WebCore::X11OffScreenWindow::setVisualInfo):
(WebCore::X11OffScreenWindow::isXRenderExtensionSupported):

  • platform/graphics/surfaces/glx/X11WindowResources.h:

(WebCore::SharedX11Resources::create):
(SharedX11Resources):
(X11OffScreenWindow):

Removed code related to surface configiration.

9:02 AM Changeset in webkit [138566] by commit-queue@webkit.org
  • 2 edits in trunk/LayoutTests

[GTK] Change bug for some inspector tests
https://bugs.webkit.org/show_bug.cgi?id=105842

Unreviewed gardening.

Bug 43332 was landed. But these tests still fail on GTK port.
And change bug to track the failures.

Patch by Seokju Kwon <Seokju Kwon> on 2012-12-29

  • platform/gtk/TestExpectations:
8:14 AM Changeset in webkit [138565] by aroben@webkit.org
  • 3 edits in trunk/Source/WebKit/mac

Web Inspector: Mac WK1 Inspector can't save/load timeline data
https://bugs.webkit.org/show_bug.cgi?id=105547

Reviewed by Pavel Feldman.

  • WebCoreSupport/WebInspectorClient.h: Declared new overrides and

member variables necessary for save support.

  • WebCoreSupport/WebInspectorClient.mm:

(WebInspectorFrontendClient::save):
(WebInspectorFrontendClient::append):
Implemented these to make saving timeline data work. (It seems to me
the InspectorFrontendClient API could be cleaned up a bit. I had to
look at Chromium source code to determine that I needed to call back
into the InspectorFrontendAPI JS object after saving/appending, and
that I needed to keep a map of passed URLs -> chosen URLs.)

(-[WebInspectorWindowController window:willPositionSheet:usingRect:]):
Tweaked the position of the sheet so that open/save sheets look a
little bette.r

(-[WebInspectorWindowController webView:runOpenPanelForFileButtonWithResultListener:allowMultipleFiles:]):
Implemented to make loading timeline data work.

4:41 AM Changeset in webkit [138564] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebCore

Web Inspector: [Timeline] cpu bar popover shows wrong duration / cpu time.
https://bugs.webkit.org/show_bug.cgi?id=105854

Patch by Eugene Klyuchnikov <eustas@chromium.org> on 2012-12-29
Reviewed by Vsevolod Vlasov.

Problem: CPU task indexes are already incremented before written to
data structure.

  • inspector/front-end/TimelinePanel.js:

(WebInspector.TimelinePanel.prototype._refreshMainThreadBars):
Use for loop to get correct indexes.

3:38 AM Changeset in webkit [138563] by loislo@chromium.org
  • 2 edits in trunk/Tools

Web Inspector: Native Memory Instrumentation plugin: do not generate 'not instrumented' warning for instrumented mutable members.
https://bugs.webkit.org/show_bug.cgi?id=105855

Reviewed by Vsevolod Vlasov.

Extract MemberExpr from ImplicitCastExpr. It happens when we instrument a mutable member because
addMember expects const T& and the mutable ref to member implicitly converts into const ref to member.

  • clang/ReportMemoryUsagePlugin/ReportMemoryUsage.cpp:

(ReportMemoryUsageVisitor):
(clang::AddMemberCallVisitor::VisitCallExpr):
(clang::ReportMemoryUsageVisitor::VisitCXXMethodDecl):
(clang::ReportMemoryUsageVisitor::checkMembersCoverage):

2:53 AM Changeset in webkit [138562] by loislo@chromium.org
  • 2 edits in trunk/Tools

Web Inspector: Native Memory Instrumentation plugin: move function bodies out of class declarations.
https://bugs.webkit.org/show_bug.cgi?id=105852

Reviewed by Alexander Pavlov.

  • clang/ReportMemoryUsagePlugin/ReportMemoryUsage.cpp:

(clang):
(AddMemberCallVisitor):
(ReportMemoryUsageVisitor):
(clang::ReportMemoryUsageAction::ParseArgs):
(clang::AddMemberCallVisitor::VisitCallExpr):
(clang::ReportMemoryUsageVisitor::VisitCXXMethodDecl):
(clang::ReportMemoryUsageVisitor::emitWarning):
(clang::ReportMemoryUsageVisitor::findInstrumentationMethod):
(clang::ReportMemoryUsageVisitor::needsToBeInstrumented):
(clang::ReportMemoryUsageVisitor::CheckMembersCoverage):

1:27 AM Changeset in webkit [138561] by loislo@chromium.org
  • 14 edits in trunk/Source

Web Inspector: Native Memory Instrumentation: instrument not instrumented members.
https://bugs.webkit.org/show_bug.cgi?id=105830

Reviewed by Vsevolod Vlasov.

In some cases we don't want to visit some class members.
As example we decided to skip pointers to interface classes such as GraphicLayerClient.
We could use addWeakPointer for them but it can't be used for nonpointer members.
In the offline discussion we came to a conclusion that we need a new instrumentation
method ignoreMember, which will be used for all the members which we won't like to visit/instrument.

DriveBy: Also I instrumented not yet instrumented members.

Source/WebCore:

  • bindings/v8/DOMWrapperMap.h:

(WebCore::DOMWrapperMap::reportMemoryUsage):

  • bindings/v8/ScriptWrappable.h:

(WebCore::ScriptWrappable::reportMemoryUsage):

  • html/HTMLMediaElement.cpp:

(WebCore::HTMLMediaElement::reportMemoryUsage):

  • platform/KURLGoogle.cpp:

(WebCore::KURLGooglePrivate::reportMemoryUsage):

  • platform/SharedBuffer.cpp:

(WebCore::SharedBuffer::reportMemoryUsage):

  • platform/audio/AudioArray.h:

(WebCore::AudioArray::reportMemoryUsage):

  • platform/graphics/GraphicsLayer.cpp:

(WebCore::GraphicsLayer::reportMemoryUsage):

  • rendering/RenderLayerCompositor.cpp:

(WebCore::RenderLayerCompositor::reportMemoryUsage):

  • rendering/RenderTableSection.cpp:

(WebCore::RenderTableSection::RowStruct::reportMemoryUsage):

  • rendering/style/RenderStyle.cpp:

(WebCore::RenderStyle::reportMemoryUsage):

  • rendering/style/StyleRareInheritedData.cpp:

(WebCore::StyleRareInheritedData::reportMemoryUsage):

Source/WTF:

  • wtf/MemoryInstrumentation.h:

(MemoryClassInfo):
(WTF::MemoryClassInfo::ignoreMember):

1:19 AM Changeset in webkit [138560] by commit-queue@webkit.org
  • 4 edits in trunk

Source/WebCore: Web Inspector: refactor DefaultTextEditor - annotate handlers and fix getters style
https://bugs.webkit.org/show_bug.cgi?id=105829

Patch by Andrey Lushnikov <lushnikov@chromium.com> on 2012-12-29
Reviewed by Pavel Feldman.

Refactor DefaultTextEditor.js: improve handler method annotations,
remove "get" prefix from method names, get rid of unnecessary
delegation (e.g. see _handleCut).

No new tests: no change in behaviour.

  • inspector/front-end/DefaultTextEditor.js:

(WebInspector.DefaultTextEditor):
(WebInspector.DefaultTextEditor.prototype._contextMenu):
(WebInspector.DefaultTextEditor.prototype.selection):
(WebInspector.DefaultTextEditor.prototype.lastSelection):
(WebInspector.DefaultTextEditor.prototype.wasShown):
(WebInspector.DefaultTextEditor.prototype.willHide):
(WebInspector.TextEditorChunkedPanel.prototype.createNewChunk):
(WebInspector.TextEditorMainPanel):
(WebInspector.TextEditorMainPanel.prototype.wasShown):
(WebInspector.TextEditorMainPanel.prototype.willHide):
(WebInspector.TextEditorMainPanel.prototype.populateContextMenu):
(WebInspector.TextEditorMainPanel.prototype.splitChunkOnALine):
(WebInspector.TextEditorMainPanel.prototype.expandChunks):
(WebInspector.TextEditorMainPanel.prototype._paintLineChunks):
(WebInspector.TextEditorMainPanel.prototype.selection):
(WebInspector.TextEditorMainPanel.prototype.lastSelection):
(WebInspector.TextEditorMainPanel.prototype._applyDomUpdates):
(WebInspector.TextEditorMainPanel.prototype._handleSelectionChange):
(WebInspector.TextEditorMainPanel.prototype._handleTextInput):
(WebInspector.TextEditorMainPanel.prototype.handleKeyDown):
(WebInspector.TextEditorMainPanel.prototype._handleCut):

LayoutTests: Web Inspector: refactor DefaultTextEditor
https://bugs.webkit.org/show_bug.cgi?id=105829

Patch by Andrey Lushnikov <lushnikov@chromium.com> on 2012-12-29
Reviewed by Pavel Feldman.

Fix test to correlate with refactoring of DefaultTextEditor.

  • inspector/editor/text-editor-selection.html:
12:02 AM Changeset in webkit [138559] by zandobersek@gmail.com
  • 2 edits in trunk/Tools

[webkitpy] Omit webkitpy/thirdparty/BeautifulSoup.py from code coverage checking
https://bugs.webkit.org/show_bug.cgi?id=105835

Reviewed by Dirk Pranke.

The BeautifulSoup module is a third-party import and as such we don't have much control
over its testing. Because of that it should be omitted from code coverage checking.

  • Scripts/webkitpy/test/main.py:

(Tester._run_tests):

Note: See TracTimeline for information about the timeline view.