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

Timeline



Sep 1, 2010:

11:49 PM Changeset in webkit [66645] by Philippe Normand
  • 2 edits in trunk/WebCore

2010-09-01 Philippe Normand <​pnormand@igalia.com>

Reviewed by Martin Robinson.

[GTK] warning: unused variable "queryResult"
​https://bugs.webkit.org/show_bug.cgi?id=44998

Fix the misbehavior introduced in r66573. The gst_element_query()
call needs to be done whatever the GStreamer version used.

  • platform/graphics/gstreamer/GStreamerGWorld.cpp: (WebCore::GStreamerGWorld::enterFullscreen):
11:45 PM Changeset in webkit [66644] by mitz@apple.com
  • 3 edits in trunk

Updated Radar link

11:43 PM Changeset in webkit [66643] by mitz@apple.com
  • 27 edits in trunk

Add support for autocorrection UI on Mac OS X.
​https://bugs.webkit.org/show_bug.cgi?id=44958
<rdar://problem/7299621>

Patch by Jia Pu <​jpu@apple.com> on 2010-09-01
Reviewed by Dan Bernstein.

WebCore:

Several new member methods are added to EditorClient for communication
between WebCore and WebKit. A new handler, executeCancelOperation(), is
added to EditorCommand.cpp so that WebCore can intercept the ESC key event
to dismiss autocorrection UI. A new DocumentMarker value, RejectedCorrection,
is added to keep track of the corrections that user has rejected, so that it
will not be suggested again later. The autocorrection is driven by a timer.
Every time the editor inserts a new letter, the timer is reset. If the timer
fires, it means neither has user entered any new letter for current word, nor
has he entered whitespace or punctuation to complete the word. In this case,
we query for autocorrection.

  • WebCore.exp.in: Updated
  • dom/DocumentMarker.h: Added RejectedCorrection to indicate word on which user has rejected autocorrection.
  • editing/Editor.cpp:

(WebCore::Editor::Editor): Initialize autocorrection timer.
(WebCore::Editor::~Editor): Make sure autocorrection UI is dismissed before

destroying Editor object.

(WebCore::Editor::markMisspellingsAfterTypingToPosition): Adopted new signature

of markAllMisspellingsAndBadGrammarInRanges().

(WebCore::Editor::markAllMisspellingsAndBadGrammarInRanges): Consolidated all

boolean arguments into one bitfield. This improves readability and allows us to
pass in a bit to indicate whether we want to show autocorrection UI. Also added
code to show autocorrection UI if it is necessary.

(WebCore::Editor::markMisspellingsAndBadGrammar): Adopted new signature of

markAllMisspellingsAndBadGrammarInRanges().

(WebCore::Editor::correctionPanelTimerFired): Check to see if we should show

autocorrection UI when user pauses during typing.

(WebCore::Editor::handleRejectedCorrection): Update user dictionary when an autocorrection is rejected.
(WebCore::Editor::startCorrectionPanelTimer): Start autocorrection timer.
(WebCore::Editor::handleCancelOperation): Dismiss autocorrection UI when ESC key is pressed.

  • editing/Editor.h: Added autocorrection related member methods and variables.
  • editing/EditorCommand.cpp:

(WebCore::executeCancelOperation): Handle ESC key event. Dismiss autocorrection UI.
(WebCore::createCommandMap): Updated for executeCancelOperation().

  • editing/TypingCommand.cpp:

(WebCore::TypingCommand::markMisspellingsAfterTyping): Reset autocorrection timer after insertion.

  • loader/EmptyClients.h:

(WebCore::EmptyEditorClient::showCorrectionPanel): Dummy implementation.
(WebCore::EmptyEditorClient::dismissCorrectionPanel): Ditto

  • page/EditorClient.h: Added methods for communication between WebCore and WebKit regarding autocorrection.
  • page/Frame.cpp:

(WebCore::Frame::respondToChangedSelection): Remove Replacement marker where we used to only remove Spelling

mark, because whenever we need to remove spelling underline we also should remove autocorrection underline.

  • platform/graphics/GraphicsContext.h: Added enum TextCheckingLineStyle to specify the underline for different type of text checking result. Replaced drawLineForMisspellingOrBadGrammar() with drawLineForTextChecking(), which allows autocorrection to have different indication from misspelling and grammatical error.
  • platform/graphics/cairo/GraphicsContextCairo.cpp:

(WebCore::GraphicsContext::drawLineForTextChecking): Replaced drawLineForMisspellingOrBadGrammar() with drawLineForTextChecking().

  • platform/graphics/haiku/GraphicsContextHaiku.cpp:

(WebCore::GraphicsContext::drawLineForTextChecking): Ditto

  • platform/graphics/mac/GraphicsContextMac.mm:

(WebCore::GraphicsContext::drawLineForTextChecking): Ditto

  • platform/graphics/openvg/GraphicsContextOpenVG.cpp:

(WebCore::GraphicsContext::drawLineForTextChecking): Ditto

  • platform/graphics/qt/GraphicsContextQt.cpp:

(WebCore::GraphicsContext::drawLineForTextChecking): Ditto

  • platform/graphics/skia/GraphicsContextSkia.cpp:

(WebCore::GraphicsContext::drawLineForTextChecking): Ditto

  • platform/graphics/win/GraphicsContextCGWin.cpp:

(WebCore::GraphicsContext::drawLineForTextChecking): Ditto

  • platform/graphics/wince/GraphicsContextWince.cpp:

(WebCore::GraphicsContext::drawLineForTextChecking): Ditto

  • platform/graphics/wx/GraphicsContextWx.cpp:

(WebCore::GraphicsContext::drawLineForTextChecking): Ditto

  • rendering/InlineTextBox.cpp:

(WebCore::textCheckingLineStyleForMarkerType): Return line style for a given document marker type.
(WebCore::InlineTextBox::paintSpellingOrGrammarMarker): Replaced drawLineForMisspellingOrBadGrammar() with drawLineForTextChecking().
(WebCore::InlineTextBox::paintDocumentMarkers): Handle new marker value, RejectedCorrection.

WebKit/mac:

See detailed high level description in WebCore/ChangeLog.

  • WebCoreSupport/WebEditorClient.h: Added new member methods declared in EditorClient. Added m_correctionPanelTag to store the ID of current autocorrection UI object.
  • WebCoreSupport/WebEditorClient.mm:

(WebEditorClient::WebEditorClient): Initialize m_correctionPanelTag.
(WebEditorClient::~WebEditorClient): Make sure the autocorrection UI is

dismissed before destroying the object.

(WebEditorClient::respondToChangedSelection): Dismiss autocorrection UI whenever the selection changes.
(WebEditorClient::showCorrectionPanel): Show autocorrection UI.
(WebEditorClient::dismissCorrectionPanel): Dismiss autocorrection UI.

WebKit2:

See detailed high level description in WebCore/ChangeLog.

  • WebProcess/WebCoreSupport/WebEditorClient.cpp:

(WebKit::WebEditorClient::showCorrectionPanel): Dummy implementation.
(WebKit::WebEditorClient::dismissCorrectionPanel): Ditto

  • WebProcess/WebCoreSupport/WebEditorClient.h: Added new methods declared in base class.
11:17 PM Changeset in webkit [66642] by kinuko@chromium.org
  • 4 edits in trunk/WebCore

2010-09-01 Kinuko Yasuda <​kinuko@google.com>

Reviewed by Adam Barth.

[V8] Support readonly Date attribute
​https://bugs.webkit.org/show_bug.cgi?id=43943

Adds Date to non_wrapper_types in CodeGeneratorV8.pm.

New tests will be added for Metadata when we add FileSystem implementation.

  • bindings/scripts/CodeGeneratorV8.pm:
  • fileapi/Metadata.h:
  • fileapi/Metadata.idl:
10:27 PM Changeset in webkit [66641] by commit-queue@webkit.org
  • 5 edits in trunk

2010-09-01 Mihai Parparita <​mihaip@chromium.org>

Reviewed by Chris Marrin.

animations/suspend-resume-animation.html does not pass when accelerated compositing is off
​https://bugs.webkit.org/show_bug.cgi?id=45057

animations/suspend-resume-animation.html now passes with the Chromium DRT
(it still fails with test_shell, since that doesn't have the suspend/resumeAnimations
API implemented)

  • platform/chromium/drt_expectations.txt:
  • platform/chromium/test_expectations.txt:

2010-09-01 Mihai Parparita <​mihaip@chromium.org>

Reviewed by Chris Marrin.

animations/suspend-resume-animation.html does not pass when accelerated compositing is off
​https://bugs.webkit.org/show_bug.cgi?id=45057

Fix incorrect logic when resuming the animation (KeyframeAnimation::startAnimation
and RenderLayerBacking::startAnimation return true when running an accelerated
animation).

  • page/animation/AnimationBase.cpp: (WebCore::AnimationBase::updateStateMachine):
9:47 PM Changeset in webkit [66640] by dpranke@chromium.org
  • 11 edits
    4 adds in trunk/WebKitTools

2010-09-01 Dirk Pranke <​dpranke@chromium.org>

Reviewed by Tony Chang.

Land a patched version of r66542 - change TestRunner to have an
separate cleanup method and fix the ordering of cleanup between
TestRunner and the printing module, and then wrap everything in a
try/finally block to ensure reliable cleanup without needing to
rely on stuff happening in the destructor of the TestRunner.

Also refactor run_webkit_tests.run() to be much smaller and cleaner
by creating a bunch of helper methods and moving more stuff into
the TestRunner class.

This fixes the crash at the end of the linux test run of
new-run-webkit-tests (and undoes the rollout in 66547).

​https://bugs.webkit.org/show_bug.cgi?id=44902

  • Scripts/webkitpy/layout_tests/data/failures/expected/exception.html: Added.
  • Scripts/webkitpy/layout_tests/data/failures/expected/keyboard.html: Added.
  • Scripts/webkitpy/layout_tests/data/passes/error-expected.txt: Added.
  • Scripts/webkitpy/layout_tests/data/passes/error.html: Added.
  • Scripts/webkitpy/layout_tests/data/platform/test/test_expectations.txt:
  • Scripts/webkitpy/layout_tests/layout_package/dump_render_tree_thread.py:
  • Scripts/webkitpy/layout_tests/layout_package/printing.py:
  • Scripts/webkitpy/layout_tests/layout_package/printing_unittest.py:
  • Scripts/webkitpy/layout_tests/port/base.py:
  • Scripts/webkitpy/layout_tests/port/test.py:
  • Scripts/webkitpy/layout_tests/run_webkit_tests.py:
  • Scripts/webkitpy/layout_tests/run_webkit_tests_unittest.py:
9:00 PM Changeset in webkit [66639] by weinig@apple.com
  • 6 edits in trunk/WebKit2

Fix the build.

As with CF, const struct OpaqueType* should be the default type
so that const_casts are not necessary in user code.

  • UIProcess/API/C/WKArray.cpp:

(WKArrayCreate):

  • UIProcess/API/C/WKBase.h:
  • UIProcess/API/C/WKType.cpp:

(WKGetTypeID):
(WKRetain):
(WKRelease):

  • WebProcess/InjectedBundle/API/c/WKBundleAPICast.h:

(toWK):

  • WebProcess/InjectedBundle/API/c/WKBundleBase.h:
8:37 PM Changeset in webkit [66638] by weinig@apple.com
  • 8 edits
    4 adds in trunk/WebKit2

Add mutable array API to WebKit2
​https://bugs.webkit.org/show_bug.cgi?id=45082

Reviewed by Adam Roben.

Add a mutable API to WebKit2 called WKMutableArrayRef. Make it "inherit" from
WKArrayRef in the same way JSObjectRef can inherit from JSValueRef in the JSC
API, by abusing the c type system (WKArrayRef is defined as a const version of
WKMutableArrayRef). Add very basic mutable API to go along with it.

  • Shared/ImmutableArray.h:

(WebKit::ImmutableArray::isMutable):
Add ability to ask it an array is mutable.

  • Shared/MutableArray.cpp: Added.

(WebKit::MutableArray::MutableArray):
(WebKit::MutableArray::~MutableArray):
(WebKit::MutableArray::append):

  • Shared/MutableArray.h: Added.

(WebKit::MutableArray::create):
(WebKit::MutableArray::isMutable):
Sketch out the mutable array interface.

  • UIProcess/API/C/WKAPICast.h:

(toWK):
Now that the opaque types can be const, use template magic to const_cast
it away.

  • UIProcess/API/C/WKBase.h:

Add new type and re-type WKArrayRef as const.

  • UIProcess/API/C/WKMutableArray.cpp: Added.

(WKMutableArrayCreate):
(WKArrayIsMutable):
(WKArrayAppendItem):

  • UIProcess/API/C/WKMutableArray.h: Added.

Fill in basic API.

  • WebKit2.pro:
  • WebKit2.xcodeproj/project.pbxproj:
  • win/WebKit2.vcproj:
  • win/WebKit2Generated.make:

Add files.

8:31 PM Changeset in webkit [66637] by mnaganov@chromium.org
  • 2 edits in trunk/WebKit/chromium

2010-09-01 Mikhail Naganov <​mnaganov@chromium.org>

Reviewed by Yury Semikhatsky.

Remove dependency on JS files for V8 logs processing.

​https://bugs.webkit.org/show_bug.cgi?id=45045

  • WebKit.gyp:
8:14 PM Changeset in webkit [66636] by commit-queue@webkit.org
  • 2 edits in trunk/WebKit2

2010-09-01 Balazs Kelemen <​kb@inf.u-szeged.hu>

Reviewed by Kenneth Rohde Christiansen.

[Qt] Implement PluginInfoStore for UNIX
​https://bugs.webkit.org/show_bug.cgi?id=45038

This implementation is generally just reusing the logic in WebCore since
there is no way to get informations from a UNIX plugin without loading it

  • in contrast to mac bundles and windows dll-s.
  • UIProcess/Plugins/qt/PluginInfoStoreQt.cpp: (WebKit::PluginInfoStore::pluginsDirectories): (WebKit::PluginInfoStore::pluginPathsInDirectory): (WebKit::PluginInfoStore::getPluginInfo): (WebKit::PluginInfoStore::shouldUsePlugin):
7:57 PM Changeset in webkit [66635] by commit-queue@webkit.org
  • 1 edit
    3 adds in trunk/LayoutTests

2010-09-01 Matthew Willis <​lilmatt@flock.com>

Reviewed by Alexey Proskuryakov.

Add tests to ensure any h1-h6 element can close any other h1-h6 element.
​https://bugs.webkit.org/show_bug.cgi?id=15136
<rdar://problem/5762882>

  • fast/html/numbered-header-element-expected.txt: Added.
  • fast/html/numbered-header-element.html: Added.
  • fast/html/script-tests/numbered-header-element.js: Added.
7:23 PM Changeset in webkit [66634] by mrowe@apple.com
  • 2 edits in trunk/WebKitTools

Fix TestNetscapePlugIn to compile without access to QuickDraw private headers.

Reviewed by Adam Roben.

  • DumpRenderTree/TestNetscapePlugIn/main.cpp:
7:23 PM Changeset in webkit [66633] by mrowe@apple.com
  • 4 edits
    1 add in trunk/WebKit/mac

<rdar://problem/8374711> WebKit needs to compile without access to QuickDraw private headers.

Reviewed by Adam Roben.

  • Carbon/HIViewAdapter.m:
  • Carbon/HIWebView.mm:
  • Misc/QuickDrawCompatibility.h: Added.
  • Plugins/WebNetscapePluginView.mm:
7:13 PM Changeset in webkit [66632] by eric.carlson@apple.com
  • 2 edits in trunk/WebCore

2010-09-01 Eric Carlson <​eric.carlson@apple.com>

Reviewed by Darin Adler.

Question mark QuickTime icon when navigating to this MP3 file
​https://bugs.webkit.org/show_bug.cgi?id=44846

Do not disable 'tmet' movie tracks, they carry meta data.

No DRT test because I am not sure it is possible to have one of these tracks in
non-streaming movie. I will add a test later if I figure out how to create a movie.

  • platform/graphics/mac/MediaPlayerPrivateQTKit.mm: (WebCore::MediaPlayerPrivate::disableUnsupportedTracks):
7:11 PM Changeset in webkit [66631] by eric@webkit.org
  • 14 edits in trunk/WebCore

2010-09-01 Eric Seidel <​eric@webkit.org>

Reviewed by Dimitri Glazkov.

Remove rendering types from SubframeLoader and split RenderEmbeddedObject::updateWidget in preparation for moving to DOM
​https://bugs.webkit.org/show_bug.cgi?id=45007

This is just moving around code (and attempting to make it more sane).
I also tried to use more specific types where possible.
The SubframeLoader no longer uses any rendering types in its API.

I split RenderEmbeddedObject::updateWidget into a bunch of functions.
It was obvious before, but now super-obvious after the split that these
new functions do not belong in RenderEmbeddedObject. Instead they
need to be moved to their respective DOM classes. I've added FIXMEs
next to each of the functions which should be moved, but I've not moved
any of the logic out of RenderEmbeddedObject.cpp yet in an effort to
keep this patch as simple as possible.

No functional change, thus no tests.

  • html/HTMLEmbedElement.cpp: (WebCore::HTMLEmbedElement::updateWidget):
  • html/HTMLFrameElementBase.cpp: (WebCore::HTMLFrameElementBase::attach): (WebCore::HTMLFrameElementBase::width): (WebCore::HTMLFrameElementBase::height):
  • html/HTMLFrameOwnerElement.cpp: (WebCore::HTMLFrameOwnerElement::renderPart):
  • html/HTMLFrameOwnerElement.h:
  • html/HTMLObjectElement.cpp: (WebCore::HTMLObjectElement::renderWidgetForJSBindings): (WebCore::HTMLObjectElement::createRenderer): (WebCore::HTMLObjectElement::updateWidget):
  • html/HTMLPlugInElement.cpp: (WebCore::HTMLPlugInElement::renderEmbeddedObject):
  • html/HTMLPlugInElement.h:
  • loader/PluginDocument.cpp: (WebCore::PluginDocumentParser::appendBytes):
  • loader/SubframeLoader.cpp: (WebCore::SubframeLoader::requestObject): (WebCore::SubframeLoader::document): (WebCore::SubframeLoader::loadPlugin):
  • loader/SubframeLoader.h:
  • rendering/RenderEmbeddedObject.cpp: (WebCore::isNetscapePlugin): (WebCore::hasFallbackContent): (WebCore::parametersFromObject): (WebCore::updateWidgetForObjectElement): (WebCore::parametersFromEmbed): (WebCore::updateWidgetForEmbedElement): (WebCore::updateWidgetForMediaElement): (WebCore::RenderEmbeddedObject::updateWidget):
  • rendering/RenderEmbeddedObject.h: (WebCore::RenderEmbeddedObject::setHasFallbackContent):
  • rendering/RenderLayerCompositor.h:
7:03 PM Changeset in webkit [66630] by bweinstein@apple.com
  • 2 edits in trunk/WebKitTools

Have kill-old-processes kill debug and release variants of WebKit2WebWebProcess
and WebKitTestRunner.

Reviewed by Adam Roben.

  • BuildSlaveSupport/win/kill-old-processes:
6:20 PM Changeset in webkit [66629] by kinuko@chromium.org
  • 3 edits in trunk/WebCore

2010-09-01 Kinuko Yasuda <​kinuko@chromium.org>

Reviewed by Jian Li.

Add more FileError and FileException code for FileWriter and FileSystem API
​https://bugs.webkit.org/show_bug.cgi?id=45048

Adding SYNTAX_ERR for FileWriter.
​http://dev.w3.org/2009/dap/file-system/file-writer.html

Adding INVALID_STATE_ERR, INVALID_MODIFICATION_ERR and QUOTA_EXCEEDED_ERR for FileSystem.
​http://dev.w3.org/2009/dap/file-system/file-dir-sys.html

Adding NO_MODIFICATION_ALLOWED_ERR (to FileException) for both.

  • fileapi/FileError.idl:
  • fileapi/FileException.idl:
6:00 PM Changeset in webkit [66628] by commit-queue@webkit.org
  • 23 edits
    6 adds in trunk

2010-09-01 Mihai Parparita <​mihaip@chromium.org>

Reviewed by Brady Eidson.

popstate event is not fired until document opts in by calling pushstate.
​https://bugs.webkit.org/show_bug.cgi?id=41372

Add two tests to check how often popstate is being fired (for both
fragment changes and page changes, especially with the page cache
enabled).

Update existing state object tests to handle popstate being fired for
the page being navigated to (right after onload).

  • fast/loader/stateobjects/document-destroyed-navigate-back-with-fragment-scroll.html:
  • fast/loader/stateobjects/document-destroyed-navigate-back.html:
  • fast/loader/stateobjects/popstate-after-load-complete-addeventlistener.html:
  • fast/loader/stateobjects/popstate-after-load-complete-body-attribute.html:
  • fast/loader/stateobjects/popstate-after-load-complete-body-inline-attribute.html:
  • fast/loader/stateobjects/popstate-after-load-complete-window-attribute.html:
  • fast/loader/stateobjects/popstate-fires-on-history-traversal-expected.txt: Added.
  • fast/loader/stateobjects/popstate-fires-on-history-traversal.html: Added.
  • fast/loader/stateobjects/popstate-fires-with-page-cache-expected.txt: Added.
  • fast/loader/stateobjects/popstate-fires-with-page-cache.html: Added.
  • fast/loader/stateobjects/pushstate-object-types.html:
  • fast/loader/stateobjects/pushstate-then-replacestate.html:
  • fast/loader/stateobjects/pushstate-with-fragment-urls-and-hashchange.html:
  • fast/loader/stateobjects/replacestate-in-iframe.html:
  • fast/loader/stateobjects/replacestate-then-pushstate.html:
  • fast/loader/stateobjects/resources/popstate-fires-with-page-cache-1.html: Added.
  • fast/loader/stateobjects/resources/popstate-fires-with-page-cache-2.html: Added.
  • fast/loader/stateobjects/resources/replacestate-in-iframe-window-child.html:

2010-09-01 Mihai Parparita <​mihaip@chromium.org>

Reviewed by Brady Eidson.

popstate event is not fired until document opts in by calling pushstate.
​https://bugs.webkit.org/show_bug.cgi?id=41372

Fire popstate even when we don't have a state object when a page is
loaded (for both regular loads and those from the page cache). Also
fire popstate when doing in-document navigation via fragment changes.
This is consistent with both Gecko and recent the HTML5 spec change:
​http://html5.org/tools/web-apps-tracker?from=5376&to=5377

Tests: fast/loader/stateobjects/popstate-fires-on-history-traversal.html

fast/loader/stateobjects/popstate-fires-with-page-cache.html

  • bindings/js/SerializedScriptValue.cpp:
  • bindings/js/SerializedScriptValue.h: (WebCore::SerializedScriptValue::create):
  • bindings/v8/SerializedScriptValue.cpp:
  • bindings/v8/SerializedScriptValue.h:
  • dom/Document.cpp: (WebCore::Document::implicitClose):
  • dom/Document.h:
  • history/CachedFrame.cpp: (WebCore::CachedFrameBase::restore):
  • loader/FrameLoader.cpp: (WebCore::FrameLoader::loadInSameDocument): (WebCore::FrameLoader::transitionToCommitted):
  • loader/HistoryController.cpp: (WebCore::HistoryController::pushState): (WebCore::HistoryController::replaceState):
5:25 PM Changeset in webkit [66627] by tkent@chromium.org
  • 2 edits in trunk/WebKit/chromium

2010-09-01 Kent Tamura <​tkent@chromium.org>

Unreviewed, rolling out r66576.
​http://trac.webkit.org/changeset/66576
​https://bugs.webkit.org/show_bug.cgi?id=45016

This change is not needed.

  • public/WebKitClient.h:
5:20 PM Changeset in webkit [66626] by mrowe@apple.com
  • 2 edits in trunk/WebKit2

Allow the web process to load frameworks other than WebKit2.framework from the build directory
when running engineering builds.

Reviewed by Sam Weinig.

  • WebProcess/mac/WebProcessMainMac.mm:

(WebKit::WebProcessMain): Pass the path containing WebKit2.framework to the sandbox rather than
the path of the WebKit2.framework itself.

4:26 PM Changeset in webkit [66625] by Joseph Pecoraro
  • 2 edits in trunk/LayoutTests

2010-09-01 Matthew Willis <​lilmatt@flock.com>

Reviewed by Joseph Pecoraro.

Fix failing test: inspector/console-format.html (regression from r66616)
​https://bugs.webkit.org/show_bug.cgi?id=45067

  • inspector/console-format-expected.txt:
4:18 PM Changeset in webkit [66624] by rniwa@webkit.org
  • 3 edits
    2 adds in trunk

2010-09-01 Ryosuke Niwa <​rniwa@webkit.org>

Reviewed by Dimitri Glazkov.

REGRESSION(r66431): WebCore::Editor::selectionStartCSSPropertyValue crashes if no background color is specified
​https://bugs.webkit.org/show_bug.cgi?id=45068

The crash was caused by the assumption that every node has an ancestor with some background color,
which was obviously false. Fixed this by adding a null for ancestor.

Test: editing/execCommand/backcolor-crash.html

  • editing/Editor.cpp: (WebCore::Editor::selectionStartCSSPropertyValue):

2010-09-01 Ryosuke Niwa <​rniwa@webkit.org>

Reviewed by Dimitri Glazkov.

REGRESSION(r66431): WebCore::Editor::selectionStartCSSPropertyValue crashes if no background color is specified
​https://bugs.webkit.org/show_bug.cgi?id=45068

Added a test to ensure WebKit doesn't crash when backColor is queried on a node without any ancestors
with a background color.

  • editing/execCommand/backcolor-crash-expected.txt: Added.
  • editing/execCommand/backcolor-crash.html: Added.
4:14 PM Changeset in webkit [66623] by jberlin@webkit.org
  • 2 edits in trunk/WebKit/win

2010-09-01 Jessie Berlin <​jberlin@apple.com>

Reviewed by Adam Roben.

WebViews should allow their parent windows to handle WM_MOUSEACTIVATE messages.
​https://bugs.webkit.org/show_bug.cgi?id=45047

  • WebView.cpp: (WebView::WebViewWndProc):
3:55 PM Changeset in webkit [66622] by jer.noble@apple.com
  • 3 edits
    3 adds in trunk

2010-08-31 Jer Noble <​jer.noble@apple.com>

Reviewed by Darin Adler.

WebKit should not accept PDFs as video.
​https://bugs.webkit.org/show_bug.cgi?id=45013


QuickTime will accept a PDF as the source for a movie, and happily
convert that PDF into a .mov. WebKit should disable the eat and grip
PDF components which allow this.

Test: http/tests/media/pdf-served-as-pdf.html

  • platform/graphics/mac/MediaPlayerPrivateQTKit.mm: (WebCore::MediaPlayerPrivate::createQTMovie): Add PDF eat and grip components to the components which are disabled before opening a QTMovie.

2010-08-31 Jer Noble <​jer.noble@apple.com>

Reviewed by Darin Adler.

WebKit should not accept PDFs as video.
​https://bugs.webkit.org/show_bug.cgi?id=45013


Added a new test, similar to the text-served-as-text.html test, which
expects an error thrown when a PDF is set as the source of a <video>
element.

  • http/tests/media/pdf-served-as-pdf-expected.txt: Added.
  • http/tests/media/pdf-served-as-pdf.html: Added.
  • http/tests/media/resources/test.pdf: Added.
3:35 PM Changeset in webkit [66621] by weinig@apple.com
  • 10 edits in trunk/WebKit2

Change ImmutableArray to be backed by a Vector
​https://bugs.webkit.org/show_bug.cgi?id=45064

Reviewed by John Sullivan.

Change ImmutableArray to be backed by a Vector to work toward having
a mutable subclass and to improve the API when working with it in WebKit2
itself.

  • Also remove WKArrayCreateAdoptingValues which was confusing and unused.
  • Shared/ImmutableArray.cpp:

(WebKit::ImmutableArray::ImmutableArray):
(WebKit::ImmutableArray::~ImmutableArray):

  • Shared/ImmutableArray.h:

(WebKit::ImmutableArray::adopt):
(WebKit::ImmutableArray::at):
(WebKit::ImmutableArray::size):

  • Shared/ImmutableDictionary.cpp:

(WebKit::ImmutableDictionary::keys):

  • UIProcess/API/C/WKArray.cpp:
  • UIProcess/API/C/WKArray.h:
  • UIProcess/WebBackForwardList.cpp:

(WebKit::WebBackForwardList::backListAsImmutableArrayWithLimit):
(WebKit::WebBackForwardList::forwardListAsImmutableArrayWithLimit):

  • UIProcess/WebContext.cpp:

(WebKit::PostMessageEncoder::PostMessageDecoder::decode):

  • WebProcess/InjectedBundle/InjectedBundle.cpp:

(WebKit::PostMessageEncoder::PostMessageDecoder::decode):

  • WebProcess/WebPage/WebFrame.cpp:

(WebKit::WebFrame::childFrames):

3:00 PM Changeset in webkit [66620] by tony@chromium.org
  • 3 edits in trunk/LayoutTests

2010-09-01 Tony Chang <​tony@chromium.org>

Unreviewed, updating a chromium baseline, the only difference is in
the anti-aliasing of the rounded border (a result of r66615).

  • platform/chromium-win/fast/borders/border-radius-huge-assert-expected.checksum:
  • platform/chromium-win/fast/borders/border-radius-huge-assert-expected.png:
2:36 PM Changeset in webkit [66619] by weinig@apple.com
  • 8 edits
    11 adds in trunk/WebKit2

Add URLResponse API for WebKit2
​https://bugs.webkit.org/show_bug.cgi?id=45062

Reviewed by Anders Carlsson.

Add WebURLResponse class and WKURLResponseRef C API to go with it. Right now,
instance of WKURLResponseRef are manipulated via platform types just like
WKURLRequestRef.

This just adds the type and conversion functions, no uses yet.

  • Shared/APIObject.h:
  • Shared/WebCoreArgumentCoders.h:
  • Shared/WebURLResponse.cpp: Added.

(WebKit::WebURLResponse::WebURLResponse):

  • Shared/WebURLResponse.h: Added.

(WebKit::WebURLResponse::create):
(WebKit::WebURLResponse::resourceResponse):
(WebKit::WebURLResponse::type):

  • Shared/mac/WebCoreArgumentCodersMac.mm:

(CoreIPC::encodeWithNSKeyedArchiver):
(CoreIPC::decodeWithNSKeyedArchiver):
(CoreIPC::encodeResourceRequest):
(CoreIPC::decodeResourceRequest):

  • Shared/mac/WebURLResponseMac.mm: Added.

(WebKit::WebURLResponse::WebURLResponse):
(WebKit::WebURLResponse::platformResponse):

  • Shared/qt/WebURLResponseQt.cpp: Added.

(WebKit::WebURLResponse::WebURLResponse):
(WebKit::WebURLResponse::platformResponse):

  • Shared/win/WebURLResponseWin.cpp: Added.

(WebKit::WebURLResponse::WebURLResponse):
(WebKit::WebURLResponse::platformResponse):

  • UIProcess/API/C/WKURLResponse.cpp: Added.

(WKURLResponseGetTypeID):

  • UIProcess/API/C/WKURLResponse.h: Added.
  • UIProcess/API/C/cf/WKURLResponseCF.cpp: Added.

(WKURLResponseCreateWithCFURLResponse):
(WKURLResponseCopyCFURLResponse):

  • UIProcess/API/C/cf/WKURLResponseCF.h: Added.
  • UIProcess/API/C/mac/WKURLResponseNS.h: Added.
  • UIProcess/API/C/mac/WKURLResponseNS.mm: Added.

(WKURLResponseCreateWithNSURLResponse):
(WKURLResponseCopyNSURLResponse):

  • WebKit2.pro:
  • WebKit2.xcodeproj/project.pbxproj:
  • win/WebKit2.vcproj:
  • win/WebKit2Generated.make:
2:35 PM Changeset in webkit [66618] by crogers@google.com
  • 48 edits
    1 copy
    2 adds in branches/audio/WebCore

audio branch: lots of miscellaneous changes for code reviews - landing in trunk

2:06 PM Changeset in webkit [66617] by tony@chromium.org
  • 4 edits
    1 delete in trunk/LayoutTests

2010-09-01 Tony Chang <​tony@chromium.org>

Unreviewed, updating chromium test expectations.

  • platform/chromium-mac/fast/js/kde/RegExp-expected.txt: Rebaseline
  • platform/chromium-win/fast/forms/focus2-expected.txt: Removed because it's wrong.
  • platform/chromium-win/fast/js/kde/RegExp-expected.txt: Rebaseline
  • platform/chromium/test_expectations.txt:
1:16 PM Changeset in webkit [66616] by barraclough@apple.com
  • 13 edits in trunk

JavaScriptCore: Ecma-262 15.11.1.1 states that if the argument is undefined then an
Error object's message property should be set to the empty string.

Rubber stamped by Oliver Hunt.

  • runtime/ErrorInstance.cpp:

(JSC::ErrorInstance::ErrorInstance):
(JSC::ErrorInstance::create):

  • runtime/ErrorInstance.h:
  • runtime/ErrorPrototype.cpp:

(JSC::ErrorPrototype::ErrorPrototype):

LayoutTests: Fix three tests in fast/js/kde are failing; fix them.

Rubber stamped by Oliver Hunt.

exceptions.js:

This is failing because it redefines the method 'should be'.
Rename the test's version of this method.

RegExp.js:

This tests erroneously expects a failed match to update the
cached results on the RegExp object; these are only updated
on a successful match.

evil-n.js:

This test erroneously expects the message property of a newly
constructed Error with no arguments to be undefined; the ECMA
262-5 spec requires this to be "". Two sputnik tests contain
the same error (the sputnik tests is actually testing 262-3
compatibility, which is not what we're interested in).

  • fast/js/kde/RegExp-expected.txt:
  • fast/js/kde/evil-n-expected.txt:
  • fast/js/kde/exceptions-expected.txt:
  • fast/js/kde/script-tests/RegExp.js:
  • fast/js/kde/script-tests/evil-n.js:
  • fast/js/kde/script-tests/exceptions.js:

(kdeShouldBe):
(testThrow):
(testThrow2):
(testReferenceError):
(testFunctionError):
(testMathFunctionError):
(testWhileAbortion):

  • fast/js/sputnik/Conformance/15_Native_Objects/15.11_Error/15.11.1/S15.11.1.1_A1_T1.html:
  • fast/js/sputnik/Conformance/15_Native_Objects/15.11_Error/15.11.2/S15.11.2.1_A1_T1.html:
12:51 PM Changeset in webkit [66615] by rwlbuis@webkit.org
  • 14 edits in trunk

2010-09-01 Rob Buis <​rwlbuis@gmail.com>

Reviewed by Darin Adler.

Support for Percentage Values in border-radius
​https://bugs.webkit.org/show_bug.cgi?id=38354

Support percentage values on the border-radius family
of CSS properties.

  • css/CSSComputedStyleDeclaration.cpp: (WebCore::getBorderRadiusCornerValue):
  • css/CSSParser.cpp: (WebCore::CSSParser::parseValue): (WebCore::CSSParser::parseBorderRadius):
  • css/CSSPrimitiveValue.cpp:
  • css/CSSStyleSelector.cpp: (WebCore::CSSStyleSelector::applyProperty):
  • page/animation/AnimationBase.cpp: (WebCore::AnimationBase::ensurePropertyMap):
  • platform/Length.h:
  • rendering/RenderObject.cpp: (WebCore::RenderObject::repaintAfterLayoutIfNeeded):
  • rendering/style/BorderData.h: (WebCore::BorderData::BorderData): (WebCore::BorderData::hasBorderRadius): (WebCore::BorderData::topLeft): (WebCore::BorderData::topRight): (WebCore::BorderData::bottomLeft): (WebCore::BorderData::bottomRight):
  • rendering/style/RenderStyle.cpp: (WebCore::RenderStyle::getBorderRadiiForRect): (WebCore::RenderStyle::getInnerBorderRadiiForRectWithBorderWidths):
  • rendering/style/RenderStyle.h: (WebCore::InheritedFlags::borderTopLeftRadius): (WebCore::InheritedFlags::borderTopRightRadius): (WebCore::InheritedFlags::borderBottomLeftRadius): (WebCore::InheritedFlags::borderBottomRightRadius): (WebCore::InheritedFlags::setBorderTopLeftRadius): (WebCore::InheritedFlags::setBorderTopRightRadius): (WebCore::InheritedFlags::setBorderBottomLeftRadius): (WebCore::InheritedFlags::setBorderBottomRightRadius): (WebCore::InheritedFlags::setBorderRadius): (WebCore::InheritedFlags::initialBorderRadius):
12:45 PM Changeset in webkit [66614] by tony@chromium.org
  • 2 edits
    6 adds in trunk/LayoutTests

2010-09-01 Tony Chang <​tony@chromium.org>

Unreviewed, add chromium win/linux baselines for fast/multicol/float-paginate.html.

  • platform/chromium-linux/fast/multicol/float-paginate-expected.checksum: Added.
  • platform/chromium-linux/fast/multicol/float-paginate-expected.png: Added.
  • platform/chromium-linux/fast/multicol/float-paginate-expected.txt: Added.
  • platform/chromium-win/fast/multicol/float-paginate-expected.checksum: Added.
  • platform/chromium-win/fast/multicol/float-paginate-expected.png: Added.
  • platform/chromium-win/fast/multicol/float-paginate-expected.txt: Added.
  • platform/chromium/test_expectations.txt:
12:37 PM Changeset in webkit [66613] by caseq@chromium.org
  • 2 edits in trunk/WebCore

2010-09-01 Andrey Kosyakov <​caseq@chromium.org>

Reviewed by Yury Semikhatsky.

Web Inspector: webInspector.inspectedWindow.evaluate() should not fail if not passed callback.
​https://bugs.webkit.org/show_bug.cgi?id=45037

  • inspector/front-end/ExtensionAPI.js: (WebInspector.injectedExtensionAPI.InspectedWindow.prototype.evaluate):
11:14 AM Changeset in webkit [66612] by hyatt@apple.com
  • 3 edits
    4 adds
    4 deletes in trunk

WebCore: ​https://bugs.webkit.org/show_bug.cgi?id=44963, overhanging floats aren't positioned properly when they occur
in multi-column layout. Don't let floats overhang in multi-column blocks, since they are part of the column layout.

Reviewed by Dan Bernstein and Simon Fraser.

fast/multicol/float-pagination-failure now succeeds, so will rename and move it.

  • rendering/RenderBlock.cpp:

(WebCore::RenderBlock::addOverhangingFloats):

LayoutTests: ​https://bugs.webkit.org/show_bug.cgi?id=44963, rename the float-paginate-failure test since it now
succeeds.

Reviewed by Dan Bernstein and Simon Fraser.

  • fast/multicol/float-paginate-failure.html: Removed.
  • fast/multicol/float-paginate.html: Added.
  • platform/mac/fast/multicol/float-paginate-expected.checksum: Added.
  • platform/mac/fast/multicol/float-paginate-expected.png: Added.
  • platform/mac/fast/multicol/float-paginate-expected.txt: Added.
  • platform/mac/fast/multicol/float-paginate-failure-expected.checksum: Removed.
  • platform/mac/fast/multicol/float-paginate-failure-expected.png: Removed.
  • platform/mac/fast/multicol/float-paginate-failure-expected.txt: Removed.
9:59 AM Changeset in webkit [66611] by kbr@google.com
  • 3 edits in trunk/WebCore

2010-08-31 Kenneth Russell <​kbr@google.com>

Reviewed by Simon Fraser.

Add helpers to FloatRect to compute bounding box from points
​https://bugs.webkit.org/show_bug.cgi?id=45015

Added fitToPoints, including a few overloaded variants for the
cases of two, three and four points. I opted not to add a
generalized version taking const FloatRect* or Vector<FloatRect>
at the present time because for my own purposes they are
unnecessary, and the specific versions can be more tightly coded.
Also added left() and top() for symmetry with right() and bottom().
These changes have been tested with new code to be added later.

  • platform/graphics/FloatRect.cpp: (WebCore::FloatRect::intersect): (WebCore::FloatRect::unite): (WebCore::FloatRect::fitToPoints):
  • platform/graphics/FloatRect.h: (WebCore::FloatRect::left): (WebCore::FloatRect::top): (WebCore::FloatRect::setLocationAndSizeFromEdges):
9:50 AM Changeset in webkit [66610] by antonm@chromium.org
  • 2 edits in trunk/WebCore

2010-09-01 Anton Muhin <​antonm@chromium.org>

Reviewed by Dimitri Glazkov.

[v8] Use Vector and DEFINE_STATIC_LOCAL instead of plain static array.
​https://bugs.webkit.org/show_bug.cgi?id=45040

By code guidelines all the statics should be wrapped into appropriate macro.

  • bindings/v8/V8Binding.cpp: (WebCore::int32ToWebCoreString):
9:34 AM Changeset in webkit [66609] by commit-queue@webkit.org
  • 2 edits in trunk

2010-09-01 Ryuan Choi <​ryuan.choi@samsung.com>

Reviewed by Antonio Gomes.

[EFL] Need to check LibSoup version
​https://bugs.webkit.org/show_bug.cgi?id=44658

Add version check of LibSoup.

  • cmake/FindLibSoup2.cmake:
9:18 AM Changeset in webkit [66608] by alex
  • 1 edit in trunk/LayoutTests/ChangeLog

2010-09-01 Alejandro G. Castro <​alex@igalia.com>

Reviewed by Martin Robinson.

[Gtk] very slow page scrolling on big -webkit-box-shadow areas
​https://bugs.webkit.org/show_bug.cgi?id=39582

Add a new drawTiledShadow method to render a big shadow doing
tiling of parts of a smaller shadow. We create the minimum shadow
required to get the tiles we will use to render the real shadow
and we use drawPattern with those tiles to create the shadow.

Pixel tests added.

  • platform/gtk/fast/box-shadow/basic-shadows-expected.checksum:
  • platform/gtk/fast/box-shadow/basic-shadows-expected.png:
  • platform/gtk/fast/css/shadow-multiple-expected.checksum:
  • platform/gtk/fast/css/shadow-multiple-expected.png:
  • platform/gtk/svg/css/stars-with-shadow-expected.checksum:
  • platform/gtk/svg/css/stars-with-shadow-expected.png:
9:10 AM Changeset in webkit [66607] by alex
  • 11 edits in trunk

2010-09-01 Alejandro G. Castro <​alex@igalia.com>

Reviewed by Martin Robinson.

[Gtk] very slow page scrolling on big -webkit-box-shadow areas
​https://bugs.webkit.org/show_bug.cgi?id=39582

Add a new drawTiledShadow method to render a big shadow doing
tiling of parts of a smaller shadow. We create the minimum shadow
required to get the tiles we will use to render the real shadow
and we use drawPattern with those tiles to create the shadow.

  • platform/graphics/GraphicsContext.h:
  • platform/graphics/cairo/FontCairo.cpp: (WebCore::Font::drawGlyphs):
  • platform/graphics/cairo/GraphicsContextCairo.cpp: (WebCore::drawPathShadow): (WebCore::drawBorderlessRectShadow): (WebCore::GraphicsContext::fillRect): (WebCore::GraphicsContext::applyPlatformShadow): (WebCore::GraphicsContext::createShadowMask): (WebCore::getPhase): (WebCore::GraphicsContext::drawTiledShadow): (WebCore::GraphicsContext::fillRoundedRect):
  • platform/graphics/cairo/ImageCairo.cpp: (WebCore::BitmapImage::draw):
9:01 AM Changeset in webkit [66606] by Martin Robinson
  • 2 edits in trunk/LayoutTests

2010-09-01 Martin Robinson <​mrobinson@igalia.com>

Updating one more baseline, which seems to differ on the bots.

  • platform/gtk/fast/frames/onlyCommentInIFrame-expected.txt:
8:29 AM Changeset in webkit [66605] by Martin Robinson
  • 2 edits
    43 adds in trunk/LayoutTests

2010-09-01 Martin Robinson <​mrobinson@igalia.com>

Add baselines and unskip non-failing tests in preparation for
landing scrollbar theme changes.

  • platform/gtk/Skipped:
  • platform/gtk/fast/dynamic/window-resize-scrollbars-test-expected.checksum: Added.
  • platform/gtk/fast/dynamic/window-resize-scrollbars-test-expected.png: Added.
  • platform/gtk/fast/dynamic/window-resize-scrollbars-test-expected.txt: Added.
  • platform/gtk/fast/dynamic/window-scrollbars-test-expected.checksum: Added.
  • platform/gtk/fast/dynamic/window-scrollbars-test-expected.png: Added.
  • platform/gtk/fast/dynamic/window-scrollbars-test-expected.txt: Added.
  • platform/gtk/fast/events/attempt-scroll-with-no-scrollbars-expected.txt: Added.
  • platform/gtk/fast/frames/001-expected.txt: Added.
  • platform/gtk/fast/frames/002-expected.txt: Copied from LayoutTests/platform/mac/fast/frames/002-expected.txt.
  • platform/gtk/fast/frames/calculate-fixed-expected.txt: Copied from LayoutTests/platform/qt/fast/frames/calculate-fixed-expected.txt.
  • platform/gtk/fast/frames/calculate-order-expected.txt: Copied from LayoutTests/platform/qt/fast/frames/calculate-order-expected.txt.
  • platform/gtk/fast/frames/calculate-percentage-expected.txt: Copied from LayoutTests/platform/qt/fast/frames/calculate-percentage-expected.txt.
  • platform/gtk/fast/frames/calculate-relative-expected.txt: Copied from LayoutTests/platform/qt/fast/frames/calculate-relative-expected.txt.
  • platform/gtk/fast/frames/calculate-round-expected.txt: Copied from LayoutTests/platform/mac/fast/frames/calculate-round-expected.txt.
  • platform/gtk/fast/frames/content-opacity-1-expected.txt: Copied from LayoutTests/platform/mac/fast/frames/content-opacity-1-expected.txt.
  • platform/gtk/fast/frames/content-opacity-2-expected.txt: Copied from LayoutTests/platform/mac/fast/frames/content-opacity-2-expected.txt.
  • platform/gtk/fast/frames/contentWindow_Frame-expected.txt: Added.
  • platform/gtk/fast/frames/contentWindow_iFrame-expected.txt: Added.
  • platform/gtk/fast/frames/empty-cols-attribute-expected.txt: Copied from LayoutTests/platform/mac/fast/frames/empty-cols-attribute-expected.txt.
  • platform/gtk/fast/frames/empty-frame-src-expected.txt: Copied from LayoutTests/platform/mac/fast/frames/empty-frame-src-expected.txt.
  • platform/gtk/fast/frames/frame-element-name-expected.txt: Added.
  • platform/gtk/fast/frames/frame-navigation-expected.txt: Added.
  • platform/gtk/fast/frames/frame-scrolling-attribute-expected.checksum: Added.
  • platform/gtk/fast/frames/frame-scrolling-attribute-expected.png: Added.
  • platform/gtk/fast/frames/frame-scrolling-attribute-expected.txt: Added.
  • platform/gtk/fast/frames/frame-set-whitespace-attributes-expected.txt: Copied from LayoutTests/platform/mac/fast/frames/frame-set-whitespace-attributes-expected.txt.
  • platform/gtk/fast/frames/frame-src-attribute-expected.txt: Added.
  • platform/gtk/fast/frames/frameElement-frame-expected.txt: Added.
  • platform/gtk/fast/frames/frameElement-iframe-expected.txt: Added.
  • platform/gtk/fast/frames/frameset-style-recalc-expected.txt: Added.
  • platform/gtk/fast/frames/iframe-option-crash-expected.txt: Added.
  • platform/gtk/fast/frames/iframe-scrolling-attribute-expected.checksum: Added.
  • platform/gtk/fast/frames/iframe-scrolling-attribute-expected.png: Added.
  • platform/gtk/fast/frames/iframe-scrolling-attribute-expected.txt: Added.
  • platform/gtk/fast/frames/iframe-text-contents-expected.txt: Copied from LayoutTests/platform/mac/fast/frames/iframe-text-contents-expected.txt.
  • platform/gtk/fast/frames/iframe-with-frameborder-expected.txt: Added.
  • platform/gtk/fast/frames/inline-object-inside-frameset-expected.txt: Added.
  • platform/gtk/fast/frames/invalid-expected.txt: Copied from LayoutTests/platform/qt/fast/frames/invalid-expected.txt.
  • platform/gtk/fast/frames/no-frame-borders-expected.txt: Copied from LayoutTests/platform/mac/fast/frames/no-frame-borders-expected.txt.
  • platform/gtk/fast/frames/onlyCommentInIFrame-expected.txt: Added.
  • platform/gtk/fast/frames/valid-expected.txt: Copied from LayoutTests/platform/qt/fast/frames/valid-expected.txt.
  • platform/gtk/fast/frames/viewsource-attribute-expected.txt: Added.
  • platform/gtk/fast/frames/viewsource-on-image-file-expected.txt: Added.
8:16 AM Changeset in webkit [66604] by antonm@chromium.org
  • 2 edits in trunk/LayoutTests

2010-09-01 Anton Muhin <​antonm@chromium.org>

Unreviewed fix to update test expectations to recent SVG changes.

  • platform/chromium/test_expectations.txt:
8:04 AM Changeset in webkit [66603] by antonm@chromium.org
  • 2 edits in trunk/LayoutTests

2010-09-01 'Anton Muhin' <​antonm@chromium.org>

Unreviewed expectations fix.

Due to recent changes in V8, Chromium now passes more ietestcentertests.

  • platform/chromium/test_expectations.txt:
7:46 AM Changeset in webkit [66602] by abecsi@webkit.org
  • 2 edits in trunk/WebKitTools

2010-09-01 Gabor Rapcsanyi <​rgabor@inf.u-szeged.hu>

Reviewed by Antonio Gomes.

[Qt] Remove obsolete include path from DumpRenderTree.pro

  • DumpRenderTree/qt/DumpRenderTree.pro:
7:19 AM Changeset in webkit [66601] by caseq@chromium.org
  • 10 edits
    4 adds in trunk

2010-09-01 Andrey Kosyakov <​caseq@chromium.org>

Reviewed by Yury Semikhatsky.

Web Inspector, Extension API: Panel.onSelectionChanged event is never fired
Added command line API to the lexical context of code evaluated by webInspector.inspectedWindow.evaluate()
Fixed webInspector.inspectedWindow.evaluate() to return object rather than its stringified representation
​https://bugs.webkit.org/show_bug.cgi?id=44957

Test: inspector/extensions-eval.html

  • inspector/front-end/ExtensionAPI.js: (WebInspector.injectedExtensionAPI.InspectedWindow.prototype.evaluate):
  • inspector/front-end/ExtensionServer.js: (WebInspector.ExtensionServer.prototype._onEvaluateOnInspectedPage):

2010-09-01 Andrey Kosyakov <​caseq@chromium.org>

Reviewed by Yury Semikhatsky.

Web Inspector, Extension API: Panel.onSelectionChanged event is never fired
Added tests for webInspector.inspectedWindow.eval() and event dispatchers.
​https://bugs.webkit.org/show_bug.cgi?id=44957

  • inspector/extensions-eval-expected.txt: Added.
  • inspector/extensions-eval.html: Added.
  • inspector/extensions-expected.txt:
  • inspector/extensions.html:
  • inspector/resources/extension-main.js: (fetchTests.callback): (fetchTests):
  • platform/chromium/test_expectations.txt:
7:12 AM Changeset in webkit [66600] by Philippe Normand
  • 3 edits in trunk/WebCore

2010-08-27 Philippe Normand <​pnormand@igalia.com>

Reviewed by Xan Lopez.

[GStreamer] ImageGStreamer doesn't need to hold a Cairo surface
​https://bugs.webkit.org/show_bug.cgi?id=44757

Pass the cairo surface to the BitmapImage constructor which will
handle its life-cycle.

  • platform/graphics/gstreamer/ImageGStreamer.h:
  • platform/graphics/gstreamer/ImageGStreamerCairo.cpp: (ImageGStreamer::ImageGStreamer): (ImageGStreamer::~ImageGStreamer):
7:01 AM Changeset in webkit [66599] by Nikolas Zimmermann
  • 19 edits in trunk

2010-08-30 Patrick Gansterer <​paroga@paroga.com>

Reviewed by Dirk Schulze.

RenderImage::imageChanged invalidates wrong area
​https://bugs.webkit.org/show_bug.cgi?id=43779

Remove RenderImage inheritance from RenderSVGImage, it now inherits from RenderSVGModelObject.

  • rendering/RenderSVGImage.cpp: (WebCore::RenderSVGImage::RenderSVGImage): (WebCore::RenderSVGImage::~RenderSVGImage): (WebCore::RenderSVGImage::layout): (WebCore::RenderSVGImage::paint): (WebCore::RenderSVGImage::imageChanged):
  • rendering/RenderSVGImage.h: (WebCore::RenderSVGImage::imageResource):
  • rendering/RenderSVGModelObject.h:
  • svg/SVGImageElement.cpp:

2010-09-01 Nikolas Zimmermann <​nzimmermann@rim.com>

Reviewed by Dirk Schulze.

RenderImage::imageChanged invalidates wrong area
​https://bugs.webkit.org/show_bug.cgi?id=43779

Rebaseline SVG image dumpAsText() tests.
A newline is gone from each of the results, after switching from RenderImage to RenderSVGModelObject in RenderSVGImage.

  • svg/custom/loadevents-capturing-expected.txt:
  • svg/custom/loadevents-externalresourcesrequired-expected.txt:
  • svg/custom/loadevents-normal-expected.txt:
  • svg/dynamic-updates/SVGImageElement-dom-height-attr-expected.txt:
  • svg/dynamic-updates/SVGImageElement-dom-preserveAspectRatio-attr-expected.txt:
  • svg/dynamic-updates/SVGImageElement-dom-width-attr-expected.txt:
  • svg/dynamic-updates/SVGImageElement-dom-x-attr-expected.txt:
  • svg/dynamic-updates/SVGImageElement-dom-y-attr-expected.txt:
  • svg/dynamic-updates/SVGImageElement-svgdom-height-prop-expected.txt:
  • svg/dynamic-updates/SVGImageElement-svgdom-preserveAspectRatio-prop-expected.txt:
  • svg/dynamic-updates/SVGImageElement-svgdom-width-prop-expected.txt:
  • svg/dynamic-updates/SVGImageElement-svgdom-x-prop-expected.txt:
  • svg/dynamic-updates/SVGImageElement-svgdom-y-prop-expected.txt:
5:32 AM Changeset in webkit [66598] by Nikolas Zimmermann
  • 21 edits in trunk/WebCore

2010-09-01 Nikolas Zimmermann <​nzimmermann@rim.com>

Reviewed by Dirk Schulze.

Add culling to RenderSVGContainer
​https://bugs.webkit.org/show_bug.cgi?id=44896

Cache object/strokeBoundingBox and repaintRectInLocalCoordinates() in RenderSVGContainer & RenderSVGRoot, just like it's done in RenderPath.
Propagate boundary changes up the render tree, in layout phase, in order to notify parents that they should update their cached boundaries.

This is a huge speed benefit when interacting with an already layouted document, as there's no need anymore to traverse all children
of any container or <svg> element, just to retrieve the repaint rect.

  • rendering/RenderForeignObject.cpp: (WebCore::RenderForeignObject::layout): Propagate boundary changes to parent, if neeeded.
  • rendering/RenderObject.cpp: (WebCore::RenderObject::setNeedsBoundariesUpdate): Ditto.
  • rendering/RenderObject.h:
  • rendering/RenderObjectChildList.cpp: (WebCore::RenderObjectChildList::removeChildNode): Ditto.
  • rendering/RenderPath.cpp: (WebCore::RenderPath::layout): Ditto. (WebCore::RenderPath::paint): Use new helper function to intersect the paintInfo.rect with the repaintRectInLocalCoordinates().
  • rendering/RenderPath.h: Remove styleWillChange(), it's in RenderSVGModelObject now. (WebCore::RenderPath::setNeedsBoundariesUpdate): Mark as virtual, just like in RenderObject.
  • rendering/RenderSVGContainer.cpp: (WebCore::RenderSVGContainer::RenderSVGContainer): Initialize m_needsBoundariesUpdate to true. (WebCore::RenderSVGContainer::layout): Only update cached boundaries, if needed, once by traversing the child tree. (WebCore::RenderSVGContainer::paint): Use new helper function to intersect the paintInfo.rect with the repaintRectInLocalCoordinates(). (WebCore::RenderSVGContainer::updateCachedBoundaries): Ask children for their boundaries, and cache them.
  • rendering/RenderSVGContainer.h: (WebCore::RenderSVGContainer::setNeedsBoundariesUpdate): Set m_needsBoundariesUpdate when called. (WebCore::RenderSVGContainer::objectBoundingBox): Inlined, just returns a member variable now. (WebCore::RenderSVGContainer::strokeBoundingBox): Ditto. (WebCore::RenderSVGContainer::repaintRectInLocalCoordinates): Ditto. (WebCore::RenderSVGContainer::calculateLocalTransform): Changed to return a boolean, wheter the transform was modified.
  • rendering/RenderSVGGradientStop.h: (WebCore::RenderSVGGradientStop::strokeBoundingBox): Add missing default implementation, return just FloatRect().
  • rendering/RenderSVGHiddenContainer.cpp: Remove repaintRectInLocalCoordinates() override, it's now needed for painting markers.
  • rendering/RenderSVGHiddenContainer.h: (WebCore::RenderSVGHiddenContainer::clippedOverflowRectForRepaint): Inlined, returning IntRect().
  • rendering/RenderSVGImage.cpp: (WebCore::RenderSVGImage::layout): Propagate boundary changes to parent, if neeeded.
  • rendering/RenderSVGRoot.cpp: Cache boundaries here as well, just like in RenderSVGContainer. (WebCore::RenderSVGRoot::RenderSVGRoot): (WebCore::RenderSVGRoot::layout): (WebCore::RenderSVGRoot::updateCachedBoundaries):
  • rendering/RenderSVGRoot.h: (WebCore::RenderSVGRoot::setNeedsBoundariesUpdate): (WebCore::RenderSVGRoot::setNeedsTransformUpdate): (WebCore::RenderSVGRoot::objectBoundingBox): (WebCore::RenderSVGRoot::strokeBoundingBox): (WebCore::RenderSVGRoot::repaintRectInLocalCoordinates):
  • rendering/RenderSVGText.cpp: (WebCore::RenderSVGText::layout): Propagate boundary changes to parent, if neeeded.
  • rendering/RenderSVGTransformableContainer.cpp: (WebCore::RenderSVGTransformableContainer::calculateLocalTransform): Return true, if the localTransform changed.
  • rendering/RenderSVGTransformableContainer.h:
  • rendering/RenderSVGViewportContainer.cpp: (WebCore::RenderSVGViewportContainer::calcViewport): Propagate boundary changes to parent, if neeeded.
  • rendering/SVGRenderSupport.cpp: (WebCore::SVGRenderSupport::computeContainerBoundingBoxes): Helper function calculating object/strokeBoundingBox and repaintRectInLocalCoordinates in one shot. (WebCore::SVGRenderSupport::paintInfoIntersectsRepaintRect): Moved from RenderPath to share the culling functionality.
  • rendering/SVGRenderSupport.h:
5:31 AM Changeset in webkit [66597] by commit-queue@webkit.org
  • 15 edits
    2 adds in trunk

2010-09-01 Mahesh Kulkarni <​mahesh.kulkarni@nokia.com>

Reviewed by Laszlo Gombos.

[Qt] QWebPage::allowGeolocationRequest should be async API
​https://bugs.webkit.org/show_bug.cgi?id=41364

Enable all geolocation delayed permission layout test for Qt port

  • platform/qt/Skipped:

2010-09-01 Mahesh Kulkarni <​mahesh.kulkarni@nokia.com>

Reviewed by Laszlo Gombos.

[Qt] QWebPage::allowGeolocationRequest should be async API
​https://bugs.webkit.org/show_bug.cgi?id=41364

Adding GeolocationPermissionClientQt.cpp/h to build script

  • WebCore.pro:

2010-09-01 Mahesh Kulkarni <​mahesh.kulkarni@nokia.com>

Reviewed by Laszlo Gombos.

[Qt] QWebPage::allowGeolocationRequest should be async API
​https://bugs.webkit.org/show_bug.cgi?id=41364

Provides a new async API for geolocation permission. Using
Notification API approach from qtwebkit. GeolocationPermissionClientQt
maintains list of pending requests from WebCore and intimates them
when client either allowes/denies the request. Also implements
ChromeClientQt::cancelGeolocationPermissionRequestForFrame.

  • Api/qwebpage.cpp: (QWebPage::setUserPermission):
  • Api/qwebpage.h:
  • WebCoreSupport/ChromeClientQt.cpp: (WebCore::ChromeClientQt::requestGeolocationPermissionForFrame): (WebCore::ChromeClientQt::cancelGeolocationPermissionRequestForFrame):
  • WebCoreSupport/ChromeClientQt.h:
  • WebCoreSupport/GeolocationPermissionClientQt.cpp: Added. (WebCore::GeolocationPermissionClientQt::geolocationPermissionClient): (WebCore::GeolocationPermissionClientQt::GeolocationPermissionClientQt): (WebCore::GeolocationPermissionClientQt::~GeolocationPermissionClientQt): (WebCore::GeolocationPermissionClientQt::requestGeolocationPermissionForFrame): (WebCore::GeolocationPermissionClientQt::cancelGeolocationPermissionRequestForFrame): (WebCore::GeolocationPermissionClientQt::setPermission):
  • WebCoreSupport/GeolocationPermissionClientQt.h: Added.
  • tests/qwebpage/tst_qwebpage.cpp: (JSTestPage::requestPermission): (tst_QWebPage::geolocationRequestJS):

2010-09-01 Mahesh Kulkarni <​mahesh.kulkarni@nokia.com>

Reviewed by Laszlo Gombos.

[Qt] QWebPage::allowGeolocationRequest should be async API
​https://bugs.webkit.org/show_bug.cgi?id=41364

Implements new async API for geolocation permission similar to
Notification. WebPage maintains list of geolocation permission request
QtWebkit and set's when LayoutTestController gets the access from test JS

  • DumpRenderTree/qt/DumpRenderTreeQt.cpp: (WebCore::WebPage::WebPage): (WebCore::WebPage::resetSettings): (WebCore::WebPage::requestPermission): (WebCore::WebPage::cancelPermission): (WebCore::WebPage::permissionSet): (WebCore::DumpRenderTree::DumpRenderTree): (WebCore::DumpRenderTree::processLine): (WebCore::DumpRenderTree::geolocationPermissionSet):
  • DumpRenderTree/qt/DumpRenderTreeQt.h:
  • DumpRenderTree/qt/LayoutTestControllerQt.cpp: (LayoutTestController::reset): (LayoutTestController::setGeolocationPermission): (LayoutTestController::setGeolocationPermissionCommon):
  • DumpRenderTree/qt/LayoutTestControllerQt.h:
5:13 AM Changeset in webkit [66596] by commit-queue@webkit.org
  • 30 edits
    1 delete in trunk/LayoutTests

2010-09-01 Pavel Podivilov <​podivilov@chromium.org>

Reviewed by Yury Semikhatsky.

Web Inspector: move debugger tests to the new test harness
​https://bugs.webkit.org/show_bug.cgi?id=44939

  • http/tests/inspector/debugger-test.js: Removed.
  • http/tests/inspector/debugger-test2.js: (initialize_DebuggerTest.InspectorTest.startDebuggerTest.startTest): (initialize_DebuggerTest.InspectorTest.startDebuggerTest): (initialize_DebuggerTest.InspectorTest.completeDebuggerTest.disableDebugger): (initialize_DebuggerTest.InspectorTest.runTestFunctionAndWaitUntilPaused): (initialize_DebuggerTest.InspectorTest.waitUntilPaused): (initialize_DebuggerTest.InspectorTest.waitUntilResumed): (initialize_DebuggerTest.InspectorTest.resumeExecution): (initialize_DebuggerTest.InspectorTest.captureStackTrace): (initialize_DebuggerTest.InspectorTest._pausedScript): (initialize_DebuggerTest.InspectorTest._resumedScript): (initialize_DebuggerTest.InspectorTest.waitUntilCurrentSourceFrameIsLoaded): (initialize_DebuggerTest.InspectorTest._showScriptSource): (initialize_DebuggerTest.InspectorTest.expandProperties.expandNextPath): (initialize_DebuggerTest.InspectorTest.expandProperties): (initialize_DebuggerTest.InspectorTest._expandProperty): (initialize_DebuggerTest.InspectorTest._findChildPropertyTreeElement): (initialize_DebuggerTest):
  • http/tests/inspector/inspector-test2.js: (initialize_InspectorTest.InspectorTest.completeTest): (initialize_InspectorTest.InspectorTest.evaluateInPage): (initialize_InspectorTest.InspectorTest.addResult.addResultToPage): (initialize_InspectorTest.InspectorTest.addResult): (initialize_InspectorTest.InspectorTest.addObject): (initialize_InspectorTest.InspectorTest.reloadPage): (initialize_InspectorTest.InspectorTest.pageReloaded): (initialize_InspectorTest.InspectorTest.runAfterPendingDispatches): (initialize_InspectorTest.InspectorTest.enableResourceTracking): (initialize_InspectorTest.InspectorTest.disableResourceTracking): (initialize_InspectorTest.InspectorTest._addSniffer): (runTest.runTestInFrontend): (runTest): (didEvaluateForTestInFrontend):
  • inspector/debugger-breakpoints-not-activated-on-reload.html:
  • inspector/debugger-cyclic-ref-expected.txt:
  • inspector/debugger-cyclic-ref.html:
  • inspector/debugger-eval-while-paused-expected.txt:
  • inspector/debugger-eval-while-paused.html:
  • inspector/debugger-no-nested-pause-expected.txt:
  • inspector/debugger-no-nested-pause.html:
  • inspector/debugger-pause-in-eval-script-expected.txt:
  • inspector/debugger-pause-in-eval-script.html:
  • inspector/debugger-pause-on-debugger-statement-expected.txt:
  • inspector/debugger-pause-on-debugger-statement.html:
  • inspector/debugger-proto-property-expected.txt:
  • inspector/debugger-proto-property.html:
  • inspector/debugger-step-in-expected.txt:
  • inspector/debugger-step-in.html:
  • inspector/debugger-step-out-expected.txt:
  • inspector/debugger-step-out.html:
  • inspector/debugger-step-over-expected.txt:
  • inspector/debugger-step-over.html:
  • inspector/dom-breakpoints-expected.txt:
  • inspector/dom-breakpoints.html:
  • inspector/extensions-api-expected.txt:
  • inspector/extensions-audits-api-expected.txt:
  • inspector/extensions-audits-expected.txt:
  • inspector/extensions-expected.txt:
  • inspector/report-protocol-errors.html:
  • platform/chromium/inspector/debugger-step-in-expected.txt:
4:36 AM Changeset in webkit [66595] by antonm@chromium.org
  • 7 edits
    3 adds in trunk/LayoutTests

2010-09-01 Anton Muhin <​antonm@chromium.org>

Unreviewed, updating expected results for Chromium.

  • platform/chromium-linux/fast/css/getComputedStyle/computed-style-without-renderer-expected.txt
  • platform/chromium-linux/fast/css/getComputedStyle/computed-style-expected.txt
  • platform/chromium-linux/svg/css/getComputedStyle-basic-expected.txt
  • platform/chromium-mac/fast/css/getComputedStyle/computed-style-without-renderer-expected.txt
  • platform/chromium-mac/fast/css/getComputedStyle/computed-style-expected.txt
  • platform/chromium-mac/svg/css/getComputedStyle-basic-expected.txt
  • platform/chromium-win/fast/css/getComputedStyle/computed-style-without-renderer-expected.txt
  • platform/chromium-win/fast/css/getComputedStyle/computed-style-expected.txt
  • platform/chromium-win/svg/css/getComputedStyle-basic-expected.txt
4:30 AM Changeset in webkit [66594] by antonm@chromium.org
  • 1 edit
    8 adds in trunk/LayoutTests

2010-09-01 Anton Muhin <​antonm@chromium.org>

Unreviewed, updating expected results for Chromium.

4:01 AM Changeset in webkit [66593] by loislo@chromium.org
  • 2 edits in trunk/LayoutTests

2010-09-01 Ilya Tikhonovsky <​loislo@chromium.org>

Reviewed by Yury Semikhatsky.

WebInspector: disable inspector tests on mac-wk2.
The tests should be skipped until ​https://bugs.webkit.org/show_bug.cgi?id=42318 get fixed.

​https://bugs.webkit.org/show_bug.cgi?id=44975

  • platform/mac-wk2/Skipped:
3:55 AM Changeset in webkit [66592] by abecsi@webkit.org
  • 5 edits
    1 add
    1 delete in trunk

2010-09-01 Andras Becsi <​abecsi@webkit.org>

Reviewed by Antti Koivisto.

[Qt] Auto-generate WebKit2 forwarding headers
​https://bugs.webkit.org/show_bug.cgi?id=44692

  • MiniBrowser/qt/BrowserWindow.h: use source style includes
  • MiniBrowser/qt/MiniBrowser.pro: add missing include paths
  • Scripts/enumerate-included-framework-headers: Removed.

2010-09-01 Andras Becsi <​abecsi@webkit.org>

Reviewed by Antti Koivisto.

[Qt] Auto-generate WebKit2 forwarding headers
​https://bugs.webkit.org/show_bug.cgi?id=44692

  • DerivedSources.pro: let the generate-forwarding-headers.pl script generate forwarding headers.
  • generate-forwarding-headers.pl: Added.
3:37 AM Changeset in webkit [66591] by Csaba Osztrogonác
  • 3 edits in trunk/LayoutTests

Unreviewed, updating expected results for r66581.

  • platform/mac/fast/css/getComputedStyle/computed-style-expected.txt:
  • platform/mac/fast/css/getComputedStyle/computed-style-without-renderer-expected.txt:
3:23 AM Changeset in webkit [66590] by andreas.kling@nokia.com
  • 5 edits
    3 adds in trunk

2010-09-01 Andreas Kling <​andreas.kling@nokia.com>

Reviewed by Darin Adler.

DOM Selection: collapseToStart() and collapseToEnd() should throw INVALID_STATE_ERR if there's no selection
​https://bugs.webkit.org/show_bug.cgi?id=44598

Spec links:
​http://www.whatwg.org/specs/web-apps/current-work/#dom-selection-collapsetostart
​http://www.whatwg.org/specs/web-apps/current-work/#dom-selection-collapsetoend

Test: fast/dom/Selection/collapseToX-empty-selection.html

  • page/DOMSelection.cpp: (WebCore::DOMSelection::collapseToEnd): (WebCore::DOMSelection::collapseToStart):
  • page/DOMSelection.h:
  • page/DOMSelection.idl:

2010-09-01 Andreas Kling <​andreas.kling@nokia.com>

Reviewed by Darin Adler.

DOM Selection: collapseToStart() and collapseToEnd() should throw INVALID_STATE_ERR if there's no selection
​https://bugs.webkit.org/show_bug.cgi?id=44598

Spec links:
​http://www.whatwg.org/specs/web-apps/current-work/#dom-selection-collapsetostart
​http://www.whatwg.org/specs/web-apps/current-work/#dom-selection-collapsetoend

  • fast/dom/Selection/collapseToX-empty-selection-expected.txt: Added.
  • fast/dom/Selection/collapseToX-empty-selection.html: Added.
  • fast/dom/Selection/script-tests/collapseToX-empty-selection.js: Added.
2:35 AM Changeset in webkit [66589] by alex
  • 4 edits in trunk/LayoutTests

2010-09-01 Alejandro G. Castro <​alex@igalia.com>

Unreviewed, updating expected results for r66581.

  • editing/pasteboard/onpaste-text-html-expected.txt:
  • fast/events/ondrop-text-html-expected.txt:
  • platform/gtk/svg/css/getComputedStyle-basic-expected.txt:
1:20 AM Changeset in webkit [66588] by kinuko@chromium.org
  • 2 edits in trunk/WebCore

2010-09-01 Kinuko Yasuda <​kinuko@chromium.org>

Unreviewed; another build fix.

  • WebCore.pri:
1:15 AM Changeset in webkit [66587] by kinuko@chromium.org
  • 2 edits in trunk/WebCore

2010-09-01 Kinuko Yasuda <​kinuko@chromium.org>

Unreviewed; qt build fix.

  • CMakeLists.txt:
1:05 AM Changeset in webkit [66586] by kinuko@chromium.org
  • 7 edits
    38 moves in trunk/WebCore

2010-08-31 Kinuko Yasuda <​kinuko@chromium.org>

Reviewed by Adam Barth.

Move FileSystem related files from WebCore/html to WebCore/fileapi.
​https://bugs.webkit.org/show_bug.cgi?id=44925

No new tests; there should be no functional changes.

  • CMakeLists.txt:
  • GNUmakefile.am:
  • WebCore.gypi:
  • WebCore.pro:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • fileapi/DOMFilePath.cpp: Renamed from WebCore/storage/DOMFilePath.cpp.
  • fileapi/DOMFilePath.h: Renamed from WebCore/storage/DOMFilePath.h.
  • fileapi/DOMFileSystem.cpp: Renamed from WebCore/storage/DOMFileSystem.cpp.
  • fileapi/DOMFileSystem.h: Renamed from WebCore/storage/DOMFileSystem.h.
  • fileapi/DOMFileSystem.idl: Renamed from WebCore/storage/DOMFileSystem.idl.
  • fileapi/DirectoryEntry.cpp: Renamed from WebCore/storage/DirectoryEntry.cpp.
  • fileapi/DirectoryEntry.h: Renamed from WebCore/storage/DirectoryEntry.h.
  • fileapi/DirectoryEntry.idl: Renamed from WebCore/storage/DirectoryEntry.idl.
  • fileapi/DirectoryReader.cpp: Renamed from WebCore/storage/DirectoryReader.cpp.
  • fileapi/DirectoryReader.h: Renamed from WebCore/storage/DirectoryReader.h.
  • fileapi/DirectoryReader.idl: Renamed from WebCore/storage/DirectoryReader.idl.
  • fileapi/EntriesCallback.h: Renamed from WebCore/storage/EntriesCallback.h.
  • fileapi/EntriesCallback.idl: Renamed from WebCore/storage/EntriesCallback.idl.
  • fileapi/Entry.cpp: Renamed from WebCore/storage/Entry.cpp.
  • fileapi/Entry.h: Renamed from WebCore/storage/Entry.h.
  • fileapi/Entry.idl: Renamed from WebCore/storage/Entry.idl.
  • fileapi/EntryArray.cpp: Renamed from WebCore/storage/EntryArray.cpp.
  • fileapi/EntryArray.h: Renamed from WebCore/storage/EntryArray.h.
  • fileapi/EntryArray.idl: Renamed from WebCore/storage/EntryArray.idl.
  • fileapi/EntryCallback.h: Renamed from WebCore/storage/EntryCallback.h.
  • fileapi/EntryCallback.idl: Renamed from WebCore/storage/EntryCallback.idl.
  • fileapi/ErrorCallback.h: Renamed from WebCore/storage/ErrorCallback.h.
  • fileapi/ErrorCallback.idl: Renamed from WebCore/storage/ErrorCallback.idl.
  • fileapi/FileEntry.cpp: Renamed from WebCore/storage/FileEntry.cpp.
  • fileapi/FileEntry.h: Renamed from WebCore/storage/FileEntry.h.
  • fileapi/FileEntry.idl: Renamed from WebCore/storage/FileEntry.idl.
  • fileapi/FileSystemCallback.h: Renamed from WebCore/storage/FileSystemCallback.h.
  • fileapi/FileSystemCallback.idl: Renamed from WebCore/storage/FileSystemCallback.idl.
  • fileapi/FileSystemCallbacks.cpp: Renamed from WebCore/storage/FileSystemCallbacks.cpp.
  • fileapi/FileSystemCallbacks.h: Renamed from WebCore/storage/FileSystemCallbacks.h.
  • fileapi/Flags.h: Renamed from WebCore/storage/Flags.h.
  • fileapi/Flags.idl: Renamed from WebCore/storage/Flags.idl.
  • fileapi/LocalFileSystem.cpp: Renamed from WebCore/storage/LocalFileSystem.cpp.
  • fileapi/LocalFileSystem.h: Renamed from WebCore/storage/LocalFileSystem.h.
  • fileapi/Metadata.h: Renamed from WebCore/storage/Metadata.h.
  • fileapi/Metadata.idl: Renamed from WebCore/storage/Metadata.idl.
  • fileapi/MetadataCallback.h: Renamed from WebCore/storage/MetadataCallback.h.
  • fileapi/MetadataCallback.idl: Renamed from WebCore/storage/MetadataCallback.idl.
12:43 AM Changeset in webkit [66585] by justin.garcia@apple.com
  • 5 edits in trunk/LayoutTests

Unreviewed, updating expected results for recent check-in.

  • editing/execCommand/5770834-1-expected.txt:
  • fast/css/getComputedStyle/computed-style-expected.txt:
  • fast/css/getComputedStyle/computed-style-without-renderer-expected.txt:
  • svg/css/getComputedStyle-basic-expected.txt:
12:15 AM Changeset in webkit [66584] by Nikolas Zimmermann
  • 13 edits in trunk/WebCore

2010-08-31 Nikolas Zimmermann <​nzimmermann@rim.com>

Reviewed by Dirk Schulze.

Prepare RenderSVGContainer culling patch
​https://bugs.webkit.org/show_bug.cgi?id=44955

Add styleWillChange methods for all SVG renderers, that call setNeedsBoundariesUpdate().
It's currently a no-op, but will be used once the containers cache their boundaries, which
happens in a follow-up patch.

  • rendering/RenderSVGBlock.cpp: (WebCore::RenderSVGBlock::styleWillChange): Call setNeedsBoundariesUpdate().
  • rendering/RenderSVGBlock.h:
  • rendering/RenderSVGImage.cpp: (WebCore::RenderSVGImage::styleWillChange): Ditto.
  • rendering/RenderSVGImage.h:
  • rendering/RenderSVGInline.cpp: (WebCore::RenderSVGInline::styleWillChange): Ditto.
  • rendering/RenderSVGInline.h:
  • rendering/RenderSVGModelObject.cpp: (WebCore::RenderSVGModelObject::styleWillChange): Ditto.
  • rendering/RenderSVGModelObject.h:
  • rendering/RenderSVGRoot.cpp: (WebCore::RenderSVGRoot::styleWillChange): Ditto.
  • rendering/RenderSVGRoot.h:
  • rendering/SVGRenderSupport.cpp: (WebCore::SVGRenderSupport::computeContainerBoundingBox): Skip hidden containers, not necessary to traverse them.
  • rendering/style/SVGRenderStyle.cpp: (WebCore::SVGRenderStyle::diff): svg-shadow changes need to cause relayouts, not only repaints, once container bounds are cached.

Aug 31, 2010:

11:44 PM Changeset in webkit [66583] by tkent@chromium.org
  • 2 edits in trunk/LayoutTests

2010-08-31 Kent Tamura <​tkent@chromium.org>

Unreviewed, test expectation update.

  • platform/chromium/drt_expectations.txt: Remove a duplicated expectation.
11:16 PM Changeset in webkit [66582] by tkent@chromium.org
  • 2 edits in trunk/LayoutTests

2010-08-31 Kent Tamura <​tkent@chromium.org>

Unreviewed, test expectation update.

  • platform/chromium/drt_expectations.txt: Add recent failures.
11:14 PM Changeset in webkit [66581] by justin.garcia@apple.com
  • 3 edits
    2 adds in trunk

WebCore: ​https://bugs.webkit.org/show_bug.cgi?id=44915
Need to handle CSSValueAuto in CSSPrimitiveValue::operator ETextAlign()

Reviewed by Dan Bernstein.

  • css/CSSPrimitiveValueMappings.h:

(WebCore::CSSPrimitiveValue::CSSPrimitiveValue): Map TAAUTO to -webkit-auto. "auto" is
not a valid text-align value. We use TAAUTO internally for the nameless value that acts
as left if direction:ltr and right for direction:rtl.

LayoutTests: ​https://bugs.webkit.org/show_bug.cgi?id=44915
Need to handle CSSValueAuto in CSSPrimitiveValue::operator ETextAlign()

Reviewed by Dan Bernstein.

  • fast/css/text-align-webkit-auto-expected.txt: Added.
  • fast/css/text-align-webkit-auto.html: Added.
11:11 PM Changeset in webkit [66580] by hyatt@apple.com
  • 1 edit in trunk/WebKit/win/WebFrame.cpp

Fix Win build bustage.

11:08 PM Changeset in webkit [66579] by hyatt@apple.com
  • 1 edit in trunk/WebCore/page/Frame.cpp

Fix Qt build bustage. Why are these tiled backing store functions defined on Frame and not FrameView anyway? Fail.

11:04 PM Changeset in webkit [66578] by commit-queue@webkit.org
  • 3 edits
    1 add in trunk/WebCore

2010-08-31 Gyuyoung Kim <​gyuyoung.kim@samsung.com>

Reviewed by Martin Robinson.

[EFL] Add PlatformVideoWindowEfl.cpp for WebKit EFL
​https://bugs.webkit.org/show_bug.cgi?id=44508

Add PlatformVideoWindowEfl.cpp in order to use gstreamer by
WebKit EFL.

  • platform/graphics/gstreamer/PlatformVideoWindow.h: (WebCore::PlatformVideoWindow::window): (WebCore::PlatformVideoWindow::videoWindowId):
  • platform/graphics/gstreamer/PlatformVideoWindowEfl.cpp: Added. (PlatformVideoWindow::PlatformVideoWindow): (PlatformVideoWindow::~PlatformVideoWindow):
  • platform/graphics/gstreamer/PlatformVideoWindowGtk.cpp: Include <gtk/gtk.h> directly.
10:59 PM Changeset in webkit [66577] by hyatt@apple.com
  • 33 edits in trunk

WebCore: ​https://bugs.webkit.org/show_bug.cgi?id=44863, disentangle style recalc from layout, so that
the former can occur in more places without having to do the latter.

Reviewed by Sam Weinig.

  • WebCore.exp.in:
  • page/FocusController.cpp:

(WebCore::FocusController::setActive):

  • page/Frame.cpp:

(WebCore::Frame::Frame):

  • page/Frame.h:
  • page/FrameView.cpp:

(WebCore::FrameView::enterCompositingMode):
(WebCore::FrameView::layout):
(WebCore::FrameView::needsLayout):
(WebCore::FrameView::updateLayoutAndStyleIfNeededRecursive):

  • page/FrameView.h:
  • page/Page.cpp:

(WebCore::Page::scheduleForcedStyleRecalcForAllPages):

  • page/Page.h:
  • page/Settings.cpp:

(WebCore::setNeedsRecalcStyleInAllFrames):
(WebCore::setLoadsImagesAutomaticallyInAllFrames):
(WebCore::Settings::setStandardFontFamily):
(WebCore::Settings::setFixedFontFamily):
(WebCore::Settings::setSerifFontFamily):
(WebCore::Settings::setSansSerifFontFamily):
(WebCore::Settings::setCursiveFontFamily):
(WebCore::Settings::setFantasyFontFamily):
(WebCore::Settings::setMinimumFontSize):
(WebCore::Settings::setMinimumLogicalFontSize):
(WebCore::Settings::setDefaultFontSize):
(WebCore::Settings::setDefaultFixedFontSize):
(WebCore::Settings::setLoadsImagesAutomatically):
(WebCore::Settings::setTextAreasAreResizable):
(WebCore::Settings::setAuthorAndUserStylesEnabled):
(WebCore::Settings::setFontRenderingMode):
(WebCore::Settings::setZoomMode):
(WebCore::Settings::setAcceleratedCompositingEnabled):
(WebCore::Settings::setShowDebugBorders):
(WebCore::Settings::setShowRepaintCounter):

  • rendering/RenderTheme.cpp:

(WebCore::RenderTheme::platformColorsDidChange):

WebKit/chromium: ​https://bugs.webkit.org/show_bug.cgi?id=44863, disentangle style recalc from layout, so that
the former can occur in more places without having to do the latter.

Reviewed by Sam Weinig.

  • src/WebFrameImpl.cpp:

(WebKit::WebFrameImpl::layout):

WebKit/efl: ​https://bugs.webkit.org/show_bug.cgi?id=44863, disentangle style recalc from layout, so that
the former can occur in more places without having to do the latter.

Reviewed by Sam Weinig.

  • ewk/ewk_view.cpp:

(ewk_view_layout_if_needed_recursive):

WebKit/gtk: ​https://bugs.webkit.org/show_bug.cgi?id=44863, disentangle style recalc from layout, so that
the former can occur in more places without having to do the latter.

Reviewed by Sam Weinig.

  • webkit/webkitwebview.cpp:

(webkit_web_view_expose_event):

WebKit/mac: ​https://bugs.webkit.org/show_bug.cgi?id=44863, disentangle style recalc from layout, so that
the former can occur in more places without having to do the latter.

Reviewed by Sam Weinig.

Eliminate Mac-specific code for style re-application and recursive layout/style updating in
favor of the cross-platform code that all the other ports use.

  • Carbon/HIWebView.mm:

(Draw):

  • WebCoreSupport/WebFrameLoaderClient.mm:

(WebFrameLoaderClient::forceLayout):

  • WebView/WebHTMLView.mm:

(-[WebHTMLView _propagateDirtyRectsToOpaqueAncestors]):
(-[WebHTMLView viewWillDraw]):
(-[WebHTMLView _recursiveDisplayRectIfNeededIgnoringOpacity:isVisibleRect:rectIsVisibleRectForView:topView:]):
(-[WebHTMLView _recursiveDisplayAllDirtyWithLockFocus:visRect:]):
(-[WebHTMLView _recursive:displayRectIgnoringOpacity:inContext:topView:]):
(-[WebHTMLView reapplyStyles]):
(-[WebHTMLView layoutToMinimumPageWidth:height:maximumPageWidth:adjustingViewSize:]):
(-[WebHTMLView setNeedsToApplyStyles:]):
(-[WebHTMLView _setPrinting:minimumPageWidth:height:maximumPageWidth:adjustViewSize:paginateScreenContent:]):
(-[WebHTMLView _layoutIfNeeded]):
(-[WebHTMLView _web_updateLayoutAndStyleIfNeededRecursive]):

  • WebView/WebHTMLViewInternal.h:
  • WebView/WebView.mm:

(-[WebView _viewWillDrawInternal]):

WebKit/qt: ​https://bugs.webkit.org/show_bug.cgi?id=44863, disentangle style recalc from layout, so that
the former can occur in more places without having to do the latter.

Reviewed by Sam Weinig.

  • Api/qwebelement.cpp:

(QWebElement::render):

  • Api/qwebframe.cpp:

(QWebFramePrivate::renderRelativeCoords):

WebKit/win: ​https://bugs.webkit.org/show_bug.cgi?id=44863, disentangle style recalc from layout, so that
the former can occur in more places without having to do the latter.

Reviewed by Sam Weinig.

  • WebFrame.cpp:

(WebFrame::paintDocumentRectToContext):

  • WebView.cpp:

(WebView::updateBackingStore):

WebKit/wx: ​https://bugs.webkit.org/show_bug.cgi?id=44863, disentangle style recalc from layout, so that
the former can occur in more places without having to do the latter.

Reviewed by Sam Weinig.

  • WebView.cpp:

(wxWebView::OnPaint):

WebKit2: ​https://bugs.webkit.org/show_bug.cgi?id=44863

Reviewed by Sam Weinig.

  • WebProcess/WebPage/WebPage.cpp:

(WebKit::WebPage::layoutIfNeeded):

10:46 PM Changeset in webkit [66576] by tkent@chromium.org
  • 2 edits in trunk/WebKit/chromium

2010-08-31 Kent Tamura <​tkent@chromium.org>

Reviewed by Dimitri Glazkov.

[Chromium] Add WebKitClient::callOnMainThreadWithDelay()
​https://bugs.webkit.org/show_bug.cgi?id=45016

It is required to remove base/task.h and base/timer.h dependency
from DRT/Chromium.

  • public/WebKitClient.h: (WebKit::WebKitClient::callOnMainThreadWithDelay): Added.
10:21 PM Changeset in webkit [66575] by hyatt@apple.com
  • 4 adds in trunk/LayoutTests

Add test case I forgot to add from previous multicol checkin.

9:49 PM Changeset in webkit [66574] by jeremy@chromium.org
  • 4 edits
    2 adds
    1 delete in trunk/WebCore

2010-08-31 Jeremy Moskovich <​jeremy@chromium.org>

Reviewed by Dimitri Glazkov.

Unfork Chromium's FontPlatformDataChromiumMac.mm.

Modified FontPlatformDataCocoa to provide hooks needed by Chromium.
Moved Chromium's mechanics for OOP font loading to CrossProcessFontLoading.mm.
Added ::create() function to MemoryActivatedFont to match new guidelines for
RefCounted<> classes.

Not possible to test due to sandbox interaction.

  • WebCore.gypi: Update files for chromium build.
  • platform/graphics/chromium/CrossProcessFontLoading.h: Added. (WebCore::MemoryActivatedFont::cgFont): (WebCore::MemoryActivatedFont::atsFontRef):
  • platform/graphics/chromium/CrossProcessFontLoading.mm: Added. (WebCore::MemoryActivatedFont::create): (WebCore::MemoryActivatedFont::MemoryActivatedFont): (WebCore::MemoryActivatedFont::~MemoryActivatedFont): (WebCore::FontPlatformData::loadFont):
  • platform/graphics/chromium/FontPlatformDataChromiumMac.mm: Removed.
  • platform/graphics/cocoa/FontPlatformData.h:
  • platform/graphics/cocoa/FontPlatformDataCocoa.mm: (WebCore::FontPlatformData::loadFont): (WebCore::FontPlatformData::FontPlatformData): (WebCore::FontPlatformData::operator=): (WebCore::FontPlatformData::setFont):
9:27 PM Changeset in webkit [66573] by commit-queue@webkit.org
  • 2 edits in trunk/WebCore

2010-08-31 Martin Robinson <​mrobinson@igalia.com>

Reviewed by Adam Barth.

[GTK] warning: unused variable "queryResult"
​https://bugs.webkit.org/show_bug.cgi?id=44998

No new tests as functionality has not changed.

  • platform/graphics/gstreamer/GStreamerGWorld.cpp: (WebCore::GStreamerGWorld::enterFullscreen): Move queryResult within the gstreamer version guards, where it is used.
8:32 PM Changeset in webkit [66572] by hyatt@apple.com
  • 2 edits in trunk/WebCore

Refinement to previous checkin for pagination of block-level replaced elements. Don't bother to
paginate floating block-level replaced elements for now.

Reviewed by Adam Roben.

Added fast/multicol/float-pagination-failure.html

  • rendering/RenderBlock.cpp:

(WebCore::RenderBlock::paintChildren):

8:30 PM Changeset in webkit [66571] by commit-queue@webkit.org
  • 2 edits in trunk/WebKitTools

2010-08-31 Ademar de Souza Reis Jr <Ademar Reis>

Reviewed by Adam Barth.

Allow Ctrl+C inside Bugzilla::fetch_bug_dictionary
​https://bugs.webkit.org/show_bug.cgi?id=44789

  • Scripts/webkitpy/common/net/bugzilla.py:
8:27 PM Changeset in webkit [66570] by kinuko@chromium.org
  • 12 edits in trunk

2010-08-31 Kinuko Yasuda <​kinuko@chromium.org>

Reviewed by Jian Li.

Add LocalFileSystem.requestFileSystem interface to DOMWindow
​https://bugs.webkit.org/show_bug.cgi?id=44734

  • public/WebRuntimeFeatures.h:
  • src/WebRuntimeFeatures.cpp: (WebKit::WebRuntimeFeatures::enableFileSystem): (WebKit::WebRuntimeFeatures::isFileSystemEnabled):

2010-08-31 Kinuko Yasuda <​kinuko@chromium.org>

Reviewed by Jian Li.

Add LocalFileSystem.requestFileSystem interface to DOMWindow
​https://bugs.webkit.org/show_bug.cgi?id=44734

No new tests; tests will be added later.

  • bindings/generic/RuntimeEnabledFeatures.cpp:
  • bindings/generic/RuntimeEnabledFeatures.h: (WebCore::RuntimeEnabledFeatures::fileSystemEnabled): (WebCore::RuntimeEnabledFeatures::setFileSystemEnabled): (WebCore::RuntimeEnabledFeatures::requestFileSystemEnabled):
  • page/DOMWindow.cpp: (WebCore::DOMWindow::requestFileSystem):
  • page/DOMWindow.h: (WebCore::DOMWindow::):
  • page/DOMWindow.idl:
  • page/Settings.cpp: (WebCore::Settings::setFileSystemRootPath):
  • page/Settings.h: (WebCore::Settings::fileSystemRootPath):
8:12 PM Changeset in webkit [66569] by commit-queue@webkit.org
  • 2 edits in trunk/WebCore

2010-08-31 Kwang Yul Seo <​skyul@company100.net>

Reviewed by Adam Barth.

[BREWMP] Add PLATFORM(BREWMP) guard to PluginViewNone
​https://bugs.webkit.org/show_bug.cgi?id=44768

Build fix. Add PLATFORM(BREWMP) guard to add dummy PluginView::privateBrowsingStateChanged
and PluginView::setJavaScriptPaused.

  • plugins/PluginViewNone.cpp:
7:47 PM Changeset in webkit [66568] by ap@apple.com
  • 5 edits
    2 adds in trunk

2010-08-31 Alexey Proskuryakov <​ap@apple.com>

Reviewed by Adam Barth.

​https://bugs.webkit.org/show_bug.cgi?id=40947
<rdar://problem/8124723> REGRESSION (r57544): XMLHttpRequest POSTed multipart/form-data
wrongfully gets a charset appended

  • http/tests/xmlhttprequest/access-control-basic-whitelist-request-headers-expected.txt:
  • http/tests/xmlhttprequest/request-encoding2.html: Updated tests that has wrong expectations.
  • http/tests/xmlhttprequest/post-with-boundary-expected.txt: Added.
  • http/tests/xmlhttprequest/post-with-boundary.html: Added. Added a new test, sending a valid multipart form with boundary.

2010-08-31 Alexey Proskuryakov <​ap@apple.com>

Reviewed by Adam Barth.

​https://bugs.webkit.org/show_bug.cgi?id=40947
<rdar://problem/8124723> REGRESSION (r57544): XMLHttpRequest POSTed multipart/form-data
wrongfully gets a charset appended

Test: http/tests/xmlhttprequest/post-with-boundary.html

  • xml/XMLHttpRequest.cpp: (WebCore::replaceCharsetInMediaType): Changed to only set charset if it was already present, and never append. (WebCore::XMLHttpRequest::send): Updated for replaceCharsetInMediaType renaming.
7:11 PM Changeset in webkit [66567] by commit-queue@webkit.org
  • 3 edits
    1 move
    5 adds in trunk/WebCore

2010-08-31 Daniel Cheng <​dcheng@chromium.org>

Reviewed by Tony Chang.

[chromium] Implement Readable/Writable versions of ChromiumDataObjectNew
​https://bugs.webkit.org/show_bug.cgi?id=44917

This is the second of several patches to update Chromium to support
arbitrary data types in event.dataTransfer.
ReadableDataObject simply calls through to several Chromium IPCs to
retrieve the appropriate data.
WritableDataObject ostensibly buffers arbitrary data internally until
a call is made to flush the data to the platform-specific abstraction;
unfortunately, a special exception is currently made for pasteboard
handling due to how it is handled in WebCore.

No new tests.

  • WebCore.gypi:
  • platform/chromium/ChromiumDataObjectNew.h: Removed.
  • platform/chromium/ClipboardChromium.cpp:
  • platform/chromium/ClipboardMimeTypes.cpp: Added.
  • platform/chromium/ClipboardMimeTypes.h: Added.
  • platform/chromium/ReadableDataObject.cpp: Added. (WebCore::clipboardBuffer): (WebCore::ReadableDataObject::create): (WebCore::ReadableDataObject::ReadableDataObject): (WebCore::ReadableDataObject::hasData): (WebCore::ReadableDataObject::types): (WebCore::ReadableDataObject::getData): (WebCore::ReadableDataObject::getURL): (WebCore::ReadableDataObject::getHTML): (WebCore::ReadableDataObject::hasFilenames): (WebCore::ReadableDataObject::filenames): (WebCore::ReadableDataObject::ensureTypeCacheInitialized):
  • platform/chromium/ReadableDataObject.h: Added.
  • platform/chromium/WritableDataObject.cpp: Added. (WebCore::WritableDataObject::create): (WebCore::WritableDataObject::WritableDataObject): (WebCore::WritableDataObject::clearData): (WebCore::WritableDataObject::clearAllExceptFiles): (WebCore::WritableDataObject::clearAll): (WebCore::WritableDataObject::setData): (WebCore::WritableDataObject::setURL): (WebCore::WritableDataObject::setHTML): (WebCore::WritableDataObject::dataMap): (WebCore::WritableDataObject::urlTitle): (WebCore::WritableDataObject::htmlBaseURL): (WebCore::WritableDataObject::fileExtension): (WebCore::WritableDataObject::fileContentFilename): (WebCore::WritableDataObject::fileContent): (WebCore::WritableDataObject::setFileExtension): (WebCore::WritableDataObject::setFileContentFilename): (WebCore::WritableDataObject::setFileContent):
  • platform/chromium/WritableDataObject.h: Added.
7:00 PM Changeset in webkit [66566] by Darin Adler
  • 2 edits in trunk/LayoutTests

Fix gtk bot.

  • platform/gtk/Skipped: Skip the webarchive/loading/missing-data.html for the same reason

the nearly identical webarchive/loading/mainresource-null-mimetype-crash.html is skipped.

6:55 PM Changeset in webkit [66565] by benjamin.poulain@nokia.com
  • 2 edits in trunk/WebKit/qt

2010-08-31 Benjamin Poulain <​benjamin.poulain@nokia.com>

Reviewed by Kenneth Rohde Christiansen.

[Qt] tst_QWebPage::originatingObjectInNetworkRequests() fails on trunk
​https://bugs.webkit.org/show_bug.cgi?id=45001

Fix the test, we should not use data encoded URL since we are not
setting an URL but setting the substituteData directly.

  • tests/qwebpage/tst_qwebpage.cpp: (tst_QWebPage::originatingObjectInNetworkRequests):
6:16 PM Changeset in webkit [66564] by commit-queue@webkit.org
  • 5 edits in trunk/WebKitTools

2010-08-31 Leandro Pereira <​leandro@profusion.mobi>

Reviewed by Adam Barth.

ews: Add support for EFL-EWS
​https://bugs.webkit.org/show_bug.cgi?id=44982

  • QueueStatusServer/model/queues.py: Add "efl-ews" to queues list.
  • Scripts/webkitpy/common/config/ports.py: Define a EflPort class and add it to the ports dict.
  • Scripts/webkitpy/tool/commands/earlywarningsystem.py: Define a EflEWS class.
  • Scripts/webkitpy/tool/commands/earlywarningsystem_unittest.py: Add a test case for the EFL EWS.
6:07 PM Changeset in webkit [66563] by tony@chromium.org
  • 8 edits in trunk/LayoutTests

2010-08-31 Tony Chang <​tony@chromium.org>

Unreviewed, update pngs where the checksum is correct, but the png doesn't match.

  • platform/chromium-linux/fast/dom/attr_dead_doc-expected.png:
  • platform/chromium-linux/fast/frames/content-opacity-1-expected.png:
  • platform/chromium-linux/fast/frames/content-opacity-2-expected.png:
  • platform/chromium-linux/fast/frames/iframe-text-contents-expected.png:
  • platform/chromium-linux/tables/mozilla/bugs/bug137388-1-expected.png:
  • platform/chromium-linux/tables/mozilla/bugs/bug137388-3-expected.png:
  • platform/chromium-linux/tables/mozilla/bugs/bug50695-2-expected.png:
6:02 PM Changeset in webkit [66562] by Darin Adler
  • 3 edits
    2 copies
    1 add in trunk

Null-deref when decoding web archive with no data key in LegacyWebArchive::createResource
​https://bugs.webkit.org/show_bug.cgi?id=45000

Reviewed by Sam Weinig.

WebCore:

Test: webarchive/loading/missing-data.html

  • loader/archive/cf/LegacyWebArchive.cpp:

(WebCore::LegacyWebArchive::createResource): Use wrapCFData instead of CFDataGetBytePtr/GetLength,
which is both more efficient, and can handle null. Also fixed incorrect use of ParsedURLString,
since the string is coming from a file and so there are no guarantees about its format.

LayoutTests:

  • webarchive/loading/missing-data-expected.txt: Added.
  • webarchive/loading/missing-data.html: Added.
  • webarchive/loading/resources/missing-data.webarchive: Added.
5:56 PM Changeset in webkit [66561] by tony@chromium.org
  • 1 edit
    303 adds in trunk/LayoutTests

2010-08-31 Tony Chang <​tony@chromium.org>

Unreviewed. Add pngs for chromium linux results that have checksums.

  • platform/chromium-linux/css2.1/t0505-c16-descendant-01-e-expected.png: Added.
  • platform/chromium-linux/css2.1/t0509-c15-ids-01-e-expected.png: Added.
  • platform/chromium-linux/css2.1/t1205-c563-list-type-00-b-expected.png: Added.
  • platform/chromium-linux/css2.1/t1205-c564-list-img-00-b-g-expected.png: Added.
  • platform/chromium-linux/editing/deleting/4845371-expected.png: Added.
  • platform/chromium-linux/editing/deleting/4922367-expected.png: Added.
  • platform/chromium-linux/editing/deleting/5156801-2-expected.png: Added.
  • platform/chromium-linux/editing/deleting/5300379-expected.png: Added.
  • platform/chromium-linux/editing/deleting/delete-3608430-fix-expected.png: Added.
  • platform/chromium-linux/editing/deleting/delete-3608445-fix-expected.png: Added.
  • platform/chromium-linux/editing/deleting/delete-3608462-fix-expected.png: Added.
  • platform/chromium-linux/editing/deleting/delete-4083333-fix-expected.png: Added.
  • platform/chromium-linux/editing/deleting/delete-block-contents-003-expected.png: Added.
  • platform/chromium-linux/editing/deleting/delete-block-merge-contents-001-expected.png: Added.
  • platform/chromium-linux/editing/deleting/delete-block-merge-contents-002-expected.png: Added.
  • platform/chromium-linux/editing/deleting/delete-block-merge-contents-003-expected.png: Added.
  • platform/chromium-linux/editing/deleting/delete-block-merge-contents-004-expected.png: Added.
  • platform/chromium-linux/editing/deleting/delete-block-merge-contents-005-expected.png: Added.
  • platform/chromium-linux/editing/deleting/delete-block-merge-contents-006-expected.png: Added.
  • platform/chromium-linux/editing/deleting/delete-block-merge-contents-007-expected.png: Added.
  • platform/chromium-linux/editing/deleting/delete-block-merge-contents-008-expected.png: Added.
  • platform/chromium-linux/editing/deleting/delete-block-merge-contents-009-expected.png: Added.
  • platform/chromium-linux/editing/deleting/delete-block-merge-contents-010-expected.png: Added.
  • platform/chromium-linux/editing/deleting/delete-block-merge-contents-011-expected.png: Added.
  • platform/chromium-linux/editing/deleting/delete-block-merge-contents-012-expected.png: Added.
  • platform/chromium-linux/editing/deleting/delete-block-merge-contents-013-expected.png: Added.
  • platform/chromium-linux/editing/deleting/delete-block-merge-contents-014-expected.png: Added.
  • platform/chromium-linux/editing/deleting/delete-block-merge-contents-015-expected.png: Added.
  • platform/chromium-linux/editing/deleting/delete-block-merge-contents-016-expected.png: Added.
  • platform/chromium-linux/editing/deleting/delete-block-merge-contents-017-expected.png: Added.
  • platform/chromium-linux/editing/deleting/delete-block-table-expected.png: Added.
  • platform/chromium-linux/editing/deleting/delete-br-001-expected.png: Added.
  • platform/chromium-linux/editing/deleting/delete-br-002-expected.png: Added.
  • platform/chromium-linux/editing/deleting/delete-br-003-expected.png: Added.
  • platform/chromium-linux/editing/deleting/delete-br-004-expected.png: Added.
  • platform/chromium-linux/editing/deleting/delete-br-005-expected.png: Added.
  • platform/chromium-linux/editing/deleting/delete-br-006-expected.png: Added.
  • platform/chromium-linux/editing/deleting/delete-br-007-expected.png: Added.
  • platform/chromium-linux/editing/deleting/delete-character-001-expected.png: Added.
  • platform/chromium-linux/editing/deleting/delete-contiguous-ws-001-expected.png: Added.
  • platform/chromium-linux/editing/deleting/delete-hr-expected.png: Added.
  • platform/chromium-linux/editing/deleting/delete-image-001-expected.png: Added.
  • platform/chromium-linux/editing/deleting/delete-image-002-expected.png: Added.
  • platform/chromium-linux/editing/deleting/delete-image-003-expected.png: Added.
  • platform/chromium-linux/editing/deleting/delete-leading-ws-001-expected.png: Added.
  • platform/chromium-linux/editing/deleting/delete-listitem-001-expected.png: Added.
  • platform/chromium-linux/editing/deleting/delete-listitem-002-expected.png: Added.
  • platform/chromium-linux/editing/deleting/delete-selection-001-expected.png: Added.
  • platform/chromium-linux/editing/deleting/delete-tab-001-expected.png: Added.
  • platform/chromium-linux/editing/deleting/delete-tab-002-expected.png: Added.
  • platform/chromium-linux/editing/deleting/delete-tab-003-expected.png: Added.
  • platform/chromium-linux/editing/deleting/delete-tab-004-expected.png: Added.
  • platform/chromium-linux/editing/deleting/delete-trailing-ws-001-expected.png: Added.
  • platform/chromium-linux/editing/deleting/delete-trailing-ws-002-expected.png: Added.
  • platform/chromium-linux/editing/deleting/delete-ws-fixup-003-expected.png: Added.
  • platform/chromium-linux/editing/deleting/delete-ws-fixup-004-expected.png: Added.
  • platform/chromium-linux/editing/deleting/list-item-1-expected.png: Added.
  • platform/chromium-linux/editing/deleting/merge-different-styles-expected.png: Added.
  • platform/chromium-linux/editing/deleting/merge-into-empty-block-2-expected.png: Added.
  • platform/chromium-linux/editing/deleting/merge-unrendered-space-expected.png: Added.
  • platform/chromium-linux/editing/deleting/merge-whitespace-pre-expected.png: Added.
  • platform/chromium-linux/editing/deleting/paragraph-in-preserveNewline-expected.png: Added.
  • platform/chromium-linux/editing/deleting/pruning-after-merge-2-expected.png: Added.
  • platform/chromium-linux/editing/deleting/smart-delete-003-expected.png: Added.
  • platform/chromium-linux/editing/deleting/table-cells-expected.png: Added.
  • platform/chromium-linux/editing/deleting/whitespace-pre-1-expected.png: Added.
  • platform/chromium-linux/editing/execCommand/4916402-expected.png: Added.
  • platform/chromium-linux/editing/execCommand/5569741-expected.png: Added.
  • platform/chromium-linux/editing/execCommand/format-block-with-braces-expected.png: Added.
  • platform/chromium-linux/editing/execCommand/format-block-with-trailing-br-expected.png: Added.
  • platform/chromium-linux/editing/execCommand/indent-empty-root-expected.png: Added.
  • platform/chromium-linux/editing/execCommand/indent-list-item-expected.png: Added.
  • platform/chromium-linux/editing/execCommand/insert-list-and-stitch-expected.png: Added.
  • platform/chromium-linux/editing/execCommand/nsresponder-indent-expected.png: Added.
  • platform/chromium-linux/editing/execCommand/nsresponder-outdent-expected.png: Added.
  • platform/chromium-linux/editing/execCommand/remove-list-item-1-expected.png: Added.
  • platform/chromium-linux/editing/inserting/4875189-1-expected.png: Added.
  • platform/chromium-linux/editing/inserting/4959067-expected.png: Added.
  • platform/chromium-linux/editing/inserting/4960120-2-expected.png: Added.
  • platform/chromium-linux/editing/inserting/5002441-expected.png: Added.
  • platform/chromium-linux/editing/inserting/5156401-2-expected.png: Added.
  • platform/chromium-linux/editing/inserting/5549929-3-expected.png: Added.
  • platform/chromium-linux/editing/inserting/break-blockquote-after-delete-expected.png: Added.
  • platform/chromium-linux/editing/inserting/editable-inline-element-expected.png: Added.
  • platform/chromium-linux/editing/inserting/insert-3659587-fix-expected.png: Added.
  • platform/chromium-linux/editing/inserting/insert-3775316-fix-expected.png: Added.
  • platform/chromium-linux/editing/inserting/insert-3778059-fix-expected.png: Added.
  • platform/chromium-linux/editing/inserting/insert-3800346-fix-expected.png: Added.
  • platform/chromium-linux/editing/inserting/insert-after-delete-001-expected.png: Added.
  • platform/chromium-linux/editing/inserting/insert-br-001-expected.png: Added.
  • platform/chromium-linux/editing/inserting/insert-br-002-expected.png: Added.
  • platform/chromium-linux/editing/inserting/insert-br-003-expected.png: Added.
  • platform/chromium-linux/editing/inserting/insert-br-004-expected.png: Added.
  • platform/chromium-linux/editing/inserting/insert-br-005-expected.png: Added.
  • platform/chromium-linux/editing/inserting/insert-br-006-expected.png: Added.
  • platform/chromium-linux/editing/inserting/insert-br-007-expected.png: Added.
  • platform/chromium-linux/editing/inserting/insert-br-008-expected.png: Added.
  • platform/chromium-linux/editing/inserting/insert-br-at-tabspan-001-expected.png: Added.
  • platform/chromium-linux/editing/inserting/insert-br-at-tabspan-002-expected.png: Added.
  • platform/chromium-linux/editing/inserting/insert-br-at-tabspan-003-expected.png: Added.
  • platform/chromium-linux/editing/inserting/insert-space-in-empty-doc-expected.png: Added.
  • platform/chromium-linux/editing/inserting/insert-tab-001-expected.png: Added.
  • platform/chromium-linux/editing/inserting/insert-tab-002-expected.png: Added.
  • platform/chromium-linux/editing/inserting/insert-tab-003-expected.png: Added.
  • platform/chromium-linux/editing/inserting/insert-tab-004-expected.png: Added.
  • platform/chromium-linux/editing/inserting/insert-text-at-tabspan-001-expected.png: Added.
  • platform/chromium-linux/editing/inserting/insert-text-at-tabspan-002-expected.png: Added.
  • platform/chromium-linux/editing/inserting/insert-text-at-tabspan-003-expected.png: Added.
  • platform/chromium-linux/editing/inserting/insert-text-with-newlines-expected.png: Added.
  • platform/chromium-linux/editing/inserting/line-break-expected.png: Added.
  • platform/chromium-linux/editing/inserting/paragraph-separator-01-expected.png: Added.
  • platform/chromium-linux/editing/inserting/paragraph-separator-02-expected.png: Added.
  • platform/chromium-linux/editing/inserting/paragraph-separator-in-table-1-expected.png: Added.
  • platform/chromium-linux/editing/inserting/paragraph-separator-in-table-2-expected.png: Added.
  • platform/chromium-linux/editing/inserting/redo-expected.png: Added.
  • platform/chromium-linux/editing/inserting/typing-001-expected.png: Added.
  • platform/chromium-linux/editing/inserting/typing-002-expected.png: Added.
  • platform/chromium-linux/editing/inserting/typing-around-br-001-expected.png: Added.
  • platform/chromium-linux/editing/inserting/typing-around-image-001-expected.png: Added.
  • platform/chromium-linux/editing/pasteboard/4076267-2-expected.png: Added.
  • platform/chromium-linux/editing/pasteboard/4076267-3-expected.png: Added.
  • platform/chromium-linux/editing/pasteboard/4076267-expected.png: Added.
  • platform/chromium-linux/editing/pasteboard/4700297-expected.png: Added.
  • platform/chromium-linux/editing/pasteboard/5006779-expected.png: Added.
  • platform/chromium-linux/editing/pasteboard/5065605-expected.png: Added.
  • platform/chromium-linux/editing/pasteboard/5071074-2-expected.png: Added.
  • platform/chromium-linux/editing/pasteboard/5075944-expected.png: Added.
  • platform/chromium-linux/editing/pasteboard/5134759-expected.png: Added.
  • platform/chromium-linux/editing/pasteboard/5156401-1-expected.png: Added.
  • platform/chromium-linux/editing/pasteboard/5483567-expected.png: Added.
  • platform/chromium-linux/editing/pasteboard/5601583-1-expected.png: Added.
  • platform/chromium-linux/editing/pasteboard/8145-1-expected.png: Added.
  • platform/chromium-linux/editing/pasteboard/cut-text-001-expected.png: Added.
  • platform/chromium-linux/editing/pasteboard/displaced-placeholder-expected.png: Added.
  • platform/chromium-linux/editing/pasteboard/merge-after-delete-expected.png: Added.
  • platform/chromium-linux/editing/pasteboard/merge-end-3-expected.png: Added.
  • platform/chromium-linux/editing/pasteboard/merge-end-4-expected.png: Added.
  • platform/chromium-linux/editing/pasteboard/merge-start-list-expected.png: Added.
  • platform/chromium-linux/editing/pasteboard/paste-blockquote-1-expected.png: Added.
  • platform/chromium-linux/editing/pasteboard/paste-blockquote-2-expected.png: Added.
  • platform/chromium-linux/editing/pasteboard/paste-blockquote-after-blockquote-expected.png: Added.
  • platform/chromium-linux/editing/pasteboard/paste-blockquote-into-blockquote-4-expected.png: Added.
  • platform/chromium-linux/editing/pasteboard/paste-blockquote-into-blockquote-expected.png: Added.
  • platform/chromium-linux/editing/pasteboard/paste-table-001-expected.png: Added.
  • platform/chromium-linux/editing/pasteboard/paste-text-001-expected.png: Added.
  • platform/chromium-linux/editing/pasteboard/paste-text-at-tabspan-001-expected.png: Added.
  • platform/chromium-linux/editing/pasteboard/paste-text-at-tabspan-002-expected.png: Added.
  • platform/chromium-linux/editing/pasteboard/paste-text-at-tabspan-003-expected.png: Added.
  • platform/chromium-linux/editing/pasteboard/paste-unrendered-select-expected.png: Added.
  • platform/chromium-linux/editing/pasteboard/paste-xml-expected.png: Added.
  • platform/chromium-linux/editing/selection/4866671-expected.png: Added.
  • platform/chromium-linux/editing/selection/4932260-1-expected.png: Added.
  • platform/chromium-linux/editing/selection/4932260-2-expected.png: Added.
  • platform/chromium-linux/editing/selection/4932260-3-expected.png: Added.
  • platform/chromium-linux/editing/selection/5076323-2-expected.png: Added.
  • platform/chromium-linux/editing/selection/5076323-3-expected.png: Added.
  • platform/chromium-linux/editing/selection/contenteditable-click-inside-expected.png: Added.
  • platform/chromium-linux/editing/selection/drag-text-delay-expected.png: Added.
  • platform/chromium-linux/editing/selection/editable-non-editable-crash-expected.png: Added.
  • platform/chromium-linux/editing/selection/extend-by-character-006-expected.png: Added.
  • platform/chromium-linux/editing/selection/extend-by-word-001-expected.png: Added.
  • platform/chromium-linux/editing/selection/fake-doubleclick-expected.png: Added.
  • platform/chromium-linux/editing/selection/fake-drag-expected.png: Added.
  • platform/chromium-linux/editing/selection/move-backwords-by-word-001-expected.png: Added.
  • platform/chromium-linux/editing/selection/move-between-blocks-no-001-expected.png: Added.
  • platform/chromium-linux/editing/selection/move-by-character-001-expected.png: Added.
  • platform/chromium-linux/editing/selection/move-by-character-002-expected.png: Added.
  • platform/chromium-linux/editing/selection/move-by-character-004-expected.png: Added.
  • platform/chromium-linux/editing/selection/move-by-character-6-expected.png: Added.
  • platform/chromium-linux/editing/selection/move-by-line-002-expected.png: Added.
  • platform/chromium-linux/editing/selection/move-by-sentence-001-expected.png: Added.
  • platform/chromium-linux/editing/selection/move-by-word-001-expected.png: Added.
  • platform/chromium-linux/editing/selection/move-past-trailing-space-expected.png: Added.
  • platform/chromium-linux/editing/selection/selection-3748164-fix-expected.png: Added.
  • platform/chromium-linux/editing/selection/table-caret-1-expected.png: Added.
  • platform/chromium-linux/editing/selection/table-caret-2-expected.png: Added.
  • platform/chromium-linux/editing/selection/table-caret-3-expected.png: Added.
  • platform/chromium-linux/editing/selection/wrapped-line-caret-1-expected.png: Added.
  • platform/chromium-linux/editing/selection/wrapped-line-caret-2-expected.png: Added.
  • platform/chromium-linux/editing/spelling/spelling-expected.png: Added.
  • platform/chromium-linux/editing/style/5065910-expected.png: Added.
  • platform/chromium-linux/editing/style/fontsize-1-expected.png: Added.
  • platform/chromium-linux/editing/style/highlight-expected.png: Added.
  • platform/chromium-linux/editing/style/style-3681552-fix-001-expected.png: Added.
  • platform/chromium-linux/editing/style/style-3681552-fix-002-expected.png: Added.
  • platform/chromium-linux/editing/style/typing-style-001-expected.png: Added.
  • platform/chromium-linux/editing/style/typing-style-002-expected.png: Added.
  • platform/chromium-linux/editing/undo/4063751-expected.png: Added.
  • platform/chromium-linux/editing/undo/redo-typing-001-expected.png: Added.
  • platform/chromium-linux/editing/undo/undo-typing-001-expected.png: Added.
  • platform/chromium-linux/fast/block/positioning/absolute-in-inline-short-ltr-expected.png: Added.
  • platform/chromium-linux/fast/block/positioning/absolute-in-inline-short-rtl-expected.png: Added.
  • platform/chromium-linux/fast/css/outline-auto-empty-rects-expected.png: Added.
  • platform/chromium-linux/fast/css/outline-auto-location-expected.png: Added.
  • platform/chromium-linux/fast/events/label-focus-expected.png: Added.
  • platform/chromium-linux/fast/forms/textfield-outline-expected.png: Added.
  • platform/chromium-linux/fast/forms/textfield-overflow-expected.png: Added.
  • platform/chromium-linux/fast/lists/004-expected.png: Added.
  • platform/chromium-linux/fast/repaint/4776765-expected.png: Added.
  • platform/chromium-linux/fast/repaint/backgroundSizeRepaint-expected.png: Added.
  • platform/chromium-linux/fast/repaint/caret-outside-block-expected.png: Added.
  • platform/chromium-linux/fast/repaint/delete-into-nested-block-expected.png: Added.
  • platform/chromium-linux/fast/repaint/float-overflow-expected.png: Added.
  • platform/chromium-linux/fast/repaint/float-overflow-right-expected.png: Added.
  • platform/chromium-linux/fast/repaint/focus-layers-expected.png: Added.
  • platform/chromium-linux/fast/repaint/layout-state-scrolloffset-expected.png: Added.
  • platform/chromium-linux/fast/repaint/layout-state-scrolloffset2-expected.png: Added.
  • platform/chromium-linux/fast/repaint/layout-state-scrolloffset3-expected.png: Added.
  • platform/chromium-linux/fast/repaint/list-marker-2-expected.png: Added.
  • platform/chromium-linux/fast/repaint/stacked-diacritics-expected.png: Added.
  • platform/chromium-linux/fast/table/empty-table-percent-height-expected.png: Added.
  • platform/chromium-linux/fast/text/justified-selection-expected.png: Added.
  • platform/chromium-linux/media/audio-controls-rendering-expected.png: Added.
  • platform/chromium-linux/media/controls-after-reload-expected.png: Added.
  • platform/chromium-linux/media/controls-strict-expected.png: Added.
  • platform/chromium-linux/media/video-display-toggle-expected.png: Added.
  • platform/chromium-linux/media/video-empty-source-expected.png: Added.
  • platform/chromium-linux/media/video-no-audio-expected.png: Added.
  • platform/chromium-linux/svg/W3C-SVG-1.1/animate-elem-85-t-expected.png: Added.
  • platform/chromium-linux/svg/W3C-SVG-1.1/color-prop-03-t-expected.png: Added.
  • platform/chromium-linux/svg/W3C-SVG-1.1/coords-coord-01-t-expected.png: Added.
  • platform/chromium-linux/svg/W3C-SVG-1.1/coords-coord-02-t-expected.png: Added.
  • platform/chromium-linux/svg/W3C-SVG-1.1/painting-fill-05-b-expected.png: Added.
  • platform/chromium-linux/svg/W3C-SVG-1.1/painting-stroke-07-t-expected.png: Added.
  • platform/chromium-linux/svg/W3C-SVG-1.1/paths-data-13-t-expected.png: Added.
  • platform/chromium-linux/svg/W3C-SVG-1.1/paths-data-15-t-expected.png: Added.
  • platform/chromium-linux/svg/W3C-SVG-1.1/pservers-grad-15-b-expected.png: Added.
  • platform/chromium-linux/svg/W3C-SVG-1.1/pservers-grad-16-b-expected.png: Added.
  • platform/chromium-linux/svg/W3C-SVG-1.1/pservers-grad-18-b-expected.png: Added.
  • platform/chromium-linux/svg/W3C-SVG-1.1/shapes-circle-01-t-expected.png: Added.
  • platform/chromium-linux/svg/W3C-SVG-1.1/shapes-circle-02-t-expected.png: Added.
  • platform/chromium-linux/svg/W3C-SVG-1.1/shapes-ellipse-01-t-expected.png: Added.
  • platform/chromium-linux/svg/W3C-SVG-1.1/shapes-ellipse-02-t-expected.png: Added.
  • platform/chromium-linux/svg/W3C-SVG-1.1/shapes-line-01-t-expected.png: Added.
  • platform/chromium-linux/svg/W3C-SVG-1.1/shapes-polyline-01-t-expected.png: Added.
  • platform/chromium-linux/svg/W3C-SVG-1.1/shapes-rect-01-t-expected.png: Added.
  • platform/chromium-linux/svg/W3C-SVG-1.1/shapes-rect-02-t-expected.png: Added.
  • platform/chromium-linux/svg/W3C-SVG-1.1/struct-defs-01-t-expected.png: Added.
  • platform/chromium-linux/svg/W3C-SVG-1.1/struct-frag-04-t-expected.png: Added.
  • platform/chromium-linux/svg/W3C-SVG-1.1/struct-group-01-t-expected.png: Added.
  • platform/chromium-linux/svg/W3C-SVG-1.1/struct-group-02-b-expected.png: Added.
  • platform/chromium-linux/svg/W3C-SVG-1.1/struct-image-04-t-expected.png: Added.
  • platform/chromium-linux/svg/W3C-SVG-1.1/struct-image-05-b-expected.png: Added.
  • platform/chromium-linux/svg/W3C-SVG-1.1/struct-image-09-t-expected.png: Added.
  • platform/chromium-linux/svg/W3C-SVG-1.1/struct-image-10-t-expected.png: Added.
  • platform/chromium-linux/svg/W3C-SVG-1.1/styling-inherit-01-b-expected.png: Added.
  • platform/chromium-linux/svg/W3C-SVG-1.1/styling-pres-01-t-expected.png: Added.
  • platform/chromium-linux/svg/custom/focus-ring-expected.png: Added.
  • platform/chromium-linux/svg/custom/foreignObject-crash-on-hover-expected.png: Added.
  • platform/chromium-linux/svg/custom/js-update-path-changes-expected.png: Added.
  • platform/chromium-linux/svg/custom/js-update-path-removal-expected.png: Added.
  • platform/chromium-linux/svg/custom/linking-a-03-b-preserveAspectRatio-expected.png: Added.
  • platform/chromium-linux/svg/custom/linking-a-03-b-viewTarget-expected.png: Added.
  • platform/chromium-linux/svg/custom/linking-a-03-b-zoomAndPan-expected.png: Added.
  • platform/chromium-linux/svg/custom/path-textPath-simulation-expected.png: Added.
  • platform/chromium-linux/svg/custom/relative-sized-inner-svg-expected.png: Added.
  • platform/chromium-linux/svg/custom/relative-sized-use-without-attributes-on-symbol-expected.png: Added.
  • platform/chromium-linux/svg/custom/use-clipped-hit-expected.png: Added.
  • platform/chromium-linux/svg/custom/use-elementInstance-event-target-expected.png: Added.
  • platform/chromium-linux/svg/custom/use-elementInstance-methods-expected.png: Added.
  • platform/chromium-linux/svg/custom/use-event-handler-on-referenced-element-expected.png: Added.
  • platform/chromium-linux/svg/custom/use-event-handler-on-use-element-expected.png: Added.
  • platform/chromium-linux/svg/custom/use-nested-transform-expected.png: Added.
  • platform/chromium-linux/svg/custom/use-on-symbol-inside-pattern-expected.png: Added.
  • platform/chromium-linux/svg/custom/width-full-percentage-expected.png: Added.
  • platform/chromium-linux/svg/filters/subRegion-one-effect-expected.png: Added.
  • platform/chromium-linux/svg/filters/subRegion-two-effects-expected.png: Added.
  • platform/chromium-linux/svg/hixie/error/003-expected.png: Added.
  • platform/chromium-linux/svg/hixie/perf/001-expected.png: Added.
  • platform/chromium-linux/svg/hixie/perf/002-expected.png: Added.
  • platform/chromium-linux/svg/hixie/shapes/path/001-expected.png: Added.
  • platform/chromium-linux/svg/text/textPathBoundsBug-expected.png: Added.
  • platform/chromium-linux/tables/mozilla/bugs/bug2516-expected.png: Added.
  • platform/chromium-linux/tables/mozilla/bugs/bug3191-expected.png: Added.
  • platform/chromium-linux/tables/mozilla/bugs/bug4284-expected.png: Added.
  • platform/chromium-linux/tables/mozilla/bugs/bug4527-expected.png: Added.
  • platform/chromium-linux/tables/mozilla/core/col_widths_auto_autoFix-expected.png: Added.
  • platform/chromium-linux/tables/mozilla/core/misc-expected.png: Added.
  • platform/chromium-linux/tables/mozilla/marvin/tbody_valign_baseline-expected.png: Added.
  • platform/chromium-linux/tables/mozilla/marvin/tbody_valign_bottom-expected.png: Added.
  • platform/chromium-linux/tables/mozilla/marvin/tbody_valign_middle-expected.png: Added.
  • platform/chromium-linux/tables/mozilla/marvin/tbody_valign_top-expected.png: Added.
  • platform/chromium-linux/tables/mozilla/marvin/td_valign_baseline-expected.png: Added.
  • platform/chromium-linux/tables/mozilla/marvin/td_valign_bottom-expected.png: Added.
  • platform/chromium-linux/tables/mozilla/marvin/td_valign_middle-expected.png: Added.
  • platform/chromium-linux/tables/mozilla/marvin/td_valign_top-expected.png: Added.
  • platform/chromium-linux/tables/mozilla/marvin/tfoot_valign_baseline-expected.png: Added.
  • platform/chromium-linux/tables/mozilla/marvin/tfoot_valign_bottom-expected.png: Added.
  • platform/chromium-linux/tables/mozilla/marvin/tfoot_valign_middle-expected.png: Added.
  • platform/chromium-linux/tables/mozilla/marvin/tfoot_valign_top-expected.png: Added.
  • platform/chromium-linux/tables/mozilla/marvin/th_valign_baseline-expected.png: Added.
  • platform/chromium-linux/tables/mozilla/marvin/th_valign_bottom-expected.png: Added.
  • platform/chromium-linux/tables/mozilla/marvin/th_valign_middle-expected.png: Added.
  • platform/chromium-linux/tables/mozilla/marvin/th_valign_top-expected.png: Added.
  • platform/chromium-linux/tables/mozilla/marvin/thead_valign_baseline-expected.png: Added.
  • platform/chromium-linux/tables/mozilla/marvin/thead_valign_bottom-expected.png: Added.
  • platform/chromium-linux/tables/mozilla/marvin/thead_valign_middle-expected.png: Added.
  • platform/chromium-linux/tables/mozilla/marvin/thead_valign_top-expected.png: Added.
  • platform/chromium-linux/tables/mozilla/marvin/tr_valign_baseline-expected.png: Added.
  • platform/chromium-linux/tables/mozilla/marvin/tr_valign_bottom-expected.png: Added.
  • platform/chromium-linux/tables/mozilla/marvin/tr_valign_middle-expected.png: Added.
  • platform/chromium-linux/tables/mozilla/marvin/tr_valign_top-expected.png: Added.
  • platform/chromium-linux/tables/mozilla_expected_failures/bugs/bug6933-expected.png: Added.
5:38 PM Changeset in webkit [66560] by kbr@google.com
  • 5 edits in trunk/WebCore

2010-08-31 Kenneth Russell <​kbr@google.com>

Reviewed by Chris Marrin.

Add length and related operations to FloatPoint and FloatSize
​https://bugs.webkit.org/show_bug.cgi?id=44994

Added length and squared length operations to both FloatPoint and
FloatSize, and added set(x, y), dot and normalize operations to
FloatPoint. These changes have been tested with new code to be
added later.

  • platform/graphics/FloatPoint.cpp: (WebCore::FloatPoint::normalize): (WebCore::FloatPoint::length):
  • platform/graphics/FloatPoint.h: (WebCore::FloatPoint::set): (WebCore::FloatPoint::dot): (WebCore::FloatPoint::lengthSquared): (WebCore::operator*):
  • platform/graphics/FloatSize.cpp: (WebCore::FloatSize::diagonalLength):
  • platform/graphics/FloatSize.h: (WebCore::FloatSize::diagonalLengthSquared):
5:34 PM Changeset in webkit [66559] by zmo@google.com
  • 4 edits in trunk

2010-08-31 Zhenyao Mo <​zmo@google.com>

Reviewed by Kenneth Russell.

gl-teximage.html and tex-image-and-sub-image-2d-with-image.html failed on Chromium Mac bots (10.5)
​https://bugs.webkit.org/show_bug.cgi?id=44993

  • platform/graphics/cg/GraphicsContext3DCG.cpp: (WebCore::GraphicsContext3D::getImageData): Take into consideration the source image pixel row alignment.

2010-08-31 Zhenyao Mo <​zmo@google.com>

Reviewed by Kenneth Russell.

gl-teximage.html and tex-image-and-sub-image-2d-with-image.html failed on Chromium Mac bots (10.5)
​https://bugs.webkit.org/show_bug.cgi?id=44993

  • platform/chromium/test_expectations.txt: Enabling the failing tests with the fix.
5:20 PM Changeset in webkit [66558] by jhoneycutt@apple.com
  • 2 edits in trunk/WebKit2

Fix some copy/paste errors.

Reviewed by Steve Falkenburg.

  • WebProcess/InjectedBundle/InjectedBundlePageFormClient.cpp:

(WebKit::InjectedBundlePageFormClient::textFieldDidEndEditing):
Null check the correct callback.
(WebKit::InjectedBundlePageFormClient::textDidChangeInTextField):
Ditto.
(WebKit::InjectedBundlePageFormClient::textDidChangeInTextArea):
Ditto.

5:18 PM Changeset in webkit [66557] by jhoneycutt@apple.com
  • 5 edits in trunk/WebKit2

Crash when closing a page with a form field when using WebKit2.

Reviewed by Anders Carlsson.

The crash came from accessing a BundlePageFormClient that had been
destroyed. The client was intended to be cleared by a call to
WKBundlePageSetFormClient, but a null check there caused it not to be
cleared.

I fixed this and all of the other WK "set client" functions.

  • UIProcess/API/C/WKContext.cpp:

(WKContextSetInjectedBundleClient):
Return early if the caller passed a client with a version not equal to
0.
(WKContextSetHistoryClient):
Ditto.

  • UIProcess/API/C/WKPage.cpp:

(WKPageSetPageLoaderClient):
Ditto.
(WKPageSetPagePolicyClient):
Ditto.
(WKPageSetPageFormClient):
Ditto.
(WKPageSetPageUIClient):
Ditto.

  • WebProcess/InjectedBundle/API/c/WKBundle.cpp:

(WKBundleSetClient):
Ditto.

  • WebProcess/InjectedBundle/API/c/WKBundlePage.cpp:

(WKBundlePageSetEditorClient):
Ditto.
(WKBundlePageSetFormClient):
Ditto.
(WKBundlePageSetLoaderClient):
Ditto.
(WKBundlePageSetUIClient):
Ditto.

5:17 PM Changeset in webkit [66556] by commit-queue@webkit.org
  • 2 edits in trunk/WebCore

2010-08-31 Patrick Gansterer <​paroga@paroga.com>

Reviewed by Kenneth Rohde Christiansen.

[CMake] Unify GENERATE_DOM_NAMES code
​https://bugs.webkit.org/show_bug.cgi?id=44842

  • CMakeLists.txt: Added property svn:eol-style.
4:49 PM Changeset in webkit [66555] by Beth Dakin
  • 3 edits in trunk/WebCore

Variable name change. m_shouldFirePostLayoutTimer is now
m_hasPendingPostLayoutTasks

Rubber-stamped by Dan Bernstein..

  • page/FrameView.cpp:

(WebCore::FrameView::FrameView):
(WebCore::FrameView::~FrameView):
(WebCore::FrameView::reset):
(WebCore::FrameView::layout):
(WebCore::FrameView::performPostLayoutTasks):

  • page/FrameView.h:
4:48 PM Changeset in webkit [66554] by ap@apple.com
  • 2 edits in trunk/WebKit2

Build fix.

  • Shared/WebPreferencesStore.h: Actually remove encode(), which has been moved to WebPreferencesStore.cpp.
4:31 PM Changeset in webkit [66553] by abarth@webkit.org
  • 3 edits in trunk/BugsSite

2010-08-31 Adam Barth <​abarth@webkit.org>

Reviewed by Eric Seidel.

[reviewtool] Make it easy to scroll through review comments
​https://bugs.webkit.org/show_bug.cgi?id=45002

This patch lets you scroll through review comments using "n" (for next)
and "p" (for previous). It also attributes comments to their authors.

  • PrettyPatch/PrettyPatch.rb:
  • code-review.js:
4:21 PM Changeset in webkit [66552] by Beth Dakin
  • 4 edits
    2 adds in trunk

Fix for ​https://bugs.webkit.org/show_bug.cgi?id=44828 FrameView
should make more calls to postLayoutTasks() using the timer
-and corresponding-
<rdar://problem/8064938>

Reviewed by Dave Hyatt.

WebCore:

This change only allows synchronous calls to postLayoutTasks() if
we are not already in a synchronous call to postLayoutTasks().
Furthermore, it de-couples the notion of "the post layout timer
should fire" from m_postLayoutTasksTimer.isActive(), instead using
a boolean to track the need to use the timer.

  • page/FrameView.cpp:

(WebCore::FrameView::FrameView):
(WebCore::FrameView::~FrameView):
(WebCore::FrameView::reset):
(WebCore::FrameView::layout):
(WebCore::FrameView::unscheduleRelayout):
(WebCore::FrameView::performPostLayoutTasks):

  • page/FrameView.h:

LayoutTests:

  • fast/events/change-overflow-on-overflow-change-expected.txt: Added.
  • fast/events/change-overflow-on-overflow-change.html: Added.
4:13 PM Changeset in webkit [66551] by ap@apple.com
  • 20 edits in trunk

Reviewed by Sam Weinig.

​https://bugs.webkit.org/show_bug.cgi?id=42195
<rdar://problem/8186761> WebKitTestRunner needs to support layoutTestController.setXSSAuditorEnabled

LayoutTests:

  • platform/mac-wk2/Skipped: Unskip XSS Auditor tests.

WebKitTools:

  • WebKitTestRunner/InjectedBundle/Bindings/LayoutTestController.idl:
  • WebKitTestRunner/InjectedBundle/LayoutTestController.cpp: (WTR::LayoutTestController::setXSSAuditorEnabled):
  • WebKitTestRunner/InjectedBundle/LayoutTestController.h: Added setXSSAuditorEnabled, calling through to an WKBundle private method. Calling this method makes WebProcess use a different value for this preference than UIProcess thinks it uses.
  • WebKitTestRunner/TestController.cpp: (WTR::TestController::resetStateToConsistentValues): Reset xssAuditorEnabled. Even though the value doesn't change, this calls through to WebProcess, which forgets its override.

WebKit2:

  • Shared/WebPreferencesStore.cpp: (WebKit::WebPreferencesStore::WebPreferencesStore): Initialize xssAuditorEnabled. (WebKit::WebPreferencesStore::decode): Moved from header, added xssAuditorEnabled. (WebKit::WebPreferencesStore::encode): Ditto. (WebKit::WebPreferencesStore::overrideXSSAuditorEnabledForTestRunner): Set an override, so that decoding a WebPreferencesStore in the process will give the override result. WebProcess doesn't hold to WebPreferencesStore, so there are no existing ones to patch. (WebKit::WebPreferencesStore::removeTestRunnerOverrides): Remove the override.
  • Shared/WebPreferencesStore.h: Added xssAuditorEnabled.
  • UIProcess/API/C/WKPreferences.cpp: (WKPreferencesSetXSSAuditorEnabled): (WKPreferencesGetXSSAuditorEnabled):
  • UIProcess/API/C/WKPreferences.h: Added an API method to change xssAuditorEnabled preference.
  • UIProcess/WebPageProxy.cpp: (WebKit::WebPageProxy::preferencesDidChange): Added a comment explaining how making this method smarter could affect WebKitTestRunner.
  • UIProcess/WebPreferences.cpp: (WebKit::WebPreferences::setXSSAuditorEnabled): (WebKit::WebPreferences::xssAuditorEnabled):
  • UIProcess/WebPreferences.h: Implemented xssAuditorEnabled preference accessors.
  • WebProcess/InjectedBundle/API/c/WKBundle.cpp: (WKBundleOverrideXSSAuditorEnabledForTestRunner):
  • WebProcess/InjectedBundle/API/c/WKBundlePrivate.h: Added a private methods to temporarily change xssAuditor setting from an injected bundle, not notifying UIProcess about the change.
  • WebProcess/InjectedBundle/InjectedBundle.cpp: (WebKit::InjectedBundle::overrideXSSAuditorEnabledForTestRunner):
  • WebProcess/InjectedBundle/InjectedBundle.h: Change the setting in existing pages, and override it for future ones.
  • WebProcess/WebPage/WebPage.cpp: (WebKit::WebPage::WebPage): Take xssAuditorEnabled from preferences store. (WebKit::WebPage::preferencesDidChange): If preferences change, forget our override. This only happens in WKRT when the next test begins.
4:12 PM Changeset in webkit [66550] by andersca@apple.com
  • 2 edits in trunk/WebCore

Fix 32-bit WebKit2 build.

Reviewed by Sam Weinig.

  • WebCore.exp.in:
3:53 PM Changeset in webkit [66549] by Nate Chapin
  • 2 edits in trunk/WebCore

2010-08-31 Nate Chapin <Nate Chapin>

Reviewed by Darin Fisher.

Regression in r63528: Custom http headers for loads of type
FrameLoadTypeReloadFromOrigin are not being correctly
propagated to subresource loads.

​https://bugs.webkit.org/show_bug.cgi?id=44987

No tests, since DRT doesn't seem to have a way to trigger a
FrameLoadTypeReloadFromOrigin load.

  • loader/FrameLoader.cpp: (WebCore::FrameLoader::addExtraFieldsToRequest):
3:48 PM Changeset in webkit [66548] by weinig@apple.com
  • 11 edits in trunk

WebKitTestRunner needs layoutTestController.addUserStyleSheet
​https://bugs.webkit.org/show_bug.cgi?id=42680

Reviewed by Gavin Barraclough.

WebKitTestRunner needs layoutTestController.addUserScript
​https://bugs.webkit.org/show_bug.cgi?id=42681

WebKit2:

Improve UserContent APIs to allow passing null in the same places
the WebKit1 API allowed. Add convenience function toWTFString to do
WKStringRef conversion with correct null string behavior.

  • UIProcess/API/C/WKAPICast.h:

(toWTFString):

  • WebProcess/InjectedBundle/API/c/WKBundle.cpp:

(WKBundleAddUserScript):
(WKBundleAddUserStyleSheet):
(WKBundleRemoveUserScript):
(WKBundleRemoveUserStyleSheet):

  • WebProcess/InjectedBundle/InjectedBundle.cpp:

(WebKit::toStringVector):

WebKitTools:

  • WebKitTestRunner/InjectedBundle/Bindings/LayoutTestController.idl:
  • WebKitTestRunner/InjectedBundle/InjectedBundle.cpp:

(WTR::InjectedBundle::beginTesting):

  • WebKitTestRunner/InjectedBundle/LayoutTestController.cpp:

(WTR::LayoutTestController::addUserScript):
(WTR::LayoutTestController::addUserStyleSheet):

  • WebKitTestRunner/InjectedBundle/LayoutTestController.h:

LayoutTests:

  • platform/mac-wk2/Skipped: Update skipped list.
3:25 PM Changeset in webkit [66547] by dpranke@chromium.org
  • 9 edits
    4 deletes in trunk/WebKitTools

2010-08-31 Dirk Pranke <​dpranke@chromium.org>

Unreviewed, rolling out r66542.
​http://trac.webkit.org/changeset/66542
​https://bugs.webkit.org/show_bug.cgi?id=44902

r66542 - the weird logging dependencies in Python stuck again ...

  • Scripts/webkitpy/layout_tests/data/failures/expected/exception.html: Removed.
  • Scripts/webkitpy/layout_tests/data/failures/expected/keyboard.html: Removed.
  • Scripts/webkitpy/layout_tests/data/passes/error-expected.txt: Removed.
  • Scripts/webkitpy/layout_tests/data/passes/error.html: Removed.
  • Scripts/webkitpy/layout_tests/data/platform/test/test_expectations.txt:
  • Scripts/webkitpy/layout_tests/layout_package/dump_render_tree_thread.py:
  • Scripts/webkitpy/layout_tests/layout_package/printing.py:
  • Scripts/webkitpy/layout_tests/layout_package/printing_unittest.py:
  • Scripts/webkitpy/layout_tests/port/base.py:
  • Scripts/webkitpy/layout_tests/port/test.py:
  • Scripts/webkitpy/layout_tests/run_webkit_tests.py:
  • Scripts/webkitpy/layout_tests/run_webkit_tests_unittest.py:
3:18 PM Changeset in webkit [66546] by dumi@chromium.org
  • 3 edits in trunk/WebKitTools

Implementing LayoutTestController::markerTextForListItem() in Chromium's DRT.

Reviewed by Tony Chang.

  • DumpRenderTree/chromium/LayoutTestController.cpp:

(LayoutTestController::LayoutTestController):
(LayoutTestController::markerTextForListItem):

  • DumpRenderTree/chromium/LayoutTestController.h:
3:13 PM Changeset in webkit [66545] by tony@chromium.org
  • 2 edits in trunk/LayoutTests

2010-08-31 Tony Chang <​tony@chromium.org>

Unreviewed, updating test_expectations.txt.

  • platform/chromium/test_expectations.txt:
3:10 PM Changeset in webkit [66544] by weinig@apple.com
  • 13 edits in trunk

Add ability to count text matches without marking
​https://bugs.webkit.org/show_bug.cgi?id=43996

Reviewed by Darin Adler.

WebCore:

Safari needs to be able to count text matches without triggering lots of repainting.
Rename markAllMatchesForText() to countMatchesForText() and add a markMatches parameter.

  • WebCore.exp.in:
  • page/Frame.cpp:

(WebCore::Frame::countMatchesForText):

  • page/Frame.h:
  • page/Page.cpp:

(WebCore::Page::markAllMatchesForText):

WebKit/efl:

  • ewk/ewk_frame.cpp:

(ewk_frame_text_matches_mark): Switched to call
countMatchesForText() instead of markAllMatchesForText().

WebKit/mac:

Safari needs to be able to count text matches without triggering lots of repainting.
Rename markAllMatchesForText: to countMatchesForText: and add a markMatches:
parameter. For backwards compatibility markAllMatchesForText: calls
countMatchesForText: and passes YES for markMatches:.

  • WebView/WebDocumentInternal.h:
  • WebView/WebHTMLView.mm:

(-[WebHTMLView markAllMatchesForText:caseSensitive:limit:]):
(-[WebHTMLView countMatchesForText:caseSensitive:limit:markMatches:]):

  • WebView/WebPDFView.mm:

(-[WebPDFView markAllMatchesForText:caseSensitive:limit:]):
(-[WebPDFView countMatchesForText:caseSensitive:limit:markMatches:]):

  • WebView/WebView.mm:

(-[WebView markAllMatchesForText:caseSensitive:highlight:limit:]):
(-[WebView countMatchesForText:caseSensitive:highlight:limit:markMatches:]):

  • WebView/WebViewPrivate.h:
3:09 PM Changeset in webkit [66543] by sfalken@apple.com
  • 2 edits in trunk/WebKit2

Remove library directive from def file to fix Debug_All build.
Rubber stamped by Sam Weinig.

  • win/WebKit2.def:
2:53 PM Changeset in webkit [66542] by dpranke@chromium.org
  • 9 edits
    4 adds in trunk/WebKitTools

2010-08-31 Dirk Pranke <​dpranke@chromium.org>

Reviewed by Ojan Vafai.

new-run-webkit-tests: add more unit tests

Add more unit tests for new-run-webkit-tests; we now cover all but
the most obscure code paths in the generic code. We still need to
add coverage for the http server and web socket paths, and add better
coverage of the platform-specific logic. Note that the rebaselining
tool is still not well tested.

Also clean up some of the configuration logic for the printing
module and the way it interacts with the Python logging module; that
is a crufty interface, to be certain.

​https://bugs.webkit.org/show_bug.cgi?id=44902

  • Scripts/webkitpy/layout_tests/data/failures/expected/exception.html: Added.
  • Scripts/webkitpy/layout_tests/data/failures/expected/keyboard.html: Added.
  • Scripts/webkitpy/layout_tests/data/passes/error-expected.txt: Added.
  • Scripts/webkitpy/layout_tests/data/passes/error.html: Added.
  • Scripts/webkitpy/layout_tests/data/platform/test/test_expectations.txt:
  • Scripts/webkitpy/layout_tests/layout_package/dump_render_tree_thread.py:
  • Scripts/webkitpy/layout_tests/layout_package/printing.py:
  • Scripts/webkitpy/layout_tests/layout_package/printing_unittest.py:
  • Scripts/webkitpy/layout_tests/port/base.py:
  • Scripts/webkitpy/layout_tests/port/test.py:
  • Scripts/webkitpy/layout_tests/run_webkit_tests.py:
  • Scripts/webkitpy/layout_tests/run_webkit_tests_unittest.py:
2:51 PM Changeset in webkit [66541] by eric.carlson@apple.com
  • 5 edits in trunk

2010-08-31 Eric Carlson <​eric.carlson@apple.com>

Reviewed by Darin Adler.

canPlayType("application/octet-stream") must return ""
​https://bugs.webkit.org/show_bug.cgi?id=44985

  • platform/graphics/MediaPlayer.cpp: (WebCore::MediaPlayer::supportsType): Always return IsNotSupported for application/octet-stream

2010-08-31 Eric Carlson <​eric.carlson@apple.com>

Reviewed by Darin Adler.

canPlayType("application/octet-stream") must return ""
​https://bugs.webkit.org/show_bug.cgi?id=44985

  • media/media-can-play-octet-stream-expected.txt:
  • media/media-can-play-octet-stream.html:
2:47 PM Changeset in webkit [66540] by Martin Robinson
  • 5 edits in trunk/WebCore

2010-08-31 Martin Robinson <​mrobinson@igalia.com>

Reviewed by Gustavo Noronha Silva.

[GTK] ScrollbarThemeGtk should respond to theme changes
​https://bugs.webkit.org/show_bug.cgi?id=44782

Scrollbar is currently written with the assumption that scrollbars never change
thickness. In GTK+ changing the current theme can alter scrollbar thickness. This
change manually resizes interior frame scrollbar widgets when the GTK+ theme changes.
Since we are now tracking style changes, we may also cache theme properties to avoid
having to call moz_gtk_get_scrollbar_metrics repeatedly during rendering.

  • platform/gtk/RenderThemeGtk.cpp: (WebCore::RenderThemeGtk::gtkScrollbar): Added accessor for the Mozilla theming code's scrollbar widget. We access it here, because RenderThemGtk takes care of initializing the Mozilla code.
  • platform/gtk/RenderThemeGtk.h: Added accessor declaration.
  • platform/gtk/ScrollbarThemeGtk.cpp: (WebCore::ScrollbarThemeGtk::ScrollbarThemeGtk): Update the style property cache during construction. (WebCore::ScrollbarThemeGtk::registerScrollbar): Track all interior frame scrollbars. (WebCore::ScrollbarThemeGtk::unregisterScrollbar): Ditto. (WebCore::ScrollbarThemeGtk::updateThemeProperties): Added this method which updates the style cache. (WebCore::gtkStyleSetCallback): Added this signal handler for when the style changes. (WebCore::ScrollbarThemeGtk::backButtonRect): Use the style cache instead of calling moz_gtk_get_scrollbar_metrics. (WebCore::ScrollbarThemeGtk::forwardButtonRect): Ditto. (WebCore::ScrollbarThemeGtk::trackRect): Ditto. (WebCore::ScrollbarThemeGtk::paintTrackBackground): Ditto. (WebCore::ScrollbarThemeGtk::thumbRect): Ditto. (WebCore::ScrollbarThemeGtk::paint): Ditto. (WebCore::ScrollbarThemeGtk::scrollbarThickness): Ditto. (WebCore::ScrollbarThemeGtk::buttonSize): Ditto. (WebCore::ScrollbarThemeGtk::minimumThumbLength): Ditto.
  • platform/gtk/ScrollbarThemeGtk.h: Added new method declarations.
2:39 PM Changeset in webkit [66539] by Darin Adler
  • 5 edits in trunk

2010-08-31 Darin Adler <Darin Adler>

Reviewed by Anders Carlsson.

Web archives are created with a size of 0 due to PassRefPtr mistake
​https://bugs.webkit.org/show_bug.cgi?id=44984
rdar://problem/8042283

  • loader/archive/ArchiveResource.cpp: (WebCore::ArchiveResource::ArchiveResource): Merge the three constructors into one. (WebCore::ArchiveResource::create): Merge the two create functions into one, and fix the logic so there is no use of a PassRefPtr after it has been passed.
  • loader/archive/ArchiveResource.h: Use default arguments instead of overloading to cut down from three create functions to two. Remove two of the constructors.

2010-08-31 Darin Adler <Darin Adler>

Reviewed by Anders Carlsson.

Web archives are created with a size of 0 due to PassRefPtr mistake
​https://bugs.webkit.org/show_bug.cgi?id=44984
rdar://problem/8042283

  • webarchive/test-link-rel-icon-expected.webarchive: Expect a correct size, not 0.
2:34 PM Changeset in webkit [66538] by Darin Adler
  • 3 edits in trunk/JavaScriptCore
  • wtf/FastMalloc.cpp:

(WTF::TCMalloc_PageHeap::scavenge): Replaced somewhat-quirky code that
mixed types with code that uses size_t.

Reviewed by Anders Carlsson.

  • wtf/TCPageMap.h: Removed names of unused arguments to avoid warning.
2:33 PM Changeset in webkit [66537] by jianli@chromium.org
  • 6 edits
    3 adds in trunk

Expose WorkerContext.createBlobURL and WorkerContext.revokeBlobURL.
​https://bugs.webkit.org/show_bug.cgi?id=44972

Reviewed by David Levin.

WebCore:

Test: fast/files/workers/worker-apply-blob-url-to-xhr.html

  • page/SecurityOrigin.cpp:

(WebCore::SecurityOrigin::canRequest):

  • workers/WorkerContext.cpp:

(WebCore::WorkerContext::createBlobURL):
(WebCore::WorkerContext::revokeBlobURL):

  • workers/WorkerContext.h:
  • workers/WorkerContext.idl:

LayoutTests:

  • fast/files/workers/resources/worker-apply-blob-url-to-xhr.js: Added.
  • fast/files/workers/worker-apply-blob-url-to-xhr-expected.txt: Added.
  • fast/files/workers/worker-apply-blob-url-to-xhr.html: Added.
2:33 PM Changeset in webkit [66536] by Darin Adler
  • 3 edits in trunk/WebCore
  • editing/TextIterator.cpp:

(WebCore::TextIterator::handleTextBox):
Added a cast so we can mix int and unsigned in a single ?: expression.
This warns with clang, but strangely not with gcc.

Reviewed by Anders Carlsson.

  • html/parser/HTMLTreeBuilder.cpp: Removed unused functions.

The clang compiler notices and complains about these. If we need them
later we should use Subversion to bring them back.

2:31 PM Changeset in webkit [66535] by Darin Adler
  • 2 edits in trunk/WebKit/mac
  • WebInspector/WebInspectorFrontend.mm:

(-[WebInspectorFrontend initWithFrontendClient:]): Remove a stray semicolon.

Reviewed by Anders Carlsson.

2:29 PM Changeset in webkit [66534] by Darin Adler
  • 2 edits in trunk/WebKitTools
  • Scripts/check-for-global-initializers: Add a file that ends up having global initializers

in a debug build when built with certain compilers.

Reviewed by Anders Carlsson.

2:27 PM Changeset in webkit [66533] by kbr@google.com
  • 3 edits in trunk/WebCore

2010-08-31 Kenneth Russell <​kbr@google.com>

Reviewed by Simon Fraser.

Add cross product and arithmetic operations to FloatPoint3D
​https://bugs.webkit.org/show_bug.cgi?id=44970

Added set, move, and scale operations similar to FloatPoint's,
addition and subtraction operators, length, lengthSquared and
cross product. These changes have been tested with new code to be
added later.

  • platform/graphics/FloatPoint3D.cpp: (WebCore::FloatPoint3D::normalize): (WebCore::FloatPoint3D::length):
  • platform/graphics/FloatPoint3D.h: (WebCore::FloatPoint3D::set): (WebCore::FloatPoint3D::move): (WebCore::FloatPoint3D::scale): (WebCore::FloatPoint3D::dot): (WebCore::FloatPoint3D::cross): (WebCore::FloatPoint3D::lengthSquared): (WebCore::operator +=): (WebCore::operator -=): (WebCore::operator+): (WebCore::operator-): (WebCore::operator*):
2:25 PM Changeset in webkit [66532] by abarth@webkit.org
  • 3 edits in trunk/BugsSite

2010-08-31 Adam Barth <​abarth@webkit.org>

Reviewed by Eric Seidel.

[reviewtool] Show previous comments inline in diff
​https://bugs.webkit.org/show_bug.cgi?id=44977

This patch adds basic support for showing previous comments inline in
the diff. We crawl the bugs.webkit.org comments about this attachment
and extract comments related to specific lines. We then show the
comments inline in the diff.

This part of the tool needs a bunch of polish, but this at least is a
starting point for further work.

  • PrettyPatch/PrettyPatch.rb:
  • code-review.js:
2:23 PM Changeset in webkit [66531] by Martin Robinson
  • 22 edits
    1 add in trunk

2010-08-31 Martin Robinson <​mrobinson@igalia.com>

Reviewed by Gustavo Noronha Silva.

[GTK] Isolate all GTK+ typedefs into one file
​https://bugs.webkit.org/show_bug.cgi?id=44900

  • GNUmakefile.am: Add GtkTypedefs.h to the source lists.
  • wtf/Platform.h: #include GtkTypedefs.h for the GTK+ build.
  • wtf/ThreadingPrimitives.h: Remove GTK+ typedefs.
  • wtf/gobject/GOwnPtr.h: Ditto.
  • wtf/gobject/GRefPtr.h: Ditto.
  • wtf/gtk/GtkTypedefs.h: Added.

2010-08-31 Martin Robinson <​mrobinson@igalia.com>

Reviewed by Gustavo Noronha Silva.

[GTK] Isolate all GTK+ typedefs into one file
​https://bugs.webkit.org/show_bug.cgi?id=44900

  • platform/graphics/gtk/CairoUtilities.h: Remove GTK+ typedefs.
  • platform/gtk/ClipboardGtk.h: Ditto.
  • platform/gtk/DataObjectGtk.h: Ditto.
  • platform/gtk/GOwnPtrGtk.h: Ditto.
  • platform/gtk/GRefPtrGtk.h: Ditto.
  • platform/gtk/PasteboardHelper.h: Ditto.
  • platform/gtk/PopupMenuGtk.h: Ditto.
  • platform/gtk/RenderThemeGtk.h: Ditto.
  • platform/gtk/ScrollbarGtk.h: Ditto.
  • platform/gtk/gtk2drawing.c: Remove unused Assertions.h.

2010-08-31 Martin Robinson <​mrobinson@igalia.com>

Reviewed by Gustavo Noronha Silva.

[GTK] Isolate all GTK+ typedefs into one file
​https://bugs.webkit.org/show_bug.cgi?id=44900

  • WebCoreSupport/EditorClientGtk.h: Remove GTK+ typedefs.
  • WebCoreSupport/FullscreenVideoController.h: Ditto.

2010-08-31 Martin Robinson <​mrobinson@igalia.com>

Reviewed by Gustavo Noronha Silva.

[GTK] Isolate all GTK+ typedefs into one file
​https://bugs.webkit.org/show_bug.cgi?id=44900

  • DumpRenderTree/gtk/EventSender.h: Remove GTK+ typedefs.
2:15 PM Changeset in webkit [66530] by tony@chromium.org
  • 2 edits in trunk/WebCore

2010-08-31 Tony Chang <​tony@chromium.org>

Unreviewed, fix chromium compile.

  • bindings/v8/custom/V8DeviceOrientationEventCustom.cpp:
2:02 PM Changeset in webkit [66529] by crogers@google.com
  • 1 edit
    2 adds in trunk/WebCore

2010-08-31 Chris Rogers <​crogers@google.com>

Reviewed by Kenneth Russell.

audio engine: add FFTConvolver class
​https://bugs.webkit.org/show_bug.cgi?id=34907

No new tests since audio API is not yet implemented.

  • platform/audio: Added.
  • platform/audio/FFTConvolver.cpp: Added. (WebCore::FFTConvolver::FFTConvolver): (WebCore::FFTConvolver::process): (WebCore::FFTConvolver::reset):
  • platform/audio/FFTConvolver.h: Added. (WebCore::FFTConvolver::fftSize):
1:57 PM Changeset in webkit [66528] by jianli@chromium.org
  • 2 edits in trunk/WebCore

Fix ThreadableBlobRegistry methods not to rely on WorkerContext.
​https://bugs.webkit.org/show_bug.cgi?id=44971

Reviewed by Darin Fisher and David Levin.

We could hit an ASSERT when we're performing some blob related cleanup
in ScriptExecutionContext destructor when WorkerContext dies. The fix
is to use isMainThread and callOnMainThread.

  • fileapi/ThreadableBlobRegistry.cpp:

(WebCore::BlobRegistryContext::BlobRegistryContext):
(WebCore::registerBlobURLTask):
(WebCore::ThreadableBlobRegistry::registerBlobURL):
(WebCore::registerBlobURLFromTask):
(WebCore::unregisterBlobURLTask):
(WebCore::ThreadableBlobRegistry::unregisterBlobURL):

1:50 PM Changeset in webkit [66527] by Martin Robinson
  • 4 edits in trunk

2010-08-31 Martin Robinson <​mrobinson@igalia.com>

Reviewed by Gustavo Noronha Silva.

[GTK] Fix 'make dist' in preparation of the 1.3.3 release
​https://bugs.webkit.org/show_bug.cgi?id=44978

  • GNUmakefile.am: Adding missing headers to the sources list.

2010-08-31 Martin Robinson <​mrobinson@igalia.com>

Reviewed by Gustavo Noronha Silva.

[GTK] Fix 'make dist' in preparation of the 1.3.3 release
​https://bugs.webkit.org/show_bug.cgi?id=44978

  • GNUmakefile.am: Add missing and moved headers to the sources list.
1:32 PM Changeset in webkit [66526] by commit-queue@webkit.org
  • 4 edits in trunk/WebCore

2010-08-31 Ryuan Choi <​ryuan.choi@samsung.com>

Reviewed by Antonio Gomes.

[EFL] crashes while dragging any contents
​https://bugs.webkit.org/show_bug.cgi?id=44940

Fix crashes caused by returning newly assigned value instead of adoptRef
value.

  • page/efl/EventHandlerEfl.cpp: (WebCore::EventHandler::createDraggingClipboard):
  • platform/efl/ClipboardEfl.cpp: (WebCore::Editor::newGeneralClipboard):
  • platform/efl/ClipboardEfl.h: (WebCore::ClipboardEfl::create):
12:53 PM Changeset in webkit [66525] by kov@webkit.org
  • 25 edits in trunk/WebKit/gtk/po

2010-08-31 Sheriff Bot <​webkit.review.bot@gmail.com>

Unreviewed, rolling out r66519.
​http://trac.webkit.org/changeset/66519
​https://bugs.webkit.org/show_bug.cgi?id=44973

Broke GTK+. (Requested by kov on #webkit).

  • cs.po:
  • de.po:
  • en_GB.po:
  • es.po:
  • et.po:
  • gl.po:
  • gu.po:
  • he.po:
  • it.po:
  • lt.po:
  • lv.po:
  • nb.po:
  • nl.po:
  • pa.po:
  • pt.po:
  • pt_BR.po:
  • ru.po:
  • sl.po:
  • sr.po:
  • ​sr@latin.po:
  • sv.po:
  • uk.po:
  • vi.po:
  • zh_CN.po:
12:45 PM Changeset in webkit [66524] by barraclough@apple.com
  • 3 edits in trunk/JavaScriptCore

Support emit_op_mod() for MIPS
​https://bugs.webkit.org/show_bug.cgi?id=42855

Patch by Chao-ying Fu <​fu@mips.com> on 2010-08-31
Reviewed by Oliver Hunt.

This patch uses MIPS div instructions for op_mod to improve performance.

  • assembler/MIPSAssembler.h:

(JSC::MIPSAssembler::div):

  • jit/JITArithmetic.cpp:

(JSC::JIT::emit_op_mod):
(JSC::JIT::emitSlow_op_mod):

12:40 PM Changeset in webkit [66523] by loislo@chromium.org
  • 4 edits
    2 adds in trunk

2010-08-31 Ilya Tikhonovsky <​loislo@chromium.org>

Reviewed by Yury Semikhatsky.

WebInspector: The parser of Inspector protocol messages should be covered by a test.

The inspector protocol is based on messages in JSON format.
Each message should be a JSON object with a number of properties.
InspectorBackendDispatcher is parsing these messages and should be
able to detect wrong formatted messages.
It has these functionality but no tests for it.

​https://bugs.webkit.org/show_bug.cgi?id=44947

  • inspector/CodeGeneratorInspector.pm:

2010-08-31 Ilya Tikhonovsky <​loislo@chromium.org>

Reviewed by Yury Semikhatsky.

WebInspector: The parser of Inspector protocol messages should be covered by a test.

The inspector protocol is based on messages in JSON format.
Each message should be a JSON object with a number of properties.
InspectorBackendDispatcher is parsing these messages and should be
able to detect wrong formatted messages.
It has these functionality but no tests for it.

​https://bugs.webkit.org/show_bug.cgi?id=44947

  • http/tests/inspector/inspector-test2.js: (dump): (dumpArray):
  • inspector/report-protocol-errors-expected.txt: Added.
  • inspector/report-protocol-errors.html: Added.
12:39 PM Changeset in webkit [66522] by kov@webkit.org
  • 5 edits in trunk

2010-08-31 Gustavo Noronha Silva <​gustavo.noronha@collabora.co.uk>

Reviewed by Martin Robinson.

Preparations for the 1.3.4 release. Bump webkit version to 543.7,
as well.

  • configure.ac:

WebKit/gtk

2010-08-31 Gustavo Noronha Silva <​gustavo.noronha@collabora.co.uk>

Reviewed by Martin Robinson.

Preparations for the 1.3.4 release.

  • NEWS:
  • docs/webkitgtk-docs.sgml:
12:35 PM Changeset in webkit [66521] by antonm@chromium.org
  • 7 edits in trunk/WebCore

2010-08-31 Anton Muhin <​antonm@chromium.org>

Reviewed by Adam Barth.

[v8] More correct and faster error handling when converting v8 objects to various WebCore strings
​https://bugs.webkit.org/show_bug.cgi?id=44678

Split v8 object conversion into two phase: 1st, which can throw an exception, and
2nd, which must always succeed. That allows to report correctly the case when exception
happens.

  • bindings/scripts/CodeGeneratorV8.pm:
  • bindings/v8/V8Binding.cpp: (WebCore::int32ToWebCoreString): (WebCore::v8NonStringValueToWebCoreString):
  • bindings/v8/V8Binding.h: (WebCore::V8ParameterBase::operator String): (WebCore::V8ParameterBase::operator AtomicString): (WebCore::V8ParameterBase::V8ParameterBase): (WebCore::V8ParameterBase::prepareBase): (WebCore::V8ParameterBase::object): (WebCore::V8ParameterBase::setString): (WebCore::V8ParameterBase::toString): (WebCore::): (WebCore::::prepare):
  • bindings/v8/custom/V8BindingMacros.h:
  • bindings/v8/custom/V8DeviceMotionEventCustom.cpp: (WebCore::V8DeviceMotionEvent::initDeviceMotionEventCallback):
  • bindings/v8/cstom/V8DeviceOrientationEventCustom.cpp: (WebCore::V8DeviceMotionEvent::initDeviceOrientationEventCallback):
12:25 PM Changeset in webkit [66520] by kov@webkit.org
  • 2 edits in trunk/WebKit/gtk

2010-08-31 Gustavo Noronha Silva <​gustavo.noronha@collabora.co.uk>

Unreviewed. Minor documentation typo fix.

  • webkit/webkitwebview.cpp:
12:21 PM Changeset in webkit [66519] by kov@webkit.org
  • 25 edits in trunk/WebKit/gtk/po

2010-08-31 Gustavo Noronha Silva <Gustavo Noronha Silva>

Refreshed all po files with the latest potfile.

  • cs.po:
  • de.po:
  • en_GB.po:
  • es.po:
  • et.po:
  • gl.po:
  • gu.po:
  • he.po:
  • it.po:
  • lt.po:
  • lv.po:
  • nb.po:
  • nl.po:
  • pa.po:
  • pt.po:
  • pt_BR.po:
  • ru.po:
  • sl.po:
  • sr.po:
  • ​sr@latin.po:
  • sv.po:
  • uk.po:
  • vi.po:
  • zh_CN.po:
12:09 PM Changeset in webkit [66518] by eric@webkit.org
  • 4 edits in trunk/WebKitTools

2010-08-31 Eric Seidel <​eric@webkit.org>

Reviewed by Adam Barth.

Show more status on queue status pages
​https://bugs.webkit.org/show_bug.cgi?id=44938

I upped the number of status lines from 6 to 15 --
this has been bugging me for a while...

I also disabled showing the chromium-win EWS queue
since it's been down for almost 5 months now.

  • QueueStatusServer/handlers/dashboard.py:
  • QueueStatusServer/handlers/queuestatus.py:
  • QueueStatusServer/model/queues.py:
12:03 PM Changeset in webkit [66517] by tony@chromium.org
  • 2 edits
    6 adds in trunk/LayoutTests

2010-08-31 Tony Chang <​tony@chromium.org>

Unreviewed, updating chromium test_expectations.txt.

  • platform/chromium-linux/fast/multicol/paginate-block-replaced-expected.checksum: Added.
  • platform/chromium-linux/fast/multicol/paginate-block-replaced-expected.png: Added.
  • platform/chromium-linux/fast/multicol/paginate-block-replaced-expected.txt: Added.
  • platform/chromium-win/fast/multicol/paginate-block-replaced-expected.checksum: Added.
  • platform/chromium-win/fast/multicol/paginate-block-replaced-expected.png: Added.
  • platform/chromium-win/fast/multicol/paginate-block-replaced-expected.txt: Added.
  • platform/chromium/test_expectations.txt: Mark notifications-click-event.html as failing and rebase paginate-block-replaced on linux/win.
11:55 AM Changeset in webkit [66516] by loislo@chromium.org
  • 4 edits
    2 adds in trunk

2010-08-30 Ilya Tikhonovsky <​loislo@chromium.org>

Reviewed by Joseph Pecoraro.

WebInspector: it'd be better to introduce inspector API related tests.

As far as we have some kind of API for Inspector
it'd be better to have API related tests. This is the test
for API wrappers. These wrappers are tracking the types of arguments
of API functions.

​https://bugs.webkit.org/show_bug.cgi?id=44875

Test: inspector/protocol-error.html

  • inspector/CodeGeneratorInspector.pm:
  • inspector/front-end/inspector.js: (WebInspector_syncDispatch): (WebInspector.reportProtocolError):

2010-08-30 Ilya Tikhonovsky <​loislo@chromium.org>

Reviewed by Joseph Pecoraro.

WebInspector: it'd be better to introduce inspector API related tests.

As far as we have some kind of API for Inspector
it'd be better to have API related tests. This is the test
for API wrappers. These wrappers are tracking the types of arguments
of API functions.

​https://bugs.webkit.org/show_bug.cgi?id=44875

  • inspector/protocol-error-expected.txt: Added.
  • inspector/protocol-error.html: Added.
11:55 AM Changeset in webkit [66515] by kov@webkit.org
  • 2 edits in trunk/WebKit/gtk/po

2010-08-31 Jorge González <​aloriel@gmail.com>

Reviewed by Gustavo Noronha.

[GTK] Spanish translation for trunk
​https://bugs.webkit.org/show_bug.cgi?id=44395

  • es.po: Updated.
11:47 AM Changeset in webkit [66514] by Adam Roben
  • 3 edits in trunk/WebKit2

Handle WM_PRINTCLIENT in WebKit2 on Windows

Reviewed by Sam Weinig.

  • UIProcess/win/WebView.cpp:

(WebKit::WebView::wndProc): Call through to onPrintClientEvent when we
get a WM_PRINTCLIENT message.
(WebKit::WebView::onPrintClientEvent): Added. Paints the entire view
into the HDC provided by Windows.

  • UIProcess/win/WebView.h: Added onPrintClientEvent.
11:46 AM Changeset in webkit [66513] by commit-queue@webkit.org
  • 2 edits in trunk/WebCore

2010-08-31 Sheriff Bot <​webkit.review.bot@gmail.com>

Unreviewed, rolling out r66484.
​http://trac.webkit.org/changeset/66484
​https://bugs.webkit.org/show_bug.cgi?id=44966

Broke EFL build (Requested by acidx on #webkit).

  • CMakeLists.txt:
11:39 AM Changeset in webkit [66512] by commit-queue@webkit.org
  • 1 edit
    2 adds in trunk/WebKit/wince

2010-08-31 Patrick Gansterer <​paroga@paroga.com>

Reviewed by Kenneth Rohde Christiansen.

Add EditorClientWinCE
​https://bugs.webkit.org/show_bug.cgi?id=44822

  • WebCoreSupport/EditorClientWinCE.cpp: Added. (WebKit::EditorClient::EditorClient): (WebKit::EditorClient::~EditorClient): (WebKit::EditorClient::setInputMethodState): (WebKit::EditorClient::shouldDeleteRange): (WebKit::EditorClient::shouldShowDeleteInterface): (WebKit::EditorClient::isContinuousSpellCheckingEnabled): (WebKit::EditorClient::isGrammarCheckingEnabled): (WebKit::EditorClient::spellCheckerDocumentTag): (WebKit::EditorClient::shouldBeginEditing): (WebKit::EditorClient::shouldEndEditing): (WebKit::EditorClient::shouldInsertText): (WebKit::EditorClient::shouldChangeSelectedRange): (WebKit::EditorClient::shouldApplyStyle): (WebKit::EditorClient::shouldMoveRangeAfterDelete): (WebKit::EditorClient::didBeginEditing): (WebKit::EditorClient::respondToChangedContents): (WebKit::EditorClient::respondToChangedSelection): (WebKit::EditorClient::didEndEditing): (WebKit::EditorClient::didWriteSelectionToPasteboard): (WebKit::EditorClient::didSetSelectionTypesForPasteboard): (WebKit::EditorClient::isEditable): (WebKit::EditorClient::registerCommandForUndo): (WebKit::EditorClient::registerCommandForRedo): (WebKit::EditorClient::clearUndoRedoOperations): (WebKit::EditorClient::canUndo): (WebKit::EditorClient::canRedo): (WebKit::EditorClient::undo): (WebKit::EditorClient::redo): (WebKit::EditorClient::shouldInsertNode): (WebKit::EditorClient::pageDestroyed): (WebKit::EditorClient::smartInsertDeleteEnabled): (WebKit::EditorClient::isSelectTrailingWhitespaceEnabled): (WebKit::EditorClient::toggleContinuousSpellChecking): (WebKit::EditorClient::toggleGrammarChecking): (WebKit::EditorClient::handleKeyboardEvent): (WebKit::EditorClient::handleInputMethodKeydown): (WebKit::EditorClient::textFieldDidBeginEditing): (WebKit::EditorClient::textFieldDidEndEditing): (WebKit::EditorClient::textDidChangeInTextField): (WebKit::EditorClient::doTextFieldCommandFromEvent): (WebKit::EditorClient::textWillBeDeletedInTextField): (WebKit::EditorClient::textDidChangeInTextArea): (WebKit::EditorClient::ignoreWordInSpellDocument): (WebKit::EditorClient::learnWord): (WebKit::EditorClient::checkSpellingOfString): (WebKit::EditorClient::getAutoCorrectSuggestionForMisspelledWord): (WebKit::EditorClient::checkGrammarOfString): (WebKit::EditorClient::updateSpellingUIWithGrammarString): (WebKit::EditorClient::updateSpellingUIWithMisspelledWord): (WebKit::EditorClient::showSpellingUI): (WebKit::EditorClient::spellingUIIsShowing): (WebKit::EditorClient::getGuessesForWord): (WebKit::EditorClient::willSetInputMethodState):
  • WebCoreSupport/EditorClientWinCE.h: Added.
11:22 AM Changeset in webkit [66511] by ojan@chromium.org
  • 2 edits in trunk/WebCore

2010-08-31 Ojan Vafai <​ojan@chromium.org>

Reviewed by Tony Chang.

deduplicate code from Node::checkReplaceChild and Node::checkAddChild
​https://bugs.webkit.org/show_bug.cgi?id=44962

Remove duplicate code in preparation for fixing ​https://bugs.webkit.org/show_bug.cgi?id=19524
No new tests since there is no change in functionality.

  • dom/Node.cpp: (WebCore::isChildTypeAllowed): (WebCore::Node::canReplaceChild): (WebCore::checkAcceptChild): (WebCore::transferOwnerDocument): (WebCore::Node::checkReplaceChild): (WebCore::Node::checkAddChild):
11:19 AM Changeset in webkit [66510] by commit-queue@webkit.org
  • 1 edit
    2 adds in trunk/WebKit/wince

2010-08-31 Patrick Gansterer <​paroga@paroga.com>

Reviewed by Kenneth Rohde Christiansen.

Add DragClientWinCE
​https://bugs.webkit.org/show_bug.cgi?id=44821

  • WebCoreSupport/DragClientWinCE.cpp: Added. (WebKit::DragClient::willPerformDragDestinationAction): (WebKit::DragClient::willPerformDragSourceAction): (WebKit::DragClient::actionMaskForDrag): (WebKit::DragClient::dragSourceActionMaskForPoint): (WebKit::DragClient::startDrag): (WebKit::DragClient::createDragImageForLink): (WebKit::DragClient::dragControllerDestroyed):
  • WebCoreSupport/DragClientWinCE.h: Added.
11:01 AM Changeset in webkit [66509] by zmo@google.com
  • 4 edits in trunk/WebCore

2010-08-30 Zhenyao Mo <​zmo@google.com>

Reviewed by Kenneth Russell.

[Chromium] bindTexture(TEXTURE_CUBE_MAP) shouldn't generate errors with valid cubemap texture
​https://bugs.webkit.org/show_bug.cgi?id=44922

  • html/canvas/WebGLRenderingContext.cpp: Remove TEXTURE_WRAP_R code. (WebCore::WebGLRenderingContext::bindTexture):
  • html/canvas/WebGLTexture.cpp: Remove the code for tracking TEXTURE_WRAP_R. (WebCore::WebGLTexture::WebGLTexture):
  • html/canvas/WebGLTexture.h: Ditto.
10:54 AM Changeset in webkit [66508] by Adam Roben
  • 2 edits in trunk/WebKit2

Don't send messages to the injected bundle if the web process has exited

Fixes <​http://webkit.org/b/43046> <rdar://problem/8239455> Crash in
WKContextPostMessageToInjectedBundle if the web process has exited

Reviewed by Sam Weinig.

  • UIProcess/WebContext.cpp:

(WebKit::WebContext::postMessageToInjectedBundle): Only send messages
when we have a valid web process.

10:53 AM Changeset in webkit [66507] by kov@webkit.org
  • 1 edit
    1 add in trunk/WebKit/gtk/po

2010-08-31 Kjartan Maraas <​kmaraas@gnome.org>

Reviewed by Gustavo Noronha Silva.

WebKitGTK+ translation needed
​https://bugzilla.gnome.org/show_bug.cgi?id=610099

Initial Norwegian (bokmal) translation.

  • nb.po: Added.
10:53 AM Changeset in webkit [66506] by Adam Roben
  • 3 edits in trunk/WebKit2

Use the Windows thread pool instead of a dedicated thread for WorkQueue on Windows

WorkQueue now uses ::RegisterWaitForSingleObject to find out when work
items need to be executed. This causes Windows to wait on the objects
on a thread pool wait thread, and then to spawn a thread pool worker
thread when an object is signaled. This is more efficient than using
our own dedicated thread to perform the wait and the work because
multiple WorkQueues (and even other parts of WebKit or other modules)
can all share the same wait thread and worker thread pool.

Each time WorkQueue::m_performWorkEvent or any handle in
WorkQueue::m_handles is signaled, a worker thread will be spawned.
To maintain WorkQueue's serial nature, only one worker thread is
allowed to perform work items at a time. (The worker thread that is
actually performing work items is called the queue's "work thread".)
To accomplish this, worker threads must register as the queue's work
thread before performing work items.
WorkQueue::m_isWorkThreadRegistered is used as an atomic guard to make
sure that only one worker thread is registered at a time.

Fixes <​http://webkit.org/b/43150> <rdar://problem/8247280>.

Reviewed by Anders Carlsson.

  • Platform/WorkQueue.h:
    • Added the WorkItemWin class, which is used to wrap WorkItems for WorkQueue's Windows implementation
    • Changed m_workItemQueue and m_handles to hold RefPtr<WorkItemWin>s
    • Replaced "work queue thread"-related members with new members that handle our thread pool code
  • Platform/win/WorkQueueWin.cpp:

(WorkQueue::WorkItemWin::WorkItemWin):
(WorkQueue::WorkItemWin::create):
Added simple constructor/creator.

(WorkQueue::handleCallback): Added. This function is called whenever a
handle in WorkQueue::m_handles is signaled. We add the WorkItemWin
that corresponds to the handle (passed via the context parameter) to
the work item queue, then try to register as the work thread and
perform any queued work. If another thread is already registered as
the work thread, we just exit and let that thread handle the work we
queued.
(WorkQueue::registerHandle): Changed to wrap the WorkItem in a
WorkItemWin, and to use ::RegisterWaitForSingleObject to wait on the
handle.
(WorkQueue::eventCallback): Added. This function is called whenever
m_performWorkEvent is signaled. We try to register as the work thread
and perfom any queued work. If another thread is already registered as
the work thread, we just exit and let that thread handle the work.
(WorkQueue::performWorkOnRegisteredWorkThread): Added. Performs any
queued work in a loop until either the queue becomes invalid or no
work is left to perform. Unregisters as the work thread before exiting
so that other threads can perform work in the future.
(WorkQueue::platformInitialize): Added initialization of
m_isWorkThreadRegistered. Replaced code to spawn the old work queue
thread with a call to ::RegisterWaitForSingleObject so that a worker
thread from the thread pool will be spawned when m_performWorkEvent is
signaled.
(WorkQueue::tryRegisterAsWorkThread): Added. Attempts an atomic
compare-and-swap to change m_isWorkThreadRegistered from 0 to 1. If
sucessful, we return true to indicate that this thread is now
registered as the work thread.
(WorkQueue::unregisterAsWorkThread): Added. Uses an atomic
compare-and-swap to change m_isWorkThreadRegistered back from 1 to 0.
(WorkQueue::scheduleWork): Changed to wrap the WorkItem in a
WorkItemWin. Also added an optimization to avoid signaling
m_performWorkEvent when a work thread is already performing work, as
it will pick up the item we just queued without us having to do
anything.

10:52 AM Changeset in webkit [66505] by jianli@chromium.org
  • 2 edits in trunk/WebCore

Rename addTwoCStrings to concatenateTwoCStrings.

Rubber-stamped by David Levin.

  • fileapi/BlobBuilder.cpp:

(WebCore::concatenateTwoCStrings):
(WebCore::BlobBuilder::append):

10:46 AM Changeset in webkit [66504] by jianli@chromium.org
  • 1 edit in trunk/WebCore/ChangeLog

Update WebCore/ChangeLog to fix the missing bug title.

10:37 AM Changeset in webkit [66503] by commit-queue@webkit.org
  • 1 edit
    2 adds in trunk/WebKit/wince

2010-08-31 Patrick Gansterer <​paroga@paroga.com>

Reviewed by Kenneth Rohde Christiansen.

Add ContextMenuClientWinCE
​https://bugs.webkit.org/show_bug.cgi?id=44820

  • WebCoreSupport/ContextMenuClientWinCE.cpp: Added. (WebKit::ContextMenuClient::ContextMenuClient): (WebKit::ContextMenuClient::contextMenuDestroyed): (WebKit::ContextMenuClient::getCustomMenuFromDefaultItems): (WebKit::ContextMenuClient::contextMenuItemSelected): (WebKit::ContextMenuClient::downloadURL): (WebKit::ContextMenuClient::copyImageToClipboard): (WebKit::ContextMenuClient::searchWithGoogle): (WebKit::ContextMenuClient::lookUpInDictionary): (WebKit::ContextMenuClient::speak): (WebKit::ContextMenuClient::stopSpeaking): (WebKit::ContextMenuClient::isSpeaking):
  • WebCoreSupport/ContextMenuClientWinCE.h: Added.
10:25 AM Changeset in webkit [66502] by evan@chromium.org
  • 4 edits in trunk/LayoutTests

2010-08-31 Evan Martin <​evan@chromium.org>

Reviewed by Tony Chang.

[chromium] rebaseline linux fast/backgrounds/001
​https://bugs.webkit.org/show_bug.cgi?id=44899

Only difference is font antialiasing.

  • platform/chromium-linux/fast/backgrounds/001-expected.checksum:
  • platform/chromium-linux/fast/backgrounds/001-expected.png:
  • platform/chromium/test_expectations.txt:
10:24 AM Changeset in webkit [66501] by abecsi@webkit.org
  • 2 edits in trunk/WebKitTools

2010-08-31 Zoltan Horvath <​zoltan@webkit.org>

Reviewed by Antonio Gomes.

[Qt] Read command line arguments and open them as URLs in new windows
​https://bugs.webkit.org/show_bug.cgi?id=44944

Extend MiniBrowser to accept command line arguments and open them as URLs in new browser windows.

  • MiniBrowser/qt/main.cpp: (main):
10:23 AM Changeset in webkit [66500] by evan@chromium.org
  • 2 edits
    3 adds in trunk/LayoutTests

2010-08-31 Evan Martin <​evan@chromium.org>

Reviewed by Tony Chang.

[chromium] new linux baseline for fast/text/capitalize-boundaries
​https://bugs.webkit.org/show_bug.cgi?id=44898

This differs from the Windows baseline in antialiasing and that the Windows
result has a missing-glyph box.

  • platform/chromium-linux/fast/text/capitalize-boundaries-expected.checksum: Added.
  • platform/chromium-linux/fast/text/capitalize-boundaries-expected.png: Added.
  • platform/chromium-linux/fast/text/capitalize-boundaries-expected.txt: Added.
  • platform/chromium/test_expectations.txt:
10:17 AM Changeset in webkit [66499] by jianli@chromium.org
  • 2 edits in trunk/WebCore

Improve BlobBuilder to combine adjacent strings.
​https://bugs.webkit.org/post_bug.cgi

Reviewed by Darin Fisher.

  • fileapi/BlobBuilder.cpp:

(WebCore::addTwoCStrings):
(WebCore::BlobBuilder::append):

10:08 AM Changeset in webkit [66498] by Darin Adler
  • 119 edits in trunk/WebCore

2010-08-31 Darin Adler <Darin Adler>

Reviewed by Simon Fraser.

Make DOM classes start with a reference count of 1, like all other RefCounted
​https://bugs.webkit.org/show_bug.cgi?id=28068

  • dom/Document.cpp: (WebCore::Document::accessSVGExtensions): Use adoptPtr instead of set.
  • dom/Document.h: Removed initialRefCount in Node constructor. It's always 1 now.
  • dom/Node.h: Removed CreateWithZeroRefCountFlag, CreateStyledElementZeroRefCount, CreateSVGElementZeroRefCount, and initialRefCount.
  • platform/TreeShared.h: Removed initialRefCount argument. It's always 1 now.
  • rendering/RenderSVGShadowTreeRootContainer.cpp: (WebCore::RenderSVGShadowTreeRootContainer::updateFromElement): Use create instead of new.
  • rendering/SVGResources.h: Inherit from Noncopyable.
  • rendering/SVGResourcesCache.cpp: (WebCore::SVGResourcesCache::~SVGResourcesCache): Removed unneeded special case code.
  • rendering/SVGShadowTreeElements.cpp: (WebCore::SVGShadowTreeContainerElement::create): Added. (WebCore::SVGShadowTreeRootElement::SVGShadowTreeRootElement): Made argument type more specific. (WebCore::SVGShadowTreeRootElement::create): Added.
  • rendering/SVGShadowTreeElements.h: Added create functions, and made the argument type more specific.
  • svg/SVGDocumentExtensions.cpp: (WebCore::SVGDocumentExtensions::SVGDocumentExtensions): Use adoptPtr.
  • svg/graphics/SVGImage.cpp: (WebCore::SVGImage::dataChanged): Use adoptPtr instead of set.
  • svg/SVGElement.cpp: (WebCore::SVGElement::SVGElement): Use CreateSVGElement instead of CreateSVGElementZeroRefCount. (WebCore::SVGElement::create): Use adoptRef.
  • wml/WMLElement.cpp: (WebCore::WMLElement::WMLElement): Use CreateStyledElement instead of CreateStyledElementZeroRefCount. (WebCore::WMLElement::create): Use adoptRef.
  • svg/SVGAElement.cpp: (WebCore::SVGAElement::create):
  • svg/SVGAltGlyphElement.cpp: (WebCore::SVGAltGlyphElement::create):
  • svg/SVGAnimateColorElement.cpp: (WebCore::SVGAnimateColorElement::create):
  • svg/SVGAnimateElement.cpp: (WebCore::SVGAnimateElement::create):
  • svg/SVGAnimateMotionElement.cpp: (WebCore::SVGAnimateMotionElement::create):
  • svg/SVGAnimateTransformElement.cpp: (WebCore::SVGAnimateTransformElement::create):
  • svg/SVGCircleElement.cpp: (WebCore::SVGCircleElement::create):
  • svg/SVGClipPathElement.cpp: (WebCore::SVGClipPathElement::create):
  • svg/SVGCursorElement.cpp: (WebCore::SVGCursorElement::create):
  • svg/SVGDefsElement.cpp: (WebCore::SVGDefsElement::create):
  • svg/SVGDescElement.cpp: (WebCore::SVGDescElement::create):
  • svg/SVGEllipseElement.cpp: (WebCore::SVGEllipseElement::create):
  • svg/SVGFEBlendElement.cpp: (WebCore::SVGFEBlendElement::create):
  • svg/SVGFEColorMatrixElement.cpp: (WebCore::SVGFEColorMatrixElement::create):
  • svg/SVGFEComponentTransferElement.cpp: (WebCore::SVGFEComponentTransferElement::create):
  • svg/SVGFECompositeElement.cpp: (WebCore::SVGFECompositeElement::create):
  • svg/SVGFEConvolveMatrixElement.cpp: (WebCore::SVGFEConvolveMatrixElement::create):
  • svg/SVGFEDiffuseLightingElement.cpp: (WebCore::SVGFEDiffuseLightingElement::create):
  • svg/SVGFEDisplacementMapElement.cpp: (WebCore::SVGFEDisplacementMapElement::create):
  • svg/SVGFEDistantLightElement.cpp: (WebCore::SVGFEDistantLightElement::create):
  • svg/SVGFEFloodElement.cpp: (WebCore::SVGFEFloodElement::create):
  • svg/SVGFEFuncAElement.cpp: (WebCore::SVGFEFuncAElement::create):
  • svg/SVGFEFuncBElement.cpp: (WebCore::SVGFEFuncBElement::create):
  • svg/SVGFEFuncGElement.cpp: (WebCore::SVGFEFuncGElement::create):
  • svg/SVGFEFuncRElement.cpp: (WebCore::SVGFEFuncRElement::create):
  • svg/SVGFEGaussianBlurElement.cpp: (WebCore::SVGFEGaussianBlurElement::create):
  • svg/SVGFEImageElement.cpp: (WebCore::SVGFEImageElement::create):
  • svg/SVGFEMergeElement.cpp: (WebCore::SVGFEMergeElement::create):
  • svg/SVGFEMergeNodeElement.cpp: (WebCore::SVGFEMergeNodeElement::create):
  • svg/SVGFEMorphologyElement.cpp: (WebCore::SVGFEMorphologyElement::create):
  • svg/SVGFEOffsetElement.cpp: (WebCore::SVGFEOffsetElement::create):
  • svg/SVGFEPointLightElement.cpp: (WebCore::SVGFEPointLightElement::create):
  • svg/SVGFESpecularLightingElement.cpp: (WebCore::SVGFESpecularLightingElement::create):
  • svg/SVGFESpotLightElement.cpp: (WebCore::SVGFESpotLightElement::create):
  • svg/SVGFETileElement.cpp: (WebCore::SVGFETileElement::create):
  • svg/SVGFETurbulenceElement.cpp: (WebCore::SVGFETurbulenceElement::create):
  • svg/SVGFilterElement.cpp: (WebCore::SVGFilterElement::create):
  • svg/SVGFontElement.cpp: (WebCore::SVGFontElement::create):
  • svg/SVGFontFaceElement.cpp: (WebCore::SVGFontFaceElement::create):
  • svg/SVGFontFaceFormatElement.cpp: (WebCore::SVGFontFaceFormatElement::create):
  • svg/SVGFontFaceNameElement.cpp: (WebCore::SVGFontFaceNameElement::create):
  • svg/SVGFontFaceSrcElement.cpp: (WebCore::SVGFontFaceSrcElement::create):
  • svg/SVGFontFaceUriElement.cpp: (WebCore::SVGFontFaceUriElement::create):
  • svg/SVGForeignObjectElement.cpp: (WebCore::SVGForeignObjectElement::create):
  • svg/SVGGElement.cpp: (WebCore::SVGGElement::create):
  • svg/SVGGlyphElement.cpp: (WebCore::SVGGlyphElement::create):
  • svg/SVGHKernElement.cpp: (WebCore::SVGHKernElement::create):
  • svg/SVGImageElement.cpp: (WebCore::SVGImageElement::create):
  • svg/SVGLineElement.cpp: (WebCore::SVGLineElement::create):
  • svg/SVGLinearGradientElement.cpp: (WebCore::SVGLinearGradientElement::create):
  • svg/SVGMPathElement.cpp: (WebCore::SVGMPathElement::create):
  • svg/SVGMarkerElement.cpp: (WebCore::SVGMarkerElement::create):
  • svg/SVGMaskElement.cpp: (WebCore::SVGMaskElement::create):
  • svg/SVGMetadataElement.cpp: (WebCore::SVGMetadataElement::create):
  • svg/SVGMissingGlyphElement.cpp: (WebCore::SVGMissingGlyphElement::create):
  • svg/SVGPathElement.cpp: (WebCore::SVGPathElement::create):
  • svg/SVGPatternElement.cpp: (WebCore::SVGPatternElement::create):
  • svg/SVGPolygonElement.cpp: (WebCore::SVGPolygonElement::create):
  • svg/SVGPolylineElement.cpp: (WebCore::SVGPolylineElement::create):
  • svg/SVGRadialGradientElement.cpp: (WebCore::SVGRadialGradientElement::create):
  • svg/SVGRectElement.cpp: (WebCore::SVGRectElement::create):
  • svg/SVGSVGElement.cpp: (WebCore::SVGSVGElement::create): (WebCore::SVGSVGElement::currentView):
  • svg/SVGScriptElement.cpp: (WebCore::SVGScriptElement::create):
  • svg/SVGSetElement.cpp: (WebCore::SVGSetElement::create):
  • svg/SVGStopElement.cpp: (WebCore::SVGStopElement::create):
  • svg/SVGStyleElement.cpp: (WebCore::SVGStyleElement::create):
  • svg/SVGSwitchElement.cpp: (WebCore::SVGSwitchElement::create):
  • svg/SVGSymbolElement.cpp: (WebCore::SVGSymbolElement::create):
  • svg/SVGTRefElement.cpp: (WebCore::SVGTRefElement::create):
  • svg/SVGTSpanElement.cpp: (WebCore::SVGTSpanElement::create):
  • svg/SVGTextElement.cpp: (WebCore::SVGTextElement::create):
  • svg/SVGTextPathElement.cpp: (WebCore::SVGTextPathElement::create):
  • svg/SVGTitleElement.cpp: (WebCore::SVGTitleElement::create):
  • svg/SVGUseElement.cpp: (WebCore::SVGUseElement::create): (WebCore::SVGUseElement::expandUseElementsInShadowTree):
  • svg/SVGVKernElement.cpp: (WebCore::SVGVKernElement::create):
  • svg/SVGViewElement.cpp: (WebCore::SVGViewElement::create):
  • wml/WMLAElement.cpp: (WebCore::WMLAElement::create):
  • wml/WMLAccessElement.cpp: (WebCore::WMLAccessElement::create):
  • wml/WMLAnchorElement.cpp: (WebCore::WMLAnchorElement::create):
  • wml/WMLBRElement.cpp: (WebCore::WMLBRElement::create):
  • wml/WMLCardElement.cpp: (WebCore::WMLCardElement::create):
  • wml/WMLDoElement.cpp: (WebCore::WMLDoElement::create):
  • wml/WMLDocument.h: (WebCore::WMLDocument::create):
  • wml/WMLFieldSetElement.cpp: (WebCore::WMLFieldSetElement::create):
  • wml/WMLFormControlElement.cpp: (WebCore::WMLFormControlElement::create):
  • wml/WMLGoElement.cpp: (WebCore::WMLGoElement::create):
  • wml/WMLImageElement.cpp: (WebCore::WMLImageElement::create):
  • wml/WMLInputElement.cpp: (WebCore::WMLInputElement::create):
  • wml/WMLInsertedLegendElement.cpp: (WebCore::WMLInsertedLegendElement::create):
  • wml/WMLIntrinsicEvent.cpp: (WebCore::WMLIntrinsicEvent::create):
  • wml/WMLMetaElement.cpp: (WebCore::WMLMetaElement::create):
  • wml/WMLNoopElement.cpp: (WebCore::WMLNoopElement::create):
  • wml/WMLOnEventElement.cpp: (WebCore::WMLOnEventElement::create):
  • wml/WMLOptGroupElement.cpp: (WebCore::WMLOptGroupElement::create):
  • wml/WMLOptionElement.cpp: (WebCore::WMLOptionElement::create):
  • wml/WMLPElement.cpp: (WebCore::WMLPElement::create):
  • wml/WMLPostfieldElement.cpp: (WebCore::WMLPostfieldElement::create):
  • wml/WMLPrevElement.cpp: (WebCore::WMLPrevElement::create):
  • wml/WMLRefreshElement.cpp: (WebCore::WMLRefreshElement::create):
  • wml/WMLSelectElement.cpp: (WebCore::WMLSelectElement::create):
  • wml/WMLSetvarElement.cpp: (WebCore::WMLSetvarElement::create):
  • wml/WMLTableElement.cpp: (WebCore::WMLTableElement::create):
  • wml/WMLTaskElement.cpp: (WebCore::WMLTaskElement::create):
  • wml/WMLTemplateElement.cpp: (WebCore::WMLTemplateElement::create):
  • wml/WMLTimerElement.cpp: (WebCore::WMLTimerElement::create): Use adoptRef.
10:05 AM Changeset in webkit [66497] by evan@chromium.org
  • 2 edits
    2 deletes in trunk/LayoutTests

2010-08-31 Evan Martin <​evan@chromium.org>

Reviewed by Tony Chang.

[chromium] incorrect baseline for fast/css/font-face-implicit-local-font
​https://bugs.webkit.org/show_bug.cgi?id=44893

The test is verifying that the CSS unicode-range: property is correctly
modifying the font for a substring of text. The checked-in image shows
the wrong font.

Also, update the comment in test_expectations to reflect this
information.

  • platform/chromium-linux/fast/css/font-face-implicit-local-font-expected.checksum: Removed.
  • platform/chromium-linux/fast/css/font-face-implicit-local-font-expected.png: Removed.
  • platform/chromium/test_expectations.txt:
10:04 AM Changeset in webkit [66496] by jschuh@chromium.org
  • 3 edits
    2 adds in trunk

2010-08-31 Justin Schuh <​jschuh@chromium.org>

Reviewed by Nate Chapin.

Local frame loads should check against origin (not referrer)
​https://bugs.webkit.org/show_bug.cgi?id=44888

Test: fast/frames/iframe-no-src-local-origin-allow.html

  • loader/SubframeLoader.cpp: (WebCore::SubframeLoader::loadSubframe):

2010-08-31 Justin Schuh <​jschuh@chromium.org>

Reviewed by Nate Chapin.

Local frame loads should check against origin (not referrer)
​https://bugs.webkit.org/show_bug.cgi?id=44888

  • fast/frames/iframe-no-src-local-origin-allow-expected.txt: Added.
  • fast/frames/iframe-no-src-local-origin-allow.html: Added.
10:01 AM Changeset in webkit [66495] by Csaba Osztrogonác
  • 4 edits in trunk/WebKit2

2010-08-31 Csaba Osztrogonác <Csaba Osztrogonác>

Reviewed by Antonio Gomes.

[Qt] Fix warnings in WebKit2 directory
​https://bugs.webkit.org/show_bug.cgi?id=44593

  • Platform/CoreIPC/qt/ConnectionQt.cpp: Mark unused variables for compiler. (CoreIPC::Connection::readyReadHandler): (CoreIPC::Connection::sendOutgoingMessage):
  • Shared/qt/WebEventFactoryQt.cpp: (WebKit::WebEventFactory::createWebTouchEvent): Initialize state variable. Default case added.
  • WebProcess/Plugins/Netscape/qt/NetscapePluginQt.cpp: (WebKit::toNP): Return initialized NPEvent.
9:57 AM Changeset in webkit [66494] by zmo@google.com
  • 5 edits in trunk

2010-08-25 Zhenyao Mo <​zmo@google.com>

Reviewed by Kenneth Russell.

Passing premultiplyAlpha=false to tex{Sub}Image2D loses information (cg)
​https://bugs.webkit.org/show_bug.cgi?id=44566

  • platform/graphics/cg/GraphicsContext3DCG.cpp: (WebCore::GraphicsContext3D::getImageData): Fix the premultiplyAlpha issue for cg.

2010-08-25 Zhenyao Mo <​zmo@google.com>

Reviewed by Kenneth Russell.

Passing premultiplyAlpha=false to tex{Sub}Image2D loses information (cg)
​https://bugs.webkit.org/show_bug.cgi?id=44566

  • fast/canvas/webgl/gl-teximage-expected.txt: Fix a typo in the file.
  • platform/chromium/test_expectations.txt: Re-enable gl-teximage.html test.
  • platform/mac/Skipped: Ditto.
9:52 AM Changeset in webkit [66493] by abecsi@webkit.org
  • 2 edits in trunk/WebKit2

2010-08-31 Andras Becsi <​abecsi@webkit.org>

Reviewed by Antonio Gomes.

[Qt] Determine application path of MiniBrowser at runtime and use that path
for QtWebProcess if the executable exists in that path.

  • UIProcess/Launcher/qt/ProcessLauncherQt.cpp: (WebKit::ProcessLauncherHelper::launch):
9:30 AM Changeset in webkit [66492] by hyatt@apple.com
  • 3 edits
    4 adds in trunk

WebCore: ​https://bugs.webkit.org/show_bug.cgi?id=44956, paginate block-level replaced elements.

<rdar://problem/8000991> Images with display:block get split between columns or pages

Reviewed by Adam Roben.

Added fast/multicol/paginate-block-replaced.html

  • rendering/RenderBlock.cpp:

(WebCore::RenderBlock::paintChildren):

LayoutTests: ​https://bugs.webkit.org/show_bug.cgi?id=44956, paginate block-level replaced elements.

Reviewed by Adam Roben.

  • fast/multicol/paginate-block-replaced.html: Added.
  • platform/mac/fast/multicol/paginate-block-replaced-expected.checksum: Added.
  • platform/mac/fast/multicol/paginate-block-replaced-expected.png: Added.
  • platform/mac/fast/multicol/paginate-block-replaced-expected.txt: Added.
9:24 AM Changeset in webkit [66491] by commit-queue@webkit.org
  • 1 edit
    18 adds in trunk/WebKitTools

2010-08-31 Adrian Perez <Adrian Perez de Castro>

Reviewed by Martin Robinson.

Support scripts to run Buildbot slaves under daemontools (gtk related)
​https://bugs.webkit.org/show_bug.cgi?id=40053

  • BuildSlaveSupport/gtk: Added.
  • BuildSlaveSupport/gtk/README: Added.
  • BuildSlaveSupport/gtk/buildbot: Added.
  • BuildSlaveSupport/gtk/buildbot/log: Added.
  • BuildSlaveSupport/gtk/buildbot/log/run: Added.
  • BuildSlaveSupport/gtk/buildbot/run: Added.
  • BuildSlaveSupport/gtk/crashmon: Added.
  • BuildSlaveSupport/gtk/crashmon/crashmon: Added.
  • BuildSlaveSupport/gtk/crashmon/log: Added.
  • BuildSlaveSupport/gtk/crashmon/log/run: Added.
  • BuildSlaveSupport/gtk/crashmon/run: Added.
  • BuildSlaveSupport/gtk/daemontools-buildbot.conf: Added.
  • BuildSlaveSupport/gtk/pulseaudio: Added.
  • BuildSlaveSupport/gtk/pulseaudio/run: Added.
  • BuildSlaveSupport/gtk/xvfb: Added.
  • BuildSlaveSupport/gtk/xvfb/log: Added.
  • BuildSlaveSupport/gtk/xvfb/log/run: Added.
  • BuildSlaveSupport/gtk/xvfb/run: Added.
9:23 AM Changeset in webkit [66490] by Adam Roben
  • 5 adds in trunk/WebKit/wince

Add InspectorClientWinCE

​https://bugs.webkit.org/show_bug.cgi?id=44819

Patch by Patrick Gansterer <​paroga@paroga.com> on 2010-08-28
Reviewed by Adam Roben.

  • WebCoreSupport/InspectorClientWinCE.cpp: Added.

(WebKit::InspectorClient::InspectorClient):
(WebKit::InspectorClient::~InspectorClient):
(WebKit::InspectorClient::inspectorDestroyed):
(WebKit::InspectorClient::openInspectorFrontend):
(WebKit::InspectorClient::releaseFrontendPage):
(WebKit::InspectorClient::highlight):
(WebKit::InspectorClient::hideHighlight):
(WebKit::InspectorClient::populateSetting):
(WebKit::InspectorClient::storeSetting):
(WebKit::InspectorClient::sendMessageToFrontend):

  • WebCoreSupport/InspectorClientWinCE.h: Added.
9:23 AM Changeset in webkit [66489] by Csaba Osztrogonác
  • 3 edits in trunk/JavaScriptCore

Modify ASSERT_UNUSED and UNUSED_PARAM similar to Qt's Q_UNUSED.
​https://bugs.webkit.org/show_bug.cgi?id=44870

Reviewed by Darin Adler.

  • wtf/Assertions.h:
  • wtf/UnusedParam.h:
8:58 AM Changeset in webkit [66488] by commit-queue@webkit.org
  • 2 edits in trunk/WebKit/efl

2010-08-31 Gyuyoung Kim <​gyuyoung.kim@samsung.com>

Unreviewed build fix.

[EFL] Build break for r66456
​https://bugs.webkit.org/post_bug.cgi

Replace firstRectsForMarkers with renderedRectsForMarkers.

  • ewk/ewk_frame.cpp: (ewk_frame_text_matches_nth_pos_get):
8:55 AM Changeset in webkit [66487] by Philippe Normand
  • 2 edits in trunk/LayoutTests

2010-08-31 Philippe Normand <​pnormand@igalia.com>

Unreviewed, unskip fixed test for the GTK port.

media/video-loop.html fails intermittently on GTK and Chromium bots
​https://bugs.webkit.org/show_bug.cgi?id=33671

  • platform/gtk/Skipped: Unskip media/video-loop.html
8:49 AM Changeset in webkit [66486] by ap@apple.com
  • 3 edits
    2 adds in trunk

Reviewed by Darin Adler.

​https://bugs.webkit.org/show_bug.cgi?id=36515
CSS style definitions are ignored if they just follow a "@charset" which appears in lines
other than the first line in stylesheet.

Test: fast/css/misplaced-charset.html

  • css/CSSGrammar.y: Ignore a misplaced charset rule without consuming extra characters and thus breaking subsequent rules.
8:43 AM Changeset in webkit [66485] by commit-queue@webkit.org
  • 2 edits in trunk/WebCore

2010-08-31 Patrick Gansterer <​paroga@paroga.com>

Reviewed by Antonio Gomes.

[CMake] Add IF (ENABLE_MATHML) around MathML sources
​https://bugs.webkit.org/show_bug.cgi?id=44927

  • CMakeLists.txt:
7:59 AM Changeset in webkit [66484] by commit-queue@webkit.org
  • 2 edits in trunk/WebCore

2010-08-31 Patrick Gansterer <​paroga@paroga.com>

Reviewed by Kenneth Rohde Christiansen.

[CMake] Unify GENERATE_DOM_NAMES code
​https://bugs.webkit.org/show_bug.cgi?id=44842

  • CMakeLists.txt: Added property svn:eol-style.
7:38 AM Changeset in webkit [66483] by commit-queue@webkit.org
  • 3 edits
    1 add in trunk/WebCore

2010-08-31 Daniel Cheng <​dcheng@chromium.org>

Reviewed by Tony Chang.

[chromium] Add an interface for platform copy/paste drag/drop data objects
​https://bugs.webkit.org/show_bug.cgi?id=44914

Currently, Chromium only supports a few hardcoded data types in
event.dataTransfer. This is the first of several patches to add support
for arbitrary data types.

No new tests.

  • WebCore.gypi:
  • platform/chromium/ChromiumDataObjectNew.h: Added.
  • platform/chromium/ClipboardChromium.cpp:
7:22 AM Changeset in webkit [66482] by abarth@webkit.org
  • 13 edits in trunk/LayoutTests

2010-08-31 Adam Barth <​abarth@webkit.org>

Reviewed by Eric Seidel.

Update in-test expectations to account for HTML5 parsing changes
​https://bugs.webkit.org/show_bug.cgi?id=44889

We've already updated the -expected.txt files for these tests to
account for the behavior changes with the HTML5 parser. This patch
updates the redundant "in-test" expectations so that the results say
"PASS" instead of "FAIL".

  • fast/forms/datalist-nonoption-child.html:
  • fast/forms/state-restore-to-non-edited-controls-expected.txt:
  • fast/forms/state-restore-to-non-edited-controls.html:
  • fast/parser/fragment-parser-expected.txt:
  • fast/parser/residual-style-close-across-n-blocks-expected.txt:
  • fast/parser/residual-style-close-across-n-blocks.html:
  • fast/parser/script-tests/fragment-parser.js:
  • http/tests/misc/isindex-formdata-expected.txt:
  • http/tests/misc/isindex-formdata.html:
  • http/tests/misc/isindex-with-no-form-base-href-expected.txt:
  • http/tests/misc/resources/isindex-with-no-form-base-href.html:
7:06 AM Changeset in webkit [66481] by commit-queue@webkit.org
  • 3 edits in trunk/LayoutTests

2010-08-31 Mihai Parparita <​mihaip@chromium.org>

Reviewed by Tony Chang.

Remove passing tests from Chromium's test_expectations.txt
​https://bugs.webkit.org/show_bug.cgi?id=44906

Remove test expectations from Chromium's test_expectations.txt that are
incorrect (the tests now pass).

fast/events/special-key-events-in-input-text.html

fixed by ​http://crrev.com/57503

fast/events/tabindex-focus-chain.html

fixed by ​http://trac.webkit.org/changeset/66311

fast/forms/tabs-with-modifiers.html

fixed by ​http://trac.webkit.org/changeset/65748

editing/selection/drag-in-iframe.html

fixed by ​http://trac.webkit.org/changeset/66269/ (appeared twice in
the file)

  • platform/chromium/drt_expectations.txt:
  • platform/chromium/test_expectations.txt:
6:49 AM Changeset in webkit [66480] by caseq@chromium.org
  • 3 edits in trunk/LayoutTests

2010-08-31 Andrey Kosyakov <​caseq@chromium.org>

Unreviewed. Attempt to fix flakiness of inspector extensions API audits tests on Qt.
Synchronize last output() with signalling test completion.

  • inspector/resources/extension-main.html:
  • inspector/resources/extension-main.js: (onTestsDone):
6:26 AM Changeset in webkit [66479] by Philippe Normand
  • 2 edits in trunk/LayoutTests

2010-08-31 Philippe Normand <​pnormand@igalia.com>

Rubber-stamped by Xan Lopez.

fast/parser/xml-declaration-missing-ending-mark.html crashed on Gtk bot
​https://bugs.webkit.org/show_bug.cgi?id=33027

  • platform/gtk/Skipped: Unskip now passing test.
5:57 AM Changeset in webkit [66478] by benjamin.poulain@nokia.com
  • 2 edits in trunk/WebKit/qt

2010-08-31 Benjamin Poulain <​benjamin.poulain@nokia.com>

Reviewed by Kenneth Rohde Christiansen.

JSC TimeoutChecker::didTimeOut overflows on ARM
​https://bugs.webkit.org/show_bug.cgi?id=38538

Re-enabled the test that was skipped for Maemo 5.

  • tests/qwebpage/tst_qwebpage.cpp: (tst_QWebPage::infiniteLoopJS):
5:38 AM Changeset in webkit [66477] by caseq@chromium.org
  • 22 edits
    10 adds
    1 delete in trunk

2010-08-30 Andrey Kosyakov <​caseq@chromium.org>

Reviewed by Yury Semikhatsky.

Web Inspector: add audits support to extension API
Exposed (late) adding of categories from AuditPanel.
Removed indexOfObjectInListSortedByFunction in favor of
insertionIndexForObjectInListSortedByFunction (the former had
weird interface always returning negative numbers and was only used in
the latter).
​https://bugs.webkit.org/show_bug.cgi?id=44518

Tests: inspector/extensions-audits-api.html

inspector/extensions-audits.html

  • WebCore.gypi:
  • WebCore.vcproj/WebCore.vcproj:
  • inspector/front-end/AuditFormatters.js: Added. (WebInspector.applyFormatters): (WebInspector.AuditFormatters.text): (WebInspector.AuditFormatters.snippet): (WebInspector.AuditFormatters.concat): (WebInspector.AuditFormatters.url):
  • inspector/front-end/AuditLauncherView.js: (WebInspector.AuditLauncherView): (WebInspector.AuditLauncherView.prototype.addCategory.compareCategories): (WebInspector.AuditLauncherView.prototype.addCategory): (WebInspector.AuditLauncherView.prototype._launchButtonClicked): (WebInspector.AuditLauncherView.prototype._selectAllClicked): (WebInspector.AuditLauncherView.prototype._categoryClicked): (WebInspector.AuditLauncherView.prototype._createCategoryElement): (WebInspector.AuditLauncherView.prototype._createLauncherUI):
  • inspector/front-end/AuditResultView.js: (WebInspector.AuditCategoryResultPane.prototype._appendResult):
  • inspector/front-end/AuditsPanel.js: (WebInspector.AuditsPanel): (WebInspector.AuditsPanel.prototype.addCategory): (WebInspector.AuditsPanel.prototype.getCategory): (WebInspector.AuditsPanel.prototype._executeAudit): (WebInspector.AuditCategory.prototype.run):
  • inspector/front-end/ExtensionAPI.js: (WebInspector.injectedExtensionAPI): (WebInspector.injectedExtensionAPI.EventSinkImpl.prototype.addListener): (WebInspector.injectedExtensionAPI.EventSinkImpl.prototype._fire): (WebInspector.injectedExtensionAPI.EventSinkImpl.prototype._dispatch): (WebInspector.injectedExtensionAPI.EventSink): (WebInspector.injectedExtensionAPI.InspectorExtensionAPI): (WebInspector.injectedExtensionAPI.Panels.prototype.create): (WebInspector.injectedExtensionAPI.Audits): (WebInspector.injectedExtensionAPI.Audits.prototype.addCategory): (WebInspector.injectedExtensionAPI.AuditCategory.customDispatch): (WebInspector.injectedExtensionAPI.AuditCategory): (WebInspector.injectedExtensionAPI.AuditCategoryImpl): (WebInspector.injectedExtensionAPI.AuditResult): (WebInspector.injectedExtensionAPI.AuditResult.prototype.get Severity): (WebInspector.injectedExtensionAPI.AuditResultImpl): (WebInspector.injectedExtensionAPI.AuditResultImpl.prototype.addResult): (WebInspector.injectedExtensionAPI.AuditResultImpl.prototype.createResult): (WebInspector.injectedExtensionAPI.AuditResultImpl.prototype.done): (WebInspector.injectedExtensionAPI.AuditResultImpl.prototype._nodeFactory): (WebInspector.injectedExtensionAPI.AuditResultNode): (WebInspector.injectedExtensionAPI.AuditResultNode.prototype.addChild):
  • inspector/front-end/ExtensionAuditCategory.js: Added. (WebInspector.ExtensionAuditCategory): (WebInspector.ExtensionAuditCategory.prototype.get id): (WebInspector.ExtensionAuditCategory.prototype.get displayName): (WebInspector.ExtensionAuditCategory.prototype.get ruleCount): (WebInspector.ExtensionAuditCategory.prototype.run): (WebInspector.ExtensionAuditCategoryResults): (WebInspector.ExtensionAuditCategoryResults.prototype.get complete): (WebInspector.ExtensionAuditCategoryResults.prototype.cancel): (WebInspector.ExtensionAuditCategoryResults.prototype.addResult): (WebInspector.ExtensionAuditCategoryResults.prototype._addNode): (WebInspector.ExtensionAuditCategoryResults.prototype._addResult):
  • inspector/front-end/ExtensionCommon.js: Added. (WebInspector.commonExtensionSymbols):
  • inspector/front-end/ExtensionServer.js: (WebInspector.ExtensionServer): (WebInspector.ExtensionServer.prototype.startAuditRun): (WebInspector.ExtensionServer.prototype.stopAuditRun): (WebInspector.ExtensionServer.prototype._postNotification): (WebInspector.ExtensionServer.prototype._onAddAuditCategory): (WebInspector.ExtensionServer.prototype._onAddAuditResult): (WebInspector.ExtensionServer.prototype._onStopAuditCategoryRun): (WebInspector.ExtensionServer.prototype._addExtensions): (WebInspector.ExtensionServer.prototype._buildExtensionAPIInjectedScript): (WebInspector.ExtensionStatus):
  • inspector/front-end/WebKit.qrc:
  • inspector/front-end/inspector.html:
  • inspector/front-end/utilities.js: ():

2010-08-30 Andrey Kosyakov <​caseq@chromium.org>

Reviewed by Yury Semikhatsky.

Web Inspector: add audits support to extension API
​https://bugs.webkit.org/show_bug.cgi?id=44518

  • http/tests/inspector/inspector-test2.js: (initialize_InspectorTest): (runTest.runTestInFrontend): (runTest):
  • inspector/audits-panel-functional.html:
  • inspector/audits-tests.js: Added. (frontend_collectAuditResults): (frontend_collectTextContent):
  • inspector/audits-tests2.js: Added. (initialize_AuditTests.InspectorTest.collectAuditResults): (initialize_AuditTests.InspectorTest.collectTextContent): (initialize_AuditTests):
  • inspector/extensions-api-expected.txt:
  • inspector/extensions-api.html:
  • inspector/extensions-audits-api-expected.txt: Added.
  • inspector/extensions-audits-api.html: Added.
  • inspector/extensions-audits-expected.txt: Added.
  • inspector/extensions-audits-tests.js: Added. (extension_runAudits.onMessage): (extension_runAudits): (initialize_ExtensionsAuditsTest.InspectorTest.startExtensionAudits.onAuditsDone): (initialize_ExtensionsAuditsTest.InspectorTest.startExtensionAudits): (initialize_ExtensionsAuditsTest): (test):
  • inspector/extensions-audits.html: Added.
  • inspector/extensions-expected.txt:
  • inspector/extensions-test.js: (extensionFunctions): (initialize_ExtensionsTest.InspectorTest.dispatchOnMessage): (initialize_ExtensionsTest.InspectorTest.runExtensionTests): (test):
  • inspector/extensions.html:
  • inspector/resources/audits-script3.js: Removed.
  • inspector/resources/extension-main.js: (onTestsDone):
  • platform/chromium/test_expectations.txt:
5:26 AM Changeset in webkit [66476] by jschuh@chromium.org
  • 2 edits in trunk/WebCore

2010-08-31 Justin Schuh <​jschuh@chromium.org>

Reviewed by Nate Chapin.

Remove unused canReferToParentFrameEncoding() function from FrameLoader.cpp
​https://bugs.webkit.org/show_bug.cgi?id=44895

No new tests; no behavior changed.

  • loader/FrameLoader.cpp:
5:02 AM Changeset in webkit [66475] by benjamin.poulain@nokia.com
  • 2 edits in trunk/JavaScriptCore

2010-08-31 Benjamin Poulain <​benjamin.poulain@nokia.com>

Reviewed by Kenneth Rohde Christiansen.

JSC TimeoutChecker::didTimeOut overflows on ARM
​https://bugs.webkit.org/show_bug.cgi?id=38538

Make getCPUTime() return values relative to the first call.
The previous implementation relied on simply on currentTime(), which
return a time since epoch and not a time since the thread started. This
made the return value of getCPUTime() overflow on 32 bits.

  • runtime/TimeoutChecker.cpp: (JSC::getCPUTime):
4:33 AM Changeset in webkit [66474] by jorlow@chromium.org
  • 2 edits in trunk/WebCore

2010-08-31 Jeremy Orlow <​jorlow@chromium.org>

Build fix.

  • storage/IDBObjectStoreBackendImpl.cpp: (WebCore::IDBObjectStoreBackendImpl::openCursor):
3:46 AM Changeset in webkit [66473] by jorlow@chromium.org
  • 18 edits
    2 adds in trunk

2010-08-24 Jeremy Orlow <​jorlow@chromium.org>

Reviewed by Steve Block.

Implement the rest of IDBCursors + make them persistent
​https://bugs.webkit.org/show_bug.cgi?id=44546

  • storage/indexeddb/keyrange-expected.txt:
  • storage/indexeddb/objectstore-cursor.html: Added.
  • storage/indexeddb/open-cursor-expected.txt:
  • storage/indexeddb/script-tests/keyrange.js: (checkLeftBoundKeyRange): ():
  • storage/indexeddb/script-tests/open-cursor.js: (emptyCursorSuccess):

2010-08-24 Jeremy Orlow <​jorlow@chromium.org>

Reviewed by Steve Block.

Implement the rest of IDBCursors + make them persistent
​https://bugs.webkit.org/show_bug.cgi?id=44546

Because the old (memory only) backend didn't support ranges, IDBCursors
were never really finished. This completes them on top of SQLite like
the rest of IndexedDB.

Test: storage/indexeddb/objectstore-cursor.html

  • storage/IDBCursor.cpp: (WebCore::IDBCursor::value):
  • storage/IDBCursorBackendImpl.cpp: (WebCore::IDBCursorBackendImpl::IDBCursorBackendImpl): (WebCore::IDBCursorBackendImpl::key): (WebCore::IDBCursorBackendImpl::value): (WebCore::IDBCursorBackendImpl::update): (WebCore::IDBCursorBackendImpl::continueFunction): (WebCore::IDBCursorBackendImpl::remove): (WebCore::IDBCursorBackendImpl::loadCurrentRow):
  • storage/IDBCursorBackendImpl.h: (WebCore::IDBCursorBackendImpl::create):
  • storage/IDBCursorBackendInterface.h:
  • storage/IDBKey.cpp: (WebCore::IDBKey::isEqual):
  • storage/IDBKey.h:
  • storage/IDBKeyRange.cpp: (WebCore::IDBKeyRange::leftBound): (WebCore::IDBKeyRange::rightBound): (WebCore::IDBKeyRange::bound):
  • storage/IDBObjectStoreBackendImpl.cpp: (WebCore::cursorWhereFragment): (WebCore::IDBObjectStoreBackendImpl::openCursor):

2010-08-24 Jeremy Orlow <​jorlow@chromium.org>

Reviewed by Steve Block.

Implement the rest of IDBCursors + make them persistent
​https://bugs.webkit.org/show_bug.cgi?id=44546

  • src/IDBCursorBackendProxy.cpp: (WebCore::IDBCursorBackendProxy::value):
  • src/IDBCursorBackendProxy.h:
  • src/WebIDBCursorImpl.cpp: (WebKit::WebIDBCursorImpl::value):
2:56 AM Changeset in webkit [66472] by commit-queue@webkit.org
  • 6 edits in trunk/LayoutTests

2010-08-31 Mihai Parparita <​mihaip@chromium.org>

Reviewed by Adam Barth.

fast/dom/frame-loading-via-document-write.html fails on Chromium and is otherwise flaky
​https://bugs.webkit.org/show_bug.cgi?id=44894

Detect all frames loading more reliably (add a counter to count down when each of
the three frames has loaded). Install onload handler on images instead of the document
to not tickle bug 29615.

  • fast/dom/frame-loading-via-document-write.html:
  • fast/dom/resources/frame-loading-via-document-write.js: (didImageLoad):
  • platform/chromium/test_expectations.txt:
  • platform/mac-leopard/Skipped:
  • platform/mac-snowleopard/Skipped:
2:40 AM Changeset in webkit [66471] by abarth@webkit.org
  • 3 edits in trunk/BugsSite

2010-08-31 Adam Barth <​abarth@webkit.org>

Reviewed by Eric Seidel.

[reviewtool] Allow reviewer to select lines of context by dragging over the line numbers
​https://bugs.webkit.org/show_bug.cgi?id=44936

  • PrettyPatch/PrettyPatch.rb:
    • Add a version number to bust through bugs.webkit.org's aggressive cacheing.
  • code-review.js:
1:58 AM Changeset in webkit [66470] by johnnyg@google.com
  • 13 edits
    2 adds in trunk

2010-08-27 John Gregg <​johnnyg@google.com>

Reviewed by David Levin.

Notifications should support a click event
​https://bugs.webkit.org/show_bug.cgi?id=44800

  • public/WebNotification.h:
  • src/WebNotification.cpp: (WebKit::WebNotification::dispatchClickEvent):

2010-08-27 John Gregg <​johnnyg@google.com>

Reviewed by David Levin.

Notifications should support a click event
​https://bugs.webkit.org/show_bug.cgi?id=44800

Test: fast/notifications/notifications-click-event.html

  • notifications/Notification.h:
  • notifications/Notification.idl:

2010-08-27 John Gregg <​johnnyg@google.com>

Reviewed by David Levin.

Notifications should support a click event
​https://bugs.webkit.org/show_bug.cgi?id=44800

  • fast/notifications/notifications-click-event-expected.txt: Added.
  • fast/notifications/notifications-click-event.html: Added.
  • platform/qt/Skipped:

2010-08-27 John Gregg <​johnnyg@google.com>

Reviewed by David Levin.

Notifications should support a click event.
Adds necessary hooks to chromium's DRT so that clicks on desktop notifications
can be simulated during a layout test. Requires storing a list of active
notifications so that they can be referred to later for clicking.
​https://bugs.webkit.org/show_bug.cgi?id=44800

  • DumpRenderTree/chromium/LayoutTestController.cpp: (LayoutTestController::LayoutTestController): (LayoutTestController::simulateDesktopNotificationClick):
  • DumpRenderTree/chromium/LayoutTestController.h:
  • DumpRenderTree/chromium/NotificationPresenter.cpp: (NotificationPresenter::simulateClick): (NotificationPresenter::show): (NotificationPresenter::cancel): (NotificationPresenter::objectDestroyed):
  • DumpRenderTree/chromium/NotificationPresenter.h:
1:25 AM Changeset in webkit [66469] by alex
  • 2 edits in trunk/LayoutTests

2010-08-31 Alejandro G. Castro <​alex@igalia.com>

Unreviewed.

[GTK] canvas/philip/tests/2d.path.isPointInPath.edge-actual.txt
fails in the bots
​https://bugs.webkit.org/show_bug.cgi?id=44932

Skip a test that is failing in the bots.

  • platform/gtk/Skipped:
1:21 AM Changeset in webkit [66468] by jianli@chromium.org
  • 2 edits in trunk/LayoutTests

Skip fast/files/apply-blob-url-to-img.html for now until finding out
how to generate the right result.

  • platform/mac/Skipped:
1:10 AM Changeset in webkit [66467] by tkent@chromium.org
  • 7 edits
    1 add in trunk

[V8] Custom binding for "dataset"
​https://bugs.webkit.org/show_bug.cgi?id=44930

Reviewed by Adam Barth.

WebCore:

  • WebCore.gyp/WebCore.gyp: Remove the exclusion of DOMStringMap.idl.
  • WebCore.gypi: Add V8DOMStringMapCustom.cpp.
  • bindings/v8/custom/V8DOMStringMapCustom.cpp: Added.

(WebCore::V8DOMStringMap::namedPropertyQuery):
(WebCore::V8DOMStringMap::namedPropertyGetter):
(WebCore::V8DOMStringMap::namedPropertyEnumerator):
(WebCore::V8DOMStringMap::namedPropertyDeleter):
(WebCore::V8DOMStringMap::namedPropertySetter):

  • dom/Element.idl: Remove V8 exclusion.
  • page/DOMWindow.idl: ditto.

LayoutTests:

  • platform/chromium/test_expectations.txt: Remove dataset.html and dataset-xhtml.xhtml.
1:06 AM Changeset in webkit [66466] by zoltan@webkit.org
  • 3 edits in trunk/WebKit2

Build fix after r66448 on WebKit2.

[Qt] qt_wk_didFinishDocumentLoadForFrame needs to be implemented
​https://bugs.webkit.org/show_bug.cgi?id=44934

  • UIProcess/API/qt/ClientImpl.cpp:

(qt_wk_didFinishDocumentLoadForFrame):

  • UIProcess/API/qt/ClientImpl.h:
12:56 AM Changeset in webkit [66465] by jianli@chromium.org
  • 2 edits in trunk/LayoutTests

Update chromium test result.

  • platform/chromium/test_expectations.txt:
12:39 AM Changeset in webkit [66464] by jianli@chromium.org
  • 2 edits in trunk/WebCore

Fix chromium build break.

  • page/DOMWindow.h:
12:35 AM Changeset in webkit [66463] by abarth@webkit.org
  • 4 edits
    2 adds in trunk

2010-08-31 Adam Barth <​abarth@webkit.org>

Reviewed by Eric Seidel.

Stale document crashes at multiple URLs (with new parser)
​https://bugs.webkit.org/show_bug.cgi?id=44885

We're not supposed to load resources in detached documents. According
to several locations I examined, it's the job of the caller of
DocLoader to check this invariant. Like most code dealing with
detached documents, this check isn't overly consistent. At some point,
we'll need to rationalize all the different patterns here. However, at
this point, this patch appears correct because it matches what we do
for <link rel="stylesheet">.

Test: fast/dom/HTMLLinkElement/prefetch-detached.html

  • html/HTMLLinkElement.cpp: (WebCore::HTMLLinkElement::process):
  • loader/DocLoader.cpp: (WebCore::DocLoader::requestLinkPrefetch):

2010-08-31 Adam Barth <​abarth@webkit.org>

Reviewed by Eric Seidel.

Stale document crashes at multiple URLs (with new parser)
​https://bugs.webkit.org/show_bug.cgi?id=44885

Test what happens when you insert a link prefetch element into a
detached document.

  • fast/dom/HTMLLinkElement/prefetch-detached-expected.txt: Added.
  • fast/dom/HTMLLinkElement/prefetch-detached.html: Added.
12:25 AM Changeset in webkit [66462] by jianli@chromium.org
  • 14 edits
    9 adds in trunk

Expose window.createBlobURL and window.revokeBlobURL.
​https://bugs.webkit.org/show_bug.cgi?id=44908

Reviewed by David Levin.

WebCore:

Tests: fast/files/apply-blob-url-to-img.html

fast/files/apply-blob-url-to-xhr.html

  • page/DOMWindow.cpp:

(WebCore::DOMWindow::createBlobURL):
(WebCore::DOMWindow::revokeBlobURL):

  • page/DOMWindow.h:
  • page/DOMWindow.idl:
  • xml/XMLHttpRequest.cpp:

(WebCore::XMLHttpRequest::createRequest):

LayoutTests:

  • fast/dom/Window/script-tests/window-property-descriptors.js:
  • fast/dom/Window/window-properties.html:
  • fast/dom/script-tests/prototype-inheritance.js:
  • fast/files/apply-blob-url-to-img.html: Added.
  • fast/files/apply-blob-url-to-xhr-expected.txt: Added.
  • fast/files/apply-blob-url-to-xhr.html: Added.
  • fast/files/resources/abe.png: Copied from LayoutTests/editing/resources/abe.png.
  • platform/chromium/test_expectations.txt:
  • platform/gtk/Skipped:
  • platform/mac-wk2/Skipped:
  • platform/mac/fast/files/apply-blob-url-to-img-expected.checksum: Added.
  • platform/mac/fast/files/apply-blob-url-to-img-expected.png: Added.
  • platform/mac/fast/files/apply-blob-url-to-img-expected.txt: Added.
  • platform/qt/Skipped:
  • platform/win/Skipped:
12:06 AM Changeset in webkit [66461] by jianli@chromium.org
  • 24 edits
    3 copies
    16 adds in trunk

Support FileReaderSync in workers.
​https://bugs.webkit.org/show_bug.cgi?id=44657

Reviewed by David Levin.

WebCore:

Also add FileException interface and make Blob/File/FileError useable in
workers.

Tests: fast/files/workers/worker-read-blob-async.html

fast/files/workers/worker-read-blob-sync.html
fast/files/workers/worker-read-file-async.html
fast/files/workers/worker-read-file-sync.html

  • DerivedSources.cpp:
  • DerivedSources.make:
  • WebCore.gypi:
  • WebCore.pro:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • fileapi/Blob.idl: Added NoStaticTables attribute.
  • fileapi/File.idl: Added NoStaticTables attribute.
  • fileapi/FileError.idl: Added NoStaticTables attribute.
  • fileapi/FileException.h: Added.
  • fileapi/FileException.idl: Added.
  • fileapi/FileReaderSync.cpp: Added.
  • fileapi/FileReaderSync.h: Added.
  • fileapi/FileReaderSync.idl: Added.
  • workers/WorkerContext.idl: Expose FileReaderSync and BlodBuilder in workers.

LayoutTests:

Added 4 test files to test FileReaderSync in workers.

  • fast/files/read-blob-async.html:
  • fast/files/read-file-async.html:
  • fast/files/resources/read-common.js:

(readBlobAsBinaryString):
(readBlobAsText):
(readBlobAsDataURL):
(_readBlobAsBinaryStringAsync):
(_readBlobAsTextAsync):
(_readBlobAsDataURLAsync):
(_readBlobAsBinaryStringSync):
(_readBlobAsTextSync):
(_readBlobAsDataURLSync):
(createReaderSync):

  • fast/files/resources/read-blob-test-cases.js:

(runNextTest):

  • fast/files/resources/read-file-test-cases.js:

(runNextTest):
(testMultipleReads):

  • fast/files/resources/setup-for-read-common.js:

(log):
(startWorker.worker.onmessage):
(startWorker.worker.onerror):
(startWorker):

  • fast/files/workers/resources/worker-read-blob-async.js: Added.
  • fast/files/workers/resources/worker-read-blob-sync.js: Added.
  • fast/files/workers/resources/worker-read-common.js: Added.
  • fast/files/workers/resources/worker-read-file-async.js: Added.
  • fast/files/workers/resources/worker-read-file-sync.js: Added.
  • fast/files/workers/worker-read-blob-async-expected.txt: Added.
  • fast/files/workers/worker-read-blob-async.html: Added.
  • fast/files/workers/worker-read-blob-sync-expected.txt: Added.
  • fast/files/workers/worker-read-blob-sync.html: Added.
  • fast/files/workers/worker-read-file-async-expected.txt: Added.
  • fast/files/workers/worker-read-file-async.html: Added.
  • fast/files/workers/worker-read-file-sync-expected.txt: Added.
  • fast/files/workers/worker-read-file-sync.html: Added.
  • platform/chromium/test_expectations.txt:
  • platform/gtk/Skipped:
  • platform/mac-wk2/Skipped:
  • platform/qt/Skipped:
  • platform/win/Skipped:
Note: See TracTimeline for information about the timeline view.