Timeline
Dec 24, 2010:
- 10:00 PM Changeset in webkit [74658] by
-
- 2 edits in trunk/LayoutTests
2010-12-24 Kenneth Russell <kbr@google.com>
Unreviewed, Chromium test expectations update.
svg/batik/paints/gradientLimit.svg fails on Linux Debug as well.
- platform/chromium/test_expectations.txt:
- 5:52 PM Changeset in webkit [74657] by
-
- 6 edits in trunk
REGRESSION (r74648): XHR layout test failures
https://bugs.webkit.org/show_bug.cgi?id=51603
Reviewed by Alexey Proskuryakov.
Reverted r74648.
WebCore:
- WebCore.exp.in:
WebKit2:
- UIProcess/WebPageProxy.cpp:
(WebKit::WebPageProxy::didReceiveAuthenticationChallenge):
- WebProcess/WebCoreSupport/WebFrameLoaderClient.cpp:
(WebKit::WebFrameLoaderClient::shouldUseCredentialStorage):
- WebProcess/mac/WebProcessMainMac.mm:
(WebKit::WebProcessMain):
- 2:33 PM Changeset in webkit [74656] by
-
- 2 edits in trunk/LayoutTests
2010-12-24 Philippe Normand <pnormand@igalia.com>
Unreviewed, skip failing media test on GTK.
- platform/gtk/Skipped: Skip media/controls-after-reload.html.
- 1:43 PM Changeset in webkit [74655] by
-
- 3 edits in trunk/Tools
2010-12-24 Sheriff Bot <webkit.review.bot@gmail.com>
Unreviewed, rolling out r74632.
http://trac.webkit.org/changeset/74632
https://bugs.webkit.org/show_bug.cgi?id=51600
makes test-webkitpy hang forever on GTK (Requested by philn-tp
on #webkit).
- Scripts/webkitpy/layout_tests/layout_package/message_broker.py:
- Scripts/webkitpy/layout_tests/layout_package/message_broker_unittest.py:
- 1:26 PM Changeset in webkit [74654] by
-
- 2 edits in trunk/WebKit2
2010-12-24 Simon Fraser <Simon Fraser>
Fix the Qt build after r74650.
- WebProcess/WebCoreSupport/WebPlatformStrategies.cpp:
- 1:24 PM Changeset in webkit [74653] by
-
- 1 edit2 adds in trunk/LayoutTests
2010-12-24 Andrei Popescu <andreip@google.com>
Unreviewed, build fix.
[Chromium] http/tests/xmlhttprequest/basic-auth-nopassword.html needs Mac, Windows and Linux specific test expectations
https://bugs.webkit.org/show_bug.cgi?id=51589
- platform/chromium-win/http/tests/xmlhttprequest/basic-auth-nopassword-expected.txt: Added.
- platform/chromium-win/http/tests/xmlhttprequest/basic-auth-nouser-expected.txt: Added.
- 1:13 PM Changeset in webkit [74652] by
-
- 3 edits1 add in trunk/WebCore
Try to fix the Qt build after r74648.
- WebCore.gypi:
- WebCore.pro:
- platform/network/qt/CredentialStorageQt.cpp: Added.
(WebCore::CredentialStorage::getFromPersistentStorage):
- 12:57 PM Changeset in webkit [74651] by
-
- 9 edits in trunk/LayoutTests
2010-12-24 Philippe Normand <pnormand@igalia.com>
Unreviewed, updated GTK media tests baselines after r74643.
- platform/gtk/media/audio-controls-rendering-expected.txt:
- platform/gtk/media/controls-after-reload-expected.txt:
- platform/gtk/media/controls-strict-expected.txt:
- platform/gtk/media/controls-styling-expected.txt:
- platform/gtk/media/video-controls-rendering-expected.txt:
- platform/gtk/media/video-display-toggle-expected.txt:
- platform/gtk/media/video-empty-source-expected.txt:
- platform/gtk/media/video-no-audio-expected.txt:
- 12:55 PM Changeset in webkit [74650] by
-
- 2 edits in trunk/WebKit2
2010-12-24 Benjamin Poulain <ikipou@gmail.com>
Reviewed by Darin Adler.
[Qt] [WK2] WebKit 2 does not build on mac, missing reference to RetainPtr
https://bugs.webkit.org/show_bug.cgi?id=51598
Fix the build, add the missing header.
- WebProcess/WebCoreSupport/WebPlatformStrategies.cpp:
- 12:46 PM Changeset in webkit [74649] by
-
- 3 edits in trunk/WebCore
2010-12-24 Simon Fraser <Simon Fraser>
Reviewed by Dan Bernstein.
CSSStyleSelector cleanup
https://bugs.webkit.org/show_bug.cgi?id=51597
Make some methods const. Add assertion to checkOneSelector, with the
intention of later removing the if (!e) check (this is hot code).
- css/CSSStyleSelector.cpp: (WebCore::CSSStyleSelector::locateCousinList): (WebCore::CSSStyleSelector::canShareStyleWithElement): (WebCore::CSSStyleSelector::locateSharedStyle): (WebCore::CSSStyleSelector::SelectorChecker::checkOneSelector): (WebCore::CSSStyleSelector::mmLength): (WebCore::CSSStyleSelector::inchLength): (WebCore::CSSStyleSelector::getColorFromPrimitiveValue): (WebCore::CSSStyleSelector::hasSelectorForAttribute):
- css/CSSStyleSelector.h: (WebCore::CSSStyleSelector::fontSelector):
- 12:24 PM Changeset in webkit [74648] by
-
- 6 edits in trunk
<rdar://problem/8758386> The web process uses its own credential storage
https://bugs.webkit.org/show_bug.cgi?id=51599
Reviewed by Anders Carlsson.
WebCore:
- WebCore.exp.in: Export CredentialStorage::getFromPersistentStorage(), Credential::hasPassword(),
and Credential::isEmpty().
WebKit2:
- UIProcess/WebPageProxy.cpp:
(WebKit::WebPageProxy::didReceiveAuthenticationChallenge): Try to answer the challenge using
the UI process’s credential storage first.
- WebProcess/WebCoreSupport/WebFrameLoaderClient.cpp:
(WebKit::WebFrameLoaderClient::shouldUseCredentialStorage): Prevent the web process from using
its own credential storage.
- WebProcess/mac/WebProcessMainMac.mm:
(WebKit::WebProcessMain): Ditto.
- 11:36 AM Changeset in webkit [74647] by
-
- 1 edit1 copy in trunk/LayoutTests
Rebased Leopard to expect the same kind of stack failure
as Windows, rather than the success expected on Snow Leopard
and many other platforms.
- platform/mac-leopard/fast/js/large-expressions-expected.txt:
Copied from platform/win/fast/js/large-expressions-expected.txt.
- 11:36 AM Changeset in webkit [74646] by
-
- 9 edits2 moves in trunk/WebCore
2010-12-24 Yury Semikhatsky <yurys@chromium.org>
Reviewed by Darin Adler.
[V8] Rename V8CustomEventListener.{h,cpp} to V8EventListener.{h,cpp}
https://bugs.webkit.org/show_bug.cgi?id=51595
- WebCore.gypi:
- bindings/v8/V8DOMWrapper.cpp:
- bindings/v8/V8EventListener.cpp: Renamed from WebCore/bindings/v8/custom/V8CustomEventListener.cpp. (WebCore::V8EventListener::V8EventListener): (WebCore::V8EventListener::getListenerFunction): (WebCore::V8EventListener::callListenerFunction):
- bindings/v8/V8EventListener.h: Renamed from WebCore/bindings/v8/custom/V8CustomEventListener.h. (WebCore::V8EventListener::create):
- bindings/v8/V8EventListenerList.h:
- bindings/v8/V8WorkerContextEventListener.h:
- bindings/v8/custom/V8DOMWindowCustom.cpp:
- bindings/v8/custom/V8LocationCustom.cpp:
- bindings/v8/custom/V8NodeCustom.cpp:
- bindings/v8/custom/V8NotificationCenterCustom.cpp:
- 11:26 AM Changeset in webkit [74645] by
-
- 1 edit2 adds in trunk/LayoutTests
Add Windows-specific (failing) expected results for
http/tests/xmlhttprequest/basic-auth-nouser.html and basic-auth-nopassword.html in order to
get the Windows 7 Release test bot green again. Unreviewed.
https://bugs.webkit.org/show_bug.cgi?id=51596
- platform/win/http/tests/xmlhttprequest/basic-auth-nopassword-expected.txt: Added.
- platform/win/http/tests/xmlhttprequest/basic-auth-nouser-expected.txt: Added.
- 11:26 AM Changeset in webkit [74644] by
-
- 2 edits1 add in trunk
Updated makefiles used by Apple engineers to fix build with the
new JavaScriptGlue location.
- Makefile: Told this to build JavaScriptGlue. Some day we might
want it to descend into Sources instead and use the Makefile there,
but there are problems with sequencing the project builds, so that
can wait.
- Sources/Makefile.shared: Added.
- 11:02 AM Changeset in webkit [74643] by
-
- 4 edits in trunk/WebCore
2010-12-24 Carlos Garcia Campos <cgarcia@igalia.com>
Reviewed by Xan Lopez.
[GTK] Show current time / total in media player
https://bugs.webkit.org/show_bug.cgi?id=51535
- css/mediaControlsGtk.css: (audio::-webkit-media-controls-current-time-display, video::-webkit-media-controls-current-time-display):
- platform/gtk/RenderThemeGtk.cpp: (WebCore::RenderThemeGtk::formatMediaControlsCurrentTime): (WebCore::RenderThemeGtk::paintMediaCurrentTime):
- platform/gtk/RenderThemeGtk.h:
- 10:55 AM Changeset in webkit [74642] by
-
- 3 edits in trunk/WebCore
2010-12-24 Jan Erik Hanssen <jhanssen@sencha.com>
Reviewed by Eric Seidel.
Clean up CSSRuleData in CSSStyleSelector.h
https://bugs.webkit.org/show_bug.cgi?id=27753
Move CSSRuleData and CSSRuleDataList from CSSStyleSelector.h to the .cpp file.
- css/CSSStyleSelector.cpp: (WebCore::CSSRuleData::CSSRuleData): (WebCore::CSSRuleData::~CSSRuleData): (WebCore::CSSRuleData::position): (WebCore::CSSRuleData::rule): (WebCore::CSSRuleData::selector): (WebCore::CSSRuleData::next): (WebCore::CSSRuleDataList::CSSRuleDataList): (WebCore::CSSRuleDataList::~CSSRuleDataList): (WebCore::CSSRuleDataList::first): (WebCore::CSSRuleDataList::last): (WebCore::CSSRuleDataList::append):
- css/CSSStyleSelector.h:
- 10:54 AM Changeset in webkit [74641] by
-
- 2 edits in trunk/LayoutTests
Skip the new editing/input/ime-composition-clearpreedit.html test since textInputController
is unimplemented in the DRT for Windows. Unreviewed.
https://bugs.webkit.org/show_bug.cgi?id=32021
- platform/win/Skipped:
- 10:43 AM Changeset in webkit [74640] by
-
- 2 edits in trunk/Tools
2010-12-24 Ryuan Choi <ryuan.choi@samsung.com>
Reviewed by Kenneth Rohde Christiansen.
[EFL] Screen doesn't be rendered when changing size of EWebLauncher
https://bugs.webkit.org/show_bug.cgi?id=43528
Remove on_resize because ewk_view_fixed_layout_size_set should not be
called on callback of EVAS_CALLBACK_RESIZE. As it change size of
FrameView, rendering was ignored when _ewk_view_smart_calculate was
called.
- EWebLauncher/main.c: (browserCreate):
- 10:21 AM Changeset in webkit [74639] by
-
- 12 edits2 copies in trunk/Tools
2010-12-24 Eric Seidel <eric@webkit.org>
Reviewed by Adam Barth.
webkit-patch (or a pre-commit hook) needs to prevent bad ChangeLog changes
https://bugs.webkit.org/show_bug.cgi?id=28291
This is a start. At least now webkit-patch will prompt when your ChangeLog looks questionable.
We could do more advanced things, like parsing the ChangeLog (with changelog.py) and comparing that
to strings with find in the diff.
Since non-interactive always returns the default, this should cause patches with bad changelogs to fail on the commit-queue.
- Scripts/webkitpy/common/checkout/api.py:
- Scripts/webkitpy/common/checkout/diff_parser.py:
- Scripts/webkitpy/tool/steps/abstractstep.py:
- Scripts/webkitpy/tool/steps/cleanworkingdirectory.py:
- Scripts/webkitpy/tool/steps/validatechangelogs.py: Copied from Tools/Scripts/webkitpy/tool/steps/validatereviewer.py.
- Scripts/webkitpy/tool/steps/validatechangelogs_unittest.py: Copied from Tools/Scripts/webkitpy/tool/steps/cleanworkingdirectory.py.
- Scripts/webkitpy/tool/steps/validatereviewer.py:
- 10:20 AM Changeset in webkit [74638] by
-
- 3 edits in trunk/WebCore
2010-12-24 Yury Semikhatsky <yurys@chromium.org>
Unreviewed. Fix compilation on Windows.
- inspector/InspectorStyleSheet.cpp: (WebCore::InspectorStyle::~InspectorStyle): added explicit empty destructor
- inspector/InspectorStyleSheet.h:
- 9:51 AM Changeset in webkit [74637] by
-
- 4 edits in trunk/WebCore
2010-12-24 Yury Semikhatsky <yurys@chromium.org>
Reviewed by Darin Adler.
inspector/debugger-step-out.html crashing intermittently in the bots
https://bugs.webkit.org/show_bug.cgi?id=50868
Use RefPtr rather instead of raw pointers when keeping references
to refcounted objects.
- inspector/InspectorCSSAgent.cpp: (WebCore::InspectorCSSAgent::getStylesForNode2): (WebCore::InspectorCSSAgent::getComputedStyleForNode2):
- inspector/InspectorStyleSheet.cpp: (WebCore::InspectorStyle::create): (WebCore::InspectorStyle::InspectorStyle): (WebCore::InspectorStyle::buildObjectForStyle): (WebCore::InspectorStyle::setPropertyText): (WebCore::InspectorStyle::toggleProperty): (WebCore::InspectorStyle::styleText): (WebCore::InspectorStyle::populateAllProperties): (WebCore::InspectorStyle::replacePropertyInStyleText): (WebCore::InspectorStyleSheet::create): (WebCore::InspectorStyleSheet::InspectorStyleSheet): (WebCore::InspectorStyleSheetForInlineStyle::create): (WebCore::InspectorStyleSheetForInlineStyle::InspectorStyleSheetForInlineStyle): (WebCore::InspectorStyleSheetForInlineStyle::getStyleAttributeRanges):
- inspector/InspectorStyleSheet.h: (WebCore::InspectorStyle::cssStyle): (WebCore::InspectorStyleSheet::pageStyleSheet):
- 9:31 AM Changeset in webkit [74636] by
-
- 6 edits2 adds in trunk
2010-12-24 Justin Schuh <jschuh@chromium.org>
Reviewed by Nikolas Zimmermann.
SVGElementInstance::m_useElement should be cleared when use element is removed from document
https://bugs.webkit.org/show_bug.cgi?id=51486
Test: svg/custom/use-instanceRoot-with-use-removed.svg
- svg/SVGElementInstance.cpp: (WebCore::SVGElementInstance::invalidateAllInstancesOfElement):
- svg/SVGElementInstance.h: (WebCore::SVGElementInstance::clearUseElement):
- svg/SVGUseElement.cpp: (WebCore::SVGUseElement::removedFromDocument): (WebCore::ShadowTreeUpdateBlocker::if): (WebCore::SVGUseElement::detachInstance): (WebCore::SVGUseElement::detach):
- svg/SVGUseElement.h:
2010-12-24 Justin Schuh <jschuh@chromium.org>
Reviewed by Nikolas Zimmermann.
SVGElementInstance::m_useElement should be cleared when use element is removed from document
https://bugs.webkit.org/show_bug.cgi?id=51486
- svg/custom/use-instanceRoot-with-use-removed-expected.txt: Added.
- svg/custom/use-instanceRoot-with-use-removed.svg: Added.
- 9:01 AM Changeset in webkit [74635] by
-
- 2 edits in trunk/WebCore
2010-12-23 Joseph Pecoraro <Joseph Pecoraro>
Reviewed by Yury Semikhatsky.
Web Inspector: Missing Localized Strings
https://bugs.webkit.org/show_bug.cgi?id=51542
Added missing strings: "WebSocket", "Frames", "Text", "Transfer",
"Preserve Log upon Navigation", and "Go to Line".
- English.lproj/localizedStrings.js:
- 8:39 AM Changeset in webkit [74634] by
-
- 11 edits3 adds in trunk
2010-12-24 Alexander Pavlov <apavlov@chromium.org>
Reviewed by Yury Semikhatsky.
Web Inspector: Inline HTML style property out of sync with element.style in Sidebar
https://bugs.webkit.org/show_bug.cgi?id=51478
Now the element attribute modifications (including "style") are tracked, which results in the
source offsets update (in the backend) and Styles/Metrics pane updates (in the frontend)
on the "style" attribute modification through the Elements tree. Also, it appears that
a complete "style" attribute removal destroys the inline style declaration
(StyledElement::m_inlineStyleDecl), while the code used to rely on it being immutable (hence a crash).
Test: inspector/elements-delete-inline-style.html
WebCore:
- inspector/InspectorCSSAgent.cpp: (WebCore::InspectorCSSAgent::didModifyDOMAttr):
- inspector/InspectorCSSAgent.h:
- inspector/InspectorDOMAgent.cpp: (WebCore::InspectorDOMAgent::didModifyDOMAttr):
- inspector/InspectorDOMAgent.h:
- inspector/InspectorStyleSheet.cpp: (WebCore::InspectorStyleSheetForInlineStyle::InspectorStyleSheetForInlineStyle): (WebCore::InspectorStyleSheetForInlineStyle::didModifyElementAttribute): (WebCore::InspectorStyleSheetForInlineStyle::text): (WebCore::InspectorStyleSheetForInlineStyle::setStyleText): (WebCore::InspectorStyleSheetForInlineStyle::getStyleAttributeRanges):
- inspector/InspectorStyleSheet.h:
- inspector/front-end/ElementsTreeOutline.js: (WebInspector.ElementsTreeElement.prototype._attributeEditingCommitted):
LayoutTests:
- http/tests/inspector/inspector-test2.js: (initialize_InspectorTest.InspectorTest.addResults): (initialize_InspectorTest.InspectorTest.createKeyEvent):
- inspector/elements-delete-inline-style-expected.txt: Added.
- inspector/elements-delete-inline-style.html: Added.
- inspector/elements-tests2.js: Added. (initialize_ElementTest.InspectorTest.expandDOMSubtree.processChildren): (initialize_ElementTest.InspectorTest.expandDOMSubtree): (initialize_ElementTest.InspectorTest.expandDOMSubtreeAndRun): (initialize_ElementTest.InspectorTest.nodeForId): (initialize_ElementTest.InspectorTest.selectElementAndRun): (initialize_ElementTest.InspectorTest.dumpSelectedElementStyles): (initialize_ElementTest.InspectorTest.getSelectedElementStyles): (initialize_ElementTest.InspectorTest.getElementStylePropertyTreeItem): (initialize_ElementTest.InspectorTest.dumpStyleTreeOutline): (initialize_ElementTest.InspectorTest.dumpStyleTreeItem): (initialize_ElementTest):
- inspector/styles-add-blank-property.html: Remove unnecessary function.
- 4:18 AM Changeset in webkit [74633] by
-
- 1 edit18 adds in trunk/LayoutTests
2010-12-24 Benjamin Kalman <kalman@chromium.org>
Reviewed by Ryosuke Niwa.
editing/selection/caret-* tests need rebaseline for gtk
https://bugs.webkit.org/show_bug.cgi?id=51578
Rebaseline editing/selection/caret-* tests for gtk.
- platform/gtk/editing/selection/caret-ltr-2-expected.checksum: Added.
- platform/gtk/editing/selection/caret-ltr-2-expected.png: Added.
- platform/gtk/editing/selection/caret-ltr-2-expected.txt: Added.
- platform/gtk/editing/selection/caret-ltr-2-left-expected.checksum: Added.
- platform/gtk/editing/selection/caret-ltr-2-left-expected.png: Added.
- platform/gtk/editing/selection/caret-ltr-2-left-expected.txt: Added.
- platform/gtk/editing/selection/caret-ltr-expected.checksum: Added.
- platform/gtk/editing/selection/caret-ltr-expected.png: Added.
- platform/gtk/editing/selection/caret-ltr-expected.txt: Added.
- platform/gtk/editing/selection/caret-ltr-right-expected.checksum: Added.
- platform/gtk/editing/selection/caret-ltr-right-expected.png: Added.
- platform/gtk/editing/selection/caret-ltr-right-expected.txt: Added.
- platform/gtk/editing/selection/caret-rtl-2-left-expected.checksum: Added.
- platform/gtk/editing/selection/caret-rtl-2-left-expected.png: Added.
- platform/gtk/editing/selection/caret-rtl-2-left-expected.txt: Added.
- platform/gtk/editing/selection/caret-rtl-right-expected.checksum: Added.
- platform/gtk/editing/selection/caret-rtl-right-expected.png: Added.
- platform/gtk/editing/selection/caret-rtl-right-expected.txt: Added.
- 4:12 AM Changeset in webkit [74632] by
-
- 3 edits in trunk/Tools
2010-12-24 Dirk Pranke <dpranke@chromium.org>
Reviewed by Kenneth Russell.
Modify new-run-webkit-tests to shut up and exit when all the threads
are wedged so that test runs complete faster and we don't end up
w/ enormous log files.
- Scripts/webkitpy/layout_tests/layout_package/message_broker.py:
- Scripts/webkitpy/layout_tests/layout_package/message_broker_unittest.py:
- 4:05 AM Changeset in webkit [74631] by
-
- 2 edits in trunk/WebCore
2010-12-24 Noel Gordon <noel.gordon@gmail.com>
Reviewed by Andreas Kling.
[chromium] Fix a style nit in the JPEG encoder
https://bugs.webkit.org/show_bug.cgi?id=51574
- platform/image-encoders/skia/JPEGImageEncoder.cpp: nit begone.
- 4:02 AM Changeset in webkit [74630] by
-
- 5 edits in trunk
2010-12-23 Alexander Pavlov <apavlov@chromium.org>
Reviewed by Maciej Stachowiak.
Web Inspector: CSS shorthand properties expand unnecessarily
https://bugs.webkit.org/show_bug.cgi?id=36304
"border-radius" is a shorthand property that has a "non-standard" format and longhands, see
http://www.w3.org/TR/css3-background/#border-radius for reference. CSSParser does not use the
standard parseShorthand() or parse4Values() methods but instead a custom parseBorderRadius() method.
This method didn't use to create a ShorthandScope instance and set the m_implicitShorthand value
appropriately when adding implicit longhands, thus they were added as normal properties
found in the CSS (neither implicit, nor longhands). This is now fixed.
WebCore:
- css/CSSParser.cpp: (WebCore::CSSParser::parseBorderRadius):
LayoutTests:
- inspector/elements-panel-styles-expected.txt:
- inspector/resources/elements-panel-styles.css: (.foo):
- 3:54 AM Changeset in webkit [74629] by
-
- 3 edits2 adds in trunk
2010-12-24 Justin Schuh <jschuh@chromium.org>
Reviewed by Darin Adler.
setTargetAttributeAnimatedValue should check for a null shadowTreeElement.
https://bugs.webkit.org/show_bug.cgi?id=51576
- svg/custom/animate-disallowed-use-element-expected.txt: Added.
- svg/custom/animate-disallowed-use-element.svg: Added.
2010-12-24 Justin Schuh <jschuh@chromium.org>
Reviewed by Darin Adler.
setTargetAttributeAnimatedValue should check for a null shadowTreeElement.
https://bugs.webkit.org/show_bug.cgi?id=51576
Disallowed elements may be pruned from the shadow tree, leaving a null
shadowTreeElement. We should check for that before setting attributes.
Test: svg/custom/animate-disallowed-use-element.svg
- svg/SVGAnimationElement.cpp: (WebCore::SVGAnimationElement::setTargetAttributeAnimatedValue):
- 3:50 AM Changeset in webkit [74628] by
-
- 2 edits in trunk/WebCore
2010-12-24 Evan Martin <evan@chromium.org>
Reviewed by Darin Adler.
TextMetrics::width is an integer, but the member is a float
https://bugs.webkit.org/show_bug.cgi?id=51566
Change the return type to a float.
No tests, fixing a compiler warning.
(I was unable to produce a difference in the values exposed to pages
with a float versus the previous conversion to an integer.)
- html/TextMetrics.h: (WebCore::TextMetrics::width):
- 3:47 AM Changeset in webkit [74627] by
-
- 1 edit1 add in trunk/LayoutTests
2010-12-24 Andrei Popescu <andreip@google.com>
Unreviewed build fix.
Add Chromium-mac test expectation for the layout test below.
[Chromium] http/tests/xmlhttprequest/basic-auth-nopassword.html needs Mac specific test expectations
https://bugs.webkit.org/show_bug.cgi?id=51589
- platform/chromium-mac/http/tests/xmlhttprequest/basic-auth-nopassword-expected.txt: Added.
- 3:27 AM Changeset in webkit [74626] by
-
- 12 edits2 adds in trunk
2010-12-24 Ryuan Choi <ryuan.choi@samsung.com>
Reviewed by Kenneth Rohde Christiansen.
[EFL] Add option to enable Touch Events.
https://bugs.webkit.org/show_bug.cgi?id=49125
Add ENABLE_TOUCH_EVENTS option.
- cmake/OptionsEfl.cmake:
- cmakeconfig.h.cmake:
2010-12-24 Ryuan Choi <ryuan.choi@samsung.com>
Reviewed by Kenneth Rohde Christiansen.
[EFL] Add option to enable Touch Events.
https://bugs.webkit.org/show_bug.cgi?id=49125
Add files for EFL to support TOUCH_EVENTS.
- CMakeLists.txt:
- CMakeListsEfl.txt:
- platform/PlatformTouchEvent.h: Add EFL related constructor.
- platform/PlatformTouchPoint.h: ditto.
- platform/efl/PlatformTouchEventEfl.cpp: Added. (WebCore::PlatformTouchEvent::PlatformTouchEvent):
- platform/efl/PlatformTouchPointEfl.cpp: Added. (WebCore::PlatformTouchPoint::PlatformTouchPoint):
2010-12-24 Ryuan Choi <ryuan.choi@samsung.com>
Reviewed by Kenneth Rohde Christiansen.
[EFL] Add option to enable Touch Events.
https://bugs.webkit.org/show_bug.cgi?id=49125
Implement ewk_frame_feed_touch_event to feed touch events. Because EFL
doesn't have touch events, Application should generate and pass it.
- WebCoreSupport/ChromeClientEfl.h: (WebCore::ChromeClientEfl::needTouchEvents):
- ewk/ewk_frame.cpp: (ewk_frame_feed_touch_event):
- ewk/ewk_frame.h:
- 3:23 AM Changeset in webkit [74625] by
-
- 2 edits in trunk/WebCore
2010-12-24 Gyuyoung Kim <gyuyoung.kim@samsung.com>
Reviewed by Kenneth Rohde Christiansen.
[EFL] Return an empty string instead of 'application/octet-stream' in getMIMETypeForExtension()
https://bugs.webkit.org/show_bug.cgi?id=51457
WebKit EFL can't play html5 audio because of wrong mime type checking.
So, 'application/octet-stream' is changed with an empty string.
- platform/efl/MIMETypeRegistryEfl.cpp: (WebCore::MIMETypeRegistry::getMIMETypeForExtension): Returns an empty string instead of application/octet-stream when it can't find proper mime type.
- 3:09 AM Changeset in webkit [74624] by
-
- 2 edits in trunk/LayoutTests
2010-12-24 Andrei Popescu <andreip@google.com>
Unreviewed. Update Chromium test expectations.
Mark editing/input/ime-composition-clearpreedit.html.html as TEXT.
This test was added in http://trac.webkit.org/changeset/74605 and
fails in Chromium.
- platform/chromium/test_expectations.txt:
- 3:05 AM Changeset in webkit [74623] by
-
- 3 edits in trunk/Tools
2010-12-24 Eric Seidel <eric@webkit.org>
Reviewed by David Levin.
commit-queue should better explain how it will re-open bugs and follow duplicate chains
https://bugs.webkit.org/show_bug.cgi?id=51549
I've had multiple people ask me about the commit-queue's behavior
regarding reporting flaky tests. I figured it would be best to just
have the queue explain itself in the bugs it files.
- Scripts/webkitpy/tool/bot/flakytestreporter.py:
- Scripts/webkitpy/tool/bot/flakytestreporter_unittest.py:
- 2:38 AM Changeset in webkit [74622] by
-
- 3 edits2 adds in trunk
2010-12-24 Justin Schuh <jschuh@chromium.org>
Reviewed by Darin Adler.
SVGFontFaceElement::rebuildFontFace() should exit when not in document
https://bugs.webkit.org/show_bug.cgi?id=51571
- svg/custom/use-invalid-font-face-expected.txt: Added.
- svg/custom/use-invalid-font-face.svg: Added.
2010-12-24 Justin Schuh <jschuh@chromium.org>
Reviewed by Darin Adler.
SVGFontFaceElement::rebuildFontFace() should exit when not in document
https://bugs.webkit.org/show_bug.cgi?id=51571
We were hitting a NULL deref crash. Since most of the callers checked
inDocument() anyway, I moved it into the start of rebuildFontFace.
Test: svg/custom/use-invalid-font-face.svg
- svg/SVGFontFaceElement.cpp: (WebCore::SVGFontFaceElement::parseMappedAttribute): (WebCore::SVGFontFaceElement::rebuildFontFace): (WebCore::SVGFontFaceElement::childrenChanged):
- 2:19 AM Changeset in webkit [74621] by
-
- 3 edits in trunk/Tools
2010-12-24 Shinichiro Hamaji <hamaji@chromium.org>
Reviewed by Eric Seidel.
check-webkit-style: should ignore "wrong" namings in JavaScriptCore/qt
https://bugs.webkit.org/show_bug.cgi?id=51467
- Scripts/webkitpy/style/checker.py:
- Scripts/webkitpy/style/checker_unittest.py:
- 2:12 AM Changeset in webkit [74620] by
-
- 2 edits in trunk/WebKit/gtk
2010-12-21 Philippe Normand <pnormand@igalia.com>
Reviewed by Eric Seidel.
[GTK] minimal build with --database fails
https://bugs.webkit.org/show_bug.cgi?id=51394
- webkit/webkitprivate.cpp: (webkit_init): Use cacheStorage() only if the OFFLINE_WEB_APPLICATIONS feature is enabled.
- 2:00 AM Changeset in webkit [74619] by
-
- 2 edits in trunk/WebCore
2010-12-24 Justin Schuh <jschuh@chromium.org>
Reviewed by Darin Adler.
Remove unnecessary check in CSSCanvasValue::canvasDestroyed().
https://bugs.webkit.org/show_bug.cgi?id=51564
This check was duplicating an ASSERT for a case fixed in r73927.
Now that the bug is fixed the check is no longer necessary.
Removing dead code, so no new tests needed.
- css/CSSCanvasValue.cpp: (WebCore::CSSCanvasValue::canvasDestroyed):
- 1:01 AM Changeset in webkit [74618] by
-
- 10 edits1 move2 adds in trunk
Move JavaScriptGlue into Sources
https://bugs.webkit.org/show_bug.cgi?id=51583
Rubber-stamped by Eric Seidel.
Actually perform the move.
- JavaScriptGlue: Removed.
- JavaScriptGlue/Configurations: Removed.
- JavaScriptGlue/Configurations/Base.xcconfig: Removed.
- JavaScriptGlue/Configurations/DebugRelease.xcconfig: Removed.
- JavaScriptGlue/Configurations/JavaScriptGlue.xcconfig: Removed.
- JavaScriptGlue/Configurations/Version.xcconfig: Removed.
- JavaScriptGlue/English.lproj: Removed.
- JavaScriptGlue/English.lproj/InfoPlist.strings: Removed.
- JavaScriptGlue/ForwardingHeaders: Removed.
- JavaScriptGlue/ForwardingHeaders/masm: Removed.
- JavaScriptGlue/ForwardingHeaders/masm/X86Assembler.h: Removed.
- JavaScriptGlue/ForwardingHeaders/profiler: Removed.
- JavaScriptGlue/ForwardingHeaders/profiler/Profiler.h: Removed.
- JavaScriptGlue/ForwardingHeaders/runtime: Removed.
- JavaScriptGlue/ForwardingHeaders/runtime/CallFrame.h: Removed.
- JavaScriptGlue/ForwardingHeaders/wtf: Removed.
- JavaScriptGlue/ForwardingHeaders/wtf/ASCIICType.h: Removed.
- JavaScriptGlue/ForwardingHeaders/wtf/AlwaysInline.h: Removed.
- JavaScriptGlue/ForwardingHeaders/wtf/Assertions.h: Removed.
- JavaScriptGlue/ForwardingHeaders/wtf/Atomics.h: Removed.
- JavaScriptGlue/ForwardingHeaders/wtf/Bitmap.h: Removed.
- JavaScriptGlue/ForwardingHeaders/wtf/BumpPointerAllocator.h: Removed.
- JavaScriptGlue/ForwardingHeaders/wtf/CrossThreadRefCounted.h: Removed.
- JavaScriptGlue/ForwardingHeaders/wtf/CurrentTime.h: Removed.
- JavaScriptGlue/ForwardingHeaders/wtf/DateInstanceCache.h: Removed.
- JavaScriptGlue/ForwardingHeaders/wtf/DateMath.h: Removed.
- JavaScriptGlue/ForwardingHeaders/wtf/FastAllocBase.h: Removed.
- JavaScriptGlue/ForwardingHeaders/wtf/FastMalloc.h: Removed.
- JavaScriptGlue/ForwardingHeaders/wtf/FixedArray.h: Removed.
- JavaScriptGlue/ForwardingHeaders/wtf/Forward.h: Removed.
- JavaScriptGlue/ForwardingHeaders/wtf/HashCountedSet.h: Removed.
- JavaScriptGlue/ForwardingHeaders/wtf/HashFunctions.h: Removed.
- JavaScriptGlue/ForwardingHeaders/wtf/HashMap.h: Removed.
- JavaScriptGlue/ForwardingHeaders/wtf/HashSet.h: Removed.
- JavaScriptGlue/ForwardingHeaders/wtf/HashTraits.h: Removed.
- JavaScriptGlue/ForwardingHeaders/wtf/ListHashSet.h: Removed.
- JavaScriptGlue/ForwardingHeaders/wtf/ListRefPtr.h: Removed.
- JavaScriptGlue/ForwardingHeaders/wtf/Locker.h: Removed.
- JavaScriptGlue/ForwardingHeaders/wtf/MainThread.h: Removed.
- JavaScriptGlue/ForwardingHeaders/wtf/MathExtras.h: Removed.
- JavaScriptGlue/ForwardingHeaders/wtf/Noncopyable.h: Removed.
- JavaScriptGlue/ForwardingHeaders/wtf/NotFound.h: Removed.
- JavaScriptGlue/ForwardingHeaders/wtf/OSAllocator.h: Removed.
- JavaScriptGlue/ForwardingHeaders/wtf/OwnArrayPtr.h: Removed.
- JavaScriptGlue/ForwardingHeaders/wtf/OwnArrayPtrCommon.h: Removed.
- JavaScriptGlue/ForwardingHeaders/wtf/OwnFastMallocPtr.h: Removed.
- JavaScriptGlue/ForwardingHeaders/wtf/OwnPtr.h: Removed.
- JavaScriptGlue/ForwardingHeaders/wtf/OwnPtrCommon.h: Removed.
- JavaScriptGlue/ForwardingHeaders/wtf/PageAllocation.h: Removed.
- JavaScriptGlue/ForwardingHeaders/wtf/PageAllocationAligned.h: Removed.
- JavaScriptGlue/ForwardingHeaders/wtf/PageBlock.h: Removed.
- JavaScriptGlue/ForwardingHeaders/wtf/PageReservation.h: Removed.
- JavaScriptGlue/ForwardingHeaders/wtf/PassOwnArrayPtr.h: Removed.
- JavaScriptGlue/ForwardingHeaders/wtf/PassOwnPtr.h: Removed.
- JavaScriptGlue/ForwardingHeaders/wtf/PassRefPtr.h: Removed.
- JavaScriptGlue/ForwardingHeaders/wtf/Platform.h: Removed.
- JavaScriptGlue/ForwardingHeaders/wtf/PossiblyNull.h: Removed.
- JavaScriptGlue/ForwardingHeaders/wtf/PtrAndFlags.h: Removed.
- JavaScriptGlue/ForwardingHeaders/wtf/RandomNumber.h: Removed.
- JavaScriptGlue/ForwardingHeaders/wtf/RefCounted.h: Removed.
- JavaScriptGlue/ForwardingHeaders/wtf/RefPtr.h: Removed.
- JavaScriptGlue/ForwardingHeaders/wtf/StackBounds.h: Removed.
- JavaScriptGlue/ForwardingHeaders/wtf/StdLibExtras.h: Removed.
- JavaScriptGlue/ForwardingHeaders/wtf/StringHasher.h: Removed.
- JavaScriptGlue/ForwardingHeaders/wtf/ThreadSafeShared.h: Removed.
- JavaScriptGlue/ForwardingHeaders/wtf/ThreadSpecific.h: Removed.
- JavaScriptGlue/ForwardingHeaders/wtf/Threading.h: Removed.
- JavaScriptGlue/ForwardingHeaders/wtf/ThreadingPrimitives.h: Removed.
- JavaScriptGlue/ForwardingHeaders/wtf/UnusedParam.h: Removed.
- JavaScriptGlue/ForwardingHeaders/wtf/VMTags.h: Removed.
- JavaScriptGlue/ForwardingHeaders/wtf/ValueCheck.h: Removed.
- JavaScriptGlue/ForwardingHeaders/wtf/Vector.h: Removed.
- JavaScriptGlue/ForwardingHeaders/wtf/VectorTraits.h: Removed.
- JavaScriptGlue/ForwardingHeaders/wtf/WTFThreadData.h: Removed.
- JavaScriptGlue/ForwardingHeaders/wtf/text: Removed.
- JavaScriptGlue/ForwardingHeaders/wtf/text/CString.h: Removed.
- JavaScriptGlue/ForwardingHeaders/wtf/text/StringHash.h: Removed.
- JavaScriptGlue/ForwardingHeaders/wtf/text/StringImpl.h: Removed.
- JavaScriptGlue/ForwardingHeaders/wtf/text/StringImplBase.h: Removed.
- JavaScriptGlue/ForwardingHeaders/wtf/text/TextPosition.h: Removed.
- JavaScriptGlue/ForwardingHeaders/wtf/text/WTFString.h: Removed.
- JavaScriptGlue/ForwardingHeaders/wtf/unicode: Removed.
- JavaScriptGlue/ForwardingHeaders/wtf/unicode/Unicode.h: Removed.
- JavaScriptGlue/ForwardingHeaders/wtf/unicode/icu: Removed.
- JavaScriptGlue/ForwardingHeaders/wtf/unicode/icu/UnicodeIcu.h: Removed.
- JavaScriptGlue/Info.plist: Removed.
- JavaScriptGlue/JSBase.cpp: Removed.
- JavaScriptGlue/JSBase.h: Removed.
- JavaScriptGlue/JSObject.cpp: Removed.
- JavaScriptGlue/JSObject.h: Removed.
- JavaScriptGlue/JSRun.cpp: Removed.
- JavaScriptGlue/JSRun.h: Removed.
- JavaScriptGlue/JSUtils.cpp: Removed.
- JavaScriptGlue/JSUtils.h: Removed.
- JavaScriptGlue/JSValueWrapper.cpp: Removed.
- JavaScriptGlue/JSValueWrapper.h: Removed.
- JavaScriptGlue/JavaScriptGlue.cpp: Removed.
- JavaScriptGlue/JavaScriptGlue.exp: Removed.
- JavaScriptGlue/JavaScriptGlue.h: Removed.
- JavaScriptGlue/JavaScriptGlue.xcodeproj: Removed.
- JavaScriptGlue/JavaScriptGlue.xcodeproj/project.pbxproj: Removed.
- JavaScriptGlue/LICENSE: Removed.
- JavaScriptGlue/Makefile: Removed.
- JavaScriptGlue/UserObjectImp.cpp: Removed.
- JavaScriptGlue/UserObjectImp.h: Removed.
- JavaScriptGlue/config.h: Removed.
- JavaScriptGlue/icu: Removed.
- JavaScriptGlue/icu/LICENSE: Removed.
- JavaScriptGlue/icu/README: Removed.
- JavaScriptGlue/icu/unicode: Removed.
- JavaScriptGlue/icu/unicode/platform.h: Removed.
- JavaScriptGlue/icu/unicode/putil.h: Removed.
- JavaScriptGlue/icu/unicode/uchar.h: Removed.
- JavaScriptGlue/icu/unicode/ucnv.h: Removed.
- JavaScriptGlue/icu/unicode/ucnv_err.h: Removed.
- JavaScriptGlue/icu/unicode/uconfig.h: Removed.
- JavaScriptGlue/icu/unicode/uenum.h: Removed.
- JavaScriptGlue/icu/unicode/uiter.h: Removed.
- JavaScriptGlue/icu/unicode/umachine.h: Removed.
- JavaScriptGlue/icu/unicode/urename.h: Removed.
- JavaScriptGlue/icu/unicode/ustring.h: Removed.
- JavaScriptGlue/icu/unicode/utf.h: Removed.
- JavaScriptGlue/icu/unicode/utf16.h: Removed.
- JavaScriptGlue/icu/unicode/utf8.h: Removed.
- JavaScriptGlue/icu/unicode/utf_old.h: Removed.
- JavaScriptGlue/icu/unicode/utypes.h: Removed.
- JavaScriptGlue/icu/unicode/uversion.h: Removed.
- JavaScriptGlue/testjsglue.cpp: Removed.
- Makefile:
- Sources: Added.
- Sources/JavaScriptGlue: Copied from JavaScriptGlue.
- Sources/Makefile: Added.
Tools:
These tools refer to JavaScriptGlue and need to be updated to its new
location.
- Scripts/build-webkit:
- Scripts/do-file-rename:
- Scripts/do-webcore-rename:
- Scripts/webkitpy/common/config/build.py:
- Scripts/webkitpy/common/config/build_unittest.py:
Websites/bugs.webkit.org:
PrettyPatch keeps a list of the top-level directories that exist in
trac.
- PrettyPatch/PrettyPatch.rb:
- 12:06 AM Changeset in webkit [74617] by
-
- 2 edits in trunk/JavaScriptCore
2010-12-24 Patrick Gansterer <Patrick Gansterer>
Reviewed by Eric Seidel.
[WINCE] Add CPU(MIPS) detection
https://bugs.webkit.org/show_bug.cgi?id=51342
WinCE usually defines MIPS and _MIPS_.
- wtf/Platform.h:
Dec 23, 2010:
- 10:14 PM Changeset in webkit [74616] by
-
- 3 edits in trunk/Tools
2010-12-23 Lucas Forschler <Lucas Forschler>
Reviewed by Eric Seidel.
Bug 51539 - update iexploder scripts to point to new location
- Scripts/run-iexploder-tests:
- Scripts/update-iexploder-cssproperties:
- 10:11 PM Changeset in webkit [74615] by
-
- 2 edits in trunk/WebKit2
Make dragging work again after r74591.
Reviewed by Sam Weinig.
- UIProcess/WebPageProxy.cpp:
(WebKit::WebPageProxy::didReceiveEvent): Fixed a typo.
- 10:05 PM Changeset in webkit [74614] by
-
- 2 edits in trunk/LayoutTests
2010-12-23 Yuta Kitamura <yutak@chromium.org>
Unreviewed. Update GTK's Skipped file.
[GTK] http/tests/inspector/console-websocket-error.html hangs
https://bugs.webkit.org/show_bug.cgi?id=51380
- platform/gtk/Skipped:
- 9:26 PM Changeset in webkit [74613] by
-
- 1 edit in trunk/LayoutTests/platform/gtk/Skipped
Unreviewed GTK fix after r74609: put the right file name in the Skipped file.
- 8:59 PM Changeset in webkit [74612] by
-
- 2 edits in trunk/LayoutTests
2010-12-23 Yuta Kitamura <yutak@chromium.org>
Unreviewed. Update Chromium test expectations.
Mark http/tests/inspector/console-websocket-error.html as SLOW on WIN DEBUG.
- platform/chromium/test_expectations.txt:
- 8:50 PM Changeset in webkit [74611] by
-
- 5 edits in trunk/LayoutTests
2010-12-22 Yuta Kitamura <yutak@chromium.org>
Reviewed by Alexey Proskuryakov.
[GTK] http/tests/inspector/console-websocket-error.html hangs
https://bugs.webkit.org/show_bug.cgi?id=51380
The test should call dumpConsoleMessages() even if WebSocket is not supported
or a WebSocket object goes to an unexpected state.
- http/tests/inspector/console-websocket-error.html:
- platform/gtk/Skipped:
- platform/mac-leopard/http/tests/inspector/console-websocket-error-expected.txt:
- platform/mac/http/tests/inspector/console-websocket-error-expected.txt:
- 8:31 PM Changeset in webkit [74610] by
-
- 2 edits in trunk/WebCore
2010-12-23 Evan Martin <evan@chromium.org>
Reviewed by Darin Adler.
ViewportArguments.h uses an enum to intialize a bool
https://bugs.webkit.org/show_bug.cgi?id=50982
Initialize userScalable to true, rather than -1.
No tests; fixes a compiler warning.
- dom/ViewportArguments.h: (WebCore::ViewportArguments::ViewportArguments):
- 8:06 PM Changeset in webkit [74609] by
-
- 5 edits13 adds in trunk
[Qt] User name/password stripped from URLs
https://bugs.webkit.org/show_bug.cgi?id=36232
Patch by Jan Erik Hanssen <jhanssen@sencha.com> on 2010-12-23
Reviewed by Adam Barth.
WebCore:
If a username but no password or no username but a password is present
in a URL, QtWebKit won't send any authentication information down to Qt
as it should.
Tests: http/tests/xmlhttprequest/basic-auth-nopassword.html
http/tests/xmlhttprequest/basic-auth-nouser.html
- platform/network/qt/ResourceHandleQt.cpp:
(WebCore::ResourceHandle::start):
(WebCore::ResourceHandle::loadResourceSynchronously):
LayoutTests:
Test authentication cases where a username but no password and
no username but a password is present.
- http/tests/xmlhttprequest/basic-auth-nopassword-expected.txt: Added.
- http/tests/xmlhttprequest/basic-auth-nopassword.html: Added.
- http/tests/xmlhttprequest/basic-auth-nouser-expected.txt: Added.
- http/tests/xmlhttprequest/basic-auth-nouser.html: Added.
- http/tests/xmlhttprequest/resources/basic-auth-nouserpass/basic-auth-nouserpass.php: Added.
- platform/gtk/Skipped:
- platform/mac-leopard/http/tests/xmlhttprequest/basic-auth-nopassword-expected.txt: Added.
- platform/mac-snowleopard/http/tests/xmlhttprequest/basic-auth-nopassword-expected.txt: Added.
- platform/mac-tiger/http/tests/xmlhttprequest/basic-auth-nopassword-expected.txt: Added.
- platform/qt/Skipped:
- 7:40 PM Changeset in webkit [74608] by
-
- 2 edits in trunk/LayoutTests
2010-12-23 Ryosuke Niwa <rniwa@webkit.org>
Unreviewed Qt rebaseline for r74593.
Added the forgotten blank line at the end.
- platform/qt/editing/pasteboard/copy-in-password-field-expected.txt:
- 7:18 PM Changeset in webkit [74607] by
-
- 1 edit1 add in trunk/LayoutTests
2010-12-23 Ryosuke Niwa <rniwa@webkit.org>
Unreviewed Qt rebaseline for r74593.
- platform/qt/editing/pasteboard/copy-in-password-field-expected.txt: Added.
- 6:56 PM Changeset in webkit [74606] by
-
- 4 edits in trunk/LayoutTests
2010-12-23 Ryosuke Niwa <rniwa@webkit.org>
More unreviewed Chromium rebaselines for r74593.
- platform/chromium-linux/editing/deleting/smart-delete-002-expected.txt:
- platform/chromium-linux/editing/pasteboard/subframe-dragndrop-1-expected.txt:
- platform/chromium-win/editing/pasteboard/paste-xml-expected.txt:
- 6:53 PM Changeset in webkit [74605] by
-
- 6 edits2 adds in trunk
2010-12-23 Jan Erik Hanssen <jhanssen@sencha.com>
Reviewed by Alexey Proskuryakov.
[Qt] Composition text is not removed from the editor when composition is cancelled
https://bugs.webkit.org/show_bug.cgi?id=29391
Ensure that the composition text is cleared when an empty QInputMethodEvent
is received and the editor is in composition mode.
- editing/input/ime-composition-clearpreedit-expected.txt: Added.
- editing/input/ime-composition-clearpreedit.html: Added.
- platform/gtk/Skipped:
- platform/mac-wk2/Skipped:
- platform/qt-wk2/Skipped:
2010-12-23 Jan Erik Hanssen <jhanssen@sencha.com>
Reviewed by Alexey Proskuryakov.
[Qt] Composition text is not removed from the editor when composition is cancelled
https://bugs.webkit.org/show_bug.cgi?id=29391
Ensure that the composition text is cleared when an empty QInputMethodEvent
is received and the editor is in composition mode.
- Api/qwebpage.cpp: (QWebPagePrivate::inputMethodEvent):
- 6:38 PM Changeset in webkit [74604] by
-
- 54 edits in trunk/LayoutTests
2010-12-23 Benjamin Kalman <kalman@chromium.org>
Reviewed by Ryosuke Niwa.
Selection becomes stale when CharacterData is manipulated directly
https://bugs.webkit.org/show_bug.cgi?id=51389
Rebaseline GTK tests.
- platform/gtk/editing/deleting/collapse-whitespace-3587601-fix-expected.txt:
- platform/gtk/editing/deleting/delete-3608430-fix-expected.txt:
- platform/gtk/editing/deleting/delete-3608445-fix-expected.txt:
- platform/gtk/editing/deleting/delete-3608462-fix-expected.txt:
- platform/gtk/editing/deleting/delete-and-undo-expected.txt:
- platform/gtk/editing/deleting/delete-block-merge-contents-010-expected.txt:
- platform/gtk/editing/deleting/delete-br-007-expected.txt:
- platform/gtk/editing/deleting/delete-contiguous-ws-001-expected.txt:
- platform/gtk/editing/deleting/delete-leading-ws-001-expected.txt:
- platform/gtk/editing/deleting/delete-line-end-ws-001-expected.txt:
- platform/gtk/editing/deleting/delete-line-end-ws-002-expected.txt:
- platform/gtk/editing/deleting/delete-listitem-001-expected.txt:
- platform/gtk/editing/deleting/delete-tab-002-expected.txt:
- platform/gtk/editing/deleting/delete-tab-004-expected.txt:
- platform/gtk/editing/deleting/delete-trailing-ws-002-expected.txt:
- platform/gtk/editing/deleting/delete-ws-fixup-001-expected.txt:
- platform/gtk/editing/deleting/delete-ws-fixup-002-expected.txt:
- platform/gtk/editing/deleting/forward-delete-expected.txt:
- platform/gtk/editing/deleting/smart-delete-003-expected.txt:
- platform/gtk/editing/deleting/smart-delete-004-expected.txt:
- platform/gtk/editing/execCommand/insertHorizontalRule-expected.txt:
- platform/gtk/editing/inserting/insert-3654864-fix-expected.txt:
- platform/gtk/editing/inserting/insert-after-delete-001-expected.txt:
- platform/gtk/editing/inserting/insert-div-003-expected.txt:
- platform/gtk/editing/inserting/insert-div-024-expected.txt:
- platform/gtk/editing/inserting/insert-space-in-empty-doc-expected.txt:
- platform/gtk/editing/inserting/insert-text-with-newlines-expected.txt:
- platform/gtk/editing/inserting/typing-001-expected.txt:
- platform/gtk/editing/inserting/typing-002-expected.txt:
- platform/gtk/editing/inserting/typing-003-expected.txt:
- platform/gtk/editing/inserting/typing-around-br-001-expected.txt:
- platform/gtk/editing/inserting/typing-around-image-001-expected.txt:
- platform/gtk/editing/pasteboard/4076267-2-expected.txt:
- platform/gtk/editing/pasteboard/4076267-3-expected.txt:
- platform/gtk/editing/pasteboard/cut-text-001-expected.txt:
- platform/gtk/editing/pasteboard/paste-text-001-expected.txt:
- platform/gtk/editing/pasteboard/paste-xml-expected.txt:
- platform/gtk/editing/pasteboard/subframe-dragndrop-1-expected.txt:
- platform/gtk/editing/spelling/spelling-expected.txt:
- platform/gtk/editing/style/style-3681552-fix-001-expected.txt:
- platform/gtk/editing/style/style-3681552-fix-002-expected.txt:
- platform/gtk/editing/style/typing-style-001-expected.txt:
- platform/gtk/editing/style/typing-style-002-expected.txt:
- platform/gtk/editing/style/unbold-in-bold-expected.txt:
- platform/gtk/editing/undo/4063751-expected.txt:
- platform/gtk/editing/undo/redo-typing-001-expected.txt:
- platform/gtk/editing/undo/undo-combined-delete-boundary-expected.txt:
- platform/gtk/editing/undo/undo-combined-delete-expected.txt:
- platform/gtk/editing/undo/undo-delete-boundary-expected.txt:
- platform/gtk/editing/undo/undo-delete-expected.txt:
- platform/gtk/editing/undo/undo-forward-delete-boundary-expected.txt:
- platform/gtk/editing/undo/undo-forward-delete-expected.txt:
- platform/gtk/editing/undo/undo-misspellings-expected.txt:
- 6:33 PM Changeset in webkit [74603] by
-
- 5 edits in trunk
2010-12-23 Ryosuke Niwa <rniwa@webkit.org>
Reviewed by Darin Adler.
queryCommandValue("FormatBlock") does not respect editing boundaries
https://bugs.webkit.org/show_bug.cgi?id=51219
Added test cases to ensure queryCommandValue("FormatBlock") respects editing boundaries.
- editing/execCommand/query-format-block-expected.txt:
- editing/execCommand/script-tests/query-format-block.js:
2010-12-23 Ryosuke Niwa <rniwa@webkit.org>
Reviewed by Darin Adler.
queryCommandValue("FormatBlock") does not respect editing boundaries
https://bugs.webkit.org/show_bug.cgi?id=51219
Fixed the bug by checking the element we found in elementForFormatBlockCommand
does not contain the editable root.
- editing/FormatBlockCommand.cpp: (WebCore::FormatBlockCommand::elementForFormatBlockCommand):
- 6:17 PM Changeset in webkit [74602] by
-
- 104 edits in trunk/LayoutTests
2010-12-23 Ryosuke Niwa <rniwa@webkit.org>
Unreviewed Chromium rebaselines for r74593.
- platform/chromium-linux/editing/deleting/smart-delete-001-expected.txt:
- platform/chromium-linux/editing/deleting/smart-delete-003-expected.txt:
- platform/chromium-linux/editing/deleting/smart-delete-004-expected.txt:
- platform/chromium-win/editing/deleting/collapse-whitespace-3587601-fix-expected.txt:
- platform/chromium-win/editing/deleting/delete-3608430-fix-expected.txt:
- platform/chromium-win/editing/deleting/delete-3608445-fix-expected.txt:
- platform/chromium-win/editing/deleting/delete-3608462-fix-expected.txt:
- platform/chromium-win/editing/deleting/delete-after-span-ws-001-expected.txt:
- platform/chromium-win/editing/deleting/delete-after-span-ws-002-expected.txt:
- platform/chromium-win/editing/deleting/delete-after-span-ws-003-expected.txt:
- platform/chromium-win/editing/deleting/delete-and-undo-expected.txt:
- platform/chromium-win/editing/deleting/delete-block-merge-contents-010-expected.txt:
- platform/chromium-win/editing/deleting/delete-block-table-expected.txt:
- platform/chromium-win/editing/deleting/delete-br-007-expected.txt:
- platform/chromium-win/editing/deleting/delete-character-001-expected.txt:
- platform/chromium-win/editing/deleting/delete-contiguous-ws-001-expected.txt:
- platform/chromium-win/editing/deleting/delete-hr-expected.txt:
- platform/chromium-win/editing/deleting/delete-leading-ws-001-expected.txt:
- platform/chromium-win/editing/deleting/delete-line-end-ws-001-expected.txt:
- platform/chromium-win/editing/deleting/delete-line-end-ws-002-expected.txt:
- platform/chromium-win/editing/deleting/delete-listitem-001-expected.txt:
- platform/chromium-win/editing/deleting/delete-selection-001-expected.txt:
- platform/chromium-win/editing/deleting/delete-tab-001-expected.txt:
- platform/chromium-win/editing/deleting/delete-tab-002-expected.txt:
- platform/chromium-win/editing/deleting/delete-tab-003-expected.txt:
- platform/chromium-win/editing/deleting/delete-tab-004-expected.txt:
- platform/chromium-win/editing/deleting/delete-trailing-ws-001-expected.txt:
- platform/chromium-win/editing/deleting/delete-trailing-ws-002-expected.txt:
- platform/chromium-win/editing/deleting/delete-ws-fixup-001-expected.txt:
- platform/chromium-win/editing/deleting/delete-ws-fixup-002-expected.txt:
- platform/chromium-win/editing/deleting/forward-delete-expected.txt:
- platform/chromium-win/editing/deleting/non-smart-delete-expected.txt:
- platform/chromium-win/editing/deleting/smart-delete-001-expected.txt:
- platform/chromium-win/editing/deleting/smart-delete-003-expected.txt:
- platform/chromium-win/editing/deleting/smart-delete-004-expected.txt:
- platform/chromium-win/editing/execCommand/insertHorizontalRule-expected.txt:
- platform/chromium-win/editing/inserting/4278698-expected.txt:
- platform/chromium-win/editing/inserting/editing-empty-divs-expected.txt:
- platform/chromium-win/editing/inserting/insert-3654864-fix-expected.txt:
- platform/chromium-win/editing/inserting/insert-after-delete-001-expected.txt:
- platform/chromium-win/editing/inserting/insert-br-003-expected.txt:
- platform/chromium-win/editing/inserting/insert-br-006-expected.txt:
- platform/chromium-win/editing/inserting/insert-br-007-expected.txt:
- platform/chromium-win/editing/inserting/insert-br-008-expected.txt:
- platform/chromium-win/editing/inserting/insert-br-009-expected.txt:
- platform/chromium-win/editing/inserting/insert-br-at-tabspan-002-expected.txt:
- platform/chromium-win/editing/inserting/insert-br-at-tabspan-003-expected.txt:
- platform/chromium-win/editing/inserting/insert-div-003-expected.txt:
- platform/chromium-win/editing/inserting/insert-div-006-expected.txt:
- platform/chromium-win/editing/inserting/insert-div-008-expected.txt:
- platform/chromium-win/editing/inserting/insert-div-024-expected.txt:
- platform/chromium-win/editing/inserting/insert-space-in-empty-doc-expected.txt:
- platform/chromium-win/editing/inserting/insert-tab-003-expected.txt:
- platform/chromium-win/editing/inserting/insert-text-at-tabspan-001-expected.txt:
- platform/chromium-win/editing/inserting/insert-text-at-tabspan-002-expected.txt:
- platform/chromium-win/editing/inserting/insert-text-with-newlines-expected.txt:
- platform/chromium-win/editing/inserting/typing-001-expected.txt:
- platform/chromium-win/editing/inserting/typing-002-expected.txt:
- platform/chromium-win/editing/inserting/typing-003-expected.txt:
- platform/chromium-win/editing/inserting/typing-around-br-001-expected.txt:
- platform/chromium-win/editing/inserting/typing-around-image-001-expected.txt:
- platform/chromium-win/editing/pasteboard/4076267-2-expected.txt:
- platform/chromium-win/editing/pasteboard/4076267-3-expected.txt:
- platform/chromium-win/editing/pasteboard/bad-placeholder-expected.txt:
- platform/chromium-win/editing/pasteboard/cut-text-001-expected.txt:
- platform/chromium-win/editing/pasteboard/interchange-newline-1-expected.txt:
- platform/chromium-win/editing/pasteboard/merge-end-1-expected.txt:
- platform/chromium-win/editing/pasteboard/merge-end-2-expected.txt:
- platform/chromium-win/editing/pasteboard/merge-end-4-expected.txt:
- platform/chromium-win/editing/pasteboard/merge-end-list-expected.txt:
- platform/chromium-win/editing/pasteboard/merge-end-table-expected.txt:
- platform/chromium-win/editing/pasteboard/paste-match-style-001-expected.txt:
- platform/chromium-win/editing/pasteboard/paste-text-001-expected.txt:
- platform/chromium-win/editing/pasteboard/paste-text-019-expected.txt:
- platform/chromium-win/editing/pasteboard/paste-text-at-tabspan-001-expected.txt:
- platform/chromium-win/editing/pasteboard/paste-text-at-tabspan-002-expected.txt:
- platform/chromium-win/editing/pasteboard/paste-text-at-tabspan-003-expected.txt:
- platform/chromium-win/editing/pasteboard/pasting-object-expected.txt:
- platform/chromium-win/editing/pasteboard/smart-paste-002-expected.txt:
- platform/chromium-win/editing/pasteboard/smart-paste-006-expected.txt:
- platform/chromium-win/editing/pasteboard/smart-paste-007-expected.txt:
- platform/chromium-win/editing/spelling/spelling-expected.txt:
- platform/chromium-win/editing/style/style-3681552-fix-001-expected.txt:
- platform/chromium-win/editing/style/style-3681552-fix-002-expected.txt:
- platform/chromium-win/editing/style/style-boundary-002-expected.txt:
- platform/chromium-win/editing/style/style-boundary-005-expected.txt:
- platform/chromium-win/editing/style/typing-style-001-expected.txt:
- platform/chromium-win/editing/style/typing-style-002-expected.txt:
- platform/chromium-win/editing/style/unbold-in-bold-expected.txt:
- platform/chromium-win/editing/undo/4063751-expected.txt:
- platform/chromium-win/editing/undo/redo-typing-001-expected.txt:
- platform/chromium-win/editing/undo/undo-combined-delete-boundary-expected.txt:
- platform/chromium-win/editing/undo/undo-combined-delete-expected.txt:
- platform/chromium-win/editing/undo/undo-delete-boundary-expected.txt:
- platform/chromium-win/editing/undo/undo-delete-expected.txt:
- platform/chromium-win/editing/undo/undo-forward-delete-boundary-expected.txt:
- platform/chromium-win/editing/undo/undo-forward-delete-expected.txt:
- platform/chromium-win/editing/undo/undo-misspellings-expected.txt:
- platform/chromium-win/editing/undo/undo-typing-001-expected.txt:
- platform/chromium-win/editing/unsupported-content/list-type-after-expected.txt:
- platform/chromium-win/editing/unsupported-content/list-type-before-expected.txt:
- platform/chromium-win/editing/unsupported-content/table-type-after-expected.txt:
- platform/chromium-win/editing/unsupported-content/table-type-before-expected.txt:
- 6:14 PM Changeset in webkit [74601] by
-
- 3 edits2 adds in trunk
2010-12-23 Ryosuke Niwa <rniwa@webkit.org>
Reviewed by Darin Adler.
FormatBlock doesn't preserve typing style but it should
https://bugs.webkit.org/show_bug.cgi?id=51140
Added a test to type text after bolding the typing style and executing a FormatBlock command.
Because FormatBlock command preserves typing style, the typed text should be in bold.
- editing/execCommand/format-block-typing-style.html: Added.
2010-12-23 Ryosuke Niwa <rniwa@webkit.org>
Reviewed by Darin Adler.
FormatBlock doesn't preserve typing style but it should
https://bugs.webkit.org/show_bug.cgi?id=51140
Fixed the bug by making preserveTypingStyle return true.
Test: editing/execCommand/format-block-typing-style.html
- editing/FormatBlockCommand.h: (WebCore::FormatBlockCommand::preservesTypingStyle):
- 5:40 PM Changeset in webkit [74600] by
-
- 12 edits1 move1 delete in trunk/JavaScriptCore
Rename RegexCompiler.cpp to RegexPattern.cpp.
Implicitly call compileRegex from RegexPattern's constructor.
Reviewed by Sam Weinig.
- Android.mk:
- CMakeLists.txt:
- GNUmakefile.am:
- JavaScriptCore.gypi:
- JavaScriptCore.pro:
- JavaScriptCore.vcproj/JavaScriptCore/JavaScriptCore.vcproj:
- JavaScriptCore.xcodeproj/project.pbxproj:
- runtime/RegExp.cpp:
(JSC::RegExp::compile):
- yarr/RegexCompiler.cpp: Removed.
- yarr/RegexCompiler.h: Removed.
- yarr/RegexInterpreter.cpp:
- yarr/RegexJIT.cpp:
- yarr/RegexPattern.cpp: Copied from JavaScriptCore/yarr/RegexCompiler.cpp.
(JSC::Yarr::compileRegex):
(JSC::Yarr::RegexPattern::RegexPattern):
- yarr/RegexPattern.h:
- 5:36 PM Changeset in webkit [74599] by
-
- 55 edits in trunk/LayoutTests
2010-12-23 Ryosuke Niwa <rniwa@webkit.org>
Unreviewed Qt rebaselines for r74593.
- platform/qt/editing/deleting/delete-3608445-fix-expected.txt:
- platform/qt/editing/deleting/delete-after-span-ws-001-expected.txt:
- platform/qt/editing/deleting/delete-after-span-ws-002-expected.txt:
- platform/qt/editing/deleting/delete-after-span-ws-003-expected.txt:
- platform/qt/editing/deleting/delete-block-merge-contents-010-expected.txt:
- platform/qt/editing/deleting/delete-character-001-expected.txt:
- platform/qt/editing/deleting/delete-hr-expected.txt:
- platform/qt/editing/deleting/delete-line-end-ws-001-expected.txt:
- platform/qt/editing/deleting/delete-selection-001-expected.txt:
- platform/qt/editing/deleting/delete-trailing-ws-001-expected.txt:
- platform/qt/editing/deleting/delete-trailing-ws-002-expected.txt:
- platform/qt/editing/deleting/non-smart-delete-expected.txt:
- platform/qt/editing/deleting/smart-delete-001-expected.txt:
- platform/qt/editing/deleting/smart-delete-003-expected.txt:
- platform/qt/editing/deleting/smart-delete-004-expected.txt:
- platform/qt/editing/inserting/insert-3654864-fix-expected.txt:
- platform/qt/editing/inserting/insert-after-delete-001-expected.txt:
- platform/qt/editing/inserting/insert-br-003-expected.txt:
- platform/qt/editing/inserting/insert-br-006-expected.txt:
- platform/qt/editing/inserting/insert-br-007-expected.txt:
- platform/qt/editing/inserting/insert-br-008-expected.txt:
- platform/qt/editing/inserting/insert-br-009-expected.txt:
- platform/qt/editing/inserting/insert-br-at-tabspan-002-expected.txt:
- platform/qt/editing/inserting/insert-br-at-tabspan-003-expected.txt:
- platform/qt/editing/inserting/insert-div-003-expected.txt:
- platform/qt/editing/inserting/insert-div-006-expected.txt:
- platform/qt/editing/inserting/insert-div-008-expected.txt:
- platform/qt/editing/inserting/insert-div-024-expected.txt:
- platform/qt/editing/inserting/insert-space-in-empty-doc-expected.txt:
- platform/qt/editing/inserting/insert-text-at-tabspan-001-expected.txt:
- platform/qt/editing/inserting/insert-text-at-tabspan-002-expected.txt:
- platform/qt/editing/inserting/typing-001-expected.txt:
- platform/qt/editing/inserting/typing-002-expected.txt:
- platform/qt/editing/inserting/typing-003-expected.txt:
- platform/qt/editing/inserting/typing-around-br-001-expected.txt:
- platform/qt/editing/inserting/typing-around-image-001-expected.txt:
- platform/qt/editing/pasteboard/4076267-3-expected.txt:
- platform/qt/editing/pasteboard/cut-text-001-expected.txt:
- platform/qt/editing/pasteboard/interchange-newline-1-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-4-expected.txt:
- platform/qt/editing/pasteboard/merge-end-list-expected.txt:
- platform/qt/editing/pasteboard/merge-end-table-expected.txt:
- platform/qt/editing/pasteboard/paste-match-style-001-expected.txt:
- platform/qt/editing/pasteboard/paste-text-019-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/pasting-object-expected.txt:
- platform/qt/editing/pasteboard/smart-paste-002-expected.txt:
- platform/qt/editing/pasteboard/smart-paste-006-expected.txt:
- platform/qt/editing/pasteboard/smart-paste-007-expected.txt:
- platform/qt/editing/style/remove-underline-from-stylesheet-expected.txt:
- platform/qt/editing/undo/redo-style-expected.txt:
- 5:19 PM Changeset in webkit [74598] by
-
- 2 edits in trunk/Tools
2010-12-23 Evan Martin <evan@chromium.org>
Reviewed by Ryosuke Niwa.
[chromium] LayoutTestController warning in float->int conversion
https://bugs.webkit.org/show_bug.cgi?id=51553
Rather than reading an int, converting to a float, then getting a compiler
warning on passing the float to something that expects an int, instead
just pass around ints.
No tests, fixes a compiler warning.
- DumpRenderTree/chromium/LayoutTestController.cpp: (parsePageSizeParameters): (LayoutTestController::pageNumberForElementById): (LayoutTestController::numberOfPages):
- 5:16 PM Changeset in webkit [74597] by
-
- 8 edits in trunk
Need WebKit2 API to create a SecurityOrigin from protocol, host, and port
https://bugs.webkit.org/show_bug.cgi?id=51563
Reviewed by Sam Weinig.
WebCore:
Add SecurityOrigin::create(const String& protocol, const String& host, int port).
- WebCore.exp.in:
Export the symbols for WebCore::SecurityOrigin::create(String, String, int).
- page/SecurityOrigin.cpp:
(WebCore::SecurityOrigin::createFromDatabaseIdentifier):
Use the constant MaxAllowedPort.
(WebCore::SecurityOrigin::create):
Sanitizing based off of that done in createFromDatabaseIdentifier.
- page/SecurityOrigin.h:
WebKit2:
Add WKSecurityOriginCreate(WKStringRef protocol, WKStringRef host, int port).
- Shared/API/c/WKSecurityOrigin.cpp:
(WKSecurityOriginCreateFromIdentifier):
Renamed so as not to clash with the new WKSecurityOriginCreate.
(WKSecurityOriginCreate):
- Shared/API/c/WKSecurityOrigin.h:
- Shared/WebSecurityOrigin.h:
(WebKit::WebSecurityOrigin::create):
- 5:11 PM Changeset in webkit [74596] by
-
- 2 edits in trunk/WebCore
2010-12-23 Patrick Gansterer <Patrick Gansterer>
Unreviewed build fix for WinCE after r73802.
Reapply r70812:
Port ContextMenuItemWin.cpp to WinCE
https://bugs.webkit.org/show_bug.cgi?id=48407
- platform/win/ContextMenuItemWin.cpp:
- 5:03 PM Changeset in webkit [74595] by
-
- 2 edits in trunk/JavaScriptCore
2010-12-23 Patrick Gansterer <Patrick Gansterer>
Unreviewed build fix for WinCE after r74360.
Move the OS(WINDOWS) section after the OS(WINCE) section
and add missing argument to the getStackMax call.
- wtf/StackBounds.cpp: (WTF::StackBounds::initialize):
- 4:55 PM Changeset in webkit [74594] by
-
- 2 edits in trunk/WebCore
2010-12-23 Adam Barth <abarth@webkit.org>
Remove include of non-existant file.
- bindings/v8/custom/V8DOMWindowCustom.cpp:
- 4:22 PM Changeset in webkit [74593] by
-
- 128 edits4 adds in trunk
2010-12-23 Ryosuke Niwa <rniwa@webkit.org>
Reviewed by Darin Adler.
Selection becomes stale when CharacterData is manipulated directly
https://bugs.webkit.org/show_bug.cgi?id=51389
The bug was caused by SelectionController's not adjusting selection when CharacterData's
m_data is modified by its API.
Fixed the bug by adding textWillBeReplaced to SelectionController and calling it in
CharacterData::textWillBeReplaced. Added newLength to setDataAndUpdate, which is passed to
textWillBeReplaced because inserting, replacing, or removing data may require moving
the offset of the selection end points (done by shouldRemovePositionAfterAdoptingTextReplacement).
Also fixed the condition for determining whether or not a node intersects the selection
in respondToNodeModification, which is extracted from nodeWillBeRemoved. The old condition
failed to recognize certain cases when selection started and ended with non-zero offsets.
Tests: editing/selection/character-data-mutation-crash.html
editing/selection/character-data-mutation.html
- dom/CharacterData.cpp: (WebCore::CharacterData::setData): Passes newLegnth to setDataAndUpdate. (WebCore::CharacterData::appendData): Ditto. (WebCore::CharacterData::insertData): Ditto. (WebCore::CharacterData::deleteData): Ditto. (WebCore::CharacterData::replaceData): Ditto. (WebCore::CharacterData::setDataAndUpdate): Added newLegnth to the argument list, and renamed lengthOfReplacedData to oldLength; calls textWillBeReplaced.
- dom/CharacterData.h:
- editing/SelectionController.cpp: (WebCore::removingNodeRemovesPosition): Removed indentations from blank lines. (WebCore::SelectionController::nodeWillBeRemoved): Calls nodeWillBeRemoved. (WebCore::SelectionController::respondToNodeModification): Extracted from nodeWillBeRemoved; fixed the logic to figure out whether a node is inside the range or not. (WebCore::shouldRemovePositionAfterAdoptingTextReplacement): Added. (WebCore::SelectionController::textWillBeReplaced): Added.
- editing/SelectionController.h:
2010-12-23 Ryosuke Niwa <rniwa@webkit.org>
Reviewed by Darin Adler.
Selection becomes stale when CharacterData is manipulated directly
https://bugs.webkit.org/show_bug.cgi?id=51389
Added tests to ensure WebKit adjusts or clears selection when the start container
or the end container of the selection's first range is modified by CharacterData's API.
Many editing tests had to be rebaselined due to different editing delegate dumps
caused by WebKit's properly adjusting selection's offsets as editing commands modify the DOM.
- editing/deleting/delete-by-word-001-expected.txt:
- editing/deleting/delete-by-word-002-expected.txt:
- editing/deleting/delete-ligature-001-expected.txt:
- editing/deleting/delete-ligature-002-expected.txt:
- editing/deleting/delete-ligature-003-expected.txt:
- editing/deleting/skip-virama-001-expected.txt:
- editing/execCommand/createLink-expected.txt:
- editing/execCommand/hilitecolor-expected.txt:
- editing/execCommand/unlink-expected.txt:
- editing/inserting/insert-thai-characters-001-expected.txt:
- editing/pasteboard/copy-in-password-field-expected.txt:
- editing/pasteboard/paste-into-anchor-text-expected.txt:
- editing/selection/character-data-mutation-crash-expected.txt: Added.
- editing/selection/character-data-mutation-crash.html: Added.
- editing/selection/character-data-mutation-expected.txt: Added.
- editing/selection/character-data-mutation.html: Added.
- editing/style/remove-underline-from-stylesheet-expected.txt:
- editing/style/typing-style-003-expected.txt:
- editing/undo/redo-style-expected.txt:
- platform/mac/editing/deleting/collapse-whitespace-3587601-fix-expected.txt:
- platform/mac/editing/deleting/delete-3608430-fix-expected.txt:
- platform/mac/editing/deleting/delete-3608445-fix-expected.txt:
- platform/mac/editing/deleting/delete-3608462-fix-expected.txt:
- platform/mac/editing/deleting/delete-after-span-ws-001-expected.txt:
- platform/mac/editing/deleting/delete-after-span-ws-002-expected.txt:
- platform/mac/editing/deleting/delete-after-span-ws-003-expected.txt:
- platform/mac/editing/deleting/delete-and-undo-expected.txt:
- platform/mac/editing/deleting/delete-block-merge-contents-010-expected.txt:
- platform/mac/editing/deleting/delete-block-table-expected.txt:
- platform/mac/editing/deleting/delete-br-007-expected.txt:
- platform/mac/editing/deleting/delete-character-001-expected.txt:
- platform/mac/editing/deleting/delete-contiguous-ws-001-expected.txt:
- platform/mac/editing/deleting/delete-hr-expected.txt:
- platform/mac/editing/deleting/delete-leading-ws-001-expected.txt:
- platform/mac/editing/deleting/delete-line-end-ws-001-expected.txt:
- platform/mac/editing/deleting/delete-line-end-ws-002-expected.txt:
- platform/mac/editing/deleting/delete-listitem-001-expected.txt:
- platform/mac/editing/deleting/delete-selection-001-expected.txt:
- platform/mac/editing/deleting/delete-tab-001-expected.txt:
- platform/mac/editing/deleting/delete-tab-002-expected.txt:
- platform/mac/editing/deleting/delete-tab-003-expected.txt:
- platform/mac/editing/deleting/delete-tab-004-expected.txt:
- platform/mac/editing/deleting/delete-trailing-ws-001-expected.txt:
- platform/mac/editing/deleting/delete-trailing-ws-002-expected.txt:
- platform/mac/editing/deleting/delete-ws-fixup-001-expected.txt:
- platform/mac/editing/deleting/delete-ws-fixup-002-expected.txt:
- platform/mac/editing/deleting/forward-delete-expected.txt:
- platform/mac/editing/deleting/non-smart-delete-expected.txt:
- platform/mac/editing/deleting/smart-delete-001-expected.txt:
- platform/mac/editing/deleting/smart-delete-002-expected.txt:
- platform/mac/editing/deleting/smart-delete-003-expected.txt:
- platform/mac/editing/deleting/smart-delete-004-expected.txt:
- platform/mac/editing/execCommand/insertHorizontalRule-expected.txt:
- platform/mac/editing/input/firstrectforcharacterrange-styled-expected.txt:
- platform/mac/editing/input/text-input-controller-expected.txt:
- platform/mac/editing/inserting/4278698-expected.txt:
- platform/mac/editing/inserting/editing-empty-divs-expected.txt:
- platform/mac/editing/inserting/insert-3654864-fix-expected.txt:
- platform/mac/editing/inserting/insert-after-delete-001-expected.txt:
- platform/mac/editing/inserting/insert-br-003-expected.txt:
- platform/mac/editing/inserting/insert-br-006-expected.txt:
- platform/mac/editing/inserting/insert-br-007-expected.txt:
- platform/mac/editing/inserting/insert-br-008-expected.txt:
- platform/mac/editing/inserting/insert-br-009-expected.txt:
- platform/mac/editing/inserting/insert-br-at-tabspan-002-expected.txt:
- platform/mac/editing/inserting/insert-br-at-tabspan-003-expected.txt:
- platform/mac/editing/inserting/insert-div-003-expected.txt:
- platform/mac/editing/inserting/insert-div-006-expected.txt:
- platform/mac/editing/inserting/insert-div-008-expected.txt:
- platform/mac/editing/inserting/insert-div-024-expected.txt:
- platform/mac/editing/inserting/insert-space-in-empty-doc-expected.txt:
- platform/mac/editing/inserting/insert-tab-003-expected.txt:
- platform/mac/editing/inserting/insert-text-at-tabspan-001-expected.txt:
- platform/mac/editing/inserting/insert-text-at-tabspan-002-expected.txt:
- platform/mac/editing/inserting/insert-text-with-newlines-expected.txt:
- platform/mac/editing/inserting/typing-001-expected.txt:
- platform/mac/editing/inserting/typing-002-expected.txt:
- platform/mac/editing/inserting/typing-003-expected.txt:
- platform/mac/editing/inserting/typing-around-br-001-expected.txt:
- platform/mac/editing/inserting/typing-around-image-001-expected.txt:
- platform/mac/editing/pasteboard/4076267-2-expected.txt:
- platform/mac/editing/pasteboard/4076267-3-expected.txt:
- platform/mac/editing/pasteboard/bad-placeholder-expected.txt:
- platform/mac/editing/pasteboard/cut-text-001-expected.txt:
- platform/mac/editing/pasteboard/emacs-ctrl-a-k-y-expected.txt:
- platform/mac/editing/pasteboard/emacs-ctrl-k-y-001-expected.txt:
- platform/mac/editing/pasteboard/interchange-newline-1-expected.txt:
- platform/mac/editing/pasteboard/merge-end-1-expected.txt:
- platform/mac/editing/pasteboard/merge-end-2-expected.txt:
- platform/mac/editing/pasteboard/merge-end-4-expected.txt:
- platform/mac/editing/pasteboard/merge-end-list-expected.txt:
- platform/mac/editing/pasteboard/merge-end-table-expected.txt:
- platform/mac/editing/pasteboard/paste-match-style-001-expected.txt:
- platform/mac/editing/pasteboard/paste-text-001-expected.txt:
- platform/mac/editing/pasteboard/paste-text-019-expected.txt:
- platform/mac/editing/pasteboard/paste-text-at-tabspan-001-expected.txt:
- platform/mac/editing/pasteboard/paste-text-at-tabspan-002-expected.txt:
- platform/mac/editing/pasteboard/paste-text-at-tabspan-003-expected.txt:
- platform/mac/editing/pasteboard/paste-xml-expected.txt:
- platform/mac/editing/pasteboard/pasting-object-expected.txt:
- platform/mac/editing/pasteboard/smart-paste-002-expected.txt:
- platform/mac/editing/pasteboard/smart-paste-006-expected.txt:
- platform/mac/editing/pasteboard/smart-paste-007-expected.txt:
- platform/mac/editing/pasteboard/subframe-dragndrop-1-expected.txt:
- platform/mac/editing/spelling/spelling-expected.txt:
- platform/mac/editing/style/style-3681552-fix-001-expected.txt:
- platform/mac/editing/style/style-3681552-fix-002-expected.txt:
- platform/mac/editing/style/style-boundary-002-expected.txt:
- platform/mac/editing/style/style-boundary-005-expected.txt:
- platform/mac/editing/style/typing-style-001-expected.txt:
- platform/mac/editing/style/typing-style-002-expected.txt:
- platform/mac/editing/style/unbold-in-bold-expected.txt:
- platform/mac/editing/undo/4063751-expected.txt:
- platform/mac/editing/undo/redo-typing-001-expected.txt:
- platform/mac/editing/undo/undo-combined-delete-boundary-expected.txt:
- platform/mac/editing/undo/undo-combined-delete-expected.txt:
- platform/mac/editing/undo/undo-delete-boundary-expected.txt:
- platform/mac/editing/undo/undo-delete-expected.txt:
- platform/mac/editing/undo/undo-forward-delete-boundary-expected.txt:
- platform/mac/editing/undo/undo-forward-delete-expected.txt:
- platform/mac/editing/undo/undo-misspellings-expected.txt:
- platform/mac/editing/undo/undo-typing-001-expected.txt:
- platform/mac/editing/unsupported-content/list-type-after-expected.txt:
- platform/mac/editing/unsupported-content/list-type-before-expected.txt:
- platform/mac/editing/unsupported-content/table-type-after-expected.txt:
- platform/mac/editing/unsupported-content/table-type-before-expected.txt:
- 4:00 PM Changeset in webkit [74592] by
-
- 2 edits2 deletes in trunk/WebCore
2010-12-23 Adam Barth <abarth@webkit.org>
Reviewed by Darin Adler.
Remove unneeded V8 bindings files
https://bugs.webkit.org/show_bug.cgi?id=51570
- WebCore.gypi:
- bindings/generic/BindingDOMWindow.h: Removed.
- bindings/v8/specialization/V8BindingDOMWindow.h: Removed.
- 3:55 PM Changeset in webkit [74591] by
-
- 3 edits in trunk/WebKit2
2010-12-23 Anders Carlsson <andersca@apple.com>
Reviewed by Simon Fraser.
Ensure that we are not getting too many mouse move events
https://bugs.webkit.org/show_bug.cgi?id=51568
<rdar://problem/7881470>
<rdar://problem/8735512>
Implement throttling of mouse move events.
- UIProcess/WebPageProxy.cpp: (WebKit::WebPageProxy::WebPageProxy): Initialize m_processingMouseMoveEvent to false.
(WebKit::WebPageProxy::handleMouseEvent):
If we're already processing a mouse move events, just update
m_nextMouseMoveEvent and return.
(WebKit::WebPageProxy::didReceiveEvent):
If we received a mouse move event, send the next mouse move event if there is one.
- 3:43 PM Changeset in webkit [74590] by
-
- 2 edits in trunk/WebCore
Build fix.
- editing/EditorCommand.cpp:
(WebCore::enabledDismissCorrectionPanel):
- 3:42 PM Changeset in webkit [74589] by
-
- 3 edits in trunk/WebCore
Try to fix crash seen on GTK bot.
- editing/EditorCommand.cpp:
(WebCore::internalCommand): Add back empty string check I accidentally deleted in r74580.
- WebCore.xcodeproj/project.pbxproj: Let Xcode edit the file.
- 3:36 PM Changeset in webkit [74588] by
-
- 3 edits in trunk/WebKit2
Assertion when loading www.yahoo.com in WebKit2 under decodeResourceRequest
https://bugs.webkit.org/show_bug.cgi?id=51562
Reviewed by Anders Carlsson.
Don't assert that our data reference isn't empty when we try to get the data from it.
If it is empty, just return 0.
All callers are safe with this function returning 0.
- Platform/CoreIPC/DataReference.h:
(CoreIPC::DataReference::data): Change the assert to an early return.
- Shared/cf/ArgumentCodersCF.cpp:
(CoreIPC::decode): Add an assert, and remove a ternarny operator because the function now does
this for us (if the data is empty, return 0).
- 3:15 PM Changeset in webkit [74587] by
-
- 5 edits in trunk
2010-12-23 Sheriff Bot <webkit.review.bot@gmail.com>
Unreviewed, rolling out r74561.
http://trac.webkit.org/changeset/74561
https://bugs.webkit.org/show_bug.cgi?id=51565
"Broke Chromium UI tests on Vista" (Requested by kbr_google on
#webkit).
- src/GraphicsContext3DChromium.cpp: (WebCore::GraphicsContext3DInternal::paintRenderingResultsToCanvas):
2010-12-23 Sheriff Bot <webkit.review.bot@gmail.com>
Unreviewed, rolling out r74561.
http://trac.webkit.org/changeset/74561
https://bugs.webkit.org/show_bug.cgi?id=51565
"Broke Chromium UI tests on Vista" (Requested by kbr_google on
#webkit).
- platform/graphics/chromium/ImageLayerChromium.cpp: (WebCore::ImageLayerChromium::updateContentsIfDirty):
- platform/graphics/skia/GraphicsContext3DSkia.cpp: (WebCore::GraphicsContext3D::getImageData):
- 3:10 PM Changeset in webkit [74586] by
-
- 2 edits in trunk/WebKit/mac
2010-12-23 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
WebScriptDebugger in WebKit has empty implementations for willExecuteProgram and didExecuteProgram. As a result,
if the top call frame is from a program, WebKitScriptDebugger doesn't record that callframe as the top frame, and
WebScriptDebugger's callframe stack is wrong from this point. That could cause crash if we trying to access the top
call frame from this stack when an exception throws because the saved top frame could be invalid.
To fix that, we need to maintain the call frame stack in willExecuteProgram and didExecuteProgram, as we did in
callEvent and returnEvent.
- WebView/WebScriptDebugger.mm: (WebScriptDebugger::willExecuteProgram): (WebScriptDebugger::didExecuteProgram):
- 3:08 PM Changeset in webkit [74585] by
-
- 2 edits in trunk/LayoutTests
2010-12-23 Kenneth Russell <kbr@google.com>
Unreviewed Chromium test expectations update. Fix botched update
of fast/css/box-shadow.html expectation; needed to be FAIL, not
IMAGE+TEXT IMAGE.
- platform/chromium/test_expectations.txt:
- 2:49 PM Changeset in webkit [74584] by
-
- 2 edits in trunk/WebKit2
Fix incorrect assertion causing all pages with subframes to crash.
- UIProcess/WebFrameProxy.cpp:
(WebKit::WebFrameProxy::appendChild):
- 2:40 PM Changeset in webkit [74583] by
-
- 1 edit in trunk/WebKit2/ChangeLog
Fixed out-of-order ChangeLog.
- 2:34 PM Changeset in webkit [74582] by
-
- 18 edits1 add in trunk/WebKit2
2010-12-23 Darin Adler <Darin Adler>
Reviewed by Sam Weinig.
WKView should not try to do asynchronous validation for selectors that are not editor commands
https://bugs.webkit.org/show_bug.cgi?id=51555
- UIProcess/API/mac/WKView.mm: (-[WKView validateUserInterfaceItem:]): Removed the special case for startSpeaking. Added call to commandIsSupportedFromMenuOrKeyBinding so we only try to do validation for commands that are supported. Tweaked comments and added some bug numbers. (-[WKView _setUserInterfaceItemState:enabled:state:]): Tweaked comment and added bug number.
- 2:23 PM Changeset in webkit [74581] by
-
- 7 edits1 add in trunk
2010-12-23 Mihai Parparita <mihaip@chromium.org>
Reviewed by Kent Tamura.
[Chromium] Add WebThemeEngine for Mac to allow scrollbar rendering to be overridden for the DRT
https://bugs.webkit.org/show_bug.cgi?id=51507
Adds WebThemeEngine for the Mac (it already has parallel definitions for
Windows and Linux) so that scrollbar thumb rendering can be overridden
for the DRT (to be consistent with the NSScroller-based rendering
used by the Mac port).
No new tests since the functionality is not exposed yet. Eventually will
result in the Chromium/Mac port being able to use the same pixel
baselines for layout tests as the Mac port.
- platform/chromium/ChromiumBridge.h:
- platform/chromium/ScrollbarThemeChromiumMac.mm: (WebCore::scrollbarStateToThemeState): (WebCore::ScrollbarThemeChromiumMac::paint):
2010-12-23 Mihai Parparita <mihaip@chromium.org>
Reviewed by Kent Tamura.
[Chromium] Add WebThemeEngine for Mac to allow scrollbar rendering to be overridden for the DRT
https://bugs.webkit.org/show_bug.cgi?id=51507
Adds WebThemeEngine for the Mac (it already has parallel definitions for
Windows and Linux) so that scrollbar thumb rendering can be overridden
for the DRT (to be consistent with the NSScroller-based rendering
used by the Mac port).
- public/mac/WebThemeEngine.h: Added. (WebKit::WebThemeEngine::paintScrollbarThumb):
- src/AssertMatchingEnums.cpp:
- src/ChromiumBridge.cpp: (WebCore::ChromiumBridge::paintScrollbarThumb):
- 2:20 PM Changeset in webkit [74580] by
-
- 6 edits in trunk
2010-12-23 Darin Adler <Darin Adler>
Reviewed by Sam Weinig.
WKView should not try to do asynchronous validation for selectors that are not editor commands
https://bugs.webkit.org/show_bug.cgi?id=51555
- WebCore.exp.in: Added commandIsSupportedFromMenuOrKeyBinding.
- editing/Editor.h: Reordered arguments in the Editor::Command constructor and the data members too so the frame is last. Added commandIsSupportedFromMenuOrKeyBinding.
- editing/EditorCommand.cpp: (WebCore::supported): Removed the EditorCommandSource argument. These functions are now only used when called from DOM. (WebCore::supportedFromMenuOrKeyBinding): Ditto. (WebCore::supportedCopyCut): Ditto. (WebCore::supportedPaste): Ditto. (WebCore::enabledDismissCorrectionPanel): Changed the supported function to an enabled function. It was incorrect to say that this is "supported" only when the correction panel is up. Correct to say that it is "enabled" only then. And also probably OK to enable it even when the selection is not in editable text, as long as the panel is up. (WebCore::createCommandMap): Moved conditional commands out of the main array into a separate section at the end. (WebCore::internalCommand): Added. (WebCore::Editor::command): Changed to use the new internalCommand function and simplified by relying on the null check in the Command constructor. (WebCore::Editor::commandIsSupportedFromMenuOrKeyBinding): Added. (WebCore::Editor::Command::Command): Removed unneeded initialization of m_source, which is never looked at if m_command is 0. Added feature of passing a null command pointer to the non-default constructor. (WebCore::Editor::Command::isSupported): Changed to only call the per-command isSupported function when the command source is DOM. Accordingly that function is now called isSupportedFromDOM.
2010-12-23 Darin Adler <Darin Adler>
Reviewed by Sam Weinig.
WKView should not try to do asynchronous validation for selectors that are not editor commands
https://bugs.webkit.org/show_bug.cgi?id=51555
- UIProcess/API/mac/WKView.mm: (-[WKView validateUserInterfaceItem:]): Removed the special case for startSpeaking. Added call to commandIsSupportedFromMenuOrKeyBinding so we only try to do validation for commands that are supported. Tweaked comments and added some bug numbers. (-[WKView _setUserInterfaceItemState:enabled:state:]): Tweaked comment and added bug number.
- 2:15 PM Changeset in webkit [74579] by
-
- 2 edits in trunk/WebCore
2010-12-23 Matthew Delaney <mdelaney@apple.com>
Reviewed by Simon Fraser.
Incorrect layer property used noted as changed in GraphicsLayerCA::setAcceleratesDrawing
https://bugs.webkit.org/show_bug.cgi?id=51560
No new tests.
- platform/graphics/ca/GraphicsLayerCA.cpp: (WebCore::GraphicsLayerCA::setAcceleratesDrawing):
- 1:55 PM WebKit Team edited by
- add myself to commiter list (diff)
- 1:28 PM Changeset in webkit [74578] by
-
- 79 edits in trunk/LayoutTests
Unreviewed. Chromium layout test expectations update.
https://bugs.webkit.org/show_bug.cgi?id=51500
Rebaseline media control layout tests due to pixel diff from r74535
- platform/chromium-linux/media/audio-controls-rendering-expected.checksum:
- platform/chromium-linux/media/audio-controls-rendering-expected.png:
- platform/chromium-linux/media/controls-after-reload-expected.checksum:
- platform/chromium-linux/media/controls-after-reload-expected.png:
- platform/chromium-linux/media/controls-strict-expected.checksum:
- platform/chromium-linux/media/controls-strict-expected.png:
- platform/chromium-linux/media/controls-styling-expected.checksum:
- platform/chromium-linux/media/controls-styling-expected.png:
- platform/chromium-linux/media/controls-without-preload-expected.checksum:
- platform/chromium-linux/media/controls-without-preload-expected.png:
- platform/chromium-linux/media/video-controls-rendering-expected.checksum:
- platform/chromium-linux/media/video-controls-rendering-expected.png:
- platform/chromium-linux/media/video-display-toggle-expected.checksum:
- platform/chromium-linux/media/video-display-toggle-expected.png:
- platform/chromium-linux/media/video-empty-source-expected.checksum:
- platform/chromium-linux/media/video-empty-source-expected.png:
- platform/chromium-linux/media/video-no-audio-expected.checksum:
- platform/chromium-linux/media/video-no-audio-expected.png:
- platform/chromium-linux/media/video-volume-slider-expected.checksum:
- platform/chromium-linux/media/video-volume-slider-expected.png:
- platform/chromium-mac/media/audio-controls-rendering-expected.checksum:
- platform/chromium-mac/media/audio-controls-rendering-expected.png:
- platform/chromium-mac/media/audio-controls-rendering-expected.txt:
- platform/chromium-mac/media/controls-after-reload-expected.checksum:
- platform/chromium-mac/media/controls-after-reload-expected.png:
- platform/chromium-mac/media/controls-after-reload-expected.txt:
- platform/chromium-mac/media/controls-strict-expected.checksum:
- platform/chromium-mac/media/controls-strict-expected.png:
- platform/chromium-mac/media/controls-strict-expected.txt:
- platform/chromium-mac/media/controls-styling-expected.checksum:
- platform/chromium-mac/media/controls-styling-expected.png:
- platform/chromium-mac/media/controls-styling-expected.txt:
- platform/chromium-mac/media/controls-without-preload-expected.checksum:
- platform/chromium-mac/media/controls-without-preload-expected.png:
- platform/chromium-mac/media/controls-without-preload-expected.txt:
- platform/chromium-mac/media/video-display-toggle-expected.checksum:
- platform/chromium-mac/media/video-display-toggle-expected.png:
- platform/chromium-mac/media/video-display-toggle-expected.txt:
- platform/chromium-mac/media/video-empty-source-expected.checksum:
- platform/chromium-mac/media/video-empty-source-expected.png:
- platform/chromium-mac/media/video-empty-source-expected.txt:
- platform/chromium-mac/media/video-no-audio-expected.checksum:
- platform/chromium-mac/media/video-no-audio-expected.png:
- platform/chromium-mac/media/video-no-audio-expected.txt:
- platform/chromium-mac/media/video-volume-slider-expected.checksum:
- platform/chromium-mac/media/video-volume-slider-expected.png:
- platform/chromium-mac/media/video-volume-slider-expected.txt:
- platform/chromium-win/media/audio-controls-rendering-expected.checksum:
- platform/chromium-win/media/audio-controls-rendering-expected.png:
- platform/chromium-win/media/audio-controls-rendering-expected.txt:
- platform/chromium-win/media/controls-after-reload-expected.checksum:
- platform/chromium-win/media/controls-after-reload-expected.png:
- platform/chromium-win/media/controls-after-reload-expected.txt:
- platform/chromium-win/media/controls-strict-expected.checksum:
- platform/chromium-win/media/controls-strict-expected.png:
- platform/chromium-win/media/controls-strict-expected.txt:
- platform/chromium-win/media/controls-styling-expected.checksum:
- platform/chromium-win/media/controls-styling-expected.png:
- platform/chromium-win/media/controls-styling-expected.txt:
- platform/chromium-win/media/controls-without-preload-expected.checksum:
- platform/chromium-win/media/controls-without-preload-expected.png:
- platform/chromium-win/media/controls-without-preload-expected.txt:
- platform/chromium-win/media/video-controls-rendering-expected.checksum:
- platform/chromium-win/media/video-controls-rendering-expected.png:
- platform/chromium-win/media/video-controls-rendering-expected.txt:
- platform/chromium-win/media/video-display-toggle-expected.checksum:
- platform/chromium-win/media/video-display-toggle-expected.png:
- platform/chromium-win/media/video-display-toggle-expected.txt:
- platform/chromium-win/media/video-empty-source-expected.checksum:
- platform/chromium-win/media/video-empty-source-expected.png:
- platform/chromium-win/media/video-empty-source-expected.txt:
- platform/chromium-win/media/video-no-audio-expected.checksum:
- platform/chromium-win/media/video-no-audio-expected.png:
- platform/chromium-win/media/video-no-audio-expected.txt:
- platform/chromium-win/media/video-volume-slider-expected.checksum:
- platform/chromium-win/media/video-volume-slider-expected.png:
- platform/chromium-win/media/video-volume-slider-expected.txt:
- platform/chromium/test_expectations.txt:
- 1:08 PM Changeset in webkit [74577] by
-
- 4 edits in trunk
2010-12-23 Jan Erik Hanssen <jhanssen@sencha.com>
Reviewed by Andreas Kling.
[Qt] copy and paste greek symbols to Word, TextEdit results in ?'s being shown
https://bugs.webkit.org/show_bug.cgi?id=35635
Explicitly set text/html data on QClipboard as UTF-8 and also include a <meta>
tag informing the OS that the data is in UTF-8 format.
- platform/qt/PasteboardQt.cpp: (WebCore::Pasteboard::writeSelection):
2010-12-23 Jan Erik Hanssen <jhanssen@sencha.com>
Reviewed by Andreas Kling.
[Qt] copy and paste greek symbols to Word, TextEdit results in ?'s being shown
https://bugs.webkit.org/show_bug.cgi?id=35635
Test that QClipboard contains the charset information and unicode data
after copying unicode text.
- tests/qwebpage/tst_qwebpage.cpp: (tst_QWebPage::macCopyUnicodeToClipboard):
- 1:02 PM Changeset in webkit [74576] by
-
- 2 edits in trunk/LayoutTests
2010-12-23 Kenneth Russell <kbr@google.com>
Unreviewed Chromium expectations update. Suppress crashes seen on
Windows with fast/canvas/webgl/canvas-test.html and
fast/canvas/webgl/gl-pixelstorei.html.
- platform/chromium/test_expectations.txt:
- 12:58 PM Changeset in webkit [74575] by
-
- 2 edits in trunk/WebKit2
<rdar://problem/8799032>
REGRESSION(r74436) Crash opening many pages with WebKit2
Reviewed by Anders Carlsson.
- Shared/cf/ArgumentCodersCF.cpp:
(CoreIPC::decode):
Fix paste-o. Decode into a CFDataRef not a CFDictionaryRef
when decoding CFDataRefs.
- 12:38 PM Changeset in webkit [74574] by
-
- 6 edits2 adds in trunk
2010-12-23 Justin Schuh <jschuh@chromium.org>
Reviewed by Eric Seidel.
CSSCursorImageValue destructor should clear referenced element.
https://bugs.webkit.org/show_bug.cgi?id=51417
Calls correct method to clear image and renames method to avoid future confusion.
Test: svg/css/cursor-image-replace.svg
- css/CSSCursorImageValue.cpp: (WebCore::CSSCursorImageValue::~CSSCursorImageValue):
- svg/SVGElement.cpp: (WebCore::SVGElement::cursorImageValueRemoved):
- svg/SVGElement.h:
2010-12-23 Justin Schuh <jschuh@chromium.org>
Reviewed by Eric Seidel.
CSSCursorImageValue destructor should clear referenced element.
https://bugs.webkit.org/show_bug.cgi?id=51417
- svg/css/cursor-image-replace-expected.txt: Added.
- svg/css/cursor-image-replace.svg: Added.
- 12:22 PM Changeset in webkit [74573] by
-
- 2 edits in trunk/LayoutTests
2010-12-23 Kenneth Russell <kbr@google.com>
Unreviewed Chromium expectations update. Allow
fast/css/box-shadow.html to fail with IMAGE as well as IMAGE+TEXT.
- platform/chromium/test_expectations.txt:
- 12:11 PM Changeset in webkit [74572] by
-
- 2 edits in trunk/WebKit/chromium
Fix build.
- src/FrameLoaderClientImpl.cpp:
(WebKit::FrameLoaderClientImpl::didSaveToPageCache):
(WebKit::FrameLoaderClientImpl::didRestoreFromPageCache):
- 12:10 PM Changeset in webkit [74571] by
-
- 43 edits in trunk
WebKit2 needs to mirror the frame tree in the UIProcess
https://bugs.webkit.org/show_bug.cgi?id=51546
Reviewed by Darin Adler.
WebCore:
- Add client functions to notify that a frame has been added or removed from the page cache.
- history/CachedFrame.cpp:
(WebCore::CachedFrameBase::restore):
(WebCore::CachedFrame::CachedFrame):
- history/CachedFrame.h:
- loader/EmptyClients.h:
(WebCore::EmptyFrameLoaderClient::didSaveToPageCache):
(WebCore::EmptyFrameLoaderClient::didRestoreFromPageCache):
- loader/FrameLoaderClient.h:
WebKit/chromium:
- Add client functions to notify that a frame has been added or removed from the page cache.
- src/FrameLoaderClientImpl.cpp:
(WebKit::WebFrameLoaderClientImpl::didSaveToPageCache):
(WebKit::WebFrameLoaderClientImpl::didRestoreFromPageCache):
- src/FrameLoaderClientImpl.h:
WebKit/efl:
- Add client functions to notify that a frame has been added or removed from the page cache.
- WebCoreSupport/FrameLoaderClientEfl.cpp:
(WebCore::FrameLoaderClientEfl::didSaveToPageCache):
(WebCore::FrameLoaderClientEfl::didRestoreFromPageCache):
- WebCoreSupport/FrameLoaderClientEfl.h:
WebKit/gtk:
- Add client functions to notify that a frame has been added or removed from the page cache.
- WebCoreSupport/FrameLoaderClientGtk.cpp:
(WebKit::FrameLoaderClient::didSaveToPageCache):
(WebKit::FrameLoaderClient::didRestoreFromPageCache):
- WebCoreSupport/FrameLoaderClientGtk.h:
WebKit/haiku:
- Add client functions to notify that a frame has been added or removed from the page cache.
- WebCoreSupport/FrameLoaderClientHaiku.cpp:
(WebCore::FrameLoaderClientHaiku::didSaveToPageCache):
(WebCore::FrameLoaderClientHaiku::didRestoreFromPageCache):
- WebCoreSupport/FrameLoaderClientHaiku.h:
WebKit/mac:
- Add client functions to notify that a frame has been added or removed from the page cache.
- WebCoreSupport/WebFrameLoaderClient.h:
- WebCoreSupport/WebFrameLoaderClient.mm:
(WebFrameLoaderClient::didSaveToPageCache):
(WebFrameLoaderClient::didRestoreFromPageCache):
WebKit/qt:
- Add client functions to notify that a frame has been added or removed from the page cache.
- WebCoreSupport/FrameLoaderClientQt.cpp:
(WebCore::FrameLoaderClientQt::didSaveToPageCache):
(WebCore::FrameLoaderClientQt::didRestoreFromPageCache):
- WebCoreSupport/FrameLoaderClientQt.h:
WebKit/win:
- Add client functions to notify that a frame has been added or removed from the page cache.
- WebCoreSupport/WebFrameLoaderClient.cpp:
(WebFrameLoaderClient::didSaveToPageCache):
(WebFrameLoaderClient::didRestoreFromPageCache):
- WebCoreSupport/WebFrameLoaderClient.h:
WebKit/wince:
- Add client functions to notify that a frame has been added or removed from the page cache.
- WebCoreSupport/FrameLoaderClientWinCE.cpp:
(WebKit::FrameLoaderClientWinCE::didSaveToPageCache):
(WebKit::FrameLoaderClientWinCE::didRestoreFromPageCache):
- WebCoreSupport/FrameLoaderClientWinCE.h:
WebKit/wx:
- Add client functions to notify that a frame has been added or removed from the page cache.
- WebKitSupport/FrameLoaderClientWx.cpp:
(WebCore::FrameLoaderClientWx::didSaveToPageCache):
(WebCore::FrameLoaderClientWx::didRestoreFromPageCache):
- WebKitSupport/FrameLoaderClientWx.h:
WebKit2:
- UIProcess/API/C/WKFrame.cpp:
(WKFrameCopyChildFrames):
- UIProcess/API/C/WKFrame.h:
Add API to get the child frames of a frame.
- UIProcess/WebFrameProxy.cpp:
(WebKit::WebFrameProxy::WebFrameProxy):
(WebKit::WebFrameProxy::disconnect):
(WebKit::WebFrameProxy::appendChild):
(WebKit::WebFrameProxy::removeChild):
(WebKit::WebFrameProxy::isDescendantOf):
(WebKit::WebFrameProxy::dumpFrameTree):
(WebKit::WebFrameProxy::didRemoveFromHierarchy):
(WebKit::WebFrameProxy::childFrames):
- UIProcess/WebFrameProxy.h:
(WebKit::WebFrameProxy::parentFrame): Add getter.
(WebKit::WebFrameProxy::nextSibling): Add getter.
(WebKit::WebFrameProxy::previousSibling): Add getter.
(WebKit::WebFrameProxy::firstChild): Add getter.
(WebKit::WebFrameProxy::lastChild): Add getter.
Make frames keep track of their subframes.
- UIProcess/WebPageProxy.cpp:
(WebKit::WebPageProxy::didCreateSubframe):
Use the now passed parentID to insert the frame into the hierarchy
(WebKit::WebPageProxy::didSaveFrameToPageCache):
When a frame is added to the page cache, remove it from the hierarchy.
(WebKit::WebPageProxy::didRestoreFrameFromPageCache):
When a frame is restored from the page cache, add it back to the hierarchy.
(WebKit::WebPageProxy::didRemoveFrameFromHierarchy):
Detached frames should also be removed from the hierarchy.
- UIProcess/WebPageProxy.h:
Add new declarations for didSaveFrameToPageCache and didRestoreFrameFromPageCache.
- UIProcess/WebPageProxy.messages.in:
Add messages for DidSaveFrameToPageCache and DidRestoreFrameFromPageCache.
- WebProcess/WebCoreSupport/WebFrameLoaderClient.cpp:
(WebKit::WebFrameLoaderClient::savePlatformDataToCachedFrame):
(WebKit::WebFrameLoaderClient::transitionToCommittedFromCachedFrame):
Remove notImplemented, there is no work we need to do here.
(WebKit::WebFrameLoaderClient::didSaveToPageCache):
(WebKit::WebFrameLoaderClient::didRestoreFromPageCache):
Send message to UIProcess to indicate this happened.
- WebProcess/WebCoreSupport/WebFrameLoaderClient.h:
- WebProcess/WebPage/WebFrame.cpp:
(WebKit::WebFrame::createSubframe):
Send parent frameID in addition to our own in the message to UIProcess.
- 11:57 AM Changeset in webkit [74570] by
-
- 6 edits6 adds in trunk
2010-12-22 Matthew Delaney <mdelaney@apple.com>
Reviewed by Simon Fraser.
Add setNeedsStyleRecalc for getting canvas contexts that use IOSurfaces
This patch also changes USE_IOSURFACE to instead follow USE(IOSURFACE)
https://bugs.webkit.org/show_bug.cgi?id=51279
- html/HTMLCanvasElement.cpp: For IOSurface, use setNeedsStyleRecalc
- html/canvas/CanvasRenderingContext2D.cpp: Use new flag
- platform/graphics/ImageBuffer.h: Use new flag
- platform/graphics/cg/ImageBufferCG.cpp: Use new flag
2010-12-22 Matthew Delaney <mdelaney@apple.com>
Reviewed by Simon Fraser.
Add setNeedsStyleRecalc for getting canvas contexts that use IOSurfaces
https://bugs.webkit.org/show_bug.cgi?id=51279
- platform/mac-snowleopard/platform/mac/compositing: Added.
- platform/mac-snowleopard/platform/mac/compositing/canvas: Added.
- platform/mac-snowleopard/platform/mac/compositing/canvas/accelerated-canvas-compositing-expected.txt: Added.
- platform/mac/compositing/canvas: Added.
- platform/mac/compositing/canvas/accelerated-canvas-compositing-expected.txt: Added.
- platform/mac/compositing/canvas/accelerated-canvas-compositing.html: Added.
- 11:33 AM Changeset in webkit [74569] by
-
- 3 edits in trunk/WebKit2
2010-12-23 Anders Carlsson <andersca@apple.com>
Reviewed by Sam Weinig.
New popup windows open disproportionately big
https://bugs.webkit.org/show_bug.cgi?id=51547
<rdar://problem/8637029>
- UIProcess/WebPageProxy.cpp: (WebKit::WebPageProxy::createNewPage): Pass the correct size back to the UI process.
- WebProcess/WebCoreSupport/WebChromeClient.cpp: (WebKit::WebChromeClient::pageRect): Implement this and return the size of the page.
- 11:16 AM Changeset in webkit [74568] by
-
- 11 edits2 adds in trunk
2010-12-20 Adrienne Walker <enne@google.com>
Reviewed by Kenneth Russell.
[chromium] Tile root layer of the compositor.
https://bugs.webkit.org/show_bug.cgi?id=49947
Refactor root layer update and drawing from LayerRendererChromium into
LayerTilerChromium. The root layer is now drawn as multiple tiles
rather than as one single large texture. Scrollbars are now drawn
separately rather than as part of the root layer.
Test: LayoutTests/compositing/
- WebCore.gypi:
- page/FrameView.cpp: (WebCore::FrameView::repaintContentRectangle):
- platform/ScrollView.cpp: (WebCore::ScrollView::ScrollView): (WebCore::ScrollView::setClipsRepaints):
- platform/ScrollView.h: (WebCore::ScrollView::clipsRepaints):
- platform/graphics/chromium/LayerChromium.h:
- platform/graphics/chromium/LayerRendererChromium.cpp: (WebCore::LayerRendererChromium::LayerRendererChromium): (WebCore::LayerRendererChromium::~LayerRendererChromium): (WebCore::LayerRendererChromium::useShader): (WebCore::LayerRendererChromium::verticalScrollbarRect): (WebCore::LayerRendererChromium::horizontalScrollbarRect): (WebCore::LayerRendererChromium::invalidateRootLayerRect): (WebCore::LayerRendererChromium::updateAndDrawRootLayer): (WebCore::LayerRendererChromium::drawLayers): (WebCore::LayerRendererChromium::setRootLayer): (WebCore::LayerRendererChromium::initializeSharedObjects): (WebCore::LayerRendererChromium::cleanupSharedObjects):
- platform/graphics/chromium/LayerRendererChromium.h:
- platform/graphics/chromium/LayerTilerChromium.cpp: Added. (WebCore::LayerTilerChromium::create): (WebCore::LayerTilerChromium::LayerTilerChromium): (WebCore::LayerTilerChromium::~LayerTilerChromium): (WebCore::LayerTilerChromium::layerRendererContext): (WebCore::LayerTilerChromium::setTileSize): (WebCore::LayerTilerChromium::reset): (WebCore::LayerTilerChromium::createTile): (WebCore::LayerTilerChromium::invalidateTiles): (WebCore::LayerTilerChromium::contentRectToTileIndices): (WebCore::LayerTilerChromium::contentRectToLayerRect): (WebCore::LayerTilerChromium::layerRectToContentRect): (WebCore::LayerTilerChromium::tileIndex): (WebCore::LayerTilerChromium::tileContentRect): (WebCore::LayerTilerChromium::tileLayerRect): (WebCore::LayerTilerChromium::invalidateRect): (WebCore::LayerTilerChromium::invalidateEntireLayer): (WebCore::LayerTilerChromium::update): (WebCore::LayerTilerChromium::setLayerPosition): (WebCore::LayerTilerChromium::draw): (WebCore::LayerTilerChromium::resizeLayer): (WebCore::LayerTilerChromium::growLayerToContain): (WebCore::LayerTilerChromium::Tile::~Tile): (WebCore::LayerTilerChromium::Tile::releaseTextureId):
- platform/graphics/chromium/LayerTilerChromium.h: Added. (WebCore::LayerTilerChromium::Tile::Tile): (WebCore::LayerTilerChromium::Tile::textureId): (WebCore::LayerTilerChromium::Tile::dirty): (WebCore::LayerTilerChromium::Tile::clearDirty): (WebCore::LayerTilerChromium::layerRenderer):
2010-12-20 Adrienne Walker <enne@google.com>
Reviewed by Kenneth Russell.
[chromium] Tile root layer of the compositor.
https://bugs.webkit.org/show_bug.cgi?id=49947
Refactor root layer logic out of WebViewImpl and into
LayerTilerChromium. The painting is now done through an interface
rather than directly in WebViewImpl.
- src/WebViewImpl.cpp: (WebKit::WebViewImpl::scrollRootLayerRect): (WebKit::WebViewImpl::invalidateRootLayerRect): (WebKit::WebViewImpl::setIsAcceleratedCompositingActive): (WebKit::WebViewImplTilePaintInterface::WebViewImplTilePaintInterface): (WebKit::WebViewImplTilePaintInterface::paint): (WebKit::WebViewImplScrollbarPaintInterface::WebViewImplScrollbarPaintInterface): (WebKit::WebViewImplScrollbarPaintInterface::paint): (WebKit::WebViewImpl::doComposite):
- src/WebViewImpl.h:
- 11:06 AM Changeset in webkit [74567] by
-
- 2 edits in trunk/LayoutTests
2010-12-23 Kenneth Russell <kbr@google.com>
Unreviewed Chromium expectations update. Fix incorrect syntax for
fast/css/box-shadow.html suppression and state that
fast/events/touch/touch-target.html may timeout.
- platform/chromium/test_expectations.txt:
- 10:59 AM Changeset in webkit [74566] by
-
- 29 edits1 add in trunk
2010-12-22 Ryosuke Niwa <rniwa@webkit.org>
Reviewed by Eric Seidel.
Editor.h doesn't need to include SelectionController.h
https://bugs.webkit.org/show_bug.cgi?id=51441
Renamed SelectionController::EDirection to SelectionDirection.
- src/WebFrameImpl.cpp: (WebKit::WebFrameImpl::executeCommand):
2010-12-22 Ryosuke Niwa <rniwa@webkit.org>
Reviewed by Eric Seidel.
Editor.h doesn't need to include SelectionController.h
https://bugs.webkit.org/show_bug.cgi?id=51441
Renamed SelectionController::EDirection to SelectionDirection.
- WebCoreSupport/EditorClientQt.cpp: (WebCore::EditorClientQt::handleKeyboardEvent):
2010-12-22 Ryosuke Niwa <rniwa@webkit.org>
Reviewed by Eric Seidel.
Editor.h doesn't need to include SelectionController.h
https://bugs.webkit.org/show_bug.cgi?id=51441
Renamed SelectionController::EDirection to SelectionDirection.
- WebView/WebFrame.mm: (-[WebFrame _rangeByAlteringCurrentSelection:SelectionController::direction:granularity:]): Takes SelectionDirection instead of SelectionController::EDirection.
- WebView/WebFrameInternal.h:
- WebView/WebTextCompletionController.mm: (-[WebTextCompletionController doCompletion]): Calls _rangeByAlteringCurrentSelection:SelectionController.
2010-12-22 Ryosuke Niwa <rniwa@webkit.org>
Reviewed by Eric Seidel.
Editor.h doesn't need to include SelectionController.h
https://bugs.webkit.org/show_bug.cgi?id=51441
Renamed SelectionController::EDirection to SelectionDirection.
- WebCoreSupport/EditorClientHaiku.cpp: (WebCore::EditorClientHaiku::handleKeyboardEvent):
2010-12-22 Ryosuke Niwa <rniwa@webkit.org>
Reviewed by Eric Seidel.
Editor.h doesn't need to include SelectionController.h
https://bugs.webkit.org/show_bug.cgi?id=51441
Renamed SelectionController::EDirection to SelectionDirection.
- WebCoreSupport/EditorClientWinCE.cpp: (WebKit::EditorClientWinCE::handleEditingKeyboardEvent):
2010-12-22 Ryosuke Niwa <rniwa@webkit.org>
Reviewed by Eric Seidel.
Editor.h doesn't need to include SelectionController.h
https://bugs.webkit.org/show_bug.cgi?id=51441
Renamed SelectionController::EDirection to SelectionDirection.
- WebCoreSupport/EditorClientEfl.cpp: (WebCore::EditorClientEfl::handleEditingKeyboardEvent):
2010-12-22 Ryosuke Niwa <rniwa@webkit.org>
Reviewed by Eric Seidel.
Editor.h doesn't need to include SelectionController.h
https://bugs.webkit.org/show_bug.cgi?id=51441
Removed #include <Selection.h> from Editor.h.
Also extracted WritingDirection.h from EditingStyle.h and renamed SelectionController::EDirection
to SelectionDirection and moved its declaration from SelectionController.h to VisibleSelection.h
- WebCore.exp.in: Signature changes.
- WebCore.xcodeproj/project.pbxproj: Added WritingDirection.h
- accessibility/AccessibilityRenderObject.cpp: (WebCore::AccessibilityRenderObject::visiblePositionRangeForLine): Change due to SelectionDirection.
- editing/EditingStyle.h: Removed WritingDirection and included WritingDirection.h
- editing/Editor.cpp: (WebCore::Editor::deleteWithDirection): Change due to SelectionDirection. (WebCore::Editor::markAllMisspellingsAndBadGrammarInRanges): Ditto. (WebCore::Editor::markMisspellingsAfterTypingToWord): Ditto.
- editing/Editor.h: Removed SelectionController.h; added DocumentMarker.h, Timer.h, VisibleSelection.h and WritingDirection.h; forward declared CSSMutableStyleDeclaration.
- editing/EditorCommand.cpp: (WebCore::executeDeleteBackward): Change due to SelectionDirection. (WebCore::executeDeleteBackwardByDecomposingPreviousCharacter): Ditto. (WebCore::executeDeleteForward): Ditto. (WebCore::executeDeleteToBeginningOfLine): Ditto. (WebCore::executeDeleteToBeginningOfParagraph): Ditto. (WebCore::executeDeleteToEndOfLine): Ditto. (WebCore::executeDeleteToEndOfParagraph): Ditto. (WebCore::executeDeleteWordBackward): Ditto. (WebCore::executeDeleteWordForward): Ditto. (WebCore::executeForwardDelete): Ditto. (WebCore::executeMoveBackward): Ditto. (WebCore::executeMoveBackwardAndModifySelection): Ditto. (WebCore::executeMoveDown): Ditto. (WebCore::executeMoveDownAndModifySelection): Ditto. (WebCore::executeMoveForward): Ditto. (WebCore::executeMoveForwardAndModifySelection): Ditto. (WebCore::executeMoveLeft): Ditto. (WebCore::executeMoveLeftAndModifySelection): Ditto. (WebCore::executeMoveRight): Ditto. (WebCore::executeMoveRightAndModifySelection): Ditto. (WebCore::executeMoveToBeginningOfDocument): Ditto. (WebCore::executeMoveToBeginningOfDocumentAndModifySelection): Ditto. (WebCore::executeMoveToBeginningOfLine): Ditto. (WebCore::executeMoveToBeginningOfLineAndModifySelection): Ditto. (WebCore::executeMoveToBeginningOfParagraph): Ditto. (WebCore::executeMoveToBeginningOfParagraphAndModifySelection): Ditto. (WebCore::executeMoveToBeginningOfSentence): Ditto. (WebCore::executeMoveToBeginningOfSentenceAndModifySelection): Ditto. (WebCore::executeMoveToEndOfDocument): Ditto. (WebCore::executeMoveToEndOfDocumentAndModifySelection): Ditto. (WebCore::executeMoveToEndOfSentence): Ditto. (WebCore::executeMoveToEndOfSentenceAndModifySelection): Ditto. (WebCore::executeMoveToEndOfLine): Ditto. (WebCore::executeMoveToEndOfLineAndModifySelection): Ditto. (WebCore::executeMoveToEndOfParagraph): Ditto. (WebCore::executeMoveToEndOfParagraphAndModifySelection): Ditto. (WebCore::executeMoveParagraphBackwardAndModifySelection): Ditto. (WebCore::executeMoveParagraphForwardAndModifySelection): Ditto. (WebCore::executeMoveUp): Ditto. (WebCore::executeMoveUpAndModifySelection): Ditto. (WebCore::executeMoveWordBackward): Ditto. (WebCore::executeMoveWordBackwardAndModifySelection): Ditto. (WebCore::executeMoveWordForward): Ditto. (WebCore::executeMoveWordForwardAndModifySelection): Ditto. (WebCore::executeMoveWordLeft): Ditto. (WebCore::executeMoveWordLeftAndModifySelection): Ditto. (WebCore::executeMoveWordRight): Ditto. (WebCore::executeMoveWordRightAndModifySelection): Ditto. (WebCore::executeMoveToLeftEndOfLine): Ditto. (WebCore::executeMoveToLeftEndOfLineAndModifySelection): Ditto. (WebCore::executeMoveToRightEndOfLine): Ditto. (WebCore::executeMoveToRightEndOfLineAndModifySelection): Ditto.
- editing/SelectionController.cpp: (WebCore::SelectionController::willBeModified): Takes SelectionDirection instead of EDirection. (WebCore::SelectionController::modify): Ditto.
- editing/SelectionController.h: Change due to SelectionDirection.
- editing/TypingCommand.cpp: (WebCore::TypingCommand::deleteKeyPressed): Ditto. (WebCore::TypingCommand::forwardDeleteKeyPressed): Ditto.
- editing/VisibleSelection.h: Added SelectionDirection, renamed and moved from EDirection in SelectionController.h
- editing/WritingDirection.h: Added.
- page/DOMSelection.cpp: (WebCore::DOMSelection::modify): Change due to SelectionDirection. (WebCore::DOMSelection::deleteFromDocument): Ditto.
- page/EventHandler.cpp: (WebCore::EventHandler::handleKeyboardSelectionMovement): Ditto.
- page/Frame.h: Includes SelectionController.h.
- 10:17 AM Changeset in webkit [74565] by
-
- 1 edit13 deletes in trunk/Tools
2010-12-23 Lucas Forschler <Lucas Forschler>
Reviewed by Darin Adler.
Bug 51537 - iexploder needs cleanup
- iExploder/htdocs: Removed.
- iExploder/htdocs/config.rb: Removed.
- iExploder/htdocs/cssproperties.in: Removed.
- iExploder/htdocs/cssvalues.in: Removed.
- iExploder/htdocs/htmlattrs.in: Removed.
- iExploder/htdocs/htmltags.in: Removed.
- iExploder/htdocs/htmlvalues.in: Removed.
- iExploder/htdocs/iexploder.cgi: Removed.
- iExploder/htdocs/iexploder.rb: Removed.
- iExploder/htdocs/index.html: Removed.
- iExploder/htdocs/webserver.rb: Removed.
- iExploder/iexploder-1.7.2/output: Removed.
- iExploder/tools: Removed.
- iExploder/tools/lasthit.rb: Removed.
- iExploder/tools/osx_last_crash.rb: Removed.
- iExploder/tools/showtest.rb: Removed.
- 10:16 AM Changeset in webkit [74564] by
-
- 5 edits in trunk/WebKit2
2010-12-23 Anders Carlsson <andersca@apple.com>
Reviewed by John Sullivan.
Nothing happens clicking "launch live player" on live.cnn.com
https://bugs.webkit.org/show_bug.cgi?id=51540
<rdar://problem/8791086>
Implement NPN_PushPopupsEnabledState/NPN_PopPopupsEnabledState.
- WebProcess/Plugins/Netscape/NetscapeBrowserFuncs.cpp: (WebKit::NPN_PushPopupsEnabledState): Call NetscapePlugin::pushPopupsEnabledState.
(WebKit::NPN_PopPopupsEnabledState):
Call NetscapePlugin::popPopupsEnabledState.
- WebProcess/Plugins/Netscape/NetscapePlugin.cpp: (WebKit::NetscapePlugin::pushPopupsEnabledState): Append the boolean to the vector.
(WebKit::NetscapePlugin::popPopupsEnabledState):
Remove the last boolean from the vector.
(WebKit::NetscapePlugin::allowPopups):
Return the last boolean from the vector.
- WebProcess/Plugins/Netscape/NetscapePlugin.h:
- WebProcess/Plugins/PluginView.cpp: (WebKit::PluginView::performJavaScriptURLRequest): Update the "allowPopupsFromPlugin" state.
- 9:45 AM Changeset in webkit [74563] by
-
- 5 edits in trunk/WebCore
2010-12-23 Adam Barth <abarth@webkit.org>
Reviewed by Darin Adler.
Move V8 to WebCore's implementation of showModalDialog
https://bugs.webkit.org/show_bug.cgi?id=51527
This code is basically verbatim translation from the JavaScriptCore
bindings. The only intentional difference is in the world selection
for the dialog's frame. I suspect JavaScriptCore's bindings have a
subtle bug there.
In this patch, I also remove a bunch of now-unneeded code in the
generic bindings.
- bindings/generic/BindingDOMWindow.h:
- bindings/js/JSDOMWindowCustom.cpp: (WebCore::DialogHandler::dialogCreated): (WebCore::setUpDialog):
- bindings/v8/custom/V8DOMWindowCustom.cpp: (WebCore::DialogHandler::DialogHandler): (WebCore::DialogHandler::dialogCreated): (WebCore::DialogHandler::returnValue): (WebCore::setUpDialog): (WebCore::V8DOMWindow::showModalDialogCallback): (WebCore::V8DOMWindow::openCallback):
- bindings/v8/specialization/V8BindingDOMWindow.h:
- 9:26 AM Changeset in webkit [74562] by
-
- 2 edits in trunk/WebCore
2010-12-23 Alexander Pavlov <apavlov@chromium.org>
Reviewed by Joseph Pecoraro.
Web Inspector: Can't backspace from CSS value to CSS key field
https://bugs.webkit.org/show_bug.cgi?id=51476
- inspector/front-end/StylesSidebarPane.js: (WebInspector.StylePropertyTreeElement.prototype.): (WebInspector.StylePropertyTreeElement.prototype):
- 8:56 AM Changeset in webkit [74561] by
-
- 5 edits in trunk
2010-12-23 W. James MacLean <wjmaclean@chromium.org>
Reviewed by Kenneth Russell.
[chromium] Add asserts to test for contiguous-pixel Skia bitmaps.
https://bugs.webkit.org/show_bug.cgi?id=51186
No new tests. Behaviour not changed, but need the asserts to detect when assumptions violated.
- platform/graphics/chromium/ImageLayerChromium.cpp: (WebCore::ImageLayerChromium::updateContentsIfDirty):
- platform/graphics/skia/GraphicsContext3DSkia.cpp: (WebCore::GraphicsContext3D::getImageData):
2010-12-23 W. James MacLean <wjmaclean@chromium.org>
Reviewed by Kenneth Russell.
[chromium] Add asserts to test for contiguous-pixel Skia bitmaps.
https://bugs.webkit.org/show_bug.cgi?id=51186
Add asserts to detect if assumptions (about contiguous pixels in Skia bitmaps) are violated.
- src/GraphicsContext3DChromium.cpp: (WebCore::GraphicsContext3DInternal::paintRenderingResultsToCanvas):
- 8:53 AM Changeset in webkit [74560] by
-
- 1 edit2 adds in trunk/LayoutTests
Add Windows-XP-specific (failing) expected results for
fast/profiler/throw-exception-from-eval-expected.html in order to get the Windows XP Debug
test bot green again. Unreviewed.
https://bugs.webkit.org/show_bug.cgi?id=49801
- platform/win-xp/fast/profiler: Added.
- platform/win-xp/fast/profiler/throw-exception-from-eval-expected.txt: Added.
- 8:39 AM Changeset in webkit [74559] by
-
- 2 edits in trunk/Tools
2010-12-23 Lucas De Marchi <lucas.demarchi@profusion.mobi>
Reviewed by Kenneth Rohde Christiansen.
[EFL] Make 'single' the default backing store in EWebLauncher
https://bugs.webkit.org/show_bug.cgi?id=51534
'single' is the simplest backing store and the one intended to be the
default in the test browser. By mistake the tiled backing store was
left as default in r72617.
- EWebLauncher/main.c: (browserCreate): ditto.
- 8:23 AM Changeset in webkit [74558] by
-
- 6 edits in trunk/WebKit/qt
2010-12-23 Dawit Alemayehu <adawit@kde.org>
Reviewed by Kenneth Rohde Christiansen.
[Qt] Add a 'hasSelection' function to QWebView and QWebPage.
https://bugs.webkit.org/show_bug.cgi?id=48722
This is a convenience API that optimizes the case where checking for
the presence of selected content and accessing the selected content
are two separate actions in the client. See comment #12 in the above
bug report link for details.
- Api/qwebpage.cpp: (QWebPage::hasSelection):
- Api/qwebpage.h:
- Api/qwebview.cpp: (QWebView::hasSelection):
- Api/qwebview.h:
- tests/qwebpage/tst_qwebpage.cpp: (tst_QWebPage::textSelection):
- 6:45 AM Changeset in webkit [74557] by
-
- 13 edits in trunk/LayoutTests
2010-12-23 Jeremy Moskovich <jeremy@chromium.org>
Unreviewed new Chromium baselines and test expectations update.
- platform/chromium-linux/fast/layers/video-layer-expected.checksum:
- platform/chromium-linux/fast/layers/video-layer-expected.png:
- platform/chromium-mac/fast/layers/video-layer-expected.checksum:
- platform/chromium-mac/fast/layers/video-layer-expected.png:
- platform/chromium-mac/fast/layers/video-layer-expected.txt:
- platform/chromium-win/fast/css/getComputedStyle/computed-style-expected.txt:
- platform/chromium-win/fast/css/getComputedStyle/computed-style-without-renderer-expected.txt:
- platform/chromium-win/fast/layers/video-layer-expected.checksum:
- platform/chromium-win/fast/layers/video-layer-expected.png:
- platform/chromium-win/fast/layers/video-layer-expected.txt:
- platform/chromium-win/svg/css/getComputedStyle-basic-expected.txt:
- platform/chromium/test_expectations.txt:
- 6:41 AM Changeset in webkit [74556] by
-
- 6 edits in trunk
2010-12-23 Lucas De Marchi <lucas.demarchi@profusion.mobi>
Reviewed by Kenneth Rohde Christiansen.
[EFL] Fix ContextMenu removal
https://bugs.webkit.org/show_bug.cgi?id=51530
Fix ContextMenu removal
In r74207 the support for ContextMenu was removed from EFL while the
code is being ported to the new CROSS_PLATFORM_MENUS due to build
breakage. However there were some missing removals in the shutdown
logic that introduced a bug, leading to a segmentation fault when
ContexController was destroyed.
The best way to effectively disable the ContextMenu is to conditionally
compile the meaningful stuff by surrounding with "#if
ENABLE(CONTEXT_MENUS)" and then disabling it in CMake.
- cmake/OptionsEfl.cmake: Disable ContextMenu and don't pretend we implement CROSS_PLATFORM_MENUS yet.
2010-12-23 Lucas De Marchi <lucas.demarchi@profusion.mobi>
Reviewed by Kenneth Rohde Christiansen.
[EFL] Fix ContextMenu removal
https://bugs.webkit.org/show_bug.cgi?id=51530
Fix ContextMenu removal
In r74207 the support for ContextMenu was removed from EFL while the
code is being ported to the new CROSS_PLATFORM_MENUS due to build
breakage. However there were some missing removals in the shutdown
logic that introduced a bug, leading to a segmentation fault when
ContexController was destroyed.
The best way to effectively disable the ContextMenu is to conditionally
compile the meaningful stuff by surrounding with "#if
ENABLE(CONTEXT_MENUS)" and then disabling it in CMake.
- ewk/ewk_contextmenu.cpp: surround need parts with #if ENABLE(CONTEXT_MENUS). (ewk_context_menu_destroy): ditto. (ewk_context_menu_item_select): ditto.
- ewk/ewk_private.h: ditto.
- ewk/ewk_view.cpp: ditto.
- 6:24 AM Changeset in webkit [74555] by
-
- 2 edits in trunk/WebCore
2010-12-23 Lucas De Marchi <lucas.demarchi@profusion.mobi>
Reviewed by Kenneth Rohde Christiansen.
Fix check by undefined macro
https://bugs.webkit.org/show_bug.cgi?id=51531
Fix check by undefined macro
When building EFL port with libsoup, BUILDING_GTK is not defined,
leading to warnings during compile. We actually have to use an #ifdef
instead of an #if.
No change in functionality so no new tests.
- platform/network/soup/cache/webkit/soup-cache.h:
- 6:08 AM Changeset in webkit [74554] by
-
- 3 edits in trunk/WebKit/efl
2010-12-23 Lucas De Marchi <lucas.demarchi@profusion.mobi>
Reviewed by Kenneth Rohde Christiansen.
[EFL] Clean warnings in Tiled Backing Store
https://bugs.webkit.org/show_bug.cgi?id=51529
Tiled backing store in EFL was giving the following warnings that are
gone now:
- Static function defined but not used;
- Comparison between signed and unsigned integer expressions;
- ewk/ewk_tiled_backing_store.c: (_ewk_tiled_backing_store_item_fill): (_ewk_tiled_backing_store_smart_calculate):
- ewk/ewk_tiled_matrix.c: (_ewk_tile_matrix_slicer_setup):
- 5:49 AM Changeset in webkit [74553] by
-
- 8 edits3 adds in trunk
2010-12-23 Jonathan Dixon <joth@chromium.org>
Reviewed by Steve Block.
TouchEvents does not support multi-touch on a page with multiple touch targets
https://bugs.webkit.org/show_bug.cgi?id=51258
- fast/events/touch/multi-touch-grouped-targets-expected.txt: Added.
- fast/events/touch/multi-touch-grouped-targets.html: Added.
- fast/events/touch/script-tests/multi-touch-grouped-targets.js: Added. (declareTouchHandler.return.shouldBe): (declareTouchHandler): (touchEndHandler): Added new test that ensures both test targets receive events when hit by multiple touch points from the same platform event.
- fast/events/touch/script-tests/touch-target-limited.js: (declareTouchStart.return.touchStartHandler): (declareTouchStart): (declareTouchMove.return.touchMoveHandler): (declareTouchMove):
- fast/events/touch/script-tests/touch-target.js: (declareTouchStart.return.touchStartHandler): (declareTouchStart): (declareTouchMove.return.touchMoveHandler): (declareTouchMove):
- fast/events/touch/touch-target-expected.txt:
- fast/events/touch/touch-target-limited-expected.txt: Updated & rebaselined existing tests to reflect new expectations.
2010-12-23 Jonathan Dixon <joth@chromium.org>
Reviewed by Steve Block.
TouchEvents does not support multi-touch on a page with multiple touch targets
https://bugs.webkit.org/show_bug.cgi?id=51258
Test: fast/events/touch/multi-touch-grouped-targets.html
- page/EventHandler.cpp: (WebCore::eventNameForTouchPointState): (WebCore::EventHandler::handleTouchEvent): Redesigned event handler to ensure all event targets involved in a given multitouch event get the appropriate event(s) fired.
- platform/PlatformTouchPoint.h: Added end-stop marker for State enum.
- 5:23 AM Changeset in webkit [74552] by
-
- 2 edits in trunk/Tools
2010-12-23 Lucas De Marchi <lucas.demarchi@profusion.mobi>
Reviewed by Kenneth Rohde Christiansen.
[EFL] Subject: [webkit 4/4] [EFL] Remove non-sense workaround
https://bugs.webkit.org/show_bug.cgi?id=51533
There's no point in concatenating env("HOME") with "blah", so remove
option from EWebLauncher.
- EWebLauncher/main.c: (main): ditto.
- 5:14 AM Changeset in webkit [74551] by
-
- 2 edits in trunk/WebCore
2010-12-23 Philippe Normand <pnormand@igalia.com>
Reviewed by Xan Lopez.
[GTK] media/audio-data-url.html fails since r75504
https://bugs.webkit.org/show_bug.cgi?id=51525
Make the GStreamer media-player advertize 3gpp mime-types as
supported if the 3gp caps are in the typefind factories.
- platform/graphics/gstreamer/MediaPlayerPrivateGStreamer.cpp: (WebCore::mimeTypeCache):
- 5:09 AM Changeset in webkit [74550] by
-
- 2 edits in trunk/LayoutTests
2010-12-23 Jeremy Moskovich <jeremy@chromium.org>
Unreviewed test expectations update.
Fix duplicate expectation warning for media/video-controls-rendering.html introduced in r74535.
- platform/chromium/test_expectations.txt:
- 4:56 AM Changeset in webkit [74549] by
-
- 31 edits in trunk/WebCore
Refactor HTMLInputElement: Move a part of HTMLInputElement::defaultEventHandler()
to InputTypes.
https://bugs.webkit.org/show_bug.cgi?id=51093
Reviewed by Dimitri Glazkov.
Move BeforeTextInserted and Wheel event handling, event forwarding to
renderers, and the isindex form submission quirk code to InputTypes.
Also, change the return types of InputType event handlers from 'bool'
to 'void'. We use Event::defaultHandled() to decide whether event
handling should be stopped.
No new tests because this is just a refactoring.
- html/BaseButtonInputType.cpp: bool->void changes
(WebCore::BaseButtonInputType::handleKeydownEvent):
(WebCore::BaseButtonInputType::handleKeypressEvent):
(WebCore::BaseButtonInputType::handleKeyupEvent):
- html/BaseButtonInputType.h: ditto.
- html/BaseCheckableInputType.cpp: ditto.
(WebCore::BaseCheckableInputType::handleKeydownEvent):
(WebCore::BaseCheckableInputType::handleKeypressEvent):
- html/BaseCheckableInputType.h: ditto.
- html/BaseDateAndTimeInputType.cpp:
(WebCore::BaseDateAndTimeInputType::handleKeydownEvent): ditto.
(WebCore::BaseDateAndTimeInputType::handleWheelEvent): Just calls handleWheelEventForSpinButton().
- html/BaseDateAndTimeInputType.h: bool->void change and handleWhellEvent() addition.
- html/CheckboxInputType.cpp: bool->void change
(WebCore::CheckboxInputType::handleKeyupEvent):
- html/CheckboxInputType.h: ditto.
- html/FileInputType.cpp: ditto.
(WebCore::FileInputType::handleDOMActivateEvent):
- html/FileInputType.h: ditto.
- html/HTMLInputElement.cpp:
(WebCore::HTMLInputElement::defaultEventHandler):
- Change return-value handling of InputType event handlers
- Move out beforeTextInserted/wheel event handling and isindex form quirk.
- html/HTMLInputElement.h:
(WebCore::HTMLInputElement::handleBeforeTextInsertedEvent):
Added so that InputType can call InputElement::handleBeforeTextInsertedEvent().
- html/ImageInputType.cpp: bool->void change
(WebCore::ImageInputType::handleDOMActivateEvent):
- html/ImageInputType.h: ditto.
- html/InputType.cpp: ditto.
(WebCore::InputType::handleClickEvent):
(WebCore::InputType::handleDOMActivateEvent):
(WebCore::InputType::handleKeydownEvent):
(WebCore::InputType::handleKeypressEvent):
(WebCore::InputType::handleKeyupEvent):
(WebCore::InputType::handleBeforeTextInsertedEvent):
(WebCore::InputType::handleWheelEvent): Added default implementation.
(WebCore::InputType::forwardEvent): ditto.
(WebCore::InputType::formForSubmission): ditto.
- html/InputType.h:
- html/IsIndexInputType.cpp:
(WebCore::IsIndexInputType::formForSubmission): Moved from HTMLInputElement.cpp.
- html/IsIndexInputType.h:
- html/NumberInputType.cpp:
(WebCore::isNumberCharacter): Moved from HTMLInputElement.cpp.
(WebCore::NumberInputType::handleKeydownEvent): bool->void change.
(WebCore::NumberInputType::handleBeforeTextInsertedEvent): Moved from HTMInputElement.cpp.
(WebCore::NumberInputType::handleWheelEvent): Added. Just calls handleWheelEventForSpinButton().
- html/NumberInputType.h:
- html/RadioInputType.cpp: bool->void changes.
(WebCore::RadioInputType::handleClickEvent):
(WebCore::RadioInputType::handleKeydownEvent):
(WebCore::RadioInputType::handleKeyupEvent):
- html/RadioInputType.h:
- html/RangeInputType.cpp:
(WebCore::RangeInputType::handleKeydownEvent): ditto.
(WebCore::RangeInputType::forwardEvent): Moved from HTMLInputElement.cpp.
- html/RangeInputType.h:
- html/ResetInputType.cpp: bool->void change.
(WebCore::ResetInputType::handleDOMActivateEvent):
- html/ResetInputType.h: ditto.
- html/SubmitInputType.cpp: ditto.
(WebCore::SubmitInputType::handleDOMActivateEvent):
- html/SubmitInputType.h: ditto.
- html/TextFieldInputType.cpp:
(WebCore::TextFieldInputType::handleKeydownEvent): bool->void change.
(WebCore::TextFieldInputType::handleKeydownEventForSpinButton): ditto.
(WebCore::TextFieldInputType::handleWheelEventForSpinButton): Moved from HTMLInputElement.cpp.
(WebCore::TextFieldInputType::forwardEvent): ditto.
- html/TextFieldInputType.h:
- 3:12 AM Changeset in webkit [74548] by
-
- 2 edits in trunk/WebCore
2010-12-23 Alexander Pavlov <apavlov@chromium.org>
Reviewed by Joseph Pecoraro.
Web Inspector: After scrolling new property value and cancelling, removed property remains active in page style
https://bugs.webkit.org/show_bug.cgi?id=51390
- inspector/front-end/StylesSidebarPane.js: (WebInspector.StylePropertyTreeElement.prototype):
- 2:38 AM Changeset in webkit [74547] by
-
- 2 edits in trunk/LayoutTests
Unreviewed. Update Qt specific expected file after r74539.
- platform/qt/fast/dom/Window/window-properties-expected.txt:
- 2:20 AM Changeset in webkit [74546] by
-
- 1 edit3 adds in trunk/LayoutTests
2010-12-23 Benjamin Kalman <kalman@chromium.org>
Reviewed by Eric Seidel.
QT linux has RTL line movement bug that causes editing/selection/extend-to-line-boundary.html to fail
https://bugs.webkit.org/show_bug.cgi?id=51514
Add different baseline to editing/selection/extend-to-line-boundary.html for qt-linux that
takes into account the RTL bug.
- platform/qt-linux/editing/selection/extend-to-line-boundary-expected.txt: Added.
- 1:54 AM Changeset in webkit [74545] by
-
- 2 edits in trunk/LayoutTests
Not reviewed.
Another attempt.
- http/tests/local/slow-css-pass.cgi:
- 1:20 AM Changeset in webkit [74544] by
-
- 2 edits in trunk/Tools
2010-12-23 Ilya Tikhonovsky <loislo@chromium.org>
Reviewed by Eric Seidel.
Web Inspector: chromium: inspector's tests are quite slow especially at windows(Debug).
Almost all inspector's tests become flaky on chromium Win(dbg) after switching to DRT.
I just discovered that we have two additional GC runs per each WebViewHost.
Test_shell had no such calls.
These gc runs eat 15% of cpu time.
- DumpRenderTree/chromium/WebViewHost.cpp: (WebViewHost::~WebViewHost):
- 1:18 AM Changeset in webkit [74543] by
-
- 2 edits in trunk/LayoutTests
Not reviewed.
Flaky Test: http/tests/local/stylesheet-and-script-load-order-media-print.html
Try to fix flakiness.
- http/tests/local/slow-css-pass.cgi:
- 12:17 AM Changeset in webkit [74542] by
-
- 3 edits in trunk/WebKit/efl
2010-12-23 Gyuyoung Kim <gyuyoung.kim@samsung.com>
Unreviewed build warning fix.
[EFL] Remove warning messages during the webkit efl compiling
https://bugs.webkit.org/show_bug.cgi?id=51520
Remove build warning messages.
- ewk/ewk_view.cpp: (_ewk_view_smart_show): (_ewk_view_smart_hide):
- ewk/ewk_view.h:
- 12:00 AM Changeset in webkit [74541] by
-
- 2 edits in trunk/WebCore
2010-12-23 Noel Gordon <noel.gordon@gmail.com>
Reviewed by Adam Barth.
[chromium] Reduce canvas.toDataURL("image/png") run-time costs 300%
https://bugs.webkit.org/show_bug.cgi?id=51455
Chromium uses the libpng compression defaults. Good compression results,
but at much larger run-time cost compared to other browsers. Chrome is 2
times slower than Mozilla 3.6.12 and 3-4 times slower than Safari 5.0.3.
libpng has supported alternative compression methods for some time, they
better meet the dual goals of good compression and run-time cost for the
on-line case, and are well-documented. Here use Huffman encoding. It's
2-20% faster than other encodings (Z_RLE, Z_FILTERED) and the compressed
size is within 1% of the Safari 5.0.3 result independent of image size.
No change in behaviour, so no new tests.
- platform/image-encoders/skia/PNGImageEncoder.cpp: (WebCore::PNGImageEncoder::encode): Apply Huffman image encoding.