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

Timeline



Apr 4, 2007:

11:04 PM Changeset in webkit [20717] by thatcher
  • 4 edits in branches/WebInspectorRefresh/WebKit/WebInspector

Move the tree splitter resize code down into JavaScript. Some other cleanup work.

10:24 PM Changeset in webkit [20716] by bdash
  • 2 edits in trunk/WebKitSite

2007-04-05 Mark Malone <​markmalone@mac.com>

Reviewed by Darin.

​http://bugs.webkit.org/show_bug.cgi?id=13273
Bug 13273: DOM Interfaces Nav link should go somewhere more useful

  • nav.inc: Link directly to the hierarchy.
10:22 PM Changeset in webkit [20715] by thatcher
  • 1 edit in branches/WebInspectorRefresh/WebKit/WebInspector/webInspector/inspector.js

Make selecting a new root element from the root popup menu work. Also reverse the order of the parent nodes shown in the popup, with the document at the top. Fixes bug 6615.

9:40 PM Changeset in webkit [20714] by ap
  • 3 edits
    2 adds in trunk

Reviewed by Darin.

​http://bugs.webkit.org/show_bug.cgi?id=13278
REGRESSION: cannot change SELECT option

Test: fast/forms/option-change-single-selected.html

  • html/HTMLSelectElement.cpp: (WebCore::HTMLSelectElement::setSelectedIndex): If multiple selection is not allowed, be sure to deselect the previous option.
9:39 PM Changeset in webkit [20713] by thatcher
  • 1 edit in branches/WebInspectorRefresh/WebKit/WebInspector/WebInspector.m

Remove my machine specific path.

9:37 PM Changeset in webkit [20712] by thatcher
  • 1 edit in branches/WebInspectorRefresh/WebKit/WebInspector/webInspector/AppleScrollbar.js

Only call this.refresh() if this.hidden is true. This was causing a 'maximum call stack size' exception.

9:26 PM Changeset in webkit [20711] by thatcher
  • 3 edits in branches/WebInspectorRefresh/WebKit/WebInspector/webInspector

Move our override scrollbar and scroll-area code to our copies of AppleScrollbar.js and AppleScrollArea.js.

8:43 PM Changeset in webkit [20710] by thatcher
  • 2 edits
    2 adds in branches/WebInspectorRefresh/WebKit/WebInspector

Keep our own copy of AppleScrollbar.js and AppleScrollArea.js, copied /System/Library/WidgetResources/AppleClasses.

6:39 PM Changeset in webkit [23228] by oliver
  • 2 edits in branches/WindowsMerge/WebKitWin

2007-04-04 Oliver Hunt <​oliver@apple.com>

Reviewed by Ada.


Fix <rdar://problem/5111027> Dragging a link on page
shows URL thumbnail as nearly-transparent instead
semi-transparent

Error in createDragImageForLink led me to get wrong
maximum drag image size, now logic and maximum drag
image size corrected.

  • WebDragClient.cpp: (WebDragClient::createDragImageForLink):


6:05 PM Changeset in webkit [20709] by adele
  • 9 edits in trunk/WebCore

Reviewed by Oliver.

Fix for <rdar://5105009> Prepare for fix to send mousedown and mouseup (but not click) events when right-clicking

  • page/EventHandler.cpp: Don't ever send click events for right-clicks. (WebCore::EventHandler::handleMouseDoubleClickEvent): (WebCore::EventHandler::handleMouseReleaseEvent):


Check that the mousedown and mouseup are not with the right button before performing default behavior.

  • html/HTMLAnchorElement.cpp: (WebCore::HTMLAnchorElement::defaultEventHandler):
  • html/HTMLInputElement.cpp: (WebCore::HTMLInputElement::defaultEventHandler):
  • html/HTMLSelectElement.cpp: (WebCore::HTMLSelectElement::menuListDefaultEventHandler): (WebCore::HTMLSelectElement::listBoxDefaultEventHandler):
  • html/HTMLTextFieldInnerElement.cpp: (WebCore::HTMLSearchFieldResultsButtonElement::defaultEventHandler): (WebCore::HTMLSearchFieldCancelButtonElement::defaultEventHandler):
  • ksvg2/svg/SVGAElement.cpp:(WebCore::SVGAElement::defaultEventHandler):
  • rendering/RenderFrameSet.cpp: (WebCore::RenderFrameSet::userResize):
  • rendering/RenderSlider.cpp: (WebCore::HTMLSliderThumbElement::defaultEventHandler):
2:40 PM Changeset in webkit [23227] by aroben
  • 2 edits in branches/WindowsMerge/WebKitWin

Release build fix for TestNetscapePlugin.

  • WebKit.vcproj/WebKit.vcproj: Make the Release post-build step the same as the Debug one so that the npapi headers get copied.
1:52 PM Changeset in webkit [20708] by sfalken
  • 19 edits in branches/Safari-522/WebCore

Re-merge fix from r20674

1:49 PM Changeset in webkit [20707] by justing
  • 4 edits
    12 adds in trunk

LayoutTests:

Reviewed by darin


<rdar://problem/5026848> Can cut/delete To Do subunits

This tests to make sure that non-editable content
that is in both the start and end editable roots
is removed:

  • editing/deleting/5026848-1-expected.checksum: Added.
  • editing/deleting/5026848-1-expected.png: Added.
  • editing/deleting/5026848-1-expected.txt: Added.
  • editing/deleting/5026848-1.html: Added. This tests to make sure that non-editable content that is not in both the start and end editable roots is not removed:
  • editing/deleting/5026848-2-expected.checksum: Added.
  • editing/deleting/5026848-2-expected.png: Added.
  • editing/deleting/5026848-2-expected.txt: Added.
  • editing/deleting/5026848-2.html: Added. This tests to make sure that editable regions that are not in both the start and end editable roots are cleared (not removed, unless they are in another editable region):
  • editing/deleting/5026848-3-expected.checksum: Added.
  • editing/deleting/5026848-3-expected.png: Added.
  • editing/deleting/5026848-3-expected.txt: Added.
  • editing/deleting/5026848-3.html: Added.

WebCore:

Reviewed by darin


<rdar://problem/5026848> Can cut/delete ToDo subunits


We were removing non-editable content during a delete.

  • editing/DeleteSelectionCommand.cpp: (WebCore::DeleteSelectionCommand::initializePositionData): Set m_startRoot and m_endRoot. (WebCore::DeleteSelectionCommand::removeNode): If a node is not inside both the start and end roots, remove it only if it is in editable content.
  • editing/DeleteSelectionCommand.h: Added m_startRoot and m_endRoot, the editable roots that contain the start and end of the selection, respectively. We consult these every time we remove a node, and don't want to recompute them on every call to removeNode().
1:39 PM Changeset in webkit [20706] by sfalken
  • 19 edits in branches/Safari-522/WebCore

Roll out r20705,r20701

11:24 AM Changeset in webkit [20705] by sfalken
  • 1 edit in branches/Safari-522/WebCore/loader/DocLoader.cpp

Fix merge from r20674

10:33 AM Changeset in webkit [20704] by zbujtas
  • 3 edits in S60/trunk/WebCore

2007-04-04 bujtas <​zbujtas@gmail.com>

Reviewed by Sriram.
DESC: change nested <q> behavior so that q q {} gets ' instead of " -antti(c).
​http://bugs.webkit.org/show_bug.cgi?id=13280


  • khtml/css/html4.css:
  • khtml/css/html4_gcce.css:
10:30 AM Changeset in webkit [20703] by andersca
  • 2 edits in trunk/WebKit

Reviewed by John.

<rdar://problem/5107536>
​http://bugs.webkit.org/show_bug.cgi?id=13264
REGRESSION: Crash when canceling about:blank in Activity viewer


  • WebView/WebFrame.mm: (-[WebFrame stopLoading]): Add a null check for the frame loader - it can be null when the frame has been disconnected from the web page.
10:28 AM Changeset in webkit [20702] by zbujtas
  • 3 edits in S60/branches/3.1m/WebCore

2007-04-04 bujtas <​zbujtas@gmail.com>

Reviewed by Sriram.
DESC: change nested <q> behavior so that q q {} gets ' instead of "
​http://bugs.webkit.org/show_bug.cgi?id=13280


  • khtml/css/html4.css:
  • khtml/css/html4_gcce.css:
10:28 AM Changeset in webkit [20701] by sfalken
  • 19 edits in branches/Safari-522/WebCore

Merged fix from r20674

10:12 AM Changeset in webkit [20700] by sfalken
  • 2 edits in branches/Safari-522/WebCore

Merged fix from r20669

9:47 AM Changeset in webkit [20699] by andersca
  • 1 edit in trunk/JavaScriptCore/ChangeLog

Turn's out Brady's not the only one who can rewrite history!

1:25 AM Changeset in webkit [23226] by aroben
  • 2 edits in branches/WindowsMerge/LayoutTests
  • Skipped: Re-enable some tests that no longer crash.

Apr 3, 2007:

11:39 PM Changeset in webkit [20698] by thatcher
  • 6 edits
    2 deletes in branches/WebInspectorRefresh/WebKit/WebInspector

Convert the Web Inspector NSOutlineView into an HTML and JavaScript based tree view. Not everything is working yet.

11:36 PM Changeset in webkit [20697] by thatcher
  • 1 copy in branches/WebInspectorRefresh

New branch to update the Web Inspector to use more HTML and JavaScript and remove the NSOutlineView.

10:55 PM Changeset in webkit [23225] by aroben
  • 6 edits in branches/WindowsMerge

WebKitToolsWin:

Reviewed by Anders.

DRT part of fix for <rdar://problem/5103999> All dom/svg tests are
failing due to extra/missing newlines

  • DumpRenderTree/DumpRenderTree.vcproj/DumpRenderTree/DumpRenderTree.cpp: (dump): Call IDOMElementPrivate::innerText to match Mac DRT.

WebKitWin:

Reviewed by Anders.

WebKit part of fix for <rdar://problem/5103999> All dom/svg tests are
failing due to extra/missing newlines

  • DOMCoreClasses.cpp: (DOMElement::innerText): Added.
  • DOMCoreClasses.h: Added declaration.
  • Interfaces/DOMPrivate.idl: Ditto.
10:32 PM Changeset in webkit [23224] by aroben
  • 2 edits in branches/WindowsMerge/WebKitTools

Rubberstamped by Adele.

Make all layout tests paint to match Mac DRT. This fixes many tests in
svg/hixie.

  • DumpRenderTree/DumpRenderTree.vcproj/DumpRenderTree/DumpRenderTree.cpp: (dump): Always force a paint.
7:12 PM Changeset in webkit [20696] by kmccullo
  • 2 edits in trunk/JavaScriptCore

Reviewed by Adam.

  • Testing a post-commit hook.
6:32 PM Changeset in webkit [20695] by andersca
  • 4 edits in trunk

JavaScriptCore:

Reviewed by Adam.

<rdar://problem/5107534>
​http://bugs.webkit.org/show_bug.cgi?id=13265
REGRESSION: Crash in KJS::Bindings::convertValueToNPVariant


  • bindings/NP_jsobject.cpp: (_NPN_InvokeDefault): Return false if the object isn't a function.

LayoutTests:

Reviewed by Adam.

<rdar://problem/5107534>
​http://bugs.webkit.org/show_bug.cgi?id=13265
REGRESSION: Crash in KJS::Bindings::convertValueToNPVariant

  • plugins/netscape-invoke-default.html: Test calling NPN_InvokeDefault on an object that isn't a function.
5:22 PM Changeset in webkit [20694] by andersca
  • 7 edits
    2 adds in trunk

LayoutTests:

Reviewed by Darin.

<rdar://problem/5028178>
Crash occurs at WebCore::FrameLoader::activeDocumentLoader() after loading Froggster widget

  • plugins/get-url-that-the-resource-load-delegate-will-disallow-expected.txt: Added.
  • plugins/get-url-that-the-resource-load-delegate-will-disallow.html: Added.

WebKit:

Reviewed by Darin.

<rdar://problem/5028178>
Crash occurs at WebCore::FrameLoader::activeDocumentLoader() after loading Froggster widget


  • Plugins/WebNetscapePluginStream.mm: (-[WebNetscapePluginStream start]): If load returns no the plugin loader has already been removed by the didFail callback.

WebKitTools:

Reviewed by Darin.

Add the notion of a "disallowed URL", which the resource loader won't allow to be loaded.


  • DumpRenderTree/DumpRenderTree.h:
  • DumpRenderTree/DumpRenderTree.m: (dumpRenderTree): (+[LayoutTestController isSelectorExcludedFromWebScript:]): (+[LayoutTestController webScriptNameForSelector:]): (-[LayoutTestController addDisallowedURL:]): (runTest):
  • DumpRenderTree/ResourceLoadDelegate.m: (-[ResourceLoadDelegate webView:resource:willSendRequest:redirectResponse:fromDataSource:]):
4:31 PM Changeset in webkit [20693] by hyatt
  • 2 edits in trunk/WebCore

Fix for <rdar://problem/5108896>, make sure the assert for setEncodedSize
deals with the error case.

Reviewed by andersca

  • loader/CachedResource.cpp: (WebCore::CachedResource::setEncodedSize):
4:22 PM Changeset in webkit [20692] by andersca
  • 2 edits in trunk/WebCore

Reviewed by Dave Hyatt.

  • loader/ImageDocument.cpp: Don't call error() in stopParsing.
3:50 PM Changeset in webkit [20691] by justing
  • 5 edits
    1 delete in trunk/LayoutTests

Reviewed by darin


<​http://bugs.webkit.org/show_bug.cgi?id=13268>
Make editing/selection/drag-in-iframe.html valid and deterministic

  • editing/resources/select-and-drag-iframe.html: Removed.
  • editing/selection/drag-in-iframe-expected.checksum:
  • editing/selection/drag-in-iframe-expected.png:
  • editing/selection/drag-in-iframe-expected.txt:
  • editing/selection/drag-in-iframe.html:
3:12 PM Changeset in webkit [23223] by oliver
  • 2 edits in branches/WindowsMerge/WebCore

2007-04-03 Oliver Hunt <​oliver@apple.com>

Buildfix

  • platform/network/cf/ResourceResponseCFNet.cpp: (WebCore::ResourceResponse::doUpdateResourceResponse):

Adding .get() calls

3:07 PM Changeset in webkit [20690] by adele
  • 4 edits in trunk/LayoutTests

Updated results for recent change to always lighten disabled text color for black text.

  • fast/forms/input-disabled-color-expected.checksum:
  • fast/forms/input-disabled-color-expected.png:
  • fast/forms/input-disabled-color-expected.txt:
2:57 PM Changeset in webkit [20689] by justing
  • 119 edits in trunk

LayoutTests:

Reviewed by darin


<rdar://problem/5107422>
TOT REGRESSION: Delete key fails to delete text, and cursor disappears in Mail.app

moveParagraphs no longer calls shouldInsert:

  • editing/deleting/delete-3857753-fix-expected.txt:
  • editing/deleting/delete-at-paragraph-boundaries-003-expected.txt:
  • editing/deleting/delete-at-paragraph-boundaries-007-expected.txt:
  • editing/deleting/delete-at-paragraph-boundaries-009-expected.txt:
  • editing/deleting/delete-block-merge-contents-001-expected.txt:
  • editing/deleting/delete-block-merge-contents-002-expected.txt:
  • editing/deleting/delete-block-merge-contents-003-expected.txt:
  • editing/deleting/delete-block-merge-contents-004-expected.txt:
  • editing/deleting/delete-block-merge-contents-005-expected.txt:
  • editing/deleting/delete-block-merge-contents-006-expected.txt:
  • editing/deleting/delete-block-merge-contents-007-expected.txt:
  • editing/deleting/delete-block-merge-contents-008-expected.txt:
  • editing/deleting/delete-block-merge-contents-009-expected.txt:
  • editing/deleting/delete-block-merge-contents-010-expected.txt:
  • editing/deleting/delete-block-merge-contents-012-expected.txt:
  • editing/deleting/delete-block-merge-contents-013-expected.txt:
  • editing/deleting/delete-block-merge-contents-014-expected.txt:
  • editing/deleting/delete-block-merge-contents-015-expected.txt:
  • editing/deleting/delete-block-merge-contents-016-expected.txt:
  • editing/deleting/delete-block-merge-contents-017-expected.txt:
  • editing/deleting/delete-block-merge-contents-018-expected.txt:
  • editing/deleting/delete-block-merge-contents-019-expected.txt:
  • editing/deleting/delete-block-merge-contents-020-expected.txt:
  • editing/deleting/delete-block-merge-contents-021-expected.txt:
  • editing/deleting/delete-block-merge-contents-022-expected.txt:
  • editing/deleting/delete-block-table-expected.txt:
  • editing/deleting/delete-br-010-expected.txt:
  • editing/deleting/delete-line-002-expected.txt:
  • editing/deleting/delete-line-004-expected.txt:
  • editing/deleting/delete-line-006-expected.txt:
  • editing/deleting/delete-line-007-expected.txt:
  • editing/deleting/delete-line-008-expected.txt:
  • editing/deleting/delete-line-011-expected.txt:
  • editing/deleting/delete-line-012-expected.txt:
  • editing/deleting/delete-listitem-001-expected.txt:
  • editing/deleting/delete-to-end-of-paragraph-expected.txt:
  • editing/deleting/merge-different-styles-expected.txt:
  • editing/deleting/merge-no-br-expected.txt:
  • editing/deleting/pruning-after-merge-1-expected.txt:
  • editing/deleting/pruning-after-merge-2-expected.txt:
  • editing/execCommand/create-list-from-range-selection-expected.txt:
  • editing/execCommand/create-list-with-hr-expected.txt:
  • editing/execCommand/format-block-expected.txt:
  • editing/execCommand/format-block-from-range-selection-expected.txt:
  • editing/execCommand/format-block-with-braces-expected.txt:
  • editing/execCommand/format-block-with-trailing-br-expected.txt:
  • editing/execCommand/indent-list-item-expected.txt:
  • editing/execCommand/indent-selection-expected.txt:
  • editing/execCommand/insert-list-and-stitch-expected.txt:
  • editing/execCommand/insert-list-with-id-expected.txt:
  • editing/execCommand/nsresponder-indent-expected.txt:
  • editing/execCommand/outdent-selection-expected.txt:
  • editing/execCommand/queryCommandState-01-expected.txt:
  • editing/execCommand/remove-list-1-expected.txt:
  • editing/execCommand/remove-list-from-range-selection-expected.txt:
  • editing/execCommand/remove-list-item-1-expected.txt:
  • editing/execCommand/remove-list-items-expected.txt:
  • editing/execCommand/switch-list-type-expected.txt:
  • editing/inserting/insert-3907422-fix-expected.txt:
  • editing/inserting/insert-div-026-expected.txt:
  • editing/inserting/multiple-lines-selected-expected.txt:
  • editing/inserting/return-key-with-selection-001-expected.txt:
  • editing/inserting/return-key-with-selection-002-expected.txt:
  • editing/pasteboard/4076267-2-expected.txt:
  • editing/pasteboard/4700297-expected.txt:
  • editing/pasteboard/8145-3-expected.txt:
  • editing/pasteboard/display-block-on-spans-expected.txt:
  • editing/pasteboard/emacs-ctrl-k-y-001-expected.txt:
  • editing/pasteboard/interchange-newline-1-expected.txt:
  • editing/pasteboard/interchange-newline-2-expected.txt:
  • editing/pasteboard/merge-after-delete-1-expected.txt:
  • editing/pasteboard/merge-after-delete-2-expected.txt:
  • editing/pasteboard/merge-after-delete-expected.txt:
  • editing/pasteboard/merge-end-1-expected.txt:
  • editing/pasteboard/merge-end-2-expected.txt:
  • editing/pasteboard/merge-end-3-expected.txt:
  • editing/pasteboard/merge-end-4-expected.txt:
  • editing/pasteboard/merge-end-5-expected.txt:
  • editing/pasteboard/merge-end-blockquote-expected.txt:
  • editing/pasteboard/merge-end-borders-expected.txt:
  • editing/pasteboard/merge-end-list-expected.txt:
  • editing/pasteboard/merge-end-table-expected.txt:
  • editing/pasteboard/paste-line-endings-007-expected.txt:
  • editing/pasteboard/paste-line-endings-008-expected.txt:
  • editing/pasteboard/paste-line-endings-009-expected.txt:
  • editing/pasteboard/paste-match-style-001-expected.txt:
  • editing/pasteboard/paste-text-002-expected.txt:
  • editing/pasteboard/paste-text-003-expected.txt:
  • editing/pasteboard/paste-text-004-expected.txt:
  • editing/pasteboard/paste-text-005-expected.txt:
  • editing/pasteboard/paste-text-006-expected.txt:
  • editing/pasteboard/paste-text-008-expected.txt:
  • editing/pasteboard/paste-text-009-expected.txt:
  • editing/pasteboard/paste-text-010-expected.txt:
  • editing/pasteboard/smart-paste-001-expected.txt:
  • editing/pasteboard/smart-paste-002-expected.txt:
  • editing/pasteboard/smart-paste-003-expected.txt:
  • editing/pasteboard/smart-paste-004-expected.txt:
  • editing/pasteboard/smart-paste-005-expected.txt:
  • editing/pasteboard/smart-paste-006-expected.txt:
  • editing/pasteboard/smart-paste-007-expected.txt:
  • editing/pasteboard/smart-paste-008-expected.txt:
  • editing/style/create-block-for-style-001-expected.txt:
  • editing/style/create-block-for-style-002-expected.txt:
  • editing/style/create-block-for-style-003-expected.txt:
  • editing/style/create-block-for-style-004-expected.txt:
  • editing/style/create-block-for-style-007-expected.txt:
  • editing/style/create-block-for-style-009-expected.txt:
  • editing/style/create-block-for-style-010-expected.txt:
  • editing/style/create-block-for-style-011-expected.txt:
  • editing/style/create-block-for-style-012-expected.txt:
  • editing/style/create-block-for-style-013-expected.txt:
  • editing/style/smoosh-styles-001-expected.txt:
  • editing/style/smoosh-styles-002-expected.txt:
  • editing/style/style-boundary-005-expected.txt:
  • editing/unsupported-content/list-delete-001-expected.txt:

WebCore:

Reviewed by darin


<rdar://problem/5107422>
TOT REGRESSION: Delete key fails to delete text, and cursor disappears in Mail.app
<rdar://problem/5107413>
TOT REGRESSION: Crash deleting text in email (infinite recursion in shouldInsertNode)

  • editing/CompositeEditCommand.cpp: (WebCore::CompositeEditCommand::moveParagraphs): Back out the call to shouldInsert until we add WebViewInsertActionMoved, because Mail returns NO from shouldInsert for WebViewInsertActionPasted to override our Paste operation.
2:39 PM Changeset in webkit [20688] by andersca
  • 5 edits
    4 adds in trunk

LayoutTests:

Reviewed by Mitz.

Add tests for innerWidth/innerHeight and clientWidth/clientHeight in quirks mode.

Modify the strict clientWidth/clientHeight test to test the behavior with scrollbars visible.


  • fast/dom/client-width-height-quirks-expected.txt: Added.
  • fast/dom/client-width-height-quirks.html: Added.
  • fast/dom/client-width-height.html:
  • fast/dom/inner-width-height-expected.txt: Added.
  • fast/dom/inner-width-height.html: Added.

WebCore:

Reviewed by Mitz.

Make window.innerWidth/innerHeight include the size of any present scrollbar to match Firefox.


Make document.body.clientWidth/clientHeight return the width/height of the visible content when in
quirks mode to match WinIE.


  • bindings/js/kjs_window.cpp: (KJS::Window::getValueProperty): Return the width and height of the scroll view, not the visible width and height.


  • dom/Element.cpp: (WebCore::Element::clientWidth): (WebCore::Element::clientHeight): Return the visible size if the element is the body element and we're in quirks mode.
1:51 PM Changeset in webkit [20687] by spadma
  • 4 edits in S60/branches/3.1m/WebKit

2007-03-30 ligman <​joseph.ligman@nokia.com>

Reviewed by Sachin.
DESC: When the request dispatcher does not define a loader

no pending transaction will exist, so set this undefined status
to KErrCancel so the callers don't wait indefinitely. Also, create, use,
and close the download property string when it's needed to avoid closing
the rstring after the string pool which it's using has somehow gone away.

​http://bugs.webkit.org/show_bug.cgi?id=13232

  • BrowserView/src/WebKitLoader.cpp: (CWebKitLoader::LoadPageL):
  • ResourceLoader/inc/BaseTransaction.h:
  • ResourceLoader/src/HttpTransaction.cpp: (CHttpTransaction::~CHttpTransaction): (CHttpTransaction::MHFRunL): (CHttpTransaction::MHFRunError): (CHttpTransaction::HandleGotResponseHeadersL):
1:50 PM Changeset in webkit [20686] by spadma
  • 4 edits in S60/trunk/WebKit

2007-03-30 ligman <​joseph.ligman@nokia.com>

Reviewed by Sachin.
DESC: When the request dispatcher does not define a loader

no pending transaction will exist, so set this undefined status
to KErrCancel so the callers don't wait indefinitely. Also, create, use,
and close the download property string when it's needed to avoid closing
the rstring after the string pool which it's using has somehow gone away.

​http://bugs.webkit.org/show_bug.cgi?id=13232

  • BrowserView/src/WebKitLoader.cpp: (CWebKitLoader::LoadPageL):
  • ResourceLoader/inc/BaseTransaction.h:
  • ResourceLoader/src/HttpTransaction.cpp: (CHttpTransaction::~CHttpTransaction): (CHttpTransaction::MHFRunL): (CHttpTransaction::MHFRunError): (CHttpTransaction::HandleGotResponseHeadersL):
1:15 PM Changeset in webkit [23222] by mjs
  • 2 edits in branches/WindowsMerge/WebCore

Reviewed by Alice.

  • fixed <rdar://problem/5107069> WebCore leaks a copy of the CFHTTPMessage CFDictionary (from CFHTTPMessageCopyAllHeaderFields)
  • platform/network/cf/ResourceResponseCFNet.cpp: (WebCore::ResourceResponse::doUpdateResourceResponse): Use RetainPtr (once to avoid a leak and once just as style cleanup).
12:22 PM Changeset in webkit [20685] by adele
  • 3 edits in trunk/WebCore

Reviewed by Darin.

Updated fix for <rdar://problem/5107095> REGRESSION: disabled text color calculations are slowing down launchtime measurements

  • platform/graphics/Color.cpp: Moved optimization for lightening black and darkening white into the Color class. (WebCore::Color::light): (WebCore::Color::dark):
  • rendering/RenderTextControl.cpp: (WebCore::disabledTextColor): Skip the difference calculation for black text, which will always be lightened.
11:35 AM Changeset in webkit [20684] by andersca
  • 2 edits in trunk/WebCore

Reviewed by Tim.

<rdar://problem/4649516>
Turn off slow script dialog or crank up time that makes it come up


  • bindings/js/kjs_binding.cpp: (KJS::ScriptInterpreter::ScriptInterpreter): Increase the timeout to 10 seconds to match Firefox.
11:30 AM Changeset in webkit [23221] by andersca
  • 3 edits in branches/WindowsMerge/WebCore

Reviewed by Adam.

Fix crashes seen when running the plugin layout tests.


  • plugins/win/PluginPackageWin.cpp: (WebCore::PluginPackageWin::load): Fill in the enumerate pointer, not the evaluate pointer (it's already been filled in).


  • plugins/win/PluginStreamWin.cpp: (WebCore::PluginStreamWin::didFinishLoading): Handle the case where there was no plugin stream data at all.
7:51 AM Changeset in webkit [20683] by yongjzha
  • 3 edits in S60/branches/3.1m/WebKit

2007-03-27 raalexan <​rasvan.alexandrescu@nokia.com>

Reviewed by Yongjun.
DESC: Connection shown as Normal in secure page
​http://bugs.webkit.org/show_bug.cgi?id=13206
TSW ID: HMNN-6ZGH6J

  • BrowserControl/src/BrCtl.cpp: (CBrCtl::CertInfo):
  • BrowserControl/src/WmlDispatcher.cpp: (CWmlDispatcher::HeadersL):
7:44 AM Changeset in webkit [20682] by yongjzha
  • 2 edits in S60/trunk/WebKit

rathnasa, reviewed by Yongjun.

DESC: Reset the iDownloadContent flag properly.

TSW Id: TKWG-6Z477F-Browser auto exit when browsing web Page

​http://bugs.webkit.org/show_bug.cgi?id=13207

  • ResourceLoader/src/DefaultContentHandler.cpp: (CDefaultContentHandler::ProcessHeadersL):
7:39 AM Changeset in webkit [20681] by yongjzha
  • 4 edits in S60/branches/3.1m/WebKit

2007-04-02 hongzhao <set EMAIL_ADDRESS environment variable>

Reviewed by ​yongjun.zhang@nokia.com
DESC: [S60]Animated images with DRM content won't be displayed in Browser in 3.1m.
Fix: Handle DRM image content during construct the content.
​http://bugs.webkit.org/show_bug.cgi?id=13258

  • BrowserCore/Renderers/inc/AnimationDecoder.h:
  • BrowserCore/Renderers/inc/StaticImageDecoder.h:
  • BrowserCore/Renderers/src/StaticImageDecoder.cpp: (CRawData::ConstructL): (CRawData::DecodeDRMImageContentL): (CStaticImageDecoder::LoadNextImage):
7:29 AM Changeset in webkit [20680] by yongjzha
  • 3 edits in S60/branches/3.1m/WebKit

2007-04-02 ligman <​joseph.ligman@nokia.com>

Reviewed by ​yongjun.zhang@nokia.com
DESC: Animated gif in xhtml fails. Start the animation timer for single scaled animated bitmaps.
​http://bugs.webkit.org/show_bug.cgi?id=13254

  • BrowserCore/Renderers/inc/ImageRenderer.h: (CImageRenderer::GetParent):
  • BrowserCore/Renderers/src/ImageRenderer.cpp: (CImageRenderer::DrawImageInRect): (NextFrameCb): (CImageRenderer::NextFrame): (CImageRenderer::InitAnimationCallback):
7:26 AM Changeset in webkit [20679] by yongjzha
  • 3 edits in S60/trunk/WebKit

ligman, reviewed by <reviewer>

DESC: Animated gif in xhtml fails. Start animation timer for single animated gifs
​http://bugs.webkit.org/show_bug.cgi?id=13255

  • BrowserCore/Renderers/inc/ImageRenderer.h: (CImageRenderer::GetParent):
  • BrowserCore/Renderers/src/ImageRenderer.cpp: (CImageRenderer::DrawImageInRect): (NextFrameCb): (CImageRenderer::NextFrame): (CImageRenderer::InitAnimationCallback):
5:56 AM Changeset in webkit [20678] by bdash
  • 8 edits in trunk

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

Reviewed by Darin.

Covered by existing editing tests.

  • css/CSSPrimitiveValue.cpp: (WebCore::CSSPrimitiveValue::getFloatValue): Added ASSERTs that the requested conversion is from a scalar type to a scalar type. The intermittent failures in the test were the result of trying to convert an identifier type to a scalar type.
  • editing/ApplyStyleCommand.cpp: (WebCore::StyleChange::checkForLegacyHTMLStyleChange): Added a check that the font-size property value is an absolute length. If it is not (such as in the case of relative units or keywords like "x-large"), then no corresponding 'size' attribute will be generated for legacy HTML.

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

Reviewed by Darin.

  • editing/style/block-style-004-expected.txt:
  • editing/style/block-style-005-expected.txt:
  • editing/style/block-style-006-expected.txt:
  • editing/style/fontsize-1-expected.txt:
5:51 AM Changeset in webkit [20677] by bdash
  • 2 edits
    1 add in trunk/WebCore

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

Reviewed by Darin.

  • manual-tests/stale-currentEvent.html: Added.
  • page/EventHandler.cpp: (WebCore::EventHandler::hoverTimerFired): Instead of faking a mouse event, just hit-test the last mouse event coordinates and update for hover state changes. This eliminates the use of currentEvent, which can be a non-mouse event or even a stale event.
4:24 AM Changeset in webkit [20676] by bdash
  • 2 edits in trunk/WebKitTools

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

Reviewed by Antti.

  • BuildSlaveSupport/build-launcher-dmg: Use bzip2 compression in nightly build disk images rather than gzip.
Note: See TracTimeline for information about the timeline view.