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

Timeline



Mar 17, 2008:

11:12 PM Changeset in webkit [31120] by timothy@apple.com
  • 2 edits in trunk/WebCore

2008-03-17 Timothy Hatcher <​timothy@apple.com>

Reviewed by Mark Rowe.

Bug 17908: Various bugs in the Console completion code
​http://bugs.webkit.org/show_bug.cgi?id=17908

  • page/inspector/ConsolePanel.js: (WebInspector.ConsolePanel.complete): Moved the code that checked for the caret being at the end of the prompt into the _caretAtEndOfPrompt helper function. (WebInspector.ConsolePanel.messagesSelectStart): Clear and redo the auto complete when the selection changes. (WebInspector.ConsolePanel._caretInsidePrompt): Fixed a logic error that always caused a false result. (WebInspector.ConsolePanel._caretAtEndOfPrompt): Added. Tests if the selection is a caret at the end of the prompt. (WebInspector.ConsolePanel._moveCaretToEndOfPrompt): Changed the offset to use the childNodes length. This makes sure the caret is at the end when there are multiple text nodes in the prompt.
10:36 PM Changeset in webkit [31119] by oliver@apple.com
  • 3 edits in trunk/JavaScriptCore

Optimise multi-scope function call resolution

Reviewed by Geoff

Refactor multiscope variable resolution and use to add
optimised FunctionCallResolveNode subclasses.

2.6% gain in sunspider performance, *25%* gain in controlflow-recursive

10:03 PM Changeset in webkit [31118] by eric@webkit.org
  • 2 edits in trunk/WebKitTools

Reviewed by Dan Bernstein.

Fix bogus argCount check breaking plugin test.

  • DumpRenderTree/TestNetscapePlugIn.subproj/PluginObject.cpp:
9:36 PM Changeset in webkit [31117] by mitz@apple.com
  • 4 edits in trunk/WebCore

2008-03-17 Dan Bernstein <​mitz@apple.com>

Rubber-stamped by Dave Hyatt.

  • FloatingObject cleanup

Renamed FloatingObject's data members as follows: node -> m_renderer,
startY -> m_top, endY -> m_bottom, left -> m_left, width -> m_width,
and noPaint -> !m_shouldPaint, reversing the meaning of the flag.

Also addressed the FIXME in RenderBlock::containsFloat().

  • rendering/RenderBlock.cpp: (WebCore::RenderBlock::repaintOverhangingFloats): (WebCore::RenderBlock::paintFloats): (WebCore::RenderBlock::insertFloatingObject): (WebCore::RenderBlock::removeFloatingObject): (WebCore::RenderBlock::positionNewFloats): (WebCore::RenderBlock::leftRelOffset): (WebCore::RenderBlock::rightRelOffset): (WebCore::RenderBlock::nextFloatBottomBelow): (WebCore::RenderBlock::floatBottom): (WebCore::RenderBlock::floatRect): (WebCore::RenderBlock::lowestPosition): (WebCore::RenderBlock::rightmostPosition): (WebCore::RenderBlock::leftmostPosition): (WebCore::RenderBlock::leftBottom): (WebCore::RenderBlock::rightBottom): (WebCore::RenderBlock::clearFloats): (WebCore::RenderBlock::addOverhangingFloats): (WebCore::RenderBlock::addIntrudingFloats): (WebCore::RenderBlock::containsFloat): Changed to return false if the floats lists exists but is empty, since line layout code no longer relies on the buggy behavior. (WebCore::RenderBlock::nodeAtPoint): (WebCore::RenderBlock::adjustForBorderFit):
  • rendering/RenderBlock.h: (WebCore::RenderBlock::containsFloats): (WebCore::RenderBlock::FloatingObject::FloatingObject):
  • rendering/bidi.cpp: (WebCore::RenderBlock::layoutInlineChildren): (WebCore::RenderBlock::matchedEndLine):
9:04 PM Changeset in webkit [31116] by mitz@apple.com
  • 6 edits
    38 adds in trunk

WebCore:

2008-03-17 Dan Bernstein <​mitz@apple.com>

Reviewed by Dave Hyatt.

  • allow incremental relayout of blocks that contain floats

Tests: fast/repaint/line-flow-with-floats-[1-9].html

  • rendering/RenderBlock.cpp: (WebCore::RenderBlock::clearFloatsIfNeeded): Cleaned up by moving most of the function body out of an if statement which was replaced with an early return. (WebCore::RenderBlock::insertFloatingObject): Cleaned up by moving most of the function body out of an if statement and moving the ASSERT, which is a crash in release builds, to the beginning. Made this function set the m_isDescendant flag of floating objects it creates. (WebCore::RenderBlock::removeFloatingObject): Added a call to markLinesDirtyInVerticalRange() when removing a float from a block with inline children. (WebCore::RenderBlock::markLinesDirtyInVerticalRange): Added. Marks the lines in the given range as dirty. (WebCore::RenderBlock::clearFloats): Added code to detect changes to the geometry of floats intruding into this block from other blocks and mark any lines whose available width has changed as a result as dirty.
  • rendering/RenderBlock.h: (WebCore::RenderBlock::FloatWithRect::FloatWithRect): Added a structure to cache a float with its position and size. (WebCore::RenderBlock::FloatingObject::FloatingObject): Added an m_isDescendant flag, used by clearFloats() to distinguish between floats entering the block from outside and floats internal to the block.
  • rendering/RootInlineBox.h: (WebCore::RootInlineBox::floats): Added. (WebCore::RootInlineBox::floatsPtr): Added. (WebCore::RootInlineBox::Overflow::Overflow): Added a data member to hold the floats originating on the line.
  • rendering/bidi.cpp: (WebCore::RenderBlock::layoutInlineChildren): Made the existence of floats not force a full layout. Changed to cache the geometry of floats in the block and detect changes to it. If a float's size or position changes, all lines from that point on are treated as dirty. An exception is a change in the dimensions of a float on an otherwise-clean line, which only dirties lines potentially affected by the change (see determineStartPosition()). Added code to update each RootInlineBox's set of floats as lines are laid out. Added code to shift floats belonging to clean lines in the end along with the lines. (WebCore::RenderBlock::determineStartPosition): Made this function look for changes to floats' dimensions and mark lines as dirty accordingly. Also look for new floats and if found, cause a full layout. Added code to re-add floats belonging to clean lines. (WebCore::RenderBlock::matchedEndLine): Added checking that the clean lines in the end can be shifted vertically as needed, i.e. that the available width along the way is uniform.

LayoutTests:

2008-03-17 Dan Bernstein <​mitz@apple.com>

Reviewed by Dave Hyatt.

  • test incremental relayout of blocks that contain floats
  • fast/repaint/line-flow-with-floats-1.html: Added.
  • fast/repaint/line-flow-with-floats-2.html: Added.
  • fast/repaint/line-flow-with-floats-3.html: Added.
  • fast/repaint/line-flow-with-floats-4.html: Added.
  • fast/repaint/line-flow-with-floats-5.html: Added.
  • fast/repaint/line-flow-with-floats-6.html: Added.
  • fast/repaint/line-flow-with-floats-7.html: Added.
  • fast/repaint/line-flow-with-floats-8.html: Added.
  • fast/repaint/line-flow-with-floats-9.html: Added.
  • fast/repaint/resources/line-flow-with-floats.html: Added.
  • fast/repaint/resources/line-flow-with-floats.js: Added.
  • platform/mac/fast/repaint/line-flow-with-floats-1-expected.checksum: Added.
  • platform/mac/fast/repaint/line-flow-with-floats-1-expected.png: Added.
  • platform/mac/fast/repaint/line-flow-with-floats-1-expected.txt: Added.
  • platform/mac/fast/repaint/line-flow-with-floats-2-expected.checksum: Added.
  • platform/mac/fast/repaint/line-flow-with-floats-2-expected.png: Added.
  • platform/mac/fast/repaint/line-flow-with-floats-2-expected.txt: Added.
  • platform/mac/fast/repaint/line-flow-with-floats-3-expected.checksum: Added.
  • platform/mac/fast/repaint/line-flow-with-floats-3-expected.png: Added.
  • platform/mac/fast/repaint/line-flow-with-floats-3-expected.txt: Added.
  • platform/mac/fast/repaint/line-flow-with-floats-4-expected.checksum: Added.
  • platform/mac/fast/repaint/line-flow-with-floats-4-expected.png: Added.
  • platform/mac/fast/repaint/line-flow-with-floats-4-expected.txt: Added.
  • platform/mac/fast/repaint/line-flow-with-floats-5-expected.checksum: Added.
  • platform/mac/fast/repaint/line-flow-with-floats-5-expected.png: Added.
  • platform/mac/fast/repaint/line-flow-with-floats-5-expected.txt: Added.
  • platform/mac/fast/repaint/line-flow-with-floats-6-expected.checksum: Added.
  • platform/mac/fast/repaint/line-flow-with-floats-6-expected.png: Added.
  • platform/mac/fast/repaint/line-flow-with-floats-6-expected.txt: Added.
  • platform/mac/fast/repaint/line-flow-with-floats-7-expected.checksum: Added.
  • platform/mac/fast/repaint/line-flow-with-floats-7-expected.png: Added.
  • platform/mac/fast/repaint/line-flow-with-floats-7-expected.txt: Added.
  • platform/mac/fast/repaint/line-flow-with-floats-8-expected.checksum: Added.
  • platform/mac/fast/repaint/line-flow-with-floats-8-expected.png: Added.
  • platform/mac/fast/repaint/line-flow-with-floats-8-expected.txt: Added.
  • platform/mac/fast/repaint/line-flow-with-floats-9-expected.checksum: Added.
  • platform/mac/fast/repaint/line-flow-with-floats-9-expected.png: Added.
  • platform/mac/fast/repaint/line-flow-with-floats-9-expected.txt: Added.
7:36 PM Google Summer of Code 2008 edited by eric@webkit.org
(diff)
7:34 PM Google Summer of Code 2008 edited by eric@webkit.org
(diff)
7:33 PM Google Summer of Code 2008 edited by eric@webkit.org
(diff)
7:32 PM Google Summer of Code 2008 edited by eric@webkit.org
(diff)
7:32 PM Google Summer of Code 2008 edited by eric@webkit.org
(diff)
7:31 PM Google Summer of Code 2008 edited by eric@webkit.org
(diff)
6:54 PM Changeset in webkit [31115] by ddkilzer@apple.com
  • 2 edits in trunk/JavaScriptCore

Don't define PLATFORM(MIDDLE_ENDIAN) on little endian ARM.

Reviewed by Darin.

See <​http://bugs.webkit.org/show_bug.cgi?id=15416#c13>.

  • wtf/Platform.h: Added check for !defined(ARMEL) when defining PLATFORM(MIDDLE_ENDIAN).
4:33 PM Changeset in webkit [31114] by oliver@apple.com
  • 11 edits in trunk/JavaScriptCore

Add fast multi-level scope lookup

Reviewed by Geoff, Darin and Weinig

Add logic and AST nodes to provide rapid variable resolution across
static scope boundaries. This also adds logic that allows us to skip
any static scopes that do not contain the variable to be resolved.

This results in a ~2.5% speedup in SunSpider, and gives a 25-30% speedup
in some simple and ad hoc closure and global variable access tests.

4:22 PM Google Summer of Code 2008 edited by eric@webkit.org
(diff)
4:08 PM Changeset in webkit [31113] by mitz@apple.com
  • 2 edits in trunk/WebKitSite

2008-03-17 Dan Bernstein <​mitz@apple.com>

Reviewed by Sam Weinig.

  • link "backtrace" to the page that explains how to get one
  • quality/bugwriting.html:
2:47 PM Changeset in webkit [31112] by eric@webkit.org
  • 8 edits
    4 adds in trunk

Reviewed by darin.

Fix _NPN_IntFromIdentifier (and export the symbol for use!)

Test: plugins/netscape-identifier-conversion.html

  • WebCore.NPAPI.exp:
  • bridge/npruntime.cpp: (_NPN_IntFromIdentifier):
2:38 PM Changeset in webkit [31111] by weinig@apple.com
  • 4 edits in trunk

WebCore:

2008-03-17 Sam Weinig <​sam@webkit.org>

Reviewed by Dan Bernstein.

Add HTMLCollection constructor to the Window object. (Omission noticed by Harri Porten)

  • page/DOMWindow.idl:

LayoutTests:

2008-03-17 Sam Weinig <​sam@webkit.org>

Reviewed by Dan Bernstein.

Add HTMLCollection constructor to the Window object. (Omission noticed by Harri Porten)

  • fast/dom/Window/window-properties-expected.txt:
2:34 PM Changeset in webkit [31110] by zecke@webkit.org
  • 2 edits in trunk/WebCore

2008-03-17 Holger Hans Peter Freyther <​zecke@selfish.org>

Unreviewed Gtk+ build fix.

  • platform/ScrollView.h: make it public
  • platform/gtk/ScrollViewGtk.cpp: remove const
2:33 PM Changeset in webkit [31109] by zecke@webkit.org
  • 5 edits in trunk/WebCore

2008-03-17 Julien Chaffraix <​julien.chaffraix@gmail.com>

Reviewed by Holger.

<​http://bugs.webkit.org/show_bug.cgi?id=17754>

  • Implement ResourceHandle::loadResourceSynchronously to dispatch synchronous requests.
  • Implement WebCoreSynchronousLoader, the ResourceHandleClient which holds the network data, response and error for us during the transfert.
  • platform/network/ResourceHandleInternal.h: Remove trailing white space.
  • platform/network/curl/ResourceHandleCurl.cpp: Add WebCoreSynchronousLoader (WebCore::WebCoreSynchronousLoader::resourceResponse): (WebCore::WebCoreSynchronousLoader::resourceError): (WebCore::WebCoreSynchronousLoader::data): (WebCore::WebCoreSynchronousLoader::WebCoreSynchronousLoader): (WebCore::WebCoreSynchronousLoader::didReceiveResponse): (WebCore::WebCoreSynchronousLoader::didReceiveData): (WebCore::WebCoreSynchronousLoader::didFinishLoading): (WebCore::WebCoreSynchronousLoader::didFail): (WebCore::ResourceHandle::loadResourceSynchronously): Implement method using WebCoreSynchronousLoader.
  • platform/network/curl/ResourceHandleManager.cpp: (WebCore::ResourceHandleManager::dispatchSynchronousJob): (WebCore::ResourceHandleManager::startJob): (WebCore::ResourceHandleManager::initializeHandle): Handle initialization method used both for synchronous and asynchronous job.
  • platform/network/curl/ResourceHandleManager.h:
2:27 PM Changeset in webkit [31108] by Darin Adler
  • 2 edits in trunk/WebCore

2008-03-17 Darin Adler <Darin Adler>

  • try to fix GTK build
  • platform/ScrollView.h: Make setGtkAdjustments adjustment. And non-virtual (why was it virtual?).
1:35 PM Changeset in webkit [31107] by mrowe@apple.com
  • 4 edits in branches/Safari-3-1-branch

Versioning.

1:34 PM Changeset in webkit [31106] by Adam Roben
  • 3 edits
    2 adds in trunk/WebCore

2008-03-17 Rodney Dawes <​dobey@wayofthemonkey.com>

Fix Bug 17898: Split PluginMessageThrottlerWin into its own files

<​http://bugs.webkit.org/show_bug.cgi?id=17898>

Move the PluginMessageThrottlerWin class into its own files, in
preparation for refactoring PluginView code to be shared.

Reviewed and tweaked by Adam Roben.

  • WebCore.vcproj/WebCore.vcproj:
  • plugins/PluginView.h:
  • plugins/win/PluginMessageThrottlerWin.cpp: Added.
  • plugins/win/PluginMessageThrottlerWin.h: Added.
  • plugins/win/PluginViewWin.cpp:
1:33 PM Changeset in webkit [31105] by mrowe@apple.com
  • 1 copy in tags/Safari-5525.15

New tag.

1:21 PM Changeset in webkit [31104] by Adam Roben
  • 2 edits in trunk/WebCore

More Windows build fixes after r31098

  • platform/ScrollView.h: Make some more methods public.
1:18 PM Changeset in webkit [31103] by Adam Roben
  • 2 edits in trunk/WebCore

Windows and Qt build fixes after r31098

  • platform/ScrollView.h: Mark methods public that still need to be so.
12:28 PM Changeset in webkit [31102] by mrowe@apple.com
  • 2 edits in branches/Safari-3-1-branch/WebCore

Merge r31069.

11:58 AM Changeset in webkit [31101] by timothy@apple.com
  • 2 edits in trunk/WebCore

2008-03-16 Timothy Hatcher <​timothy@apple.com>

Reviewed by Darin Adler.

Bug 17883: Console completion should support bracket notation
​http://bugs.webkit.org/show_bug.cgi?id=17883

Also fixes a bug where the Inspector's window object was used instead of the
inspected window object.

  • page/inspector/ConsolePanel.js: (WebInspector.ConsolePanel.complete): Add a comment about the _backwardsRange call. (WebInspector.ConsolePanel.completions): Add a comment about the _backwardsRange call. Check the last character of the expression for a dot or bracket. Fallback to the InspectorController.inspectedWindow() instead of window, this was a bad bug. If the expression caused an exception, just consider the prefix a window property. When bracket notation is used remember what quote was used and compared property names with that quote surrounding it. Also escape the property name for the quote and backslash.
11:01 AM Changeset in webkit [31100] by Darin Adler
  • 12 edits
    22 adds
    7 deletes in trunk

WebCore:

2008-03-17 Robert Blaut <​webkit@blaut.biz>

Reviewed by Darin.

Fix for bug ​http://bugs.webkit.org/show_bug.cgi?id=17696
Set default margin-bottom for form element in quirk mode
and be compatible with Gecko.

Tests: fast/css/margin-bottom-form-element-quirk.html

fast/css/margin-bottom-form-element-strict.html

  • css/html4.css:
  • css/quirks.css:

LayoutTests:

2008-03-17 Robert Blaut <​webkit@blaut.biz>

Reviewed by Darin.

Fix for bug ​http://bugs.webkit.org/show_bug.cgi?id=17696
Set default margin-bottom for form element in quirk mode
and be compatible with Gecko.

  • fast/css/margin-bottom-form-element-quirk.html: Added.
  • fast/css/margin-bottom-form-element-strict.html: Added.
  • fast/frames/viewsource-empty-attribute-value-expected.txt:
  • platform/mac/fast/block/margin-collapse/103-expected.checksum:
  • platform/mac/fast/block/margin-collapse/103-expected.png:
  • platform/mac/fast/block/margin-collapse/103-expected.txt:
  • platform/mac/fast/css/margin-bottom-form-element-quirk-expected.checksum: Added.
  • platform/mac/fast/css/margin-bottom-form-element-quirk-expected.png: Added.
  • platform/mac/fast/css/margin-bottom-form-element-quirk-expected.txt: Added.
  • platform/mac/fast/css/margin-bottom-form-element-strict-expected.checksum: Added.
  • platform/mac/fast/css/margin-bottom-form-element-strict-expected.png: Added.
  • platform/mac/fast/css/margin-bottom-form-element-strict-expected.txt: Added.
  • platform/mac/tables/mozilla/bugs/bug44505-expected.checksum:
  • platform/mac/tables/mozilla/bugs/bug44505-expected.png:
  • platform/mac/tables/mozilla/bugs/bug44505-expected.txt:
  • platform/mac/tables/mozilla/bugs/bug51727-expected.checksum:
  • platform/mac/tables/mozilla/bugs/bug51727-expected.png:
  • platform/mac/tables/mozilla/bugs/bug51727-expected.txt:
  • platform/mac/tables/mozilla/bugs/bug52505-expected.checksum:
  • platform/mac/tables/mozilla/bugs/bug52505-expected.png:
  • platform/mac/tables/mozilla/bugs/bug52505-expected.txt:
  • platform/mac/tables/mozilla/bugs/bug52506-expected.checksum:
  • platform/mac/tables/mozilla/bugs/bug52506-expected.png:
  • platform/mac/tables/mozilla/bugs/bug52506-expected.txt:
  • platform/mac/tables/mozilla_expected_failures/bugs/bug2479-2-expected.checksum:
  • platform/mac/tables/mozilla_expected_failures/bugs/bug2479-2-expected.png:
  • platform/mac/tables/mozilla_expected_failures/bugs/bug2479-2-expected.txt:
  • platform/mac/tables/mozilla_expected_failures/bugs/bug56024-expected.checksum:
  • platform/mac/tables/mozilla_expected_failures/bugs/bug56024-expected.png:
  • platform/mac/tables/mozilla_expected_failures/bugs/bug56024-expected.txt:
  • platform/qt/fast/block/margin-collapse/103-expected.txt: Removed.
  • platform/qt/tables/mozilla/bugs/bug44505-expected.txt: Removed.
  • platform/qt/tables/mozilla/bugs/bug51727-expected.txt: Removed.
  • platform/qt/tables/mozilla/bugs/bug52505-expected.txt: Removed.
  • platform/qt/tables/mozilla/bugs/bug52506-expected.txt: Removed.
  • platform/qt/tables/mozilla_expected_failures/bugs/bug2479-2-expected.txt: Removed.
  • platform/qt/tables/mozilla_expected_failures/bugs/bug56024-expected.txt: Removed.
10:38 AM Changeset in webkit [31099] by Antti Koivisto
  • 2 edits in trunk/WebCore

2008-03-17 Antti Koivisto <Antti Koivisto>

Reviewed by Darin.

Speculative fix for ​http://bugs.webkit.org/show_bug.cgi?id=17878
Bug 17878: REGRESSION: Acid3 sometimes crashes Webkit under WebCore::Loader::Host::cancelRequests


I can't reproduce the crash or make a test case for this one but I'm pretty sure this
is the problem. Essentially the same bug as ​http://bugs.webkit.org/show_bug.cgi?id=17862
except in didFail() instead of didFinishLoading().

  • loader/loader.cpp: (WebCore::Loader::Host::didFail):
9:56 AM Changeset in webkit [31098] by Darin Adler
  • 2 edits in trunk/WebCore

2008-03-17 Jan Michael Alonzo <​jmalonzo@unpluggable.com>

Reviewed and tweaked by Darin.

​http://bugs.webkit.org/show_bug.cgi?id=17172
Refactor platform checks in ScrollView.h

  • platform/ScrollView.h: Change #ifs around.
9:43 AM Changeset in webkit [31097] by Darin Adler
  • 3 edits
    3 adds in trunk

WebCore:

2008-03-17 Yuzhu Shen <​yuzhu.shen@gmail.com>

Reviewed by Darin.

Fix the bug: ​http://bugs.webkit.org/show_bug.cgi?id=17760
It is necessary to load the image even when src="".

Test: fast/images/load-img-with-empty-src.html

  • html/HTMLImageLoader.cpp: (WebCore::HTMLImageLoader::updateFromElement): isNull rather than isEmpty.

LayoutTests:

2008-03-17 Yuzhu Shen <​yuzhu.shen@gmail.com>

Reviewed by Darin.

Bug: ​http://bugs.webkit.org/show_bug.cgi?id=17760
Test whether <img> tries to load image with empty src attribute.

  • fast/images/load-img-with-empty-src-expected.txt: Added.
  • fast/images/load-img-with-empty-src.html: Added.
  • fast/images/resources/test-load.jpg: Added.
8:27 AM Changeset in webkit [31096] by Adam Roben
  • 2 edits in trunk/WebCore

2008-03-17 Rodney Dawes <​dobey@wayofthemonkey.com>

Fix GTK+ build from r31094.

  • plugins/gtk/PluginDatabaseGtk.cpp: (PluginDatabase::getPluginsInPaths):
8:25 AM Changeset in webkit [31095] by Adam Roben
  • 3 edits
    2 adds in trunk

Fix Bug 17876: REGRESSION (r31060): Attempting to visit Ofcom page causes crash

WebCore:

Fix Bug 17876: REGRESSION (r31060): Attempting to visit Ofcom page causes crash

<​http://bugs.webkit.org/show_bug.cgi?id=17876>

Reviewed by John.

Test: fast/dom/remove-named-attribute-crash.html

  • dom/NamedAttrMap.cpp: (WebCore::NamedAttrMap::removeAttribute): Store the Attribute we're going to remove from the m_attributes Vector in a RefPtr so it doesn't get deleted when it is removed from the Vector.

LayoutTests:

Test for Bug 17876: REGRESSION (r31060): Attempting to visit Ofcom page causes crash

<​http://bugs.webkit.org/show_bug.cgi?id=17876>

Reviewed by John.

  • fast/dom/remove-named-attribute-crash-expected.txt: Added.
  • fast/dom/remove-named-attribute-crash.html: Added.
7:51 AM Changeset in webkit [31094] by Adam Roben
  • 3 edits
    2 adds in trunk/WebCore

2008-03-17 Rodney Dawes <​dobey@wayofthemonkey.com>

Reviewed by Adam Roben.

Implement PluginDatabase for GTK+ with PluginDatabaseGtk.cpp.
Remove implemented methods from TemporaryLinkStubs.

  • GNUmakefile.am:
  • plugins/gtk:
  • plugins/gtk/PluginDatabaseGtk.cpp:
  • platform/gtk/TemporaryLinkStubs.cpp:
2:47 AM WebKit Team edited by zecke2@selfish.org
Correct my information. I'm not employed by Trolltech (and wasn't) (diff)
1:27 AM Changeset in webkit [31093] by Simon Hausmann
  • 3 edits in trunk/WebCore

Fix the Qt build.

Mar 16, 2008:

9:05 PM Changeset in webkit [31092] by mjs@apple.com
  • 2 edits in trunk/WebCore

2008-03-16 Maciej Stachowiak <​mjs@apple.com>

Not reviewed, just fixing an incomplete comment from the last commit.

  • dom/Range.cpp: (WebCore::Range::surroundContents):
8:59 PM Changeset in webkit [31091] by kevino@webkit.org
  • 3 edits in trunk/WebKit/wx

wx build fix. Make sure we link png/jpeg libraries before wx libraries to get the right symbols.

8:50 PM Changeset in webkit [31090] by mjs@apple.com
  • 3 edits
    2 adds in trunk

WebCore:

2008-03-16 Maciej Stachowiak <​mjs@apple.com>

Reviewed by Darin.


This gets us to 92/100

  • dom/Range.cpp: (WebCore::Range::surroundContents): Check for HIERARCHY_REQUEST_ERR before BAD_BOUNDARYPOINTS_ERR, since Acid3 expects exceptional conditions to be tested in the order that the spec lists them. Also, adjust the HIERARCHY_REQUEST_ERR check. If the start point of the range is in a comment node, the node that would be the parent of a partial replacement is actually the comment node's parent (since comment nodes have character indices), so we should do the HIERARCHY_REQUEST_ERR check based on the parent of the comment node, as for text nodes, even though it will fail later with a different exception because it is not allowed to surround a partially selected non-text node.

LayoutTests:

2008-03-16 Maciej Stachowiak <​mjs@apple.com>

Reviewed by Darin.


  • fast/dom/Range/acid3-surround-contents-expected.txt: Added.
  • fast/dom/Range/acid3-surround-contents.html: Added.
8:47 PM Changeset in webkit [31089] by Darin Adler
  • 21 edits
    2 adds in trunk

WebCore:

2008-03-16 Marvin Decker <​marv.decker@gmail.com>

Reviewed by Darin.

Fix bug 15119: URL query characters that are unencodable in the
request's character set should be converted to XML entities with
non-alphanumeric characters escaped.

Test: http/tests/uri/escaped-entity.html

  • html/FormDataList.cpp: (WebCore::FormDataList::appendString):
  • html/HTMLFormElement.cpp: (WebCore::HTMLFormElement::formData):
  • platform/KURL.cpp: (WebCore::encodeRelativeString):
  • platform/text/String.cpp: (WebCore::String::latin1): (WebCore::String::utf8):
  • platform/text/TextCodec.cpp: (WebCore::TextCodec::unencodableCharReplacement):
  • platform/text/TextCodec.h: (WebCore::):
  • platform/text/TextCodecICU.cpp: (WebCore::urlEscapedEntityCallback): (WebCore::gbkUrlEscapedEntityCallack): (WebCore::TextCodecICU::encode):
  • platform/text/TextCodecICU.h: (WebCore::TextCodecICU::setNeedsGBKFallbacks):
  • platform/text/TextCodecLatin1.cpp: (WebCore::encodeComplexWindowsLatin1): (WebCore::TextCodecLatin1::encode):
  • platform/text/TextCodecLatin1.h:
  • platform/text/TextCodecUTF16.cpp: (WebCore::TextCodecUTF16::encode):
  • platform/text/TextCodecUTF16.h:
  • platform/text/TextCodecUserDefined.cpp: (WebCore::encodeComplexUserDefined): (WebCore::TextCodecUserDefined::encode):
  • platform/text/TextCodecUserDefined.h:
  • platform/text/TextEncoding.cpp: (WebCore::TextEncoding::encode):
  • platform/text/TextEncoding.h:
  • platform/text/mac/TextCodecMac.cpp: (WebCore::TextCodecMac::encode):
  • platform/text/mac/TextCodecMac.h:
  • xml/XMLHttpRequest.cpp: (WebCore::XMLHttpRequest::send):

LayoutTests:

2008-03-16 Marvin Decker <​marv.decker@gmail.com>

Reviewed by Darin.

Fix bug 15119, unencodable characters in URLs should be entity-escaped.

  • http/tests/uri/escaped-entity-expected.txt: Added.
  • http/tests/uri/escaped-entity.html: Added.
8:26 PM Changeset in webkit [31088] by Darin Adler
  • 2 edits in trunk/JavaScriptCore

2008-03-16 weihongzeng <​weihong.zeng@hotmail.com>


Reviewed by Darin Adler.


​http://bugs.webkit.org/show_bug.cgi?id=15416
Add support for mixed-endian processors


  • kjs/dtoa.cpp: Add IEEE_ARM, triggered by PLATFORM(MIDDLE_ENDIAN).
6:29 PM Changeset in webkit [31087] by kevino@webkit.org
  • 12 edits in trunk

Rubber stamped by Darin.

Add set-webkit-configuration support for wx port, and centralize build dir location setting.

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

3:21 PM Changeset in webkit [31086] by Darin Adler
  • 7 edits
    2 copies
    2 adds in trunk

WebCore:

2008-03-16 Darin Adler <Darin Adler>

Reviewed by Mark Rowe.

Tests: fast/forms/textarea-default-value-leading-newline.html

fast/forms/textarea-linewrap-dynamic.html

  • html/HTMLTextAreaElement.cpp: (WebCore::HTMLTextAreaElement::HTMLTextAreaElement): Tweaked names/ (WebCore::HTMLTextAreaElement::selectionStart): Ditto, also use early return and check for < 0 rather than -1 specifically. (WebCore::HTMLTextAreaElement::selectionEnd): Ditto. (WebCore::HTMLTextAreaElement::setSelectionStart): Early return. (WebCore::HTMLTextAreaElement::setSelectionEnd): Ditto. (WebCore::HTMLTextAreaElement::select): Ditto. (WebCore::HTMLTextAreaElement::setSelectionRange): Ditto. (WebCore::HTMLTextAreaElement::parseMappedAttribute): Changed value parsing to treat unknown values as meaning "default" rather than "leave value as-is". Only call setNeedsLayoutAndPrefWidthsRecalc when mode changed. (WebCore::HTMLTextAreaElement::createRenderer): Removed name of unused argument. (WebCore::HTMLTextAreaElement::appendFormData): Updated for name changes. (WebCore::HTMLTextAreaElement::isKeyboardFocusable): Got rid of unnneeded explicit class name in isFocusable call. (WebCore::HTMLTextAreaElement::isMouseFocusable): Ditto. (WebCore::HTMLTextAreaElement::updateFocusAppearance): Tweaked formatting. (WebCore::HTMLTextAreaElement::defaultEventHandler): Ditto. (WebCore::HTMLTextAreaElement::updateValue): Early return. (WebCore::HTMLTextAreaElement::setValue): Got rid of intermediate value to eliminate on small refcount churn. (WebCore::HTMLTextAreaElement::defaultValue): Got rid of unneeded redundant string length checks, since String already checks all indexing and returns 0. (WebCore::HTMLTextAreaElement::setDefaultValue): Added code to normalize line endings and add a leading line ending to fix cases where the first character is a newline. (WebCore::HTMLTextAreaElement::accessKeyAction): Removed name of unused arg. (WebCore::HTMLTextAreaElement::accessKey): Changed return value to avoid refcount churn. (WebCore::HTMLTextAreaElement::selection): Updated names and use < 0 instead of -1 specifically.
  • html/HTMLTextAreaElement.h: Replaced wrap function with more-specific shouldWrapText one. Changed return value of accessKey. Made WrapMethod enum provate and renamed the values. Renamed cachedSelStart and cachedSelEnd to m_cachedSelectionStart and m_cachedSelectionEnd.
  • rendering/RenderTextControl.cpp: (WebCore::RenderTextControl::createInnerTextStyle): Updated for change to HTMLTextAreaElement wrap function.

LayoutTests:

2008-03-16 Darin Adler <Darin Adler>

Reviewed by Mark Rowe.

  • fast/forms/textarea-default-value-leading-newline-expected.txt: Added.
  • fast/forms/textarea-default-value-leading-newline.html: Copied from fast/forms/textarea-crlf.html.
  • fast/forms/textarea-hard-linewrap-expected.txt: Updated.
  • fast/forms/textarea-hard-linewrap.html: Cleaned up a bit.
  • fast/forms/textarea-linewrap-dynamic-expected.txt: Added.
  • fast/forms/textarea-linewrap-dynamic.html: Copied from fast/forms/textarea-hard-linewrap.html.
2:47 PM Changeset in webkit [31085] by Antti Koivisto
  • 6 edits in trunk/WebCore

Reviewed by Mitz.

Fix http/tests/security/frame-loading-via-document-write.html
which was broken by the preload patch.


Don't print error message when preload fails local file security check.
Some minor refactoring.

  • html/PreloadScanner.cpp: (WebCore::PreloadScanner::emitTag): (WebCore::PreloadScanner::emitCSSRule):
  • loader/Cache.cpp: (WebCore::Cache::requestResource):
  • loader/Cache.h:
  • loader/DocLoader.cpp: (WebCore::DocLoader::requestResource): (WebCore::DocLoader::preload): (WebCore::DocLoader::printPreloadStats):
  • loader/DocLoader.h:
1:53 PM Changeset in webkit [31084] by Antti Koivisto
  • 3 edits
    3 adds in trunk

WebCore:

Reviewed by Darin.


Fix ​http://bugs.webkit.org/show_bug.cgi?id=17862
REGRESSION (r31038): Reproducible crash under DocLoader::checkForReload() at marware.com


This was a memory smasher introduced by the preloading patch.
If a script resource was marked uncacheable, early deletion of the
Request object would cause deletion of the CachedResource too if
it was referred more than once in a single document.

Test: http/tests/misc/uncacheable-script-repeated.html

  • loader/loader.cpp: (WebCore::Loader::Host::servePendingRequests): (WebCore::Loader::Host::didFinishLoading):

LayoutTests:

Reviewed by Darin.


Test for ​http://bugs.webkit.org/show_bug.cgi?id=17862
REGRESSION (r31038): Reproducible crash under DocLoader::checkForReload() at marware.com

  • http/tests/misc/resources/uncacheable-script.cgi: Added.
  • http/tests/misc/uncacheable-script-repeated-expected.txt: Added.
  • http/tests/misc/uncacheable-script-repeated.html: Added.
1:35 PM Changeset in webkit [31083] by Darin Adler
  • 3 edits
    2 adds in trunk

WebCore:

2008-03-16 Yuzhu Shen <​yuzhu.shen@gmail.com>

Reviewed by Darin.

Fix bug ​http://bugs.webkit.org/show_bug.cgi?id=17714
<img href="#"> should go to top of the page.

Test: fast/html/empty-fragment-id-goto-top.html

  • loader/FrameLoader.cpp: (WebCore::FrameLoader::gotoAnchor):

LayoutTests:

2008-03-16 Yuzhu Shen <​yuzhu.shen@gmail.com>

Reviewed by Darin.

Bug: ​http://bugs.webkit.org/show_bug.cgi?id=17714
Test whether clicking <img href="#"> goes to top of the page.

  • fast/html/empty-fragment-id-goto-top-expected.txt: Added.
  • fast/html/empty-fragment-id-goto-top.html: Added.
1:25 PM Changeset in webkit [31082] by Darin Adler
  • 6 edits in trunk

WebCore:

2008-03-16 Thiago Macieira <​thiago.macieira@trolltech.com>

Reviewed by Darin.

Don't use RefPtr in classes you haven't seen the implementation of.

Forward-declaration and declaration of RefPtr<Foo> is ok. But you
cannot *use* said objects until Foo is defined. This is true even for
initialisation with a 0.

Seems the HP aCC compiler is more strict here than gcc.

  • editing/SplitTextNodeCommand.h:
  • page/FrameTree.h:
  • xml/XPathExpressionNode.h:

WebKit/qt:

2008-03-16 Thiago Macieira <​thiago.macieira@trolltech.com>

Reviewed by Simon.

Don't use RefPtr in classes you haven't seen the implementation of.


Forward-declaration and declaration of RefPtr<Foo> is ok. But you
cannot *use* said objects until Foo is defined. This is true even for
initialisation with a 0.


Seems the HP aCC compiler is more strict here than gcc.

  • Api/qwebframe_p.h:
1:17 PM Changeset in webkit [31081] by Darin Adler
  • 4 edits
    2 adds in trunk

WebCore:

2008-03-16 Darin Adler <Darin Adler>

Reviewed by Mitz.

Test: fast/forms/textarea-trailing-newline.html

  • rendering/RenderTextControl.cpp: (WebCore::RenderTextControl::finishText): Added code to strip the trailing newline. It's possible there are some obscure cases where this is not wanted, but I couldn't find any. If someone finds a case where this is bad, we can make the code conditional.

LayoutTests:

2008-03-16 Darin Adler <Darin Adler>

Reviewed by Mitz.

  • fast/forms/textarea-paste-newline.html: Updated test to expect correct behavior instead of expecting the bug.
  • fast/forms/textarea-trailing-newline-expected.txt: Added.
  • fast/forms/textarea-trailing-newline.html: Added.
1:15 PM Changeset in webkit [31080] by Darin Adler
  • 2 edits in trunk/WebCore

2008-03-16 Darin Adler <Darin Adler>

Reviewed by Mitz.

This is causing intermittent crashes on some existing test cases on the buildbot
too; I don't have a 100% test case right now, but it should be easy to add one later
and this does fix a crash in a test we already have.

  • dom/CharacterData.cpp: (WebCore::CharacterData::CharacterData): Initialize m_data to the empty string, not the null string. The class assumes the string can never being null. (WebCore::CharacterData::setData): If asked to set the data to the null string, set it to the empty string instead. This matches what the (non-empty) constructor has always done.
Note: See TracTimeline for information about the timeline view.