Index: Source/WebCore/ChangeLog |
=================================================================== |
--- Source/WebCore/ChangeLog (revision 85349) |
+++ Source/WebCore/ChangeLog (working copy) |
@@ -1,4319 +1,3 @@ |
-2011-04-27 Nate Chapin <japhet@chromium.org> |
- |
- Reviewed by Adam Barth. |
- |
- Don't use XMLTreeViewer when script is disabled (it uses Javascript |
- to build a prettified version of the xml document). |
- https://bugs.webkit.org/show_bug.cgi?id=59113 |
- |
- * xml/XMLTreeViewer.cpp: |
- (WebCore::XMLTreeViewer::transformDocumentToTreeView): |
- |
-2011-04-26 James Robinson <jamesr@chromium.org> |
- |
- Reviewed by Kenneth Russell. |
- |
- [chromium] Only reset tiles if the LayerRendererChromium is actually changing |
- https://bugs.webkit.org/show_bug.cgi?id=59572 |
- |
- http://trac.webkit.org/changeset/84981/ caused us to reset the tiles |
- when LayerRendererChromium was set. We set this pointer every frame |
- and should only reset the tiles if the new LayerRendererChromium is |
- different from the old one (indicating a context loss), not just reset |
- on every frame. |
- |
- * platform/graphics/chromium/LayerTilerChromium.cpp: |
- (WebCore::LayerTilerChromium::setLayerRenderer): |
- * platform/graphics/chromium/LayerTilerChromium.h: |
- |
-2011-04-26 Dinesh K Garg <dineshg@codeaurora.org> |
- |
- Reviewed by Eric Seidel. |
- |
- beginElement broken by setAttribute |
- https://bugs.webkit.org/show_bug.cgi?id=26019 |
- |
- Test: svg/animations/animate-beginElementAt.svg |
- |
- * svg/SVGAnimationElement.cpp: |
- (WebCore::SVGAnimationElement::attributeChanged): |
- Reset the animation state here as the rest of the code reads it and would still |
- think we are animating when we are not. |
- |
- * svg/animation/SVGSMILElement.h: |
- (WebCore::SVGSMILElement::setInactive): Helper function to reset the state. |
- |
-2011-04-26 Sheriff Bot <webkit.review.bot@gmail.com> |
- |
- Unreviewed, rolling out r84989. |
- http://trac.webkit.org/changeset/84989 |
- https://bugs.webkit.org/show_bug.cgi?id=59566 |
- |
- REGRESSION (r84989): Lots of vertical text tests failing on |
- windows (Requested by weinig on #webkit). |
- |
- * platform/graphics/FontPlatformData.h: |
- * platform/graphics/win/FontCacheWin.cpp: |
- (WebCore::FontCache::getFontDataForCharacters): |
- (WebCore::FontCache::createFontPlatformData): |
- * platform/graphics/win/FontCustomPlatformData.cpp: |
- (WebCore::FontCustomPlatformData::fontPlatformData): |
- * platform/graphics/win/FontPlatformDataCGWin.cpp: |
- (WebCore::FontPlatformData::FontPlatformData): |
- * platform/graphics/win/FontPlatformDataWin.cpp: |
- (WebCore::FontPlatformData::FontPlatformData): |
- |
-2011-04-26 Sam Weinig <sam@webkit.org> |
- |
- Reviewed by Eric Seidel. |
- |
- Try again: |
- Remove JSDOMWrapperWithGlobalPointer now that all JSDOMWrappers have global objects |
- https://bugs.webkit.org/show_bug.cgi?id=59310 |
- |
- * bindings/js/JSDOMBinding.h: |
- (WebCore::DOMConstructorObject::DOMConstructorObject): |
- * bindings/js/JSDOMWrapper.h: |
- (WebCore::JSDOMWrapper::globalObject): |
- (WebCore::JSDOMWrapper::scriptExecutionContext): |
- (WebCore::JSDOMWrapper::createStructure): |
- (WebCore::JSDOMWrapper::JSDOMWrapper): |
- * bindings/scripts/CodeGeneratorJS.pm: |
- |
-2011-04-26 Sam Weinig <sam@webkit.org> |
- |
- Reviewed by David Hyatt. |
- |
- Remove Datagrid from the tree |
- https://bugs.webkit.org/show_bug.cgi?id=59543 |
- |
- * Android.derived.jscbindings.mk: |
- * Android.derived.v8bindings.mk: |
- * Android.jscbindings.mk: |
- * Android.v8bindings.mk: |
- * CMakeLists.txt: |
- * CodeGenerators.pri: |
- * Configurations/FeatureDefines.xcconfig: |
- * DerivedSources.cpp: |
- * DerivedSources.make: |
- * GNUmakefile.am: |
- * GNUmakefile.list.am: |
- * UseJSC.cmake: |
- * WebCore.gypi: |
- * WebCore.pro: |
- * WebCore.vcproj/WebCore.vcproj: |
- * WebCore.xcodeproj/project.pbxproj: |
- * bindings/js/JSBindingsAllInOne.cpp: |
- * bindings/js/JSDataGridColumnListCustom.cpp: Removed. |
- * bindings/js/JSDataGridDataSource.cpp: Removed. |
- * bindings/js/JSDataGridDataSource.h: Removed. |
- * bindings/js/JSHTMLDataGridElementCustom.cpp: Removed. |
- * bindings/v8/V8DataGridDataSource.cpp: Removed. |
- * bindings/v8/V8DataGridDataSource.h: Removed. |
- * bindings/v8/custom/V8DataGridColumnListCustom.cpp: Removed. |
- * bindings/v8/custom/V8HTMLDataGridElementCustom.cpp: Removed. |
- * css/CSSStyleSelector.cpp: |
- * css/CSSStyleSelector.h: |
- * css/html.css: |
- (input, textarea, keygen, select, button, isindex): |
- (select:disabled, keygen:disabled, optgroup:disabled, option:disabled): |
- * editing/htmlediting.cpp: |
- (WebCore::canHaveChildrenForEditing): |
- * features.pri: |
- * html/DOMDataGridDataSource.cpp: Removed. |
- * html/DOMDataGridDataSource.h: Removed. |
- * html/DataGridColumn.cpp: Removed. |
- * html/DataGridColumn.h: Removed. |
- * html/DataGridColumn.idl: Removed. |
- * html/DataGridColumnList.cpp: Removed. |
- * html/DataGridColumnList.h: Removed. |
- * html/DataGridColumnList.idl: Removed. |
- * html/DataGridDataSource.h: Removed. |
- * html/HTMLDataGridCellElement.cpp: Removed. |
- * html/HTMLDataGridCellElement.h: Removed. |
- * html/HTMLDataGridCellElement.idl: Removed. |
- * html/HTMLDataGridColElement.cpp: Removed. |
- * html/HTMLDataGridColElement.h: Removed. |
- * html/HTMLDataGridColElement.idl: Removed. |
- * html/HTMLDataGridElement.cpp: Removed. |
- * html/HTMLDataGridElement.h: Removed. |
- * html/HTMLDataGridElement.idl: Removed. |
- * html/HTMLDataGridRowElement.cpp: Removed. |
- * html/HTMLDataGridRowElement.h: Removed. |
- * html/HTMLDataGridRowElement.idl: Removed. |
- * html/HTMLElement.cpp: |
- (WebCore::HTMLElement::ieForbidsInsertHTML): |
- * html/HTMLElementsAllInOne.cpp: |
- * html/HTMLTagNames.in: |
- * page/DOMWindow.idl: |
- * rendering/RenderBox.cpp: |
- (WebCore::RenderBox::sizesToIntrinsicLogicalWidth): |
- * rendering/RenderDataGrid.cpp: Removed. |
- * rendering/RenderDataGrid.h: Removed. |
- * rendering/RenderingAllInOne.cpp: |
- |
-2011-04-26 Chun-Lung Huang <alvincl.huang@gmail.com> |
- |
- Reviewed by Adele Peterson. |
- |
- On WebKit (Windows), glyphs in vertical text tests are rotated 90 |
- degrees clockwise. https://bugs.webkit.org/show_bug.cgi?id=48459 |
- |
- This platform dependent patch makes WebKit (Windows) show the |
- vertical writing text correctly. Job was done by adding a prefix '@' |
- in front of the font family name (Windows Only). No new tests added. |
- Some layout tests images: |
- http://www.flickr.com/photos/burorly/sets/72157625585506341/ |
- |
- * platform/graphics/FontPlatformData.h: |
- * platform/graphics/win/FontCacheWin.cpp: |
- (WebCore::FontCache::getFontDataForCharacters): |
- (WebCore::FontCache::createFontPlatformData): |
- * platform/graphics/win/FontCustomPlatformData.cpp: |
- (WebCore::FontCustomPlatformData::fontPlatformData): |
- * platform/graphics/win/FontPlatformDataCGWin.cpp: |
- (WebCore::FontPlatformData::FontPlatformData): |
- * platform/graphics/win/FontPlatformDataWin.cpp: |
- (WebCore::FontPlatformData::FontPlatformData): |
- |
-2011-04-26 Jer Noble <jer.noble@apple.com> |
- |
- Reviewed by Brady Eidson. |
- |
- HTML5 video fullscreen transition causes an audio stutter |
- https://bugs.webkit.org/show_bug.cgi?id=59544 |
- |
- AVFoundation will momentarily stutter when you disconnect a AVPlayerLayer |
- from its AVPlayer. Since you shouldn't necessarily have to destroy the |
- layer to render it invisible, instead just set the layer as hidden. |
- |
- * platform/graphics/avfoundation/MediaPlayerPrivateAVFoundation.cpp: |
- (WebCore::MediaPlayerPrivateAVFoundation::setVisible): Call through to platformSetVisible(). |
- * platform/graphics/avfoundation/MediaPlayerPrivateAVFoundation.h: |
- * platform/graphics/avfoundation/MediaPlayerPrivateAVFoundationObjC.h: |
- * platform/graphics/avfoundation/MediaPlayerPrivateAVFoundationObjC.mm: |
- (WebCore::MediaPlayerPrivateAVFoundationObjC::platformSetVisible): Added. |
- |
-2011-04-26 Christian Dywan <christian@lanedo.com> |
- |
- Reviewed by Eric Seidel. |
- |
- Extra separator after Select All in context menu |
- https://bugs.webkit.org/show_bug.cgi?id=54092 |
- |
- * page/ContextMenuController.cpp: Move separator to non-GTK+ |
- port guards since it is for the spelling options menu |
- which the GTK+ port doesn't use. |
- |
-2011-04-26 James Robinson <jamesr@chromium.org> |
- |
- Reviewed by Kenneth Russell. |
- |
- [chromium] Clear out LayerTilerChromium's tiles on lost context |
- https://bugs.webkit.org/show_bug.cgi?id=59233 |
- |
- Clears out the LayerTilerChromium's tile set when the LayerRendererChromium is reset. In this case the tiles are |
- no longer valid since they exist in the previous LayerRendererChromium. |
- |
- Will be tested by a chrome test since we don't have any way to exercise the context lost case in layout tests. |
- |
- * platform/graphics/chromium/LayerTilerChromium.h: |
- (WebCore::LayerTilerChromium::setLayerRenderer): |
- |
-2011-04-26 Martin Robinson <mrobinson@igalia.com> |
- |
- Reviewed by Dirk Schulze. |
- |
- [GTK] Flash in divs with overflow: auto is not positioned and clipped properly |
- https://bugs.webkit.org/show_bug.cgi?id=57644 |
- |
- Manual test: plugins/windowed-in-iframe-2.html |
- |
- * manual-tests/plugins/windowed-in-iframe-2.html: Added. |
- * platform/gtk/WidgetGtk.cpp: |
- (WebCore::Widget::setFrameRect): Like the Qt port, we notify the widget via frameRectsChanged. |
- * plugins/PluginView.h: Added helper method to set the allocation and clip. |
- * plugins/gtk/PluginViewGtk.cpp: |
- (WebCore::PluginView::setNPWindowIfNeeded):Call the helper method instead of setting |
- the allocation immediately. Bring this method into sync with the implementation in |
- the Qt port. |
- (WebCore::PluginView::updateWidgetAllocationAndClip): Added. |
- (WebCore::PluginView::plugAddedCallback): Call the new helper method. |
- |
-2011-04-26 Martin Robinson <mrobinson@igalia.com> |
- |
- Reviewed by Eric Seidel. |
- |
- [GTK] fast/block/float/overhanging-tall-block.html crashes in the bots |
- https://bugs.webkit.org/show_bug.cgi?id=58818 |
- |
- Prevent allocating scratch buffers larger than the target GdkDrawable |
- when creating a WidgetRenderingContext. This prevents incredibly large |
- widgets from crashing the X Server. This change also allowed greatly |
- simplifying the way that the painting offset is calculated. |
- |
- * platform/gtk/WidgetRenderingContext.cpp: |
- (WebCore::WidgetRenderingContext::WidgetRenderingContext): Never allocate a pixmap |
- larger than the target GdkDrawable. |
- (WebCore::WidgetRenderingContext::~WidgetRenderingContext): Calculate the offset of the |
- blit by looking at the location of the target rect itself, rather than delaying the |
- calculation up to this point. |
- (WebCore::WidgetRenderingContext::calculateClipRect): Added. |
- (WebCore::WidgetRenderingContext::gtkPaintBox): Use the new calculateClipRectHelper. |
- (WebCore::WidgetRenderingContext::gtkPaintFlatBox): Ditto. |
- (WebCore::WidgetRenderingContext::gtkPaintFocus): Ditto. |
- (WebCore::WidgetRenderingContext::gtkPaintSlider): Ditto. |
- (WebCore::WidgetRenderingContext::gtkPaintCheck): Ditto. |
- (WebCore::WidgetRenderingContext::gtkPaintOption): Ditto. |
- (WebCore::WidgetRenderingContext::gtkPaintShadow): Ditto. |
- (WebCore::WidgetRenderingContext::gtkPaintArrow): Ditto. |
- (WebCore::WidgetRenderingContext::gtkPaintVLine): Ditto. |
- * platform/gtk/WidgetRenderingContext.h: Remove some now unused members. Add |
- a member to store the IntSize mapping from the coordinates of the target to the coordinates |
- of the scratch buffer. |
- |
-2011-04-26 Patrick Gansterer <paroga@webkit.org> |
- |
- Reviewed by Adam Barth. |
- |
- Respect charset in handleDataURL |
- https://bugs.webkit.org/show_bug.cgi?id=47746 |
- |
- Original patch by Kwang Yul Seo <skyul@company100.net> |
- |
- We must use TextEncoding(charset) to encode data URL again. Otherwise TextEncoding::encode() |
- returns an empty string because encoding name is not specified. |
- |
- Also use "US-ASCII" as fallback charset as extractCharsetFromMediaType can be empty. |
- |
- This change fixes over 130 layout tests when running GTK port with cURL network backend. |
- |
- * platform/network/DataURL.cpp: |
- (WebCore::handleDataURL): |
- |
-2011-04-26 Anders Carlsson <andersca@apple.com> |
- |
- Fix clang++ build. |
- |
- You can never have too many casts! |
- |
- * accessibility/mac/AccessibilityObjectWrapper.mm: |
- (-[AccessibilityObjectWrapper position]): |
- * editing/mac/SelectionControllerMac.mm: |
- (WebCore::accessibilityConvertScreenRect): |
- |
-2011-04-26 Chris Rogers <crogers@google.com> |
- |
- Reviewed by Kenneth Russell. |
- |
- Fix web audio build on mac port |
- https://bugs.webkit.org/show_bug.cgi?id=59355 |
- |
- No new tests since audio API is not yet implemented. |
- |
- * WebCore.xcodeproj/project.pbxproj: |
- * bindings/js/JSAudioContextCustom.cpp: |
- (WebCore::JSAudioContext::visitChildren): |
- * bindings/js/JSJavaScriptAudioNodeCustom.cpp: |
- (WebCore::JSJavaScriptAudioNode::visitChildren): |
- * webaudio/AudioContext.idl: |
- |
-2011-04-26 Chris Rogers <crogers@google.com> |
- |
- Reviewed by Kenneth Russell. |
- |
- Add FFTFrame implementation for FFmpeg |
- https://bugs.webkit.org/show_bug.cgi?id=59408 |
- |
- No new tests since audio API is not yet implemented. |
- |
- * WebCore.gyp/WebCore.gyp: |
- * WebCore.gypi: |
- * platform/audio/FFTFrame.h: |
- * platform/audio/FFTFrameStub.cpp: |
- * platform/audio/ffmpeg: Added. |
- * platform/audio/ffmpeg/FFTFrameFFMPEG.cpp: Added. |
- (WebCore::FFTFrame::FFTFrame): |
- (WebCore::FFTFrame::initialize): |
- (WebCore::FFTFrame::cleanup): |
- (WebCore::FFTFrame::~FFTFrame): |
- (WebCore::FFTFrame::multiply): |
- (WebCore::FFTFrame::doFFT): |
- (WebCore::FFTFrame::doInverseFFT): |
- (WebCore::FFTFrame::realData): |
- (WebCore::FFTFrame::imagData): |
- (WebCore::FFTFrame::getUpToDateComplexData): |
- (WebCore::FFTFrame::contextForSize): |
- * platform/audio/mac/FFTFrameMac.cpp: |
- * webaudio/ConvolverNode.cpp: |
- |
-2011-04-26 Jeff Miller <jeffm@apple.com> |
- |
- Add a newline at the end of SVGFEDropShadowElement.idl. |
- |
- * svg/SVGFEDropShadowElement.idl: |
- |
-2011-04-26 Mihai Parparita <mihaip@chromium.org> |
- |
- Reviewed by Eric Seidel. |
- |
- InlineBox.h shouldn't include RenderBR.h |
- https://bugs.webkit.org/show_bug.cgi?id=59480 |
- |
- Per bug 59348 RenderBR.h is a "top header" by include * size, because |
- InlineBox.h includes it. There's no reason why it has to. |
- |
- * rendering/InlineBox.h: |
- * rendering/InlineTextBox.cpp: |
- * rendering/RenderBlock.h: |
- |
-2011-04-26 Sam Weinig <sam@webkit.org> |
- |
- Nope, roll out r84950 and r84952. Working on a laptop is clearly not something |
- I should do. |
- |
- * bindings/js/JSDOMBinding.h: |
- (WebCore::JSDOMWrapperWithGlobalPointer::globalObject): |
- (WebCore::JSDOMWrapperWithGlobalPointer::scriptExecutionContext): |
- (WebCore::JSDOMWrapperWithGlobalPointer::createStructure): |
- (WebCore::JSDOMWrapperWithGlobalPointer::JSDOMWrapperWithGlobalPointer): |
- (WebCore::DOMConstructorObject::DOMConstructorObject): |
- * bindings/js/JSDOMWrapper.h: |
- (WebCore::JSDOMWrapper::JSDOMWrapper): |
- * bindings/scripts/CodeGeneratorJS.pm: |
- |
-2011-04-26 Sam Weinig <sam@webkit.org> |
- |
- Fix build. |
- |
- * bindings/js/JSDOMWrapper.h: |
- (WebCore::JSDOMWrapper::globalObject): |
- |
-2011-04-26 Sam Weinig <sam@webkit.org> |
- |
- Roll r84945 back in with infinite recursion removed. |
- |
- * bindings/js/JSDOMBinding.h: |
- (WebCore::DOMConstructorObject::DOMConstructorObject): |
- * bindings/js/JSDOMWrapper.h: |
- (WebCore::JSDOMWrapper::globalObject): |
- (WebCore::JSDOMWrapper::scriptExecutionContext): |
- (WebCore::JSDOMWrapper::createStructure): |
- (WebCore::JSDOMWrapper::JSDOMWrapper): |
- * bindings/scripts/CodeGeneratorJS.pm: |
- |
-2011-04-26 Sam Weinig <sam@webkit.org> |
- |
- Rollout previous patch. It broke stuff. |
- |
- * bindings/js/JSDOMBinding.h: |
- (WebCore::JSDOMWrapperWithGlobalPointer::globalObject): |
- (WebCore::JSDOMWrapperWithGlobalPointer::scriptExecutionContext): |
- (WebCore::JSDOMWrapperWithGlobalPointer::createStructure): |
- (WebCore::JSDOMWrapperWithGlobalPointer::JSDOMWrapperWithGlobalPointer): |
- (WebCore::DOMConstructorObject::DOMConstructorObject): |
- * bindings/js/JSDOMWrapper.h: |
- (WebCore::JSDOMWrapper::JSDOMWrapper): |
- * bindings/scripts/CodeGeneratorJS.pm: |
- |
-2011-04-26 Sam Weinig <sam@webkit.org> |
- |
- Reviewed by Maciej Stachowiak. |
- |
- Remove JSDOMWrapperWithGlobalPointer now that all JSDOMWrappers have global objects |
- https://bugs.webkit.org/show_bug.cgi?id=59310 |
- |
- * bindings/js/JSDOMBinding.h: |
- (WebCore::DOMConstructorObject::DOMConstructorObject): |
- * bindings/js/JSDOMWrapper.h: |
- (WebCore::JSDOMWrapper::globalObject): |
- (WebCore::JSDOMWrapper::scriptExecutionContext): |
- (WebCore::JSDOMWrapper::createStructure): |
- (WebCore::JSDOMWrapper::JSDOMWrapper): |
- Merge JSDOMWrapperWithGlobalPointer with JSDOMWrapper. |
- |
- * bindings/scripts/CodeGeneratorJS.pm: |
- Make JSDOMWrapper the base class. |
- |
-2011-04-26 Dan Bernstein <mitz@apple.com> |
- |
- Reviewed by Mark Rowe. |
- |
- Choose the compiler based on the Xcode version for Snow Leopard debug builds. |
- |
- * Configurations/Base.xcconfig: |
- * Configurations/CompilerVersion.xcconfig: Added. |
- |
-2011-04-26 Adrienne Walker <enne@google.com> |
- |
- Reviewed by Geoffrey Garen. |
- |
- Use OwnArrayPtr<T> instead of OwnPtr<T*> in ScopeTracer |
- https://bugs.webkit.org/show_bug.cgi?id=59469 |
- |
- * platform/chromium/TraceEvent.h: |
- (WebCore::internal::ScopeTracer::ScopeTracer): |
- |
-2011-04-25 Geoffrey Garen <ggaren@apple.com> |
- |
- Reviewed by Oliver Hunt. |
- |
- Nixed special finalizer handling for WebCore strings |
- https://bugs.webkit.org/show_bug.cgi?id=59425 |
- |
- Not needed anymore, since weak handles have finalizers. |
- |
- * WebCore.exp.in: Exports! |
- |
- * bindings/js/DOMWrapperWorld.cpp: |
- (WebCore::JSStringOwner::finalize): |
- (WebCore::DOMWrapperWorld::DOMWrapperWorld): Use a weak handle finalizer, |
- so we don't need special treatment anymore. |
- |
- * bindings/js/DOMWrapperWorld.h: |
- (WebCore::JSStringOwner::JSStringOwner): |
- (WebCore::DOMWrapperWorld::stringWrapperOwner): Use a HashMap of Weak<T> |
- instead of a WeakGCMap, so we can specify a custom finalizer. |
- |
- * bindings/js/JSDOMBinding.cpp: |
- (WebCore::jsStringSlowCase): |
- * bindings/js/JSDOMBinding.h: |
- (WebCore::jsString): Updated for string map change. |
- |
-2011-04-26 David Kilzer <ddkilzer@apple.com> |
- |
- <http://webkit.org/b/59372> Restore user default for enabling HTTP pipelining |
- |
- Reviewed by Antti Koivisto. |
- |
- To enable HTTP pipelining post-SnowLeopard, use this command: |
- |
- defaults write BUNDLE.ID WebKitEnableHTTPPipelining -bool YES |
- |
- * platform/network/cf/ResourceRequestCFNet.cpp: |
- (WebCore::readBooleanPreference): Re-added. Originally added in |
- r76756 and removed in r84120. |
- (WebCore::initializeMaximumHTTPConnectionCountPerHost): Enable |
- HTTP pipelining if use default is set to true. Added macro |
- guards to match those in ResourceRequestMac.mm. |
- |
-2011-04-26 Antti Koivisto <antti@apple.com> |
- |
- Reviewed by Adele Peterson. |
- |
- Don't strip fragments from file: urls in memory cache |
- https://bugs.webkit.org/show_bug.cgi?id=59463 |
- <rdar://problem/9231130> |
- |
- Some clients expect file: resources to be unique based on fragment identifier. |
- Limit the changes in http://trac.webkit.org/changeset/74107 to the HTTP protocol only |
- |
- Test: fast/loader/file-protocol-fragment.html |
- |
- * loader/cache/MemoryCache.cpp: |
- (WebCore::MemoryCache::removeFragmentIdentifierIfNeeded): |
- |
-2011-04-26 David Levin <levin@chromium.org> |
- |
- Reviewed by Darin Adler. |
- |
- Fix more strict OwnPtr in WebCore (round N). |
- https://bugs.webkit.org/show_bug.cgi?id=59464 |
- |
- * dom/Document.cpp: |
- (WebCore::Document::Document): |
- (WebCore::Document::attach): |
- * fileapi/FileReader.cpp: |
- (WebCore::FileReader::terminate): |
- * loader/DocumentThreadableLoader.cpp: |
- (WebCore::DocumentThreadableLoader::preflightFailure): |
- * workers/DefaultSharedWorkerRepository.cpp: |
- (WebCore::SharedWorkerConnectTask::create): |
- (WebCore::SharedWorkerScriptLoader::load): |
- |
-2011-04-26 Ruben <chromium@hybridsource.org> |
- |
- Reviewed by Tony Chang. |
- |
- Change linux ifdefs for Chromium to Unix instead, allowing |
- new Unix ports to be enabled by default |
- https://bugs.webkit.org/show_bug.cgi?id=59297 |
- |
- No new tests, just changing preprocessor defines. |
- |
- * bindings/js/JSInspectorFrontendHostCustom.cpp: |
- * bindings/v8/custom/V8InspectorFrontendHostCustom.cpp: |
- * dom/SelectElement.cpp: |
- * loader/cache/CachedFont.cpp: |
- * page/EventHandler.cpp: |
- * page/chromium/EventHandlerChromium.cpp: |
- * platform/Scrollbar.cpp: |
- * platform/chromium/PlatformBridge.h: |
- * platform/graphics/chromium/FontPlatformData.h: |
- * platform/graphics/skia/FontCustomPlatformData.cpp: |
- * platform/graphics/skia/FontCustomPlatformData.h: |
- |
-2011-04-26 Ryosuke Niwa <rniwa@webkit.org> |
- |
- Reviewed by Dan Bernstein. |
- |
- [RTL] Arabic/AB - after typing a date, cursors doesn't go back |
- https://bugs.webkit.org/show_bug.cgi?id=49111 |
- |
- Even when the offset corresponds to a position visually left of the box |
- and there aren't any inline boxes on the left, the previous/next position |
- may still correspond to some position in the same line. |
- |
- The bug was caused by our ignoring such cases. Fixed the bug by comparing |
- previous/next position's inline box to the current box. If they match, |
- then we stay on the same position because moving to the left visually at |
- the left edge should not result in a position on the same line. |
- |
- Also fixed a bug that WebKit uses offsets that are not extrema when moved to |
- the left edge or to the right edge, and a bug that WebKit could not move to |
- the left from 12^3 CBA abc to 123 C^BA abc (there is no offset between 3 and C). |
- |
- Test cases are added to editing/selection/move-left-right.html |
- |
- * editing/VisiblePosition.cpp: |
- (WebCore::VisiblePosition::leftVisuallyDistinctCandidate): |
- (WebCore::VisiblePosition::rightVisuallyDistinctCandidate): |
- |
-2011-04-26 Gabor Loki <loki@webkit.org> |
- |
- Reviewed by Csaba Osztrogonác. |
- |
- Speeding up SVG filters with multicore (SMP) support |
- https://bugs.webkit.org/show_bug.cgi?id=43903 |
- |
- Some SVG filters execute a huge number of pixel manipulations, which |
- cannot be sped up by graphics accelerators, since their algorithm is |
- too complex. Using the power of Symmetric Multi Processing (SMP) we |
- can split up a task to smaller (data independent) tasks, which can be |
- executed independently. |
- |
- The ParallelJobs framework provides a simple way for distributed |
- programming. The framework is based on WebKit's threading infrastructure, |
- Open Multi-Processing's (OpenMP) API, and libdispatch API. |
- |
- * ForwardingHeaders/wtf/ParallelJobs.h: Added. |
- * platform/graphics/filters/FETurbulence.cpp: |
- (WebCore::FETurbulence::PaintingData::PaintingData): |
- (WebCore::FETurbulence::noise2D): |
- (WebCore::FETurbulence::calculateTurbulenceValueForPoint): |
- (WebCore::FETurbulence::fillRegion): |
- (WebCore::FETurbulence::fillRegionWorker): |
- (WebCore::FETurbulence::apply): |
- * platform/graphics/filters/FETurbulence.h: |
- |
-2011-04-26 Pavel Feldman <pfeldman@google.com> |
- |
- Reviewed by Yury Semikhatsky. |
- |
- Web Inspector: close button is off in docked mode on a mac. |
- https://bugs.webkit.org/show_bug.cgi?id=59453 |
- |
- * inspector/front-end/inspector.css: |
- (.toolbar-item.close-left): |
- |
-2011-04-26 Pavel Feldman <pfeldman@google.com> |
- |
- Reviewed by Yury Semikhatsky. |
- |
- Web Inspector: New Style is not working (UI glitch) |
- https://bugs.webkit.org/show_bug.cgi?id=59451 |
- |
- * inspector/InspectorDOMStorageAgent.cpp: |
- (WebCore::InspectorDOMStorageAgent::setFrontend): |
- * inspector/front-end/AuditRules.js: |
- (WebInspector.AuditRules.ImageDimensionsRule.prototype.doRun.getStyles): |
- (WebInspector.AuditRules.CssInHeadRule.prototype.doRun.externalStylesheetsReceived): |
- (WebInspector.AuditRules.CssInHeadRule.prototype.doRun.inlineStylesReceived): |
- (WebInspector.AuditRules.StylesScriptsOrderRule.prototype.doRun.cssBeforeInlineReceived): |
- (WebInspector.AuditRules.StylesScriptsOrderRule.prototype.doRun.lateStylesReceived): |
- * inspector/front-end/CSSStyleModel.js: |
- (WebInspector.CSSStyleModel.prototype.setRuleSelector): |
- (WebInspector.CSSStyleModel.prototype.setRuleSelector.callback): |
- (WebInspector.CSSStyleModel.prototype.addRule): |
- (WebInspector.CSSStyleModel.prototype.addRule.callback): |
- (WebInspector.CSSStyleModel.prototype._documentElementId): |
- * inspector/front-end/DOMAgent.js: |
- (WebInspector.DOMNode.prototype._addAttribute): |
- (WebInspector.DOMNode.prototype.documentElement): |
- * inspector/front-end/ResourceTreeModel.js: |
- (WebInspector.PageDispatcher.prototype.frameDetached): |
- * inspector/front-end/StylesSidebarPane.js: |
- (WebInspector.StylesSidebarPane.prototype._createNewRule): |
- |
-2011-04-26 Pavel Feldman <pfeldman@google.com> |
- |
- Reviewed by Yury Semikhatsky. |
- |
- Web Inspector: introduce DOMStorage::enable and disable. |
- https://bugs.webkit.org/show_bug.cgi?id=59445 |
- |
- * inspector/Inspector.json: |
- * inspector/InspectorAgent.cpp: |
- (WebCore::InspectorAgent::InspectorAgent): |
- (WebCore::InspectorAgent::restoreInspectorStateFromCookie): |
- * inspector/InspectorDOMStorageAgent.cpp: |
- (WebCore::InspectorDOMStorageAgent::InspectorDOMStorageAgent): |
- (WebCore::InspectorDOMStorageAgent::restore): |
- (WebCore::InspectorDOMStorageAgent::enable): |
- (WebCore::InspectorDOMStorageAgent::disable): |
- * inspector/InspectorDOMStorageAgent.h: |
- (WebCore::InspectorDOMStorageAgent::create): |
- * inspector/front-end/DOMStorage.js: |
- (WebInspector.DOMStorageDispatcher.prototype.addDOMStorage): |
- * inspector/front-end/ResourcesPanel.js: |
- (WebInspector.ResourcesPanel.prototype._reset): |
- * inspector/front-end/inspector.js: |
- |
-2011-04-26 Pavel Feldman <pfeldman@google.com> |
- |
- Reviewed by Yury Semikhatsky. |
- |
- Web Inspector: extract Page agent dispatcher into the ResourceTreeModel. |
- https://bugs.webkit.org/show_bug.cgi?id=59440 |
- |
- * inspector/Inspector.json: |
- * inspector/InspectorAgent.cpp: |
- (WebCore::InspectorAgent::restoreInspectorStateFromCookie): |
- * inspector/InspectorPageAgent.cpp: |
- (WebCore::InspectorPageAgent::setFrontend): |
- (WebCore::InspectorPageAgent::frameNavigated): |
- * inspector/InspectorPageAgent.h: |
- * inspector/front-end/AuditsPanel.js: |
- (WebInspector.AuditsPanel): |
- (WebInspector.AuditsPanel.prototype._onLoadEventFired): |
- (WebInspector.AuditsPanel.prototype._domContentLoadedEventFired): |
- * inspector/front-end/CSSStyleModel.js: |
- (WebInspector.CSSStyleModelResourceBinding): |
- (WebInspector.CSSStyleModelResourceBinding.prototype._inspectedURLChanged): |
- * inspector/front-end/DOMBreakpointsSidebarPane.js: |
- (WebInspector.DOMBreakpointsSidebarPane): |
- (WebInspector.DOMBreakpointsSidebarPane.prototype._inspectedURLChanged): |
- * inspector/front-end/DOMStorage.js: |
- (WebInspector.DOMStorageDispatcher.prototype.addDOMStorage): |
- * inspector/front-end/ExtensionServer.js: |
- (WebInspector.ExtensionServer.prototype._inspectedURLChanged): |
- (WebInspector.ExtensionServer.prototype.initExtensions): |
- * inspector/front-end/NetworkPanel.js: |
- (WebInspector.NetworkPanel): |
- (WebInspector.NetworkPanel.prototype._onLoadEventFired): |
- (WebInspector.NetworkPanel.prototype._domContentLoadedEventFired): |
- (WebInspector.NetworkPanel.prototype._frameNavigated): |
- * inspector/front-end/ResourceTreeModel.js: |
- (WebInspector.ResourceTreeModel): |
- (WebInspector.ResourceTreeModel.prototype._processCachedResources): |
- (WebInspector.ResourceTreeModel.prototype._dispatchInspectedURLChanged): |
- (WebInspector.ResourceTreeModel.prototype._frameNavigated): |
- (WebInspector.ResourceTreeModel.prototype._frameDetached): |
- (WebInspector.ResourceTreeModel.prototype._clearChildFramesAndResources): |
- (WebInspector.PageDispatcher): |
- (WebInspector.PageDispatcher.prototype.domContentEventFired): |
- (WebInspector.PageDispatcher.prototype.loadEventFired): |
- (WebInspector.PageDispatcher.prototype.frameNavigated): |
- (WebInspector.PageDispatcher.prototype.frameDetached): |
- * inspector/front-end/ResourcesPanel.js: |
- (WebInspector.ResourcesPanel): |
- (WebInspector.ResourcesPanel.prototype._onLoadEventFired): |
- (WebInspector.ResourcesPanel.prototype._frameNavigated): |
- * inspector/front-end/inspector.js: |
- |
-2011-04-26 Csaba Osztrogonác <ossy@webkit.org> |
- |
- Unreviewed rollout r84877 and StyleRareInheritedData.cpp changes of r84892, |
- because it broke transitions/multiple-text-shadow-transition.html |
- |
- Fix OwnPtr strict errors in RenderStyle and make StyleRareInheritedData::textShadow an OwnPtr |
- https://bugs.webkit.org/show_bug.cgi?id=59377 |
- |
- * css/CSSStyleSelector.cpp: |
- (WebCore::CSSStyleSelector::applyProperty): |
- * page/animation/AnimationBase.cpp: |
- (WebCore::PropertyWrapperShadow::PropertyWrapperShadow): |
- (WebCore::PropertyWrapperShadow::blend): |
- * rendering/style/RenderStyle.cpp: |
- (WebCore::RenderStyle::setTextShadow): |
- (WebCore::RenderStyle::setBoxShadow): |
- * rendering/style/RenderStyle.h: |
- (WebCore::InheritedFlags::textShadow): |
- * rendering/style/StyleRareInheritedData.cpp: |
- (WebCore::StyleRareInheritedData::StyleRareInheritedData): |
- (WebCore::StyleRareInheritedData::~StyleRareInheritedData): |
- * rendering/style/StyleRareInheritedData.h: |
- * rendering/style/StyleRareNonInheritedData.cpp: |
- (WebCore::StyleRareNonInheritedData::StyleRareNonInheritedData): |
- |
-2011-04-26 Mihai Parparita <mihaip@chromium.org> |
- |
- Reviewed by Adam Barth. |
- |
- Turn off make built-in implicit rules for derived sources makefile |
- https://bugs.webkit.org/show_bug.cgi?id=59418 |
- |
- We don't use any of make's built-in implicit rules, turning them off |
- speeds up parsing of the makefile. |
- |
- * WebCore.xcodeproj/project.pbxproj: |
- * gyp/generate-derived-sources.sh: |
- |
-2011-04-26 Leo Yang <leo.yang@torchmobile.com.cn> |
- |
- Reviewed by Nikolas Zimmermann. |
- |
- ASSERT failure in SVGUseElement |
- https://bugs.webkit.org/show_bug.cgi?id=59313 |
- |
- In SVGUseElement::insertedIntoDocument(), ASSERT(!m_isPendingResource) |
- was wrong because the document may not be well-formed. |
- |
- This patch asserts the element is not pending on resource or the |
- document is not well-formed. |
- |
- Test: svg/custom/use-crash-in-non-wellformed-document.svg |
- |
- * svg/SVGUseElement.cpp: |
- (WebCore::isWellFormedDocument): |
- (WebCore::SVGUseElement::insertedIntoDocument): |
- |
-2011-04-26 Mikhail Naganov <mnaganov@chromium.org> |
- |
- Reviewed by Pavel Feldman. |
- |
- Web Inspector: [Chromium] Fix columns resizing in the detailed heap snapshot grids. |
- https://bugs.webkit.org/show_bug.cgi?id=59438 |
- |
- * inspector/front-end/DetailedHeapshotView.js: |
- (WebInspector.DetailedHeapshotView.prototype._changeView): |
- |
-2011-04-26 David Levin <levin@chromium.org> |
- |
- Reviewed by Adam Barth. |
- |
- Fix a few OwnPtr strict errors in WebCore headers. |
- https://bugs.webkit.org/show_bug.cgi?id=59431 |
- |
- * bindings/js/ScheduledAction.cpp: |
- (WebCore::ScheduledAction::create): |
- * css/MediaQueryMatcher.cpp: |
- (WebCore::MediaQueryMatcher::prepareEvaluator): |
- * dom/NodeRareData.h: |
- (WebCore::NodeListsNodeData::create): |
- * dom/ScriptRunner.h: |
- (WebCore::ScriptRunner::create): |
- * inspector/WorkerInspectorController.cpp: |
- (WebCore::WorkerInspectorController::connectFrontend): |
- * loader/icon/IconDatabase.h: |
- (WebCore::IconDatabase::create): |
- * platform/graphics/SimpleFontData.cpp: |
- (WebCore::SimpleFontData::verticalRightOrientationFontData): |
- (WebCore::SimpleFontData::uprightOrientationFontData): |
- (WebCore::SimpleFontData::brokenIdeographFontData): |
- * rendering/style/StyleRareInheritedData.cpp: |
- (WebCore::StyleRareInheritedData::StyleRareInheritedData): |
- * rendering/svg/SVGResources.h: |
- (WebCore::SVGResources::ClipperFilterMaskerData::create): |
- (WebCore::SVGResources::MarkerData::create): |
- (WebCore::SVGResources::FillStrokeData::create): |
- * storage/StorageSyncManager.cpp: |
- (WebCore::StorageSyncManager::close): |
- * workers/WorkerContext.cpp: |
- (WebCore::CloseWorkerContextTask::create): |
- * workers/WorkerMessagingProxy.cpp: |
- (WebCore::MessageWorkerContextTask::create): |
- (WebCore::MessageWorkerTask::create): |
- (WebCore::WorkerExceptionTask::create): |
- (WebCore::WorkerContextDestroyedTask::create): |
- (WebCore::WorkerTerminateTask::create): |
- (WebCore::WorkerThreadActivityReportTask::create): |
- * workers/WorkerRunLoop.cpp: |
- (WebCore::WorkerRunLoop::Task::create): |
- * workers/WorkerThread.cpp: |
- (WebCore::WorkerThreadStartupData::create): |
- (WebCore::WorkerThreadShutdownFinishTask::create): |
- (WebCore::WorkerThreadShutdownStartTask::create): |
- |
-2011-04-26 Adam Klein <adamk@chromium.org> |
- |
- Reviewed by Adam Barth. |
- |
- Fix last strict OwnPtr violation under svg/... |
- https://bugs.webkit.org/show_bug.cgi?id=59429 |
- |
- * svg/SVGDocumentExtensions.cpp: |
- (WebCore::SVGDocumentExtensions::removePendingResource): |
- |
-2011-04-26 Adam Barth <abarth@webkit.org> |
- |
- Reviewed by David Levin. |
- |
- Fix more strict OwnPtr violations in WebCore |
- https://bugs.webkit.org/show_bug.cgi?id=59433 |
- |
- These manifest in the Chromium build. |
- |
- * bindings/v8/ScriptController.cpp: |
- (WebCore::ScriptController::ScriptController): |
- * bindings/v8/ScriptDebugServer.cpp: |
- (WebCore::ScriptDebugServer::editScriptSource): |
- * bindings/v8/V8DOMMap.cpp: |
- (WebCore::DOMDataStoreHandle::DOMDataStoreHandle): |
- * bindings/v8/V8Proxy.cpp: |
- (WebCore::V8Proxy::precompileScript): |
- * bindings/v8/WorkerScriptController.cpp: |
- (WebCore::WorkerScriptController::WorkerScriptController): |
- * page/Page.cpp: |
- (WebCore::Page::Page): |
- (WebCore::Page::speechInput): |
- * platform/PlatformGestureRecognizer.cpp: |
- (WebCore::PlatformGestureRecognizer::create): |
- * platform/PlatformGestureRecognizer.h: |
- * platform/graphics/chromium/DrawingBufferChromium.cpp: |
- (WebCore::DrawingBuffer::DrawingBuffer): |
- * platform/graphics/chromium/cc/CCLayerImpl.cpp: |
- (WebCore::CCLayerImpl::CCLayerImpl): |
- * platform/graphics/gpu/Texture.cpp: |
- (WebCore::Texture::create): |
- * platform/image-decoders/bmp/BMPImageDecoder.cpp: |
- (WebCore::BMPImageDecoder::decodeHelper): |
- * platform/image-decoders/gif/GIFImageDecoder.cpp: |
- (WebCore::GIFImageDecoder::decode): |
- * platform/image-decoders/ico/ICOImageDecoder.cpp: |
- (WebCore::ICOImageDecoder::decodeAtIndex): |
- * platform/image-decoders/jpeg/JPEGImageDecoder.cpp: |
- (WebCore::JPEGImageDecoder::decode): |
- * platform/image-decoders/png/PNGImageDecoder.cpp: |
- (WebCore::PNGImageDecoder::decode): |
- * storage/chromium/DatabaseTrackerChromium.cpp: |
- (WebCore::DatabaseTracker::addOpenDatabase): |
- |
-2011-04-25 David Levin <levin@chromium.org> |
- |
- Reviewed by Adam Barth. |
- |
- Fix OwnPtr strict errors in CrossThreadTask.h |
- https://bugs.webkit.org/show_bug.cgi?id=59427 |
- |
- * dom/CrossThreadTask.h: |
- (WebCore::CrossThreadTask1::create): |
- (WebCore::CrossThreadTask2::create): |
- (WebCore::CrossThreadTask3::create): |
- (WebCore::CrossThreadTask4::create): |
- (WebCore::CrossThreadTask5::create): |
- (WebCore::CrossThreadTask6::create): |
- (WebCore::CrossThreadTask7::create): |
- (WebCore::CrossThreadTask8::create): |
- |
-2011-04-25 Mark Rowe <mrowe@apple.com> |
- |
- Reviewed by Dan Bernstein. |
- |
- <rdar://problem/9330337> Leak of 'WebCore::ApplyPropertyFillLayer<WebCore::FillSize>' |
- |
- * css/CSSStyleApplyProperty.cpp: |
- (WebCore::CSSStyleApplyProperty::CSSStyleApplyProperty): Don't initialize CSSPropertyWebkitMaskSize twice. |
- * css/CSSStyleApplyProperty.h: |
- (WebCore::CSSStyleApplyProperty::setPropertyValue): Add some asserts to catch this class of leak. |
- |
-2011-04-25 Levi Weintraub <leviw@chromium.org> |
- |
- Reviewed by Ryosuke Niwa. |
- |
- REGRESSION: a character appears after tab span when typing immediately before the tab span |
- https://bugs.webkit.org/show_bug.cgi?id=58132 |
- |
- Correcting the order of operations we use to avoid inserting into invalid positions in |
- ReplaceSelectionCommand to avoid inserting into tab-spans. |
- |
- Test: editing/pasteboard/paste-before-tab-span.html |
- |
- * editing/ReplaceSelectionCommand.cpp: |
- (WebCore::ReplaceSelectionCommand::doApply): Moving positionOutsideTabSpan to after |
- positionAvoidingPrecedingNodes, since that function could once again put our position |
- inside a tab span. |
- |
-2011-04-25 David Levin <levin@chromium.org> |
- |
- Reviewed by Adam Barth. |
- |
- Fix OwnPtr strict error in ImageBuffer.h |
- https://bugs.webkit.org/show_bug.cgi?id=59422 |
- |
- * platform/graphics/ImageBuffer.h: |
- (WebCore::ImageBuffer::create): |
- |
-2011-04-25 Adam Barth <abarth@webkit.org> |
- |
- Reviewed by David Levin. |
- |
- Fix strict OwnPtr violations and whitelist known-tricky files |
- https://bugs.webkit.org/show_bug.cgi?id=59421 |
- |
- With this patch, WebCore builds cleanly with strict OwnPtrs. |
- |
- * dom/MessagePort.cpp: |
- (WebCore::MessagePort::MessagePort): |
- * dom/XMLDocumentParserLibxml2.cpp: |
- * page/ContextMenuController.cpp: |
- (WebCore::ContextMenuController::ContextMenuController): |
- (WebCore::ContextMenuController::clearContextMenu): |
- * page/PluginHalter.cpp: |
- * platform/graphics/MediaPlayer.cpp: |
- * platform/network/ResourceRequestBase.cpp: |
- (WebCore::ResourceRequestBase::adopt): |
- (WebCore::ResourceRequestBase::copyData): |
- * svg/SVGDocumentExtensions.cpp: |
- |
-2011-04-25 David Levin <levin@chromium.org> |
- |
- Reviewed by Adam Barth. |
- |
- Fix OwnPtr strict errors in DatasetDOMStringMap.h. |
- https://bugs.webkit.org/show_bug.cgi?id=59419 |
- |
- * dom/DatasetDOMStringMap.h: |
- (WebCore::DatasetDOMStringMap::create): |
- |
-2011-04-25 Jocelyn Turcotte <jocelyn.turcotte@nokia.com> |
- |
- Reviewed by Eric Seidel. |
- |
- KeyboardEvent.cpp should work with strict OwnPtrs. |
- https://bugs.webkit.org/show_bug.cgi?id=59403 |
- |
- * dom/KeyboardEvent.cpp: |
- (WebCore::KeyboardEvent::KeyboardEvent): |
- |
-2011-04-25 James Robinson <jamesr@chromium.org> |
- |
- Reviewed by Eric Seidel. |
- |
- Fix OwnPtr strict errors in RenderStyle and make StyleRareInheritedData::textShadow an OwnPtr |
- https://bugs.webkit.org/show_bug.cgi?id=59377 |
- |
- * css/CSSStyleSelector.cpp: |
- (WebCore::CSSStyleSelector::applyProperty): |
- * page/animation/AnimationBase.cpp: |
- (WebCore::PropertyWrapperShadow::PropertyWrapperShadow): |
- (WebCore::PropertyWrapperShadow::blend): |
- * rendering/style/RenderStyle.cpp: |
- (WebCore::RenderStyle::setTextShadow): |
- (WebCore::RenderStyle::setBoxShadow): |
- * rendering/style/RenderStyle.h: |
- (WebCore::InheritedFlags::textShadow): |
- * rendering/style/StyleRareInheritedData.cpp: |
- (WebCore::StyleRareInheritedData::StyleRareInheritedData): |
- (WebCore::StyleRareInheritedData::~StyleRareInheritedData): |
- * rendering/style/StyleRareInheritedData.h: |
- * rendering/style/StyleRareNonInheritedData.cpp: |
- (WebCore::StyleRareNonInheritedData::StyleRareNonInheritedData): |
- |
-2011-04-25 Sheriff Bot <webkit.review.bot@gmail.com> |
- |
- Unreviewed, rolling out r84864. |
- http://trac.webkit.org/changeset/84864 |
- https://bugs.webkit.org/show_bug.cgi?id=59413 |
- |
- Expect result of use-crash-in-non-wellformed-document.svg is |
- platform dependent (Requested by leoyang on #webkit). |
- |
- * svg/SVGUseElement.cpp: |
- (WebCore::SVGUseElement::insertedIntoDocument): |
- |
-2011-04-25 Leo Yang <leo.yang@torchmobile.com.cn> |
- |
- Reviewed by Nikolas Zimmermann. |
- |
- ASSERT failure in SVGUseElement |
- https://bugs.webkit.org/show_bug.cgi?id=59313 |
- |
- In SVGUseElement::insertedIntoDocument(), ASSERT(!m_isPendingResource) |
- was wrong because the document may not be well-formed. |
- |
- This patch asserts the element is not pending on resource or the |
- document is not well-formed. |
- |
- Test: svg/custom/use-crash-in-non-wellformed-document.svg |
- |
- * svg/SVGUseElement.cpp: |
- (WebCore::isWellFormedDocument): |
- (WebCore::SVGUseElement::insertedIntoDocument): |
- |
-2011-04-25 Igor Oliveira <igor.oliveira@openbossa.org> |
- |
- Reviewed by Tony Chang. |
- |
- Move complexity from DataTransferItemsChromium and DataTransferItemChromium for base class |
- https://bugs.webkit.org/show_bug.cgi?id=59028 |
- |
- The DataTransferItemsChromium and DataTransferItemChromium code can be reused by other platforms. |
- Moving the code for base class we can reduce the efforts to support DataTransferItems |
- element in different platforms. |
- |
- * Android.mk: |
- * CMakeLists.txt: |
- * GNUmakefile.list.am: |
- * WebCore.gypi: |
- * WebCore.pro: |
- * WebCore.vcproj/WebCore.vcproj: |
- * WebCore.xcodeproj/project.pbxproj: |
- * dom/DataTransferItem.cpp: |
- (WebCore::DataTransferItem::DataTransferItem): |
- (WebCore::DataTransferItem::owner): |
- (WebCore::DataTransferItem::kind): |
- (WebCore::DataTransferItem::type): |
- * dom/DataTransferItem.h: |
- * dom/DataTransferItems.cpp: Copied from Source/WebCore/platform/chromium/DataTransferItemsChromium.cpp. |
- (WebCore::DataTransferItems::DataTransferItems): |
- (WebCore::DataTransferItems::length): |
- (WebCore::DataTransferItems::item): |
- (WebCore::DataTransferItems::deleteItem): |
- (WebCore::DataTransferItems::clear): |
- (WebCore::DataTransferItems::add): |
- * dom/DataTransferItems.h: |
- * platform/chromium/DataTransferItemChromium.cpp: |
- (WebCore::DataTransferItem::create): |
- (WebCore::DataTransferItemChromium::DataTransferItemChromium): |
- (WebCore::DataTransferItemChromium::getAsString): |
- (WebCore::DataTransferItemChromium::getAsFile): |
- * platform/chromium/DataTransferItemChromium.h: |
- * platform/chromium/DataTransferItemsChromium.cpp: |
- (WebCore::DataTransferItemsChromium::DataTransferItemsChromium): |
- * platform/chromium/DataTransferItemsChromium.h: |
- |
-2011-04-25 Chris Marrin <cmarrin@apple.com> |
- |
- Reviewed by Simon Fraser. |
- |
- REGRESSION(75137): directly composited images are double-drawn sometimes |
- https://bugs.webkit.org/show_bug.cgi?id=58632 |
- |
- Depending on the order of operations (layout, creating compositing |
- layers and creating simple image layers) a layer can have both image |
- content and have m_drawsContent=true. In this case 2 copies of the image |
- are drawn in the layer. If the image has alpha, it looks wrong. |
- |
- I added an updateDrawsContent() call after setting the image contents |
- to make sure it's correct. |
- |
- I couldn't create a test case that shows the problem because layerTreeAsText() |
- doesn't emit enough info to show this level of detail. I've opened |
- https://bugs.webkit.org/show_bug.cgi?id=59352 to fix that. |
- |
- * rendering/RenderLayerBacking.cpp: |
- (WebCore::RenderLayerBacking::updateImageContents): |
- |
-2011-04-25 Kenneth Russell <kbr@google.com> |
- |
- Reviewed by David Levin. |
- |
- Fix OwnPtr strict errors in InlineFlowBox.cpp |
- https://bugs.webkit.org/show_bug.cgi?id=59392 |
- |
- * rendering/InlineFlowBox.cpp: |
- (WebCore::InlineFlowBox::setLayoutOverflow): |
- (WebCore::InlineFlowBox::setVisualOverflow): |
- |
-2011-04-25 Kenneth Russell <kbr@google.com> |
- |
- Reviewed by David Levin. |
- |
- Fix OwnPtr strict errors in TransformState.cpp |
- https://bugs.webkit.org/show_bug.cgi?id=59380 |
- |
- * rendering/TransformState.cpp: |
- (WebCore::TransformState::applyTransform): |
- |
-2011-04-25 Adam Barth <abarth@webkit.org> |
- |
- Reviewed by James Robinson. |
- |
- DOMWindow.cpp should work with strict OwnPtrs |
- https://bugs.webkit.org/show_bug.cgi?id=59356 |
- |
- * page/DOMWindow.cpp: |
- (WebCore::PostMessageTimer::fired): |
- (WebCore::DOMWindow::postMessageTimerFired): |
- * page/DOMWindow.h: |
- |
-2011-04-25 Adam Klein <adamk@chromium.org> |
- |
- Reviewed by James Robinson. |
- |
- Fix OwnPtr strict errors in some SVG classes. |
- https://bugs.webkit.org/show_bug.cgi?id=59395 |
- |
- * rendering/style/SVGRenderStyleDefs.cpp: |
- (WebCore::StyleShadowSVGData::StyleShadowSVGData): |
- * svg/SVGSVGElement.cpp: |
- (WebCore::SVGSVGElement::SVGSVGElement): |
- * svg/SVGStyledTransformableElement.cpp: |
- (WebCore::SVGStyledTransformableElement::supplementalTransform): |
- * svg/SVGTextElement.cpp: |
- (WebCore::SVGTextElement::supplementalTransform): |
- * svg/graphics/SVGImage.cpp: |
- (WebCore::SVGImage::dataChanged): |
- |
-2011-04-25 Adam Barth <abarth@webkit.org> |
- |
- Re-land changes to ValidationMessage. |
- |
- * html/ValidationMessage.cpp: |
- (WebCore::ValidationMessage::setMessage): |
- (WebCore::ValidationMessage::setMessageDOMAndStartTimer): |
- (WebCore::ValidationMessage::requestToHideMessage): |
- |
-2011-04-25 Ojan Vafai <ojan@chromium.org> |
- |
- Reviewed by James Robinson. |
- |
- fix OwnPtr strict error in FrameView.cpp |
- https://bugs.webkit.org/show_bug.cgi?id=59402 |
- |
- * page/FrameView.cpp: |
- (WebCore::FrameView::FrameView): |
- (WebCore::FrameView::addWidgetToUpdate): |
- |
-2011-04-25 Adam Klein <adamk@chromium.org> |
- |
- Unreviewed, rolling out r84844. |
- http://trac.webkit.org/changeset/84844 |
- https://bugs.webkit.org/show_bug.cgi?id=59395 |
- |
- Checked in too much. |
- |
- * rendering/style/SVGRenderStyleDefs.cpp: |
- (WebCore::StyleShadowSVGData::StyleShadowSVGData): |
- * svg/SVGDocumentExtensions.cpp: |
- (WebCore::SVGDocumentExtensions::removePendingResource): |
- * svg/SVGSVGElement.cpp: |
- (WebCore::SVGSVGElement::SVGSVGElement): |
- * svg/SVGStyledTransformableElement.cpp: |
- (WebCore::SVGStyledTransformableElement::supplementalTransform): |
- * svg/SVGTextElement.cpp: |
- (WebCore::SVGTextElement::supplementalTransform): |
- * svg/graphics/SVGImage.cpp: |
- (WebCore::SVGImage::dataChanged): |
- |
-2011-04-25 Adrienne Walker <enne@google.com> |
- |
- Reviewed by David Levin. |
- |
- Make ActiveDOMCallback play nice with OwnPtr strict mode |
- https://bugs.webkit.org/show_bug.cgi?id=59398 |
- |
- * bindings/generic/ActiveDOMCallback.cpp: |
- (WebCore::ActiveDOMCallback::ActiveDOMCallback): |
- |
-2011-04-25 Adam Klein <adamk@chromium.org> |
- |
- Reviewed by James Robinson. |
- |
- Fix OwnPtr strict errors in some SVG classes. |
- https://bugs.webkit.org/show_bug.cgi?id=59395 |
- |
- * rendering/style/SVGRenderStyleDefs.cpp: |
- (WebCore::StyleShadowSVGData::StyleShadowSVGData): |
- * svg/SVGSVGElement.cpp: |
- (WebCore::SVGSVGElement::SVGSVGElement): |
- * svg/SVGStyledTransformableElement.cpp: |
- (WebCore::SVGStyledTransformableElement::supplementalTransform): |
- * svg/SVGTextElement.cpp: |
- (WebCore::SVGTextElement::supplementalTransform): |
- * svg/graphics/SVGImage.cpp: |
- (WebCore::SVGImage::dataChanged): |
- |
-2011-04-25 James Robinson <jamesr@chromium.org> |
- |
- Unreviewed, rolling out r84835. |
- http://trac.webkit.org/changeset/84835 |
- https://bugs.webkit.org/show_bug.cgi?id=59394 |
- |
- Breaks compile because Deque<OwnPtr> doesn't work |
- |
- * dom/XMLDocumentParserLibxml2.cpp: |
- (WebCore::PendingCallbacks::~PendingCallbacks): |
- (WebCore::PendingCallbacks::appendStartElementNSCallback): |
- (WebCore::PendingCallbacks::appendEndElementNSCallback): |
- (WebCore::PendingCallbacks::appendCharactersCallback): |
- (WebCore::PendingCallbacks::appendProcessingInstructionCallback): |
- (WebCore::PendingCallbacks::appendCDATABlockCallback): |
- (WebCore::PendingCallbacks::appendCommentCallback): |
- (WebCore::PendingCallbacks::appendInternalSubsetCallback): |
- (WebCore::PendingCallbacks::appendErrorCallback): |
- (WebCore::XMLDocumentParser::XMLDocumentParser): |
- * html/ValidationMessage.cpp: |
- (WebCore::ValidationMessage::setMessage): |
- (WebCore::ValidationMessage::setMessageDOMAndStartTimer): |
- (WebCore::ValidationMessage::requestToHideMessage): |
- |
-2011-04-25 James Robinson <jamesr@chromium.org> |
- |
- Reviewed by Sam Weinig. |
- |
- Fix OwnPtr strict mode violation in ImageBufferCG |
- https://bugs.webkit.org/show_bug.cgi?id=59396 |
- |
- * platform/graphics/cg/ImageBufferCG.cpp: |
- (WebCore::ImageBuffer::ImageBuffer): |
- |
-2011-04-25 Huang Dongsung <luxtella@company100.net> |
- |
- Reviewed by Dirk Schulze. |
- |
- [Skia] Remove an unused local variable in PlatformContextSkia::readbackHardwareToSoftware(). |
- https://bugs.webkit.org/show_bug.cgi?id=59308 |
- |
- An unused local variable is created and even allocates heap memory. |
- |
- * platform/graphics/skia/PlatformContextSkia.cpp: |
- (WebCore::PlatformContextSkia::readbackHardwareToSoftware): |
- |
-2011-04-25 Geoffrey Garen <ggaren@apple.com> |
- |
- Qt build fix: Updated files not used by other ports for rename. |
- |
- * bindings/js/JSTouchCustom.cpp: |
- (WebCore::toJSNewlyCreated): |
- * bindings/js/JSTouchListCustom.cpp: |
- (WebCore::toJSNewlyCreated): |
- |
-2011-04-25 Adrienne Walker <enne@google.com> |
- |
- Reviewed by David Levin. |
- |
- WebCore/css/CSS* files should play nice with strict OwnPtrs |
- https://bugs.webkit.org/show_bug.cgi?id=59382 |
- |
- * css/CSSFontFaceSource.cpp: |
- (WebCore::CSSFontFaceSource::getFontData): |
- * css/CSSParser.cpp: |
- (WebCore::CSSParser::addProperty): |
- (WebCore::CSSParser::parseValue): |
- |
-2011-04-25 Adam Barth <abarth@webkit.org> |
- |
- Reviewed by James Robinson. |
- |
- ValidationMessage and XMLDocumentParserLibxml2 should play nice with strict OwnPtrs |
- https://bugs.webkit.org/show_bug.cgi?id=59394 |
- |
- * dom/XMLDocumentParserLibxml2.cpp: |
- (WebCore::PendingCallbacks::PendingCallbacks): |
- (WebCore::PendingCallbacks::appendStartElementNSCallback): |
- (WebCore::PendingCallbacks::appendEndElementNSCallback): |
- (WebCore::PendingCallbacks::appendCharactersCallback): |
- (WebCore::PendingCallbacks::appendProcessingInstructionCallback): |
- (WebCore::PendingCallbacks::appendCDATABlockCallback): |
- (WebCore::PendingCallbacks::appendCommentCallback): |
- (WebCore::PendingCallbacks::appendInternalSubsetCallback): |
- (WebCore::PendingCallbacks::appendErrorCallback): |
- (WebCore::XMLDocumentParser::XMLDocumentParser): |
- * html/ValidationMessage.cpp: |
- (WebCore::ValidationMessage::setMessage): |
- (WebCore::ValidationMessage::setMessageDOMAndStartTimer): |
- (WebCore::ValidationMessage::requestToHideMessage): |
- |
-2011-04-25 James Robinson <jamesr@chromium.org> |
- |
- Reviewed by Adam Barth. |
- |
- Fix OwnPtr strictness issues in WebCore/inspector/ |
- https://bugs.webkit.org/show_bug.cgi?id=59387 |
- |
- * inspector/InspectorAgent.cpp: |
- (WebCore::InspectorAgent::InspectorAgent): |
- * inspector/InspectorController.cpp: |
- (WebCore::InspectorController::InspectorController): |
- * inspector/InspectorStyleSheet.cpp: |
- (WebCore::InspectorStyleSheet::ensureSourceData): |
- |
-2011-04-25 Geoffrey Garen <ggaren@apple.com> |
- |
- Reviewed by Beth Dakin. |
- |
- Removed XPathNamespace because it was unused |
- https://bugs.webkit.org/show_bug.cgi?id=59381 |
- |
- This allows us to make Node::ownerDocument() non-virtual, though it has |
- the unhappy side-effect of rebuilding the world. |
- |
- * CMakeLists.txt: |
- * GNUmakefile.list.am: |
- * WebCore.gypi: |
- * WebCore.pro: |
- * WebCore.vcproj/WebCore.vcproj: |
- * WebCore.xcodeproj/project.pbxproj: Bye bye dead code. |
- |
- * dom/Node.h: Hello fast code! |
- |
- * xml/XPathNamespace.cpp: Removed. |
- * xml/XPathNamespace.h: Removed. |
- |
-2011-04-25 Adam Barth <abarth@webkit.org> |
- |
- Reviewed by Darin Adler. |
- |
- HistoryItemMac should play nice with OwnPtrs |
- https://bugs.webkit.org/show_bug.cgi?id=59388 |
- |
- * history/mac/HistoryItemMac.mm: |
- (WebCore::HistoryItem::setTransientProperty): |
- |
-2011-04-25 David Levin <levin@chromium.org> |
- |
- Reviewed by Kenneth Russell. |
- |
- Fix strict OwnPtrs in ContextMenuController.cpp |
- https://bugs.webkit.org/show_bug.cgi?id=59375 |
- |
- * page/ContextMenuController.cpp: |
- (WebCore::ContextMenuController::createContextMenu): |
- (WebCore::separatorItem): |
- |
-2011-04-25 Adam Barth <abarth@webkit.org> |
- |
- Reviewed by Darin Adler. |
- |
- CheckedRadioButton should play nice with strict OwnPtr |
- https://bugs.webkit.org/show_bug.cgi?id=59386 |
- |
- * dom/CheckedRadioButtons.cpp: |
- (WebCore::CheckedRadioButtons::addButton): |
- |
-2011-04-25 Adam Barth <abarth@webkit.org> |
- |
- Reviewed by David Levin. |
- |
- Resource*.cpp should play nice with strict OwnPtrs |
- https://bugs.webkit.org/show_bug.cgi?id=59383 |
- |
- * bridge/runtime_method.cpp: |
- (JSC::RuntimeMethod::RuntimeMethod): |
- * platform/network/ResourceHandle.cpp: |
- (WebCore::ResourceHandle::ResourceHandle): |
- * platform/network/ResourceResponseBase.cpp: |
- (WebCore::ResourceResponseBase::adopt): |
- (WebCore::ResourceResponseBase::copyData): |
- |
-2011-04-25 Adam Barth <abarth@webkit.org> |
- |
- Reviewed by David Levin. |
- |
- Page*.cpp and objc_runtime.mm should play nice with strict OwnPtrs |
- https://bugs.webkit.org/show_bug.cgi?id=59374 |
- |
- * bridge/objc/objc_runtime.mm: |
- (JSC::Bindings::callObjCFallbackObject): |
- * page/Page.cpp: |
- (WebCore::Page::Page): |
- (WebCore::Page::initGroup): |
- * page/PageGroup.cpp: |
- (WebCore::PageGroup::addUserScriptToWorld): |
- (WebCore::PageGroup::addUserStyleSheetToWorld): |
- * page/mac/PageMac.cpp: |
- (WebCore::Page::addSchedulePair): |
- |
-2011-04-25 Geoffrey Garen <ggaren@apple.com> |
- |
- Build fix: This time for sure! |
- |
- * dom/make_names.pl: |
- |
-2011-04-25 Geoffrey Garen <ggaren@apple.com> |
- |
- Build fix: Let's edit the script instead of the autogenrated file. That |
- way, our change will stand the test of time. |
- |
- * dom/make_names.pl: |
- |
-2011-04-25 Adam Barth <abarth@webkit.org> |
- |
- Reviewed by David Levin. |
- |
- Fix strict OwnPtrs in HTMLMediaElement and HTTPHeaderMap |
- https://bugs.webkit.org/show_bug.cgi?id=59368 |
- |
- * html/HTMLMediaElement.cpp: |
- (WebCore::HTMLMediaElement::HTMLMediaElement): |
- * platform/network/HTTPHeaderMap.cpp: |
- (WebCore::HTTPHeaderMap::copyData): |
- |
-2011-04-25 James Robinson <jamesr@chromium.org> |
- |
- Reviewed by Adam Barth. |
- |
- Fix strict OwnPtr violations in Render*.cpp |
- https://bugs.webkit.org/show_bug.cgi?id=59361 |
- |
- * rendering/RenderBlock.cpp: |
- (WebCore::RenderBlock::RenderBlock): |
- (WebCore::RenderBlock::finishDelayUpdateScrollInfo): |
- * rendering/RenderBox.cpp: |
- (WebCore::RenderBox::addLayoutOverflow): |
- (WebCore::RenderBox::addVisualOverflow): |
- * rendering/RenderButton.cpp: |
- (WebCore::RenderButton::styleDidChange): |
- * rendering/RenderLayer.cpp: |
- (WebCore::RenderLayer::updateTransform): |
- (WebCore::RenderLayer::paintChildLayerIntoColumns): |
- (WebCore::RenderLayer::hitTestChildLayerColumns): |
- (WebCore::RenderLayer::ensureBacking): |
- * rendering/RenderTable.cpp: |
- (WebCore::RenderTable::styleDidChange): |
- * rendering/RenderView.cpp: |
- (WebCore::RenderView::compositor): |
- * rendering/style/RenderStyle.cpp: |
- (WebCore::RenderStyle::addCachedPseudoStyle): |
- (WebCore::RenderStyle::accessCounterDirectives): |
- (WebCore::RenderStyle::accessAnimations): |
- (WebCore::RenderStyle::accessTransitions): |
- * rendering/svg/RenderSVGResourceFilter.cpp: |
- (WebCore::RenderSVGResourceFilter::applyResource): |
- * rendering/svg/SVGResources.cpp: |
- (WebCore::SVGResources::SVGResources): |
- |
-2011-04-25 Adam Barth <abarth@webkit.org> |
- |
- Attempt to fix the compile. |
- |
- * platform/graphics/MediaPlayer.cpp: |
- (WebCore::MediaPlayer::loadWithNextMediaEngine): |
- |
-2011-04-25 Geoffrey Garen <ggaren@apple.com> |
- |
- Rubber-stamped by Beth Dakin. |
- |
- Merged CREATE_DOM_NODE_WRAPPER and CREATE_DOM_OBJECT_WRAPPER into |
- CREATE_DOM_WRAPPER because there's no meaningful difference between |
- them anymore. |
- |
- * bindings/js/JSCDATASectionCustom.cpp: |
- (WebCore::toJSNewlyCreated): |
- * bindings/js/JSCSSRuleCustom.cpp: |
- (WebCore::toJS): |
- * bindings/js/JSCSSValueCustom.cpp: |
- (WebCore::toJS): |
- * bindings/js/JSDOMBinding.h: |
- * bindings/js/JSDocumentCustom.cpp: |
- (WebCore::toJS): |
- * bindings/js/JSElementCustom.cpp: |
- (WebCore::toJSNewlyCreated): |
- * bindings/js/JSEventCustom.cpp: |
- (WebCore::toJS): |
- * bindings/js/JSHTMLCollectionCustom.cpp: |
- (WebCore::toJS): |
- * bindings/js/JSImageDataCustom.cpp: |
- (WebCore::toJS): |
- * bindings/js/JSNodeCustom.cpp: |
- (WebCore::createWrapperInline): |
- * bindings/js/JSSVGPathSegCustom.cpp: |
- (WebCore::toJS): |
- * bindings/js/JSStyleSheetCustom.cpp: |
- (WebCore::toJS): |
- * bindings/js/JSTextCustom.cpp: |
- (WebCore::toJSNewlyCreated): |
- * bindings/js/JSWebKitCSSMatrixCustom.cpp: |
- (WebCore::JSWebKitCSSMatrixConstructor::constructJSWebKitCSSMatrix): |
- * bindings/js/JSWebSocketCustom.cpp: |
- (WebCore::JSWebSocketConstructor::constructJSWebSocket): |
- * bindings/js/JSXMLHttpRequestCustom.cpp: |
- (WebCore::JSXMLHttpRequestConstructor::constructJSXMLHttpRequest): |
- * bindings/js/JSXSLTProcessorCustom.cpp: |
- (WebCore::JSXSLTProcessorConstructor::constructJSXSLTProcessor): |
- |
-2011-04-25 Adam Barth <abarth@webkit.org> |
- |
- Reviewed by James Robinson. |
- |
- HTMLDocumentParser should play nice with OwnPtrs |
- https://bugs.webkit.org/show_bug.cgi?id=59363 |
- |
- * html/parser/HTMLDocumentParser.cpp: |
- (WebCore::HTMLDocumentParser::pumpTokenizer): |
- |
-2011-04-25 Adam Barth <abarth@webkit.org> |
- |
- Reviewed by James Robinson. |
- |
- Frame.cpp should work with strict OwnPtrs |
- https://bugs.webkit.org/show_bug.cgi?id=59360 |
- |
- * page/Frame.cpp: |
- (WebCore::createRegExpForLabels): |
- |
-2011-04-25 Adam Barth <abarth@webkit.org> |
- |
- Reviewed by David Levin. |
- |
- Fix strict OwnPtr in Element.cpp |
- https://bugs.webkit.org/show_bug.cgi?id=59357 |
- |
- * dom/NodeRareData.h: |
- (WebCore::NodeRareData::ensureEventTargetData): |
- |
-2011-04-25 Adam Barth <abarth@webkit.org> |
- |
- Reviewed by David Levin. |
- |
- Fix strict OwnPtr issues in Media*.cpp |
- https://bugs.webkit.org/show_bug.cgi?id=59354 |
- |
- There's still a tricky case in MediaPlayer.cpp, but we'll worry about |
- that later. |
- |
- * css/MediaQueryMatcher.cpp: |
- (WebCore::MediaQueryMatcher::addListener): |
- * dom/Document.cpp: |
- * platform/graphics/MediaPlayer.cpp: |
- (WebCore::createNullMediaPlayer): |
- |
-2011-04-25 James Robinson <jamesr@chromium.org> |
- |
- Reviewed by David Levin. |
- |
- Fix strict OwnPtr violations in ListHashSet and RenderLayerCompositor |
- https://bugs.webkit.org/show_bug.cgi?id=59353 |
- |
- * rendering/RenderLayerCompositor.cpp: |
- (WebCore::RenderLayerCompositor::RenderLayerCompositor): |
- |
-2011-04-25 Adam Barth <abarth@webkit.org> |
- |
- Reviewed by James Robinson. |
- |
- Worker*.cpp should work with strict OwnPtrs |
- https://bugs.webkit.org/show_bug.cgi?id=59346 |
- |
- * inspector/WorkerInspectorController.cpp: |
- (WebCore::WorkerInspectorController::WorkerInspectorController): |
- * workers/WorkerContext.cpp: |
- (WebCore::WorkerContext::WorkerContext): |
- * workers/WorkerRunLoop.cpp: |
- (WebCore::WorkerRunLoop::WorkerRunLoop): |
- * workers/WorkerScriptLoader.cpp: |
- (WebCore::WorkerScriptLoader::createResourceRequest): |
- |
-2011-04-25 James Robinson <jamesr@chromium.org> |
- |
- Reviewed by Adam Barth. |
- |
- Remove bad use of OwnPtr::set() in IconDatabase.cpp |
- https://bugs.webkit.org/show_bug.cgi?id=59344 |
- |
- Needed for strict OwnPtr compliance. |
- |
- * loader/icon/IconDatabase.cpp: |
- (WebCore::readySQLiteStatement): |
- |
-2011-04-25 Adam Barth <abarth@webkit.org> |
- |
- Reviewed by Maciej Stachowiak. |
- |
- ImageBuffer.h should work with strict OwnPtrs |
- https://bugs.webkit.org/show_bug.cgi?id=59341 |
- |
- This patch is to prepare for the strict OwnPtr hack-a-thon. |
- |
- * platform/graphics/ImageBuffer.h: |
- (WebCore::ImageBuffer::create): |
- |
-2011-04-25 Steve Falkenburg <sfalken@apple.com> |
- |
- Reviewed by Brian Weinstein. |
- |
- WebCore build on Windows should include windows.h in its precompiled header to speed builds |
- https://bugs.webkit.org/show_bug.cgi?id=59339 |
- |
- windows.h is included by several of the underlying wtf headers, including Atomics.h and |
- ThreadingPrimitives.h anyway, so include it in the WebCorePrefix.h to speed up compilation. |
- |
- On my 8 core Mac Pro, this reduces a clean rebuild of debug WebCore from 8 minutes to 7 minutes |
- (a 12% speedup). |
- |
- * WebCorePrefix.h: Include windows.h |
- |
-2011-04-25 Martin Robinson <mrobinson@igalia.com> |
- |
- Reviewed by Xan Lopez. |
- |
- [GTK] Crash in WebCore::FrameView::notifyPageThatContentAreaWillPaint() |
- https://bugs.webkit.org/show_bug.cgi?id=59311 |
- |
- * platform/gtk/MainFrameScrollbarGtk.cpp: |
- (MainFrameScrollbarGtk::attachAdjustment): Before connecting an adjustment to a scrollbar |
- disconnect any lingering signal handlers. This prevents an adjustment from controlling the |
- active ScrollView and some zombie ScrollView. |
- |
-2011-04-25 Geoffrey Garen <ggaren@apple.com> |
- |
- Another shot at fixing the EFL build. |
- |
- * UseJSC.cmake: |
- * bindings/js/JSDOMBinding.cpp: |
- |
-2011-04-25 Geoffrey Garen <ggaren@apple.com> |
- |
- Try to fix EFL build. |
- |
- * UseJSC.cmake: Added a missing file. |
- |
-2011-04-25 Geoffrey Garen <ggaren@apple.com> |
- |
- Reviewed by Oliver Hunt. |
- |
- Removed a use of markDOMObjectWrapper: ActiveDOMObjects, Workers, and MessagePorts |
- https://bugs.webkit.org/show_bug.cgi?id=59333 |
- |
- * WebCore.xcodeproj/project.pbxproj: Added .idl files for easier editing. |
- |
- * bindings/js/JSDOMBinding.cpp: |
- * bindings/js/JSDOMBinding.h: |
- * bindings/js/JSDocumentCustom.cpp: |
- * bindings/js/JSWorkerContextCustom.cpp: |
- (WebCore::JSWorkerContext::visitChildren): Removed now-unused functions. |
- |
- * bindings/scripts/CodeGeneratorJS.pm: Added support for ActiveDOMObject |
- lifetime management. |
- |
- * dom/Document.idl: No need for a custom mark function anymore, since |
- ActiveDOMObjets now manage their own lifetimes. |
- |
- * dom/MessagePort.cpp: |
- (WebCore::MessagePort::hasPendingActivity): Correctly report that we |
- have pending activity when we're remotely entangled, instead of relying |
- on our clients to know this about us and do the check for us. |
- |
- * dom/MessagePort.h: FIXME! |
- |
- * dom/MessagePort.idl: |
- * fileapi/DOMFileSystem.idl: |
- * fileapi/FileReader.idl: |
- * fileapi/FileWriter.idl: |
- * notifications/Notification.idl: |
- * notifications/NotificationCenter.idl: |
- * page/EventSource.idl: |
- * storage/IDBDatabase.idl: |
- * storage/IDBRequest.idl: |
- * storage/IDBTransaction.idl: |
- * webaudio/AudioContext.idl: |
- * websockets/WebSocket.idl: |
- * workers/AbstractWorker.idl: |
- * workers/SharedWorker.idl: |
- * workers/Worker.idl: Opt in to ActiveDOMObject lifetime management. |
- |
- * workers/WorkerContext.cpp: |
- (WebCore::WorkerContext::hasPendingActivity): No need to make up for |
- MessagePort::hasPendingActivity being wrong anymore, since it's now right. |
- |
- * xml/XMLHttpRequest.idl: Opt in to ActiveDOMObject lifetime management. |
- |
-2011-04-25 Yury Semikhatsky <yurys@chromium.org> |
- |
- Reviewed by Pavel Feldman. |
- |
- Web Inspector: worker messages should be routed to corresponding worker front-end |
- https://bugs.webkit.org/show_bug.cgi?id=59323 |
- |
- Added WorkerManager which is responsible for routing messages between |
- Page inspector front-end and Worker inspector front-ends. |
- |
- * WebCore.gypi: |
- * inspector/front-end/WebKit.qrc: |
- * inspector/front-end/WorkerManager.js: Added. |
- (WebInspector.WorkerManager): |
- (WebInspector.WorkerManager.loaded.InspectorFrontendHost.sendMessageToBackend): |
- (WebInspector.WorkerManager.loaded.InspectorFrontendHost.loaded): |
- (WebInspector.WorkerManager.loaded): |
- (WebInspector.WorkerFrontendStub): |
- (WebInspector.WorkerFrontendStub.prototype._receiveMessage): |
- (WebInspector.WorkerDispatcher): |
- (WebInspector.WorkerDispatcher.prototype._receiveMessage): |
- (WebInspector.WorkerDispatcher.prototype.workerCreated): |
- (WebInspector.WorkerDispatcher.prototype.dispatchMessageFromWorker): |
- * inspector/front-end/inspector.html: |
- * inspector/front-end/inspector.js: |
- (WebInspector.loaded): |
- |
-2011-04-25 Annie Sullivan <sullivan@chromium.org> |
- |
- Reviewed by Pavel Feldman. |
- |
- Web Inspector: [Resources panel] Should be easy to copy data. |
- https://bugs.webkit.org/show_bug.cgi?id=45662 |
- |
- Adds context menu items to copy resource location, resource request headers, and resource response headers. |
- |
- * inspector/front-end/NetworkPanel.js: |
- (WebInspector.NetworkPanel.prototype._contextMenu): Add new items to context menu. |
- (WebInspector.NetworkPanel.prototype._exportLocation): Copy resource location to clipboard. |
- (WebInspector.NetworkPanel.prototype._exportRequestHeaders): Copy resource request headers to clipboard. |
- (WebInspector.NetworkPanel.prototype._exportResponseHeaders): Copy resource response headers to clipboard. |
- * inspector/front-end/Resource.js: |
- (WebInspector.Resource.prototype.get requestHeadersText): If _requestHeadersText is undefined, generate it from requestHeaders. |
- (WebInspector.Resource.prototype.get responseHeadersText): If _responseHeadersText is undefined, generate it form responseHeaders. |
- |
-2011-04-25 Alexander Pavlov <apavlov@chromium.org> |
- |
- Reviewed by Yury Semikhatsky. |
- |
- Web Inspector: CSS is parsed improperly when saved in UTF-8 with a BOM |
- https://bugs.webkit.org/show_bug.cgi?id=59322 |
- |
- Use TextResourceDecoder to determine external stylesheet charsets correctly. |
- |
- Test: inspector/styles/parse-utf8-bom.html |
- |
- * inspector/InspectorStyleSheet.cpp: |
- (WebCore::InspectorStyleSheet::originalStyleSheetText): |
- |
-2011-04-25 Pavel Feldman <pfeldman@google.com> |
- |
- Not reviewed: Inspector.json cleanup. |
- |
- * inspector/Inspector.json: |
- |
-2011-04-25 Yury Semikhatsky <yurys@chromium.org> |
- |
- Reviewed by Pavel Feldman. |
- |
- Web Inspector: introduce InspectorWorkerAgent |
- https://bugs.webkit.org/show_bug.cgi?id=59320 |
- |
- InspectorWorkerAgent is responsible for routing inspector messages |
- between worker context inspector agents and corresponding worker |
- inspector front-end. |
- |
- * CMakeLists.txt: |
- * GNUmakefile.list.am: |
- * WebCore.gypi: |
- * WebCore.pro: |
- * WebCore.vcproj/WebCore.vcproj: |
- * WebCore.xcodeproj/project.pbxproj: |
- * inspector/CodeGeneratorInspector.pm: |
- * inspector/Inspector.json: |
- * inspector/InspectorAgent.cpp: |
- (WebCore::InspectorAgent::InspectorAgent): |
- (WebCore::InspectorAgent::setFrontend): |
- (WebCore::InspectorAgent::disconnectFrontend): |
- * inspector/InspectorAgent.h: |
- (WebCore::InspectorAgent::workerAgent): |
- * inspector/InspectorController.cpp: |
- (WebCore::InspectorController::connectFrontend): |
- (WebCore::InspectorController::disconnectFrontend): |
- * inspector/InspectorInstrumentation.cpp: |
- (WebCore::InspectorInstrumentation::didStartWorkerContextImpl): |
- * inspector/InspectorInstrumentation.h: |
- (WebCore::InspectorInstrumentation::willStartWorkerContext): |
- (WebCore::InspectorInstrumentation::didStartWorkerContext): |
- * inspector/InspectorWorkerAgent.cpp: Added. |
- (WebCore::InspectorWorkerAgent::WorkerFrontendChannel::WorkerFrontendChannel): |
- (WebCore::InspectorWorkerAgent::WorkerFrontendChannel::~WorkerFrontendChannel): |
- (WebCore::InspectorWorkerAgent::WorkerFrontendChannel::id): |
- (WebCore::InspectorWorkerAgent::WorkerFrontendChannel::inspectorProxy): |
- (WebCore::InspectorWorkerAgent::WorkerFrontendChannel::sendMessageToFrontend): |
- (WebCore::InspectorWorkerAgent::create): |
- (WebCore::InspectorWorkerAgent::InspectorWorkerAgent): |
- (WebCore::InspectorWorkerAgent::~InspectorWorkerAgent): |
- (WebCore::InspectorWorkerAgent::setFrontend): |
- (WebCore::InspectorWorkerAgent::clearFrontend): |
- (WebCore::InspectorWorkerAgent::dispatchMessageOnWorkerInspector): |
- (WebCore::InspectorWorkerAgent::didStartWorkerContext): |
- * inspector/InspectorWorkerAgent.h: Added. |
- * inspector/InstrumentingAgents.h: |
- (WebCore::InstrumentingAgents::InstrumentingAgents): |
- (WebCore::InstrumentingAgents::inspectorWorkerAgent): |
- (WebCore::InstrumentingAgents::setInspectorWorkerAgent): |
- * inspector/WorkerInspectorController.cpp: |
- (WebCore::WorkerInspectorController::connectFrontend): |
- * inspector/generate-inspector-idl: |
- * workers/Worker.cpp: |
- (WebCore::Worker::notifyFinished): |
- * workers/WorkerContextInspectorProxy.h: Added. Platform-specific transport for inspector |
- messages sent from the inspected page worker agent to the worker context inspector agents. |
- (WebCore::WorkerContextInspectorProxy::connectFrontend): |
- (WebCore::WorkerContextInspectorProxy::disconnectFrontend): |
- (WebCore::WorkerContextInspectorProxy::sendMessageToWorkerContextInspector): |
- (WebCore::WorkerContextInspectorProxy::~WorkerContextInspectorProxy): |
- * workers/WorkerContextProxy.h: |
- (WebCore::WorkerContextProxy::inspectorProxy): |
- |
-2011-04-25 Pavel Feldman <pfeldman@google.com> |
- |
- Reviewed by Yury Semikhatsky. |
- |
- Web Inspector: move Frame and Resource Tree management into the Page agent. |
- https://bugs.webkit.org/show_bug.cgi?id=59321 |
- |
- Network agent should only handle network-related activities. |
- Frame structure should be managed by the Page agent instead. |
- |
- * inspector/Inspector.json: |
- * inspector/InspectorAgent.cpp: |
- (WebCore::InspectorAgent::InspectorAgent): |
- * inspector/InspectorDOMAgent.cpp: |
- (WebCore::InspectorDOMAgent::InspectorDOMAgent): |
- (WebCore::InspectorDOMAgent::setFrontend): |
- (WebCore::InspectorDOMAgent::restore): |
- (WebCore::InspectorDOMAgent::highlightFrame): |
- * inspector/InspectorDOMAgent.h: |
- (WebCore::InspectorDOMAgent::create): |
- * inspector/InspectorInstrumentation.cpp: |
- (WebCore::InspectorInstrumentation::domContentLoadedEventFiredImpl): |
- (WebCore::InspectorInstrumentation::loadEventFiredImpl): |
- (WebCore::InspectorInstrumentation::frameDetachedFromParentImpl): |
- (WebCore::InspectorInstrumentation::didCommitLoadImpl): |
- * inspector/InspectorPageAgent.cpp: |
- (WebCore::InspectorPageAgent::create): |
- (WebCore::InspectorPageAgent::resourceContent): |
- (WebCore::InspectorPageAgent::resourceContentBase64): |
- (WebCore::InspectorPageAgent::resourceData): |
- (WebCore::InspectorPageAgent::cachedResource): |
- (WebCore::InspectorPageAgent::resourceTypeString): |
- (WebCore::InspectorPageAgent::cachedResourceType): |
- (WebCore::InspectorPageAgent::cachedResourceTypeString): |
- (WebCore::InspectorPageAgent::InspectorPageAgent): |
- (WebCore::InspectorPageAgent::setFrontend): |
- (WebCore::InspectorPageAgent::reload): |
- (WebCore::InspectorPageAgent::open): |
- (WebCore::InspectorPageAgent::getCookies): |
- (WebCore::InspectorPageAgent::deleteCookie): |
- (WebCore::InspectorPageAgent::getResourceTree): |
- (WebCore::InspectorPageAgent::getResourceContent): |
- (WebCore::InspectorPageAgent::restore): |
- (WebCore::InspectorPageAgent::domContentEventFired): |
- (WebCore::InspectorPageAgent::loadEventFired): |
- (WebCore::InspectorPageAgent::frameNavigated): |
- (WebCore::InspectorPageAgent::frameDetached): |
- (WebCore::InspectorPageAgent::didClearWindowObjectInWorld): |
- (WebCore::InspectorPageAgent::mainFrame): |
- (WebCore::pointerAsId): |
- (WebCore::InspectorPageAgent::frameForId): |
- (WebCore::InspectorPageAgent::frameId): |
- (WebCore::InspectorPageAgent::loaderId): |
- (WebCore::InspectorPageAgent::buildObjectForFrame): |
- (WebCore::InspectorPageAgent::buildObjectForFrameTree): |
- * inspector/InspectorPageAgent.h: |
- * inspector/InspectorResourceAgent.cpp: |
- (WebCore::buildObjectForCachedResource): |
- (WebCore::InspectorResourceAgent::willSendRequest): |
- (WebCore::InspectorResourceAgent::didReceiveResponse): |
- (WebCore::InspectorResourceAgent::didLoadResourceFromMemoryCache): |
- (WebCore::InspectorResourceAgent::setInitialScriptContent): |
- (WebCore::InspectorResourceAgent::setInitialXHRContent): |
- (WebCore::InspectorResourceAgent::InspectorResourceAgent): |
- * inspector/InspectorResourceAgent.h: |
- (WebCore::InspectorResourceAgent::create): |
- * inspector/InspectorStyleSheet.cpp: |
- (WebCore::InspectorStyleSheet::resourceStyleSheetText): |
- * inspector/front-end/NetworkManager.js: |
- (WebInspector.NetworkManager.prototype.requestContent): |
- * inspector/front-end/NetworkPanel.js: |
- (WebInspector.NetworkPanel): |
- (WebInspector.NetworkPanel.prototype.frameNavigated): |
- * inspector/front-end/ResourceTreeModel.js: |
- (WebInspector.ResourceTreeModel): |
- (WebInspector.ResourceTreeModel.prototype.frontendReused): |
- (WebInspector.ResourceTreeModel.prototype.frameNavigated): |
- (WebInspector.ResourceTreeModel.prototype.frameDetached): |
- * inspector/front-end/inspector.js: |
- (WebInspector.domContentEventFired): |
- (WebInspector.loadEventFired): |
- (WebInspector.frameNavigated): |
- (WebInspector.frameDetached): |
- |
-2011-04-25 Pavel Feldman <pfeldman@google.com> |
- |
- Reviewed by Yury Semikhatsky. |
- |
- Web Inspector: few protocol improvements. |
- https://bugs.webkit.org/show_bug.cgi?id=59319 |
- |
- - Page domain is documented |
- - setUserAgentOverride is moved to the Network agent |
- - setSearchingForNode -> setInspectModeEnabled |
- - highlightDOMNode -> highlightNode |
- - reloadPage -> reload |
- - openInNewWindow -> open with optional parameter |
- |
- * inspector/Inspector.json: |
- * inspector/InspectorDOMAgent.cpp: |
- (WebCore::InspectorDOMAgent::setInspectModeEnabled): |
- (WebCore::InspectorDOMAgent::highlightNode): |
- * inspector/InspectorDOMAgent.h: |
- (WebCore::InspectorDOMAgent::hideNodeHighlight): |
- * inspector/InspectorInstrumentation.cpp: |
- (WebCore::InspectorInstrumentation::applyUserAgentOverrideImpl): |
- * inspector/InspectorPageAgent.cpp: |
- (WebCore::InspectorPageAgent::clearFrontend): |
- (WebCore::InspectorPageAgent::reload): |
- (WebCore::InspectorPageAgent::open): |
- * inspector/InspectorPageAgent.h: |
- * inspector/InspectorResourceAgent.cpp: |
- (WebCore::InspectorResourceAgent::clearFrontend): |
- (WebCore::InspectorResourceAgent::applyUserAgentOverride): |
- (WebCore::InspectorResourceAgent::setUserAgentOverride): |
- * inspector/InspectorResourceAgent.h: |
- * inspector/front-end/AuditsPanel.js: |
- (WebInspector.AuditsPanel.prototype._reloadResources): |
- * inspector/front-end/ElementsPanel.js: |
- (WebInspector.ElementsPanel.prototype.setSearchingForNode): |
- * inspector/front-end/ExtensionServer.js: |
- (WebInspector.ExtensionServer.prototype._onReload): |
- * inspector/front-end/NetworkPanel.js: |
- (WebInspector.NetworkDataGridNode.prototype._openInNewTab): |
- * inspector/front-end/ResourcesPanel.js: |
- (WebInspector.FrameResourceTreeElement.prototype.ondblclick): |
- * inspector/front-end/inspector.js: |
- (WebInspector.highlightDOMNode): |
- (WebInspector.openResource): |
- (WebInspector.documentKeyDown): |
- |
-2011-04-25 Jon Lee <jonlee@apple.com> |
- |
- Reviewed by Maciej Stachowiak. |
- |
- Overlay scroller hard to see on pages with dark background (59183) |
- https://bugs.webkit.org/show_bug.cgi?id=59183 |
- <rdar://problem/8975367> |
- |
- Switch the scrollbar's overlay style depending on its frame's background color. |
- This refactors the getDocumentBackgroundColor method needed for gestures. The style |
- is determined and set on every paint() call to the Mac scrollbar theme. |
- |
- * WebCore.exp.in: adding method to allow changing style |
- * page/Frame.cpp: |
- (WebCore::Frame::getDocumentBackgroundColor): moving code from WebFrame for reuse by FrameView |
- * page/Frame.h: |
- * page/FrameView.cpp: |
- (WebCore::FrameView::recommendedScrollbarOverlayStyle): overridden to suggest a style based on CSS background color |
- * page/FrameView.h: |
- * platform/ScrollTypes.h: new enum to represent different overlay scrollbar styles |
- * platform/ScrollableArea.h: |
- (WebCore::ScrollableArea::recommendedScrollbarOverlayStyle): new virtual function to return a suggested overlay style |
- * platform/mac/ScrollbarThemeMac.mm: |
- (WebCore::ScrollbarThemeMac::paint): |
- * platform/mac/WebCoreSystemInterface.h: |
- * platform/mac/WebCoreSystemInterface.mm: |
- |
-2011-04-25 Dan Bernstein <mitz@apple.com> |
- |
- Reviewed by Maciej Stachowiak. |
- |
- <rdar://problem/9113516> REGRESSION (WK2): Holding down Option while dragging scrollbar thumb has no effect (should make scroll distance equal mouse translation) |
- https://bugs.webkit.org/show_bug.cgi?id=59315 |
- |
- * platform/Scrollbar.cpp: |
- (WebCore::Scrollbar::Scrollbar): Initialize new member variables. |
- (WebCore::Scrollbar::moveThumb): Added a boolean draggingDocument parameter. When true, the document |
- is scrolled a distance equal to the change in the mouse position. |
- (WebCore::Scrollbar::mouseMoved): Check with the theme whether the mouse move event should drag |
- the document rather than the thumb, and pass the result to moveThumb. |
- (WebCore::Scrollbar::mouseUp): Reset m_draggingDocument. |
- * platform/Scrollbar.h: |
- * platform/ScrollbarTheme.h: |
- (WebCore::ScrollbarTheme::shouldDragDocumentInsteadOfThumb): Added. The base class implementation |
- returns false. |
- * platform/mac/ScrollbarThemeMac.h: |
- * platform/mac/ScrollbarThemeMac.mm: |
- (WebCore::ScrollbarThemeMac::shouldDragDocumentInsteadOfThumb): Overridden to return true if the |
- Option key is down. |
- |
-2011-04-24 Geoffrey Garen <ggaren@apple.com> |
- |
- Reviewed by Sam Weinig. |
- |
- Removed a use of markDOMObjectWrapper: WebGL, XHR, workers; plus, more autogeneration |
- https://bugs.webkit.org/show_bug.cgi?id=59307 |
- |
- * WebCore.xcodeproj/project.pbxproj: Added Blob.idl, so it's easier to edit. |
- |
- * bindings/js/JSCSSRuleCustom.cpp: |
- * bindings/js/JSCSSStyleDeclarationCustom.cpp: |
- * bindings/js/JSCanvasRenderingContextCustom.cpp: |
- * bindings/js/JSDOMApplicationCacheCustom.cpp: Autogenerate these instead |
- of hand-coding them. |
- |
- * bindings/js/JSDOMBinding.h: |
- (WebCore::root): Moved some GC helpers here from JSNodeCustom.h, because |
- they're reasonably generic. |
- |
- * bindings/js/JSDOMImplementationCustom.cpp: |
- * bindings/js/JSDOMStringMapCustom.cpp: |
- * bindings/js/JSDOMTokenListCustom.cpp: |
- * bindings/js/JSHTMLCollectionCustom.cpp: |
- * bindings/js/JSMediaListCustom.cpp: Autogenerate these instead of |
- hand-coding them. |
- |
- * bindings/js/JSMessageChannelCustom.cpp: |
- (WebCore::JSMessageChannel::visitChildren): |
- * bindings/js/JSMessagePortCustom.cpp: |
- (WebCore::JSMessagePort::visitChildren): Use addOpaqueRoot instead of |
- markDOMObjectWrapper. This is one of the few cases where a DOM object is |
- considered a stand-alone root. It's not as efficient as the shared root |
- case, but workers and message ports are very rare objects, so it's no |
- big deal. |
- |
- * bindings/js/JSNamedNodeMapCustom.cpp: Autogenerate! |
- |
- * bindings/js/JSNodeCustom.h: Moved to JSDOMBinding.h. |
- |
- * bindings/js/JSSharedWorkerCustom.cpp: |
- (WebCore::JSSharedWorker::visitChildren): Use addOpaqueRoot instead of |
- markDOMObjectWrapper, as above. Once again, a stand-alone root. |
- |
- * bindings/js/JSStyleSheetCustom.cpp: |
- * bindings/js/JSStyleSheetListCustom.cpp: Autogenerate! |
- |
- * bindings/js/JSWebGLRenderingContextCustom.cpp: |
- (WebCore::JSWebGLRenderingContext::visitChildren): Use the opaque roots |
- system for marking WebGL contexts and their associated satellite objects. |
- |
- * bindings/js/JSWorkerContextCustom.cpp: |
- (WebCore::JSWorkerContext::visitChildren): |
- * bindings/js/JSXMLHttpRequestCustom.cpp: Use addOpaqueRoot instead of |
- markDOMObjectWrapper, as above. Once again, stand-alone roots. |
- |
- (WebCore::JSXMLHttpRequest::visitChildren): |
- * bindings/js/JSXMLHttpRequestUploadCustom.cpp: No need to mark our owner |
- XHR because it is not reachable from us. |
- |
- * bindings/scripts/CodeGeneratorJS.pm: Added autogeneration support for a |
- bunch of repetitive cases of isReachableFromOpaqueRoots callbacks. |
- |
- * css/CSSRule.idl: |
- * css/CSSStyleDeclaration.idl: |
- * css/MediaList.idl: |
- * css/StyleMedia.idl: |
- * css/StyleSheet.idl: |
- * css/StyleSheetList.idl: |
- * dom/DOMImplementation.idl: |
- * dom/DOMStringMap.idl: |
- * dom/MessagePort.idl: |
- * dom/NamedNodeMap.idl: |
- * fileapi/Blob.idl: |
- * html/DOMTokenList.idl: Opt in to autogeneration. |
- |
- * html/HTMLCanvasElement.cpp: |
- (WebCore::HTMLCanvasElement::getContext): Standardized on PassOwnPtr/OwnPtr |
- usage, to reduce human error and make ownership rules more obvious. |
- |
- * html/HTMLCollection.idl: |
- * html/canvas/ArrayBuffer.idl: Opt in to autogeneration. |
- |
- * html/canvas/CanvasRenderingContext.cpp: |
- * html/canvas/CanvasRenderingContext.h: |
- (WebCore::CanvasRenderingContext::ref): |
- (WebCore::CanvasRenderingContext::deref): |
- * html/canvas/CanvasRenderingContext2D.h: |
- (WebCore::CanvasRenderingContext2D::create): |
- (WebCore::CanvasRenderingContext2D::state): Standardized on PassOwnPtr/OwnPtr |
- usage, to reduce human error and make ownership rules more obvious. |
- |
- Inlined some functions to match our standard idiom and to make the code |
- in the header document itself. |
- |
- * html/canvas/CanvasRenderingContext.idl: Opt in to autogeneration. |
- |
- * html/canvas/OESStandardDerivatives.cpp: |
- (WebCore::OESStandardDerivatives::OESStandardDerivatives): |
- (WebCore::OESStandardDerivatives::create): |
- * html/canvas/OESStandardDerivatives.h: |
- * html/canvas/OESStandardDerivatives.idl: |
- * html/canvas/OESTextureFloat.cpp: |
- (WebCore::OESTextureFloat::OESTextureFloat): |
- (WebCore::OESTextureFloat::create): |
- * html/canvas/OESTextureFloat.h: |
- * html/canvas/OESTextureFloat.idl: |
- * html/canvas/OESVertexArrayObject.cpp: |
- (WebCore::OESVertexArrayObject::OESVertexArrayObject): |
- (WebCore::OESVertexArrayObject::create): |
- * html/canvas/OESVertexArrayObject.h: |
- * html/canvas/OESVertexArrayObject.idl: |
- * html/canvas/WebGLExtension.cpp: |
- (WebCore::WebGLExtension::WebGLExtension): |
- * html/canvas/WebGLExtension.h: |
- (WebCore::WebGLExtension::ref): |
- (WebCore::WebGLExtension::deref): |
- (WebCore::WebGLExtension::context): |
- * html/canvas/WebGLRenderingContext.cpp: |
- (WebCore::WebGLRenderingContext::~WebGLRenderingContext): |
- (WebCore::WebGLRenderingContext::getExtension): |
- * html/canvas/WebGLRenderingContext.h: |
- * html/canvas/WebKitLoseContext.cpp: |
- (WebCore::WebKitLoseContext::WebKitLoseContext): |
- (WebCore::WebKitLoseContext::create): |
- (WebCore::WebKitLoseContext::loseContext): |
- * html/canvas/WebKitLoseContext.h: |
- * html/canvas/WebKitLoseContext.idl: Updated the ownership model for WebGL |
- canavs rendering contexts to match the model for 2D canvas rendering |
- contexts. This makes garbage collection a lot more straightforward, but |
- it also makes the behavior of these objects more reliable. (Previously, |
- satellite objects would magically stop working when the last reference |
- to their owner object was dropped. Now, satellite objects keep their owner |
- alive through reference counting.) |
- |
- * loader/appcache/DOMApplicationCache.idl: |
- * page/BarInfo.idl: |
- * page/Console.idl: |
- * page/DOMSelection.idl: |
- * page/Geolocation.idl: |
- * page/History.idl: |
- * page/Location.idl: |
- * page/Navigator.idl: |
- * page/Screen.idl: |
- * page/WorkerNavigator.idl: |
- * plugins/DOMMimeTypeArray.h: |
- (WebCore::DOMMimeTypeArray::frame): |
- * plugins/DOMMimeTypeArray.idl: |
- * plugins/DOMPluginArray.h: |
- (WebCore::DOMPluginArray::frame): |
- * plugins/DOMPluginArray.idl: |
- * storage/Storage.idl: |
- * workers/WorkerLocation.idl: Opt in to autogeneration. |
- |
- * xml/XMLHttpRequest.cpp: |
- (WebCore::XMLHttpRequest::~XMLHttpRequest): |
- * xml/XMLHttpRequest.h: |
- * xml/XMLHttpRequestUpload.cpp: |
- (WebCore::XMLHttpRequestUpload::scriptExecutionContext): |
- * xml/XMLHttpRequestUpload.h: |
- (WebCore::XMLHttpRequestUpload::create): |
- (WebCore::XMLHttpRequestUpload::ref): |
- (WebCore::XMLHttpRequestUpload::deref): |
- (WebCore::XMLHttpRequestUpload::xmlHttpRequest): |
- (WebCore::XMLHttpRequestUpload::toXMLHttpRequestUpload): |
- * xml/XMLHttpRequestUpload.idl: Updated the ownership model for XHR and |
- its associated upload object to match the model for canvas. This makes |
- garbage collection a lot more straightforward, but it also makes the |
- behavior of these objects more reliable. |
- |
-2011-04-24 Sheriff Bot <webkit.review.bot@gmail.com> |
- |
- Unreviewed, rolling out r84759. |
- http://trac.webkit.org/changeset/84759 |
- https://bugs.webkit.org/show_bug.cgi?id=59306 |
- |
- Caused gc-shadow.html to start crashing in V8. (Requested by |
- dglazkov|away on #webkit). |
- |
- * dom/Node.cpp: |
- (WebCore::NodeRendererFactory::createRendererAndStyle): |
- (WebCore::Node::styleForRenderer): |
- |
-2011-04-24 Maciej Stachowiak <mjs@apple.com> |
- |
- Reviewed by George Staikos. |
- |
- Handling of URLs like http:/example.com/ is incorrect |
- https://bugs.webkit.org/show_bug.cgi?id=59300 |
- <rdar://problem/9231956> |
- |
- URLs like http:/example.com/ or http:example.com/ are now correctly |
- canonicalized as http://example.com/ |
- |
- The code still doesn't quite match other browsers - at least some |
- other browsers seem to base parsing behavior on whether they |
- recongize a scheme from a fixed list, and ignore whether // is |
- present in the URL or not. |
- |
- * platform/KURL.cpp: |
- (WebCore::isNonFileHierarchicalScheme): New helper function. |
- (WebCore::KURL::parse): For a particular list of whitelisted schemes, |
- assume they are hierarchical and need an authority even if there is no // |
- after the :/ |
- |
-2011-04-24 Dan Bernstein <mitz@apple.com> |
- |
- LLVM Compiler build fix. |
- |
- * page/ContentSecurityPolicy.cpp: Removed an unused function. |
- |
-2011-04-24 Dominic Cooney <dominicc@chromium.org> |
- |
- Reviewed by Dimitri Glazkov. |
- |
- Crash when adding a text node to a shadow root |
- https://bugs.webkit.org/show_bug.cgi?id=59304 |
- |
- Text nodes need to consult their host element for style. |
- |
- Test: fast/dom/shadow/append-child-text.html |
- |
- * dom/Node.cpp: |
- (WebCore::NodeRendererFactory::styleForRenderer): |
- (WebCore::NodeRendererFactory::createRendererAndStyle): |
- (WebCore::Node::styleForRenderer): forward to NodeRendererFactory |
- |
-2011-04-24 Adam Barth <abarth@webkit.org> |
- |
- Reviewed by Eric Seidel. |
- |
- Update Content-Security-Policy syntax to match new version of spec |
- https://bugs.webkit.org/show_bug.cgi?id=59291 |
- |
- Brandon removed disable-xss-protection in favor of unsafe-inline and |
- allow-eval in favor of unsafe-eval. This change in syntax also means |
- the options directive no longer exists. |
- |
- * page/ContentSecurityPolicy.cpp: |
- (WebCore::CSPSourceList::allowInline): |
- (WebCore::CSPSourceList::allowEval): |
- (WebCore::CSPSourceList::CSPSourceList): |
- (WebCore::CSPSourceList::parseSource): |
- (WebCore::CSPSourceList::addSourceUnsafeInline): |
- (WebCore::CSPSourceList::addSourceUnsafeEval): |
- (WebCore::CSPDirective::allowInline): |
- (WebCore::CSPDirective::allowEval): |
- (WebCore::ContentSecurityPolicy::allowJavaScriptURLs): |
- (WebCore::ContentSecurityPolicy::allowInlineEventHandlers): |
- (WebCore::ContentSecurityPolicy::allowInlineScript): |
- (WebCore::ContentSecurityPolicy::allowEval): |
- (WebCore::ContentSecurityPolicy::addDirective): |
- * page/ContentSecurityPolicy.h: |
- |
-2011-04-24 Dan Bernstein <mitz@apple.com> |
- |
- Reviewed by Maciej Stachowiak. |
- |
- Manual test for <rdar://problem/9329741> Reproducible crash in WebChromeClient::invalidateContentsAndWindow() on simonscat.com |
- https://bugs.webkit.org/show_bug.cgi?id=59299 |
- |
- * manual-tests/back-from-document-with-scrollbar.html: Added. |
- |
-2011-04-24 Rik Cabanier <cabanier@adobe.com> |
- |
- Reviewed by Simon Fraser. |
- |
- Tables are not rendered correctly |
- https://bugs.webkit.org/show_bug.cgi?id=59138 |
- |
- Test: fast/table/auto-100-percent-width.html |
- |
- * rendering/AutoTableLayout.cpp: |
- (WebCore::AutoTableLayout::computePreferredLogicalWidths): |
- |
-2011-04-23 Simon Fraser <simon.fraser@apple.com> |
- |
- Fix Windows build, which complains about unreachable code. |
- |
- * platform/ScrollableArea.cpp: |
- (WebCore::ScrollableArea::hasLayerForHorizontalScrollbar): |
- (WebCore::ScrollableArea::hasLayerForVerticalScrollbar): |
- (WebCore::ScrollableArea::hasLayerForScrollCorner): |
- |
-2011-04-23 Simon Fraser <simon.fraser@apple.com> |
- |
- Reviewed by Dan Bernstein. |
- |
- Area under composited scrollbars not repainted in WebKit2 on scrolling |
- https://bugs.webkit.org/show_bug.cgi?id=59294 |
- <rdar://problem/9299062> |
- |
- When computing the rect that can be copied on scrolling, we normally |
- exclude the areas occupied by overlay scrollbars to avoid smeared scrollbars; |
- the assumption is that other code will repaint these areas. However, when |
- scrollbars are in their own compositing layers, we can, and should |
- copy the entire area. |
- |
- Not testable in DRT because it does not allow tests to use overlay scrollbars. |
- |
- * platform/ScrollView.h: |
- * platform/ScrollView.cpp: |
- (WebCore::ScrollView::rectToCopyOnScroll): |
- New method, with code moved from scrollContents and fixed to look |
- for scrollbars in layers. |
- (WebCore::ScrollView::scrollContents): |
- Call rectToCopyOnScroll(). |
- |
- * platform/ScrollableArea.h: |
- * platform/ScrollableArea.cpp: |
- (WebCore::ScrollableArea::hasLayerForHorizontalScrollbar): |
- (WebCore::ScrollableArea::hasLayerForVerticalScrollbar): |
- (WebCore::ScrollableArea::hasLayerForScrollCorner): |
- Methods we can call outside of the ACCELERATED_COMPOSITING #ifdef. |
- |
-2011-04-23 Sheriff Bot <webkit.review.bot@gmail.com> |
- |
- Unreviewed, rolling out r84740. |
- http://trac.webkit.org/changeset/84740 |
- https://bugs.webkit.org/show_bug.cgi?id=59290 |
- |
- change breaks apple internal builds and is incorrect |
- (Requested by smfr on #webkit). |
- |
- * WebCore.exp.in: |
- * page/FrameView.cpp: |
- * page/FrameView.h: |
- * platform/ScrollTypes.h: |
- * platform/ScrollableArea.h: |
- * platform/mac/ScrollbarThemeMac.mm: |
- (WebCore::ScrollbarThemeMac::paint): |
- * platform/mac/WebCoreSystemInterface.h: |
- * platform/mac/WebCoreSystemInterface.mm: |
- |
-2011-04-23 Dan Bernstein <mitz@apple.com> |
- |
- Reviewed by Simon Fraser. |
- |
- <rdar://problem/8970549> WebFindOptionsAtWordStarts fails with Japanese words |
- https://bugs.webkit.org/show_bug.cgi?id=59288 |
- |
- * platform/text/mac/TextBoundaries.mm: |
- (WebCore::wordStringTokenizer): Added this helper function. |
- (WebCore::findNextWordFromIndex): Changed to use a CFStringTokenizer with kCFStringTokenizerUnitWord, |
- whose behavior is not locale-dependent. |
- |
-2011-04-23 Dominic Cooney <dominicc@chromium.org> |
- |
- Reviewed by Dimitri Glazkov. |
- |
- [V8] Nodes in shadow DOM should not be GCed while their hosts are alive |
- https://bugs.webkit.org/show_bug.cgi?id=59284 |
- |
- Test: fast/dom/shadow/gc-shadow.html |
- |
- * bindings/v8/V8GCController.cpp: |
- (WebCore::calculateGroupId): group shadow nodes with their hosts |
- |
-2011-04-23 MORITA Hajime <morrita@google.com> |
- |
- Reviewed by Dimitri Glazkov. |
- |
- [Refactoring] NodeRenderParentDetector should be NodeRenderFactory |
- https://bugs.webkit.org/show_bug.cgi?id=59280 |
- |
- - Rename NodeRenderParentDetector to NodeRendererFactory |
- - move createRendererAndStyle() and createRendererIfNeeded to |
- NodeRendererFactory |
- |
- No new test, no behavioral change. |
- |
- * dom/Node.cpp: |
- (WebCore::NodeRendererFactory::NodeRendererFactory): |
- (WebCore::NodeRendererFactory::document): |
- (WebCore::NodeRendererFactory::nextRenderer): |
- (WebCore::NodeRendererFactory::findVisualParent): |
- (WebCore::NodeRendererFactory::shouldCreateRenderer): |
- (WebCore::NodeRendererFactory::createRendererAndStyle): |
- (WebCore::NodeRendererFactory::createRendererIfNeeded): |
- (WebCore::Node::parentNodeForRenderingAndStyle): |
- (WebCore::Node::createRendererIfNeeded): |
- * dom/Node.h: |
- * html/HTMLDetailsElement.cpp: |
- (WebCore::DetailsSummaryElement::detailsElement): Added const_cast due to the chage on parentNodeForRenderingAndStyle() |
- |
-2011-04-23 MORITA Hajime <morrita@google.com> |
- |
- Reviewed by Kent Tamura. |
- |
- REGRESSION: (non-Mac) <meter>'s bar part isn't rendered for with -webkit-writing-mode: vertical-lr from r82899 |
- https://bugs.webkit.org/show_bug.cgi?id=59281 |
- |
- Styles for -webkit-meter-bar and -webkit-progress-bar missed width property, which caused unexpected layout results |
- for vertical writing modes. This fix added a width property for each of them. |
- |
- Tests: fast/dom/HTMLMeterElement/meter-writing-mode.html |
- fast/dom/HTMLProgressElement/progress-writing-mode.html |
- |
- * css/html.css: |
- (meter::-webkit-meter-bar): |
- (progress::-webkit-progress-bar): |
- |
-2011-04-23 Kevin Ollivier <kevino@theolliviers.com> |
- |
- [wx] Unreviewed build fix. Added missing header. |
- |
- * platform/graphics/filters/FEDropShadow.h: |
- |
-2011-04-23 Alexey Proskuryakov <ap@apple.com> |
- |
- Reviewed by Maciej Stachowiak. |
- |
- REGRESSION (r80812): window.print();window.close() doesn't work in WebKit2 |
- https://bugs.webkit.org/show_bug.cgi?id=59241 |
- <rdar://problem/9150861> |
- |
- Covered by manual-tests/print-after-window-close.html. |
- |
- Re-fixing <https://bugs.webkit.org/show_bug.cgi?id=51357> in a way that doesn't cause the |
- regression. |
- |
- * loader/MainResourceLoader.cpp: (WebCore::MainResourceLoader::didCancel): We shouldn't be |
- doing complicated work while the loader is half-canceled. |
- |
- * manual-tests/print-after-window-close.html: Extended to actually print a non-empty page, |
- thus being more practical. |
- |
- * page/Chrome.cpp: (WebCore::Chrome::print): Added a FIXME about a need for PageGroupLoadDeferrer. |
- It's too scary for me to try now. |
- |
- * page/DOMWindow.cpp: |
- (WebCore::DOMWindow::DOMWindow): |
- (WebCore::DOMWindow::finishedLoading): |
- * page/DOMWindow.h: |
- Reverted the previous fix for <https://bugs.webkit.org/show_bug.cgi?id=51357>. |
- |
-2011-04-19 Vitaly Repeshko <vitalyr@chromium.org> |
- |
- Reviewed by Adam Barth. |
- |
- [V8] Use implicit references for V8 listeners on DOM nodes. |
- https://bugs.webkit.org/show_bug.cgi?id=58953 |
- |
- Instead of allocating an auxiliary V8 array referencing V8 |
- listener objects associated with a DOM node and using an extra |
- pointer in every DOM node wrapper, we can register implicit |
- references between nodes and their listeners during GC. This also |
- makes V8 bindings more aligned with JSC bindings. |
- |
- No new tests because this is a refactoring. |
- |
- * bindings/scripts/CodeGeneratorV8.pm: Stopped generating the |
- listener cache internal field for DOM nodes. |
- * bindings/v8/V8AbstractEventListener.h: Exposed the lister handle |
- to the GC. |
- (WebCore::V8AbstractEventListener::existingListenerObjectPeristentHandle): |
- * bindings/v8/V8GCController.cpp: Started using implicit references. |
- (WebCore::GrouperVisitor::visitDOMWrapper): |
- * dom/EventTarget.h: Implemented an iterator over all listeners. |
- (WebCore::EventListenerIterator): |
- |
-2011-04-22 Jon Lee <jonlee@apple.com> |
- |
- Reviewed by Beth Dakin. |
- |
- Overlay scroller hard to see on pages with dark background (59183) |
- https://bugs.webkit.org/show_bug.cgi?id=59183 |
- <rdar://problem/8975367> |
- |
- * WebCore.exp.in: adding method to allow changing style |
- * page/FrameView.cpp: |
- (WebCore::FrameView::recommendedScrollbarOverlayStyle): overridden to suggest a style based on CSS background color |
- * page/FrameView.h: |
- * platform/ScrollTypes.h: new enum to represent different overlay scrollbar styles |
- * platform/ScrollableArea.cpp: |
- (WebCore::ScrollableArea::recommendedScrollbarOverlayStyle): new virtual function to return a suggested overlay style |
- * platform/ScrollableArea.h: |
- * platform/mac/ScrollbarThemeMac.mm: |
- (WebCore::ScrollbarThemeMac::paint): |
- * platform/mac/WebCoreSystemInterface.h: |
- * platform/mac/WebCoreSystemInterface.mm: |
- |
-2011-04-22 Chris Evans <cevans@chromium.org> |
- |
- Reviewed by Adam Barth. |
- |
- Upgrade CSS loads from mixed content warning (displayed) to mixed |
- content error (ran) |
- https://bugs.webkit.org/show_bug.cgi?id=59056 |
- |
- Test: http/tests/security/mixedContent/insecure-css-in-iframe.html |
- Test: http/tests/security/mixedContent/insecure-css-in-main-frame.html |
- |
- * loader/cache/CachedResourceLoader.cpp: |
- (WebCore::CachedResourceLoader::canRequest): CSS loads are running rather than displaying content. |
- |
-2011-04-22 Geoffrey Garen <ggaren@apple.com> |
- |
- Fixed an upside-down conditional in my last check-in. |
- |
- * bindings/js/JSDOMApplicationCacheCustom.cpp: |
- (WebCore::isObservable): |
- |
-2011-04-22 Geoffrey Garen <ggaren@apple.com> |
- |
- Rolled back in r84725 and r84728 with appcache crash fixed. |
- |
- Reviewed by Oliver Hunt. |
- |
- Removed a use of markDOMObjectWrapper: DOMWindow |
- https://bugs.webkit.org/show_bug.cgi?id=59260 |
- |
- * bindings/js/JSDOMWindowCustom.cpp: |
- (WebCore::JSDOMWindow::visitChildren): No need to mark explicitly, since |
- we use the opaque roots system now. |
- |
- * bindings/js/JSNavigatorCustom.cpp: Ditto. |
- |
- * bindings/scripts/CodeGeneratorJS.pm: Added autogeneration for objects |
- whose opaque roots are frames. |
- |
- * css/StyleMedia.h: |
- (WebCore::StyleMedia::frame): Added an accessor for the sake of GC. |
- |
- * css/StyleMedia.idl: |
- * loader/appcache/DOMApplicationCache.idl: |
- * page/BarInfo.idl: |
- * page/Console.idl: |
- * page/DOMSelection.idl: |
- * page/Geolocation.idl: |
- * page/History.idl: |
- * page/Location.idl: |
- * page/Navigator.idl: |
- * page/Screen.idl: |
- * storage/Storage.idl: Updated to use the opaque roots system. |
- |
-2011-04-22 Nate Chapin <japhet@chromium.org> |
- |
- Reviewed by Adam Barth. |
- |
- Don't use XMLTreeViewer when script is disabled (it uses Javascript |
- to build a prettified version of the xml document). |
- https://bugs.webkit.org/show_bug.cgi?id=59113 |
- |
- * xml/XMLTreeViewer.cpp: |
- (WebCore::XMLTreeViewer::transformDocumentToTreeView): |
- |
-2011-04-22 Sheriff Bot <webkit.review.bot@gmail.com> |
- |
- Unreviewed, rolling out r84725 and r84728. |
- http://trac.webkit.org/changeset/84725 |
- http://trac.webkit.org/changeset/84728 |
- https://bugs.webkit.org/show_bug.cgi?id=59270 |
- |
- It broke http/tests/appcache tests (Requested by Ossy on |
- #webkit). |
- |
- * bindings/js/JSDOMWindowCustom.cpp: |
- (WebCore::JSDOMWindow::visitChildren): |
- * bindings/js/JSNavigatorCustom.cpp: |
- (WebCore::JSNavigator::visitChildren): |
- * bindings/scripts/CodeGeneratorJS.pm: |
- * css/StyleMedia.h: |
- * css/StyleMedia.idl: |
- * loader/appcache/DOMApplicationCache.idl: |
- * page/BarInfo.idl: |
- * page/Console.idl: |
- * page/DOMSelection.idl: |
- * page/DOMWindow.h: |
- (WebCore::DOMWindow::optionalScreen): |
- (WebCore::DOMWindow::optionalSelection): |
- (WebCore::DOMWindow::optionalHistory): |
- (WebCore::DOMWindow::optionalCrypto): |
- (WebCore::DOMWindow::optionalLocationbar): |
- (WebCore::DOMWindow::optionalMenubar): |
- (WebCore::DOMWindow::optionalPersonalbar): |
- (WebCore::DOMWindow::optionalScrollbars): |
- (WebCore::DOMWindow::optionalStatusbar): |
- (WebCore::DOMWindow::optionalToolbar): |
- (WebCore::DOMWindow::optionalConsole): |
- (WebCore::DOMWindow::optionalNavigator): |
- (WebCore::DOMWindow::optionalLocation): |
- (WebCore::DOMWindow::optionalMedia): |
- (WebCore::DOMWindow::optionalSessionStorage): |
- (WebCore::DOMWindow::optionalLocalStorage): |
- (WebCore::DOMWindow::optionalPerformance): |
- * page/Geolocation.idl: |
- * page/History.idl: |
- * page/Location.idl: |
- * page/Navigator.h: |
- (WebCore::Navigator::optionalGeolocation): |
- * page/Navigator.idl: |
- * page/Screen.idl: |
- * storage/Storage.idl: |
- |
-2011-04-22 Dimitri Glazkov <dglazkov@chromium.org> |
- |
- Reviewed by Kent Tamura. |
- |
- Change SVG-related shadow root checks to use isSVGShadowRoot. |
- https://bugs.webkit.org/show_bug.cgi?id=59237 |
- |
- Covered by existing tests, no behavior change. |
- |
- * css/CSSStyleSelector.cpp: |
- (WebCore::CSSStyleSelector::SelectorChecker::checkSelector): Use isSVGShadowRoot. |
- * svg/SVGUseElement.cpp: |
- (WebCore::ShadowTreeUpdateBlocker::while): Ditto. |
- * svg/animation/SVGSMILElement.cpp: |
- (WebCore::SVGSMILElement::insertedIntoDocument): Ditto. |
- |
-2011-04-22 Geoffrey Garen <ggaren@apple.com> |
- |
- Mac build fix. |
- |
- * page/DOMWindow.h: |
- (WebCore::DOMWindow::optionalApplicationCache): Restored this function, |
- which is used after all. |
- |
-2011-04-22 Geoffrey Garen <ggaren@apple.com> |
- |
- Mac build fix. |
- |
- * page/DOMWindow.h: |
- (WebCore::DOMWindow::optionalApplicationCache): Restored this function, |
- which is used after all. |
- |
-2011-04-22 Geoffrey Garen <ggaren@apple.com> |
- |
- Reviewed by Oliver Hunt. |
- |
- Removed a use of markDOMObjectWrapper: DOMWindow |
- https://bugs.webkit.org/show_bug.cgi?id=59260 |
- |
- * bindings/js/JSDOMWindowCustom.cpp: |
- (WebCore::JSDOMWindow::visitChildren): No need to mark explicitly, since |
- we use the opaque roots system now. |
- |
- * bindings/js/JSNavigatorCustom.cpp: Ditto. |
- |
- * bindings/scripts/CodeGeneratorJS.pm: Added autogeneration for objects |
- whose opaque roots are frames. |
- |
- * css/StyleMedia.h: |
- (WebCore::StyleMedia::frame): Added an accessor for the sake of GC. |
- |
- * css/StyleMedia.idl: |
- * loader/appcache/DOMApplicationCache.idl: |
- * page/BarInfo.idl: |
- * page/Console.idl: |
- * page/DOMSelection.idl: |
- * page/Geolocation.idl: |
- * page/History.idl: |
- * page/Location.idl: |
- * page/Navigator.idl: |
- * page/Screen.idl: |
- * storage/Storage.idl: Updated to use the opaque roots system. |
- |
-2011-04-22 Xiaomei Ji <xji@chromium.org> |
- |
- Reviewed by Ryosuke Niwa. |
- |
- move caret by word in visual order returns wrong result when caret itself is at word boundary. |
- https://bugs.webkit.org/show_bug.cgi?id=58504. |
- |
- positionBeforeNextWord should take care that the current position is after current word. |
- positionAfterPreviousWord should take care that the current positin is before current word. |
- |
- * editing/visible_units.cpp: |
- (WebCore::positionBeforeNextWord): |
- (WebCore::positionAfterPreviousWord): |
- |
-2011-04-19 Jer Noble <jer.noble@apple.com> |
- |
- Reviewed by Daniel Bates. |
- |
- Full Screen from within an <iframe> does not cause <iframe> to resize. |
- https://bugs.webkit.org/show_bug.cgi?id=58638 |
- |
- Tests: fullscreen/full-screen-frameset.html |
- |
- * css/CSSStyleSelector.cpp: |
- (WebCore::CSSStyleSelector::SelectorChecker::checkOneSelector): Add the |
- -webkit-full-screen pseudo class to frame elements which contain |
- full screen elements. |
- * css/fullscreen.css: |
- (iframe:-webkit-full-screen): Add iframe-specific styling rules. |
- * dom/Document.cpp: |
- (WebCore::Document::setContainsFullScreenElementRecursively): Added. Walk up each |
- successive document owner element, calling setContainsFullScreenElement() if |
- owner is a frame element. |
- (WebCore::Document::fullScreenIsAllowedForElement): Use the new Element::isFrameElementBase |
- call instead of checking the element's tag name. |
- (WebCore::Document::webkitWillEnterFullScreenForElement): Mark any containing |
- iframes as containing a full screen element. |
- (WebCore::Document::webkitWillExitFullScreenForElement): Ditto. |
- * dom/Element.h: |
- (WebCore::Element::isFrameElementBase): Added. |
- * html/HTMLFrameElement.h: |
- (WebCore::HTMLFrameElement::allowFullScreen): Disable full screen from within |
- HTMLFrameElements. |
- * html/HTMLFrameElementBase.cpp: |
- (WebCore::HTMLFrameElementBase::setContainsFullScreenElement): Added. |
- * html/HTMLFrameElementBase.h: |
- (WebCore::HTMLFrameElementBase::containsFullScreenElement): Added. |
- (WebCore::HTMLFrameElementBase::isFrameElementBase): Added. |
- |
-2011-04-22 Michael Saboff <msaboff@apple.com> |
- |
- Reviewed by Maciej Stachowiak. |
- |
- Creating copy of ContainerNode's when inserting or removing is inefficient |
- https://bugs.webkit.org/show_bug.cgi?id=58695 |
- |
- Eliminated node copying in willRemove() and insertedIntoDocument(). |
- |
- No new tests as this is a more efficient implementation of |
- existing code that is covered by existing tests. |
- |
- * dom/ContainerNode.cpp: |
- (WebCore::ContainerNode::willRemove): Changed method to use |
- RefPtr<> to protect against modification during removal. |
- (WebCore::ContainerNode::insertedIntoDocument): Changed method to use |
- RefPtr<> and two other deletion checks to protect against |
- modification during insertion. |
- |
-2011-04-22 Geoffrey Garen <ggaren@apple.com> |
- |
- Reviewed by Sam Weinig. |
- |
- We have an autogenerator, so use it |
- https://bugs.webkit.org/show_bug.cgi?id=59247 |
- |
- * bindings/js/JSCSSRuleCustom.cpp: |
- * bindings/js/JSCSSRuleListCustom.cpp: |
- * bindings/js/JSCSSStyleDeclarationCustom.cpp: |
- * bindings/js/JSCSSValueCustom.cpp: |
- * bindings/js/JSCanvasRenderingContextCustom.cpp: |
- * bindings/js/JSDOMImplementationCustom.cpp: |
- * bindings/js/JSDOMStringMapCustom.cpp: |
- * bindings/js/JSHTMLCollectionCustom.cpp: |
- * bindings/js/JSNamedNodeMapCustom.cpp: |
- * bindings/js/JSNodeCustom.h: |
- * bindings/js/JSNodeListCustom.cpp: |
- * bindings/js/JSStyleSheetCustom.cpp: |
- * bindings/js/JSStyleSheetListCustom.cpp: Removed repetitive hand-written code. |
- |
- * bindings/scripts/CodeGeneratorJS.pm: Augmented autogenerator to do the job. |
- |
- * css/CSSRule.idl: |
- * css/CSSRuleList.idl: |
- * css/CSSStyleDeclaration.idl: |
- * css/CSSValue.idl: |
- * css/MediaList.idl: |
- * css/StyleSheet.idl: |
- * css/StyleSheetList.idl: |
- * dom/DOMImplementation.idl: |
- * dom/DOMStringMap.idl: |
- * dom/NamedNodeMap.idl: |
- * dom/Node.idl: |
- * dom/NodeList.idl: |
- * html/DOMTokenList.idl: |
- * html/HTMLCollection.idl: |
- * html/canvas/CanvasRenderingContext.idl: Updated attributes to plug into |
- the autogenerator. |
- |
-2011-04-22 Beth Dakin <bdakin@apple.com> |
- |
- Reviewed by Maciej Stachowiak. |
- |
- https://bugs.webkit.org/show_bug.cgi?id=59223 |
- REGRESSION (r82185): iframe not displayed after navigation back to cached page |
- |
- Manual test added. |
- * manual-tests/frame-layout-on-back.html: Added. |
- * manual-tests/resources/empty-document-goes-back.html: Added. |
- * manual-tests/resources/simple-iframe.html: Added. |
- |
-2011-04-22 Yael Aharon <yael.aharon@nokia.com> |
- |
- Reviewed by Sam Weinig. |
- |
- HTMLFormControlElement::labels should allow custom attributes. |
- https://bugs.webkit.org/show_bug.cgi?id=59128 |
- |
- Cache the labels list when it is created and keep using it for future |
- labels calls. |
- |
- Test: fast/forms/labels-custom-property.html |
- |
- * html/HTMLFormControlElement.cpp: |
- (WebCore::HTMLFormControlElement::labels): |
- |
-2011-04-22 Adrienne Walker <enne@google.com> |
- |
- Reviewed by James Robinson. |
- |
- [chromium] Remove minimum texture size for compositor tiling |
- https://bugs.webkit.org/show_bug.cgi?id=59236 |
- |
- This was mistakenly committed in r83915 and causes bad filtering |
- on very small textures. |
- |
- Tests: compositing/images/content-image-change.html |
- |
- * platform/graphics/chromium/LayerTilerChromium.cpp: |
- (WebCore::LayerTilerChromium::setTileSize): |
- |
-2011-04-22 Beth Dakin <bdakin@apple.com> |
- |
- Reviewed by Maciej Stachowiak. |
- |
- https://bugs.webkit.org/show_bug.cgi?id=59223 |
- REGRESSION (r82185): iframe not displayed after navigation back to cached page |
- -and corresponding- |
- <rdar://problem/9323763> |
- |
- Cache the scroll position in Document::setInPageCache() instead of |
- FrameView::resetScrollbarsAndClearContentsSize(), and only clear the contents size |
- for the mainFrame(). Still resetScrollbars() for other frames, which matches pre- |
- r82185 code. |
- * dom/Document.cpp: |
- (WebCore::Document::setInPageCache): |
- * page/FrameView.cpp: |
- (WebCore::FrameView::resetScrollbarsAndClearContentsSize): |
- |
-2011-04-22 Joseph Pecoraro <joepeck@webkit.org> |
- |
- Reviewed by Kent Tamura. |
- |
- Let the RenderTheme decide if an input type should have a spinner |
- https://bugs.webkit.org/show_bug.cgi?id=59246 |
- |
- Whether an input should have a spinner or not should |
- be a UI decision made by the RenderTheme. Remove |
- WebCore's default hasSpinButton values and defer |
- to the RenderTheme. |
- |
- No change in functionality, covered by existing tests. |
- |
- * html/HTMLInputElement.cpp: |
- (WebCore::HTMLInputElement::isRangeControl): expose isRangeControl |
- for possible RenderTheme queries. |
- * html/HTMLInputElement.h: |
- |
- * rendering/RenderTextControlSingleLine.cpp: |
- (WebCore::RenderTextControlSingleLine::createSubtreeIfNeeded): |
- (WebCore::RenderTextControlSingleLine::createInnerTextStyle): |
- For both of these ask the render theme if the input should |
- have a spinner. |
- |
- * rendering/RenderTheme.cpp: |
- (WebCore::RenderTheme::shouldHaveSpinButtonForInputElement): |
- Steppable inputs (number, date, and range) but not range |
- should have spinners. This does not change existing behavior. |
- * rendering/RenderTheme.h: |
- |
- * dom/InputElement.h: |
- * html/BaseDateAndTimeInputType.cpp: remove hasSpinButton. |
- * html/BaseDateAndTimeInputType.h: remove hasSpinButton. |
- * html/InputType.cpp: remove hasSpinButton. |
- * html/InputType.h: remove hasSpinButton. |
- * html/NumberInputType.cpp: remove hasSpinButton. |
- * html/NumberInputType.h: remove hasSpinButton. |
- |
-2011-04-22 Joseph Pecoraro <joepeck@webkit.org> |
- |
- Reviewed by Kent Tamura. |
- |
- Don't rely on hasSpinButton to check if an input type can be stepped |
- https://bugs.webkit.org/show_bug.cgi?id=59224 |
- |
- Add isSteppable to know for certain if an input type can |
- be stepped or not. Previously we were checking if the input |
- has a spin control or is the range input type. These are UI |
- details, but the step functionality does not require there |
- to be a UI. |
- |
- No change in behavior, covered by existing tests. |
- |
- * html/BaseDateAndTimeInputType.cpp: |
- (WebCore::BaseDateAndTimeInputType::isSteppable): |
- * html/BaseDateAndTimeInputType.h: |
- * html/HTMLInputElement.cpp: |
- (WebCore::HTMLInputElement::isSteppable): |
- (WebCore::HTMLInputElement::stepUpFromRenderer): remove the |
- check for a spinner and range control and instead ASSERT |
- that the type is steppable. |
- * html/HTMLInputElement.h: |
- * html/InputType.cpp: |
- (WebCore::InputType::isSteppable): |
- * html/InputType.h: |
- * html/NumberInputType.cpp: |
- (WebCore::NumberInputType::isSteppable): |
- * html/NumberInputType.h: |
- * html/RangeInputType.cpp: |
- (WebCore::RangeInputType::isSteppable): |
- * html/RangeInputType.h: |
- |
-2011-04-22 Adam Barth <abarth@webkit.org> |
- |
- Reviewed by Eric Seidel. |
- |
- CSP frame-src is missing |
- https://bugs.webkit.org/show_bug.cgi?id=58643 |
- |
- In this patch, I've audited all the callers to |
- SecurityOrigin::canDisplay to make sure they all the appropriate |
- Content-Security-Policy method as well. I'm not sure how to test all |
- these cases, but making this changes fixed the frame-src test. |
- |
- * loader/SubframeLoader.cpp: |
- (WebCore::SubframeLoader::loadMediaPlayerProxyPlugin): |
- (WebCore::SubframeLoader::createJavaAppletWidget): |
- (WebCore::SubframeLoader::loadSubframe): |
- (WebCore::SubframeLoader::loadPlugin): |
- * loader/SubresourceLoader.cpp: |
- (WebCore::SubresourceLoader::create): |
- * loader/cache/CachedResourceLoader.cpp: |
- (WebCore::CachedResourceLoader::canRequest): |
- (WebCore::CachedResourceLoader::requestResource): |
- - While I was understanding this code, I fixed the FIXME here. |
- * loader/cache/CachedResourceLoader.h: |
- |
-2011-04-22 Adrienne Walker <enne@google.com> |
- |
- Reviewed by Simon Fraser. |
- |
- Clear shared graphics context when destroying CanvasRenderingContext2D. |
- https://bugs.webkit.org/show_bug.cgi?id=59212 |
- |
- * html/canvas/CanvasRenderingContext2D.cpp: |
- (WebCore::CanvasRenderingContext2D::~CanvasRenderingContext2D): |
- |
-2011-04-22 Levi Weintraub <leviw@chromium.org> |
- |
- Reviewed by Ryosuke Niwa. |
- |
- End/Home keys do not work in a contentEditable element |
- https://bugs.webkit.org/show_bug.cgi?id=50564 |
- |
- Changing logical[Start/End]OfLine to return a valid VisiblePosition for for editable contexts that are |
- inline and share a line with non-editable content. |
- |
- Test: editing/selection/modify-by-lineboundary-in-inline-editable-contexts.html |
- |
- * editing/visible_units.cpp: |
- (WebCore::logicalStartOfLine): |
- (WebCore::logicalEndOfLine): |
- |
-2011-04-22 Dimitri Glazkov <dglazkov@chromium.org> |
- |
- Reviewed by Kent Tamura. |
- |
- Make showTree/showTreeForThis aware of the shadow DOM |
- https://bugs.webkit.org/show_bug.cgi?id=52965 |
- |
- No new tests, because this code is only compiled in DEBUG. |
- |
- * dom/Node.cpp: |
- (WebCore::traverseTreeAndMark): Moved tree traversal logic here, adding |
- recursive sub-traversal of the shadow tree. |
- (WebCore::Node::showTreeAndMark): Changed to use the new tree traversal function. |
- |
-2011-04-22 Sam Weinig <sam@webkit.org> |
- |
- Reviewed by Gavin Barraclough and Oliver Hunt. |
- |
- Arrays should participate in global object forwarding fun |
- https://bugs.webkit.org/show_bug.cgi?id=59215 |
- |
- Change callers of constructArray to use the new variant that takes |
- a global object. |
- |
- * bindings/js/JSClipboardCustom.cpp: |
- (WebCore::JSClipboard::types): |
- * bindings/js/JSConsoleCustom.cpp: |
- (WebCore::JSConsole::profiles): |
- * bindings/js/JSJavaScriptCallFrameCustom.cpp: |
- (WebCore::JSJavaScriptCallFrame::scopeChain): |
- * bindings/js/JSMessageEventCustom.cpp: |
- (WebCore::JSMessageEvent::ports): |
- * bindings/js/JSScriptProfileNodeCustom.cpp: |
- (WebCore::JSScriptProfileNode::children): |
- * bindings/js/JSWebGLRenderingContextCustom.cpp: |
- (WebCore::toJS): |
- (WebCore::JSWebGLRenderingContext::getAttachedShaders): |
- (WebCore::JSWebGLRenderingContext::getSupportedExtensions): |
- |
-2011-04-22 Csaba Osztrogonác <ossy@webkit.org> |
- |
- [Qt] Unreviewed typo fix after r84522. |
- |
- * WebCore.pro: |
- |
-2011-04-22 Daniel Bates <dbates@rim.com> |
- |
- Fix misspelled word in comment; tagets => targets. |
- |
- * page/EventHandler.cpp: |
- (WebCore::EventHandler::handleTouchEvent): |
- |
-2011-04-21 Geoffrey Garen <ggaren@apple.com> |
- |
- Reviewed by Oliver Hunt. |
- |
- Removed a use of markDOMObjectWrapper: Element and Document |
- https://bugs.webkit.org/show_bug.cgi?id=59178 |
- |
- * GNUmakefile.list.am: |
- * UseJSC.cmake: |
- * WebCore.gypi: |
- * WebCore.pro: |
- * WebCore.vcproj/WebCore.vcproj: |
- * WebCore.xcodeproj/project.pbxproj: |
- * bindings/js/JSBindingsAllInOne.cpp: Build! |
- |
- * bindings/js/JSDOMImplementationCustom.cpp: |
- (WebCore::JSDOMImplementationOwner::isReachableFromOpaqueRoots): |
- (WebCore::JSDOMImplementationOwner::finalize): |
- (WebCore::wrapperOwner): |
- (WebCore::wrapperContext): |
- (WebCore::toJS): Use the opaque roots system for DOMImplementation. Also, |
- DOMImplementation does not need to keep its document wrapper alive, since |
- there's no DOM API for getting a DOMImplementation's document. |
- |
- * bindings/js/JSDOMStringMapCustom.cpp: |
- (WebCore::JSDOMStringMapOwner::isReachableFromOpaqueRoots): |
- (WebCore::JSDOMStringMapOwner::finalize): |
- (WebCore::wrapperOwner): |
- (WebCore::wrapperContext): |
- (WebCore::toJS): Use the opaque roots system for DOMStringMap. |
- |
- * bindings/js/JSDOMTokenListCustom.cpp: Added. |
- (WebCore::JSDOMTokenListOwner::isReachableFromOpaqueRoots): |
- (WebCore::JSDOMTokenListOwner::finalize): |
- (WebCore::wrapperOwner): |
- (WebCore::wrapperContext): |
- (WebCore::toJS): Use the opaque roots system for DOMTokenList. |
- |
- * bindings/js/JSDocumentCustom.cpp: |
- (WebCore::JSDocument::visitChildren): |
- * bindings/js/JSElementCustom.cpp: No more need for explicit marking. |
- |
- * bindings/scripts/CodeGeneratorV8.pm: Boo on whoever made this design decision. |
- |
- * dom/DOMImplementation.cpp: |
- (WebCore::DOMImplementation::DOMImplementation): |
- (WebCore::DOMImplementation::createDocument): |
- (WebCore::DOMImplementation::createHTMLDocument): |
- * dom/DOMImplementation.h: |
- (WebCore::DOMImplementation::create): |
- (WebCore::DOMImplementation::ref): |
- (WebCore::DOMImplementation::deref): |
- (WebCore::DOMImplementation::document): Fixed the DOMImplementation |
- ownership model to match other satellite objects in the DOM. This was |
- necessary to fix some crashes due to incorrect reference counting. |
- |
- * dom/DOMImplementation.idl: |
- * dom/DOMStringMap.idl: |
- * dom/Document.cpp: |
- (WebCore::Document::~Document): |
- * dom/Document.h: |
- * dom/Element.cpp: |
- * dom/Element.h: |
- * dom/Element.idl: |
- * html/DOMTokenList.idl: Updated to match changes above. |
- |
-2011-04-22 Simon Fraser <simon.fraser@apple.com> |
- |
- Reviewed by Dan Bernstein. |
- |
- Rounded corners don't looks smooth when background-image is a gradient |
- https://bugs.webkit.org/show_bug.cgi?id=59213 |
- |
- Add the "shrink background" behavior to the slow background |
- drawing code path. |
- |
- Test: fast/backgrounds/gradient-background-leakage.html |
- |
- * rendering/RenderBoxModelObject.cpp: |
- (WebCore::backgroundRectAdjustedForBleedAvoidance): |
- Utility function that returns a shrunk border rect based |
- on the context scale. |
- (WebCore::RenderBoxModelObject::paintFillLayerExtended): |
- Use backgroundRectAdjustedForBleedAvoidance() in both the fast |
- and slow paths. |
- |
-2011-04-22 Xiaomei Ji <xji@chromium.org> |
- |
- Reviewed by David Hyatt. |
- |
- Local caret rectangle calculation should be relative to its containing block. |
- https://bugs.webkit.org/show_bug.cgi?id=49508. |
- |
- InlineBox and caret rectangle are positioned relative to its containing block. |
- So its left and right edge value should be relative to its containing block as well. |
- |
- Test: fast/forms/cursor-at-editable-content-boundary.html |
- |
- * rendering/RenderText.cpp: |
- (WebCore::RenderText::localCaretRect): |
- |
-2011-04-22 Levi Weintraub <leviw@chromium.org> |
- |
- Reviewed by Darin Fisher. |
- |
- REGRESSION: left property broken with position:fixed elements in RTL documents |
- https://bugs.webkit.org/show_bug.cgi?id=59204 |
- http://code.google.com/p/chromium/issues/detail?id=80216 |
- |
- Test: fast/block/positioning/rtl-fixed-positioning.html |
- fast/block/positioning/vertical-rl/fixed-positioning.html |
- |
- Returning the proper scroll offsets for fixed position content in RTL documents. |
- We regressed to X and Y offsets not updating while scrolling when we clamped to zero, |
- but RTL documents scroll in negative space. |
- |
- * page/FrameView.cpp: |
- (WebCore::FrameView::scrollXForFixedPosition): Properly handling RTL documents where |
- the scroll origin and offsets are negative. |
- (WebCore::FrameView::scrollYForFixedPosition): Properly handling vertical writing- |
- mode RTL documents, like above. |
- * platform/ScrollView.h: |
- (WebCore::ScrollView::scrollOrigin): Changing this accessor to const. |
- |
-2011-04-22 Justin Schuh <jschuh@chromium.org> |
- |
- Reviewed by Eric Seidel. |
- |
- Should assert on invalid type in FEComponentTransfer::apply |
- https://bugs.webkit.org/show_bug.cgi?id=57959 |
- |
- No functional change. No new tests needed. |
- |
- * platform/graphics/filters/FEComponentTransfer.cpp: |
- (WebCore::FEComponentTransfer::apply): |
- |
-2011-04-22 MORITA Hajime <morrita@google.com> |
- |
- Unreviewed crash fix. |
- |
- * html/HTMLDetailsElement.cpp: |
- (WebCore::DetailsSummaryElement::shouldInclude): |
- |
-2011-04-22 Dan Bernstein <mitz@apple.com> |
- |
- Reviewed by Sam Weinig. |
- |
- Exclude scrollbars from the dirty rect passed to the document |
- https://bugs.webkit.org/show_bug.cgi?id=59206 |
- |
- No test added because functionality is unchanged. |
- |
- * platform/ScrollView.cpp: |
- (WebCore::ScrollView::paint): Compute a clip rectangle (analogous to the frame of the clip view |
- in an NSScrollView) by subtracting non-overlay scrollbars, and intersect the document dirty rect |
- with it. If the result is empty, avoid calling paintContents(). |
- |
-2011-04-22 Jer Noble <jer.noble@apple.com> |
- |
- Reviewed by Maciej Stachowiak. |
- |
- Disable MediaPlayerPrivateAVFoundation when using old full screen mode. |
- https://bugs.webkit.org/show_bug.cgi?id=59147 |
- |
- Add a new isAVFoundationEnabled setting and check it before adding |
- MediaPlayerPrivateAVFoundationObjC to the media engine registry. |
- |
- * WebCore.exp.in: |
- * page/Settings.cpp: |
- * page/Settings.h: |
- (WebCore::Settings::setAVFoundationEnabled): Added. |
- (WebCore::Settings::isAVFoundationEnabled): Added. |
- * platform/graphics/MediaPlayer.cpp: |
- (WebCore::installedMediaEngines): Check the settings added above. |
- |
-2011-04-22 Sam Weinig <sam@webkit.org> |
- |
- Reviewed by Gavin Barraclough. |
- |
- Remove remaining uses of deprecated version of toJS |
- https://bugs.webkit.org/show_bug.cgi?id=59170 |
- |
- * bindings/js/JSCanvasRenderingContext2DCustom.cpp: |
- (WebCore::toJS): |
- (WebCore::JSCanvasRenderingContext2D::strokeStyle): |
- (WebCore::JSCanvasRenderingContext2D::fillStyle): |
- (WebCore::JSCanvasRenderingContext2D::createPattern): |
- * bindings/js/JSConsoleCustom.cpp: |
- (WebCore::JSConsole::profiles): |
- * bindings/js/JSCustomSQLStatementErrorCallback.cpp: |
- (WebCore::JSSQLStatementErrorCallback::handleEvent): |
- * bindings/js/JSDOMBinding.h: |
- * bindings/js/JSDOMMimeTypeArrayCustom.cpp: |
- (WebCore::JSDOMMimeTypeArray::nameGetter): |
- * bindings/js/JSDOMPluginArrayCustom.cpp: |
- (WebCore::JSDOMPluginArray::nameGetter): |
- * bindings/js/JSDOMPluginCustom.cpp: |
- (WebCore::JSDOMPlugin::nameGetter): |
- * bindings/js/JSDOMWindowCustom.cpp: |
- (WebCore::namedItemGetter): |
- (WebCore::JSDOMWindow::event): |
- * bindings/js/JSEventTarget.cpp: |
- (WebCore::toJS): |
- * bindings/js/JSInjectedScriptHostCustom.cpp: |
- Propagate stored global object down access chain. |
- |
- (WebCore::InjectedScriptHost::nodeAsScriptValue): |
- Replace use of deprecated toJS overload with explicit call to deprecatedGlobalObjectForPrototype. |
- |
- (WebCore::JSInjectedScriptHost::currentCallFrame): |
- (WebCore::JSInjectedScriptHost::inspectedNode): |
- * bindings/js/JSNamedNodeMapCustom.cpp: |
- (WebCore::JSNamedNodeMap::nameGetter): |
- * bindings/js/JSNodeListCustom.cpp: |
- (WebCore::callNodeList): |
- (WebCore::JSNodeList::nameGetter): |
- * bindings/js/JSScriptProfileNodeCustom.cpp: |
- (WebCore::JSScriptProfileNode::children): |
- * bindings/js/JSStyleSheetListCustom.cpp: |
- (WebCore::JSStyleSheetList::nameGetter): |
- * bindings/js/JSXSLTProcessorCustom.cpp: |
- (WebCore::JSXSLTProcessor::transformToFragment): |
- (WebCore::JSXSLTProcessor::transformToDocument): |
- * bindings/scripts/CodeGeneratorJS.pm: |
- Propagate stored global object down access chain. |
- |
- * css/MediaQueryListListener.cpp: |
- (WebCore::MediaQueryListListener::queryChanged): |
- Replace use of deprecated toJS overload with explicit call to deprecatedGlobalObjectForPrototype, |
- we will want refactor this to match other callbacks so we can remove this. |
- |
-2011-04-22 Naoki Takano <takano.naoki@gmail.com> |
- |
- Reviewed by Eric Seidel. |
- |
- [Chromium]Mac UI polish to add left/right padding space for autofill popup window. |
- https://bugs.webkit.org/show_bug.cgi?id=58960 |
- http://code.google.com/p/chromium/issues/detail?id=51077 |
- |
- No new test because Chromium Autofill popup window doesn't have any test framework. |
- This assumes only AutofillPopupMenuClient gives TexFieldPart appearance. |
- Only if Autofill popup window has right/left padding. |
- The code should be removed after Autofill popup window logic is separated from WebKit to Chromium. |
- |
- * rendering/RenderThemeChromiumMac.h: Add popupInternalPaddingRight() and popupInternalPaddingLeft(). |
- * rendering/RenderThemeChromiumMac.mm: |
- (WebCore::RenderThemeChromiumMac::popupInternalPaddingLeft): Return padding size only if TextFieldPart is specified. |
- (WebCore::RenderThemeChromiumMac::popupInternalPaddingRight): Return padding size only if TextFieldPart is specified. |
- |
-2011-04-21 Adam Roben <aroben@apple.com> |
- |
- Add code to enumerate all the functions imported by a particular Windows binary |
- |
- PEImage is where all the logic starts. It represents a single Portable Executable (PE) |
- binary. The various enumerator classes are used in conjunction with PEImage to enumerate the |
- modules and functions imported by that image. |
- |
- I couldn't figure out how to write a test for this. :-( |
- |
- Part of <http://webkit.org/b/51063> <rdar://problem/8769281> REGRESSION (WebKit2): No |
- context menu appears when right-clicking on windowless Flash plugin |
- |
- Reviewed by Sam Weinig, and given a good once-over by Jeff Miller. |
- |
- * WebCore.vcproj/WebCore.vcproj: Added the new files. |
- |
- * platform/win/PEImage.cpp: Added. |
- (WebCore::PEImage::PEImage): Checks that the module contains the expected magic numbers, |
- then initializes m_ntHeaders using the module. |
- (WebCore::PEImage::convertRVAToAddress): Converts a relative virtual address (RVA) to a |
- non-relative address, which can then be dereferenced. (The PE format uses RVAs in lots of |
- places.) |
- (WebCore::PEImage::dataDirectoryEntryAddress): Returns the address of the data associated |
- with the given entry in the image's DataDirectory. |
- |
- * platform/win/PEImage.h: Added. |
- (WebCore::PEImage::isValid): We're a valid image if the validation code in our constructor |
- succeeded, in which case m_ntHeaders will be set. |
- |
- * platform/win/ImportedFunctionsEnumerator.cpp: Added. This class enumerates all the |
- functions imported by the image from a single module by walking the Import Name Table (INT) |
- and Import Address Table (IAT). |
- (WebCore::ImportedFunctionsEnumerator::ImportedFunctionsEnumerator): Initialize our members. |
- (WebCore::ImportedFunctionsEnumerator::isAtEnd): The INT and IAT are both 0-terminated |
- arrays. |
- (WebCore::ImportedFunctionsEnumerator::next): Step to the next entries in the INT and IAT. |
- (WebCore::ImportedFunctionsEnumerator::currentFunctionName): Gets the address of the |
- function's name, if possible, using the INT. |
- (WebCore::ImportedFunctionsEnumerator::addressOfCurrentFunctionPointer): Returns the address |
- of the function pointer for the current function. This function pointer is used whenever |
- code in m_image calls the current function. (A future patch will take advantage of this by |
- overwriting this function pointer with another, thereby redirecting calls to that function.) |
- |
- * platform/win/ImportedFunctionsEnumerator.h: Added. |
- |
- * platform/win/ImportedModulesEnumeratorBase.h: Added. This base class is used by |
- ImportedModulesEnumerator and DelayLoadedModulesEnumerator, and allows code to be written |
- that works with either one. |
- |
- * platform/win/ImportedModulesEnumerator.cpp: Added. This class enumerates all the |
- non-delay-loaded modules imported by the image by walking the image's import array. |
- (WebCore::ImportedModulesEnumerator::ImportedModulesEnumerator): Module import information |
- is stored in the IMAGE_DIRECTORY_ENTRY_IMPORT entry of the image's DataDirectory. |
- (WebCore::ImportedModulesEnumerator::isAtEnd): The import array is 0-terminated. |
- (WebCore::ImportedModulesEnumerator::next): Steps along the import array. |
- (WebCore::ImportedModulesEnumerator::currentModuleName): Gets the address of the module's |
- name. |
- (WebCore::ImportedModulesEnumerator::functionsEnumerator): Get's the addresses of the INT |
- and IAT for this module and wraps them in an ImportedFunctionsEnumerator. |
- |
- * platform/win/ImportedModulesEnumerator.h: Added. |
- |
- * platform/win/DelayLoadedModulesEnumerator.cpp: Added. This class enumerates all the |
- delay-loaded modules that are imported by the given PEImage by walking the image's |
- ImageDelayDescr array. |
- (WebCore::DelayLoadedModulesEnumerator::DelayLoadedModulesEnumerator): Delay-load |
- information is stored in the IMAGE_DIRECTORY_ENTRY_DELAY_IMPORT entry of the image's |
- DataDirectory. |
- (WebCore::DelayLoadedModulesEnumerator::isAtEnd): The delay-loaded modules array is |
- 0-terminated. |
- (WebCore::DelayLoadedModulesEnumerator::next): Steps along the delay-loaded modules array. |
- (WebCore::DelayLoadedModulesEnumerator::currentModuleName): Gets the address of the module's |
- name out of the descriptor. |
- (WebCore::DelayLoadedModulesEnumerator::functionsEnumerator): Gets the INT and IAT for the |
- current module out of the descriptor and makes an ImportedFunctionsEnumerator from them. |
- (WebCore::DelayLoadedModulesEnumerator::convertPotentialRVAToAddress): Pre-VC7.0 images |
- stored non-relative addresses in the ImageDelayDescr structures, while VC7.0 and later |
- images store RVAs. This function abstracts away the difference. |
- |
- * platform/win/DelayLoadedModulesEnumerator.h: Added. |
- |
-2011-04-21 Adam Roben <aroben@apple.com> |
- |
- Add a shared do-nothing window class that anyone can use |
- |
- Fixes <http://webkit.org/b/59102> WebKit has too many dummy window classes |
- |
- Reviewed by Anders Carlsson. |
- |
- * WebCore.vcproj/WebCore.vcproj: Added DefWndProcWindowClass.{cpp,h}. Moved Arena.cpp out of |
- platform/win and into platform. Let VS reorder other files. |
- |
- * platform/graphics/ca/win/CACFLayerTreeHost.cpp: |
- (WebCore::CACFLayerTreeHost::acceleratedCompositingAvailable): Changed to use the shared |
- dummy window class instead of creating our own. |
- |
- * platform/win/DefWndProcWindowClass.cpp: Added. |
- (WebCore::registerClass): Register it! |
- (WebCore::defWndProcWindowClassName): Register it and return it! |
- |
- * platform/win/DefWndProcWindowClass.h: Added. |
- |
-2011-04-21 MORITA Hajime <morrita@google.com> |
- |
- Reviewed by Dimitri Glazkov. |
- |
- Content of <details> should be forwarded through the shadow DOM |
- https://bugs.webkit.org/show_bug.cgi?id=59055 |
- |
- - Made ShadowContentElement abstract and added shouldInclude() pure virtual method. |
- - Added SummaryContentElement, DetailsContentElement, DetailsSummaryElement as |
- subclasses of ShadowContentElement. |
- - Gave two shadow children for HTMLDetailsElement. One is DetailsContentElement, that |
- handles non-summary contents of <details>. Another is SummaryContentElement, that |
- handles the main summary element. |
- - Removed most of RenderDetails custom layout mechanism, which is replaced by |
- ShadowContentElement subclasses. |
- |
- Tests: fast/html/details-add-child-1.html |
- fast/html/details-add-child-2.html |
- fast/html/details-remove-child-1.html |
- fast/html/details-remove-child-2.html |
- |
- * dom/ShadowRoot.cpp: |
- (WebCore::ShadowRoot::contentContainerFor): |
- (WebCore::ShadowRoot::hostChildrenChanged): |
- (WebCore::ShadowRoot::hasContentElement): |
- * dom/ShadowRoot.h: |
- * html/HTMLDetailsElement.cpp: |
- (WebCore::DetailsContentElement::DetailsContentElement): |
- (WebCore::DetailsContentElement::create): |
- (WebCore::DetailsContentElement::shouldInclude): |
- (WebCore::DetailsSummaryElement::DetailsSummaryElement): |
- (WebCore::DetailsSummaryElement::create): |
- (WebCore::DetailsSummaryElement::shouldInclude): |
- (WebCore::HTMLDetailsElement::create): |
- (WebCore::HTMLDetailsElement::HTMLDetailsElement): |
- (WebCore::HTMLDetailsElement::ensureShadowSubtreeOf): |
- (WebCore::findSummaryFor): |
- (WebCore::HTMLDetailsElement::ensureMainSummary): |
- (WebCore::HTMLDetailsElement::refreshMainSummary): |
- (WebCore::HTMLDetailsElement::createShadowSubtree): |
- * html/HTMLDetailsElement.h: |
- * html/HTMLSummaryElement.cpp: |
- (WebCore::SummaryContentElement::SummaryContentElement): |
- (WebCore::SummaryContentElement::shouldInclude): |
- (WebCore::SummaryContentElement::create): |
- (WebCore::HTMLSummaryElement::createShadowSubtree): |
- * html/shadow/ShadowContentElement.h: |
- * rendering/RenderDetails.cpp: |
- (WebCore::RenderDetails::RenderDetails): |
- (WebCore::RenderDetails::addChild): |
- * rendering/RenderDetails.h: |
- * rendering/RenderSummary.cpp: |
- * rendering/RenderSummary.h: |
- |
-2011-04-21 Simon Fraser <simon.fraser@apple.com> |
- |
- Reviewed by Dan Bernstein. |
- |
- Extra GraphicsContext::save() in InlineTextBox::paintDecoration() |
- https://bugs.webkit.org/show_bug.cgi?id=59109 |
- |
- Remove an extraneous GraphicsContext::save() in InlineTextBox::paintDecoration(). |
- |
- Add an assertion that fires when a GraphicsContext() is destroyed with a non- |
- empty state stack. Fix canvas to avoid this assertion, since bad content might |
- trigger it. |
- |
- Also add an assertion that fires for mismatched begin/endTransparencyLayers. |
- |
- Test: fast/text/multiple-text-shadow-alpha.html |
- |
- * html/HTMLCanvasElement.cpp: |
- (WebCore::HTMLCanvasElement::~HTMLCanvasElement): |
- Make sure that the CanvasRenderingContext goes away before the ImageBuffer, |
- so that CanvasRenderingContext2D has a chance to unwind the state stack in |
- the ImageBuffer's GraphicsContext. |
- |
- * html/canvas/CanvasRenderingContext2D.cpp: |
- (WebCore::CanvasRenderingContext2D::~CanvasRenderingContext2D): |
- Balance unmatched state saves in the canvas context. |
- |
- * platform/graphics/GraphicsContext.cpp: |
- (WebCore::GraphicsContext::~GraphicsContext): |
- Assert that the stack is empty. |
- |
- * platform/graphics/cg/GraphicsContextPlatformPrivateCG.h: |
- (WebCore::GraphicsContextPlatformPrivate::GraphicsContextPlatformPrivate): |
- (WebCore::GraphicsContextPlatformPrivate::~GraphicsContextPlatformPrivate): |
- (WebCore::GraphicsContextPlatformPrivate::beginTransparencyLayer): |
- (WebCore::GraphicsContextPlatformPrivate::endTransparencyLayer): |
- Changes to allow for asserting for unmatched transparency layer begin/ends. |
- |
- * rendering/InlineTextBox.cpp: |
- (WebCore::InlineTextBox::paintDecoration): |
- Remove an unbalanced save(). |
- |
-2011-04-22 Pavel Feldman <pfeldman@google.com> |
- |
- Reviewed by Yury Semikhatsky. |
- |
- Web Inspector: add Save As context menu item to the TextViewer. |
- https://bugs.webkit.org/show_bug.cgi?id=59196 |
- |
- * inspector/front-end/ResourceView.js: |
- (WebInspector.ResourceSourceFrame.prototype.requestContent): |
- (WebInspector.ResourceSourceFrame.prototype.suggestedFileName): |
- (WebInspector.RevisionSourceFrame.prototype.requestContent): |
- (WebInspector.RevisionSourceFrame.prototype.suggestedFileName): |
- * inspector/front-end/ScriptsPanel.js: |
- (WebInspector.ScriptsPanel.prototype._createSourceFrame): |
- (WebInspector.SourceFrameDelegateForScriptsPanel): |
- (WebInspector.SourceFrameDelegateForScriptsPanel.prototype.formatSourceFilesToggled): |
- (WebInspector.SourceFrameDelegateForScriptsPanel.prototype.suggestedFileName): |
- * inspector/front-end/SourceFrame.js: |
- (WebInspector.SourceFrame.prototype._initializeTextViewer): |
- (WebInspector.SourceFrame.prototype.populateTextAreaContextMenu): |
- (WebInspector.SourceFrame.prototype.suggestedFileName): |
- (WebInspector.SourceFrameDelegate.prototype.formatSourceFilesToggled): |
- (WebInspector.SourceFrameDelegate.prototype.suggestedFileName): |
- * inspector/front-end/TextViewer.js: |
- (WebInspector.TextViewer.prototype._contextMenu): |
- (WebInspector.TextViewerDelegate.prototype.cancelEditing): |
- (WebInspector.TextViewerDelegate.prototype.populateLineGutterContextMenu): |
- (WebInspector.TextViewerDelegate.prototype.populateTextAreaContextMenu): |
- (WebInspector.TextViewerDelegate.prototype.suggestedFileName): |
- |
-2011-04-22 Mikhail Naganov <mnaganov@chromium.org> |
- |
- Reviewed by Pavel Feldman. |
- |
- Web Inspector: [Chromium] Move detailed heap snapshots storage and processing into workers. |
- https://bugs.webkit.org/show_bug.cgi?id=59094 |
- |
- * WebCore.gypi: |
- * WebCore.vcproj/WebCore.vcproj: |
- * gyp/streamline-inspector-source.sh: |
- * inspector/front-end/DetailedHeapshotGridNodes.js: |
- (WebInspector.HeapSnapshotGridNode.prototype.dispose): |
- (WebInspector.HeapSnapshotObjectNode.prototype._createProvider): |
- (WebInspector.HeapSnapshotInstanceNode.prototype._createProvider): |
- (WebInspector.HeapSnapshotConstructorNode.prototype._createNodesProvider): |
- (WebInspector.HeapSnapshotIteratorsTuple.prototype.dispose): |
- (WebInspector.HeapSnapshotDiffNode.prototype.calculateDiff.diffCalculated): |
- (WebInspector.HeapSnapshotDiffNode.prototype._createNodesProvider.createProvider): |
- (WebInspector.HeapSnapshotDiffNode.prototype._createNodesProvider): |
- (WebInspector.HeapSnapshotDominatorObjectNode.prototype._createProvider): |
- (MixInSnapshotNodeFunctions): |
- * inspector/front-end/DetailedHeapshotView.js: |
- (WebInspector.HeapSnapshotSortableDataGrid.prototype.dispose): |
- (WebInspector.HeapSnapshotSortableDataGrid.prototype.resetSortingCache): |
- (WebInspector.HeapSnapshotDiffDataGrid.prototype.setBaseDataSource): |
- (WebInspector.HeapSnapshotRetainingPathsList.prototype.dispose): |
- (WebInspector.HeapSnapshotRetainingPathsList.prototype._resetPaths): |
- (WebInspector.HeapSnapshotRetainingPathsList.prototype.setDataSource): |
- (WebInspector.HeapSnapshotRetainingPathsList.prototype.showNext): |
- (WebInspector.HeapSnapshotRetainingPathsList.prototype.searchCancelled): |
- (WebInspector.HeapSnapshotRetainingPathsList.prototype._setRootChildrenForFinder): |
- (WebInspector.DetailedHeapshotView.prototype.dispose): |
- (WebInspector.DetailedHeapshotView.prototype.get profileWrapper): |
- (WebInspector.DetailedHeapshotView.prototype.get baseProfileWrapper): |
- * inspector/front-end/HeapSnapshot.js: |
- (WebInspector.HeapSnapshotLoader): |
- (WebInspector.HeapSnapshotLoader.prototype.finishLoading): |
- (WebInspector.HeapSnapshotLoader.prototype.pushJSONChunk): |
- (WebInspector.HeapSnapshot.prototype.pushBaseIds): |
- (WebInspector.HeapSnapshot.prototype.createDiff): |
- (WebInspector.HeapSnapshot.prototype._parseFilter): |
- (WebInspector.HeapSnapshot.prototype.createEdgesProvider): |
- (WebInspector.HeapSnapshot.prototype.createNodesProvider): |
- (WebInspector.HeapSnapshot.prototype.createPathFinder): |
- (WebInspector.HeapSnapshot.prototype.updateStaticData): |
- (WebInspector.HeapSnapshotPathFinder): |
- (WebInspector.HeapSnapshotPathFinder.prototype.updateRoots): |
- (WebInspector.HeapSnapshotPathFinder.prototype._reset): |
- * inspector/front-end/HeapSnapshotProxy.js: |
- (WebInspector.HeapSnapshotRealWorker): |
- (WebInspector.HeapSnapshotRealWorker.prototype._messageReceived): |
- (WebInspector.HeapSnapshotRealWorker.prototype.postMessage): |
- (WebInspector.HeapSnapshotRealWorker.prototype.terminate): |
- (WebInspector.HeapSnapshotFakeWorker): |
- (WebInspector.HeapSnapshotFakeWorker.prototype.postMessage): |
- (WebInspector.HeapSnapshotFakeWorker.prototype.terminate): |
- (WebInspector.HeapSnapshotFakeWorker.prototype._postMessageFromWorker): |
- (WebInspector.HeapSnapshotWorker): |
- (WebInspector.HeapSnapshotWorker.prototype.createObject): |
- (WebInspector.HeapSnapshotWorker.prototype.dispose): |
- (WebInspector.HeapSnapshotWorker.prototype.disposeObject): |
- (WebInspector.HeapSnapshotWorker.prototype.callGetter): |
- (WebInspector.HeapSnapshotWorker.prototype.callFactoryMethod.wrapCallback): |
- (WebInspector.HeapSnapshotWorker.prototype.callFactoryMethod): |
- (WebInspector.HeapSnapshotWorker.prototype.callMethod): |
- (WebInspector.HeapSnapshotWorker.prototype._findFunction): |
- (WebInspector.HeapSnapshotWorker.prototype._messageReceived): |
- (WebInspector.HeapSnapshotWorker.prototype._postMessage): |
- (WebInspector.HeapSnapshotProxyObject): |
- (WebInspector.HeapSnapshotProxyObject.prototype._callWorker): |
- (WebInspector.HeapSnapshotProxyObject.prototype.dispose): |
- (WebInspector.HeapSnapshotProxyObject.prototype.disposeWorker): |
- (WebInspector.HeapSnapshotProxyObject.prototype.callFactoryMethod): |
- (WebInspector.HeapSnapshotProxyObject.prototype.callGetter): |
- (WebInspector.HeapSnapshotProxyObject.prototype.callMethod): |
- (WebInspector.HeapSnapshotLoaderProxy): |
- (WebInspector.HeapSnapshotLoaderProxy.prototype.finishLoading.callLoadCallbacks): |
- (WebInspector.HeapSnapshotLoaderProxy.prototype.finishLoading.updateStaticData): |
- (WebInspector.HeapSnapshotLoaderProxy.prototype.finishLoading): |
- (WebInspector.HeapSnapshotLoaderProxy.prototype.get loaded): |
- (WebInspector.HeapSnapshotLoaderProxy.prototype.startLoading): |
- (WebInspector.HeapSnapshotLoaderProxy.prototype.pushJSONChunk): |
- (WebInspector.HeapSnapshotProxy): |
- (WebInspector.HeapSnapshotProxy.prototype.aggregates): |
- (WebInspector.HeapSnapshotProxy.prototype.createDiff): |
- (WebInspector.HeapSnapshotProxy.prototype.createEdgesProvider): |
- (WebInspector.HeapSnapshotProxy.prototype.createNodesProvider): |
- (WebInspector.HeapSnapshotProxy.prototype.createPathFinder): |
- (WebInspector.HeapSnapshotProxy.prototype.dispose): |
- (WebInspector.HeapSnapshotProxy.prototype.finishLoading): |
- (WebInspector.HeapSnapshotProxy.prototype.get loaded): |
- (WebInspector.HeapSnapshotProxy.prototype.get nodeCount): |
- (WebInspector.HeapSnapshotProxy.prototype.nodeFieldValuesByIndex): |
- (WebInspector.HeapSnapshotProxy.prototype.pushBaseIds): |
- (WebInspector.HeapSnapshotProxy.prototype.get rootNodeIndex): |
- (WebInspector.HeapSnapshotProxy.prototype.updateStaticData): |
- (WebInspector.HeapSnapshotProxy.prototype.startLoading): |
- (WebInspector.HeapSnapshotProxy.prototype.get totalSize): |
- (WebInspector.HeapSnapshotProxy.prototype.get uid): |
- (WebInspector.HeapSnapshotProviderProxy): |
- (WebInspector.HeapSnapshotProviderProxy.prototype.isEmpty): |
- (WebInspector.HeapSnapshotProviderProxy.prototype.serializeNextItems): |
- (WebInspector.HeapSnapshotProviderProxy.prototype.sortAndRewind): |
- (WebInspector.HeapSnapshotPathFinderProxy): |
- (WebInspector.HeapSnapshotPathFinderProxy.prototype.findNext): |
- (WebInspector.HeapSnapshotPathFinderProxy.prototype.updateRoots): |
- (WebInspector.HeapSnapshotsDiffProxy): |
- (WebInspector.HeapSnapshotsDiffProxy.prototype.calculate): |
- (WebInspector.HeapSnapshotsDiffProxy.prototype.pushBaseIds): |
- (WebInspector.HeapSnapshotsDiffProxy.prototype.pushBaseSelfSizes): |
- * inspector/front-end/HeapSnapshotWorker.js: Added. |
- (WebInspector.UIString): |
- (postMessageWrapper): |
- * inspector/front-end/HeapSnapshotWorkerDispatcher.js: Added. |
- (WebInspector.HeapSnapshotWorkerDispatcher): |
- (WebInspector.HeapSnapshotWorkerDispatcher.prototype._findFunction): |
- (WebInspector.HeapSnapshotWorkerDispatcher.prototype.dispatchMessage): |
- * inspector/front-end/ProfilesPanel.js: |
- (WebInspector.ProfilesPanel.prototype.loadHeapSnapshot): |
- (WebInspector.ProfilesPanel.prototype._finishHeapSnapshot.else.parsed): |
- (WebInspector.ProfilesPanel.prototype._finishHeapSnapshot): |
- * inspector/front-end/WebKit.qrc: |
- * inspector/front-end/inspector.html: |
- |
-2011-04-22 Pavel Podivilov <podivilov@chromium.org> |
- |
- Reviewed by Pavel Feldman. |
- |
- Web Inspector: fix de-obfuscation of scripts inlined in html. |
- https://bugs.webkit.org/show_bug.cgi?id=58538 |
- |
- Use html tokenizer to find script boundaries instead of relying on scriptParsed events. |
- |
- Test: inspector/debugger/script-formatter.html |
- |
- * WebCore.gypi: |
- * WebCore.vcproj/WebCore.vcproj: |
- * inspector/front-end/JavaScriptFormatter.js: Renamed from Source/WebCore/inspector/front-end/UglifyJS/JavaScriptFormatter.js. |
- * inspector/front-end/ScriptFormatter.js: |
- (WebInspector.ScriptFormatter): |
- (WebInspector.ScriptFormatter.prototype.formatContent): |
- (WebInspector.ScriptFormatter.prototype._didFormatContent): |
- * inspector/front-end/ScriptFormatterWorker.js: |
- (onmessage): |
- (formatScript): |
- (HTMLScriptFormatter): |
- (HTMLScriptFormatter.prototype.format): |
- (HTMLScriptFormatter.prototype.scriptStarted): |
- (HTMLScriptFormatter.prototype.scriptEnded): |
- (HTMLScriptFormatter.prototype.styleSheetStarted): |
- (HTMLScriptFormatter.prototype.styleSheetEnded): |
- * inspector/front-end/SourceFile.js: |
- (WebInspector.FormattedSourceFile.prototype._didRequestContent): |
- * inspector/front-end/SourceHTMLTokenizer.js: |
- (WebInspector.SourceHTMLTokenizer.prototype.scriptStarted): |
- (WebInspector.SourceHTMLTokenizer.prototype.scriptEnded): |
- (WebInspector.SourceHTMLTokenizer.prototype.styleSheetStarted): |
- (WebInspector.SourceHTMLTokenizer.prototype.styleSheetEnded): |
- (WebInspector.SourceHTMLTokenizer.prototype.nextToken): |
- * inspector/front-end/SourceHTMLTokenizer.re2js: |
- * inspector/front-end/WebKit.qrc: |
- |
-2011-04-22 Andrey Kosyakov <caseq@chromium.org> |
- |
- Reviewed by Pavel Feldman. |
- |
- Web Inspector: [Extensions API] misc API cleanup |
- https://bugs.webkit.org/show_bug.cgi?id=59194 |
- |
- * inspector/front-end/ExtensionAPISchema.json: |
- * inspector/front-end/ExtensionPanel.js: |
- (WebInspector.ExtensionSidebarPane.prototype._setObject): |
- |
-2011-04-21 Pavel Feldman <pfeldman@google.com> |
- |
- Reviewed by Yury Semikhatsky. |
- |
- Web Inspector: group content scripts in the scripts panel drop down. |
- https://bugs.webkit.org/show_bug.cgi?id=59108 |
- |
- * English.lproj/localizedStrings.js: |
- * inspector/front-end/ScriptsPanel.js: |
- (WebInspector.ScriptsPanel.prototype._addOptionToFilesSelect.optionCompare): |
- (WebInspector.ScriptsPanel.prototype._addOptionToFilesSelect): |
- |
-2011-04-22 Mike West <mkwst@google.com> |
- |
- Reviewed by Pavel Feldman. |
- |
- Web Inspector: Tiny improvement to UI for adding an XHR breakpoint |
- https://bugs.webkit.org/show_bug.cgi?id=59100 |
- |
- Improve web inspector UI around adding XHR breakpoints by adding |
- a textual label to the input-like `span` used to specify which |
- XHR requests the breakpoint ought to target. |
- |
- * inspector/front-end/BreakpointsSidebarPane.js: |
- (WebInspector.XHRBreakpointsSidebarPane.prototype._addButtonClicked.finishEditing): |
- (WebInspector.XHRBreakpointsSidebarPane.prototype._addButtonClicked): |
- * inspector/front-end/inspector.css: |
- (#breakpoint-condition-input): |
- |
-2011-04-22 Pavel Podivilov <podivilov@chromium.org> |
- |
- Reviewed by Pavel Feldman. |
- |
- Web Inspector: js formatter should never insert new tokens or remove existing tokens and comments. |
- https://bugs.webkit.org/show_bug.cgi?id=59191 |
- |
- * WebCore.gypi: |
- * WebCore.vcproj/WebCore.vcproj: |
- * inspector/front-end/ScriptFormatter.js: |
- (WebInspector.ScriptFormatter.prototype.formatContent.didFormatChunks): |
- (WebInspector.ScriptFormatter.prototype.formatContent): |
- * inspector/front-end/ScriptFormatterWorker.js: |
- (onmessage): |
- * inspector/front-end/SourceFile.js: |
- (WebInspector.SourceFile.prototype._loadResourceContent): |
- (WebInspector.FormattedSourceFile.prototype._didRequestContent): |
- (WebInspector.FormattedSourceMapping): |
- (WebInspector.FormattedSourceMapping.prototype.scriptLocationToSourceLine): |
- (WebInspector.FormattedSourceMapping.prototype.sourceLineToScriptLocation): |
- (WebInspector.FormattedSourceMapping.prototype._convertPosition): |
- * inspector/front-end/UglifyJS/JavaScriptFormatter.js: Added. |
- (FormattedContentBuilder): Helper class for building formatted script content. |
- (FormattedContentBuilder.prototype.addToken): |
- (FormattedContentBuilder.prototype.addSpace): |
- (FormattedContentBuilder.prototype.addNewLine): |
- (FormattedContentBuilder.prototype.increaseNestingLevel): |
- (FormattedContentBuilder.prototype.decreaseNestingLevel): |
- (FormattedContentBuilder.prototype.content): |
- (FormattedContentBuilder.prototype.mapping): |
- (Tokenizer): Wrapper class for UglifyJS tokenizer. |
- (Tokenizer.prototype.content): |
- (Tokenizer.prototype.next): |
- (Tokenizer.prototype._convertUglifyToken): |
- (JavaScriptFormatter): New formatter based on UglifyJS tokenizer. |
- (JavaScriptFormatter.prototype.format): |
- * inspector/front-end/UglifyJS/process.js: Removed. |
- * inspector/front-end/WebKit.qrc: |
- |
-2011-04-22 Mikhail Naganov <mnaganov@chromium.org> |
- |
- Reviewed by Pavel Feldman. |
- |
- Web Inspector: [Chromium] Refactor code prior to moving detailed heap snapshots into workers. |
- https://bugs.webkit.org/show_bug.cgi?id=59189 |
- |
- * WebCore.gypi: |
- * WebCore.vcproj/WebCore.vcproj: |
- * inspector/front-end/BinarySearch.js: Added. |
- * inspector/front-end/DetailedHeapshotGridNodes.js: |
- (WebInspector.HeapSnapshotGridNode.prototype._populate.sorted): |
- (WebInspector.HeapSnapshotGridNode.prototype._populate): |
- (WebInspector.HeapSnapshotGridNode.prototype.populateChildren): |
- (WebInspector.HeapSnapshotGridNode.prototype.sort.afterSort.afterPopulate): |
- (WebInspector.HeapSnapshotGridNode.prototype.sort.afterSort): |
- (WebInspector.HeapSnapshotGridNode.prototype.sort): |
- (WebInspector.HeapSnapshotObjectNode): |
- (WebInspector.HeapSnapshotObjectNode.prototype._createChildNode): |
- (WebInspector.HeapSnapshotInstanceNode.prototype._createChildNode): |
- (WebInspector.HeapSnapshotInstanceNode.prototype._enhanceData): |
- (WebInspector.HeapSnapshotInstanceNode.prototype.get isDeletedNode): |
- (WebInspector.HeapSnapshotDiffNode.prototype.calculateDiff.diffCalculated): |
- (WebInspector.HeapSnapshotDiffNode.prototype.calculateDiff.baseIdsReceived): |
- (WebInspector.HeapSnapshotDiffNode.prototype.get data): |
- * inspector/front-end/DetailedHeapshotView.js: |
- (WebInspector.HeapSnapshotSortableDataGrid.prototype.sortingChanged.sort): |
- (WebInspector.HeapSnapshotSortableDataGrid.prototype.sortingChanged): |
- (WebInspector.HeapSnapshotSortableDataGrid.prototype._performSorting): |
- (WebInspector.HeapSnapshotSortableDataGrid.prototype.recursiveSortingEnter): |
- (WebInspector.HeapSnapshotSortableDataGrid.prototype.recursiveSortingLeave): |
- (WebInspector.HeapSnapshotContainmentDataGrid): |
- (WebInspector.HeapSnapshotContainmentDataGrid.prototype._defaultPopulateCount.100.setDataSource): |
- (WebInspector.HeapSnapshotContainmentDataGrid.prototype.sortingChanged): |
- (WebInspector.HeapSnapshotDiffDataGrid.prototype.setBaseDataSource): |
- (WebInspector.HeapSnapshotDiffDataGrid.prototype.populateChildren.baseAggregatesReceived.aggregatesReceived.addNodeIfNonZeroDiff): |
- (WebInspector.HeapSnapshotDiffDataGrid.prototype.populateChildren.baseAggregatesReceived.aggregatesReceived): |
- (WebInspector.HeapSnapshotDiffDataGrid.prototype.populateChildren): |
- (WebInspector.HeapSnapshotDominatorsDataGrid): |
- (WebInspector.HeapSnapshotDominatorsDataGrid.prototype._defaultPopulateCount.25.setDataSource): |
- (WebInspector.HeapSnapshotDominatorsDataGrid.prototype.sortingChanged): |
- (WebInspector.HeapSnapshotRetainingPathsList.prototype._performSorting): |
- (WebInspector.DetailedHeapshotView.prototype._changeBase.baseProfileLoaded): |
- (WebInspector.DetailedHeapshotView.prototype._changeBase): |
- (WebInspector.DetailedHeapshotView.prototype._changeView): |
- (WebInspector.DetailedHeapshotView.prototype._showStringContentPopup.displayString): |
- (WebInspector.DetailedHeapshotView.prototype._showStringContentPopup): |
- * inspector/front-end/HeapSnapshot.js: |
- (WebInspector.HeapSnapshot.prototype.dispose): |
- (WebInspector.HeapSnapshot.prototype.pushBaseNodeIds): |
- (WebInspector.HeapSnapshotFilteredOrderedIterator.prototype.serializeNextItems): |
- (WebInspector.HeapSnapshotFilteredOrderedIterator.prototype.sortAndRewind): |
- (WebInspector.HeapSnapshotEdgesProvider.prototype._serialize): |
- (WebInspector.HeapSnapshotNodesProvider.prototype._serialize): |
- (WebInspector.HeapSnapshotsDiff.prototype.calculate): |
- (WebInspector.HeapSnapshotsDiff.prototype.pushBaseIds): |
- (WebInspector.HeapSnapshotsDiff.prototype.pushBaseSelfSizes): |
- * inspector/front-end/HeapSnapshotProxy.js: |
- (WebInspector.HeapSnapshotProxy.prototype.createEdgesProvider): |
- (WebInspector.HeapSnapshotProxy.prototype.createNodesProvider): |
- (WebInspector.HeapSnapshotProxy.prototype.pushBaseIds): |
- (WebInspector.HeapSnapshotProviderProxy): |
- (WebInspector.HeapSnapshotProviderProxy.prototype.serializeNextItems): |
- (WebInspector.HeapSnapshotProviderProxy.prototype.sortAndRewind): |
- (WebInspector.HeapSnapshotsDiffProxy.prototype.pushBaseIds): |
- (WebInspector.HeapSnapshotsDiffProxy.prototype.pushBaseSelfSizes): |
- * inspector/front-end/PleaseWaitMessage.js: |
- (WebInspector.PleaseWaitMessage.prototype.showAndWaitFor): |
- (WebInspector.PleaseWaitMessage.prototype.unlistenAndHide): |
- * inspector/front-end/WebKit.qrc: |
- * inspector/front-end/inspector.html: |
- * inspector/front-end/utilities.js: |
- |
-2011-04-22 Mikhail Naganov <mnaganov@chromium.org> |
- |
- Unreviewed fix after r84484 -- conditional statement was inverted incorrectly. |
- |
- * inspector/front-end/ProfilesPanel.js: |
- (WebInspector.ProfilesPanel.prototype.handleShortcut): |
- |
-2011-04-22 Annie Sullivan <sullivan@chromium.org> |
- |
- Reviewed by Pavel Feldman. |
- |
- Web Inspector: Timeline is too narrow |
- https://bugs.webkit.org/show_bug.cgi?id=58399 |
- |
- Changed Network Panel widths so that Timeline is 50%. |
- Added titles to the values in the other columns, so that a tooltip will show the value if the column is too narrow. |
- |
- * inspector/front-end/NetworkPanel.js: |
- (WebInspector.NetworkPanel.prototype._createTable): Changed column widths. |
- (WebInspector.NetworkPanel.prototype._toggleGridMode): Changed column widths. |
- (WebInspector.NetworkDataGridNode.prototype.refreshResource): Added titles to column values. |
- (WebInspector.NetworkDataGridNode.prototype._refreshStatusCell): Ditto |
- (WebInspector.NetworkDataGridNode.prototype._refreshSizeCell): Ditto |
- (WebInspector.NetworkDataGridNode.prototype._refreshTimeCell): Ditto |
- * inspector/front-end/utilities.js: |
- (Element.prototype.setTextAndTitle): Added new method to set the textContent and title of an Element at the same time, so that it can have a tooltip. |
- |
-2011-04-22 Dan Bernstein <mitz@apple.com> |
- |
- Reviewed by Maciej Stachowiak. |
- |
- <rdar://problem/9322486> REGRESSION (r83397): Default controls fade out when playing audio-only media |
- https://bugs.webkit.org/show_bug.cgi?id=59180 |
- |
- Test: media/audio-controls-do-not-fade-out.html |
- |
- * html/HTMLMediaElement.cpp: |
- (WebCore::HTMLMediaElement::playbackProgressTimerFired): Only start fading out the controls if there is |
- a video track. |
- |
-2011-04-22 Luke Macpherson <macpherson@chromium.org> |
- |
- Reviewed by Eric Seidel. |
- |
- Add FontItalics and FontSmallCaps enums to be used instead of boolean values. |
- https://bugs.webkit.org/show_bug.cgi?id=59080 |
- |
- No new tests as no new functionality added. |
- |
- * css/CSSPrimitiveValueMappings.h: |
- Add casts to/from FontItalics and FontSmallCaps |
- * css/CSSStyleSelector.cpp: |
- Use new casts defined in PrimitiveValueMappings.h |
- * platform/graphics/Font.h: |
- Use FontItalics enum. |
- * platform/graphics/FontDescription.h: |
- Define and use FontItalics and FontSmallCaps enums. |
- |
-2011-04-21 Andrey Kosyakov <caseq@chromium.org> |
- |
- Reviewed by Yury Semikhatsky. |
- |
- Web Inspector: exception in front-end when user selects resource info for resource that does not have response headers yet |
- https://bugs.webkit.org/show_bug.cgi?id=59123 |
- |
- * inspector/front-end/ResourceCookiesView.js: |
- (WebInspector.ResourceCookiesView): |
- (WebInspector.ResourceCookiesView.prototype.show): |
- (WebInspector.ResourceCookiesView.prototype.get _gotCookies): |
- (WebInspector.ResourceCookiesView.prototype._buildCookiesTable): |
- |
-2011-04-21 Beth Dakin <bdakin@apple.com> |
- |
- Reviewed by Dan Bernstein. |
- |
- https://bugs.webkit.org/show_bug.cgi?id=57898 |
- REGRESSION (r82185): Scroll position not restored on navigation back to a page in |
- the page cache |
- -and corresponding- |
- <rdar://problem/9226652> |
- |
- The original fix for this change broke scroll-position restore for pages that are |
- *not* in the page cache. This patch fixes both cached and non-cached pages by |
- using the cachedScrollPosition() only for cached pages. |
- * loader/HistoryController.cpp: |
- (WebCore::HistoryController::saveScrollPositionAndViewStateToItem): |
- |
-2011-04-21 Luke Macpherson <macpherson@chromium.org> |
- |
- Reviewed by Eric Seidel. |
- |
- Remove redundant calls to RenderStyle::setDisplay() |
- https://bugs.webkit.org/show_bug.cgi?id=59060 |
- |
- No new tests / no new functionality added. |
- |
- * css/CSSStyleSelector.cpp: |
- (WebCore::CSSStyleSelector::applyProperty): |
- Remove redundant calls to RenderStyle::setDisplay() |
- |
-2011-04-21 John Bauman <jbauman@chromium.org> |
- |
- Reviewed by Kenneth Russell. |
- |
- WebGL context drawing buffer may not be cleared if FBO bound |
- https://bugs.webkit.org/show_bug.cgi?id=58956 |
- |
- Clear the drawing buffer even if it's not currently bound, by |
- switching between FBOs around the clear. Also, make sure that we |
- handle depth and stencil masks properly. |
- |
- No new tests, because testing this under DRT doesn't work. |
- |
- * html/canvas/WebGLRenderingContext.cpp: |
- (WebCore::WebGLRenderingContext::initializeNewContext): |
- (WebCore::WebGLRenderingContext::clearIfComposited): |
- (WebCore::WebGLRenderingContext::depthMask): |
- * html/canvas/WebGLRenderingContext.h: |
- * |
-2011-04-21 Chris Fleizach <cfleizach@apple.com> |
- |
- Reviewed by Beth Dakin. |
- |
- WebKit is erroneously modifying the DOM; adding 'aria-selected' to tables |
- https://bugs.webkit.org/show_bug.cgi?id=59140 |
- |
- WebKit should not modify the DOM when the screen reader requests certain things be set. There is no |
- specification that allows this behavior yet, but we are working on something for the future. |
- |
- * accessibility/AccessibilityRenderObject.cpp: |
- (WebCore::AccessibilityRenderObject::setARIAGrabbed): |
- (WebCore::AccessibilityRenderObject::setIsExpanded): |
- (WebCore::AccessibilityRenderObject::setSelected): |
- (WebCore::AccessibilityRenderObject::setValue): |
- |
-2011-04-21 Chris Fleizach <cfleizach@apple.com> |
- |
- Reviewed by Beth Dakin. |
- |
- AX: order AccessibilityRole alphabetically |
- https://bugs.webkit.org/show_bug.cgi?id=31524 |
- |
- * accessibility/AccessibilityObject.h: |
- |
-2011-04-21 Jia Pu <jpu@apple.com> |
- |
- Reviewed by Maciej Stachowiak. |
- |
- [Mac] correctionPanelTimerFired() crashes due to rangeToBeReplaced being cleared. |
- https://bugs.webkit.org/show_bug.cgi?id=59155 |
- <rdar://problem/9261698> |
- |
- Make sure the range isn't cleared before calling windowRectForRange() on it. Also removed unused functions in Editor.h, |
- and fixed a typo in function name. |
- |
- * editing/Editor.h: |
- * editing/SpellingCorrectionController.cpp: |
- (WebCore::SpellingCorrectionController::correctionPanelTimerFired): |
- (WebCore::SpellingCorrectionController::respondToChangedSelection): |
- * editing/SpellingCorrectionController.h: |
- (WebCore::SpellingCorrectionController::shouldStartTimerFor): |
- |
-2011-04-21 Maciej Stachowiak <mjs@apple.com> |
- |
- Reviewed by Adam Roben. |
- |
- Add a feature define to allow <details> and <summary> to be disabled |
- https://bugs.webkit.org/show_bug.cgi?id=59118 |
- <rdar://problem/9257045> |
- |
- No new tests; no behavior change. |
- |
- * Configurations/FeatureDefines.xcconfig: |
- * DerivedSources.make: |
- * GNUmakefile.am: |
- * features.pri: |
- * html/HTMLDetailsElement.cpp: |
- * html/HTMLSummaryElement.cpp: |
- * html/HTMLTagNames.in: |
- * html/shadow/DetailsMarkerControl.cpp: |
- * rendering/RenderBlock.cpp: |
- (WebCore::canMergeContiguousAnonymousBlocks): |
- * rendering/RenderDetails.cpp: |
- * rendering/RenderDetails.h: |
- * rendering/RenderDetailsMarker.cpp: |
- * rendering/RenderDetailsMarker.h: |
- * rendering/RenderObject.h: |
- * rendering/RenderSummary.cpp: |
- * rendering/RenderSummary.h: |
- * rendering/RenderTreeAsText.cpp: |
- (WebCore::RenderTreeAsText::writeRenderObject): |
- |
-2011-04-21 Dimitri Glazkov <dglazkov@chromium.org> |
- |
- Reviewed by Adele Peterson. |
- |
- REGRESSION (r83386): Typing in WSJ.com search field doesn't clear placeholder text |
- https://bugs.webkit.org/show_bug.cgi?id=59053 |
- |
- Turns out, the relatedTarget of an event could be also an ancestor of an event, |
- not just its parent. Duh. |
- |
- * dom/EventDispatcher.cpp: |
- (WebCore::EventDispatcher::adjustToShadowBoundaries): Changed to use shadowHost |
- explicitly. |
- |
-2011-04-21 Michael Saboff <msaboff@apple.com> |
- |
- Reviewed by Maciej Stachowiak. |
- |
- Qualified names used for all TagName access, yet namespace usage is rare |
- https://bugs.webkit.org/show_bug.cgi?id=58997 |
- |
- The methods getElementsByTagName and getElementsByTagNameNS where |
- always creating and using QualifiedNames. QualifiedName::init |
- was consistently in the top 3 routines when running the Dromaeo |
- DOM-query benchmark. Split out the functionality so that |
- getElementsByTagName uses just the local name, an implied "*" |
- namespace and a separate TagNodeListCache keyed by an atomic name |
- instead of a QualifiedName. Access to elements via |
- getElementsByTagNameNS that have "*" namespace are forwarded to |
- getElementsByTagName as well. This provides ~10% speed up in that |
- Dromaeo test. |
- |
- No new tests added, existing tests have coverage. The changes are |
- an optimization of existing functionality. |
- |
- * dom/Node.cpp: |
- (WebCore::Node::removeCachedTagNodeList): |
- (WebCore::Node::getElementsByTagName): |
- (WebCore::Node::getElementsByTagNameNS): |
- (WebCore::NodeListsNodeData::invalidateCaches): |
- (WebCore::NodeListsNodeData::isEmpty): |
- * dom/Node.h: |
- * dom/NodeRareData.h: |
- * dom/TagNodeList.cpp: |
- (WebCore::TagNodeList::~TagNodeList): |
- |
-2011-04-21 Geoffrey Garen <ggaren@apple.com> |
- |
- Reviewed by Oliver Hunt. |
- |
- Removed a use of markDOMObjectWrapper: more style- and node-related stuff |
- https://bugs.webkit.org/show_bug.cgi?id=59127 |
- |
- * bindings/js/JSCanvasRenderingContextCustom.cpp: |
- * bindings/js/JSCanvasRenderingContextCustom.cpp: |
- (WebCore::JSCanvasRenderingContextOwner::isReachableFromOpaqueRoots): |
- (WebCore::JSCanvasRenderingContextOwner::finalize): |
- (WebCore::wrapperOwner): |
- (WebCore::wrapperContext): Use opaque roots for marking canvas rendering |
- contexts, so JSNode isn't responsible for it. |
- |
- * bindings/js/JSElementCustom.cpp: |
- (WebCore::JSElement::markChildren): |
- * bindings/js/JSHTMLCanvasElementCustom.cpp: |
- * bindings/js/JSHTMLLinkElementCustom.cpp: |
- * bindings/js/JSHTMLStyleElementCustom.cpp: Removed obsolete marking code. |
- |
- * bindings/js/JSNodeCustom.cpp: |
- (WebCore::isObservable): |
- (WebCore::isReachableFromDOM): |
- (WebCore::JSNodeOwner::isReachableFromOpaqueRoots): Removed a bunch of |
- CSS object model related special cases, since the CSS object model |
- now tracks its own lifetime through the opaque roots system. Removed |
- canvas special case, covered above. |
- |
- * bindings/js/JSProcessingInstructionCustom.cpp: |
- * bindings/js/JSWebKitAnimationListCustom.cpp: |
- * dom/ProcessingInstruction.idl: |
- * html/HTMLCanvasElement.idl: |
- * html/HTMLLinkElement.idl: |
- * html/HTMLStyleElement.idl: |
- * page/WebKitAnimationList.idl: Removed obsolete marking code. |
- |
-2011-04-21 Vangelis Kokkevis <vangelis@chromium.org> |
- |
- Reviewed by James Robinson. |
- |
- [chromium] Only update the contents of layers that are going to |
- be drawn. Remove individual texture unreserves that happen |
- after the layer draw calls with a single unreserve for all the |
- textures in the texture manager. |
- https://bugs.webkit.org/show_bug.cgi?id=59129 |
- |
- Tests: No new functionality. Covered by existing layout tests. |
- |
- * platform/graphics/chromium/ContentLayerChromium.cpp: |
- (WebCore::ContentLayerChromium::draw): |
- * platform/graphics/chromium/ContentLayerChromium.h: |
- * platform/graphics/chromium/ImageLayerChromium.cpp: |
- (WebCore::ImageLayerChromium::paintContentsIfDirty): |
- (WebCore::ImageLayerChromium::updateCompositorResources): |
- * platform/graphics/chromium/LayerRendererChromium.cpp: |
- (WebCore::LayerRendererChromium::drawRootLayer): |
- (WebCore::LayerRendererChromium::updateAndDrawLayers): |
- (WebCore::LayerRendererChromium::updateLayers): |
- (WebCore::LayerRendererChromium::paintLayerContents): |
- * platform/graphics/chromium/LayerRendererChromium.h: |
- * platform/graphics/chromium/LayerTexture.cpp: |
- * platform/graphics/chromium/LayerTexture.h: |
- * platform/graphics/chromium/LayerTilerChromium.cpp: |
- * platform/graphics/chromium/LayerTilerChromium.h: |
- * platform/graphics/chromium/RenderSurfaceChromium.cpp: |
- (WebCore::RenderSurfaceChromium::drawSurface): |
- * platform/graphics/chromium/TextureManager.cpp: |
- (WebCore::TextureManager::unprotectAllTextures): |
- * platform/graphics/chromium/TextureManager.h: |
- * platform/graphics/chromium/cc/CCHeadsUpDisplay.cpp: |
- (WebCore::CCHeadsUpDisplay::draw): |
- * platform/graphics/chromium/cc/CCLayerImpl.h: |
- (WebCore::CCLayerImpl::owner): |
- |
-2011-04-21 Ryosuke Niwa <rniwa@webkit.org> |
- |
- Reviewed by Sam Weinig. |
- |
- Add Frame* to the argument lists of canCopyCut and canPaste |
- https://bugs.webkit.org/show_bug.cgi?id=59153 |
- |
- canCopyCut and canPaste now takes Frame* for which supportedCopyCut and supportedPaste are called. |
- This allows editor client to grant copy/cut and paste permissions per origin. |
- |
- * editing/EditorCommand.cpp: |
- (WebCore::supportedCopyCut): |
- (WebCore::supportedPaste): |
- * loader/EmptyClients.h: |
- (WebCore::EmptyEditorClient::canCopyCut): |
- (WebCore::EmptyEditorClient::canPaste): |
- * page/EditorClient.h: |
- |
-2011-04-21 Oliver Hunt <oliver@apple.com> |
- |
- Whoops, change lost during merge |
- |
- * bindings/js/JSEventListener.h: |
- |
-2011-04-21 Geoffrey Garen <ggaren@apple.com> |
- |
- Reviewed by Sam Weinig. |
- |
- Fixed a failure seen in fast/dom/prototype-inheritance-2.html due to |
- wrong global object use. |
- |
- * bindings/js/JSHTMLDocumentCustom.cpp: |
- (WebCore::JSHTMLDocument::nameGetter): |
- (WebCore::JSHTMLDocument::all): |
- * bindings/js/JSHTMLFormElementCustom.cpp: |
- (WebCore::JSHTMLFormElement::nameGetter): Use the right global object. |
- |
-2011-04-21 Oliver Hunt <oliver@apple.com> |
- |
- Reviewed by Geoffrey Garen. |
- |
- Start moving to a general visitor pattern for GC traversal |
- https://bugs.webkit.org/show_bug.cgi?id=59141 |
- |
- This is just a rename: |
- markChildren -> visitChildren |
- markAggregate -> visitAggregate |
- markStack -> visitor |
- MarkStack -> typedef'd to SlotVisitor |
- |
- * bindings/js/JSAttrCustom.cpp: |
- (WebCore::JSAttr::visitChildren): |
- * bindings/js/JSAudioConstructor.cpp: |
- (WebCore::constructAudio): |
- * bindings/js/JSCSSRuleCustom.cpp: |
- (WebCore::JSCSSRule::visitChildren): |
- * bindings/js/JSCSSStyleDeclarationCustom.cpp: |
- (WebCore::JSCSSStyleDeclaration::visitChildren): |
- * bindings/js/JSDOMBinding.cpp: |
- (WebCore::visitActiveObjectsForContext): |
- (WebCore::markDOMObjectWrapper): |
- * bindings/js/JSDOMBinding.h: |
- * bindings/js/JSDOMGlobalObject.cpp: |
- (WebCore::JSDOMGlobalObject::visitChildren): |
- * bindings/js/JSDOMGlobalObject.h: |
- * bindings/js/JSDOMImplementationCustom.cpp: |
- (WebCore::JSDOMImplementation::visitChildren): |
- * bindings/js/JSDOMWindowCustom.cpp: |
- (WebCore::JSDOMWindow::visitChildren): |
- * bindings/js/JSDOMWindowShell.cpp: |
- (WebCore::JSDOMWindowShell::visitChildren): |
- * bindings/js/JSDOMWindowShell.h: |
- * bindings/js/JSDocumentCustom.cpp: |
- (WebCore::JSDocument::visitChildren): |
- * bindings/js/JSElementCustom.cpp: |
- (WebCore::JSElement::visitChildren): |
- * bindings/js/JSEventListener.cpp: |
- (WebCore::JSEventListener::markJSFunction): |
- * bindings/js/JSEventListener.h: |
- * bindings/js/JSHTMLCanvasElementCustom.cpp: |
- (WebCore::JSHTMLCanvasElement::visitChildren): |
- * bindings/js/JSHTMLLinkElementCustom.cpp: |
- (WebCore::JSHTMLLinkElement::visitChildren): |
- * bindings/js/JSHTMLStyleElementCustom.cpp: |
- (WebCore::JSHTMLStyleElement::visitChildren): |
- * bindings/js/JSImageConstructor.cpp: |
- (WebCore::constructImage): |
- * bindings/js/JSJavaScriptAudioNodeCustom.cpp: |
- (WebCore::JSJavaScriptAudioNode::visitChildren): |
- * bindings/js/JSMessageChannelCustom.cpp: |
- (WebCore::JSMessageChannel::visitChildren): |
- * bindings/js/JSMessagePortCustom.cpp: |
- (WebCore::JSMessagePort::visitChildren): |
- * bindings/js/JSNamedNodeMapCustom.cpp: |
- (WebCore::JSNamedNodeMapOwner::isReachableFromOpaqueRoots): |
- (WebCore::JSNamedNodeMap::visitChildren): |
- * bindings/js/JSNavigatorCustom.cpp: |
- (WebCore::JSNavigator::visitChildren): |
- * bindings/js/JSNodeCustom.cpp: |
- (WebCore::isObservable): |
- (WebCore::isReachableFromDOM): |
- (WebCore::JSNodeOwner::isReachableFromOpaqueRoots): |
- (WebCore::JSNode::visitChildren): |
- * bindings/js/JSNodeCustom.h: |
- * bindings/js/JSNodeFilterCondition.cpp: |
- (WebCore::JSNodeFilterCondition::WeakOwner::isReachableFromOpaqueRoots): |
- * bindings/js/JSNodeFilterCondition.h: |
- * bindings/js/JSNodeFilterCustom.cpp: |
- (WebCore::JSNodeFilter::visitChildren): |
- * bindings/js/JSNodeIteratorCustom.cpp: |
- (WebCore::JSNodeIterator::visitChildren): |
- * bindings/js/JSNodeListCustom.cpp: |
- (WebCore::JSNodeListOwner::isReachableFromOpaqueRoots): |
- * bindings/js/JSProcessingInstructionCustom.cpp: |
- (WebCore::JSProcessingInstruction::visitChildren): |
- * bindings/js/JSSVGElementInstanceCustom.cpp: |
- (WebCore::JSSVGElementInstance::visitChildren): |
- * bindings/js/JSSharedWorkerCustom.cpp: |
- (WebCore::JSSharedWorker::visitChildren): |
- * bindings/js/JSStyleSheetCustom.cpp: |
- (WebCore::JSStyleSheet::visitChildren): |
- * bindings/js/JSTreeWalkerCustom.cpp: |
- (WebCore::JSTreeWalker::visitChildren): |
- * bindings/js/JSWebGLRenderingContextCustom.cpp: |
- (WebCore::JSWebGLRenderingContext::visitChildren): |
- * bindings/js/JSWebKitAnimationListCustom.cpp: |
- (WebCore::JSWebKitAnimationList::visitChildren): |
- * bindings/js/JSWorkerContextCustom.cpp: |
- (WebCore::JSWorkerContext::visitChildren): |
- * bindings/js/JSXMLHttpRequestCustom.cpp: |
- (WebCore::JSXMLHttpRequest::visitChildren): |
- * bindings/js/JSXMLHttpRequestUploadCustom.cpp: |
- (WebCore::JSXMLHttpRequestUpload::visitChildren): |
- * bindings/scripts/CodeGeneratorJS.pm: |
- * bridge/qt/qt_instance.cpp: |
- (JSC::Bindings::QtRuntimeObject::visitChildren): |
- (JSC::Bindings::QtInstance::markAggregate): |
- * bridge/qt/qt_instance.h: |
- * bridge/qt/qt_pixmapruntime.cpp: |
- * bridge/qt/qt_runtime.cpp: |
- (JSC::Bindings::QtRuntimeMetaMethod::visitChildren): |
- * bridge/qt/qt_runtime.h: |
- * dom/EventListener.h: |
- (WebCore::EventListener::visitJSFunction): |
- * dom/EventTarget.h: |
- (WebCore::EventTarget::visitJSEventListeners): |
- * dom/Node.h: |
- * dom/NodeFilterCondition.h: |
- (WebCore::NodeFilterCondition::visitAggregate): |
- * page/DOMWindow.h: |
- * workers/WorkerContext.h: |
- |
-2011-04-21 Beth Dakin <bdakin@apple.com> |
- |
- Reviewed by Dan Bernstein. |
- |
- https://bugs.webkit.org/show_bug.cgi?id=59048 |
- Need to track whether overlay scrollbar is currently visible and in lower- |
- righthand corner |
- -and corresponding- |
- <rdar://problem/9211232> |
- |
- Whenever we start a new ScrollbarPartAnimation, check if the scrollbar is vertical |
- and showing (alpha of 1), and if so, call setVisibleScrollerThumbRect with the |
- knob rect, otherwise with an empty rect. ScrollAnimatorMac has a local rect to |
- track this value so that it can avoid sending messages across the ChromeClient to |
- the UIProcess if the rect has not changed. |
- * platform/mac/ScrollAnimatorMac.h: |
- * platform/mac/ScrollAnimatorMac.mm: |
- (-[ScrollbarPainterDelegate setUpAnimation:scrollerPainter:part:WebCore::animateAlphaTo:duration:]): |
- (WebCore::ScrollAnimatorMac::ScrollAnimatorMac): |
- (WebCore::ScrollAnimatorMac::setVisibleScrollerThumbRect): |
- |
- Plumbing for the chrome client. |
- * page/ChromeClient.h: |
- (WebCore::ChromeClient::notifyScrollerThumbIsVisibleInRect): |
- * page/FrameView.cpp: |
- (WebCore::FrameView::setVisibleScrollerThumbRect): |
- * page/FrameView.h: |
- * platform/ScrollableArea.h: |
- (WebCore::ScrollableArea::setVisibleScrollerThumbRect): |
- |
- New WebCoreSystemInterface function to get the knob rect. |
- * WebCore.exp.in: |
- * platform/mac/WebCoreSystemInterface.h: |
- * platform/mac/WebCoreSystemInterface.mm: |
- |
2011-04-21 Nat Duca <nduca@chromium.org> |
Reviewed by James Robinson. |