Index: Source/WebCore/ChangeLog |
=================================================================== |
--- Source/WebCore/ChangeLog (revision 76822) |
+++ Source/WebCore/ChangeLog (working copy) |
@@ -1,3 +1,2499 @@ |
+2011-01-25 Adam Barth <abarth@webkit.org> |
+ |
+ Reviewed by Eric Seidel. |
+ |
+ Empty URLs are never display isolated |
+ https://bugs.webkit.org/show_bug.cgi?id=53053 |
+ |
+ This check mirrors the check in the local case. There isn't a good way |
+ to test this with a LayoutTest, but it is tested in Chromium (where |
+ this registry is used). |
+ |
+ * platform/SchemeRegistry.cpp: |
+ (WebCore::SchemeRegistry::shouldTreatURLSchemeAsDisplayIsolated): |
+ |
+2011-01-25 Sam Weinig <sam@webkit.org> |
+ |
+ Reviewed by Anders Carlsson. |
+ |
+ Pipe a timestamp down into the PlatformWheelEvent for the Mac. |
+ https://bugs.webkit.org/show_bug.cgi?id=53111 |
+ |
+ * platform/PlatformWheelEvent.h: |
+ (WebCore::PlatformWheelEvent::PlatformWheelEvent): |
+ (WebCore::PlatformWheelEvent::timestamp): |
+ Add timestamp member. |
+ |
+ * platform/mac/WheelEventMac.mm: |
+ (WebCore::PlatformWheelEvent::PlatformWheelEvent): |
+ Initialize the timestamp from the event. |
+ |
+2011-01-25 Sheriff Bot <webkit.review.bot@gmail.com> |
+ |
+ Unreviewed, rolling out r76625. |
+ http://trac.webkit.org/changeset/76625 |
+ https://bugs.webkit.org/show_bug.cgi?id=53119 |
+ |
+ "incorrect pixel test results" (Requested by rniwa on |
+ #webkit). |
+ |
+ * rendering/RenderBox.cpp: |
+ (WebCore::RenderBox::localCaretRect): |
+ |
+2011-01-25 Patrick Gansterer <paroga@webkit.org> |
+ |
+ Unreviewed WinCE build fix for r76614. |
+ |
+ * platform/FileSystem.h: |
+ |
+2011-01-21 Levi Weintraub <leviw@chromium.org> |
+ |
+ Reviewed by Darin Adler. |
+ |
+ Adding border and padding to the calculation of the local caret rect for RenderBoxes. |
+ |
+ Undo moves caret to invalid position |
+ https://bugs.webkit.org/show_bug.cgi?id=49744 |
+ |
+ Tests: editing/selection/caret-painting-after-paste-undo-rtl.html |
+ editing/selection/caret-painting-after-paste-undo.html |
+ |
+ * rendering/RenderBox.cpp: |
+ (WebCore::RenderBox::localCaretRect): |
+ |
+2011-01-25 Sam Weinig <sam@webkit.org> |
+ |
+ Reviewed by Anders Carlsson. |
+ |
+ Add hasPreciseScrollingDeltas bit to PlatformWheelEvent on the Mac. |
+ https://bugs.webkit.org/show_bug.cgi?id=53107 |
+ |
+ * platform/PlatformWheelEvent.h: |
+ (WebCore::PlatformWheelEvent::PlatformWheelEvent): |
+ (WebCore::PlatformWheelEvent::hasPreciseScrollingDeltas): |
+ * platform/mac/WheelEventMac.mm: |
+ (WebCore::PlatformWheelEvent::PlatformWheelEvent): |
+ |
+2011-01-20 Jer Noble <jer.noble@apple.com> |
+ |
+ Reviewed by Eric Carlson. |
+ |
+ REGRESSION (r72119): Audio never plays on Star Wars intro animation |
+ https://bugs.webkit.org/show_bug.cgi?id=52467 |
+ |
+ QuickTime's eat/m4a movie importer compontent doesn't list audio/m4a as a mime |
+ type which it supports, though it handles .m4a files just fine. Change the way |
+ we build the list of supported MIME Types through a new WebKitSystemInterface |
+ function. |
+ |
+ Caused by r72119, which adds system-specific extension->MIME entries to the cache |
+ before global entries, and the system-specific entries include QuickTime's registry |
+ entries which contain the audio/m4a MIME type, while its components do not. |
+ |
+ Test: media/audio-mpeg4-supported.html |
+ |
+ * WebCore.vcproj/QTMovieWinCommon.vsprops: |
+ * platform/graphics/win/QTMovie.cpp: |
+ (getMIMETypeCallBack): |
+ (initializeSupportedTypes): |
+ (QTMovie::countSupportedTypes): gSupportedTypes is now a CFArrayRef. |
+ (QTMovie::getSupportedType): Ditto. |
+ |
+2011-01-25 Hans Wennborg <hans@chromium.org> |
+ |
+ Reviewed by Jeremy Orlow. |
+ |
+ IndexedDB: Remove PlatformBridge::idbShutdown() |
+ https://bugs.webkit.org/show_bug.cgi?id=53077 |
+ |
+ Since Chromium r72157, this is not used anymore. The previous use was |
+ to signal to the embedder that IndexedDB was shutting down, but we |
+ agreed it is better if the embedder keeps track of that itself. |
+ |
+ No new tests: this does not change any functionality. |
+ |
+ * platform/chromium/PlatformBridge.h: |
+ * storage/IDBFactoryBackendInterface.cpp: |
+ * storage/IDBFactoryBackendInterface.h: |
+ (WebCore::IDBFactoryBackendInterface::~IDBFactoryBackendInterface): |
+ * storage/chromium/IDBFactoryBackendInterface.cpp: |
+ |
+2011-01-25 Darin Adler <darin@apple.com> |
+ |
+ Reviewed by Anders Carlsson. |
+ |
+ WebKit is using CSBackupSetItemExcluded incorrectly |
+ https://bugs.webkit.org/show_bug.cgi?id=53095 |
+ rdar://problem/8790540 |
+ |
+ * loader/icon/IconDatabase.cpp: |
+ (WebCore::IconDatabase::performOpenInitialization): Added code to |
+ exclude the database from backup one time, and record inside the |
+ database that this has been done. |
+ (WebCore::IconDatabase::wasExcludedFromBackup): Added. |
+ (WebCore::IconDatabase::setWasExcludedFromBackup): Added. |
+ * loader/icon/IconDatabase.h: Added new functions above. |
+ |
+ * platform/FileSystem.cpp: |
+ (WebCore::canExcludeFromBackup): Added. |
+ (WebCore::excludeFromBackup): Added. |
+ |
+ * platform/FileSystem.h: Added canExcludeFromBackup, excludeFromBackup, |
+ and pathAsURL functions. Cleaned up ifdefs and comments a bit and sorted |
+ things alphabetically, particularly platform-specific sections. |
+ |
+ * platform/cf/FileSystemCF.cpp: |
+ (WebCore::pathAsURL): Added. |
+ |
+ * platform/mac/FileSystemMac.mm: |
+ (WebCore::canExcludeFromBackup): Added. |
+ (WebCore::excludeFromBackup): Added. |
+ |
+ * platform/network/cf/FormDataStreamCFNet.cpp: |
+ (WebCore::advanceCurrentStream): Changed to call pathAsURL. |
+ * platform/network/mac/FormDataStreamMac.mm: |
+ (WebCore::advanceCurrentStream): Ditto. |
+ |
+2011-01-25 Helder Correia <helder@sencha.com> |
+ |
+ Reviewed by Dirk Schulze. |
+ |
+ REGRESSION(75139): SVG gradients are not applied to texts |
+ https://bugs.webkit.org/show_bug.cgi?id=52531 |
+ |
+ CGContextConcatCTM should use the CGLayer context, not the |
+ GraphicsContext. Also, the CTM needs to be adjusted (translated). |
+ This fixes SVG text gradient fill. |
+ |
+ Test: svg/css/composite-shadow-text.svg |
+ |
+ * platform/graphics/cg/GraphicsContextCG.cpp: |
+ (WebCore::GraphicsContext::fillRect): |
+ |
+2011-01-25 Benjamin Kalman <kalman@chromium.org> |
+ |
+ Reviewed by Ryosuke Niwa. |
+ |
+ Cannot extend or modify forward by word over a non-contenteditable region |
+ https://bugs.webkit.org/show_bug.cgi?id=53070 |
+ |
+ Test: editing/selection/extend-forward-by-word-over-non-editable.html |
+ |
+ Change all instances of honorEditableBoundaryAtOrAfter to honorEditableBoundaryAtOrBefore and vice versa in the |
+ functions which determine the end/start of words/lines/sentences in visible_units.cpp. |
+ |
+ This fixes the bug where moving forwards by a word over a non-contenteditable region would place the cursor |
+ inside that region, and then get moved back to the start of the word due to honorEditableBoundaryAtOrBefore. |
+ The cursor is now moved to the end of the region (which is effectively a noop in this case). |
+ |
+ * editing/visible_units.cpp: Change all instances of honorEditableBoundaryAtOrAfter to |
+ honorEditableBoundaryAtOrBefore and vice versa. |
+ (WebCore::previousWordPosition): |
+ (WebCore::nextWordPosition): |
+ (WebCore::startOfLine): |
+ (WebCore::endOfLine): |
+ (WebCore::previousSentencePosition): |
+ (WebCore::nextSentencePosition): |
+ (WebCore::logicalStartOfLine): |
+ (WebCore::logicalEndOfLine): |
+ |
+2011-01-25 Yael Aharon <yael.aharon@nokia.com> |
+ |
+ Unreviewed build fix. |
+ After r76466, efsrv.lib is used unconditionally. |
+ Don't guard it with ENABLE_NETSCAPE_PLUGIN_API. |
+ |
+ * WebCore.pro: |
+ |
+2011-01-24 Zhenyao Mo <zmo@google.com> |
+ |
+ Reviewed by Kenneth Russell. |
+ |
+ Style cleanup for WebGLRenderingContext |
+ https://bugs.webkit.org/show_bug.cgi?id=52352 |
+ |
+ * html/canvas/WebGLBuffer.cpp: |
+ * html/canvas/WebGLBuffer.h: |
+ * html/canvas/WebGLObject.cpp: |
+ * html/canvas/WebGLProgram.cpp: |
+ * html/canvas/WebGLRenderbuffer.cpp: |
+ * html/canvas/WebGLRenderingContext.cpp: |
+ (WebCore::WebGLRenderingContext::create): |
+ (WebCore::WebGLRenderingContext::blendFunc): |
+ (WebCore::WebGLRenderingContext::createShader): |
+ (WebCore::WebGLRenderingContext::deleteBuffer): |
+ (WebCore::WebGLRenderingContext::deleteShader): |
+ (WebCore::WebGLRenderingContext::deleteTexture): |
+ (WebCore::WebGLRenderingContext::disableVertexAttribArray): |
+ (WebCore::WebGLRenderingContext::validateIndexArrayPrecise): |
+ (WebCore::WebGLRenderingContext::validateRenderingState): |
+ (WebCore::WebGLRenderingContext::enableVertexAttribArray): |
+ (WebCore::WebGLRenderingContext::getUniform): |
+ (WebCore::WebGLRenderingContext::detachAndRemoveAllObjects): |
+ * html/canvas/WebGLRenderingContext.h: |
+ * html/canvas/WebGLShader.cpp: |
+ * html/canvas/WebGLTexture.cpp: |
+ * html/canvas/WebGLUniformLocation.cpp: |
+ * html/canvas/WebGLUniformLocation.h: |
+ |
+2011-01-24 Chris Marrin <cmarrin@apple.com> |
+ |
+ Reviewed by Eric Seidel. |
+ |
+ Change ENABLE_3D_CANVAS to ENABLE_WEBGL |
+ https://bugs.webkit.org/show_bug.cgi?id=53041 |
+ |
+ * Configurations/FeatureDefines.xcconfig: |
+ * DerivedSources.cpp: |
+ * GNUmakefile.am: |
+ * WebCore.pro: |
+ * bindings/generic/RuntimeEnabledFeatures.h: |
+ * bindings/js/JSArrayBufferCustom.cpp: |
+ * bindings/js/JSCanvasRenderingContextCustom.cpp: |
+ (WebCore::toJS): |
+ * bindings/js/JSDOMWindowCustom.cpp: |
+ * bindings/js/JSDataViewCustom.cpp: |
+ * bindings/js/JSDocumentCustom.cpp: |
+ * bindings/js/JSFloat32ArrayCustom.cpp: |
+ * bindings/js/JSHTMLCanvasElementCustom.cpp: |
+ (WebCore::JSHTMLCanvasElement::getContext): |
+ * bindings/js/JSInt16ArrayCustom.cpp: |
+ * bindings/js/JSInt32ArrayCustom.cpp: |
+ * bindings/js/JSInt8ArrayCustom.cpp: |
+ * bindings/js/JSUint16ArrayCustom.cpp: |
+ * bindings/js/JSUint32ArrayCustom.cpp: |
+ * bindings/js/JSUint8ArrayCustom.cpp: |
+ * bindings/js/JSWebGLRenderingContextCustom.cpp: |
+ * bindings/js/JSXMLHttpRequestCustom.cpp: |
+ (WebCore::JSXMLHttpRequest::markChildren): |
+ (WebCore::JSXMLHttpRequest::send): |
+ (WebCore::JSXMLHttpRequest::response): |
+ * bindings/v8/custom/V8ArrayBufferCustom.cpp: |
+ * bindings/v8/custom/V8ArrayBufferViewCustom.h: |
+ * bindings/v8/custom/V8DataViewCustom.cpp: |
+ * bindings/v8/custom/V8DocumentCustom.cpp: |
+ (WebCore::V8Document::getCSSCanvasContextCallback): |
+ * bindings/v8/custom/V8Float32ArrayCustom.cpp: |
+ * bindings/v8/custom/V8HTMLCanvasElementCustom.cpp: |
+ (WebCore::V8HTMLCanvasElement::getContextCallback): |
+ * bindings/v8/custom/V8Int16ArrayCustom.cpp: |
+ * bindings/v8/custom/V8Int32ArrayCustom.cpp: |
+ * bindings/v8/custom/V8Int8ArrayCustom.cpp: |
+ * bindings/v8/custom/V8Uint16ArrayCustom.cpp: |
+ * bindings/v8/custom/V8Uint32ArrayCustom.cpp: |
+ * bindings/v8/custom/V8Uint8ArrayCustom.cpp: |
+ * bindings/v8/custom/V8WebGLRenderingContextCustom.cpp: |
+ * bindings/v8/custom/V8XMLHttpRequestCustom.cpp: |
+ (WebCore::V8XMLHttpRequest::responseAccessorGetter): |
+ (WebCore::V8XMLHttpRequest::sendCallback): |
+ * features.pri: |
+ * html/HTMLCanvasElement.cpp: |
+ (WebCore::HTMLCanvasElement::getContext): |
+ (WebCore::HTMLCanvasElement::reset): |
+ * html/HTMLCanvasElement.h: |
+ * html/canvas/ArrayBuffer.cpp: |
+ * html/canvas/ArrayBuffer.idl: |
+ * html/canvas/ArrayBufferView.cpp: |
+ * html/canvas/ArrayBufferView.idl: |
+ * html/canvas/DataView.cpp: |
+ * html/canvas/DataView.idl: |
+ * html/canvas/Float32Array.cpp: |
+ * html/canvas/Float32Array.idl: |
+ * html/canvas/Int16Array.cpp: |
+ * html/canvas/Int16Array.idl: |
+ * html/canvas/Int32Array.cpp: |
+ * html/canvas/Int32Array.idl: |
+ * html/canvas/Int8Array.cpp: |
+ * html/canvas/Int8Array.idl: |
+ * html/canvas/OESStandardDerivatives.cpp: |
+ * html/canvas/OESStandardDerivatives.idl: |
+ * html/canvas/OESTextureFloat.cpp: |
+ * html/canvas/OESTextureFloat.idl: |
+ * html/canvas/Uint16Array.cpp: |
+ * html/canvas/Uint16Array.idl: |
+ * html/canvas/Uint32Array.cpp: |
+ * html/canvas/Uint32Array.idl: |
+ * html/canvas/Uint8Array.cpp: |
+ * html/canvas/Uint8Array.idl: |
+ * html/canvas/WebGLActiveInfo.idl: |
+ * html/canvas/WebGLBuffer.cpp: |
+ * html/canvas/WebGLBuffer.idl: |
+ * html/canvas/WebGLContextAttributes.cpp: |
+ * html/canvas/WebGLContextAttributes.idl: |
+ * html/canvas/WebGLContextEvent.idl: |
+ * html/canvas/WebGLExtension.cpp: |
+ * html/canvas/WebGLFramebuffer.cpp: |
+ * html/canvas/WebGLFramebuffer.idl: |
+ * html/canvas/WebGLGetInfo.cpp: |
+ * html/canvas/WebGLObject.cpp: |
+ * html/canvas/WebGLProgram.cpp: |
+ * html/canvas/WebGLProgram.idl: |
+ * html/canvas/WebGLRenderbuffer.cpp: |
+ * html/canvas/WebGLRenderbuffer.idl: |
+ * html/canvas/WebGLRenderingContext.cpp: |
+ * html/canvas/WebGLRenderingContext.idl: |
+ * html/canvas/WebGLShader.cpp: |
+ * html/canvas/WebGLShader.idl: |
+ * html/canvas/WebGLTexture.cpp: |
+ * html/canvas/WebGLTexture.idl: |
+ * html/canvas/WebGLUniformLocation.cpp: |
+ * html/canvas/WebGLUniformLocation.idl: |
+ * html/canvas/WebKitLoseContext.cpp: |
+ * html/canvas/WebKitLoseContext.idl: |
+ * page/DOMWindow.idl: |
+ * platform/graphics/ANGLEWebKitBridge.cpp: |
+ * platform/graphics/GraphicsContext3D.cpp: |
+ * platform/graphics/cg/GraphicsContext3DCG.cpp: |
+ * platform/graphics/gpu/DrawingBuffer.cpp: |
+ * platform/graphics/gpu/mac/DrawingBufferMac.mm: |
+ * platform/graphics/mac/GraphicsContext3DMac.mm: |
+ * platform/graphics/mac/WebGLLayer.mm: |
+ * platform/graphics/opengl/Extensions3DOpenGL.cpp: |
+ * platform/graphics/opengl/GraphicsContext3DOpenGL.cpp: |
+ * platform/graphics/qt/Extensions3DQt.cpp: |
+ * platform/graphics/qt/GraphicsContext3DQt.cpp: |
+ * platform/graphics/qt/GraphicsLayerQt.cpp: |
+ (WebCore::GraphicsLayerQtImpl::GraphicsLayerQtImpl): |
+ (WebCore::GraphicsLayerQtImpl::paint): |
+ (WebCore::GraphicsLayerQtImpl::flushChanges): |
+ * platform/graphics/qt/GraphicsLayerQt.h: |
+ * platform/graphics/skia/GraphicsContext3DSkia.cpp: |
+ * platform/graphics/texmap/GraphicsLayerTextureMapper.h: |
+ * rendering/RenderLayerBacking.cpp: |
+ (WebCore::isAcceleratedCanvas): |
+ (WebCore::RenderLayerBacking::updateGraphicsLayerConfiguration): |
+ (WebCore::RenderLayerBacking::containsPaintedContent): |
+ (WebCore::RenderLayerBacking::contentChanged): |
+ * webaudio/AudioBuffer.cpp: |
+ * webaudio/AudioBuffer.idl: |
+ * webaudio/RealtimeAnalyser.cpp: |
+ * webaudio/RealtimeAnalyser.h: |
+ * webaudio/RealtimeAnalyserNode.h: |
+ * webaudio/RealtimeAnalyserNode.idl: |
+ * xml/XMLHttpRequest.cpp: |
+ (WebCore::XMLHttpRequest::setResponseType): |
+ (WebCore::XMLHttpRequest::clearResponse): |
+ (WebCore::XMLHttpRequest::didReceiveData): |
+ * xml/XMLHttpRequest.h: |
+ |
+2011-01-25 Mikhail Naganov <mnaganov@chromium.org> |
+ |
+ Reviewed by Pavel Feldman. |
+ |
+ Web Inspector: [Chromium] Prepare for landing of detailed heap snapshots. |
+ |
+ - Introduce Preferences.detailedHeapProfiles flag for controlling |
+ querying of detailed heap snapshots. |
+ - Add boilerplate code for the new view. |
+ - Factor out common code. |
+ |
+ https://bugs.webkit.org/show_bug.cgi?id=52624 |
+ |
+ * WebCore.gypi: |
+ * WebCore.vcproj/WebCore.vcproj: |
+ * bindings/js/ScriptProfiler.h: |
+ (WebCore::ScriptProfiler::HeapSnapshotControl::~HeapSnapshotControl): |
+ (WebCore::ScriptProfiler::takeHeapSnapshot): |
+ * bindings/v8/ScriptProfiler.cpp: |
+ (WebCore::ScriptProfiler::takeHeapSnapshot): |
+ * bindings/v8/ScriptProfiler.h: |
+ (WebCore::ScriptProfiler::HeapSnapshotControl::~HeapSnapshotControl): |
+ * inspector/Inspector.idl: |
+ * inspector/InspectorProfilerAgent.cpp: |
+ (WebCore::InspectorProfilerAgent::takeHeapSnapshot): |
+ * inspector/InspectorProfilerAgent.h: |
+ * inspector/front-end/DetailedHeapshotView.js: Added. |
+ (WebInspector.DetailedHeapshotView): |
+ (WebInspector.DetailedHeapshotView.prototype.get statusBarItems): |
+ (WebInspector.DetailedHeapshotView.prototype.get profile): |
+ (WebInspector.DetailedHeapshotView.prototype.set profile): |
+ (WebInspector.DetailedHeapshotView.prototype.show): |
+ (WebInspector.DetailedHeapshotView.prototype.hide): |
+ (WebInspector.DetailedHeapshotProfileType): |
+ (WebInspector.DetailedHeapshotProfileType.prototype.get buttonTooltip): |
+ (WebInspector.DetailedHeapshotProfileType.prototype.get buttonStyle): |
+ (WebInspector.DetailedHeapshotProfileType.prototype.buttonClicked): |
+ (WebInspector.DetailedHeapshotProfileType.prototype.get welcomeMessage): |
+ (WebInspector.DetailedHeapshotProfileType.prototype.createSidebarTreeElementForProfile): |
+ (WebInspector.DetailedHeapshotProfileType.prototype.createView): |
+ * inspector/front-end/HeapSnapshot.js: Added. |
+ (WebInspector.HeapSnapshotEdgesIterator): |
+ (WebInspector.HeapSnapshotEdgesIterator.prototype.get done): |
+ (WebInspector.HeapSnapshotEdgesIterator.prototype.get isElement): |
+ (WebInspector.HeapSnapshotEdgesIterator.prototype.get isHidden): |
+ (WebInspector.HeapSnapshotEdgesIterator.prototype.get name): |
+ (WebInspector.HeapSnapshotEdgesIterator.prototype.next): |
+ (WebInspector.HeapSnapshotEdgesIterator.prototype.get node): |
+ (WebInspector.HeapSnapshotEdgesIterator.prototype.get nodeIndex): |
+ (WebInspector.HeapSnapshotEdgesIterator.prototype._getNameOrIndex): |
+ (WebInspector.HeapSnapshotEdgesIterator.prototype._getType): |
+ (WebInspector.HeapSnapshotNodeWrapper): |
+ (WebInspector.HeapSnapshotNodeWrapper.prototype.get edges): |
+ (WebInspector.HeapSnapshotNodeWrapper.prototype.get edgesCount): |
+ (WebInspector.HeapSnapshotNodeWrapper.prototype.get instancesCount): |
+ (WebInspector.HeapSnapshotNodeWrapper.prototype.get isHidden): |
+ (WebInspector.HeapSnapshotNodeWrapper.prototype.get name): |
+ (WebInspector.HeapSnapshotNodeWrapper.prototype.get selfSize): |
+ (WebInspector.HeapSnapshotNodeWrapper.prototype._getName): |
+ (WebInspector.HeapSnapshotNodeWrapper.prototype._getEdges): |
+ (WebInspector.HeapSnapshotNodeWrapper.prototype._getType): |
+ (WebInspector.HeapSnapshot): |
+ (WebInspector.HeapSnapshot.prototype._init): |
+ (WebInspector.HeapSnapshot.prototype.get rootEdges): |
+ * inspector/front-end/HeapSnapshotView.js: |
+ (WebInspector.HeapSnapshotProfileType.prototype.buttonClicked): |
+ * inspector/front-end/ProfilesPanel.js: |
+ (WebInspector.ProfilesPanel.prototype._setRecordingProfile): |
+ (WebInspector.ProfilesPanel.prototype._reportHeapSnapshotProgress): |
+ (WebInspector.ProfilerDispatcher.prototype.setRecordingProfile): |
+ (WebInspector.ProfilerDispatcher.prototype.reportHeapSnapshotProgress): |
+ * inspector/front-end/Settings.js: |
+ * inspector/front-end/WebKit.qrc: |
+ * inspector/front-end/inspector.html: |
+ * inspector/front-end/inspector.js: |
+ (WebInspector._createPanels): |
+ |
+2011-01-25 Yury Semikhatsky <yurys@chromium.org> |
+ |
+ Reviewed by Pavel Feldman. |
+ |
+ Web Inspector: remove "attached" state related methods from InspectorAgent |
+ https://bugs.webkit.org/show_bug.cgi?id=53086 |
+ |
+ * WebCore.exp.in: |
+ * inspector/InspectorAgent.cpp: |
+ * inspector/InspectorAgent.h: |
+ * inspector/InspectorFrontendClientLocal.cpp: |
+ (WebCore::InspectorFrontendClientLocal::changeAttachedWindowHeight): |
+ (WebCore::InspectorFrontendClientLocal::restoreAttachedWindowHeight): |
+ * inspector/InspectorSettings.cpp: |
+ (WebCore::InspectorSettings::InspectorSettings): |
+ * inspector/InspectorSettings.h: |
+ |
+2011-01-25 Qi Zhang <qi.2.zhang@nokia.com> |
+ |
+ Reviewed by Laszlo Gombos. |
+ |
+ [Symbian] RVCT fails to compile [U|I]nt[8|16|32]Array.h |
+ https://bugs.webkit.org/show_bug.cgi?id=51873 |
+ |
+ Replace the using declaration, because it doesn't compile on RVCT, and WINSCW has the same issue. |
+ No new functionality so no new tests. |
+ |
+ * html/canvas/Int32Array.h: |
+ (WebCore::Int32Array::set): |
+ * html/canvas/Int8Array.h: |
+ (WebCore::Int8Array::set): |
+ * html/canvas/Uint16Array.h: |
+ (WebCore::Uint16Array::set): |
+ * html/canvas/Uint32Array.h: |
+ (WebCore::Uint32Array::set): |
+ * html/canvas/Uint8Array.h: |
+ (WebCore::Uint8Array::set): |
+ |
+2011-01-25 Zoltan Herczeg <zherczeg@webkit.org> |
+ |
+ Reviewed by Dirk Schulze. |
+ |
+ Repaint SVG elements with filter instead of relayout where possible |
+ https://bugs.webkit.org/show_bug.cgi?id=52200 |
+ |
+ This patch allows repainting of filters, when their |
+ attribute changes does not require relayout. |
+ |
+ Existing dynamic-update tests cover this feature. |
+ |
+ * platform/graphics/filters/FilterEffect.cpp: |
+ (WebCore::FilterEffect::clearResult): Clearing the currently stored image |
+ before repainting. |
+ * platform/graphics/filters/FilterEffect.h: |
+ * rendering/RenderObject.h: |
+ (WebCore::RenderObject::isSVGResourceFilter): Not only the filter primitives, |
+ but filters should also be detected to allow safe testing and casting. |
+ * rendering/svg/RenderSVGResourceFilter.cpp: |
+ (WebCore::RenderSVGResourceFilter::buildPrimitives): Passing the renderer. |
+ (WebCore::RenderSVGResourceFilter::applyResource): |
+ determineFilterPrimitiveSubregion does not require the filter anymore. |
+ (WebCore::RenderSVGResourceFilter::postApplyResource): Repaint |
+ if lastEffect->hasResult() is false. |
+ (WebCore::RenderSVGResourceFilter::primitiveAttributeChanged): |
+ Searching for all FilterEffects, whose created by the current FilterElement, |
+ and clearing all resulting images depending on those FilterEffects. |
+ * rendering/svg/RenderSVGResourceFilter.h: |
+ (WebCore::RenderSVGResourceFilter::isSVGResourceFilter): |
+ * rendering/svg/RenderSVGResourceFilterPrimitive.cpp: |
+ (WebCore::RenderSVGResourceFilterPrimitive::determineFilterPrimitiveSubregion): |
+ The filter argument is unnecessary anymore. But is is still kept as a static |
+ member, since the primitive renderer still does not know about the |
+ FilterEffect objects. |
+ * rendering/svg/RenderSVGResourceFilterPrimitive.h: |
+ (WebCore::RenderSVGResourceFilterPrimitive::RenderSVGResourceFilterPrimitive): |
+ (WebCore::RenderSVGResourceFilterPrimitive::primitiveAttributeChanged): |
+ Calls RenderSVGResourceFilter::primitiveAttributeChanged. |
+ * svg/SVGFEDiffuseLightingElement.cpp: |
+ (WebCore::SVGFEDiffuseLightingElement::setFilterEffectAttribute): |
+ Setting the new attribute value for each FilterEffect. |
+ (WebCore::SVGFEDiffuseLightingElement::svgAttributeChanged): |
+ * svg/SVGFEDiffuseLightingElement.h: |
+ Calling primitiveAttributeChanged. |
+ * svg/SVGFilterPrimitiveStandardAttributes.cpp: |
+ (WebCore::SVGFilterPrimitiveStandardAttributes::setFilterEffectAttribute): |
+ This function will be removed when all FilterElement implements |
+ their setFilterEffectAttribute |
+ * svg/SVGFilterPrimitiveStandardAttributes.h: |
+ (WebCore::SVGFilterPrimitiveStandardAttributes::primitiveAttributeChanged): |
+ * svg/graphics/filters/SVGFilterBuilder.cpp: |
+ (WebCore::SVGFilterBuilder::appendEffectToEffectReferences): The |
+ renderers are assigned to the filter effects. |
+ (WebCore::SVGFilterBuilder::clearEffects): |
+ (WebCore::SVGFilterBuilder::clearResultsRecursive): Recursively |
+ clearing the result images for those filters, whose depend on |
+ the starting filter. |
+ * svg/graphics/filters/SVGFilterBuilder.h: |
+ (WebCore::SVGFilterBuilder::effectReferences): 'get' is unnecessary |
+ (WebCore::SVGFilterBuilder::effectByRenderer): returns the |
+ FilterEffect belongs to this RenderObject. |
+ |
+2011-01-25 Dirk Schulze <krit@webkit.org> |
+ |
+ Reviewed by Nikolas Zimmermann. |
+ |
+ SVG is missing to-animation support for Path |
+ https://bugs.webkit.org/show_bug.cgi?id=52982 |
+ |
+ SVG was missing 'to' animation support for SVG paths. Even the fallback to discrete |
+ animation did not work and an assert was thrown, because of the missing m_fromPath. |
+ This also influences a test of the W3C test suite. Subtest 2 of animate-elem-83-t.svg passes now. |
+ |
+ Test: svg/animations/animate-path-to-animation.html |
+ |
+ * svg/SVGAnimateElement.cpp: |
+ (WebCore::SVGAnimateElement::calculateAnimatedValue): Take the value of the last SVGAnimateElement for |
+ m_fromPath, since 'to' animations are accumulative. |
+ (WebCore::SVGAnimateElement::calculateFromAndToValues): Added support for 'to' animations. |
+ (WebCore::SVGAnimateElement::resetToBaseValue): Set m_animatedPath on the first animation element to baseVal. |
+ * svg/SVGPathByteStream.h: |
+ (WebCore::SVGPathByteStream::copySVGPathByteStream): Return copy of current byte stream. |
+ |
+2011-01-25 Pavel Feldman <pfeldman@chromium.org> |
+ |
+ Reviewed by Yury Semikhatsky. |
+ |
+ Web Inspector: bind resources to URLs upon adding them into the tree. |
+ https://bugs.webkit.org/show_bug.cgi?id=53013 |
+ |
+ * inspector/front-end/AuditRules.js: |
+ (WebInspector.AuditRules.UnusedCssRule.prototype.doRun.evalCallback.selectorsCallback): |
+ * inspector/front-end/CSSStyleModel.js: |
+ (WebInspector.CSSStyleModel.prototype._styleSheetChanged.callback): |
+ (WebInspector.CSSStyleModel.prototype._styleSheetChanged): |
+ * inspector/front-end/DebuggerModel.js: |
+ (WebInspector.DebuggerModel.prototype._parsedScriptSource): |
+ * inspector/front-end/NetworkManager.js: |
+ (WebInspector.NetworkManager.prototype._processCachedResources): |
+ (WebInspector.NetworkManager.prototype.inflightResourceForURL): |
+ (WebInspector.NetworkDispatcher): |
+ (WebInspector.NetworkDispatcher.prototype.willSendRequest): |
+ (WebInspector.NetworkDispatcher.prototype.markResourceAsCached): |
+ (WebInspector.NetworkDispatcher.prototype.didReceiveResponse): |
+ (WebInspector.NetworkDispatcher.prototype.didReceiveContentLength): |
+ (WebInspector.NetworkDispatcher.prototype.didFinishLoading): |
+ (WebInspector.NetworkDispatcher.prototype.didFailLoading): |
+ (WebInspector.NetworkDispatcher.prototype.willSendWebSocketHandshakeRequest): |
+ (WebInspector.NetworkDispatcher.prototype.didReceiveWebSocketHandshakeResponse): |
+ (WebInspector.NetworkDispatcher.prototype.didCloseWebSocket): |
+ (WebInspector.NetworkDispatcher.prototype._appendRedirect): |
+ (WebInspector.NetworkDispatcher.prototype._startResource): |
+ (WebInspector.NetworkDispatcher.prototype._finishResource): |
+ (WebInspector.NetworkDispatcher.prototype._createResource): |
+ * inspector/front-end/ResourceTreeModel.js: |
+ (WebInspector.ResourceTreeModel.prototype.addResourceToFrame): |
+ (WebInspector.ResourceTreeModel.prototype._bindResourceURL): |
+ (WebInspector.ResourceTreeModel.prototype._clearResources): |
+ (WebInspector.ResourceTreeModel.prototype._unbindResourceURL): |
+ * inspector/front-end/ResourcesPanel.js: |
+ (WebInspector.ResourcesPanel.prototype.canShowSourceLine): |
+ (WebInspector.ResourcesPanel.prototype.showSourceLine): |
+ * inspector/front-end/ScriptsPanel.js: |
+ (WebInspector.ScriptsPanel.prototype._parsedScriptSource): |
+ (WebInspector.ScriptsPanel.prototype._addScript): |
+ |
+2011-01-25 Nikolas Zimmermann <nzimmermann@rim.com> |
+ |
+ Reviewed by Dirk Schulze. |
+ |
+ Introduce FontMetrics abstraction |
+ https://bugs.webkit.org/show_bug.cgi?id=51456 |
+ |
+ Use accurate floating-point metrics for SVG Font calculations instead of casting float -> integer. |
+ This hopefully fixes differences between 32/64 bit bots. |
+ |
+ Needs new layout test results for several platforms, which need to be taken from the bots landing afterwards. |
+ |
+ * platform/graphics/SimpleFontData.cpp: |
+ (WebCore::SimpleFontData::SimpleFontData): |
+ (WebCore::SimpleFontData::initCharWidths): |
+ |
+2011-01-24 Pavel Podivilov <podivilov@chromium.org> |
+ |
+ Reviewed by Pavel Feldman. |
+ |
+ Web Inspector: debugger and browser debugger agents should manage sticky breakpoints independently. |
+ https://bugs.webkit.org/show_bug.cgi?id=52999 |
+ |
+ * inspector/Inspector.idl: |
+ * inspector/InspectorAgent.cpp: |
+ (WebCore::InspectorAgent::didCommitLoad): |
+ (WebCore::InspectorAgent::enableDebugger): |
+ (WebCore::InspectorAgent::inspectedURLWithoutFragment): |
+ * inspector/InspectorAgent.h: |
+ * inspector/InspectorBrowserDebuggerAgent.cpp: |
+ (WebCore::InspectorBrowserDebuggerAgent::setAllBrowserBreakpoints): |
+ (WebCore::InspectorBrowserDebuggerAgent::inspectedURLChanged): |
+ (WebCore::InspectorBrowserDebuggerAgent::restoreStickyBreakpoint): |
+ * inspector/InspectorBrowserDebuggerAgent.h: |
+ * inspector/InspectorDebuggerAgent.cpp: |
+ (WebCore::InspectorDebuggerAgent::InspectorDebuggerAgent): |
+ (WebCore::InspectorDebuggerAgent::setAllJavaScriptBreakpoints): |
+ (WebCore::InspectorDebuggerAgent::inspectedURLChanged): |
+ (WebCore::InspectorDebuggerAgent::restoreBreakpoints): |
+ * inspector/InspectorDebuggerAgent.h: |
+ * inspector/InspectorState.cpp: |
+ (WebCore::InspectorState::InspectorState): |
+ * inspector/InspectorState.h: |
+ * inspector/front-end/BreakpointManager.js: |
+ (WebInspector.BreakpointManager): |
+ (WebInspector.BreakpointManager.prototype._projectChanged): |
+ (WebInspector.BreakpointManager.prototype._saveBreakpoints): |
+ (WebInspector.BreakpointManager.prototype._pushBreakpointsToBackend): |
+ |
+2011-01-25 Carlos Garcia Campos <cgarcia@igalia.com> |
+ |
+ Reviewed by Martin Robinson. |
+ |
+ [GTK] Crash in some pages containing flash |
+ https://bugs.webkit.org/show_bug.cgi?id=53016 |
+ |
+ Flash plugin can produce X errors that are handled by the GDK X |
+ error handler, which exits the process. Since we don't want to |
+ crash due to flash bugs, we install a custom error handler to show |
+ a warning when a X error happens without aborting. |
+ |
+ * plugins/gtk/PluginPackageGtk.cpp: |
+ (WebCore::webkitgtkXError): |
+ (WebCore::PluginPackage::load): |
+ |
+2011-01-25 Carlos Garcia Campos <cgarcia@igalia.com> |
+ |
+ Reviewed by Martin Robinson. |
+ |
+ [GTK] Implement spin buttons in RenderThemeGtk |
+ https://bugs.webkit.org/show_bug.cgi?id=51454 |
+ |
+ Paint inner up/down buttons for spin button elements when building |
+ with GTK+ 3.x. |
+ |
+ Test results will land with the GTK+ 2.x version of this patch. |
+ |
+ * platform/gtk/RenderThemeGtk.h: |
+ * platform/gtk/RenderThemeGtk2.cpp: |
+ (WebCore::RenderThemeGtk::adjustInnerSpinButtonStyle): |
+ (WebCore::RenderThemeGtk::paintInnerSpinButton): |
+ * platform/gtk/RenderThemeGtk3.cpp: |
+ (WebCore::spinButtonArrowSize): |
+ (WebCore::RenderThemeGtk::adjustInnerSpinButtonStyle): |
+ (WebCore::paintSpinArrowButton): |
+ (WebCore::RenderThemeGtk::paintInnerSpinButton): |
+ |
+2011-01-24 Mihai Parparita <mihaip@chromium.org> |
+ |
+ Unreviewed. Missed move of one m_deleted use to be outside NDEBUG block |
+ in r76575. |
+ |
+ * loader/cache/CachedResource.cpp: |
+ (WebCore::CachedResource::~CachedResource): |
+ |
+2011-01-24 Mihai Parparita <mihaip@chromium.org> |
+ |
+ Reviewed by Tony Chang. |
+ |
+ Add runtime checks for invariants in memory cache |
+ https://bugs.webkit.org/show_bug.cgi?id=53059 |
+ |
+ To help track down bug 53045, add some CRASH calls in addition to |
+ ASSERTs, so that we can track down failures in reliability bots. |
+ |
+ Just some checks, no new tests necessary. |
+ |
+ * css/CSSImageValue.cpp: |
+ (WebCore::CSSImageValue::cachedImage): |
+ * loader/cache/CachedResource.h: |
+ * loader/cache/CachedResource.cpp: |
+ (WebCore::CachedResource::~CachedResource): |
+ |
+2011-01-24 Simon Fraser <simon.fraser@apple.com> |
+ |
+ Reviewed by Dan Bernstein. |
+ |
+ Leaking CSSRuleDataList objects |
+ https://bugs.webkit.org/show_bug.cgi?id=53062 |
+ |
+ Fix leaked CSSRuleDataLists added to the m_pseudoRules hash. |
+ |
+ * css/CSSStyleSelector.cpp: |
+ (WebCore::CSSRuleSet::~CSSRuleSet): |
+ |
+2011-01-24 Simon Fraser <simon.fraser@apple.com> |
+ |
+ Reviewed by Eric Seidel. |
+ |
+ Refcount Images used in rendering code |
+ https://bugs.webkit.org/show_bug.cgi?id=52701 |
+ |
+ Change StyleImage::image(RenderObject*, const IntSize&) and |
+ CSSImageGeneratorValue::image(RenderObject*, const IntSize&) to |
+ return PassRefPtr<Image>, and adjust other code accordingly. |
+ |
+ This allows us to return one-time images, for example for CSS gradients |
+ whose appearance may change depending on factors other than the renderer |
+ and the destination size. |
+ |
+ * css/CSSCanvasValue.cpp: |
+ (WebCore::CSSCanvasValue::image): |
+ * css/CSSCanvasValue.h: |
+ * css/CSSGradientValue.cpp: |
+ (WebCore::CSSGradientValue::image): |
+ * css/CSSGradientValue.h: |
+ * css/CSSImageGeneratorValue.h: |
+ * rendering/RenderBoxModelObject.cpp: |
+ (WebCore::RenderBoxModelObject::paintFillLayerExtended): |
+ (WebCore::RenderBoxModelObject::paintNinePieceImage): |
+ * rendering/RenderImage.cpp: |
+ (WebCore::RenderImage::paintReplaced): |
+ (WebCore::RenderImage::paintIntoRect): |
+ * rendering/RenderImageResource.h: |
+ (WebCore::RenderImageResource::image): |
+ * rendering/RenderImageResourceStyleImage.h: |
+ (WebCore::RenderImageResourceStyleImage::image): |
+ * rendering/RenderListMarker.cpp: |
+ (WebCore::RenderListMarker::paint): |
+ * rendering/style/StyleCachedImage.cpp: |
+ (WebCore::StyleCachedImage::image): |
+ * rendering/style/StyleCachedImage.h: |
+ * rendering/style/StyleGeneratedImage.cpp: |
+ (WebCore::StyleGeneratedImage::image): |
+ * rendering/style/StyleGeneratedImage.h: |
+ * rendering/style/StyleImage.h: |
+ * rendering/style/StylePendingImage.h: |
+ (WebCore::StylePendingImage::image): |
+ * rendering/svg/RenderSVGImage.cpp: |
+ (WebCore::RenderSVGImage::paint): |
+ |
+2011-01-24 Tony Chang <tony@chromium.org> |
+ |
+ Unreviewed, round ascent and descent to match old code. |
+ |
+ * platform/graphics/chromium/SimpleFontDataLinux.cpp: |
+ (WebCore::SimpleFontData::platformInit): |
+ |
+2011-01-24 Simon Fraser <simon.fraser@apple.com> |
+ |
+ Reviewed by Chris Marrin. |
+ |
+ perspective() transform function should take lengths |
+ https://bugs.webkit.org/show_bug.cgi?id=52683 |
+ |
+ The argument to the perspective() transform function should |
+ be a Length, rather than a bare number. Bare numbers are still |
+ accepted (and treated as px), but this behavior is deprecated. |
+ |
+ Test: animations/3d/transform-perspective.html |
+ transforms/3d/general/3dtransform-values.html |
+ |
+ * css/CSSParser.cpp: |
+ (WebCore::CSSParser::parseTransform): Check the units for the perspective() |
+ function. Allow bare numbers for backwards compatibility. |
+ * css/CSSStyleSelector.cpp: |
+ (WebCore::CSSStyleSelector::createTransformOperations): Convert |
+ value to Length. |
+ * platform/graphics/transforms/PerspectiveTransformOperation.cpp: |
+ (WebCore::clampToPostiveInteger): Helper. |
+ (WebCore::PerspectiveTransformOperation::blend): Blend via Lengths. |
+ * platform/graphics/transforms/PerspectiveTransformOperation.h: |
+ (WebCore::PerspectiveTransformOperation::create): double -> Length. |
+ (WebCore::PerspectiveTransformOperation::perspective): Ditto. |
+ (WebCore::PerspectiveTransformOperation::isIdentity): Ditto. |
+ (WebCore::PerspectiveTransformOperation::apply): Ditto. |
+ (WebCore::PerspectiveTransformOperation::PerspectiveTransformOperation): Assert |
+ that the Length is a fixed type. |
+ |
+2011-01-24 Kent Tamura <tkent@chromium.org> |
+ |
+ Reviewed by Dimitri Glazkov. |
+ |
+ [Windows] Textfield <input>s have different widths. |
+ https://bugs.webkit.org/show_bug.cgi?id=48405 |
+ |
+ - Apply the padding override for type=text to other textfield types too. |
+ - Reset margin value of outer spin button, which is not used in Windows. |
+ - Don't add inner spin button width to the preferred text field width. |
+ Text fields should have the identical widths regardless of the |
+ existence of spin buttons. |
+ |
+ Test: fast/forms/input-widths.html |
+ |
+ * css/themeWin.css: |
+ (input[type="week"]): |
+ (input[type="week"]:disabled): |
+ (input[type="search"]::-webkit-search-cancel-button): |
+ (input::-webkit-outer-spin-button): |
+ * rendering/RenderTextControlSingleLine.cpp: |
+ (WebCore::RenderTextControlSingleLine::preferredContentWidth): |
+ Don't add inner spin button width to the preferred width. |
+ |
+2011-01-24 Kent Tamura <tkent@chromium.org> |
+ |
+ Reviewed by Dimitri Glazkov. |
+ |
+ Some bugs of search cancel button and spin button about state change in |
+ an event handler. |
+ https://bugs.webkit.org/show_bug.cgi?id=46950 |
+ |
+ Fix the following problems: |
+ * Type=search field didn't release event capturing |
+ * Assertion failure when an input field with spin buttons was changed |
+ to another type on focus event. |
+ * A input field with spin button didn't release event capturing when it |
+ was changed to another type on focus event. |
+ |
+ Tests: fast/forms/input-number-change-type-on-focus.html |
+ fast/forms/search-hide-cancel-on-cancel.html |
+ |
+ * rendering/TextControlInnerElements.cpp: |
+ (WebCore::SearchFieldCancelButtonElement::defaultEventHandler): |
+ - Make the variable 'input' RefPtr. It makes the code simpler. |
+ - Remove visibility check on mouseup event. We should release capturing |
+ anyway because the cancel button may be invisible if JavaScript code |
+ called by the focus event removes the input value. |
+ (WebCore::SpinButtonElement::detach): |
+ - Release capturing on detach because it is possible that a spin button |
+ node is detached while it is capturing events. |
+ (WebCore::SpinButtonElement::defaultEventHandler): |
+ Take a reference to this and check renderer() after some functions which |
+ may run JavaScript code. |
+ (WebCore::InputFieldSpeechButtonElement::defaultEventHandler): |
+ Make the variable 'input' RefPtr to align other functions in this file. |
+ (WebCore::InputFieldSpeechButtonElement::setRecognitionResult): ditto. |
+ * rendering/TextControlInnerElements.h: Declare SpinButtonElement::detach(). |
+ |
+2011-01-24 Ryosuke Niwa <rniwa@webkit.org> |
+ |
+ Reviewed by Ojan Vafai. |
+ |
+ Inserting multiple whitespace using text composition (IME) should insert interleaved nbsp and whitespace. |
+ https://bugs.webkit.org/show_bug.cgi?id=52781 |
+ |
+ The bug was caused by stringWithRebalancedWhitespace's replacing the space at the beginning of a paragraph |
+ and the end of a paragraph by a non-breaking space after it replaced two consecutive spaces by a space and |
+ non-breaking space pattern, thereby replacing more spaces by non-breaking spaces than needed. |
+ |
+ Rewrote the function using Vector<UChar> to fix the bug. New function no longer calls String::replace |
+ multiple times but instead it traverses through the string and replaces a space that immediately follows |
+ another space or appears at the beginning of a paragraph or at the end of a paragraph by a non-break space. |
+ |
+ * editing/CompositeEditCommand.cpp: |
+ * editing/htmlediting.cpp: |
+ (WebCore::stringWithRebalancedWhitespace): Written. |
+ * editing/htmlediting.h: |
+ (WebCore::isWhitespace): Removed from CompositeEditCommand.cpp |
+ |
+2011-01-24 Kenneth Russell <kbr@google.com> |
+ |
+ Reviewed by James Robinson. |
+ |
+ Web Audio API: port FFTFrame to FFTW |
+ https://bugs.webkit.org/show_bug.cgi?id=52989 |
+ |
+ Ported FFTFrame class to the open-source FFTW library. Tested with |
+ unit tests from Chris Rogers. Made preliminary changes to GYP |
+ files for conditional compilation of these files; will need to be |
+ adjusted once FFTW is integrated as third-party source. |
+ |
+ * WebCore.gyp/WebCore.gyp: |
+ * WebCore.gypi: |
+ * platform/audio/FFTFrame.h: |
+ * platform/audio/fftw: Added. |
+ * platform/audio/fftw/FFTFrameFFTW.cpp: Added. |
+ (WebCore::FFTFrame::FFTFrame): |
+ (WebCore::FFTFrame::~FFTFrame): |
+ (WebCore::FFTFrame::multiply): |
+ (WebCore::FFTFrame::doFFT): |
+ (WebCore::FFTFrame::doInverseFFT): |
+ (WebCore::FFTFrame::cleanup): |
+ (WebCore::FFTFrame::realData): |
+ (WebCore::FFTFrame::imagData): |
+ (WebCore::FFTFrame::fftwPlanForSize): |
+ |
+2011-01-24 Anders Carlsson <andersca@apple.com> |
+ |
+ Reviewed by Dan Bernstein. |
+ |
+ Reset the page scale factor on standard frame loads |
+ https://bugs.webkit.org/show_bug.cgi?id=53058 |
+ <rdar://problem/8908844> |
+ |
+ Add a symbol needed by WebKit2. |
+ |
+ * WebCore.exp.in: |
+ |
+2011-01-24 Ryosuke Niwa <rniwa@webkit.org> |
+ |
+ Reviewed by Eric Seidel. |
+ |
+ Stop instantiating legacy editing positions in InsertTextCommand, MoveSelectionCommand, |
+ ReplaceSelectionCommand, SelectionController, SpellChecker, TypingCommand, and markup.cpp |
+ https://bugs.webkit.org/show_bug.cgi?id=52676 |
+ |
+ Stop instantiating legacy editing positions in the following files. |
+ |
+ * editing/InsertTextCommand.cpp: |
+ (WebCore::InsertTextCommand::prepareForTextInsertion): |
+ (WebCore::InsertTextCommand::performTrivialReplace): |
+ (WebCore::InsertTextCommand::input): |
+ (WebCore::InsertTextCommand::insertTab): |
+ * editing/MoveSelectionCommand.cpp: |
+ (WebCore::MoveSelectionCommand::doApply): |
+ * editing/ReplaceSelectionCommand.cpp: |
+ (WebCore::ReplaceSelectionCommand::removeUnrenderedTextNodesAtEnds): |
+ (WebCore::ReplaceSelectionCommand::mergeEndIfNeeded): |
+ (WebCore::ReplaceSelectionCommand::doApply): |
+ (WebCore::ReplaceSelectionCommand::shouldRemoveEndBR): |
+ (WebCore::ReplaceSelectionCommand::performTrivialReplace): |
+ * editing/SelectionController.cpp: |
+ (WebCore::SelectionController::setSelectionFromNone): |
+ * editing/SpellChecker.cpp: |
+ (WebCore::SpellChecker::didCheck): |
+ * editing/TypingCommand.cpp: |
+ (WebCore::TypingCommand::makeEditableRootEmpty): |
+ (WebCore::TypingCommand::deleteKeyPressed): |
+ (WebCore::TypingCommand::forwardDeleteKeyPressed): |
+ * editing/markup.cpp: |
+ (WebCore::StyledMarkupAccumulator::appendText): |
+ (WebCore::StyledMarkupAccumulator::serializeNodes): |
+ (WebCore::highestAncestorToWrapMarkup): |
+ (WebCore::createMarkup): |
+ |
+2011-01-24 Peter Kasting <pkasting@google.com> |
+ |
+ Reviewed by Darin Adler. |
+ |
+ Roll back r67261 ("Don't fire onclick on middle clicks") due to |
+ regressions. |
+ https://bugs.webkit.org/show_bug.cgi?id=46733 |
+ |
+ * html/HTMLAnchorElement.cpp: |
+ (WebCore::isLinkClick): |
+ * html/HTMLInputElement.cpp: |
+ (WebCore::HTMLInputElement::preDispatchEventHandler): |
+ (WebCore::HTMLInputElement::postDispatchEventHandler): |
+ (WebCore::HTMLInputElement::defaultEventHandler): |
+ * page/EventHandler.cpp: |
+ (WebCore::EventHandler::handleMouseDoubleClickEvent): |
+ (WebCore::EventHandler::handleMouseReleaseEvent): |
+ |
+2011-01-24 Martin Robinson <mrobinson@igalia.com> |
+ |
+ Reviewed by Eric Seidel. |
+ |
+ [GTK] Many DOM XHTML tests time out |
+ https://bugs.webkit.org/show_bug.cgi?id=52553 |
+ |
+ Properly handle the situation where a synchronous load fails before the inner |
+ event loop has started. In this case, we simply do not run the inner event loop, |
+ or else it will block indefinitely (since no GIO or libsoup callbacks will fire). |
+ |
+ * platform/network/soup/ResourceHandleSoup.cpp: |
+ (WebCore::ResourceHandle::loadResourceSynchronously): Bail out of a synchronous |
+ load if it fails up front. |
+ |
+2011-01-24 Zhenyao Mo <zmo@google.com> |
+ |
+ Reviewed by Eric Seidel. |
+ |
+ Cleanup WebGLGetInfo and related get*Parameter helpers in WebGLRenderingContext |
+ https://bugs.webkit.org/show_bug.cgi?id=52338 |
+ |
+ Removed the long/unsigned long types and corresponding get functions |
+ and use int/unsigned int instead. |
+ |
+ * bindings/js/JSWebGLRenderingContextCustom.cpp: |
+ (WebCore::toJS): |
+ * bindings/v8/custom/V8WebGLRenderingContextCustom.cpp: |
+ (WebCore::toV8Object): |
+ * html/canvas/WebGLGetInfo.cpp: |
+ (WebCore::WebGLGetInfo::WebGLGetInfo): |
+ (WebCore::WebGLGetInfo::getInt): |
+ (WebCore::WebGLGetInfo::getUnsignedInt): |
+ * html/canvas/WebGLGetInfo.h: |
+ * html/canvas/WebGLRenderingContext.cpp: |
+ (WebCore::WebGLRenderingContext::getBufferParameter): |
+ (WebCore::WebGLRenderingContext::getFramebufferAttachmentParameter): |
+ (WebCore::WebGLRenderingContext::getParameter): |
+ (WebCore::WebGLRenderingContext::getProgramParameter): |
+ (WebCore::WebGLRenderingContext::getRenderbufferParameter): |
+ (WebCore::WebGLRenderingContext::getShaderParameter): |
+ (WebCore::WebGLRenderingContext::getTexParameter): |
+ (WebCore::WebGLRenderingContext::getUniform): |
+ (WebCore::WebGLRenderingContext::getVertexAttrib): |
+ (WebCore::WebGLRenderingContext::getIntParameter): |
+ (WebCore::WebGLRenderingContext::getUnsignedIntParameter): |
+ * html/canvas/WebGLRenderingContext.h: |
+ |
+2011-01-24 Zhenyao Mo <zmo@google.com> |
+ |
+ Reviewed by Darin Adler. |
+ |
+ Remove sizeInBytes from GraphicsContext3D's various implementations |
+ https://bugs.webkit.org/show_bug.cgi?id=52339 |
+ |
+ * html/canvas/WebGLRenderingContext.cpp: |
+ (WebCore::WebGLRenderingContext::sizeInBytes): |
+ * platform/graphics/GraphicsContext3D.h: |
+ * platform/graphics/opengl/GraphicsContext3DOpenGL.cpp: |
+ * platform/graphics/qt/GraphicsContext3DQt.cpp: |
+ |
+2011-01-24 Adam Roben <aroben@apple.com> |
+ |
+ Windows Production build fix |
+ |
+ * WebCore.vcproj/WebCore.make: Update for move of WebCore into Source. |
+ |
+2011-01-24 Anton Muhin <antonm@chromium.org> |
+ |
+ Reviewed by Nate Chapin. |
+ |
+ [v8] Refactoring: extract IntrusiveDOMWrapperMap into a seprate class and files. |
+ https://bugs.webkit.org/show_bug.cgi?id=52911 |
+ |
+ Plain refactoring, covered by the existing tests. |
+ |
+ * WebCore.gypi: IntrusiveDOMWrapperMap.h added. |
+ * bindings/v8/DOMDataStore.cpp: IntrusiveDOMWrapperMap is moved out of DOMDataStore class. |
+ * bindings/v8/DOMDataStore.h: IntrusiveDOMWrapperMap is moved out of DOMDataStore class. |
+ * bindings/v8/IntrusiveDOMWrapperMap.h: Added. |
+ (WebCore::ChunkedTable::ChunkedTable): |
+ (WebCore::ChunkedTable::add): |
+ (WebCore::ChunkedTable::remove): |
+ (WebCore::ChunkedTable::clear): |
+ (WebCore::ChunkedTable::visit): |
+ (WebCore::ChunkedTable::Chunk::Chunk): |
+ (WebCore::ChunkedTable::clearEntries): |
+ (WebCore::ChunkedTable::visitEntries): |
+ (WebCore::IntrusiveDOMWrapperMap::IntrusiveDOMWrapperMap): |
+ (WebCore::IntrusiveDOMWrapperMap::get): |
+ (WebCore::IntrusiveDOMWrapperMap::set): |
+ (WebCore::IntrusiveDOMWrapperMap::contains): |
+ (WebCore::IntrusiveDOMWrapperMap::visit): |
+ (WebCore::IntrusiveDOMWrapperMap::removeIfPresent): |
+ (WebCore::IntrusiveDOMWrapperMap::clear): |
+ (WebCore::IntrusiveDOMWrapperMap::ChunkedTableTraits::move): |
+ (WebCore::IntrusiveDOMWrapperMap::ChunkedTableTraits::clear): |
+ (WebCore::IntrusiveDOMWrapperMap::ChunkedTableTraits::visit): |
+ * bindings/v8/StaticDOMDataStore.h: include added. |
+ |
+2011-01-24 Shane Stephens <shanestephens@google.com> |
+ |
+ Reviewed by Chris Marrin. |
+ |
+ TransformationMatrix multiply operations apply operands in wrong order. |
+ https://bugs.webkit.org/show_bug.cgi?id=52780 |
+ |
+ Rename TranformationMatrix::multLeft into multiply (the method does a multRight, |
+ not a multLeft). |
+ |
+ Remove TransformationMatrix::multiply, which was actually doing a multLeft. |
+ |
+ Fix TransformationMatrix::operator* and operator*= such that the operand is |
+ applied to the right-hand side of the matrix that the method is called on. |
+ i.e., previously "a * b" used to compute "b * a", and "a *= b" used to store |
+ "b * a" in "a". This has now been fixed so "a * b" computes "a * b" and |
+ "a *= b" stores "a * b" in "a". |
+ |
+ Convert all call sites for these methods to provide operands in the correct order. |
+ |
+ No new tests as patch adds no new functionality. |
+ |
+ * css/WebKitCSSMatrix.cpp: |
+ (WebCore::WebKitCSSMatrix::multiply): |
+ * platform/graphics/transforms/Matrix3DTransformOperation.h: |
+ (WebCore::Matrix3DTransformOperation::apply): |
+ * platform/graphics/transforms/MatrixTransformOperation.h: |
+ (WebCore::MatrixTransformOperation::apply): |
+ * platform/graphics/transforms/TransformationMatrix.cpp: |
+ (WebCore::TransformationMatrix::scaleNonUniform): |
+ (WebCore::TransformationMatrix::scale3d): |
+ (WebCore::TransformationMatrix::rotate3d): |
+ (WebCore::TransformationMatrix::skew): |
+ (WebCore::TransformationMatrix::applyPerspective): |
+ (WebCore::TransformationMatrix::multiply): |
+ (WebCore::TransformationMatrix::recompose): |
+ * platform/graphics/transforms/TransformationMatrix.h: |
+ (WebCore::TransformationMatrix::operator*=): |
+ (WebCore::TransformationMatrix::operator*): |
+ * rendering/RenderLayer.cpp: |
+ (WebCore::transparencyClipBox): |
+ * rendering/RenderObject.cpp: |
+ (WebCore::RenderObject::getTransformFromContainer): |
+ * rendering/TransformState.cpp: |
+ (WebCore::TransformState::applyTransform): |
+ (WebCore::HitTestingTransformState::applyTransform): |
+ |
+2011-01-24 Andrei Popescu <andreip@google.com> |
+ |
+ Reviewed by Nate Chapin. |
+ |
+ IndexedDatabase methods should not take arguments of type OptionsObject |
+ https://bugs.webkit.org/show_bug.cgi?id=53012 |
+ |
+ This patch reverts all IDB methods, except IDBDatabase::createObjectStore and |
+ IDBObjectStore::createIndex, to using a plain list of arguments instead of |
+ grouping the various parameters inside a single OptionsObject argument. |
+ This decision was made on public-webapps@w3.org mailing list. |
+ |
+ We also add support (v8 only for now) for passing DOMStringList objects as arguments to native |
+ methods. The code for obtaining a DOMStringList object from a JS array of strings existed already |
+ in OptionsObject.cpp, I just copied it to V8Bindings.cpp and taught the v8 code generator how to |
+ use it. |
+ |
+ * bindings/scripts/CodeGeneratorV8.pm: |
+ * bindings/v8/V8Binding.cpp: |
+ (WebCore::v8ValueToWebCoreDOMStringList): |
+ * bindings/v8/V8Binding.h: |
+ * storage/IDBDatabase.cpp: |
+ (WebCore::IDBDatabase::transaction): |
+ * storage/IDBDatabase.h: |
+ (WebCore::IDBDatabase::transaction): |
+ * storage/IDBDatabase.idl: |
+ * storage/IDBIndex.cpp: |
+ (WebCore::IDBIndex::openCursor): |
+ (WebCore::IDBIndex::openKeyCursor): |
+ * storage/IDBIndex.h: |
+ (WebCore::IDBIndex::openCursor): |
+ (WebCore::IDBIndex::openKeyCursor): |
+ * storage/IDBIndex.idl: |
+ * storage/IDBObjectStore.cpp: |
+ (WebCore::IDBObjectStore::openCursor): |
+ * storage/IDBObjectStore.h: |
+ (WebCore::IDBObjectStore::openCursor): |
+ * storage/IDBObjectStore.idl: |
+ |
+2011-01-24 Pavel Feldman <pfeldman@chromium.org> |
+ |
+ Reviewed by Yury Semikhatsky. |
+ |
+ Web Inspector: extract dispatch API from network manager. |
+ https://bugs.webkit.org/show_bug.cgi?id=53009 |
+ |
+ * WebCore.xcodeproj/project.pbxproj: |
+ * inspector/front-end/NetworkManager.js: |
+ (WebInspector.NetworkManager): |
+ (WebInspector.NetworkManager.prototype.reset): |
+ (WebInspector.NetworkManager.prototype.requestContent): |
+ (WebInspector.NetworkManager.prototype._processCachedResources): |
+ (WebInspector.NetworkDispatcher): |
+ (WebInspector.NetworkDispatcher.prototype._updateResourceWithRequest): |
+ (WebInspector.NetworkDispatcher.prototype._updateResourceWithResponse): |
+ (WebInspector.NetworkDispatcher.prototype._updateResourceWithCachedResource): |
+ (WebInspector.NetworkDispatcher.prototype.identifierForInitialRequest): |
+ (WebInspector.NetworkDispatcher.prototype.willSendRequest): |
+ (WebInspector.NetworkDispatcher.prototype.didReceiveResponse): |
+ (WebInspector.NetworkDispatcher.prototype.didLoadResourceFromMemoryCache): |
+ (WebInspector.NetworkDispatcher.prototype.didCommitLoadForFrame): |
+ (WebInspector.NetworkDispatcher.prototype.didCreateWebSocket): |
+ (WebInspector.NetworkDispatcher.prototype._appendRedirect): |
+ (WebInspector.NetworkDispatcher.prototype._startResource): |
+ (WebInspector.NetworkDispatcher.prototype._updateResource): |
+ (WebInspector.NetworkDispatcher.prototype._finishResource): |
+ (WebInspector.NetworkDispatcher.prototype._addFramesRecursively): |
+ (WebInspector.NetworkDispatcher.prototype._dispatchEventToListeners): |
+ (WebInspector.NetworkDispatcher.prototype._createResource): |
+ * inspector/front-end/Resource.js: |
+ (WebInspector.Resource.prototype._innerRequestContent): |
+ * inspector/front-end/ResourceTreeModel.js: |
+ (WebInspector.ResourceTreeModel): |
+ (WebInspector.ResourceTreeModel.prototype.reset): |
+ (WebInspector.ResourceTreeModel.prototype.unbindResourceURL): |
+ |
+2011-01-24 Yury Semikhatsky <yurys@chromium.org> |
+ |
+ Reviewed by Pavel Feldman. |
+ |
+ Web Inspector: [JSC] remove ScriptDebugServer::pageCreated |
+ https://bugs.webkit.org/show_bug.cgi?id=53007 |
+ |
+ It's impossible to create "provisional" inspector for pages which are |
+ not yet created so there is no need to listen for page creation in ScriptDebugServer. |
+ |
+ * bindings/js/ScriptDebugServer.cpp: |
+ * bindings/js/ScriptDebugServer.h: |
+ * page/Page.cpp: |
+ (WebCore::Page::Page): |
+ |
+2011-01-24 Pavel Podivilov <podivilov@chromium.org> |
+ |
+ Reviewed by Pavel Feldman. |
+ |
+ Web Inspector: [REGRESSION] AppCache view on resources panel is broken. |
+ https://bugs.webkit.org/show_bug.cgi?id=53002 |
+ |
+ |
+ * inspector/front-end/ApplicationCacheItemsView.js: |
+ (WebInspector.ApplicationCacheItemsView.prototype._update): |
+ * inspector/front-end/DOMAgent.js: |
+ (WebInspector.ApplicationCacheDispatcher.getApplicationCachesAsync): |
+ |
+2011-01-24 Andreas Kling <kling@webkit.org> |
+ |
+ Reviewed by Kenneth Rohde Christiansen. |
+ |
+ [Qt] Fix warnings about unregistering an invalid timer |
+ https://bugs.webkit.org/show_bug.cgi?id=53006 |
+ |
+ The ShadowBuffer's purge timer ID was initialized with 0 which lead |
+ to us calling QObject::killTimer(0), causing some qWarnings. |
+ |
+ * platform/graphics/qt/ContextShadowQt.cpp: |
+ (WebCore::ShadowBuffer::ShadowBuffer): |
+ (WebCore::ShadowBuffer::schedulePurge): |
+ |
+2011-01-21 Mikhail Naganov <mnaganov@chromium.org> |
+ |
+ Reviewed by Pavel Feldman. |
+ |
+ Web Inspector: [Chromium] Fix heap snapshot table sorting. |
+ |
+ Fix table sorting to avoid qsort instability artefacts. |
+ |
+ https://bugs.webkit.org/show_bug.cgi?id=52914 |
+ |
+ * inspector/front-end/HeapSnapshotView.js: |
+ (WebInspector.HeapSnapshotView.prototype._sortData): |
+ (WebInspector.HeapSnapshotDataGridList.propertyComparator.comparator): |
+ (WebInspector.HeapSnapshotDataGridList.propertyComparator): |
+ |
+2011-01-24 Mikhail Naganov <mnaganov@chromium.org> |
+ |
+ Reviewed by Yury Semikhatsky. |
+ |
+ Web Inspector: Find duplicate strings in localizedStrings.js, update it. |
+ |
+ https://bugs.webkit.org/show_bug.cgi?id=53005 |
+ |
+ * English.lproj/localizedStrings.js: |
+ |
+2011-01-24 Pavel Podivilov <podivilov@chromium.org> |
+ |
+ Unreviewed, build fix for r76509. |
+ |
+ * inspector/InspectorDebuggerAgent.cpp: |
+ (WebCore::InspectorDebuggerAgent::setBreakpoint): |
+ |
+2011-01-24 Pavel Podivilov <podivilov@chromium.org> |
+ |
+ Unreviewed, build fix for r76509. |
+ |
+ * bindings/js/ScriptDebugServer.h: |
+ |
+2011-01-24 Pavel Podivilov <podivilov@chromium.org> |
+ |
+ Reviewed by Yury Semikhatsky. |
+ |
+ Web Inspector: set breakpoints by line:column. |
+ https://bugs.webkit.org/show_bug.cgi?id=52615 |
+ |
+ * CMakeLists.txt: |
+ * GNUmakefile.am: |
+ * WebCore.gypi: |
+ * WebCore.pro: |
+ * WebCore.vcproj/WebCore.vcproj: |
+ * WebCore.xcodeproj/project.pbxproj: |
+ * bindings/js/ScriptDebugServer.cpp: |
+ (WebCore::ScriptDebugServer::setBreakpoint): |
+ (WebCore::ScriptDebugServer::removeBreakpoint): |
+ (WebCore::ScriptDebugServer::hasBreakpoint): |
+ (WebCore::ScriptDebugServer::clearBreakpoints): |
+ * bindings/js/ScriptDebugServer.h: |
+ * bindings/v8/DebuggerScript.js: |
+ (): |
+ * bindings/v8/ScriptDebugServer.cpp: |
+ (WebCore::ScriptDebugServer::setBreakpoint): |
+ * bindings/v8/ScriptDebugServer.h: |
+ * inspector/Inspector.idl: |
+ * inspector/InspectorBrowserDebuggerAgent.cpp: |
+ (WebCore::InspectorBrowserDebuggerAgent::restoreStickyBreakpoint): |
+ * inspector/InspectorDebuggerAgent.cpp: |
+ (WebCore::InspectorDebuggerAgent::setStickyBreakpoint): |
+ (WebCore::InspectorDebuggerAgent::setBreakpoint): |
+ (WebCore::InspectorDebuggerAgent::restoreBreakpoint): |
+ (WebCore::InspectorDebuggerAgent::didParseSource): |
+ * inspector/InspectorDebuggerAgent.h: |
+ * inspector/ScriptBreakpoint.cpp: Removed. |
+ * inspector/ScriptBreakpoint.h: |
+ (WebCore::ScriptBreakpoint::ScriptBreakpoint): |
+ * inspector/front-end/Breakpoint.js: |
+ (WebInspector.Breakpoint): |
+ (WebInspector.Breakpoint.prototype.get url): |
+ * inspector/front-end/BreakpointManager.js: |
+ (WebInspector.JavaScriptBreakpoint.prototype._serializeToJSON): |
+ * inspector/front-end/DebuggerModel.js: |
+ (WebInspector.DebuggerModel.prototype.continueToLine): |
+ (WebInspector.DebuggerModel.prototype.setBreakpoint): |
+ (WebInspector.DebuggerModel.prototype._breakpointSetOnBackend): |
+ (WebInspector.DebuggerDispatcher.prototype.breakpointResolved): |
+ |
+2011-01-24 Peter Beverloo <peter@chromium.org> |
+ |
+ Reviewed by Pavel Feldman. |
+ |
+ Web Inspector: recognize application/x-font-woff as a valid mime-type. |
+ https://bugs.webkit.org/show_bug.cgi?id=52977 |
+ |
+ No new tests. |
+ |
+ * inspector/front-end/inspector.js: |
+ |
+2011-01-24 Pavel Podivilov <podivilov@chromium.org> |
+ |
+ Unreviewed, fix for r76497. |
+ |
+ * inspector/front-end/ResourcesPanel.js: |
+ (WebInspector.ResourcesPanel.prototype.show): |
+ |
+2011-01-24 Andreas Kling <kling@webkit.org> |
+ |
+ Reviewed by Tor Arne Vestbø. |
+ |
+ [Qt] Use rad2deg() from WTF instead of rolling our own. |
+ https://bugs.webkit.org/show_bug.cgi?id=52993 |
+ |
+ * platform/graphics/qt/GraphicsContextQt.cpp: |
+ (WebCore::GraphicsContext::rotate): |
+ * platform/graphics/qt/PathQt.cpp: |
+ (WebCore::Path::addArc): |
+ |
+2011-01-24 Sergio Villar Senin <svillar@igalia.com> |
+ |
+ Reviewed by Martin Robinson. |
+ |
+ [Gtk] ResourceHandleSoup: do not wait for streams to close to issue didFinishLoading |
+ https://bugs.webkit.org/show_bug.cgi?id=52885 |
+ |
+ No new tests as it does not change functionality. We will not wait for |
+ the the input stream to close to issue didFinishLoading to |
+ WebCore. We expect a subtle performance improvement with this |
+ patch when loading complex web pages, as we wouldn't have to wait |
+ for the stream to close (which involves creating a thread among |
+ other things). |
+ |
+ * platform/network/soup/ResourceHandleSoup.cpp: |
+ (WebCore::closeCallback): |
+ (WebCore::readCallback): |
+ |
+2011-01-21 Pavel Podivilov <podivilov@chromium.org> |
+ |
+ Reviewed by Pavel Feldman. |
+ |
+ Web Inspector: eliminate SourceView and ScriptView classes. |
+ https://bugs.webkit.org/show_bug.cgi?id=52896 |
+ |
+ SourceView and ScriptView delegate everything to SourceFrame and should be eliminated. |
+ |
+ * WebCore.gypi: |
+ * WebCore.vcproj/WebCore.vcproj: |
+ * inspector/front-end/GoToLineDialog.js: |
+ (WebInspector.GoToLineDialog.prototype._highlightSelectedLine): |
+ * inspector/front-end/ResourceView.js: |
+ (WebInspector.ResourceView.createResourceView): |
+ (WebInspector.ResourceView.resourceViewTypeMatchesResource): |
+ (WebInspector.SourceFrameContentProviderForResource): Moved from SourceView. |
+ * inspector/front-end/ResourcesPanel.js: |
+ (WebInspector.ResourcesPanel.prototype.show): |
+ (WebInspector.ResourcesPanel.prototype._applyDiffMarkup): |
+ (WebInspector.FrameResourceTreeElement.prototype._errorsWarningsUpdated): |
+ * inspector/front-end/ScriptView.js: Removed. |
+ * inspector/front-end/ScriptsPanel.js: |
+ (WebInspector.ScriptsPanel.prototype._scriptSourceChanged): |
+ (WebInspector.ScriptsPanel.prototype._sourceFrameForScriptOrResource): |
+ (WebInspector.ScriptsPanel.prototype._showScriptOrResource): |
+ (WebInspector.ScriptsPanel.prototype._setPauseOnExceptions): |
+ (WebInspector.ScriptsPanel.prototype._formatScript): |
+ (WebInspector.SourceFrameContentProviderForScript): Moved from ScriptView. |
+ * inspector/front-end/SourceFrame.js: |
+ (WebInspector.SourceFrame): |
+ (WebInspector.SourceFrame.prototype.show): |
+ (WebInspector.SourceFrame.prototype.hide): |
+ (WebInspector.SourceFrame.prototype.hasContent): |
+ (WebInspector.SourceFrame.prototype._createTextViewer): |
+ (WebInspector.SourceFrame.prototype.performSearch.doFindSearchMatches): |
+ (WebInspector.SourceFrame.prototype.performSearch): |
+ * inspector/front-end/SourceView.js: Removed. |
+ * inspector/front-end/WebKit.qrc: |
+ * inspector/front-end/inspector.html: |
+ |
+2011-01-24 Andras Becsi <abecsi@webkit.org> |
+ |
+ Reviewed by Csaba Osztrogonác. |
+ |
+ [Qt] Move project files into Source |
+ https://bugs.webkit.org/show_bug.cgi?id=52891 |
+ |
+ No new tests needed. |
+ |
+ * WebCore.pri: |
+ * WebCore.pro: |
+ |
+2011-01-24 Kent Tamura <tkent@chromium.org> |
+ |
+ Unreviewed, trivial fix. |
+ |
+ Fix a Chromium-only assertion failure by r76491. |
+ https://bugs.webkit.org/show_bug.cgi?id=38982 |
+ |
+ Separate icon loading from the FileChooser constructor in order to avoid |
+ ref() before adoptRef(). |
+ |
+ * platform/FileChooser.cpp: |
+ (WebCore::FileChooser::FileChooser): |
+ (WebCore::FileChooser::initialize): |
+ (WebCore::FileChooser::create): |
+ * platform/FileChooser.h: |
+ |
+2011-01-24 MORITA Hajime <morrita@google.com> |
+ |
+ Reviewed by Kent Tamura. |
+ |
+ TextControlInnerElement::m_shadowParent should be removed. |
+ https://bugs.webkit.org/show_bug.cgi?id=52998 |
+ |
+ No new tests. No behavioral change. |
+ |
+ * rendering/TextControlInnerElements.cpp: |
+ (WebCore::TextControlInnerElement::TextControlInnerElement): |
+ * rendering/TextControlInnerElements.h: |
+ |
+2011-01-24 Yury Semikhatsky <yurys@chromium.org> |
+ |
+ Reviewed by Pavel Feldman. |
+ |
+ REGRESSION (r72895): console.trace crashes |
+ https://bugs.webkit.org/show_bug.cgi?id=52981 |
+ |
+ - Use local RefPtr variable for accessing PassRefPtr arguments to make sure the PassRefPtr has |
+ non-null value. |
+ - Use PassRefPtr for objects whos ownership is passed to the inspector. |
+ |
+ * GNUmakefile.am: |
+ * WebCore.gypi: |
+ * WebCore.pro: |
+ * WebCore.vcproj/WebCore.vcproj: |
+ * WebCore.xcodeproj/project.pbxproj: |
+ * inspector/InspectorConsoleInstrumentation.h: Added. Extract inline methods called from the console into their |
+ own header to reduce compilation time of other classes that depend on the inspector instrumentation. |
+ (WebCore::InspectorInstrumentation::addMessageToConsole): |
+ (WebCore::InspectorInstrumentation::consoleCount): |
+ (WebCore::InspectorInstrumentation::startConsoleTiming): |
+ (WebCore::InspectorInstrumentation::stopConsoleTiming): |
+ (WebCore::InspectorInstrumentation::consoleMarkTimeline): |
+ (WebCore::InspectorInstrumentation::addStartProfilingMessageToConsole): |
+ (WebCore::InspectorInstrumentation::addProfile): |
+ (WebCore::InspectorInstrumentation::profilerEnabled): |
+ (WebCore::InspectorInstrumentation::getCurrentUserInitiatedProfileName): |
+ * inspector/InspectorDatabaseInstrumentation.h: Added. Extract inline methods called from the datanase code into their |
+ own header to reduce compilation time of other classes that depend on the inspector instrumentation. |
+ (WebCore::InspectorInstrumentation::didOpenDatabase): |
+ * inspector/InspectorInstrumentation.cpp: |
+ (WebCore::InspectorInstrumentation::addMessageToConsoleImpl): |
+ (WebCore::InspectorInstrumentation::consoleCountImpl): |
+ (WebCore::InspectorInstrumentation::stopConsoleTimingImpl): |
+ (WebCore::InspectorInstrumentation::consoleMarkTimelineImpl): |
+ (WebCore::InspectorInstrumentation::addProfileImpl): |
+ (WebCore::InspectorInstrumentation::didOpenDatabaseImpl): |
+ * inspector/InspectorInstrumentation.h: |
+ * page/Console.cpp: use RefPtr local variable instead of accessing PassRefPtr argument directly. |
+ (WebCore::Console::addMessage): |
+ (WebCore::Console::trace): |
+ (WebCore::Console::count): |
+ (WebCore::Console::markTimeline): |
+ (WebCore::Console::profileEnd): |
+ (WebCore::Console::timeEnd): |
+ (WebCore::Console::group): |
+ (WebCore::Console::groupCollapsed): |
+ * storage/Database.cpp: |
+ (WebCore::Database::openDatabase): |
+ |
+2010-01-24 Kent Tamura <tkent@chromium.org> |
+ |
+ Reviewed by Darin Fisher. |
+ |
+ [Chromium] Support icon loading for <input type=file> |
+ https://bugs.webkit.org/show_bug.cgi?id=38982 |
+ |
+ An icon data in Chromium port is represented as WebCore::Image. We |
+ don't need OS-specific code anymore. |
+ |
+ * WebCore.gypi: |
+ * platform/graphics/Icon.h: |
+ (WebCore::Icon::create): Add Chromium-specific factory. |
+ * platform/graphics/chromium/IconChromium.cpp: Added. |
+ * platform/graphics/chromium/IconChromiumLinux.cpp: Removed. |
+ * platform/graphics/chromium/IconChromiumMac.cpp: Removed. |
+ * platform/graphics/chromium/IconChromiumWin.cpp: Removed. |
+ * platform/graphics/chromium/PlatformIcon.h: |
+ |
+2011-01-21 Pavel Podivilov <podivilov@chromium.org> |
+ |
+ Reviewed by Pavel Feldman. |
+ |
+ Web Inspector: move search functions from SourceView to SourceFrame. |
+ https://bugs.webkit.org/show_bug.cgi?id=52895 |
+ |
+ This is the last step before eliminating SourceView and ScriptView since |
+ this classes just delegate everything to SourceFrame. |
+ |
+ * inspector/front-end/SourceFrame.js: |
+ (WebInspector.SourceFrame.prototype.set visible): |
+ (WebInspector.SourceFrame.prototype._clearLineHighlight): |
+ (WebInspector.SourceFrame.prototype._createTextViewer): |
+ (WebInspector.SourceFrame.prototype.performSearch.doFindSearchMatches): |
+ (WebInspector.SourceFrame.prototype.performSearch): |
+ (WebInspector.SourceFrame.prototype.searchCanceled): |
+ (WebInspector.SourceFrame.prototype.jumpToFirstSearchResult): |
+ (WebInspector.SourceFrame.prototype.jumpToLastSearchResult): |
+ (WebInspector.SourceFrame.prototype.jumpToNextSearchResult): |
+ (WebInspector.SourceFrame.prototype.jumpToPreviousSearchResult): |
+ (WebInspector.SourceFrame.prototype.showingFirstSearchResult): |
+ (WebInspector.SourceFrame.prototype.showingLastSearchResult): |
+ (WebInspector.SourceFrame.prototype._jumpToSearchResult): |
+ * inspector/front-end/SourceView.js: |
+ (WebInspector.SourceView.prototype.hide): |
+ (WebInspector.SourceView.prototype.searchCanceled): |
+ (WebInspector.SourceView.prototype.performSearch): |
+ (WebInspector.SourceView.prototype.jumpToFirstSearchResult): |
+ (WebInspector.SourceView.prototype.jumpToLastSearchResult): |
+ (WebInspector.SourceView.prototype.jumpToNextSearchResult): |
+ (WebInspector.SourceView.prototype.jumpToPreviousSearchResult): |
+ (WebInspector.SourceView.prototype.showingFirstSearchResult): |
+ (WebInspector.SourceView.prototype.showingLastSearchResult): |
+ (WebInspector.SourceView.prototype.clearMessages): |
+ |
+2011-01-24 Sheriff Bot <webkit.review.bot@gmail.com> |
+ |
+ Unreviewed, rolling out r76463. |
+ http://trac.webkit.org/changeset/76463 |
+ https://bugs.webkit.org/show_bug.cgi?id=52993 |
+ |
+ It broke canvas/philip/tests/2d.path.stroke.skew.html |
+ (Requested by Ossy on #webkit). |
+ |
+ * platform/graphics/qt/GraphicsContextQt.cpp: |
+ (WebCore::GraphicsContext::rotate): |
+ * platform/graphics/qt/PathQt.cpp: |
+ (WebCore::Path::addArc): |
+ |
+2011-01-23 Andrey Kosyakov <caseq@chromium.org> |
+ |
+ Reviewed by Pavel Feldman. |
+ |
+ Web Inspector: summary bar is not resized properly with the rest of network panel |
+ https://bugs.webkit.org/show_bug.cgi?id=52881 |
+ |
+ * inspector/front-end/NetworkPanel.js: |
+ (WebInspector.NetworkPanel.prototype.updateSidebarWidth): |
+ (WebInspector.NetworkPanel.prototype._positionSummaryBar): |
+ (WebInspector.NetworkPanel.prototype._toggleGridMode): |
+ |
+2011-01-19 MORITA Hajime <morrita@google.com> |
+ |
+ Reviewed by Ryosuke Niwa. |
+ |
+ Space and tab characters "sent" by an input method give totally different results than typing them directly |
+ https://bugs.webkit.org/show_bug.cgi?id=5241 |
+ |
+ * Introduced TextEvent::InputTypeComposition and TypingCommand::TextCompositionType to |
+ distinguish text input which is originated by composition. |
+ * Generalized rebalanceWhitespaceAt() to rebalanceWhitespaceOnTextSubstring() to rebalancing |
+ range of string on text node, instead of surrounding part of that. |
+ |
+ Test: editing/inserting/insert-composition-whitespace.html |
+ |
+ * dom/TextEvent.h: |
+ (WebCore::TextEvent::isComposition): |
+ * dom/TextEventInputType.h: Added TextEventInputComposition as a member of TextEvent::InputType |
+ * editing/CompositeEditCommand.cpp: |
+ (WebCore::containsOnlyWhitespace): |
+ (WebCore::CompositeEditCommand::shouldRebalanceLeadingWhitespaceFor): |
+ (WebCore::CompositeEditCommand::canRebalance): |
+ (WebCore::CompositeEditCommand::rebalanceWhitespaceAt): |
+ (WebCore::CompositeEditCommand::rebalanceWhitespaceOnTextSubstring): Added: A generalized version of rebalanceWhitespaceAt(), which takes a range inside Text string. |
+ * editing/CompositeEditCommand.h: |
+ * editing/Editor.cpp: |
+ (WebCore::Editor::insertTextForConfirmedComposition): Added. |
+ (WebCore::Editor::insertTextWithoutSendingTextEvent): |
+ (WebCore::Editor::confirmComposition): Now uses insertTextForConfirmedComposition(). |
+ (WebCore::Editor::setComposition): |
+ * editing/Editor.h: |
+ * editing/InsertTextCommand.cpp: |
+ (WebCore::InsertTextCommand::input): |
+ * editing/InsertTextCommand.h: |
+ * editing/TypingCommand.cpp: |
+ (WebCore::TypingCommand::TypingCommand): |
+ (WebCore::TypingCommand::insertText): |
+ (WebCore::TypingCommand::insertTextRunWithoutNewlines): |
+ * editing/TypingCommand.h: Added TypingCommand::m_compositionType and TypingCommand::TextCompositionType |
+ (WebCore::TypingCommand::setCompositionType): Added. |
+ (WebCore::TypingCommand::create): |
+ |
+2011-01-23 Mark Rowe <mrowe@apple.com> |
+ |
+ Follow-up to r76477. |
+ |
+ Fix the scripts that detect problematic code such as static initializers |
+ and destructors, weak vtables, inappropriate files in the framework wrappers, |
+ and public headers including private headers. These had all been broken |
+ since the projects were moved in to the Source directory as the paths to the |
+ scripts were not updated at that time. |
+ |
+ Stop copying an IDL file in to the framework wrapper. Doing that doesn't even make sense. |
+ |
+ * WebCore.xcodeproj/project.pbxproj: |
+ |
+2011-01-23 Mark Rowe <mrowe@apple.com> |
+ |
+ Build fix after r76459. |
+ |
+ Static member variables or globals of types that have constructors or destructors are bad as |
+ they generate static initializers and destructors. This is code that is run either at link time |
+ when the library is loaded in to memory or at application termination time. Both of these are |
+ terrible for performance and are thus outlawed in WebKit code. |
+ |
+ The typical solution is to replace the static member or global with a function that allocates |
+ the necessary variable on the heap. The variable is leaked to prevent it from being destroyed |
+ at application termination time. The DEFINE_STATIC_LOCAL macro wraps this in to a concise little |
+ package, but sadly fails to work in this case due to the type containing multiple template |
+ parameters. |
+ |
+ * inspector/InspectorInstrumentation.cpp: |
+ (WebCore::InspectorInstrumentation::inspectorAgents): |
+ * inspector/InspectorInstrumentation.h: |
+ (WebCore::InspectorInstrumentation::bindInspectorAgent): |
+ (WebCore::InspectorInstrumentation::unbindInspectorAgent): |
+ (WebCore::InspectorInstrumentation::inspectorAgentForPage): |
+ |
+2011-01-21 Vangelis Kokkevis <vangelis@chromium.org> |
+ |
+ Reviewed by Kenneth Russell. |
+ |
+ [chromium] Add support for -webkit-mask properties to the |
+ accelerated compositing path. |
+ https://bugs.webkit.org/show_bug.cgi?id=49780 |
+ |
+ Tests: Existing tests in LayoutTests/compositing/masks |
+ |
+ * platform/graphics/chromium/ContentLayerChromium.cpp: |
+ (WebCore::ContentLayerChromium::draw): |
+ (WebCore::ContentLayerChromium::unreserveContentsTexture): |
+ (WebCore::ContentLayerChromium::bindContentsTexture): |
+ * platform/graphics/chromium/ContentLayerChromium.h: |
+ * platform/graphics/chromium/GraphicsLayerChromium.cpp: |
+ (WebCore::GraphicsLayerChromium::setMaskLayer): |
+ * platform/graphics/chromium/GraphicsLayerChromium.h: |
+ * platform/graphics/chromium/LayerChromium.cpp: |
+ (WebCore::LayerChromium::LayerChromium): |
+ * platform/graphics/chromium/LayerChromium.h: |
+ (WebCore::LayerChromium::setMaskLayer): |
+ (WebCore::LayerChromium::maskLayer): |
+ (WebCore::LayerChromium::unreserveContentsTexture): |
+ (WebCore::LayerChromium::bindContentsTexture): |
+ * platform/graphics/chromium/LayerRendererChromium.cpp: |
+ (WebCore::LayerRendererChromium::updateLayersRecursive): |
+ * platform/graphics/chromium/LayerTexture.cpp: |
+ (WebCore::LayerTexture::bindTexture): |
+ * platform/graphics/chromium/RenderSurfaceChromium.cpp: |
+ (WebCore::RenderSurfaceChromium::SharedValues::SharedValues): |
+ (WebCore::RenderSurfaceChromium::SharedValues::~SharedValues): |
+ (WebCore::RenderSurfaceChromium::RenderSurfaceChromium): |
+ (WebCore::RenderSurfaceChromium::draw): |
+ * platform/graphics/chromium/RenderSurfaceChromium.h: |
+ (WebCore::RenderSurfaceChromium::SharedValues::maskShaderProgram): |
+ (WebCore::RenderSurfaceChromium::SharedValues::maskShaderSamplerLocation): |
+ (WebCore::RenderSurfaceChromium::SharedValues::maskShaderMaskSamplerLocation): |
+ (WebCore::RenderSurfaceChromium::SharedValues::maskShaderMatrixLocation): |
+ (WebCore::RenderSurfaceChromium::SharedValues::maskShaderAlphaLocation): |
+ |
+2011-01-23 Patrick Gansterer <paroga@webkit.org> |
+ |
+ Reviewed by Darin Adler. |
+ |
+ Use WTF::StringHasher in WebCore |
+ https://bugs.webkit.org/show_bug.cgi?id=52934 |
+ |
+ * loader/appcache/ApplicationCacheStorage.cpp: |
+ (WebCore::urlHostHash): |
+ * platform/LinkHash.cpp: |
+ (WebCore::visitedLinkHashInline): |
+ * platform/cf/BinaryPropertyList.cpp: |
+ (WebCore::IntegerArrayHash::hash): |
+ * platform/graphics/wx/FontPlatformDataWx.cpp: |
+ (WebCore::FontPlatformData::computeHash): |
+ * platform/network/ProtectionSpaceHash.h: |
+ (WebCore::ProtectionSpaceHash::hash): |
+ |
+2011-01-23 Patrick Gansterer <paroga@webkit.org> |
+ |
+ Reviewed by David Kilzer. |
+ |
+ Add an overload to base64Encode with String output |
+ https://bugs.webkit.org/show_bug.cgi?id=50122 |
+ |
+ This change removes duplicated code. |
+ |
+ * inspector/InspectorResourceAgent.cpp: |
+ (WebCore::InspectorResourceAgent::resourceContentBase64): |
+ * page/DOMWindow.cpp: |
+ (WebCore::DOMWindow::btoa): |
+ * platform/graphics/skia/FontCustomPlatformData.cpp: |
+ (WebCore::createUniqueFontName): |
+ * platform/graphics/win/FontCustomPlatformData.cpp: |
+ (WebCore::createUniqueFontName): |
+ * platform/graphics/wince/FontCustomPlatformData.cpp: |
+ (WebCore::createUniqueFontName): |
+ * platform/network/cf/ResourceHandleCFNet.cpp: |
+ (WebCore::encodeBasicAuthorization): |
+ * platform/network/mac/ResourceHandleMac.mm: |
+ (WebCore::encodeBasicAuthorization): |
+ * platform/text/Base64.cpp: |
+ (WebCore::base64Encode): |
+ * platform/text/Base64.h: |
+ (WebCore::base64Encode): |
+ * platform/wince/KeygenWinCE.cpp: |
+ (WebCore::WebCore::signedPublicKeyAndChallengeString): |
+ |
+2011-01-23 Patrick Gansterer <paroga@webkit.org> |
+ |
+ Reviewed by David Kilzer. |
+ |
+ Add String::containsOnlyLatin1() |
+ https://bugs.webkit.org/show_bug.cgi?id=52979 |
+ |
+ Use String::containsOnlyLatin1() instead of isSafeToConvertCharList(). |
+ |
+ * page/DOMWindow.cpp: |
+ (WebCore::DOMWindow::btoa): |
+ (WebCore::DOMWindow::atob): |
+ |
+2011-01-23 Patrick Gansterer <paroga@webkit.org> |
+ |
+ Reviewed by Andreas Kling. |
+ |
+ [CMake] Add missing CodeGenerator dependencies |
+ https://bugs.webkit.org/show_bug.cgi?id=52976 |
+ |
+ * CMakeLists.txt: |
+ |
+2011-01-23 Yael Aharon <yael.aharon@nokia.com> |
+ |
+ Reviewed by Kenneth Rohde Christiansen. |
+ |
+ [Qt][Symbian] Fix --minimal build |
+ https://bugs.webkit.org/show_bug.cgi?id=52839 |
+ |
+ Some of the plugins' functionality is reimplemented in |
+ Symbian specific files. |
+ Some of this functionality should be compiled in even if |
+ the guard NETSCAPE_PLUGIN_API is turned off. |
+ This patch moves that functionality out of the |
+ NETSCAPE_PLUGIN_API guard. |
+ |
+ Build fix so no new tests. |
+ |
+ * WebCore.pro: |
+ * plugins/symbian/PluginPackageSymbian.cpp: |
+ |
+2011-01-23 Andreas Kling <kling@webkit.org> |
+ |
+ Reviewed by Tor Arne Vestbø. |
+ |
+ [Qt] Use rad2deg() from WTF instead of rolling our own. |
+ |
+ * platform/graphics/qt/GraphicsContextQt.cpp: |
+ (WebCore::GraphicsContext::rotate): |
+ * platform/graphics/qt/PathQt.cpp: |
+ (WebCore::Path::addArc): |
+ |
+2011-01-23 Kenneth Rohde Christiansen <kenneth@webkit.org> |
+ |
+ Reviewed by Andreas Kling. |
+ |
+ Viewport meta: Always adjust device dimensions to the device pixel ratio. |
+ |
+ * dom/ViewportArguments.cpp: |
+ (WebCore::computeViewportAttributes): |
+ |
+2011-01-23 Andreas Kling <kling@webkit.org> |
+ |
+ Reviewed by Kenneth Rohde Christiansen. |
+ |
+ Don't scale absolute geometry specified by viewport meta tag. |
+ |
+ * dom/ViewportArguments.cpp: |
+ (WebCore::computeViewportAttributes): |
+ |
+2011-01-23 Ilya Tikhonovsky <loislo@chromium.org> |
+ |
+ Unreviewed build fix. |
+ |
+ Web Inspector: looks like efl doesn't regenerate InspectorBackendDispatcher. |
+ |
+ Touch Inspector.idl. |
+ |
+ * inspector/Inspector.idl: |
+ |
+2011-01-22 Ilya Tikhonovsky <loislo@chromium.org> |
+ |
+ Reviewed by Pavel Feldman. |
+ |
+ Web Inspector: next step in splitting InspectorController. |
+ |
+ Splitting InspectorController is a bit tricky process. |
+ |
+ As a first step I'll do the next things: |
+ 1) rename existing InspectorController to InspectorAgent; |
+ 2) s/nspectorController/nspectorAgent/g everywhere in WebCore/inspector but InspectorInstrumentation; |
+ 3) create a fake InspectorController derived from the InspectorAgent for the rest of WebCore and WebKit; |
+ |
+ The second step is a migration of a small set of functions described in bug 52510 from InspectorAgent to InspectorController. |
+ |
+ As far as the second step will have significant changes in the functions' implementation I'll do it as a separate patch. |
+ |
+ https://bugs.webkit.org/show_bug.cgi?id=52955 |
+ |
+ * CMakeLists.txt: |
+ * GNUmakefile.am: |
+ * WebCore.exp.in: |
+ * WebCore.gypi: |
+ * WebCore.pro: |
+ * WebCore.vcproj/WebCore.vcproj: |
+ * WebCore.xcodeproj/project.pbxproj: |
+ * bindings/js/JSInjectedScriptHostCustom.cpp: |
+ (WebCore::JSInjectedScriptHost::nodeForId): |
+ (WebCore::JSInjectedScriptHost::selectDOMStorage): |
+ * bindings/v8/custom/V8InjectedScriptHostCustom.cpp: |
+ (WebCore::V8InjectedScriptHost::nodeForIdCallback): |
+ * inspector/CodeGeneratorInspector.pm: |
+ * inspector/InjectedScriptHost.cpp: |
+ (WebCore::InjectedScriptHost::InjectedScriptHost): |
+ (WebCore::InjectedScriptHost::clearConsoleMessages): |
+ (WebCore::InjectedScriptHost::databaseForId): |
+ (WebCore::InjectedScriptHost::selectDatabase): |
+ (WebCore::InjectedScriptHost::selectDOMStorage): |
+ (WebCore::InjectedScriptHost::injectedScriptForMainFrame): |
+ (WebCore::InjectedScriptHost::inspectorDOMAgent): |
+ (WebCore::InjectedScriptHost::frontend): |
+ (WebCore::InjectedScriptHost::didCreateWorker): |
+ (WebCore::InjectedScriptHost::didDestroyWorker): |
+ * inspector/InjectedScriptHost.h: |
+ (WebCore::InjectedScriptHost::create): |
+ (WebCore::InjectedScriptHost::inspectorAgent): |
+ (WebCore::InjectedScriptHost::disconnectController): |
+ * inspector/InspectorAgent.cpp: Copied from Source/WebCore/inspector/InspectorController.cpp. |
+ (WebCore::InspectorAgent::InspectorAgent): |
+ (WebCore::InspectorAgent::~InspectorAgent): |
+ (WebCore::InspectorAgent::inspectedPageDestroyed): |
+ (WebCore::InspectorAgent::enabled): |
+ (WebCore::InspectorAgent::inspectorStartsAttached): |
+ (WebCore::InspectorAgent::setInspectorStartsAttached): |
+ (WebCore::InspectorAgent::setInspectorAttachedHeight): |
+ (WebCore::InspectorAgent::inspectorAttachedHeight): |
+ (WebCore::InspectorAgent::searchingForNodeInPage): |
+ (WebCore::InspectorAgent::restoreInspectorStateFromCookie): |
+ (WebCore::InspectorAgent::inspect): |
+ (WebCore::InspectorAgent::focusNode): |
+ (WebCore::InspectorAgent::highlight): |
+ (WebCore::InspectorAgent::highlightDOMNode): |
+ (WebCore::InspectorAgent::highlightFrame): |
+ (WebCore::InspectorAgent::hideHighlight): |
+ (WebCore::InspectorAgent::mouseDidMoveOverElement): |
+ (WebCore::InspectorAgent::handleMousePress): |
+ (WebCore::InspectorAgent::setInspectorFrontendClient): |
+ (WebCore::InspectorAgent::didClearWindowObjectInWorld): |
+ (WebCore::InspectorAgent::setSearchingForNode): |
+ (WebCore::InspectorAgent::connectFrontend): |
+ (WebCore::InspectorAgent::show): |
+ (WebCore::InspectorAgent::showPanel): |
+ (WebCore::InspectorAgent::close): |
+ (WebCore::InspectorAgent::disconnectFrontend): |
+ (WebCore::InspectorAgent::resourceAgent): |
+ (WebCore::InspectorAgent::releaseFrontendLifetimeAgents): |
+ (WebCore::InspectorAgent::populateScriptObjects): |
+ (WebCore::InspectorAgent::pushDataCollectedOffline): |
+ (WebCore::InspectorAgent::restoreDebugger): |
+ (WebCore::InspectorAgent::restoreProfiler): |
+ (WebCore::InspectorAgent::unbindAllResources): |
+ (WebCore::InspectorAgent::didCommitLoad): |
+ (WebCore::InspectorAgent::mainResourceFiredDOMContentEvent): |
+ (WebCore::InspectorAgent::mainResourceFiredLoadEvent): |
+ (WebCore::InspectorAgent::isMainResourceLoader): |
+ (WebCore::InspectorAgent::willSendRequest): |
+ (WebCore::InspectorAgent::ensureSettingsLoaded): |
+ (WebCore::InspectorAgent::startTimelineProfiler): |
+ (WebCore::InspectorAgent::stopTimelineProfiler): |
+ (WebCore::PostWorkerNotificationToFrontendTask::create): |
+ (WebCore::PostWorkerNotificationToFrontendTask::PostWorkerNotificationToFrontendTask): |
+ (WebCore::PostWorkerNotificationToFrontendTask::performTask): |
+ (WebCore::InspectorAgent::postWorkerNotificationToFrontend): |
+ (WebCore::InspectorAgent::didCreateWorker): |
+ (WebCore::InspectorAgent::didDestroyWorker): |
+ (WebCore::InspectorAgent::didOpenDatabase): |
+ (WebCore::InspectorAgent::getCookies): |
+ (WebCore::InspectorAgent::buildArrayForCookies): |
+ (WebCore::InspectorAgent::buildObjectForCookie): |
+ (WebCore::InspectorAgent::deleteCookie): |
+ (WebCore::InspectorAgent::didUseDOMStorage): |
+ (WebCore::InspectorAgent::didCreateWebSocket): |
+ (WebCore::InspectorAgent::willSendWebSocketHandshakeRequest): |
+ (WebCore::InspectorAgent::didReceiveWebSocketHandshakeResponse): |
+ (WebCore::InspectorAgent::didCloseWebSocket): |
+ (WebCore::InspectorAgent::isRecordingUserInitiatedProfile): |
+ (WebCore::InspectorAgent::startUserInitiatedProfiling): |
+ (WebCore::InspectorAgent::stopUserInitiatedProfiling): |
+ (WebCore::InspectorAgent::profilerEnabled): |
+ (WebCore::InspectorAgent::enableProfiler): |
+ (WebCore::InspectorAgent::disableProfiler): |
+ (WebCore::InspectorAgent::showAndEnableDebugger): |
+ (WebCore::InspectorAgent::enableDebugger): |
+ (WebCore::InspectorAgent::disableDebugger): |
+ (WebCore::InspectorAgent::resume): |
+ (WebCore::InspectorAgent::setAllBrowserBreakpoints): |
+ (WebCore::InspectorAgent::evaluateForTestInFrontend): |
+ (WebCore::InspectorAgent::didEvaluateForTestInFrontend): |
+ (WebCore::quadToPath): |
+ (WebCore::drawOutlinedQuad): |
+ (WebCore::drawOutlinedQuadWithClip): |
+ (WebCore::drawHighlightForBox): |
+ (WebCore::drawHighlightForLineBoxesOrSVGRenderer): |
+ (WebCore::convertFromFrameToMainFrame): |
+ (WebCore::frameToMainFrameOffset): |
+ (WebCore::InspectorAgent::drawNodeHighlight): |
+ (WebCore::InspectorAgent::drawElementTitle): |
+ (WebCore::InspectorAgent::openInInspectedWindow): |
+ (WebCore::InspectorAgent::addScriptToEvaluateOnLoad): |
+ (WebCore::InspectorAgent::removeAllScriptsToEvaluateOnLoad): |
+ (WebCore::InspectorAgent::setInspectorExtensionAPI): |
+ (WebCore::InspectorAgent::inspectedURL): |
+ (WebCore::InspectorAgent::reloadPage): |
+ (WebCore::InspectorAgent::setExtraHeaders): |
+ * inspector/InspectorAgent.h: Copied from Source/WebCore/inspector/InspectorController.h. |
+ (WebCore::InspectorAgent::inspectorBackendDispatcher): |
+ (WebCore::InspectorAgent::inspectorClient): |
+ (WebCore::InspectorAgent::injectedScriptHost): |
+ (WebCore::InspectorAgent::inspectedPage): |
+ (WebCore::InspectorAgent::hideDOMNodeHighlight): |
+ (WebCore::InspectorAgent::hideFrameHighlight): |
+ (WebCore::InspectorAgent::frontend): |
+ (WebCore::InspectorAgent::inspectorController): |
+ (WebCore::InspectorAgent::inspectorAgent): |
+ (WebCore::InspectorAgent::consoleAgent): |
+ (WebCore::InspectorAgent::cssAgent): |
+ (WebCore::InspectorAgent::domAgent): |
+ (WebCore::InspectorAgent::injectedScriptAgent): |
+ (WebCore::InspectorAgent::runtimeAgent): |
+ (WebCore::InspectorAgent::timelineAgent): |
+ (WebCore::InspectorAgent::databaseAgent): |
+ (WebCore::InspectorAgent::domStorageAgent): |
+ (WebCore::InspectorAgent::fileSystemAgent): |
+ (WebCore::InspectorAgent::browserDebuggerAgent): |
+ (WebCore::InspectorAgent::debuggerAgent): |
+ (WebCore::InspectorAgent::profilerAgent): |
+ (WebCore::InspectorAgent::applicationCacheAgent): |
+ (WebCore::InspectorAgent::hasInspectorFrontendClient): |
+ (WebCore::InspectorAgent::hasFrontend): |
+ (WebCore::InspectorAgent::startProfiling): |
+ (WebCore::InspectorAgent::stopProfiling): |
+ (WebCore::InspectorAgent::debuggerEnabled): |
+ (WebCore::InspectorAgent::state): |
+ (WebCore::InspectorAgent::settings): |
+ * inspector/InspectorApplicationCacheAgent.cpp: |
+ (WebCore::InspectorApplicationCacheAgent::InspectorApplicationCacheAgent): |
+ (WebCore::InspectorApplicationCacheAgent::getApplicationCaches): |
+ * inspector/InspectorApplicationCacheAgent.h: |
+ * inspector/InspectorBrowserDebuggerAgent.cpp: |
+ (WebCore::InspectorBrowserDebuggerAgent::InspectorBrowserDebuggerAgent): |
+ (WebCore::InspectorBrowserDebuggerAgent::inspectedURLChanged): |
+ (WebCore::InspectorBrowserDebuggerAgent::restoreStickyBreakpoint): |
+ (WebCore::InspectorBrowserDebuggerAgent::setDOMBreakpoint): |
+ (WebCore::InspectorBrowserDebuggerAgent::removeDOMBreakpoint): |
+ (WebCore::InspectorBrowserDebuggerAgent::willInsertDOMNode): |
+ (WebCore::InspectorBrowserDebuggerAgent::willRemoveDOMNode): |
+ (WebCore::InspectorBrowserDebuggerAgent::willModifyDOMAttr): |
+ (WebCore::InspectorBrowserDebuggerAgent::descriptionForDOMEvent): |
+ (WebCore::InspectorBrowserDebuggerAgent::pauseOnNativeEventIfNeeded): |
+ (WebCore::InspectorBrowserDebuggerAgent::willSendXMLHttpRequest): |
+ * inspector/InspectorBrowserDebuggerAgent.h: |
+ (WebCore::InspectorBrowserDebuggerAgent::create): |
+ * inspector/InspectorConsoleAgent.cpp: |
+ (WebCore::InspectorConsoleAgent::InspectorConsoleAgent): |
+ (WebCore::InspectorConsoleAgent::~InspectorConsoleAgent): |
+ (WebCore::InspectorConsoleAgent::clearConsoleMessages): |
+ (WebCore::InspectorConsoleAgent::addMessageToConsole): |
+ (WebCore::InspectorConsoleAgent::resourceRetrievedByXMLHttpRequest): |
+ (WebCore::InspectorConsoleAgent::didReceiveResponse): |
+ (WebCore::InspectorConsoleAgent::didFailLoading): |
+ (WebCore::InspectorConsoleAgent::setMonitoringXHREnabled): |
+ (WebCore::InspectorConsoleAgent::setConsoleMessagesEnabled): |
+ (WebCore::InspectorConsoleAgent::addConsoleMessage): |
+ * inspector/InspectorConsoleAgent.h: |
+ * inspector/InspectorController.cpp: |
+ (WebCore::InspectorController::InspectorController): |
+ * inspector/InspectorController.h: |
+ * inspector/InspectorDatabaseAgent.cpp: |
+ * inspector/InspectorDebuggerAgent.cpp: |
+ (WebCore::InspectorDebuggerAgent::create): |
+ (WebCore::InspectorDebuggerAgent::InspectorDebuggerAgent): |
+ (WebCore::InspectorDebuggerAgent::~InspectorDebuggerAgent): |
+ (WebCore::InspectorDebuggerAgent::evaluateOnCallFrame): |
+ (WebCore::InspectorDebuggerAgent::getCompletionsOnCallFrame): |
+ (WebCore::InspectorDebuggerAgent::currentCallFrames): |
+ * inspector/InspectorDebuggerAgent.h: |
+ * inspector/InspectorFileSystemAgent.cpp: |
+ (WebCore::InspectorFileSystemAgentCallbacks::didOpenFileSystem): |
+ (WebCore::InspectorFileSystemAgent::InspectorFileSystemAgent): |
+ (WebCore::InspectorFileSystemAgent::stop): |
+ (WebCore::InspectorFileSystemAgent::getFileSystemPathAsync): |
+ (WebCore::InspectorFileSystemAgent::didGetFileSystemPath): |
+ (WebCore::InspectorFileSystemAgent::didGetFileSystemError): |
+ * inspector/InspectorFileSystemAgent.h: |
+ (WebCore::InspectorFileSystemAgent::create): |
+ * inspector/InspectorFrontendHost.cpp: |
+ (WebCore::InspectorFrontendHost::setExtensionAPI): |
+ * inspector/InspectorProfilerAgent.cpp: |
+ (WebCore::InspectorProfilerAgent::create): |
+ (WebCore::InspectorProfilerAgent::InspectorProfilerAgent): |
+ (WebCore::InspectorProfilerAgent::addProfileFinishedMessageToConsole): |
+ (WebCore::InspectorProfilerAgent::addStartProfilingMessageToConsole): |
+ (WebCore::InspectorProfilerAgent::startUserInitiatedProfiling): |
+ (WebCore::InspectorProfilerAgent::stopUserInitiatedProfiling): |
+ * inspector/InspectorProfilerAgent.h: |
+ |
+2011-01-22 Chris Rogers <crogers@google.com> |
+ |
+ Rubber-stamped by Anders Carlsson. |
+ |
+ Fix FFTFrameStub to compile properly |
+ https://bugs.webkit.org/show_bug.cgi?id=52969 |
+ |
+ No new tests since this is a build fix. |
+ |
+ * platform/audio/FFTFrameStub.cpp: |
+ (WebCore::FFTFrame::FFTFrame): |
+ |
+2011-01-22 Andrei Popescu <andreip@google.com> |
+ |
+ Reviewed by Jeremy Orlow. |
+ |
+ IndexedDB corrupts data on disk |
+ https://bugs.webkit.org/show_bug.cgi?id=52890 |
+ |
+ We need to store the SerializedScriptValues on disk in a BLOB column rather than TEXT. |
+ Test: storage/indexeddb/data-corruption.html |
+ |
+ * platform/sql/SQLiteStatement.cpp: |
+ (WebCore::SQLiteStatement::bindBlob): |
+ (WebCore::SQLiteStatement::getColumnBlobAsString): |
+ (WebCore::SQLiteStatement::getColumnBlobAsVector): |
+ * platform/sql/SQLiteStatement.h: |
+ * storage/IDBCursorBackendImpl.cpp: |
+ (WebCore::IDBCursorBackendImpl::loadCurrentRow): |
+ * storage/IDBFactoryBackendImpl.cpp: |
+ (WebCore::runCommands): |
+ (WebCore::createTables): |
+ (WebCore::createMetaDataTable): |
+ (WebCore::migrateDatabase): |
+ * storage/IDBObjectStoreBackendImpl.cpp: |
+ (WebCore::IDBObjectStoreBackendImpl::getInternal): |
+ (WebCore::putObjectStoreData): |
+ |
+2011-01-22 Nikolas Zimmermann <nzimmermann@rim.com> |
+ |
+ Not reviewed. Fix WinCE build. |
+ |
+ * platform/graphics/wince/GraphicsContextWinCE.cpp: |
+ (WebCore::GraphicsContext::drawText): |
+ * rendering/RenderThemeWinCE.cpp: |
+ (WebCore::RenderThemeWinCE::adjustMenuListButtonStyle): |
+ |
+2011-01-22 Nikolas Zimmermann <nzimmermann@rim.com> |
+ |
+ Reviewed by Dirk Schulze. |
+ |
+ REGRESSION: Vertical line metrics incorrect |
+ https://bugs.webkit.org/show_bug.cgi?id=52960 |
+ |
+ SimpleFontDataMac.mm contains a hack to modifiy lineGap/descent for the 'Hiragino' font. |
+ That didn't influence the lineSpacing so far, but does now, causing regressions. |
+ |
+ Restore old line spacing behaviour to fix the regression. |
+ Covered by existing fast/blockflow, fast/repaint and fast/text/international test cases. |
+ |
+ * platform/graphics/FontMetrics.h: |
+ (WebCore::FontMetrics::FontMetrics): |
+ (WebCore::FontMetrics::floatLineSpacing): |
+ (WebCore::FontMetrics::setLineSpacing): |
+ (WebCore::FontMetrics::lineSpacing): |
+ (WebCore::FontMetrics::reset): |
+ * platform/graphics/SimpleFontData.cpp: |
+ (WebCore::SimpleFontData::SimpleFontData): |
+ * platform/graphics/chromium/SimpleFontDataChromiumWin.cpp: |
+ (WebCore::SimpleFontData::platformInit): |
+ * platform/graphics/chromium/SimpleFontDataLinux.cpp: |
+ (WebCore::SimpleFontData::platformInit): |
+ * platform/graphics/freetype/SimpleFontDataFreeType.cpp: |
+ (WebCore::SimpleFontData::platformInit): |
+ * platform/graphics/haiku/SimpleFontDataHaiku.cpp: |
+ (WebCore::SimpleFontData::platformInit): |
+ * platform/graphics/mac/SimpleFontDataMac.mm: |
+ (WebCore::SimpleFontData::platformInit): |
+ * platform/graphics/pango/SimpleFontDataPango.cpp: |
+ (WebCore::SimpleFontData::platformInit): |
+ * platform/graphics/qt/SimpleFontDataQt.cpp: |
+ (WebCore::SimpleFontData::platformInit): |
+ * platform/graphics/win/SimpleFontDataCGWin.cpp: |
+ (WebCore::SimpleFontData::platformInit): |
+ * platform/graphics/win/SimpleFontDataCairoWin.cpp: |
+ (WebCore::SimpleFontData::platformInit): |
+ * platform/graphics/win/SimpleFontDataWin.cpp: |
+ (WebCore::SimpleFontData::initGDIFont): |
+ * platform/graphics/wince/SimpleFontDataWinCE.cpp: |
+ (WebCore::SimpleFontData::platformInit): |
+ * platform/graphics/wx/SimpleFontDataWx.cpp: |
+ (WebCore::SimpleFontData::platformInit): |
+ |
+2011-01-22 Andreas Kling <kling@webkit.org> |
+ |
+ Reviewed by Kenneth Rohde Christiansen. |
+ |
+ [Qt] fast/backgrounds/svg-as-mask.html fails |
+ https://bugs.webkit.org/show_bug.cgi?id=52906 |
+ |
+ Transparency layers should start out with in SourceOver mode with |
+ alpha 1.0 (modeled after CGContextBeginTransparencyLayer.) |
+ |
+ * platform/graphics/qt/TransparencyLayer.h: |
+ (WebCore::TransparencyLayer::TransparencyLayer): |
+ |
+2011-01-22 Nikolas Zimmermann <nzimmermann@rim.com> |
+ |
+ Not reviewed. |
+ |
+ Introduce FontMetrics abstraction |
+ https://bugs.webkit.org/show_bug.cgi?id=51456 |
+ |
+ Fix Chromium/Win build. |
+ |
+ * platform/graphics/chromium/UniscribeHelperTextRun.cpp: s/->/./ |
+ (WebCore::UniscribeHelperTextRun::UniscribeHelperTextRun): |
+ |
+2011-01-21 Nikolas Zimmermann <nzimmermann@rim.com> |
+ |
+ Reviewed by Dirk Schulze. |
+ |
+ Introduce FontMetrics abstraction |
+ https://bugs.webkit.org/show_bug.cgi?id=51456 |
+ |
+ Encapsulate ascent/descent/lineHeight/lineGap methods in a single FontMetrics class, instead of |
+ having to define them in both Font & SimpleFontData. Changed to store floating point values |
+ as default, in order to get accurate information for small sized fonts. All these methods |
+ now have floating-point and integer versions. Whenever an integer variant of these functions |
+ is called, lroundf() is used to round the value. |
+ |
+ This makes it possible to support small font-sizes for SVG in a follow-up patch, as well |
+ as fixing rounding issues when using SVG Fonts. |
+ |
+ Shouldn't affect existing tests. |
+ |
+ * GNUmakefile.am: Add FontMetrics.h to build. |
+ * WebCore.gypi: Ditto. |
+ * WebCore.pro: Ditto. |
+ * WebCore.vcproj/WebCore.vcproj: Ditto. |
+ * WebCore.xcodeproj/project.pbxproj: Ditto. |
+ * accessibility/gtk/AccessibilityObjectWrapperAtk.cpp: Use style->fontMetrics() instead of style->font() to access the metrics. |
+ (baselinePositionForAccessibilityRenderObject): |
+ * css/CSSPrimitiveValue.cpp: |
+ (WebCore::CSSPrimitiveValue::computeLengthDouble): |
+ * html/canvas/CanvasRenderingContext2D.cpp: Ditto. |
+ (WebCore::CanvasRenderingContext2D::drawTextInternal): |
+ * inspector/InspectorController.cpp: Ditto. |
+ (WebCore::InspectorController::drawElementTitle): |
+ * platform/chromium/PopupMenuChromium.cpp: Ditto. |
+ (WebCore::PopupListBox::paintRow): |
+ (WebCore::PopupListBox::getRowHeight): |
+ * platform/graphics/Font.h: Remove ascent/descent/height/lineGap/lineSpacing/xHeight/unitsPerEm accessor... |
+ (WebCore::Font::fontMetrics): ... and only expose a single FontMetrics object here. |
+ * platform/graphics/FontFastPath.cpp: Use fontMetrics() to query metrics information. |
+ (WebCore::Font::emphasisMarkAscent): |
+ (WebCore::Font::emphasisMarkDescent): |
+ (WebCore::Font::emphasisMarkHeight): |
+ (WebCore::Font::floatWidthForSimpleText): |
+ * platform/graphics/FontMetrics.h: Added. |
+ (WebCore::FontMetrics::FontMetrics): Creates a FontMetrics object, stored in SimpleFontData. |
+ (WebCore::FontMetrics::unitsPerEm): Returns an unsigned describing the unitsPerEm. |
+ (WebCore::FontMetrics::setUnitsPerEm): Sets the unitsPerEm value. |
+ (WebCore::FontMetrics::floatAscent): Returns the stored m_ascent float. |
+ (WebCore::FontMetrics::setAscent): Sets the stored m_ascent float. |
+ (WebCore::FontMetrics::floatDescent): Returns the stored m_descent float. |
+ (WebCore::FontMetrics::setDescent): Sets the stored m_descent float. |
+ (WebCore::FontMetrics::floatHeight): Returns floatAscent() + floatDescent(). |
+ (WebCore::FontMetrics::floatLineGap): Returns the stored m_lineGap float. |
+ (WebCore::FontMetrics::setLineGap): Sets the stored m_lineGap float. |
+ (WebCore::FontMetrics::floatLineSpacing): Returns the stored m_lineSpacing float. |
+ (WebCore::FontMetrics::setLineSpacing): Sets the stored m_lineSpacing float. |
+ (WebCore::FontMetrics::xHeight): Returns the stored m_xHeight float (no integer version available, hence no 'float' prefix). |
+ (WebCore::FontMetrics::setXHeight): Sets the stored m_xHeight float. |
+ (WebCore::FontMetrics::ascent): Returns a rounded version of ascent(). |
+ (WebCore::FontMetrics::descent): Ditto (for descent). |
+ (WebCore::FontMetrics::height): Returns ascent() + descent(). |
+ (WebCore::FontMetrics::lineGap): Returns a rounded version of lineGap(). |
+ (WebCore::FontMetrics::lineSpacing): Ditto (for lineSpacing). |
+ (WebCore::FontMetrics::reset): Nulls all members, used only by the platform variants of SimpleFontData. |
+ * platform/graphics/SimpleFontData.cpp: Adapt SVG Fonts code, to initialize the FontMetrics object, as the m_ascent/etc.. members are gone. |
+ (WebCore::SimpleFontData::SimpleFontData): |
+ (WebCore::SimpleFontData::initCharWidths): |
+ * platform/graphics/SimpleFontData.h: Remove ascent/descent/height/lineSpacing/lineGap/xHeight/unitsPerEm accessors, and members, just store a FontMetrics object and expose it. |
+ (WebCore::SimpleFontData::fontMetrics): |
+ (WebCore::SimpleFontData::avgCharWidth): |
+ * platform/graphics/chromium/FontChromiumWin.cpp: Use fontMetrics() to query font metrics. |
+ (WebCore::TransparencyAwareFontPainter::TransparencyAwareGlyphPainter::estimateTextBounds): |
+ (WebCore::TransparencyAwareFontPainter::TransparencyAwareGlyphPainter::drawGlyphs): |
+ (WebCore::TransparencyAwareFontPainter::TransparencyAwareUniscribePainter::estimateTextBounds): |
+ (WebCore::Font::drawComplexText): |
+ * platform/graphics/chromium/SimpleFontDataChromiumWin.cpp: Adapt platform code, to initialize the FontMetrics object. |
+ (WebCore::SimpleFontData::platformInit): |
+ * platform/graphics/chromium/SimpleFontDataLinux.cpp: Ditto. |
+ (WebCore::SimpleFontData::platformInit): |
+ * platform/graphics/chromium/UniscribeHelperTextRun.cpp: Use fontMetrics() to query font metrics. |
+ (WebCore::UniscribeHelperTextRun::UniscribeHelperTextRun): |
+ (WebCore::UniscribeHelperTextRun::nextWinFontData): |
+ * platform/graphics/freetype/SimpleFontDataFreeType.cpp: Adapt platform code, to initialize the FontMetrics object. |
+ (WebCore::SimpleFontData::platformInit): |
+ * platform/graphics/haiku/SimpleFontDataHaiku.cpp: Ditto. |
+ (WebCore::SimpleFontData::platformInit): |
+ * platform/graphics/mac/FontComplexTextMac.cpp: Use fontMetrics() to query font metrics. |
+ (WebCore::Font::floatWidthForComplexText): |
+ * platform/graphics/mac/FontMac.mm: Ditto. |
+ (WebCore::showGlyphsWithAdvances): |
+ * platform/graphics/mac/SimpleFontDataMac.mm: Adapt platform code, to initialize the FontMetrics object. |
+ (WebCore::SimpleFontData::platformInit): |
+ (WebCore::SimpleFontData::platformCharWidthInit): |
+ * platform/graphics/pango/SimpleFontDataPango.cpp: Ditto. |
+ (WebCore::SimpleFontData::platformInit): |
+ * platform/graphics/qt/SimpleFontDataQt.cpp: Ditto. (+ Switch to QFontMetricsF to get floating-point accurancy.) |
+ (WebCore::SimpleFontData::platformInit): |
+ * platform/graphics/win/FontCGWin.cpp: Use fontMetrics() to query font metrics. |
+ (WebCore::drawGDIGlyphs): |
+ * platform/graphics/win/FontWin.cpp: Ditto. |
+ (WebCore::Font::floatWidthForComplexText): |
+ * platform/graphics/win/SimpleFontDataCGWin.cpp: Adapt platform code, to initialize the FontMetrics object. |
+ (WebCore::SimpleFontData::platformInit): |
+ (WebCore::SimpleFontData::platformBoundsForGlyph): |
+ * platform/graphics/win/SimpleFontDataCairoWin.cpp: Ditto. |
+ (WebCore::SimpleFontData::platformInit): |
+ * platform/graphics/win/SimpleFontDataWin.cpp: Ditto. |
+ (WebCore::SimpleFontData::initGDIFont): |
+ * platform/graphics/wince/GraphicsContextWinCE.cpp: Use fontMetrics() to query font metrics. |
+ (WebCore::GraphicsContext::drawText): |
+ * platform/graphics/wince/SimpleFontDataWinCE.cpp: Adapt platform code, to initialize the FontMetrics object. |
+ (WebCore::SimpleFontData::platformInit): |
+ * platform/graphics/wx/SimpleFontDataWx.cpp: Ditto. |
+ (WebCore::SimpleFontData::platformInit): |
+ * platform/win/PopupMenuWin.cpp: Use style->fontMetrics() instead of style->font() to access the metrics. |
+ (WebCore::PopupMenuWin::calculatePositionAndSize): |
+ (WebCore::PopupMenuWin::paint): |
+ * rendering/EllipsisBox.cpp: Ditto. |
+ (WebCore::EllipsisBox::paint): |
+ (WebCore::EllipsisBox::nodeAtPoint): |
+ * rendering/InlineBox.cpp: Ditto. |
+ (WebCore::InlineBox::logicalHeight): |
+ * rendering/InlineFlowBox.cpp: Ditto. |
+ (WebCore::verticalPositionForBox): |
+ (WebCore::InlineFlowBox::computeLogicalBoxHeights): |
+ (WebCore::InlineFlowBox::placeBoxesInBlockDirection): |
+ * rendering/InlineTextBox.cpp: Ditto. |
+ (WebCore::InlineTextBox::paint): |
+ (WebCore::InlineTextBox::paintDecoration): |
+ (WebCore::InlineTextBox::paintSpellingOrGrammarMarker): |
+ (WebCore::InlineTextBox::paintCompositionUnderline): |
+ * rendering/RenderBlock.cpp: Ditto. |
+ (WebCore::RenderBlock::baselinePosition): |
+ (WebCore::RenderBlock::firstLineBoxBaseline): |
+ (WebCore::RenderBlock::lastLineBoxBaseline): |
+ * rendering/RenderBox.cpp: Ditto. |
+ (WebCore::RenderBox::localCaretRect): |
+ * rendering/RenderEmbeddedObject.cpp: Ditto. |
+ (WebCore::RenderEmbeddedObject::paintReplaced): |
+ * rendering/RenderImage.cpp: Ditto. |
+ (WebCore::RenderImage::setImageSizeForAltText): |
+ (WebCore::RenderImage::paintReplaced): |
+ * rendering/RenderInline.cpp: Ditto. |
+ (WebCore::RenderInline::baselinePosition): |
+ * rendering/RenderListBox.cpp: Ditto. |
+ (WebCore::RenderListBox::paintItemForeground): |
+ (WebCore::RenderListBox::itemHeight): |
+ * rendering/RenderListMarker.cpp: Ditto. |
+ (WebCore::RenderListMarker::paint): |
+ (WebCore::RenderListMarker::layout): |
+ (WebCore::RenderListMarker::computePreferredLogicalWidths): |
+ (WebCore::RenderListMarker::updateMargins): |
+ (WebCore::RenderListMarker::getRelativeMarkerRect): |
+ * rendering/RenderTextControl.cpp: Ditto. |
+ (WebCore::RenderTextControl::paintPlaceholder): |
+ * rendering/RenderTextControlSingleLine.cpp: Ditto. |
+ (WebCore::RenderTextControlSingleLine::createInnerTextStyle): |
+ * rendering/RenderThemeWin.cpp: Ditto. |
+ (WebCore::RenderThemeWin::adjustMenuListButtonStyle): |
+ * rendering/mathml/RenderMathMLFraction.cpp: Ditto. |
+ (WebCore::RenderMathMLFraction::baselinePosition): |
+ * rendering/style/RenderStyle.h: Add "const FontMetrics& fontMetrics() const" accessor. |
+ (WebCore::InheritedFlags::fontMetrics): |
+ (WebCore::InheritedFlags::computedLineHeight): |
+ * rendering/svg/RenderSVGInlineText.cpp: Use style->fontMetrics() instead of style->font() to access the metrics. |
+ (WebCore::RenderSVGInlineText::positionForPoint): |
+ * rendering/svg/SVGInlineTextBox.cpp: Ditto. |
+ (WebCore::SVGInlineTextBox::selectionRectForTextFragment): |
+ (WebCore::positionOffsetForDecoration): |
+ (WebCore::SVGInlineTextBox::paintDecorationWithStyle): |
+ (WebCore::SVGInlineTextBox::paintTextWithShadows): |
+ (WebCore::SVGInlineTextBox::calculateBoundaries): |
+ * rendering/svg/SVGTextLayoutEngineBaseline.cpp: Ditto. |
+ (WebCore::SVGTextLayoutEngineBaseline::calculateBaselineShift): |
+ (WebCore::SVGTextLayoutEngineBaseline::calculateAlignmentBaselineShift): |
+ (WebCore::SVGTextLayoutEngineBaseline::calculateGlyphAdvanceAndOrientation): |
+ * rendering/svg/SVGTextLayoutEngineSpacing.cpp: Ditto. |
+ (WebCore::SVGTextLayoutEngineSpacing::calculateSVGKerning): |
+ * rendering/svg/SVGTextMetrics.cpp: Ditto. |
+ (WebCore::SVGTextMetrics::SVGTextMetrics): |
+ * rendering/svg/SVGTextQuery.cpp: Ditto. |
+ (WebCore::calculateGlyphBoundaries): |
+ * svg/SVGFontFaceElement.cpp: |
+ (WebCore::SVGFontFaceElement::unitsPerEm): Rename defaultUnitsPerEm global to gDefaultUnitsPerEm. |
+ * svg/SVGLength.cpp: Use style->fontMetrics() instead of style->font() to access the metrics. |
+ (WebCore::SVGLength::convertValueFromUserUnitsToEXS): |
+ (WebCore::SVGLength::convertValueFromEXSToUserUnits): |
+ |
+2011-01-22 Ryosuke Niwa <rniwa@webkit.org> |
+ |
+ Reviewed by Eric Seidel. |
+ |
+ Stop instantiating legacy editing positions in VisibleSelection, visible_units.cpp, Frame, and RenderBlock |
+ https://bugs.webkit.org/show_bug.cgi?id=52759 |
+ |
+ Stopped instantiating legacy editing positions in the following files. |
+ |
+ * editing/VisibleSelection.cpp: |
+ (WebCore::VisibleSelection::adjustSelectionToAvoidCrossingEditingBoundaries): |
+ * editing/visible_units.cpp: |
+ (WebCore::previousBoundary): |
+ (WebCore::previousLinePosition): |
+ (WebCore::nextLinePosition): |
+ (WebCore::startOfBlock): |
+ * page/Frame.cpp: |
+ (WebCore::Frame::visiblePositionForPoint): |
+ * rendering/RenderBlock.cpp: Removed RenderBlock::positionForRenderer because it was not called anywhere. |
+ * rendering/RenderBlock.h: Ditto. |
+ |
+2011-01-22 Adrienne Walker <enne@google.com> |
+ |
+ Reviewed by James Robinson. |
+ |
+ [chromium] Fix compositor repaints for offscreen fixed elements |
+ https://bugs.webkit.org/show_bug.cgi?id=52681 |
+ |
+ The fast path scrolling was clipping offscreen invalidations. |
+ Additionally, the compositor was ignoring all invalidations that were |
+ entirely out of the content rect. |
+ |
+ Test: compositing/ to verify this caused no regressions. |
+ Tested fix manually in browser as I could not generate a LayoutTest that reproduced the bug in run-webkit-tests. |
+ |
+ * page/FrameView.cpp: |
+ (WebCore::FrameView::scrollContentsFastPath): |
+ * platform/graphics/chromium/LayerRendererChromium.cpp: |
+ (WebCore::LayerRendererChromium::invalidateRootLayerRect): |
+ |
+2011-01-22 Alexey Proskuryakov <ap@apple.com> |
+ |
+ Reviewed by Dan Bernstein. |
+ |
+ https://bugs.webkit.org/show_bug.cgi?id=52951 |
+ WebKit2 generates a bad PDF for cross process messaging |
+ |
+ * page/PrintContext.cpp: (WebCore::PrintContext::spoolRect): Use a correct offset to actually |
+ draw inside the requested rectangle. |
+ |
+2011-01-21 Yury Semikhatsky <yurys@chromium.org> |
+ |
+ Reviewed by Adam Barth. |
+ |
+ Regression: new window.onerror() implementation leaks cross-origin Javascript errors |
+ https://bugs.webkit.org/show_bug.cgi?id=52903 |
+ |
+ In case of an exception in a script from different domain only generic message |
+ will be passed to window.onerror hander. |
+ |
+ Tests: http/tests/security/cross-origin-script-window-onerror-redirected.html |
+ http/tests/security/cross-origin-script-window-onerror.html |
+ |
+ * bindings/js/CachedScriptSourceProvider.h: use URL from the resource response to make sure we do all |
+ cross origin checks agains real script URL, not the original URL which may have resulted in a sequence |
+ of redirects to different domains. |
+ (WebCore::CachedScriptSourceProvider::CachedScriptSourceProvider): |
+ * bindings/v8/ScriptSourceCode.h: same for v8. |
+ (WebCore::ScriptSourceCode::url): |
+ * dom/ScriptExecutionContext.cpp: |
+ (WebCore::ScriptExecutionContext::dispatchErrorEvent): in case the error occurred in a script we cannot |
+ access provide concise "Script error." message without any information about the error source. This is |
+ what Firefox does in this case. |
+ |
+2011-01-21 Andreas Kling <kling@webkit.org> |
+ |
+ Reviewed by Kenneth Rohde Christiansen. |
+ |
+ [Qt] Always set composition mode through GraphicsContext |
+ https://bugs.webkit.org/show_bug.cgi?id=52940 |
+ |
+ GraphicsContext tracks the current composition mode so we should |
+ never call through to the QPainter directly. |
+ |
+ * platform/graphics/GraphicsContext.h: |
+ * platform/graphics/qt/GraphicsContextQt.cpp: |
+ (WebCore::toQtCompositionMode): Changed this method to a static inline |
+ since it's only used by GraphicsContextQt.cpp now. |
+ |
+ * platform/graphics/qt/ImageQt.cpp: |
+ (WebCore::Image::drawPattern): |
+ (WebCore::BitmapImage::draw): |
+ |
+2011-01-21 Dan Bernstein <mitz@apple.com> |
+ |
+ Reviewed by Adele Peterson. |
+ |
+ Inconsistent handling of no-break space in justification logic |
+ https://bugs.webkit.org/show_bug.cgi?id=52938 |
+ |
+ Test: fast/text/justify-nbsp.html |
+ |
+ * rendering/RenderBlockLineLayout.cpp: |
+ (WebCore::RenderBlock::computeInlineDirectionPositionsForLine): Count no-break space as a |
+ space when computing the distribution of space between text boxes on the line. |
+ |
2011-01-21 Charlie Reis <creis@chromium.org> |
Reviewed by Darin Fisher. |