Timeline
Apr 28, 2008:
- 11:14 PM Changeset in webkit [32672] by
-
- 2 edits in trunk/WebKit/mac
Reviewed by Maciej.
Build fix for Tiger.
- 11:05 PM Changeset in webkit [32671] by
-
- 13 edits in trunk
WebCore:
Reviewed by Maciej Stachowiak.
- fix https://bugs.webkit.org/show_bug.cgi?id=18795 REGRESSION (r32660): Text heavier with text-shadow
- rendering/InlineTextBox.cpp: (WebCore::paintTextWithShadows): The loop was doing one extra pass without shadow even in cases where the last pass with a shadow was painting the text at the right position (rather than outside the clip). Fixed that. The only case where a separate pass without shadow is needed is when there is text-stroke, since in that case all shadow passes paint the text outside the clip.
LayoutTests:
Reviewed by Maciej Stachowiak.
- updated results for https://bugs.webkit.org/show_bug.cgi?id=18795 REGRESSION (r32660): Text heavier with text-shadow
- platform/mac/fast/css/shadow-multiple-expected.checksum:
- platform/mac/fast/css/shadow-multiple-expected.png:
- platform/mac/fast/repaint/shadow-multiple-horizontal-expected.checksum:
- platform/mac/fast/repaint/shadow-multiple-horizontal-expected.png:
- platform/mac/fast/repaint/shadow-multiple-strict-horizontal-expected.checksum:
- platform/mac/fast/repaint/shadow-multiple-strict-horizontal-expected.png:
- platform/mac/fast/repaint/shadow-multiple-strict-vertical-expected.checksum:
- platform/mac/fast/repaint/shadow-multiple-strict-vertical-expected.png:
- platform/mac/fast/repaint/shadow-multiple-vertical-expected.checksum:
- platform/mac/fast/repaint/shadow-multiple-vertical-expected.png:
- 10:53 PM SquirrelFishPerfIdeas edited by
- (diff)
- 10:51 PM SquirrelFishPerfIdeas edited by
- (diff)
- 10:30 PM SquirrelFishPerfIdeas edited by
- (diff)
- 9:10 PM Changeset in webkit [32670] by
-
- 2 edits in branches/squirrelfish/JavaScriptCore
Restore base pointer when popping a global RegisterFile
Reviewed by Geoff
- 8:57 PM Changeset in webkit [32669] by
-
- 5 edits in branches/squirrelfish/JavaScriptCore
Bug 18643: SQUIRRELFISH: need to support implicit function calls (valueOf, toString, getters/setters)
<https://bugs.webkit.org/show_bug.cgi?id=18643>
Reviewed by Geoff
Partial fix. This results in all implicit calls to toString or valueOf
executing in a separate RegisterFile, so ensuring that the the pointers
in the triggering interpreter don't get trashed. This still leaves the
task of preventing new global re-entry from toString and valueOf from
clobbering the RegisterFile.
- 8:14 PM Changeset in webkit [32668] by
-
- 11 edits4 adds in trunk
WebCore:
2008-04-28 Brady Eidson <beidson@apple.com>
Reviewed by Sam Weinig
Add Storage.clear(), per updates to HTML5
Tests: storage/domstorage/localstorage/clear.html
storage/domstorage/sessionstorage/clear.html
- storage/LocalStorageArea.cpp: (WebCore::LocalStorageArea::areaCleared):
- storage/LocalStorageArea.h:
- storage/SessionStorageArea.cpp: (WebCore::SessionStorageArea::areaCleared):
- storage/SessionStorageArea.h:
- storage/Storage.cpp: (WebCore::Storage::clear):
- storage/Storage.h:
- storage/Storage.idl:
- storage/StorageArea.cpp: (WebCore::StorageArea::internalClear):
- storage/StorageArea.h: (WebCore::StorageArea::clear):
LayoutTests:
2008-04-28 Brady Eidson <beidson@apple.com>
Reviewed by Sam Weinig
Add Storage.clear(), per updates to HTML5
- storage/domstorage/localstorage/clear-expected.txt: Added.
- storage/domstorage/localstorage/clear.html: Added.
- storage/domstorage/sessionstorage/clear-expected.txt: Added.
- storage/domstorage/sessionstorage/clear.html: Added.
- 6:54 PM Changeset in webkit [32667] by
-
- 4 edits in branches/squirrelfish/JavaScriptCore
2008-04-28 Geoffrey Garen <ggaren@apple.com>
Reviewed by Maciej Stachowiak.
Simplified activation object a bit: No need to store the callee
in the activation object -- we can pull it out of the call frame
when needed, instead.
SunSpider reports no change.
- 6:30 PM Changeset in webkit [32666] by
-
- 21 edits in trunk
WebCore:
2008-04-28 Adele Peterson <adele@apple.com>
Reviewed by Dan Bernstein, Tim Hatcher, Anders Carlsson, and Darin Adler.
WebCore part of fix for <rdar://problem/3709505>
Safari should have a way to upload bundles from the file upload control (as zip)
- WebCore.base.exp: Added symbols.
- html/HTMLFormElement.cpp: (WebCore::HTMLFormElement::formData): Ask the application if a file will need to be replaced before it's uploaded. It will also give the replacement filename which is used to determine the correct mime-type and to construct the correct header.
- loader/FrameLoader.cpp: (WebCore::FrameLoader::submitForm): Asks the application to generate any files for the form data before a form submission starts. (WebCore::FrameLoader::loadItem): ditto.
- loader/ResourceLoader.cpp: (WebCore::ResourceLoader::didReceiveResponse): Tells the FormData from the request to remove any generated files if it needs to. (WebCore::ResourceLoader::didCancel): ditto. (WebCore::ResourceLoader::didFail): ditto.
- platform/network/FormData.cpp: (WebCore::FormData::FormData): Initializes m_hasGeneratedFiles, which keeps track of whether there are files that will need to be removed. (WebCore::FormData::~FormData): Added. Assert that there are no files that need to be removed, but if there are, release builds will still remove them here. (WebCore::FormData::appendFile): Passes along a shouldGenerateFile flag to the FormDataElement. (WebCore::FormData::generateFiles): Added. Iterates through the FormDataElements, and using the ChromeClient pointer, asks the application to create any compressed files so the FormDataElements can store the paths. (WebCore::FormData::removeGeneratedFilesIfNeeded): Added. Removes generated files and their directories (if empty).
- platform/network/FormData.h: (WebCore::FormDataElement::FormDataElement): Added a boolean to track whether the file will need to be generated, and a string to hold the path of the generated file.
- platform/network/mac/FormDataStreamMac.mm: (WebCore::advanceCurrentStream): Uses the generated file path instead of the original file path when streaming the file. (WebCore::setHTTPBody): Uses the generated file path when determining the size of the file to be uploaded.
- platform/FileSystem.h: Added directoryName.
- platform/posix/FileSystemPOSIX.cpp: (WebCore::directoryName): Added.
- platform/gtk/FileSystemGtk.cpp: (WebCore::directoryName): Added empty implementation.
- platform/qt/FileSystemQt.cpp: (WebCore::directoryName): ditto.
- platform/win/FileSystemWin.cpp: (WebCore::directoryName): ditto.
- platform/wx/FileSystemWx.cpp: (WebCore::directoryName): ditto.
Added new ChromeClient methods to give the application control over the file compression for uploading.
- page/Chrome.cpp: (WebCore::ChromeClient::shouldReplaceWithGeneratedFileForUpload): Added a default implementation. (WebCore::ChromeClient::generateReplacementFile): ditto.
- page/ChromeClient.h:
WebKit/mac:
2008-04-28 Adele Peterson <adele@apple.com>
Reviewed by Dan Bernstein, Tim Hatcher, Anders Carlsson, and Darin Adler.
WebKit part of fix for <rdar://problem/3709505>
Safari should have a way to upload bundles from the file upload control (as zip)
Added UIDelegate methods to let the application handle generating replacement files for uploads.
In this case, Safari will create archived files for bundles so they can be uploaded properly.
- DefaultDelegates/WebDefaultUIDelegate.m: (-[WebDefaultUIDelegate webView:shouldReplaceUploadFile:usingGeneratedFilename:]): (-[WebDefaultUIDelegate webView:generateReplacementFile:]):
- WebCoreSupport/WebChromeClient.h:
- WebCoreSupport/WebChromeClient.mm: (WebChromeClient::shouldReplaceWithGeneratedFileForUpload): (WebChromeClient::generateReplacementFile):
- WebView/WebUIDelegatePrivate.h:
- 6:28 PM Changeset in webkit [32665] by
-
- 6 edits in trunk
WebCore:
2008-04-28 Anders Carlsson <andersca@apple.com>
Reviewed by Sam, Mark, Adele and Darin.
Add a (currently not called) method for opening the application cache database.
- WebCore.base.exp:
- loader/appcache/ApplicationCacheStorage.cpp: (WebCore::ApplicationCacheStorage::setCacheDirectory): (WebCore::ApplicationCacheStorage::openDatabase):
- loader/appcache/ApplicationCacheStorage.h:
WebKit/mac:
2008-04-28 Anders Carlsson <andersca@apple.com>
Reviewed by Sam, Mark, Adele and Darin.
Initialize the application cache path.
- WebView/WebView.mm: (WebKitInitializeApplicationCachePathIfNecessary): (-[WebView _commonInitializationWithFrameName:groupName:]):
- 6:18 PM Changeset in webkit [32664] by
-
- 34 edits8 adds in trunk
WebCore:
2008-04-28 Alice Liu <alice.liu@apple.com>
Reviewed by Darin.
Fix <rdar://problem/4911289> Add tabindex property to all children
of HTMLElement (7138)
http://bugs.webkit.org/show_bug.cgi?id=7138
Test: fast/events/tabindex-focus-blur-all.html
- WebCore.xcodeproj/project.pbxproj:
- bindings/objc/DOMPrivate.h: -DOMHTMLLabelElementPrivate.h and DOMHTMLLegendElementPrivate.h are no longer needed
- bindings/objc/PublicDOMInterfaces.h: -Moving focus, blur, tabindex to HTMLElement from its descendants.
- dom/Node.cpp: (WebCore::Node::Node): (WebCore::Node::isFocusable): (WebCore::Node::isKeyboardFocusable): -The concept of being focusable now needs to account for tabindex attribute.
- dom/Node.h: -Need to use a bit to keep track of whether tabindex was explicitly set to 0 or was just initialized to 0. (WebCore::Node::tabIndex): -Make this virtual so that Anchors and GenericFormElements can determine tabindex without accounting for focusability (WebCore::Node::setTabIndexExplicitly): -Renamed setTabIndex() to setTabIndexExplicitly() so as not to hide behind and/or be confused for HTMLElement::setTabIndex
- html/HTMLAnchorElement.cpp:
- html/HTMLAnchorElement.h: (WebCore::HTMLAnchorElement::tabIndex):
- html/HTMLGenericFormElement.cpp:
- html/HTMLGenericFormElement.h: (WebCore::HTMLGenericFormElement::tabIndex): -Anchors and form elements are normally focusable, and need to avert the check for focusability in HTMLElement::tabIndex because tabIndex could be queried before a first layout happens.
- html/HTMLElement.cpp: (WebCore::HTMLElement::parseMappedAttribute): -Function Node::setTabIndex changed to Node::setTabIndexExplicitly (WebCore::HTMLElement::isFocusable): -The concept of being focusable now needs to account for tabindex attribute. (WebCore::HTMLElement::tabIndex): -HTML5 specifies that querying tabindex for elements that aren't focusable shall return -1 (WebCore::HTMLElement::setTabIndex): -New default implementation
- html/HTMLElement.h:
- html/HTMLLegendElement.cpp: (WebCore::HTMLLegendElement::focus): -Now with HTML5's enhanced specification of tabindex, legend elements themselves shall focus (if they should), and not the first form element they encapsulate. This matches Firefox behavior. HTMLLabelElements still forward focus to the encapsulated element. This also matches other browsers.
- html/HTMLAreaElement.cpp: (WebCore::HTMLAreaElement::isFocusable):
- html/HTMLAreaElement.h: -HTMLAreaElement isn't subject to the same focusability requirements as its parent, HTMLAnchorElement. Default HTMLElement implementation is desired.
- html/HTMLFieldSetElement.cpp: (WebCore::HTMLFieldSetElement::isFocusable):
- html/HTMLLegendElement.cpp: (WebCore::HTMLLegendElement::isFocusable):
- html/HTMLOptGroupElement.cpp: (WebCore::HTMLOptGroupElement::isFocusable):
- html/HTMLOptionElement.cpp: (WebCore::HTMLOptionElement::isFocusable): -These elements aren't subject to the same focusability requirements as their parent, HTMLGenericFormElement. Default HTMLElement implementation is desired.
- html/HTMLAnchorElement.idl:
- html/HTMLAreaElement.idl:
- html/HTMLButtonElement.idl:
- html/HTMLElement.idl:
- html/HTMLInputElement.idl:
- html/HTMLLabelElement.idl:
- html/HTMLLegendElement.idl:
- html/HTMLObjectElement.idl:
- html/HTMLSelectElement.idl:
- html/HTMLTextAreaElement.idl: -Moving tabindex attribute, focus and blur methods from descendants of HTMLElement to HTMLElement
WebKit/mac:
2008-04-28 Alice Liu <alice.liu@apple.com>
Reviewed by Darin.
Fix <rdar://problem/4911289> Add tabindex property to all children
of HTMLElement (7138)
http://bugs.webkit.org/show_bug.cgi?id=7138
- MigrateHeaders.make: Removing DOMHTMLLabelElementPrivate.h and DOMHTMLLegendElementPrivate.h because now that focus() has been moved to DOMHTMLElement.h, these files are no longer needed.
LayoutTests:
2008-04-28 Alice Liu <alice.liu@apple.com>
Reviewed by Sam Weinig.
Fix <rdar://problem/4911289> Add tabindex property to all children
of HTMLElement (7138)
http://bugs.webkit.org/show_bug.cgi?id=7138
minor updates to these existing tests:
- fast/dom/domListEnumeration-expected.txt:
- fast/dom/plugin-attributes-enumeration-expected.txt:
- fast/dom/resources/domListEnumeration.js:
- fast/events/resources/abe.png: Added.
- fast/events/resources/tabindex-focus-blur-all-frame1.html: Added.
- fast/events/resources/tabindex-focus-blur-all-frame2.html: Added.
- fast/events/resources/tabindex-focus-blur-all-iframe1.html: Added.
- fast/events/resources/tabindex-focus-blur-all-iframe2.html: Added.
- fast/events/resources/tabindex-focus-blur-all.js: Added.
- fast/events/tabindex-focus-blur-all-expected.txt: Added.
- fast/events/tabindex-focus-blur-all.html: Added.
- 5:26 PM Changeset in webkit [32663] by
-
- 4 edits in branches/squirrelfish/JavaScriptCore
2008-04-28 Geoffrey Garen <ggaren@apple.com>
Reviewed by Maciej Stachowiak.
RS by Oliver Hunt on moving JSArguments.cpp out of AllInOneFile.cpp.
Substantially more handling of "arguments": "arguments" works fully
now, but "f.arguments" still doesn't work.
Fixes 10 regression tests.
SunSpider reports no regression.
- kjs/JSActivation.cpp: (KJS::JSActivation::createArgumentsObject): Reconstruct an arguments List to pass to the arguments object constructor.
- JavaScriptCore.xcodeproj/project.pbxproj:
- kjs/AllInOneFile.cpp: Removed JSActivation.cpp from AllInOneFile.cpp because that seems to make GCC happy. (Previously, I had added JSActivation.cpp to AllInOneFile.cpp because *that* seemed to make GCC happy. So it goes.)
- 4:39 PM Changeset in webkit [32662] by
-
- 4 edits in branches/squirrelfish/JavaScriptCore
2008-04-28 Geoffrey Garen <ggaren@apple.com>
Reviewed by Maciej Stachowiak.
Groundwork for more handling of "arguments". I'm not checking in the
actual handling of "arguments" yet, because it still needs a little
fiddling to avoid a performance regression.
SunSpider reports no change.
- VM/Machine.cpp: (KJS::initializeCallFrame): Put argc in the register file, so the arguments object can find it later, to determine arguments.length.
- kjs/nodes.h: (KJS::FunctionBodyNode::): Added a special code accessor for when you know the code has already been generated, and you don't have a scopeChain to supply for potential code generation. (This is the case when the activation object creates the arguments object.)
- 4:04 PM Changeset in webkit [32661] by
-
- 3 edits in branches/squirrelfish/JavaScriptCore
Replace unsafe use of auto_ptr in Vector with manual memory
management.
Reviewed by Geoff
- 2:28 PM Changeset in webkit [32660] by
-
- 15 edits20 adds in trunk
WebCore:
Reviewed by Dave Hyatt.
- support multiple box- and text-shadows
Tests: fast/css/shadow-multiple.html
fast/repaint/shadow-multiple-horizontal.html
fast/repaint/shadow-multiple-strict-horizontal.html
fast/repaint/shadow-multiple-strict-vertical.html
fast/repaint/shadow-multiple-vertical.html
- css/CSSComputedStyleDeclaration.cpp: (WebCore::valueForShadow): Changed to account for reversing the order of the shadow values in the ShadowData list.
- css/CSSValueList.cpp: (WebCore::CSSValueList::prepend): Added.
- css/CSSValueList.h:
- rendering/InlineFlowBox.cpp: (WebCore::InlineFlowBox::placeBoxesHorizontally): Changed to account for all shadows in overflow calculation. (WebCore::InlineFlowBox::placeBoxesVertically): Ditto. (WebCore::InlineFlowBox::paint): Changed to account for all shadows when testing for intersection with the damage rect. (WebCore::InlineFlowBox::paintTextDecorations): Changed to paint all shadows.
- rendering/InlineTextBox.cpp: (WebCore::paintTextWithShadows): Factored out from paint() and changed to paint all shadows. (WebCore::InlineTextBox::paint): Moved the text painting code out to paintTextWithShadows(). Changed to not paint shadows for markers and composition underlines and in "force black text" mode. (WebCore::InlineTextBox::paintSelection): (WebCore::InlineTextBox::paintCompositionBackground): (WebCore::InlineTextBox::paintDecoration): Changed to paint all shadows.
- rendering/InlineTextBox.h: Changed some public methods to private or protected.
- rendering/RenderBlock.cpp: (WebCore::RenderBlock::overflowHeight): Changed to account for all shadows. (WebCore::RenderBlock::overflowWidth): Ditto. (WebCore::RenderBlock::overflowLeft): Ditto. (WebCore::RenderBlock::overflowTop): Ditto. (WebCore::RenderBlock::overflowRect): Ditto. (WebCore::RenderBlock::layoutBlock): Ditto.
- rendering/RenderFlexibleBox.cpp: (WebCore::RenderFlexibleBox::layoutBlock): Ditto.
- rendering/RenderLayer.cpp: (WebCore::RenderLayer::calculateRects): Ditto.
- rendering/RenderObject.cpp: (WebCore::RenderObject::paintBoxShadow): Changed to paint all shadows. Changed to avoid clipping out the box if it has a fully opaque background. (WebCore::RenderObject::repaintAfterLayoutIfNeeded): Changed to account for all shadows. (WebCore::RenderObject::selectionForegroundColor): Cleaned up. (WebCore::RenderObject::adjustRectForOutlineAndShadow): Changed to account for all shadows.
- rendering/RenderReplaced.cpp: (WebCore::RenderReplaced::adjustOverflowForBoxShadow): Ditto.
- rendering/RenderStyle.cpp: (WebCore::RenderStyle::setTextShadow): Changed to prepend when adding so that the stacking order of shadows when painting will be "first o top". (WebCore::RenderStyle::setBoxShadow): Ditto.
- rendering/RenderTable.cpp: (WebCore::RenderTable::layout): Changed to account for all shadows.
LayoutTests:
Reviewed by Dave Hyatt.
- add rendering and invalidation tests for multiple shadows
- fast/css/shadow-multiple.html: Added.
- fast/repaint/shadow-multiple-horizontal.html: Added.
- fast/repaint/shadow-multiple-strict-horizontal.html: Added.
- fast/repaint/shadow-multiple-strict-vertical.html: Added.
- fast/repaint/shadow-multiple-vertical.html: Added.
- platform/mac/fast/css/shadow-multiple-expected.checksum: Added.
- platform/mac/fast/css/shadow-multiple-expected.png: Added.
- platform/mac/fast/css/shadow-multiple-expected.txt: Added.
- platform/mac/fast/repaint/shadow-multiple-horizontal-expected.checksum: Added.
- platform/mac/fast/repaint/shadow-multiple-horizontal-expected.png: Added.
- platform/mac/fast/repaint/shadow-multiple-horizontal-expected.txt: Added.
- platform/mac/fast/repaint/shadow-multiple-strict-horizontal-expected.checksum: Added.
- platform/mac/fast/repaint/shadow-multiple-strict-horizontal-expected.png: Added.
- platform/mac/fast/repaint/shadow-multiple-strict-horizontal-expected.txt: Added.
- platform/mac/fast/repaint/shadow-multiple-strict-vertical-expected.checksum: Added.
- platform/mac/fast/repaint/shadow-multiple-strict-vertical-expected.png: Added.
- platform/mac/fast/repaint/shadow-multiple-strict-vertical-expected.txt: Added.
- platform/mac/fast/repaint/shadow-multiple-vertical-expected.checksum: Added.
- platform/mac/fast/repaint/shadow-multiple-vertical-expected.png: Added.
- platform/mac/fast/repaint/shadow-multiple-vertical-expected.txt: Added.
- 12:47 PM Changeset in webkit [32659] by
-
- 2 edits in trunk/WebCore
Fix some more Windows build errors in COMPtr
- platform/win/COMPtr.h: (WTF::): Define DefaultHash<COMPtr<P> > by using PtrHash<COMPtr<P> >, which is just a copy of PtrHash<RefPtr<P> >.
- 12:39 PM Changeset in webkit [32658] by
-
- 3 edits in trunk/WebKit/win
Fix some Windows build errors in WebKit
- Interfaces/WebKit.idl: Touched to force Interfaces to rebuild.
- WebKitPrefix.cpp: Touched to force WebKit to rebuild.
- 12:28 PM Changeset in webkit [32657] by
-
- 2 edits in trunk/WebCore
Fix some Windows build errors in COMPtr
- platform/win/COMPtr.h: Explicitly name the WTF namespace when using HashTableDeletedValueType.
- 12:22 PM Changeset in webkit [32656] by
-
- 2 edits in trunk/WebCore
Fix some Windows build errors in FontPlatfomrData
- platform/graphics/win/FontPlatformData.h: Removed the Deleted constructor and replaced it with a HashTableDeletedValueType constructor. Also added isHashTableDeletedValue() and hashTableDeletedFontValue().
- 12:14 PM Changeset in webkit [32655] by
-
- 3 edits4 adds in trunk
WebCore:
2008-04-28 David Hyatt <hyatt@apple.com>
Canvas can't draw itself into its own buffer. Make sure the dirtying call comes after the image()
has been accessed. Technically willDraw should be renamed to didDraw and all calls should be moved
to after draws have occurred (since if the draw op itself wants to reference the image it shouldn't get
a dirty copy either), but I'll take that up with olliej when he wakes up. :)
Reviewed by Dan Bernstein
Added fast/canvas/canvasDrawsIntoSelf.html
- html/CanvasRenderingContext2D.cpp: (WebCore::CanvasRenderingContext2D::drawImage):
LayoutTests:
2008-04-28 David Hyatt <hyatt@apple.com>
Add layout test for canvas self-drawing bug.
Reviewed by Dan Bernstein
- fast/canvas/canvasDrawingIntoSelf.html: Added.
- platform/mac/fast/canvas/canvasDrawingIntoSelf-expected.checksum: Added.
- platform/mac/fast/canvas/canvasDrawingIntoSelf-expected.png: Added.
- platform/mac/fast/canvas/canvasDrawingIntoSelf-expected.txt: Added.
- 12:12 PM Changeset in webkit [32654] by
-
- 2 edits in trunk/JavaScriptCore
Windows build fix.
- kjs/ExecState.h: For whatever reason, MSVC couldn't generate a default constructor for a struct that had a "const List" member. Removing the const qualifier makes the problem go away.
- 11:58 AM Changeset in webkit [32653] by
-
- 2 edits in trunk/WebCore
Reviewed by Geoff Garen.
<rdar://problem/5825192> Replace use of Gestalt with BSD sysctl
- platform/graphics/mac/MediaPlayerPrivateQTKit.mm: (WebCore::MediaPlayerPrivate::isAvailable): There is no need to check QuickTime version post-Tiger, as it will never be below 7.3 again. This change may make library loading happen earlier in some cases, hopefully, that's not a big deal.
- 11:22 AM Changeset in webkit [32652] by
-
- 64 edits in trunk
Reviewed by Darin.
Fix run-webkit-tests --threading
and provisionally fix <https://bugs.webkit.org/show_bug.cgi?id=18661>
Proxy server issue in Sunday's Nightly
Changed ClassInfo objects for built-in objects to hold a getter function returning
a per-thread instance. This makes it safe to share these ClassInfo objects between threads -
and these are the only ones that need to be shared.
- 11:15 AM Changeset in webkit [32651] by
-
- 3 edits4 adds in trunk
WebCore:
2008-04-28 David Hyatt <hyatt@apple.com>
The arguments in drawImage were reversed when I switched away from the image buffer method.
Reviewed by Tim Hatcher
Added fast/canvas/drawImage.html
- html/CanvasRenderingContext2D.cpp: (WebCore::CanvasRenderingContext2D::drawImage):
LayoutTests:
2008-04-28 David Hyatt <hyatt@apple.com>
Add layout test for canvas.
Reviewed by Tim Hatcher
- fast/canvas/drawImage.html: Added.
- platform/mac/fast/canvas/drawImage-expected.checksum: Added.
- platform/mac/fast/canvas/drawImage-expected.png: Added.
- platform/mac/fast/canvas/drawImage-expected.txt: Added.
- 10:18 AM Changeset in webkit [32650] by
-
- 7 edits in trunk/JavaScriptCore
2008-04-28 Darin Adler <Darin Adler>
Reviewed by Adam.
- make sure RefPtr's default hash doesn't ref/deref when computing the hash
- remove remnants of the hash table storage type optimization
- wtf/HashFunctions.h: Used "using" to get the hash and equal functions from PtrHash<P*> into PtrHash<RefPtr<P>>.
- wtf/HashMap.h: Replaced uses of PairBaseHashTraits with PairHashTraits. Eliminated storage-related typedefs. Removed constructor, destructor, copy constructor, and destructor since the compiler-generated ones are fine. Removed refAll and derefAll. Took out unnnecessary typecasts. Removed use of RefCounter.
- wtf/HashSet.h: Eliminated storage-related typedefs. Removed constructor, destructor, copy constructor, and destructor since the compiler-generated ones are fine. Removed refAll and derefAll. Removed unneeded template arguents from HashSetTranslatorAdapter. Eliminated unneeded HashSetTranslator template.
- wtf/HashTable.h: Tweaked formatting. Removed NeedsRef, RefCounterBase, RefCounter, HashTableRefCounterBase, HashTableRefCounter, and Assigner class templates.
- wtf/HashTraits.h: Removed StorageTraits, needsRef, PairBaseHashTraits, and HashKeyStorageTraits.
- wtf/RefPtrHashMap.h: Made all the same fixes as in HashMap. Also made the corresponding changes to RefPtrHashMapRawKeyTranslator.
- 10:06 AM Changeset in webkit [32649] by
-
- 2 edits in trunk/WebKitLibraries
2008-04-28 Darin Adler <Darin Adler>
Reviewed by Adam.
- fix Windows build
- win/tools/vsprops/common.vsprops: Add warning 4344 to the list of disabled warnings. It's really a warning about a bug they fixed in MSVC -- not helpful to us in WebKit.
- 9:54 AM Changeset in webkit [32648] by
-
- 2 edits in trunk/JavaScriptCore
2008-04-28 Darin Adler <Darin Adler>
Reviewed by Mitz.
- fix assertion hit every time you view www.apple.com
- kjs/PropertyNameArray.cpp: (KJS::PropertyNameArray::add): Changed assertion to allow null and empty strings. Now to find out why we have a property named "" and if that's a bug!
- 8:25 AM Changeset in webkit [32647] by
-
- 3 edits4 adds in trunk
Holger Hans Peter Freyther <zecke@selfish.org>
Fix the crash on http://www.orad.pl. We called end too early.
- we need a ::write that will pause the parser due a pending script
- we need a second write to queue data (tricky part)
- we need to get a finish call (document loaded)
- we resume the script and parse, schedule loading of a new script and have called end
- we resume the new script and we get a crash
This bug is hard to trigger, e.g. if the web server is only allowing
one connection per client/IP. This is why we need to sleep in the scripts. We
want to make sure that the page is first fully received before serving the
scripts.
- 8:24 AM Changeset in webkit [32646] by
-
- 2 edits in trunk/LayoutTests
Holger Hans Peter Freyther <zecke@selfish.org>
Update the Skipped list, we pass most of the http/tests/misc
- 8:24 AM Changeset in webkit [32645] by
-
- 2 edits in trunk/WebKitTools
Holger Hans Peter Freyther <zecke@selfish.org>
Reset dumping resource load callbacks to false for the next test
- 8:14 AM Changeset in webkit [32644] by
-
- 2 edits in trunk/WebCore
Ariya Hidayat <ahidayat@trolltech.com>
Fix rounded corner on combo box with CSS border
- 8:14 AM Changeset in webkit [32643] by
-
- 13 edits in trunk
Tor Arne Vestbø <tavestbo@trolltech.com>
Fix QWebView::loadFinished isn't always emitted
Replaced loadDone() with loadFinished(bool) and moved the signals for progress
tracking (start, progres, and finish) to the page instead of the frame. This
ensures that we emit loadFinished even when a subframe started the actual load.
This causes a few regressions in the layout tests that we for now accept for
the sake of the correct API. Layout tests we can fix any time though, including
patch release, the API however we can't change anymore in patch releases.
- 8:09 AM Changeset in webkit [32642] by
-
- 2 edits in trunk/WebCore
Ariya Hidayat <ahidayat@trolltech.com>
Fix CSS border does not show up for combo boxes
Do not clear the border when painting the combo box for 'select' form element
- 8:09 AM Changeset in webkit [32641] by
-
- 4 edits in trunk/WebKit/qt
Kavindra Devi Palaraja <kdpalara@trolltech.com>
Added more documentation for QWebSettings, QWebPluginFactory and QWebFrame
- 8:09 AM Changeset in webkit [32640] by
-
- 2 edits in trunk/WebKit/qt
Ariya Hidayat <ahidayat@trolltech.com>
Fix document/frame title not reset when loading a new URI
We should assume first the frame has no title. If it has, then the dispatchDidReceiveTitle()
will be called very soon with the correct title.
This properly resets the title when we navigate to a URI without a title.
- 8:08 AM Changeset in webkit [32639] by
-
- 2 edits in trunk/WebCore
Olivier Goffart <ogoffart@trolltech.com>
Avoid comparing to 0 in qFuzzyCompare for increased
numeric stability.
- 7:51 AM Changeset in webkit [32638] by
-
- 2 edits in trunk/WebKit/qt
David Boddie <dboddie@trolltech.com>
Minor documentation changes.
- 7:51 AM Changeset in webkit [32637] by
-
- 2 edits in trunk/WebCore
Benjamin Meyer <bmeyer@trolltech.com>
Fixes: Pages returning 404 with content are not displayed.
A web page that returns 404 can still have content.
- 7:50 AM Changeset in webkit [32636] by
-
- 2 edits in trunk/WebCore
Paul Olav Tvete <paul@trolltech.com>
Fix compiler warning
- 7:29 AM Changeset in webkit [32635] by
-
- 4 edits in trunk
Tor Arne Vestbø <tavestbo@trolltech.com>
Prevent middle-click from triggering open URL from clipboard when the event has already been accepted.
- 7:29 AM Changeset in webkit [32634] by
-
- 5 edits in trunk
Tor Arne Vestbø <tavestbo@trolltech.com>
Fix scrollbar behavior in QtWebKit to match QScrollBar.
Right click context menu is now supported, along with
middle click to center slider thumb over mouse cursor.
- 7:24 AM Changeset in webkit [32633] by
-
- 5 edits1 add in trunk
Tor Arne Vestbø <tavestbo@trolltech.com>
Implemented channel-based logging for QtWebKit.
Comma-separated log channels are read from the QT_WEBKIT_LOG environment variable.
Warnings for notImplemented() is still output by default, but can be disabled
by setting DISABLE_NI_WARNINGS=1.
- 6:52 AM Changeset in webkit [32632] by
-
- 5 edits in trunk/WebKit/qt
Simon Hausmann <shausman@trolltech.com>
Removed setHtml(const QByteArray &) overload as it breaks the common use of the setHtml() API.
- 6:52 AM Changeset in webkit [32631] by
-
- 4 edits in trunk/WebKit/qt
Kavindra Devi Palaraja <kdpalara@trolltech.com>
Doc - adding more documentation to QWebFrame, QWebPage and QWebView
- 6:50 AM Changeset in webkit [32630] by
-
- 4 edits in trunk
Simon Hausmann <shausman@trolltech.com>
Fixes: Popups/Context menu in WebKit appearing at the wrong location when embedded in the graphics view or using multiple screens
Give the popups the right parent widget and the right coordinates relative within the parent.
- 6:45 AM Changeset in webkit [32629] by
-
- 4 edits in trunk/WebKit/qt
Benjamin Meyer <bmeyer@trolltech.com>
Doc: Add see also's (and a few minor whitespace/typo corrections)
- 6:45 AM Changeset in webkit [32628] by
-
- 2 edits in trunk/WebKit/qt
Kavindra Devi Palaraja <kdpalara@trolltech.com>
Submitting more documentation for QWebPage
- 6:45 AM Changeset in webkit [32627] by
-
- 2 edits in trunk/WebCore
Tor Arne Vestbø <tavestbo@trolltech.com>
Fixes: Tabbing to a checkbox/button doesn't work on Win/X11
Respect platform settings for tabbing to all controls.
- 6:45 AM Changeset in webkit [32626] by
-
- 2 edits in trunk/WebCore
Tor Arne Vestbø <tavestbo@trolltech.com>
First part of the fix for tabbing to a checkbox/button doesn't work on Win/X11
Draw controls instead of pimitive elements, to get focus rects for free.
- 6:44 AM Changeset in webkit [32625] by
-
- 5 edits in trunk
Lincoln Ramsay <lincoln.ramsay@trolltech.com>
Compile when QT_NO_CLIPBOARD is defined.
- 6:44 AM Changeset in webkit [32624] by
-
- 2 edits in trunk/WebKit/qt
David Boddie <dboddie@trolltech.com>
Minor documentation fix.
- 6:34 AM Changeset in webkit [32623] by
-
- 3 edits in trunk/WebCore
Ariya Hidayat <ariya.hidayat@trolltech.com>
Draw the arrow button on the styled combo box using the fallback ("Windows") style
- 6:30 AM Changeset in webkit [32622] by
-
- 2 edits in trunk/WebCore
Fix positioning of native widgets (plugins).
Correctly convert the coordinates of the Widget in Widget::setFrameGeometry()
to be relative to the containing toplevel QWidget using convertToContainingWindow().
Note: convertToContainingWindow() takes child coordinates.
- 6:30 AM Changeset in webkit [32621] by
-
- 3 edits in trunk/WebCore
Holger Hans Peter Freyther <zecke@selfish.org>
plugins: fix scrolling issues
When scrolling you can see artefacts that remain on the screen, they
seem to originate from the backingstore handling. I'm not entirely sure
where they come from, my workarounds involved a
data->m_widget->parentWidget()->update() call in Widget::geometryChanged or
handling widgets with a native QWidget embedded like web pages with a
static background. This needs more investigations, I think the ScrollView
change is cleaner than the brute force in Widget::geometryChanged
Focus handling seems to be broken as well and needs further work, we don't
clip plugin QWidgets and they can draw out of their bounds, e.g. on the ScrollBar,
the z-order is likely not honored as well.
- 6:29 AM Changeset in webkit [32620] by
-
- 1 edit3 adds in trunk/WebCore
Holger Hans Peter Freyther <zecke@selfish.org>
Add manual test to test placement of QWidgets in iframe's
- 6:29 AM Changeset in webkit [32619] by
-
- 3 edits in trunk/WebKit/qt
Holger Hans Peter Freyther <zecke@selfish.org>
Implement QWebPage::createPlugin
The code is coming from the demo browser and needed here
for some manual tests.
- 4:59 AM Changeset in webkit [32618] by
-
- 2 edits in trunk/WebKit/qt
Qt/Win build fix. Include config.h to get the implicit MathExtras.h
inclusion correct with regards to rand_s.
- 4:21 AM Changeset in webkit [32617] by
-
- 2 edits in trunk/WebCore
Fix the build with ENABLE_DATABASE=0 (Qt/Windows) by including
SecurityOrigin.h. It's used in the file and also implicitly included
through the database headers.
- 3:52 AM Changeset in webkit [32616] by
-
- 2 edits in trunk/WebCore
Prospective Wx build fix.
- 3:52 AM Changeset in webkit [32615] by
-
- 2 edits in trunk/WebCore
2008-04-28 Simon Hausmann <Simon Hausmann>
Reviewed by Alp.
Gtk build fix, similar to the changes done in r32609.
- 3:24 AM Changeset in webkit [32614] by
-
- 2 edits in trunk/WebCore
2008-04-28 Simon Hausmann <Simon Hausmann>
Prospective build fix. After r32609 it seems the workaround here
is not needed anymore. With gcc 4.2/4.3 before 32609 the workaround
is needed for compilation, after 32609 compilation only succeeds
without the workaround.
- plugins/PluginPackage.h: Removed workaround of extra StorageTraits specification for the custom hash function of PluginPackage used in PluginSet.
- 3:10 AM Changeset in webkit [32613] by
-
- 6 edits in trunk
2008-04-28 Tor Arne Vestbø <tavestbo@trolltech.com>
Reviewed by Simon.
Fix scrollbar painting issues in QtWebKit.
Hovering and click-draging outside of the scrollbar would
produce unexpected and inconcistent results. We also didn't
pass on leave-events to the underlying WebKit code, which
was nessecary to implement the paint fix.
- 12:41 AM Changeset in webkit [32612] by
-
- 8 edits1 add in trunk
Bug 17173: HTML5 Canvas API requires us to ignore certain operations
<https://bugs.webkit.org/show_bug.cgi?id=17173>
Reviewed by Maciej
This fixes the semantics of a number of Canvas functions to match
the new HTML5 non-throwing semantics.
Test: fast/canvas/canvas-path-with-inf-nan-dimensions.html
Apr 27, 2008:
- 11:57 PM Changeset in webkit [32611] by
-
- 2 edits in trunk/JavaScriptCore
2008-04-27 Mark Rowe <mrowe@apple.com>
Reviewed by Maciej Stachowiak.
Fix crash inside PtrHash::hash when loading a page.
- wtf/HashFunctions.h: Explicitly use the superclass implementation of hash to avoid infinite recursion.
- 11:49 PM Changeset in webkit [32610] by
-
- 3 edits2 adds in trunk
Reviewed by Adele.
https://bugs.webkit.org/show_bug.cgi?id=18652
onchange events don't seem to fire for input[type=range] controls.
Fire changeEvent when dragging the slider.
- 10:59 PM Changeset in webkit [32609] by
-
- 42 edits in trunk
2008-04-25 Darin Adler <Darin Adler>
Reviewed by Maciej.
- fix <rdar://problem/5657459> REGRESSION: JavaScriptCore no longer builds with GCC 4.2 due to pointer aliasing warnings
Fix this by removing the HashTable optimizations that allowed us to share a back end
implementation between hash tables with integers, pointers, RefPtr, and String objects
as keys. The way it worked was incompatible with strict aliasing.
This increases code size. On Mac OS X we'll have to regenerate .order files to avoid
slowing down Safari startup times.
This creates a slight slowdown in SunSpider, mitigated by the following four speedups:
- speed up array put slightly by moving a branch (was already done for get)
- speed up symbol table access by adding a function named inlineGet to HashMap and using that in symbolTableGet/Put
- speed up PropertyNameArray creation by reducing the amount of reference count churn and uniqueness checking when adding names and not doing any allocation at all when building small arrays
- speed up conversion of strings to floating point numbers by eliminating the malloc/free of the buffer for the ASCII copy of the string; a way to make things even faster would be to change strtod to take a UTF-16 string
Note that there is considerable unused complexity now in HashSet/Map/Table to support
"storage types", which is no longer used. Will do in a separate patch.
- API/JSCallbackObjectFunctions.h: (KJS::JSCallbackObject<Base>::getPropertyNames): Removed explicit cast to Identifier to take advantage of the new PropertyNameArray::add overload and avoid reference count churn.
- API/JSObjectRef.cpp: (JSPropertyNameAccumulatorAddName): Ditto.
- JavaScriptCore.exp: Updated PropertyNameArray::add entry point name.
- kjs/JSVariableObject.cpp: Removed now-unneeded IdentifierRepHashTraits::nullRepPtr definition (see below). (KJS::JSVariableObject::getPropertyNames): Removed explicit cast to Identifier.
- kjs/JSVariableObject.h: (KJS::JSVariableObject::symbolTableGet): Use inlineGet for speed. Also changed to do early exit instead of nesting the body inside an if. (KJS::JSVariableObject::symbolTablePut): Ditto.
- kjs/PropertyNameArray.cpp: (KJS::PropertyNameArray::add): Changed implementation to take a raw pointer instead of a reference to an identifier. Do uniqueness checking by searching the vector when the vector is short, only building the set once the vector is large enough.
- kjs/PropertyNameArray.h: Added an overload of add for a raw pointer, and made the old add function call that one. Added an addKnownUnique function for use when the new name is known to be different from any other in the array. Changed the vector to have an inline capacity of 20.
- kjs/SymbolTable.h: Changed IdentifierRepHash to inherit from the default hash for a RefPtr so we don't have to define so much. Added an overload of the hash function for a raw pointer as required by the new RefPtrHashMap. Got rid of the now-unneeded IdentifierRepHashTraits -- the default traits now work fine. Added a definition of empthValueIsZero to SymbolTableIndexHashTraits; not having it was incorrect, but harmless.
- kjs/array_instance.cpp: (KJS::ArrayInstance::put): Move the maxArrayIndex check inside the branch that checks the index against the length, as done in the get function.
- kjs/function.cpp: (KJS::globalFuncKJSPrint): Changed to use the new getCString instead of cstring.
- kjs/internal.cpp: Removed printInfo debugging function, a client of cstring. If we need a debugging function we can easily make a better one and we haven't used this one in a long time.
- kjs/internal.h: Ditto.
- kjs/object.cpp: (KJS::JSObject::getPropertyNames): Removed explicit cast to Identifier.
- kjs/property_map.cpp: (KJS::PropertyMap::getEnumerablePropertyNames): Ditto. Also added a special case for the case where the propertyNames array is empty -- in that case we know we're adding a set of names that are non-overlapping so we can use addKnownUnique.
- kjs/ustring.cpp: (KJS::UString::getCString): Replaces cstring. Puts the C string into a CStringBuffer, which is a char Vector with an inline capacity. Also returns a boolean to indicate if the converion was lossy, which eliminates the need for a separate is8Bit call. (KJS::UString::toDouble): Changed to call getCString instead of cstring.
- kjs/ustring.h: Ditto.
- wtf/HashFunctions.h: Overload the hash and equal functions for RefPtr's default hash to take raw pointers. This works with the changes to RefPtrHashMap to avoid introducing refcount churn.
- wtf/HashMap.h: Removed special code to convert the deleted value to the empty value when writing a new value into the map. This is now handled elsewhere. (WTF::HashMap::get): Removed code that checks for an empty hash table before calling HashTable::lookup; it's slightly more efficient to do this check inside lookup.
- wtf/HashTable.h: (WTF::HashTable::isDeletedBucket): Changed to use isDeletedValue instead of using deletedValue and the equality operator. (WTF::HashTable::deleteBucket): Changed to use constructDeletedValue instead of using deletedValue and the assignment operator. (WTF::HashTable::checkKey): Added. Factors out the check for values that are empty or deleted keys that's used in various functions below. (WTF::HashTable::lookup): Changed to use checkKey, check for a 0 table, and also made public for use by RefPtrHashMap. (WTF::HashTable::lookupForWriting): Changed to use checkKey. (WTF::HashTable::fullLookupForWriting): Changed to use checkKey. (WTF::HashTable::add): Changed to use checkKey, and call initializeBucket on a deleted bucket before putting a new entry into it. (WTF::HashTable::addPassingHashCode): Ditto. (WTF::HashTable::deallocateTable): Check isDeletedBucket before calling ~ValueType.
- wtf/HashTraits.h: Got ridd of all the HashTraits specialization for the integer types, since GeneicHashTraitsBase already deals with integers separately. Put the deleted value support into GenericHashTraitsBase. Changed FloatHashTraits to inherit from GenericHashTraits, and define construct/isDeletedValue rather than deletedValue. Removed the ref and deref functions from RefPtr's HashTraits, and defined construct/isDeletedValue. Eliminated DeletedValueAssigner. Changed PairHashTraits to define construct/isDeletedValue, and also merged PairBaseHashTraits in with PairHashTraits. Got rid of all specialization of HashKeyStorageTraits. We'll remove that, and the needsRef data member, later.
- wtf/RefPtr.h: Added HashTableDeletedValueType, an enum type with a single value, HashTableDeletedValue. Used that type to make a new constructor to construct deleted values and also added an isHashTableDeletedValue function.
- wtf/RefPtrHashMap.h: Added RefPtrHashMapRawKeyTranslator and used it to implement the raw pointer functions. This is a way to continue to avoid refcount thrash. We can't use the old way because it depended on the underlying map using a non-RefPtr type. (WTF::HashMap::find): Use find with RefPtrHashMapRawKeyTranslator. (WTF::HashMap::contains): Use contains with RefPtrHashMapRawKeyTranslator. (WTF::HashMap::inlineAdd): Use add with RefPtrHashMapRawKeyTranslator. (WTF::HashMap::get): Removed code that checks for an empty hash table before calling HashTable::lookup; it's slightly more efficient to do this check inside lookup. (WTF::HashMap::inlineGet): Added. Just like get, but marked inline for use in the symbol table code.
WebCore:
2008-04-25 Darin Adler <Darin Adler>
Reviewed by Maciej.
- update for compatibility with HashTable that no longer has optimization to share implementation between hash tables with integers, pointers, RefPtr, and String objects as keys
- bindings/js/JSSVGPODTypeWrapper.h: (WebCore::PODTypeReadWriteHashInfo::PODTypeReadWriteHashInfo): Added constructor for HashTableDeletedValue. (WebCore::PODTypeReadWriteHashInfo::isHashTableDeletedValue): Added. (WebCore::PODTypeReadWriteHashInfoTraits::constructDeletedValue): Added. (WebCore::PODTypeReadWriteHashInfoTraits::isDeletedValue): Added.
- dom/Document.cpp: Made changedDocuments internal to the file rather than a static data member of Document. (WebCore::FormElementKey::ref): Removed unneeded check for deleted value -- this will never be called on a deleted element. (WebCore::FormElementKey::deref): Ditto.
- dom/Document.h: Added HashTableDeletedValue constructor and isHashTableDeletedValue to FormElementKey. Changed FormElementKeyHashTraits to use construct/isDeletedValue. Got rid of the changedDocuments data member. Changed iconURL to be an inline that returns a const String&.
- dom/StyledElement.cpp: Changed MappedAttributeKeyTraits to use construct/isDeletedValue.
- page/mac/AXObjectCacheMac.mm: (WebCore::AXObjectCache::getAXID): Call isDeletedValue instead of deletedValue.
- platform/SecurityOriginHash.h: Added overload so that SecurityOriginHash can work with raw pointers as well as RefPt (helpful with the new RefPtrHashMap). Eliminated SecurityOriginTraits, since we can now use the default traits. Changed the value of safeToCompareToEmptyOrDeleted to false, since it's not safe to compare a deleted value using this hash function. I don't think it was safe before either; I'm not sure why it didn't cause a problem before.
- platform/cf/SchedulePair.h: Removed SchedulePairTraits -- custom traits are no longer needed.
- platform/graphics/FontCache.cpp: (WebCore::FontPlatformDataCacheKey::FontPlatformDataCacheKey): Added constructor for HashTableDeletedValue. (WebCore::FontPlatformDataCacheKey::isHashTableDeletedValue): Added. (WebCore::FontPlatformDataCacheKey::hashTableDeletedSize): Added. (WebCore::FontPlatformDataCacheKeyTraits::constructDeletedValue): Added. (WebCore::FontPlatformDataCacheKeyTraits::isDeletedValue): Added. (WebCore::FontDataCacheKeyTraits::constructDeletedValue): Added. (WebCore::FontDataCacheKeyTraits::isDeletedValue): Added.
- platform/graphics/IntSizeHash.h: Changed HashTraits<IntSize> to use construct/isDeletedValue.
- platform/graphics/mac/FontPlatformData.h: (WebCore::FontPlatformData::FontPlatformData): Added constructor for HashTableDeletedValue. (WebCore::FontPlatformData::isHashTableDeletedValue): Added. (WebCore::FontPlatformData::hashTableDeletedFontValue): Added.
- platform/text/PlatformString.h: (WebCore::String::swap): Added. Avoids any refcount churn when swapping two strings. (WebCore::String::String): Added constructor for HashTableDeletedValue. (WebCore::String::isHashTableDeletedValue): Added. (WebCore::swap): Added. Avoids any refcount churn when swapping two strings.
- platform/text/StringHash.h: Changed specialization of HashTraits for WebCore::String to use the deleted value now defined in that class and removed the code to do ref/deref. Removed HashKeyStorageTraits specializations.
- platform/win/COMPtr.h: Changed specialization of HashTraits for COMPtr to use the deleted value now defined in that class and removed the code to do ref/deref. Removed HashKeyStorageTraits specializations. (COMPtr::COMPtr): Added constructor for HashTableDeletedValue. (COMPtr::isHashTableDeletedValue): Added. (COMPtr::query): Removed inline keyword not needed since functions defined in the class definition are automatically marked inline. (COMPtr::hashTableDeletedValue): Added.
- storage/DatabaseTracker.h: Removed now-unneeded SecurityOriginTraits.
- storage/LocalStorage.h: Ditto.
- storage/OriginQuotaManager.h: Ditto.
- storage/SessionStorage.h: Ditto.
- svg/SVGAnimatedTemplate.h: (WebCore::SVGAnimatedTypeWrapperKey::SVGAnimatedTypeWrapperKey): Added constructor for HashTableDeletedValue. (WebCore::SVGAnimatedTypeWrapperKey::isHashTableDeletedValue): Added. (WebCore::SVGAnimatedTypeWrapperKeyHashTraits::constructDeletedValue): Added. (WebCore::SVGAnimatedTypeWrapperKeyHashTraits::isDeletedValue): Added.
- 10:44 PM Changeset in webkit [32608] by
-
- 2 edits in trunk/WebCore
2008-04-27 Mark Rowe <mrowe@apple.com>
Reviewed by Tim Hatcher.
Initialize the extra member of the xmlEntity struct when using libxml2 >= 2.6.27.
- dom/XMLTokenizer.cpp: (WebCore::):
- 10:37 PM Changeset in webkit [32607] by
-
- 3 edits in trunk/WebKit/wx
Add methods to check if there is a previous/next page in the history. Also some coding style cleanup.
https://bugs.webkit.org/show_bug.cgi?id=18757
- 10:08 PM Changeset in webkit [32606] by
-
- 3 edits in branches/squirrelfish/JavaScriptCore
2008-04-27 Cameron Zwarich <cwzwarich@uwaterloo.ca>
Reviewed by Maciej.
Bug 18746: SQUIRRELFISH: indirect eval used when direct eval should be used
<https://bugs.webkit.org/show_bug.cgi?id=18746>
Change the base to the correct value of the 'this' object after the direct
eval test instead of before.
Fixes 5 layout tests.
- VM/Machine.cpp: (KJS::Machine::privateExecute):
- kjs/nodes.cpp: (KJS::EvalFunctionCallNode::emitCode):
- 9:46 PM Changeset in webkit [32605] by
-
- 6 edits3 adds in trunk
WebCore:
Reviewed by Darin Adler.
- fix https://bugs.webkit.org/show_bug.cgi?id=3729 <rdar://problem/4036353> REGRESSION: arrow keys move insertion bar backwards in RTL text
Test: editing/selection/move-left-right.html
- editing/SelectionController.cpp: (WebCore::SelectionController::modifyMovingRight): Added. Currently implemented for character granularity, all other being treated as "forward". (WebCore::SelectionController::modifyMovingForward): Renamed modifyMovingRightForward() to this. (WebCore::SelectionController::modifyMovingLeft): Added. Currently implemented for character granularity, all other being treated as "backward". (WebCore::SelectionController::modifyMovingBackward): Renamed modifyMovingLeftBackward() to this. (WebCore::SelectionController::modify): Changed to call either the visual (left/right) or logical (backward/forward) methods depending on the 'dir' argument for moves.
- editing/SelectionController.h:
- editing/VisiblePosition.cpp: (WebCore::VisiblePosition::leftVisuallyDistinctCandidate): Added. (WebCore::VisiblePosition::left): Added. (WebCore::VisiblePosition::rightVisuallyDistinctCandidate): Added. (WebCore::VisiblePosition::right): Added.
- editing/VisiblePosition.h:
LayoutTests:
Reviewed by Darin Adler.
- test for https://bugs.webkit.org/show_bug.cgi?id=3729 <rdar://problem/4036353> REGRESSION: arrow keys move insertion bar backwards in RTL text
- editing/selection/move-left-right-expected.txt: Added.
- editing/selection/move-left-right.html: Added.
- platform/mac/editing/selection/move-left-right-expected.txt: Added.
- 9:32 PM Changeset in webkit [32604] by
-
- 5 edits in trunk
WebCore:
2008-04-27 Sam Weinig <sam@webkit.org>
Reviewed by Maciej Stachowiak.
Fix for <rdar://problem/5893385> Need to provide access to the CanvasRenderingContext2D prototype
Add a JS constructor for CanvasRenderingContext2D.
- html/CanvasRenderingContext2D.idl:
- page/DOMWindow.idl:
LayoutTests:
2008-04-27 Sam Weinig <sam@webkit.org>
Reviewed by Maciej Stachowiak.
Update test to reflect add the CanvasRenderingContext2d constructor.
- fast/dom/Window/window-properties-expected.txt:
Apr 26, 2008:
- 10:27 PM Changeset in webkit [32603] by
-
- 11 edits in tags/Safari-6526.2
Merge r32530.
- 10:27 PM Changeset in webkit [32602] by
-
- 8 edits in tags/Safari-6526.2
Merge r32583.
- 8:10 PM Changeset in webkit [32601] by
-
- 4 edits41 deletes in trunk/BugsSite
Removed temp files and added appropriate svn:ignore properties.
Rubber-stamped by Mark Rowe.
- data: Added svn:ignore property for "versioncache" and "versioncache.*" files.
- data/template: Added svn:ignore property for "template" subdirectory.
- data/template/template: Removed precompiled template subdirectory.
- data/versioncache: Removed.
- data/versioncache.*: Removed backup versioncache files.
- data/webdot: Added svn:ignore property for "*.dot" files.
- data/webdot/*.dot: Removed cached webdot files.
- 7:29 PM Changeset in webkit [32600] by
-
- 2 edits in trunk/WebKitTools
Delete the DerivedSources after make clean has been done so that the DerivedSouces don't get re-created. Also, use the proper extension for the Win wxPython extension.
- 7:24 PM Changeset in webkit [32599] by
-
- 4 edits in trunk
Versioning.
- 7:23 PM Changeset in webkit [32598] by
-
- 1 copy in tags/Safari-6526.2
New tag.
- 6:56 PM Changeset in webkit [32597] by
-
- 29 edits6 moves4 adds in trunk
WebCore:
2008-04-26 Adam Barth <hk9565@gmail.com>
Reviewed by Adam Roben and Sam Weinig.
Fixes bug http://bugs.webkit.org/show_bug.cgi?id=17331
Fixes bug http://bugs.webkit.org/show_bug.cgi?id=16848
Updates postMessage implementation to match HTML 5 specification:
1) Adds origin parameter to postMessage.
2) Removes domain and uri attributes of MessageEvent in favor of
origin attribute.
In order to do this correctly, we need to distinguish between hosts and domains
in the SecurityOrigin class. There are now three ways to compare security origins:
1) isSameSchemeHostPort compares scheme, host, and port, and is used for postMessage
2) equal compares all aspects of the security origin, and is used for hash keys
3) canAccess understands the semantics of schemes such as file:// and data:// URLs,
and should be used for scripting access checks.
Changed SecurityOrigin::toString() to generate identifiers that are suitable for
being used as a MessageEvent's origin property. In the future, they could be used
as database string identifiers as well. Re-used KURL parser to parse serialized
SecurityOrigins.
Collin Jackson <collinj-webkit@collinjackson.com> also contributed to this patch.
Tests: http/tests/security/postMessage/invalid-origin-throws-exception.html
http/tests/security/postMessage/javascript-page-still-sends-origin.html
http/tests/security/postMessage/origin-unaffected-by-base-tag.html
http/tests/security/postMessage/origin-unaffected-by-document-domain.html
http/tests/security/postMessage/target-origin.html
- WebCore.base.exp:
- bindings/js/JSDOMWindowCustom.cpp: (WebCore::JSDOMWindow::postMessage):
- dom/Document.cpp: (WebCore::Document::domain):
- dom/MessageEvent.cpp: (WebCore::MessageEvent::MessageEvent): (WebCore::MessageEvent::initMessageEvent):
- dom/MessageEvent.h: (WebCore::MessageEvent::origin):
- dom/MessageEvent.idl:
- html/CanvasRenderingContext2D.cpp: (WebCore::CanvasRenderingContext2D::checkOrigin): (WebCore::CanvasRenderingContext2D::createPattern):
- page/DOMWindow.cpp: (WebCore::DOMWindow::postMessage):
- page/DOMWindow.h:
- page/DOMWindow.idl:
- platform/SecurityOrigin.cpp: (WebCore::SecurityOrigin::SecurityOrigin): (WebCore::SecurityOrigin::create): (WebCore::SecurityOrigin::createForFrame): (WebCore::SecurityOrigin::copy): (WebCore::SecurityOrigin::setDomainFromDOM): (WebCore::SecurityOrigin::canAccess): (WebCore::SecurityOrigin::isSecureTransitionTo): (WebCore::SecurityOrigin::toString): (WebCore::SecurityOrigin::createFromString): (WebCore::SecurityOrigin::createFromDatabaseIdentifier): (WebCore::SecurityOrigin::databaseIdentifier): (WebCore::SecurityOrigin::equal): (WebCore::SecurityOrigin::isSameSchemeHostPort):
- platform/SecurityOrigin.h: (WebCore::SecurityOrigin::host): (WebCore::SecurityOrigin::domain):
- platform/SecurityOriginHash.h: (WebCore::SecurityOriginTraits::deletedValue):
- storage/DatabaseTracker.cpp: (WebCore::DatabaseTracker::hasEntryForDatabase): (WebCore::DatabaseTracker::originPath): (WebCore::DatabaseTracker::fullPathForDatabase): (WebCore::DatabaseTracker::populateOrigins): (WebCore::DatabaseTracker::databaseNamesForOrigin): (WebCore::DatabaseTracker::detailsForNameAndOrigin): (WebCore::DatabaseTracker::setDatabaseDetails): (WebCore::DatabaseTracker::setQuota): (WebCore::DatabaseTracker::addDatabase): (WebCore::DatabaseTracker::deleteOrigin): (WebCore::DatabaseTracker::deleteDatabase):
WebKit/mac:
2008-04-20 Adam Barth <hk9565@gmail.com>
Reviewed by Adam Roben and Sam Weinig.
Updated WebSecurityOrigin to match new SecurityOrigin API.
Collin Jackson <collinj-webkit@collinjackson.com> also contributed to this patch.
- Storage/WebSecurityOrigin.mm: (-[WebSecurityOrigin host]): (-[WebSecurityOrigin domain]):
- Storage/WebSecurityOriginPrivate.h:
WebKit/win:
2008-04-26 Adam Barth <hk9565@gmail.com>
Reviewed by Adam Roben and Sam Weinig.
Renamed "domain" method to "host" to match SecurityOrigin.
- Interfaces/IWebSecurityOrigin.idl:
- WebSecurityOrigin.cpp: (WebSecurityOrigin::host):
- WebSecurityOrigin.h:
WebKitTools:
2008-04-26 Adam Barth <hk9565@gmail.com>
Reviewed by Adam Roben and Sam Weinig.
Updates LayoutTestController to use host instead of domain.
Collin Jackson <collinj-webkit@collinjackson.com> also contributed to this patch.
- DumpRenderTree/mac/LayoutTestControllerMac.mm: (LayoutTestController::setDatabaseQuota):
- DumpRenderTree/mac/UIDelegate.mm: (-[UIDelegate webView:frame:exceededDatabaseQuotaForSecurityOrigin:database:]):
LayoutTests:
2008-04-26 Adam Barth <hk9565@gmail.com>
Reviewed by Adam Roben and Sam Weinig.
Update LayoutTests for new postMessage API.
Collin Jackson <collinj-webkit@collinjackson.com> also contributed to this patch.
- http/tests/security/postMessage/domain-and-uri-unaffected-by-base-tag-expected.txt: Removed.
- http/tests/security/postMessage/domain-and-uri-unaffected-by-base-tag.html: Removed.
- http/tests/security/postMessage/domain-unaffected-by-document-domain-expected.txt: Removed.
- http/tests/security/postMessage/domain-unaffected-by-document-domain.html: Removed.
- http/tests/security/postMessage/invalid-origin-throws-exception-expected.txt: Added.
- http/tests/security/postMessage/invalid-origin-throws-exception.html: Added.
- http/tests/security/postMessage/javascript-page-still-sends-domain-expected.txt: Removed.
- http/tests/security/postMessage/javascript-page-still-sends-domain.html: Removed.
- http/tests/security/postMessage/javascript-page-still-sends-origin-expected.txt: Copied from LayoutTests/http/tests/security/postMessage/javascript-page-still-sends-domain-expected.txt.
- http/tests/security/postMessage/javascript-page-still-sends-origin.html: Copied from LayoutTests/http/tests/security/postMessage/javascript-page-still-sends-domain.html.
- http/tests/security/postMessage/origin-unaffected-by-base-tag-expected.txt: Copied from LayoutTests/http/tests/security/postMessage/domain-and-uri-unaffected-by-base-tag-expected.txt.
- http/tests/security/postMessage/origin-unaffected-by-base-tag.html: Copied from LayoutTests/http/tests/security/postMessage/domain-and-uri-unaffected-by-base-tag.html.
- http/tests/security/postMessage/origin-unaffected-by-document-domain-expected.txt: Copied from LayoutTests/http/tests/security/postMessage/domain-unaffected-by-document-domain-expected.txt.
- http/tests/security/postMessage/origin-unaffected-by-document-domain.html: Copied from LayoutTests/http/tests/security/postMessage/domain-unaffected-by-document-domain.html.
- http/tests/security/postMessage/target-origin-expected.txt: Added.
- http/tests/security/postMessage/target-origin.html: Added.
- 6:52 PM Changeset in webkit [32596] by
-
- 4 edits in trunk
Another round of build fixes, hopefully the last this time.
- 5:35 PM Changeset in webkit [32595] by
-
- 2 edits in trunk/WebKitTools
Reviewed by Kevin Ollivier.
Allow the user to set the path to SWIG using an environment variable.
- 5:28 PM Changeset in webkit [32594] by
-
- 2 edits in trunk/WebCore
wx build fix. Add needed wx includes for compilation.
https://bugs.webkit.org/show_bug.cgi?id=18758
- 5:21 PM Changeset in webkit [32593] by
-
- 2 edits in trunk/WebKitTools
wx build fix. Download the latest libpng version for building the dependencies.
- 1:12 PM Changeset in webkit [32592] by
-
- 5 edits in branches/squirrelfish/JavaScriptCore
2008-04-26 Maciej Stachowiak <mjs@apple.com>
Reviewed by Oliver.
- document all property getting, setting and deleting opcodes
(And fix function parameter names to match corresponding opcode parameter names.)
- VM/CodeGenerator.cpp: (KJS::CodeGenerator::emitResolve): (KJS::CodeGenerator::emitResolveBase): (KJS::CodeGenerator::emitResolveBaseAndProperty): (KJS::CodeGenerator::emitResolveBaseAndFunc): (KJS::CodeGenerator::emitGetPropId): (KJS::CodeGenerator::emitPutPropId): (KJS::CodeGenerator::emitDeletePropId): (KJS::CodeGenerator::emitPutPropVal):
- VM/CodeGenerator.h:
- VM/Machine.cpp: (KJS::resolve): (KJS::resolveBase): (KJS::resolveBaseAndProperty): (KJS::resolveBaseAndFunc): (KJS::Machine::privateExecute):
- kjs/nodes.cpp: (KJS::ResolveNode::emitCode): (KJS::ArrayNode::emitCode): (KJS::PropertyListNode::emitCode): (KJS::BracketAccessorNode::emitCode): (KJS::EvalFunctionCallNode::emitCode): (KJS::FunctionCallResolveNode::emitCode): (KJS::FunctionCallBracketNode::emitCode): (KJS::PostIncResolveNode::emitCode): (KJS::PostDecResolveNode::emitCode): (KJS::PostIncBracketNode::emitCode): (KJS::PostDecBracketNode::emitCode): (KJS::PostIncDotNode::emitCode): (KJS::PostDecDotNode::emitCode): (KJS::DeleteResolveNode::emitCode): (KJS::TypeOfResolveNode::emitCode): (KJS::PreIncResolveNode::emitCode): (KJS::PreDecResolveNode::emitCode): (KJS::PreIncBracketNode::emitCode): (KJS::PreDecBracketNode::emitCode): (KJS::AssignResolveNode::emitCode): (KJS::AssignDotNode::emitCode): (KJS::ReadModifyDotNode::emitCode): (KJS::AssignBracketNode::emitCode): (KJS::ReadModifyBracketNode::emitCode): (KJS::ConstDeclNode::emitCodeSingle):
- 9:41 AM Changeset in webkit [32591] by
-
- 6 edits8 adds in trunk
WebCore:
2008-04-26 Anatoli Papirovski <apapirovski@mac.com>
Reviewed by Dave Hyatt.
Fix for https://bugs.webkit.org/show_bug.cgi?id=18583
WebKit should ignore declarations with a negative value for
line-height and font-size, including in the shorthand font form.
Tests: fast/css/font-size-negative.html
fast/css/line-height-negative.html
- css/CSSParser.cpp: (WebCore::CSSParser::parseValue): (WebCore::CSSParser::parseFont):
LayoutTests:
2008-04-26 Anatoli Papirovski <apapirovski@mac.com>
Reviewed by Dave Hyatt.
Test for https://bugs.webkit.org/show_bug.cgi?id=18583
Negative values are invalid for font-size and line-height,
including in the shorthand "font".
- fast/css/font-size-negative.html: Added.
- fast/css/line-height-negative.html: Added.
- platform/mac/css2.1/t100801-c548-ln-ht-02-b-ag-expected.checksum:
- platform/mac/css2.1/t100801-c548-ln-ht-02-b-ag-expected.png:
- platform/mac/css2.1/t100801-c548-ln-ht-02-b-ag-expected.txt:
- platform/mac/fast/css/font-size-negative-expected.checksum: Added.
- platform/mac/fast/css/font-size-negative-expected.png: Added.
- platform/mac/fast/css/font-size-negative-expected.txt: Added.
- platform/mac/fast/css/line-height-negative-expected.checksum: Added.
- platform/mac/fast/css/line-height-negative-expected.png: Added.
- platform/mac/fast/css/line-height-negative-expected.txt: Added.
- 2:07 AM Changeset in webkit [32590] by
-
- 9 edits2 adds in branches/squirrelfish
Bug 18628: SQUIRRELFISH: need to support recursion limit
<https://bugs.webkit.org/show_bug.cgi?id=18628>
Reviewed by Maciej.
Basically completes recursion limiting. There is still some
tuning we may want to do to make things better in the face of
very bad code, but certainly nothing worse than anything already
possible in trunk.
Also fixes a WebKit test by fixing the exception text :D
- 2:07 AM Changeset in webkit [32589] by
-
- 1 edit in branches/squirrelfish/JavaScriptCore/ChangeLog
Fix the changelog
Apr 25, 2008:
- 8:28 PM Changeset in webkit [32588] by
-
- 2 edits in trunk/JavaScriptGlue
<rdar://problem/5891264> Don't install the JavaScriptGlue headers
Reviewed by Adele Peterson.
- 7:02 PM Changeset in webkit [32587] by
-
- 11 edits1 delete in trunk/JavaScriptCore
2008-04-25 Sam Weinig <sam@webkit.org>
Rubber-stamped by Mark Rowe.
Remove SavedBuiltins and SavedProperties classes and the methods used to
save data to them. The CachedPage now stores a the JSGlobalObject in full.
- JavaScriptCore.exp:
- JavaScriptCore.vcproj/JavaScriptCore/JavaScriptCore.vcproj:
- JavaScriptCore.xcodeproj/project.pbxproj:
- kjs/JSGlobalObject.cpp:
- kjs/JSGlobalObject.h:
- kjs/JSVariableObject.cpp:
- kjs/JSVariableObject.h: (KJS::JSVariableObject::localStorage):
- kjs/SavedBuiltins.h: Removed.
- kjs/object.h:
- kjs/property_map.cpp:
- kjs/property_map.h:
- 5:55 PM Changeset in webkit [32586] by
-
- 3 edits2 adds in trunk/WebCore
2008-04-25 Anders Carlsson <andersca@apple.com>
Reviewed by Sam.
Add empty ApplicationCacheStorage class.
- WebCore.vcproj/WebCore.vcproj:
- WebCore.xcodeproj/project.pbxproj:
- loader/appcache/ApplicationCacheStorage.cpp: Added. (WebCore::cacheStorage):
- loader/appcache/ApplicationCacheStorage.h: Added.
- 5:45 PM Changeset in webkit [32585] by
-
- 7 edits in trunk/WebCore
2008-04-24 Sam Weinig <sam@webkit.org>
Reviewed by Geoffrey Garen.
Replace the JSDOMWindow with a new one instead of clearing it.
- bindings/js/JSDOMWindowBase.cpp: (WebCore::JSDOMWindowBase::clear): Don't reset the window.
- bindings/js/kjs_proxy.cpp: (WebCore::KJSProxy::clear): Replace the current JSDOMWindow stored in the wrapper window with a new one.
- history/CachedPage.cpp: (WebCore::CachedPage::CachedPage): (WebCore::CachedPage::restore): (WebCore::CachedPage::clear):
- history/CachedPage.h: Instead of storing the builtins, properties and localstorage in special holding structs, store the entire JSDOMWindow.
For the case of a page that has no JSDOMWindow before it's first
navigation, has one created while it is in the cache and is then
navigated back, create a new JSDOMWindow to keep the assumption
in JSDOMWindowWrapper that it always has the correct window.
- loader/FrameLoader.cpp: (WebCore::FrameLoader::clear): We must clear the DOMWindow first, so that we create a new JSDOMWindow, it gets a new one.
- page/Frame.cpp: (WebCore::Frame::clearDOMWindow): Null out the domWindow so that on the next access, a new one is created.
- 5:08 PM Changeset in webkit [32584] by
-
- 7 edits in trunk/WebCore
2008-04-25 Sam Weinig <sam@webkit.org>
Reviewed by Mark Rowe.
The EventListeners need to operate and store JSDOMWindows, not JSDOMWrapperWindows.
- bindings/js/JSDOMWindowBase.cpp: (WebCore::JSDOMWindowBase::~JSDOMWindowBase): (WebCore::JSDOMWindowBase::findOrCreateJSEventListener): (WebCore::JSDOMWindowBase::findOrCreateJSUnprotectedEventListener):
- bindings/js/JSSVGLazyEventListener.cpp: (WebCore::JSSVGLazyEventListener::JSSVGLazyEventListener):
- bindings/js/JSSVGLazyEventListener.h:
- bindings/js/kjs_events.cpp: (WebCore::JSAbstractEventListener::handleEvent): (WebCore::JSUnprotectedEventListener::JSUnprotectedEventListener): (WebCore::JSUnprotectedEventListener::~JSUnprotectedEventListener): (WebCore::JSUnprotectedEventListener::window): (WebCore::JSUnprotectedEventListener::clearWindow): (WebCore::JSEventListener::JSEventListener): (WebCore::JSEventListener::~JSEventListener): (WebCore::JSEventListener::window): (WebCore::JSEventListener::clearWindow): (WebCore::JSLazyEventListener::JSLazyEventListener): (WebCore::JSLazyEventListener::parseCode):
- bindings/js/kjs_events.h:
- bindings/js/kjs_proxy.cpp: (WebCore::KJSProxy::createHTMLEventHandler): (WebCore::KJSProxy::createSVGEventHandler):
- 5:03 PM Changeset in webkit [32583] by
-
- 8 edits in trunk
Add some content to an empty ICU header file to prevent verification errors.
Rubber-stamped by Sam Weinig.
- 4:38 PM Changeset in webkit [32582] by
-
- 2 edits in trunk/WebCore
2008-04-25 Anders Carlsson <andersca@apple.com>
Fix tyop.
- loader/DocumentLoader.cpp: (WebCore::DocumentLoader::scheduleApplicationCacheLoad):
- 4:36 PM Changeset in webkit [32581] by
-
- 12 edits in trunk
WebCore:
2008-04-25 Anders Carlsson <andersca@apple.com>
Reviewed by Sam.
Add and honor offlineWebApplicationCacheEnabled setting.
- WebCore.base.exp:
- loader/DocumentLoader.cpp: (WebCore::DocumentLoader::scheduleApplicationCacheLoad):
- loader/MainResourceLoader.cpp: (WebCore::MainResourceLoader::load):
- loader/appcache/ApplicationCacheGroup.cpp: (WebCore::ApplicationCacheGroup::selectCache): (WebCore::ApplicationCacheGroup::selectCacheWithoutManifestURL):
- page/Settings.cpp: (WebCore::Settings::Settings): (WebCore::Settings::setOfflineWebApplicationCacheEnabled):
- page/Settings.h: (WebCore::Settings::offlineWebApplicationCacheEnabled):
WebKit/mac:
2008-04-25 Anders Carlsson <andersca@apple.com>
Reviewed by Sam.
Add offlineWebApplicationCacheEnabled preference.
- WebView/WebPreferenceKeysPrivate.h:
- WebView/WebPreferences.m: (+[WebPreferences initialize]): (-[WebPreferences offlineWebApplicationCacheEnabled]): (-[WebPreferences setOfflineWebApplicationCacheEnabled:]):
- WebView/WebPreferencesPrivate.h:
- WebView/WebView.mm: (-[WebView _updateSettingsFromPreferences:]):
- 3:44 PM Changeset in webkit [32580] by
-
- 98 edits in trunk/WebKitSite
2008-04-25 Mark Rowe <mrowe@apple.com>
Upgrade to WordPress 2.5.1. Another day, another security vulnerability in WordPress.
- blog:
- 3:41 PM Changeset in webkit [32579] by
-
- 3 edits in branches/squirrelfish/JavaScriptCore
Bug 18736: SQUIRRELFISH: switch statements with no default have incorrect codegen
<https://bugs.webkit.org/show_bug.cgi?id=18736>
Reviewed by Geoff
Put a limit on the level of reentry recursion. 128 levels of re-entrant recursion
seems reasonable as it is greater than the old eval limit, and a long way short of
the reentry depth needed to overflow the stack.
- 2:44 PM Changeset in webkit [32578] by
-
- 2 edits in trunk/JavaScriptCore
<rdar://problem/5819422> REGRESSION: Wrong line number passed to -willLeaveCallFrame
Patch by George Dicker and Michael Kahl. Reviewed by Darin.
When -[NSObject(WebScriptDebugDelegate) webView:willLeaveCallFrame:sourceId:line:forWebFrame:]
is invoked, the first line number of the function is returned instead of the last
line number. This regressed in r28458.
- kjs/nodes.cpp: (KJS::FunctionBodyNodeWithDebuggerHooks::execute): Pass lastLine() instead of lineNo() when calling Debugger::returnEvent().
- 2:11 PM Changeset in webkit [32577] by
-
- 2 edits in trunk/WebCore
2008-04-25 Anders Carlsson <andersca@apple.com>
Reviewed by Adam.
<rdar://problem/5629139>
PluginInfoStoreWin needs to implement pluginNameForMIMEType().
- plugins/PluginInfoStore.cpp: (WebCore::PluginInfoStore::pluginNameForMIMEType):
- 1:09 PM Changeset in webkit [32576] by
-
- 2 edits in trunk/WebKit/win
2008-04-25 Anders Carlsson <andersca@apple.com>
Reviewed by Adam.
Fix internal debug build.
- WebKit.vcproj/WebKit.vcproj:
- 11:11 AM Changeset in webkit [32575] by
-
- 5 edits in branches/squirrelfish/JavaScriptCore
2008-04-25 Geoffrey Garen <ggaren@apple.com>
Reviewed by Sam Weinig.
A tiny bit of cleanup to the regexp code.
Removed some static_cast.
Removed createRegExpImp because it's no longer used.
- 11:00 AM SquirrelFish edited by
- (diff)
- 10:02 AM Changeset in webkit [32574] by
-
- 2 edits in trunk/JavaScriptCore
2008-04-25 Darin Adler <Darin Adler>
Done with Stephanie Lewis.
- JavaScriptCore.xcodeproj/project.pbxproj: Prepare for compilation with gcc 4.2 by adding -fno-strict-aliasing to CollatorICU.cpp.
- 7:43 AM Changeset in webkit [32573] by
-
- 2 edits in trunk/WebKit/qt
Benjamin Meyer <bmeyer@trolltech.com>
Fixed focus handling when a node is focused while the corresponding QWebPage does not have the focus.
- Correctly de- and reactivate the focused frame in focusOut/focusInEvent
without telling the focus controller. We don't want to change the focused frame
- the controller has to remember it in fact - but instead just deactivate the frame for correct painting as RenderTheme::isFocused() uses the activation state of the frame.
- 5:37 AM Changeset in webkit [32572] by
-
- 2 edits in trunk/WebCore
Tor Arne Vestbø <tavestbo@trolltech.com>
Respect antialiasing hint when drawing focus rects.
- 5:37 AM Changeset in webkit [32571] by
-
- 2 edits in trunk/WebKit/qt
Kavindra Devi Palaraja <kdpalara@trolltech.com>
completed documentation for the Detailed Description section for QWebView
- 5:37 AM Changeset in webkit [32570] by
-
- 2 edits in trunk/WebKit/qt
Denis Dzyubenko <denis.dzyubenko@trolltech.com>
Fixed the way QWebHistory works - when you call back(), forward() or goToItem() functions it changes the current item in history *and* loads the corresponding page.
- 5:36 AM Changeset in webkit [32569] by
-
- 2 edits in trunk/WebKit/qt
Tor Arne Vestbø <tavestbo@trolltech.com>
Fix resubmit of HTML forms when initially denied by QWebPage::acceptNavigationRequest().
- 3:49 AM Changeset in webkit [32568] by
-
- 4 edits in trunk/WebKit/qt
Simon Hausmann <Simon Hausmann>
When pressing backspace in a line edit in a webpage we should not go back to the previous page.
The shortcut for back on Windows is backspace. Implemented shortcut override
handling in QWebView/QWebPage to prevent this.
- 3:49 AM Changeset in webkit [32567] by
-
- 3 edits in trunk/WebKit/qt
Ariya Hidayat <ariya.hidayat@trolltech.com>
Fix triple-clicking does not work in a web page
- 3:49 AM Changeset in webkit [32566] by
-
- 2 edits in trunk/WebCore
Holger Hans Peter Freyther <zecke@selfish.org>
networking: Use Qt::QueuedConnection in QNetworkReplyHandler
Make sure we leave the networking backend before calling
into WebCore. Otheriwse we might return to the networking
backend when it is already deleted.
- 3:49 AM Changeset in webkit [32565] by
-
- 2 edits in trunk/WebKit/qt
Benjamin Meyer <bmeyer@trolltech.com>
When pressing Ctrl-Up the keyboard modifiers could include other modifiers
- 3:48 AM Changeset in webkit [32564] by
-
- 2 edits in trunk/WebKit/qt
Tor Arne Vestbø <tavestbo@trolltech.com>
Fix handling of Javascript's confirm() function in QtWebKit.
- 3:41 AM Changeset in webkit [32563] by
-
- 2 edits in trunk/WebKit/qt
Kavindra Devi Palaraja <kdpalara@trolltech.com>
Doc - added a screenshot, flowchart, and a snippet to the QWebView documentation to improve clarity
- 3:32 AM Changeset in webkit [32562] by
-
- 4 edits in trunk/WebKit/qt
Benjamin Meyer <bmeyer@trolltech.com>
QWebPage: missing signal when window.print() is requested from javascript
- 3:32 AM Changeset in webkit [32561] by
-
- 2 edits in trunk/WebCore
Tor Arne Vestbø <tavestbo@trolltech.com>
Fix propagation of mouse double click events.
Treat a mouse double click as a regular mouse press with just a different click count.
- 3:32 AM Changeset in webkit [32560] by
-
- 2 edits in trunk/WebKit/qt
Benjamin Meyer <bmeyer@trolltech.com>
Fixes: "Save Image" action wasn't doing anything.
- 3:31 AM Changeset in webkit [32559] by
-
- 3 edits in trunk/WebCore
Tor Arne Vestbø <tavestbo@trolltech.com>
Fixes: Fix strange button sizes (push, radio, and check) in QtWebKit.
We now use pixel metrics instead of fixed sizes for the
button margins, or if the style supports layout rects we
use the delta between the content rect and the layout rect.
In adition we compensate for the difference between the size
for content and the actual layout rect (if supported) by
reporting back the layout rect to WebKit for page layout,
and then inflating the rect we actually paint to.
- 3:31 AM Changeset in webkit [32558] by
-
- 2 edits in trunk/WebKit/qt
Benjamin Meyer <bmeyer@trolltech.com>
Apply key event changes to the current frame, not the main frame.
Example: hitting space bar should scroll current frame, not the main frame
which doesn't even have a scrollbar.
- 3:00 AM Changeset in webkit [32557] by
-
- 2 edits in trunk/WebKit/qt
Benjamin Meyer <bmeyer@trolltech.com>
Fixes: QWebFrame crash when fetching the icon
Just call QWebSettings::iconForUrl to not duplicate code and obey the mutex lock.
- 2:59 AM Changeset in webkit [32556] by
-
- 2 edits in trunk/WebCore
Benjamin Meyer <bmeyer@trolltech.com>
Fixes: Valgrind warnings about uninitilized variables used in jumps
- 2:59 AM Changeset in webkit [32555] by
-
- 4 edits in trunk
Warwick Allison <warwick@trolltech.com>
Fixes: Scrollbars did not report correct maximum.
- 2:59 AM Changeset in webkit [32554] by
-
- 3 edits in trunk/WebCore
Benjamin Meyer <bmeyer@trolltech.com>
Implement NoDrop, ZoomIn, and ZoomOut cursors
- 2:01 AM Changeset in webkit [32553] by
-
- 2 edits in trunk/WebCore
Holger Hans Peter Freyther <zecke@selfish.org>
Handle the ListboxAppearance as 'TextField' as well. We are mainly interested in the background.
RenderBox will call us to paint the outline/background, it is handled the same as a
TextAreaAppearance in RenderTheme, handle it the same in the RenderThemeQt
- 2:01 AM Changeset in webkit [32552] by
-
- 2 edits in trunk/WebCore
Holger Hans Peter Freyther <zecke@selfish.org>
No reason to exit the application when we can not paint something. Replace the Q_ASSERT
with a runtime check. Return true to indicate that we did not handle the painting
of this element.
- 2:01 AM Changeset in webkit [32551] by
-
- 2 edits in trunk/WebCore
Holger Hans Peter Freyther <zecke@selfish.org>
Correct the comment. We are in painTextField and don't paint a button.
- 2:00 AM Changeset in webkit [32550] by
-
- 2 edits in trunk/WebCore
Holger Hans Peter Freyther <zecke@selfish.org>
Allow ListboxAppearance to take focus as well. Stolen from Tor Arne
- 2:00 AM Changeset in webkit [32549] by
-
- 2 edits in trunk/LayoutTests
Holger Hans Peter Freyther <zecke@selfish.org>
Do not execute most of the http tests as they hang or crash.
- 2:00 AM Changeset in webkit [32548] by
-
- 2 edits in trunk/WebCore
Simon Hausmann <Simon Hausmann>
Remove debug output.
- 1:28 AM Changeset in webkit [32547] by
-
- 2 edits in branches/squirrelfish/JavaScriptCore
Bug 18736: SQUIRRELFISH: switch statements with no default have incorrect codegen
<https://bugs.webkit.org/show_bug.cgi?id=18736>
Reviewed by Maciej
Ensure the "default" target is correct in the absence of an explicit default handler.
- 1:10 AM Changeset in webkit [32546] by
-
- 5 edits in trunk/WebKit/qt
David Boddie <dboddie@trolltech.com>
Documentation updates for some of the QWeb classes
- 1:09 AM Changeset in webkit [32545] by
-
- 1 edit in branches/squirrelfish/JavaScriptCore/ChangeLog
Fixing the ChangeLog
- 1:08 AM Changeset in webkit [32544] by
-
- 4 edits in branches/squirrelfish/JavaScriptCore
Bug 18732: SQUIRRELFISH: exceptions thrown by native constructors are ignored
<https://bugs.webkit.org/show_bug.cgi?id=18732>
Reviewed by Maciej
More bounds checking.
- 12:53 AM Changeset in webkit [32543] by
-
- 5 edits in trunk
Holger Hans Peter Freyther <zecke@selfish.org>
Implement dumping of resource load callbacks to pass http/tests/xmlhttprequest/abort-should-cancel-load.html
Similar to Editing and Frameloading we do the dumping within WebCore
- 12:52 AM Changeset in webkit [32542] by
-
- 2 edits in trunk/WebCore
Holger Hans Peter Freyther <zecke@selfish.org>
Change the string to match the mac and pass http/tests/misc/isindex-formdata.html
- 12:52 AM Changeset in webkit [32541] by
-
- 2 edits in trunk/WebCore
Simon Hausmann <Simon Hausmann>
When we encounter a new/unknown HTTP request type report it back to WebCore as loading error.
- 12:51 AM Changeset in webkit [32540] by
-
- 3 edits in branches/squirrelfish/JavaScriptCore
2008-04-25 Maciej Stachowiak <mjs@apple.com>
Reviewed by Oliver.
- fix signal catching magic
The signal handlers are restored to _exit but are only set when
running under run-javascriptcore-tests. fprintf from a signal
handler is not safe.
- kjs/testkjs.cpp: (main): (parseArguments):
- tests/mozilla/jsDriver.pl:
- 12:44 AM Changeset in webkit [32539] by
-
- 2 edits in trunk/WebCore
Benjamin Meyer <bmeyer@trolltech.com>
Fix crash in the networking layer.
Set the m_reply to null right after calling deleteLater().
- 12:27 AM Changeset in webkit [32538] by
-
- 2 edits in branches/squirrelfish/JavaScriptCore
2008-04-25 Cameron Zwarich <cwzwarich@uwaterloo.ca>
Reviewed by Maciej.
Bug 18732: SQUIRRELFISH: exceptions thrown by native constructors are ignored
<https://bugs.webkit.org/show_bug.cgi?id=18732>
Fixes another regression test.
- VM/Machine.cpp: (KJS::Machine::privateExecute):
- 12:24 AM Changeset in webkit [32537] by
-
- 3 edits in branches/squirrelfish/JavaScriptCore
2008-04-25 Cameron Zwarich <cwzwarich@uwaterloo.ca>
Reviewed by Maciej.
Bug 18728: SQUIRRELFISH: invalid regular expression constants should throw exceptions
<https://bugs.webkit.org/show_bug.cgi?id=18728>
Fixes another regression test.
- kjs/nodes.cpp: (KJS::RegExpNode::emitCode):
Apr 24, 2008:
- 11:45 PM Changeset in webkit [32536] by
-
- 3 edits in branches/squirrelfish/JavaScriptCore
2008-04-24 Cameron Zwarich <cwzwarich@uwaterloo.ca>
Reviewed by Geoffrey Garen.
Bug 18735: SQUIRRELFISH: closures are sometimes given an incorrect 'this' value when called
<https://bugs.webkit.org/show_bug.cgi?id=18735>
The overloaded toThisObject method was not copied over to JSActivation.
Fixes two regression tests.
- kjs/JSActivation.cpp: (KJS::JSActivation::toThisObject):
- kjs/JSActivation.h:
- 11:44 PM Changeset in webkit [32535] by
-
- 5 edits in branches/squirrelfish/JavaScriptCore
2008-04-24 Geoffrey Garen <ggaren@apple.com>
Reviewed by Oliver Hunt.
Added support for arguments.callee.
- 11:42 PM Changeset in webkit [32534] by
-
- 2 edits in trunk/WebCore
2008-04-24 Mark Rowe <mrowe@apple.com>
Rubber-stamped by Oliver Hunt.
- WebCore.base.exp: Remove two symbols from the export list that don't need to be exported.
- 10:00 PM Changeset in webkit [32533] by
-
- 6 edits in branches/squirrelfish/JavaScriptCore
Bug 18628: SQUIRRELFISH: need to support recursion limit
<https://bugs.webkit.org/show_bug.cgi?id=18628>
Reviewed by Maciej
Partial fix -- this gets us some of the required bounds checking, but not
complete coverage. But it does manage to do them without regressing :D
- 9:04 PM Changeset in webkit [32532] by
-
- 3 edits2 adds in trunk
WebCore:
Reviewed by Dave Hyatt.
- fix https://bugs.webkit.org/show_bug.cgi?id=18722 <rdar://problem/5888360> REGRESSION (3.1 -> TOT): Webkit Nightly Build crashes when visiting i has a hotdog
Test: fast/dynamic/float-remove-above-line.html
- rendering/bidi.cpp: (WebCore::RenderBlock::layoutInlineChildren): Avoid adding floats that do not intrude into the line to its floats vector. When such floats go away, they do not dirty the line (because they do not intersect with it) and having it keep them in its floats vector is what caused the crash.
LayoutTests:
Reviewed by Dave Hyatt.
- test for https://bugs.webkit.org/show_bug.cgi?id=18722 <rdar://problem/5888360> REGRESSION (3.1 -> TOT): Webkit Nightly Build crashes when visiting i has a hotdog
- fast/dynamic/float-remove-above-line-expected.txt: Added.
- fast/dynamic/float-remove-above-line.html: Added.
- 7:04 PM Changeset in webkit [32531] by
-
- 2 edits in trunk/WebCore
2008-04-24 Justin Garcia <justin.garcia@apple.com>
Reviewed by John Sullivan.
It is possible, despite our safeguards, for createMarkup to iterate past the end of the Range
that is passed to it. Added a null check to prevent crashes in this situation (we won't crash but
we will create too much markup), and added an ASSERT to hopefully catch the scenario in a debugger
and help us understand what's going on.
- editing/markup.cpp: (WebCore::createMarkup):
- 6:29 PM Changeset in webkit [32530] by
-
- 11 edits in trunk
Remove code for calculating the glyph cache size.
Reviewed by Sam Weinig.
WebCore:
- WebCore.base.exp: Remove unused symbol.
- platform/graphics/mac/FontCacheMac.mm: Remove code that is unused now that we do not compute the glyph cache size.
- platform/mac/WebCoreSystemInterface.h: Remove unused symbol.
- platform/mac/WebCoreSystemInterface.mm: Ditto.
WebKit:
- WebCoreSupport/WebSystemInterface.m:
(InitWebCoreSystemInterface): Remove unused symbol.
WebKitSystemInterface:
- WebKitSystemInterface.h: Remove unused symbol.
- libWebKitSystemInterfaceLeopard.a:
- libWebKitSystemInterfaceTiger.a:
- 6:29 PM Changeset in webkit [32529] by
-
- 4 edits in trunk
Add a definition of BUILDING_ON_LEOPARD to complement BUILDING_ON_TIGER.
Reviewed by Sam Weinig.
- 5:58 PM Changeset in webkit [32528] by
-
- 3 edits3 adds in trunk
2008-04-24 Cameron McCormack <cam@mcc.id.au>
Reviewed by Maciej Stachowiak.
Ensure that changes to rx="" and ry="" attributes on an SVG <rect>
element cause a relayout.
Test: svg/dom/SVGRectElement/rect-modify-rx.svg
- svg/SVGRectElement.cpp: (WebCore::SVGRectElement::svgAttributeChanged): Call needsLayout() when the attribute is rx or ry. (WebCore::SVGRectElement::hasRelativeValues): Return true when the attribute is rx or ry, since they also can have relative values.
2008-04-24 Cameron McCormack <cam@mcc.id.au>
Reviewed by Maciej Stachowiak.
Test for:
http://bugs.webkit.org/show_bug.cgi?id=18328
Modifying rx="" or ry="" on a <rect> doesn't relayout
- svg/dom/SVGRectElement: Added.
- svg/dom/SVGRectElement/rect-modify-rx-expected.txt: Added.
- svg/dom/SVGRectElement/rect-modify-rx.svg: Added.
- 5:50 PM Changeset in webkit [32527] by
-
- 2 edits in trunk/WebKitTools
2008-04-24 Jan Michael Alonzo <jmalonzo@unpluggable.com>
Reviewed by Maciej Stachowiak.
http://bugs.webkit.org/show_bug.cgi?id=18485
Typo and documentation fix for build-webkit
- Scripts/build-webkit:
- 5:47 PM Changeset in webkit [32526] by
-
- 5 edits in trunk
2008-04-24 Adam Barth <hk9565@gmail.com>
Reviewed by Sam Weinig.
Update the XMLHttpRequest header black list to match the spec.
- xml/XMLHttpRequest.cpp: (WebCore::isSafeRequestHeader):
2008-04-24 Adam Barth <hk9565@gmail.com>
Reviewed by Sam Weinig.
Test that we block headers beginning with "Sec-" as per spec.
- http/tests/xmlhttprequest/set-dangerous-headers-expected.txt:
- http/tests/xmlhttprequest/set-dangerous-headers.html:
- 5:47 PM Changeset in webkit [32525] by
-
- 3 edits2 adds in trunk
2008-04-22 Feng Qian <ian.eng.webkit@gmail.com>
Reviewed by Geoff Garen.
Fix https://bugs.webkit.org/show_bug.cgi?id=18677
Bug 18677: Synchronous event dispatch confuses the popup blocker into incorrectly blocking chat popups in gmail.com.
Test: fast/events/nested-window-event.html
- bindings/js/kjs_events.cpp: (WebCore::JSAbstractEventListener::handleEvent): Save and restore the current event around the event dispatch.
2008-04-22 Feng Qian <ian.eng.webkit@gmail.com>
Reviewed by Geoff Garen.
Test for nested event handlers.
- fast/events/nested-window-event-expected.txt: Added.
- fast/events/nested-window-event.html: Added.
- 5:26 PM Changeset in webkit [32524] by
-
- 6 edits in trunk/WebCore
2008-04-24 Brady Eidson <beidson@apple.com>
Reviewed by Teh Mitzpettel
Make it so PageGroups know what their own name is.
- WebCore.base.exp:
- page/Page.cpp: (WebCore::Page::setGroupName): (WebCore::Page::groupName):
- page/Page.h:
- page/PageGroup.cpp: Move the PageGroupMap into the PageGroup class. (WebCore::PageGroup::PageGroup): Added a c'tor that takes the group name as an argument instead of the initial Page. (WebCore::PageGroup::pageGroup): Handles creating and mapping names to groups
- page/PageGroup.h: (WebCore::PageGroup::name):
- 4:20 PM Changeset in webkit [32523] by
-
- 4 edits in trunk/WebKit
WebKit/gtk:
- build fix
- webkit/webkitwebview.cpp:
WebKit/win:
- build fix
- WebView.cpp: (WebView::handleContextMenuEvent):
- 4:18 PM Changeset in webkit [32522] by
-
- 1 edit in branches/squirrelfish/JavaScriptCore/ChangeLog
Fixed up ChangeLog
- 4:18 PM Changeset in webkit [32521] by
-
- 4 edits in branches/squirrelfish/JavaScriptCore
2008-04-24 Cameron Zwarich <cwzwarich@uwaterloo.ca>
Reviewed by Oliver.
Bug 18717: SQUIRRELFISH: eval returns the wrong value for a variable declaration statement
<https://bugs.webkit.org/show_bug.cgi?id=18717>
Fixes a regression test, but exposes the failure of another due to the
lack of getters and setters.
- kjs/nodes.cpp: (KJS::ConstDeclNode::emitCodeSingle): (KJS::ConstDeclNode::emitCode): (KJS::ConstStatementNode::emitCode): (KJS::VarStatementNode::emitCode):
- kjs/nodes.h:
- 3:58 PM Changeset in webkit [32520] by
-
- 3 edits in branches/squirrelfish/JavaScriptCore
Cameron Zwarich <cwzwarich@uwaterloo.ca>
Reviewed by Oliver.
Bug 18717: SQUIRRELFISH: eval returns the wrong value for a variable declaration statement
<https://bugs.webkit.org/show_bug.cgi?id=18717>
Fixes a regression test, but exposes the failure of another due to the
lack of getters and setters.
- 3:56 PM Changeset in webkit [32519] by
-
- 5 edits in trunk/WebCore
2008-04-24 Adele Peterson <adele@apple.com>
Reviewed by Darin.
Consolidate two versions of pathGetFileName.
- html/HTMLFormElement.cpp: (WebCore::HTMLFormElement::formData): Call FileSystem.h version of pathGetFileName and removed the static helper function.
- platform/posix/FileSystemPOSIX.cpp: (WebCore::pathGetFileName): Moved general case from HTMLFormElement version here. If other platforms relied on that default they should implement this function for their platform specific version of FileSystem.cpp
- platform/win/FileSystemWin.cpp: (WebCore::pathGetFileName): Moved win implementation from HTMLFormElement here.
- platform/wx/FileSystemWx.cpp: (WebCore::pathGetFileName): Moved wx implementation from HTMLFormElement here.
- 2:49 PM Changeset in webkit [32518] by
-
- 2 edits in trunk/WebKit/mac
2008-04-24 Brady Eidson <beidson@apple.com>
Reviewed by Darin
Fix layout test regressions from my earlier preferences/settings tweak.
- WebView/WebView.mm: (-[WebView _commonInitializationWithFrameName:groupName:]): Even if we're not posting the notification to update the settings, each WebView still needs to register for the notification - restore that behavior.
- 2:45 PM Changeset in webkit [32517] by
-
- 2 edits in trunk/WebCore
2008-04-24 Anders Carlsson <andersca@apple.com>
Windows build fix.
- html/HTMLFormElement.cpp: (WebCore::pathGetFilename):
- 12:43 PM Changeset in webkit [32516] by
-
- 1 add in trunk/WebKitSite/blog-files/kate-circle.png
Add svg mask example.
- 12:42 PM Changeset in webkit [32515] by
-
- 1 add in trunk/WebKitSite/blog-files/circle.svg
Add svg file of a circle.
- 12:26 PM Changeset in webkit [32514] by
-
- 1 add in trunk/WebKitSite/blog-files/kate-gradient-rounded.png
Add more files.
- 12:21 PM Changeset in webkit [32513] by
-
- 1 add in trunk/WebKitSite/blog-files/kate-gradient.png
Add kate gradient pic.
- 12:16 PM Changeset in webkit [32512] by
-
- 2 edits in trunk/WebKitTools
2008-04-24 Anders Carlsson <andersca@apple.com>
Reviewed by Sam.
Don't call fprintf from the signal handler.
- DumpRenderTree/mac/DumpRenderTree.mm: (crashHandler):
- 12:16 PM Changeset in webkit [32511] by
-
- 1 edit in trunk/WebKitSite/blog-files/kate-vignette-mask.png
Adjust files.
- 12:15 PM Changeset in webkit [32510] by
-
- 2 edits in trunk/WebCore
2008-04-24 Anders Carlsson <andersca@apple.com>
Don't crash when the string is empty.
- html/HTMLMediaElement.cpp: (WebCore::parseTimeOffset):
- 12:12 PM Changeset in webkit [32509] by
-
- 1 add in trunk/WebKitSite/blog-files/kate-vignette-mask.png
Add more files.
- 12:09 PM Changeset in webkit [32508] by
-
- 38 edits2 adds in trunk
WebCore:
Reviewed by Darin Adler.
- preparation for https://bugs.webkit.org/show_bug.cgi?id=3729 <rdar://problem/4036353> REGRESSION: arrow keys move insertion bar backwards in RTL text
The three main changes in this patch are:
1) Making all inline boxes know their bidi level, instead of just text
boxes knowing whether their bidi level is odd or even. This is
required for the next change.
2) Replacing RenderObject::inlineBox() with
Position::getInlineBoxAndOffset() in recognition of the fact that the
inline box containing the primary caret for a position in a given
node may belong to a different node's renderer.
3) Changing RenderObject::caretRect() to take an InlineBox parameter,
and changing callers to call VisiblePosition::caretRect(), which
locates the inline box, then calls caretRect() on the renderer for
that box. This, combined with the previous change, ensures that the
primary caret is rendered at the right place for positions that
lie on a directionality boundary.
Test: platform/mac/editing/input/caret-primary-bidi.html
- WebCore.base.exp: Added the VisiblePosition(Node*, int, EAffinity) constructor and VisiblePosition::caretRect(), and sorted.
- dom/Position.cpp: (WebCore::nextRenderedEditable): Adjusted for the removal of RenderObject::inlineBox(). (WebCore::previousRenderedEditable): Ditto. (WebCore::Position::rendersInDifferentPosition): Ditto. (WebCore::Position::getInlineBoxAndOffset): Added. Gets the inline box and the offset within that box at which the primary caret for this position should render.
- dom/Position.h:
- editing/DeleteSelectionCommand.cpp: (WebCore::DeleteSelectionCommand::mergeParagraphs): Changed to call VisiblePosition::caretRect() instead of calling the RenderObject method.
- editing/SelectionController.cpp: (WebCore::caretY): Ditto. (WebCore::SelectionController::xPosForVerticalArrowNavigation): Ditto. (WebCore::SelectionController::layout): Ditto.
- editing/VisiblePosition.cpp: (WebCore::VisiblePosition::caretRect): Changed to call getInlineBoxAndOffset() to get the correct inline box and call the renderer for that box.
- editing/VisiblePosition.h: (WebCore::VisiblePosition::getInlineBoxAndOffset): Added convenience methods for getting the inline box and caret offset for a visible position, accounting for its affinity.
- editing/visible_units.cpp: (WebCore::rootBoxForLine): Changed to use getInlineBoxAndOffset() instead of RenderObject::inlineBox(). (WebCore::startPositionForLine): (WebCore::endPositionForLine): (WebCore::previousLinePosition): Ditto. (WebCore::nextLinePosition): Ditto.
- page/AccessibilityObject.cpp: (WebCore::updateAXLineStartForVisiblePosition): Ditto.
- page/Frame.cpp: (WebCore::Frame::firstRectForRange): Ditto.
- rendering/InlineBox.cpp: (WebCore::InlineBox::caretMinOffset): Changed to forward to the renderer. (WebCore::InlineBox::caretMaxOffset): Ditto.
- rendering/InlineBox.h: Replaced the m_reversed bit, intended for use in InlineTextBox only, with six bits of the bidi level of the box, intended for use in all leaf inline boxes. (WebCore::InlineBox::InlineBox): Added missing initializer for m_dirOverride and initialized the bidi level. (WebCore::InlineBox::bidiLevel): Added this accessor. (WebCore::InlineBox::setBidiLevel): Ditto. (WebCore::InlineBox::direction): Ditto. (WebCore::InlineBox::caretLeftmostOffset): Added this convenience method. (WebCore::InlineBox::caretRightmostOffset): Ditto.
- rendering/InlineTextBox.cpp: Replaced all references to m_reversed with checking of direction(). (WebCore::InlineTextBox::selectionRect): (WebCore::InlineTextBox::placeEllipsisBox): (WebCore::InlineTextBox::paint): (WebCore::InlineTextBox::paintSelection): (WebCore::InlineTextBox::paintCompositionBackground): (WebCore::InlineTextBox::paintSpellingOrGrammarMarker): (WebCore::InlineTextBox::paintTextMatchMarker): (WebCore::InlineTextBox::textPos): (WebCore::InlineTextBox::offsetForPosition): (WebCore::InlineTextBox::positionForOffset):
- rendering/RenderBR.cpp: Removed inlineBox().
- rendering/RenderBR.h: Ditto.
- rendering/RenderBox.cpp: (WebCore::RenderBox::caretRect): Changed to take an inline box and account for the direction of the box (or the renderer) in positioning the caret: in right-to-left boxes, the "before" position is to the right while "after" is to the left.
- rendering/RenderBox.h:
- rendering/RenderFlow.cpp: (WebCore::RenderFlow::caretRect): Updated the signature.
- rendering/RenderFlow.h:
- rendering/RenderObject.cpp: (WebCore::RenderObject::caretRect): Updated the signature. (WebCore::RenderObject::caretMaxOffset): Changed to return the child node count (or 1 if there are no children) for replaced elements, such as <select>s.
- rendering/RenderObject.h:
- rendering/RenderReplaced.cpp: Removed caretMinOffset() and caretMaxOffset() because the base class implementation does the right thing for replaced objects now.
- rendering/RenderReplaced.h:
- rendering/RenderSVGInlineText.cpp: (WebCore::RenderSVGInlineText::caretRect): Updated the signature. (WebCore::RenderSVGInlineText::positionForCoordinates): Updated for the change from m_reversed to direction().
- rendering/RenderSVGInlineText.h:
- rendering/RenderText.cpp: (WebCore::RenderText::caretRect): Changed to take an inline box and removed the code that used to find the inline for the given position. Changed use of m_reversed to use direction(). (WebCore::RenderText::position): Changed use of m_reversed to use direction().
- rendering/RenderText.h:
- rendering/RenderTextControl.cpp: (WebCore::RenderTextControl::textWithHardLineBreaks): Adjusted for the removal of RenderObject::inlineBox().
- rendering/RenderTreeAsText.cpp: (WebCore::writeTextRun): Changed to use direction() instead of m_reversed.
- rendering/SVGInlineTextBox.cpp: Ditto. (WebCore::SVGInlineTextBox::calculateGlyphBoundaries): (WebCore::SVGInlineTextBoxClosestCharacterToPositionWalker::chunkPortionCallback): (WebCore::SVGInlineTextBox::svgCharacterHitsPosition):
- rendering/SVGRenderTreeAsText.cpp: Ditto. (WebCore::writeSVGInlineTextBox):
- rendering/SVGRootInlineBox.cpp: Ditto. (WebCore::svgTextRunForInlineTextBox): (WebCore::cummulatedWidthOrHeightOfTextChunk): (WebCore::SVGRootInlineBox::buildLayoutInformationForTextBox):
- rendering/bidi.cpp: (WebCore::RenderBlock::constructLine): Made this function set the bidi level on all leaf boxes.
- svg/SVGTextContentElement.cpp: Changed to use direction() instead of m_reversed. (WebCore::cumulativeCharacterRangeLength): (WebCore::SVGInlineTextBoxQueryWalker::chunkPortionCallback):
WebKit/mac:
Reviewed by Darin Adler.
- preparation for https://bugs.webkit.org/show_bug.cgi?id=3729 <rdar://problem/4036353> REGRESSION: arrow keys move insertion bar backwards in RTL text
- WebView/WebFrame.mm: (-[WebFrame _caretRectAtNode:offset:affinity:]): Changed to use VisiblePosition::caretRect() instead of the RenderObject method which was removed.
LayoutTests:
Reviewed by Darin Adler.
- test the visual position of the primary caret in bidirectional text
- platform/mac/editing/input/caret-primary-bidi-expected.txt: Added.
- platform/mac/editing/input/caret-primary-bidi.html: Added.
- 12:09 PM Changeset in webkit [32507] by
-
- 1 add in trunk/WebKitSite/blog-files/kate.png
Check in kate image for blog post.
- 12:00 PM Changeset in webkit [32506] by
-
- 1 add in trunk/WebKitSite/blog-files/vignette-mask.png
Add blog files for mask post.
- 11:57 AM Changeset in webkit [32505] by
-
- 4 edits in trunk/WebKit
../mac:
2008-04-24 Brady Eidson <beidson@apple.com>
Reviewed by Darin
Rework the Settings population again.
- WebView/WebView.mm: (-[WebView _updateSettingsFromPreferences:]): This method is called both from _preferencesChangedNotification and directly from WebView's common init function. (-[WebView _preferencesChangedNotification:]): (-[WebView _commonInitializationWithFrameName:groupName:]): Call _updateSettingsFromPreferences immediately after creating the new Page
../win:
2008-04-24 Brady Eidson <beidson@apple.com>
Reviewed by Darin
Revert my previous change to WebKit/Win until I can make it act more like the new Mac code
- WebView.cpp: (WebView::initWithFrame):
- 11:53 AM Changeset in webkit [32504] by
-
- 2 edits in trunk/WebCore
2008-04-24 Sam Weinig <sam@webkit.org>
Fix the world.
- bindings/js/kjs_proxy.cpp: (WebCore::KJSProxy::clear):
- 11:51 AM Changeset in webkit [32503] by
-
- 1 add in trunk/WebKitSite/blog-files/maskspeedracer.png
Add blog images.
- 11:45 AM Changeset in webkit [32502] by
-
- 2 edits in trunk/JavaScriptCore
2008-04-24 Sam Weinig <sam@webkit.org>
Reviewed by Geoffrey Garen.
Add a #define to easily enable collecting on every allocation to aid
debugging GC bugs.
- kjs/collector.cpp: (KJS::Collector::heapAllocate):
- 11:22 AM Changeset in webkit [32501] by
-
- 2 edits in branches/squirrelfish/JavaScriptCore
2008-04-24 Geoffrey Garen <ggaren@apple.com>
Reviewed by Sam Weinig.
Print a CRASH statement when crashing, so test failures are not a
mystery.
- kjs/testkjs.cpp: (handleCrash): (main):
- 10:54 AM Changeset in webkit [32500] by
-
- 4 edits in trunk
WebCore:
2008-04-24 Darin Adler <Darin Adler>
Reviewed by Geoff.
- fix crash in regression test where we'd ask a frame for a user agent string after the WebView was already closed
- loader/FrameLoader.cpp: (WebCore::FrameLoader::scheduleHTTPRedirection): Quietly do nothing if this is called on a frame that's already detached from its page. (WebCore::FrameLoader::scheduleLocationChange): Ditto. (WebCore::FrameLoader::scheduleRefresh): Ditto. (WebCore::FrameLoader::scheduleHistoryNavigation): Ditto. (WebCore::FrameLoader::redirectionTimerFired): Assert that this is never called for a frame that's already detached from its page. (WebCore::FrameLoader::scheduleRedirection): Ditto. (WebCore::FrameLoader::startRedirectionTimer): Ditto.
WebKit/mac:
2008-04-24 Darin Adler <Darin Adler>
Reviewed by Geoff.
- fix crash in regression test where we'd ask a frame for a user agent string after the WebView was already closed
- WebCoreSupport/WebFrameLoaderClient.mm: (WebFrameLoaderClient::userAgent): Assert that the WebView is not nil. Also added some code to prevent the crash in release builds if this problem happens again.
- 10:50 AM Changeset in webkit [32499] by
-
- 3 edits in trunk/WebCore
Add a new callFunction helper function to remove duplicate code
for calling JavaScript functions with arguments.
Reviewed by Darin Adler.
- page/InspectorController.cpp:
(WebCore::InspectorController::callSimpleFunction): Call callFunction.
(WebCore::InspectorController::callFunction): Extended version of the original
callSimpleFunction, with script function arguments and an exception argument.
(WebCore::InspectorController::focusNode): Use callFunction.
(WebCore::InspectorController::addScriptResource): Ditto.
(WebCore::InspectorController::removeScriptResource): Ditto.
(WebCore::InspectorController::addDatabaseScriptResource): Ditto.
(WebCore::InspectorController::removeDatabaseScriptResource): Ditto.
(WebCore::InspectorController::addScriptConsoleMessage): Ditto.
- page/InspectorController.h: Add callFunction.
- 10:45 AM Changeset in webkit [32498] by
-
- 30 edits in trunk
WebCore:
2008-04-24 Anders Carlsson <andersca@apple.com>
Reviewed by Sam.
Change some String arguments to be const references instead.
- css/CSSRule.cpp: (WebCore::CSSRule::setCssText):
- css/CSSRule.h:
- html/HTMLFormElement.cpp: (WebCore::pathGetFilename):
- html/HTMLMediaElement.cpp: (WebCore::parseTimeOffset):
- page/EditorClient.h:
- platform/SecurityOrigin.cpp: (WebCore::isDefaultPortForProtocol):
- platform/graphics/MediaPlayer.cpp: (WebCore::MediaPlayer::load):
- platform/graphics/MediaPlayer.h:
- rendering/MediaControlElements.cpp: (WebCore::MediaControlInputElement::MediaControlInputElement):
- rendering/MediaControlElements.h:
- svg/animation/SVGSMILElement.cpp: (WebCore::SVGSMILElement::Condition::Condition):
- svg/animation/SVGSMILElement.h:
- svg/graphics/SVGImageEmptyClients.h: (WebCore::SVGEmptyEditorClient::shouldInsertText):
- xml/XSLTProcessor.cpp: (WebCore::createFragmentFromSource):
WebKit/gtk:
2008-04-24 Anders Carlsson <andersca@apple.com>
Reviewed by Sam.
Change some String arguments to be const references instead.
- WebCoreSupport/EditorClientGtk.cpp: (WebKit::EditorClient::shouldInsertText):
- WebCoreSupport/EditorClientGtk.h:
WebKit/mac:
2008-04-24 Anders Carlsson <andersca@apple.com>
Reviewed by Sam.
Change some String arguments to be const references instead.
- WebCoreSupport/WebEditorClient.h:
- WebCoreSupport/WebEditorClient.mm: (WebEditorClient::shouldInsertText):
WebKit/qt:
2008-04-24 Anders Carlsson <andersca@apple.com>
Reviewed by Sam.
Change some String arguments to be const references instead.
- WebCoreSupport/EditorClientQt.cpp: (WebCore::EditorClientQt::shouldInsertText):
- WebCoreSupport/EditorClientQt.h:
WebKit/win:
2008-04-24 Anders Carlsson <andersca@apple.com>
Reviewed by Sam.
Change some String arguments to be const references instead.
- WebCoreSupport/WebEditorClient.cpp: (WebEditorClient::shouldInsertText):
- WebCoreSupport/WebEditorClient.h:
WebKit/wx:
2008-04-24 Anders Carlsson <andersca@apple.com>
Reviewed by Sam.
Change some String arguments to be const references instead.
- WebKitSupport/EditorClientWx.cpp: (WebCore::EditorClientWx::shouldInsertText):
- WebKitSupport/EditorClientWx.h:
- 10:41 AM Changeset in webkit [32497] by
-
- 3 edits2 adds in trunk
WebCore:
2008-04-24 Justin Garcia <justin.garcia@apple.com>
Reviewed by Darin Adler.
<rdar://problem/5604313> FormatBlock to apply an h1 also inserts a style span around the first letter (16004)
- editing/TextIterator.cpp: (WebCore::TextIterator::advance): We emit an extra newline when leaving a block element that has atypical margin/padding. When we emit the extra newline, it needs to be positioned after that block's contents, not after the block. This is how we position the first newline, too, since the range for emitted newlines should start where the line break begins visually.
LayoutTests:
2008-04-24 Justin Garcia <justin.garcia@apple.com>
Reviewed by Darin Adler.
<rdar://problem/5604313> FormatBlock leaves first letter incorrectly styled (16004)
- editing/execCommand/5604313-expected.txt: Added.
- editing/execCommand/5604313.html: Added.
- 10:25 AM Changeset in webkit [32496] by
-
- 2 edits in trunk/WebCore
2008-04-24 Anders Carlsson <andersca@apple.com>
Reviewed by Adam.
Only avoid implicit entries when doing the initial caching.
- loader/appcache/ApplicationCacheGroup.cpp: (WebCore::ApplicationCacheGroup::didReceiveResponse): (WebCore::ApplicationCacheGroup::startLoadingEntry):
- 10:20 AM Changeset in webkit [32495] by
-
- 11 edits in trunk
2008-04-24 Kevin McCullough <kmccullough@apple.com>
Reviewed by Adam and Sam.
-<rdar://problem/5770054> JavaScript profiler (10928)
-Only profile the page group that starts profiling to avoid profiling
tools that shouldn't be profiled unless explicitly requested to.
- JavaScriptCore.exp: Export new signature.
- kjs/JSGlobalObject.cpp: Add unique identifiers to the JSGlobalObject. (KJS::JSGlobalObject::init):
- kjs/JSGlobalObject.h: Ditto. (KJS::JSGlobalObject::setPageGroupIdentifier): (KJS::JSGlobalObject::pageGroupIdentifier):
- profiler/Profiler.cpp: Check the identifier of the page group of the lexical global exec state and only profile if it matches the given page group identifier. (KJS::Profiler::startProfiling): (KJS::Profiler::willExecute): (KJS::Profiler::didExecute):
- profiler/Profiler.h: Ditto. (KJS::Profiler::Profiler):
WebCore:
2008-04-24 Kevin McCullough <kmccullough@apple.com>
Reviewed by Adam and Sam.
-<rdar://problem/5770054> JavaScript profiler (10928)
-Only profile the page group that starts profiling to avoid profiling
tools that shouldn't be profiled unless explicitly requested to.
- bindings/js/kjs_proxy.cpp: When a new global object is created set its page group identifier. (WebCore::KJSProxy::clear): (WebCore::KJSProxy::initScript):
- page/Console.cpp: When calling console.profile set the identifier of the page group being profiled. (WebCore::Console::profile):
- page/PageGroup.cpp: Implement unique identifiers. (WebCore::getUniqueIdentifier): (WebCore::PageGroup::PageGroup):
- page/PageGroup.h: Ditto. (WebCore::PageGroup::identifier):
- 10:11 AM Changeset in webkit [32494] by
-
- 2 edits in branches/squirrelfish/JavaScriptCore
2008-04-24 Cameron Zwarich <cwzwarich@uwaterloo.ca>
Reviewed by Geoffrey Garen.
Bug 18716: SQUIRRELFISH: typeof should return undefined for an undefined variable reference
<https://bugs.webkit.org/show_bug.cgi?id=18716>
This fixes 2 more regression tests.
- kjs/nodes.cpp: (KJS::TypeOfResolveNode::emitCode):
- 10:09 AM SquirrelFish edited by
- (diff)
- 10:05 AM Changeset in webkit [32493] by
-
- 4 edits in branches/squirrelfish/JavaScriptCore
2008-04-24 Geoffrey Garen <ggaren@apple.com>
Reviewed by Sam Weinig.
Put the callee in the call frame.
Necessary in order to support "arguments" and "arguments.callee".
Also fixes a latent GC bug, where an executing function could be
subject to GC if the register holding it were overwritten. Here's
an example that would have caused problems:
function f()
{
Flood the machine stack to eliminate any old pointers to f.
g.call({});
Overwrite f in the register file.
f = 1;
Force a GC.
for (var i = 0; i < 5000; ++i) {
({});
}
Welcome to crash-ville.
}
function g()
{
}
f();
- VM/Machine.h: Changed the order of arguments to execute(FunctionBodyNode*...) to match the other execute functions.
- kjs/function.cpp: Updated to match new argument requirements from execute(FunctionBodyNode*...). Renamed newObj to thisObj to match the rest of JavaScriptCore.
SunSpider reports no change.
- 9:55 AM Changeset in webkit [32492] by
-
- 5 edits in trunk
WebCore:
2008-04-24 John Sullivan <sullivan@apple.com>
Mac buid fix
- page/mac/WebCoreViewFactory.h: add declarations of six new AX methods that were added to LocalizedStrings.h
- platform/mac/LocalizedStringsMac.mm: (WebCore::AXButtonActionVerb): implement by calling through to WebCoreViewFactory as we do with the other functions here (WebCore::AXRadioButtonActionVerb): ditto (WebCore::AXTextFieldActionVerb): ditto (WebCore::AXCheckedCheckBoxActionVerb): ditto (WebCore::AXUncheckedCheckBoxActionVerb): ditto (WebCore::AXLinkActionVerb): ditto
WebKit/mac:
2008-04-24 John Sullivan <sullivan@apple.com>
Mac build fix
- WebCoreSupport/WebViewFactory.mm: (-[WebViewFactory AXButtonActionVerb]): implement this method using the text in WebCoreLocalizedStrings.cpp (-[WebViewFactory AXRadioButtonActionVerb]): ditto (-[WebViewFactory AXTextFieldActionVerb]): ditto (-[WebViewFactory AXCheckedCheckBoxActionVerb]): ditto (-[WebViewFactory AXUncheckedCheckBoxActionVerb]): ditto (-[WebViewFactory AXLinkActionVerb]): ditto
- 9:33 AM Changeset in webkit [32491] by
-
- 4 edits in trunk/WebKit
../mac:
2008-04-23 Brady Eidson <beidson@apple.com>
Reviewed by Sam Weinig
In some current work I noticed that when a new Page is created, it is possible that it requires info from its Settings
object before the Settings object is initialized. It seems quite prudent to post the preferences changed notification,
thereby populating the Settings object, immediately after the Page is created.
- WebView/WebView.mm: (-[WebView _commonInitializationWithFrameName:groupName:]): Post the notification right after the Page is created
../win:
2008-04-23 Brady Eidson <beidson@apple.com>
Reviewed by Sam Weinig
In some current work I noticed that when a new Page is created, it is possible that it requires info from its Settings
object before the Settings object is initialized. It seems quite prudent to post the preferences changed notification,
thereby populating the Settings object, immediately after the Page is created.
- WebView.cpp: (WebView::initWithFrame): Post the notification right after the Page is created
- 8:23 AM Changeset in webkit [32490] by
-
- 2 edits in trunk/WebKit/mac
2008-04-24 John Sullivan <sullivan@apple.com>
Reviewed by Jess
- fixed <rdar://problem/5886655> JavaScript input panel automatic resizing doesn't work right with HiDPI
- Misc/WebNSControlExtras.m: (-[NSControl sizeToFitAndAdjustWindowHeight]): deploy userSpaceScaleFactor when using view distances on the window
- 7:07 AM Changeset in webkit [32489] by
-
- 2 edits in trunk/WebKit/qt
Holger Hans Peter Freyther <zecke@selfish.org>
Cosmetic changes to make the code more readable.
-Early exit if we don't have a webview
-handle the empty tooltip and non-empty tooltip case separately
- 7:07 AM Changeset in webkit [32488] by
-
- 2 edits in trunk/WebCore
Fix text rendering in -reverse mode on Qt/Mac.
For Font::width() don't use a plain QFontMetrics object but also the properly
setup QTextLayout that has the RTL/LTR force flags set.
- 7:06 AM Changeset in webkit [32487] by
-
- 2 edits in trunk/WebCore
Tor Arne Vestbø <tavestbo@trolltech.com>
Don't do anti-aliasing for arcs, lines and rects, but continue to anti-alias
all drawing primitives in the HTML Canvas element.
This speeds up drawing of dashed outlines significantly.
Also preserve the anti-aliasing renderhint for transparency layers and
get rid of the mostly unused device pointer.
- 6:45 AM Changeset in webkit [32486] by
-
- 4 edits in trunk/WebCore
Paul Olav Tvete <paul@trolltech.com>
Fix various compiler warnings related to QString(const char *)
construction by using QLatin1String.
- 6:44 AM Changeset in webkit [32485] by
-
- 2 edits in trunk/WebKit/qt
Paul Olav Tvete <paul@trolltech.com>
Automatically wrap tooltip text and hide a shown tooltip when it is empty.
QWidget::setTooltip("") will still show the old tooltip for up to 10 seconds.
Workaround as discussed with Matthias.
- 6:30 AM Changeset in webkit [32484] by
-
- 2 edits in trunk/WebCore
Morten Johan Sørvig <msorvig@trolltech.com>
Work around QMacStyle bug in repainting the vertical scrollbar.
We draw the scrollbar using HITheme but somehow the translation and the
clipping we previously set using ClipCGContextToRegion is ignored. Drawing
without the translation fixes the clip.
- 4:20 AM Changeset in webkit [32483] by
-
- 2 edits in trunk/WebKit/qt
Holger Hans Peter Freyther <zecke@selfish.org>
Allow to disable caching completeley by calling setObjectCacheCapacities(0, 0, 0)
- 4:20 AM Changeset in webkit [32482] by
-
- 3 edits in trunk/WebCore
Holger Hans Peter Freyther <zecke@selfish.org>
- Set an expiration date for local files as we do not know what will happen to these files (in contrast to a http server with max-age and other headers)
- Remember the time when we started the job and use that for the expiration date
- QNetworkManager does not handle expiration at all, we might want to set the expiration for all downloads but this will be a performance nightmare.
- 4:16 AM Changeset in webkit [32481] by
-
- 3 edits in trunk/WebCore
Fix the Gtk and Qt builds.
Added missing localization stubs for accessibility.
- 3:35 AM Changeset in webkit [32480] by
-
- 2 edits in trunk/WebKit/qt
Benjamin Meyer <bmeyer@trolltech.com>
Improve keyboard scrolling
Match Down/Up keys scroll distance with Safari (i.e. faster) and add Home and End shortcuts to scroll to the top/botom.
- 3:20 AM Changeset in webkit [32479] by
-
- 9 edits in trunk
Olivier Goffart <ogoffart@trolltech.com>
Fix various compiler warnings in the Qt port.
- 3:20 AM Changeset in webkit [32478] by
-
- 2 edits in trunk/WebCore
Andre Poenitz <andre.poenitz@trolltech.com>
Removed spurious QHideEvent forward declaration.
- 2:01 AM Changeset in webkit [32477] by
-
- 2 edits1 add in trunk/JavaScriptCore
2008-04-24 Julien Chaffraix <jchaffraix@webkit.org>
Reviewed by Simon.
Bug 15940: Implement threading API for Qt
https://bugs.webkit.org/show_bug.cgi?id=15940
Original patch by Justin Haygood, tweaked by me.
- JavaScriptCore.pri:
- wtf/ThreadingQt.cpp: Added. (WTF::threadMapMutex): (WTF::threadMap): (WTF::establishIdentifierForThread): (WTF::clearThreadForIdentifier): (WTF::threadForIdentifier): (WTF::initializeThreading): (WTF::ThreadPrivate::getReturnValue): (WTF::ThreadPrivate::ThreadPrivate): (WTF::ThreadPrivate::run): (WTF::createThread): (WTF::waitForThreadCompletion): return !res to return 0 on success (to match the pthreads implementation). (WTF::detachThread): (WTF::identifierByQthreadHandle): (WTF::currentThread): (WTF::Mutex::Mutex): (WTF::Mutex::~Mutex): (WTF::Mutex::lock): (WTF::Mutex::tryLock): (WTF::Mutex::unlock): (WTF::ThreadCondition::ThreadCondition): (WTF::ThreadCondition::~ThreadCondition): (WTF::ThreadCondition::wait): (WTF::ThreadCondition::timedWait): (WTF::ThreadCondition::signal):
- 1:53 AM Changeset in webkit [32476] by
-
- 3 edits in trunk/WebCore
Tor Arne Vestbø <tavestbo@trolltech.com>
Render text areas using Qt (ensures proper style).
- 1:53 AM Changeset in webkit [32475] by
-
- 2 edits in trunk/WebCore
Tor Arne Vestbø <tavestbo@trolltech.com>
Make mouse wheel events over scrollbars behave the same as in Qt, i.e. both
horizontal and vertical wheel events will scroll the view in the orientation
of the scrollbar underneath the cursor.
- 1:25 AM Changeset in webkit [32474] by
-
- 5 edits in trunk
2008-04-23 Jon Honeycutt <jhoneycutt@apple.com>
Reviewed by Brady.
Implement accDoDefaultAction().
- AccessibleBase.cpp: (AccessibleBase::accDoDefaultAction): Call the object's performDefaultAction() method. Return S_FALSE if the call returns false, indicating that the object has no default action associated with it.
- 1:24 AM Changeset in webkit [32473] by
-
- 2 edits in trunk/WebCore
2008-04-23 Jon Honeycutt <jhoneycutt@apple.com>
Reviewed by Adam.
Don't allow a hit test to return an AccessibilityObject that should be
ignored.
- page/AccessibilityObject.cpp: (WebCore::AccessibilityObject::doAccessibilityHitTest): If this result object's accessibility is ignored, return the parent unignored object.
- 1:24 AM Changeset in webkit [32472] by
-
- 2 edits in trunk/WebCore
2008-04-23 Jon Honeycutt <jhoneycutt@apple.com>
Reviewed by Adam.
Fix a crash when performing a hit test on an AccessibilityObject with
no layer (e.g., an AccessibilityObject for a RenderText object).
- page/AccessibilityObject.cpp: (WebCore::AccessibilityObject::doAccessibilityHitTest): Added a null check for layer. Removed a null check for m_renderer, which shouldn't occur.
- 1:23 AM Changeset in webkit [32471] by
-
- 2 edits in trunk/WebKit/win
2008-04-23 Jon Honeycutt <jhoneycutt@apple.com>
Reviewed by Adam.
Implement accHitTest().
- AccessibleBase.cpp: (AccessibleBase::accHitTest): Perform a hit test for a child object. If none is found, report whether the point was within our own bounding box.
- 1:23 AM Changeset in webkit [32470] by
-
- 2 edits in trunk/WebKit/win
2008-04-23 Jon Honeycutt <jhoneycutt@apple.com>
Reviewed by Adam and Darin.
Implement accNavigate().
- AccessibleBase.cpp: (AccessibleBase::accNavigate): This method navigates between elements, given a start point and a direction. The directions UP, DOWN, LEFT, RIGHT are not are not implemented, matching Mozilla and IE. The directions FIRSTCHILD and LASTCHILD are only allowed from self. NEXT and PREVIOUS are allowed from self or any of our children.
- 1:22 AM Changeset in webkit [32469] by
-
- 4 edits in trunk
2008-04-23 Jon Honeycutt <jhoneycutt@apple.com>
Reviewed by Adam.
Implement accLocation().
- AccessibleBase.cpp: (AccessibleBase::accLocation): Report the screen coordinates for the object's bounding box.
- 1:21 AM Changeset in webkit [32468] by
-
- 3 edits in trunk/WebCore
2008-04-23 Jon Honeycutt <jhoneycutt@apple.com>
Reviewed by Adam.
Implement contentsToScreen() and screenToContents() on Windows.
- platform/ScrollView.h: Make these declarations conditional on HAVE(ACCESSIBILITY) instead of PLATFORM(MAC).
- platform/win/ScrollViewWin.cpp: (WebCore::ScrollView::contentsToScreen): Translate from coordinates relative to the ScrollView to coordinates relative to the screen. Do this by getting the offset of the ScrollView in screen coordinates and moving the rect by this amount. (WebCore::ScrollView::screenToContents): Translate from coordinates relative to the screen to coordinates relative to the ScrollView.
- 1:20 AM Changeset in webkit [32467] by
-
- 8 edits in trunk
2008-04-23 Jon Honeycutt <jhoneycutt@apple.com>
Reviewed by Adam, Darin, Oliver.
Implement get_accDefaultAction().
- AccessibleBase.cpp: (AccessibleBase::get_accDefaultAction): Report the result of calling the object's actionVerb method.
- English.lproj/Localizable.strings: Updated with new strings.
- WebCoreLocalizedStrings.cpp: (WebCore::AXButtonActionVerb): Return a localized string representing the object's default action. (WebCore::AXRadioButtonActionVerb): Same. (WebCore::AXTextFieldActionVerb): Same. (WebCore::AXCheckedCheckBoxActionVerb): Same (WebCore::AXUncheckedCheckBoxActionVerb): Same. (WebCore::AXLinkActionVerb): Same.
- 1:19 AM Changeset in webkit [32466] by
-
- 2 edits in trunk/WebKit/win
2008-04-22 Jon Honeycutt <jhoneycutt@apple.com>
Reviewed by Adam.
Implement get_accFocus().
- AccessibleBase.cpp: (AccessibleBase::get_accRole): Remove redundant initialize; VariantInit does this. (AccessibleBase::get_accState): Same. (AccessibleBase::get_accSelection): Relocated next to accSelect(). (AccessibleBase::get_accFocus): If the focused object is this object or a child of this object, report it. Otherwise, report VT_EMPTY.
- 1:15 AM Changeset in webkit [32465] by
-
- 2 edits in trunk/WebKit/gtk
Reviewed by Alp Toker.
Emit a warning about an unknown setting only if it is
not a valid property. Otherwise applications can't
subclass WebKitWebSettings to add new properties.
- webkit/webkitwebview.cpp:
- 12:55 AM Changeset in webkit [32464] by
-
- 25 edits in trunk
Tor Arne Vestbø <tavestbo@trolltech.com>
Cleaned up copyright headers in the Qt port (removed misplaced class
descriptions and fixed inconsistent whitespace and indentation).
- 12:44 AM Changeset in webkit [32463] by
-
- 2 edits in trunk/WebKit/qt
Tor Arne Vestbø <tavestbo@trolltech.com>
Added basic URL guessing to QtLauncher (same as in the demo browser).
- 12:44 AM Changeset in webkit [32462] by
-
- 2 edits in trunk/WebCore
Tor Arne Vestbø <tavestbo@trolltech.com>
Disable vanlilla focus rings since Qt provides this as part of the style.
- 12:38 AM Changeset in webkit [32461] by
-
- 2 edits in trunk/WebCore
George Staikos <george@staikos.net>
This optimization in BitmapImage::drawPattern for the identity
transform is bogus and causes incorrect results on sites like youtube.
- 12:31 AM Changeset in webkit [32460] by
-
- 2 edits in trunk/WebKit/qt
Benjamin Meyer <bmeyer@trolltech.com>
Prevent double deletions of the default web interface.
