- 18 Apr, 2012 28 commits
-
-
commit-queue@webkit.org authored
https://bugs.webkit.org/show_bug.cgi?id=83264 Patch by Dominik Röttsches <dominik.rottsches@linux.intel.com> on 2012-04-18 Reviewed by Dimitri Glazkov. STORE_FONT_CUSTOM_PLATFORM_DATA was not defined for EFL. Fixing that by removing a long standing FIXME with the intention to turn the whitelist for this switch into a blacklist. * loader/cache/CachedFont.cpp: git-svn-id: http://svn.webkit.org/repository/webkit/trunk@114523 268f45cc-cd09-0410-ab3c-d52691b4dbfc
-
benjamin@webkit.org authored
https://bugs.webkit.org/show_bug.cgi?id=83841 Patch by Benjamin Poulain <bpoulain@apple.com> on 2012-04-18 Reviewed by Geoffrey Garen. We do not need to obtain the replaceValue until we have a match. By moving the intialization of replaceValue when needed, we save a few instructions when there is no match. * runtime/StringPrototype.cpp: (JSC::replaceUsingRegExpSearch): (JSC::replaceUsingStringSearch): (JSC::stringProtoFuncReplace): git-svn-id: http://svn.webkit.org/repository/webkit/trunk@114521 268f45cc-cd09-0410-ab3c-d52691b4dbfc
-
leviw@chromium.org authored
https://bugs.webkit.org/show_bug.cgi?id=84098 Reviewed by Eric Seidel. Shadows do not flow with the page, so sub-pixel layout doesn't actually offer any benefit that couldn't have been attained before that conversion. With that in mind, this patch reverts drop shadow offsets to integers, but also cleans up the code by switching the x/y location pair to be an IntPoint. No new tests. No change in behavior. * css/CSSComputedStyleDeclaration.cpp: (WebCore::CSSComputedStyleDeclaration::valueForFilter): * css/CSSStyleSelector.cpp: (WebCore::CSSStyleSelector::collectMatchingRulesForList): * css/SVGCSSStyleSelector.cpp: (WebCore::CSSStyleSelector::applySVGProperty): * page/animation/AnimationBase.cpp: (WebCore::blendFunc): (WebCore::shadowForBlending): * platform/animation/AnimationUtilities.h: (WebCore::blend): New blend function that operates on IntPoints. (WebCore): * platform/chromium/support/WebFilterOperations.cpp: (WebKit::WebDropShadowFilterOperation): * platform/graphics/filters/FilterOperation.cpp: (WebCore::DropShadowFilterOperation::blend): * platform/graphics/filters/FilterOperation.h: (WebCore::DropShadowFilterOperation::clone): (WebCore::DropShadowFilterOperation::x): (WebCore::DropShadowFilterOperation::y): (WebCore::DropShadowFilterOperation::location): Preserved the comment about lengths. (WebCore::DropShadowFilterOperation::operator==): (WebCore::DropShadowFilterOperation::DropShadowFilterOperation): (DropShadowFilterOperation): * rendering/RenderBoxModelObject.cpp: (WebCore::areaCastingShadowInHole): Reverted to integers since this operates on the IntRect from a RoundedRect. (WebCore::RenderBoxModelObject::paintBoxShadow): Reduced the complexity and unnecessary conversion between LayoutUnits and integers by using all integers after we calculate the pixel-snapped RoundedRect that we use for painting. * rendering/style/RenderStyle.cpp: (WebCore::RenderStyle::getShadowExtent): (WebCore::RenderStyle::getShadowHorizontalExtent): (WebCore::RenderStyle::getShadowVerticalExtent): * rendering/style/ShadowData.cpp: (WebCore::ShadowData::ShadowData): (WebCore::ShadowData::operator==): (WebCore::calculateShadowExtent): (WebCore::ShadowData::adjustRectForShadow): * rendering/style/ShadowData.h: (WebCore::ShadowData::ShadowData): (WebCore::ShadowData::x): (WebCore::ShadowData::y): (WebCore::ShadowData::location): (ShadowData): git-svn-id: http://svn.webkit.org/repository/webkit/trunk@114520 268f45cc-cd09-0410-ab3c-d52691b4dbfc
-
haraken@chromium.org authored
https://bugs.webkit.org/show_bug.cgi?id=84202 Reviewed by Nate Chapin. The final objective is to pass Isolate around in V8 bindings. This patch adds an optional Isolate argument to wrap(). After rewriting all wrap() callers so that they pass Isolate to wrap(), I'll make the Isolate argument non-optional. No tests. No change in behavior. * bindings/scripts/CodeGeneratorV8.pm: Modified as described above. (GenerateHeader): * bindings/v8/custom/V8DocumentCustom.cpp: Ditto. (WebCore::toV8): * bindings/v8/custom/V8HTMLDocumentCustom.cpp: Ditto. (WebCore::toV8): * bindings/v8/custom/V8NodeCustom.cpp: Ditto. (WebCore::toV8Slow): * bindings/v8/custom/V8SVGDocumentCustom.cpp: Ditto. (WebCore::toV8): * dom/make_names.pl: Ditto. (printWrapperFactoryCppFile): * bindings/scripts/test/V8/V8Float64Array.h: Updated run-bindings-tests results. (V8Float64Array): (WebCore::V8Float64Array::wrap): * bindings/scripts/test/V8/V8TestActiveDOMObject.h: (V8TestActiveDOMObject): (WebCore::V8TestActiveDOMObject::wrap): (WebCore::toV8): * bindings/scripts/test/V8/V8TestCustomNamedGetter.h: (V8TestCustomNamedGetter): (WebCore::V8TestCustomNamedGetter::wrap): (WebCore::toV8): * bindings/scripts/test/V8/V8TestEventConstructor.h: (V8TestEventConstructor): (WebCore::V8TestEventConstructor::wrap): (WebCore::toV8): * bindings/scripts/test/V8/V8TestEventTarget.h: (V8TestEventTarget): (WebCore::V8TestEventTarget::wrap): (WebCore::toV8): * bindings/scripts/test/V8/V8TestInterface.h: (V8TestInterface): (WebCore::V8TestInterface::wrap): (WebCore::toV8): * bindings/scripts/test/V8/V8TestMediaQueryListListener.h: (V8TestMediaQueryListListener): (WebCore::V8TestMediaQueryListListener::wrap): (WebCore::toV8): * bindings/scripts/test/V8/V8TestNamedConstructor.h: (V8TestNamedConstructor): (WebCore::V8TestNamedConstructor::wrap): (WebCore::toV8): * bindings/scripts/test/V8/V8TestNode.h: (V8TestNode): (WebCore::V8TestNode::wrap): (WebCore::toV8): * bindings/scripts/test/V8/V8TestObj.h: (V8TestObj): (WebCore::V8TestObj::wrap): (WebCore::toV8): * bindings/scripts/test/V8/V8TestSerializedScriptValueInterface.h: (V8TestSerializedScriptValueInterface): (WebCore::V8TestSerializedScriptValueInterface::wrap): (WebCore::toV8): git-svn-id: http://svn.webkit.org/repository/webkit/trunk@114519 268f45cc-cd09-0410-ab3c-d52691b4dbfc
-
achicu@adobe.com authored
https://bugs.webkit.org/show_bug.cgi?id=83960 Reviewed by Simon Fraser. The local clip rect should not be used when calculating the bounds of a filter area. Otherwise drop-shadow might not know about the pixels outside the clipping rectangle, even though the actual shadow might be inside it. No new tests added in this patch, but this patch fixes two existing tests that fail. LayoutTests/css3/filters/filter-repaint-shadow-clipped.html LayoutTests/css3/filters/filter-repaint-shadow-rotated.html * rendering/RenderLayer.cpp: (WebCore::RenderLayer::paintLayerContents): (WebCore::RenderLayer::calculateLayerBounds): * rendering/RenderLayer.h: git-svn-id: http://svn.webkit.org/repository/webkit/trunk@114518 268f45cc-cd09-0410-ab3c-d52691b4dbfc
-
pilgrim@chromium.org authored
https://bugs.webkit.org/show_bug.cgi?id=83109 Reviewed by Kentaro Hara. Now that incrementStatsCounter has been moved from PlatformSupport.h to Platform.h, we need to be able to call it directly from WebCore/bindings/v8/v8Proxy.h (for the INC_STATS macro). That means we need a new StatsCounter class in WebCore/platform/ and an implementation in WebCore/platform/chromium/StatsCounterChromium.cpp. Other ports are welcome to implement their own stats counters if they wish. This pattern was based on the HistogramSupport/HistogramSupportChromium classes. * CMakeLists.txt: * GNUmakefile.list.am: * Target.pri: * WebCore.gypi: * WebCore.vcproj/WebCore.vcproj: * WebCore.xcodeproj/project.pbxproj: * bindings/v8/V8Proxy.h: * bindings/v8/custom/V8DedicatedWorkerContextCustom.cpp: (WebCore::V8DedicatedWorkerContext::postMessageCallback): (WebCore::V8DedicatedWorkerContext::webkitPostMessageCallback): * bindings/v8/custom/V8NotificationCenterCustom.cpp: (WebCore::V8NotificationCenter::createHTMLNotificationCallback): (WebCore::V8NotificationCenter::createNotificationCallback): (WebCore::V8NotificationCenter::requestPermissionCallback): * bindings/v8/custom/V8WorkerContextCustom.cpp: (WebCore::V8WorkerContext::importScriptsCallback): (WebCore::V8WorkerContext::setTimeoutCallback): (WebCore::V8WorkerContext::setIntervalCallback): * platform/StatsCounter.cpp: Added. (WebCore): (WebCore::StatsCounter::incrementStatsCounter): * platform/StatsCounter.h: Added. (WebCore): (StatsCounter): * platform/chromium/StatsCounterChromium.cpp: Added. (WebCore): (WebCore::StatsCounter::incrementStatsCounter): git-svn-id: http://svn.webkit.org/repository/webkit/trunk@114515 268f45cc-cd09-0410-ab3c-d52691b4dbfc
-
apavlov@chromium.org authored
https://bugs.webkit.org/show_bug.cgi?id=84139 http://code.google.com/p/chromium/issues/detail?id=123432 Source/WebCore: Do not update the window rect on the wrong client but instead return the popup widget's new screen coordinates from the refresh() method. Reviewed by Kent Tamura. * platform/chromium/PopupContainer.cpp: (WebCore::PopupContainer::refresh): * platform/chromium/PopupContainer.h: (PopupContainer): Source/WebKit/chromium: Do not use the PopupContainer's frameRect() to update the popup client's window rect, since PopupContainer is always positioned at (0, 0) inside the client's window rect, and the latter one is positioned relative to the screen origin. Reviewed by Kent Tamura. * src/WebViewImpl.cpp: (WebKit::WebViewImpl::refreshAutofillPopup): git-svn-id: http://svn.webkit.org/repository/webkit/trunk@114513 268f45cc-cd09-0410-ab3c-d52691b4dbfc
-
mhahnenberg@apple.com authored
https://bugs.webkit.org/show_bug.cgi?id=83919 Reviewed by Geoffrey Garen. * API/JSContextRef.cpp: Used the new didAbandonObjectGraph callback to indicate that now that we've released a global object, we're abandoning a potentially large number of objects that JSC might want to collect. * heap/CopiedSpace.cpp: (JSC::CopiedSpace::tryAllocateSlowCase): Added the call to timer's willAllocate function to indicate that we've hit a slow path and are allocating now, so schedule the timer. * heap/Heap.cpp: (JSC::Heap::Heap): (JSC::Heap::collectAllGarbage): Removed the call to discardAllCompiledCode because it was causing us to throw away too much code during our benchmarks (especially vp8, which is very large and thus has large amounts of compiled code). (JSC::Heap::collect): Added the new call to didCollect at the conclusion of a collection so that we can cancel the timer if we no longer need to run a collection. Also added a check at the beginning of a collection to see if we should throw away our compiled code. Currently this is set to happen about once every minute. * heap/Heap.h: Added field to keep track of the last time we threw away our compiled code. * heap/MarkedAllocator.cpp: (JSC::MarkedAllocator::allocateSlowCase): Added call to willAllocate on the allocation slow path, just like in CopiedSpace. * runtime/GCActivityCallback.cpp: Added default stubs for non-CF platforms. (JSC::DefaultGCActivityCallback::willAllocate): (JSC): (JSC::DefaultGCActivityCallback::didCollect): (JSC::DefaultGCActivityCallback::didAbandonObjectGraph): * runtime/GCActivityCallback.h: Added new functions to make JSC's GC timer less arcane. This includes replacing the operator () with willAllocate() and adding an explicit didCollect() to cancel the timer after a collection occurs rather than relying on the way the timer is invoked to cancel itself. Also added a callback for when somebody else (e.g. WebCore or the JSC API) to notify JSC that they have just abandoned an entire graph of objects and that JSC might want to clean them up. (JSC::GCActivityCallback::~GCActivityCallback): (JSC::GCActivityCallback::willAllocate): (JSC::GCActivityCallback::didCollect): (JSC::GCActivityCallback::didAbandonObjectGraph): (JSC::GCActivityCallback::synchronize): (DefaultGCActivityCallback): * runtime/GCActivityCallbackCF.cpp: Re-wired all the run loop stuff to implement the aforementioned functions. We added a flag to check whether the timer was active because the call to CFRunLoopTimerSetNextFireDate actually turned out to be quite expensive (although Instruments couldn't tell us this). (DefaultGCActivityCallbackPlatformData): (JSC): (JSC::DefaultGCActivityCallbackPlatformData::timerDidFire): (JSC::DefaultGCActivityCallback::commonConstructor): (JSC::scheduleTimer): (JSC::cancelTimer): (JSC::DefaultGCActivityCallback::willAllocate): (JSC::DefaultGCActivityCallback::didCollect): (JSC::DefaultGCActivityCallback::didAbandonObjectGraph): git-svn-id: http://svn.webkit.org/repository/webkit/trunk@114511 268f45cc-cd09-0410-ab3c-d52691b4dbfc
-
simon.fraser@apple.com authored
https://bugs.webkit.org/show_bug.cgi?id=84221 Source/WebCore: Reviewed by Dean Jackson. When a RenderLayerBacking doesn't require its own backing store, and is in "paintIntoCompositingAncestor" mode, we would assert when trying to paint its m_foregroundLayer if it had one (because of a negative z-index child). The fix is to set the 'drawsContent' state on the m_foregroundLayer as well as m_graphicsLayer. Test: compositing/backing/no-backing-foreground-layer.html * rendering/RenderLayerBacking.cpp: (WebCore::RenderLayerBacking::updateDrawsContent): LayoutTests: Reviewed by Dean Jackson. New testcase. * compositing/backing/no-backing-foreground-layer-expected.txt: Added. * compositing/backing/no-backing-foreground-layer.html: Added. git-svn-id: http://svn.webkit.org/repository/webkit/trunk@114509 268f45cc-cd09-0410-ab3c-d52691b4dbfc
-
commit-queue@webkit.org authored
https://bugs.webkit.org/show_bug.cgi?id=84229 CMake caches ONLY_BUILD_JAVASCRIPTCORE=1 and will reuse the setting for subsequent builds, stopping WebCore from being built ever again. Instead of trying to emulate a build target using command line options, just build the existing jsc target directly. Patch by Thiago Marcos P. Santos <thiago.santos@intel.com> on 2012-04-18 Reviewed by Rob Buis. .: * CMakeLists.txt: Tools: * Scripts/build-jsc: git-svn-id: http://svn.webkit.org/repository/webkit/trunk@114508 268f45cc-cd09-0410-ab3c-d52691b4dbfc
-
commit-queue@webkit.org authored
https://bugs.webkit.org/show_bug.cgi?id=83874 Patch by Simon Pena <spena@igalia.com> on 2012-04-18 Reviewed by Philippe Normand. Source/WebKit/gtk: When the MediaPlaybackRequiresUserGesture setting was added, no gtk API was implemented to access it. This adds a property to gtk WebKit WebSettings, and connects it to WebKit WebView, thus allowing clients to programmatically modify it. * webkit/webkitwebsettings.cpp: (webkit_web_settings_class_init): (webkit_web_settings_set_property): (webkit_web_settings_get_property): * webkit/webkitwebsettingsprivate.h: * webkit/webkitwebview.cpp: (webkit_web_view_update_settings): (webkit_web_view_settings_notify): Tools: Now that there is gtk API to update the MediaPlaybackRequiresUserGesture property, set it to false from the DumpRenderTree tool when resetting defaults to consistent values. * DumpRenderTree/gtk/DumpRenderTree.cpp: (resetDefaultsToConsistentValues): LayoutTests: media/video-load-require-user-gesture.html and media/video-play-require-user-gesture.html where enabling MediaPlaybackRequiresUserGesture when run, so tests running concurrently would fail -they would pass when run independently. Since new API is added to ensure DumpRenderTree is able to reset that setting, having media/video-load-require-user-gesture.html and media/video-play-require-user-gesture.html no longer causes flakiness. * platform/gtk/test_expectations.txt: Unskip media/video-load-require-user-gesture.html and media/video-play-require-user-gesture.html git-svn-id: http://svn.webkit.org/repository/webkit/trunk@114506 268f45cc-cd09-0410-ab3c-d52691b4dbfc
-
yurys@chromium.org authored
https://bugs.webkit.org/show_bug.cgi?id=84240 DataGrid now has an explicit root node. All children are added/removed to that node. Reviewed by Pavel Feldman. Source/WebCore: * inspector/front-end/ApplicationCacheItemsView.js: (WebInspector.ApplicationCacheItemsView.prototype._createDataGrid): (WebInspector.ApplicationCacheItemsView.prototype._populateDataGrid): * inspector/front-end/CSSSelectorProfileView.js: (WebInspector.CSSSelectorProfileView): (WebInspector.CSSSelectorProfileView.prototype.rebuildGridItems): (WebInspector.CSSSelectorProfileView.prototype.refreshData): * inspector/front-end/CookieItemsView.js: (WebInspector.SimpleCookiesTable): (WebInspector.SimpleCookiesTable.prototype.setCookies): * inspector/front-end/CookiesTable.js: (WebInspector.CookiesTable): (WebInspector.CookiesTable.prototype._rebuildTable): * inspector/front-end/DOMStorageItemsView.js: (WebInspector.DOMStorageItemsView.prototype._dataGridForDOMStorageEntries): * inspector/front-end/DataGrid.js: (WebInspector.DataGrid): (WebInspector.DataGrid.createSortableDataGrid.sortDataGrid): (WebInspector.DataGrid.createSortableDataGrid): (WebInspector.DataGrid.prototype.setRootNode): (WebInspector.DataGrid.prototype.rootNode): (WebInspector.DataGrid.prototype.autoSizeColumns): (WebInspector.DataGrid.prototype._enumerateChildren): (WebInspector.DataGrid.prototype._keyDown): (WebInspector.DataGrid.prototype._contextMenuInDataTable): (WebInspector.DataGridNode.prototype.get revealed): (WebInspector.DataGridNode.prototype.get depth): (WebInspector.DataGridNode.prototype.appendChild): (WebInspector.DataGridNode.prototype.insertChild): (WebInspector.DataGridNode.prototype.removeChild): (WebInspector.DataGridNode.prototype.removeChildren): (WebInspector.DataGridNode.prototype.collapse): (WebInspector.DataGridNode.prototype.expand): (WebInspector.DataGridNode.prototype.reveal): (WebInspector.DataGridNode.prototype.revealAndSelect): (WebInspector.DataGridNode.prototype.traverseNextNode): (WebInspector.DataGridNode.prototype.traversePreviousNode): * inspector/front-end/HeapSnapshotDataGrids.js: (WebInspector.HeapSnapshotSortableDataGrid): (WebInspector.HeapSnapshotSortableDataGrid.prototype.dispose): (WebInspector.HeapSnapshotSortableDataGrid.prototype.changeNameFilter): (WebInspector.HeapSnapshotSortableDataGrid.prototype.updateVisibleNodes): (WebInspector.HeapSnapshotSortableDataGrid.prototype._performSorting): (WebInspector.HeapSnapshotContainmentDataGrid.prototype._defaultPopulateCount.100.expandRoute): (WebInspector.HeapSnapshotContainmentDataGrid.prototype.setDataSource): (WebInspector.HeapSnapshotContainmentDataGrid.prototype.sortingChanged): (WebInspector.HeapSnapshotRetainmentDataGrid.prototype.reset): (WebInspector.HeapSnapshotConstructorsDataGrid.prototype.populateChildren): (WebInspector.HeapSnapshotDiffDataGrid.prototype.setBaseDataSource): (WebInspector.HeapSnapshotDiffDataGrid.prototype.populateChildren.baseAggregatesReceived.aggregatesReceived.addNodeIfNonZeroDiff): (WebInspector.HeapSnapshotDiffDataGrid.prototype.populateChildren.baseAggregatesReceived.aggregatesReceived): (WebInspector.HeapSnapshotDiffDataGrid.prototype.populateChildren): (WebInspector.HeapSnapshotDominatorsDataGrid.prototype._defaultPopulateCount.25.setDataSource): (WebInspector.HeapSnapshotDominatorsDataGrid.prototype.sortingChanged): * inspector/front-end/HeapSnapshotGridNodes.js: (WebInspector.HeapSnapshotGenericObjectNode): * inspector/front-end/HeapSnapshotView.js: (WebInspector.HeapSnapshotView.prototype.performSearch): (WebInspector.HeapSnapshotView.prototype.refreshVisibleData): * inspector/front-end/IndexedDBViews.js: (WebInspector.IDBDataView.prototype._createDataGrid): (WebInspector.IDBDataView.prototype._updateData.callback): (WebInspector.IDBDataView.prototype._updateData): (WebInspector.IDBDataView.prototype.clear): * inspector/front-end/NetworkPanel.js: (WebInspector.NetworkLogView.prototype._createTable): (WebInspector.NetworkLogView.prototype.refresh): (WebInspector.NetworkLogView.prototype._reset): * inspector/front-end/ProfileDataGridTree.js: (WebInspector.ProfileDataGridNode.prototype.insertChild): (WebInspector.ProfileDataGridNode.prototype.removeChild): (WebInspector.ProfileDataGridNode.prototype.removeChildren): * inspector/front-end/ProfileView.js: (WebInspector.CPUProfileView.prototype.refresh): * inspector/front-end/treeoutline.js: LayoutTests: * inspector/profiler/heap-snapshot-test.js: (initialize_HeapSnapshotTest): * inspector/storage-panel-dom-storage.html: git-svn-id: http://svn.webkit.org/repository/webkit/trunk@114504 268f45cc-cd09-0410-ab3c-d52691b4dbfc
-
schenney@chromium.org authored
https://bugs.webkit.org/show_bug.cgi?id=84203 Unreviewed test expectation fix. The svg/transforms/transform-origin-css-property.xhtml test has a ref-test for it, so there should not be any image expectations. If the test if failing, we need to look at why the ref test is not valid, or why we differ. The expectations file update here is just some cleanup for https://bugs.webkit.org/show_bug.cgi?id=84230. * platform/chromium-mac/svg/transforms/transform-origin-css-property-expected.png: Removed. * platform/chromium-win/svg/transforms/transform-origin-css-property-expected.png: Removed. * platform/chromium/test_expectations.txt: git-svn-id: http://svn.webkit.org/repository/webkit/trunk@114503 268f45cc-cd09-0410-ab3c-d52691b4dbfc
-
commit-queue@webkit.org authored
https://bugs.webkit.org/show_bug.cgi?id=84232 Patch by Vineet Chaudhary <rgf748@motorola.com> on 2012-04-18 Reviewed by Kentaro Hara. Adding PassThisToCallback=XXX to attributes will be useful for the callbacks which requires to pass "this". This will help to identify the type(XXX) of "this" value in codegenerator. Tests: bindings/scripts/test/TestCallback.idl * bindings/scripts/CodeGeneratorJS.pm: (GenerateCallbackImplementation): Modified codegenerator to support [PassThisToCallback] * bindings/scripts/CodeGeneratorV8.pm: (GenerateCallbackImplementation): Ditto. * bindings/scripts/IDLAttributes.txt: Added PassThisToCallback=* * bindings/scripts/test/CPP/WebDOMTestCallback.cpp: Modofied results from run-bindings-tests. (WebDOMTestCallback::callbackWithBoolean): (WebDOMTestCallback::callbackRequiresThisToPass): * bindings/scripts/test/CPP/WebDOMTestCallback.h: Ditto. (WebDOMTestCallback): * bindings/scripts/test/GObject/WebKitDOMTestCallback.cpp: Ditto. (webkit_dom_test_callback_callback_with_boolean): (webkit_dom_test_callback_callback_requires_this_to_pass): * bindings/scripts/test/GObject/WebKitDOMTestCallback.h: Ditto. * bindings/scripts/test/JS/JSTestCallback.cpp: Ditto. (WebCore::JSTestCallback::callbackWithBoolean): (WebCore): (WebCore::JSTestCallback::callbackRequiresThisToPass): * bindings/scripts/test/JS/JSTestCallback.h: Ditto. (JSTestCallback): * bindings/scripts/test/ObjC/DOMTestCallback.h: Ditto. * bindings/scripts/test/ObjC/DOMTestCallback.mm: Ditto. (-[DOMTestCallback callbackWithBoolean:]): (-[DOMTestCallback callbackRequiresThisToPass:thisClassParam:]): * bindings/scripts/test/TestCallback.idl: Added test to verify generated code. * bindings/scripts/test/V8/V8TestCallback.cpp: Ditto. (WebCore::V8TestCallback::callbackWithBoolean): (WebCore): (WebCore::V8TestCallback::callbackRequiresThisToPass): * bindings/scripts/test/V8/V8TestCallback.h: Ditto. (V8TestCallback): git-svn-id: http://svn.webkit.org/repository/webkit/trunk@114502 268f45cc-cd09-0410-ab3c-d52691b4dbfc
-
pfeldman@chromium.org authored
* bindings/v8/V8IsolatedContext.cpp: git-svn-id: http://svn.webkit.org/repository/webkit/trunk@114501 268f45cc-cd09-0410-ab3c-d52691b4dbfc
-
macpherson@chromium.org authored
https://bugs.webkit.org/show_bug.cgi?id=84204 Reviewed by Kentaro Hara. Code calls next and then current instead of reusing the result of next. No new tests / no functionality changed. * css/CSSParser.cpp: (WebCore::CSSParser::parseImageSet): git-svn-id: http://svn.webkit.org/repository/webkit/trunk@114500 268f45cc-cd09-0410-ab3c-d52691b4dbfc
-
commit-queue@webkit.org authored
https://bugs.webkit.org/show_bug.cgi?id=83557 Patch by Vivek Galatage <vivekgalatage@gmail.com> on 2012-04-18 Reviewed by Pavel Feldman. The calls to inspector APIs are not put under ENABLE(INSPECTOR) guard. This change brings all such API calls under the guard. Source/WebKit/win: * WebCoreSupport/WebInspectorClient.cpp: * WebInspector.cpp: * WebNodeHighlight.cpp: (WebNodeHighlight::WebNodeHighlight): (WebNodeHighlight::update): * WebNodeHighlight.h: (WebNodeHighlight): * WebView.cpp: (WebView::WebView): (WebView::close): (WebView::initWithFrame): (WebView::inspector): * WebView.h: (WebView): Tools: * WebKitTestRunner/InjectedBundle/LayoutTestController.cpp: (WTR::LayoutTestController::showWebInspector): (WTR::LayoutTestController::closeWebInspector): (WTR::LayoutTestController::evaluateInWebInspector): (WTR::LayoutTestController::setJavaScriptProfilingEnabled): * WebKitTestRunner/TestInvocation.cpp: (WTR::TestInvocation::invoke): git-svn-id: http://svn.webkit.org/repository/webkit/trunk@114499 268f45cc-cd09-0410-ab3c-d52691b4dbfc
-
commit-queue@webkit.org authored
https://bugs.webkit.org/show_bug.cgi?id=84223 Patch by Jason Liu <jason.liu@torchmobile.com.cn> on 2012-04-18 Reviewed by George Staikos. .: * ManualTests/blackberry/clear-cookie-refresh-result.php: Added. * ManualTests/blackberry/clear-cookie-refresh.php: Added. Source/WebCore: Webkit uses the old ResourceRequest when refreshing. Its cookies' header isn't removed after clicking "clear cookies" button. We need to set cookies for this request again. We must click "clear button" to test, so have to write a manual test case. Test: ManualTests/blackberry/clear-cookie-refresh.php * platform/network/blackberry/ResourceRequestBlackBerry.cpp: (WebCore::ResourceRequest::initializePlatformRequest): git-svn-id: http://svn.webkit.org/repository/webkit/trunk@114498 268f45cc-cd09-0410-ab3c-d52691b4dbfc
-
pfeldman@chromium.org authored
Web Inspector: [Chromium] Module's require injects scripts before the debug id is set to the context. https://bugs.webkit.org/show_bug.cgi?id=84166 Reviewed by Yury Semikhatsky. V8IsolatedContext was getting debug id assigned post-construction. At the same time, it was compiling all its module scripts within the constructor. As a result, scripts ended up in the main world's list. * bindings/v8/V8IsolatedContext.cpp: (WebCore::setInjectedScriptContextDebugId): (WebCore): (WebCore::V8IsolatedContext::V8IsolatedContext): * bindings/v8/V8Proxy.cpp: (WebCore::V8Proxy::evaluateInIsolatedWorld): * bindings/v8/V8Proxy.h: (V8Proxy): git-svn-id: http://svn.webkit.org/repository/webkit/trunk@114497 268f45cc-cd09-0410-ab3c-d52691b4dbfc
-
commit-queue@webkit.org authored
[Qt][WK2] Minibrowser asserts on startup at QtViewportInteractionEngine::ensureContentWithinViewportBoundary() https://bugs.webkit.org/show_bug.cgi?id=84172 Patch by Zalan Bujtas <zbujtas@gmail.com> on 2012-04-18 Reviewed by Simon Hausmann. After the viewport computing refactor, ensureContentWithinViewportBoundary() can be called with resumed page. QtViewportInteractionEngine::ensureContentWithinViewportBoundary() functionality does not necessarily require the page to be suspended. It's the caller's context that determines whether the page needs to be suspended, so it's the caller's responsibility to enforce the ASSERT(). Remove the invalid ASSERT on m_suspendCount at QtViewportInteractionEngine::ensureContentWithinViewportBoundary() * UIProcess/qt/QtViewportInteractionEngine.cpp: (WebKit::QtViewportInteractionEngine::ensureContentWithinViewportBoundary): git-svn-id: http://svn.webkit.org/repository/webkit/trunk@114496 268f45cc-cd09-0410-ab3c-d52691b4dbfc
-
toyoshim@chromium.org authored
https://bugs.webkit.org/show_bug.cgi?id=84230 * platform/chromium/test_expectations.txt: git-svn-id: http://svn.webkit.org/repository/webkit/trunk@114495 268f45cc-cd09-0410-ab3c-d52691b4dbfc
-
pfeldman@chromium.org authored
https://bugs.webkit.org/show_bug.cgi?id=84179 Reviewed by Yury Semikhatsky. Source/WebCore: Perform total update upon modifications of the document node. Test: inspector/elements/delete-from-document.html * inspector/InspectorDOMAgent.cpp: (WebCore::InspectorDOMAgent::didRemoveDOMNode): * inspector/front-end/ElementsTreeOutline.js: LayoutTests: * http/tests/inspector/elements-test.js: (initialize_ElementTest.InspectorTest.dumpElementsTree.print): (initialize_ElementTest.InspectorTest.dumpElementsTree): * inspector/elements/delete-from-document-expected.txt: Added. * inspector/elements/delete-from-document.html: Added. git-svn-id: http://svn.webkit.org/repository/webkit/trunk@114494 268f45cc-cd09-0410-ab3c-d52691b4dbfc
-
commit-queue@webkit.org authored
https://bugs.webkit.org/show_bug.cgi?id=82930 Support for concurrent HTML5 audio improvements being made in the platform library, which need to be aware of tabs and tab visibility. PR96004. Patch by Max Feil <mfeil@rim.com> on 2012-04-18 Reviewed by George Staikos. Test: media/audio-concurrent-supported.html * platform/blackberry/PageClientBlackBerry.h: * platform/graphics/blackberry/MediaPlayerPrivateBlackBerry.cpp: (WebCore::MediaPlayerPrivate::MediaPlayerPrivate): (WebCore::MediaPlayerPrivate::showErrorDialog): (WebCore::MediaPlayerPrivate::isTabVisible): (WebCore): * platform/graphics/blackberry/MediaPlayerPrivateBlackBerry.h: (MediaPlayerPrivate): Source/WebKit/blackberry: [BlackBerry] Tab awareness for HTML5 concurrent audio https://bugs.webkit.org/show_bug.cgi?id=82930 Support for concurrent HTML5 audio improvements being made in the platform library, which need to be aware of tabs and tab visibility. PR96004. Patch by Max Feil <mfeil@rim.com> on 2012-04-18 Reviewed by George Staikos. * Api/WebPageClient.h: * Api/WebPage_p.h: (BlackBerry::WebKit::WebPagePrivate::isVisible): LayoutTests: [BlackBerry] Tab awareness for HTML5 concurrent audio https://bugs.webkit.org/show_bug.cgi?id=82930 Layout test for concurrent HTML5 audio. This is a simple test to confirm that multiple audio elements can play at the same time. Patch by Max Feil <mfeil@rim.com> on 2012-04-18 Reviewed by George Staikos. * media/audio-concurrent-supported-expected.txt: Added. * media/audio-concurrent-supported.html: Added. git-svn-id: http://svn.webkit.org/repository/webkit/trunk@114493 268f45cc-cd09-0410-ab3c-d52691b4dbfc
-
kkristof@inf.u-szeged.hu authored
Patch by Zoltan Arvai <zarvai@inf.u-szeged.hu> on 2012-04-18 * platform/qt/Skipped: fast/dom/shadow/adopt-node-with-shadow-root.html git-svn-id: http://svn.webkit.org/repository/webkit/trunk@114492 268f45cc-cd09-0410-ab3c-d52691b4dbfc
-
noel.gordon@gmail.com authored
https://bugs.webkit.org/show_bug.cgi?id=84022 Reviewed by Eric Seidel. ImageBuffer::getPremultipliedImageData() can fail. toDataURL() should check for a failure return, and return "data:," if so. No new tests. Covered by existing toDataURL tests, in particular: canvas/philip/tests/toDataURL.jpeg.alpha.html * platform/graphics/cg/ImageBufferCG.cpp: (WebCore::ImageBuffer::toDataURL): Rename |arr| variable to premultipliedData and return "data:," if premultipliedData is empty. Clean up whitespace and a comment. git-svn-id: http://svn.webkit.org/repository/webkit/trunk@114491 268f45cc-cd09-0410-ab3c-d52691b4dbfc
-
mario@webkit.org authored
https://bugs.webkit.org/show_bug.cgi?id=84010 Unreviewed, unskipping test after bug being fixed by r114311. * Scripts/run-gtk-tests: (TestRunner): Unskip now passing test. git-svn-id: http://svn.webkit.org/repository/webkit/trunk@114490 268f45cc-cd09-0410-ab3c-d52691b4dbfc
-
commit-queue@webkit.org authored
https://bugs.webkit.org/show_bug.cgi?id=84199 Patch by Jacky Jiang <zhajiang@rim.com> on 2012-04-17 Reviewed by George Staikos. Source/WebKit/blackberry: PR: 148279 When we set "user-scalable=no" in viewport meta-tag and change any other WebSettings, the WebPage can turn to be scalable. This is a regression of the change master_32/SHA:fa0f8ee9. In that change, we used setUserScalable(webSettings->isUserScalable()) in WebPagePrivate::didChangeSettings which was incorrect for non-DRT case, as webSettings->isUserScalable() would always return true by default. Create a new API WebPage::setUserScalable which will be used in DumpRenderTree.cpp. * Api/WebPage.cpp: (BlackBerry::WebKit::WebPage::setUserScalable): (WebKit): (BlackBerry::WebKit::WebPagePrivate::didChangeSettings): * Api/WebPage.h: Tools: Revert a part of the change master_32/SHA:fa0f8ee9 and use WebPage::setUserScalable instead. * DumpRenderTree/blackberry/DumpRenderTree.cpp: (BlackBerry::WebKit::DumpRenderTree::resetToConsistentStateBeforeTesting): git-svn-id: http://svn.webkit.org/repository/webkit/trunk@114488 268f45cc-cd09-0410-ab3c-d52691b4dbfc
-
commit-queue@webkit.org authored
https://bugs.webkit.org/show_bug.cgi?id=84225 Patch by Rachel Blum <groby@chromium.org> on 2012-04-17 Reviewed by David Levin. Tested with manual tests. * platform/graphics/skia/NativeImageSkia.cpp: (WebCore::NativeImageSkia::shouldCacheResampling): git-svn-id: http://svn.webkit.org/repository/webkit/trunk@114487 268f45cc-cd09-0410-ab3c-d52691b4dbfc
-
- 17 Apr, 2012 12 commits
-
-
toyoshim@chromium.org authored
* platform/chromium-win-vista/svg/W3C-SVG-1.1/masking-intro-01-f-expected.png: * platform/chromium-win/svg/W3C-SVG-1.1/masking-intro-01-f-expected.png: git-svn-id: http://svn.webkit.org/repository/webkit/trunk@114486 268f45cc-cd09-0410-ab3c-d52691b4dbfc
-
jer.noble@apple.com authored
https://bugs.webkit.org/show_bug.cgi?id=42337 Update wk2 Skipped list to skip new failing test. * platform/wk2/Skipped: git-svn-id: http://svn.webkit.org/repository/webkit/trunk@114485 268f45cc-cd09-0410-ab3c-d52691b4dbfc
-
jer.noble@apple.com authored
https://bugs.webkit.org/show_bug.cgi?id=84227 Update wk2 Skipped list to skip new failing test. * platform/wk2/Skipped: git-svn-id: http://svn.webkit.org/repository/webkit/trunk@114484 268f45cc-cd09-0410-ab3c-d52691b4dbfc
-
toyoshim@chromium.org authored
* platform/chromium-linux/svg/as-background-image/svg-as-background-6-expected.png: Added. * platform/chromium-mac-leopard/svg/as-background-image/svg-as-background-6-expected.png: Added. * svg/as-background-image/svg-as-background-6-expected.txt: Renamed from LayoutTests/platform/efl/svg/as-background-image/svg-as-background-6-expected.txt. git-svn-id: http://svn.webkit.org/repository/webkit/trunk@114483 268f45cc-cd09-0410-ab3c-d52691b4dbfc
-
morrita@google.com authored
https://bugs.webkit.org/show_bug.cgi?id=84127 Reviewed by Dimitri Glazkov. Source/WebCore: ShadowRoot cannot cannot be removed from its host, which means ShadowRoot cannot be adopted by any Document directly because the adoptNode() tries to remove it from its parent but it doesn't make sense for ShadowRoot. This change adds a guard to check such a case. Test: fast/dom/shadow/adopt-node-with-shadow-root.html * dom/Document.cpp: (WebCore::Document::adoptNode): LayoutTests: * fast/dom/shadow/adopt-node-with-shadow-root-expected.txt: Added. * fast/dom/shadow/adopt-node-with-shadow-root.html: Added. git-svn-id: http://svn.webkit.org/repository/webkit/trunk@114481 268f45cc-cd09-0410-ab3c-d52691b4dbfc
-
jer.noble@apple.com authored
REGRESSION (r114406-r114417): media/video-playing-and-pause.html failing on Lion Release (WebKit2 Tests) https://bugs.webkit.org/show_bug.cgi?id=84226 Remove expected failing results ſtor media/video-playing-and-pause.html. * platform/mac-wk2/media/video-playing-and-pause-expected.txt: Removed. git-svn-id: http://svn.webkit.org/repository/webkit/trunk@114480 268f45cc-cd09-0410-ab3c-d52691b4dbfc
-
toyoshim@chromium.org authored
https://bugs.webkit.org/show_bug.cgi?id=84024 * platform/chromium/test_expectations.txt: git-svn-id: http://svn.webkit.org/repository/webkit/trunk@114479 268f45cc-cd09-0410-ab3c-d52691b4dbfc
-
fpizlo@apple.com authored
https://bugs.webkit.org/show_bug.cgi?id=84218 Reviewed by Geoff Garen. * dfg/DFGByteCodeParser.cpp: (JSC::DFG::ByteCodeParser::makeSafe): * dfg/DFGCommon.h: (JSC::DFG::isX86): (DFG): git-svn-id: http://svn.webkit.org/repository/webkit/trunk@114478 268f45cc-cd09-0410-ab3c-d52691b4dbfc
-
commit-queue@webkit.org authored
https://bugs.webkit.org/show_bug.cgi?id=80912 Patch by Myles Maxfield <mmaxfield@google.com> on 2012-04-17 Reviewed by Hajime Morita. * wtf/BumpPointerAllocator.h: (WTF::BumpPointerPool::create): git-svn-id: http://svn.webkit.org/repository/webkit/trunk@114477 268f45cc-cd09-0410-ab3c-d52691b4dbfc
-
ap@apple.com authored
https://bugs.webkit.org/show_bug.cgi?id=84099 <rdar://problem/11133877> Reviewed by Sam Weinig. Use PDFDocuemnt instead of CGPDFDocument when printing PDFs, because PDFKit knows how to draw PDF buttons, and CG does not. * Shared/mac/PDFKitImports.h: Added. * Shared/mac/PDFKitImports.mm: Added. * UIProcess/API/mac/WKPrintingView.mm: Moved code for dealing with dynamically loaded PDFKit from WKPrintingView, as we now also need it elsewhere. * WebKit2.xcodeproj/project.pbxproj: Added * WebProcess/Plugins/PDF/BuiltInPDFView.cpp: Rebamed to .mm. * WebProcess/Plugins/PDF/BuiltInPDFView.h: Changed m_pdfDocument to PDFDocument. * WebProcess/Plugins/PDF/BuiltInPDFView.mm: Copied from Source/WebKit2/WebProcess/Plugins/PDF/BuiltInPDFView.cpp. (WebKit::BuiltInPDFView::pdfDocumentDidLoad): (WebKit::BuiltInPDFView::calculateSizes): (WebKit::BuiltInPDFView::paintContent): Changed m_pdfDocument to PDFDocument, and updated for the changes. We still use CGPDF when drawing to screen though, because that doesn't affect USPS. * WebProcess/Plugins/Plugin.h: (WebKit::Plugin::pdfDocumentForPrinting): Updated the type, and changed ifsed from CG to MAC, as we're now fully dependent on Cocoa. * WebProcess/Plugins/PluginView.h: (WebKit::PluginView::pdfDocumentForPrinting): Ditto. * WebProcess/WebPage/WebPage.cpp: (WebKit::WebPage::pdfDocumentForPrintingFrame): (WebKit::WebPage::beginPrinting): (WebKit::WebPage::computePagesForPrinting): (WebKit::WebPage::drawRectToPDF): (WebKit::WebPage::drawPagesToPDF): * WebProcess/WebPage/WebPage.h: Moved PDF document printing code to WebPageMac.mm, as it's now Objective C. * WebProcess/WebPage/mac/WebPageMac.mm: (WebKit::WebPage::computePagesForPrintingPDFDocument): (WebKit::roundCGFloat): (WebKit::drawPDFPage): While moving, also fixed a logic error in calculating the box (it used to intersect with an empty box). (WebKit::WebPage::drawRectToPDFFromPDFDocument): (WebKit::WebPage::drawPagesToPDFFromPDFDocument): git-svn-id: http://svn.webkit.org/repository/webkit/trunk@114476 268f45cc-cd09-0410-ab3c-d52691b4dbfc
-
commit-queue@webkit.org authored
https://bugs.webkit.org/show_bug.cgi?id=83649 Patch by John Bauman <jbauman@chromium.org> on 2012-04-17 Reviewed by James Robinson. We were waiting only on the canvas context, which with the new GPU scheduling was causing the RateLimiter not to ratelimit enough. We need to insert no-op commands in the compositor context, so that we'll wait for the canvas context and SwapBuffers as well. * platform/graphics/chromium/LayerRendererChromium.cpp: (WebCore::LayerRendererChromium::doNoOp): (WebCore): * platform/graphics/chromium/LayerRendererChromium.h: (LayerRendererChromium): * platform/graphics/chromium/RateLimiter.cpp: (WebCore::RateLimiter::create): (WebCore::RateLimiter::RateLimiter): (WebCore::RateLimiter::rateLimitContext): * platform/graphics/chromium/RateLimiter.h: (RateLimiterClient): (WebCore): (RateLimiter): * platform/graphics/chromium/cc/CCLayerTreeHost.cpp: (WebCore::CCLayerTreeHost::startRateLimiter): (WebCore::CCLayerTreeHost::rateLimit): (WebCore): * platform/graphics/chromium/cc/CCLayerTreeHost.h: (CCLayerTreeHost): * platform/graphics/chromium/cc/CCProxy.h: (CCProxy): * platform/graphics/chromium/cc/CCSingleThreadProxy.cpp: (WebCore::CCSingleThreadProxy::forceNoOpCommand): (WebCore): * platform/graphics/chromium/cc/CCSingleThreadProxy.h: (CCSingleThreadProxy): * platform/graphics/chromium/cc/CCThreadProxy.cpp: (WebCore::CCThreadProxy::forceNoOpCommand): (WebCore): (WebCore::CCThreadProxy::forceNoOpCommandOnImplThread): * platform/graphics/chromium/cc/CCThreadProxy.h: (CCThreadProxy): git-svn-id: http://svn.webkit.org/repository/webkit/trunk@114475 268f45cc-cd09-0410-ab3c-d52691b4dbfc
-
kling@webkit.org authored
<http://webkit.org/b/84219> Reviewed by Antti Koivisto. Change the identifier CSSPrimitiveValue cache in CSSValuePool from a HashMap to a fixed-size array of RefPtr<CSSPrimitiveValue>s. We have ~700 values total, so this is quite space efficient now that the CSSValuePool is globally shared. More importantly it avoids a hash lookup every time we need an identifier value. * css/CSSValuePool.h: * css/CSSValuePool.cpp: (WebCore::CSSValuePool::createIdentifierValue): git-svn-id: http://svn.webkit.org/repository/webkit/trunk@114474 268f45cc-cd09-0410-ab3c-d52691b4dbfc
-