Timeline
05/04/10:
- 23:06 Changeset [58803] by
-
2010-05-04 Eric Seidel <eric@webkit.org>
Reviewed by Shinichiro Hamaji.
PrettyPatch.pretty_diff("") should not hang
https://bugs.webkit.org/show_bug.cgi?id=38552
Also found a bug in PrettyPatch.pretty_diff where it would
hang when passed "" as input.
I suspect there may be bugs in prettify.rb (or our use there-of)
where it can hang, which would then cause the testing thread to hang.
- Scripts/webkitpy/common/prettypatch.py:
- Don't hang when passed ""
- Scripts/webkitpy/common/prettypatch_unittest.py:
- Test that we don't hang when passed ""
- Scripts/webkitpy/layout_tests/port/base.py:
- Add a FIXME that we should share code with prettypatch.rb
- Scripts/webkitpy/common/prettypatch.py:
- 22:09 Changeset [58802] by
-
2010-05-04 Simon Fraser <simon.fraser@apple.com>
Reviewed by Dan Bernstein.
Improve check for drawing into the window that was added in r58623
https://bugs.webkit.org/show_bug.cgi?id=38562
Rather than assuming that any non-bitmap context is the window's context,
compare the current graphics context with -[NSWindow graphicsContext] to
determine that we're drawing into the window.
- WebView/WebHTMLView.mm: (-[WebHTMLView drawRect:]):
- 20:24 Changeset [58801] by
-
Auto-generate all Database callbacks.
https://bugs.webkit.org/show_bug.cgi?id=38503
Reviewed by Adam Barth.
- DerivedSources.cpp:
- DerivedSources.make:
- GNUmakefile.am:
- WebCore.gypi:
- WebCore.pri:
- WebCore.pro:
- WebCore.vcproj/WebCore.vcproj:
- WebCore.xcodeproj/project.pbxproj:
- bindings/js/JSBindingsAllInOne.cpp:
- bindings/js/JSCustomSQLStatementCallback.cpp: Removed.
- bindings/js/JSCustomSQLStatementCallback.h: Removed.
- bindings/js/JSCustomSQLStatementErrorCallback.cpp:
(WebCore::JSSQLStatementErrorCallback::handleEvent):
- bindings/js/JSCustomSQLStatementErrorCallback.h: Removed.
- bindings/js/JSCustomSQLTransactionCallback.cpp: Removed.
- bindings/js/JSCustomSQLTransactionCallback.h: Removed.
- bindings/js/JSCustomSQLTransactionErrorCallback.cpp: Removed.
- bindings/js/JSCustomSQLTransactionErrorCallback.h: Removed.
- bindings/js/JSDatabaseCallback.cpp: Removed.
- bindings/js/JSDatabaseCallback.h: Removed.
- bindings/js/JSDatabaseCustom.cpp:
(WebCore::JSDatabase::changeVersion):
(WebCore::createTransaction):
- bindings/js/JSSQLTransactionCustom.cpp:
(WebCore::JSSQLTransaction::executeSql):
- bindings/scripts/CodeGeneratorJS.pm:
- bindings/scripts/test/JS/JSTestCallback.cpp:
- bindings/v8/custom/V8CustomSQLStatementCallback.cpp: Removed.
- bindings/v8/custom/V8CustomSQLStatementCallback.h: Removed.
- bindings/v8/custom/V8CustomSQLStatementErrorCallback.cpp:
(WebCore::V8SQLStatementErrorCallback::handleEvent):
- bindings/v8/custom/V8CustomSQLStatementErrorCallback.h: Removed.
- bindings/v8/custom/V8CustomSQLTransactionCallback.cpp: Removed.
- bindings/v8/custom/V8CustomSQLTransactionCallback.h: Removed.
- bindings/v8/custom/V8CustomSQLTransactionErrorCallback.cpp: Removed.
- bindings/v8/custom/V8CustomSQLTransactionErrorCallback.h: Removed.
- bindings/v8/custom/V8DatabaseCallback.cpp: Removed.
- bindings/v8/custom/V8DatabaseCallback.h: Removed.
- bindings/v8/custom/V8DatabaseCustom.cpp:
(WebCore::V8Database::changeVersionCallback):
(WebCore::createTransaction):
- bindings/v8/custom/V8SQLTransactionCustom.cpp:
(WebCore::V8SQLTransaction::executeSqlCallback):
- storage/DatabaseCallback.h:
- storage/DatabaseCallback.idl: Added.
- storage/SQLStatement.cpp:
(WebCore::SQLStatement::performCallback):
- storage/SQLStatementCallback.h:
- storage/SQLStatementCallback.idl: Added.
- storage/SQLStatementErrorCallback.idl: Added.
- storage/SQLTransaction.cpp:
(WebCore::SQLTransaction::deliverTransactionCallback):
- storage/SQLTransactionCallback.h:
- storage/SQLTransactionCallback.idl: Added.
- storage/SQLTransactionErrorCallback.h:
- storage/SQLTransactionErrorCallback.idl: Added.
- 20:06 Changeset [58800] by
-
results for new test checked in with http://trac.webkit.org/changeset/58797
- 19:43 Changeset [58799] by
-
2010-05-04 Dirk Pranke <dpranke@chromium.org>
Unreviewed, expectations fix.
Re-add the two expectations lines that got mistakenly deleted in
r58798.
- platform/chromium/test_expectations.txt:
- 19:05 Changeset [58798] by
-
2010-05-04 Chris Marrin <cmarrin@apple.com>
Reviewed by Simon Fraser.
Made composited iframes work on Windows
https://bugs.webkit.org/show_bug.cgi?id=32446
I've added one new test (composited-parent-iframe.html) which
shows both the parent iframe and the iframe contents document
with compositing. It also shows correct behavior with clipping,
borders, margins, padding and shadows. It also sets the iframe
size as a percent of the page to show correct resizing behavior.
The existing composited-iframe.html has been changed similarly
but without the page resizing. The existing preserve-3d-switching.html
test needed new results because of changes to the layer dumper.
- compositing/geometry/preserve-3d-switching-expected.txt:
- compositing/iframes/composited-iframe.html:
- compositing/iframes/composited-parent-iframe.html: Added.
- compositing/iframes/resources/composited-subframe.html:
- platform/win/compositing/iframes/composited-iframe-expected.txt:
- platform/win/compositing/iframes/composited-parent-iframe-expected.txt: Added.
- 18:50 Changeset [58797] by
-
Reviewed by Simon Fraser.
Made composited iframes work on Windows
https://bugs.webkit.org/show_bug.cgi?id=32446
This completes the work in http://trac.webkit.org/changeset/57919
to create compositing layers in the parent document when an iframe has
a compositing layer. The parent document has a layer for the iframe
element and builds a layer tree to the page root. The layer tree for the
iframe document is then parented to the iframe element's GraphicsLayer.
The RenderLayerCompositor for the iframe document (which owns the
root of the layer tree) now has a clippingLayer which is the
parent of the layer tree root so it can be clipped to the parent
iframe's bounds, taking into account borders, padding, etc. in
the parent iframe element.
I also got rid of a no longer used function: RenderLayerCompositor::parentInRootLayer
Test: compositing/iframes/composited-parent-iframe.html
- rendering/RenderLayerBacking.cpp:Make calls to RenderLayerCompositor to set the clipping bounds for iframe content
- rendering/RenderLayerCompositor.cpp:Hook the iframe content to the parent iframe element
- rendering/RenderLayerCompositor.h:
- 18:24 Changeset [58796] by
-
Not reviewed.
Updated test results - the test still passes.
- http/tests/security/postMessage/invalid-origin-throws-exception-expected.txt:
- 18:22 Changeset [58795] by
-
Fix the sticky-notes demo.
https://bugs.webkit.org/show_bug.cgi?id=38550
Reviewed by Darin Adler.
- Exceptions thrown by openDatabase() should be treated as
failures.
- Do not allow users to create new notes when openDatabase()
failed.
- demos/sticky-notes/index.html:
- 18:11 Changeset [58794] by
-
AX: layout tests need to be updated to reflect what is being tested
https://bugs.webkit.org/show_bug.cgi?id=38556
Reviewed by Beth Dakin.
Many of the initial AX layout tests just dumped a lot of attributes and did not
have a focused test. This updates some of those tests (more to follow in the future).
- accessibility/image-map1.html:
- accessibility/img-aria-button-alt-tag-expected.txt:
- accessibility/img-aria-button-alt-tag.html:
- accessibility/input-image-alt-expected.txt:
- accessibility/input-image-alt.html:
- accessibility/placeholder-expected.txt:
- accessibility/placeholder.html:
- platform/mac/accessibility/image-map1-expected.txt:
- 18:08 Changeset [58793] by
-
2010-05-04 Vangelis Kokkevis <vangelis@chromium.org>
Reviewed by Darin Fisher.
Provide an API for querying whether a WebWidget is using gpu accelerated compositing.
Also changed: isAcceleratedCompositing to: isAcceleratedCompositingActive.
https://bugs.webkit.org/show_bug.cgi?id=38220
- public/WebWidget.h:
- src/WebPopupMenuImpl.h: (WebKit::WebPopupMenuImpl::isAcceleratedCompositingActive):
- src/WebViewImpl.cpp: (WebKit::WebViewImpl::WebViewImpl): (WebKit::WebViewImpl::paint): (WebKit::WebViewImpl::isAcceleratedCompositingActive): (WebKit::WebViewImpl::setRootGraphicsLayer): (WebKit::WebViewImpl::setIsAcceleratedCompositingActive): (WebKit::WebViewImpl::updateRootLayerContents):
- src/WebViewImpl.h:
- 17:57 Changeset [58792] by
-
Reviewed by Adam Barth.
https://bugs.webkit.org/show_bug.cgi?id=38497
<rdar://problem/7759438> Make sure that http URLs always have a host in SecurityOrigin
This is a hardening fix, and behavior really depends on what an underlying networking layer
does. So, no test.
- page/SecurityOrigin.cpp: (WebCore::schemeRequiresAuthority): List schemes that need an authority for successful loading. (WebCore::SecurityOrigin::SecurityOrigin): Never let e.g. http origins with empty authorities have the same security origin.
- 17:49 Changeset [58791] by
-
2010-05-04 Dirk Pranke <dpranke@chromium.org>
Unreviewed, expectations change.
Mark fast/forms/placeholder-pseudo-style.html as an IMAGE diff on
chromium-linux . This was mistaken deleted earlier today. Mark
fast/dom/clone-node-z-index.html as a TEXT diff on chromium-win; I
haven't looked at this yet but I need to roll past this to get the
new-run-webkit-tests change I checked in in r58789 downstream to
the Chromium bots.
- platform/chromium/test_expectations.txt:
- 17:30 Changeset [58790] by
-
2010-05-04 Zhenyao Mo <zmo@google.com>
Reviewed by Dimitri Glazkov.
getFramebufferAttachmentParameter should return the original WebGLTexture/WebGLRenderbuffer instead of creating new ones sharing names.
https://bugs.webkit.org/show_bug.cgi?id=38236
- fast/canvas/webgl/gl-object-get-calls-expected.txt: Check if getFramebufferAttachmentParameter return a texture/renderbuffer that matches the original one.
- fast/canvas/webgl/script-tests/gl-object-get-calls.js: Ditto.
2010-05-04 Zhenyao Mo <zmo@google.com>
Reviewed by Dimitri Glazkov.
getFramebufferAttachmentParameter should return the original WebGLTexture/WebGLRenderbuffer instead of creating new ones sharing names.
https://bugs.webkit.org/show_bug.cgi?id=38236
- html/canvas/CanvasObject.h: Add type check functions. (WebCore::CanvasObject::isBuffer): (WebCore::CanvasObject::isFramebuffer): (WebCore::CanvasObject::isProgram): (WebCore::CanvasObject::isRenderbuffer): (WebCore::CanvasObject::isShader): (WebCore::CanvasObject::isTexture):
- html/canvas/WebGLBuffer.h: Add type check functions. (WebCore::WebGLBuffer::isBuffer):
- html/canvas/WebGLFramebuffer.h: Add type check functions. (WebCore::WebGLFramebuffer::isFramebuffer):
- html/canvas/WebGLProgram.h: Add type check functions. (WebCore::WebGLProgram::isProgram):
- html/canvas/WebGLRenderbuffer.cpp: remove constructor using existing name.
- html/canvas/WebGLRenderbuffer.h: Add type check functions; remove constructor using existing name. (WebCore::WebGLRenderbuffer::isRenderbuffer):
- html/canvas/WebGLRenderingContext.cpp: (WebCore::WebGLRenderingContext::getFramebufferAttachmentParameter): Return original Texture/Renderbuffer instead of creating new ones. (WebCore::WebGLRenderingContext::findTexture): Find a WebGLTexture using a name. (WebCore::WebGLRenderingContext::findRenderbuffer): Find a WebGLRenderbuffer using a name.
- html/canvas/WebGLRenderingContext.h: Add find* functions.
- html/canvas/WebGLShader.h: Add type check functions. (WebCore::WebGLShader::isShader):
- html/canvas/WebGLTexture.cpp: remove constructor using existing name.
- html/canvas/WebGLTexture.h: Add type check functions; remove constructor using existing name. (WebCore::WebGLTexture::isTexture):
- 17:16 Changeset [58789] by
-
2010-05-04 Dirk Pranke <dpranke@chromium.org>
Reviewed by Eric Seidel.
new-run-webkit-tests: turn off threading on the Chromium Mac port until
we can stabilize the port more and figure out why it is hanging so
frequently.
- Scripts/webkitpy/layout_tests/port/chromium_mac.py:
- override default_child_processes() and log a warning
- Scripts/webkitpy/layout_tests/run_webkit_tests.py:
- fix a typo that caused us to print a method object instead of the value the method object returns in the case where there is only one child process.
- Scripts/webkitpy/layout_tests/run_webkit_tests_unittest.py:
- Add unit tests for the output of run_webkit_tests - in this case, the handling of --child-processes and --print config
- Scripts/webkitpy/layout_tests/port/chromium_mac.py:
- 17:16 Changeset [58788] by
-
2010-05-04 Luiz Agostini <luiz.agostini@openbossa.org>
Reviewed by Simon Hausmann.
[Qt] QT_MOBILE_THEME compile time flag
https://bugs.webkit.org/show_bug.cgi?id=38439
Replacing preprocessor conditional used in RenderThemeQt from Q_WS_MAEMO_5 to
USE(QT_MOBILE_THEME).
- WebCore.pro:
- platform/qt/RenderThemeQt.cpp: (WebCore::RenderThemeQt::RenderThemeQt): (WebCore::RenderThemeQt::qStyle): (WebCore::RenderThemeQt::extraDefaultStyleSheet): (WebCore::RenderThemeQt::adjustMenuListButtonStyle): (WebCore::RenderThemeQt::setPaletteFromPageClientIfExists):
- platform/qt/RenderThemeQt.h:
- 16:57 Changeset [58787] by
-
https://bugs.webkit.org/show_bug.cgi?id=38555
Reviewed by David Kilzer.
Small code refactoring: move the logic to figure out the path to the
databases directory to another method.
- Storage/WebDatabaseManager.mm:
(databasesDirectoryPath):
(WebKitInitializeDatabasesIfNecessary):
- 16:17 Changeset [58786] by
-
2010-05-04 Joseph Pecoraro <joepeck@webkit.org>
Reviewed by Dan Bernstein.
SVG fonts trigger GlyphPage::fill with null font
https://bugs.webkit.org/show_bug.cgi?id=38530
SVG fonts do not use the glyph page cache. This change detects when
attempting to fill a GlyphPage with an SVG font and indicates that
the glyphs were not available.
- platform/graphics/GlyphPageTreeNode.cpp: (WebCore::fill): helper method which handles attempts to fill a GlyphPage with SVG or non-SVG fonts (WebCore::GlyphPageTreeNode::initializePage): use the helper where appropriate
- 15:59 Changeset [58785] by
-
Fixes to V8DOMWrapper to handle CachedAudio
- 15:58 Changeset [58784] by
-
Initial changes to gyp files to build audio engine
- 15:58 Changeset [58783] by
-
Add custom V8 bindings for AudioContext and AudioNode
- 15:46 Changeset [58782] by
-
Unreviewed tests skips for qt/gtk.
Add the recently added drag and drop tests to the
skipped lists for both gtk and qt to go along with
the other drag and drop tests that are skipped on
those platforms.
- platform/gtk/Skipped:
- platform/qt/Skipped:
- 15:24 Changeset [58781] by
-
2010-05-04 Anders Carlsson <andersca@apple.com>
Reviewed by Dan Bernstein.
[WebKit2] The web process doesn't need to paint when the web view is hidden.
https://bugs.webkit.org/show_bug.cgi?id=38549
- Shared/CoreIPCSupport/DrawingAreaMessageKinds.h: (DrawingAreaMessage::): Add SuspendPainting/ResumePainting messages.
- UIProcess/DrawingAreaProxyUpdateChunk.cpp: (WebKit::DrawingAreaProxyUpdateChunk::setPageIsVisible): Suspend and resume painting accordingly.
- WebProcess/WebPage/DrawingAreaUpdateChunk.cpp: (WebKit::DrawingAreaUpdateChunk::DrawingAreaUpdateChunk): Initialize m_shouldPaint to true.
(WebKit::DrawingAreaUpdateChunk::display):
Return if m_shouldPaint is false.
(WebKit::DrawingAreaUpdateChunk::scheduleDisplay):
Ditto.
(WebKit::DrawingAreaUpdateChunk::setSize):
Assert that we should paint here.
(WebKit::DrawingAreaUpdateChunk::suspendPainting):
Set m_shouldPaint to false and stop the timer.
(WebKit::DrawingAreaUpdateChunk::resumePainting):
Set m_shouldPaint to true and paint if needed.
(WebKit::DrawingAreaUpdateChunk::didReceiveMessage):
handle SuspendPainting/ResumePainting messages.
- WebProcess/WebPage/DrawingAreaUpdateChunk.h:
- 15:04 Changeset [58780] by
-
REGRESSION(r54368): Text drag-and-drop from input/textarea doesn't work
if the text is like a URL
https://bugs.webkit.org/show_bug.cgi?id=38374
Patch by Kent Tamura <tkent@chromium.org> on 2010-05-04
Reviewed by David Levin (and unofficially by Enrica Casucci).
WebCore:
Since r54368, documentFragmentFromDragData() creates an <a> element
with no anchor text for URL-like text dragged from input/textarea
element. If such text is dropped to input/textarea element, the text
is not inserted.
To fix this problem, use the original text or the URL as an anchor
text.
Test: editing/pasteboard/drag-drop-url-text.html
- page/DragController.cpp:
(WebCore::documentFragmentFromDragData):
LayoutTests:
- editing/pasteboard/drag-drop-url-text-expected.txt: Added.
- editing/pasteboard/drag-drop-url-text.html: Added.
- 15:01 Changeset [58779] by
-
2010-05-04 Yongjun Zhang <yongjun_zhang@apple.com>
Reviewed by Darin Adler.
WebKit crashes at DebuggerCallFrame::functionName() if m_callFrame is the top global callframe.
https://bugs.webkit.org/show_bug.cgi?id=38535
Don't call asFunction if callee is not a FunctionType to prevent assertion failure
in JSCell::isObject().
- debugger/DebuggerCallFrame.cpp: (JSC::DebuggerCallFrame::functionName): (JSC::DebuggerCallFrame::calculatedFunctionName):
- 14:50 Changeset [58778] by
-
Remove ChangeLog from bad merge.
- 14:49 Changeset [58777] by
-
Fix the find command in extract-localizable-strings so skip
directories are correctly skipped and header files are searched.
https://bugs.webkit.org/show_bug.cgi?id=38545
rdar://problem/7941295
Reviewed by Darin Adler.
- Scripts/extract-localizable-strings: Append -o after each -prune
so -and isn't implied. Surround all the -name arguments so they get
an implied -print action. Removed check for "icu". Skip any header that
ends in LocalizableStrings.h, so SafariLocalizableStrings.h is skipped.
- Scripts/update-webkit-localizable-strings: Add the two icu directories
to the skip list.
- 14:49 Changeset [58776] by
-
Updated the localizable strings to run without errors.
Rubber-stamped by John Sullivan.
- English.lproj/Localizable.strings: Updated.
- StringsNotToBeLocalized.txt: Updated.
- 14:13 Changeset [58775] by
-
JavaInstanceJSC.cpp and JNIUtilityPrivate.cpp need to include jni_jsobject.h for jlong_to_pt() and ptr_to_jlong()
https://bugs.webkit.org/show_bug.cgi?id=38525
Reviewed by Darin Adler.
No new tests, build fix only.
- bridge/jni/jni_jsobject.h: Guard Mac-specific code with PLATFORM(MAC)
- bridge/jni/jsc/JNIUtilityPrivate.cpp: Include jni_jsobject.h
- bridge/jni/jsc/JavaInstanceJSC.cpp: Include jni_jsobject.h
- 14:02 Changeset [58774] by
-
2010-05-04 Steve Block <steveblock@google.com>
Reviewed by Darin Adler.
New FileSystemPOSIX functions cause linker errors on Android
https://bugs.webkit.org/show_bug.cgi?id=38521
No new tests, build fix only.
- platform/android/FileSystemAndroid.cpp: Remove closeFile, which is provided in FileSystemPOSIX.cpp
- platform/posix/FileSystemPOSIX.cpp: Guard writeToFile with !PLATFORM(ANDROID)
- 13:49 Changeset [58773] by
-
Remove redundant exports to fix Windows build warnings.
- JavaScriptCore.vcproj/JavaScriptCore/JavaScriptCore.def:
- 13:37 Changeset [58772] by
-
2010-05-04 Jesus Sanchez-Palencia <jesus@webkit.org>
Reviewed by Eric Seidel.
Wrong documentation on 'webkit-patch help land'.
https://bugs.webkit.org/show_bug.cgi?id=37871
Small fix on the help documentation for webkit-patch
land.
- Scripts/webkitpy/tool/commands/download.py:
- 13:16 Changeset [58771] by
-
Fix for <rdar://problem/7818509> Crash occurs when exiting Safari
Reviewed by Mike Thole.
We can avoid this crash if we call [self _removeFromAllWebViewsSet]
even in the case when we are doing a fastDocumentTeardown. This is
a much safer approach.
- WebView/WebView.mm:
(-[WebView _close]):
- 13:13 Changeset [58770] by
-
Unreviewed.
[Qt] http/tests/security/cross-frame-access-name-getter.html timed out
https://bugs.webkit.org/show_bug.cgi?id=38540
- platform/qt/Skipped: http/tests/security/cross-frame-access-name-getter.html skipped until bug fixed.
- 13:02 Changeset [58769] by
-
2010-05-04 Dirk Pranke <dpranke@chromium.org>
Unreviewed, expectations change.
Remove a bunch of test failures that are no longer failing (likely fixed in r58762).
- platform/chromium/test_expectations.txt:
- 11:49 Changeset [58768] by
-
2010-05-04 Pavel Feldman <pfeldman@chromium.org>
Reviewed by Timothy Hatcher.
Web Inspector: restore main and sidebar scroller positions
when switching between panels.
- inspector/front-end/ElementsPanel.js: (WebInspector.ElementsPanel.prototype.elementsToRestoreScrollPositionsFor):
- inspector/front-end/Panel.js: (WebInspector.Panel.prototype.show): (WebInspector.Panel.prototype.hide): (WebInspector.Panel.prototype.elementsToRestoreScrollPositionsFor): (WebInspector.Panel.prototype._storeScrollPositions): (WebInspector.Panel.prototype._restoreScrollPositions):
- inspector/front-end/ResourcesPanel.js: (WebInspector.ResourcesPanel.prototype.elementsToRestoreScrollPositionsFor):
- inspector/front-end/ScriptsPanel.js: (WebInspector.ScriptsPanel.prototype.elementsToRestoreScrollPositionsFor):
- 11:12 Changeset [58767] by
-
Unreviewed, build fix for Qt skipping new notifications tests.
- 10:42 Changeset [58766] by
-
2010-05-04 Steven Lai <steven_lai@asia.apple.com>
Reviewed by Darin Adler.
https://bugs.webkit.org/show_bug.cgi?id=37900
<rdar://problem/7806164> cloneNode() does not preserve z-index with more than six digits
- fast/dom/HTMLInputElement/clone-input-checked-expected.txt: Added.
- fast/dom/HTMLInputElement/clone-input-checked.html: Added.
- fast/dom/clone-node-z-index-expected.txt: Added.
- fast/dom/clone-node-z-index.html: Added.
2010-05-04 Steven Lai <steven_lai@asia.apple.com>
Reviewed by Darin Adler.
https://bugs.webkit.org/show_bug.cgi?id=37900
<rdar://problem/7806164> cloneNode() does not preserve z-index with more than six digits
Tests: fast/dom/HTMLInputElement/clone-input-checked.html
fast/dom/clone-node-z-index.html
- dom/Element.cpp: undid the order change of calling copyNonAttributeProperties() before setAttributes() (WebCore::Element::cloneElementWithoutChildren): uncheck the previous radio button in the same radio button group only when the checked radio box is appended to the dom tree
- html/HTMLInputElement.cpp: (WebCore::HTMLInputElement::updateCheckedRadioButtons): (WebCore::HTMLInputElement::attach): (WebCore::HTMLInputElement::setChecked):
- html/HTMLInputElement.h:
- 10:20 Changeset [58765] by
-
2010-04-29 John Gregg <johnnyg@google.com>
Reviewed by Dmitry Titov.
notifications should have dir and replaceId attributes
https://bugs.webkit.org/show_bug.cgi?id=38336
- public/WebNotification.h:
- src/WebNotification.cpp: (WebKit::WebNotification::dir): (WebKit::WebNotification::replaceId):
2010-04-29 John Gregg <johnnyg@google.com>
Reviewed by Dmitry Titov.
notifications should have dir and replaceId attributes.
Note that tests are only enabled in Chromium, skipped elsewhere
since notifications aren't implemented.
https://bugs.webkit.org/show_bug.cgi?id=38336
Tests: fast/notifications/notifications-replace.html
fast/notifications/notifications-rtl.html
- notifications/Notification.h: (WebCore::Notification::dir): (WebCore::Notification::setDir): (WebCore::Notification::replaceId): (WebCore::Notification::setReplaceId):
- notifications/Notification.idl:
2010-04-29 John Gregg <johnnyg@google.com>
Reviewed by Dmitry Titov.
notifications should have dir and replaceId attributes
Tests are disabled until the attributes are correctly handled
in Chromium test_shell.
https://bugs.webkit.org/show_bug.cgi?id=38336
- fast/notifications/notifications-replace-expected.txt: Added.
- fast/notifications/notifications-replace.html: Added.
- fast/notifications/notifications-rtl-expected.txt: Added.
- fast/notifications/notifications-rtl.html: Added.
- platform/chromium/test_expectations.txt:
- 09:31 Changeset [58764] by
-
2010-05-04 Laszlo Gombos <laszlo.1.gombos@nokia.com>
Reviewed by Kenneth Rohde Christiansen.
[Qt] QWebPage viewMode property
https://bugs.webkit.org/show_bug.cgi?id=38119
Rename the property from wrt_viewMode to _q_viewMode.
- Api/qwebpage.cpp: (QWebPagePrivate::dynamicPropertyChangeEvent):
- tests/qwebpage/tst_qwebpage.cpp: (tst_QWebPage::viewModes):
2010-05-04 Laszlo Gombos <laszlo.1.gombos@nokia.com>
Reviewed by Kenneth Rohde Christiansen.
[Qt] QWebPage viewMode property
https://bugs.webkit.org/show_bug.cgi?id=38119
Rename the property from wrt_viewMode to _q_viewMode.
- DumpRenderTree/qt/LayoutTestControllerQt.cpp: (LayoutTestController::setViewModeMediaFeature):
- 09:24 Changeset [58763] by
-
2010-05-04 Alejandro G. Castro <alex@igalia.com>
Reviewed by Xan Lopez.
Fixed error defining the float/double minimum in the object
properties, the smallest value of a float/double is
-G_MAXDOUBLE/-G_MAXFLOAT.
- bindings/scripts/CodeGeneratorGObject.pm:
- bindings/scripts/CodeGeneratorGObject.pm:
- 08:55 Changeset [58762] by
-
2010-05-03 Joseph Pecoraro <joepeck@webkit.org>
Reviewed by Dan Bernstein.
REGRESSION: Text clipped in absolutely positioned search inputs
https://bugs.webkit.org/show_bug.cgi?id=38468
Update the test to include a second search field and show
the clipping of a search input is like a text input.
- fast/css/input-search-padding.html:
- platform/mac/fast/css/input-search-padding-expected.checksum:
- platform/mac/fast/css/input-search-padding-expected.png:
- platform/mac/fast/css/input-search-padding-expected.txt:
2010-05-03 Joseph Pecoraro <joepeck@webkit.org>
Reviewed by Dan Bernstein.
REGRESSION: Text clipped in absolutely positioned search inputs
https://bugs.webkit.org/show_bug.cgi?id=38468
Previously I incorrectly added x() and y() to the tx and ty positioning.
This clips a search input like a text input, respecting the vertically
centered text.
- rendering/RenderTextControlSingleLine.cpp: (WebCore::RenderTextControlSingleLine::layout): only the Y changes, so change setLocation to setY (WebCore::RenderTextControlSingleLine::controlClipRect):
- 08:41 Changeset [58761] by
-
2010-05-04 Alexander Pavlov <apavlov@chromium.org>
Reviewed by Timothy Hatcher.
Web Inspector: REGRESSION: Up/Down, PgUp/PgDown keys do not change numeric style properties
https://bugs.webkit.org/show_bug.cgi?id=38516
- inspector/front-end/StylesSidebarPane.js: (WebInspector.StylePropertyTreeElement.prototype):
- 08:30 Changeset [58760] by
-
2010-05-04 Ben Murdoch <benm@google.com>
Reviewed by Simon Hausmann.
Crash in handleTouchEvent: using dangling node ptrs in hashmap
https://bugs.webkit.org/show_bug.cgi?id=38514
- fast/events/touch/resources/touch-stale-node-crash.js: Added.
- fast/events/touch/resources/send-touch-up.html: Added.
- fast/events/touch/touch-stale-node-crash-expected.txt: Added.
- fast/events/touch/touch-stale-node-crash.html: Added.
2010-05-04 Ben Murdoch <benm@google.com>
Reviewed by Simon Hausmann.
Crash in handleTouchEvent: using dangling node ptrs in hashmap
https://bugs.webkit.org/show_bug.cgi?id=38514
When navigating away from a page, if you have your finger still
pressed and then lift it on the new page we see a crash if the
node got deleted as we still have a dangling pointer in the
m_originatingTouchPointTargets hashmap and try to use it as the
receiver to dispatch a touchend event.
Test: fast/events/touch/touch-stale-node-crash.html
- page/EventHandler.cpp: (WebCore::EventHandler::clear): Clear the hashmap of touch targets.
- 07:54 QtWebKitPackaging edited by
- (diff)
- 07:53 QtWebKitPackaging edited by
- (diff)
- 07:52 QtWebKitPackaging edited by
- (diff)
- 07:29 Changeset [58759] by
-
2010-05-04 Jochen Eisinger <jochen@chromium.org>
Reviewed by Jeremy Orlow.
Fix typo in run_webkit_tests.py: s/_print\./_printer./
https://bugs.webkit.org/show_bug.cgi?id=38515
- Scripts/webkitpy/layout_tests/run_webkit_tests.py:
- 07:15 Changeset [58758] by
-
2010-05-04 Joseph Pecoraro <joepeck@webkit.org>
Reviewed by Pavel Feldman.
Web Inspector: Drawer Misbehaving when Docking / Undocking in Console Panel
https://bugs.webkit.org/show_bug.cgi?id=38510
- inspector/front-end/inspector.js: (WebInspector.set attached): resize the drawer after docking/undocking
- 07:04 Changeset [58757] by
-
2010-05-04 Jay Civelli <jcivelli@chromium.org>
Reviewed by David Levin.
[chromium] Make the select popup not close on right/middle clicks.
https://bugs.webkit.org/show_bug.cgi?id=38473
- src/WebViewImpl.cpp: (WebKit::WebViewImpl::mouseDown):
- 07:04 Changeset [58756] by
-
2010-05-04 Laszlo Gombos <laszlo.1.gombos@nokia.com>
Unreviewed, build fix for Symbian.
[Symbian] Symbian builds does not support shadow builds
Revision r54715 broke the Symbian build. For Symbian
the include directory is generated in the root of the source tree.
This patch sets the INCLUDEPATH accordingly for Symbian.
- WebKit.pri:
- 06:53 Changeset [58755] by
-
2010-05-04 Jedrzej Nowacki <jedrzej.nowacki@nokia.com>
Reviewed by Holger Freyther.
Fix a memory leak inside the QScriptValue.
QSciptValuePrivate::toString should release all temporary variables.
[Qt] QScriptValue::toString have a memory leak.
https://bugs.webkit.org/show_bug.cgi?id=38112
- qt/api/qscriptvalue_p.h: (QScriptValuePrivate::toString):
- 06:30 Changeset [58754] by
-
[Qt] Preserve binary compatibility with qtwebkit-2.0 branch
Rubber-stamped by Tor Arne Vestbø.
Moved the WebGL attribute in QWebSettings to the end of the enum,
to ensure that the numeric values of the enum values following it
are the same as in the release branch.
- Api/qwebsettings.h:
- 05:34 Changeset [58753] by
-
2010-05-04 Gustavo Noronha Silva <gustavo.noronha@collabora.co.uk>
Reviewed by Darin Adler.
Does not work on SPARC because of alignment issues
https://bugs.webkit.org/show_bug.cgi?id=38455
Use AllocAlignmentInteger instead of void* on pageheap_memory to
avoid crashing at runtime in some architectures. Investigated by
Mike Hommey <mh@glandium.org>, change suggested by Darin Adler.
- wtf/FastMalloc.cpp:
- 05:31 Changeset [58752] by
-
2010-05-04 Laszlo Gombos <laszlo.1.gombos@nokia.com>
Unreviewed, build fix for Symbian.
[Symbian] Build fix after r58598.
Use C99 integer types for the Symbian plugin
implementation.
No new tests, as there is no new functionality.
- plugins/symbian/PluginPackageSymbian.cpp: (WebCore::PluginPackage::NPVersion):
- plugins/symbian/PluginViewSymbian.cpp: (WebCore::PluginView::handlePostReadFile):
- 05:07 Changeset [58751] by
-
2010-05-04 Nikolas Zimmermann <nzimmermann@rim.com>
Reviewed by Dirk Schulze.
Split SVGCharacterLayoutInfo in smaller pieces
https://bugs.webkit.org/show_bug.cgi?id=38513
Split SVGCharacterLayoutInfo into SVGCharacterLayoutInfo/SVGCharacterData and SVGTextChunkLayoutInfo.
This is a preparation for more work in the text area.
- Android.mk: Add SVGCharacterData.(cpp|h) and SVGTextChunkLayoutInfo.h to build.
- GNUmakefile.am: Ditto.
- WebCore.gypi: Ditto.
- WebCore.pro: Ditto
- WebCore.vcproj/WebCore.vcproj: Ditto.
- WebCore.xcodeproj/project.pbxproj: Ditto.
- rendering/SVGCharacterData.cpp: Copied from rendering/SVGCharacterLayoutInfo.cpp.
- rendering/SVGCharacterData.h: Copied from rendering/SVGCharacterLayoutInfo.h. (WebCore::SVGChar::SVGChar):
- rendering/SVGCharacterLayoutInfo.cpp: (WebCore::SVGCharacterLayoutInfo::isInitialLayout): Introduced new helper function to share code between addLayoutInformation/addStackContent. (WebCore::SVGCharacterLayoutInfo::addLayoutInformation): Use new helper function. (WebCore::SVGCharacterLayoutInfo::addStackContent): Ditto
- rendering/SVGCharacterLayoutInfo.h:
- rendering/SVGRootInlineBox.h: Include new files.
- rendering/SVGTextChunkLayoutInfo.h: Copied from rendering/SVGCharacterLayoutInfo.h.
- 04:49 BuildingQtOnOSX edited by
- (diff)
- 04:48 Changeset [58750] by
-
Revert r58747 - postponing Skia roll, at which time
we'll need to add these expected failures back.
https://bugs.webkit.org/show_bug.cgi?id=38512
- platform/chromium/test_expectations.txt:
- 04:16 Changeset [58749] by
-
2010-05-04 Xan Lopez <xlopez@igalia.com>
Reviewed by Holger Freyther.
[GTK] GObject DOM bindings
https://bugs.webkit.org/show_bug.cgi?id=33590
Use helper functions from CodeGenerator.pm to figure out whether a
type is "fundamental" or not (basically whether it's anything
other than a string or a non-pointer type).
- bindings/scripts/CodeGeneratorGObject.pm:
- 03:25 Changeset [58748] by
-
2010-04-30 Alexander Pavlov <apavlov@chromium.org>
Reviewed by Yury Semikhatsky.
Web Inspector: REGRESSION: Disabled style properties are absent in Styles sidebar after WebInspector is re-opened
https://bugs.webkit.org/show_bug.cgi?id=38255
Moved stylesheet-related mappings into a separate object stored
in InspectorController rather than InspectorDOMAgent (which gets reset
on every frontend [dis]connect).
- GNUmakefile.am:
- WebCore.gypi:
- WebCore.pro:
- WebCore.vcproj/WebCore.vcproj:
- WebCore.xcodeproj/project.pbxproj:
- inspector/InspectorCSSStore.cpp: Added. (WebCore::InspectorCSSStore::InspectorCSSStore): (WebCore::InspectorCSSStore::~InspectorCSSStore): (WebCore::InspectorCSSStore::reset):
- inspector/InspectorCSSStore.h: Added.
- inspector/InspectorController.cpp: (WebCore::InspectorController::InspectorController): (WebCore::InspectorController::setFrontend): (WebCore::InspectorController::didCommitLoad):
- inspector/InspectorController.h:
- inspector/InspectorDOMAgent.cpp: (WebCore::InspectorDOMAgent::InspectorDOMAgent): (WebCore::InspectorDOMAgent::discardBindings): (WebCore::InspectorDOMAgent::applyStyleText): (WebCore::InspectorDOMAgent::setStyleText): (WebCore::InspectorDOMAgent::setStyleProperty): (WebCore::InspectorDOMAgent::toggleStyleEnabled): (WebCore::InspectorDOMAgent::setRuleSelector): (WebCore::InspectorDOMAgent::addRule): (WebCore::InspectorDOMAgent::bindStyle): (WebCore::InspectorDOMAgent::bindRule): (WebCore::InspectorDOMAgent::buildObjectForStyle): (WebCore::InspectorDOMAgent::buildObjectForRule):
- inspector/InspectorDOMAgent.h: (WebCore::InspectorDOMAgent::create): (WebCore::InspectorDOMAgent::cssStore):
- 02:55 Changeset [58747] by
-
2010-05-04 Jeremy Moskovich <jeremy@chromium.org>
Reviewed by Pavel Feldman.
Preparing for chromium skia roll incorporating changes up to r561.
Add expected failures so bots don't turn red while rolling.
- platform/chromium/test_expectations.txt:
- 02:36 Changeset [58746] by
-
Unreviewed fix to make Qt bot happier.
r58712 made fast/workers/worker-gc2.html crash. It seems to be a DRT sideeffect,
skipping worker-event-listener.html solve the problem until we find a real fix.
https://bugs.webkit.org/show_bug.cgi?id=38502
- platform/qt/Skipped: fast/workers/worker-event-listener.html skipped.
- 02:25 Changeset [58745] by
-
2010-05-04 Mikhail Naganov <mnaganov@chromium.org>
Reviewed by Pavel Feldman.
Display "Recording..." item when recording an user-initiated CPU profile.
- English.lproj/localizedStrings.js:
- inspector/front-end/ProfilesPanel.js: (WebInspector.ProfilesPanel.prototype.addProfileHeader): (WebInspector.ProfilesPanel.prototype.removeProfileHeader): (WebInspector.ProfilesPanel.prototype.showProfile):
- inspector/front-end/inspector.js: (WebInspector.setRecordingProfile):
- 02:15 Changeset [58744] by
-
2010-05-04 Tucker Jay <jay.tucker@nokia.com>
Reviewed by Holger Freyther.
Animated GIF images does not animate 10x as expected by default.
https://bugs.webkit.org/show_bug.cgi?id=36818
Added test case to existing manual test to test the
fixed functionality.
- manual-tests/qt/qt-10loop-anim.gif: Added.
- manual-tests/qt/qt-gif-test.html:
- platform/graphics/qt/ImageDecoderQt.cpp: (WebCore::ImageDecoderQt::repetitionCount):
- 01:27 Changeset [58743] by
-
2010-05-04 Dirk Schulze <krit@webkit.org>
Unreviewed sort of XCodes project file.
- WebCore.xcodeproj/project.pbxproj:
- 01:06 Changeset [58742] by
-
2010-05-04 Chris Jerdonek <cjerdonek@webkit.org>
Reviewed by Eric Seidel.
This revision completes the terminology change from "processor" to
"checker" for the CarriageReturnProcessor, CppProcessor,
PythonProcessor, and TextProcessor classes, etc.
The word "processor" currently has two uses in our code. This
revision renames the lower-level use to "checker" and preserves the
word "processor" for higher-level, more general uses. This
revision also makes whatever other name changes that logically
followed from this change.
- Scripts/check-webkit-style:
- Updated references to PatchChecker.
- Scripts/webkitpy/style/checker.py:
- Renamed the StyleCheckerConfiguration class to StyleProcessorConfiguration.
- Renamed the ProcessorDispatcher class to CheckerDispatcher, and made similar changes for the class's method names.
- Renamed the PatchChecker class to PatchReader.
- Updated the file as necessary to accommodate the other class renames in this patch.
- Scripts/webkitpy/style/checker_unittest.py:
- Updated the unit test code as necessary.
- Scripts/webkitpy/style/checkers/common.py:
- Renamed the CarriageReturnProcessor class to CarriageReturnChecker, and changed its process() method to check().
- Scripts/webkitpy/style/checkers/common_unittest.py:
- Updated the unit test code as necessary.
- Scripts/webkitpy/style/checkers/cpp.py:
- Renamed the CppProcessor class to CppChecker, and renamed its process() method to check().
- Scripts/webkitpy/style/checkers/cpp_unittest.py:
- Updated the unit test code as necessary.
- Scripts/webkitpy/style/checkers/python.py:
- Renamed the PythonProcessor class to PythonChecker, and renamed its process() method to check().
- Scripts/webkitpy/style/checkers/python_unittest.py:
- Updated the unit test code as necessary.
- Scripts/webkitpy/style/checkers/text.py:
- Renamed the TextProcessor class to TextChecker, and renamed its process() method to check().
- Scripts/webkitpy/style/checkers/text_unittest.py:
- Updated the unit test code as necessary.
- Scripts/webkitpy/style/error_handlers.py:
- Updated the code as necessary.
- Scripts/webkitpy/style/error_handlers_unittest.py:
- Updated the unit test code as necessary.
- Scripts/check-webkit-style:
- 01:00 Changeset [58741] by
-
2010-05-04 Chris Jerdonek <cjerdonek@webkit.org>
Reviewed by Eric Seidel.
Adjusted svn-apply and -unapply to accept git diffs generated
using the --no-prefix flag.
- Scripts/VCSUtils.pm:
- Loosened the regular expression for the "diff --git" line to match when the --no-prefix flag is used with "git diff".
- Also refactored the code parsing the first line so that the script exits with an error message if the first line cannot be parsed.
- Scripts/webkitperl/VCSUtils_unittest/parseGitDiffHeader.pl:
- Added a unit test case for the --no-prefix case.
- Scripts/VCSUtils.pm:
- 00:57 Changeset [58740] by
-
2010-05-04 Adam Barth <abarth@webkit.org>
Reviewed by Maciej Stachowiak.
Add first WTFURL file
https://bugs.webkit.org/show_bug.cgi?id=38470
This class represents a component of a URL, such as the scheme or path.
Essentially, the class is an index into a string buffer.
- wtf/url: Added.
- There was some discussion about where to put the URL library. There are a number of reasonable options, but putting the library in the WTF directory seems like a reasonable alternative. This location will make more sense once WTF is moved out of JavaScriptCore and into a top-level directory.
- wtf/url/src: Added.
- I've added this extra directory to the usual structure because we eventually want a "public" folder that defines the public interface to this library that clients outside of svn.webkit.org can rely upon, much like the various subfolders of the WebKit directory often define a public API.
- wtf/url/src/URLComponent.h: Added.
- I've added this somewhat trivial class to get the ball rolling.
(WTF::URLComponent::URLComponent):
(WTF::URLComponent::isValid):
(WTF::URLComponent::isNonempty):
(WTF::URLComponent::reset):
(WTF::URLComponent::operator==):
(WTF::URLComponent::begin):
(WTF::URLComponent::setBegin):
(WTF::URLComponent::length):
(WTF::URLComponent::setLength):
(WTF::URLComponent::end):
- wtf/url/wtfurl.gyp: Added.
- WTFURL will be an independently buildable library, hence the beginnings of a wtfurl.gyp file.
- wtf/url: Added.
- 00:55 Changeset [58739] by
-
2010-05-04 Chris Jerdonek <cjerdonek@webkit.org>
Reviewed by Daniel Bates.
Changed VCSUtils.pm's parseDiffHeader() to call the new
parseGitDiffHeader() method.
This revision makes more progress towards adding executable-bit
support to svn-apply and svn-unapply. It also makes more progress
towards refactoring the code into a more maintainable form.
- Scripts/VCSUtils.pm:
- Removed gitdiff2svndiff().
- Removed the Git-specific logic from parseDiffHeader() and renamed it parseSvnDiffHeader().
- Added a new parseDiffHeader() subroutine which calls parseSvnDiffHeader() or parseGitDiffHeader() depending on the format of the first header line.
- Scripts/webkitperl/VCSUtils_unittest/gitdiff2svndiff.pl: Removed.
- Scripts/webkitperl/VCSUtils_unittest/parseDiffHeader.pl:
- Removed most of the test cases since these cases are now covered by the unit tests for parseSvnDiffHeader() and parseGitDiffHeader().
- Scripts/webkitperl/VCSUtils_unittest/parseSvnDiffHeader.pl: Added.
- Copied the SVN unit tests from parseDiffHeader.pl and updated them as necessary.
- Scripts/VCSUtils.pm:
- 00:22 Changeset [58738] by
-
2010-05-04 Alejandro G. Castro <alex@igalia.com>
Unreviewed, skip test.
Skipping inspector/timeline-parse-html.html
https://bugs.webkit.org/show_bug.cgi?id=38509
- platform/gtk/Skipped:
- 00:03 Changeset [58737] by
-
2010-05-03 Eric Seidel <eric@webkit.org>
Reviewed by Adam Barth.
Builders should run the perl and python unit tests on every commit
https://bugs.webkit.org/show_bug.cgi?id=37976
The unit tests take a few seconds to run, so they should not
have any noticable effect on builder speed.
We're running the tests everywhere but Tiger as Tiger's
Python version is ancient.
I would have rather have detected the python version of the
slave but I couldn't find any API to do that, and I didn't want
to hack version detection into test-webkitpy (at least not yet).
- BuildSlaveSupport/build.webkit.org-config/master.cfg:
05/03/10:
- 23:52 Changeset [58736] by
-
2010-05-03 Steven Lai <steven_lai@asia.apple.com>
Reviewed by Brady Eidson.
Test hashchange() event is fired in asynchronous manner.
https://bugs.webkit.org/show_bug.cgi?id=36201
- fast/loader/hashchange-event-async-expected.txt: Added.
- fast/loader/hashchange-event-async.html: Added.
2010-05-03 Steven Lai <steven_lai@asia.apple.com>
Reviewed by Brady Eidson.
Reverted hashchange() event back to async.
(This change does not update HashChangeEvent to its new proposed interface)
https://bugs.webkit.org/show_bug.cgi?id=36201
rdar://problem/7780794
rdar://problem/7761278 (partial fix)
Tests: fast/loader/hashchange-event-async.html
- dom/Document.cpp: reverted hashchange() event back to async (WebCore::Document::enqueueHashchangeEvent):
- 23:14 Changeset [58735] by
-
2010-05-03 Chris Jerdonek <cjerdonek@webkit.org>
Reviewed by Adam Barth.
Fixed a recent REGRESSION that caused svn-apply and -unapply to
skip over changes to the first file in a diff if leading junk was
present (like in an e-mail diff) and if the --force option was used.
- Scripts/svn-apply:
- Removed the bit of code at the beginning of the patch() subroutine that checks for the "Index:" line at the beginning of a file diff (since the parsePatch() subroutine already checks this).
- Scripts/svn-unapply:
- Removed the bit of code at the beginning of the patch() subroutine that checks for the "Index:" line at the beginning of a file diff (since the parsePatch() subroutine already checks this).
- Scripts/svn-apply:
- 21:44 Changeset [58734] by
-
[Cairo,WX] Stop leaking a FontPlatformData.
Allocate the FontPlatformData on the stack and fix
a memory leak by doing so.
2010-05-03 Holger Hans Peter Freyther <zecke@selfish.org>
Rubber-stamped by Xan Lopez.
[Cairo,WX] Stop leaking a FontPlatformData.
https://bugs.webkit.org/show_bug.cgi?id=37500
Stephan Aßmus pointed out that the pango font backend
is leaking memory and fixed it. The WX font backend
and the Cairo/Fontconfig backend have the same snippet
of code and are leaking memory as well. This commit is
fixing that.
- platform/graphics/cairo/SimpleFontDataCairo.cpp: (WebCore::SimpleFontData::smallCapsFontData):
- platform/graphics/wx/SimpleFontDataWx.cpp: (WebCore::SimpleFontData::smallCapsFontData):
- 21:16 Changeset [58733] by
-
Unreviewed.
Adjusted the ChangeLog entry below for r58732 (bug 35804) to reflect
the fact that the change will not become active until the patch
for bug 38454 lands.
- 21:00 Changeset [58732] by
-
2010-05-03 Chris Jerdonek <cjerdonek@webkit.org>
Reviewed by Eric Seidel.
The svn-apply and svn-unapply scripts now display an instructive
error message if the --binary flag is left off the "git diff" command
for diffs containing binary file differences.
- Scripts/VCSUtils.pm:
- Adjusted parseDiffHeader() to exit with an appropriate error message if it encounters a line of the form "Binary files <path1> and <path2> differ".
- Scripts/VCSUtils.pm:
- 19:53 Changeset [58731] by
-
2010-05-03 Dirk Pranke <dpranke@chromium.org>
Unreviewed, build fix.
new-run-webkit-tests: r58728 broke the buildbot logic for parsing the
output of the log; specifying --verbose should basically be equivalent
to --print everything, but instead it was equivalent to not specifying
--print and getting the default set. Now, --verbose acts as if
--print everything was implicitly specified as the default (you can
still override it if you specify both; this is a somewhat debatable
call).
- Scripts/webkitpy/layout_tests/layout_package/printing.py:
- 19:39 Changeset [58730] by
-
https://bugs.webkit.org/show_bug.cgi?id=38368
<rdar://problem/7834433> REGRESSSION: 1.5% PLT regression due to 56028
(return memory quicker).
Instead of returning everything but the smallest spans spread out
the spans kept over several size lists.
Reviewed by Geoff Garen.
- wtf/FastMalloc.cpp:
(WTF::TCMalloc_PageHeap::scavenge):
- 19:26 Changeset [58729] by
-
2010-05-03 Dirk Pranke <dpranke@chromium.org>
Unreviewed, build fix.
new-run-webkit-tests: Fix minor precedence bug introduced in r58728 where we printed
"-\n" 78 times instead of "-" 78 times followed by a single "\n".
- Scripts/webkitpy/layout_tests/layout_package/printing.py:
- 18:59 Changeset [58728] by
-
2010-05-03 Dirk Pranke <dpranke@chromium.org>
Reviewed by Eric Seidel.
new-run-webkit-tests: refactor a large chunk of the printing/logging
code out of run-webkit-tests py (almost 300 lines out of 1900).
This change also renames --log to --print (to be slightly more
descriptive). I've also added better help messages for printing.
The new code has unit tests!
There is still more code to be moved, but this patch is big enough as
it is. Namely, still to move are the printing of the actual results
and the timing statistics, which should move another 300-400 lines
out of the file.
Notable changes to run_webkit_tests.py beyond code simply moving:
- MeteredStream is now hidden under the new printing.Printer class. All the references to self._meter now point to self._printer.
- All logging configuration is done in printing.configure_logging()
- Instead of using write() lambdas to control what is and isn't printed, we use separate methods on the printer object. This will make it easier to grep which print statements are printed under protection of each flag.
- The print_results flag I added a few revs back to suppress printing in the unit tests has been replaced with --print nothing.
- The ResultSummary class now stores the entire TestResult for each test, not just the actual result type.
- summarize_unexpected_results() got moved from a method on TestRunner to a standalone function. This should move into a separate file along with the ResultSummary class and the TestResult class
- The --trace option added recently has been replaced by '--print trace-everything' and '--print trace-unexpected'
- Scripts/new-run-webkit-tests:
- update to new entry points in run_webkit_tests.py
- Scripts/webkitpy/layout_tests/layout_package/metered_stream.py:
- fix a minor nit where we were printing an empty string where we didn't need to
- Scripts/webkitpy/layout_tests/layout_package/printing.py: Added.
- Scripts/webkitpy/layout_tests/layout_package/printing_unittest.py: Added
- Scripts/webkitpy/layout_tests/port/test.py:
- implement relative_test_filename() and expected_filename() so we can test printing unexpected results in a platform-neutral way
- Scripts/webkitpy/run_webkit_test.py:
- move a lot of the printing code into printing.py
- change the signatures of the exported entry points for easier unit testing
- Scripts/webkitpy/run_webkit_tests_unittest.py:
- update w/ changes to run_webkit_tests entry points.
- 18:55 Changeset [58727] by
-
2010-05-03 Laszlo Gombos <laszlo.1.gombos@nokia.com>
Reviewed by Simon Hausmann.
[Qt] Expose HTMLTokenizer yielding parameters
https://bugs.webkit.org/show_bug.cgi?id=37023
Enables to set TimeDelay and ChunkSize for
HTMLTokenizer.
- Api/qwebpage.cpp: (QWebPagePrivate::dynamicPropertyChangeEvent):
- 18:37 Changeset [58726] by
-
2010-05-03 Dirk Pranke <dpranke@chromium.org>
Unreviewed, expectations change.
Fix expectation for fast/forms/searchfield-heights.html - IMAGE+TEXT
on Linux, IMAGE on win and mac.
- platform/chromium/test_expectations.txt:
- 18:32 Changeset [58725] by
-
2010-05-03 Dirk Pranke <dpranke@chromium.org>
Unreviewed, expectations change.
Restore editing/pasteboard/paste-xml.xhtml as failing w/ a TEXT
failure (was failing prior to landing fix for 27751).
Mark editiing/selection/caret-rtl-2.html as failing w/ IMAGE; need
to investigate further.
Mark svg/text/kerning.svg as failing on mac w/ IMAGE as well as Linux.
- platform/chromium/test_expectations.txt:
- 18:08 Changeset [58724] by
-
2010-05-03 Abhishek Arya <inferno@chromium.org>
Reviewed by Dirk Pranke.
Roll back test expectations change, initially needed to land bug 27751.
- platform/chromium/test_expectations.txt:
- 17:45 Changeset [58723] by
-
Touch WebKit.idl to force a rebuild
- 17:42 Changeset [58722] by
-
2010-05-03 Dirk Pranke <dpranke@chromium.org>
Unreviewed, expectations change.
Roll in Chromium expectations from overrides file.
- platform/chromium/test_expectations.txt:
- 17:35 Changeset [58721] by
-
2010-05-03 Dirk Pranke <dpranke@chromium.org>
Unreviewed, expectations change.
Mark the SVG test failures as linux-only.
Mark fast/forms/searchfield-heights.html as IMAGE+TEXT, not IMAGE.
- platform/chromium/test_expectations.txt:
- 17:15 Changeset [58720] by
-
2010-05-03 Anders Carlsson <andersca@apple.com>
Reviewed by Jon Honeycutt.
[WebKit2] WKView should respond to WM_SHOWWINDOW messages
https://bugs.webkit.org/show_bug.cgi?id=38496
- UIProcess/win/WebView.cpp: (WebKit::WebView::wndProc): Add case for WM_SHOWWINDOW.
(WebKit::WebView::onShowWindowEvent):
Update the page visibility accordingly.
- UIProcess/win/WebView.h:
- 17:07 Changeset [58719] by
-
2010-05-03 Dirk Pranke <dpranke@chromium.org>
Unreviewed, expectations change.
And add one more Linux test (fast/forms/search-placeholder-value-changed.html)
from r58700. Plus, fix sorting of tests from that rev.
- platform/chromium/test_expectations.txt:
- 17:03 Changeset [58718] by
-
2010-05-03 James Robinson <jamesr@chromium.org>
Reviewed by Eric Seidel.
Clean up a few compiler warnings
https://bugs.webkit.org/show_bug.cgi?id=38073
- wtf/text/StringImpl.cpp: (WebCore::StringImpl::ascii):
2010-05-03 James Robinson <jamesr@chromium.org>
Reviewed by Eric Seidel.
Clean up a few compiler warnings
https://bugs.webkit.org/show_bug.cgi?id=38073
- html/TextMetrics.h: (WebCore::TextMetrics::width):
- rendering/style/StyleRareInheritedData.h:
- rendering/style/StyleRareNonInheritedData.h:
- 17:01 Changeset [58717] by
-
2010-05-03 Dirk Pranke <dpranke@chromium.org>
Unreviewed, expectations change.
Mark yet more mac-specific tests as failling from r58700.
- platform/chromium/test_expectations.txt:
- 16:48 Changeset [58716] by
-
2010-05-03 Dirk Pranke <dpranke@chromium.org>
Unreviewed, expectations change.
Mark more tests as failing from r58700 and change the sandbox test from SLOW to a
flaky PASS TIMEOUT.
- platform/chromium/test_expectations.txt:
- 16:43 Changeset [58715] by
-
- English.lproj/Localizable.strings: Regenerated.
- StringsNotToBeLocalized.txt: Updated for recent changes.
- 16:09 Changeset [58714] by
-
2010-05-03 Anders Carlsson <andersca@apple.com>
Reviewed by Dan Bernstein.
Get rid of PageClient::isPageVisible and pass visibility directly in setPageIsVisible
https://bugs.webkit.org/show_bug.cgi?id=38493
- UIProcess/API/mac/PageClientImpl.h:
- UIProcess/API/mac/PageClientImpl.mm: Remove isPageVisible.
- UIProcess/API/mac/WKView.mm: (isViewVisible): New function (moved here from PageClientImpl).
(-[WKView _updateVisibility]):
Call didChangeVisibility.
(-[WKView viewDidMoveToWindow]):
(-[WKView viewDidHide]):
(-[WKView viewDidUnhide]):
Call _updateVisibility.
- UIProcess/DrawingAreaProxy.h: Rename didChangeVisibility to setPageIsVisible and add an isVisible parameter.
- UIProcess/DrawingAreaProxyUpdateChunk.cpp: (WebKit::DrawingAreaProxyUpdateChunk::setPageIsVisible): Don't call WebPageProxy::isVisible.
- UIProcess/DrawingAreaProxyUpdateChunk.h:
- UIProcess/PageClient.h: Remove isPageVisible.
- UIProcess/WebPageProxy.cpp:
- UIProcess/WebPageProxy.h: Remove isVisible.
- 16:07 Changeset [58713] by
-
2010-05-03 Dirk Pranke <dpranke@chromium.org>
Unreviewed, expectations change.
Mark more editing/* tests from r58703 as failing. Also, (re-?) add the SVG tests again
as they do fail on some platforms.
- platform/chromium/test_expectations.txt:
- 16:03 Changeset [58712] by
-
Rolling out r58114 - this introduced memory leaks of
AtomicStrings then workers terminated.
Reviewed by NOBODY (reverting previous commit).
- GNUmakefile.am:
- JavaScriptCore.gypi:
- JavaScriptCore.vcproj/WTF/WTF.vcproj:
- JavaScriptCore.xcodeproj/project.pbxproj:
- jit/ThunkGenerators.cpp:
(JSC::ThunkHelpers::stringImplDataOffset):
- runtime/Identifier.cpp:
(JSC::IdentifierTable::~IdentifierTable):
(JSC::IdentifierTable::add):
(JSC::IdentifierCStringTranslator::hash):
(JSC::IdentifierCStringTranslator::equal):
(JSC::IdentifierCStringTranslator::translate):
(JSC::Identifier::add):
(JSC::IdentifierUCharBufferTranslator::hash):
(JSC::IdentifierUCharBufferTranslator::equal):
(JSC::IdentifierUCharBufferTranslator::translate):
(JSC::Identifier::addSlowCase):
- runtime/Identifier.h:
- runtime/JSGlobalData.cpp:
(JSC::JSGlobalData::JSGlobalData):
(JSC::JSGlobalData::~JSGlobalData):
- runtime/JSGlobalData.h:
- wtf/WTFThreadData.cpp:
(WTF::WTFThreadData::WTFThreadData):
(WTF::WTFThreadData::~WTFThreadData):
- wtf/WTFThreadData.h:
(JSC::IdentifierTable::remove):
(JSC::IdentifierTable::literalTable):
(WTF::WTFThreadData::atomicStringTable):
- wtf/text/AtomicString.cpp:
(WebCore::AtomicStringTable::create):
(WebCore::AtomicStringTable::table):
(WebCore::AtomicStringTable::destroy):
(WebCore::stringTable):
(WebCore::CStringTranslator::hash):
(WebCore::CStringTranslator::equal):
(WebCore::CStringTranslator::translate):
(WebCore::operator==):
(WebCore::AtomicString::add):
(WebCore::equal):
(WebCore::UCharBufferTranslator::hash):
(WebCore::UCharBufferTranslator::equal):
(WebCore::UCharBufferTranslator::translate):
(WebCore::HashAndCharactersTranslator::hash):
(WebCore::HashAndCharactersTranslator::equal):
(WebCore::HashAndCharactersTranslator::translate):
(WebCore::AtomicString::find):
(WebCore::AtomicString::remove):
- wtf/text/AtomicStringTable.h: Removed.
- wtf/text/StringImpl.cpp:
(WebCore::StringImpl::~StringImpl):
- wtf/text/StringImpl.h:
(WebCore::StringImpl::inTable):
(WebCore::StringImpl::setInTable):
(WebCore::equal):
- wtf/text/StringImplBase.h:
(WTF::StringImplBase::StringImplBase):
- 15:46 Changeset [58711] by
-
[Qt] fast/frames/flattening/frameset-flattening-subframesets.html fails intermittently on Qt bot
Fails caused by layoutTestController.display() in fast/frames/flattening/frameset-flattening-subframe-resize.html
https://bugs.webkit.org/show_bug.cgi?id=37334
- platform/qt/Skipped:
- fast/frames/flattening/frameset-flattening-subframe-resize.html skipped until fix
- fast/frames/flattening/frameset-flattening-subframesets.html unskipped, because it works.
- platform/qt/Skipped:
- 15:37 Changeset [58710] by
-
Add the ability to auto-generate callbacks to all code generators.
https://bugs.webkit.org/show_bug.cgi?id=38414
Reviewed by Adam Barth.
- bindings/scripts/CodeGeneratorJS.pm:
- bindings/scripts/CodeGeneratorV8.pm:
- bindings/scripts/test/JS/JSTestCallback.cpp: Added.
(WebCore::JSTestCallback::JSTestCallback):
(WebCore::JSTestCallback::~JSTestCallback):
(WebCore::JSTestCallback::callbackWithClass1Param):
(WebCore::JSTestCallback::callbackWithClass2Param):
- bindings/scripts/test/JS/JSTestCallback.h: Added.
(WebCore::JSTestCallback::create):
- bindings/scripts/test/V8/JSTestCallback.cpp: Added.
(WebCore::V8TestCallback::V8TestCallback):
(WebCore::V8TestCallback::~V8TestCallback):
(WebCore::V8TestCallback::callbackWithClass1Param):
(WebCore::V8TestCallback::callbackWithClass2Param):
- bindings/scripts/test/V8/V8TestCallback.h: Added.
(WebCore::V8TestCallback::create):
- bindings/scripts/test/TestCallback.idl: Added.
- 15:37 Changeset [58709] by
-
Reviewed by Kevin Ollivier.
[wx] Implement scheduleDispatchFunctionsOnMainThread for wx port.
https://bugs.webkit.org/show_bug.cgi?id=38480
- 15:34 Changeset [58708] by
-
Fix missing bug URL.
- 15:31 Changeset [58707] by
-
Reviewed by Kevin Ollivier.
[wx] Build and use Mac's ComplexTextController to support complex text in wx.
https://bugs.webkit.org/show_bug.cgi?id=38482
- 15:27 Changeset [58706] by
-
2010-05-03 Dirk Pranke <dpranke@chromium.org>
Unreviewed, expectations change.
Mark the editing/* tests from r58703 as temporarily expected to fail
until the other half of the two-sided patch can land.
- platform/chromium/test_expectations.txt:
- 15:18 Changeset [58705] by
-
2010-05-03 Oliver Hunt <oliver@apple.com>
Reviewed by Maciej Stachowiak.
Interpreter crashes due to incorrect refcounting of cached structures.
https://bugs.webkit.org/show_bug.cgi?id=38491
rdar://problem/7926160
Make sure we ref/deref structures used for cached custom property getters
- bytecode/CodeBlock.cpp: (JSC::CodeBlock::derefStructures): (JSC::CodeBlock::refStructures):
2010-05-03 Oliver Hunt <oliver@apple.com>
Reviewed by Maciej Stachowiak.
Interpreter crashes due to incorrect refcounting of cached structures.
https://bugs.webkit.org/show_bug.cgi?id=38491
Add test for cached structure chains used for custom getters.
- fast/js/pic/cached-named-property-getter.html:
- 14:51 Changeset [58704] by
-
2010-05-03 Dirk Pranke <dpranke@chromium.org>
Unreviewed, expectations change.
Fix duplicate expectation, remove the SVG tests added in r58700.
- platform/chromium/test_expectations.txt:
- 14:50 Changeset [58703] by
-
2010-05-03 Abhishek Arya <inferno@chromium.org>
Reviewed by Adam Barth.
Tests that javascript cannot access clipboard.
https://bugs.webkit.org/show_bug.cgi?id=27751
- editing/execCommand/clipboard-access-expected.txt: Added.
- editing/execCommand/clipboard-access.html: Added.
- editing/execCommand/script-tests/clipboard-access.js: Added. (enabled): (whenEnabled):
2010-05-03 Abhishek Arya <inferno@chromium.org>
Reviewed by Adam Barth.
Add support for controlling clipboard access from javascript.
Clipboard access from javascript is disabled by default.
https://bugs.webkit.org/show_bug.cgi?id=27751
Test: editing/execCommand/clipboard-access.html
- WebCore.base.exp:
- editing/EditorCommand.cpp: (WebCore::supportedCopyCut): (WebCore::supportedPaste): (WebCore::createCommandMap):
- page/Settings.cpp: (WebCore::Settings::Settings): (WebCore::Settings::setJavaScriptCanAccessClipboard):
- page/Settings.h: (WebCore::Settings::javaScriptCanAccessClipboard):
2010-05-03 Abhishek Arya <inferno@chromium.org>
Reviewed by Adam Barth.
Add support for controlling clipboard access from javascript.
Clipboard access from javascript is disabled by default.
https://bugs.webkit.org/show_bug.cgi?id=27751
- public/WebSettings.h:
- src/WebSettingsImpl.cpp: (WebKit::WebSettingsImpl::setJavaScriptCanAccessClipboard):
- src/WebSettingsImpl.h:
2010-05-03 Abhishek Arya <inferno@chromium.org>
Reviewed by Adam Barth.
Add support for controlling clipboard access from javascript.
Clipboard access from javascript is disabled by default.
https://bugs.webkit.org/show_bug.cgi?id=27751
- webkit/webkitwebsettings.cpp: (webkit_web_settings_class_init): (webkit_web_settings_set_property): (webkit_web_settings_get_property): (webkit_web_settings_copy):
- webkit/webkitwebview.cpp: (webkit_web_view_update_settings): (webkit_web_view_settings_notify):
2010-05-03 Abhishek Arya <inferno@chromium.org>
Reviewed by Adam Barth.
Add support for controlling clipboard access from javascript.
Clipboard access from javascript is disabled by default.
https://bugs.webkit.org/show_bug.cgi?id=27751
- WebView/WebPreferenceKeysPrivate.h:
- WebView/WebPreferences.mm: (+[WebPreferences initialize]): (-[WebPreferences javaScriptCanAccessClipboard]): (-[WebPreferences setJavaScriptCanAccessClipboard:]):
- WebView/WebPreferencesPrivate.h:
- WebView/WebView.mm: (-[WebView _preferencesChangedNotification:]):
2010-05-03 Abhishek Arya <inferno@chromium.org>
Reviewed by Adam Barth.
Add support for controlling clipboard access from javascript.
Clipboard access from javascript is disabled by default.
https://bugs.webkit.org/show_bug.cgi?id=27751
- Api/qwebsettings.cpp: (QWebSettingsPrivate::apply):
- Api/qwebsettings.h:
2010-05-03 Abhishek Arya <inferno@chromium.org>
Reviewed by Adam Barth.
Add support for controlling clipboard access from javascript.
Clipboard access from javascript is disabled by default.
https://bugs.webkit.org/show_bug.cgi?id=27751
- Interfaces/IWebPreferencesPrivate.idl:
- WebPreferenceKeysPrivate.h:
- WebPreferences.cpp: (WebPreferences::initializeDefaultSettings): (WebPreferences::javaScriptCanAccessClipboard): (WebPreferences::setJavaScriptCanAccessClipboard):
- WebPreferences.h:
- WebView.cpp: (WebView::notifyPreferencesChanged):
2010-05-03 Abhishek Arya <inferno@chromium.org>
Reviewed by Adam Barth.
Add support for controlling clipboard access from javascript.
Clipboard access from javascript is enabled in test framework.
https://bugs.webkit.org/show_bug.cgi?id=27751
- DumpRenderTree/LayoutTestController.cpp: (setJavaScriptCanAccessClipboardCallback): (LayoutTestController::staticFunctions):
- DumpRenderTree/LayoutTestController.h:
- DumpRenderTree/chromium/LayoutTestController.cpp: (LayoutTestController::LayoutTestController): (LayoutTestController::setJavaScriptCanAccessClipboard): (LayoutTestController::overridePreference):
- DumpRenderTree/chromium/LayoutTestController.h:
- DumpRenderTree/chromium/TestShell.cpp: (TestShell::resetWebSettings):
- DumpRenderTree/gtk/DumpRenderTree.cpp: (resetDefaultsToConsistentValues):
- DumpRenderTree/gtk/LayoutTestControllerGtk.cpp: (LayoutTestController::setJavaScriptCanAccessClipboard):
- DumpRenderTree/mac/DumpRenderTree.mm: (resetDefaultsToConsistentValues):
- DumpRenderTree/mac/LayoutTestControllerMac.mm: (LayoutTestController::setJavaScriptCanAccessClipboard):
- DumpRenderTree/qt/DumpRenderTreeQt.cpp: (WebCore::WebPage::WebPage): (WebCore::WebPage::resetSettings):
- DumpRenderTree/qt/LayoutTestControllerQt.cpp: (LayoutTestController::setJavaScriptCanAccessClipboard):
- DumpRenderTree/qt/LayoutTestControllerQt.h:
- DumpRenderTree/win/DumpRenderTree.cpp: (resetDefaultsToConsistentValues):
- DumpRenderTree/win/LayoutTestControllerWin.cpp: (LayoutTestController::setJavaScriptCanAccessClipboard):
- DumpRenderTree/wx/LayoutTestControllerWx.cpp: (LayoutTestController::setJavaScriptCanAccessClipboard):
- 14:43 Changeset [58702] by
-
2010-05-03 Dirk Pranke <dpranke@chromium.org>
Unreviewed, expectations change.
Mark a sandbox test as SLOW and change one of the fast/forms tests I
just changed from IMAGE+TEXT->IMAGE back to IMAGE.
- platform/chromium/test_expectations.txt:
- 14:34 Changeset [58701] by
-
2010-05-03 Anders Carlsson <andersca@apple.com>
Reviewed by Adam Roben.
Maui MiniBrowser: Add an option to show/hide the web view
https://bugs.webkit.org/show_bug.cgi?id=38486
- MiniBrowser/win/BrowserWindow.cpp: (BrowserWindow::createWindow): Set the background brush to something other than null.
(BrowserWindow::onCommand):
Show and hide the web view accordingly.
- MiniBrowser/win/MiniBrowser.rc:
- MiniBrowser/win/resource.h: Add new menu item.
- 14:15 Changeset [58700] by
-
2010-05-03 Dirk Pranke <dpranke@chromium.org>
Unreviewed, expectations change.
Change expectations for some of the fast/forms tests that just started
failing from IMAGE+TEXT to just IMAGE, and add some failing SVG tests
so that we can get the bots to green up.
- platform/chromium/test_expectations.txt:
- 13:45 Changeset [58699] by
-
2010-05-03 Kevin Watters <kevinwatters@gmail.com>
Reviewed by Kevin Ollivier.
Provide access to GrantUniversalAccess to allow enabling of XSS support.
- WebFrame.cpp: (wxWebFrame::GrantUniversalAccess):
- WebFrame.h:
- WebView.cpp: (wxWebView::GetParseMode): (wxWebView::GrantUniversalAccess):
- WebView.h:
- 13:36 Changeset [58698] by
-
2010-05-03 Yaar Schnitman <yaar@chromium.org>
Not Reviewed.
Preparing for chromium webkit roll 58626:TBD: Rebaselines
https://bugs.webkit.org/show_bug.cgi?id=38485
- platform/chromium-linux/svg/W3C-SVG-1.1/masking-mask-01-b-expected.checksum:
- platform/chromium-linux/svg/W3C-SVG-1.1/masking-mask-01-b-expected.png:
- platform/chromium-win/svg/W3C-SVG-1.1/masking-mask-01-b-expected.checksum:
- platform/chromium-win/svg/W3C-SVG-1.1/masking-mask-01-b-expected.png:
- platform/chromium/test_expectations.txt:
- 13:19 Changeset [58697] by
-
2010-05-03 Anders Carlsson <andersca@apple.com>
Reviewed by Adam Roben.
Implement PageClient::isPageVisible on Windows.
https://bugs.webkit.org/show_bug.cgi?id=38483
- UIProcess/PageClient.h:
- UIProcess/win/WebView.cpp: (WebKit::WebView::isPageVisible):
- UIProcess/win/WebView.h:
- 13:12 Changeset [58696] by
-
Fixed a typo in ChangeLog.
- 13:04 Changeset [58695] by
-
Reviewed by Adam Barth.
https://bugs.webkit.org/show_bug.cgi?id=38285
<rdar://problem/7903453> REGRESSION: Javascript command window.open does not work in empty tab
Cannot be tested, because new windows created in DRT always have an opener, and thus inherit
its security origin. Only new windows and tabs created by browser chrome had this problem.
- loader/FrameLoader.cpp: (WebCore::FrameLoader::init): Moved updateSandboxFlags() call to the beginning, so that an initial document would get correct flags.
LayoutTests:
These tests don't verify that this bug is fixed (that cannpt be tested automatically),
but test closely related behavior that hasn't been covered before.
- http/tests/security/resources/iframe-no-src.html: Added.
- http/tests/security/sandbox-inherit-to-initial-document-2-expected.txt: Added.
- http/tests/security/sandbox-inherit-to-initial-document-2.html: Added.
- http/tests/security/sandbox-inherit-to-initial-document-expected.txt: Added.
- http/tests/security/sandbox-inherit-to-initial-document.html: Added.
- 12:51 Changeset [58694] by
-
2010-05-03 Noam Rosenthal <noam.rosenthal@nokia.com>
Reviewed by Darin Adler.
WebGL compile issue.
Added ExceptionCode.h to JSWebGLArrayBufferConstructor.cpp, for some reason it was missing.
https://bugs.webkit.org/show_bug.cgi?id=38453
No new tests: compile fix.
- bindings/js/JSWebGLArrayBufferConstructor.cpp:
- 12:47 Changeset [58693] by
-
Fix Windows build.
- Platform/CoreIPC/win/ConnectionWin.cpp:
(CoreIPC::Connection::sendOutgoingMessage):
- Shared/win/UpdateChunk.cpp:
(WebKit::UpdateChunk::UpdateChunk):
(WebKit::UpdateChunk::encode):
(WebKit::UpdateChunk::decode):
- Shared/win/UpdateChunk.h:
(WebKit::UpdateChunk::rect):
- UIProcess/DrawingAreaProxy.h:
- UIProcess/DrawingAreaProxyUpdateChunk.h:
- UIProcess/win/DrawingAreaProxyUpdateChunkWin.cpp:
(WebKit::DrawingAreaProxyUpdateChunk::drawUpdateChunkIntoBackingStore):
- WebProcess/WebPage/win/DrawingAreaUpdateChunkWin.cpp:
(WebKit::DrawingAreaUpdateChunk::paintIntoUpdateChunk):
- WebProcess/win/WebProcessMain.cpp:
- 12:11 Changeset [58692] by
-
2010-05-03 Yaar Schnitman <yaar@chromium.org>
Not Reviewed.
Preparing for Chromium WebKit roll 58626:TBD
https://bugs.webkit.org/show_bug.cgi?id=38479
- platform/chromium/test_expectations.txt: added missing expectation
- 11:51 Changeset [58691] by
-
2010-05-03 Yaar Schnitman <yaar@chromium.org>
Not Reviewed.
Added missing chromium baselines for svg text kerning
https://bugs.webkit.org/show_bug.cgi?id=38476
- platform/chromium-win/svg/W3C-SVG-1.1/fonts-kern-01-t-expected.checksum:
- platform/chromium-win/svg/W3C-SVG-1.1/fonts-kern-01-t-expected.png:
- platform/chromium-win/svg/W3C-SVG-1.1/fonts-kern-01-t-expected.txt:
- platform/chromium-win/svg/W3C-SVG-1.1/render-groups-01-b-expected.checksum:
- platform/chromium-win/svg/W3C-SVG-1.1/render-groups-01-b-expected.png: Added.
- platform/chromium-win/svg/W3C-SVG-1.1/render-groups-03-t-expected.checksum:
- platform/chromium-win/svg/W3C-SVG-1.1/render-groups-03-t-expected.png: Added.
- platform/chromium-win/svg/text/kerning-expected.checksum:
- platform/chromium-win/svg/text/kerning-expected.png:
- platform/chromium-win/svg/text/text-hkern-expected.checksum: Added.
- platform/chromium-win/svg/text/text-hkern-expected.png: Added.
- platform/chromium/test_expectations.txt:
- 11:49 Changeset [58690] by
-
2010-05-03 Eric Seidel <eric@webkit.org>
Unreviewed, rolling out r58685.
http://trac.webkit.org/changeset/58685
https://bugs.webkit.org/show_bug.cgi?id=38461
Broke a test on Gtk
- platform/graphics/gstreamer/MediaPlayerPrivateGStreamer.cpp: (WebCore::MediaPlayerPrivateGStreamer::updateStates):
- 11:23 Changeset [58689] by
-
2010-05-03 Yaar Schnitman <yaar@chromium.org>
Not Reviewed.
Preparing for chromium webkit roll 58626:TBD
https://bugs.webkit.org/show_bug.cgi?id=38474
- platform/chromium/test_expectations.txt: Added temporary expectations.
- 11:19 Changeset [58688] by
-
2010-04-30 Anders Carlsson <andersca@apple.com>
Reviewed by Sam Weinig.
https://bugs.webkit.org/show_bug.cgi?id=38415
Have the WKView notify the DrawingAreaProxy when its visibility changes.
- UIProcess/API/mac/PageClientImpl.h:
- UIProcess/API/mac/PageClientImpl.mm: (WebKit::PageClientImpl::isPageVisible):
- UIProcess/API/mac/WKView.mm: (-[WKView viewDidMoveToWindow]): (-[WKView viewDidHide]): (-[WKView viewDidUnhide]):
- UIProcess/DrawingAreaProxy.h:
- UIProcess/DrawingAreaProxyUpdateChunk.cpp: (WebKit::DrawingAreaProxyUpdateChunk::DrawingAreaProxyUpdateChunk): (WebKit::DrawingAreaProxyUpdateChunk::didChangeVisibility):
- UIProcess/DrawingAreaProxyUpdateChunk.h:
- UIProcess/PageClient.h:
- UIProcess/WebPageProxy.cpp: (WebKit::WebPageProxy::isVisible):
- UIProcess/WebPageProxy.h:
- 11:16 Changeset [58687] by
-
Use HTML5 number parsing in HTMLProgressElement
https://bugs.webkit.org/show_bug.cgi?id=38434
Reviewed by Darin Adler.
WebCore:
Use parseToDoubleForNumberType instead of toDouble.
Throw an exception when the number is NaN or Infinity.
- html/HTMLProgressElement.cpp:
(WebCore::HTMLProgressElement::value):
(WebCore::HTMLProgressElement::setValue):
(WebCore::HTMLProgressElement::max):
(WebCore::HTMLProgressElement::setMax):
- html/HTMLProgressElement.h:
- html/HTMLProgressElement.idl:
LayoutTests:
- fast/dom/HTMLProgressElement/script-tests/set-progress-properties.js:
- fast/dom/HTMLProgressElement/set-progress-properties-expected.txt:
- 11:12 Changeset [58686] by
-
2010-05-03 Jens Alfke <snej@chromium.org>
Reviewed by Darin Fisher.
[chromium] Add "willSendSubmitEvent" hook to WebFrameClient and FrameLoaderClient
https://bugs.webkit.org/show_bug.cgi?id=38397
No tests (functionality is exposed only through native WebKit API.)
- html/HTMLFormElement.cpp: (WebCore::HTMLFormElement::prepareSubmit): Call frame loader's dispatchWillSendSubmitEvent
- loader/EmptyClients.h:
- loader/FrameLoaderClient.h: (WebCore::FrameLoaderClient::dispatchWillSendSubmitEvent): New empty method
2010-05-03 Jens Alfke <snej@chromium.org>
Reviewed by Darin Fisher.
[chromium] Add "willSendSubmitEvent" hook to WebFrameClient and FrameLoaderClient
https://bugs.webkit.org/show_bug.cgi?id=38397
No tests (functionality is exposed only through native WebKit API.)
- efl/WebCoreSupport/FrameLoaderClientEfl.h: (WebCore::FrameLoaderClientEfl::dispatchWillSendSubmitEvent):
2010-05-03 Jens Alfke <snej@chromium.org>
Reviewed by Darin Fisher.
[chromium] Add "willSendSubmitEvent" hook to WebFrameClient and FrameLoaderClient
https://bugs.webkit.org/show_bug.cgi?id=38397
No tests (functionality is exposed only through native WebKit API.)
- public/WebFrameClient.h: (WebKit::WebFrameClient::willSendSubmitEvent): New empty method
- src/FrameLoaderClientImpl.cpp: (WebKit::FrameLoaderClientImpl::dispatchWillSendSubmitEvent): Delegate to client
- src/FrameLoaderClientImpl.h:
2010-05-03 Jens Alfke <snej@chromium.org>
Reviewed by Darin Fisher.
[chromium] Add "willSendSubmitEvent" hook to WebFrameClient and FrameLoaderClient
https://bugs.webkit.org/show_bug.cgi?id=38397
No tests (functionality is exposed only through native WebKit API.)
- WebCoreSupport/FrameLoaderClientGtk.h: (WebKit::FrameLoaderClient::dispatchWillSendSubmitEvent):
2010-05-03 Jens Alfke <snej@chromium.org>
Reviewed by Darin Fisher.
[chromium] Add "willSendSubmitEvent" hook to WebFrameClient and FrameLoaderClient
https://bugs.webkit.org/show_bug.cgi?id=38397
No tests (functionality is exposed only through native WebKit API.)
- WebCoreSupport/FrameLoaderClientHaiku.h: (WebCore::FrameLoaderClientHaiku::dispatchWillSendSubmitEvent):
2010-05-03 Jens Alfke <snej@chromium.org>
Reviewed by Darin Fisher.
[chromium] Add "willSendSubmitEvent" hook to WebFrameClient and FrameLoaderClient
https://bugs.webkit.org/show_bug.cgi?id=38397
No tests (functionality is exposed only through native WebKit API.)
- WebCoreSupport/WebFrameLoaderClient.h: (WebFrameLoaderClient::dispatchWillSendSubmitEvent):
2010-05-03 Jens Alfke <snej@chromium.org>
Reviewed by Darin Fisher.
[chromium] Add "willSendSubmitEvent" hook to WebFrameClient and FrameLoaderClient
https://bugs.webkit.org/show_bug.cgi?id=38397
No tests (functionality is exposed only through native WebKit API.)
- WebCoreSupport/FrameLoaderClientQt.h: (WebCore::FrameLoaderClientQt::dispatchWillSendSubmitEvent):
2010-05-03 Jens Alfke <snej@chromium.org>
Reviewed by Darin Fisher.
[chromium] Add "willSendSubmitEvent" hook to WebFrameClient and FrameLoaderClient
https://bugs.webkit.org/show_bug.cgi?id=38397
No tests (functionality is exposed only through native WebKit API.)
- WebFrame.h:
2010-05-03 Jens Alfke <snej@chromium.org>
Reviewed by Darin Fisher.
[chromium] Add "willSendSubmitEvent" hook to WebFrameClient and FrameLoaderClient
https://bugs.webkit.org/show_bug.cgi?id=38397
No tests (functionality is exposed only through native WebKit API.)
- WebKitSupport/FrameLoaderClientWx.h: (WebCore::FrameLoaderClientWx::dispatchWillSendSubmitEvent):
2010-05-03 Jens Alfke <snej@chromium.org>
Reviewed by Darin Fisher.
[chromium] Add "willSendSubmitEvent" hook to WebFrameClient and FrameLoaderClient
https://bugs.webkit.org/show_bug.cgi?id=38397
No tests (functionality is exposed only through native WebKit API.)
- WebProcess/WebCoreSupport/WebFrameLoaderClient.h: (WebKit::WebFrameLoaderClient::dispatchWillSendSubmitEvent):
- 10:58 Changeset [58685] by
-
2010-05-03 Philippe Normand <pnormand@igalia.com>
Reviewed by Eric Carlson.
[GStreamer] forgotten call to durationChanged in updateStates()
https://bugs.webkit.org/show_bug.cgi?id=38461
Notify MediaPlayer if duration is known after playback started.
- platform/graphics/gstreamer/MediaPlayerPrivateGStreamer.cpp: (WebCore::MediaPlayerPrivateGStreamer::updateStates):
- 10:28 Changeset [58684] by
-
Fix for https://bugs.webkit.org/show_bug.cgi?id=38471
Add generic callback mechanism
Reviewed by Anders Carlsson.
Added GenericCallback class replacing RenderTreeExternalRepresentationCallback
and ScriptReturnValueCallback.
Also,
- Standardize C API callbacks to take the context last.
- Standardize C API callbacks to not have the _f suffix (now the block variants have a _b suffix).
- Re-write toWK and toRef methods as a set of template functions using the generic API->implementation mapping information.
- UIProcess/API/C/WKAPICast.h:
- UIProcess/API/C/WKPage.cpp:
(WKPageRunJavaScriptInMainFrame):
(callRunJavaScriptBlockAndRelease):
(disposeRunJavaScriptBlock):
(WKPageRunJavaScriptInMainFrame_b):
(WKPageRenderTreeExternalRepresentation):
(WKPageRenderTreeExternalRepresentation_b):
- UIProcess/API/C/WKPage.h:
- UIProcess/API/C/WKPagePrivate.h:
- UIProcess/GenericCallback.h: Added.
(WebKit::GenericCallback::create):
(WebKit::GenericCallback::~GenericCallback):
(WebKit::GenericCallback::performCallbackWithReturnValue):
(WebKit::GenericCallback::invalidate):
(WebKit::GenericCallback::callbackID):
(WebKit::GenericCallback::generateCallbackID):
(WebKit::GenericCallback::GenericCallback):
- UIProcess/RenderTreeExternalRepresentationCallback.cpp: Removed.
- UIProcess/RenderTreeExternalRepresentationCallback.h: Removed.
- UIProcess/ScriptReturnValueCallback.cpp: Removed.
- UIProcess/ScriptReturnValueCallback.h: Removed.
- UIProcess/WebPageProxy.cpp:
(WebKit::WebPageProxy::didRunJavaScriptInMainFrame):
(WebKit::WebPageProxy::didGetRenderTreeExternalRepresentation):
- UIProcess/WebPageProxy.h:
- WebKit2.xcodeproj/project.pbxproj:
- win/WebKit2.vcproj:
- 10:25 Changeset [58683] by
-
Remove URLCore top-level directory. The diff wasn't clear where this folder was going and Darin Adler would prefer that we not put this code in a top-level directory.
- 10:24 Changeset [58682] by
-
2010-05-03 Ryuan Choi <ryuan.choi@gmail.com>
Reviewed by Darin Adler.
fixing build break due to clearWatch() when Geolocation feature is
disabled.
no test because this is a build fix only
- page/Geolocation.cpp: (WebCore::Geolocation::clearWatch):
- 10:09 Changeset [58681] by
-
2010-05-03 Chris Jerdonek <cjerdonek@webkit.org>
Reviewed by Daniel Bates.
Added a parseGitDiffHeader() subroutine to VCSUtils.pm that
parses any changes to the executable bit in a Git diff.
This revision is more preparation towards adding "executable bit"
support to svn-apply and svn-unapply. No code is going "live" in
this change except for the new unit tests in test-webkitperl.
- Scripts/VCSUtils.pm:
- Added isExecutable() to determine whether a file mode has the executable bit set or not.
- Added parseGitDiffHeader() to parse the header of a Git diff.
- Scripts/webkitperl/VCSUtils_unittest/parseGitDiffHeader.pl: Added.
- Added unit tests for parseGitDiffHeader().
- Scripts/VCSUtils.pm:
- 10:00 Changeset [58680] by
-
Add empty folder to house URL parser
https://bugs.webkit.org/show_bug.cgi?id=38470
Reviewed by Darin Adler.
Add URLCore directory.
- 09:56 Changeset [58679] by
-
2010-05-03 Stephan Aßmus <superstippi@gmx.de>
Reviewed by Holger Freyther.
[Gtk] Fix leaking the FontPlatformData instance used to create the the
small caps font data.
https://bugs.webkit.org/show_bug.cgi?id=37500
No new tests needed.
- platform/graphics/gtk/SimpleFontDataPango.cpp:
(WebCore::SimpleFontData::smallCapsFontData):
- Use a stack allocated FontPlatformData instead of a heap allocated one that is never freed.
- platform/graphics/gtk/SimpleFontDataPango.cpp:
(WebCore::SimpleFontData::smallCapsFontData):
- 09:37 Changeset [58678] by
-
2010-05-03 Jarkko Sakkinen <jarkko.j.sakkinen@gmail.com>
Reviewed by Simon Hausmann.
[Qt] GraphicsLayer: support webGL
https://bugs.webkit.org/show_bug.cgi?id=35388
Added support GraphicsContext3D to GraphicsLayer.
Added paint method to GraphicsContext3D for Qt platform that
uses drawTexture() when QGLWidget is used as viewport of
QGraphicsWebView.
Fine-tuned texture and handling and image to texture conversion to
work also when drawTexture() blitting is used.
- platform/graphics/GraphicsContext3D.h:
- platform/graphics/qt/GraphicsContext3DQt.cpp: (WebCore::GraphicsContext3DInternal::GraphicsContext3DInternal): (WebCore::GraphicsContext3D::beginPaint): (WebCore::GraphicsContext3D::paint): (WebCore::GraphicsContext3D::texImage2D): (WebCore::GraphicsContext3D::texSubImage2D): (WebCore::GraphicsContext3D::getImageData):
- platform/graphics/qt/GraphicsLayerQt.cpp: (WebCore::GraphicsLayerQtImpl::): (WebCore::GraphicsLayerQtImpl::GraphicsLayerQtImpl): (WebCore::GraphicsLayerQtImpl::paint): (WebCore::GraphicsLayerQtImpl::flushChanges): (WebCore::GraphicsLayerQt::setContentsToGraphicsContext3D): (WebCore::GraphicsLayerQt::setGraphicsContext3DNeedsDisplay):
- platform/graphics/qt/GraphicsLayerQt.h:
- 09:20 Changeset [58677] by
-
2010-05-03 Adam Barth <abarth@webkit.org>
Reviewed by Dimitri Glazkov.
Add some more Chromium-specific files to gitignore
https://bugs.webkit.org/show_bug.cgi?id=38469
These files are generated as part of the update-webkit --chromium
script.
- .gitignore:
- 08:59 Changeset [58676] by
-
Unreviewed. Qt specific expected files added after r58675.
- platform/qt/editing/pasteboard/4944770-2-expected.txt: Added.
- platform/qt/editing/pasteboard/5028447-expected.txt: Added.
- platform/qt/editing/pasteboard/5089327-expected.txt: Added.
- platform/qt/editing/pasteboard/5601583-1-expected.txt: Added.
- platform/qt/editing/pasteboard/paste-blockquote-after-blockquote-expected.txt: Added.
- platform/qt/editing/pasteboard/paste-blockquote-into-blockquote-2-expected.txt: Added.
- platform/qt/editing/pasteboard/paste-blockquote-into-blockquote-3-expected.txt: Added.
- platform/qt/editing/pasteboard/paste-blockquote-into-blockquote-4-expected.txt: Added.
- platform/qt/editing/pasteboard/paste-blockquote-into-blockquote-expected.txt: Added.
- platform/qt/editing/pasteboard/paste-text-013-expected.txt: Added.
- platform/qt/editing/pasteboard/select-element-1-expected.txt: Added.
- platform/qt/editing/pasteboard/testcase-9507-expected.txt: Added.
- platform/qt/editing/pasteboard/undoable-fragment-removes-expected.txt: Added.
- platform/qt/editing/pasteboard/unrendered-br-expected.txt: Added.
- 08:39 Changeset [58675] by
-
2010-05-03 Csaba Osztrogonác <ossy@webkit.org>
Unreviewed. Qt specific expected files added and updated.
[Qt] Unskip editing/pasteboard tests that fail due to 'x,y' differences
https://bugs.webkit.org/show_bug.cgi?id=38435
- platform/qt/editing/pasteboard/3976872-expected.txt:
- platform/qt/editing/pasteboard/4076267-3-expected.txt:
- platform/qt/editing/pasteboard/4242293-expected.txt:
- platform/qt/editing/pasteboard/4631972-expected.txt:
- platform/qt/editing/pasteboard/4641033-expected.txt:
- platform/qt/editing/pasteboard/4700297-expected.txt:
- platform/qt/editing/pasteboard/4806874-expected.txt:
- platform/qt/editing/pasteboard/4840662-expected.txt:
- platform/qt/editing/pasteboard/4944770-1-expected.txt:
- platform/qt/editing/pasteboard/4944770-2-expected.txt: Added.
- platform/qt/editing/pasteboard/4989774-expected.txt:
- platform/qt/editing/pasteboard/5006779-expected.txt:
- platform/qt/editing/pasteboard/5027857-expected.txt:
- platform/qt/editing/pasteboard/5028447-expected.txt: Added.
- platform/qt/editing/pasteboard/5032095-expected.txt:
- platform/qt/editing/pasteboard/5065605-expected.txt:
- platform/qt/editing/pasteboard/5071074-2-expected.txt:
- platform/qt/editing/pasteboard/5071074-expected.txt:
- platform/qt/editing/pasteboard/5075944-2-expected.txt:
- platform/qt/editing/pasteboard/5075944-3-expected.txt:
- platform/qt/editing/pasteboard/5075944-expected.txt:
- platform/qt/editing/pasteboard/5089327-expected.txt: Added.
- platform/qt/editing/pasteboard/5134759-expected.txt:
- platform/qt/editing/pasteboard/5156401-1-expected.txt:
- platform/qt/editing/pasteboard/5245519-expected.txt:
- platform/qt/editing/pasteboard/5247341-expected.txt:
- platform/qt/editing/pasteboard/5368833-expected.txt:
- platform/qt/editing/pasteboard/5478250-expected.txt:
- platform/qt/editing/pasteboard/5483567-expected.txt:
- platform/qt/editing/pasteboard/5601583-1-expected.txt: Added.
- platform/qt/editing/pasteboard/5780697-2-expected.txt:
- platform/qt/editing/pasteboard/7955-expected.txt:
- platform/qt/editing/pasteboard/8145-1-expected.txt:
- platform/qt/editing/pasteboard/8145-2-expected.txt:
- platform/qt/editing/pasteboard/8145-3-expected.txt:
- platform/qt/editing/pasteboard/copy-paste-bidi-expected.txt:
- platform/qt/editing/pasteboard/cut-text-001-expected.txt:
- platform/qt/editing/pasteboard/displaced-generic-placeholder-expected.txt:
- platform/qt/editing/pasteboard/displaced-placeholder-expected.txt:
- platform/qt/editing/pasteboard/display-block-on-spans-expected.txt:
- platform/qt/editing/pasteboard/emacs-cntl-y-001-expected.txt:
- platform/qt/editing/pasteboard/innerText-inline-table-expected.txt:
- platform/qt/editing/pasteboard/interchange-newline-1-expected.txt:
- platform/qt/editing/pasteboard/interchange-newline-3-expected.txt:
- platform/qt/editing/pasteboard/interchange-newline-4-expected.txt:
- platform/qt/editing/pasteboard/merge-after-delete-1-expected.txt:
- platform/qt/editing/pasteboard/merge-after-delete-2-expected.txt:
- platform/qt/editing/pasteboard/merge-after-delete-expected.txt:
- platform/qt/editing/pasteboard/merge-end-1-expected.txt:
- platform/qt/editing/pasteboard/merge-end-2-expected.txt:
- platform/qt/editing/pasteboard/merge-end-3-expected.txt:
- platform/qt/editing/pasteboard/merge-end-4-expected.txt:
- platform/qt/editing/pasteboard/merge-end-5-expected.txt:
- platform/qt/editing/pasteboard/merge-end-blockquote-expected.txt:
- platform/qt/editing/pasteboard/merge-end-borders-expected.txt:
- platform/qt/editing/pasteboard/merge-end-list-expected.txt:
- platform/qt/editing/pasteboard/merge-end-table-expected.txt:
- platform/qt/editing/pasteboard/merge-start-blockquote-expected.txt:
- platform/qt/editing/pasteboard/merge-start-list-expected.txt:
- platform/qt/editing/pasteboard/paste-4035648-fix-expected.txt:
- platform/qt/editing/pasteboard/paste-4038267-fix-expected.txt:
- platform/qt/editing/pasteboard/paste-4039777-fix-expected.txt:
- platform/qt/editing/pasteboard/paste-blockquote-1-expected.txt:
- platform/qt/editing/pasteboard/paste-blockquote-2-expected.txt:
- platform/qt/editing/pasteboard/paste-blockquote-3-expected.txt:
- platform/qt/editing/pasteboard/paste-blockquote-after-blockquote-expected.txt: Added.
- platform/qt/editing/pasteboard/paste-blockquote-into-blockquote-2-expected.txt: Added.
- platform/qt/editing/pasteboard/paste-blockquote-into-blockquote-3-expected.txt: Added.
- platform/qt/editing/pasteboard/paste-blockquote-into-blockquote-4-expected.txt: Added.
- platform/qt/editing/pasteboard/paste-blockquote-into-blockquote-expected.txt: Added.
- platform/qt/editing/pasteboard/paste-line-endings-001-expected.txt:
- platform/qt/editing/pasteboard/paste-line-endings-002-expected.txt:
- platform/qt/editing/pasteboard/paste-line-endings-003-expected.txt:
- platform/qt/editing/pasteboard/paste-line-endings-004-expected.txt:
- platform/qt/editing/pasteboard/paste-line-endings-005-expected.txt:
- platform/qt/editing/pasteboard/paste-line-endings-006-expected.txt:
- platform/qt/editing/pasteboard/paste-line-endings-007-expected.txt:
- platform/qt/editing/pasteboard/paste-line-endings-008-expected.txt:
- platform/qt/editing/pasteboard/paste-line-endings-009-expected.txt:
- platform/qt/editing/pasteboard/paste-line-endings-010-expected.txt:
- platform/qt/editing/pasteboard/paste-list-001-expected.txt:
- platform/qt/editing/pasteboard/paste-match-style-001-expected.txt:
- platform/qt/editing/pasteboard/paste-match-style-002-expected.txt:
- platform/qt/editing/pasteboard/paste-pre-001-expected.txt:
- platform/qt/editing/pasteboard/paste-pre-002-expected.txt:
- platform/qt/editing/pasteboard/paste-table-001-expected.txt:
- platform/qt/editing/pasteboard/paste-table-003-expected.txt:
- platform/qt/editing/pasteboard/paste-table-cells-expected.txt:
- platform/qt/editing/pasteboard/paste-text-002-expected.txt:
- platform/qt/editing/pasteboard/paste-text-003-expected.txt:
- platform/qt/editing/pasteboard/paste-text-004-expected.txt:
- platform/qt/editing/pasteboard/paste-text-005-expected.txt:
- platform/qt/editing/pasteboard/paste-text-006-expected.txt:
- platform/qt/editing/pasteboard/paste-text-007-expected.txt:
- platform/qt/editing/pasteboard/paste-text-009-expected.txt:
- platform/qt/editing/pasteboard/paste-text-010-expected.txt:
- platform/qt/editing/pasteboard/paste-text-011-expected.txt:
- platform/qt/editing/pasteboard/paste-text-012-expected.txt:
- platform/qt/editing/pasteboard/paste-text-013-expected.txt: Added.
- platform/qt/editing/pasteboard/paste-text-014-expected.txt:
- platform/qt/editing/pasteboard/paste-text-015-expected.txt:
- platform/qt/editing/pasteboard/paste-text-016-expected.txt:
- platform/qt/editing/pasteboard/paste-text-017-expected.txt:
- platform/qt/editing/pasteboard/paste-text-018-expected.txt:
- platform/qt/editing/pasteboard/paste-text-019-expected.txt:
- platform/qt/editing/pasteboard/paste-text-at-tabspan-001-expected.txt:
- platform/qt/editing/pasteboard/paste-text-at-tabspan-002-expected.txt:
- platform/qt/editing/pasteboard/paste-text-at-tabspan-003-expected.txt:
- platform/qt/editing/pasteboard/paste-unrendered-select-expected.txt:
- platform/qt/editing/pasteboard/pasting-object-expected.txt:
- platform/qt/editing/pasteboard/prevent-block-nesting-01-expected.txt:
- platform/qt/editing/pasteboard/quirks-mode-br-1-expected.txt:
- platform/qt/editing/pasteboard/select-element-1-expected.txt: Added.
- platform/qt/editing/pasteboard/styled-element-markup-expected.txt:
- platform/qt/editing/pasteboard/testcase-9507-expected.txt: Added.
- platform/qt/editing/pasteboard/undoable-fragment-removes-expected.txt: Added.
- platform/qt/editing/pasteboard/unrendered-br-expected.txt: Added.
2010-05-03 Robert Hogan <robert@webkit.org>
Reviewed by Eric Seidel.
[Qt] Unskip editing/pasteboard tests that fail due to 'x,y' differences
Unskip a large number of editing/pasteboard tests that fail only due
to rendertree differences in the height/width of rendered text.
Reduce the skipped editing/pasteboard tests to those that fail for
a good reason and add some initial commentary on each.
Unskip:
editing/pasteboard/3976872.html
editing/pasteboard/4076267-3.html
editing/pasteboard/4242293.html
editing/pasteboard/4631972.html
editing/pasteboard/4641033.html
editing/pasteboard/4700297.html
editing/pasteboard/4840662.html
editing/pasteboard/4944770-1.html
editing/pasteboard/4944770-2.html
editing/pasteboard/4989774.html
editing/pasteboard/5006779.html
editing/pasteboard/5027857.html
editing/pasteboard/5028447.html
editing/pasteboard/5032095.html
editing/pasteboard/5065605.html
editing/pasteboard/5071074-2.html
editing/pasteboard/5071074.html
editing/pasteboard/5075944-2.html
editing/pasteboard/5075944-3.html
editing/pasteboard/5075944.html
editing/pasteboard/5089327.html
editing/pasteboard/5134759.html
editing/pasteboard/5156401-1.html
editing/pasteboard/5245519.html
editing/pasteboard/5247341.html
editing/pasteboard/5368833.html
editing/pasteboard/5478250.html
editing/pasteboard/5483567.html
editing/pasteboard/5601583-1.html
editing/pasteboard/5780697-2.html
editing/pasteboard/7955.html
editing/pasteboard/8145-1.html
editing/pasteboard/8145-2.html
editing/pasteboard/8145-3.html
editing/pasteboard/copy-paste-bidi.html
editing/pasteboard/cut-text-001.html
editing/pasteboard/displaced-generic-placeholder.html
editing/pasteboard/displaced-placeholder.html
editing/pasteboard/display-block-on-spans.html
editing/pasteboard/emacs-cntl-y-001.html
editing/pasteboard/innerText-inline-table.html
editing/pasteboard/interchange-newline-1.html
editing/pasteboard/interchange-newline-3.html
editing/pasteboard/interchange-newline-4.html
editing/pasteboard/merge-after-delete-1.html
editing/pasteboard/merge-after-delete-2.html
editing/pasteboard/merge-after-delete.html
editing/pasteboard/merge-end-1.html
editing/pasteboard/merge-end-2.html
editing/pasteboard/merge-end-3.html
editing/pasteboard/merge-end-4.html
editing/pasteboard/merge-end-5.html
editing/pasteboard/merge-end-blockquote.html
editing/pasteboard/merge-end-borders.html
editing/pasteboard/merge-end-list.html
editing/pasteboard/merge-end-table.html
editing/pasteboard/merge-start-blockquote.html
editing/pasteboard/merge-start-list.html
editing/pasteboard/paste-4035648-fix.html
editing/pasteboard/paste-4038267-fix.html
editing/pasteboard/paste-4039777-fix.html
editing/pasteboard/paste-blockquote-1.html
editing/pasteboard/paste-blockquote-2.html
editing/pasteboard/paste-blockquote-3.html
editing/pasteboard/paste-blockquote-after-blockquote.html
editing/pasteboard/paste-blockquote-into-blockquote-2.html
editing/pasteboard/paste-blockquote-into-blockquote-3.html
editing/pasteboard/paste-blockquote-into-blockquote-4.html
editing/pasteboard/paste-blockquote-into-blockquote.html
editing/pasteboard/paste-line-endings-001.html
editing/pasteboard/paste-line-endings-002.html
editing/pasteboard/paste-line-endings-003.html
editing/pasteboard/paste-line-endings-004.html
editing/pasteboard/paste-line-endings-005.html
editing/pasteboard/paste-line-endings-006.html
editing/pasteboard/paste-line-endings-007.html
editing/pasteboard/paste-line-endings-008.html
editing/pasteboard/paste-line-endings-009.html
editing/pasteboard/paste-line-endings-010.html
editing/pasteboard/paste-list-001.html
editing/pasteboard/paste-match-style-001.html
editing/pasteboard/paste-match-style-002.html
editing/pasteboard/paste-pre-001.html
editing/pasteboard/paste-pre-002.html
editing/pasteboard/paste-table-001.html
editing/pasteboard/paste-table-003.html
editing/pasteboard/paste-table-cells.html
editing/pasteboard/paste-text-002.html
editing/pasteboard/paste-text-003.html
editing/pasteboard/paste-text-004.html
editing/pasteboard/paste-text-005.html
editing/pasteboard/paste-text-006.html
editing/pasteboard/paste-text-007.html
editing/pasteboard/paste-text-009.html
editing/pasteboard/paste-text-010.html
editing/pasteboard/paste-text-011.html
editing/pasteboard/paste-text-012.html
editing/pasteboard/paste-text-013.html
editing/pasteboard/paste-text-014.html
editing/pasteboard/paste-text-015.html
editing/pasteboard/paste-text-016.html
editing/pasteboard/paste-text-017.html
editing/pasteboard/paste-text-018.html
editing/pasteboard/paste-text-019.html
editing/pasteboard/paste-text-at-tabspan-002.html
editing/pasteboard/paste-text-at-tabspan-003.html
editing/pasteboard/paste-unrendered-select.html
editing/pasteboard/pasting-object.html
editing/pasteboard/prevent-block-nesting-01.html
editing/pasteboard/quirks-mode-br-1.html
editing/pasteboard/select-element-1.html
editing/pasteboard/styled-element-markup.html
editing/pasteboard/testcase-9507.html
editing/pasteboard/undoable-fragment-removes.html
editing/pasteboard/unrendered-br.html
- platform/qt/Skipped:
- 08:18 Changeset [58674] by
-
2010-05-03 Philippe Normand <pnormand@igalia.com>
Rubber-stamped by Csaba Osztrogonac.
media/video-duration-known-after-eos.html failing
https://bugs.webkit.org/show_bug.cgi?id=38466
- platform/qt/Skipped: Skipping failing test. A lot of media tests are already skipped on this platform. Tor Arne said he'd look at them, at some point.
- 07:13 Changeset [58673] by
-
[Qt] Fix qtlibraryinfix not to contain space
Patch by Janne Koskinen <janne.p.koskinen@digia.com> on 2010-05-03
Reviewed by Simon Hausmann.
List catenation with += adds whitespace cutting the infix
from the final target.
- WebCore.pro:
- 07:10 Changeset [58672] by
-
[Qt] Fix rendering of <button> elements on Mac OS X
Reviewed by Simon Hausmann.
The <button> element has ButtonPart appearance, not PushButton part,
so we have to include ButtonPart when we decide if we should draw the
button as raised on Mac OS X.
- 07:09 Changeset [58671] by
-
2010-04-30 Philippe Normand <pnormand@igalia.com>
Reviewed by Eric Seidel.
[GStreamer] endless loop after playback ended
https://bugs.webkit.org/show_bug.cgi?id=38384
At playback end ensure duration() will return a valid duration if
we managed to calculate it based on current position.
Test: media/video-duration-known-after-eos.html
- platform/graphics/gstreamer/MediaPlayerPrivateGStreamer.cpp: (WebCore::MediaPlayerPrivateGStreamer::didEnd):
2010-04-30 Philippe Normand <pnormand@igalia.com>
Reviewed by Eric Seidel.
[GStreamer] endless loop after playback ended
https://bugs.webkit.org/show_bug.cgi?id=38384
- media/video-duration-known-after-eos-expected.txt: Added.
- media/video-duration-known-after-eos.html: Added.
- 05:00 Changeset [58670] by
-
[Qt] Prune dead code in QWebPage
Reviewed by Simon Hausmann.
- Api/qwebpage.cpp:
- 04:17 Changeset [58669] by
-
[Qt] Fix build break on Mac OS X
- plugins/mac/PluginPackageMac.cpp: Use correct type
- WebCore.pro: Remove duplicate symbol, we now have a Qt implementation
- 04:00 Changeset [58668] by
-
2010-05-03 Daniel Bates <dbates@rim.com>
Reviewed by Chris Jerdonek.
Refactor the unit tests in VCSUtils_unittest/parseDiff.pl to use
Test::More::is_deeply like we do in VCSUtils_unittest/parseDiffHeader.pl.
- Scripts/webkitperl/VCSUtils_unittest/parseDiff.pl:
- 02:09 QtWebKitRelease20 edited by
- (diff)
- 01:57 QtWebKitRelease20 edited by
- (diff)
- 01:22 QtWebKitRelease20 edited by
- (diff)
- 01:16 QtWebKitRelease20 edited by
- (diff)
- 01:15 QtWebKitRelease20 edited by
- (diff)
- 00:59 QtWebKitTriageRoster edited by
- (diff)
- 00:12 Changeset [58667] by
-
[Qt] Fix library infix usage when compiling inside of Qt
Patch by Thomas Zander <t.zander@nokia.com> on 2010-05-03
Reviewed by Simon Hausmann.
Don't apply the infix when building inside Qt, as that's done through the
inclusion of qbase.pri.
- WebCore.pro:
05/02/10:
- 22:40 Changeset [58666] by
-
Unreviewed.
Corrected a file path in the ChangeLog entry for r58663 (bug 38319) below.
- 21:25 Changeset [58665] by
-
Another case of <rdar://problem/7552959> REGRESSION: Infinite recursion in Position::getInlineBoxAndOffset()
https://bugs.webkit.org/show_bug.cgi?id=38445
Reviewed by Simon Fraser.
WebCore:
Test: editing/selection/mixed-editability-11.html
- dom/Position.cpp:
(WebCore::downstreamIgnoringEditingBoundaries): Added. Returns the furthest visually equivalent
position downstream, crossing any editability boundaries.
(WebCore::upstreamIgnoringEditingBoundaries): Similarly for upstream.
(WebCore::Position::getInlineBoxAndOffset): Changed the logic for finding an inline box for positions
whose node is a block flow. Instead of traversing the DOM, advance downstream or upstream as far as
possible, crossing any editability boudaries. Infinite recursion is avoided by advancing all the way
and checking that the new position is different from the starting position. Also replaced the specific
test for buttons with the generic and more comprehensive canHaveChildrenForEditing().
LayoutTests:
- editing/selection/mixed-editability-11-expected.txt: Added.
- editing/selection/mixed-editability-11.html: Added.
- 19:21 Changeset [58664] by
-
2010-05-02 Geoff Levand <geoff.levand@am.sony.com>
Reviewed by Eric Seidel.
[GTK] Fix out of source build failure
https://bugs.webkit.org/show_bug.cgi?id=38051
Add a preprocessor include path for generated GTK header files.
Fixes build errors like these when building GTK out of source:
webkit.h: error: webkit/webkitversion.h: No such file or directory
- GNUmakefile.am:
- 18:49 Changeset [58663] by
-
2010-05-02 Chris Jerdonek <cjerdonek@webkit.org>
Reviewed by Eric Seidel.
- Scripts/VCSUtils.pm:
- In parseDiffHeader()--
- Added an "scmFormat" hash key to the return value to represent whether the diff is Git or SVN formatted.
- Adjusted the code so the value of "copiedFromPath" will be undef rather than "does not exist" if the file was not copied.
- In parseDiffHeader()--
- Scripts/webkitperl/VCSUtils_unittest/parseDiffHeader.pl:
- Added a FIXME to refactor these unit tests to use is_deeply().
- Scripts/webkitperl/VCSUtils_unittest/parseDiffHeader.pl:
- Updated the unit tests to test the "scmFormat" value.
- Simplified the unit tests by refactoring them to use is_deeply().
- Scripts/VCSUtils.pm:
- 15:18 Changeset [58662] by
-
2010-05-02 Tasuku Suzuki <tasuku.suzuki@nokia.com>
Reviewed by Simon Hausmann.
[Qt] Fix compilation with QT_NO_BEARERMANAGEMENT
https://bugs.webkit.org/show_bug.cgi?id=38324
- platform/network/NetworkStateNotifier.h:
- platform/network/qt/NetworkStateNotifierQt.cpp:
2010-05-02 Tasuku Suzuki <tasuku.suzuki@nokia.com>
Reviewed by Simon Hausmann.
[Qt] Fix compilation with QT_NO_BEARERMANAGEMENT
https://bugs.webkit.org/show_bug.cgi?id=38324
- Api/qwebsettings.cpp:
- 14:48 Changeset [58661] by
-
[Qt] QtWebKit versioning added
https://bugs.webkit.org/show_bug.cgi?id=37207
Patch by Janne Koskinen <janne.p.koskinen@digia.com> on 2010-04-29
Reviewed by Simon Hausmann.
QtWebkit releases separated from Qt release cycle.
WebCore:
- WebCore.pro:
WebKit/qt:
- qtwebkit_version.pri: Added.
- 14:42 Changeset [58660] by
-
2010-05-02 Joseph Pecoraro <joepeck@webkit.org>
Reviewed by Timothy Hatcher.
Web Inspector: Old Style trimWhitespace() should be trim()
https://bugs.webkit.org/show_bug.cgi?id=38441
- inspector/front-end/SourceFrame.js: (WebInspector.SourceFrame.prototype._evalSelectionInCallFrame):
- 14:31 Changeset [58659] by
-
2010-05-02 Tasuku Suzuki <tasuku.suzuki@nokia.com>
Reviewed by Simon Hausmann.
[Qt] Fix compilation with QT_NO_LINEEDIT
https://bugs.webkit.org/show_bug.cgi?id=38324
- platform/qt/RenderThemeQt.cpp: (WebCore::RenderThemeQt::~RenderThemeQt): (WebCore::RenderThemeQt::findFrameLineWidth):
- 13:18 Changeset [58658] by
-
2010-05-02 Pavel Feldman <pfeldman@chromium.org>
Not reviewed: Touch inspector controller to kick windows tests.
- inspector/InspectorController.cpp:
- 11:32 Changeset [58657] by
-
2010-05-02 Pavel Feldman <pfeldman@chromium.org>
Reviewed by Timothy Hatcher.
Web Inspector: IMG nodes are being added to the DOM tree late, sometimes hiding the revealed element.
- inspector/front-end/ElementsTreeOutline.js: (WebInspector.ElementsTreeOutline.prototype._onmousemove): (WebInspector.ElementsTreeElement.prototype._createTooltipForNode.setTooltip): (WebInspector.ElementsTreeElement.prototype._createTooltipForNode): (WebInspector.ElementsTreeElement.prototype.updateTitle): (WebInspector.ElementsTreeElement.prototype._attributeHTML): ():
- 10:08 WebInspector edited by
- (diff)
- 09:30 Changeset [58656] by
-
2010-05-02 Dirk Schulze <krit@webkit.org>
Reviewed by Nikolas Zimmermann.
SVG hkern implementation incomplete
https://bugs.webkit.org/show_bug.cgi?id=38407
Test: svg/text/text-hkern.svg
The current SVG hkern implementation is incomplete and partly wrong. We pass the ACID3 test
by accident.
The new implementation supports all glyph and unicode combinations that are allowed by the Spec
and fixes various of bugs. The parser moved from SVGFontElement to the general parsing code in
SVGParserUtilities.
Some clean-up makes the code more readable and reuseable for the upcoming vkern implementation.
hkern support for text on path is missing and will be added by a following patch.
Unicode strings of hkern elements are just parsed once and not on every glyph again anymore.
- rendering/SVGRootInlineBox.cpp: (WebCore::calculateCSSKerning): (WebCore::applySVGKerning): (WebCore::SVGRootInlineBox::buildLayoutInformationForTextBox):
- svg/SVGFontElement.cpp: (WebCore::stringMatchesUnicodeRange): (WebCore::stringMatchesGlyphName): (WebCore::matches): (WebCore::SVGFontElement::getHorizontalKerningPairForStringsAndGlyphs):
- svg/SVGFontElement.h:
- svg/SVGHKernElement.cpp: (WebCore::SVGHKernElement::buildHorizontalKerningPair):
- svg/SVGHKernElement.h:
- svg/SVGParserUtilities.cpp: (WebCore::parseGlyphName): (WebCore::parseUnicodeRange): (WebCore::parseKerningUnicodeString):
- svg/SVGParserUtilities.h:
2010-05-02 Dirk Schulze <krit@webkit.org>
Reviewed by Nikolas Zimmermann.
SVG hkern implementation incomplete
https://bugs.webkit.org/show_bug.cgi?id=38407
The hkern patch fixes some very old bugs in some tests of the official
SVG Test suite. Added a new test to check correct behavior on different
use cases of textin SVG.
- platform/mac/svg/W3C-SVG-1.1/fonts-kern-01-t-expected.checksum:
- platform/mac/svg/W3C-SVG-1.1/fonts-kern-01-t-expected.png:
- platform/mac/svg/W3C-SVG-1.1/fonts-kern-01-t-expected.txt:
- platform/mac/svg/W3C-SVG-1.1/masking-mask-01-b-expected.checksum:
- platform/mac/svg/W3C-SVG-1.1/masking-mask-01-b-expected.png:
- platform/mac/svg/W3C-SVG-1.1/render-groups-01-b-expected.checksum:
- platform/mac/svg/W3C-SVG-1.1/render-groups-01-b-expected.png:
- platform/mac/svg/W3C-SVG-1.1/render-groups-03-t-expected.checksum:
- platform/mac/svg/W3C-SVG-1.1/render-groups-03-t-expected.png:
- platform/mac/svg/text/kerning-expected.checksum:
- platform/mac/svg/text/kerning-expected.png:
- platform/mac/svg/text/text-hkern-expected.checksum: Added.
- platform/mac/svg/text/text-hkern-expected.png: Added.
- platform/mac/svg/text/text-hkern-expected.txt: Added.
- svg/text/text-hkern.svg: Added.
2010-05-02 Dirk Schulze <krit@webkit.org>
Reviewed by Nikolas Zimmermann.
SVG hkern implementation incomplete
https://bugs.webkit.org/show_bug.cgi?id=38407
We support hkern now. Mention this in the SVG status page.
- projects/svg/status.xml:
- 08:27 Changeset [58655] by
-
2010-05-02 Pavel Feldman <pfeldman@chromium.org>
Reviewed by Timothy Hatcher.
Web Inspector: debugger shortcuts are processed twice if source frame has focus.
- inspector/front-end/SourceFrame.js: (WebInspector.SourceFrame): (WebInspector.SourceFrame.prototype._createViewerIfNeeded):
- 08:03 Changeset [58654] by
-
Unreviewed. Build fix on Japanese Windows.
Replace a non-ASCII character in a comment with an ASCII character.
- 05:47 Changeset [58653] by
-
2010-05-02 Kartikaya Gupta <kagupta@rim.com>
Reviewed by George Staikos.
When running the IDL file through the preprocessor, wait for the preprocessor subcommand to fully terminate before continuing. Without this, if multiple IDL files are parsed in a tight loop, the code dies after 64 IDL files because it exhausts the available system resources.
No new tests. None needed.
- bindings/scripts/IDLParser.pm:
- 05:26 Changeset [58652] by
-
2010-05-02 Laszlo Gombos <laszlo.1.gombos@nokia.com>
Reviewed by Eric Seidel.
[Qt] Enable JIT for QtWebKit on Symbian
https://bugs.webkit.org/show_bug.cgi?id=38339
JIT on Symbian has been stable for quite some time, it
is time to turn it on by default.
- wtf/Platform.h:
- 05:05 Changeset [58651] by
-
2010-05-02 Jarkko Sakkinen <jarkko.j.sakkinen@gmail.com>
Reviewed by Eric Seidel.
[Qt] Build error in GraphicsContext3DQt.cpp
https://bugs.webkit.org/show_bug.cgi?id=38382
Removed duplicate implementation of isGLES2Compliant from
GraphicsContext3DQt.cpp. Removed deprecated API stuff for
texImage2D/texSubImage2D.
- platform/graphics/qt/GraphicsContext3DQt.cpp:
- 04:54 Changeset [58650] by
-
2010-05-02 Garret Kelly <gdk@chromium.org>
Reviewed by David Levin.
Make the Touch RuntimeEnabledFeature disabled by default.
https://bugs.webkit.org/show_bug.cgi?id=38392
- bindings/generic/RuntimeEnabledFeatures.cpp: Disable the Touch feature by default.
- 04:29 Changeset [58649] by
-
Unreviewed. Fix a build failure by r58647.
- 01:55 Changeset [58648] by
-
2010-05-02 Benjamin Poulain <benjamin.poulain@nokia.com>
Reviewed by Kenneth Rohde Christiansen.
[Qt] QWebPage::userAgentForUrl is terrible API
https://bugs.webkit.org/show_bug.cgi?id=33875
Simplify the creation of the user agent string to avoid some
overhead for each loaded url.
The static part of the user agent is cached so it only have
to be made once.
This creation has been made in order to simplify the code.
The two variable: application name and current language are
set dynamically when needed.
The default locale is non longer created if the widget locale
is used.
- Api/qwebpage.cpp: (QWebPage::userAgentForUrl):
- tests/qwebpage/tst_qwebpage.cpp: (tst_QWebPage::userAgentApplicationName): (tst_QWebPage::userAgentLocaleChange):
- 01:39 Changeset [58647] by
-
2010-05-02 Michael Nordman <michaeln@google.com>
Reviewed by Dmitry Titov.
Define two new ResourceRequestBase TargetTypes for worker and shared worker
main resources. Use the new target types where appropiate. Add logic to marshal
the target type specified by requests initiated on a background worker thread.
No new tests. This doesn't have script visible artifacts.
- platform/network/ResourceRequestBase.cpp: marshal the values (WebCore::ResourceRequestBase::adopt): (WebCore::ResourceRequestBase::copyData):
- platform/network/ResourceRequestBase.h: define the types (WebCore::ResourceRequestBase::):
- workers/DefaultSharedWorkerRepository.cpp: use TargetIsSharedWorker (WebCore::SharedWorkerScriptLoader::load):
- workers/Worker.cpp: use TargetIsWorker (WebCore::Worker::Worker):
- workers/WorkerContext.cpp: use TargetIsScript for importScripts (WebCore::WorkerContext::importScripts):
- workers/WorkerScriptLoader.cpp: add a data member for the target type (WebCore::WorkerScriptLoader::WorkerScriptLoader): (WebCore::WorkerScriptLoader::createResourceRequest):
- workers/WorkerScriptLoader.h:
2010-05-02 Michael Nordman <michaeln@google.com>
Reviewed by Dmitry Titov.
Define two new ResourceRequestBase TargetTypes for worker and shared worker
main resources. Use the new target types where appropiate. Add logic to marshal
the target type specified by requests initiated on a background worker thread.
- src/SharedWorkerRepository.cpp: use TargetIsSharedWorker (WebCore::SharedWorkerScriptLoader::SharedWorkerScriptLoader):
- 01:12 Changeset [58646] by
-
2010-05-02 Marcus Bulach <bulach@chromium.org>
Reviewed by Jeremy Orlow.
[chromium] Upstream test expectations for:
http/tests/loading/basic-auth-resend-wrong-credentials.html
https://bugs.webkit.org/show_bug.cgi?id=38387
- platform/chromium/test_expectations.txt:
- 01:01 Changeset [58645] by
-
2010-05-02 Noam Rosenthal <noam.rosenthal@nokia.com>
Reviewed by Kenneth Rohde Christiansen.
[Qt] GraphicsLayer: animation incorrect when scrolling
https://bugs.webkit.org/show_bug.cgi?id=38371
This is a regression introduced with the invalidate-on-scroll code path, that uses QGraphicsWebViewPrivate::update()
instead of QGraphicsWebViewPrivate::scroll(). The patch makes sure that the scrolling position is correct on
each content update - this shouldn't have a performance impact - the only overhead is an additional value-test on each update.
- Api/qgraphicswebview.cpp: (QGraphicsWebViewPrivate::update):
- 00:50 Changeset [58644] by
-
2010-05-02 Ben Murdoch <benm@google.com>
Reviewed by Darin Adler.
fast/events/touch has non-standard script test wrappers
https://bugs.webkit.org/show_bug.cgi?id=38084
Tidy up the touch event layout tests. Make the script-tests TEMPLATE.html
generate asynchronous style tests using js-test-post-function.js and move
the non-standard synchronous test into the resources directory.
- fast/events/touch/basic-multi-touch-events.html: Regenerate from TEMPLATE.html
- fast/events/touch/basic-single-touch-events.html: ditto.
- fast/events/touch/send-oncancel-event.html: ditto.
- fast/events/touch/touch-target.html: ditto.
- fast/events/touch/create-touch-event.html: Link to script in resources rather than
script-tests as this test is not generated from TEMPLATE.html.
- fast/events/touch/resources/create-touch-event.js: Copied from LayoutTests/fast/events/touch/script-tests/create-touch-event.js.
- fast/events/touch/script-tests/TEMPLATE.html: Update to generate asynchronous tests.
- fast/events/touch/script-tests/create-touch-event.js: Removed.
- fast/events/touch/script-tests/touch-target.js: Refactor to work with new wrapper
generated from TEMPLATE.html.
- 00:23 Changeset [58643] by
-
2010-05-02 Rafael Weinstein <rafaelw@chromium.org>
Reviewed by Darin Fisher.
Include WebWindowFeatures in call to WebViewClient::createView.
https://bugs.webkit.org/show_bug.cgi?id=38301
- public/WebViewClient.h: (WebKit::WebViewClient::createView):
- public/WebWindowFeatures.h:
- src/ChromeClientImpl.cpp: (WebKit::ChromeClientImpl::createWindow):
- 00:12 Changeset [58642] by
-
2010-05-02 Noam Rosenthal <noam.rosenthal@nokia.com>
Reviewed by Adele Peterson.
Webkit doesn't compile with 3D-canvas enabled and video disabled
https://bugs.webkit.org/show_bug.cgi?id=38297
Added a #ifdef ENABLE(VIDEO) to WebGL code
No new tests: compile fix.
- bindings/js/JSWebGLRenderingContextCustom.cpp: (WebCore::JSWebGLRenderingContext::texImage2D): (WebCore::JSWebGLRenderingContext::texSubImage2D):
- 00:01 Changeset [58641] by
-
2010-05-01 Evan Stade <estade@chromium.org>
Reviewed by David Levin.
[chromium] Skia needs to fade DragImages
https://bugs.webkit.org/show_bug.cgi?id=38008
tested by DragImageTest
- platform/chromium/DragImageChromiumSkia.cpp: (WebCore::dissolveDragImageToFraction):implement (WebCore::createDragImageFromImage):deep copy instead of shallow
2010-05-01 Evan Stade <estade@chromium.org>
Reviewed by David Levin
[chromium] Skia needs to fade DragImages
https://bugs.webkit.org/show_bug.cgi?id=38008
- tests/DragImageTest.cpp: (DragImageTest.CreateDragImage): test that the drag image is a deep copy
05/01/10:
- 22:27 Changeset [58640] by
-
Not reviewed.
More shortening of Sputnik paths.
- fast/js/sputnik/Conformance/07_Lexical_Conventions/7.9_Automatic_Semicolon_Insertion/7.9.2_Examples: Copied from fast/js/sputnik/Conformance/07_Lexical_Conventions/7.9_Automatic_Semicolon_Insertion/7.9.2_Examples_of_Automatic_Semicolon_Insertion.
- fast/js/sputnik/Conformance/07_Lexical_Conventions/7.9_Automatic_Semicolon_Insertion/7.9.2_Examples_of_Automatic_Semicolon_Insertion: Removed.
- 22:07 Changeset [58639] by
-
2010-05-01 James Robinson <jamesr@chromium.org>
Reviewed by Eric Seidel.
Add fast/repaint/fixed-move-after-keyboard-scroll.html to skipped list pending investigation
https://bugs.webkit.org/show_bug.cgi?id=38416
- platform/mac-snowleopard/Skipped:
- 18:25 Changeset [58638] by
-
Land test case that was supposed to be in r53607.
2010-01-20 Adam Barth <abarth@webkit.org>
Reviewed by Darin Adler.
Stylesheet href property shows redirected URL unlike other browsers
https://bugs.webkit.org/show_bug.cgi?id=33683
Test whether the href property of style sheets contains the original or
final URL of the redirect chain that lead to the style sheet.
I couldn't figure out how to test the XSLStyleSheet parts of this
change. There didn't seem to be DOM bindings for the href property
here. If I missed it, please let me know.
- http/tests/security/stylesheet-href-redirect-expected.txt: Added.
- http/tests/security/stylesheet-href-redirect.html: Added.
- 15:44 Changeset [58637] by
-
2010-05-01 Daniel Bates <dbates@rim.com>
Reviewed by Chris Jerdonek.
Adds infrastructure to change the file mode of a file using
the appropriate SCM-specific routines.
No functionality was changed, so no new tests.
- Scripts/VCSUtils.pm: Added subroutines scmToggleExecutableBit, scmAddExecutableBit, and scmRemoveExecutableBit.
- Scripts/svn-apply: Check for the hash key executableBitDelta and toggle the executable bit.
- Scripts/svn-unapply: Ditto.
- 13:23 Changeset [58636] by
-
2010-05-01 Maciej Stachowiak <mjs@apple.com>
Reviewed by Sam Weinig.
REGRESSION (r58273): Visited links do not change color immediately when Cmd-clicked
https://bugs.webkit.org/show_bug.cgi?id=38422
<rdar://problem/7921778>
Tests:
manual-tests/visited-link-new-window.html
- css/CSSStyleSelector.cpp: (WebCore::CSSStyleSelector::initElement): Only cache the visited link state when invoked as part of a helper call to styleForElement or pseudoStyleForElement, to avoid caching the visited link state beyond the scope of a single style lookup. (WebCore::CSSStyleSelector::styleForElement): Adjust for above change. (WebCore::CSSStyleSelector::pseudoStyleForElement): Adjust for above change.
- css/CSSStyleSelector.h: (WebCore::CSSStyleSelector::styleForElement): Change so "visited link helper mode" can't accidentally be called from outside CSSStyleSelector itself. (WebCore::CSSStyleSelector::pseudoStyleForElement): ditto
- manual-tests/visited-link-new-window.html: Added. I could not figure out any way to make an automated test that supports visited link coloring.
- 05:06 Changeset [58635] by
-
Move number parsing code out of HTMLInputElement.
https://bugs.webkit.org/show_bug.cgi?id=38203
Reviewed by Darin Adler.
The numebr parsing code follows HTML5 parsing rules and should be available outside of HTMLInputElement.
No new tests as no new functionality was introduced.
- html/HTMLInputElement.cpp:
- html/HTMLInputElement.h:
- html/HTMLParser.cpp:
- html/HTMLParser.h:
- html/StepRange.cpp:
- html/ValidityState.cpp:
- rendering/RenderSlider.cpp:
- 03:42 Changeset [58634] by
-
2010-05-01 Xan Lopez <xlopez@igalia.com>
Reviewed by Oliver Hunt.
[GTK] GObject DOM bindings
https://bugs.webkit.org/show_bug.cgi?id=33590
Test webkit_dom_document_get_links.
- tests/testdomdocument.c: (test_dom_document_get_links): (main):
- 03:40 Changeset [58633] by
-
2010-05-01 Xan Lopez <xlopez@igalia.com>
Reviewed by Oliver Hunt.
[GTK] GObject DOM bindings
https://bugs.webkit.org/show_bug.cgi?id=33590
Unify more logic to skip functions into SkipFunction, and
whitelist the two [Custom] methods in HTMLCollection.idl.
- bindings/scripts/CodeGeneratorGObject.pm:
- 03:20 Changeset [58632] by
-
[Qt] Fix up width/height differences in smart-paste-00* results
Unreviewed fix to height/width rendering differences in results added
by r58631.
[Qt] Fix up width/height differences in smart-paste-00* results
- platform/qt/editing/pasteboard/smart-paste-001-expected.txt:
- platform/qt/editing/pasteboard/smart-paste-002-expected.txt:
- platform/qt/editing/pasteboard/smart-paste-003-expected.txt:
- platform/qt/editing/pasteboard/smart-paste-004-expected.txt:
- platform/qt/editing/pasteboard/smart-paste-005-expected.txt:
- platform/qt/editing/pasteboard/smart-paste-006-expected.txt:
- platform/qt/editing/pasteboard/smart-paste-007-expected.txt:
- 03:02 Changeset [58631] by
-
[Qt] Add smart paste support
2010-05-01 Robert Hogan <robert@webkit.org>
Reviewed by Simon Hausmann.
[Qt] Add smart paste support
Add platform specific results.
Unskip tests:
editing/pasteboard/smart-paste-001.html
editing/pasteboard/smart-paste-002.html
editing/pasteboard/smart-paste-003.html
editing/pasteboard/smart-paste-004.html
editing/pasteboard/smart-paste-005.html
editing/pasteboard/smart-paste-006.html
editing/pasteboard/smart-paste-007.html
- platform/qt/Skipped:
- platform/qt/editing/pasteboard/smart-paste-001-expected.txt: Added.
- platform/qt/editing/pasteboard/smart-paste-002-expected.txt: Added.
- platform/qt/editing/pasteboard/smart-paste-003-expected.txt: Added.
- platform/qt/editing/pasteboard/smart-paste-004-expected.txt: Added.
- platform/qt/editing/pasteboard/smart-paste-005-expected.txt: Added.
- platform/qt/editing/pasteboard/smart-paste-006-expected.txt:
- platform/qt/editing/pasteboard/smart-paste-007-expected.txt: Added.
- Api/qwebpage.cpp: (QWebPagePrivate::QWebPagePrivate): Enable smart paste support by default.
- WebCore.pro:
- editing/qt/SmartReplaceQt.cpp: Added. (WebCore::isCharacterSmartReplaceExempt):
- platform/qt/PasteboardQt.cpp: (WebCore::Pasteboard::writeSelection): (WebCore::Pasteboard::canSmartReplace):
- 00:07 Changeset [58630] by
-
2010-04-30 Yoshiki Hayashi <yhayashi@google.com>
Reviewed by Shinichiro Hamaji.
Fixes an issue where border height and padding height are ignored when computing vertically shrinking flexbox's height.
- fast/flexbox/child-flexing-expected.txt: Added.
- fast/flexbox/child-flexing.html: Added.
2010-04-30 Yoshiki Hayashi <yhayashi@google.com>
Reviewed by Shinichiro Hamaji.
Fixes an issue where border height and padding height are ignored when computing vertically shrinking flexbox's height.
Test: fast/flexbox/child-flexing.html
- rendering/RenderFlexibleBox.cpp: (WebCore::RenderFlexibleBox::allowedChildFlex):
04/30/10:
- 22:34 Changeset [58629] by
-
2010-04-30 Shinichiro Hamaji <hamaji@chromium.org>
Reviewed by Darin Adler.
Add layoutTestController.setPrinting()
https://bugs.webkit.org/show_bug.cgi?id=37203
- platform/mac/Skipped:
- platform/mac/printing/media-queries-print-expected.txt:
2010-04-30 Shinichiro Hamaji <hamaji@chromium.org>
Reviewed by Darin Adler.
Add layoutTestController.setPrinting()
https://bugs.webkit.org/show_bug.cgi?id=37203
Use the renderer's width insteead of screen's width as the width of
a screen depends on machines.
- rendering/RenderTreeAsText.cpp: (WebCore::externalRepresentation):
- 20:09 Changeset [58628] by
-
2010-04-30 Joseph Pecoraro <joepeck@webkit.org>
Reviewed by Timothy Hatcher.
Web Inspector: Missing INSPECTOR Guard in Console::lastWMLErrorMessage
https://bugs.webkit.org/show_bug.cgi?id=38366
Console::lastWMLErrorMessage is only available if WML is enabled, however
its implementation only makes sense as long as INSPECTOR is enabled
as well. So this adds the ENABLE(INSPECTOR) guard in the function. A
browser without ENABLE(INSPECTOR) will always get an empty result.
- page/Console.cpp:
- 19:54 Changeset [58627] by
-
2010-04-28 Joseph Pecoraro <joepeck@webkit.org>
Reviewed by Oliver Hunt.
<input type="search"> with uneven padding causes text clipping
https://bugs.webkit.org/show_bug.cgi?id=38160
- fast/css/input-search-padding.html: Added.
- platform/mac/fast/css/input-search-padding-expected.checksum: Added.
- platform/mac/fast/css/input-search-padding-expected.png: Added.
- platform/mac/fast/css/input-search-padding-expected.txt: Added.
2010-04-28 Joseph Pecoraro <joepeck@webkit.org>
Reviewed by Oliver Hunt.
<input type="search"> with uneven padding causes text clipping
https://bugs.webkit.org/show_bug.cgi?id=38160
Test: fast/css/input-search-padding.html
An <input type="search"> contains an inner block, which is explicitly
centered in RenderTextControlSingleLine based on the height of the element.
However, the clipping rect was not using the set location, and instead
calculated off of the top border and padding alone. This also vertically
centers the Caps Lock indicator.
- rendering/RenderTextControl.cpp: moved controlClipRect implementation to RenderTextControlSingleLine
- rendering/RenderTextControl.h: allow a subclass implementation of controlClipRect, removed redundant hasControlClip implementation, and moved controlClipRect
- rendering/RenderTextControlSingleLine.cpp: (WebCore::RenderTextControlSingleLine::paint): vertically center the Caps Lock indicator (WebCore::RenderTextControlSingleLine::controlClipRect): use the set location of the anonymous inner block instead
- rendering/RenderTextControlSingleLine.h: allow for an implementation of controlClipRect for <input type="search">
- 18:01 Changeset [58626] by
-
Fix the build.
- Platform/CoreIPC/mac/ConnectionMac.cpp: Add missing #include.
- 17:49 Changeset [58625] by
-
Caret may fail to blink if a focus handler brings up a modal dialog
https://bugs.webkit.org/show_bug.cgi?id=38372
Reviewed by Darin Adler.
WebCore:
- manual-tests/onfocus-alert-blinking-caret.html: Added.
- page/EventHandler.cpp:
(WebCore::EventHandler::handleMousePressEvent):
Moved the call to setCaretBlinkingSuspended() from here...
(WebCore::EventHandler::handleMousePressEvent):
... to here. This makes us suspend caret blinking before dispatching the
mouse down event. If dispatching the mouse down event allows the message
loop to run, we want mouse up events received in that message loop to be
able to resume caret blinking.
(WebCore::EventHandler::lostMouseCapture):
We've lost mouse capture and won't be notified of mouse up events;
resume caret blinking.
- page/EventHandler.h:
Declare lostMouseCapture().
WebKit/win:
- WebView.cpp:
(WebView::handleMouseEvent):
If the message is WM_CANCELMODE, which indicates that we our capturing
of mouse events has been cancelled, tell the EventHandler.
It's possible to re-enter this function if handling a mouse event allows
the message loop to run; moved up the call to setMouseActivated(), so
that if we do re-enter this function, the later mouse event will not be
considered as activating the window.
(WebView::WebViewWndProc):
Handle WM_CANCELMODE by calling handleMouseEvent().
- 17:22 Changeset [58624] by
-
https://bugs.webkit.org/show_bug.cgi?id=38413
Add callback based API to get the textual representation of the RenderTree.
Reviewed by Anders Carlsson.
- Also ensures that any pending callbacks are invalidated if the WebPage closes (expectedly or unexpectedly).
- A follow up patch will unify the callback mechanism with a common base class.
- Shared/CoreIPCSupport/WebPageMessageKinds.h:
(WebPageMessage::):
- Shared/CoreIPCSupport/WebPageProxyMessageKinds.h:
(WebPageProxyMessage::):
- UIProcess/API/C/WKPage.cpp:
(WKPageRunJavaScriptInMainFrame_f):
(WKPageRenderTreeExternalRepresentation_f):
(callRenderTreeExternalRepresentationBlockAndDispose):
(disposeRenderTreeExternalRepresentationBlock):
(WKPageRenderTreeExternalRepresentation):
- UIProcess/API/C/WKPagePrivate.h: Added.
- UIProcess/RenderTreeExternalRepresentationCallback.cpp: Added.
(WebKit::generateCallbackID):
(WebKit::RenderTreeExternalRepresentationCallback::RenderTreeExternalRepresentationCallback):
(WebKit::RenderTreeExternalRepresentationCallback::~RenderTreeExternalRepresentationCallback):
(WebKit::RenderTreeExternalRepresentationCallback::performCallbackWithReturnValue):
(WebKit::RenderTreeExternalRepresentationCallback::invalidate):
- UIProcess/RenderTreeExternalRepresentationCallback.h: Added.
(WebKit::RenderTreeExternalRepresentationCallback::create):
(WebKit::RenderTreeExternalRepresentationCallback::callbackID):
- UIProcess/ScriptReturnValueCallback.cpp:
(WebKit::ScriptReturnValueCallback::~ScriptReturnValueCallback):
(WebKit::ScriptReturnValueCallback::performCallbackWithReturnValue):
(WebKit::ScriptReturnValueCallback::invalidate):
- UIProcess/ScriptReturnValueCallback.h:
- UIProcess/WebPageProxy.cpp:
(WebKit::WebPageProxy::close):
(WebKit::WebPageProxy::getRenderTreeExternalRepresentation):
(WebKit::WebPageProxy::didReceiveMessage):
(WebKit::WebPageProxy::didRunJavaScriptInMainFrame):
(WebKit::WebPageProxy::didGetRenderTreeExternalRepresentation):
(WebKit::WebPageProxy::processDidExit):
- UIProcess/WebPageProxy.h:
- WebKit2.xcodeproj/project.pbxproj:
- WebProcess/WebPage/WebPage.cpp:
(WebKit::WebPage::getRenderTreeExternalRepresentation):
(WebKit::WebPage::didReceiveMessage):
- WebProcess/WebPage/WebPage.h:
- win/WebKit2.vcproj:
- 16:54 Changeset [58623] by
-
2010-04-30 Simon Fraser <simon.fraser@apple.com>
Reviewed by Dan Bernstein.
<rdar://problem/7477071> REGRESSION: Bad flicker when wheel-scrolling Google Maps, iPad gallery and other sites
Sites that frequently toggle content in and out of compositing layers (like http://www.tumblr.com/boothed)
can cause flickering because of unsychronized compositing layer and view-based updates. There were two
underlying issues:
- On SnowLeopard, AppKit can throttle window updates, thus breaking an assumption that NSView drawing will happen on the runloop cycle after a repaint. This provided a window for the layerSyncRunLoopObserver to fire and commit layer changes too early.
Fix this by having the layerSyncRunLoopObserver in WebView check to see if a display is pending,
and not commit layer changes in that case. We'll commit layer changes later when we
finally draw.
- The change in r49269 was wrong; it was attempting to fix an issue that was actually caused by -drawRects: coming in for page snapshots. The correct approach is to avoid hitting the synchronization and update disabling code in WebHTMLView for draws that are not to the screen.
- WebView/WebHTMLView.mm: (-[WebHTMLView drawRect:]):
- WebView/WebView.mm: (layerSyncRunLoopObserverCallBack): (-[WebView _scheduleCompositingLayerSync]):
- 16:42 Changeset [58622] by
-
Enabled accelerated compositing in DRT for Windows
https://bugs.webkit.org/show_bug.cgi?id=38404
- 16:30 Changeset [58621] by
-
Unreviewed, build fix.
[Chromium] Added a simple IdentifierRep struct and removed dependency on
IdentifierRep.h.
- bindings/v8/V8NPObject.cpp: Added simple IdentifierRep struct.
- 16:30 Changeset [58620] by
-
Unreviewed, expectations update.
[Chromium] More missing pixel test results.
- platform/chromium-mac/fast/borders/svg-as-border-image-2-expected.checksum: Added.
- platform/chromium-mac/fast/borders/svg-as-border-image-expected.checksum: Added.
- platform/chromium/test_expectations.txt:
- 16:25 Changeset [58619] by
-
2010-04-30 James Robinson <jamesr@chromium.org>
Unreviewed. Add Chromium expectations for a bunch of repaint tests that we now pass (yay!)
- platform/chromium-linux/fast/repaint/fixed-child-move-after-scroll-expected.checksum: Added.
- platform/chromium-linux/fast/repaint/fixed-child-move-after-scroll-expected.png: Added.
- platform/chromium-linux/fast/repaint/fixed-child-of-fixed-move-after-scroll-expected.checksum: Added.
- platform/chromium-linux/fast/repaint/fixed-child-of-fixed-move-after-scroll-expected.png: Added.
- platform/chromium-linux/fast/repaint/fixed-child-of-transformed-move-after-scroll-expected.checksum: Added.
- platform/chromium-linux/fast/repaint/fixed-child-of-transformed-move-after-scroll-expected.png: Added.
- platform/chromium-linux/fast/repaint/fixed-move-after-keyboard-scroll-expected.checksum: Added.
- platform/chromium-linux/fast/repaint/fixed-move-after-keyboard-scroll-expected.png: Added.
- platform/chromium-linux/fast/repaint/fixed-move-after-scroll-expected.checksum: Added.
- platform/chromium-linux/fast/repaint/fixed-move-after-scroll-expected.png: Added.
- platform/chromium-mac/fast/repaint/fixed-move-after-keyboard-scroll-expected.checksum: Added.
- platform/chromium-mac/fast/repaint/fixed-move-after-keyboard-scroll-expected.png: Added.
- platform/chromium-win/fast/repaint/fixed-child-move-after-scroll-expected.checksum: Added.
- platform/chromium-win/fast/repaint/fixed-child-move-after-scroll-expected.png: Added.
- platform/chromium-win/fast/repaint/fixed-child-move-after-scroll-expected.txt: Added.
- platform/chromium-win/fast/repaint/fixed-child-of-fixed-move-after-scroll-expected.checksum: Added.
- platform/chromium-win/fast/repaint/fixed-child-of-fixed-move-after-scroll-expected.png: Added.
- platform/chromium-win/fast/repaint/fixed-child-of-fixed-move-after-scroll-expected.txt: Added.
- platform/chromium-win/fast/repaint/fixed-child-of-transformed-move-after-scroll-expected.checksum: Added.
- platform/chromium-win/fast/repaint/fixed-child-of-transformed-move-after-scroll-expected.png: Added.
- platform/chromium-win/fast/repaint/fixed-child-of-transformed-move-after-scroll-expected.txt: Added.
- platform/chromium-win/fast/repaint/fixed-move-after-keyboard-scroll-expected.checksum: Added.
- platform/chromium-win/fast/repaint/fixed-move-after-keyboard-scroll-expected.png: Added.
- platform/chromium-win/fast/repaint/fixed-move-after-keyboard-scroll-expected.txt: Added.
- platform/chromium-win/fast/repaint/fixed-move-after-scroll-expected.checksum: Added.
- platform/chromium-win/fast/repaint/fixed-move-after-scroll-expected.png: Added.
- platform/chromium-win/fast/repaint/fixed-move-after-scroll-expected.txt: Added.
- platform/chromium/test_expectations.txt:
- 16:06 Changeset [58618] by
-
Fix for https://bugs.webkit.org/show_bug.cgi?id=38406
Add support for sending messages with a size greater than 4096 bytes
Reviewed by Anders Carlsson.
Adds support by putting message bodies that are larger than 4096 bytes
in OOL memory.
- Platform/CoreIPC/Connection.cpp:
(CoreIPC::Connection::sendMessage):
(CoreIPC::Connection::waitForMessage):
(CoreIPC::Connection::sendSyncMessage):
(CoreIPC::Connection::dispatchMessages):
- Platform/CoreIPC/Connection.h:
(CoreIPC::Connection::OutgoingMessage::OutgoingMessage):
(CoreIPC::Connection::OutgoingMessage::messageID):
(CoreIPC::Connection::send):
(CoreIPC::Connection::sendSync):
- Platform/CoreIPC/MessageID.h:
(CoreIPC::MessageID::):
(CoreIPC::MessageID::MessageID):
(CoreIPC::MessageID::equalIgnoringFlags):
(CoreIPC::MessageID::copyAddingFlags):
(CoreIPC::MessageID::fromInt):
(CoreIPC::MessageID::toInt):
(CoreIPC::MessageID::isMessageBodyOOL):
- Platform/CoreIPC/mac/ConnectionMac.cpp:
(CoreIPC::Connection::sendOutgoingMessage):
(CoreIPC::createArgumentDecoder):
- 16:00 Changeset [58617] by
-
Unreviewed, build fix.
[Chromium] Remove bridge/ from include paths.
- WebCore.gyp/WebCore.gyp: Removed bridge/ from include path.
- 15:54 Changeset [58616] by
-
WebCore: Convert m_documentUnderMouse, m_dragInitiator to RefPtr.
Eliminated unused m_dragInitiator accessor to prevent dereferencing.
https://bugs.webkit.org/show_bug.cgi?id=37618
Patch by Abhishek Arya <inferno@chromium.org> on 2010-04-30
Reviewed by David Kilzer.
Test: editing/pasteboard/drag-drop-iframe-refresh-crash.html
- page/DragController.cpp:
(WebCore::DragController::tryDocumentDrag):
(WebCore::DragController::concludeEditDrag):
- page/DragController.h:
(WebCore::DragController::draggingImageURL):
(WebCore::DragController::documentUnderMouse):
LayoutTests: Tests for a crash when an image drag-drop operation happens inside a continuously refreshing iframe.
https://bugs.webkit.org/show_bug.cgi?id=37618
Patch by Abhishek Arya <inferno@chromium.org> on 2010-04-30
Reviewed by David Kilzer.
- editing/pasteboard/drag-drop-iframe-refresh-crash-expected.txt: Added.
- editing/pasteboard/drag-drop-iframe-refresh-crash.html: Added.
- editing/resources/drag-drop.html: Added.
- platform/qt/Skipped:
- 15:26 Changeset [58615] by
-
2010-04-29 James Robinson <jamesr@chromium.org>
Reviewed by Simon Fraser.
Calls FrameView::scrollPositionChanged whenever a ScrollView is scrolled
https://bugs.webkit.org/show_bug.cgi?id=38286
When a ScrollView's scroll position is changed, we have to call
FrameView::scrollPositionChanged to generate repaint invalidation for
fixed position elements. This ends up getting called indirectly when
the ScrollView has a platformWidget through the port layer
(see WebHTMLView.mm's _frameOrBoundsChanged method for how the mac
port does it) but not when there is no platformWidget.
This is tested by the fast/repaint/fixed-* tests when run in pixel
mode.
Test: fast/repaint/fixed-move-after-keyboard-scroll.html
- page/FrameView.h:
- platform/ScrollView.cpp: (WebCore::ScrollView::valueChanged):
- platform/ScrollView.h: (WebCore::ScrollView::scrollPositionChanged):
- 15:22 Changeset [58614] by
-
Part of the previous part (forgot to save).
- Plugins/WebNetscapePluginView.mm:
(-[WebNetscapePluginView stopTimers]):
(-[WebNetscapePluginView startTimers]):
(-[WebNetscapePluginView checkIfAllowedToLoadURL:frame:callbackFunc:context:]):
(-[WebNetscapePluginView _containerCheckResult:contextInfo:]):
(-[WebNetscapePluginView cancelCheckIfAllowedToLoadURL:]):
(-[WebNetscapePluginView scheduleTimerWithInterval:repeat:timerFunc:]):
(-[WebNetscapePluginView unscheduleTimer:]):
(-[WebNetscapePluginView getVariable:forURL:value:length:]):
(-[WebNetscapePluginView setVariable:forURL:value:length:]):
- 15:20 Changeset [58613] by
-
Use C99 integer types in more places.
Reviewed by Darin Adler.
WebCore:
- manual-tests/NPN_Invoke/main.c:
(NPP_New):
(NPP_NewStream):
(NPP_WriteReady):
(NPP_Write):
(NPP_HandleEvent):
(functionPointerForTVector):
- plugins/mac/PluginViewMac.cpp:
(WebCore::PluginView::platformGetValueStatic):
(WebCore::PluginView::handlePostReadFile):
WebKit/mac:
- Plugins/WebNetscapePluginView.mm:
(getNPRect):
- 15:18 Changeset [58612] by
-
Unreviewed, expectations update.
[Chromium] Add more missing pixel results.
- platform/chromium-win/transforms/svg-vs-css-expected.checksum: Added.
- platform/chromium-win/transforms/svg-vs-css-expected.png: Added.
- 15:03 Changeset [58611] by
-
2010-04-30 Darin Adler <darin@apple.com>
Reviewed by Oliver Hunt.
Remove unused scrollRectIntoViewRecursively function
https://bugs.webkit.org/show_bug.cgi?id=38403
- page/Chrome.cpp: (WebCore::Chrome::scrollRectIntoView): Moved comment here that was previously in the scrollRectIntoViewRecursively function.
- platform/ScrollView.cpp: Get rid scrollRectIntoViewRecursively.
- platform/ScrollView.h: Ditto. Fix comment that refers to the two functions. Also correct all uses of the term "method" to use the C++ term "function" and got rid of double spaces after periods.
- 14:59 Changeset [58610] by
-
Fix Tiger build.
- Plugins/WebBaseNetscapePluginStream.h:
- Plugins/WebBaseNetscapePluginStream.mm:
(WebNetscapePluginStream::startStream):
- 14:42 Changeset [58609] by
-
Another Qt build fix.
- plugins/qt/PluginViewQt.cpp:
(WebCore::PluginView::platformGetValueStatic):
- 14:35 Changeset [58608] by
-
Another 32-bit build fix.
- Plugins/WebNetscapePluginView.mm:
(-[WebNetscapePluginView saveAndSetNewPortStateForUpdate:]):
- 14:34 Changeset [58607] by
-
Unreviewed, expectations updated.
[Chromium] Add missing pixel results.
- platform/chromium-linux/fast/borders/svg-as-border-image-2-expected.checksum: Added.
- platform/chromium-linux/fast/borders/svg-as-border-image-expected.checksum: Added.
- platform/chromium-linux/transforms/svg-vs-css-expected.checksum: Added.
- platform/chromium-linux/transforms/svg-vs-css-expected.png: Added.
- platform/chromium-mac/svg/custom/mask-changes-expected.checksum: Added.
- platform/chromium-mac/svg/custom/mask-changes-expected.png: Added.
- platform/chromium-mac/transforms/svg-vs-css-expected.checksum: Added.
- platform/chromium-win/fast/borders/svg-as-border-image-2-expected.checksum: Added.
- platform/chromium-win/fast/borders/svg-as-border-image-2-expected.png: Added.
- platform/chromium-win/fast/borders/svg-as-border-image-expected.checksum: Added.
- platform/chromium-win/fast/borders/svg-as-border-image-expected.png: Added.
- 14:27 BuildingOnWindows edited by
- Clarification about CFNetwork.resources contents needed (diff)
- 14:23 Changeset [58606] by
-
Try to fix GTK+ build.
- DumpRenderTree/unix/TestNetscapePlugin/TestNetscapePlugin.cpp:
(webkit_test_plugin_new_stream):
- 14:13 Changeset [58605] by
-
Add back TRUE, FALSE and NULL macros. They were not meant to be removed!
- bridge/npapi.h:
- 14:04 Changeset [58604] by
-
<http://webkit.org/b/38364> MIME typo in LayoutTests/http/tests/security/xss-DENIED-mime-type-execute-as-html.html
Reviewed by NOBODY (OOPS!).
- http/tests/security/xss-DENIED-mime-type-execute-as-html.html:
Fixed MIME typo from "application-javascript" to
"application/javascript".
- 14:04 Changeset [58603] by
-
Try to fix the Qt build this time.
- plugins/qt/PluginPackageQt.cpp:
(WebCore::staticPluginQuirkRequiresGtkToolKit_NPN_GetValue):
(WebCore::PluginPackage::NPVersion):
- 14:03 Changeset [58602] by
-
Yet another build fix.
- plugins/gtk/PluginPackageGtk.cpp:
(WebCore::PluginPackage::NPVersion):
- 14:02 Changeset [58601] by
-
Fix 32-bit build (again).
- Plugins/WebNetscapeContainerCheckContextInfo.h:
- Plugins/WebNetscapeContainerCheckContextInfo.mm:
(-[WebNetscapeContainerCheckContextInfo initWithCheckRequestID:callbackFunc:context:]):
(-[WebNetscapeContainerCheckContextInfo checkRequestID]):
(-[WebNetscapeContainerCheckContextInfo callback]):
- Plugins/WebNetscapePluginPackage.h:
- 13:55 Changeset [58600] by
-
Fix build.
- bridge/npapi.h:
- 13:50 Changeset [58599] by
-
[chromium] Add WebFileSystem interface and hook up with all FileSystem methods.
https://bugs.webkit.org/show_bug.cgi?id=38228
Reviewed by Darin Fisher.
WebCore:
- platform/chromium/ChromiumBridge.h:
- platform/chromium/FileSystemChromium.cpp:
(WebCore::openFile):
(WebCore::closeFile):
(WebCore::seekFile):
(WebCore::truncateFile):
(WebCore::readFromFile):
(WebCore::writeToFile):
WebKit/chromium:
- WebKit.gyp:
- public/WebFileSystem.h: Added.
- public/WebKitClient.h:
(WebKit::WebKitClient::fileSystem):
- src/ChromiumBridge.cpp:
(WebCore::ChromiumBridge::fileExists):
(WebCore::ChromiumBridge::deleteFile):
(WebCore::ChromiumBridge::deleteEmptyDirectory):
(WebCore::ChromiumBridge::getFileSize):
(WebCore::ChromiumBridge::getFileModificationTime):
(WebCore::ChromiumBridge::directoryName):
(WebCore::ChromiumBridge::pathByAppendingComponent):
(WebCore::ChromiumBridge::makeAllDirectories):
(WebCore::ChromiumBridge::getAbsolutePath):
(WebCore::ChromiumBridge::isDirectory):
(WebCore::ChromiumBridge::filePathToURL):
(WebCore::ChromiumBridge::openFile):
(WebCore::ChromiumBridge::closeFile):
(WebCore::ChromiumBridge::seekFile):
(WebCore::ChromiumBridge::truncateFile):
(WebCore::ChromiumBridge::readFromFile):
(WebCore::ChromiumBridge::writeToFile):
- 13:49 Changeset [58598] by
-
2010-04-30 Anders Carlsson <andersca@apple.com>
Reviewed by Dan Bernstein.
Final part of
https://bugs.webkit.org/show_bug.cgi?id=20784
move npapi.h to C99 integer types
- bridge/npapi.h: Remove the old types.
- 13:25 Changeset [58597] by
-
2010-04-30 Sheriff Bot <webkit.review.bot@gmail.com>
Unreviewed, rolling out r58569.
http://trac.webkit.org/changeset/58569
https://bugs.webkit.org/show_bug.cgi?id=38399
This broke the GTK bots due to bad GC behavior (Requested by
ericu on #webkit).
- fast/workers/change-version-handle-reuse-worker-expected.txt: Removed.
- fast/workers/change-version-handle-reuse-worker.html: Removed.
- fast/workers/execute-sql-args-worker-expected.txt: Removed.
- fast/workers/execute-sql-args-worker.html: Removed.
- fast/workers/resources/database-worker-controller.js: Removed.
- fast/workers/resources/database-worker.js: Removed.
- storage/change-version-handle-reuse-expected.txt:
- storage/change-version-handle-reuse.html:
- storage/change-version-handle-reuse.js: Removed.
- storage/execute-sql-args.html:
- storage/execute-sql-args.js: Removed.
2010-04-30 Sheriff Bot <webkit.review.bot@gmail.com>
Unreviewed, rolling out r58569.
http://trac.webkit.org/changeset/58569
https://bugs.webkit.org/show_bug.cgi?id=38399
This broke the GTK bots due to bad GC behavior (Requested by
ericu on #webkit).
- bindings/js/JSWorkerContextCustom.cpp:
- bindings/v8/custom/V8WorkerContextCustom.cpp:
- storage/Database.idl:
- storage/SQLError.idl:
- storage/SQLResultSet.idl:
- storage/SQLResultSetRowList.idl:
- storage/SQLTransaction.idl:
- workers/WorkerContext.cpp: (WebCore::WorkerContext::openDatabase):
- workers/WorkerContext.h: (WebCore::WorkerContext::databaseExceededQuota):
- workers/WorkerContext.idl:
- 13:11 Changeset [58596] by
-
Fix 32-bit build.
- Plugins/WebNetscapeContainerCheckContextInfo.h:
- Plugins/WebNetscapeContainerCheckContextInfo.mm:
(-[WebNetscapeContainerCheckContextInfo initWithCheckRequestID:callbackFunc:context:]):
- Plugins/WebNetscapeContainerCheckPrivate.h:
- Plugins/WebNetscapePluginPackage.mm:
(functionPointerForTVector):
- Plugins/WebNetscapePluginView.mm:
(PluginTimer::PluginTimer):
(-[WebNetscapePluginView checkIfAllowedToLoadURL:frame:callbackFunc:context:]):
(-[WebNetscapePluginView scheduleTimerWithInterval:repeat:timerFunc:]):
- 12:36 Changeset [58595] by
-
2010-04-30 Yaar Schnitman <yaar@chromium.org>
Not Reviewed.
Updating expectations for svg-as-border-*
https://bugs.webkit.org/show_bug.cgi?id=38398
- platform/chromium-win/fast/borders/svg-as-border-image-2-expected.txt: Added.
- platform/chromium-win/fast/borders/svg-as-border-image-expected.txt: Added.
- platform/chromium-win/transforms/svg-vs-css-expected.txt: Added.
- 12:25 Changeset [58594] by
-
Not reviewed.
- platform/mac-tiger/Skipped: Added two tests that fail on buidbot.
- 12:25 Changeset [58593] by
-
Fix GTK+ build.
- plugins/gtk/PluginViewGtk.cpp:
(WebCore::PluginView::handlePostReadFile):
(WebCore::PluginView::platformGetValueStatic):
- 12:23 Changeset [58592] by
-
Fix build.
- DumpRenderTree/unix/TestNetscapePlugin/TestNetscapePlugin.cpp:
- 12:18 Changeset [58591] by
-
Fix Qt build.
- plugins/qt/PluginViewQt.cpp:
(WebCore::PluginView::handlePostReadFile):
(WebCore::PluginView::platformGetValue):
- 12:17 Changeset [58590] by
-
2010-04-30 Anders Carlsson <andersca@apple.com>
Reviewed by Timothy Hatcher.
Next step towards fixing
https://bugs.webkit.org/show_bug.cgi?id=20784
move npapi.h to C99 integer types
Use the C99 types everywhere. The "old" types are still around but will be removed
in a subsequent commit.
- Plugins/WebBaseNetscapePluginStream.h:
- Plugins/WebBaseNetscapePluginStream.mm: (WebNetscapePluginStream::deliverData):
- Plugins/WebNetscapePluginView.h:
- Plugins/WebNetscapePluginView.mm: (-[WebNetscapePluginView saveAndSetNewPortStateForUpdate:]): (-[WebNetscapePluginView getAuthenticationInfoWithProtocol:host:port:scheme:realm:username:usernameLength:password:passwordLength:]):
- Plugins/npapi.mm: (NPN_MemAlloc): (NPN_MemFlush): (NPN_PostURLNotify): (NPN_PostURL): (NPN_Write): (NPN_ScheduleTimer): (NPN_UnscheduleTimer): (NPN_GetValueForURL): (NPN_SetValueForURL): (NPN_GetAuthenticationInfo): (WKN_CheckIfAllowedToLoadURL): (WKN_CancelCheckIfAllowedToLoadURL):
2010-04-30 Anders Carlsson <andersca@apple.com>
Reviewed by Timothy Hatcher.
Next step towards fixing
https://bugs.webkit.org/show_bug.cgi?id=20784
move npapi.h to C99 integer types
Use the C99 types everywhere. The "old" types are still around but will be removed
in a subsequent commit.
- bridge/npapi.h: (_NPCocoaEvent::):
- plugins/PluginPackage.h:
- plugins/PluginStream.cpp: (WebCore::PluginStream::deliverData):
- plugins/PluginStream.h:
- plugins/PluginView.cpp: (WebCore::PluginView::postURLNotify): (WebCore::PluginView::postURL): (WebCore::PluginView::write): (WebCore::PluginView::handlePost):
- plugins/PluginView.h:
- plugins/PluginViewNone.cpp: (WebCore::PluginView::handlePostReadFile):
- plugins/npapi.cpp: (NPN_MemAlloc): (NPN_MemFlush): (NPN_PostURLNotify): (NPN_PostURL): (NPN_Write):
- plugins/npfunctions.h:
- plugins/win/PluginPackageWin.cpp: (WebCore::PluginPackage::NPVersion):
- plugins/win/PluginViewWin.cpp: (WebCore::PluginView::handlePostReadFile):
2010-04-30 Anders Carlsson <andersca@apple.com>
Reviewed by Timothy Hatcher.
Next step towards fixing
https://bugs.webkit.org/show_bug.cgi?id=20784
move npapi.h to C99 integer types
Use the C99 types everywhere. The "old" types are still around but will be removed
in a subsequent commit.
- DumpRenderTree/TestNetscapePlugIn.subproj/PluginObject.cpp: (int32VariantToIdentifier): (doubleVariantToIdentifier): (testIdentifierToInt): (testGetIntIdentifier):
- DumpRenderTree/TestNetscapePlugIn.subproj/main.cpp: (NPP_New): (NPP_NewStream): (NPP_WriteReady): (NPP_Write): (NPP_HandleEvent):
- DumpRenderTree/win/TestNetscapePlugin/main.cpp: (NPP_New): (NPP_NewStream): (NPP_WriteReady): (NPP_Write): (NPP_HandleEvent):
- 12:00 Changeset [58589] by
-
Make all image decoders set the "failed" bit if an image could not be
completely decoded, but no more data is coming. The ICO and BMP
decoders already did this.
https://bugs.webkit.org/show_bug.cgi?id=35411
Reviewed by David Levin.
"Failed" does not cause the image to not be displayed, it simply causes
us to not bother to try to decode again if future requests are made, and
for some decoders, lets the decoder clean up some of its temporary
objects.
No layout tests because this does not change the visible output of decoding in any way.
- platform/image-decoders/gif/GIFImageDecoder.cpp:
(WebCore::GIFImageDecoder::frameComplete): Return whether the frame could be marked as complete.
(WebCore::GIFImageDecoder::decode): Fail if read() needs more data (and thus returns false) and no more is coming.
- platform/image-decoders/gif/GIFImageDecoder.h:
- platform/image-decoders/gif/GIFImageReader.cpp:
(GIFImageReader::do_lzw): Instead of returning true for buffer underrun and false for failure, return false for both and set the failure flag on failure.
(GIFImageReader::read): Ditto.
- platform/image-decoders/gif/GIFImageReader.h:
- platform/image-decoders/jpeg/JPEGImageDecoder.cpp:
(WebCore::):
(WebCore::JPEGImageReader::decode): See do_lzw() comment above.
(WebCore::JPEGImageDecoder::decode): Fail if decode() needs more data (and thus returns false) and no more is coming.
- platform/image-decoders/jpeg/JPEGImageDecoder.h:
- platform/image-decoders/png/PNGImageDecoder.cpp:
(WebCore::PNGImageReader::decode): Return true for decode success, false for buffer underrun or decode failure, and set the failure flag on decode failure.
(WebCore::PNGImageDecoder::decode): See JPEGImageDecoder::decode() comment above.
- platform/image-decoders/png/PNGImageDecoder.h:
- 11:39 Changeset [58588] by
-
[Qt] fast/frames/flattening/frameset-flattening-subframesets.html fails intermittently on Qt bot
https://bugs.webkit.org/show_bug.cgi?id=37334
- platform/qt/Skipped: fast/frames/flattening/frameset-flattening-subframesets.html skipped until fix.
- 11:33 Changeset [58587] by
-
Not reviewed.
Enable Sputnik tests on Tiger.
- platform/mac-tiger/Skipped: Only skip tests that fail for me on Tiger/PPC, and Unicode tests.
- 11:24 Changeset [58586] by
-
<rdar://problem/7902467> - Audio plays upon loading of npr.org but shouldn't
Reviewed by Eric Carlson.
No new tests. (Currently no way to test such site specific hack behavior)
- html/HTMLMediaElement.cpp:
(WebCore::HTMLMediaElement::HTMLMediaElement):
(WebCore::HTMLMediaElement::asyncEventTimerFired): If the event to be dispatched is the canplay
event, wrap the dispatch with m_dispatchingCanPlayEvent set.
(WebCore::HTMLMediaElement::play): If m_dispatchingCanPlayEvent is true and the site is npr.org,
don't perform the play().
- html/HTMLMediaElement.h: Add m_dispatchingCanPlayEvent member.
- 10:50 Changeset [58585] by
-
WebCore: Part of <rdar://problem/6649734> Text repainting does not account for glyphs which draw outside the typographic bounds of the font
https://bugs.webkit.org/show_bug.cgi?id=6274
Reviewed by Adele Peterson.
Account for glyph overflow of characters in the range U+1E00..U+2000, but without sending them
through the complex text code path. Instead, introduce a variant of the fast path that tracks
glyph overflow.
- platform/graphics/Font.cpp:
(WebCore::Font::drawText): Use codePath().
(WebCore::Font::floatWidth): Use codePath(). Pass the GlyphOverflow pointer through to
floatWidthForSimpleText() if the code path is SimpleWithGlyphOverflow.
(WebCore::Font::selectionRectForText): Use codePath().
(WebCore::Font::offsetForPosition): Ditto.
- platform/graphics/Font.h: Replaced canUseGlyphCache() with codePath(). Added a GlyphOverflow
parameter to floatWidthForSimpleText().
- platform/graphics/FontFastPath.cpp:
Removed ROMAN_AND_GREEK_DIACRITICS_CAN_USE_GLYPH_CACHE.
(WebCore::Font::codePath): Renamed canUseGlyphCache() to this. Where it used to return false,
it now returns Complex. Where it used to return true, it now returns Simple, except for
the range U+1E00..U+2000, where it now returns SimpleWithGlyphOverflow.
(WebCore::Font::floatWidthForSimpleText): Added a GlyphOverflow parameter. If not 0, have the
width iterator account for glyph bounds, then update the GlyphOverflow accordingly.
- platform/graphics/WidthIterator.cpp:
(WebCore::WidthIterator::WidthIterator): Added boolean parameter telling the width iterator
whether to account for glyph bounds. Initialize m_accountForGlyphBounds accordingly. Initialize
m_maxGlyphBoundingBoxY, m_minGlyphBoundingBoxY, m_firstGlyphOverflow and m_lastGlyphOverflow.
(WebCore::WidthIterator::advance): If accounting for glyph bounds, update the above member variables.
- platform/graphics/WidthIterator.h:
(WebCore::WidthIterator::maxGlyphBoundingBoxY): Added this accessor.
(WebCore::WidthIterator::minGlyphBoundingBoxY): Ditto.
(WebCore::WidthIterator::firstGlyphOverflow): Ditto.
(WebCore::WidthIterator::lastGlyphOverflow): Ditto.
LayoutTests: Updated results for <rdar://problem/6649734> Text repainting does not account for glyphs which draw outside the typographic bounds of the font
https://bugs.webkit.org/show_bug.cgi?id=6274
Reviewed by Adele Peterson.
- platform/mac/fast/repaint/stacked-diacritics-expected.checksum:
- platform/mac/fast/repaint/stacked-diacritics-expected.png:
- 10:23 Changeset [58584] by
-
2010-04-30 Chris Marrin <cmarrin@apple.com>
Reviewed by Simon Fraser.
Reversed the order of the CSSMatrix.multiply method
https://bugs.webkit.org/show_bug.cgi?id=38337
- css/WebKitCSSMatrix.cpp: (WebCore::WebKitCSSMatrix::multiply):
- platform/graphics/transforms/TransformationMatrix.cpp:
These are the existing tests for 2D and 3D CSSMatrix and SVGMatrix.
I've added tests missing between the two styles and now do tests
that confirm that SVGMatrix and CSSMatrix both multiply the same.
This also adds a new test from Simon which compares SVG to CSS matrixes.
This is a pixel test for mac only.
- platform/mac/transforms/svg-vs-css-expected.checksum: Added.
- platform/mac/transforms/svg-vs-css-expected.png: Added.
- platform/mac/transforms/svg-vs-css-expected.txt: Added.
- svg/dom/SVGMatrix-interface-expected.txt:
- svg/dom/SVGMatrix-interface.xhtml:
- transforms/cssmatrix-2d-interface-expected.txt:
- transforms/cssmatrix-2d-interface.xhtml:
- transforms/cssmatrix-3d-interface-expected.txt:
- transforms/cssmatrix-3d-interface.xhtml:
- transforms/svg-vs-css.xhtml: Added.
- 09:56 Changeset [58583] by
-
Unreviewed. Attempt to fix the Chromium Mac build after the last commit.
- 09:35 Changeset [58582] by
-
Unreviewed, expectations update.
[Chromium] Provide expectations for svg/custom/repaint-stroke-width-changes.svg,
which was added in http://trac.webkit.org/changeset/58570/.
- platform/chromium-linux/svg/custom/repaint-stroke-width-changes-expected.checksum: Added.
- platform/chromium-mac/svg/custom/repaint-stroke-width-changes-expected.checksum: Added.
- platform/chromium-mac/svg/custom/repaint-stroke-width-changes-expected.png: Added.
- platform/chromium-win/svg/custom/repaint-stroke-width-changes-expected.checksum: Added.
- platform/chromium-win/svg/custom/repaint-stroke-width-changes-expected.png: Added.
- 09:33 Changeset [58581] by
-
Reviewed by Dan Bernstein.
Allow other ports to compile ATSUI and CoreText functions in SimpleFontData for Mac.
https://bugs.webkit.org/show_bug.cgi?id=38334
- 09:09 Changeset [58580] by
-
2010-04-30 Simon Fraser <simon.fraser@apple.com>
Reviewed by Beth Dakin.
Update image results for SVG as CSS border image tests
https://bugs.webkit.org/show_bug.cgi?id=37028
Improve the tests for SVG in border image to use an image that is not a solid color,
and by adding comparison elements with PNG-based borders.
Remove unused "green rect" SVG files.
Update Mac pixel results, and remove Chromium pixel results with the expectation
that someone will commit new ones.
- fast/borders/resources/green-fixed-size-rect.svg: Removed.
- fast/borders/resources/green-relative-size-rect.svg: Removed.
- fast/borders/resources/large-tiles.png: Added.
- fast/borders/resources/tiles-fixed-size.svg: Added.
- fast/borders/resources/tiles-relative-size.svg: Added.
- fast/borders/resources/tiles.png: Added.
- fast/borders/svg-as-border-image-2.html:
- fast/borders/svg-as-border-image.html:
- platform/chromium-win/fast/borders/svg-as-border-image-2-expected.checksum: Removed.
- platform/chromium-win/fast/borders/svg-as-border-image-2-expected.png: Removed.
- platform/chromium-win/fast/borders/svg-as-border-image-2-expected.txt: Removed.
- platform/chromium-win/fast/borders/svg-as-border-image-expected.checksum: Removed.
- platform/chromium-win/fast/borders/svg-as-border-image-expected.png: Removed.
- platform/chromium-win/fast/borders/svg-as-border-image-expected.txt: Removed.
- platform/mac/fast/borders/svg-as-border-image-2-expected.checksum:
- platform/mac/fast/borders/svg-as-border-image-2-expected.png:
- platform/mac/fast/borders/svg-as-border-image-2-expected.txt:
- platform/mac/fast/borders/svg-as-border-image-expected.checksum:
- platform/mac/fast/borders/svg-as-border-image-expected.png:
- platform/mac/fast/borders/svg-as-border-image-expected.txt:
- 08:53 Changeset [58579] by
-
Unreviewed, expectations tweak.
[Chromium] Defer Sputnik tests.
- platform/chromium/test_expectations.txt: Tagged fast/js/sputnik with DEFER.
- 08:46 Changeset [58578] by
-
2010-04-30 Shinichiro Hamaji <hamaji@chromium.org>
Reviewed by Simon Fraser.
SHOULD NEVER BE REACHED assertion loading forbes.com
https://bugs.webkit.org/show_bug.cgi?id=38272
- printing/pseudo-class-outside-page-expected.txt: Added.
- printing/pseudo-class-outside-page.html: Added.
2010-04-30 Shinichiro Hamaji <hamaji@chromium.org>
Reviewed by Simon Fraser.
SHOULD NEVER BE REACHED assertion loading forbes.com
https://bugs.webkit.org/show_bug.cgi?id=38272
Ignore page media related pseudo classes.
Test: printing/pseudo-class-outside-page.html
- css/CSSStyleSelector.cpp: (WebCore::CSSStyleSelector::SelectorChecker::checkOneSelector):
- 08:39 QtWebKitTriageRoster edited by
- (diff)
- 08:22 Changeset [58577] by
-
Changed Steve Block from committer to reviewer.
- 08:11 Changeset [58576] by
-
2010-04-30 Nikolas Zimmermann <nzimmermann@rim.com>
Not reviewed. While I'm at it: Update prototype-inheritance-2-expected.txt result, it's failing since CSSPageRule has been introduced.
- platform/win/fast/dom/prototype-inheritance-2-expected.txt:
- 08:08 Changeset [58575] by
-
2010-04-30 Nikolas Zimmermann <nzimmermann@rim.com>
Not reviewed. Windows fails on svg/custom/mask-changes.svg. Trying to fix the test which is really flawed. Shows a DRT difference between Mac/Win.
- svg/custom/mask-changes.svg:
- 08:08 WebKit Team edited by
- (diff)
- 06:34 Changeset [58574] by
-
2010-04-30 Jeremy Orlow <jorlow@chromium.org>
Unreviewed. Fixing a missed test from https://bugs.webkit.org/show_bug.cgi?id=38331
- fast/events/init-events-expected.txt:
- fast/events/script-tests/init-events.js:
- 06:31 Changeset [58573] by
-
2010-04-30 Laszlo Gombos <laszlo.1.gombos@nokia.com>
Unreviewed, build fix.
Fix compiler warning "suggest parentheses around"
No new tests as there is no new functionality.
- svg/SVGAnimateElement.cpp: (WebCore::SVGAnimateElement::calculateFromAndToValues):
- 04:49 Changeset [58572] by
-
WebCore: Unreviewed. Regression fix.
Revert a part of r58564 to be compatible with prior behavior
https://bugs.webkit.org/show_bug.cgi?id=38383
- rendering/RenderTextControlSingleLine.cpp:
(WebCore::RenderTextControlSingleLine::forwardEvent):
r58564 made a region check for the cancel button stricter, but it
made some tests failing on Chromium. So, relax the check again.
LayoutTests: Unreviewed.
Revert a part of r58564 to be compatible with prior behavior
https://bugs.webkit.org/show_bug.cgi?id=38383
- platform/chromium/test_expectations.txt: Remove expectations added by r58568.
- 04:26 Changeset [58571] by
-
2010-04-29 Jeremy Orlow <jorlow@chromium.org>
Reviewed by Darin Fisher.
Change StorageEvent.uri to StorageEvent.url to match the spec
https://bugs.webkit.org/show_bug.cgi?id=38331
Change the existing tests to test 'url' rather than 'uri'.
- storage/domstorage/events/documentURI-expected.txt:
- storage/domstorage/events/script-tests/documentURI.js: (step2): (step3):
- storage/domstorage/localstorage/resources/iframe-events-second.html:
- storage/domstorage/sessionstorage/resources/iframe-events-second.html:
2010-04-29 Jeremy Orlow <jorlow@chromium.org>
Reviewed by Darin Fisher.
Change StorageEvent.uri to StorageEvent.url to match the spec
https://bugs.webkit.org/show_bug.cgi?id=38331
As I mentioned in http://www.mail-archive.com/public-webapps@w3.org/msg08495.html
WebKit is the only one who places the document's URL in a 'uri' property
rather than a 'url' property. Even though we've shipped several versions of
browsers with the old name, we probably should change this to comply with the
spec.
This stuff is covered by existing tests.
- storage/StorageEvent.cpp: (WebCore::StorageEvent::create): (WebCore::StorageEvent::StorageEvent): (WebCore::StorageEvent::initStorageEvent):
- storage/StorageEvent.h: (WebCore::StorageEvent::url):
- storage/StorageEvent.idl:
2010-04-29 Jeremy Orlow <jorlow@chromium.org>
Reviewed by Darin Fisher.
Change StorageEvent.uri to StorageEvent.url to match the spec
https://bugs.webkit.org/show_bug.cgi?id=38331
Get rid of a stale FIXME and combine 2 lines that needn't be split.
- src/StorageEventDispatcherImpl.cpp: (WebCore::StorageEventDispatcherImpl::dispatchStorageEvent):
- 04:22 Changeset [58570] by
-
2010-04-30 Nikolas Zimmermann <nzimmermann@rim.com>
Reviewed by Dirk Schulze.
REGRESSION: RenderPath does not handle repaints correctly anymore if bounds changed
https://bugs.webkit.org/show_bug.cgi?id=38385
The last SVG performance patch broke repainting if bounds of a RenderPath get smaller.
It would only repaint the smaller part, not the original larger bounds.
Remove all lazy calculation of the repaint rects, instead calculate object/strokeBoundingBox and repaintRectInLocalCoordinates
once in layout - after LayoutRepainter grabbed the initial bounds, before calling repaintAfterLayout(). We can now inline
all these functions, and save a lot of m_path.isEmpty() checks, which are expensive. No need to store a seperated markerBoundingBox(),
combine with strokeBoundingBox() -> save one FloatRect per RenderPath. Move strokeBoundingBox() from SVGRenderBase to RenderObject,
right next to objectBoundingBox() - to save unnecessary toSVGRenderBase() calls. Completly remove this method.
Overall this is a regression fix, a performance improvement and saves memory. Something for everyone.
Tests: svg/custom/repaint-stroke-width-changes.svg
- rendering/RenderObject.cpp: Added strokeBoundingBox() here, to avoid the toSVGRenderBase() dance. (WebCore::RenderObject::strokeBoundingBox):
- rendering/RenderObject.h: Ditto.
- rendering/RenderPath.cpp: (WebCore::RenderPath::RenderPath): (WebCore::RenderPath::layout): Fix regression, do repainting correctly, by recalculating the boundaries, if needed, instead of nulling them. (WebCore::RenderPath::paint): Cache SVGRenderStyle in local variable, remove no longer valid FIXME. (WebCore::RenderPath::calculateMarkerBoundsIfNeeded): Return a FloatRect, to avoid having to store the marker bounding box seperated. (WebCore::RenderPath::styleWillChange): Mark boundaries as dirty. (WebCore::RenderPath::updateCachedBoundaries): New function to (re-)calculate all cached boundaries, only called from layout().
- rendering/RenderPath.h: Rename cached rect variables to have more sensible names. (WebCore::RenderPath::objectBoundingBox): Inlined, just returns the cached value - no more lazy creation. Huge speedup as this is hot code. (WebCore::RenderPath::strokeBoundingBox): Ditto. (WebCore::RenderPath::repaintRectInLocalCoordinates): Ditto.
- rendering/RenderSVGBlock.h: Remove toSVGRenderBase() method.
- rendering/RenderSVGImage.h: Ditto.
- rendering/RenderSVGInline.cpp: No need to call toSVGRenderBase() just to get the strokeBoundingBox(). Unifies code to retrieve bounding boxes. (WebCore::RenderSVGInline::strokeBoundingBox):
- rendering/RenderSVGInline.h: Remove toSVGRenderBase() method.
- rendering/RenderSVGModelObject.h: Ditto.
- rendering/RenderSVGResourceFilter.cpp: No need to call toSVGRenderBase() anymore, just grab the strokeBoundingBox() from the RenderObject. (WebCore::RenderSVGResourceFilter::applyResource):
- rendering/RenderSVGText.h: Remove toSVGRenderBase() method.
- rendering/SVGRenderSupport.h: Ditto. Remove markerBoundingBox() method, now combined with strokeBoundingBox(). (WebCore::SVGRenderBase::strokeBoundingBox):
- rendering/SVGRootInlineBox.h: Remove toSVGRenderBase() method.
- rendering/style/SVGRenderStyle.h: Add hasMarkers() helper method, to avoid doing unnecessary work in RenderPath. (WebCore::SVGRenderStyle::hasMarkers):
2010-04-30 Nikolas Zimmermann <nzimmermann@rim.com>
Reviewed by Dirk Schulze.
REGRESSION: RenderPath does not handle repaints correctly anymore if bounds changed
https://bugs.webkit.org/show_bug.cgi?id=38385
Add new test covering the repaint problem. Influences some marker results, that can be safely ignored.
- platform/mac/svg/custom/circular-marker-reference-1-expected.txt:
- platform/mac/svg/custom/circular-marker-reference-3-expected.txt:
- platform/mac/svg/custom/circular-marker-reference-4-expected.txt:
- platform/mac/svg/custom/repaint-stroke-width-changes-expected.checksum: Added.
- platform/mac/svg/custom/repaint-stroke-width-changes-expected.png: Added.
- platform/mac/svg/custom/repaint-stroke-width-changes-expected.txt: Added.
- svg/custom/repaint-stroke-width-changes.svg: Added.
- 04:11 Changeset [58569] by
-
2010-04-30 Eric Uhrhane <ericu@chromium.org>
Reviewed by Dmitry Titov.
Add bindings for async DB API in Workers.
https://bugs.webkit.org/show_bug.cgi?id=34992
- storage/change-version-handle-reuse.html: Refactored to share code with the worker version of the test.
- storage/change-version-handle-reuse-expected.html: Removed reference to reloading, which this test doesn't do.
- storage/change-version-handle-reuse.js: This is the extracted shared core of the test.
- storage/execute-sql-args.html: Refactored to share code with the worker version of the test.
- storage/execute-sql-args.js: This is the extracted shared core of the test.
These are the worker versions of the tests; they're tiny shims around shared code.
- fast/workers/execute-sql-args-worker-expected.txt: Added.
- fast/workers/execute-sql-args-worker.html: Added.
- fast/workers/change-version-handle-reuse-worker-expected.txt: Added.
- fast/workers/change-version-handle-reuse-worker.html: Added. This file gets included from worker database tests; it does all the boilerplate that's needed on the DOM side.
- fast/workers/resources/database-worker-controller.js: Added. This file is the boilerplate for the worker side of the database tests. It will import and run the actual test on command, and will report back completion and exceptions.
- fast/workers/resources/database-worker.js: Added.
2010-04-30 Eric Uhrhane <ericu@chromium.org>
Reviewed by Dmitry Titov.
Add bindings for async DB API in Workers.
https://bugs.webkit.org/show_bug.cgi?id=34992
Tests: storage/change-version-handle-reuse-worker.html
storage/execute-sql-args-worker.html
- bindings/js/JSWorkerContextCustom.cpp: Add openDatabase binding. (WebCore::JSWorkerContext::openDatabase):
- bindings/v8/custom/V8WorkerContextCustom.cpp: Add openDatabase stub; Chromium will need work both in V8 and in the browser process before we can turn this on there. (WebCore::V8WorkerContext::openDatabaseCallback):
Add NoStaticTables flags to all objects now shared with workers.
- storage/Database.idl:
- storage/SQLError.idl:
- storage/SQLResultSet.idl:
- storage/SQLResultSetRowList.idl:
- storage/SQLTransaction.idl:
- workers/WorkerContext.h: Add databaseExceededQuota.
- workers/WorkerContext.cpp: (WebCore::WorkerContext::databaseExceededQuota): Add stub implementation for testing; you just get 5MB for now. (WebCore::WorkerContext::openDatabase): Remove invalid assertion.
Add the IDL for the call to openDatabase.
- workers/WorkerContext.idl:
- 03:06 Changeset [58568] by
-
Unreviewd, update test expectations.
https://bugs.webkit.org/show_bug.cgi?id=38383
Start failing after WebKit r58563:r58565
- fast/forms/search-abs-pos-cancel-button.html
- fast/forms/search-cancel-button-mouseup.html
- fast/forms/search-rtl.html
- fast/forms/search-zoomed.html
- platform/chromium/test_expectations.txt:
- 02:24 Changeset [58567] by
-
Unreviewed. Fix wrong bug numbers in ChangeLogs for r58564.
- 02:14 Changeset [58566] by
-
2010-04-30 Shinichiro Hamaji <hamaji@chromium.org>
Reviewed by Darin Adler.
Unnecessary PrintContext::end() calls
https://bugs.webkit.org/show_bug.cgi?id=38247
Refactoring only, so no new tests.
- page/PrintContext.cpp: (WebCore::PrintContext::pageNumberForElement): (WebCore::PrintContext::numberOfPages):
- 01:48 Changeset [58565] by
-
Unreviewd, test expectations cleanup.
media/video-empty-source.html passes since r58480:r58515 (win), r58489 (linux), r58488:458490 (mac)
fast/backround/svg-as-background-6.html passes since r58488:r58490 (mac)
fast/repaint/caret-outside-block.html passes since r58414:r58415 (mac)
fast/replaced/border-radius-clip.html passes since r58543 (win, linux)
fast/images/svg-as-tiled-background.html passes r58488:r58490 (mac)
fast/text/line-breaks.html passes since r58480:r58515 (win)
fast/backgrounds/svg-as-background-2.html passes since r58480:r58515 (win), r58489 (linux), r58488:r58490 (mac)
- platform/chromium/test_expectations.txt:
- 01:45 Changeset [58564] by
-
Implement interactive behavior of spin buttons.
https://bugs.webkit.org/show_bug.cgi?id=27968
Reviewed by Adele Peterson.
WebCore:
Introduce SpinButtonElement. It is a shadow element class for
spin buttons. If the upper side of the element is clicked, calls
HTMLInputElement::stepUpFromRenderer(1). If the lower button is
clicked, calls HTMLInputElement::stepUpFromRenderer(-1).
SpinButtonElement tracks the mouse pointer position, and
RenderTheme sets ControlStates::SpinUpState if the pointer is on
the upper side.
Test: platform/mac/fast/forms/input-number-click.html
- dom/Element.h:
(WebCore::Element::isSpinButtonElement):
- editing/VisibleSelection.cpp:
(WebCore::VisibleSelection::adjustSelectionToAvoidCrossingEditingBoundaries):
Remove an assertion. lastEditablePositionBeforePositionInRoot() can
return null in a case that m_end is at a shadow element (a spin button)
and baseRoot is another shadow element (inner text block) in the same
node (an INPUT element).
- html/HTMLInputElement.cpp:
(WebCore::HTMLInputElement::stepUpFromRenderer):
- html/HTMLInputElement.h:
(WebCore::HTMLInputElement::hasSpinButton):
Add types supporting step attribute except RANGE.
- rendering/RenderTextControlSingleLine.cpp:
(WebCore::RenderTextControlSingleLine::nodeAtPoint):
(WebCore::RenderTextControlSingleLine::forwardEvent):
(WebCore::RenderTextControlSingleLine::preferredContentWidth):
(WebCore::RenderTextControlSingleLine::createSubtreeIfNeeded):
- rendering/RenderTextControlSingleLine.h:
- rendering/RenderTheme.cpp:
(WebCore::RenderTheme::controlStatesForRenderer):
(WebCore::RenderTheme::isSpinUpButtonPartPressed):
(WebCore::RenderTheme::isSpinUpButtonPartHovered):
- rendering/RenderTheme.h:
- rendering/TextControlInnerElements.cpp:
(WebCore::SpinButtonElement::SpinButtonElement):
(WebCore::SpinButtonElement::defaultEventHandler):
- rendering/TextControlInnerElements.h:
(WebCore::SpinButtonElement::isSpinButtonElement):
(WebCore::SpinButtonElement::isEnabledFormControl):
(WebCore::SpinButtonElement::onUpButton):
LayoutTests:
- platform/mac/fast/forms/input-number-click-expected.txt: Added.
- platform/mac/fast/forms/input-number-click.html: Added.
- 01:41 Changeset [58563] by
-
2010-04-30 Yael Aharon <yael.aharon@nokia.com>
Reviewed by Kenneth Rohde Christiansen.
[Qt] Enable DOMWindow constructor for HTMLProgressElement
https://bugs.webkit.org/show_bug.cgi?id=38333
Update expected test results after this change
- platform/qt/fast/dom/Window/window-properties-expected.txt:
- platform/qt/fast/dom/Window/window-property-descriptors-expected.txt:
- platform/qt/fast/dom/prototype-inheritance-expected.txt:
- platform/qt/fast/js/global-constructors-expected.txt:
2010-04-30 Yael Aharon <yael.aharon@nokia.com>
Reviewed by Kenneth Rohde Christiansen.
[Qt] Enable DOMWindow constructor for HTMLProgressElement
https://bugs.webkit.org/show_bug.cgi?id=38333
Add ENABLE_PROGRESS_TAG to FEATURES_DEFINES_JAVASCRIPT.
- WebCore.pri:
- 00:56 Changeset [58562] by
-
2010-04-29 Oliver Hunt <oliver@apple.com>
Reviewed by Gavin Barraclough.
Add codegen support for unsigned right shift
https://bugs.webkit.org/show_bug.cgi?id=38375
Expose unsigned right shift in the macro assembler, and make use of it
from the jit. Currently if the result is outside the range 0..231-1
we simply fall back to the slow case, even in JSVALUE64 and JSVALUE32_64
where technically we could still return an immediate value.
- assembler/MacroAssemblerARM.h: (JSC::MacroAssemblerARM::urshift32):
- assembler/MacroAssemblerARMv7.h: (JSC::MacroAssemblerARMv7::urshift32):
- assembler/MacroAssemblerX86Common.h: (JSC::MacroAssemblerX86Common::urshift32):
- assembler/X86Assembler.h: (JSC::X86Assembler::): (JSC::X86Assembler::shrl_i8r): (JSC::X86Assembler::shrl_CLr):
Add unsigned right shift to the x86 assembler
- jit/JIT.cpp: (JSC::JIT::privateCompileMainPass): (JSC::JIT::privateCompileSlowCases):
op_rshift no longer simply get thrown to a stub function
- jit/JIT.h:
- jit/JITArithmetic.cpp: (JSC::JIT::emit_op_urshift): (JSC::JIT::emitSlow_op_urshift): JSVALUE32 and JSVALUE64 implementation. Only supports double lhs in JSVALUE64.
- jit/JITArithmetic32_64.cpp: (JSC::JIT::emit_op_rshift): (JSC::JIT::emitSlow_op_rshift): (JSC::JIT::emit_op_urshift): (JSC::JIT::emitSlow_op_urshift): Refactor right shift code to have shared implementation between signed and unsigned versions.
- 00:45 Changeset [58561] by
-
2010-04-30 Kent Tamura <tkent@chromium.org>
Reviewed by Adele Peterson.
Mac implementation of outer-spin-button appearance, and anonymous
element generation for <input type=number>.
https://bugs.webkit.org/show_bug.cgi?id=32813
input-appearance-spinbutton.html checks the appearances of various
sizes of spin-buttons. input-appearance-spinbutton-size.html
checks a spin-button height is equal to or less than the
corresponding text field height.
- platform/mac/fast/forms/input-appearance-spinbutton-expected.checksum: Added.
- platform/mac/fast/forms/input-appearance-spinbutton-expected.png: Added.
- platform/mac/fast/forms/input-appearance-spinbutton-expected.txt: Added.
- platform/mac/fast/forms/input-appearance-spinbutton-size-expected.txt: Added.
- platform/mac/fast/forms/input-appearance-spinbutton-size.html: Added.
- platform/mac/fast/forms/input-appearance-spinbutton.html: Added.
2010-04-30 Kent Tamura <tkent@chromium.org>
Reviewed by Adele Peterson.
Mac implementation of outer-spin-button appearance, and anonymous
element generation for <input type=number>.
https://bugs.webkit.org/show_bug.cgi?id=32813
The implementation uses NSStepperCell. Like the other Mac
controls, it has only three candidates for sizes.
The editable block of an input element is shrunk, and the
anonymous block for a spin button is put on the right of the
editable block.
Tests: platform/mac/fast/forms/input-appearance-spinbutton-size.html
platform/mac/fast/forms/input-appearance-spinbutton.html
- dom/Element.cpp: (WebCore::Element::pseudoStyleCacheIsInvalid):
- dom/InputElement.h: (WebCore::InputElement::hasSpinButton):
- html/HTMLInputElement.h: (WebCore::HTMLInputElement::hasSpinButton): Return true for NUMBER type.
- platform/mac/ThemeMac.mm: (WebCore::sizeFromNSControlSize): Split the main part of sizeFromFont() to this in order to use stepperControlSizeForFont() instead of controlSizeForFont(). (WebCore::sizeFromFont): Just calls sizeFromNSControlSize() with sizeFromFont(). No behavior changes. (WebCore::stepperSizes): Returns sizes for mini, small, and regular. (WebCore::stepperControlSizeForFont): Dedicated version of controlSizeForFont(). (WebCore::stepper): Returns NSStepperCell object with specified settings. (WebCore::paintStepper): (WebCore::ThemeMac::controlSize): Support for OuterSpinButton. (WebCore::ThemeMac::minimumControlSize): ditto. (WebCore::ThemeMac::inflateControlPaintRect): ditto. (WebCore::ThemeMac::paint): ditto.
- rendering/RenderBox.cpp: (WebCore::RenderBox::paintBoxDecorations): Move the content to paintBoxDecorationsWithSize(). (WebCore::RenderBox::paintBoxDecorationsWithSize):
- rendering/RenderBox.h: Declare paintBoxDecorationsWithSize().
- rendering/RenderTextControlSingleLine.cpp: (WebCore::RenderTextControlSingleLine::~RenderTextControlSingleLine): (WebCore::RenderTextControlSingleLine::paintBoxDecorations): Call paintBoxDecorationsWithSize() with smaller width by decorationWidthRight(). (WebCore::RenderTextControlSingleLine::addFocusRingRects): Add a rectangle of which width is smaller by decorationWidthRight(). (WebCore::RenderTextControlSingleLine::layout): Adjust m_outerSpinButton position. (WebCore::RenderTextControlSingleLine::styleDidChange): (WebCore::RenderTextControlSingleLine::textBlockWidth): (WebCore::RenderTextControlSingleLine::decorationWidthRight): (WebCore::RenderTextControlSingleLine::preferredDecorationWidthRight): (WebCore::RenderTextControlSingleLine::createSubtreeIfNeeded): Creates an element for m_outerSpinButton if it is needed. (WebCore::RenderTextControlSingleLine::createInnerTextStyle): (WebCore::RenderTextControlSingleLine::createOuterSpinButtonStyle):
- rendering/RenderTextControlSingleLine.h: Declare new methods and m_outerSpinButton.
- rendering/RenderThemeMac.mm: (WebCore::RenderThemeMac::adjustRepaintRect): Support for OuterSpinButton.
