| 2008-08-23 Cameron Zwarich <cwzwarich@uwaterloo.ca> |
| |
| Rubber-stamped by Mark Rowe. |
| |
| Remove modelines. |
| |
| * WebCore.pro: |
| * bridge/testbindings.cpp: |
| * dom/DocPtr.h: |
| * loader/SubstituteData.h: |
| * page/Chrome.cpp: |
| * page/Chrome.h: |
| * page/ChromeClient.h: |
| * page/Frame.h: |
| * page/FrameLoadRequest.h: |
| * page/FrameTree.cpp: |
| * page/FrameTree.h: |
| * page/Page.h: |
| * page/mac/ChromeMac.mm: |
| * platform/network/HTTPHeaderMap.h: |
| * platform/network/ResourceErrorBase.cpp: |
| * platform/network/ResourceErrorBase.h: |
| * platform/network/ResourceHandleInternal.h: |
| * platform/network/ResourceRequestBase.cpp: |
| * platform/network/ResourceRequestBase.h: |
| * platform/network/ResourceResponseBase.cpp: |
| * platform/network/ResourceResponseBase.h: |
| * platform/network/cf/ResourceError.h: |
| * platform/network/cf/ResourceRequest.h: |
| * platform/network/cf/ResourceRequestCFNet.h: |
| * platform/network/cf/ResourceResponse.h: |
| * platform/network/cf/ResourceResponseCFNet.h: |
| * platform/network/curl/ResourceError.h: |
| * platform/network/curl/ResourceRequest.h: |
| * platform/network/curl/ResourceResponse.h: |
| * platform/network/mac/ResourceError.h: |
| * platform/network/mac/ResourceErrorMac.mm: |
| * platform/network/mac/ResourceRequest.h: |
| * platform/network/mac/ResourceRequestMac.mm: |
| * platform/network/mac/ResourceResponse.h: |
| * platform/network/mac/ResourceResponseMac.mm: |
| * platform/network/qt/ResourceError.h: |
| * platform/network/qt/ResourceRequest.h: |
| * platform/network/qt/ResourceResponse.h: |
| * platform/network/soup/CookieJarSoup.cpp: |
| * platform/network/soup/ResourceError.h: |
| * platform/network/soup/ResourceRequest.h: |
| * platform/network/soup/ResourceResponse.h: |
| |
| 2008-08-23 Dan Bernstein <mitz@apple.com> |
| |
| Reviewed by Dave Hyatt. |
| |
| - fix <rdar://problem/6065143> Reflowed image in first line of table doesn't affect baseline |
| |
| Test: fast/table/vertical-align-baseline.html |
| |
| * rendering/RenderTable.cpp: |
| (WebCore::RenderTable::getBaselineOfFirstLineBox): Added. Returns the |
| baseline of the first table row, if there is one, -1 otherwise. |
| * rendering/RenderTable.h: |
| * rendering/RenderTableCell.cpp: |
| (WebCore::RenderTableCell::baselinePosition): Changed to follow the |
| CSS2.1 definition of the baseline of a table cell. |
| * rendering/RenderTableSection.cpp: |
| (WebCore::RenderTableSection::getBaselineOfFirstLineBox): Added. Returns |
| the baseline of the first row in the section. |
| * rendering/RenderTableSection.h: |
| |
| 2008-08-18 Jon Honeycutt <jhoneycutt@apple.com> |
| |
| Add SPI to make a Windows WebView transparent. |
| |
| Reviewed by Dan Bernstein. |
| |
| * platform/graphics/GraphicsContext.h: Add a parameter, hasAlpha, that |
| determines whether the created context has an alpha channel. |
| * platform/graphics/win/GraphicsContextCGWin.cpp: |
| (WebCore::CGContextWithHDC): |
| |
| 2008-08-22 Timothy Hatcher <timothy@apple.com> |
| |
| Rolls out r35834 because it caused a regression in the Inspector's |
| Console where some expressions always threw a syntax error. |
| |
| https://bugs.webkit.org/show_bug.cgi?id=20487 |
| |
| * page/inspector/Console.js: |
| (Console.prototype._evalInInspectedWindow): Removes parenthesis |
| around the expression. |
| |
| 2008-08-22 Timothy Hatcher <timothy@apple.com> |
| |
| Makes getStyleProperty return a value for the overflow property |
| when overflow-x and overflow-y are equal. |
| |
| https://bugs.webkit.org/show_bug.cgi?id=20485 |
| |
| Reviewed by Dan Bernstein. |
| |
| Test: fast/css/overflow-property.html |
| |
| * css/CSSMutableStyleDeclaration.cpp: |
| (WebCore::CSSMutableStyleDeclaration::getPropertyValue): |
| |
| 2008-08-22 Timothy Hatcher <timothy@apple.com> |
| |
| Fixes a bug where while editing in the Inspector the |
| sidebar scroll position would jump to the top. |
| |
| https://bugs.webkit.org/show_bug.cgi?id=20484 |
| |
| Reviewed by Darin Adler. |
| |
| * page/inspector/treeoutline.js: |
| (TreeOutline._removeChildren): Remove the offsetTop call that |
| was forcing a layout, since layout causes scroll positions |
| to be clamped to the new scrollHeight/Width. Layout will happen |
| normally when needed. |
| |
| 2008-08-22 Kevin Ollivier <kevino@theolliviers.com> |
| |
| wx build fix. Add AccessibilityTable sources. |
| |
| * WebCoreSources.bkl: |
| |
| 2008-08-21 Dan Bernstein <mitz@apple.com> |
| |
| Reviewed by Jon Honeycutt and Alexey Proskuryakov. |
| |
| - fix <rdar://problem/6162701> WebKit should correct Geeza Pro's font metrics |
| |
| Test: platform/mac/fast/text/international/Geeza-Pro-vertical-metrics-adjustment.html |
| |
| * platform/graphics/mac/SimpleFontDataMac.mm: |
| (WebCore::SimpleFontData::platformInit): Add 8% to Geeza Pro's reported |
| ascent and 100% to its reported descent. |
| |
| 2008-08-21 Kalle Vahlman <kalle.vahlman@movial.com> |
| |
| Reviewed by Darin Adler. |
| |
| https://bugs.webkit.org/show_bug.cgi?id=20267 |
| [GTK] Crash on some pages due to a plugin |
| |
| Fix handling of badly formatted and empty plugin mime descriptions |
| |
| * plugins/gtk/PluginPackageGtk.cpp: |
| (WebCore::PluginPackage::fetchInfo): |
| |
| 2008-08-21 Kevin Watters <kevinwatters@gmail.com> |
| |
| Reviewed by Darin Adler. |
| |
| Follow other ports (and IE) in ImageSourceWx.cpp never to return an |
| animated GIF frame duration of less than 50ms. |
| |
| * platform/graphics/wx/ImageSourceWx.cpp: |
| (WebCore::ImageSource::frameDurationAtIndex): |
| |
| 2008-08-21 Maxime Britto <britto@apple.com> |
| |
| Reviewed by Kevin McCullough. |
| |
| Test: fast/events/autoscroll-nonscrollable-iframe-in-scrollable-div.html |
| https://bugs.webkit.org/show_bug.cgi?id=20451 |
| |
| rdar://problem/6166435 Inspector doesn't auto scroll when selecting text (20451) |
| When we climb up the rendering tree looking for a scrollable renderer, we need to be able to jump outside of an iframe. |
| This way we can see if what is embedding the iframe can be scrolled even if the iframe content can't. |
| |
| * page/EventHandler.cpp: |
| (WebCore::EventHandler::handleMouseDraggedEvent): when we reach the document node and it can't be scrolled we set the next parent as the document's owner element if it exists. |
| |
| (WebCore::EventHandler::handleMousePressEvent): ditto |
| |
| 2008-08-21 Anthony Ricaud <rik24d@gmail.com> |
| |
| Small optimization for when the dividers in the Inspector's |
| Resources panel are updated. |
| |
| Reviewed by Tim Hatcher. |
| |
| * page/inspector/ResourcesPanel.js: |
| (WebInsector.ResourcesPanel.prototype._updateGraphDividersIfNeeded): |
| Combind a for loop and clone the divider element. |
| |
| 2008-08-21 Chris Marrin <cmarrin@apple.com> |
| |
| Allow 0 (without units) for Time eg. duration |
| Fix for https://bugs.webkit.org/show_bug.cgi?id=20467 |
| |
| Reviewed by Dave Hyatt. |
| |
| Test: css1/units/zero-duration-without-units.html |
| |
| * css/CSSParser.cpp: |
| (WebCore::CSSParser::validUnit): |
| |
| 2008-08-21 Timothy Hatcher <timothy@apple.com> |
| |
| Make deleting all text while editing a DOM attribute in |
| the Inspector delete the attribute. This also fixes |
| an exception that would happen before. |
| |
| https://bugs.webkit.org/show_bug.cgi?id=20472 |
| |
| Reviewed by Kevin McCullough. |
| |
| * page/inspector/ElementsPanel.js: |
| (WebInspector.DOMNodeTreeElement.prototype._attributeEditingCommitted): |
| Don't check for hasAttributes on the parseElement, continuing |
| through the function will correctly remove the attribute. |
| If the parseElement is null, call _editingCancelled not |
| editingCancelled, this fixes an exception. |
| |
| 2008-08-21 Timothy Hatcher <timothy@apple.com> |
| |
| Update the Inspector's Metrics pane when editing in |
| the Styles pane. This makes sure the metrics shown |
| always match what the Styles pane shows. |
| |
| https://bugs.webkit.org/show_bug.cgi?id=20470 |
| |
| Reviewed by Kevin McCullough. |
| |
| * page/inspector/ElementsPanel.js: |
| (WebInspector.ElementsPanel): Add event listeners for |
| the "style edited" and "style property toggled" events, |
| so the Metrics pane is updated. |
| (WebInspector.ElementsPanel.prototype._stylesPaneEdited): |
| Update the Metrics pane. |
| * page/inspector/StylesSidebarPane.js: |
| (WebInspector.StylePropertyTreeElement.prototype.toggleEnabled): |
| Dispatch the "style property toggled" event. |
| (WebInspector.StylePropertyTreeElement.prototype.editingCancelled): |
| Dispatch the "style edited" event when the CSS text is restored. |
| (WebInspector.StylePropertyTreeElement.prototype.applyStyleText): |
| Dispatch the "style edited" event. |
| |
| 2008-08-21 Timothy Hatcher <timothy@apple.com> |
| |
| Make the Inspector's Metrics sidebar pane editable. |
| |
| https://bugs.webkit.org/show_bug.cgi?id=17218 |
| rdar://problem/5732818 |
| |
| Reviewed by Kevin McCullough. |
| |
| * page/inspector/ElementsPanel.js: |
| (WebInspector.ElementsPanel): Add an event listener for |
| the "metrics edited" event, so the Styles pane is updated. |
| * page/inspector/MetricsSidebarPane.js: |
| (WebInspector.MetricsSidebarPane.prototype.update): Remember the node |
| so future updates work. Add a double click event listener for the |
| metric values to start editing. |
| (WebInspector.MetricsSidebarPane.prototype.startEditing): |
| Call WebInspector.startEditing with some context. |
| (WebInspector.MetricsSidebarPane.prototype.editingCancelled): |
| (WebInspector.MetricsSidebarPane.prototype.editingCommitted): |
| Set the user input on the elements inline style. Fire the |
| "metrics edited" event. |
| |
| 2008-08-21 Steve Falkenburg <sfalken@apple.com> |
| |
| Fix a race condition in Windows timer code. |
| Timer function could end up being called with a 0 timer, leading to a Windows exception. |
| |
| Don't post a timer message if one is already pending. |
| |
| Reviewed by Ada Chan. |
| |
| * platform/win/SharedTimerWin.cpp: |
| (WebCore::TimerWindowWndProc): |
| (WebCore::clearTimer): |
| (WebCore::queueTimerProc): |
| (WebCore::setSharedTimerFireTime): |
| |
| 2008-08-21 Anthony Ricaud <rik24d@gmail.com> |
| |
| After trying to add the expression, try again with quotes for |
| easier edition. |
| |
| https://bugs.webkit.org/show_bug.cgi?id=20466 |
| |
| Reviewed by Tim Hatcher. |
| |
| * page/inspector/ObjectPropertiesSection.js: Added an |
| evaluateExpression function. |
| |
| 2008-08-21 Anthony Ricaud <rik24d@gmail.com> |
| |
| Perform Inspector searches on search event to clear results when |
| clicking the cross to empty it. Delete the lastQuery when the field |
| is emptied in order to perform the search if exactly the same query |
| is entered next. |
| |
| https://bugs.webkit.org/show_bug.cgi?id=20462 |
| |
| Reviewed by Tim Hatcher. |
| |
| * page/inspector/inspector.js: |
| |
| 2008-08-21 Marco Barisione <marco.barisione@collabora.co.uk> |
| |
| Reviewed by Mark Rowe. |
| |
| http://bugs.webkit.org/show_bug.cgi?id=19656 |
| [SOUP] The gio code should call didFail() instead of |
| didFinishLoading() in case of error |
| |
| In case of error call didFail() instead of didReceiveResponse() and |
| didFinishLoading(). |
| |
| * platform/network/soup/ResourceHandleSoup.cpp: |
| (WebCore::networkErrorForFile): |
| (WebCore::readCallback): |
| (WebCore::openCallback): |
| (WebCore::queryInfoCallback): |
| |
| 2008-08-20 Mark Rowe <mrowe@apple.com> |
| |
| Reviewed by Jon Honeycutt. |
| |
| Fix build failure. |
| |
| * bridge/c/c_instance.cpp: |
| (KJS::Bindings::CInstance::getPropertyNames): Declare count as uint32_t rather than unsigned |
| as that is what NPEnumerationFunctionPtr is declared as accepting. |
| |
| 2008-08-20 Mark Rowe <mrowe@apple.com> |
| |
| Reviewed by Dan Bernstein. |
| |
| Build fix. Handle kCGInterpolationMedium in switch statements if it is available. |
| |
| * platform/graphics/GraphicsContext.h: |
| (WebCore::): |
| * platform/graphics/cg/GraphicsContextCG.cpp: |
| (WebCore::GraphicsContext::setImageInterpolationQuality): |
| (WebCore::GraphicsContext::imageInterpolationQuality): |
| |
| 2008-08-20 Dan Bernstein <mitz@apple.com> |
| |
| Rubber-stamped by John Sullivan. |
| |
| - rename shouldUpdateWhileHidden to shouldUpdateWhileOffscreen and |
| rename related methods and variables accordingly. |
| |
| * WebCore.base.exp: |
| * page/FrameView.cpp: |
| (WebCore::FrameViewPrivate::FrameViewPrivate): |
| (WebCore::FrameView::shouldUpdateWhileOffscreen): |
| (WebCore::FrameView::setShouldUpdateWhileOffscreen): |
| * page/FrameView.h: |
| * platform/ScrollView.h: |
| * platform/mac/ScrollViewMac.mm: |
| (WebCore::ScrollView::updateContents): |
| |
| 2008-08-20 Brady Eidson <beidson@apple.com> |
| |
| Reviewed by Mitzpettel |
| |
| <rdar://problem/6163636> - Many images broken in Mail |
| |
| This can be traced back to the preload scanner. With that change, CachedResources are created a lot |
| sooner than before and confuse the WebArchive machinery. |
| |
| When referencing WebArchive subresources directly through the WebKit API it is appropriate to ignore |
| such CachedResources since they are placeholders and have not been submitted to the ResourceLoadDelegate |
| machinery and nothing is known about where the data will eventually come from. |
| |
| * loader/DocumentLoader.cpp: |
| (WebCore::DocumentLoader::subresource): Ignore a CachedResource if its preloadResult is |
| CachedResource::PreloadReferenced. |
| |
| 2008-08-20 Holger Hans Peter Freyther <zecke@selfish.org> |
| |
| Unreviewed compile fix |
| |
| Catch up with the JSValue::type elimination. |
| |
| * bridge/qt/qt_instance.cpp: |
| (KJS::Bindings::QtRuntimeObjectImp::construct): |
| (KJS::Bindings::QtInstance::defaultValue): |
| * bridge/qt/qt_runtime.cpp: |
| (KJS::Bindings::QtRuntimeConnectionMethod::call): |
| |
| 2008-08-20 Chris Teague <chris.teague@gmail.com> |
| |
| Reviewed by Mark Rowe. |
| |
| Fix https://bugs.webkit.org/show_bug.cgi?id=20449 |
| Bug 20449: Build fails if LOW_BANDWIDTH_DISPLAY is defined |
| |
| * ChangeLog: |
| * loader/FrameLoader.cpp: |
| (WebCore::FrameLoader::switchOutLowBandwidthDisplayIfReady): |
| |
| 2008-08-20 Simon Fraser <simon.fraser@apple.com> |
| |
| Reviewed by Dave Hyatt |
| |
| getComputedStyle() for webkitTransform should return a transform |
| that does not have the transform origin baked into it. |
| https://bugs.webkit.org/show_bug.cgi?id=20464 |
| |
| Test: fast/css/getComputedStyle-transform.html |
| |
| * css/CSSComputedStyleDeclaration.cpp: |
| (WebCore::computedTransform): |
| * rendering/style/RenderStyle.cpp: |
| (WebCore::RenderStyle::applyTransform): |
| * rendering/style/RenderStyle.h: |
| |
| 2008-08-20 Josh Aas <joshmoz@gmail.com> |
| |
| Reviewed and landed by Anders. |
| |
| <rdar://problem/6163636> |
| rename NPCocoaEvent's "event" struct to "data" (20446) |
| |
| * bridge/npapi.h: |
| (_NPCocoaEvent::): |
| |
| 2008-08-20 Beth Dakin <bdakin@apple.com> |
| |
| Reviewed by Darin. |
| |
| Fix for <rdar://problem/6145626> |
| This patch fixes a number of remaining problems getting |
| disconnected frames to work correctly with markAllMatchesForText() |
| and findString(). Details inline. |
| |
| This was a static helper function in Frame, but this patch requires |
| the same functionality in Editor, so I just added it as a function |
| on Node. |
| * dom/Node.cpp: |
| (WebCore::Node::isInShadowTree): |
| * dom/Node.h: |
| |
| * editing/Editor.cpp: |
| (WebCore::Editor::insideVisibleArea): Returns false if |
| excludeFromTextSearch() is true. |
| |
| In a normal (non-disconnected) frame, findString returns a range of |
| the document node if the text is not found in the frame. I changed |
| firstVisibleRange and lastVisibleRange to match that behavior when |
| the text is not found |
| (WebCore::Editor::firstVisibleRange): |
| (WebCore::Editor::lastVisibleRange): |
| |
| Here are the bulk of the changes in the patch. A lot of text was |
| not being found in disconnected frames because I failed to account |
| for all of the possible problems associated with shadow trees. That |
| is fixed here. |
| (WebCore::Editor::nextVisibleRange): |
| * editing/Editor.h: |
| |
| excludeFromTextSearch() is new. It allows a WebKit client to mark a |
| frame as not-text-searchable through SPI. |
| * WebCore.base.exp: |
| * page/Frame.cpp: |
| (WebCore::Frame::excludeFromTextSearch): |
| (WebCore::Frame::setExcludeFromTextSearch): |
| (WebCore::FramePrivate::FramePrivate): |
| * page/Frame.h: |
| * page/FramePrivate.h: |
| |
| (WebCore::Frame::findString): |
| (WebCore::Frame::markAllMatchesForText): I kept running into an |
| assertion failure in paining code because of the forced paint on |
| empty visible rects. |
| |
| 2008-08-20 Timothy Hatcher <timothy@apple.com> |
| |
| Adds a positon box to the Inspector's Metrics sidebar |
| pane. When an element is not statically positioned, there |
| is now a position box that show top, right, bottom and |
| left computed values. |
| |
| Reviewed by Kevin McCullough. |
| |
| * English.lproj/localizedStrings.js: Updated the strings. |
| * page/inspector/MetricsSidebarPane.js: |
| (WebInspector.MetricsSidebarPane.prototype.update): |
| Renamed the boxPartValue function to createBoxPartElement |
| and made it create the entire element. Made it understand |
| how to get position style properties. Don't use the figure dash |
| when 0px is used for positions, since the 0 is meaningful there. |
| Instead use the figure dash when a position is auto. |
| * page/inspector/inspector.css: Added a new rule for position. |
| |
| 2008-08-20 chris fleizach <cfleizach@apple.com> |
| |
| Reviewed by Beth Dakin |
| |
| Qt build fix |
| |
| * WebCore.pro: |
| |
| 2008-08-20 Dan Bernstein <mitz@apple.com> |
| |
| Reviewed by Anders Carlsson. |
| |
| - avoid using a deprecated NSScroller method on Leopard |
| |
| * platform/mac/PlatformScrollBarMac.mm: |
| (WebCore::PlatformScrollbar::updateThumbPosition): |
| (WebCore::PlatformScrollbar::updateThumbProportion): |
| |
| 2008-08-20 Jan Michael Alonzo <jmalonzo@webkit.org> |
| |
| Reviewed by Oliver Hunt. |
| |
| Gtk build fix |
| |
| * GNUmakefile.am: |
| * page/AccessibilityTable.cpp: Change nil to 0 |
| (WebCore::AccessibilityTable::cellForColumnAndRow): |
| |
| 2008-08-19 Alexey Proskuryakov <ap@webkit.org> |
| |
| Reviewed by Geoff Garen. |
| |
| Bring back shared JSGlobalData and implicit locking, because too many clients rely on it. |
| |
| * ForwardingHeaders/kjs/JSLock.h: Added. |
| * WebCore.vcproj/WebCore.vcproj: |
| * bindings/js/GCController.cpp: |
| (WebCore::collect): |
| (WebCore::GCController::gcTimerFired): |
| (WebCore::GCController::garbageCollectNow): |
| * bindings/js/JSCustomSQLStatementCallback.cpp: |
| (WebCore::JSCustomSQLStatementCallback::handleEvent): |
| * bindings/js/JSCustomSQLStatementErrorCallback.cpp: |
| (WebCore::JSCustomSQLStatementErrorCallback::handleEvent): |
| * bindings/js/JSCustomSQLTransactionCallback.cpp: |
| (WebCore::JSCustomSQLTransactionCallback::handleEvent): |
| * bindings/js/JSCustomSQLTransactionErrorCallback.cpp: |
| (WebCore::JSCustomSQLTransactionErrorCallback::handleEvent): |
| * bindings/js/JSCustomVoidCallback.cpp: |
| (WebCore::JSCustomVoidCallback::handleEvent): |
| * bindings/js/JSCustomXPathNSResolver.cpp: |
| (WebCore::JSCustomXPathNSResolver::lookupNamespaceURI): |
| * bindings/js/JSDOMWindowBase.cpp: |
| (WebCore::DOMWindowTimer::~DOMWindowTimer): |
| (WebCore::JSDOMWindowBase::clear): |
| (WebCore::JSDOMWindowBase::timerFired): |
| * bindings/js/JSEventCustom.cpp: |
| (WebCore::toJS): |
| * bindings/js/JSEventListener.cpp: |
| (WebCore::JSAbstractEventListener::handleEvent): |
| * bindings/js/JSNSResolver.cpp: |
| (WebCore::JSNSResolver::lookupNamespaceURI): |
| * bindings/js/JSNodeFilterCondition.cpp: |
| (WebCore::JSNodeFilterCondition::acceptNode): |
| * bindings/js/ScheduledAction.cpp: |
| (WebCore::ScheduledAction::execute): |
| * bindings/js/ScriptController.cpp: |
| (WebCore::ScriptController::evaluate): |
| (WebCore::ScriptController::clearWindowShell): |
| (WebCore::ScriptController::createHTMLEventHandler): |
| (WebCore::ScriptController::createSVGEventHandler): |
| (WebCore::ScriptController::initScript): |
| (WebCore::ScriptController::updateDocument): |
| (WebCore::ScriptController::bindingRootObject): |
| (WebCore::ScriptController::windowScriptNPObject): |
| (WebCore::ScriptController::createScriptObjectForPluginElement): |
| (WebCore::ScriptController::clearScriptObjects): |
| * bindings/js/ScriptControllerMac.mm: |
| (WebCore::ScriptController::windowScriptObject): |
| * bindings/objc/WebScriptObject.mm: |
| (_didExecute): |
| (-[WebScriptObject callWebScriptMethod:withArguments:]): |
| (-[WebScriptObject evaluateWebScript:]): |
| (-[WebScriptObject setValue:forKey:]): |
| (-[WebScriptObject valueForKey:]): |
| (-[WebScriptObject removeWebScriptKey:]): |
| (-[WebScriptObject stringRepresentation]): |
| (-[WebScriptObject webScriptValueAtIndex:]): |
| (-[WebScriptObject setWebScriptValueAtIndex:value:]): |
| (+[WebScriptObject _convertValueToObjcValue:originRootObject:rootObject:]): |
| * bridge/NP_jsobject.cpp: |
| (_NPN_InvokeDefault): |
| (_NPN_Invoke): |
| (_NPN_Evaluate): |
| (_NPN_GetProperty): |
| (_NPN_SetProperty): |
| (_NPN_RemoveProperty): |
| (_NPN_HasProperty): |
| (_NPN_HasMethod): |
| (_NPN_Enumerate): |
| * bridge/c/c_class.cpp: |
| (KJS::Bindings::CClass::~CClass): |
| (KJS::Bindings::CClass::methodsNamed): |
| (KJS::Bindings::CClass::fieldNamed): |
| * bridge/c/c_instance.cpp: |
| (KJS::Bindings::CInstance::invokeMethod): |
| (KJS::Bindings::CInstance::invokeDefaultMethod): |
| (KJS::Bindings::CInstance::getPropertyNames): |
| * bridge/c/c_runtime.cpp: |
| (KJS::Bindings::CField::valueFromInstance): |
| (KJS::Bindings::CField::setValueToInstance): |
| * bridge/c/c_utility.cpp: |
| (KJS::Bindings::convertValueToNPVariant): |
| (KJS::Bindings::convertNPVariantToValue): |
| * bridge/jni/jni_class.cpp: |
| (JavaClass::JavaClass): |
| (JavaClass::~JavaClass): |
| * bridge/jni/jni_instance.cpp: |
| (JavaInstance::stringValue): |
| * bridge/jni/jni_jsobject.mm: |
| (JavaJSObject::call): |
| (JavaJSObject::eval): |
| (JavaJSObject::getMember): |
| (JavaJSObject::setMember): |
| (JavaJSObject::removeMember): |
| (JavaJSObject::getSlot): |
| (JavaJSObject::setSlot): |
| (JavaJSObject::toString): |
| (JavaJSObject::convertValueToJObject): |
| (JavaJSObject::convertJObjectToValue): |
| * bridge/jni/jni_objc.mm: |
| (KJS::Bindings::dispatchJNICall): |
| * bridge/jni/jni_runtime.cpp: |
| (appendClassName): |
| (JavaMethod::signature): |
| * bridge/jni/jni_runtime.h: |
| (KJS::Bindings::JavaString::JavaString): |
| (KJS::Bindings::JavaString::_commonInit): |
| (KJS::Bindings::JavaString::~JavaString): |
| (KJS::Bindings::JavaString::UTF8String): |
| * bridge/jni/jni_utility.cpp: |
| (KJS::Bindings::convertValueToJValue): |
| * bridge/npruntime.cpp: |
| (_NPN_GetStringIdentifier): |
| * bridge/objc/objc_instance.mm: |
| (ObjcInstance::moveGlobalExceptionToExecState): |
| (ObjcInstance::invokeMethod): |
| (ObjcInstance::invokeDefaultMethod): |
| (ObjcInstance::setValueOfUndefinedField): |
| (ObjcInstance::getValueOfUndefinedField): |
| * bridge/objc/objc_runtime.mm: |
| (ObjcField::valueFromInstance): |
| (ObjcField::setValueToInstance): |
| * bridge/objc/objc_utility.mm: |
| (KJS::Bindings::convertValueToObjcValue): |
| (KJS::Bindings::convertNSStringToString): |
| (KJS::Bindings::convertObjcValueToValue): |
| * bridge/qt/qt_instance.cpp: |
| (KJS::Bindings::QtRuntimeObjectImp::removeFromCache): |
| (KJS::Bindings::QtInstance::~QtInstance): |
| (KJS::Bindings::QtInstance::getQtInstance): |
| (KJS::Bindings::QtInstance::getRuntimeObject): |
| * bridge/qt/qt_runtime.cpp: |
| (KJS::Bindings::convertValueToQVariant): |
| (KJS::Bindings::convertQVariantToValue): |
| (KJS::Bindings::QtRuntimeMetaMethod::call): |
| (KJS::Bindings::QtRuntimeConnectionMethod::call): |
| (KJS::Bindings::QtConnectionObject::QtConnectionObject): |
| (KJS::Bindings::QtConnectionObject::execute): |
| * bridge/runtime.cpp: |
| (KJS::Bindings::Instance::createRuntimeObject): |
| * bridge/testbindings.cpp: |
| (main): |
| * bridge/testbindings.mm: |
| (main): |
| * bridge/testqtbindings.cpp: |
| (main): |
| * dom/Document.cpp: |
| (WebCore::Document::~Document): |
| * dom/Node.cpp: |
| (WebCore::Node::setDocument): |
| * history/CachedPage.cpp: |
| (WebCore::CachedPage::CachedPage): |
| (WebCore::CachedPage::restore): |
| (WebCore::CachedPage::clear): |
| * loader/FrameLoader.cpp: |
| (WebCore::getString): |
| * page/InspectorController.cpp: |
| (WebCore::ConsoleMessage::ConsoleMessage): |
| (WebCore::XMLHttpRequestResource::XMLHttpRequestResource): |
| (WebCore::XMLHttpRequestResource::~XMLHttpRequestResource): |
| (WebCore::getResourceDocumentNode): |
| (WebCore::search): |
| (WebCore::inspectedWindow): |
| (WebCore::wrapCallback): |
| (WebCore::currentCallFrame): |
| (WebCore::profiles): |
| (WebCore::InspectorController::focusNode): |
| (WebCore::InspectorController::inspectedWindowScriptObjectCleared): |
| (WebCore::InspectorController::addDatabaseScriptResource): |
| (WebCore::InspectorController::addScriptProfile): |
| * page/JavaScriptCallFrame.cpp: |
| (WebCore::JavaScriptCallFrame::evaluate): |
| * page/JavaScriptProfileNode.cpp: |
| (WebCore::getTotalTime): |
| (WebCore::getSelfTime): |
| (WebCore::getTotalPercent): |
| (WebCore::getSelfPercent): |
| (WebCore::getNumberOfCalls): |
| (WebCore::getChildren): |
| (WebCore::getVisible): |
| * page/Page.cpp: |
| * page/mac/FrameMac.mm: |
| * plugins/PluginView.cpp: |
| (WebCore::PluginView::start): |
| (WebCore::getString): |
| (WebCore::PluginView::performRequest): |
| (WebCore::PluginView::bindingInstance): |
| * plugins/gtk/PluginViewGtk.cpp: |
| (WebCore::PluginView::paint): |
| (WebCore::PluginView::handleKeyboardEvent): |
| (WebCore::PluginView::handleMouseEvent): |
| (WebCore::PluginView::setNPWindowRect): |
| (WebCore::PluginView::stop): |
| (WebCore::PluginView::init): |
| * plugins/qt/PluginViewQt.cpp: |
| (WebCore::PluginView::setNPWindowRect): |
| (WebCore::PluginView::stop): |
| (WebCore::PluginView::init): |
| * plugins/win/PluginViewWin.cpp: |
| (WebCore::PluginView::dispatchNPEvent): |
| (WebCore::PluginView::handleKeyboardEvent): |
| (WebCore::PluginView::handleMouseEvent): |
| (WebCore::PluginView::setNPWindowRect): |
| (WebCore::PluginView::stop): |
| * xml/XMLHttpRequest.cpp: |
| (WebCore::XMLHttpRequest::clearResponse): |
| (WebCore::XMLHttpRequest::didFinishLoading): |
| (WebCore::XMLHttpRequest::didReceiveData): |
| |
| 2008-08-19 Eric Seidel <eric@webkit.org> |
| |
| Reviewed by Geoff Garen. |
| |
| Add the beginnings of Skia graphics support to WebCore |
| as I try to begin the long process of un-forking the changes |
| needed to WebCore to make Andriod's WebCore build. |
| |
| I'll follow this up with actual *Skia.cpp files in a separate patch. |
| |
| * platform/graphics/AffineTransform.h: |
| * platform/graphics/FloatPoint.h: |
| * platform/graphics/FloatRect.h: |
| * platform/graphics/Gradient.h: |
| * platform/graphics/GraphicsContext.h: |
| * platform/graphics/Image.h: |
| * platform/graphics/ImageBuffer.h: |
| * platform/graphics/ImageSource.h: |
| * platform/graphics/IntPoint.h: |
| * platform/graphics/IntRect.h: |
| * platform/graphics/Path.h: |
| * platform/graphics/Pattern.h: |
| * svg/graphics/SVGPaintServerPattern.h: |
| * svg/graphics/SVGPaintServerSolid.h: |
| |
| 2008-08-19 Steve Falkenburg <sfalken@apple.com> |
| |
| Fix Windows build more. |
| |
| * WebCore.vcproj/WebCore.vcproj: |
| |
| 2008-08-19 Steve Falkenburg <sfalken@apple.com> |
| |
| Fix Windows build. |
| |
| * page/AccessibilityRenderObject.cpp: |
| (WebCore::AccessibilityRenderObject::canSetFocusAttribute): |
| |
| 2008-08-19 chris fleizach <cfleizach@apple.com> |
| |
| Reviewed by Beth Dakin. |
| |
| <rdar://problem/4003764> Expose tables as AXTables |
| |
| Tests: accessibility/table-attributes.html |
| accessibility/table-cell-spans.html |
| accessibility/table-cells.html |
| accessibility/table-detection.html |
| accessibility/table-sections.html |
| accessibility/table-with-rules.html |
| |
| * WebCore.xcodeproj/project.pbxproj: |
| * page/AXObjectCache.cpp: |
| (WebCore::AXObjectCache::get): |
| * page/AccessibilityObject.h: |
| (WebCore::): |
| (WebCore::AccessibilityObject::isDataTable): |
| (WebCore::AccessibilityObject::isTableRow): |
| (WebCore::AccessibilityObject::isTableColumn): |
| (WebCore::AccessibilityObject::isTableCell): |
| * page/AccessibilityRenderObject.cpp: |
| (WebCore::AccessibilityRenderObject::canSetFocusAttribute): |
| * page/AccessibilityTable.cpp: Added. |
| (WebCore::AccessibilityTable::AccessibilityTable): |
| (WebCore::AccessibilityTable::~AccessibilityTable): |
| (WebCore::AccessibilityTable::create): |
| (WebCore::AccessibilityTable::isTableExposableThroughAccessibility): |
| (WebCore::AccessibilityTable::clearChildren): |
| (WebCore::AccessibilityTable::addChildren): |
| (WebCore::AccessibilityTable::headerContainer): |
| (WebCore::AccessibilityTable::columns): |
| (WebCore::AccessibilityTable::rows): |
| (WebCore::AccessibilityTable::rowHeaders): |
| (WebCore::AccessibilityTable::columnHeaders): |
| (WebCore::AccessibilityTable::cells): |
| (WebCore::AccessibilityTable::columnCount): |
| (WebCore::AccessibilityTable::rowCount): |
| (WebCore::AccessibilityTable::cellForColumnAndRow): |
| (WebCore::AccessibilityTable::roleValue): |
| (WebCore::AccessibilityTable::accessibilityIsIgnored): |
| (WebCore::AccessibilityTable::title): |
| (WebCore::AccessibilityTable::isDataTable): |
| * page/AccessibilityTable.h: Added. |
| * page/AccessibilityTableCell.cpp: Added. |
| (WebCore::AccessibilityTableCell::AccessibilityTableCell): |
| (WebCore::AccessibilityTableCell::~AccessibilityTableCell): |
| (WebCore::AccessibilityTableCell::create): |
| (WebCore::AccessibilityTableCell::accessibilityIsIgnored): |
| (WebCore::AccessibilityTableCell::rowIndexRange): |
| (WebCore::AccessibilityTableCell::columnIndexRange): |
| * page/AccessibilityTableCell.h: Added. |
| (WebCore::AccessibilityTableCell::isTableCell): |
| (WebCore::AccessibilityTableCell::roleValue): |
| * page/AccessibilityTableColumn.cpp: Added. |
| (WebCore::AccessibilityTableColumn::AccessibilityTableColumn): |
| (WebCore::AccessibilityTableColumn::~AccessibilityTableColumn): |
| (WebCore::AccessibilityTableColumn::create): |
| (WebCore::AccessibilityTableColumn::setParentTable): |
| (WebCore::AccessibilityTableColumn::elementRect): |
| (WebCore::AccessibilityTableColumn::size): |
| (WebCore::AccessibilityTableColumn::children): |
| (WebCore::AccessibilityTableColumn::headerObject): |
| (WebCore::AccessibilityTableColumn::headerObjectForSection): |
| (WebCore::AccessibilityTableColumn::addChildren): |
| * page/AccessibilityTableColumn.h: Added. |
| (WebCore::AccessibilityTableColumn::parentObject): |
| (WebCore::AccessibilityTableColumn::roleValue): |
| (WebCore::AccessibilityTableColumn::accessibilityIsIgnored): |
| (WebCore::AccessibilityTableColumn::isTableColumn): |
| (WebCore::AccessibilityTableColumn::setColumnIndex): |
| (WebCore::AccessibilityTableColumn::columnIndex): |
| * page/AccessibilityTableHeaderContainer.cpp: Added. |
| (WebCore::AccessibilityTableHeaderContainer::AccessibilityTableHeaderContainer): |
| (WebCore::AccessibilityTableHeaderContainer::~AccessibilityTableHeaderContainer): |
| (WebCore::AccessibilityTableHeaderContainer::create): |
| (WebCore::AccessibilityTableHeaderContainer::children): |
| (WebCore::AccessibilityTableHeaderContainer::elementRect): |
| (WebCore::AccessibilityTableHeaderContainer::size): |
| (WebCore::AccessibilityTableHeaderContainer::addChildren): |
| * page/AccessibilityTableHeaderContainer.h: Added. |
| (WebCore::AccessibilityTableHeaderContainer::roleValue): |
| (WebCore::AccessibilityTableHeaderContainer::setParentTable): |
| (WebCore::AccessibilityTableHeaderContainer::parentObject): |
| (WebCore::AccessibilityTableHeaderContainer::accessibilityIsIgnored): |
| * page/AccessibilityTableRow.cpp: Added. |
| (WebCore::AccessibilityTableRow::AccessibilityTableRow): |
| (WebCore::AccessibilityTableRow::~AccessibilityTableRow): |
| (WebCore::AccessibilityTableRow::create): |
| (WebCore::AccessibilityTableRow::accessibilityIsIgnored): |
| (WebCore::AccessibilityTableRow::headerObject): |
| * page/AccessibilityTableRow.h: Added. |
| (WebCore::AccessibilityTableRow::isTableRow): |
| (WebCore::AccessibilityTableRow::roleValue): |
| (WebCore::AccessibilityTableRow::setRowIndex): |
| (WebCore::AccessibilityTableRow::rowIndex): |
| * page/mac/AccessibilityObjectWrapper.mm: |
| (-[AccessibilityObjectWrapper accessibilityAttributeNames]): |
| (RoleEntry::): |
| (-[AccessibilityObjectWrapper roleDescription]): |
| (-[AccessibilityObjectWrapper accessibilityAttributeValue:]): |
| (-[AccessibilityObjectWrapper accessibilityParameterizedAttributeNames]): |
| (-[AccessibilityObjectWrapper accessibilityAttributeValue:forParameter:]): |
| |
| 2008-08-19 Steve Falkenburg <sfalken@apple.com> |
| |
| Build fix. |
| Add buildfailed support to stop builds early (preventing inaccurate error messages). |
| Add missing post-build rule to Release. |
| |
| * WebCore.vcproj/QTMovieWin.vcproj: |
| |
| 2008-08-19 Keishi Hattori <casey.hattori@gmail.com> |
| |
| Added support for console.count in the inspector. |
| |
| Reviewed by Geoff Garen. |
| |
| * page/Console.cpp: |
| (WebCore::Console::count): |
| * page/Console.h: |
| * page/Console.idl: Added console.count. |
| * page/InspectorController.cpp: |
| (WebCore::InspectorController::didCommitLoad): Clears m_counts. |
| (WebCore::InspectorController::count): Updates the count number |
| sing "title@source:line" as the identifier, and adds a |
| message to the console. |
| * page/InspectorController.h: Added m_counts. |
| |
| 2008-08-19 Keishi Hattori <casey.hattori@gmail.com> |
| |
| Clear console.time timers when changing page. |
| |
| Reviewed by Geoff Garen. |
| |
| * page/InspectorController.cpp: |
| (WebCore::InspectorController::didCommitLoad): |
| |
| 2008-08-19 Eric Carlson <eric.carlson@apple.com> |
| |
| Reviewed by Adele. |
| |
| Fix for <rdar://problem/6154695> Full-page movies flicker while playing |
| https://bugs.webkit.org/show_bug.cgi?id=20404 |
| |
| Ignore setVisible() when visibility doesn't change. |
| |
| * platform/graphics/mac/MediaPlayerPrivateQTKit.h: |
| * platform/graphics/mac/MediaPlayerPrivateQTKit.mm: |
| (WebCore::MediaPlayerPrivate::MediaPlayerPrivate): |
| (WebCore::MediaPlayerPrivate::setVisible): |
| |
| 2008-08-19 Timothy Hatcher <timothy@apple.com> |
| |
| Fixes a bug in the Profile view where switching sort order, then |
| switching from heavy to tree mode would show the tree in the |
| previous sort order. |
| |
| https://bugs.webkit.org/show_bug.cgi?id=20441 |
| |
| Reviewed by Kevin McCullough. |
| |
| * page/inspector/ProfileView.js: |
| (WebInspector.ProfileView.prototype._changeView): Call _sortProfile |
| on the next profile before assigning it to this.profile. |
| (WebInspector.ProfileView.prototype._sortData): Call _sortProfile. |
| (WebInspector.ProfileView.prototype._sortProfile): Moves from |
| _sortData and takes a profile argument. If the profile passed in |
| matches the this.profile, then call refresh. |
| |
| 2008-08-18 Timothy Hatcher <timothy@apple.com> |
| |
| Changed the default sort order now that heavy view is the default. |
| Also fixes a bug where the heavy profile was not sorted at first. |
| |
| https://bugs.webkit.org/show_bug.cgi?id=20440 |
| |
| Reviewed by Kevin McCullough. |
| |
| * page/inspector/ProfileView.js: |
| (WebInspector.ProfileView): Changed the default sort column. Also assign |
| heavyProfile to profile, so the sortSelfTimeDescending call happens |
| on the heavy profile before assigning to this.profile. |
| |
| 2008-08-18 Timothy Hatcher <timothy@apple.com> |
| |
| Add support for editing DOM properties and scope variables by double |
| clicking a property to enter edit mode. |
| |
| https://bugs.webkit.org/show_bug.cgi?id=20415 |
| |
| Reviewed by Kevin McCullough. |
| |
| * page/inspector/ObjectPropertiesSection.js: |
| (WebInspector.ObjectPropertiesSection): Set editable to true by default. |
| (WebInspector.ObjectPropertiesSection.prototype.onpopulate): |
| Factored out code into update, and calls update. |
| (WebInspector.ObjectPropertiesSection.prototype.update): Moved from onpopulate. |
| Call removeChildren since this method can be called multiple times now. |
| (WebInspector.ObjectPropertyTreeElement): Pass an empty title, the title |
| gets made later in onattach. |
| (WebInspector.ObjectPropertyTreeElement.prototype.onpopulate): Don't return early |
| if shouldRefreshChildren is true. Call removeChildren since this method can be |
| called multiple times now. |
| (WebInspector.ObjectPropertyTreeElement.prototype.ondblclick): Call startEditing. |
| (WebInspector.ObjectPropertyTreeElement.prototype.onattach): Call update. |
| (WebInspector.ObjectPropertyTreeElement.prototype.update): Update the title for |
| this element (code moved from the constructor.) |
| (WebInspector.ObjectPropertyTreeElement.prototype.updateSiblings): Recreate all |
| sibling property elements. |
| (WebInspector.ObjectPropertyTreeElement.prototype.startEditing): Call |
| WebInspector.startEditing after rememebring some context. |
| (WebInspector.ObjectPropertyTreeElement.prototype.editingEnded): Reset the scrollLeft |
| for the list element, since it might have scrolled during editing. |
| (WebInspector.ObjectPropertyTreeElement.prototype.editingCancelled): Call editingEnded |
| then restore the state from the context. Then call update to restore the title. |
| (WebInspector.ObjectPropertyTreeElement.prototype.editingCommitted): Call editingCancelled |
| if the user input and the previous input are the same. Call editingEnded, then call applyExpression |
| to commit the user input. |
| (WebInspector.ObjectPropertyTreeElement.prototype.applyExpression): Evaluates the input expression |
| and stores the result on the object for the property name of this element. If the expression is |
| empty, delete the property and remove the tree element. |
| * page/inspector/ScopeChainSidebarPane.js: |
| (WebInspector.ScopeChainSidebarPane.prototype.update): Set the editInSelectedCallFrameWhenPaused |
| property on each ObjectPropertiesSection. |
| (WebInspector.ScopeVariableTreeElement.prototype.onattach): Call ObjectPropertyTreeElement's onattach |
| since it is now implemented. |
| * page/inspector/ScriptsPanel.js: |
| (WebInspector.ScriptsPanel.prototype.evaluateInSelectedCallFrame): Added an updateInterface argument |
| that defaults to true if omitted. It specifies whether to call update on the scope chain. |
| * page/inspector/inspector.css: New styles. |
| * page/inspector/treeoutline.js: |
| (TreeElement.prototype._attach): Fixed an exception that fired when handling the shouldRefreshChildren |
| change. The nextSibling would exist but have a _listItemNode that didn't match the new parent. |
| |
| 2008-08-18 Timothy Hatcher <timothy@apple.com> |
| |
| Surround the expression to be evaluated in parenthesis so the |
| result of the eval is the result of the whole expression not |
| the last potential sub-expression. So evaluating {x: 123} |
| will show the Object not 123. |
| |
| https://bugs.webkit.org/show_bug.cgi?id=20428 |
| |
| Reviewed by Kevin McCullough. |
| |
| * page/inspector/Console.js: |
| (Console.prototype._evalInInspectedWindow): Add parenthesis |
| around the expression. And add couple comments. |
| |
| 2008-08-19 Kevin McCullough <kmccullough@apple.com> |
| |
| Reviewed by Geoff. |
| |
| -Implement a page() function to extract a common code pattern. |
| |
| * WebCore.xcodeproj/project.pbxproj: |
| * page/Console.cpp: |
| (WebCore::Console::addMessage): |
| (WebCore::Console::error): |
| (WebCore::Console::info): |
| (WebCore::Console::log): |
| (WebCore::Console::dir): |
| (WebCore::Console::assertCondition): |
| (WebCore::Console::time): |
| (WebCore::Console::timeEnd): |
| (WebCore::Console::group): |
| (WebCore::Console::groupEnd): |
| (WebCore::Console::finishedProfiling): |
| (WebCore::Console::warn): |
| (WebCore::Console::framePage): |
| * page/Console.h: |
| |
| 2008-08-12 Darin Adler <darin@apple.com> |
| |
| Reviewed by Geoff. |
| |
| - eliminate JSValue::type() |
| |
| * bridge/c/c_instance.cpp: |
| (KJS::Bindings::CInstance::defaultValue): Take PreferredPrimitiveType |
| argument instead of JSType argument. Removed unneeded code to handle |
| boolean, since that's never passed. |
| * bridge/c/c_instance.h: Ditto. |
| |
| * bridge/c/c_utility.cpp: |
| (KJS::Bindings::convertValueToNPVariant): Use JSValue::is functions |
| instead of JSValue::type(). Removed unneeded code to handle |
| "unspecified". |
| |
| * bridge/jni/jni_instance.cpp: |
| (JavaInstance::defaultValue): Take PreferredPrimitiveType argument |
| instead of JSType argument. Removed unneeded code to handle boolean. |
| * bridge/jni/jni_instance.h: Ditto. |
| |
| * bridge/jni/jni_jsobject.mm: |
| (JavaJSObject::convertValueToJObject): Use JSValue::is functions |
| instead of JSValue::type(). |
| |
| * bridge/objc/objc_instance.h: Take PreferredPrimitiveType argument |
| instead of JSType argument. Removed unused argument. |
| * bridge/objc/objc_instance.mm: |
| (ObjcInstance::getValueOfUndefinedField): Removed unused argument. |
| (ObjcInstance::defaultValue): Take PreferredPrimitiveType argument |
| instead of JSType argument. Removed unneeded code to handle boolean |
| and another dead code path for unknown types. |
| |
| * bridge/objc/objc_runtime.h: Take PreferredPrimitiveType argument |
| instead of JSType argument. Removed override of type() that caused |
| the fallback object to return "UndefinedType" when there is no |
| invokeUndefinedMethodFromWebScript:withArguments: method defined. |
| That didn't accomplish much, since most checks for undefined don't |
| ever call type(). |
| * bridge/objc/objc_runtime.mm: |
| (ObjcFallbackObjectImp::defaultValue): Ditto. |
| |
| * bridge/qt/qt_instance.cpp: |
| (KJS::Bindings::QtInstance::defaultValue): Take PreferredPrimitiveType |
| argument instead of JSType argument. Removed unneeded code to handle |
| boolean. |
| * bridge/qt/qt_instance.h: Ditto. |
| |
| * bridge/runtime.h: |
| (KJS::Bindings::Instance::getValueOfUndefinedField): Removed |
| unsed argument. |
| * bridge/runtime_object.cpp: |
| (RuntimeObjectImp::defaultValue): Take PreferredPrimitiveType |
| argument instead of JSType argument. |
| * bridge/runtime_object.h: Ditto. |
| |
| 2008-08-18 Maxime Britto <britto@apple.com> |
| |
| Reviewed by Adele. |
| |
| <rdar://6157207> Mouse pointer does not change when new window is opened after pan-scrolling original window |
| Related to the discussion from rdar://6102511 , we should disable every key event (except for the esc key which stops the panning). |
| We shouldn't be able to create another window while we are in pan scrolling. |
| Other browsers behaviors : |
| FF3 : Most of the keys are disabled, there is no way to create another window while in panscroll mode |
| IE7 : Keys are not disabled but stops immediately the panning. |
| This patch matches FF3 behavior by disabling every key but the esc key. |
| |
| * page/EventHandler.cpp: |
| (WebCore::EventHandler::stopAutoscrollTimer): Change the cursor back to the regular arrow cursor when the pannning is stopped. |
| (WebCore::EventHandler::keyEvent): When a key event is received while in panning or autoscroll we swallow the event early. |
| |
| 2008-08-18 Dan Bernstein <mitz@apple.com> |
| |
| Reviewed by Dave Hyatt. |
| |
| - fix <rdar://problem/5862634> REGRESSION (3.1.1): In iChat, inline image not resizable past current size after another IM is received |
| |
| Test: fast/replaced/max-width-percent.html |
| |
| Added an includeMaxWidth boolean to RenderBox::calcReplaedWidth(). |
| When false, max-width is not factored into the |
| calculation. |
| Changed RenderReplaced and subclasses' calcPrefWidths() to call |
| calcReplacedWidth(false) and then apply max-width only if it has a |
| fixed, positive value. |
| |
| * rendering/RenderBox.cpp: |
| (WebCore::RenderBox::calcReplacedWidth): |
| * rendering/RenderBox.h: |
| * rendering/RenderImage.cpp: |
| (WebCore::RenderImage::calcReplacedWidth): |
| (WebCore::RenderImage::calcPrefWidths): |
| * rendering/RenderImage.h: |
| * rendering/RenderReplaced.cpp: |
| (WebCore::RenderReplaced::calcPrefWidths): |
| * rendering/RenderSVGRoot.cpp: |
| (WebCore::RenderSVGRoot::calcPrefWidths): |
| * rendering/RenderVideo.cpp: |
| (WebCore::RenderVideo::calcReplacedWidth): |
| (WebCore::RenderVideo::calcPrefWidths): |
| * rendering/RenderVideo.h: |
| |
| 2008-08-18 Daniel Macks <dmacks@netspace.org> |
| |
| Reviewed by Mark Rowe. |
| |
| https://bugs.webkit.org/show_bug.cgi?id=20410 |
| More portable/self-documenting replacement for SIZE_MAX. |
| |
| * platform/network/curl/FormDataStreamCurl.cpp: |
| (WebCore::FormDataStream::read): |
| |
| 2008-08-18 Simon Fraser <simon.fraser@apple.com> |
| |
| Reviewed by Dave Hyatt |
| |
| Need to make sure we have an Animation in the AnimationList |
| before setting the initial value. |
| https://bugs.webkit.org/show_bug.cgi?id=20408 |
| |
| Test: fast/css/transition_shorthand_parsing.html |
| |
| * css/CSSStyleSelector.cpp: |
| |
| 2008-08-18 Kevin McCullough <kmccullough@apple.com> |
| |
| Reviewed by Tim. |
| |
| <rdar://problem/6150593> JSProfiler: Empty profiles disappear when there |
| is another profile. |
| |
| * page/inspector/ProfilesPanel.js: |
| |
| 2008-08-18 Kevin McCullough <kmccullough@apple.com> |
| |
| Reviewed by Geoff. |
| |
| <rdar://problem/6150642> REGRESSION: Closing the Web Inspector clears |
| all console messages |
| |
| * page/inspector/Console.js: |
| |
| 2008-08-18 Dirk Schulze <vbs85@gmx.de> |
| |
| Reviewed by Eric Seidel. |
| |
| Fixed Canvas for Cairo. Stroke and fill colors didn't work after |
| the canvas clean up. |
| |
| https://bugs.webkit.org/show_bug.cgi?id=20405 |
| |
| * html/CanvasRenderingContext2D.cpp: |
| (WebCore::CanvasRenderingContext2D::fill): |
| (WebCore::CanvasRenderingContext2D::stroke): |
| (WebCore::CanvasRenderingContext2D::fillRect): |
| |
| 2008-08-17 Timothy Hatcher <timothy@apple.com> |
| |
| Complete in scope variables in the Console when paused. |
| |
| https://bugs.webkit.org/show_bug.cgi?id=19115 |
| |
| Reviewed by Geoffrey Garen. |
| |
| * page/inspector/Console.js: |
| (WebInspector.Console.prototype.completions): If the expressionString |
| is null or empty and the debugger is paused, call variablesInScopeForSelectedCallFrame |
| to get an object that declares all the in scope variables. That way |
| "top level" expressions are completed. |
| * page/inspector/ScriptsPanel.js: |
| (WebInspector.ScriptsPanel.prototype.variablesInScopeForSelectedCallFrame): |
| Return an object that has all the variables that are in scope for the |
| selected call frame. The value of each property is just true. |
| The return object is useful for quick lookups or auto completion. |
| |
| 2008-08-17 Cameron Zwarich <cwzwarich@uwaterloo.ca> |
| |
| Not reviewed. |
| |
| Speculative Qt build fix. |
| |
| * bridge/qt/qt_runtime.cpp: |
| (KJS::Bindings::convertValueToQVariant): |
| (KJS::Bindings::QtRuntimeMethod::QtRuntimeMethod): |
| |
| 2008-08-17 Geoffrey Garen <ggaren@apple.com> |
| |
| Reviewed by Cameron Zwarich. |
| |
| Updated project files to XCode 3.1. |
| |
| * manual-tests/NPN_Invoke/NPN_Invoke.xcodeproj/project.pbxproj: |
| |
| 2008-08-17 Geoffrey Garen <ggaren@apple.com> |
| |
| Reviewed by Cameron Zwarich. |
| |
| Made room for a free word in JSCell. |
| |
| Changed JSDOMWindowBase to store its auxiliary data in a subclass of |
| JSGlobalData, so the two could share a pointer. |
| |
| Added a bunch of ASSERTs, to help catch over-sized objects. |
| |
| 2008-08-15 Mark Rowe <mrowe@apple.com> |
| |
| Reviewed by Dan Bernstein. |
| |
| Disable dead code stripping in debug builds. |
| |
| * Configurations/Base.xcconfig: |
| * WebCore.xcodeproj/project.pbxproj: |
| |
| 2008-08-15 Mark Rowe <mrowe@apple.com> |
| |
| Rubber-stamped by Geoff Garen. |
| |
| <rdar://problem/6139914> Please include a _debug version of JavaScriptCore framework |
| |
| * Configurations/Base.xcconfig: Factor out the debug-only settings so that they can shared |
| between the Debug configuration and debug Production variant. |
| * WebCore.xcodeproj/project.pbxproj: Enable the debug variant. |
| |
| 2008-08-15 Antti Koivisto <antti@apple.com> |
| |
| Reviewed by Anders. |
| |
| Don't start preloading body resources before the head is complete. This prevents |
| body preloads from slowing down initial display when there is limited amount |
| of bandwidth available. |
| |
| Works by queuing up found body preloads to DocLoader and only issuing them |
| after document has rendering. |
| |
| With bandwidth capped to 300kbit/s this speeds up cnn.com initial display by ~25% or 5s |
| without affecting complete load time. |
| |
| * html/PreloadScanner.cpp: |
| (WebCore::PreloadScanner::PreloadScanner): |
| (WebCore::PreloadScanner::scanningBody): |
| (WebCore::PreloadScanner::emitTag): |
| (WebCore::PreloadScanner::emitCSSRule): |
| * html/PreloadScanner.h: |
| * loader/DocLoader.cpp: |
| (WebCore::DocLoader::preload): |
| (WebCore::DocLoader::checkForPendingPreloads): |
| (WebCore::DocLoader::requestPreload): |
| * loader/DocLoader.h: |
| * loader/loader.cpp: |
| (WebCore::Loader::Host::didFinishLoading): |
| (WebCore::Loader::Host::didFail): |
| |
| 2008-08-15 Ada Chan <adachan@apple.com> |
| |
| Use item's computed style if the render style is 0 before falling back to the <select>'s style. |
| This way style set on an <hr> within a <select> will be honored. |
| |
| Reviewed by Dave Hyatt and Dan Bernstein. |
| |
| * rendering/RenderMenuList.cpp: |
| (WebCore::RenderMenuList::itemStyle): |
| |
| 2008-08-15 Antti Koivisto <antti@apple.com> |
| |
| Reviewed by Oliver. |
| |
| Some loader performance tweaks: |
| - Make stylesheets highest priority instead of scripts. We block script execution on stylesheets. |
| Especially if a stylesheet @imports other stylesheets it is important to get them to the front of the queue |
| to not delay rendering. |
| - Issue the first resource load for a host immediately even if the resource is low priority. TCP connection setup |
| can take long time when latency is high so it is good to get started early. |
| - When the document is fully parsed and stylesheets have been loaded there is no need to keep managing the |
| load queues. Issue remaining loads to the network layer. |
| |
| * loader/loader.cpp: |
| (WebCore::Loader::determinePriority): |
| (WebCore::Loader::load): |
| (WebCore::Loader::Host::servePendingRequests): |
| * loader/loader.h: |
| |
| 2008-08-15 Timothy Hatcher <timothy@apple.com> |
| |
| Detach the script debugger when the Web Inspector's window closes. |
| This has always been the intended design, but never fully implemented. |
| |
| https://bugs.webkit.org/show_bug.cgi?id=20402 |
| |
| Reviewed by Adam Roben. |
| |
| * page/InspectorController.cpp: |
| (WebCore::InspectorController::setWindowVisible): Call stopDebugging() |
| if the window is no longer visible. |
| |
| 2008-08-15 HÃ¥vard Wall <hwall@trolltech.com> |
| |
| Reviewed by Simon. |
| |
| Fixes: compile with QT_NO_CONTEXTMENU |
| |
| * platform/qt/PlatformMouseEventQt.cpp: |
| (WebCore::PlatformMouseEvent::PlatformMouseEvent): |
| * platform/qt/PlatformScrollBarQt.cpp: |
| (WebCore::PlatformScrollbar::handleMouseMoveEvent): |
| (WebCore::PlatformScrollbar::handleContextMenuEvent): |
| |
| 2008-08-15 HÃ¥vard Wall <hwall@trolltech.com> |
| |
| Reviewed by Simon. |
| |
| Fixes: compile with QT_NO_WHEELEVENT |
| |
| * platform/qt/WheelEventQt.cpp: |
| (WebCore::PlatformWheelEvent::PlatformWheelEvent): |
| |
| 2008-08-15 Keishi Hattori <casey.hattori@gmail.com> |
| |
| Fixed Bug 20210: Console groups are incorrect when closing and reopening the Inspector |
| |
| https://bugs.webkit.org/show_bug.cgi?id=20210 |
| |
| Reviewed by Tim Hatcher. |
| |
| * page/Console.cpp: |
| (WebCore::Console::group): |
| (WebCore::Console::groupEnd): |
| * page/Console.h: |
| (WebCore::): Removed GroupTitleMessageLevel. Added StartGroupMessaageLevel and EndGroupMessageLevel. |
| * page/InspectorController.cpp: |
| (WebCore::InspectorController::startGroup): Increments group level by |
| one and adds console message with StartGroupMessaageLevel. |
| (WebCore::InspectorController::endGroup): Decrements group level by one |
| and adds console message with EndGroupMessaageLevel. |
| * page/InspectorController.h: |
| * page/inspector/Console.js: |
| (WebInspector.Console.prototype.addMessage): Creates new ConsoleGroup |
| if the message is StartGroupMessaageLevel. |
| (WebInspector.ConsoleMessage.prototype.toMessageElement): |
| (WebInspector.ConsoleGroup.prototype.addMessage): |
| * page/inspector/inspector.js: |
| |
| 2008-08-15 Keishi Hattori <casey.hattori@gmail.com> |
| |
| Adds support for console.dir to the Inspector |
| |
| https://bugs.webkit.org/show_bug.cgi?id=19155 |
| |
| Reviewed by Tim Hatcher. |
| |
| * bindings/js/JSConsoleCustom.cpp: |
| (WebCore::JSConsole::dir): |
| * page/Console.cpp: |
| (WebCore::Console::dir): |
| * page/Console.h: Added ObjectMessageLevel. |
| * page/Console.idl: Added console.dir. |
| * page/inspector/Console.js: |
| (WebInspector.ConsoleMessage.prototypet.toMessageElement): Creates an |
| ObjectPropertiesSection if the MessageLevel is Object. |
| * page/inspector/ObjectPropertiesSection.js: "in" operator can't be |
| used on primitive data types. |
| * page/inspector/inspector.css: |
| |
| 2008-08-15 Keishi Hattori <casey.hattori@gmail.com> |
| |
| Adds support for clear() in the Inspector console. |
| |
| https://bugs.webkit.org/show_bug.cgi?id=19873 |
| |
| Reviewed by Tim Hatcher. |
| |
| * page/inspector/Console.js: |
| |
| 2008-08-15 Anthony Ricaud <rik24d@gmail.com> |
| |
| Cmd-F on Mac or Ctrl-F on other platforms now focus the search field. |
| |
| Platform distinction and modifier key matching adjusted |
| by Daniel Jalkut <jalkut@red-sweater.com> |
| |
| Bug 16313: text search (find) keybindings should work in the Web Inspector |
| https://bugs.webkit.org/show_bug.cgi?id=16313 |
| |
| Reviewed by Tim Hatcher. |
| |
| * page/inspector/inspector.js: Added a case for the F key |
| |
| 2008-08-15 Keishi Hattori <casey.hattori@gmail.com> |
| |
| Fix for error when the string doesn't contain a webkit-profile link. |
| |
| https://bugs.webkit.org/show_bug.cgi?id=20399 |
| |
| Reviewed by Tim Hatcher. |
| |
| * page/inspector/inspector.js: |
| |
| 2008-08-15 Timothy Hatcher <timothy@apple.com> |
| |
| Fixes two bugs where JavaScript could be executed from the page |
| while the debugger is paused. |
| |
| The first issue was JSLazyEventListener not checking the paused |
| state before parsing the code. |
| |
| The second issue was with the PageGroup version of |
| JavaScriptDebugServer::setJavaScriptPaused always passing false |
| to the Page version of JavaScriptDebugServer::setJavaScriptPaused, |
| and not the paused argument. |
| |
| https://bugs.webkit.org/show_bug.cgi?id=20284 |
| |
| Reviewed by Adam Roben. |
| |
| * bindings/js/JSEventListener.cpp: |
| (WebCore::JSLazyEventListener::parseCode): Check the paused |
| state of the ScriptController. Return early if paused. |
| * manual-tests/inspector/debugger-execution-while-paused.html: Added. |
| * page/JavaScriptDebugServer.cpp: |
| (WebCore::JavaScriptDebugServer::setJavaScriptPaused): |
| Pass the paused argument to the Page version of setJavaScriptPaused. |
| |
| 2008-08-15 Alexey Proskuryakov <ap@webkit.org> |
| |
| Reviewed by Geoff Garen. |
| |
| JSStringRef is created context-free, but can get linked to one via an identifier table, |
| breaking an implicit API contract. |
| |
| * page/InspectorController.cpp: |
| (WebCore::jsStringRef): |
| (WebCore::InspectorController::didParseSource): |
| (WebCore::InspectorController::failedToParseSource): |
| * page/JavaScriptProfile.cpp: |
| (WebCore::getTitleCallback): |
| Updated for JavaScriptCore changes. |
| |
| 2008-08-14 Kevin Ollivier <kevino@theolliviers.com> |
| |
| wx !USE(WXGC) build fix. This is necessary until we find a way to replace GDI with |
| an alternative that performs reasonably well. (GDI+ is too slow in many cases.) |
| |
| * platform/graphics/AffineTransform.h: |
| |
| 2008-08-14 Eric Carlson <eric.carlson@apple.com> |
| |
| Reviewed by Adele. |
| |
| Fix for https://bugs.webkit.org/show_bug.cgi?id=20388 |
| <video> elements on Windows never becomes visible when a page is restored from the cache |
| |
| Always pass "set" calls down to MediaPlayerPrivate instead of only when the |
| value is different from the cached value. Let the implementation decide when |
| to avoid work because nothing has changed. |
| |
| * platform/graphics/MediaPlayer.cpp: |
| (WebCore::MediaPlayer::setVolume): |
| (WebCore::MediaPlayer::setRate): |
| (WebCore::MediaPlayer::setRect): |
| (WebCore::MediaPlayer::setVisible): |
| |
| 2008-08-14 Keishi Hattori <casey.hattori@gmail.com> |
| |
| Make Firebug command line API respect predefined variables. |
| |
| https://bugs.webkit.org/show_bug.cgi?id=20385 |
| |
| Reviewed by Tim Hatcher. |
| |
| * page/inspector/Console.js: |
| |
| 2008-08-14 Sam Weinig <sam@webkit.org> |
| |
| Reviewed by Geoffrey Garen and Timothy Hatcher. |
| |
| Allow programatically setting the HTMLTokenizers time delay and chunk size |
| which are used for determining how aggressively we yield. |
| |
| * WebCore.base.exp: |
| * html/HTMLTokenizer.cpp: |
| (WebCore::HTMLTokenizer::begin): |
| (WebCore::HTMLTokenizer::continueProcessing): |
| * html/HTMLTokenizer.h: |
| * page/Page.cpp: |
| (WebCore::Page::Page): |
| (WebCore::Page::setCustomHTMLTokenizerTimeDelay): |
| (WebCore::Page::setCustomHTMLTokenizerChunkSize): |
| * page/Page.h: |
| (WebCore::Page::hasCustomHTMLTokenizerTimeDelay): |
| (WebCore::Page::customHTMLTokenizerTimeDelay): |
| (WebCore::Page::hasCustomHTMLTokenizerChunkSize): |
| (WebCore::Page::customHTMLTokenizerChunkSize): |
| |
| 2008-08-14 Eric Seidel <eric@webkit.org> |
| |
| Reviewed by Beth. |
| |
| Move us one step closer to cross-platform svg/graphics code |
| |
| * WebCore.xcodeproj/project.pbxproj: |
| * html/CanvasStyle.cpp: |
| * platform/graphics/Color.cpp: |
| (WebCore::colorWithOverrideAlpha): |
| * platform/graphics/Color.h: |
| * svg/graphics/cg/CgSupport.cpp: |
| (WebCore::applyStrokeStyleToContext): |
| (WebCore::strokeBoundingBox): |
| * svg/graphics/cg/SVGPaintServerSolidCg.cpp: |
| (WebCore::SVGPaintServerSolid::setup): |
| |
| 2008-08-14 Eric Seidel <eric@webkit.org> |
| |
| Reviewed by Alexey. |
| |
| Remove un-need includes from HTMLCanvas and use the |
| Gradient platform abstraction in one place in CanvasStyle |
| |
| * html/CanvasRenderingContext2D.cpp: |
| (WebCore::CanvasRenderingContext2D::fillRect): |
| * html/CanvasStyle.cpp: |
| * html/HTMLCanvasElement.cpp: |
| |
| 2008-08-14 Eric Seidel <eric@webkit.org> |
| |
| Reviewed by Alexey. |
| |
| Clean up GlyphBuffer.h, removing more #ifdefs |
| |
| * platform/graphics/GlyphBuffer.h: |
| (WebCore::GlyphBuffer::glyphAt): |
| (WebCore::GlyphBuffer::advanceAt): |
| (WebCore::GlyphBuffer::add): |
| |
| 2008-08-14 Eric Seidel <eric@webkit.org> |
| |
| Reviewed by Sam. |
| |
| Clean up AffineTransform.h, removing #ifdefs |
| |
| * platform/graphics/AffineTransform.h: |
| * platform/graphics/cairo/AffineTransformCairo.cpp: |
| * platform/graphics/cg/AffineTransformCG.cpp: |
| * platform/graphics/qt/AffineTransformQt.cpp: |
| * platform/graphics/wx/AffineTransformWx.cpp: |
| |
| 2008-08-14 Dan Bernstein <mitz@apple.com> |
| |
| - fix non-CG builds by adding an ImageSource::frameSizeAtIndex() that returns size(). |
| |
| * platform/graphics/cairo/ImageSourceCairo.cpp: |
| (WebCore::ImageSource::frameSizeAtIndex): |
| * platform/graphics/qt/ImageSourceQt.cpp: |
| (WebCore::ImageSource::frameSizeAtIndex): |
| * platform/graphics/wx/ImageSourceWx.cpp: |
| (WebCore::ImageSource::frameSizeAtIndex): |
| |
| 2008-08-14 Dan Bernstein <mitz@apple.com> |
| |
| Reviewed by Brady Eidson. |
| |
| - fix <rdar://problem/5993323> REGRESSION (r34210): Apple.com favicon appears stretched/clipped |
| |
| * platform/graphics/BitmapImage.cpp: |
| (WebCore::BitmapImage::BitmapImage): Added initialization of |
| m_hasUniformFrameSize. |
| (WebCore::BitmapImage::cacheFrame): Added code to get the size of the |
| cached frame for use in decoded size computation and for setting |
| m_hasUniformFrameSize. |
| (WebCore::BitmapImage::currentFrameSize): Added. |
| (WebCore::BitmapImage::dataChanged): Added code to reset |
| m_hasUniformFrameSize. |
| * platform/graphics/BitmapImage.h: Added currentFrameSize() and |
| m_hasUniformFrameSize. |
| * platform/graphics/ImageSource.h: Added frameSizeAtIndex(). |
| * platform/graphics/cg/ImageCG.cpp: |
| (WebCore::BitmapImage::draw): Changed to use currentFrameSize(). This |
| fixes the bug, which resulted from assuming that the frame being drawn |
| was the same size as the first frame. |
| * platform/graphics/cg/ImageSourceCG.cpp: |
| (WebCore::ImageSource::frameSizeAtIndex): Renamed size() to this and |
| changed to get the size of the frame at the given index. |
| (WebCore::ImageSource::size): Added. Returns frameSizeAtIndex(0). |
| |
| 2008-08-13 Simon Fraser <simon.fraser@apple.com> |
| |
| Reviewed by Eric Seidel |
| |
| Fix @font-face inside @media rule crash. |
| https://bugs.webkit.org/show_bug.cgi?id=20367 |
| |
| Test: fast/css/font-face-in-media-rule.html |
| |
| * css/CSSStyleSelector.cpp: |
| (WebCore::CSSRuleSet::addRulesFromSheet): |
| |
| 2008-08-14 Kevin Ollivier <kevino@theolliviers.com> |
| |
| wx build fixes after recent changes to Canvas and Image classes. |
| |
| * platform/graphics/Pattern.h: |
| * platform/graphics/wx/GradientWx.cpp: |
| (WebCore::Gradient::fill): |
| * platform/graphics/wx/GraphicsContextWx.cpp: |
| (WebCore::GraphicsContext::applyFillPattern): |
| (WebCore::GraphicsContext::applyStrokePattern): |
| * platform/graphics/wx/ImageBufferWx.cpp: |
| (WebCore::ImageBuffer::image): |
| * platform/graphics/wx/ImageWx.cpp: |
| (WebCore::Image::loadPlatformResource): |
| |
| 2008-08-14 Maxime Britto <britto@apple.com> |
| |
| Reviewed by Sam Weinig. |
| |
| rdar://6102511 |
| When pan-scrolling, typing on the keyboard should either stop the pan scroll or be ignored |
| IE and FF are both preventing the keyboard event to interact with the page while scrolling. |
| Some differences exist between them concerning the kind of key which is pressed : |
| IE7 : every key leads to a stop of the panning |
| FF3 : the ESC and TAB keys leads to a stop, the other keys are inactive. |
| For WebKit this patch is adopting the FF3 behavior except for the TAB key which is inactive too. |
| |
| * page/EventHandler.cpp: |
| (WebCore::EventHandler::keyEvent): Verifies which key has been hit and decide either to stop the pan scroll or to swallow the key event. |
| |
| 2008-08-14 Christian Dywan <christian@twotoasts.de> |
| |
| Gtk+/ Cairo build fix, patch by Dirk Schulze. |
| |
| * html/CanvasStyle.cpp: |
| * platform/graphics/cairo/PatternCairo.cpp: |
| (WebCore::Pattern::createPlatformPattern): |
| * svg/graphics/cairo/SVGResourceMaskerCairo.cpp: |
| |
| 2008-08-14 Kevin McCullough <kmccullough@apple.com> |
| |
| Reviewed by Tim. |
| |
| <rdar://problem/6115819> Notify of profile in console |
| |
| * page/InspectorController.cpp: |
| (WebCore::InspectorController::addProfile): |
| (WebCore::InspectorController::addProfileMessageToConsole): Called by |
| addProfile this is the function that adds a message to the console that |
| a profile finished. |
| * page/InspectorController.h: |
| * page/JavaScriptProfile.cpp: Expose the profiler's unique ID to match |
| the console log to the profile in the web inspector. |
| (WebCore::getUniqueIdCallback): |
| (WebCore::ProfileClass): |
| * page/inspector/ProfilesPanel.js: Created a map of all the profiles by |
| Id to bring up the requested profile. Also select and reveal the |
| profile in the profile panel. And created displayTitleForProfileLink() |
| which formats a title taking into account if it's user initiated or if |
| there are multiples. Lasty, I put the user initiated profile in a |
| variable. |
| * page/inspector/inspector.js: Make the profile title be a clickable |
| link that will take the user to the identified profile. Also expose |
| the count of user initiated profiles so they can be displayed in the |
| console with the correct count. |
| |
| 2008-08-14 Timothy Hatcher <timothy@apple.com> |
| |
| Avoid formating ConsoleMessages twice unless the message will be |
| displayed in bubbles of a SourceFrame. |
| |
| Reviewed by Kevin McCullough. |
| |
| * page/inspector/Console.js: |
| (WebInspector.ConsoleMessage): Only format the plain text message |
| if the URL and line are valid and the level is error or warning. |
| (WebInspector.ConsoleMessage.prototype.isErrorOrWarning): Added. |
| Helper to test for error or warning level. |
| * page/inspector/SourceFrame.js: |
| (WebInspector.SourceFrame.prototype.addMessage): Don't add the |
| message if there is no message or valid line or if the msg |
| isn't an error or warning. |
| |
| 2008-08-14 Jan Michael Alonzo <jmalonzo@webkit.org> |
| |
| partial Gtk build fix, not reviewed |
| |
| * platform/graphics/cairo/PatternCairo.cpp: |
| |
| 2008-08-13 Sam Weinig <sam@webkit.org> |
| |
| Reviewed by Anders Carlson. |
| |
| Fix style issue. |
| |
| * html/HTMLMediaElement.cpp: |
| (WebCore::HTMLMediaElement::pickMedia): |
| |
| 2008-08-13 Nikolas Zimmermann <zimmermann@kde.org> |
| |
| Build fix for Cairo, not reviewed. (exposed by gtk build slave) |
| Continue Erics build fixes, after the Image cleanup. |
| |
| * platform/graphics/cairo/ImageBufferCairo.cpp: |
| (WebCore::ImageBuffer::image): |
| |
| 2008-08-13 Nikolas Zimmermann <zimmermann@kde.org> |
| |
| Build fix for Qt, not reviewed. |
| Don't declare eventuallyMarkAsParserCreated in a block wrapped by !USE_QXMLSTREAM. |
| |
| * dom/XMLTokenizer.cpp: |
| (WebCore::eventuallyMarkAsParserCreated): Was erre |
| |
| 2008-08-13 Nikolas Zimmermann <zimmermann@kde.org> |
| |
| Build fix, not reviewed. |
| Add ScriptElement.cpp to Gtk build. |
| |
| * GNUmakefile.am: |
| |
| 2008-08-13 Nikolas Zimmermann <zimmermann@kde.org> |
| |
| Reviewed by Eric. |
| |
| Fixes: https://bugs.webkit.org/show_bug.cgi?id=20372 |
| |
| Refactor HTMLScriptElement's code into a common base class: ScriptElement. |
| SVGScriptElement will be converted to use ScriptElement in a follow-up patch. |
| |
| This resolves code duplications and allows us to completly replace the old |
| SVGScriptElement (which doesn't use CachedScript, no dynamic injected scripts etc..) |
| |
| As ScriptElement, doesn't actually inherit from Element, we may want to rename |
| it, though StyleElement uses the same naming convention, so I left it as is for now. |
| Eventually we'll rename both files in future. |
| |
| No functional changes yet, as SVGScriptElement doesn't yet use the new base class. |
| |
| * WebCore.pro: Add new ScriptElement.cpp to build. |
| * WebCore.vcproj/WebCore.vcproj: Ditto. |
| * WebCore.xcodeproj/project.pbxproj: Ditto. |
| * WebCoreSources.bkl: Dutto. |
| * dom/ScriptElement.cpp: Added. 1:1 based on HTMLScriptElement |
| (WebCore::ScriptElement::insertedIntoDocument): |
| (WebCore::ScriptElement::removedFromDocument): |
| (WebCore::ScriptElement::childrenChanged): |
| (WebCore::ScriptElement::finishParsingChildren): |
| (WebCore::ScriptElement::handleSourceAttribute): |
| (WebCore::isSupportedJavaScriptLanguage): |
| (WebCore::ScriptElementData::ScriptElementData): |
| (WebCore::ScriptElementData::~ScriptElementData): |
| (WebCore::ScriptElementData::requestScript): |
| (WebCore::ScriptElementData::evaluateScript): |
| (WebCore::ScriptElementData::stopLoadRequest): |
| (WebCore::ScriptElementData::notifyFinished): |
| (WebCore::ScriptElementData::ignoresLoadRequest): |
| (WebCore::ScriptElementData::shouldExecuteAsJavaScript): |
| (WebCore::ScriptElementData::scriptCharset): |
| (WebCore::ScriptElementData::scriptContent): |
| * dom/ScriptElement.h: Added. |
| (WebCore::ScriptElement::ScriptElement): |
| (WebCore::ScriptElement::~ScriptElement): |
| (WebCore::ScriptElementData::element): |
| (WebCore::ScriptElementData::createdByParser): |
| (WebCore::ScriptElementData::setCreatedByParser): |
| * dom/XMLTokenizer.cpp: |
| (WebCore::isScriptElement): |
| (WebCore::castToScriptElement): |
| (WebCore::eventuallyMarkAsParserCreated): |
| (WebCore::XMLTokenizer::startElementNs): |
| (WebCore::XMLTokenizer::endElementNs): |
| (WebCore::createXHTMLParserErrorHeader): |
| (WebCore::XMLTokenizer::insertErrorMessageBlock): |
| * html/HTMLScriptElement.cpp: Refactored code, pushed most code down to ScriptElement. |
| (WebCore::HTMLScriptElement::HTMLScriptElement): |
| (WebCore::HTMLScriptElement::~HTMLScriptElement): |
| (WebCore::HTMLScriptElement::isURLAttribute): |
| (WebCore::HTMLScriptElement::setCreatedByParser): |
| (WebCore::HTMLScriptElement::shouldExecuteAsJavaScript): |
| (WebCore::HTMLScriptElement::childrenChanged): |
| (WebCore::HTMLScriptElement::parseMappedAttribute): |
| (WebCore::HTMLScriptElement::finishParsingChildren): |
| (WebCore::HTMLScriptElement::insertedIntoDocument): |
| (WebCore::HTMLScriptElement::removedFromDocument): |
| (WebCore::HTMLScriptElement::text): |
| (WebCore::HTMLScriptElement::setText): |
| (WebCore::HTMLScriptElement::setHtmlFor): |
| (WebCore::HTMLScriptElement::setEvent): |
| (WebCore::HTMLScriptElement::charset): |
| (WebCore::HTMLScriptElement::src): |
| (WebCore::HTMLScriptElement::type): |
| (WebCore::HTMLScriptElement::scriptCharset): |
| (WebCore::HTMLScriptElement::scriptContent): |
| (WebCore::HTMLScriptElement::sourceAttributeValue): |
| (WebCore::HTMLScriptElement::charsetAttributeValue): |
| (WebCore::HTMLScriptElement::typeAttributeValue): |
| (WebCore::HTMLScriptElement::languageAttributeValue): |
| (WebCore::HTMLScriptElement::dispatchLoadEvent): |
| (WebCore::HTMLScriptElement::dispatchErrorEvent): |
| * html/HTMLScriptElement.h: |
| * svg/SVGScriptElement.cpp: Inherit from ScriptElement, don't actually use it yet. |
| * svg/SVGScriptElement.cpp: |
| (WebCore::SVGScriptElement::SVGScriptElement): |
| (WebCore::SVGScriptElement::scriptContent): |
| (WebCore::SVGScriptElement::sourceAttributeValue): |
| (WebCore::SVGScriptElement::charsetAttributeValue): |
| (WebCore::SVGScriptElement::typeAttributeValue): |
| (WebCore::SVGScriptElement::languageAttributeValue): |
| (WebCore::SVGScriptElement::dispatchLoadEvent): |
| (WebCore::SVGScriptElement::dispatchErrorEvent): |
| * svg/SVGScriptElement.h: |
| (WebCore::SVGScriptElement::setCreatedByParser): |
| |
| 2008-08-13 Eric Seidel <eric@webkit.org> |
| |
| Attempt to fix the Gtk build, no review. |
| |
| I removed the bogus GraphicsContext::translatePoint() hack for Gtk in the process. |
| |
| * platform/graphics/GraphicsContext.h: |
| * platform/graphics/cairo/GraphicsContextCairo.cpp: |
| * platform/graphics/qt/GradientQt.cpp: |
| * platform/graphics/qt/GraphicsContextQt.cpp: |
| * platform/gtk/RenderThemeGtk.cpp: |
| (WebCore::paintMozWidget): |
| |
| 2008-08-13 Eric Seidel <eric@webkit.org> |
| |
| Yet another attempt to fix the Qt build, no review. |
| |
| * platform/graphics/qt/GradientQt.cpp: |
| * platform/graphics/qt/GraphicsContextQt.cpp: |
| (WebCore::GraphicsContext::applyStrokePattern): |
| (WebCore::GraphicsContext::applyFillPattern): |
| |
| 2008-08-13 Eric Seidel <eric@webkit.org> |
| |
| Attempt to fix the Windows build, no review. |
| |
| * platform/win/CursorWin.cpp: |
| (WebCore::loadCursorByName): |
| * platform/win/ScrollViewWin.cpp: |
| (WebCore::ScrollView::paint): |
| |
| 2008-08-13 Eric Seidel <eric@webkit.org> |
| |
| Attempt to fix the Windows build, no review. |
| |
| * platform/win/CursorWin.cpp: |
| (WebCore::loadCursorByName): |
| * platform/win/ScrollViewWin.cpp: |
| (WebCore::ScrollView::paint): |
| |
| 2008-08-13 Eric Seidel <eric@webkit.org> |
| |
| Attempt to fix the Qt build, no review. |
| |
| * platform/graphics/qt/ImageBufferQt.cpp: |
| (WebCore::ImageBuffer::image): |
| |
| 2008-08-13 Eric Seidel <eric@webkit.org> |
| |
| Attempt to fix the Windows build, no review. |
| |
| * platform/graphics/win/ImageWin.cpp: |
| (WebCore::Image::loadPlatformResource): |
| * plugins/win/PluginViewWin.cpp: |
| (WebCore::PluginView::paintMissingPluginIcon): |
| |
| 2008-08-13 Eric Seidel <eric@webkit.org> |
| |
| Another attempt to fix the Qt build, no review. |
| |
| * platform/graphics/qt/ImageQt.cpp: |
| * platform/graphics/qt/StillImageQt.h: |
| |
| 2008-08-13 Eric Seidel <eric@webkit.org> |
| |
| No review, build fix only. |
| |
| Fix mac build, due to change in new code since my patch was written. |
| |
| * svg/graphics/cg/SVGResourceMaskerCg.mm: |
| (WebCore::SVGResourceMasker::applyMask): |
| |
| 2008-08-13 Eric Seidel <eric@webkit.org> |
| |
| Build fix only, no review. |
| |
| Attempt to fix the Qt build. |
| |
| * platform/graphics/qt/ImageBufferQt.cpp: |
| * platform/graphics/qt/StillImageQt.h: |
| (WebCore::StillImage::create): |
| (WebCore::StillImage::destroyDecodedData): |
| (WebCore::StillImage::decodedSize): |
| |
| 2008-08-13 Eric Seidel <eric@webkit.org> |
| |
| Build fix only, no review. |
| |
| Attempt to fix the Qt build. |
| |
| * html/CanvasStyle.cpp: |
| (WebCore::CanvasStyle::applyStrokeColor): |
| (WebCore::CanvasStyle::applyFillColor): |
| |
| 2008-08-13 Eric Seidel <eric@webkit.org> |
| |
| Reviewed by Sam. |
| |
| Match HTML5 spec by throwing INVALID_STATE_ERR when |
| createPattern is called and the HTMLImageElement is not |
| yet done loading the image (!isComplete) |
| https://bugs.webkit.org/show_bug.cgi?id=20351 |
| |
| Test: http/misc/canvas-pattern-from-incremental-image.html |
| |
| * html/CanvasRenderingContext2D.cpp: |
| (WebCore::CanvasRenderingContext2D::createPattern): |
| |
| 2008-08-13 Eric Seidel <eric@webkit.org> |
| |
| Reviewed by Niko. |
| |
| Split out a Pattern class from CanvasPattern |
| and remove all the Pattern-related #ifdefs |
| (This will break non-mac platforms! I will fix them.) |
| https://bugs.webkit.org/show_bug.cgi?id=20351 |
| |
| * GNUmakefile.am: |
| * WebCore.pro: |
| * WebCore.vcproj/WebCore.vcproj: |
| * WebCore.xcodeproj/project.pbxproj: |
| * WebCoreSources.bkl: |
| * html/CanvasPattern.cpp: |
| (WebCore::CanvasPattern::parseRepetitionType): |
| (WebCore::CanvasPattern::CanvasPattern): |
| * html/CanvasPattern.h: |
| (WebCore::CanvasPattern::create): |
| (WebCore::CanvasPattern::pattern): |
| (WebCore::CanvasPattern::originClean): |
| * html/CanvasRenderingContext2D.cpp: |
| (WebCore::CanvasRenderingContext2D::createPattern): |
| (WebCore::CanvasRenderingContext2D::applyStrokePattern): |
| (WebCore::CanvasRenderingContext2D::applyFillPattern): |
| * html/CanvasRenderingContext2D.h: |
| * html/HTMLCanvasElement.cpp: |
| * html/HTMLCanvasElement.h: |
| * loader/FrameLoader.cpp: |
| (WebCore::FrameLoader::createHistoryItem): |
| * platform/graphics/GraphicsContext.h: |
| * platform/graphics/cairo/GraphicsContextCairo.cpp: |
| (WebCore::GraphicsContext::applyStrokePattern): |
| (WebCore::GraphicsContext::applyFillPattern): |
| * platform/graphics/cg/GraphicsContextCG.cpp: |
| (WebCore::GraphicsContext::clipToImageBuffer): |
| (WebCore::GraphicsContext::applyStrokePattern): |
| (WebCore::GraphicsContext::applyFillPattern): |
| |
| 2008-08-13 Eric Seidel <eric@webkit.org> |
| |
| Reviewed by Niko. |
| |
| Make Images RefCounted (and clean up callers) |
| https://bugs.webkit.org/show_bug.cgi?id=20351 |
| |
| * editing/DeleteButtonController.cpp: |
| (WebCore::DeleteButtonController::createDeletionUI): |
| * loader/CachedImage.cpp: |
| (WebCore::CachedImage::CachedImage): |
| (WebCore::brokenImage): |
| (WebCore::nullImage): |
| (WebCore::CachedImage::image): |
| (WebCore::CachedImage::notifyObservers): |
| (WebCore::CachedImage::createImage): |
| * loader/CachedImage.h: |
| * loader/icon/IconRecord.cpp: |
| (WebCore::IconRecord::setImageData): |
| (WebCore::IconRecord::loadImageFromResource): |
| * loader/icon/IconRecord.h: |
| * platform/graphics/BitmapImage.h: |
| (WebCore::BitmapImage::create): |
| * platform/graphics/GeneratedImage.h: |
| * platform/graphics/Gradient.cpp: |
| * platform/graphics/Gradient.h: |
| * platform/graphics/Image.cpp: |
| (WebCore::Image::nullImage): |
| * platform/graphics/Image.h: |
| * platform/graphics/ImageBuffer.h: |
| * platform/graphics/cairo/ImageBufferCairo.cpp: |
| (WebCore::ImageBuffer::image): |
| * platform/graphics/cg/ImageBufferCG.cpp: |
| (WebCore::ImageBuffer::image): |
| (WebCore::ImageBuffer::getImageData): |
| (WebCore::ImageBuffer::putImageData): |
| * platform/graphics/cg/PDFDocumentImage.h: |
| (WebCore::PDFDocumentImage::create): |
| (WebCore::PDFDocumentImage::destroyDecodedData): |
| (WebCore::PDFDocumentImage::decodedSize): |
| * platform/graphics/gtk/ImageGtk.cpp: |
| (WebCore::Image::loadPlatformResource): |
| * platform/graphics/mac/ImageMac.mm: |
| (WebCore::Image::loadPlatformResource): |
| * rendering/RenderImage.cpp: |
| * rendering/RenderLayer.cpp: |
| (WebCore::RenderLayer::paintOverflowControls): |
| * svg/graphics/SVGImage.cpp: |
| (WebCore::SVGImage::nativeImageForCurrentFrame): |
| * svg/graphics/SVGImage.h: |
| (WebCore::SVGImage::create): |
| (WebCore::SVGImage::destroyDecodedData): |
| (WebCore::SVGImage::decodedSize): |
| (WebCore::SVGImage::frameAtIndex): |
| * svg/graphics/cairo/SVGPaintServerPatternCairo.cpp: |
| (WebCore::SVGPaintServerPattern::setup): |
| * svg/graphics/cairo/SVGResourceMaskerCairo.cpp: |
| (WebCore::SVGResourceMasker::applyMask): |
| * svg/graphics/cg/SVGPaintServerPatternCg.cpp: |
| (WebCore::patternCallback): |
| * svg/graphics/cg/SVGResourceMaskerCg.mm: |
| (WebCore::SVGResourceMasker::applyMask): |
| |
| 2008-08-13 Eric Seidel <eric@webkit.org> |
| |
| Reviewed by Niko. |
| |
| Remove #ifdefs from CanvasStyle by using platform/Color.h |
| https://bugs.webkit.org/show_bug.cgi?id=20351 |
| |
| There are some down-sides to this commit. |
| This commit limits us to 255 levels of grey for calls like: |
| context.setStrokeStyle(.37, 1.0) |
| previously CG might have used up to 32bits to store the grey level |
| Since setStrokeStyle is not part of HTML5, I don't suspect the web will notice. |
| |
| Likewise, setStrokeStyle/setFillStyle calls which used float colors are now limited |
| to RGBA32 (like all the rest of colors in WebCore), thus: |
| context.setStrokStyle(.37, .24, .456, .99) will now have the same precision as: |
| context.strokeStyle = "rgba(.37, .24, .456, .99)", which is to say RGBA32 |
| |
| If this is a problem for Dashboard, we can either roll out this commit |
| or add a beefier Color abstraction, which can be used internally by GraphicsContext |
| when keeping state, and then GraphicsContext can grow some additional set* routines |
| for setting the a grey/float/whatever fill and stroke. |
| |
| * html/CanvasRenderingContext2D.cpp: |
| (WebCore::CanvasRenderingContext2D::setFillStyle): |
| * html/CanvasStyle.cpp: |
| (WebCore::CanvasStyle::CanvasStyle): |
| (WebCore::colorWithOverrideAlpha): |
| (WebCore::CanvasStyle::applyStrokeColor): |
| (WebCore::CanvasStyle::applyFillColor): |
| * html/CanvasStyle.h: |
| * platform/graphics/Color.cpp: |
| (WebCore::colorFloatToRGBAByte): |
| (WebCore::makeRGBA32FromFloats): |
| * platform/graphics/Color.h: |
| |
| 2008-08-13 Eric Carlson <eric.carlson@apple.com> |
| |
| Reviewed by Adele Peterson. |
| |
| Fix for <rdar://problem/6137931> |
| https://bugs.webkit.org/show_bug.cgi?id=20360 |
| Remove all parameters from the MIME type before checking with the MIME type registry |
| |
| Tests: media/video-source-type-params.html |
| |
| * html/HTMLMediaElement.cpp: |
| (WebCore::HTMLMediaElement::pickMedia): only pass the portion before the first ';' |
| to isSupportedMediaMIMEType() |
| |
| 2008-08-13 Beth Dakin <bdakin@apple.com> |
| |
| Reviewed by Sam Weinig. |
| |
| Fix for <rdar://problem/6141345> |
| |
| This patch refines findString and markAllMatchesForText functions' |
| interactions with disconnected frames. They no longer rely on |
| knowing where a range is relative to the visible region and work |
| with disconnected frames that contain frames. |
| |
| * editing/Editor.cpp: |
| (WebCore::Editor::insideVisibleArea): Now returns a bool instead of |
| the visiblity enum. |
| (WebCore::Editor::firstVisibleRange): This now returns the very |
| first visible range in the document. It's no longer dependent on |
| searching forward. |
| (WebCore::Editor::lastVisibleRange): This now returns the very last |
| visible range in the document. It's no longer dependent on |
| searching backwards. |
| (WebCore::Editor::nextVisibleRange): This returns the next visible |
| range in the appropriate direction from the current range. |
| * editing/Editor.h: |
| * page/Frame.cpp: |
| (WebCore::Frame::findString): |
| (WebCore::Frame::markAllMatchesForText): |
| |
| 2008-08-13 Kevin Ollivier <kevino@theolliviers.com> |
| |
| wx build fix for case-sensitive platforms, like Linux. |
| |
| * WebCoreSources.bkl: |
| |
| 2008-08-13 Marco Barisione <marco.barisione@collabora.co.uk> |
| |
| Reviewed by Holger Freyther. |
| |
| http://bugs.webkit.org/show_bug.cgi?id=16881 |
| [GTK] PlatformScreenGtk is unimplemented |
| |
| Original patch by Christian Dywan. |
| |
| * platform/gtk/PlatformScreenGtk.cpp: |
| (WebCore::screenDepth): |
| (WebCore::screenDepthPerComponent): |
| (WebCore::screenIsMonochrome): |
| (WebCore::screenRect): |
| (WebCore::screenAvailableRect): |
| |
| 2008-08-13 Jan Michael Alonzo <jmalonzo@webkit.org> |
| |
| Reviewed by Holger Freyther. |
| |
| http://bugs.webkit.org/show_bug.cgi?id=20318 |
| SharedTimerGtk should use G_PRIORITY_DEFAULT_IDLE for g_idle_add |
| |
| g_idle_add is the same as g_idle_add_full with a priority of |
| G_PRIORITY_DEFAULT_IDLE, so we can safely use that. |
| |
| * platform/gtk/SharedTimerGtk.cpp: |
| (WebCore::setSharedTimerFireTime): |
| |
| 2008-08-13 Timothy Hatcher <timothy@apple.com> |
| |
| Changed the InspectorController so it can be notified when the |
| attached state of the Inspector changes in WebKit. |
| |
| Reviewed by Kevin McCullough. |
| |
| * WebCore.base.exp: Updated the symbol for setWindowVisible. |
| * page/InspectorController.cpp: |
| (WebCore::InspectorController::setWindowVisible): Added an attached argument, |
| that defaults to false.Call setAttachedWindow with the attached argument. |
| (WebCore::InspectorController::setAttachedWindow): Call the script version |
| of setAttachedWindow. |
| * page/InspectorController.h: |
| * page/inspector/inspector.js: |
| (WebInspector.setAttachedWindow): Set the attached property. |
| |
| 2008-08-12 Timothy Hatcher <timothy@apple.com> |
| |
| Added a close button to the Inspector's toolbar when docked. |
| |
| https://bugs.webkit.org/show_bug.cgi?id=14270 |
| |
| Reviewed by Kevin McCullough. |
| |
| * page/InspectorController.cpp: |
| (WebCore::closeWindow): Call InspectorController::closeWindow. |
| (WebCore::InspectorController::windowScriptObjectAvailable): |
| Added closeWindow to the script class. |
| * page/InspectorController.h: |
| * page/inspector/Images/closeButtons.png: Added. |
| * page/inspector/inspector.css: Added and changed styles. |
| * page/inspector/inspector.html: Added the close button. |
| * page/inspector/inspector.js: |
| (WebInspector.loaded): Added click event listener to the close button. |
| (WebInspector.close): Call InspectorController.closeWindow. |
| |
| 2008-08-12 Timothy Hatcher <timothy@apple.com> |
| |
| Make the docked Web Inspector resizable. This is the cross platform |
| portion of the code. Each InspectorClient needs to implement the |
| real resize code. |
| |
| https://bugs.webkit.org/show_bug.cgi?id=14282 |
| |
| Reviewed by Kevin McCullough. |
| |
| * loader/EmptyClients.h: Added an empty setAttachedWindowHeight. |
| * page/InspectorClient.h: Added setAttachedWindowHeight. |
| * page/InspectorController.cpp: |
| (WebCore::setAttachedWindowHeight): Call setAttachedWindowHeight |
| on the InspectorController. |
| (WebCore::InspectorController::setAttachedWindowHeight): Call |
| setAttachedWindowHeight on the client. |
| (WebCore::InspectorController::windowScriptObjectAvailable): |
| Added setAttachedWindowHeight to the script class. |
| * page/InspectorController.h: |
| * page/inspector/inspector.css: Make the cursor on the toolbar be |
| row-resize when docked. |
| * page/inspector/inspector.js: |
| (WebInspector.loaded): Always add the toolbarDragStart event listener. |
| (WebInspector.toolbarDragStart): Return early if we are not attached |
| and not on Leopard. Call WebInspector.elementDragStart. |
| (WebInspector.toolbarDragEnd): Call WebInspector.elementDragEnd. |
| (WebInspector.toolbarDrag): When attached call setAttachedWindowHeight, |
| otherwise call moveByUnrestricted. |
| |
| 2008-08-13 Simon Hausmann <hausmann@webkit.org> |
| |
| Reviewed by Holger. |
| |
| Initialize m_zoomsTextOnly in the Settings constructor. |
| |
| * page/Settings.cpp: |
| (WebCore::Settings::Settings): |
| |
| 2008-08-13 Brad Hughes <bhughes@trolltech.com> |
| |
| Reviewed by Simon. |
| |
| Fix compiling of QtWebKit in release mode with the Intel C++ Compiler for Linux |
| |
| The latest upgrade of the intel compiler allows us to compile all of |
| Qt with optimizations enabled (yay!). |
| |
| * WebCore.pro: |
| |
| 2008-08-13 Prasanth Ullattil <prasanth.ullattil@trolltech.com> |
| |
| Reviewed by Simon. |
| |
| Fix QtWebKit not displaying content on 403 HTTP responses |
| |
| Just like with 404 responses also display content with 403, as |
| used by http://audiio.ejamming.proteus-tech.com/audiio/profile/original_signup/ |
| |
| * platform/network/qt/QNetworkReplyHandler.cpp: |
| (WebCore::QNetworkReplyHandler::finish): |
| |
| 2008-08-13 Simon Hausmann <hausmann@webkit.org> |
| |
| Reviewed by Holger. |
| |
| Qt part of https://bugs.webkit.org/show_bug.cgi?id=18994 |
| |
| Make the formatting of String::format() locale independent through the use of QString::vsprintf. |
| |
| * platform/text/String.cpp: |
| (WebCore::String::format): |
| |
| 2008-08-13 Simon Hausmann <hausmann@webkit.org> |
| |
| Reviewed by Lars. |
| |
| Fix QWebFrame::setHtml() not setting the new contents immediately. |
| |
| Added a setter to the DocumentLoader to toggle the deferred loading of the main |
| resource when it comes from substitute data. |
| |
| Disable deferred loading of the main resource when we have valid substitute data, |
| as used by QWebFrame::setHtml. |
| |
| * loader/DocumentLoader.h: |
| |
| 2008-08-13 Mark Rowe <mrowe@apple.com> |
| |
| Speculative GTK build fix. |
| |
| * GNUmakefile.am: Add dependency info for JSSVGElementWrapperFactory.cpp. |
| |
| 2008-08-13 Thiago Macieira <tjmaciei@trolltech.com> |
| |
| Reviewed by Simon. |
| |
| Fix encoding of [ and ] in the host part of the URL |
| |
| Until QUrl is fixed (making QUrl's tolerant parser more tolerant), we have to |
| add this workaround to the QUrl <> WebCore::KURL conversion operator so that it |
| doesn't encode [ and ] when they are found in the host part. That is, the |
| following URL: |
| http://[::1]/ |
| is valid and should not be reencoded to: |
| http://%5b::1%5d/ |
| |
| * platform/qt/KURLQt.cpp: |
| (WebCore::KURL::operator QUrl): |
| |
| 2008-08-12 Mihnea Ovidenie <mihnea@adobe.com> |
| |
| Fix for https://bugs.webkit.org/show_bug.cgi?id=19891 |
| Broken HTML object elements cause de-reference of pointer to freed memory. |
| If we fail to load an image for an object tag and we no longer believe the object tag points at |
| an image, then clear m_imageLoader in the HTMLObjectElement so that we attempt to render the |
| fall back content. |
| |
| Reviewed by Dave Hyatt and Alexey Proskuryakov. |
| |
| Test: http/tests/misc/object-image-error-with-onload.html |
| |
| * html/HTMLObjectElement.cpp: |
| (WebCore::HTMLObjectElement::renderFallbackContent): |
| * page/Frame.cpp: |
| (WebCore::Frame::Frame): |
| |
| 2008-08-12 Nikolas Zimmermann <zimmermann@kde.org> |
| |
| Reviewed by Dave. |
| |
| Fixes: https://bugs.webkit.org/show_bug.cgi?id=19798 |
| Masks are translated, and the mask images are swapped on the y-axis. |
| |
| Turned out that http://trac.webkit.org/changeset/31830/trunk/WebCore/svg/graphics/cg/SVGResourceMaskerCg.mm |
| is guilty. GraphicsContext::clipToImageBuffer() does some extra transformations that SVGResourcesMaskerCg does not want. |
| |
| Long term goal is to remove the SVGResource*/SVGPaintServer* classes anyway, so it's okay to duplicate |
| the "clip to image buffer" functionality, in the CG specific SVGResourceMaskerCg class - as it was before. |
| |
| * svg/graphics/cg/SVGResourceMaskerCg.mm: |
| (WebCore::SVGResourceMasker::applyMask): Changed back to use CG clipping again. |
| |
| 2008-08-12 Dan Bernstein <mitz@apple.com> |
| |
| - WebCore part of <rdar://problem/6121636> |
| Make fast*alloc() abort() on failure and add "try" variants that |
| return NULL on failure. |
| |
| Reviewed by Darin Adler. |
| |
| * platform/Arena.cpp: |
| (WebCore::ArenaAllocate): Removed null checking of fastMalloc()'s |
| result. |
| * platform/graphics/cg/ImageBufferCG.cpp: |
| (WebCore::ImageBuffer::create): Changed to use tryFastCalloc(). |
| |
| 2008-08-12 Dan Bernstein <mitz@apple.com> |
| |
| Reviewed by Darin Adler. |
| |
| - fix https://bugs.webkit.org/show_bug.cgi?id=19348 |
| <rdar://problem/5978447> REGRESSION (r34193): Setting the size of a frame with javascript document.body.row no longer works |
| |
| Test: fast/frames/frameset-style-recalc.html |
| |
| * html/HTMLFrameSetElement.cpp: |
| (WebCore::HTMLFrameSetElement::recalcStyle): Changed to call the base |
| class implementation after marking for layout. |
| |
| 2008-08-12 Dan Bernstein <mitz@apple.com> |
| |
| Reviewed by John Sullivan. |
| |
| - move shouldUpdateWhenOffscreen from Settings to FrameView and rename it shouldUpdateWhileHidden |
| |
| * WebCore.base.exp: |
| * page/FrameView.cpp: |
| (WebCore::FrameViewPrivate::FrameViewPrivate): |
| (WebCore::FrameView::shouldUpdateWhileHidden): |
| (WebCore::FrameView::setShouldUpdateWhileHidden): |
| * page/FrameView.h: |
| * page/Settings.cpp: |
| * page/Settings.h: |
| |
| 2008-08-12 Adam Roben <aroben@apple.com> |
| |
| Windows build fix |
| |
| * bindings/js/JSSVGPODTypeWrapper.h: Align |
| JSSVGDynamicPODTypeWrapper's and JSSVGStaticPODTypeWrapperWithParent's |
| members on 16-byte boundaries to avoid an alignment warning. |
| |
| 2008-08-12 Nikolas Zimmermann <zimmermann@kde.org> |
| |
| Reviewed by Oliver. |
| |
| Add new dynamice-update layout tests covering SVGMarkerElement. |
| Fix bug: SVGMarkerElement's SVG DOM function calls don't update rendering. |
| Fix orientAngle/orientType confusion: "auto" orient should always return "0" as angle. |
| |
| Tests: svg/dynamic-updates/SVGMarkerElement-dom-markerHeight-attr.html |
| svg/dynamic-updates/SVGMarkerElement-dom-markerUnits-attr.html |
| svg/dynamic-updates/SVGMarkerElement-dom-markerWidth-attr.html |
| svg/dynamic-updates/SVGMarkerElement-dom-orient-attr.html |
| svg/dynamic-updates/SVGMarkerElement-dom-refX-attr.html |
| svg/dynamic-updates/SVGMarkerElement-dom-refY-attr.html |
| svg/dynamic-updates/SVGMarkerElement-svgdom-markerHeight-prop.html |
| svg/dynamic-updates/SVGMarkerElement-svgdom-markerUnits-prop.html |
| svg/dynamic-updates/SVGMarkerElement-svgdom-markerWidth-prop.html |
| svg/dynamic-updates/SVGMarkerElement-svgdom-orientAngle-prop.html |
| svg/dynamic-updates/SVGMarkerElement-svgdom-orientType-prop.html |
| svg/dynamic-updates/SVGMarkerElement-svgdom-refX-prop.html |
| svg/dynamic-updates/SVGMarkerElement-svgdom-refY-prop.html |
| svg/dynamic-updates/SVGMarkerElement-svgdom-setOrientToAngle-call.html |
| svg/dynamic-updates/SVGMarkerElement-svgdom-setOrientToAuto-call.html |
| |
| * svg/SVGMarkerElement.cpp: |
| (WebCore::SVGMarkerElement::SVGMarkerElement): |
| (WebCore::SVGMarkerElement::parseMappedAttribute): |
| (WebCore::SVGMarkerElement::svgAttributeChanged): |
| (WebCore::SVGMarkerElement::childrenChanged): |
| (WebCore::SVGMarkerElement::setOrientToAuto): |
| (WebCore::SVGMarkerElement::setOrientToAngle): |
| (WebCore::SVGMarkerElement::canvasResource): |
| |
| 2008-08-12 Nikolas Zimmermann <zimmermann@kde.org> |
| |
| Reviewed by Oliver. |
| |
| Add new dynamic-update layout tests covering SVGImageElement. |
| Fix bug: SVGImageElement doesn't react on 'preserveAspectRatio' changes. |
| |
| Tests: svg/dynamic-updates/SVGImageElement-dom-height-attr.html |
| svg/dynamic-updates/SVGImageElement-dom-preserveAspectRatio-attr.html |
| svg/dynamic-updates/SVGImageElement-dom-width-attr.html |
| svg/dynamic-updates/SVGImageElement-dom-x-attr.html |
| svg/dynamic-updates/SVGImageElement-dom-y-attr.html |
| svg/dynamic-updates/SVGImageElement-svgdom-height-prop.html |
| svg/dynamic-updates/SVGImageElement-svgdom-preserveAspectRatio-prop.html |
| svg/dynamic-updates/SVGImageElement-svgdom-width-prop.html |
| svg/dynamic-updates/SVGImageElement-svgdom-x-prop.html |
| svg/dynamic-updates/SVGImageElement-svgdom-y-prop.html |
| |
| * svg/SVGImageElement.cpp: |
| (WebCore::SVGImageElement::svgAttributeChanged): |
| |
| 2008-08-11 Anthony Ricaud <rik24d@gmail.com> |
| |
| Changed Option/Alt-Up or Down in CSS editing when the value is |
| near zero to jump to the next integer. |
| |
| Reviewed by Tim Hatcher. |
| |
| https://bugs.webkit.org/show_bug.cgi?id=20326 |
| |
| * page/inspector/StylesSidebarPane.js: |
| |
| 2008-08-11 Anthony Ricaud <rik24d@gmail.com> |
| |
| Changed the line highlight transition for an easier animation. |
| |
| Reviewed by Tim Hatcher. |
| |
| * page/inspector/SourceFrame.js: |
| |
| 2008-08-11 Keishi Hattori <casey.hattori@gmail.com> |
| |
| Added support for some Firebug Command Line APIs. |
| |
| Reviewed by Tim Hatcher. |
| |
| https://bugs.webkit.org/show_bug.cgi?id=19867 |
| https://bugs.webkit.org/show_bug.cgi?id=19868 |
| https://bugs.webkit.org/show_bug.cgi?id=19869 |
| https://bugs.webkit.org/show_bug.cgi?id=19875 |
| https://bugs.webkit.org/show_bug.cgi?id=19876 |
| https://bugs.webkit.org/show_bug.cgi?id=19880 |
| |
| * page/inspector/Console.js: |
| (WebInspector.Console.prototype._evalInInspectedWindow): |
| Create an object on the inspected window that holds the console |
| command line API functions. This object is used in a with statement |
| around the typed expression. |
| |
| 2008-08-11 Nikolas Zimmermann <zimmermann@kde.org> |
| |
| Reviewed by Antti. |
| |
| Fixes: http://bugs.webkit.org/show_bug.cgi?id=17736 |
| |
| JS wrapper objects around SVG POD types, that contain other SVG POD types with writable properties |
| failed to update. Modification of the values were completly ignored (ie. transform.matrix.a = 50, didn't take any effect) |
| |
| Added tests: svg/custom/svg-modify-currentTranslate.html |
| svg/custom/tearoffs-with-tearoffs.html |
| svg/custom/immutable-properties.html |
| |
| Fixed tests: svg/dynamic-updates/SVGLinearGradientElement-svgdom-gradientTransform-prop.html |
| |
| * bindings/js/JSSVGPODTypeWrapper.h: |
| (WebCore::JSSVGDynamicPODTypeWrapper::commitChange): |
| (WebCore::JSSVGStaticPODTypeWrapperWithPODTypeParent::create): |
| (WebCore::JSSVGStaticPODTypeWrapperWithPODTypeParent::commitChange): |
| (WebCore::JSSVGStaticPODTypeWrapperWithPODTypeParent::JSSVGStaticPODTypeWrapperWithPODTypeParent): |
| (WebCore::JSSVGStaticPODTypeWrapperWithParent::create): |
| (WebCore::JSSVGStaticPODTypeWrapperWithParent::operator PODType): |
| (WebCore::JSSVGStaticPODTypeWrapperWithParent::commitChange): |
| (WebCore::JSSVGStaticPODTypeWrapperWithParent::JSSVGStaticPODTypeWrapperWithParent): |
| * bindings/scripts/CodeGenerator.pm: |
| * bindings/scripts/CodeGeneratorJS.pm: |
| * svg/SVGSVGElement.idl: Add [Immutable] markers to SVG POD attributes, that contain POD types with writable attributes. |
| * svg/SVGZoomEvent.idl: SVG specification explicitely demands these attributes to be readonly, even its content. |
| |
| 2008-08-11 Brady Eidson <beidson@apple.com> |
| |
| Reviewed by John and Anders |
| |
| Fix for <rdar://problem/6141797> |
| |
| When WebArchives were entirely a WebKit concept, there was a guarantee that a WebResource |
| would never have nil data. |
| |
| When they were pushed down into WebCore, that guarantee was lost, subtly changing a few |
| semantics with some WebKit applications. |
| |
| The guarantee was a good one and should be restored. |
| |
| Note that ApplicationCacheResource doesn't need any updates to follow this rule as it already |
| creates an empty data object in the case of null data for its own purposes. |
| |
| * loader/SubstituteResource.h: |
| (WebCore::SubstituteResource::SubstituteResource): ASSERT that the data is not null. This |
| well help any future subclassers not make this mistake. |
| |
| * loader/archive/ArchiveResource.cpp: |
| (WebCore::ArchiveResource::create): Return 0 if the data is null. |
| |
| 2008-08-11 Simon Fraser <simon.fraser@apple.com> |
| |
| Reviewed by Dave Hyatt |
| |
| https://bugs.webkit.org/show_bug.cgi?id=20328 |
| Fix a problem when an 'all' transition transition with more than |
| one property changing is interrupted, and did some AnimationController |
| cleanup. |
| |
| Test: transitions/interrupted-all-transition.html |
| |
| * page/AnimationController.cpp: |
| (WebCore::ImplicitAnimation::ImplicitAnimation): |
| (WebCore::AnimationControllerPrivate::blendProperties): |
| (WebCore::CompositeAnimation::updateTransitions): |
| (WebCore::CompositeAnimation::cleanupFinishedAnimations): |
| (WebCore::CompositeAnimation::setTransitionStartTime): |
| (WebCore::CompositeAnimation::overrideImplicitAnimations): |
| (WebCore::CompositeAnimation::resumeOverriddenImplicitAnimations): |
| (WebCore::ImplicitAnimation::animate): |
| (WebCore::ImplicitAnimation::onAnimationEnd): |
| (WebCore::ImplicitAnimation::sendTransitionEvent): |
| (WebCore::ImplicitAnimation::affectsProperty): |
| (WebCore::KeyframeAnimation::endAnimation): |
| (WebCore::KeyframeAnimation::onAnimationEnd): |
| |
| 2008-08-11 Kevin McCullough <kmccullough@apple.com> |
| |
| Reviewed by Tim. |
| |
| - Because console messages have group levels now, newly created messages |
| that do not specify the level lose their message since the number of |
| arguments is wrong. |
| |
| * page/inspector/Console.js: |
| * page/inspector/Resource.js: |
| |
| 2008-08-11 Alp Toker <alp@nuanti.com> |
| |
| Build fix. Add new files from r35666 (WebKitAnimationEvent). Also take |
| the opportunity to sort the sources lists. |
| |
| * GNUmakefile.am: |
| |
| 2008-08-11 Timothy Hatcher <timothy@apple.com> |
| |
| Speed up the the JavaScript syntax highlighter by generating |
| the finders only once per script instead of per line. |
| |
| https://bugs.webkit.org/show_bug.cgi?id=20346 |
| |
| Reviewed by Adam Roben. |
| |
| * page/inspector/SourceFrame.js: |
| (WebInspector.SourceFrame.prototype._syntaxHighlightJavascriptLine): |
| Removed, factored into syntaxHighlightJavascript as an inline function. |
| (WebInspector.SourceFrame.prototype.syntaxHighlightJavascript): |
| Pulled in the _syntaxHighlightJavascriptLine so it will create a closure. |
| Generate the finders before iterating the lines. |
| |
| 2008-08-11 Adam Roben <aroben@apple.com> |
| |
| Windows build fix |
| |
| * WebCore.vcproj/WebCore.vcproj: Added JSWebKitAnimationEvent.cpp and |
| JSWebKitTransitionEvent.cpp to the project. Let VS reformat the file, |
| too. |
| |
| 2008-08-11 Dan Bernstein <mitz@apple.com> |
| |
| Reviewed by Darin Adler. |
| |
| - fix <rdar://problem/6131096> Reproducible crash in CounterNode::isReset under guard malloc |
| |
| Test: fast/css/counters/invalidate-cached-counter-node.html |
| |
| * rendering/RenderContainer.cpp: |
| (WebCore::RenderContainer::invalidateCounters): Added. Invalidates all |
| RenderCounters in :before and :after content. |
| * rendering/RenderContainer.h: |
| * rendering/RenderCounter.cpp: |
| (WebCore::RenderCounter::isCounter): Renamed isRenderCounter() to this |
| to match the RenderObject method. |
| (WebCore::RenderCounter::invalidate): Added. Resets the cached |
| CounterNode and invalidates the object's layout and preferred widths. |
| (WebCore::destroyCounterNodeChildren): Added a call to |
| invalidateCounters(). |
| * rendering/RenderCounter.h: |
| * rendering/RenderObject.h: |
| (WebCore::RenderObject::invalidateCounters): |
| |
| 2008-08-11 Dean Jackson <dino@apple.com> |
| |
| Implement CSS Animation and Transition Events |
| https://bugs.webkit.org/show_bug.cgi?id=20337 |
| |
| Implement the events defined in the CSS Animations |
| and Transitions specifications so code can react |
| to animations and transitions. |
| |
| Reviewed by Tim Hatcher and Dave Hyatt. |
| |
| * DerivedSources.make: |
| * GNUmakefile.am: |
| * WebCore.pro: |
| * WebCore.vcproj/WebCore.vcproj: |
| * WebCore.xcodeproj/project.pbxproj: |
| * WebCoreSources.bkl: |
| Build configs for new files |
| |
| * bindings/js/JSDOMWindowBase.cpp: |
| * bindings/js/JSDOMWindowBase.h: |
| * bindings/js/JSEventCustom.cpp: |
| * dom/Document.h: |
| * dom/Event.cpp: |
| * dom/Event.h: |
| * dom/EventTarget.cpp: |
| * dom/EventTargetNode.cpp: |
| * dom/EventTargetNode.h: |
| * html/HTMLElement.cpp: |
| * page/AnimationController.cpp: |
| do all the new event stuff |
| |
| * html/HTMLAttributeNames.in: |
| the onwebkitanimation* and onwebkittransitionend attrs |
| |
| * dom/WebKitAnimationEvent.cpp: Added. |
| * dom/WebKitAnimationEvent.h: Added. |
| * dom/WebKitAnimationEvent.idl: Added. |
| * dom/WebKitTransitionEvent.cpp: Added. |
| * dom/WebKitTransitionEvent.h: Added. |
| * dom/WebKitTransitionEvent.idl: Added. |
| New files for the events |
| |
| * manual-tests/transition-events.html: Added. |
| New testfile |
| |
| 2008-08-11 Adam Roben <aroben@apple.com> |
| |
| Add a ForwardingHeader for wtf/NotFound.h |
| |
| Rubberstamped by Darin Adler. |
| |
| * ForwardingHeaders/wtf/NotFound.h: Added. |
| |
| 2008-08-11 Timothy Hatcher <timothy@apple.com> |
| |
| Fixes a bug where error bubbles in JavaScript resources would |
| be clobbered by the syntax highlighter. |
| |
| https://bugs.webkit.org/show_bug.cgi?id=20345 |
| |
| Reviewed by Adam Roben. |
| |
| * manual-tests/inspector/resources/script-error.js: Added. |
| * manual-tests/inspector/styled-error-bubbles-in-scripts.html: Added. |
| * page/inspector/SourceFrame.js: |
| (WebInspector.SourceFrame.prototype._addMessageToSource): |
| Check the nodeType and not the nodeName, this is less fragile. |
| (WebInspector.SourceFrame.prototype._syntaxHighlightJavascriptLine): |
| Check if the lastChild is an error bubble, if so remove it before |
| getting the line's textContent. Add the error bubble back at the end. |
| |
| == Rolled over to ChangeLog-2008-08-10 == |