⚠ Archived content — this site is no longer maintained.   Current WebKit documentation is at docs.webkit.org.

Timeline



Apr 21, 2007:

8:15 PM Changeset in webkit [21002] by andrew
  • 3 edits
    4 adds in trunk

LayoutTests:
2007-04-21 Mitz Pettel <​mitz@webkit.org>

Reviewed by Darin.

  • fast/repaint/control-clip-expected.checksum: Added.
  • fast/repaint/control-clip-expected.png: Added.
  • fast/repaint/control-clip-expected.txt: Added.
  • fast/repaint/control-clip.html: Added.

WebCore:
2007-04-21 Mitz Pettel <​mitz@webkit.org>

Reviewed by Darin.

Test: fast/repaint/control-clip.html

Ignore controls' lightweight clip in repaint rect computation for now.

  • rendering/RenderBox.cpp: (WebCore::RenderBox::computeAbsoluteRepaintRect):
6:41 PM Changeset in webkit [21001] by darin
  • 2 edits in trunk/WebCore

Reviewed by Adam.

  • fix <rdar://problem/4816343> :-khtml-drag CSS pseudo-class changed to :-webkit-drag, need alias
  • css/CSSSelector.cpp: (WebCore::CSSSelector::extractPseudoType): Since Apple documentation claims "-khtml-drag" works, we need to keep it working, so add it back as an alias.
4:46 PM Changeset in webkit [21000] by darin
  • 3 edits in trunk/WebKit

Reviewed by Oliver.

  • fix some problems I ran into using the inspector
  • WebInspector/webInspector/inspector.js: Add some null checks.
4:46 PM Changeset in webkit [20999] by darin
  • 2 edits in trunk/WebCore

Reviewed by Oliver.

  • fix <rdar://problem/4782422> REGRESSION: -[stringByEvaluatingJavaScriptFromString] returns an empty string instead of nil for non-string results
  • page/mac/WebCoreFrameBridge.mm: (-[WebCoreFrameBridge stringByEvaluatingJavaScriptFromString:forceUserGesture:]): Added a call to nsStringNilIfEmpty to restore the old behavior. It might be nice to distinguish the empty string from a non-string result, but there's no obvious way to do that, and for compatibility it's probably best to just restore the old behavior instead.
3:31 PM Changeset in webkit [20998] by hyatt
  • 3 edits in trunk/WebCore

Fix for bug 13432, determineFlowSpacing is O(n2).

Reviewed by aroben

  • rendering/InlineBox.cpp: (WebCore::InlineBox::nextOnLineExists): (WebCore::InlineBox::prevOnLineExists):
  • rendering/InlineBox.h: (WebCore::InlineBox::InlineBox):
2:38 AM Changeset in webkit [20997] by ap
  • 8 edits
    2 adds in trunk

2007-04-21 Lamar Goddard <​lamargoddard@gmail.com>

Reviewed by Darin.

Fix for ​http://bugs.webkit.org/show_bug.cgi?id=5262
<rdar://problem/5018778>
XMLSerializer drops Namespace information

WebCore:

Updated WebCore::markup to output namespace information for elements/attributes whose namespace information
doesn't appear in its scope in the output.

  • WebCore/editing/markup.cpp: (WebCore::createMarkup(const Node*, ...)): Changed call to WebCore::markup to match parameters (WebCore::markup): Changed recursive call to match tree structure, removed ASSERT and no longer needed includeSiblings parameter. (WebCore::startMarkup): Added optional parameter to track namespaces in the current scope. (WebCore::addNamespace): Function to add namespace information to markup. (WebCore::shouldAddNamespaceAttr): (WebCore::shouldAddNamespaceElem): Functions that test whether namespace information should be added for a given node.
  • WebCore/dom/Document.idl: (createElementNS): (createAttributeNS): (getElementsByTagNameNS): Added [ConvertNullToNullString] to namespaceURI parameter.
  • WebCore/dom/Node.cpp: (Node::getElementsByTagNameNS): removed test for namespaceURI being null as null can be a valid namespace.

LayoutTests:

  • fast/dom/serialize-nodes-expected.txt: Added.
  • fast/dom/serialize-nodes.xhtml: Added.
  • dom/xhtml/level3/core/nodeisequalnode14-expected.txt: This test fails because createAttribute is supposed to create an Attr with localName of null.
  • dom/xhtml/level3/core/nodeisequalnode15-expected.txt: Now succeeds.
  • fast/innerHTML/004-expected.txt: Added namespace information to head and body nodes as xhtml nodes now serialize with namespace information.
1:50 AM Changeset in webkit [20996] by ap
  • 4 edits
    4 adds in trunk

Reviewed by Darin.

​http://bugs.webkit.org/show_bug.cgi?id=13300
Reproducible crash opening anekdot.ru

WebCore:

  • html/HTMLParser.cpp: (WebCore::HTMLParser::HTMLParser): (WebCore::HTMLParser::getNode): (WebCore::HTMLParser::handleIsindex):
  • html/HTMLParser.h: Made HTMLParser::form a RefPtr, and renamed it to m_currentFormElement. Made m_currentMapElement a RefPtr.

LayoutTests:

  • fast/parser/rewrite-form-expected.txt: Added.
  • fast/parser/rewrite-form.html: Added.
  • fast/parser/rewrite-map-expected.txt: Added.
  • fast/parser/rewrite-map.html: Added.
1:20 AM Changeset in webkit [20995] by bdash
  • 9 edits
    4 adds in trunk

2007-04-21 Mitz Pettel <​mitz@webkit.org>

Reviewed by Darin.

Test: fast/overflow/hit-test-overflow-controls.html

  • page/EventHandler.cpp: (WebCore::EventHandler::handleMouseMoveEvent): Avoid activating scrollbars during layer resize.
  • rendering/RenderBlock.cpp: (WebCore::RenderBlock::isPointInOverflowControl): Renamed isPointInScrollbar to this, to reflect that it returns true for points in the resize corner as well. Changed it to call the layer to do the actual work. (WebCore::RenderBlock::nodeAtPoint): Updated for the above rename.
  • rendering/RenderBlock.h:
  • rendering/RenderLayer.cpp: (WebCore::RenderLayer::horizontalScrollbarWidget): Fixed typo in this function's name (it was horizontaScrollbarWidget). (WebCore::RenderLayer::hitTestOverflowControls): Added. Returns whether the hit point is in one of the scrollbars or the resize corner, and updates the platform scrollbar in the hit test result if a scrollbar was hit.
  • rendering/RenderLayer.h:
  • rendering/RenderListBox.cpp: (WebCore::RenderListBox::isPointInOverflowControl): Rename.
  • rendering/RenderListBox.h:

2007-04-21 Mitz Pettel <​mitz@webkit.org>

Reviewed by Darin.

  • fast/overflow/hit-test-overflow-controls-expected.checksum: Added.
  • fast/overflow/hit-test-overflow-controls-expected.png: Added.
  • fast/overflow/hit-test-overflow-controls-expected.txt: Added.
  • fast/overflow/hit-test-overflow-controls.html: Added.
1:17 AM Changeset in webkit [20994] by bdash
  • 6 edits in trunk

2007-04-21 Mitz Pettel <​mitz@webkit.org>

Reviewed by Darin.

  • DumpRenderTree/DumpRenderTree.m: (activateAhemFont): Added. Activates the copy of Ahem included in the DumpRenderTree binary. (dumpRenderTree): Replaced the check that Ahem is available with a call to activateAhemFont().
  • DumpRenderTree/DumpRenderTree.xcodeproj/project.pbxproj: Added linker options to include Ahem in the data section of the DumpRenderTree binary.

2007-04-21 Mitz Pettel <​mitz@webkit.org>

Reviewed by Darin.

  • quality/testing.html: Removed the requirement to download and install Ahem for running the tests.
  • quality/testwriting.html: Added information about Ahem.
1:14 AM Changeset in webkit [20993] by bdash
  • 3 edits
    2 adds in trunk

2007-04-21 Mitz Pettel <​mitz@webkit.org>

Reviewed by Darin.

Test: fast/dom/Window/getMatchedCSSRules-null-crash.html

  • css/cssstyleselector.cpp: (WebCore::CSSStyleSelector::styleRulesForElement): Added null check.

2007-04-21 Mitz Pettel <​mitz@webkit.org>

Reviewed by Darin.

  • fast/dom/Window/getMatchedCSSRules-null-crash-expected.txt: Added.
  • fast/dom/Window/getMatchedCSSRules-null-crash.html: Added.
1:12 AM Changeset in webkit [20992] by bdash
  • 3 edits in trunk/WebCore

2007-04-21 Mitz Pettel <​mitz@webkit.org>

Reviewed by Darin.

No test included because DumpRenderTree does not support setting the user stylesheet.

  • css/cssstyleselector.cpp: (WebCore::CSSStyleSelector::CSSStyleSelector): (WebCore::CSSStyleSelector::~CSSStyleSelector):
  • css/cssstyleselector.h: Made m_userSheet a RefPtr.
1:09 AM Changeset in webkit [20991] by bdash
  • 2 edits in trunk/JavaScriptCore

2007-04-21 Mitz Pettel <​mitz@webkit.org>

Reviewed by Adam.

  • kjs/array_object.cpp: (ArrayInstance::sort): Free the old storage, not the new one.
1:08 AM Changeset in webkit [20990] by bdash
  • 2 edits in trunk/WebCore

2007-04-21 MorganL <​morganl.webkit@yahoo.com>

Reviewed by Adam.

NPObject should be forward declared using 'struct' instead of 'class'

  • page/Frame.h:
12:51 AM Changeset in webkit [20989] by bdash
  • 2 edits in trunk/WebCore

2007-04-21 Alp Toker <​alp@atoker.com>

Gdk build fix.

  • platform/gdk/FontPlatformDataGdk.cpp: (WebCore::FontPlatformData::FontPlatformData): Use the correct casts.

Apr 20, 2007:

11:43 PM Changeset in webkit [20988] by thatcher
  • 1 edit in trunk/WebKit/WebInspector/WebInspector.m

Fixes the build.

11:19 PM Changeset in webkit [20987] by thatcher
  • 1 delete in branches/WebInspectorRefresh

Remove the branch now that the changes were merged over to source:trunk in r20986.

11:14 PM Changeset in webkit [20986] by thatcher
  • 2 edits
    11 copies
    2 deletes in trunk/WebKit

Reviewed by Darin.

Merge over the WebInspectorRefresh branch. This change removes the NSOutlineView and
is replaced with a HTML/JavaScript tree. Most of the inspector logic is now in the JavaScript.

A few bugs are fixed by these changes:

Bug 6615: Parent node drop-down list is upside-down
​http://bugs.webkit.org/show_bug.cgi?id=6615

Bug 6643: REGRESSION: Tree view repaints lines without erasing them first
​http://bugs.webkit.org/show_bug.cgi?id=6643

Bug 6650: Web Inspector HTML Hierarchy can't be scrolled with scrollwheel
​http://bugs.webkit.org/show_bug.cgi?id=6650

Bug 6677: Can't drag inspector when tree view has focus
​http://bugs.webkit.org/show_bug.cgi?id=6677

Bug 7326: Web Inspector tree scrollbar always shows up when resizing the top pane down
​http://bugs.webkit.org/show_bug.cgi?id=7326

  • WebInspector/WebInspector.h: Removed the searchQuery methods.
  • WebInspector/WebInspector.m: Removed the DOMNode category and code for the old outline view.
  • WebInspector/WebInspectorOutlineView.h: Removed.
  • WebInspector/WebInspectorOutlineView.m: Removed.
  • WebInspector/WebInspectorInternal.h: Remove some methods and instance variables.
  • WebInspector/webInspector/Images/resize.png: Added.
  • WebInspector/webInspector/inspector.css:
  • WebInspector/webInspector/inspector.html: Include the new classes and remove the plugin.
  • WebInspector/webInspector/inspector.js: Changes to use the new tree outline and other fixes.
  • WebInspector/webInspector/scrollarea.js: Copied from the Dashboard widget resources.
  • WebInspector/webInspector/scrollbar.js: Ditto.
  • WebInspector/webInspector/treeoutline.js: New tree outline class.
  • WebInspector/webInspector/utilities.js: DOM and String prototype additions.
  • WebKit.xcodeproj/project.pbxproj: Remove WebInspectorOutlineView.
7:00 PM Changeset in webkit [20985] by bdash
  • 1 copy in tags/Safari-4522.6.1

New tag.

6:49 PM Changeset in webkit [20984] by bdash
  • 4 edits in branches/Safari-522

Versioning.

6:46 PM Changeset in webkit [20983] by bdash
  • 8 edits in branches/Safari-522

Merge r20445.

6:00 PM Changeset in webkit [20982] by bdakin
  • 1 edit
    2 adds in trunk/LayoutTests

Reviewed by Adam.

Test I meant to check in a while ago.

  • fast/events/fire-scroll-event-expected.txt: Added.
  • fast/events/fire-scroll-event.html: Added.
5:58 PM Changeset in webkit [23329] by adachan
  • 11 edits in branches/WindowsMerge
5:55 PM Changeset in webkit [23328] by bdakin
  • 2 edits in branches/WindowsMerge/WebCore

Reviewed by Adam.

Fix for <rdar://problem/5089822> Boomer does not send scroll events

  • platform/win/ScrollViewWin.cpp: (WebCore::ScrollView::ScrollViewPrivate::valueChanged):
5:26 PM Changeset in webkit [20981] by beidson
  • 6 edits in trunk

WebCore:

Reviewed by Oliver (Black Sheep)

<rdar://problem/3559794>
[WebView setMaintainsBackForwardList:] doesn't actually flush out the current page caches

  • WebCore.exp:
  • history/BackForwardList.cpp: (WebCore::BackForwardList::BackForwardList): Set the flag to true by default (WebCore::BackForwardList::addItem): Check the flag and bail if its false (WebCore::BackForwardList::setCapacity): More correct if it set the m_current to NoCurrentItemIndex if we're setting capacity to 0 (WebCore::BackForwardList::enabled): (WebCore::BackForwardList::setEnabled): If we're disabling, cycle capacity to 0 and back to flush things out
  • history/BackForwardList.h: Add enabled(), setEnabled(bool), and m_enabled

WebKit:

Reviewed by Oliver (Black Sheep)

<rdar://problem/3559794>
[WebView setMaintainsBackForwardList:] doesn't actually flush out the current page caches

  • WebView/WebView.mm: Remove _private->useBackForwardList (-[WebView _setInitiatedDrag:]): Use _private->page instead of [self page] (-[WebView initWithCoder:]): Manipulate the flag that is now in WebCore::BackForwardList (-[WebView encodeWithCoder:]): Ditto (-[WebView backForwardList]): Use _private->page instead of [self page] (-[WebView setMaintainsBackForwardList:]): Manipulate the flag that is now in WebCore::BackForwardList
5:24 PM Changeset in webkit [20980] by hyatt
  • 5 edits in trunk/WebCore

Bug 13424, firstLineStyle and verticalPositionHint together take
33% of the time on the new tag nesting PLT. Don't waste time even
looking for first-line styles if no stylesheet used them.

Reviewed by beth

  • css/CSSGrammar.y:
  • dom/Document.cpp: (WebCore::Document::Document):
  • dom/Document.h: (WebCore::Document::usesFirstLineRules): (WebCore::Document::setUsesFirstLineRules):
  • rendering/RenderObject.cpp: (WebCore::RenderObject::verticalPositionHint): (WebCore::RenderObject::firstLineStyle):
5:14 PM Changeset in webkit [20979] by thatcher
  • 1 copy in tags/Safari-4522.6

New tag.

5:13 PM Changeset in webkit [20978] by thatcher
  • 4 edits in branches/Safari-522

Versioning

3:50 PM Changeset in webkit [20977] by mjs
  • 2 edits in trunk/JavaScriptCore

Not reviewed, build fix.

  • fix build problem with last change - -O3 complains more about uninitialized variables


  • pcre/pcre_compile.c: (compile_branch): (pcre_compile2):
3:33 PM Changeset in webkit [20976] by aroben
  • 2 edits in trunk/WebKitTools

Reviewed by Darin.

​http://bugs.webkit.org/show_bug.cgi?id=13421
Bug 13421: prepare-ChangeLog should use svn-create-patch when spewing diffs

  • Scripts/prepare-ChangeLog: Use svn-create-patch instead of svn diff.
3:26 PM Changeset in webkit [20975] by oliver
  • 2 edits in trunk/WebCore

2007-04-20 Peter Kasting <​pkasting@google.com>

Reviewed by Oliver.

  • platform/TextEncodingRegistry.cpp: (WebCore::addToTextEncodingNameMap): Change a pointer comparison to a strcmp(), since the strings here are not necessarily pointer-equivalent.
3:20 PM Changeset in webkit [23327] by aroben
  • 2 edits in branches/WindowsMerge/WebKitTools
  • DumpRenderTree/DumpRenderTree.vcproj/TestNetscapePlugin/TestNetscapePlugin.vcproj: Use WebKitConfigSuffix instead of ConfigSuffix.
3:20 PM Changeset in webkit [20974] by mjs
  • 3 edits in trunk/JavaScriptCore

Reviewed by Darin.


  • <rdar://problem/5149915> use mergesort when possible, since it leads to fewer compares (2% JS iBench speedup)
  • kjs/array_object.cpp: (ArrayInstance::sort): Use mergesort(3) on platforms that have it, since it tends to do fewer compares than qsort; but avoid it very on large arrays since it uses extra memory. Also added comments identifying possibly even better sorting algorithms for sort by string value and sort by compare function.
  • kjs/config.h:
3:00 PM Changeset in webkit [20973] by mjs
  • 2 edits in trunk/JavaScriptCore

Reviewed by Darin.


  • bump optimization flags up to -O3 for 1% JS iBench speed improvement
  • Configurations/Base.xcconfig:
2:52 PM Changeset in webkit [20972] by darin
  • 7 edits
    2 adds in trunk

LayoutTests:

Reviewed by Geoff Garen and John Sullivan.

  • test changes for <rdar://problem/4671964> document property on <iframe> elements causes problems for SAP application
  • fast/dom/iframe-document-expected.txt: Added.
  • fast/dom/iframe-document.html: Added.
  • fast/dom/attr_dead_doc.html: Changed to use contentDocument instead of relying on the document property on HTMLIFrameElement.
  • fast/events/mouseover-mouseout.html: Ditto.
  • fast/frames/repaint-display-none-crash.html: Ditto.

WebCore:

Reviewed by Geoff Garen and John Sullivan.

  • fix <rdar://problem/4671964> document property on <iframe> elements causes problems for SAP application

Test: fast/dom/iframe-document.html

  • bindings/js/kjs_html.h: Removed IFrameDocument.
  • bindings/js/kjs_html.cpp: (KJS::JSHTMLElement::iFrameGetter): Ditto.
12:49 PM GtkAPIProposal edited by Kimmo Kinnunen
(diff)
12:47 PM GtkAPIProposal edited by Kimmo Kinnunen
(diff)
2:25 AM BuildingGdk edited by alp@atoker.com
Updated roadmap (diff)
2:01 AM Changeset in webkit [23326] by aroben
  • 2 edits in branches/WindowsMerge/WebKitTools

Explicitly link DRT against some Win32 libs because VC++ Express
doesn't link against them by default.

  • DumpRenderTree/DumpRenderTree.vcproj/DumpRenderTree/DumpRenderTree.vcproj:
1:47 AM Changeset in webkit [20971] by bdash
  • 2 edits in trunk/JavaScriptCore

2007-04-20 Mark Rowe <​mrowe@apple.com>

Reviewed by Maciej.

Fix bogus optimisation in the generic pthread code path.

  • kjs/collector.cpp: (KJS::currentThreadStackBase):
12:50 AM Changeset in webkit [20970] by bdash
  • 2 edits in trunk/JavaScriptCore

2007-04-20 Mark Rowe <​mrowe@apple.com>

Reviewed by Anders.

Improve FreeBSD compatibility, as suggested by Alexander Botero-Lowry.

  • kjs/collector.cpp: (KJS::currentThreadStackBase): FreeBSD requires that pthread_attr_t's are initialized via pthread_attr_init before being used in any context.
12:13 AM Changeset in webkit [20969] by andersca
  • 2 edits in trunk/WebKit

Reviewed by Maciej.

<rdar://problem/5085897> REGRESSION: Some Flash links at www.jumpskyhigh.com just reload the page


Get rid of the check that would prevent plugin requests from being loaded if a new page load was underway.
www.jumpskyhigh.com had a flash movie that was embedded inside an <a> tag and clicking on the plug-in would cause the
URL pointed to by the <a> tag to start loading and thus preventing the plug-in from loading the real URL.


This check was added by Maciej and we should be able to remove it with the loader changes that have happened now,
(mainly the fact that resource loaders are handled by the document loader instead of the frame loader).


  • Plugins/WebBaseNetscapePluginView.mm: (-[WebBaseNetscapePluginView loadRequest:inTarget:withNotifyData:sendNotification:]):
Note: See TracTimeline for information about the timeline view.