Index: WebCore/ChangeLog |
=================================================================== |
--- WebCore/ChangeLog (revision 68005) |
+++ WebCore/ChangeLog (working copy) |
@@ -1,3 +1,3308 @@ |
+2010-09-21 Cosmin Truta <ctruta@chromium.org> |
+ |
+ Reviewed by James Robinson. |
+ |
+ RenderImage::intrinsicSizeChanged crashes when m_imageResource is missing |
+ https://bugs.webkit.org/show_bug.cgi?id=46120 |
+ |
+ Avoid applying operator -> to m_imageResource when its underlying pointer is NULL. |
+ |
+ * rendering/RenderImage.h: |
+ (WebCore::RenderImage::intrinsicSizeChanged): Checked m_imageResource. |
+ * html/HTMLImageElement.cpp: |
+ (HTMLImageElement::createRenderer): Fixed indentation. |
+ |
+2010-09-21 Kenneth Russell <kbr@google.com> |
+ |
+ Reviewed by James Robinson. |
+ |
+ [chromium] Remove GLES2Context and WebGLES2Context |
+ https://bugs.webkit.org/show_bug.cgi?id=46131 |
+ |
+ Removed now-obsolete GLES2Context and WebGLES2Context classes, and |
+ dependency on command buffer client code from WebCore. Built and |
+ tested 3D CSS and WebGL content on Mac OS X to test. |
+ |
+ * WebCore.gypi: |
+ * platform/chromium/GLES2Context.h: Removed. |
+ * platform/graphics/skia/ImageSkia.cpp: |
+ |
+2010-09-21 Geoffrey Garen <ggaren@apple.com> |
+ |
+ Try to fix the Windows build. |
+ |
+ * loader/Cache.cpp: |
+ (WebCore::Cache::makeResourcePurgeable): |
+ (WebCore::Cache::evict): Cast to int before using unary '-', to avoid an |
+ MSVC warning. |
+ |
+2010-09-21 Pratik Solanki <psolanki@apple.com> |
+ |
+ Reviewed by Geoffrey Garen and Darin Adler. |
+ |
+ Use purgeable memory to keep more dead resources in cache |
+ https://bugs.webkit.org/show_bug.cgi?id=44806 |
+ <rdar://problem/8350901> |
+ |
+ This changes the behavior of dead resources in the WebCore cache to be the following if |
+ shouldMakeResourcePurgeableOnEviction() returns true. |
+ |
+ 1. Dead resources in the cache are kept in non-purgeable memory. |
+ 2. When we prune dead resources, instead of freeing them, we mark their memory as purgeable |
+ and keep the resources until the kernel reclaims the purgeable memory. |
+ |
+ By leaving the in-cache dead resources in dirty resident memory, we decrease the likelihood |
+ of the kernel claiming that memory and forcing us to refetch the resource (for example when |
+ a user presses back). |
+ |
+ And by having an unbounded number of resource objects using purgeable memory, we can use |
+ as much memory as is available on the machine. The trade-off is that the CachedResource |
+ object (and its member variables) are allocated in non-purgeable TC-malloc'd memory so |
+ we would see slightly more memory use due to this. |
+ |
+ * loader/Cache.cpp: |
+ (WebCore::Cache::resourceForURL): Adjust sizes appropriately if we made resource memory |
+ non-purgeable. |
+ (WebCore::Cache::pruneDeadResources): When removing dead resources, try first to mark their |
+ memory as purgeable. If not, evict the resource. |
+ (WebCore::Cache::makeResourcePurgeable): Added. Try to mark resource |
+ memory as purgeable. If successful, adjust the sizes so that we don't |
+ factor this resources size in the Cache size calculation. |
+ (WebCore::Cache::evict): Don't decrement size if we already did it in makeResourcePurgeable. |
+ (WebCore::Cache::dumpLRULists): Extra debug logging. |
+ * loader/Cache.h: |
+ (WebCore::Cache::shouldMakeResourcePurgeableOnEviction): Added. Indicates if the new |
+ behaviour is enabled. |
+ * loader/CachedCSSStyleSheet.cpp: |
+ (WebCore::CachedCSSStyleSheet::allClientsRemoved): Do not mark memory as purgeable. The |
+ Cache class takes care of this. |
+ * loader/CachedImage.cpp: |
+ (WebCore::CachedImage::destroyDecodedData): Ditto. |
+ * loader/CachedScript.cpp: |
+ (WebCore::CachedScript::destroyDecodedData): Ditto. |
+ |
+2010-09-21 Dan Bernstein <mitz@apple.com> |
+ |
+ Reviewed by Darin Adler. |
+ |
+ <rdar://problem/7729077> Extending the selection to sentence boundary after a line break may select extra character |
+ https://bugs.webkit.org/show_bug.cgi?id=46232 |
+ |
+ Test: editing/selection/extend-by-sentence-002.html |
+ |
+ * editing/visible_units.cpp: |
+ (WebCore::nextBoundary): The text iterator’s range end can be the position after |
+ the line break, in which case the next visible is actually after the first character |
+ of the next sentence. Instead, advance the text iterator past the newline character |
+ and return the beginning of its range, which is guaranteed to still be before the |
+ next sentence. |
+ |
+2010-09-21 Robert Hogan <robert@webkit.org> |
+ |
+ Rubber-stamped by Ariya Hidayat. |
+ |
+ [Qt] Build fix against qt trunk |
+ |
+ * platform/graphics/qt/ContextShadow.cpp: |
+ (WebCore::ContextShadow::beginShadowLayer): |
+ |
+2010-09-21 Tony Chang <tony@chromium.org> |
+ |
+ Unreviewed, QT build fix. |
+ |
+ * platform/qt/ClipboardQt.cpp: |
+ (WebCore::ClipboardQt::ClipboardQt): |
+ |
+2010-09-21 Andy Estes <aestes@apple.com> |
+ |
+ Reviewed by Darin Adler. |
+ |
+ REGRESSION (r61285): AIM 2.1.296: Code rendered as text in Welcome screen |
+ https://bugs.webkit.org/show_bug.cgi?id=46134 |
+ |
+ * WebCore.exp.in: Export __ZN7WebCore32applicationIsAOLInstantMessengerEv. |
+ * platform/mac/RuntimeApplicationChecks.h: |
+ * platform/mac/RuntimeApplicationChecks.mm: |
+ (WebCore::applicationIsAOLInstantMessenger): Determine if the embedding |
+ application is AOL Instant Messenger by checking the bundle identifier. |
+ |
+2010-09-21 Chris Guillory <chris.guillory@google.com> |
+ |
+ Reviewed by Chris Fleizach. |
+ |
+ Send webkit accessibility notifications to Chromium. |
+ https://bugs.webkit.org/show_bug.cgi?id=45156 |
+ |
+ * WebCore.gypi: |
+ * accessibility/chromium/AXObjectCacheChromium.cpp: |
+ (WebCore::AXObjectCache::postPlatformNotification): |
+ (WebCore::AXObjectCache::handleFocusedUIElementChanged): |
+ (WebCore::AXObjectCache::handleScrolledToAnchor): |
+ * dom/Document.cpp: |
+ (WebCore::Document::implicitClose): |
+ (WebCore::Document::setFocusedNode): |
+ * editing/SelectionController.h: |
+ * editing/chromium/SelectionControllerChromium.cpp: Added. |
+ (WebCore::SelectionController::notifyAccessibilityForSelectionChange): |
+ * page/FrameView.cpp: |
+ (WebCore::FrameView::layout): |
+ * page/chromium/ChromeClientChromium.h: |
+ |
+2010-09-21 Alejandro G. Castro <alex@igalia.com> |
+ |
+ Reviewed by Martin Robinson. |
+ |
+ [gtk] Move copyContextProperties outside the GraphicsContextCairo |
+ https://bugs.webkit.org/show_bug.cgi?id=46179. |
+ |
+ Moved the copyContextProperties function to the CairoUtilities so |
+ we can use it outside GraphicsContextCairo. We moved also the old |
+ gtk CairoUtilities file to GdkCairoUtilities and added the new |
+ file in the cairo directory. |
+ |
+ * GNUmakefile.am: |
+ * platform/graphics/cairo/CairoUtilities.cpp: Added. |
+ (WebCore::copyContextProperties): |
+ * platform/graphics/cairo/CairoUtilities.h: Added. |
+ * platform/graphics/cairo/GraphicsContextCairo.cpp: |
+ * platform/graphics/gtk/CairoUtilities.cpp: Removed. |
+ * platform/graphics/gtk/CairoUtilities.h: Removed. |
+ * platform/graphics/gtk/GdkCairoUtilities.cpp: Added. |
+ (getCairoSurfacePixel): |
+ (getGdkPixbufPixel): |
+ (cairoImageSurfaceToGdkPixbuf): |
+ * platform/graphics/gtk/GdkCairoUtilities.h: Added. |
+ * platform/graphics/gtk/ImageBufferGtk.cpp: |
+ * platform/graphics/gtk/ImageGtk.cpp: |
+ |
+2010-09-21 Vangelis Kokkevis <vangelis@chromium.org> |
+ |
+ Reviewed by James Robinson. |
+ |
+ [chromium] Cleaned up LayerRendererChromium::prepareToDrawLayers |
+ logic to avoid unnecessary copy operations if the existing root layer |
+ contents aren't useful for scrolling. Also removed call to updateLayersRecursive() |
+ as the composited layer update is actually happening in the drawLayers() method. |
+ https://bugs.webkit.org/show_bug.cgi?id=46199 |
+ |
+ |
+ * platform/graphics/chromium/LayerRendererChromium.cpp: |
+ (WebCore::LayerRendererChromium::prepareToDrawLayers): |
+ |
+2010-09-21 Daniel Cheng <dcheng@chromium.org> |
+ |
+ Reviewed by Tony Chang. |
+ |
+ Change Clipboard to use an enum instead of isForDragging = true/false |
+ https://bugs.webkit.org/show_bug.cgi?id=46004 |
+ |
+ No new tests, since it's just a refactoring change. |
+ |
+ * bindings/js/JSClipboardCustom.cpp: |
+ (WebCore::JSClipboard::setDragImage): |
+ * bindings/v8/custom/V8ClipboardCustom.cpp: |
+ (WebCore::V8Clipboard::setDragImageCallback): |
+ * dom/Clipboard.cpp: |
+ (WebCore::Clipboard::Clipboard): |
+ (WebCore::Clipboard::setDropEffect): |
+ (WebCore::Clipboard::setEffectAllowed): |
+ * dom/Clipboard.h: |
+ (WebCore::Clipboard::isForCopyAndPaste): |
+ (WebCore::Clipboard::isForDragAndDrop): |
+ * editing/android/EditorAndroid.cpp: |
+ (WebCore::Editor::newGeneralClipboard): |
+ * editing/brew/EditorBrew.cpp: |
+ (WebCore::Editor::newGeneralClipboard): |
+ * editing/chromium/EditorChromium.cpp: |
+ (WebCore::Editor::newGeneralClipboard): |
+ * editing/haiku/EditorHaiku.cpp: |
+ (WebCore::Editor::newGeneralClipboard): |
+ * editing/mac/EditorMac.mm: |
+ (WebCore::Editor::newGeneralClipboard): |
+ * editing/wx/EditorWx.cpp: |
+ (WebCore::Editor::newGeneralClipboard): |
+ * page/chromium/EventHandlerChromium.cpp: |
+ (WebCore::EventHandler::createDraggingClipboard): |
+ * page/efl/EventHandlerEfl.cpp: |
+ (WebCore::EventHandler::createDraggingClipboard): |
+ * page/gtk/EventHandlerGtk.cpp: |
+ (WebCore::EventHandler::createDraggingClipboard): |
+ * page/haiku/EventHandlerHaiku.cpp: |
+ (WebCore::EventHandler::createDraggingClipboard): |
+ * page/mac/EventHandlerMac.mm: |
+ (WebCore::EventHandler::createDraggingClipboard): |
+ * page/qt/EventHandlerQt.cpp: |
+ (WebCore::EventHandler::createDraggingClipboard): |
+ * page/win/EventHandlerWin.cpp: |
+ (WebCore::EventHandler::createDraggingClipboard): |
+ * page/wx/EventHandlerWx.cpp: |
+ (WebCore::EventHandler::createDraggingClipboard): |
+ * platform/android/ClipboardAndroid.cpp: |
+ (WebCore::ClipboardAndroid::ClipboardAndroid): |
+ (WebCore::ClipboardAndroid::clearData): |
+ (WebCore::ClipboardAndroid::clearAllData): |
+ (WebCore::ClipboardAndroid::setData): |
+ * platform/android/ClipboardAndroid.h: |
+ * platform/brew/ClipboardBrew.cpp: |
+ (WebCore::ClipboardBrew::ClipboardBrew): |
+ (WebCore::ClipboardBrew::clearData): |
+ (WebCore::ClipboardBrew::clearAllData): |
+ (WebCore::ClipboardBrew::setData): |
+ * platform/brew/ClipboardBrew.h: |
+ * platform/chromium/ClipboardChromium.cpp: |
+ (WebCore::Clipboard::create): |
+ (WebCore::ClipboardChromium::ClipboardChromium): |
+ (WebCore::ClipboardChromium::create): |
+ (WebCore::ClipboardChromium::getData): |
+ * platform/chromium/ClipboardChromium.h: |
+ * platform/efl/ClipboardEfl.cpp: |
+ (WebCore::Editor::newGeneralClipboard): |
+ (WebCore::ClipboardEfl::ClipboardEfl): |
+ * platform/efl/ClipboardEfl.h: |
+ (WebCore::ClipboardEfl::create): |
+ * platform/gtk/ClipboardGtk.cpp: |
+ (WebCore::Editor::newGeneralClipboard): |
+ (WebCore::Clipboard::create): |
+ (WebCore::ClipboardGtk::ClipboardGtk): |
+ (WebCore::dataObjectTypeFromHTMLClipboardType): |
+ (WebCore::ClipboardGtk::clearData): |
+ (WebCore::ClipboardGtk::getData): |
+ (WebCore::ClipboardGtk::setData): |
+ * platform/gtk/ClipboardGtk.h: |
+ (WebCore::ClipboardGtk::create): |
+ * platform/haiku/ClipboardHaiku.cpp: |
+ (WebCore::Clipboard::create): |
+ (WebCore::ClipboardHaiku::ClipboardHaiku): |
+ * platform/haiku/ClipboardHaiku.h: |
+ (WebCore::ClipboardHaiku::create): |
+ * platform/mac/ClipboardMac.h: |
+ (WebCore::ClipboardMac::create): |
+ * platform/mac/ClipboardMac.mm: |
+ (WebCore::Clipboard::create): |
+ (WebCore::ClipboardMac::ClipboardMac): |
+ * platform/qt/ClipboardQt.cpp: |
+ (WebCore::ClipboardQt::ClipboardQt): |
+ (WebCore::ClipboardQt::~ClipboardQt): |
+ (WebCore::ClipboardQt::clearData): |
+ (WebCore::ClipboardQt::clearAllData): |
+ (WebCore::ClipboardQt::setData): |
+ (WebCore::ClipboardQt::declareAndWriteDragImage): |
+ (WebCore::ClipboardQt::writeURL): |
+ (WebCore::ClipboardQt::writeRange): |
+ (WebCore::ClipboardQt::writePlainText): |
+ * platform/qt/ClipboardQt.h: |
+ (WebCore::ClipboardQt::create): |
+ * platform/win/ClipboardWin.cpp: |
+ (WebCore::Clipboard::create): |
+ (WebCore::ClipboardWin::ClipboardWin): |
+ (WebCore::ClipboardWin::clearData): |
+ (WebCore::ClipboardWin::clearAllData): |
+ (WebCore::ClipboardWin::setData): |
+ (WebCore::ClipboardWin::setExternalDataObject): |
+ * platform/win/ClipboardWin.h: |
+ (WebCore::ClipboardWin::create): |
+ * platform/win/EditorWin.cpp: |
+ (WebCore::Editor::newGeneralClipboard): |
+ * platform/wx/ClipboardWx.cpp: |
+ (WebCore::ClipboardWx::ClipboardWx): |
+ * platform/wx/ClipboardWx.h: |
+ (WebCore::ClipboardWx::create): |
+ |
+2010-09-21 Abhishek Arya <inferno@chromium.org> |
+ |
+ Reviewed by Dave Hyatt. |
+ |
+ Take isValueList() checks out of the asserts for memory safety. |
+ https://bugs.webkit.org/show_bug.cgi?id=46194 |
+ |
+ Test: editing/execCommand/apply-style-text-decoration-crash.html |
+ |
+ * editing/ApplyStyleCommand.cpp: |
+ (WebCore::StyleChange::extractTextStyles): |
+ (WebCore::ApplyStyleCommand::applyInlineStyleToPushDown): |
+ |
+2010-09-21 Vangelis Kokkevis <vangelis@chromium.org> |
+ |
+ Reviewed by Kenneth Russell. |
+ |
+ [chromium] Fixing double translation of composited layers when scrolling. |
+ The scrolling transform is now applied by a separate layer maintained in |
+ the RenderLayerCompositor. |
+ https://bugs.webkit.org/show_bug.cgi?id=46193 |
+ |
+ * platform/graphics/chromium/LayerRendererChromium.cpp: |
+ (WebCore::LayerRendererChromium::drawLayers): |
+ |
+2010-09-21 David Hyatt <hyatt@apple.com> |
+ |
+ Reviewed by Dan Bernstein. |
+ |
+ https://bugs.webkit.org/show_bug.cgi?id=46196, add marginStart/Before/End/After accessors to RenderBoxModelObject. |
+ |
+ * rendering/RenderBox.cpp: |
+ (WebCore::RenderBox::marginBefore): |
+ (WebCore::RenderBox::marginAfter): |
+ (WebCore::RenderBox::marginStart): |
+ (WebCore::RenderBox::marginEnd): |
+ * rendering/RenderBox.h: |
+ * rendering/RenderBoxModelObject.h: |
+ * rendering/RenderInline.cpp: |
+ (WebCore::computeMargin): |
+ (WebCore::RenderInline::marginLeft): |
+ (WebCore::RenderInline::marginRight): |
+ (WebCore::RenderInline::marginTop): |
+ (WebCore::RenderInline::marginBottom): |
+ (WebCore::RenderInline::marginStart): |
+ (WebCore::RenderInline::marginEnd): |
+ * rendering/RenderInline.h: |
+ (WebCore::RenderInline::marginBefore): |
+ (WebCore::RenderInline::marginAfter): |
+ |
+2010-09-21 David Hyatt <hyatt@apple.com> |
+ |
+ Reviewed by Dan Bernstein. |
+ |
+ https://bugs.webkit.org/show_bug.cgi?id=46190, add borderStart/Before/End/After accessors to RenderBoxModelObject. |
+ |
+ * rendering/RenderBoxModelObject.h: |
+ (WebCore::RenderBoxModelObject::borderBefore): |
+ (WebCore::RenderBoxModelObject::borderAfter): |
+ (WebCore::RenderBoxModelObject::borderStart): |
+ (WebCore::RenderBoxModelObject::borderEnd): |
+ (WebCore::RenderBoxModelObject::borderAndPaddingLogicalHeight): |
+ (WebCore::RenderBoxModelObject::borderAndPaddingLogicalWidth): |
+ * rendering/RenderTableCell.cpp: |
+ (WebCore::RenderTableCell::borderStart): |
+ (WebCore::RenderTableCell::borderEnd): |
+ (WebCore::RenderTableCell::borderBefore): |
+ (WebCore::RenderTableCell::borderAfter): |
+ * rendering/RenderTableCell.h: |
+ |
+2010-09-21 Martin Robinson <mrobinson@igalia.com> |
+ |
+ Reviewed by Xan Lopez. |
+ |
+ [GTK] 'make dist' should be fixed in preparation for the next release |
+ https://bugs.webkit.org/show_bug.cgi?id=46129 |
+ |
+ * GNUmakefile.am: Update the sources list to include missing headers. |
+ |
+2010-09-21 David Hyatt <hyatt@apple.com> |
+ |
+ Reviewed by John Sullivan. |
+ |
+ https://bugs.webkit.org/show_bug.cgi?id=46188, make the cross-platform setPrinting method match the Mac version of the method. |
+ Specifically don't pass the page size down to subframes of the one you're printing. |
+ |
+ * page/Frame.cpp: |
+ (WebCore::Frame::setPrinting): |
+ |
+2010-09-21 Patrick Gansterer <paroga@webkit.org> |
+ |
+ Reviewed by Adam Roben. |
+ |
+ Use WTF::Vector for storing formData in ResourceHandleWin. |
+ https://bugs.webkit.org/show_bug.cgi?id=46180 |
+ |
+ * platform/network/ResourceHandleInternal.h: |
+ (WebCore::ResourceHandleInternal::ResourceHandleInternal): |
+ * platform/network/win/ResourceHandleWin.cpp: |
+ (WebCore::ResourceHandle::onHandleCreated): |
+ (WebCore::ResourceHandle::onRequestComplete): |
+ |
+2010-09-21 Jocelyn Turcotte <jocelyn.turcotte@nokia.com> |
+ |
+ Reviewed by Andreas Kling. |
+ |
+ [Qt] Check if the reply has been deleted before finishing a network request |
+ https://bugs.webkit.org/show_bug.cgi?id=46174 |
+ |
+ A crash can happen with the following sequence: |
+ 1. QNetworkReplyHandler::abort() emits reply->deleteLater() |
+ 2. QNAM emits QNetworkReply::finished() -> calls QNetworkReplyHandler::finish() |
+ 3. event loop would call reply->deleteLater() |
+ However a crash occurs since m_reply == 0 on step 2. |
+ |
+ * platform/network/qt/QNetworkReplyHandler.cpp: |
+ (WebCore::QNetworkReplyHandler::finish): |
+ |
+2010-09-21 Jochen Eisinger <jochen@chromium.org> |
+ |
+ Reviewed by Jeremy Orlow. |
+ |
+ Use @ to concatenate the database name and the database identifier when building |
+ the filename to use for indexed databases. That character is valid for filenames |
+ but cannot occur in a database identifier derived from a security origin. |
+ https://bugs.webkit.org/show_bug.cgi?id=46178 |
+ |
+ * storage/IDBFactoryBackendImpl.cpp: |
+ (WebCore::IDBFactoryBackendImpl::databaseFileName): |
+ |
+2010-09-21 Steve Block <steveblock@google.com> |
+ |
+ Reviewed by Jeremy Orlow. |
+ |
+ DeviceOrientationClient and DeviceMotionClient should have controllerDestroyed() methods |
+ https://bugs.webkit.org/show_bug.cgi?id=45891 |
+ |
+ Tested by existing DeviceOrientation tests. |
+ |
+ * dom/DeviceMotionClient.h: |
+ * dom/DeviceMotionController.cpp: |
+ (WebCore::DeviceMotionController::~DeviceMotionController): |
+ * dom/DeviceMotionController.h: |
+ * dom/DeviceOrientationClient.h: |
+ * dom/DeviceOrientationController.cpp: |
+ (WebCore::DeviceOrientationController::~DeviceOrientationController): |
+ * dom/DeviceOrientationController.h: |
+ * loader/EmptyClients.h: |
+ (WebCore::EmptyDeviceMotionClient::deviceOrientationControllerDestroyed): |
+ (WebCore::EmptyDeviceOrientationClient::deviceOrientationControllerDestroyed): |
+ * platform/mock/DeviceOrientationClientMock.h: |
+ (WebCore::DeviceOrientationClientMock::deviceOrientationControllerDestroyed): |
+ |
+2010-09-21 Jochen Eisinger <jochen@chromium.org> |
+ |
+ Reviewed by Jeremy Orlow. |
+ |
+ Factor out the file name used for a given indexed database to a static method. |
+ https://bugs.webkit.org/show_bug.cgi?id=46090 |
+ |
+ * storage/IDBFactoryBackendImpl.cpp: |
+ (WebCore::openSQLiteDatabase): |
+ (WebCore::IDBFactoryBackendImpl::databaseFileName): |
+ * storage/IDBFactoryBackendImpl.h: |
+ |
+2010-09-21 Dirk Schulze <krit@webkit.org> |
+ |
+ SVG Filter cleanup |
+ https://bugs.webkit.org/show_bug.cgi?id=45612 |
+ |
+ Unreviewed change of epsilon back to a more tolerant value in RenderTreeAsText and made it a static const. |
+ This should turn the bots green again. |
+ |
+ * rendering/RenderTreeAsText.cpp: |
+ (WebCore::hasFractions): |
+ |
+2010-09-21 David Hyatt <hyatt@apple.com> |
+ |
+ Reviewed by Dan Bernstein. |
+ |
+ https://bugs.webkit.org/show_bug.cgi?id=46125, convert table cell intrinsic padding from top/bottom-based |
+ to before/after-based. A vertical text table can have intrinsic padding built into the left/right |
+ direction (and this allows the base class logical padding methods on RenderBoxModelObject to be safe to use). |
+ |
+ * rendering/RenderTableCell.cpp: |
+ (WebCore::RenderTableCell::RenderTableCell): |
+ (WebCore::RenderTableCell::paddingTop): |
+ (WebCore::RenderTableCell::paddingBottom): |
+ (WebCore::RenderTableCell::paddingLeft): |
+ (WebCore::RenderTableCell::paddingRight): |
+ (WebCore::RenderTableCell::paddingBefore): |
+ (WebCore::RenderTableCell::paddingAfter): |
+ * rendering/RenderTableCell.h: |
+ (WebCore::RenderTableCell::setIntrinsicPaddingBefore): |
+ (WebCore::RenderTableCell::setIntrinsicPaddingAfter): |
+ (WebCore::RenderTableCell::setIntrinsicPadding): |
+ (WebCore::RenderTableCell::intrinsicPaddingBefore): |
+ (WebCore::RenderTableCell::intrinsicPaddingAfter): |
+ * rendering/RenderTableSection.cpp: |
+ (WebCore::RenderTableSection::calcRowHeight): |
+ (WebCore::RenderTableSection::layoutRows): |
+ * rendering/RenderTreeAsText.cpp: |
+ (WebCore::RenderTreeAsText::writeRenderObject): |
+ (WebCore::writeTextRun): |
+ |
+2010-09-21 Dirk Schulze <krit@webkit.org> |
+ |
+ Reviewed by Nikolas Zimmermann. |
+ |
+ SVG Filter cleanup |
+ https://bugs.webkit.org/show_bug.cgi?id=45612 |
+ |
+ Second cleanup patch. Modified files to follow webkit style. Removed dependencies |
+ to SVG of most of the filter effects and prepared them to move from WebCore/svg/graphics/filters |
+ to WebCore/platform/graphics/filters/. |
+ Renamed calculateDrawingIntRect and calculateDrawingRect to requestedRegionOfInputImageData and |
+ drawingRegionOfInputImage to clarify the functionality and make both functions distinguishable. |
+ |
+ * platform/graphics/filters/FEBlend.cpp: |
+ (WebCore::FEBlend::apply): |
+ * platform/graphics/filters/FEBlend.h: |
+ * platform/graphics/filters/FEColorMatrix.cpp: |
+ (WebCore::FEColorMatrix::apply): |
+ * platform/graphics/filters/FEColorMatrix.h: |
+ * platform/graphics/filters/FEComponentTransfer.cpp: |
+ (WebCore::FEComponentTransfer::apply): |
+ * platform/graphics/filters/FEComponentTransfer.h: |
+ (WebCore::ComponentTransferFunction::ComponentTransferFunction): |
+ * platform/graphics/filters/FEComposite.cpp: |
+ (WebCore::FEComposite::apply): |
+ * platform/graphics/filters/FEComposite.h: |
+ * platform/graphics/filters/FEGaussianBlur.cpp: |
+ (WebCore::FEGaussianBlur::apply): |
+ * platform/graphics/filters/FEGaussianBlur.h: |
+ * platform/graphics/filters/Filter.h: |
+ * platform/graphics/filters/FilterEffect.cpp: |
+ (WebCore::FilterEffect::requestedRegionOfInputImageData): |
+ (WebCore::FilterEffect::drawingRegionOfInputImage): |
+ (WebCore::FilterEffect::effectContext): |
+ * platform/graphics/filters/FilterEffect.h: |
+ * platform/graphics/filters/ImageBufferFilter.cpp: |
+ * platform/graphics/filters/ImageBufferFilter.h: |
+ * platform/graphics/filters/SourceAlpha.cpp: |
+ (WebCore::SourceAlpha::apply): |
+ * platform/graphics/filters/SourceAlpha.h: |
+ (WebCore::SourceAlpha::isSourceInput): |
+ (WebCore::SourceAlpha::SourceAlpha): |
+ * platform/graphics/filters/SourceGraphic.cpp: |
+ (WebCore::SourceGraphic::apply): |
+ * platform/graphics/filters/SourceGraphic.h: |
+ (WebCore::SourceGraphic::isSourceInput): |
+ (WebCore::SourceGraphic::SourceGraphic): |
+ * rendering/RenderTreeAsText.cpp: |
+ (WebCore::hasFractions): |
+ (WebCore::operator<<): |
+ * rendering/RenderTreeAsText.h: |
+ (WebCore::operator<<): |
+ * rendering/SVGRenderTreeAsText.cpp: |
+ * rendering/SVGRenderTreeAsText.h: |
+ * svg/graphics/filters/SVGFEConvolveMatrix.cpp: |
+ (WebCore::FEConvolveMatrix::apply): |
+ * svg/graphics/filters/SVGFEConvolveMatrix.h: |
+ * svg/graphics/filters/SVGFEDiffuseLighting.cpp: |
+ * svg/graphics/filters/SVGFEDiffuseLighting.h: |
+ * svg/graphics/filters/SVGFEDisplacementMap.cpp: |
+ (WebCore::FEDisplacementMap::apply): |
+ * svg/graphics/filters/SVGFEDisplacementMap.h: |
+ * svg/graphics/filters/SVGFEFlood.cpp: |
+ (WebCore::FEFlood::apply): |
+ (WebCore::FEFlood::externalRepresentation): |
+ * svg/graphics/filters/SVGFEFlood.h: |
+ * svg/graphics/filters/SVGFEImage.cpp: |
+ (WebCore::FEImage::apply): |
+ * svg/graphics/filters/SVGFEImage.h: |
+ * svg/graphics/filters/SVGFELighting.cpp: |
+ (WebCore::FELighting::apply): |
+ * svg/graphics/filters/SVGFELighting.h: |
+ * svg/graphics/filters/SVGFEMerge.cpp: |
+ (WebCore::FEMerge::apply): |
+ * svg/graphics/filters/SVGFEMerge.h: |
+ * svg/graphics/filters/SVGFEMorphology.cpp: |
+ (WebCore::FEMorphology::apply): |
+ * svg/graphics/filters/SVGFEMorphology.h: |
+ * svg/graphics/filters/SVGFEOffset.cpp: |
+ (WebCore::FEOffset::apply): |
+ * svg/graphics/filters/SVGFEOffset.h: |
+ * svg/graphics/filters/SVGFESpecularLighting.cpp: |
+ * svg/graphics/filters/SVGFESpecularLighting.h: |
+ * svg/graphics/filters/SVGFETile.cpp: |
+ (WebCore::FETile::apply): |
+ * svg/graphics/filters/SVGFETile.h: |
+ * svg/graphics/filters/SVGFETurbulence.cpp: |
+ (WebCore::FETurbulence::apply): |
+ * svg/graphics/filters/SVGFETurbulence.h: |
+ |
+2010-09-20 Philippe Normand <pnormand@igalia.com> |
+ |
+ Reviewed by Eric Carlson. |
+ |
+ [GTK] enhanced context menu for media elements |
+ https://bugs.webkit.org/show_bug.cgi?id=45021 |
+ |
+ New items in the media Element context menu: |
+ - play/pause |
+ - mute/unmute |
+ - controls display control |
+ - switch to fullscreen (for video only) |
+ - loop playback control |
+ - copy media url to clipboard |
+ - open in new window |
+ |
+ Test: media/context-menu-actions.html |
+ |
+ * page/ContextMenuController.cpp: |
+ (WebCore::ContextMenuController::contextMenuItemSelected): |
+ * platform/ContextMenu.cpp: |
+ (WebCore::ContextMenu::populate): |
+ (WebCore::ContextMenu::checkOrEnableIfNeeded): |
+ * platform/ContextMenuItem.h: |
+ * platform/LocalizationStrategy.h: |
+ * platform/LocalizedStrings.cpp: |
+ (WebCore::contextMenuItemTagOpenVideoInNewWindow): |
+ (WebCore::contextMenuItemTagOpenAudioInNewWindow): |
+ (WebCore::contextMenuItemTagCopyVideoLinkToClipboard): |
+ (WebCore::contextMenuItemTagCopyAudioLinkToClipboard): |
+ (WebCore::contextMenuItemTagToggleMediaControls): |
+ (WebCore::contextMenuItemTagToggleMediaLoop): |
+ (WebCore::contextMenuItemTagEnterVideoFullscreen): |
+ (WebCore::contextMenuItemTagMediaPlay): |
+ (WebCore::contextMenuItemTagMediaPause): |
+ (WebCore::contextMenuItemTagMediaMute): |
+ * platform/LocalizedStrings.h: |
+ * platform/android/LocalizedStringsAndroid.cpp: |
+ (WebCore::contextMenuItemTagOpenVideoInNewWindow): |
+ (WebCore::contextMenuItemTagOpenAudioInNewWindow): |
+ (WebCore::contextMenuItemTagCopyVideoLinkToClipboard): |
+ (WebCore::contextMenuItemTagCopyAudioLinkToClipboard): |
+ (WebCore::contextMenuItemTagToggleMediaControls): |
+ (WebCore::contextMenuItemTagToggleMediaLoop): |
+ (WebCore::contextMenuItemTagEnterVideoFullscreen): |
+ (WebCore::contextMenuItemTagMediaPlay): |
+ (WebCore::contextMenuItemTagMediaPause): |
+ (WebCore::contextMenuItemTagMediaMute): |
+ * platform/brew/LocalizedStringsBrew.cpp: |
+ (WebCore::contextMenuItemTagOpenVideoInNewWindow): |
+ (WebCore::contextMenuItemTagOpenAudioInNewWindow): |
+ (WebCore::contextMenuItemTagCopyVideoLinkToClipboard): |
+ (WebCore::contextMenuItemTagCopyAudioLinkToClipboard): |
+ (WebCore::contextMenuItemTagToggleMediaControls): |
+ (WebCore::contextMenuItemTagToggleMediaLoop): |
+ (WebCore::contextMenuItemTagEnterVideoFullscreen): |
+ (WebCore::contextMenuItemTagMediaPlay): |
+ (WebCore::contextMenuItemTagMediaPause): |
+ (WebCore::contextMenuItemTagMediaMute): |
+ * platform/efl/LocalizedStringsEfl.cpp: |
+ (WebCore::contextMenuItemTagOpenVideoInNewWindow): |
+ (WebCore::contextMenuItemTagOpenAudioInNewWindow): |
+ (WebCore::contextMenuItemTagCopyVideoLinkToClipboard): |
+ (WebCore::contextMenuItemTagCopyAudioLinkToClipboard): |
+ (WebCore::contextMenuItemTagToggleMediaControls): |
+ (WebCore::contextMenuItemTagToggleMediaLoop): |
+ (WebCore::contextMenuItemTagEnterVideoFullscreen): |
+ (WebCore::contextMenuItemTagMediaPlay): |
+ (WebCore::contextMenuItemTagMediaPause): |
+ (WebCore::contextMenuItemTagMediaMute): |
+ * platform/gtk/ContextMenuItemGtk.cpp: |
+ (WebCore::gtkStockIDFromContextMenuAction): |
+ * platform/gtk/LocalizedStringsGtk.cpp: |
+ (WebCore::contextMenuItemTagOpenVideoInNewWindow): |
+ (WebCore::contextMenuItemTagOpenAudioInNewWindow): |
+ (WebCore::contextMenuItemTagCopyVideoLinkToClipboard): |
+ (WebCore::contextMenuItemTagCopyAudioLinkToClipboard): |
+ (WebCore::contextMenuItemTagToggleMediaControls): |
+ (WebCore::contextMenuItemTagToggleMediaLoop): |
+ (WebCore::contextMenuItemTagEnterVideoFullscreen): |
+ (WebCore::contextMenuItemTagMediaPlay): |
+ (WebCore::contextMenuItemTagMediaPause): |
+ (WebCore::contextMenuItemTagMediaMute): |
+ * platform/haiku/LocalizedStringsHaiku.cpp: |
+ (WebCore::contextMenuItemTagOpenVideoInNewWindow): |
+ (WebCore::contextMenuItemTagOpenAudioInNewWindow): |
+ (WebCore::contextMenuItemTagCopyVideoLinkToClipboard): |
+ (WebCore::contextMenuItemTagCopyAudioLinkToClipboard): |
+ (WebCore::contextMenuItemTagToggleMediaControls): |
+ (WebCore::contextMenuItemTagToggleMediaLoop): |
+ (WebCore::contextMenuItemTagEnterVideoFullscreen): |
+ (WebCore::contextMenuItemTagMediaPlay): |
+ (WebCore::contextMenuItemTagMediaPause): |
+ (WebCore::contextMenuItemTagMediaMute): |
+ * platform/wx/LocalizedStringsWx.cpp: |
+ (WebCore::contextMenuItemTagOpenVideoInNewWindow): |
+ (WebCore::contextMenuItemTagOpenAudioInNewWindow): |
+ (WebCore::contextMenuItemTagCopyVideoLinkToClipboard): |
+ (WebCore::contextMenuItemTagCopyAudioLinkToClipboard): |
+ (WebCore::contextMenuItemTagToggleMediaControls): |
+ (WebCore::contextMenuItemTagToggleMediaLoop): |
+ (WebCore::contextMenuItemTagEnterVideoFullscreen): |
+ (WebCore::contextMenuItemTagMediaPlay): |
+ (WebCore::contextMenuItemTagMediaPause): |
+ (WebCore::contextMenuItemTagMediaMute): |
+ * rendering/HitTestResult.cpp: |
+ (WebCore::HitTestResult::absoluteMediaURL): |
+ (WebCore::HitTestResult::mediaSupportsFullscreen): |
+ (WebCore::HitTestResult::mediaElement): |
+ (WebCore::HitTestResult::toggleMediaControlsDisplay): |
+ (WebCore::HitTestResult::toggleMediaLoopPlayback): |
+ (WebCore::HitTestResult::enterFullscreenForVideo): |
+ (WebCore::HitTestResult::mediaControlsEnabled): |
+ (WebCore::HitTestResult::mediaLoopEnabled): |
+ (WebCore::HitTestResult::mediaPlaying): |
+ (WebCore::HitTestResult::toggleMediaPlayState): |
+ (WebCore::HitTestResult::mediaHasAudio): |
+ (WebCore::HitTestResult::mediaMuted): |
+ (WebCore::HitTestResult::toggleMediaMuteState): |
+ * rendering/HitTestResult.h: |
+ |
+2010-09-20 Kwang Yul Seo <skyul@company100.net> |
+ |
+ Reviewed by Daniel Bates. |
+ |
+ [BREWMP] Handle tab key |
+ https://bugs.webkit.org/show_bug.cgi?id=46104 |
+ |
+ In Brew MP, AVK_FUNCTION is the key code for tab. |
+ Process AVK_FUNCTION as tab in PlatformKeyboardEvent. |
+ |
+ * platform/brew/PlatformKeyboardEventBrew.cpp: |
+ (WebCore::keyIdentifierForBrewKeyCode): |
+ (WebCore::windowsKeyCodeForKeyEvent): |
+ |
+2010-09-20 Sheriff Bot <webkit.review.bot@gmail.com> |
+ |
+ Unreviewed, rolling out r67918. |
+ http://trac.webkit.org/changeset/67918 |
+ https://bugs.webkit.org/show_bug.cgi?id=46162 |
+ |
+ Broke chromium win compile (Requested by jamesr on #webkit). |
+ |
+ * WebCore.gyp/WebCore.gyp: |
+ * WebCore.gypi: |
+ * platform/graphics/chromium/GlyphPageTreeNodeLinux.cpp: Renamed from WebCore/platform/graphics/skia/GlyphPageTreeNodeSkia.cpp. |
+ (WebCore::GlyphPage::fill): |
+ |
+2010-09-20 Daniel Bates <dbates@rim.com> |
+ |
+ Reviewed by Dan Bernstein. |
+ |
+ Cleanup: Extract common border radii expansion code in |
+ RenderBoxModelObject::paintBoxShadow() into function |
+ https://bugs.webkit.org/show_bug.cgi?id=45934 |
+ |
+ Extracted code for expanding and clamping the border radii |
+ into common function to remove duplicate code. |
+ |
+ No functionality was changed. So, no new tests. |
+ |
+ * rendering/RenderBoxModelObject.cpp: |
+ (WebCore::uniformlyExpandBorderRadii): Added. |
+ (WebCore::RenderBoxModelObject::paintBoxShadow): Moved common code to uniformlyExpandBorderRadii(). |
+ |
+2010-09-20 Kwang Yul Seo <skyul@company100.net> |
+ |
+ Reviewed by James Robinson. |
+ |
+ Make sure skia is not Chromium specific |
+ https://bugs.webkit.org/show_bug.cgi?id=39672 |
+ |
+ GGlyphPageTreeNodeLinux does not depend on Linux or Chromium. |
+ Move GGlyphPageTreeNodeLinux to platform/graphics/skia and rename it to |
+ GlyphPageTreeNodeSkia.cpp so that other ports can use it. |
+ |
+ * WebCore.gyp/WebCore.gyp: |
+ * WebCore.gypi: |
+ * platform/graphics/chromium/GlyphPageTreeNodeLinux.cpp: Removed. |
+ * platform/graphics/skia/GlyphPageTreeNodeSkia.cpp: Added. |
+ |
+2010-09-20 Kinuko Yasuda <kinuko@chromium.org> |
+ |
+ Reviewed by David Levin. |
+ |
+ Add NoStaticTables attributes to IDLs of FileSystem API for Worker support |
+ https://bugs.webkit.org/show_bug.cgi?id=46021 |
+ |
+ Added NoStaticTables attributes to IDLs except for ones for Callbacks. |
+ (As NoStaticTables only affects non-callback interfaces.) |
+ |
+ * fileapi/DOMFileSystem.idl: |
+ * fileapi/DirectoryEntry.idl: |
+ * fileapi/DirectoryReader.idl: |
+ * fileapi/Entry.idl: |
+ * fileapi/EntryArray.idl: |
+ * fileapi/FileEntry.idl: |
+ * fileapi/Flags.idl: |
+ * fileapi/Metadata.idl: |
+ |
+2010-09-20 Nico Weber <thakis@chromium.org> |
+ |
+ Reviewed by Adam Barth. |
+ |
+ Fix "operands of ? are integers of different signs" clang warnings |
+ https://bugs.webkit.org/show_bug.cgi?id=46157 |
+ |
+ * platform/graphics/mac/ComplexTextController.cpp: |
+ (WebCore::ComplexTextController::advance): |
+ * rendering/RenderBox.cpp: |
+ (WebCore::RenderBox::calcHeight): |
+ |
+2010-09-20 Vangelis Kokkevis <vangelis@chromium.org> |
+ |
+ Unreviewed, rolling out r67906. |
+ http://trac.webkit.org/changeset/67906 |
+ https://bugs.webkit.org/show_bug.cgi?id=46139 |
+ |
+ Change causes chromium to crash when switching pages |
+ |
+ * platform/graphics/chromium/ContentLayerChromium.h: |
+ * platform/graphics/chromium/LayerChromium.h: |
+ |
+2010-09-19 Antonio Gomes <agomes@rim.com> |
+ |
+ Reviewed by Ojan Vafai. |
+ |
+ SelectionController::modify should ask EditingBehavior for platform specific behavior |
+ https://bugs.webkit.org/show_bug.cgi?id=41975 |
+ |
+ As a follow up of the refactoring work in bug 39854, patch makes SelectionController::modify() |
+ stop accessing EditingBehaviorType values directly, and replaces its use by the EditingBehavior |
+ class. |
+ |
+ Since the "Settings*" parameter of the private SelectionController::modify() method becomes unneeded |
+ with this change (it was used to query the editingBehaviorType), patch merges two modify() methods. |
+ |
+ No behavior change, so no new tests. |
+ |
+ * editing/EditingBehavior.h: |
+ (WebCore::EditingBehavior::shouldAlwaysGrowSelectionWhenExtendingToBoundary): |
+ * editing/SelectionController.cpp: |
+ (WebCore::SelectionController::modify): |
+ * editing/SelectionController.h: |
+ |
+2010-09-20 Vangelis Kokkevis <vangelis@chromium.org> |
+ |
+ Reviewed by James Robinson. |
+ |
+ [chromium] Making destructors of LayerChromium and ContentLayerChromium virtual |
+ so that the derived class' destructor code gets called. |
+ https://bugs.webkit.org/show_bug.cgi?id=46139 |
+ |
+ * platform/graphics/chromium/ContentLayerChromium.h: |
+ * platform/graphics/chromium/LayerChromium.h: |
+ |
+2010-09-20 Darin Adler <darin@apple.com> |
+ |
+ Reviewed by James Robinson. |
+ |
+ Deprecate the inputType function on HTMLInputElement |
+ https://bugs.webkit.org/show_bug.cgi?id=46023 |
+ |
+ * html/HTMLInputElement.cpp: |
+ (WebCore::HTMLInputElement::HTMLInputElement): Use m_deprecatedTypeNumber. |
+ (WebCore::HTMLInputElement::isValidValue): Use deprecatedInputType. |
+ (WebCore::HTMLInputElement::typeMismatch): Ditto. |
+ (WebCore::HTMLInputElement::valueMissing): Ditto. |
+ (WebCore::HTMLInputElement::rangeUnderflow): Ditto. |
+ (WebCore::HTMLInputElement::rangeOverflow): Ditto. |
+ (WebCore::HTMLInputElement::minimum): Ditto. |
+ (WebCore::HTMLInputElement::maximum): Ditto. |
+ (WebCore::HTMLInputElement::stepBase): Ditto. |
+ (WebCore::HTMLInputElement::stepMismatch): Ditto. |
+ (WebCore::HTMLInputElement::getStepParameters): Ditto. |
+ (WebCore::HTMLInputElement::getAllowedValueStep): Ditto. |
+ (WebCore::HTMLInputElement::isKeyboardFocusable): Ditto. |
+ (WebCore::HTMLInputElement::shouldUseInputMethod): Ditto. |
+ (WebCore::HTMLInputElement::handleBlurEvent): Ditto. |
+ (WebCore::HTMLInputElement::setType): Added a comment. |
+ (WebCore::HTMLInputElement::updateType): Replaced the setInputType function |
+ with this. The type is always the type attribute value, so there is no value |
+ in passing the type string in, and this is an internal implementation detail, |
+ not a public function. |
+ (WebCore::createFormControlTypes): Use deprecatedNumberOfTypes. |
+ (WebCore::HTMLInputElement::formControlType): Use deprecatedInputType. |
+ (WebCore::HTMLInputElement::saveFormControlState): Ditto. |
+ (WebCore::HTMLInputElement::restoreFormControlState): Ditto. |
+ (WebCore::HTMLInputElement::accessKeyAction): Ditto. |
+ (WebCore::HTMLInputElement::mapToEntry): Ditto. |
+ (WebCore::HTMLInputElement::parseMappedAttribute): Ditto. |
+ (WebCore::HTMLInputElement::rendererIsNeeded): Ditto. |
+ (WebCore::HTMLInputElement::createRenderer): Ditto. |
+ (WebCore::HTMLInputElement::attach): Ditto. |
+ (WebCore::HTMLInputElement::isSuccessfulSubmitButton): Ditto. |
+ (WebCore::HTMLInputElement::appendFormData): Ditto. |
+ (WebCore::HTMLInputElement::isTextField): Ditto. |
+ (WebCore::HTMLInputElement::isTextType): Ditto. |
+ (WebCore::HTMLInputElement::setChecked): Ditto. |
+ (WebCore::HTMLInputElement::value): Ditto. |
+ (WebCore::HTMLInputElement::valueWithDefault): Ditto. |
+ (WebCore::HTMLInputElement::setSuggestedValue): Ditto. |
+ (WebCore::HTMLInputElement::setValue): Ditto. |
+ (WebCore::HTMLInputElement::parseToDouble): Ditto. |
+ (WebCore::HTMLInputElement::valueAsDate): Ditto. |
+ (WebCore::HTMLInputElement::setValueAsDate): Ditto. |
+ (WebCore::HTMLInputElement::valueAsNumber): Ditto. |
+ (WebCore::HTMLInputElement::setValueAsNumber): Ditto. |
+ (WebCore::HTMLInputElement::serializeForDateTimeTypes): Ditto. |
+ (WebCore::HTMLInputElement::serialize): Ditto. |
+ (WebCore::HTMLInputElement::setValueFromRenderer): Ditto. |
+ (WebCore::HTMLInputElement::storesValueSeparateFromAttribute): Ditto. |
+ (WebCore::HTMLInputElement::preDispatchEventHandler): Ditto. |
+ (WebCore::HTMLInputElement::postDispatchEventHandler): Ditto. |
+ (WebCore::HTMLInputElement::defaultEventHandler): Ditto. |
+ (WebCore::HTMLInputElement::handleBeforeTextInsertedEvent): Ditto. |
+ (WebCore::HTMLInputElement::files): Ditto. |
+ (WebCore::HTMLInputElement::isAcceptableValue): Ditto. |
+ (WebCore::HTMLInputElement::sanitizeValue): Ditto. |
+ (WebCore::HTMLInputElement::hasUnacceptableValue): Ditto. |
+ (WebCore::HTMLInputElement::needsActivationCallback): Ditto. |
+ (WebCore::HTMLInputElement::isRequiredFormControl): Ditto. |
+ (WebCore::HTMLInputElement::recalcWillValidate): Ditto. |
+ (WebCore::HTMLInputElement::parseToDateComponents): Ditto. |
+ (WebCore::HTMLInputElement::dataList): Ditto. |
+ (WebCore::HTMLInputElement::isSpeechEnabled): Ditto. |
+ |
+ * html/HTMLInputElement.h: Renamed InputType to DeprecatedInputType since it |
+ should not be used outside this class. Renamed numberOfTypes to |
+ dprecatedNumberOfTypes for the same reason. Changed all code to use |
+ deprecatedInputType function instead of m_type. Renamed inputType function |
+ to deprecatedInputType. Replaced public setInputType function with private |
+ updateType function. Renamed m_type data member to m_deprecatedTypeNumber. |
+ |
+ * rendering/MediaControlElements.cpp: |
+ (WebCore::MediaControlInputElement::MediaControlInputElement): |
+ * rendering/RenderFileUploadControl.cpp: |
+ (WebCore::RenderFileUploadControl::updateFromElement): |
+ Use setType instead of setInputType. |
+ |
+2010-09-20 Mihai Parparita <mihaip@chromium.org> |
+ |
+ Unreviewed; an attempt to fix Windows build. |
+ |
+ * WebCore.vcproj/WebCore.vcproj: |
+ |
+2010-09-20 Simon Fraser <simon.fraser@apple.com> |
+ |
+ Reviewed by Dan Bernstein. |
+ |
+ When PDF image is directly composited, it does not display |
+ https://bugs.webkit.org/show_bug.cgi?id=46144 |
+ |
+ In the "direct image compositing" code path, we set the image directly |
+ as the contents of the GraphicsLayer. However, this only works with |
+ bitmap images, so we have to check that the image is a bitmap image. |
+ |
+ Test: compositing/images/direct-pdf-image.html |
+ |
+ * rendering/RenderLayerBacking.cpp: |
+ (WebCore::RenderLayerBacking::isDirectlyCompositedImage): |
+ |
+2010-09-20 Simon Fraser <simon.fraser@apple.com> |
+ |
+ Reviewed by Darin Adler. |
+ |
+ Crash when div with content set to image gets composited (Vimeo). |
+ https://bugs.webkit.org/show_bug.cgi?id=46140 |
+ |
+ When CSS specifies that the content of an element is an image, we make |
+ a RenderImage, and setStyle() before setting the image resource. In this |
+ case the compositing code can attempt to access the cached image, which |
+ would crash with a null deref. |
+ |
+ Fix by null-checking m_imageResource. |
+ |
+ Test: compositing/images/content-image.html |
+ |
+ * rendering/RenderImage.h: |
+ (WebCore::RenderImage::cachedImage): |
+ |
+2010-09-20 Mihai Parparita <mihaip@chromium.org> |
+ |
+ Reviewed by Dimitri Glazkov. |
+ |
+ Hashchange event is no longer a simple event, needs to be its own interface |
+ https://bugs.webkit.org/show_bug.cgi?id=36335 |
+ |
+ Add HashChangeEvent.idl and supporting files/changes. |
+ |
+ Test: fast/loader/hashchange-event-properties.html |
+ |
+ The original version of this patch was written by |
+ Steven Lai <s3lance@hotmail.com>. |
+ |
+ * CMakeLists.txt: |
+ * DerivedSources.cpp: |
+ * DerivedSources.make: |
+ * GNUmakefile.am: |
+ * WebCore.gypi: |
+ * WebCore.pri: |
+ * WebCore.vcproj/WebCore.vcproj: |
+ * WebCore.xcodeproj/project.pbxproj: |
+ * bindings/js/JSEventCustom.cpp: |
+ (WebCore::toJS): |
+ * bindings/v8/custom/V8EventCustom.cpp: |
+ (WebCore::toV8): |
+ * dom/Document.cpp: |
+ (WebCore::Document::enqueueHashchangeEvent): |
+ * dom/Event.cpp: |
+ (WebCore::Event::isHashChangeEvent): |
+ * dom/Event.h: |
+ * dom/HashChangeEvent.h: Added. |
+ (WebCore::HashChangeEvent::isHashChangeEvent): |
+ (WebCore::HashChangeEvent::create): |
+ (WebCore::HashChangeEvent::initHashChangeEvent): |
+ (WebCore::HashChangeEvent::oldURL): |
+ (WebCore::HashChangeEvent::newURL): |
+ (WebCore::HashChangeEvent::HashChangeEvent): |
+ * dom/HashChangeEvent.idl: Added. |
+ * page/DOMWindow.idl: |
+ |
+2010-09-20 Andy Estes <aestes@apple.com> |
+ |
+ Reviewed by Adam Barth. |
+ |
+ REGRESSION (HTML5 Parser): Pages broken due to <tag<tag> parsing changes |
+ https://bugs.webkit.org/show_bug.cgi?id=40961 |
+ |
+ Add an optional quirk to the HTML5 tokenizer that reverts it to WebKit's |
+ legacy behavior when the start of a new token is encountered before the |
+ current token is finished parsing. The legacy behavior is to emit the |
+ current token as if it were properly closed and being parsing the new |
+ token. |
+ |
+ Test: fast/parser/pre-html5-parser-quirks.html |
+ |
+ * html/parser/HTMLDocumentParser.cpp: |
+ (WebCore::HTMLDocumentParser::HTMLDocumentParser): Instantiate |
+ m_tokenizer with the value of Settings::usePreHTML5ParserQuirks(). |
+ (WebCore::usePreHTMLParserQuirks): Add a helper function to return the |
+ value of Settings::usePreHTML5ParserQuirks() if Settings is non-NULL. |
+ * html/parser/HTMLPreloadScanner.cpp: |
+ (WebCore::HTMLPreloadScanner::HTMLPreloadScanner): Instantiate |
+ m_tokenizer with the value of Settings::usePreHTML5ParserQuirks(). |
+ * html/parser/HTMLTokenizer.cpp: |
+ (WebCore::HTMLTokenizer::HTMLTokenizer): |
+ (WebCore::HTMLTokenizer::nextToken): If the quirk is enabled and an |
+ unexpected '<' is encountered in certain states, emit the current token |
+ and reprocess the '<' as the start of a new token. |
+ * html/parser/HTMLTokenizer.h: |
+ (WebCore::HTMLTokenizer::create): |
+ * html/parser/HTMLViewSourceParser.cpp: |
+ (WebCore::HTMLViewSourceParser::HTMLViewSourceParser): Instantiate |
+ m_tokenizer with the value of Settings::usePreHTML5ParserQuirks(). |
+ * page/Settings.cpp: |
+ (WebCore::Settings::Settings): |
+ * page/Settings.h: |
+ (WebCore::Settings::setUsePreHTML5ParserQuirks): |
+ (WebCore::Settings::usePreHTML5ParserQuirks): |
+ |
+2010-09-20 David Hyatt <hyatt@apple.com> |
+ |
+ Reviewed by Dan Bernstein. |
+ |
+ https://bugs.webkit.org/show_bug.cgi?id=46132, add an isVerticalBlockFlow() method to RenderStyle and |
+ patch callers so that they don't have to check both top-to-bottom and bottom-to-top block flow. |
+ |
+ * rendering/RenderBox.cpp: |
+ (WebCore::RenderBox::availableLogicalWidth): |
+ * rendering/style/RenderStyle.cpp: |
+ (WebCore::RenderStyle::logicalWidth): |
+ (WebCore::RenderStyle::logicalHeight): |
+ (WebCore::RenderStyle::logicalMinWidth): |
+ (WebCore::RenderStyle::logicalMaxWidth): |
+ (WebCore::RenderStyle::logicalMinHeight): |
+ (WebCore::RenderStyle::logicalMaxHeight): |
+ (WebCore::RenderStyle::borderStartWidth): |
+ (WebCore::RenderStyle::borderEndWidth): |
+ (WebCore::RenderStyle::marginStart): |
+ (WebCore::RenderStyle::marginEnd): |
+ (WebCore::RenderStyle::paddingStart): |
+ (WebCore::RenderStyle::paddingEnd): |
+ * rendering/style/RenderStyle.h: |
+ (WebCore::InheritedFlags::isVerticalBlockFlow): |
+ |
+2010-09-20 David Hyatt <hyatt@apple.com> |
+ |
+ Reviewed by Dan Bernstein. |
+ |
+ https://bugs.webkit.org/show_bug.cgi?id=46124, add support for logical padding accessors to |
+ RenderBoxModelObject. |
+ |
+ Added fast/blockflow/percentage-padding.html even though it fails, so that when more of layout is patched |
+ we can see it start passing. |
+ |
+ * rendering/RenderBoxModelObject.cpp: |
+ (WebCore::RenderBoxModelObject::paddingTop): |
+ (WebCore::RenderBoxModelObject::paddingBottom): |
+ (WebCore::RenderBoxModelObject::paddingLeft): |
+ (WebCore::RenderBoxModelObject::paddingRight): |
+ (WebCore::RenderBoxModelObject::paddingBefore): |
+ (WebCore::RenderBoxModelObject::paddingAfter): |
+ (WebCore::RenderBoxModelObject::paddingStart): |
+ (WebCore::RenderBoxModelObject::paddingEnd): |
+ * rendering/RenderBoxModelObject.h: |
+ |
+2010-09-20 David Hyatt <hyatt@apple.com> |
+ |
+ Reviewed by Dan Bernstein. |
+ |
+ https://bugs.webkit.org/show_bug.cgi?id=46126, add availableLogicalWidth() to RenderBox. |
+ |
+ * rendering/RenderBlock.cpp: |
+ (WebCore::RenderBlock::availableWidth): |
+ (WebCore::RenderBlock::availableLogicalWidth): |
+ * rendering/RenderBlock.h: |
+ * rendering/RenderBox.cpp: |
+ (WebCore::RenderBox::availableLogicalWidth): |
+ * rendering/RenderBox.h: |
+ (WebCore::RenderBox::availableWidth): |
+ |
+2010-09-20 Kenneth Russell <kbr@google.com> |
+ |
+ Reviewed by James Robinson. |
+ |
+ [chromium] Change compositor to use GraphicsContext3D rather than GLES2Context |
+ https://bugs.webkit.org/show_bug.cgi?id=45912 |
+ |
+ Switched Chromium's compositor to use GraphicsContext3D to issue |
+ its OpenGL rendering calls rather than the Chromium-specific |
+ GLES2Context and command buffer OpenGL implementation. |
+ |
+ The in-process software rendering path for GraphicsContext3D does |
+ not yet work with the compositor, at least not on Mac OS X. This |
+ will be worked on in subsequent bugs. |
+ |
+ Tested manually with 3D CSS, WebGL and video content on Mac OS X |
+ and Linux. No new tests. |
+ |
+ * platform/graphics/chromium/Canvas2DLayerChromium.cpp: |
+ (WebCore::Canvas2DLayerChromium::~Canvas2DLayerChromium): |
+ (WebCore::Canvas2DLayerChromium::updateContents): |
+ * platform/graphics/chromium/CanvasLayerChromium.cpp: |
+ (WebCore::CanvasLayerChromium::SharedValues::SharedValues): |
+ (WebCore::CanvasLayerChromium::SharedValues::~SharedValues): |
+ (WebCore::CanvasLayerChromium::draw): |
+ * platform/graphics/chromium/CanvasLayerChromium.h: |
+ * platform/graphics/chromium/ContentLayerChromium.cpp: |
+ (WebCore::ContentLayerChromium::SharedValues::SharedValues): |
+ (WebCore::ContentLayerChromium::SharedValues::~SharedValues): |
+ (WebCore::ContentLayerChromium::~ContentLayerChromium): |
+ (WebCore::ContentLayerChromium::updateTextureRect): |
+ (WebCore::ContentLayerChromium::draw): |
+ * platform/graphics/chromium/ContentLayerChromium.h: |
+ * platform/graphics/chromium/DrawingBufferChromium.cpp: |
+ (WebCore::DrawingBuffer::publishToPlatformLayer): |
+ * platform/graphics/chromium/LayerChromium.cpp: |
+ (WebCore::loadShader): |
+ (WebCore::LayerChromium::SharedValues::SharedValues): |
+ (WebCore::LayerChromium::SharedValues::~SharedValues): |
+ (WebCore::LayerChromium::createShaderProgram): |
+ (WebCore::LayerChromium::layerRendererContext): |
+ (WebCore::LayerChromium::drawTexturedQuad): |
+ (WebCore::LayerChromium::drawDebugBorder): |
+ (WebCore::LayerChromium::drawAsMask): |
+ (WebCore::LayerChromium::prepareForDraw): |
+ * platform/graphics/chromium/LayerChromium.h: |
+ (WebCore::LayerChromium::SharedValues::context): |
+ * platform/graphics/chromium/LayerRendererChromium.cpp: |
+ (WebCore::LayerRendererChromium::create): |
+ (WebCore::LayerRendererChromium::LayerRendererChromium): |
+ (WebCore::LayerRendererChromium::context): |
+ (WebCore::LayerRendererChromium::debugGLCall): |
+ (WebCore::LayerRendererChromium::useShader): |
+ (WebCore::LayerRendererChromium::prepareToDrawLayers): |
+ (WebCore::LayerRendererChromium::updateRootLayerTextureRect): |
+ (WebCore::LayerRendererChromium::drawLayers): |
+ (WebCore::LayerRendererChromium::finish): |
+ (WebCore::LayerRendererChromium::present): |
+ (WebCore::LayerRendererChromium::getFramebufferPixels): |
+ (WebCore::LayerRendererChromium::createLayerTexture): |
+ (WebCore::LayerRendererChromium::drawLayerIntoStencilBuffer): |
+ (WebCore::LayerRendererChromium::drawLayersRecursive): |
+ (WebCore::LayerRendererChromium::drawLayer): |
+ (WebCore::LayerRendererChromium::scissorToRect): |
+ (WebCore::LayerRendererChromium::makeContextCurrent): |
+ (WebCore::LayerRendererChromium::resizeOnscreenContent): |
+ (WebCore::LayerRendererChromium::initializeSharedObjects): |
+ (WebCore::LayerRendererChromium::cleanupSharedObjects): |
+ * platform/graphics/chromium/LayerRendererChromium.h: |
+ * platform/graphics/chromium/VideoLayerChromium.cpp: |
+ (WebCore::VideoLayerChromium::createTextureRect): |
+ (WebCore::VideoLayerChromium::updateTextureRect): |
+ * platform/graphics/chromium/WebGLLayerChromium.cpp: |
+ (WebCore::WebGLLayerChromium::updateContents): |
+ * platform/graphics/gpu/SharedGraphicsContext3D.cpp: |
+ (WebCore::SharedGraphicsContext3D::flush): |
+ (WebCore::SharedGraphicsContext3D::supportsCopyTextureToParentTextureCHROMIUM): |
+ (WebCore::SharedGraphicsContext3D::copyTextureToParentTextureCHROMIUM): |
+ * platform/graphics/gpu/SharedGraphicsContext3D.h: |
+ |
+2010-09-20 David Hyatt <hyatt@apple.com> |
+ |
+ Reviewed by Dan Bernstein. |
+ |
+ https://bugs.webkit.org/show_bug.cgi?id=46119, add logical accessors to RenderStyle. Not used by any RenderObjects yet, so no tests |
+ until then. |
+ |
+ * rendering/style/RenderStyle.cpp: |
+ (WebCore::RenderStyle::logicalWidth): |
+ (WebCore::RenderStyle::logicalHeight): |
+ (WebCore::RenderStyle::logicalMinWidth): |
+ (WebCore::RenderStyle::logicalMaxWidth): |
+ (WebCore::RenderStyle::logicalMinHeight): |
+ (WebCore::RenderStyle::logicalMaxHeight): |
+ (WebCore::RenderStyle::borderBeforeWidth): |
+ (WebCore::RenderStyle::borderAfterWidth): |
+ (WebCore::RenderStyle::borderStartWidth): |
+ (WebCore::RenderStyle::borderEndWidth): |
+ (WebCore::RenderStyle::marginBefore): |
+ (WebCore::RenderStyle::marginAfter): |
+ (WebCore::RenderStyle::marginStart): |
+ (WebCore::RenderStyle::marginEnd): |
+ (WebCore::RenderStyle::paddingBefore): |
+ (WebCore::RenderStyle::paddingAfter): |
+ (WebCore::RenderStyle::paddingStart): |
+ (WebCore::RenderStyle::paddingEnd): |
+ * rendering/style/RenderStyle.h: |
+ |
+2010-09-20 David Hyatt <hyatt@apple.com> |
+ |
+ Reviewed by Dan Bernstein. |
+ |
+ https://bugs.webkit.org/show_bug.cgi?id=46116, implement block-flow-aware logical properties. |
+ |
+ This patch adds all the logical properties from the CSS3 Writing Mode draft for margins, padding, border, |
+ width and height. |
+ |
+ Added fast/css/logical-property-resolution.html |
+ |
+ * css/CSSComputedStyleDeclaration.cpp: |
+ (WebCore::CSSComputedStyleDeclaration::getPropertyCSSValue): |
+ * css/CSSParser.cpp: |
+ (WebCore::CSSParser::parseValue): |
+ * css/CSSProperty.cpp: |
+ (WebCore::resolveToPhysicalProperty): |
+ (WebCore::CSSProperty::resolveDirectionAwareProperty): |
+ * css/CSSProperty.h: |
+ * css/CSSPropertyNames.in: |
+ * css/CSSStyleSelector.cpp: |
+ (WebCore::CSSStyleSelector::applyDeclarations): |
+ (WebCore::CSSStyleSelector::applyProperty): |
+ |
+2010-09-20 David Hyatt <hyatt@apple.com> |
+ |
+ Reviewed by Beth Dakin. |
+ |
+ https://bugs.webkit.org/show_bug.cgi?id=46114, two tables repaint tests failing on bots. |
+ |
+ Patch layoutRows to apply the layout delta when setting a new cell location prior to laying out the cell. |
+ |
+ The layout delta is then removed after layout has happened when the repaint check is done. This change |
+ makes cell layout match block child layout, and now the cells have the correct new location up front |
+ when laying out (a necessity for pagination). |
+ |
+ * rendering/RenderTableSection.cpp: |
+ (WebCore::RenderTableSection::layoutRows): |
+ |
+2010-09-20 Jacob Dinu <dinu.jacob@nokia.com> |
+ |
+ Reviewed by Adam Barth. |
+ |
+ When loading a cached page, dispatchDidCommitLoad is called after FrameLoader::open so |
+ that all initialzations are done before calling client dispatchDidCommitLoad to avoid |
+ client from accessing incorrect data. |
+ https://bugs.webkit.org/show_bug.cgi?id=41155 |
+ |
+ * loader/FrameLoader.cpp: |
+ (WebCore::FrameLoader::commitProvisionalLoad): |
+ (WebCore::FrameLoader::transitionToCommitted): |
+ |
+2010-09-20 Simon Fraser <simon.fraser@apple.com> |
+ |
+ Reviewed by Adam Roben. |
+ |
+ REGRESSION: alternating animation direction doesn't work on Windows |
+ https://bugs.webkit.org/show_bug.cgi?id=46017 |
+ |
+ Animation-direction on Windows was broken because bitfields on |
+ Windows are signed. |
+ |
+ Fixed by making m_direction an unsigned in the bitfield, and |
+ casting in the accessor. |
+ |
+ Test: animations/animation-direction.html |
+ |
+ * platform/animation/Animation.h: |
+ (WebCore::Animation::direction): |
+ |
+2010-09-20 Enrica Casucci <enrica@apple.com> |
+ |
+ Reviewed by Sam Weinig. |
+ |
+ Pasteboard doesn't work in WebKit2. |
+ https://bugs.webkit.org/show_bug.cgi?id=42317 |
+ <rdar://problem/7660537> |
+ |
+ * loader/EmptyClients.h: |
+ (WebCore::EmptyEditorClient::documentFragmentFromAttributedString): Changed parameter to |
+ be a Vector of RefPtr. |
+ * page/EditorClient.h: |
+ * platform/mac/PasteboardMac.mm: |
+ (WebCore::Pasteboard::documentFragmentWithRtf): Stylistic changes. |
+ |
+2010-09-20 Yong Li <yoli@rim.com> |
+ |
+ Reviewed by Alexey Proskuryakov. |
+ |
+ https://bugs.webkit.org/show_bug.cgi?id=39966 |
+ Make compareBorders() a consistent compare function which can beused by qsort(). |
+ |
+ New test case added: LayoutTests/tables/sort-collapsed-border-styles.html |
+ |
+ * rendering/RenderTableCell.cpp: |
+ (WebCore::compareBorders): |
+ (WebCore::chooseBorder): |
+ (WebCore::RenderTableCell::collapsedLeftBorder): |
+ (WebCore::RenderTableCell::collapsedRightBorder): |
+ (WebCore::RenderTableCell::collapsedTopBorder): |
+ (WebCore::RenderTableCell::collapsedBottomBorder): |
+ (WebCore::compareBorderStylesForQSort): |
+ |
+2010-09-20 Patrick Gansterer <paroga@paroga.com> |
+ |
+ Reviewed by Adam Roben. |
+ |
+ [WINCE] Buildfix for PluginViewWin.cpp |
+ https://bugs.webkit.org/show_bug.cgi?id=46033 |
+ |
+ * plugins/win/PluginViewWin.cpp: |
+ (WebCore::PluginView::paintIntoTransformedContext): |
+ (WebCore::PluginView::snapshot): |
+ |
+2010-09-18 David Hyatt <hyatt@apple.com> |
+ |
+ Reviewed by Sam Weinig. |
+ |
+ https://bugs.webkit.org/show_bug.cgi?id=46030, aintitcool.com doesn't paginate correctly when printed. |
+ |
+ This happens because the site always has a document width that will exceed the page width. |
+ We incorrectly apply a double scale instead of clipping after the first scale still doesn't fit. |
+ The fix for the issue is to cap the right layout overflow to the page width and to just clip out |
+ any additional excess. This is the code in FrameView.cpp. |
+ |
+ This patch also cleans up table cell invalidation to reduce the # of relayouts. This change is not |
+ a correctness fix. It's just performance. |
+ |
+ I'm not sure how to write a test for this, since the double scale is an artifact of how WebKit mac calls |
+ back in when really printing. |
+ |
+ * page/FrameView.cpp: |
+ (WebCore::FrameView::forceLayoutForPagination): |
+ * rendering/RenderBlock.cpp: |
+ (WebCore::RenderBlock::layoutBlockChild): |
+ (WebCore::RenderBlock::layoutPositionedObjects): |
+ (WebCore::RenderBlock::positionNewFloats): |
+ * rendering/RenderBlock.h: |
+ (WebCore::RenderBlock::markForPaginationRelayout): |
+ * rendering/RenderFlexibleBox.cpp: |
+ (WebCore::RenderFlexibleBox::layoutHorizontalBox): |
+ (WebCore::RenderFlexibleBox::layoutVerticalBox): |
+ * rendering/RenderTableSection.cpp: |
+ (WebCore::RenderTableSection::layoutRows): |
+ |
+2010-09-20 Ilya Tikhonovsky <loislo@chromium.org> |
+ |
+ Reviewed by Pavel Feldman. |
+ |
+ Web Inspector: merge Inspector client runtime events into the serialized inspector state object. |
+ |
+ Drive-by rename getBackendSettings -> getInspectorState to better reflect the nature of the data. |
+ |
+ https://bugs.webkit.org/show_bug.cgi?id=45974 |
+ |
+ * inspector/Inspector.idl: |
+ * inspector/InspectorClient.h: |
+ (WebCore::InspectorClient::updateInspectorStateCookie): |
+ * inspector/InspectorController.cpp: |
+ (WebCore::InspectorController::saveApplicationSettings): |
+ (WebCore::InspectorController::getInspectorState): |
+ (WebCore::InspectorController::updateInspectorStateCookie): |
+ (WebCore::InspectorController::restoreInspectorStateFromCookie): |
+ (WebCore::InspectorController::getSettings): |
+ (WebCore::InspectorController::setMonitoringXHREnabled): |
+ (WebCore::InspectorController::restoreDebugger): |
+ (WebCore::InspectorController::restoreProfiler): |
+ (WebCore::InspectorController::setResourceTrackingEnabled): |
+ (WebCore::InspectorController::ensureSettingsLoaded): |
+ (WebCore::InspectorController::startTimelineProfiler): |
+ (WebCore::InspectorController::stopTimelineProfiler): |
+ (WebCore::InspectorController::enableProfiler): |
+ (WebCore::InspectorController::disableProfiler): |
+ (WebCore::InspectorController::enableDebuggerFromFrontend): |
+ (WebCore::InspectorController::disableDebugger): |
+ * inspector/InspectorController.h: |
+ * inspector/front-end/ConsoleView.js: |
+ (WebInspector.ConsoleView.prototype._handleContextMenuEvent): |
+ * inspector/front-end/ResourcesPanel.js: |
+ (WebInspector.ResourcesPanel.prototype._toggleResourceTracking): |
+ * inspector/front-end/Settings.js: |
+ (WebInspector.Settings.initialize): |
+ * inspector/front-end/inspector.js: |
+ (WebInspector.doLoadedDone.populateInspectorState): |
+ (WebInspector.doLoadedDone): |
+ |
+2010-09-20 Dirk Schulze <krit@webkit.org> |
+ |
+ Reviewed by Nikolas Zimmermann. |
+ |
+ SVG Filter cleanup |
+ https://bugs.webkit.org/show_bug.cgi?id=45612 |
+ |
+ All effect inputs are stored in a Vector in FilterEffect instead of passing them via constructors to every effect type. |
+ This simplifies the primitive subregion logic and centralizes it in determineFilterPrimitiveSubregion. |
+ Just SourceGraphic, SourceAlpha and FETile still calculate filter primitive subregions on |
+ their own. Working on it in a followup patch. |
+ The subregions code is SVG specific and moving this calculation to FilterEffect is an |
+ intermediate step before moving it to RenderSVGResourceFilterPrimitive in a following patch. |
+ The new FilterEffectVector will also make it possible to add code to determine the |
+ smallest used region of an effect and will help save memory and resources in the future. |
+ subRegion got renamed to filterPrimitiveSubregion to match the name scheme. scaledSubRegion got |
+ renamed to repaintRectInLocalCoordinates since this is its proper meaning. |
+ Removed unnecessary member variables and functions from FilterEffect. |
+ |
+ No new tests added since the functionality didn't change. |
+ |
+ * platform/graphics/cairo/GraphicsContextCairo.cpp: scaledSubRegion was renamed to repaintRectInLocalCoordinates. |
+ (WebCore::GraphicsContext::createShadowMask): |
+ * platform/graphics/filters/FEBlend.cpp: Removed Filter effect inputs from constructor. |
+ (WebCore::FEBlend::FEBlend): |
+ (WebCore::FEBlend::create): |
+ (WebCore::FEBlend::apply): Call inputFilterEffects to get input filter primitives. |
+ (WebCore::FEBlend::externalRepresentation): Call inputFilterEffects to get input filter primitives. |
+ * platform/graphics/filters/FEBlend.h: |
+ * platform/graphics/filters/FEColorMatrix.cpp: Ditto. |
+ (WebCore::FEColorMatrix::FEColorMatrix): |
+ (WebCore::FEColorMatrix::create): |
+ (WebCore::FEColorMatrix::apply): |
+ (WebCore::FEColorMatrix::externalRepresentation): |
+ * platform/graphics/filters/FEColorMatrix.h: |
+ * platform/graphics/filters/FEComponentTransfer.cpp: Ditto. |
+ (WebCore::FEComponentTransfer::FEComponentTransfer): |
+ (WebCore::FEComponentTransfer::create): |
+ (WebCore::FEComponentTransfer::apply): |
+ (WebCore::FEComponentTransfer::externalRepresentation): |
+ * platform/graphics/filters/FEComponentTransfer.h: |
+ * platform/graphics/filters/FEComposite.cpp: Ditto. |
+ (WebCore::FEComposite::FEComposite): |
+ (WebCore::FEComposite::create): |
+ (WebCore::FEComposite::apply): |
+ (WebCore::FEComposite::externalRepresentation): |
+ * platform/graphics/filters/FEComposite.h: |
+ * platform/graphics/filters/FEGaussianBlur.cpp: Ditto. |
+ (WebCore::FEGaussianBlur::FEGaussianBlur): |
+ (WebCore::FEGaussianBlur::create): |
+ (WebCore::FEGaussianBlur::apply): |
+ (WebCore::FEGaussianBlur::externalRepresentation): |
+ * platform/graphics/filters/FEGaussianBlur.h: |
+ * platform/graphics/filters/Filter.h: |
+ (WebCore::Filter::determineFilterPrimitiveSubregion): Renamed from calculateEffectSubRegion to match name scheme. |
+ * platform/graphics/filters/FilterEffect.cpp: |
+ (WebCore::FilterEffect::FilterEffect): |
+ (WebCore::FilterEffect::determineFilterPrimitiveSubregion): The main place for subregion calculation. |
+ (WebCore::FilterEffect::calculateDrawingIntRect): Takes repaintRectInLocalCoordinates now. |
+ (WebCore::FilterEffect::calculateDrawingRect): ditto. |
+ (WebCore::FilterEffect::getEffectContext): Check if ImageBuffer was created. |
+ * platform/graphics/filters/FilterEffect.h: Changed names to match name scheme. Removed unnecessary member variables and functions. |
+ Seperate SVG specific member variables and functions. Will get removed in followup patches. |
+ (WebCore::FilterEffect::resultImage): |
+ (WebCore::FilterEffect::setEffectBuffer): |
+ (WebCore::FilterEffect::inputEffect): |
+ (WebCore::FilterEffect::inputEffects): |
+ (WebCore::FilterEffect::numberOfinputEffects): |
+ (WebCore::FilterEffect::isAlphaImage): |
+ (WebCore::FilterEffect::setIsAlphaImage): |
+ (WebCore::FilterEffect::repaintRectInLocalCoordinates): |
+ (WebCore::FilterEffect::setRepaintRectInLocalCoordinates): |
+ (WebCore::FilterEffect::isSourceInput): |
+ (WebCore::FilterEffect::hasX): |
+ (WebCore::FilterEffect::setHasX): |
+ (WebCore::FilterEffect::hasY): |
+ (WebCore::FilterEffect::setHasY): |
+ (WebCore::FilterEffect::hasWidth): |
+ (WebCore::FilterEffect::setHasWidth): |
+ (WebCore::FilterEffect::hasHeight): |
+ (WebCore::FilterEffect::setHasHeight): |
+ (WebCore::FilterEffect::filterPrimitiveSubregion): |
+ (WebCore::FilterEffect::setFilterPrimitiveSubregion): |
+ (WebCore::FilterEffect::effectBoundaries): |
+ (WebCore::FilterEffect::setEffectBoundaries): |
+ * platform/graphics/filters/SourceAlpha.cpp: |
+ (WebCore::SourceAlpha::determineFilterPrimitiveSubregion): Renamed to match name scheme. |
+ * platform/graphics/filters/SourceAlpha.h: |
+ (WebCore::SourceAlpha::isSourceInput): |
+ * platform/graphics/filters/SourceGraphic.cpp: |
+ (WebCore::SourceGraphic::determineFilterPrimitiveSubregion): Ditto. |
+ * platform/graphics/filters/SourceGraphic.h: |
+ (WebCore::SourceGraphic::isSourceInput): |
+ * rendering/RenderSVGResourceFilter.cpp: Adapt to renames in FilterEffect. |
+ (WebCore::RenderSVGResourceFilter::applyResource): |
+ (WebCore::RenderSVGResourceFilter::postApplyResource): |
+ * svg/SVGFEBlendElement.cpp: Constructor of FEBlend doesn't take input effects. Adding them afterwards. |
+ (WebCore::SVGFEBlendElement::build): |
+ * svg/SVGFEColorMatrixElement.cpp: Ditto. |
+ (WebCore::SVGFEColorMatrixElement::build): |
+ * svg/SVGFEComponentTransferElement.cpp: Ditto. |
+ (WebCore::SVGFEComponentTransferElement::build): |
+ * svg/SVGFECompositeElement.cpp: Ditto. |
+ (WebCore::SVGFECompositeElement::build): |
+ * svg/SVGFEConvolveMatrixElement.cpp: Ditto. |
+ (WebCore::SVGFEConvolveMatrixElement::build): |
+ * svg/SVGFEDiffuseLightingElement.cpp: Ditto. |
+ (WebCore::SVGFEDiffuseLightingElement::build): |
+ * svg/SVGFEDisplacementMapElement.cpp: Ditto. |
+ (WebCore::SVGFEDisplacementMapElement::build): |
+ * svg/SVGFEGaussianBlurElement.cpp: Ditto. |
+ (WebCore::SVGFEGaussianBlurElement::build): |
+ * svg/SVGFEMergeElement.cpp: Ditto. |
+ (WebCore::SVGFEMergeElement::build): |
+ * svg/SVGFEMorphologyElement.cpp: Ditto. |
+ (WebCore::SVGFEMorphologyElement::build): |
+ * svg/SVGFEOffsetElement.cpp: Ditto. |
+ (WebCore::SVGFEOffsetElement::build): |
+ * svg/SVGFESpecularLightingElement.cpp: Ditto. |
+ (WebCore::SVGFESpecularLightingElement::build): |
+ * svg/SVGFETileElement.cpp: Ditto. |
+ (WebCore::SVGFETileElement::build): |
+ * svg/graphics/filters/SVGFEConvolveMatrix.cpp: Ditto. |
+ (WebCore::FEConvolveMatrix::FEConvolveMatrix): |
+ (WebCore::FEConvolveMatrix::create): |
+ (WebCore::FEConvolveMatrix::apply): |
+ (WebCore::FEConvolveMatrix::externalRepresentation): |
+ * svg/graphics/filters/SVGFEConvolveMatrix.h: |
+ * svg/graphics/filters/SVGFEDiffuseLighting.cpp: Ditto. |
+ (WebCore::FEDiffuseLighting::FEDiffuseLighting): |
+ (WebCore::FEDiffuseLighting::create): |
+ (WebCore::FEDiffuseLighting::externalRepresentation): |
+ * svg/graphics/filters/SVGFEDiffuseLighting.h: |
+ * svg/graphics/filters/SVGFEDisplacementMap.cpp: Ditto. |
+ (WebCore::FEDisplacementMap::FEDisplacementMap): |
+ (WebCore::FEDisplacementMap::create): |
+ (WebCore::FEDisplacementMap::apply): |
+ (WebCore::FEDisplacementMap::externalRepresentation): |
+ * svg/graphics/filters/SVGFEDisplacementMap.h: |
+ * svg/graphics/filters/SVGFEFlood.cpp: Ditto. |
+ (WebCore::FEFlood::apply): |
+ * svg/graphics/filters/SVGFEImage.cpp: Ditto. |
+ (WebCore::FEImage::apply): |
+ * svg/graphics/filters/SVGFELighting.cpp: Ditto. |
+ (WebCore::FELighting::FELighting): |
+ (WebCore::FELighting::apply): |
+ * svg/graphics/filters/SVGFELighting.h: |
+ * svg/graphics/filters/SVGFEMerge.cpp: Ditto. |
+ (WebCore::FEMerge::FEMerge): |
+ (WebCore::FEMerge::create): |
+ (WebCore::FEMerge::apply): |
+ (WebCore::FEMerge::externalRepresentation): |
+ * svg/graphics/filters/SVGFEMerge.h: |
+ * svg/graphics/filters/SVGFEMorphology.cpp: Ditto. |
+ (WebCore::FEMorphology::FEMorphology): |
+ (WebCore::FEMorphology::create): |
+ (WebCore::FEMorphology::apply): |
+ (WebCore::FEMorphology::externalRepresentation): |
+ * svg/graphics/filters/SVGFEMorphology.h: |
+ * svg/graphics/filters/SVGFEOffset.cpp: Ditto. |
+ (WebCore::FEOffset::FEOffset): |
+ (WebCore::FEOffset::create): |
+ (WebCore::FEOffset::apply): |
+ (WebCore::FEOffset::externalRepresentation): |
+ * svg/graphics/filters/SVGFEOffset.h: |
+ * svg/graphics/filters/SVGFESpecularLighting.cpp: Ditto. |
+ (WebCore::FESpecularLighting::FESpecularLighting): |
+ (WebCore::FESpecularLighting::create): |
+ (WebCore::FESpecularLighting::externalRepresentation): |
+ * svg/graphics/filters/SVGFESpecularLighting.h: |
+ * svg/graphics/filters/SVGFETile.cpp: Ditto. |
+ (WebCore::FETile::FETile): |
+ (WebCore::FETile::create): |
+ (WebCore::FETile::determineFilterPrimitiveSubregion): Renamed to match name scheme. |
+ (WebCore::FETile::apply): |
+ (WebCore::FETile::externalRepresentation): |
+ * svg/graphics/filters/SVGFETile.h: |
+ * svg/graphics/filters/SVGFilter.cpp: Renamed itemBox to targetBoundingBox to match name scheme. |
+ (WebCore::SVGFilter::SVGFilter): |
+ (WebCore::SVGFilter::determineFilterPrimitiveSubregion): Renamed to match name scheme. |
+ (WebCore::SVGFilter::create): |
+ * svg/graphics/filters/SVGFilter.h: |
+ (WebCore::SVGFilter::sourceImageRect): |
+ |
+2010-09-20 Pavel Podivilov <podivilov@chromium.org> |
+ |
+ Reviewed by Pavel Feldman. |
+ |
+ Web Inspector: highlight DOM node when hover on link element or DOM breakpoint |
+ https://bugs.webkit.org/show_bug.cgi?id=45897 |
+ |
+ * inspector/front-end/BreakpointsSidebarPane.js: |
+ (WebInspector.BreakpointItem): |
+ (WebInspector.JSBreakpointItem): |
+ (WebInspector.DOMBreakpointItem): |
+ (WebInspector.DOMBreakpointItem.prototype.compareTo): |
+ (WebInspector.DOMBreakpointItem.prototype._breakpointClicked): |
+ * inspector/front-end/ElementsPanel.js: |
+ (WebInspector.ElementsPanel.prototype.hide): |
+ (WebInspector.ElementsPanel.prototype.reset): |
+ (WebInspector.ElementsPanel.prototype.setDocument.selectNode): |
+ (WebInspector.ElementsPanel.prototype._mouseMovedInCrumbs): |
+ (WebInspector.ElementsPanel.prototype._mouseMovedOutOfCrumbs): |
+ (WebInspector.ElementsPanel.prototype.linkifyNodeReference): |
+ * inspector/front-end/ElementsTreeOutline.js: |
+ (WebInspector.ElementsTreeOutline.prototype.set focusedDOMNode): |
+ (WebInspector.ElementsTreeOutline.prototype._onmousemove): |
+ (WebInspector.ElementsTreeOutline.prototype._onmouseout): |
+ * inspector/front-end/inspector.js: |
+ (WebInspector.highlightDOMNode): |
+ (WebInspector.highlightDOMNodeForTwoSeconds): |
+ (WebInspector.wireElementWithDOMNode): |
+ (WebInspector._updateFocusedNode): |
+ (WebInspector.reset): |
+ (WebInspector.updateFocusedNode): |
+ |
+2010-09-19 Sheriff Bot <webkit.review.bot@gmail.com> |
+ |
+ Unreviewed, rolling out r67749. |
+ http://trac.webkit.org/changeset/67749 |
+ https://bugs.webkit.org/show_bug.cgi?id=46068 |
+ |
+ breaking ToT chromium canary build (Requested by shans on |
+ #webkit). |
+ |
+ * bindings/v8/V8DOMWindowShell.cpp: |
+ (WebCore::V8DOMWindowShell::createNewContext): |
+ * bindings/v8/V8Proxy.cpp: |
+ (WebCore::V8Proxy::registeredExtensionWithV8): |
+ (WebCore::V8Proxy::registerExtension): |
+ * bindings/v8/V8Proxy.h: |
+ * loader/EmptyClients.h: |
+ * loader/FrameLoaderClient.h: |
+ |
+2010-09-19 Gavin Barraclough <barraclough@apple.com> |
+ |
+ Reviewed by Oliver Hunt. |
+ |
+ Bug 46065 - Unify implementation of ToInt32 and ToUInt32, don't use fmod. |
+ |
+ These methods implement the same conversion (see discussion in the notes |
+ of sections of 9.5 and 9.6 of the spec), only differing in how the result |
+ is interpretted. |
+ |
+ Removing JSValue::toInt32 (since this has weird, non-spec function). |
+ A couple of places in the binding are using this method, so adding |
+ finiteInt32Value to the bindings to maintain current behaviour. |
+ |
+ Test: fast/js/toInt32UInt32.html |
+ |
+ * bindings/js/JSDOMBinding.h: |
+ (WebCore::finiteInt32Value): |
+ * bindings/js/JSHTMLOptionsCollectionCustom.cpp: |
+ (WebCore::JSHTMLOptionsCollection::add): |
+ * bindings/js/JSSQLResultSetRowListCustom.cpp: |
+ (WebCore::JSSQLResultSetRowList::item): |
+ * bindings/js/JSSVGPODListCustom.h: |
+ (WebCore::JSSVGPODListCustom::getItem): |
+ (WebCore::JSSVGPODListCustom::insertItemBefore): |
+ (WebCore::JSSVGPODListCustom::replaceItem): |
+ (WebCore::JSSVGPODListCustom::removeItem): |
+ * bindings/js/JSSVGPathSegListCustom.cpp: |
+ (WebCore::JSSVGPathSegList::getItem): |
+ (WebCore::JSSVGPathSegList::insertItemBefore): |
+ (WebCore::JSSVGPathSegList::replaceItem): |
+ (WebCore::JSSVGPathSegList::removeItem): |
+ |
+2010-09-19 Andreas Kling <andreas.kling@nokia.com> |
+ |
+ Reviewed by Kenneth Rohde Christiansen. |
+ |
+ [Qt] clipBoundingRect() should return rect relevant to current layer |
+ https://bugs.webkit.org/show_bug.cgi?id=46059 |
+ |
+ r65791 inadvertently changed clipBoundingRect() to always use the GC painter |
+ without checking the TransparencyLayer stack first. |
+ |
+ * platform/graphics/qt/GraphicsContextQt.cpp: |
+ (WebCore::GraphicsContextPlatformPrivate::p): |
+ (WebCore::GraphicsContextPlatformPrivate::clipBoundingRect): |
+ |
+2010-09-19 Sam Weinig <sam@webkit.org> |
+ |
+ Reviewed by Anders Carlsson. |
+ |
+ Replace WKBundleRangeRef with WKBundleRangeHandleRef. |
+ https://bugs.webkit.org/show_bug.cgi?id=46054 |
+ |
+ * WebCore.exp.in: |
+ * WebCore.xcodeproj/project.pbxproj: |
+ |
+2010-09-19 Patrick Gansterer <paroga@paroga.com> |
+ |
+ Unreviewed. |
+ |
+ [WINCE] Buildfix for Gradient after r67801. |
+ |
+ * platform/graphics/Gradient.h: |
+ * platform/graphics/wince/GradientWinCE.cpp: |
+ (WebCore::Gradient::getStops): |
+ |
+2010-09-19 Kent Tamura <tkent@chromium.org> |
+ |
+ Reviewed by Dimitri Glazkov. |
+ |
+ [Chromium] Add is<input type>() functions to WebInputElement |
+ https://bugs.webkit.org/show_bug.cgi?id=46035 |
+ |
+ * html/HTMLInputElement.h: |
+ (WebCore::HTMLInputElement::isText): Added. It is used by WebKit/chromium. |
+ |
+2010-09-18 Andreas Kling <andreas.kling@nokia.com> |
+ |
+ Reviewed by Antonio Gomes. |
+ |
+ [Qt] Path: Remove unused member variable |
+ https://bugs.webkit.org/show_bug.cgi?id=46048 |
+ |
+ Path::m_lastMoveToIndex is not used for anything, so remove it. |
+ |
+ * platform/graphics/Path.h: |
+ * platform/graphics/qt/PathQt.cpp: |
+ (WebCore::Path::Path): |
+ (WebCore::Path::operator=): |
+ (WebCore::Path::moveTo): |
+ |
+2010-09-18 Patrick Gansterer <paroga@paroga.com> |
+ |
+ Reviewed by Darin Adler. |
+ |
+ Use AtomicString insted of String in XMLDocumentParser |
+ https://bugs.webkit.org/show_bug.cgi?id=45990 |
+ |
+ Create AtomicString as early as possible. |
+ This can avoid unnecessary memcpy of String content. |
+ |
+ * dom/XMLDocumentParser.h: |
+ * dom/XMLDocumentParserLibxml2.cpp: |
+ (WebCore::toAtomicString): |
+ (WebCore::handleElementNamespaces): |
+ (WebCore::handleElementAttributes): |
+ (WebCore::XMLDocumentParser::startElementNs): |
+ |
+2010-09-18 Erik Arvidsson <arv@chromium.org> |
+ |
+ Reviewed by Nate Chapin. |
+ |
+ [Chromium] fast/dom/dataset-gc.html is failing |
+ https://bugs.webkit.org/show_bug.cgi?id=45426 |
+ |
+ * bindings/scripts/CodeGeneratorV8.pm: |
+ * bindings/v8/custom/V8DOMStringMapCustom.cpp: |
+ (WebCore::toV8): |
+ * dom/DOMStringMap.h: |
+ * dom/DatasetDOMStringMap.h: |
+ (WebCore::DatasetDOMStringMap::element): |
+ |
+2010-09-18 Andreas Kling <andreas.kling@nokia.com> |
+ |
+ Reviewed by Oliver Hunt. |
+ |
+ Gradient: Fast-path for the ideal case in sortStopsIfNecessary() |
+ https://bugs.webkit.org/show_bug.cgi?id=46045 |
+ |
+ Avoid calling std::stable_sort for 2-stop gradients that are already in order. |
+ |
+ * platform/graphics/Gradient.cpp: |
+ (WebCore::Gradient::sortStopsIfNecessary): |
+ |
+2010-09-18 Kwang Yul Seo <skyul@company100.net> |
+ |
+ Reviewed by Andreas Kling. |
+ |
+ [BREWMP] Fix a typo in key code name |
+ https://bugs.webkit.org/show_bug.cgi?id=46003 |
+ |
+ AVK_CLR is the right Brew MP key code name for VK_BACK. |
+ |
+ * platform/brew/PlatformKeyboardEventBrew.cpp: |
+ (WebCore::windowsKeyCodeForKeyEvent): |
+ |
+2010-09-18 Andreas Kling <andreas.kling@nokia.com> |
+ |
+ Reviewed by Oliver Hunt. |
+ |
+ Gradient: Pre-reserve space for 2 color stops |
+ https://bugs.webkit.org/show_bug.cgi?id=46044 |
+ |
+ Most gradients have only 2 color stops, pre-reserve space for these in m_stops |
+ to avoid growing the vector dynamically in the common case. |
+ |
+ * platform/graphics/Gradient.h: |
+ |
+2010-09-18 Dimitri Glazkov <dglazkov@chromium.org> |
+ |
+ Unreviewed, rolling out r67792. |
+ http://trac.webkit.org/changeset/67792 |
+ https://bugs.webkit.org/show_bug.cgi?id=45976 |
+ |
+ Broke over a hundred SVG tests. |
+ |
+ * platform/graphics/GraphicsContext.cpp: |
+ (WebCore::GraphicsContext::drawImage): |
+ (WebCore::GraphicsContext::drawImageBuffer): |
+ * platform/graphics/GraphicsContext.h: |
+ |
+2010-09-18 Mihai Parparita <mihaip@chromium.org> |
+ |
+ Reviewed by Simon Fraser. |
+ |
+ webkitAnimationEnd event doesn't fire when skipping over final iteration |
+ https://bugs.webkit.org/show_bug.cgi?id=46010 |
+ |
+ It's possible for AnimationBase::fireAnimationEventsIfNeeded to decide |
+ that the animation is ended while we're in the looping state, if the |
+ animation timer never fires during an iteration. That then fails the |
+ assert in AnimationBase::updateStateMachine (we don't expect |
+ AnimationStateInputEndTimerFired while in the looping state). Jump the |
+ state to AnimationStateEnding, which is more accurate. |
+ |
+ Test: animations/animation-end-event-short-iterations.html |
+ |
+ * page/animation/AnimationBase.cpp: |
+ (WebCore::AnimationBase::fireAnimationEventsIfNeeded): |
+ |
+2010-09-18 Kwang Yul Seo <skyul@company100.net> |
+ |
+ Reviewed by Andreas Kling. |
+ |
+ [BREWMP] Wrong key code type in keyIdentifierForBrewKeyCode |
+ https://bugs.webkit.org/show_bug.cgi?id=46005 |
+ |
+ The type of key code in Brew MP is uint16. Take uint16 instead of int16 in keyIdentifierForBrewKeyCode. |
+ |
+ * platform/brew/PlatformKeyboardEventBrew.cpp: |
+ (WebCore::keyIdentifierForBrewKeyCode): |
+ |
+2010-09-18 Matthew Delaney <mdelaney@apple.com> |
+ |
+ Reviewed by Oliver Hunt. |
+ |
+ Canvas drawImage with source or destination widths/heights of -1 renders incorrectly. |
+ https://bugs.webkit.org/show_bug.cgi?id=45976 |
+ |
+ Specifying a -1 width or height for either the source or destination rectangle in drawImage |
+ get improperly drawn with -1 replaced as the corresponding image width or height. This was because |
+ the value -1 was being passed around to signal non-specified widths and heights and thus was |
+ interpreted lower down in the drawImage implementation which disallowed any calls that actually |
+ wanted to use -1 as the widths or heights. Fixed this by moving the default logic up and not using |
+ -1 in-band signaling to denote default behavior. |
+ |
+ Tests: canvas/philip/tests/2d.drawImage.negativeOneDest.html |
+ canvas/philip/tests/2d.drawImage.negativeOneSource.html |
+ |
+ * platform/graphics/GraphicsContext.cpp: Removed -1 in-band signals and replaced cascased calls to have |
+ the desired replaces for unspecified widths and heights earlier on. |
+ * platform/graphics/GraphicsContext.h: Removed faulty default value. |
+ |
+2010-09-18 Ademar de Souza Reis Jr <ademar.reis@openbossa.org> |
+ |
+ Reviewed by Kenneth Rohde Christiansen. |
+ |
+ Enable Platform Strategies on Qt |
+ |
+ [Qt] Turn on PLATFORM_STRATEGIES |
+ https://bugs.webkit.org/show_bug.cgi?id=45831 |
+ |
+ * WebCore.pro: Adding/removing implementation files |
+ * platform/qt/Language.cpp: Added, WebCore::defaultLanguage() was |
+ moved from platform/qt/Localizations.cpp |
+ (WebCore::defaultLanguage): Moved from Localizations.cpp |
+ * platform/qt/Localizations.cpp: Removed (code is now inside |
+ WebCoreSupport/WebPlatformStrategies.cpp) |
+ * plugins/qt/PluginDataQt.cpp: Removed (idem) |
+ |
+2010-09-18 Patrick Gansterer <paroga@paroga.com> |
+ |
+ Reviewed by Darin Adler. |
+ |
+ Rename Wince files to WinCE |
+ https://bugs.webkit.org/show_bug.cgi?id=37287 |
+ |
+ * loader/icon/wince/IconDatabaseWinCE.cpp: Copied from WebCore/loader/icon/wince/IconDatabaseWince.cpp. |
+ * loader/icon/wince/IconDatabaseWince.cpp: Removed. |
+ * page/wince/FrameWinCE.cpp: Copied from WebCore/page/wince/FrameWince.cpp. |
+ * page/wince/FrameWince.cpp: Removed. |
+ * platform/graphics/MediaPlayer.cpp: |
+ * platform/graphics/wince/ColorWinCE.cpp: Copied from WebCore/platform/graphics/wince/ColorWince.cpp. |
+ * platform/graphics/wince/ColorWince.cpp: Removed. |
+ * platform/graphics/wince/FontCacheWinCE.cpp: Copied from WebCore/platform/graphics/wince/FontCacheWince.cpp. |
+ * platform/graphics/wince/FontCacheWince.cpp: Removed. |
+ * platform/graphics/wince/FontWinCE.cpp: Copied from WebCore/platform/graphics/wince/FontWince.cpp. |
+ * platform/graphics/wince/FontWince.cpp: Removed. |
+ * platform/graphics/wince/GlyphPageTreeNodeWinCE.cpp: Copied from WebCore/platform/graphics/wince/GlyphPageTreeNodeWince.cpp. |
+ * platform/graphics/wince/GlyphPageTreeNodeWince.cpp: Removed. |
+ * platform/graphics/wince/GradientWinCE.cpp: Copied from WebCore/platform/graphics/wince/GradientWince.cpp. |
+ * platform/graphics/wince/GradientWince.cpp: Removed. |
+ * platform/graphics/wince/GraphicsContextWinCE.cpp: Copied from WebCore/platform/graphics/wince/GraphicsContextWince.cpp. |
+ * platform/graphics/wince/GraphicsContextWince.cpp: Removed. |
+ * platform/graphics/wince/ImageBufferWinCE.cpp: Copied from WebCore/platform/graphics/wince/ImageBufferWince.cpp. |
+ * platform/graphics/wince/ImageBufferWince.cpp: Removed. |
+ * platform/graphics/wince/MediaPlayerPrivateWinCE.h: Copied from WebCore/platform/graphics/wince/MediaPlayerPrivateWince.h. |
+ * platform/graphics/wince/MediaPlayerPrivateWince.h: Removed. |
+ * platform/graphics/wince/PathWinCE.cpp: Copied from WebCore/platform/graphics/wince/PathWince.cpp. |
+ * platform/graphics/wince/PathWince.cpp: Removed. |
+ * platform/graphics/wince/PlatformPathWinCE.cpp: Copied from WebCore/platform/graphics/wince/PlatformPathWince.cpp. |
+ * platform/graphics/wince/PlatformPathWinCE.h: Copied from WebCore/platform/graphics/wince/PlatformPathWince.h. |
+ * platform/graphics/wince/PlatformPathWince.cpp: Removed. |
+ * platform/graphics/wince/PlatformPathWince.h: Removed. |
+ * platform/graphics/wince/SimpleFontDataWinCE.cpp: Copied from WebCore/platform/graphics/wince/SimpleFontDataWince.cpp. |
+ * platform/graphics/wince/SimpleFontDataWince.cpp: Removed. |
+ * platform/graphics/wince/WinCEGraphicsExtras.h: Copied from WebCore/platform/graphics/wince/WinceGraphicsExtras.h. |
+ * platform/graphics/wince/WinceGraphicsExtras.h: Removed. |
+ * platform/text/wince/TextBoundariesWinCE.cpp: Copied from WebCore/platform/text/wince/TextBoundariesWince.cpp. |
+ * platform/text/wince/TextBoundariesWince.cpp: Removed. |
+ * platform/text/wince/TextBreakIteratorWinCE.cpp: Copied from WebCore/platform/text/wince/TextBreakIteratorWince.cpp. |
+ * platform/text/wince/TextBreakIteratorWince.cpp: Removed. |
+ * platform/wince/CursorWinCE.cpp: Copied from WebCore/platform/wince/CursorWince.cpp. |
+ * platform/wince/CursorWince.cpp: Removed. |
+ * platform/wince/DragDataWinCE.cpp: Copied from WebCore/platform/wince/DragDataWince.cpp. |
+ * platform/wince/DragDataWince.cpp: Removed. |
+ * platform/wince/DragImageWinCE.cpp: Copied from WebCore/platform/wince/DragImageWince.cpp. |
+ * platform/wince/DragImageWince.cpp: Removed. |
+ * platform/wince/EditorWinCE.cpp: Copied from WebCore/platform/wince/EditorWince.cpp. |
+ (WebCore::Editor::newGeneralClipboard): |
+ * platform/wince/EditorWince.cpp: Removed. |
+ * platform/wince/FileChooserWinCE.cpp: Copied from WebCore/platform/wince/FileChooserWince.cpp. |
+ * platform/wince/FileChooserWince.cpp: Removed. |
+ * platform/wince/FileSystemWinCE.cpp: Copied from WebCore/platform/wince/FileSystemWince.cpp. |
+ * platform/wince/FileSystemWince.cpp: Removed. |
+ * platform/wince/KURLWinCE.cpp: Copied from WebCore/platform/wince/KURLWince.cpp. |
+ * platform/wince/KURLWince.cpp: Removed. |
+ * platform/wince/KeygenWinCE.cpp: Copied from WebCore/platform/wince/KeygenWince.cpp. |
+ * platform/wince/KeygenWince.cpp: Removed. |
+ * platform/wince/MIMETypeRegistryWinCE.cpp: Copied from WebCore/platform/wince/MIMETypeRegistryWince.cpp. |
+ * platform/wince/MIMETypeRegistryWince.cpp: Removed. |
+ * platform/wince/PasteboardWinCE.cpp: Copied from WebCore/platform/wince/PasteboardWince.cpp. |
+ * platform/wince/PasteboardWince.cpp: Removed. |
+ * platform/wince/SearchPopupMenuWinCE.cpp: Copied from WebCore/platform/wince/SearchPopupMenuWince.cpp. |
+ * platform/wince/SearchPopupMenuWince.cpp: Removed. |
+ * platform/wince/SharedTimerWinCE.cpp: Copied from WebCore/platform/wince/SharedTimerWince.cpp. |
+ * platform/wince/SharedTimerWince.cpp: Removed. |
+ * rendering/RenderThemeWinCE.cpp: Copied from WebCore/rendering/RenderThemeWince.cpp. |
+ (WebCore::RenderThemeWinCE::create): |
+ (WebCore::RenderTheme::themeForPage): |
+ (WebCore::RenderThemeWinCE::RenderThemeWinCE): |
+ (WebCore::RenderThemeWinCE::~RenderThemeWinCE): |
+ (WebCore::RenderThemeWinCE::platformActiveSelectionBackgroundColor): |
+ (WebCore::RenderThemeWinCE::platformInactiveSelectionBackgroundColor): |
+ (WebCore::RenderThemeWinCE::platformActiveSelectionForegroundColor): |
+ (WebCore::RenderThemeWinCE::platformInactiveSelectionForegroundColor): |
+ (WebCore::RenderThemeWinCE::supportsFocus): |
+ (WebCore::RenderThemeWinCE::supportsFocusRing): |
+ (WebCore::RenderThemeWinCE::determineClassicState): |
+ (WebCore::RenderThemeWinCE::getThemeData): |
+ (WebCore::RenderThemeWinCE::paintButton): |
+ (WebCore::RenderThemeWinCE::setCheckboxSize): |
+ (WebCore::RenderThemeWinCE::paintTextField): |
+ (WebCore::RenderThemeWinCE::adjustMenuListStyle): |
+ (WebCore::RenderThemeWinCE::paintMenuList): |
+ (WebCore::RenderThemeWinCE::paintMenuListButton): |
+ (WebCore::RenderThemeWinCE::systemFont): |
+ (WebCore::RenderThemeWinCE::themeChanged): |
+ (WebCore::RenderThemeWinCE::extraDefaultStyleSheet): |
+ (WebCore::RenderThemeWinCE::extraQuirksStyleSheet): |
+ (WebCore::RenderThemeWinCE::supportsHover): |
+ (WebCore::RenderThemeWinCE::systemColor): |
+ (WebCore::RenderThemeWinCE::adjustSliderThumbSize): |
+ (WebCore::RenderThemeWinCE::adjustButtonInnerStyle): |
+ (WebCore::RenderThemeWinCE::adjustSearchFieldStyle): |
+ (WebCore::RenderThemeWinCE::paintSearchField): |
+ (WebCore::RenderThemeWinCE::paintSearchFieldCancelButton): |
+ (WebCore::RenderThemeWinCE::adjustSearchFieldCancelButtonStyle): |
+ (WebCore::RenderThemeWinCE::adjustSearchFieldDecorationStyle): |
+ (WebCore::RenderThemeWinCE::adjustSearchFieldResultsDecorationStyle): |
+ (WebCore::RenderThemeWinCE::paintSearchFieldResultsDecoration): |
+ (WebCore::RenderThemeWinCE::adjustSearchFieldResultsButtonStyle): |
+ (WebCore::RenderThemeWinCE::paintSearchFieldResultsButton): |
+ (WebCore::RenderThemeWinCE::adjustMenuListButtonStyle): |
+ (WebCore::RenderThemeWinCE::paintSliderTrack): |
+ (WebCore::RenderThemeWinCE::paintSliderThumb): |
+ (WebCore::RenderThemeWinCE::paintMediaFullscreenButton): |
+ (WebCore::RenderThemeWinCE::paintMediaMuteButton): |
+ (WebCore::RenderThemeWinCE::paintMediaPlayButton): |
+ (WebCore::RenderThemeWinCE::paintMediaSeekBackButton): |
+ (WebCore::RenderThemeWinCE::paintMediaSeekForwardButton): |
+ (WebCore::RenderThemeWinCE::paintMediaSliderTrack): |
+ (WebCore::RenderThemeWinCE::paintMediaSliderThumb): |
+ * rendering/RenderThemeWinCE.h: Copied from WebCore/rendering/RenderThemeWince.h. |
+ * rendering/RenderThemeWince.cpp: Removed. |
+ * rendering/RenderThemeWince.h: Removed. |
+ * storage/wince/DatabaseThreadWinCE.cpp: Copied from WebCore/storage/wince/DatabaseThreadWince.cpp. |
+ * storage/wince/DatabaseThreadWinCE.h: Copied from WebCore/storage/wince/DatabaseThreadWince.h. |
+ * storage/wince/DatabaseThreadWince.cpp: Removed. |
+ * storage/wince/DatabaseThreadWince.h: Removed. |
+ * storage/wince/LocalStorageThreadWinCE.cpp: Copied from WebCore/storage/wince/LocalStorageThreadWince.cpp. |
+ * storage/wince/LocalStorageThreadWinCE.h: Copied from WebCore/storage/wince/LocalStorageThreadWince.h. |
+ * storage/wince/LocalStorageThreadWince.cpp: Removed. |
+ * storage/wince/LocalStorageThreadWince.h: Removed. |
+ |
+2010-09-18 Patrick Gansterer <paroga@paroga.com> |
+ |
+ Reviewed by Eric Seidel. |
+ |
+ Move Plugin*None.cpp from CMakeLists.txt into CMakeListsEfl.txt |
+ https://bugs.webkit.org/show_bug.cgi?id=45900 |
+ |
+ PluginDataNone.cpp and PluginViewNone.cpp are not used by all port. |
+ Move them into plaform sepcific build system. |
+ |
+ * CMakeLists.txt: |
+ * CMakeListsEfl.txt: |
+ |
+2010-09-18 Andreas Kling <andreas.kling@nokia.com> |
+ |
+ Reviewed by Simon Hausmann. |
+ |
+ [Qt] Don't copy clip path to TransparencyLayer |
+ https://bugs.webkit.org/show_bug.cgi?id=45965 |
+ |
+ We don't need to copy the GC's clip path to the TransparencyLayer since it will |
+ clip anyway when ending the layer. |
+ |
+ * platform/graphics/qt/TransparencyLayer.h: |
+ (WebCore::TransparencyLayer::TransparencyLayer): |
+ |
+2010-09-18 Patrick Gansterer <paroga@paroga.com> |
+ |
+ Unreviewed. |
+ |
+ [WINCE] Buildfix for FrameWince.cpp after r51353, r67762 and r67771. |
+ |
+ * page/wince/FrameWince.cpp: |
+ (WebCore::computePageRectsForFrame): |
+ (WebCore::imageFromSelection): |
+ |
+2010-09-18 Kent Tamura <tkent@chromium.org> |
+ |
+ Unreviewed, a trivial change. |
+ |
+ Run sort-Xcode-project-file. |
+ |
+ * WebCore.xcodeproj/project.pbxproj: |
+ |
+2010-09-18 Dan Bernstein <mitz@apple.com> |
+ |
+ Reviewed by Eric Seidel. |
+ |
+ REGRESSION (r67762): Over 160 layout tests failing due to incorrect zoom factors in subframes |
+ https://bugs.webkit.org/show_bug.cgi?id=46031 |
+ |
+ * page/Frame.cpp: |
+ (WebCore::parentPageZoomFactor): Return the zoom factor of the parent frame. |
+ (WebCore::parentTextZoomFactor): Ditto. |
+ |
+2010-09-17 Sam Weinig <sam@webkit.org> |
+ |
+ Reviewed by Jon Honeycutt. |
+ |
+ Need a way to load data (as plain text) in a WKPage |
+ <rdar://problem/8424239> |
+ |
+ * WebCore.exp.in: |
+ |
+2010-09-17 David Hyatt <hyatt@apple.com> |
+ |
+ Reviewed by Sam Weinig. |
+ |
+ https://bugs.webkit.org/show_bug.cgi?id=46027, image on gamespot.com paginates without bringing its enclosing border along. |
+ |
+ Make sure that unsplittable objects like replaced elements propagate their pagination strut outward to the parent block |
+ when appropriate. An image on a line did this, but a block-level image did not. |
+ |
+ Added fast/multicol/image-inside-nested-blocks-with-border.html |
+ |
+ * rendering/RenderBlock.cpp: |
+ (WebCore::RenderBlock::layoutBlockChild): |
+ |
+2010-09-17 Dimitri Glazkov <dglazkov@chromium.org> |
+ |
+ Unreviewed, build fix. |
+ |
+ Move calls from frameView->pageZoomFactor() to frame->pageZoomFactor(). |
+ |
+ * page/EventHandler.cpp: |
+ (WebCore::EventHandler::handleTouchEvent): Removed a helper function, |
+ added direct calls to frame->pageZoomFactor(). |
+ |
+2010-09-17 David Hyatt <hyatt@apple.com> |
+ |
+ Reviewed by Simon Fraser. |
+ |
+ https://bugs.webkit.org/show_bug.cgi?id=45993, convert printing to the new pagination model. |
+ |
+ Make printing store the page height in the RenderView and push that into the layout state to |
+ use the new pagination model. The old pagination model is retained because it is still used |
+ for embedded WebViews. |
+ |
+ * page/FrameView.cpp: |
+ (WebCore::FrameView::reset): |
+ (WebCore::FrameView::layout): |
+ (WebCore::FrameView::forceLayout): |
+ (WebCore::FrameView::forceLayoutForPagination): |
+ (WebCore::FrameView::adjustPageHeight): |
+ * page/FrameView.h: |
+ * page/PrintContext.cpp: |
+ (WebCore::PrintContext::computePageRectsWithPageSizeInternal): |
+ * rendering/RenderBlock.cpp: |
+ (WebCore::RenderBlock::paintChildren): |
+ * rendering/RenderBox.cpp: |
+ (WebCore::RenderBox::calcHeight): |
+ * rendering/RenderImage.cpp: |
+ (WebCore::RenderImage::paintReplaced): |
+ * rendering/RenderLineBoxList.cpp: |
+ (WebCore::RenderLineBoxList::paint): |
+ * rendering/RenderVideo.cpp: |
+ (WebCore::RenderVideo::paintReplaced): |
+ * rendering/RenderView.cpp: |
+ (WebCore::RenderView::RenderView): |
+ (WebCore::RenderView::layout): |
+ (WebCore::RenderView::paint): |
+ (WebCore::RenderView::setBestTruncatedAt): |
+ * rendering/RenderView.h: |
+ (WebCore::RenderView::popLayoutState): |
+ (WebCore::RenderView::pageHeight): |
+ (WebCore::RenderView::setPageHeight): |
+ (WebCore::RenderView::bestTruncatedAt): |
+ (WebCore::RenderView::truncatedAt): |
+ (WebCore::RenderView::setTruncatedAt): |
+ (WebCore::RenderView::printRect): |
+ (WebCore::RenderView::setPrintRect): |
+ (WebCore::RenderView::pushLayoutState): |
+ (WebCore::LayoutStateMaintainer::LayoutStateMaintainer): |
+ (WebCore::LayoutStateMaintainer::push): |
+ (WebCore::LayoutStateMaintainer::pop): |
+ |
+2010-09-17 Dan Bernstein <mitz@apple.com> |
+ |
+ Reviewed by Simon Fraser. |
+ |
+ Canvas sizing ignores intrinsic size |
+ https://bugs.webkit.org/show_bug.cgi?id=46024 |
+ |
+ Updated fast/replaced/table-percent-height.html |
+ |
+ * rendering/RenderEmbeddedObject.cpp: |
+ (WebCore::RenderEmbeddedObject::RenderEmbeddedObject): If this is used as a proxy |
+ for <video>, behave like it has an intrinsic size (e.g. preserve aspect ratio when |
+ width is auto and height is specified). |
+ * rendering/RenderImage.cpp: |
+ (WebCore::RenderImage::calcAspectRatioWidth): Changed to explicitly invoke |
+ RenderBox::calcReplacedHeight(), now that RenderReplaced has its own implementation |
+ which we don’t want. |
+ (WebCore::RenderImage::calcAspectRatioHeight): Similarly with calcReplacedWidth(). |
+ * rendering/RenderImage.h: |
+ * rendering/RenderReplaced.cpp: |
+ (WebCore::RenderReplaced::RenderReplaced): Initialize m_hasIntrinsicSize. |
+ (WebCore::lengthIsSpecified): Added this helper function, based on RenderImage’s |
+ is{Width,Height}Specified(). |
+ (WebCore::RenderReplaced::calcReplacedWidth): Moved from RenderVideo all the way |
+ up here so other replaced objects could use this logic. |
+ (WebCore::RenderReplaced::calcReplacedHeight): Ditto. |
+ (WebCore::RenderReplaced::calcAspectRatioWidth): Ditto. |
+ (WebCore::RenderReplaced::calcAspectRatioHeight): Ditto. |
+ (WebCore::RenderReplaced::calcPrefWidths): Replaced with the RenderImage version |
+ of the logic. |
+ (WebCore::RenderReplaced::setIntrinsicSize): Added an assertion. |
+ * rendering/RenderReplaced.h: |
+ (WebCore::RenderReplaced::minimumReplacedHeight): Promoted from private to protected |
+ to allow RenderVideo to call through. |
+ (WebCore::RenderReplaced::setHasIntrinsicSize): Added this setter. |
+ * rendering/RenderVideo.cpp: |
+ (WebCore::RenderVideo::updateIntrinsicSize): Factor zoom into the intrinsic size, |
+ the way other RenderReplaced objects do. |
+ (WebCore::RenderVideo::calcReplacedWidth): Invoke the RenderReplaced implementation, |
+ which matches what used to be here, skipping over the RenderImage implementation. |
+ (WebCore::RenderVideo::calcReplacedHeight): Ditto. |
+ (WebCore::RenderVideo::minimumReplacedHeight): Ditto. |
+ * rendering/RenderVideo.h: |
+ |
+2010-09-17 Darin Adler <darin@apple.com> |
+ |
+ Build fix for platforms that compile Touch. |
+ |
+ * dom/Touch.cpp: |
+ (WebCore::contentsX): |
+ (WebCore::contentsY): |
+ Get zoom factor from Frame rather than FrameView. |
+ |
+2010-09-17 Darin Adler <darin@apple.com> |
+ |
+ Reviewed by Sam Weinig. |
+ |
+ REGRESSION (r60104): Zoom level is unexpectedly reset on page reload |
+ https://bugs.webkit.org/show_bug.cgi?id=42863 |
+ |
+ Moved zoom level back from FrameView to Frame. |
+ I had forgotten that FrameView's lifetime is much shorter than |
+ Frame's, and until that is fixed it's best to leave this on Frame. |
+ |
+ * WebCore.exp.in: Updated. |
+ |
+ * css/CSSStyleSelector.cpp: |
+ (WebCore::CSSStyleSelector::styleForDocument): |
+ (WebCore::CSSStyleSelector::applyProperty): |
+ (WebCore::CSSStyleSelector::getComputedSizeFromSpecifiedSize): |
+ * dom/Document.cpp: |
+ (WebCore::Document::nodesFromRect): |
+ (WebCore::Document::elementFromPoint): |
+ (WebCore::Document::caretRangeFromPoint): |
+ * dom/MouseRelatedEvent.cpp: |
+ (WebCore::contentsX): |
+ (WebCore::contentsY): |
+ (WebCore::pageZoomFactor): |
+ * dom/Node.cpp: |
+ (WebCore::Node::dispatchMouseEvent): |
+ (WebCore::Node::dispatchWheelEvent): |
+ * html/HTMLBodyElement.cpp: |
+ (WebCore::adjustForZoom): |
+ (WebCore::HTMLBodyElement::scrollLeft): |
+ (WebCore::HTMLBodyElement::setScrollLeft): |
+ (WebCore::HTMLBodyElement::scrollTop): |
+ (WebCore::HTMLBodyElement::setScrollTop): |
+ (WebCore::HTMLBodyElement::scrollHeight): |
+ (WebCore::HTMLBodyElement::scrollWidth): |
+ * html/ImageDocument.cpp: |
+ (WebCore::pageZoomFactor): |
+ (WebCore::ImageDocument::scale): |
+ (WebCore::ImageDocument::imageFitsInWindow): |
+ * page/DOMWindow.cpp: |
+ (WebCore::DOMWindow::innerHeight): |
+ (WebCore::DOMWindow::innerWidth): |
+ (WebCore::DOMWindow::scrollX): |
+ (WebCore::DOMWindow::scrollY): |
+ (WebCore::DOMWindow::scrollTo): |
+ * page/DragController.cpp: |
+ (WebCore::elementUnderMouse): |
+ * rendering/RenderView.cpp: |
+ (WebCore::RenderView::zoomFactor): |
+ * svg/SVGSVGElement.cpp: |
+ (WebCore::SVGSVGElement::currentScale): |
+ (WebCore::SVGSVGElement::setCurrentScale): |
+ Get zoom factor from Frame rather than FrameView. |
+ |
+ * page/Frame.cpp: |
+ (WebCore::parentPageZoomFactor): Moved back here from FrameView. |
+ (WebCore::parentTextZoomFactor): Ditto. |
+ (WebCore::Frame::Frame): Ditto. |
+ (WebCore::Frame::setPageZoomFactor): Ditto. |
+ (WebCore::Frame::setTextZoomFactor): Ditto. |
+ (WebCore::Frame::setPageAndTextZoomFactors): Ditto. |
+ |
+ * page/Frame.h: Moved functions and data for zooming back here |
+ from FrameView. |
+ |
+ * page/FrameView.cpp: |
+ (WebCore::FrameView::FrameView): |
+ * page/FrameView.h: |
+ Removed code from here. |
+ |
+2010-09-17 Matthew Delaney <mdelaney@apple.com> |
+ |
+ Reviewed by Simon Fraser. |
+ |
+ Reduce minimum DOMTimer interval |
+ https://bugs.webkit.org/show_bug.cgi?id=45362 |
+ |
+ Reduced the minimum allowed DOMTimer interval to 4ms. This has us matching Chrome, |
+ which hasn't had any problem with 4ms in the past 2 years, as well as increasing our |
+ performance on perf tests that have tight frequent loops such as canvas "animation" |
+ performance tests. |
+ |
+ No new tests added. Can't reliably instrument a test in javascript to verify that |
+ the minimum clamped interval time is in fact 4ms. |
+ |
+ * WebCore.exp.in: |
+ * page/DOMTimer.cpp: Removed old comments. |
+ * page/DOMTimer.h: Removed old comments. |
+ * page/Settings.cpp: Exposted new method to set minimum DOMTimer interval. |
+ * page/Settings.h: |
+ |
+2010-09-17 Marc-Antoine Ruel <maruel@chromium.org> |
+ |
+ Reviewed by Dimitri Glazkov. |
+ |
+ [Chromium] Split webcore_platform off webcore_remaining |
+ |
+ Further reduce webcore_remaining size by moving platform/ code in its |
+ own static libary. |
+ https://bugs.webkit.org/show_bug.cgi?id=45915 |
+ |
+ * WebCore.gyp/WebCore.gyp: |
+ |
+2010-09-13 Matt Perry <mpcomplete@chromium.org> |
+ |
+ Reviewed by Darin Fisher. |
+ |
+ Have V8DOMWindowShell ask the embedder whether to run a V8 extension |
+ in a particular script context. |
+ https://bugs.webkit.org/show_bug.cgi?id=45721 |
+ |
+ * bindings/v8/V8DOMWindowShell.cpp: |
+ (WebCore::V8DOMWindowShell::createNewContext): |
+ * bindings/v8/V8Proxy.cpp: |
+ (WebCore::V8Proxy::registeredExtensionWithV8): |
+ (WebCore::V8Proxy::registerExtension): |
+ * bindings/v8/V8Proxy.h: |
+ * loader/EmptyClients.h: |
+ (WebCore::EmptyFrameLoaderClient::allowScriptExtension): |
+ * loader/FrameLoaderClient.h: |
+ |
+2010-09-17 Anders Carlsson <andersca@apple.com> |
+ |
+ Reviewed by Sam Weinig. |
+ |
+ REGRESSION (r66711): Plug-in replacement text is never shown |
+ https://bugs.webkit.org/show_bug.cgi?id=45997 |
+ <rdar://problem/8446766> |
+ |
+ Don't return when the plug-in has replacement text. |
+ |
+ * rendering/RenderEmbeddedObject.cpp: |
+ (WebCore::RenderEmbeddedObject::paintReplaced): |
+ |
+2010-09-17 Lucas De Marchi <lucas.demarchi@profusion.mobi> |
+ |
+ Unreviewed, build fix. Add files missed in r67704. |
+ |
+ * CMakeLists.txt: |
+ |
+2010-09-17 Simon Fraser <simon.fraser@apple.com> |
+ |
+ Reviewed by Chris Marrin. |
+ |
+ Remove scroll and clip layers for WKCACFLayerRenderer |
+ https://bugs.webkit.org/show_bug.cgi?id=45922 |
+ |
+ WKCACFLayerRenderer no longer needs its own layers for managing scrolling |
+ and clipping, because RenderLayerCompositor provides this functionality. |
+ |
+ * platform/graphics/win/MediaPlayerPrivateFullscreenWindow.cpp: |
+ (WebCore::MediaPlayerPrivateFullscreenWindow::setRootChildLayer): Remove call to setScrollFrame(). |
+ (WebCore::MediaPlayerPrivateFullscreenWindow::wndProc): Ditto. |
+ |
+ * platform/graphics/win/WKCACFLayerRenderer.cpp: Remove references to scroll and clip layers |
+ (WebCore::WKCACFLayerRenderer::WKCACFLayerRenderer): |
+ (WebCore::WKCACFLayerRenderer::setRootChildLayer): |
+ (WebCore::WKCACFLayerRenderer::destroyRenderer): |
+ (WebCore::WKCACFLayerRenderer::resize): |
+ * platform/graphics/win/WKCACFLayerRenderer.h: |
+ |
+ * rendering/RenderLayerCompositor.cpp: |
+ (WebCore::RenderLayerCompositor::layerTreeAsText): Dump the layers from the root platform layer. |
+ (WebCore::RenderLayerCompositor::requiresScrollLayer): On Windows, |
+ always say yes. |
+ |
+2010-09-17 Jian Li <jianli@chromium.org> |
+ |
+ Reviewed by Dmitry Titov. |
+ |
+ createBlobURL with no argument causes crash. |
+ https://bugs.webkit.org/show_bug.cgi?id=45880 |
+ |
+ The fix is to check if the passing blob argument is NULL or not. |
+ |
+ Test: fast/files/create-blob-url-crash.html |
+ |
+ * dom/ScriptExecutionContext.cpp: |
+ (WebCore::ScriptExecutionContext::createPublicBlobURL): |
+ * page/DOMWindow.idl: Add attribute to convert null string to undefined. |
+ |
+2010-09-17 Chris Marrin <cmarrin@apple.com> |
+ |
+ Reviewed by Simon Fraser. |
+ |
+ Add WebKitAccelerated2dCanvasEnabled flag to WebKit for Mac |
+ https://bugs.webkit.org/show_bug.cgi?id=45911 |
+ |
+ * WebCore.exp.in: |
+ |
+2010-09-17 Tony Gentilcore <tonyg@chromium.org> |
+ |
+ Reviewed by Adam Barth. |
+ |
+ ASSERTION FAILED: m_loadEventDelayCount |
+ https://bugs.webkit.org/show_bug.cgi?id=45790 |
+ |
+ * dom/AsyncScriptRunner.cpp: |
+ (WebCore::AsyncScriptRunner::timerFired): Protect document before we go running scripts. |
+ |
+2010-09-17 Stephen White <senorblanco@chromium.org> |
+ |
+ Reviewed by Kenneth Russell. |
+ |
+ [CHROMIUM] GPU-accelerated canvas should work in test_shell. |
+ https://bugs.webkit.org/show_bug.cgi?id=45968 |
+ |
+ In order for the shaders in GPU-accelerated canvas to work in both the |
+ GraphicsContext3D / GLES2 path as well as chromium's test_shell (which |
+ passes them to the Mesa backend unmodified), the precision specifiers |
+ in the fragment shader have to be wrapped in #if GL_ES. |
+ |
+ Soon to be covered by many layout tests. |
+ |
+ * platform/graphics/gpu/SolidFillShader.cpp: |
+ (WebCore::SolidFillShader::create): |
+ * platform/graphics/gpu/TexShader.cpp: |
+ (WebCore::TexShader::create): |
+ |
+2010-09-17 Martin Robinson <mrobinson@igalia.com> |
+ |
+ Reviewed by Xan Lopez. |
+ |
+ [GTK] [REGRESSION] Listbox rendering is incorrect |
+ https://bugs.webkit.org/show_bug.cgi?id=45941 |
+ |
+ Only apply the shadow tiling optimization to a layer if the GraphicsContext |
+ has a shadow. |
+ |
+ This is covered by manual-tests/select-element-type-select.html. |
+ |
+ * platform/graphics/cairo/GraphicsContextCairo.cpp: |
+ (WebCore::drawBorderlessRectShadow): Before applying the tiled shadow optimization |
+ first check to see if the context has a shadow at all. |
+ |
+2010-09-17 Martin Robinson <mrobinson@igalia.com> |
+ |
+ Reviewed by Oliver Hunt. |
+ |
+ [GTK] FontPlatformDataFreeType should use smart pointers to hold its members |
+ https://bugs.webkit.org/show_bug.cgi?id=45917 |
+ |
+ Have FontPlatformDataFreeType use smart pointers to hold its reference-counted |
+ members. Also move the FcPattern specialization from OwnPtrCairo to PlatformRefPtrCairo |
+ as this type is reference-counted. |
+ |
+ No new tests as this should not introduce any functionality changes. |
+ |
+ * platform/graphics/cairo/FontCacheFreeType.cpp: |
+ (WebCore::FontCache::getFontDataForCharacters): Updated to reflect smart pointer changes. |
+ (WebCore::FontCache::createFontPlatformData): Ditto. |
+ * platform/graphics/cairo/FontPlatformDataFreeType.cpp: |
+ (WebCore::FontPlatformData::FontPlatformData): Ditto. |
+ (WebCore::FontPlatformData::operator=): Ditto. |
+ (WebCore::FontPlatformData::~FontPlatformData): Ditto. |
+ (WebCore::FontPlatformData::isFixedPitch): Ditto. |
+ (WebCore::FontPlatformData::operator==): Ditto. |
+ * platform/graphics/cairo/FontPlatformDataFreeType.h: |
+ (WebCore::FontPlatformData::FontPlatformData): Ditto. |
+ (WebCore::FontPlatformData::scaledFont): Ditto. |
+ (WebCore::FontPlatformData::hash): Ditto. |
+ (WebCore::FontPlatformData::isHashTableDeletedValue): Ditto. |
+ * platform/graphics/cairo/GlyphPageTreeNodeCairo.cpp: |
+ (WebCore::GlyphPage::fill): Ditto. |
+ * platform/graphics/cairo/OwnPtrCairo.cpp: Removed FcPattern specialization. |
+ * platform/graphics/cairo/OwnPtrCairo.h: Ditto. |
+ * platform/graphics/cairo/PlatformRefPtrCairo.cpp: Added FcPattern and cairo_scaled_font_t specializations. |
+ (WTF::refPlatformPtr): Ditto. |
+ (WTF::derefPlatformPtr): Ditto. |
+ * platform/graphics/cairo/PlatformRefPtrCairo.h: Ditto. |
+ * platform/graphics/cairo/SimpleFontDataCairo.cpp: |
+ (WebCore::SimpleFontData::platformInit): Updated to reflect smart pointer changes. |
+ (WebCore::SimpleFontData::containsCharacters): Ditto. |
+ (WebCore::SimpleFontData::platformWidthForGlyph): Ditto. |
+ * platform/graphics/gtk/FontGtk.cpp: |
+ (WebCore::setPangoAttributes): Ditto. |
+ |
+2010-09-14 Pavel Podivilov <podivilov@chromium.org> |
+ |
+ Reviewed by Pavel Feldman. |
+ |
+ Web Inspector: show status message below call stack when debugger is paused on DOM breakpoint |
+ https://bugs.webkit.org/show_bug.cgi?id=45114 |
+ |
+ * English.lproj/localizedStrings.js: |
+ * inspector/InspectorDOMAgent.cpp: |
+ (WebCore::InspectorDOMAgent::performSearch): |
+ (WebCore::InspectorDOMAgent::shouldBreakOnNodeInsertion): |
+ (WebCore::InspectorDOMAgent::shouldBreakOnNodeRemoval): |
+ (WebCore::InspectorDOMAgent::shouldBreakOnAttributeModification): |
+ (WebCore::InspectorDOMAgent::descriptionForDOMEvent): |
+ * inspector/InspectorDOMAgent.h: |
+ * inspector/InspectorDebuggerAgent.cpp: |
+ (WebCore::InspectorDebuggerAgent::didPause): |
+ * inspector/front-end/CallStackSidebarPane.js: |
+ (WebInspector.CallStackSidebarPane): |
+ (WebInspector.CallStackSidebarPane.prototype.updateStatus.formatters.s): |
+ (WebInspector.CallStackSidebarPane.prototype.updateStatus.append): |
+ (WebInspector.CallStackSidebarPane.prototype.updateStatus): |
+ * inspector/front-end/ElementsPanel.js: |
+ (WebInspector.ElementsPanel.prototype.linkifyNodeById): |
+ * inspector/front-end/ScriptsPanel.js: |
+ (WebInspector.ScriptsPanel.prototype.debuggerPaused): |
+ * inspector/front-end/inspector.css: |
+ (.pane > .body .placard + .info): |
+ * inspector/front-end/inspector.js: |
+ (WebInspector.pausedScript): |
+ (WebInspector.formatLocalized): |
+ |
+2010-09-17 Johnny Ding <jnd@chromium.org> |
+ |
+ Reviewed by Adam Barth. |
+ |
+ Stop history reload navigation to bypass WebKit's popup blocker. |
+ Now history reload can only navigate the page in self frame, no matter |
+ what target frame is defined in <base> and no new window can be created. |
+ https://bugs.webkit.org/show_bug.cgi?id=45369 |
+ |
+ Test: fast/events/popup-blocked-from-history-reload.html |
+ |
+ * loader/RedirectScheduler.cpp: |
+ (WebCore::ScheduledNavigation::ScheduledNavigation): |
+ (WebCore::ScheduledNavigation::wasUserGesture): |
+ Move the m_wasUserGesture to base class ScheduledNavigation. Then all |
+ asynchronous navigation situations can restore the correct gesture state |
+ during the real navigation process. |
+ (WebCore::ScheduledURLNavigation::ScheduledURLNavigation): |
+ (WebCore::ScheduledURLNavigation::fire): |
+ (WebCore::ScheduledURLNavigation::referrer): |
+ (WebCore::ScheduledHistoryNavigation::ScheduledHistoryNavigation): |
+ (WebCore::ScheduledHistoryNavigation::fire): |
+ (WebCore::ScheduledFormSubmission::ScheduledFormSubmission): |
+ (WebCore::ScheduledFormSubmission::fire): |
+ (WebCore::RedirectScheduler::scheduleHistoryNavigation): |
+ |
+2010-09-16 Vangelis Kokkevis <vangelis@chromium.org> |
+ |
+ Reviewed by James Robinson. |
+ |
+ [chromium] Clip the update region of an ImageLayerChromium to actual size |
+ of the image bitmap to prevent uninitialized memory access when uploading |
+ the dirty region to the texture. |
+ https://bugs.webkit.org/show_bug.cgi?id=45937 |
+ |
+ |
+ * platform/graphics/chromium/ImageLayerChromium.cpp: |
+ (WebCore::ImageLayerChromium::updateContents): |
+ |
+2010-09-17 Dimitri Glazkov <dglazkov@chromium.org> |
+ |
+ Reviewed by David Hyatt. |
+ |
+ Remove extraneous RenderObject traversal. |
+ https://bugs.webkit.org/show_bug.cgi?id=45821 |
+ |
+ This code always exits in the first iteration of the loop, |
+ because in the context of this call, node->renderer()->node() != 0 |
+ is always true. |
+ |
+ No behavior change, covered by existing tests. |
+ |
+ * page/EventHandler.cpp: |
+ (WebCore::EventHandler::canMouseDownStartSelect): Removed the loop. |
+ (WebCore::EventHandler::canMouseDragExtendSelect): Ditto. |
+ |
+2010-09-17 Pavel Feldman <pfeldman@chromium.org> |
+ |
+ Reviewed by Yury Semikhatsky. |
+ |
+ Web Inspector: FrameLoader::loadedResourceFromMemoryCache reports |
+ resource as not cached to InspectorController. |
+ https://bugs.webkit.org/show_bug.cgi?id=45961 |
+ |
+ Web Inspector: FrameLoader::loadedResourceFromMemoryCache reports |
+ resource as not cached to InspectorController. |
+ There is one path in control flow that leads to resources being |
+ reported as not cached. Patch to follow. |
+ |
+ * inspector/InspectorController.cpp: |
+ (WebCore::InspectorController::markResourceAsCached): |
+ * inspector/InspectorController.h: |
+ * inspector/InspectorResource.cpp: |
+ (WebCore::InspectorResource::markAsCached): |
+ (WebCore::InspectorResource::updateResponse): |
+ * inspector/InspectorResource.h: |
+ * loader/FrameLoader.cpp: |
+ (WebCore::FrameLoader::loadedResourceFromMemoryCache): |
+ |
+2010-09-17 Alexander Pavlov <apavlov@chromium.org> |
+ |
+ Unreviewed, build fix. Add files missed in the previous commit. |
+ |
+ * GNUmakefile.am: |
+ * css/CSSPropertySourceData.cpp: Added. |
+ (WebCore::SourceRange::SourceRange): |
+ (WebCore::SourceRange::operator=): |
+ (WebCore::CSSPropertySourceData::CSSPropertySourceData): |
+ (WebCore::CSSPropertySourceData::operator=): |
+ (WebCore::CSSPropertySourceData::toString): |
+ (WebCore::CSSPropertySourceData::hash): |
+ (WebCore::CSSPropertySourceData::init): |
+ * css/CSSPropertySourceData.h: Added. |
+ (WebCore::CSSPropertySourceData::~CSSPropertySourceData): |
+ (WebCore::CSSStyleSourceData::create): |
+ |
+2010-09-15 Alexander Pavlov <apavlov@chromium.org> |
+ |
+ Reviewed by Pavel Feldman. |
+ |
+ Provide source-based properties for style declarations to CSSParser clients |
+ |
+ This change: |
+ - Enables InspectorCSSStore to extract CSS stylesheet text for all source-based stylesheets, |
+ even with the resource tracking turned off. |
+ - Adds to CSSParser a capability of source-related style declaration data extraction: |
+ * Start/end source offsets for a declaration, relative to the parent stylesheet start or the "style" attribute value. |
+ * Start/end source offsets for CSS properties, relative to the declaration start. |
+ * CSS property data: name, value, priority, WebCore parsability (i.e. if the property is understood by WebCore). |
+ - Provides the extracted data to the WebInspector frontend via InspectorDOMAgent::getStyleSourceData(). |
+ https://bugs.webkit.org/show_bug.cgi?id=44949 |
+ |
+ * WebCore.gypi: |
+ * WebCore.pro: |
+ * WebCore.vcproj/WebCore.vcproj: |
+ * WebCore.xcodeproj/project.pbxproj: |
+ * css/CSSGrammar.y: |
+ * css/CSSMutableStyleDeclaration.h: |
+ (WebCore::CSSMutableStyleDeclaration::node): |
+ * css/CSSParser.cpp: |
+ (WebCore::CSSParser::CSSParser): |
+ (WebCore::CSSParser::parseSheet): |
+ (WebCore::CSSParser::parseDeclaration): |
+ (WebCore::CSSParser::createStyleRule): |
+ (WebCore::CSSParser::markRuleBodyStart): |
+ (WebCore::CSSParser::markPropertyStart): |
+ (WebCore::CSSParser::markPropertyEnd): |
+ * css/CSSParser.h: |
+ (WebCore::CSSParser::resetPropertyMarks): |
+ * css/CSSPropertySourceData.cpp: Added. |
+ (WebCore::SourceRange::SourceRange): |
+ (WebCore::SourceRange::operator=): |
+ (WebCore::CSSPropertySourceData::CSSPropertySourceData): |
+ (WebCore::CSSPropertySourceData::operator=): |
+ (WebCore::CSSPropertySourceData::toString): |
+ (WebCore::CSSPropertySourceData::hash): |
+ (WebCore::CSSPropertySourceData::init): |
+ * css/CSSPropertySourceData.h: Added. |
+ (WebCore::CSSPropertySourceData::~CSSPropertySourceData): |
+ (WebCore::CSSStyleSourceData::create): |
+ * inspector/Inspector.idl: |
+ * inspector/InspectorCSSStore.cpp: |
+ (WebCore::InspectorCSSStore::reset): |
+ (WebCore::InspectorCSSStore::styleSheetText): |
+ (WebCore::InspectorCSSStore::resourceStyleSheetText): |
+ (WebCore::InspectorCSSStore::inlineStyleSheetText): |
+ (WebCore::InspectorCSSStore::getStyleSourceData): |
+ (WebCore::InspectorCSSStore::extractRanges): |
+ (WebCore::InspectorCSSStore::getStyleAttributeRanges): |
+ (WebCore::InspectorCSSStore::getParentStyleSheet): |
+ (WebCore::InspectorCSSStore::inlineStyleElement): |
+ * inspector/InspectorCSSStore.h: |
+ * inspector/InspectorController.cpp: |
+ (WebCore::InspectorController::resourceContentForURL): |
+ * inspector/InspectorController.h: |
+ * inspector/InspectorDOMAgent.cpp: |
+ (WebCore::InspectorDOMAgent::getStyleSourceData): |
+ (WebCore::InspectorDOMAgent::buildObjectForStyle): |
+ * inspector/InspectorDOMAgent.h: |
+ |
+2010-09-17 Alexander Pavlov <apavlov@chromium.org> |
+ |
+ Reviewed by Pavel Feldman. |
+ |
+ Web Inspector: Show node description in inspector highlight |
+ https://bugs.webkit.org/show_bug.cgi?id=20930 |
+ |
+ * inspector/InspectorController.cpp: |
+ (WebCore::InspectorController::drawNodeHighlight): |
+ (WebCore::InspectorController::drawElementTitle): |
+ * inspector/InspectorController.h: |
+ |
+2010-09-16 takano takumi <takano1@asia.apple.com> |
+ |
+ Reviewed by Kent Tamura. |
+ |
+ https://bugs.webkit.org/show_bug.cgi?id=45020 |
+ Support "block-flow" and "writing-mode": interpret properties into RenderStyle |
+ |
+ Fixed a regression in SVG test introduced by my previous change. |
+ |
+ * css/CSSPrimitiveValueMappings.h: |
+ (WebCore::CSSPrimitiveValue::operator EWritingMode): |
+ * css/SVGCSSParser.cpp: |
+ (WebCore::CSSParser::parseSVGValue): |
+ |
+2010-09-16 Kenneth Russell <kbr@google.com> |
+ |
+ Reviewed by James Robinson. |
+ |
+ Add entry points to GraphicsContext3D needed for Chromium compositor port |
+ https://bugs.webkit.org/show_bug.cgi?id=45939 |
+ |
+ Added entry points for two Chromium-specific extensions, and added |
+ a flag to the GraphicsContext3D constructor, currently unsupported |
+ by all ports (including Chromium), indicating whether the context |
+ should render directly to the passed HostWindow or off-screen per |
+ the current semantics. The switch to use GraphicsContext3D in |
+ Chromium's compositor will follow in a subsequent patch. |
+ |
+ No new tests; functionality is unchanged. Built and tested |
+ Chromium and WebKit on Mac OS X. |
+ |
+ * platform/graphics/GraphicsContext3D.cpp: |
+ (WebCore::GraphicsContext3D::supportsMapSubCHROMIUM): |
+ (WebCore::GraphicsContext3D::mapBufferSubDataCHROMIUM): |
+ (WebCore::GraphicsContext3D::unmapBufferSubDataCHROMIUM): |
+ (WebCore::GraphicsContext3D::mapTexSubImage2DCHROMIUM): |
+ (WebCore::GraphicsContext3D::unmapTexSubImage2DCHROMIUM): |
+ (WebCore::GraphicsContext3D::supportsCopyTextureToParentTextureCHROMIUM): |
+ (WebCore::GraphicsContext3D::copyTextureToParentTextureCHROMIUM): |
+ * platform/graphics/GraphicsContext3D.h: |
+ * platform/graphics/mac/GraphicsContext3DMac.mm: |
+ (WebCore::GraphicsContext3D::create): |
+ (WebCore::GraphicsContext3D::GraphicsContext3D): |
+ * platform/graphics/qt/GraphicsContext3DQt.cpp: |
+ (WebCore::GraphicsContext3D::create): |
+ (WebCore::GraphicsContext3D::GraphicsContext3D): |
+ |
+2010-09-16 Simon Fraser <simon.fraser@apple.com> |
+ |
+ Reviewed by James Robinson. |
+ |
+ In layerTreeAsText output, avoid printing properties which have their default values |
+ https://bugs.webkit.org/show_bug.cgi?id=45931 |
+ |
+ Don't dump properties that have their default values to simplify output. |
+ |
+ * platform/graphics/GraphicsLayer.cpp: |
+ (WebCore::GraphicsLayer::dumpProperties): |
+ |
+2010-09-16 Andreas Kling <andreas.kling@nokia.com> |
+ |
+ Reviewed by Ariya Hidayat. |
+ |
+ [Qt] GraphicsContext: Remove unused variable |
+ |
+ GraphicsContextPlatformPrivate::redirect wasn't used for anything. |
+ |
+ * platform/graphics/qt/GraphicsContextQt.cpp: |
+ (WebCore::GraphicsContextPlatformPrivate::p): |
+ (WebCore::GraphicsContextPlatformPrivate::GraphicsContextPlatformPrivate): |
+ |
+2010-09-16 Andreas Kling <andreas.kling@nokia.com> |
+ |
+ Reviewed by Ariya Hidayat. |
+ |
+ [Qt] Remove unnecessary QPainter::clipPath() call in TransparencyLayer() |
+ https://bugs.webkit.org/show_bug.cgi?id=45923 |
+ |
+ QPainter::clipPath() can be very expensive, so use QPainter::hasClipping() |
+ instead of QPainter::clipPath().isEmpty(). |
+ |
+ * platform/graphics/qt/TransparencyLayer.h: |
+ (WebCore::TransparencyLayer::TransparencyLayer): |
+ |
+2010-09-16 David Hyatt <hyatt@apple.com> |
+ |
+ Reviewed by Eric Seidel. |
+ |
+ Fix for regression from my previous check-in. Make sure not to move the LayoutRepainter line |
+ below the setHeight(0) call. This meant it snagged the wrong rectangle and we ended up repainting |
+ the world. |
+ |
+ * rendering/RenderBlock.cpp: |
+ (WebCore::RenderBlock::layoutBlock): |
+ |
+2010-09-16 takano takumi <takano1@asia.apple.com> |
+ |
+ Reviewed by Dave Hyatt. |
+ |
+ https://bugs.webkit.org/show_bug.cgi?id=45020 |
+ Support "block-flow" and "writing-mode": interpret properties into RenderStyle |
+ |
+ Added code that interprets "block-flow" property and "writing-mode" property settings |
+ into a RenderStyle object. |
+ |
+ Test: fast/text/international/block-flow-parser-test.html |
+ |
+ * css/CSSComputedStyleDeclaration.cpp: |
+ (WebCore::CSSComputedStyleDeclaration::getPropertyCSSValue): |
+ * css/CSSParser.cpp: |
+ (WebCore::CSSParser::parseValue): |
+ * css/CSSPrimitiveValueMappings.h: |
+ (WebCore::CSSPrimitiveValue::CSSPrimitiveValue): |
+ (WebCore::CSSPrimitiveValue::operator EBlockFlowDirection): |
+ * css/CSSPropertyNames.in: |
+ * css/CSSStyleSelector.cpp: |
+ (WebCore::CSSStyleSelector::applyProperty): |
+ * css/CSSValueKeywords.in: |
+ * css/SVGCSSValueKeywords.in: |
+ * rendering/style/RenderStyle.cpp: |
+ (WebCore::RenderStyle::diff): |
+ * rendering/style/RenderStyle.h: |
+ (WebCore::InheritedFlags::setBitDefaults): |
+ (WebCore::InheritedFlags::blockFlow): |
+ (WebCore::InheritedFlags::setBlockFlow): |
+ (WebCore::InheritedFlags::initialBlockFlow): |
+ * rendering/style/RenderStyleConstants.h: |
+ |
+2010-09-16 Martin Robinson <mrobinson@igalia.com> |
+ |
+ Reviewed by Xan Lopez. |
+ |
+ [GTK] Implement dissolveDragImageToFraction |
+ https://bugs.webkit.org/show_bug.cgi?id=45826 |
+ |
+ Convert the drag image from a GdkPixbuf to a Cairo surface. Not only |
+ might this allow code to be shared between Cairo ports it prevents a |
+ full data copy when the image is not resized and makes the implementation |
+ of dissolveDragImageToFraction more straight-forward. |
+ |
+ This feature is covered by manual-tests/drag-image.html. |
+ |
+ * page/gtk/DragControllerGtk.cpp: |
+ (WebCore::DragController::maxDragImageSize): Make the maximum size of the drag |
+ image a little smaller. |
+ * platform/DragImage.h: Changed the drag image type to be a cairo_surface_t*. |
+ * platform/gtk/DragImageGtk.cpp: |
+ (WebCore::dragImageSize): Reimplemented to reflect new drag image type. |
+ (WebCore::deleteDragImage): Ditto. |
+ (WebCore::scaleDragImage): Ditto. |
+ (WebCore::dissolveDragImageToFraction): Filled the stub of this function. |
+ (WebCore::createDragImageFromImage): Reimplemented to reflect new drag image type. |
+ * platform/gtk/GtkVersioning.h: Added definition for gtk_widget_get_realized for older GTK+s. |
+ |
+2010-09-15 David Hyatt <hyatt@apple.com> |
+ |
+ Reviewed by Darin Adler. |
+ |
+ https://bugs.webkit.org/show_bug.cgi?id=38402, paginate columns at layout time rather than at paint time. |
+ |
+ This patch adds support for column breaking at layout time rather than at paint time. New variables have been |
+ added to LayoutState and to ColumnInfo to track column information while laying out. |
+ |
+ The basic idea behind this patch is to retain the columns' paint-time hackery of transforming one long vertical strip |
+ into multiple columns. Now, however, layout is aware of the columns and will move objects up and down vertically in order |
+ to ensure they don't intersect a break. |
+ |
+ Many new tests added in fast/multicol and new results added for existing fast/multicol tests. |
+ |
+ * rendering/ColumnInfo.h: |
+ (WebCore::ColumnInfo::ColumnInfo): |
+ (WebCore::ColumnInfo::columnCount): |
+ (WebCore::ColumnInfo::columnHeight): |
+ (WebCore::ColumnInfo::setColumnCountAndHeight): |
+ (WebCore::ColumnInfo::setColumnHeight): |
+ (WebCore::ColumnInfo::updateMinimumColumnHeight): |
+ (WebCore::ColumnInfo::minimumColumnHeight): |
+ (WebCore::ColumnInfo::forcedBreaks): |
+ (WebCore::ColumnInfo::forcedBreakOffset): |
+ (WebCore::ColumnInfo::maximumDistanceBetweenForcedBreaks): |
+ (WebCore::ColumnInfo::clearForcedBreaks): |
+ (WebCore::ColumnInfo::addForcedBreak): |
+ * rendering/LayoutState.cpp: |
+ (WebCore::LayoutState::LayoutState): |
+ (WebCore::LayoutState::clearPaginationInformation): |
+ (WebCore::LayoutState::pageY): |
+ (WebCore::LayoutState::addForcedColumnBreak): |
+ * rendering/LayoutState.h: |
+ (WebCore::LayoutState::LayoutState): |
+ (WebCore::LayoutState::paginatingColumns): |
+ (WebCore::LayoutState::paginated): |
+ * rendering/RenderBlock.cpp: |
+ (WebCore::RenderBlock::RenderBlock): |
+ (WebCore::RenderBlock::~RenderBlock): |
+ (WebCore::RenderBlock::layoutBlock): |
+ (WebCore::RenderBlock::collapseMargins): |
+ (WebCore::RenderBlock::estimateVerticalPosition): |
+ (WebCore::RenderBlock::layoutBlockChild): |
+ (WebCore::RenderBlock::layoutPositionedObjects): |
+ (WebCore::RenderBlock::paintColumnRules): |
+ (WebCore::RenderBlock::paintColumnContents): |
+ (WebCore::RenderBlock::paintChildren): |
+ (WebCore::RenderBlock::insertFloatingObject): |
+ (WebCore::RenderBlock::removeFloatingObjects): |
+ (WebCore::RenderBlock::positionNewFloats): |
+ (WebCore::RenderBlock::positionNewFloatOnLine): |
+ (WebCore::RenderBlock::lowestPosition): |
+ (WebCore::RenderBlock::rightmostPosition): |
+ (WebCore::RenderBlock::leftmostPosition): |
+ (WebCore::RenderBlock::markAllDescendantsWithFloatsForLayout): |
+ (WebCore::RenderBlock::markDescendantBlocksAndLinesForLayout): |
+ (WebCore::RenderBlock::hitTestColumns): |
+ (WebCore::RenderBlock::setDesiredColumnCountAndWidth): |
+ (WebCore::RenderBlock::columnCount): |
+ (WebCore::RenderBlock::columnRectAt): |
+ (WebCore::RenderBlock::layoutColumns): |
+ (WebCore::RenderBlock::adjustPointToColumnContents): |
+ (WebCore::RenderBlock::adjustRectForColumns): |
+ (WebCore::RenderBlock::adjustForColumns): |
+ (WebCore::RenderBlock::setMaxTopMargins): |
+ (WebCore::RenderBlock::setMaxBottomMargins): |
+ (WebCore::RenderBlock::setPaginationStrut): |
+ (WebCore::RenderBlock::setPageY): |
+ (WebCore::RenderBlock::nextPageTop): |
+ (WebCore::inNormalFlow): |
+ (WebCore::RenderBlock::applyBeforeBreak): |
+ (WebCore::RenderBlock::applyAfterBreak): |
+ (WebCore::RenderBlock::adjustForUnsplittableChild): |
+ (WebCore::RenderBlock::adjustLinePositionForPagination): |
+ * rendering/RenderBlock.h: |
+ (WebCore::RenderBlock::paginationStrut): |
+ (WebCore::RenderBlock::pageY): |
+ (WebCore::RenderBlock::maxTopPosMargin): |
+ (WebCore::RenderBlock::maxTopNegMargin): |
+ (WebCore::RenderBlock::maxBottomPosMargin): |
+ (WebCore::RenderBlock::maxBottomNegMargin): |
+ (WebCore::RenderBlock::initMaxMarginValues): |
+ (WebCore::RenderBlock::FloatingObject::FloatingObject): |
+ (WebCore::RenderBlock::FloatingObject::type): |
+ (WebCore::RenderBlock::RenderBlockRareData::RenderBlockRareData): |
+ * rendering/RenderBlockLineLayout.cpp: |
+ (WebCore::RenderBlock::layoutInlineChildren): |
+ (WebCore::RenderBlock::determineStartPosition): |
+ (WebCore::RenderBlock::skipLeadingWhitespace): |
+ (WebCore::RenderBlock::findNextLineBreak): |
+ * rendering/RenderBox.cpp: |
+ (WebCore::RenderBox::mapLocalToContainer): |
+ (WebCore::RenderBox::computeRectForRepaint): |
+ (WebCore::RenderBox::markDescendantBlocksAndLinesForLayout): |
+ * rendering/RenderBox.h: |
+ * rendering/RenderFlexibleBox.cpp: |
+ (WebCore::RenderFlexibleBox::layoutBlock): |
+ (WebCore::RenderFlexibleBox::layoutHorizontalBox): |
+ (WebCore::RenderFlexibleBox::layoutVerticalBox): |
+ * rendering/RenderFlexibleBox.h: |
+ * rendering/RenderInline.cpp: |
+ (WebCore::RenderInline::computeRectForRepaint): |
+ (WebCore::RenderInline::mapLocalToContainer): |
+ * rendering/RenderLayer.cpp: |
+ (WebCore::RenderLayer::updatePagination): |
+ (WebCore::RenderLayer::paintChildLayerIntoColumns): |
+ (WebCore::RenderLayer::hitTestChildLayerColumns): |
+ * rendering/RenderLineBoxList.cpp: |
+ (WebCore::RenderLineBoxList::paint): |
+ * rendering/RenderTable.cpp: |
+ (WebCore::RenderTable::RenderTable): |
+ (WebCore::RenderTable::layout): |
+ * rendering/RenderTable.h: |
+ * rendering/RenderTableRow.cpp: |
+ (WebCore::RenderTableRow::layout): |
+ * rendering/RenderTableSection.cpp: |
+ (WebCore::RenderTableSection::layoutRows): |
+ * rendering/RenderView.cpp: |
+ (WebCore::RenderView::RenderView): |
+ (WebCore::RenderView::pushLayoutState): |
+ * rendering/RenderView.h: |
+ (WebCore::RenderView::setTruncatedAt): |
+ (WebCore::RenderView::pushLayoutState): |
+ (WebCore::RenderView::popLayoutState): |
+ (WebCore::LayoutStateMaintainer::LayoutStateMaintainer): |
+ (WebCore::LayoutStateMaintainer::push): |
+ * rendering/RootInlineBox.h: |
+ (WebCore::RootInlineBox::RootInlineBox): |
+ (WebCore::RootInlineBox::paginationStrut): |
+ (WebCore::RootInlineBox::setPaginationStrut): |
+ |
+2010-09-16 Pavel Podivilov <podivilov@chromium.org> |
+ |
+ Reviewed by Dimitri Glazkov. |
+ |
+ Fix debug crash in HTMLFrameOwnerElement caused by content frame being used after destruction. |
+ https://bugs.webkit.org/show_bug.cgi?id=45737 |
+ |
+ Test: fast/frames/frame-limit.html |
+ |
+ * html/HTMLFrameOwnerElement.cpp: |
+ (WebCore::HTMLFrameOwnerElement::willRemove): |
+ |
+2010-09-16 Thomas Zander <t.zander@nokia.com> |
+ |
+ Reviewed by Andreas Kling. |
+ |
+ https://bugs.webkit.org/show_bug.cgi?id=45878 |
+ |
+ [Qt] Fixed incorrect Symbian scoping. |
+ |
+ The missing install functionality is only true for mmp based systems. |
+ |
+ * WebCore.pro: limit functionality to mmp systems. |
+ |
+2010-09-16 Anders Carlsson <andersca@apple.com> |
+ |
+ Fix clang build. |
+ |
+ * page/animation/AnimationBase.h: |
+ Fix struct/class mismatch. |
+ |
+2010-09-16 Darin Adler <darin@apple.com> |
+ |
+ Reviewed by Andreas Kling. |
+ |
+ Reduce use of HTMLInputElement::inputType so we can remove it later |
+ https://bugs.webkit.org/show_bug.cgi?id=45903 |
+ |
+ * accessibility/AccessibilityRenderObject.cpp: |
+ (WebCore::AccessibilityRenderObject::isFileUploadButton): Use isFileUpload. |
+ (WebCore::AccessibilityRenderObject::isInputImage): Use isImageButton. |
+ (WebCore::AccessibilityRenderObject::determineAccessibilityRole): Use |
+ isCheckbox and isRadioButton. |
+ |
+ * css/CSSStyleSelector.cpp: |
+ (WebCore::CSSStyleSelector::adjustRenderStyle): Use isImageButton. |
+ |
+ * editing/Editor.cpp: |
+ (WebCore::Editor::setBaseWritingDirection): Use isTextField. |
+ |
+ * editing/ReplaceSelectionCommand.cpp: |
+ (WebCore::ReplaceSelectionCommand::doApply): Use isPasswordField. |
+ |
+ * editing/SelectionController.cpp: |
+ (WebCore::SelectionController::isInPasswordField): Use isPasswordField. |
+ |
+ * html/HTMLInputElement.h: Added isFileUpload, isImageButton, and isSubmitButton. |
+ |
+ * page/Chrome.cpp: |
+ (WebCore::Chrome::setToolTip): Use isSubmitButton and isFileUpload. |
+ |
+ * page/DragController.cpp: |
+ (WebCore::asFileInput): Use isFileUpload. |
+ |
+ * page/EventHandler.cpp: |
+ (WebCore::isSubmitImage): Use isImageButton. |
+ |
+ * rendering/RenderFileUploadControl.cpp: |
+ (WebCore::RenderFileUploadControl::updateFromElement): Use isFileUpload. |
+ |
+2010-09-16 Pavel Podivilov <podivilov@chromium.org> |
+ |
+ Reviewed by Pavel Feldman. |
+ |
+ Web Inspector: quick fix for crash in InspectorController::loadBreakpoints caused by empty main resource |
+ https://bugs.webkit.org/show_bug.cgi?id=45901 |
+ |
+ * inspector/InspectorController.cpp: |
+ (WebCore::InspectorController::breakpointsSettingKey): |
+ |
+2010-09-16 Pavel Feldman <pfeldman@chromium.org> |
+ |
+ Reviewed by Yury Semikhatsky. |
+ |
+ Web Inspector: Prevent responseReceivedTime from being > endTime in case mixture of WebCore and Network stack info is used. |
+ https://bugs.webkit.org/show_bug.cgi?id=45892 |
+ |
+ * inspector/InspectorResource.cpp: |
+ (WebCore::InspectorResource::endTiming): |
+ |
+2010-09-16 Ilya Tikhonovsky <loislo@chromium.org> |
+ |
+ Reviewed by Yury Semikhatsky. |
+ |
+ Web Inspector: move resourceTracking flag under control of BackendSettings. |
+ |
+ Four Inspector API methods about resourceTracking flag were removed. |
+ Actual state of the flag is transfered as the response of setResourceTracking request. |
+ Initial state of the flag on the frontend side is obtained from settings.backend. |
+ |
+ https://bugs.webkit.org/show_bug.cgi?id=45887 |
+ |
+ * inspector/Inspector.idl: |
+ * inspector/InspectorController.cpp: |
+ (WebCore::InspectorController::getBackendSettings): |
+ (WebCore::InspectorController::populateScriptObjects): |
+ (WebCore::InspectorController::setResourceTracking): |
+ * inspector/InspectorController.h: |
+ * inspector/front-end/ResourcesPanel.js: |
+ (WebInspector.ResourcesPanel.prototype._toggleResourceTracking): |
+ * inspector/front-end/Settings.js: |
+ (WebInspector.Settings.initialize.populateBackendSettings): |
+ (WebInspector.Settings.initialize): |
+ * inspector/front-end/inspector.js: |
+ |
+2010-09-16 Jian Li <jianli@chromium.org> |
+ |
+ Reviewed by David Levin. |
+ |
+ BlobData should be copied for it to be used cross-thread in ThreadableBlobRegistry. |
+ https://bugs.webkit.org/show_bug.cgi?id=45909 |
+ |
+ * fileapi/ThreadableBlobRegistry.cpp: |
+ (WebCore::BlobRegistryContext::BlobRegistryContext): |
+ |
+2010-09-16 Daniel Cheng <dcheng@chromium.org> |
+ |
+ Reviewed by Simon Fraser. |
+ |
+ [chromium] Fix incorrect drag node/selection painting. |
+ https://bugs.webkit.org/show_bug.cgi?id=45573 |
+ |
+ Drag selections were being incorrectly clipped when dragging a selection |
+ near the bottom of a page that was taller than the viewport. To fix |
+ this, we use paintContents() instead of paint(). |
+ |
+ * page/chromium/FrameChromium.cpp: |
+ (WebCore::Frame::nodeImage): |
+ (WebCore::Frame::dragImageForSelection): |
+ |
+2010-09-16 Daniel Cheng <dcheng@chromium.org> |
+ |
+ Reviewed by Tony Chang. |
+ |
+ Fix incorrect usage of dissolveDragImageToFraction(). |
+ https://bugs.webkit.org/show_bug.cgi?id=45835 |
+ |
+ createDragImageForSelection() was ignoring the return value of |
+ dissolveDragImageToFraction(). This didn't happen to crash on most |
+ platforms, since most implementations simply modified the image that was |
+ passed in. However, Chromium Mac's implementation actually creates a new |
+ image and returns that instead. This caused us to crash when copying the |
+ image from the renderer to the browser process, since the memory had |
+ already been freed. |
+ |
+ * manual-tests/selection-drag-crash.html: Added. |
+ * platform/DragImage.cpp: |
+ (WebCore::createDragImageForSelection): |
+ |
+2010-09-16 Sheriff Bot <webkit.review.bot@gmail.com> |
+ |
+ Unreviewed, rolling out r67628. |
+ http://trac.webkit.org/changeset/67628 |
+ https://bugs.webkit.org/show_bug.cgi?id=45904 |
+ |
+ broke the build (Requested by eric_carlson on #webkit). |
+ |
+ * page/ContextMenuController.cpp: |
+ (WebCore::ContextMenuController::contextMenuItemSelected): |
+ * platform/ContextMenu.cpp: |
+ (WebCore::ContextMenu::populate): |
+ (WebCore::ContextMenu::checkOrEnableIfNeeded): |
+ * platform/ContextMenuItem.h: |
+ * platform/LocalizationStrategy.h: |
+ * platform/LocalizedStrings.cpp: |
+ * platform/LocalizedStrings.h: |
+ * platform/android/LocalizedStringsAndroid.cpp: |
+ * platform/brew/LocalizedStringsBrew.cpp: |
+ * platform/efl/LocalizedStringsEfl.cpp: |
+ * platform/gtk/ContextMenuItemGtk.cpp: |
+ (WebCore::gtkStockIDFromContextMenuAction): |
+ * platform/gtk/LocalizedStringsGtk.cpp: |
+ * platform/haiku/LocalizedStringsHaiku.cpp: |
+ * platform/wx/LocalizedStringsWx.cpp: |
+ * rendering/HitTestResult.cpp: |
+ (WebCore::HitTestResult::absoluteMediaURL): |
+ * rendering/HitTestResult.h: |
+ |
+2010-09-16 Andreas Kling <andreas.kling@nokia.com> |
+ |
+ Reviewed by Simon Fraser. |
+ |
+ CSS: Fix crash in getTimingFunctionValue() |
+ https://bugs.webkit.org/show_bug.cgi?id=45896 |
+ |
+ Use a RefPtr to avoid deleting the TimingFunction prematurely. |
+ |
+ This is covered by existing tests, e.g transitions/inherit-other-props.html |
+ but will only actually crash on picky platforms (or in valgrind.) |
+ |
+ * css/CSSComputedStyleDeclaration.cpp: |
+ (WebCore::getTimingFunctionValue): |
+ |
+2010-09-16 Pavel Feldman <pfeldman@chromium.org> |
+ |
+ Reviewed by Yury Semikhatsky. |
+ |
+ Web Inspector: prevent resource timing popover from having scrollers. |
+ https://bugs.webkit.org/show_bug.cgi?id=45883 |
+ |
+ * inspector/front-end/ResourcesPanel.js: |
+ (WebInspector.ResourcesPanel.prototype._showPopover): |
+ |
+2010-09-15 Philippe Normand <pnormand@igalia.com> |
+ |
+ Reviewed by Eric Carlson. |
+ |
+ [GTK] enhanced context menu for media elements |
+ https://bugs.webkit.org/show_bug.cgi?id=45021 |
+ |
+ New items in the media Element context menu: |
+ - play/pause |
+ - mute/unmute |
+ - controls display control |
+ - switch to fullscreen (for video only) |
+ - loop playback control |
+ - copy media url to clipboard |
+ - open in new window |
+ |
+ Test: media/context-menu-actions.html |
+ |
+ * page/ContextMenuController.cpp: |
+ (WebCore::ContextMenuController::contextMenuItemSelected): |
+ * platform/ContextMenu.cpp: |
+ (WebCore::ContextMenu::populate): |
+ (WebCore::ContextMenu::checkOrEnableIfNeeded): |
+ * platform/ContextMenuItem.h: |
+ * platform/LocalizationStrategy.h: |
+ * platform/LocalizedStrings.cpp: |
+ (WebCore::contextMenuItemTagOpenMediaInNewWindow): |
+ (WebCore::contextMenuItemTagCopyMediaLinkToClipboard): |
+ (WebCore::contextMenuItemTagToggleMediaControls): |
+ (WebCore::contextMenuItemTagToggleMediaLoop): |
+ (WebCore::contextMenuItemTagEnterVideoFullscreen): |
+ (WebCore::contextMenuItemTagMediaPlay): |
+ (WebCore::contextMenuItemTagMediaPause): |
+ (WebCore::contextMenuItemTagMediaMute): |
+ (WebCore::contextMenuItemTagMediaUnMute): |
+ * platform/LocalizedStrings.h: |
+ * platform/android/LocalizedStringsAndroid.cpp: |
+ (WebCore::contextMenuItemTagOpenMediaInNewWindow): |
+ (WebCore::contextMenuItemTagCopyMediaLinkToClipboard): |
+ (WebCore::contextMenuItemTagToggleMediaControls): |
+ (WebCore::contextMenuItemTagToggleMediaLoop): |
+ (WebCore::contextMenuItemTagEnterVideoFullscreen): |
+ (WebCore::contextMenuItemTagMediaPlay): |
+ (WebCore::contextMenuItemTagMediaPause): |
+ (WebCore::contextMenuItemTagMediaMute): |
+ (WebCore::contextMenuItemTagMediaUnMute): |
+ * platform/brew/LocalizedStringsBrew.cpp: |
+ (WebCore::contextMenuItemTagOpenMediaInNewWindow): |
+ (WebCore::contextMenuItemTagCopyMediaLinkToClipboard): |
+ (WebCore::contextMenuItemTagToggleMediaControls): |
+ (WebCore::contextMenuItemTagToggleMediaLoop): |
+ (WebCore::contextMenuItemTagEnterVideoFullscreen): |
+ (WebCore::contextMenuItemTagMediaPlay): |
+ (WebCore::contextMenuItemTagMediaPause): |
+ (WebCore::contextMenuItemTagMediaMute): |
+ (WebCore::contextMenuItemTagMediaUnMute): |
+ * platform/efl/LocalizedStringsEfl.cpp: |
+ (WebCore::contextMenuItemTagOpenMediaInNewWindow): |
+ (WebCore::contextMenuItemTagCopyMediaLinkToClipboard): |
+ (WebCore::contextMenuItemTagToggleMediaControls): |
+ (WebCore::contextMenuItemTagToggleMediaLoop): |
+ (WebCore::contextMenuItemTagEnterVideoFullscreen): |
+ (WebCore::contextMenuItemTagMediaPlay): |
+ (WebCore::contextMenuItemTagMediaPause): |
+ (WebCore::contextMenuItemTagMediaMute): |
+ (WebCore::contextMenuItemTagMediaUnMute): |
+ * platform/gtk/ContextMenuItemGtk.cpp: |
+ (WebCore::gtkStockIDFromContextMenuAction): |
+ * platform/gtk/LocalizedStringsGtk.cpp: |
+ (WebCore::contextMenuItemTagOpenMediaInNewWindow): |
+ (WebCore::contextMenuItemTagCopyMediaLinkToClipboard): |
+ (WebCore::contextMenuItemTagToggleMediaControls): |
+ (WebCore::contextMenuItemTagToggleMediaLoop): |
+ (WebCore::contextMenuItemTagEnterVideoFullscreen): |
+ (WebCore::contextMenuItemTagMediaPlay): |
+ (WebCore::contextMenuItemTagMediaPause): |
+ (WebCore::contextMenuItemTagMediaMute): |
+ (WebCore::contextMenuItemTagMediaUnMute): |
+ * platform/haiku/LocalizedStringsHaiku.cpp: |
+ (WebCore::contextMenuItemTagOpenMediaInNewWindow): |
+ (WebCore::contextMenuItemTagCopyMediaLinkToClipboard): |
+ (WebCore::contextMenuItemTagToggleMediaControls): |
+ (WebCore::contextMenuItemTagToggleMediaLoop): |
+ (WebCore::contextMenuItemTagEnterVideoFullscreen): |
+ (WebCore::contextMenuItemTagMediaPlay): |
+ (WebCore::contextMenuItemTagMediaPause): |
+ (WebCore::contextMenuItemTagMediaMute): |
+ (WebCore::contextMenuItemTagMediaUnMute): |
+ * platform/wx/LocalizedStringsWx.cpp: |
+ (WebCore::contextMenuItemTagOpenMediaInNewWindow): |
+ (WebCore::contextMenuItemTagCopyMediaLinkToClipboard): |
+ (WebCore::contextMenuItemTagToggleMediaControls): |
+ (WebCore::contextMenuItemTagToggleMediaLoop): |
+ (WebCore::contextMenuItemTagEnterVideoFullscreen): |
+ (WebCore::contextMenuItemTagMediaPlay): |
+ (WebCore::contextMenuItemTagMediaPause): |
+ (WebCore::contextMenuItemTagMediaMute): |
+ (WebCore::contextMenuItemTagMediaUnMute): |
+ * rendering/HitTestResult.cpp: |
+ (WebCore::HitTestResult::absoluteMediaURL): |
+ (WebCore::HitTestResult::mediaSupportsFullscreen): |
+ (WebCore::HitTestResult::mediaElement): |
+ (WebCore::HitTestResult::toggleMediaControlsDisplay): |
+ (WebCore::HitTestResult::toggleMediaLoopPlayback): |
+ (WebCore::HitTestResult::enterFullscreenForVideo): |
+ (WebCore::HitTestResult::mediaControlsEnabled): |
+ (WebCore::HitTestResult::mediaLoopEnabled): |
+ (WebCore::HitTestResult::mediaPlaying): |
+ (WebCore::HitTestResult::toggleMediaPlayState): |
+ (WebCore::HitTestResult::mediaHasAudio): |
+ (WebCore::HitTestResult::mediaMuted): |
+ (WebCore::HitTestResult::toggleMediaMuteState): |
+ * rendering/HitTestResult.h: |
+ |
2010-09-16 Tony Gentilcore <tonyg@chromium.org> |
Reviewed by Adam Barth. |