Timeline



Oct 24, 2009:

9:56 PM Changeset in webkit [50039] by timothy@apple.com
  • 5 edits in trunk/WebCore

WebInspector: Fixed issue with IME inside console
https://bugs.webkit.org/show_bug.cgi?id=30660

Patch by Keishi Hattori <casey.hattori@gmail.com> on 2009-10-24
Reviewed by Timothy Hatcher.

  • inspector/front-end/ConsoleView.js:

(WebInspector.ConsoleView.prototype._promptKeyDown):

  • inspector/front-end/DatabaseQueryView.js:

(WebInspector.DatabaseQueryView.prototype._promptKeyDown):

  • inspector/front-end/inspector.js:

(WebInspector.loaded):
(WebInspector.searchKeyDown): Moved performSearch here from WebInspector.searchKeyUp.
(WebInspector.startEditing.element.handleKeyEvent):
(WebInspector.startEditing):

  • inspector/front-end/utilities.js:

(isEnterKey): Added. Check if in IME.

9:13 PM Changeset in webkit [50038] by bweinstein@apple.com
  • 3 edits in trunk/WebCore

Fixes <https://bugs.webkit.org/show_bug.cgi?id=30752>.
Web Inspector: Multiple Selection on Scope Bars by default Conflicts with other behavior on OSX.

Reviewed by Timothy Hatcher.

Have the scope bars select one scope by default, but if the multiple selection key
is pressed, allow for multiple selection.

  • inspector/front-end/ConsoleView.js:

(WebInspector.ConsoleView):
(WebInspector.ConsoleView.prototype._updateFilter):
(WebInspector.ConsoleView.prototype.filter):

  • inspector/front-end/ResourcesPanel.js:

(WebInspector.ResourcesPanel):
(WebInspector.ResourcesPanel.prototype.filter):
(WebInspector.ResourcesPanel.prototype._updateFilter):

7:44 PM Changeset in webkit [50037] by weinig@apple.com
  • 18 edits
    1 add in trunk

WebCore: Fix for https://bugs.webkit.org/show_bug.cgi?id=30751
HTMLOptionsCollection should not inherit from HTMLCollection

Reviewed by Dan Bernstein.

  • GNUmakefile.am:
  • WebCore.gypi:
  • WebCore.pro:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:

Added JSHTMLCollectionFunctions.h

  • bindings/js/JSHTMLAllCollectionCustom.cpp:

(WebCore::callHTMLAllCollection):
(WebCore::JSHTMLAllCollection::nameGetter):
(WebCore::JSHTMLAllCollection::item):
(WebCore::JSHTMLAllCollection::namedItem):

  • bindings/js/JSHTMLCollectionCustom.cpp:

(WebCore::callHTMLCollection):
(WebCore::JSHTMLCollection::nameGetter):
(WebCore::JSHTMLCollection::item):
(WebCore::JSHTMLCollection::namedItem):

  • bindings/js/JSHTMLOptionsCollectionCustom.cpp:

(WebCore::callHTMLCollection):
(WebCore::JSHTMLOptionsCollection::getCallData):
(WebCore::JSHTMLOptionsCollection::canGetItemsForName):
(WebCore::JSHTMLOptionsCollection::nameGetter):
(WebCore::JSHTMLOptionsCollection::indexSetter):
(WebCore::JSHTMLOptionsCollection::item):
(WebCore::JSHTMLOptionsCollection::namedItem):
Refactor to use generic implementation of HTMLCollection bindings functions
from JSHTMLCollectionFunctions.h

  • bindings/js/JSHTMLCollectionFunctions.h: Added.

(WebCore::getCollectionNamedItems):
(WebCore::getCollectionItems):
(WebCore::callHTMLCollectionGeneric):
Added generic functions that JSHTMLCollection, JSHTMLAllCollection and
JSHTMLOptionsCollection can all use.

  • bindings/js/JSHTMLDocumentCustom.cpp: Add include of JSHTMLAllCollection.h

for toJS on document.all.

  • bindings/objc/DOMHTML.mm:
  • bindings/objc/DOMUtility.mm:

(JSC::createDOMWrapper):
Remove special cases for HTMLOptionsCollection.

  • bindings/scripts/CodeGeneratorJS.pm:
  • html/HTMLAllCollection.idl:
  • html/HTMLCollection.idl:
  • html/HTMLOptionsCollection.idl:

LayoutTests: Update tests for https://bugs.webkit.org/show_bug.cgi?id=30751
HTMLOptionsCollection should not inherit from HTMLCollection

Reviewed by Dan Bernstein.

  • fast/dom/wrapper-classes-expected.txt:
4:08 PM Changeset in webkit [50036] by timothy@apple.com
  • 4 edits in trunk/WebCore

Make the scope bars in the Web Inspector match other scope bars in Mac OS X.
Also cleans up the CSS to use classes instead of ID selectors.

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

Reviewed by Dan Bernstein.

  • inspector/front-end/ResourcesPanel.js: Add the scope-bar class.
  • inspector/front-end/inspector.css: Lots of changes to make it look right.
  • inspector/front-end/inspector.html: Add the scope-bar class.
4:08 PM Changeset in webkit [50035] by timothy@apple.com
  • 2 edits in trunk/WebCore

Fix selecting Resources in the Web Inspector's Resources panel.

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

Reviewed by Pavel Feldman.

  • inspector/front-end/ResourcesPanel.js:

(WebInspector.ResourcesPanel.prototype.isCategoryVisible): Added.
(WebInspector.ResourceSidebarTreeElement.prototype.get selectable): Call
isCategoryVisible on the ResourcesPanel.

4:08 PM Changeset in webkit [50034] by timothy@apple.com
  • 2 edits in trunk/WebCore

Fix the Scope Bar in the Web Inspector's Resource panel, so that selecting
All will deselect the other filters.

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

Reviewed by Pavel Feldman.

  • inspector/front-end/ResourcesPanel.js:

(WebInspector.ResourcesPanel.createFilterElement): Better syntax.
(WebInspector.ResourcesPanel):
(WebInspector.ResourcesPanel.prototype.filter): Use a normal for loop,
and the child variable instead of target in the loop. Other clean up.

4:08 PM Changeset in webkit [50033] by timothy@apple.com
  • 6 edits in trunk/WebCore

Make resizing the Web Inspector's Elements panel sidebar work again.
Also makes Resources panel use the sidebar code in Panel.

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

Reviewed by Pavel Feldman.

  • inspector/front-end/Panel.js:

(WebInspector.Panel): Don't create the sidebar with an argument.
(WebInspector.Panel.prototype.show): Call updateSidebarWidth.
(WebInspector.Panel.prototype.handleKeyEvent): Call handleSidebarKeyEvent.
(WebInspector.Panel.prototype.handleSidebarKeyEvent): Added.
(WebInspector.Panel.prototype.createSidebar): Renamed from _createSidebar.
(WebInspector.Panel.prototype._sidebarDragging): Call updateSidebarWidth.
(WebInspector.Panel.prototype.updateSidebarWidth): Renamed from _updateSidebarWidth.
(WebInspector.Panel.prototype.setSidebarWidth): Added. Called by updateSidebarWidth.
(WebInspector.Panel.prototype.updateMainViewWidth): Renamed from setMainViewWidth.

  • inspector/front-end/ProfilesPanel.js:

(WebInspector.ProfilesPanel): Call createSidebar.
(WebInspector.ProfilesPanel.prototype.updateMainViewWidth): Renamed from setMainViewWidth.

  • inspector/front-end/ResourcesPanel.js:

(WebInspector.ResourcesPanel.prototype.show): Remove call to _updateSidebarWidth.
(WebInspector.ResourcesPanel.prototype.showResource): Call updateSidebarWidth
(WebInspector.ResourcesPanel.prototype.closeVisibleResource): Ditto.
(WebInspector.ResourcesPanel.prototype.setSidebarWidth): Added.
(WebInspector.ResourcesPanel.prototype.updateMainViewWidth): Added.

  • inspector/front-end/StoragePanel.js:

(WebInspector.StoragePanel): Call createSidebar.
(WebInspector.StoragePanel.prototype.updateMainViewWidth): Renamed from setMainViewWidth.

  • inspector/front-end/TimelinePanel.js:

(WebInspector.TimelinePanel): Call createSidebar.
(WebInspector.TimelinePanel.prototype.updateMainViewWidth): Renamed from setMainViewWidth.

4:08 PM Changeset in webkit [50032] by timothy@apple.com
  • 3 edits in trunk/WebCore

Call updateSelection() in a few places so hovering in the Web Inspector's
Elements panel always has the correct row selection height.

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

Reviewed by Pavel Feldman.

  • inspector/front-end/ElementsTreeOutline.js:

(WebInspector.ElementsTreeElement.prototype.toggleNewAttributeButton): Call updateSelection().
(WebInspector.ElementsTreeElement.prototype._insertInLastAttributePosition): Ditto.

  • inspector/front-end/inspector.css: Prevent text wrapping in the middle of the add attribute text.
4:02 PM Changeset in webkit [50031] by eric@webkit.org
  • 2 edits in trunk/WebCore

2009-10-24 Timothy Hatcher <timothy@apple.com>

Remove the re-root tree feature from the Web Inspector's
Elements panel. This feature wasn't super useful and gets
in the way of double-click to edit.

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

Reviewed by Pavel Feldman.

  • inspector/front-end/ElementsTreeOutline.js: (WebInspector.ElementsTreeElement.prototype.ondblclick):
3:40 PM Changeset in webkit [50030] by eric@webkit.org
  • 3 edits in trunk/WebCore

2009-10-24 Anton Muhin <antonm@chromium.org>

Reviewed by Adam Barth.

Implements ReturnsNew hint for V8 bindings generation code.
https://bugs.webkit.org/show_bug.cgi?id=30745

  • bindings/scripts/CodeGeneratorV8.pm:
  • bindings/v8/V8DOMWrapper.h: (WebCore::V8DOMWrapper::convertNewNodeToV8Object):
3:16 PM Changeset in webkit [50029] by mitz@apple.com
  • 3 edits
    1 copy
    4 adds in trunk

Missing images at 350.org due to wrong URLs
https://bugs.webkit.org/show_bug.cgi?id=30748

Reviewed by Sam Weinig.

WebCore:

Test: fast/dom/HTMLImageElement/parse-src.html

  • dom/Element.cpp:

(WebCore::Element::getURLAttribute): Pass the attribute value through deprecateParseURL().
This matches what the Objective-C bindings do, as well as what HTMLImageLoader::sourceURI()
and SVGImageLoader::sourceURI() do.

LayoutTests:

  • fast/dom/HTMLImageElement/parse-src-expected.txt: Added.
  • fast/dom/HTMLImageElement/parse-src.html: Added.
  • fast/dom/HTMLImageElement/script-tests: Added.
  • fast/dom/HTMLImageElement/script-tests/TEMPLATE.html: Copied from LayoutTests/fast/dom/Document/script-tests/TEMPLATE.html.
  • fast/dom/HTMLImageElement/script-tests/parse-src.js: Added.
1:55 PM Changeset in webkit [50028] by Laszlo Gombos
  • 2 edits in trunk/WebCore

2009-10-24 Laszlo Gombos <Laszlo Gombos>

Unreviewed.

[Qt] Build fix for QtWebKit after r50013.

  • WebCore.pro:
9:12 AM Changeset in webkit [50027] by Dimitri Glazkov
  • 5 edits
    1 add in trunk/WebCore

2009-10-24 Dimitri Glazkov <Dimitri Glazkov>

Unreviewed, build fix.

[V8] Fix up to accommodate for HTMLAllCollection changes in
http://trac.webkit.org/changeset/49998.

  • WebCore.gypi:
  • bindings/scripts/CodeGeneratorV8.pm:
  • bindings/v8/custom/V8CustomBinding.h:
  • bindings/v8/custom/V8HTMLDocumentCustom.cpp:
8:28 AM Changeset in webkit [50026] by eric@webkit.org
  • 17 edits in trunk

2009-10-24 Laszlo Gombos <Laszlo Gombos>

Reviewed by Holger Freyther.

[Qt] [Symbian] Set the capability and memory required to run QtWebKit for Symbian
https://bugs.webkit.org/show_bug.cgi?id=30476

Set the stack size to 80 Kb and heap size to the 128kB - 32MB range
to all executables linking against WebKit library.

  • WebKit.pri:

2009-10-24 Laszlo Gombos <Laszlo Gombos>

Reviewed by Holger Freyther.

[Qt] [Symbian] Set the capability and memory required to run QtWebKit for Symbian
https://bugs.webkit.org/show_bug.cgi?id=30476

Assign ReadUserData WriteUserData NetworkServices Symbian capabilities
to jsc.exe.

  • jsc.pro:

2009-10-24 Laszlo Gombos <Laszlo Gombos>

Reviewed by Holger Freyther.

[Qt] [Symbian] Set the capability and memory required to run QtWebKit for Symbian
https://bugs.webkit.org/show_bug.cgi?id=30476

Assign ReadUserData WriteUserData NetworkServices Symbian capabilities
to all QtWebkit executables.

  • QGVLauncher/QGVLauncher.pro:
  • QtLauncher/QtLauncher.pro:
  • tests/benchmarks/loading/tst_loading.pro:
  • tests/benchmarks/painting/tst_painting.pro:
  • tests/qgraphicswebview/qgraphicswebview.pro:
  • tests/qwebelement/qwebelement.pro:
  • tests/qwebframe/qwebframe.pro:
  • tests/qwebhistory/qwebhistory.pro:
  • tests/qwebhistoryinterface/qwebhistoryinterface.pro:
  • tests/qwebpage/qwebpage.pro:
  • tests/qwebplugindatabase/qwebplugindatabase.pro:
  • tests/qwebview/qwebview.pro:
5:56 AM QtWebKitPerformanceToolBackLog: edited by zecke@selfish.org
(diff)
5:48 AM QtWebKitPerformanceToolBackLog: edited by zecke@selfish.org
(diff)
5:23 AM QtBackLog edited by zecke@selfish.org
(diff)
5:21 AM QtBackLog edited by zecke@selfish.org
(diff)

Oct 23, 2009:

11:55 PM QtBackLog edited by zecke@selfish.org
(diff)
11:55 PM QtBackLog edited by zecke@selfish.org
(diff)
11:41 PM QtBackLog edited by zecke@selfish.org
(diff)
9:42 PM QtWebKitPerformanceUtilities edited by zecke@selfish.org
(diff)
9:12 PM Changeset in webkit [50025] by mitz@apple.com
  • 4 edits in trunk/WebCore

<rdar://problem/7294131> Assertion failure in AuthenticationMac at
mac(const Credential&) after authenticating to MobileMe Gallery movie

Reviewed by Alexey Proskuryakov.

  • platform/network/Credential.cpp:

(WebCore::Credential::Credential): Initialize m_persistence.
(WebCore::Credential::isEmpty): Made this method const.

  • platform/network/Credential.h:
  • platform/network/mac/AuthenticationMac.mm:

(WebCore::mac): Return nil if the Credential is empty.

8:16 PM Changeset in webkit [50024] by mrowe@apple.com
  • 2 edits in trunk/WebKit/mac

Fix engineering builds in the mysterious future.

  • Configurations/DebugRelease.xcconfig:
8:06 PM Changeset in webkit [50023] by mrowe@apple.com
  • 4 edits in branches/safari-4-branch

Versioning.

8:05 PM Changeset in webkit [50022] by mrowe@apple.com
  • 1 copy in tags/Safari-6531.21.5

New tag.

8:05 PM Changeset in webkit [50021] by bweinstein@apple.com
  • 3 edits in trunk/WebCore

Fixes <https://bugs.webkit.org/show_bug.cgi?id=30617>.
Web Inspector: Resources Scope Bar Should Allow Combinations.

Reviewed by Timothy Hatcher.

Refactor the filtering in ResourcePanel to make it match ConsoleView,
and add a divider between All, and the other choices of Resource types.

  • inspector/front-end/ResourcesPanel.js:

(WebInspector.ResourcesPanel):
(WebInspector.ResourcesPanel.prototype.filter):
(WebInspector.ResourcesPanel.prototype._updateFilter):

  • inspector/front-end/inspector.css:
8:03 PM Changeset in webkit [50020] by mrowe@apple.com
  • 2 edits in branches/safari-4-branch/WebKit/win

Merge r50019.

8:01 PM Changeset in webkit [50019] by mrowe@apple.com
  • 2 edits in trunk/WebKit/win

Build fix. Include config.h before including other files.

  • AccessibleImage.cpp:
7:06 PM Changeset in webkit [50018] by eric@webkit.org
  • 3 edits
    3 adds in trunk

2009-10-23 Chris Evans <cevans@chromium.org>

Reviewed by Adam Barth.

Added test for bug 27239 (ignore Refresh for view source mode).
https://bugs.webkit.org/show_bug.cgi?id=27239

  • http/tests/security/view-source-no-refresh.html: Added
  • http/tests/security/view-source-no-refresh-expected.txt: Added
  • http/tests/security/resources/view-source-no-refresh.php: Added

2009-10-23 Chris Evans <cevans@chromium.org>

Reviewed by Adam Barth.

Ignore the Refresh header if we're in view source mode.

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

Test: http/tests/security/view-source-no-refresh.html

  • loader/FrameLoader.cpp: ignore Refresh in view-source mode.
6:10 PM Changeset in webkit [50017] by ap@apple.com
  • 5 edits in trunk/WebCore

Reviewed by Sam Weinig.

https://bugs.webkit.org/show_bug.cgi?id=30734
Remove XMLTokenizer::m_currentNodeIsReferenced

No change in behavior, no tests.

  • dom/XMLTokenizer.cpp: (WebCore::XMLTokenizer::setCurrentNode):
  • dom/XMLTokenizer.h: Don't store a boolean for the current node being the document - we can always check for that.
  • dom/XMLTokenizerLibxml2.cpp: (WebCore::XMLTokenizer::XMLTokenizer):
  • dom/XMLTokenizerQt.cpp: (WebCore::XMLTokenizer::XMLTokenizer): A fragment passed to XMLTokenizer is never null - in fact, the first thing we do is dereference it unconditionally, and then begin checking it for null, as if it could help.
6:09 PM Changeset in webkit [50016] by Beth Dakin
  • 3 edits in trunk/WebCore

No review since this is a build fix.

Build fix for the Mac. (Build is only broken when SVG filters are
enabled.)

Labelled these function static.

  • platform/graphics/filters/FEComponentTransfer.cpp:

(WebCore::identity):
(WebCore::table):
(WebCore::discrete):
(WebCore::linear):
(WebCore::gamma):

  • platform/graphics/filters/FEGaussianBlur.cpp:

(WebCore::boxBlur):

5:40 PM Changeset in webkit [50015] by pfeldman@chromium.org
  • 9 edits
    2 adds in trunk/WebCore

2009-10-23 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Timothy Hatcher.

Web Inspector: Implement a Timeline panel.

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

  • WebCore.gypi:
  • WebCore.vcproj/WebCore.vcproj:
  • inspector/front-end/Images/timelineIcon.png: Added.
  • inspector/front-end/TimelineAgent.js: (WebInspector.addItemToTimeline): (WebInspector.timelineProfilerWasStarted): (WebInspector.timelineProfilerWasStopped):
  • inspector/front-end/TimelinePanel.js: Added. (WebInspector.TimelinePanel): (WebInspector.TimelinePanel.prototype.toolbarItemClass.get toolbarItemLabel): (WebInspector.TimelinePanel.prototype.get statusBarItems): (WebInspector.TimelinePanel.prototype.handleKeyEvent): (WebInspector.TimelinePanel.prototype.timelineWasStarted): (WebInspector.TimelinePanel.prototype.timelineWasStopped): (WebInspector.TimelinePanel.prototype.addItemToTimeline): (WebInspector.TimelinePanel.prototype._innerAddItemToTimeline): (WebInspector.TimelinePanel.prototype._toggleTimelineButton): (WebInspector.TimelinePanel.prototype.setMainViewWidth): (WebInspector.TimelinePanel.prototype.getItemTypeName): (WebInspector.TimelineRecordTreeElement): (WebInspector.TimelineRecordTreeElement.prototype.onattach):
  • inspector/front-end/WebKit.qrc:
  • inspector/front-end/inspector.css:
  • inspector/front-end/inspector.html:
  • inspector/front-end/inspector.js: (WebInspector._createPanels):
5:26 PM Changeset in webkit [50014] by Joseph Pecoraro
  • 2 edits in trunk/WebCore

2009-10-23 Joseph Pecoraro <Joseph Pecoraro>

Reviewed by Timothy Hatcher.

Web Inspector: Clash Between Search's onkeyup and incremental search events
https://bugs.webkit.org/show_bug.cgi?id=30731

  • inspector/front-end/inspector.js: (WebInspector.performSearch): prevent incremental search event from clobbering a forced search
5:24 PM Changeset in webkit [50013] by pfeldman@chromium.org
  • 11 edits
    2 adds in trunk/WebCore

2009-10-23 Kelly Norton <knorton@google.com>

Reviewed by Pavel Feldman.

Removes the need for C++ Timeline types in InspectorTimelineAgent in favor
of ScriptObjects.
https://bugs.webkit.org/show_bug.cgi?id=30707

  • GNUmakefile.am:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • inspector/DOMDispatchTimelineItem.cpp: Removed.
  • inspector/DOMDispatchTimelineItem.h: Removed.
  • inspector/InspectorTimelineAgent.cpp: (WebCore::InspectorTimelineAgent::InspectorTimelineAgent): (WebCore::InspectorTimelineAgent::willDispatchDOMEvent): (WebCore::InspectorTimelineAgent::didDispatchDOMEvent): (WebCore::InspectorTimelineAgent::willLayout): (WebCore::InspectorTimelineAgent::didLayout): (WebCore::InspectorTimelineAgent::willRecalculateStyle): (WebCore::InspectorTimelineAgent::didRecalculateStyle): (WebCore::InspectorTimelineAgent::willPaint): (WebCore::InspectorTimelineAgent::didPaint): (WebCore::InspectorTimelineAgent::willWriteHTML): (WebCore::InspectorTimelineAgent::didWriteHTML): (WebCore::InspectorTimelineAgent::reset): (WebCore::InspectorTimelineAgent::didCompleteCurrentRecord): (WebCore::InspectorTimelineAgent::pushCurrentTimelineItem):
  • inspector/InspectorTimelineAgent.h: (WebCore::InspectorTimelineAgent::TimelineItemEntry::TimelineItemEntry):
  • inspector/TimelineItem.cpp: Removed.
  • inspector/TimelineItem.h: Removed.
  • inspector/TimelineItemFactory.cpp: Added. (WebCore::TimelineItemFactory::createGenericTimelineItem): (WebCore::TimelineItemFactory::createDOMDispatchTimelineItem):
  • inspector/TimelineItemFactory.h: Added. (WebCore::): (WebCore::TimelineItemFactory::TimelineItemFactory):
5:13 PM Changeset in webkit [50012] by barraclough@apple.com
  • 10 edits in trunk

https://bugs.webkit.org/show_bug.cgi?id=30729
Provide private API to request the global context for a specific world.
Used to initialize parameters on the global object.

Patch by Gavin Barraclough <barraclough@apple.com> on 2009-10-23
Reviewed by Geoff Garen.

WebCore:

  • bindings/js/ScriptController.cpp:

(WebCore::ScriptController::globalObject):

  • bindings/js/ScriptController.h:

WebKit/mac:

  • WebView/WebFramePrivate.h:
  • WebView/WebFrame.mm:

(-[WebFrame contextForWorldID:]):

WebKit/win:

  • WebFrame.cpp:

(WebFrame::contextForWorldID):

  • WebFrame.h:
4:44 PM Changeset in webkit [50011] by mrowe@apple.com
  • 4 edits in branches/safari-4-branch

Versioning.

4:44 PM Changeset in webkit [50010] by mrowe@apple.com
  • 1 copy in tags/Safari-6531.21.4

New tag.

4:40 PM Changeset in webkit [50009] by mrowe@apple.com
  • 4 edits
    4 adds in branches/safari-4-branch

Merge r50004.

4:40 PM Changeset in webkit [50008] by mrowe@apple.com
  • 2 edits in branches/safari-4-branch/WebCore

Merge r47534.

4:35 PM Changeset in webkit [50007] by bweinstein@apple.com
  • 2 edits in trunk/WebKit/win

Pan-scrolling pointer shows up in other tabs or windows, can't "fix" it.
<http://webkit.org/b/30732> and <rdar://7332018>.

Reviewed by John Sullivan.

Turns off pan-scrolling on a WebView when it is about to lose focus.

  • WebView.cpp:

(WebViewWndProc):

3:40 PM Changeset in webkit [50006] by eric@webkit.org
  • 9 edits in trunk/WebCore

2009-10-23 Steve Block <steveblock@google.com>

Reviewed by Eric Seidel.

Fixes style problems in Android-specific files.
https://bugs.webkit.org/show_bug.cgi?id=30717

Style changes only, no new tests possible.

  • page/android/EventHandlerAndroid.cpp: Modified. (WebCore::EventHandler::passMouseDownEventToWidget): Fixes whitespace in method signature. (WebCore::EventHandler::passSubframeEventToSubframe): Fixes whitespace in method signature. (WebCore::EventHandler::passWheelEventToWidget): Fixes whitespace in method signature. (WebCore::EventHandler::passMouseMoveEventToSubframe): Fixes whitespace in method signature.
  • platform/android/ClipboardAndroid.h: Modified. Fixes header include order and indenting.
  • platform/android/CursorAndroid.cpp: Modified. (WebCore::grabCursor): Fixes function braces. (WebCore::grabbingCursor): Fixes function braces.
  • platform/android/FileChooserAndroid.cpp: Modified. (WebCore::FileChooser::basenameForWidth): Fixes braces on while statement, and updates String::copy to String::threadsafeCopy.
  • platform/android/FileSystemAndroid.cpp: Modified. Fixes header include order. (WebCore::unloadModule): Fixes zero test. (WebCore::writeToFile): Fixes if statement structure.
  • platform/android/KeyEventAndroid.cpp: Modified. (WebCore::windowsKeyCodeForKeyEvent): Fixes switch statement indenting. (WebCore::keyIdentifierForAndroidKeyCode): Fixes switch statement indenting.
  • platform/android/ScreenAndroid.cpp: Modified. Fixes header include order and switch statement indenting.
  • platform/android/TemporaryLinkStubs.cpp: Modified. Fixes header include order.
3:25 PM Changeset in webkit [50005] by eric@webkit.org
  • 4 edits
    1 add in trunk/LayoutTests

2009-10-23 Alejandro G. Castro <alex@igalia.com>

Reviewed by Xan Lopez.

[GTK] Modified the tests results according to the last changes due
to the fixes of the bug (r49985):
https://bugs.webkit.org/show_bug.cgi?id=29697.

  • platform/gtk/editing/execCommand/4916541-expected.txt:
  • platform/gtk/editing/execCommand/5138441-expected.txt:
  • platform/gtk/editing/execCommand/5481523-expected.txt:
  • platform/gtk/editing/execCommand/indent-paragraphs-expected.txt: Added.
3:16 PM Changeset in webkit [50004] by jhoneycutt@apple.com
  • 4 edits
    4 adds in trunk

MSAA: Name for image elements should use alt attribute text

https://bugs.webkit.org/show_bug.cgi?id=30727
<rdar://problem/7328291>

Reviewed by Alice Liu.

WebKit/win:

  • AccessibleBase.cpp:

(AccessibleBase::createInstance):
If the AccessibilityObject is an image, return an AccessibleImage.

  • AccessibleImage.cpp: Added.

(AccessibleImage::AccessibleImage):
Call the base constructor.
(AccessibleImage::name):
If the object is not an AccessibilityRenderObject, return the base's
name(). Look first for an ARIA "labeled-by" attribute, then look for an
alternate text attribute. Fall back to the base's name().

  • AccessibleImage.h: Added.

(AccessibleImage::~AccessibleImage):

  • WebKit.vcproj/WebKit.vcproj:

Add AccessibleImage to the project.

LayoutTests:

  • platform/win/accessibility/img-alt-attribute-expected.txt: Added.
  • platform/win/accessibility/img-alt-attribute.html: Added.
2:49 PM Changeset in webkit [50003] by eric@webkit.org
  • 2 edits in trunk/WebKitTools

2009-10-23 Eric Seidel <eric@webkit.org>

No review, only adding Alice to the list of reviewers.

  • Scripts/modules/committers.py:
2:45 PM Changeset in webkit [50002] by weinig@apple.com
  • 2 edits in trunk/WebCore

Another shot at a fixed build.

  • DerivedSources.cpp:
2:33 PM Changeset in webkit [50001] by weinig@apple.com
  • 2 edits in trunk/WebCore

Another shot at a fixed build.

  • html/HTMLOptionsCollection.idl:
2:25 PM Changeset in webkit [50000] by beidson@apple.com
  • 1 edit in trunk/WebCore/ChangeLog

50,000!

  • ChangeLog: Point out revision 50,000.
2:25 PM Changeset in webkit [49999] by weinig@apple.com
  • 2 edits in trunk/WebKit/win

Try and fix the windows build.

  • Interfaces/WebKit.idl:
2:12 PM Changeset in webkit [49998] by weinig@apple.com
  • 26 edits
    8 adds
    3 deletes in trunk

WebCore: Fix for https://bugs.webkit.org/show_bug.cgi?id=30695
Drop .tags() support from HTMLCollection

Reviewed by Anders Carlsson.

  • Make .tags() only work for HTMLAllCollections in JS. .tags() is still available for all HTMLCollection like objects in Objective-C since that is API.
  • Auto-generate HTMLAllCollection.

Test: fast/dom/HTMLDocument/document-all.html

  • DerivedSources.make:
  • GNUmakefile.am:
  • WebCore.gypi:
  • WebCore.pro:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • bindings/js/JSHTMLAllCollection.cpp: Removed.
  • bindings/js/JSHTMLAllCollection.h: Removed.
  • bindings/js/JSHTMLAllCollectionCustom.cpp: Added.

(WebCore::getNamedItems):
(WebCore::callHTMLAllCollection):
(WebCore::JSHTMLAllCollection::getCallData):
(WebCore::JSHTMLAllCollection::canGetItemsForName):
(WebCore::JSHTMLAllCollection::nameGetter):
(WebCore::JSHTMLAllCollection::item):
(WebCore::JSHTMLAllCollection::namedItem):

  • bindings/js/JSHTMLCollectionCustom.cpp:

(WebCore::toJS):

  • bindings/js/JSHTMLDocumentCustom.cpp:
  • bindings/scripts/CodeGeneratorJS.pm:
  • dom/Document.cpp:

(WebCore::Document::all):

  • dom/Document.h:
  • dom/HTMLAllCollection.idl: Removed.
  • html/HTMLAllCollection.cpp: Added.

(WebCore::HTMLAllCollection::create):
(WebCore::HTMLAllCollection::HTMLAllCollection):
(WebCore::HTMLAllCollection::~HTMLAllCollection):

  • html/HTMLAllCollection.h: Added.
  • html/HTMLAllCollection.idl: Added.
  • html/HTMLCollection.h:
  • html/HTMLCollection.idl:
  • html/HTMLDocument.idl:
  • page/DOMWindow.idl:

LayoutTests: Update and add test for https://bugs.webkit.org/show_bug.cgi?id=30695
Drop .tags() support from HTMLCollection

Reviewed by Anders Carlsson.

  • fast/dom/HTMLDocument/document-all-expected.txt: Added.
  • fast/dom/HTMLDocument/document-all.html: Added.
  • fast/dom/HTMLDocument/script-tests: Added.
  • fast/dom/HTMLDocument/script-tests/document-all.js: Added.
  • fast/dom/Window/window-properties-expected.txt:
  • fast/dom/dom-constructors-expected.txt:
  • fast/dom/dom-constructors.html:
  • fast/dom/domListEnumeration-expected.txt:
  • fast/dom/prototype-inheritance-2-expected.txt:
  • fast/dom/prototype-inheritance-expected.txt:
  • fast/dom/script-tests/domListEnumeration.js:
  • fast/dom/undetectable-document-all-expected.txt:
  • fast/js/global-constructors-expected.txt:
1:47 PM Changeset in webkit [49997] by pfeldman@chromium.org
  • 4 edits in trunk/WebCore

2009-10-23 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Timothy Hatcher.

Web Inspector: pull basic sidebar implementation into the Panel.

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

1:43 PM Changeset in webkit [49996] by eric@webkit.org
  • 2 edits in trunk/WebCore

2009-10-23 Jens Alfke <snej@chromium.org>

Reviewed by Dimitri Glazkov.

Slight optimizations to object returning and exception handling in generated V8 bindings.
https://bugs.webkit.org/show_bug.cgi?id=30599

  • bindings/scripts/CodeGeneratorV8.pm: Generate better code
  • bindings/v8/V8DOMWrapper.h: (WebCore::V8DOMWrapper::convertToV8Object): Added overload that takes a Ref<>
  • bindings/v8/custom/V8InspectorBackendCustom.cpp: (WebCore::CALLBACK_FUNC_DECL): Remove unnecessary template param to prevent compile error
1:35 PM Changeset in webkit [49995] by eric@webkit.org
  • 8 edits
    1 add in trunk

2009-10-23 Alpha Lam <hclam@chromium.org>

Reviewed by Eric Carlson.

Volume slider always starts at half volume
https://bugs.webkit.org/show_bug.cgi?id=28322

  • media/video-volume-slider.html: Added. Pixel test for the volume slider of the media control. This test assumes volume slider is implemented and usable. It moves the mouse over the right bottom corner of a video element to trigger the volume slider. If volume slider is not implemented at all, you'll only see a video element with the first frame.
  • platform/gtk/Skipped: The above test requires a volume slider so it's skipped.
  • platform/mac/Skipped: ditto.
  • platform/win/Skipped: ditto.

2009-10-23 Alpha Lam <hclam@chromium.org>

Reviewed by Eric Carlson.

Volume slider always starts at half volume
https://bugs.webkit.org/show_bug.cgi?id=28322

Sets the value attribute of volume slider to the volume attribute
of the media element. Also update the value attribute of the
input element for the volume slider when the volume attribute
of the media element is updated.

Test: media/video-volume-slider.html

  • rendering/MediaControlElements.cpp: (WebCore::MediaControlVolumeSliderElement::update): Set the value attribute of the input element for the volume slider using the current volume of the media element.
  • rendering/MediaControlElements.h: Add method definition of update().
  • rendering/RenderMedia.cpp: (WebCore::RenderMedia::createVolumeSlider): Creates a volume slider with value attribute equals to the current volume attribute of the media element.
1:26 PM Changeset in webkit [49994] by eric@webkit.org
  • 2 edits in trunk/WebCore

2009-10-23 Evan Martin <evan@chromium.org>

Reviewed by Eric Seidel.

Chrome Linux: fix caret positioning in LTR complex languages
The caret is positioned off by one in languages like Thai.

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

A group of us spent a while trying various approaches to write a test
for this; however, since it is only that the blinking cursor is
displayed incorrectly, we concluded it can not be tested.

(Patch by Hironori Bono.)

  • platform/graphics/chromium/FontLinux.cpp: (WebCore::Font::selectionRectForComplexText):
1:13 PM Changeset in webkit [49993] by eric@webkit.org
  • 1 edit
    2 adds in trunk/LayoutTests

2009-10-23 Andrew Scherkus <scherkus@chromium.org>

Reviewed by Eric Carlson.

Add a layout test to assert that media element implementors do not
advance currentTime after being paused.

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

  • media/video-pause-immediately-expected.txt: Added.
  • media/video-pause-immediately.html: Added.
1:09 PM Changeset in webkit [49992] by tony@chromium.org
  • 10 edits in trunk/WebCore

2009-10-23 Tony Chang <tony@chromium.org>

Reviewed by David Levin.

Fix Chromium build after IsoldatedWorld change (r49963).
https://bugs.webkit.org/show_bug.cgi?id=30719

  • bindings/ScriptControllerBase.cpp:
  • bindings/js/ScriptController.cpp: (WebCore::ScriptController::executeScriptInIsolatedWorld):
  • bindings/v8/ScriptController.cpp: (WebCore::mainThreadNormalWorld):
  • bindings/v8/ScriptController.h: (WebCore::ScriptController::windowShell): (WebCore::ScriptController::existingWindowShell):
  • dom/Document.cpp: (WebCore::Document::createWrapperCache):
  • dom/ScriptExecutionContext.cpp:
  • dom/ScriptExecutionContext.h:
  • xml/XMLHttpRequest.cpp:
  • xml/XMLHttpRequest.h:
1:05 PM Changeset in webkit [49991] by eric@webkit.org
  • 3 edits in trunk/LayoutTests

2009-10-23 Victor Wang <victorw@chromium.org>

Reviewed by Eric Seidel.

Update the expected pixel result for test svg-as-background-6.html

The old result is incorrect because of the wrong SVG masking offset calculation.
Patch http://trac.webkit.org/changeset/49598 fixed the issue.
The pixel result of this layout test needs to be updated.

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

  • platform/mac/fast/backgrounds/svg-as-background-6-expected.checksum:
  • platform/mac/fast/backgrounds/svg-as-background-6-expected.png:
12:48 PM Changeset in webkit [49990] by eric@webkit.org
  • 7 edits
    2 adds in trunk

2009-10-23 Xiaomei Ji <xji@chromium.org>

Reviewed by Sam Weinig.

Fix caretRangeFromPoint()/elementFromPoint() returns wrong result
for a zoomed and scrolled page.
https://bugs.webkit.org/show_bug.cgi?id=30689.

The real cause is the wrong calculation of event.clientX/Y when
page is zoomed and scolled.
After Sam fixed the event.clientX/Y in r49551, the hit test point
should be recalculated (revert r49490).

  • fast/dom/Document/CaretRangeFromPoint/caretRangeFromPoint-in-zoom-and-scroll-expected.txt: Added.
  • fast/dom/Document/CaretRangeFromPoint/caretRangeFromPoint-in-zoom-and-scroll.html: Added.
  • fast/dom/Document/CaretRangeFromPoint/hittest-relative-to-viewport-expected.txt:
  • fast/dom/Document/CaretRangeFromPoint/hittest-relative-to-viewport.html:
  • fast/dom/elementFromPoint-relative-to-viewport-expected.txt:
  • fast/dom/elementFromPoint-relative-to-viewport.html:

2009-10-23 Xiaomei Ji <xji@chromium.org>

Reviewed by Sam Weinig.

Fix caretRangeFromPoint()/elementFromPoint() returns wrong result
for a zoomed and scrolled page.
https://bugs.webkit.org/show_bug.cgi?id=30689.

The real cause is the wrong calculation of event.clientX/Y when
page is zoomed and scolled.
After Sam fixed the event.clientX/Y in r49551, the hit test point
should be recalculated (revert r49490).

Test: fast/dom/Document/CaretRangeFromPoint/caretRangeFromPoint-in-zoom-and-scroll.html

  • dom/Document.cpp: (WebCore::Document::elementFromPoint): calculate the correct point relative to document when page is zoomed and scrolled. (WebCore::Document::caretRangeFromPoint): calculate the correct point relative to document when page is zoomed and scrolled.
12:38 PM Changeset in webkit [49989] by dumi@chromium.org
  • 5 edits
    2 adds in trunk/WebCore

Adding a class that allows us to enable/disable features at
runtime. Adding a flag to enable database support.

Patch by Dumitru Daniliuc <dumi@chromium.org> on 2009-10-22
Reviewed by Dmitry Titov.

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

  • WebCore.gypi:
  • bindings/v8/RuntimeEnabledFeatures.cpp: Added.
  • bindings/v8/RuntimeEnabledFeatures.h: Added.

(WebCore::RuntimeEnabledFeatures::setDatabaseEnabled):
(WebCore::RuntimeEnabledFeatures::databaseEnabled):
(WebCore::RuntimeEnabledFeatures::RuntimeEnabledFeatures):

  • bindings/v8/custom/V8CustomBinding.h:
  • bindings/v8/custom/V8DOMWindowCustom.cpp:

(WebCore::ACCESSOR_RUNTIME_ENABLER):

  • page/DOMWindow.idl:
12:30 PM Changeset in webkit [49988] by eric@webkit.org
  • 2 edits in trunk/WebCore

2009-10-23 Keishi Hattori <casey.hattori@gmail.com>

Reviewed by Timothy Hatcher.

Inspector injects _inspectorCommandLineAPI into global object when opened
https://bugs.webkit.org/show_bug.cgi?id=30675

  • inspector/front-end/InjectedScript.js: (InjectedScript.getCompletions): (InjectedScript._evaluateOn): (InjectedScript.addInspectedNode): (InjectedScript._ensureCommandLineAPIInstalled): Moved _inspectorCommandLineAPI from window to console.
12:23 PM Changeset in webkit [49987] by eric@webkit.org
  • 2 edits in trunk/WebCore

2009-10-23 Janne Koskinen <janne.p.koskinen@digia.com>

Reviewed by Tor Arne Vestbø.

[Qt] don't undef NULL in Symbian emulator

npapi.h way of defining mwerks compiler,intel
and win32 are all true for Symbian emulator.
Changed the definions so that NULL won't be undefined
when compiling for Symbian emulator nor target HW.

  • bridge/npapi.h:
12:14 PM Changeset in webkit [49986] by eric@webkit.org
  • 2 edits in trunk/WebCore

2009-10-23 Janne Koskinen <janne.p.koskinen@digia.com>

Reviewed by Tor Arne Vestbø.

[Qt] Correct linking of sqlite3 for Symbian
https://bugs.webkit.org/show_bug.cgi?id=30569

".lib" will try to include library as static library
causing the library to be searched from wrong directory and if found
will be statically linked instead of the inteded dynamic linking.

  • WebCore.pro:
12:00 PM Changeset in webkit [49985] by adele@apple.com
  • 21 edits
    2 adds in trunk

REGRESSION: Copy is copying incorrect background-color
<rdar://problem/7248529>
https://bugs.webkit.org/show_bug.cgi?id=29697

Patch by Enrica Casucci <enrica@apple.com> on 2009-10-23
Reviewed by Adele Peterson.

WebCore:

Test: editing/execCommand/indent-paragraphs.html

This change removes the BackgroundColor from the list
of inheritable CSS properties and changes significantly
the way we perform indentation.
The new implementation of IndentOutdentCommand uses a new method
defined in CompositeEditCommand called moveParagraphWithClones.
This is an improved version of the original moveParagraph that
is aimed at preserving the original markup of the paragraph being
moved.

  • editing/ApplyStyleCommand.cpp:

(WebCore::): removed CSSPropertyBackgroundColor from editingStyleProperties.

  • editing/CompositeEditCommand.cpp:

(WebCore::CompositeEditCommand::CompositeEditCommand): Removed trailing blanks.
(WebCore::CompositeEditCommand::rebalanceWhitespaceAt): Removed trailing blanks.
(WebCore::CompositeEditCommand::cloneParagraphUnderNewElement): Added.
(WebCore::CompositeEditCommand::cleanupAfterDeletion): Added.
(WebCore::CompositeEditCommand::moveParagraphWithClones): Added.
(WebCore::CompositeEditCommand::moveParagraphs): Modified to use cleanupAfterDeletion.
(WebCore::CompositeEditCommand::breakOutOfEmptyListItem): Removed trailing blanks.
(WebCore::CompositeEditCommand::breakOutOfEmptyMailBlockquotedParagraph): Removed trailing blanks.
(WebCore::CompositeEditCommand::positionAvoidingSpecialElementBoundary): Removed trailing blanks.

  • editing/CompositeEditCommand.h: Added three new methods.
  • editing/IndentOutdentCommand.cpp:

(WebCore::IndentOutdentCommand::tryIndentingAsListItem): Changed to use moveParagraphWithClones.
(WebCore::IndentOutdentCommand::indentIntoBlockquote): Changed to use moveParagraphWithClones.
(WebCore::IndentOutdentCommand::indentRegion): Removed comment.

  • editing/IndentOutdentCommand.h: Removed prepareBlockquoteLevelForInsertion method.

LayoutTests:

All the following tests have been re-baselined because
the indent command now preserves the original markup.
A new test (indent-paragraph.html) has been added to test
several indent combinations not covered by the existing tests.

  • editing/execCommand/19087-expected.txt:
  • editing/execCommand/19653-1-expected.txt:
  • editing/execCommand/indent-div-inside-list-expected.txt:
  • editing/execCommand/indent-nested-blockquotes-expected.txt:
  • editing/execCommand/indent-nested-div-expected.txt:
  • editing/execCommand/indent-paragraphs.html: Added.
  • editing/execCommand/indent-second-paragraph-in-blockquote-expected.txt:
  • editing/execCommand/indent-with-style-expected.txt:
  • editing/inserting/5994480-2-expected.txt:
  • platform/mac/editing/execCommand/4641880-2-expected.txt:
  • platform/mac/editing/execCommand/4916541-expected.txt:
  • platform/mac/editing/execCommand/5138441-expected.txt:
  • platform/mac/editing/execCommand/5481523-expected.txt:
  • platform/mac/editing/execCommand/indent-paragraphs-expected.txt: Added.
  • platform/mac/editing/execCommand/indent-selection-expected.txt:
  • platform/mac/editing/pasteboard/select-element-1-expected.txt:
11:57 AM Changeset in webkit [49984] by eric@webkit.org
  • 2 edits in trunk/WebKitTools

2009-10-23 Eric Seidel <eric@webkit.org>

Reviewed by Eric Carlson.

fast/media/mq-transform-02.html failed on Leopard Commit Bot
https://bugs.webkit.org/show_bug.cgi?id=30700

  • DumpRenderTree/mac/DumpRenderTree.mm: (resetDefaultsToConsistentValues): Update QuickTime version check.
10:34 AM Changeset in webkit [49983] by bweinstein@apple.com
  • 2 edits in trunk/LayoutTests

Rubber-stamped by Sam Weinig.

Isolated worlds tests now pass, removing then from the Skipped
list.

  • platform/win/Skipped:
9:45 AM Changeset in webkit [49982] by kevino@webkit.org
  • 2 edits in trunk/WebKitTools

wxMac 10.4 build fix, needs to link against WebKitSystemInterfaceTiger to get
character measurement APIs that are private on Tiger.

9:42 AM Changeset in webkit [49981] by kevino@webkit.org
  • 3 edits in trunk/WebKit/wx

wx build fix. Update the globalObject calls after changes.

9:34 AM Changeset in webkit [49980] by cmarrin@apple.com
  • 1 edit in trunk/WebKitSite/blog-files/webgl/resources/blogpost.css

Changed anchor style to take a class name

8:53 AM Changeset in webkit [49979] by eric.carlson@apple.com
  • 2 edits in trunk/WebKit/mac

2009-10-22 Eric Carlson <eric.carlson@apple.com>

Reviewed by John Sullivan.

<rdar://problem/7322847>
Pressing space bar key in full screen mode should pause movie but doesn't

  • WebView/WebVideoFullscreenHUDWindowController.mm: (-[WebVideoFullscreenHUDWindowController keyDown:]): New, toggle playback on space key. (-[WebVideoFullscreenHUDWindowController windowDidLoad]): Play button action is now called togglePlaying. (-[WebVideoFullscreenHUDWindowController togglePlaying:]): Rename from playingChanged.
7:30 AM Changeset in webkit [49978] by vestbo@webkit.org
  • 1 edit in trunk/WebKit/qt/Api/qwebframe.cpp

Qt build fix

7:07 AM Changeset in webkit [49977] by benm@google.com
  • 2 edits in trunk/JavaScriptCore

Fixes a leak in createThreadInternal on Android.
https://bugs.webkit.org/show_bug.cgi?id=30698

Patch by Steve Block <steveblock@google.com> on 2009-10-23
Reviewed by Dmitry Titov.

  • wtf/ThreadingPthreads.cpp: Modified.

(WTF::createThreadInternal): Avoid leaking a ThreadData object on failure.

5:45 AM Changeset in webkit [49976] by benm@google.com
  • 3 edits in trunk/WebCore

Adds missing UnusedParam.h includes, required when INSPECTOR is not enabled.
https://bugs.webkit.org/show_bug.cgi?id=30677

Patch by Steve Block <steveblock@google.com> on 2009-10-23
Reviewed by Eric Seidel.

Build fix, no new tests required.

  • page/Console.cpp: Modified. Added UnusedParam.h include.
  • workers/WorkerContext.cpp: Modified. Added UnusedParam.h include.
5:02 AM Changeset in webkit [49975] by zecke@webkit.org
  • 2 edits in trunk/WebCore

[Qt] Plugins : Use window's winId() instead of the widget's.

2009-10-23 Girish Ramakrishnan <Girish Ramakrishnan>

Reviewed by Holger Freyther.

[Qt] Plugins : Use window's winId() instead of the widget's.

As per, https://developer.mozilla.org/en/NPN_GetValue
NPNVnetscapeWindow on Unix/X11: "Gets the browser toplevel window in which the
plug-in is displayed; returns Window".

The issue was found because NPNVNetscapeWindow winId() ends up creating
a native window resulting in flicker.

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

  • plugins/qt/PluginViewQt.cpp: (WebCore::PluginView::getValue):
5:02 AM Changeset in webkit [49974] by zecke@webkit.org
  • 2 edits in trunk/WebCore

[Qt] Windowless Plugins : Don't use m_clipRect when painting.

2009-10-23 Girish Ramakrishnan <Girish Ramakrishnan>

Reviewed by Holger Freyther.

[Qt] Windowless Plugins : Don't use m_clipRect when painting.

Though it works, it is not correct to use m_clipRect for painting
in Windowless mode. Instead, the rect paramater that is passed
as a part of PluginView::paint() must be used. This change will
also result in some speedup since previously we used to paint all
the visible parts of a windowless plugin (even if those parts were
not dirty).

Also, fix invalidateRect() to compute the correct width and height.

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

  • plugins/qt/PluginViewQt.cpp: (WebCore::PluginView::paint): (WebCore::PluginView::invalidateRect):
5:02 AM Changeset in webkit [49973] by zecke@webkit.org
  • 4 edits
    1 add in trunk/WebCore

Plugins : Use test.swf in the plugins manual tests.

They previously referenced youtube videos which among other things are
cumbersome to use without internet connection. The old youtube link is
just commented out so that testing with youtube can be done with
little effort.

4:47 AM Changeset in webkit [49972] by xan@webkit.org
  • 3 edits in trunk/LayoutTests

2009-10-23 Xan Lopez <xlopez@igalia.com>

Use the correct Skipped file ...

  • platform/gtk/Skipped:
  • platform/mac/Skipped:
4:43 AM Changeset in webkit [49971] by xan@webkit.org
  • 2 edits in trunk/LayoutTests

2009-10-23 Xan Lopez <xlopez@igalia.com>

Skip new tests missing expected results.

  • platform/mac/Skipped:
1:14 AM Changeset in webkit [49970] by bweinstein@apple.com
  • 2 edits
    1 add in trunk/WebCore

Fix the Windows Release build by moving Accessibility files
into all-in-one file and excluding them from the build.

Reviewed by Jon Honeycutt.

  • WebCore.vcproj/WebCore.vcproj:
  • accessibility/AccessibilityAllInOne.cpp: Added.

Oct 22, 2009:

11:22 PM Changeset in webkit [49969] by mrowe@apple.com
  • 16 edits in trunk

Update configurations to support the mysterious future.

Rubber-stamped by Brady Eidson.

10:27 PM Changeset in webkit [49968] by barraclough@apple.com
  • 2 edits in trunk/WebKitTools

Build fix following bug #30696.

Patch by Gavin Barraclough <barraclough@apple.com> on 2009-10-22
Reviewed by NOBODY (build fix).

  • DumpRenderTree/gtk/LayoutTestControllerGtk.cpp:

(LayoutTestController::evaluateScriptInIsolatedWorld):

10:16 PM Changeset in webkit [49967] by mrowe@apple.com
  • 23 edits
    2 deletes in trunk/WebKitSite

Update to WordPress 2.8.5.

  • blog:
10:06 PM Changeset in webkit [49966] by barraclough@apple.com
  • 3 edits in trunk/WebKit/qt

Build fix following bug #30696.

Patch by Gavin Barraclough <barraclough@apple.com> on 2009-10-22
Reviewed by NOBODY (speculative build fix - qt is currently already broken!)

  • Api/qwebelement.cpp:

(setupScriptContext):

  • Api/qwebframe.cpp:

(QWebFrame::evaluateJavaScript):

9:57 PM Changeset in webkit [49965] by barraclough@apple.com
  • 2 edits in trunk/WebKit/mac

Build fix following bug #30696.

Patch by Gavin Barraclough <barraclough@apple.com> on 2009-10-22
Reviewed by NOBODY (build fix).

  • Plugins/Hosted/NetscapePluginInstanceProxy.mm:

(WebKit::NetscapePluginInstanceProxy::getWindowNPObject):
(WebKit::NetscapePluginInstanceProxy::evaluate):
(WebKit::NetscapePluginInstanceProxy::invoke):
(WebKit::NetscapePluginInstanceProxy::invokeDefault):
(WebKit::NetscapePluginInstanceProxy::construct):
(WebKit::NetscapePluginInstanceProxy::getProperty):
(WebKit::NetscapePluginInstanceProxy::setProperty):
(WebKit::NetscapePluginInstanceProxy::removeProperty):
(WebKit::NetscapePluginInstanceProxy::hasProperty):
(WebKit::NetscapePluginInstanceProxy::hasMethod):
(WebKit::NetscapePluginInstanceProxy::enumerate):

9:48 PM Changeset in webkit [49964] by barraclough@apple.com
  • 3 edits in trunk/WebKit/gtk

Build fix following bug #30696.

Patch by Gavin Barraclough <barraclough@apple.com> on 2009-10-22
Reviewed by NOBODY (build fix).

  • WebCoreSupport/FrameLoaderClientGtk.cpp:

(WebKit::FrameLoaderClient::windowObjectCleared):

  • webkit/webkitwebframe.cpp:

(webkit_web_frame_get_global_context):

9:33 PM Changeset in webkit [49963] by barraclough@apple.com
  • 101 edits in trunk

WebCore: https://bugs.webkit.org/show_bug.cgi?id=30696
Add support for IsolatedWorlds to JSC bindings.

Patch by Gavin Barraclough <barraclough@apple.com> on 2009-10-16
Reviewed by Sam Weinig & Geoff Garen.

An IsolatedWorld is basically a parallel, separate copy of the window shells and DOM wrapper objects for
a given document. To support isolation this patch:

  • Adds support to the ScriptController to track multiple window shells, one per world.
  • Adds support to Document to support multiple separate wrapper-maps, one per world.
  • Replaces the single global DOM wrapper map (on the WebCoreJSClientData) with separate maps, stored on the (new) IsolatedWorld objects.

In addition to supporting separate copies of these objects, two other features are supported:

  • It is necessary to track the current world on entry into JSC, so that within callbacks out to WebCore we can determine which world (and as such, set of DOM bindings) we should be operating on.
  • EventListeners & Callbacks are run in the world they were registered in. This requires the handler to retain a reference to the world.

No new tests. (Enabled existing isolated world layout tests.)

  • WebCore.base.exp:
  • bindings/js/JSAbstractWorkerCustom.cpp:

(WebCore::JSAbstractWorker::addEventListener):
(WebCore::JSAbstractWorker::removeEventListener):

  • bindings/js/JSCallbackData.cpp:

(WebCore::JSCallbackData::invokeCallback):

  • bindings/js/JSCallbackData.h:

(WebCore::JSCallbackData::JSCallbackData):

  • bindings/js/JSCustomXPathNSResolver.cpp:

(WebCore::JSCustomXPathNSResolver::lookupNamespaceURI):

  • bindings/js/JSDOMApplicationCacheCustom.cpp:

(WebCore::JSDOMApplicationCache::addEventListener):
(WebCore::JSDOMApplicationCache::removeEventListener):

  • bindings/js/JSDOMBinding.cpp:

(WebCore::removeWrappers):
(WebCore::DOMObjectWrapperMap::get):
(WebCore::DOMObjectWrapperMap::set):
(WebCore::DOMObjectWrapperMap::remove):
(WebCore::DOMObjectWrapperMap::take):
(WebCore::IsolatedWorld::IsolatedWorld):
(WebCore::IsolatedWorld::~IsolatedWorld):
(WebCore::EnterIsolatedWorld::EnterIsolatedWorld):
(WebCore::EnterIsolatedWorld::~EnterIsolatedWorld):
(WebCore::JSGlobalDataWorldIterator::JSGlobalDataWorldIterator):
(WebCore::JSGlobalDataWorldIterator::operator bool):
(WebCore::JSGlobalDataWorldIterator::operator*):
(WebCore::JSGlobalDataWorldIterator::operator->):
(WebCore::JSGlobalDataWorldIterator::operator++):
(WebCore::getCurrentWorld):
(WebCore::getNormalWorld):
(WebCore::commonNormalWorld):
(WebCore::commonCurrentWorld):
(WebCore::DOMObjectHashTableMap::mapFor):
(WebCore::DOMObjectWrapperMap::mapFor):
(WebCore::forgetDOMObject):
(WebCore::getCachedDOMNodeWrapper):
(WebCore::forgetDOMNode):
(WebCore::cacheDOMNodeWrapper):
(WebCore::forgetAllDOMNodesForDocument):
(WebCore::forgetWorldOfDOMNodesForDocument):
(WebCore::isObservableThroughDOM):
(WebCore::markDOMNodesForDocument):
(WebCore::markActiveObjectsForContext):
(WebCore::takeWrappers):
(WebCore::updateDOMNodeDocument):
(WebCore::markDOMObjectWrapper):
(WebCore::allowsAccessFromFrame):
(WebCore::printErrorMessageForFrame):
(WebCore::JSC_DebuggerCallFrame_evaluateInWorld):
(WebCore::JSC_callInWorld):
(WebCore::JSC_constructInWorld):
(WebCore::JSC_evaluateInWorld):

  • bindings/js/JSDOMBinding.h:

(WebCore::IsolatedWorld::rememberDocument):
(WebCore::IsolatedWorld::forgetDocument):
(WebCore::IsolatedWorld::rememberScriptController):
(WebCore::IsolatedWorld::forgetScriptController):
(WebCore::DOMObjectHashTableMap::~DOMObjectHashTableMap):
(WebCore::DOMObjectHashTableMap::get):
(WebCore::WebCoreJSClientData::WebCoreJSClientData):
(WebCore::WebCoreJSClientData::currentWorld):
(WebCore::WebCoreJSClientData::normalWorld):
(WebCore::WebCoreJSClientData::rememberWorld):
(WebCore::WebCoreJSClientData::forgetWorld):
(WebCore::debuggerWorld):
(WebCore::pluginWorld):

  • bindings/js/JSDOMGlobalObject.cpp:

(WebCore::JSDOMGlobalObject::createJSAttributeEventListener):
(WebCore::toJSDOMGlobalObject):

  • bindings/js/JSDOMGlobalObject.h:
  • bindings/js/JSDOMWindowBase.cpp:

(WebCore::JSDOMWindowBase::printErrorMessage):
(WebCore::JSDOMWindowBase::commonJSGlobalData):
(WebCore::toJS):
(WebCore::toJSDOMWindow):

  • bindings/js/JSDOMWindowBase.h:
  • bindings/js/JSDOMWindowCustom.cpp:

(WebCore::createWindow):
(WebCore::JSDOMWindow::open):
(WebCore::JSDOMWindow::showModalDialog):
(WebCore::JSDOMWindow::setTimeout):
(WebCore::JSDOMWindow::setInterval):
(WebCore::JSDOMWindow::addEventListener):
(WebCore::JSDOMWindow::removeEventListener):

  • bindings/js/JSDOMWindowShell.cpp:

(WebCore::toJS):
(WebCore::toJSDOMWindowShell):

  • bindings/js/JSDOMWindowShell.h:
  • bindings/js/JSDesktopNotificationsCustom.cpp:

(WebCore::JSNotification::addEventListener):
(WebCore::):

  • bindings/js/JSEventListener.cpp:

(WebCore::JSEventListener::JSEventListener):
(WebCore::JSEventListener::handleEvent):
(WebCore::JSEventListener::reportError):

  • bindings/js/JSEventListener.h:

(WebCore::JSEventListener::create):

  • bindings/js/JSEventSourceCustom.cpp:

(WebCore::JSEventSource::addEventListener):
(WebCore::JSEventSource::removeEventListener):

  • bindings/js/JSEventTarget.cpp:

(WebCore::toJS):

  • bindings/js/JSHTMLDocumentCustom.cpp:

(WebCore::JSHTMLDocument::open):

  • bindings/js/JSHTMLFrameSetElementCustom.cpp:

(WebCore::JSHTMLFrameSetElement::nameGetter):

  • bindings/js/JSInspectorBackendCustom.cpp:

(WebCore::JSInspectorBackend::databaseForId):
(WebCore::JSInspectorBackend::inspectedWindow):
(WebCore::JSInspectorBackend::nodeForId):

  • bindings/js/JSLazyEventListener.cpp:

(WebCore::JSLazyEventListener::JSLazyEventListener):
(WebCore::JSLazyEventListener::parseCode):

  • bindings/js/JSLazyEventListener.h:

(WebCore::JSLazyEventListener::create):

  • bindings/js/JSMessageChannelCustom.cpp:

(WebCore::JSMessageChannel::markChildren):

  • bindings/js/JSMessagePortCustom.cpp:

(WebCore::JSMessagePort::markChildren):
(WebCore::JSMessagePort::addEventListener):
(WebCore::JSMessagePort::removeEventListener):

  • bindings/js/JSNodeCustom.cpp:

(WebCore::JSNode::addEventListener):
(WebCore::JSNode::removeEventListener):
(WebCore::JSNode::markChildren):

  • bindings/js/JSNodeFilterCondition.cpp:

(WebCore::JSNodeFilterCondition::acceptNode):

  • bindings/js/JSQuarantinedObjectWrapper.cpp:

(WebCore::JSQuarantinedObjectWrapper::construct):
(WebCore::JSQuarantinedObjectWrapper::call):

  • bindings/js/JSSVGElementInstanceCustom.cpp:

(WebCore::JSSVGElementInstance::addEventListener):
(WebCore::JSSVGElementInstance::removeEventListener):

  • bindings/js/JSSharedWorkerCustom.cpp:

(WebCore::JSSharedWorker::markChildren):

  • bindings/js/JSWebSocketCustom.cpp:

(WebCore::JSWebSocket::addEventListener):
(WebCore::JSWebSocket::removeEventListener):

  • bindings/js/JSWorkerContextCustom.cpp:

(WebCore::JSWorkerContext::addEventListener):
(WebCore::JSWorkerContext::removeEventListener):
(WebCore::JSWorkerContext::setTimeout):
(WebCore::JSWorkerContext::setInterval):

  • bindings/js/JSXMLHttpRequestConstructor.cpp:

(WebCore::constructXMLHttpRequest):

  • bindings/js/JSXMLHttpRequestCustom.cpp:

(WebCore::JSXMLHttpRequest::markChildren):
(WebCore::JSXMLHttpRequest::addEventListener):
(WebCore::JSXMLHttpRequest::removeEventListener):

  • bindings/js/JSXMLHttpRequestUploadCustom.cpp:

(WebCore::JSXMLHttpRequestUpload::markChildren):
(WebCore::JSXMLHttpRequestUpload::addEventListener):
(WebCore::JSXMLHttpRequestUpload::removeEventListener):

  • bindings/js/ScheduledAction.cpp:

(WebCore::ScheduledAction::create):
(WebCore::ScheduledAction::ScheduledAction):
(WebCore::ScheduledAction::executeFunctionInContext):
(WebCore::ScheduledAction::execute):

  • bindings/js/ScheduledAction.h:

(WebCore::ScheduledAction::ScheduledAction):

  • bindings/js/ScriptCachedFrameData.cpp:

(WebCore::ScriptCachedFrameData::ScriptCachedFrameData):
(WebCore::ScriptCachedFrameData::restore):

  • bindings/js/ScriptController.cpp:

(WebCore::ScriptController::~ScriptController):
(WebCore::ScriptController::evaluateInWorld):
(WebCore::ScriptController::evaluate):
(WebCore::ScriptController::evaluateInIsolatedWorld):
(WebCore::ScriptController::clearWindowShell):
(WebCore::ScriptController::initScript):
(WebCore::ScriptController::processingUserGestureEvent):
(WebCore::ScriptController::attachDebugger):
(WebCore::ScriptController::updateDocument):
(WebCore::ScriptController::bindingRootObject):
(WebCore::ScriptController::createRootObject):
(WebCore::ScriptController::windowScriptNPObject):
(WebCore::ScriptController::jsObjectForPluginElement):

  • bindings/js/ScriptController.h:

(WebCore::ScriptController::windowShell):
(WebCore::ScriptController::existingWindowShell):
(WebCore::ScriptController::globalObject):
(WebCore::ScriptController::forgetWorld):

  • bindings/js/ScriptControllerMac.mm:

(WebCore::ScriptController::windowScriptObject):

  • bindings/js/ScriptEventListener.cpp:

(WebCore::createAttributeEventListener):

  • bindings/js/ScriptFunctionCall.cpp:

(WebCore::ScriptFunctionCall::call):
(WebCore::ScriptFunctionCall::construct):

  • bindings/js/ScriptObjectQuarantine.cpp:

(WebCore::getQuarantinedScriptObject):

  • bindings/js/ScriptState.cpp:

(WebCore::scriptStateFromNode):
(WebCore::scriptStateFromPage):

  • bindings/js/ScriptState.h:
  • bindings/js/WorkerScriptController.cpp:

(WebCore::WorkerScriptController::WorkerScriptController):
(WebCore::WorkerScriptController::evaluate):

  • bindings/objc/DOMInternal.mm:

(-[WebScriptObject _initializeScriptDOMNodeImp]):

  • bindings/objc/WebScriptObject.mm:

(-[WebScriptObject callWebScriptMethod:withArguments:]):
(-[WebScriptObject evaluateWebScript:]):

  • bindings/scripts/CodeGeneratorJS.pm:
  • bridge/NP_jsobject.cpp:

(_NPN_InvokeDefault):
(_NPN_Invoke):
(_NPN_Evaluate):
(_NPN_Construct):

  • bridge/jni/jni_jsobject.mm:

(JavaJSObject::call):
(JavaJSObject::eval):

  • dom/Document.cpp:

(WebCore::Document::createWrapperCache):

  • dom/Document.h:

(WebCore::Document::wrapperCacheMap):
(WebCore::Document::getWrapperCache):

  • inspector/InspectorController.cpp:

(WebCore::InspectorController::startUserInitiatedProfiling):
(WebCore::InspectorController::stopUserInitiatedProfiling):

  • inspector/JavaScriptCallFrame.cpp:

(WebCore::JavaScriptCallFrame::evaluate):

  • loader/FrameLoader.cpp:

(WebCore::FrameLoader::dispatchWindowObjectAvailable):

  • platform/network/mac/AuthenticationMac.mm:
  • xml/XMLHttpRequest.cpp:

(WebCore::XMLHttpRequest::XMLHttpRequest):
(WebCore::XMLHttpRequest::dropProtection):

  • xml/XMLHttpRequest.h:

(WebCore::XMLHttpRequest::create):

WebKit/mac: https://bugs.webkit.org/show_bug.cgi?id=30696
Update to incorporate support for IsolatedWorlds in JSC bindings.

Patch by Gavin Barraclough <barraclough@apple.com> on 2009-10-16
Reviewed by Sam Weinig & Geoff Garen.

  • WebView/WebFrame.mm:

(-[WebFrame _attachScriptDebugger]):
(-[WebFrame _stringByEvaluatingJavaScriptFromString:forceUserGesture:]):
(-[WebFrame globalContext]):

  • WebView/WebScriptDebugDelegate.mm:

(-[WebScriptCallFrame evaluateWebScript:]):

  • WebView/WebView.mm:

(-[WebView aeDescByEvaluatingJavaScriptFromString:]):

WebKit/win: https://bugs.webkit.org/show_bug.cgi?id=30696
Update to incorporate support for IsolatedWorlds in JSC bindings.

Patch by Gavin Barraclough <barraclough@apple.com> on 2009-10-22
Reviewed by Sam Weinig & Geoff Garen.

  • Interfaces/IWebFramePrivate.idl:
  • WebFrame.cpp:

(WebFrame::globalContext):
(WebFrame::windowObjectCleared):
(WebFrame::stringByEvaluatingJavaScriptInIsolatedWorld):

  • WebFrame.h:

WebKitTools: https://bugs.webkit.org/show_bug.cgi?id=30696
Enable isolated-worlds tests on mac.

Patch by Gavin Barraclough <barraclough@apple.com> on 2009-10-22
Reviewed by Sam Weinig & Geoff Garen.

Add private interface for DRT to invoke execution in a given world.

  • DumpRenderTree/LayoutTestController.cpp:

(evaluateScriptInIsolatedWorldCallback):
(LayoutTestController::staticFunctions):

  • DumpRenderTree/LayoutTestController.h:
  • DumpRenderTree/mac/LayoutTestControllerMac.mm:

(LayoutTestController::evaluateScriptInIsolatedWorld):

  • DumpRenderTree/win/LayoutTestControllerWin.cpp:

(LayoutTestController::evaluateScriptInIsolatedWorld):

LayoutTests: https://bugs.webkit.org/show_bug.cgi?id=30696
Enable isolated-worlds tests on mac.

  • Update the results to switch from windows to unix line-endings.
  • Update all-window-prototypes.html to allow for properties on the global object with null prototypes (just skip over them).

Patch by Gavin Barraclough <barraclough@apple.com> on 2009-10-19
Reviewed by Sam Weinig & Geoff Garen.

  • http/tests/security/isolatedWorld/all-window-properties-expected.txt:
  • http/tests/security/isolatedWorld/all-window-prototypes-expected.txt:
  • http/tests/security/isolatedWorld/all-window-prototypes.html:
  • http/tests/security/isolatedWorld/body-properties-expected.txt:
  • http/tests/security/isolatedWorld/body-prototype-expected.txt:
  • http/tests/security/isolatedWorld/document-properties-expected.txt:
  • http/tests/security/isolatedWorld/document-prototype-expected.txt:
  • http/tests/security/isolatedWorld/global-variables-expected.txt:
  • http/tests/security/isolatedWorld/image-properties-expected.txt:
  • http/tests/security/isolatedWorld/image-prototype-expected.txt:
  • http/tests/security/isolatedWorld/location-properties-expected.txt:
  • http/tests/security/isolatedWorld/location-prototype-expected.txt:
  • http/tests/security/isolatedWorld/number-prototype-expected.txt:
  • http/tests/security/isolatedWorld/object-prototype-expected.txt:
  • http/tests/security/isolatedWorld/storage-properties-expected.txt:
  • http/tests/security/isolatedWorld/storage-prototype-expected.txt:
  • http/tests/security/isolatedWorld/string-prototype-expected.txt:
  • http/tests/security/isolatedWorld/window-properties-expected.txt:
  • platform/mac/Skipped:
9:12 PM Changeset in webkit [49962] by Joseph Pecoraro
  • 2 edits in trunk/WebCore

2009-10-22 Joseph Pecoraro <Joseph Pecoraro>

Reviewed by Timothy Hatcher.

Web Inspector: EventListenersSidebarPane TODO No Longer Needed
https://bugs.webkit.org/show_bug.cgi?id=30705

  • inspector/front-end/EventListenersSidebarPane.js:
5:22 PM Changeset in webkit [49961] by bweinstein@apple.com
  • 3 edits in trunk/LayoutTests

Updating expected results - unreviewed.

Updated expected computed style results on Windows after r49959.

  • platform/win/fast/css/getComputedStyle/computed-style-expected.txt:
  • platform/win/fast/css/getComputedStyle/computed-style-without-renderer-expected.txt:
5:00 PM Changeset in webkit [49960] by ap@apple.com
  • 3 edits in trunk/WebCore

Reviewed by Tim Hatcher.

https://bugs.webkit.org/show_bug.cgi?id=30506
<rdar://problem/7319845> Resources that the server sent as 304 not modified are not shown
in the inspectors resource pane

I don't know how to make a test for this.

The issue here was that preloaded resources weren't added to DocLoader document resource set,
but Web Inspector asked DocLoader to fetch data from CachedResource. Even when (if) document
parser eventually requested the same resource for real and it got added to resource set, it
was too late - the Inspector wasn't updated.

  • inspector/InspectorResource.cpp: (WebCore::InspectorResource::cachedResource): (WebCore::InspectorResource::type): (WebCore::InspectorResource::resourceData):
  • inspector/InspectorResource.h: Fix the issue by trying to fetch corresponding CachedResource harder - also look in global cache. This seems safe, and easier than updating the Inspector on transitions between resource states (revalidate vs. revalidate done and preload vs. non-preload).
4:34 PM Changeset in webkit [49959] by Beth Dakin
  • 17 edits
    36 adds in trunk

WebCore: Fix for <rdar://problem/6942706> ER: Add a CSS property that allows
shadows to work for SVG content
-and corresponding-
https://bugs.webkit.org/show_bug.cgi?id=30479

Reviewed by Dan Bernstein.

This patch adds a new SVG-specific CSS property called -webkit-
shadow that has the same syntax as -webkit-box-shadow

Add CSSPropertyWebkitShadow to the list of SVG CSS properties, and
make valueForShadow a member function rather than a static function

  • css/CSSComputedStyleDeclaration.cpp:

(WebCore::):
(WebCore::CSSComputedStyleDeclaration::valueForShadow):
(WebCore::CSSComputedStyleDeclaration::getPropertyCSSValue):

  • css/CSSComputedStyleDeclaration.h:

Call valueForShadow for CSSPropertyWebkitShadow

  • css/SVGCSSComputedStyleDeclaration.cpp:

(WebCore::CSSComputedStyleDeclaration::getSVGPropertyCSSValue):

Do standard CSS stuff for CSSPropertyWebkitShadow

  • css/SVGCSSParser.cpp:

(WebCore::CSSParser::parseSVGValue):

  • css/SVGCSSPropertyNames.in:
  • css/SVGCSSStyleSelector.cpp:

(WebCore::CSSStyleSelector::applySVGProperty):

Set the appropriate shadow on the paint context if we have one.
This seems to be all we have to do to have an SVG-shaped shadow.

  • rendering/SVGRenderSupport.cpp:

(WebCore::SVGRenderBase::prepareToRenderSVGContent):

Add shadow to SVGRenderStyle. I added a new macro that I don't like
just because that's the way things are done here right now.

  • rendering/style/SVGRenderStyle.cpp:

(WebCore::SVGRenderStyle::SVGRenderStyle):
(WebCore::SVGRenderStyle::operator==):

  • rendering/style/SVGRenderStyle.h:
  • rendering/style/SVGRenderStyleDefs.cpp:

(StyleShadowSVGData::StyleShadowSVGData):
(StyleShadowSVGData::operator==):

  • rendering/style/SVGRenderStyleDefs.h:

(WebCore::StyleShadowSVGData::create):
(WebCore::StyleShadowSVGData::copy):
(WebCore::StyleShadowSVGData::operator!=):

LayoutTests: Tests for <rdar://problem/6942706> ER: Add a CSS property that
allows shadows to work for SVG content
-and corresponding-
https://bugs.webkit.org/show_bug.cgi?id=30479

Reviewed by Dan Bernstein.

New results that include -webkit-shadow:

  • platform/mac/fast/css/getComputedStyle/computed-style-expected.txt:
  • platform/mac/fast/css/getComputedStyle/computed-style-without-renderer-expected.txt:

New tests and their results.

  • platform/mac/svg/css/arrow-with-shadow-expected.checksum: Added.
  • platform/mac/svg/css/arrow-with-shadow-expected.png: Added.
  • platform/mac/svg/css/arrow-with-shadow-expected.txt: Added.
  • platform/mac/svg/css/circle-in-mask-with-shadow-expected.checksum: Added.
  • platform/mac/svg/css/circle-in-mask-with-shadow-expected.png: Added.
  • platform/mac/svg/css/circle-in-mask-with-shadow-expected.txt: Added.
  • platform/mac/svg/css/clippath-with-shadow-expected.checksum: Added.
  • platform/mac/svg/css/clippath-with-shadow-expected.png: Added.
  • platform/mac/svg/css/clippath-with-shadow-expected.txt: Added.
  • platform/mac/svg/css/group-with-shadow-expected.checksum: Added.
  • platform/mac/svg/css/group-with-shadow-expected.png: Added.
  • platform/mac/svg/css/group-with-shadow-expected.txt: Added.
  • platform/mac/svg/css/mask-with-shadow-expected.checksum: Added.
  • platform/mac/svg/css/mask-with-shadow-expected.png: Added.
  • platform/mac/svg/css/mask-with-shadow-expected.txt: Added.
  • platform/mac/svg/css/path-with-shadow-expected.checksum: Added.
  • platform/mac/svg/css/path-with-shadow-expected.png: Added.
  • platform/mac/svg/css/path-with-shadow-expected.txt: Added.
  • platform/mac/svg/css/stars-with-shadow-expected.checksum: Added.
  • platform/mac/svg/css/stars-with-shadow-expected.png: Added.
  • platform/mac/svg/css/stars-with-shadow-expected.txt: Added.
  • platform/mac/svg/filters/shadow-on-filter-expected.checksum: Added.
  • platform/mac/svg/filters/shadow-on-filter-expected.png: Added.
  • platform/mac/svg/filters/shadow-on-filter-expected.txt: Added.
  • platform/mac/svg/filters/shadow-on-rect-with-filter-expected.checksum: Added.
  • platform/mac/svg/filters/shadow-on-rect-with-filter-expected.png: Added.
  • platform/mac/svg/filters/shadow-on-rect-with-filter-expected.txt: Added.
  • svg/css/arrow-with-shadow.svg: Added.
  • svg/css/circle-in-mask-with-shadow.svg: Added.
  • svg/css/clippath-with-shadow.svg: Added.
  • svg/css/getComputedStyle-basic-expected.txt:
  • svg/css/group-with-shadow.svg: Added.
  • svg/css/mask-with-shadow.svg: Added.
  • svg/css/path-with-shadow.svg: Added.
  • svg/css/stars-with-shadow.html: Added.
  • svg/filters/shadow-on-filter.svg: Added.
  • svg/filters/shadow-on-rect-with-filter.svg: Added.
4:07 PM Changeset in webkit [49958] by eric@webkit.org
  • 4 edits in trunk/WebCore

2009-10-22 Joanmarie Diggs <joanmarie.diggs@gmail.com>

Reviewed by Xan Lopez.

https://bugs.webkit.org/show_bug.cgi?id=25530
[Gtk] Implement LABEL_FOR/LABELLED_BY relationship pair for labels

Implements atk_object_ref_relation_set and LABEL_FOR/LABELLED_BY.
Also causes the accessible name for labeled controls to be based on
the label as expected, rather than based on the contents.

  • accessibility/AccessibilityRenderObject.h:
  • accessibility/AccessibilityRenderObject.cpp: (correspondingLabelForControlElement):
  • accessibility/gtk/AccessibilityObjectWrapperAtk.cpp: (setAtkRelationSetFromCoreObject): (webkit_accessible_ref_relation_set): (webkit_accessible_class_init): (webkit_accessible_get_name):
2:32 PM Changeset in webkit [49957] by eric@webkit.org
  • 4 edits
    2 adds in trunk

2009-10-22 Shu Chang <Chang.Shu@nokia.com>

Reviewed by Eric Seidel.

[Qt] Added expected result for clicked-link-is-visited.html.
Removed test case from qt/Skipped as it works now.

  • platform/qt/Skipped:
  • platform/qt/fast/history: Added.
  • platform/qt/fast/history/clicked-link-is-visited-expected.txt: Added.

2009-10-22 Shu Chang <Chang.Shu@nokia.com>

Reviewed by Eric Seidel.

[Qt] Enable track visited links in QWebPage
https://bugs.webkit.org/show_bug.cgi?id=30574

Test: fast/history/clicked-link-is-visited.html

  • Api/qwebpage.cpp: (QWebPagePrivate::QWebPagePrivate):
2:08 PM Changeset in webkit [49956] by Joseph Pecoraro
  • 2 edits in trunk/WebCore

2009-10-22 Joseph Pecoraro <Joseph Pecoraro>

Reviewed by Timothy Hatcher.

Web Inspector: windowFocused and windowBlured Fail to Clear/Mark Inspector as "inactive"
https://bugs.webkit.org/show_bug.cgi?id=30663

Correctly handle focusing/blurring on inner <iframe>'s such as Source Frames.

  • inspector/front-end/inspector.js: (WebInspector.windowFocused): fix for inner iframes (WebInspector.windowBlurred): fix for inner iframes (WebInspector.addMainEventListeners): change useCapture to false
1:33 PM Changeset in webkit [49955] by ggaren@apple.com
  • 4 edits in trunk/JavaScriptCore

Fixed ASSERT when opening Safari's Caches window while the Web Inspector
is open.

Patch by Geoffrey Garen <ggaren@apple.com> on 2009-10-22
Reviewed by Alexey Proskuryakov.

  • runtime/Collector.cpp:

(JSC::typeName): Added two new types to the type name list in the Collector.
These types have been around for a while, but nobody remembered to consider them here.

  • runtime/JSCell.h:

(JSC::JSCell::isPropertyNameIterator):

  • runtime/JSPropertyNameIterator.h:

(JSC::JSPropertyNameIterator::isPropertyNameIterator): Give the Collector
a way to tell if a cell is a JSPropertyNameIterator.

1:19 PM Changeset in webkit [49954] by eric@webkit.org
  • 3 edits in trunk/WebCore

2009-10-22 Jaime Yap <jaimeyap@google.com>

Reviewed by Timothy Hatcher.

Makes a page reload optional when activating resource monitoring in
InspectorController. It leaves the default behavior as performing the reload.
Reload is an optional param so callsites do not have to change.

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

No new tests added.

  • inspector/InspectorController.cpp: (WebCore::InspectorController::enableResourceTracking):
  • inspector/InspectorController.h:
1:10 PM Changeset in webkit [49953] by eric@webkit.org
  • 2 edits in trunk/WebKit/qt

2009-10-22 Girish Ramakrishnan <Girish Ramakrishnan>

Reviewed by Eric Seidel.

[Qt] Add Print Shortcut to QtLauncher

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

  • QtLauncher/main.cpp: (MainWindow::setupUI):
1:06 PM Changeset in webkit [49952] by sfalken@apple.com
  • 2 edits
    1 delete in trunk/JavaScriptCore

https://bugs.webkit.org/show_bug.cgi?id=30686
Remove debug-specific def file.
Only Debug_All target uses JavaScriptCore_debug.dll naming, and since
that target is only used internally, maintaining two files just to
suppress a single link warning isn't worthwhile.

Reviewed by Jon Honeycutt.

12:56 PM Changeset in webkit [49951] by eric@webkit.org
  • 2 edits in trunk/WebCore

2009-10-22 Shu Chang <Chang.Shu@nokia.com>

Reviewed by Eric Seidel.

[Qt] Fix memory leak in QNetworkReplyHandler::abort().
In QNetworkReplyHandler::release(), m_reply should no longer point to its parent
after being released.
https://bugs.webkit.org/show_bug.cgi?id=30167

No layout test available as a non-functional fix.

  • platform/network/qt/QNetworkReplyHandler.cpp: (WebCore::QNetworkReplyHandler::abort): (WebCore::QNetworkReplyHandler::release):
12:15 PM Changeset in webkit [49950] by pfeldman@chromium.org
  • 4 edits in trunk/WebCore

2009-10-22 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Timothy Hatcher.

Web Inspector: Fix DOMAgent leak.

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

  • inspector/InspectorController.cpp: (WebCore::InspectorController::~InspectorController): (WebCore::InspectorController::setFrontendProxyObject): (WebCore::InspectorController::close): (WebCore::InspectorController::releaseDOMAgent):
  • inspector/InspectorController.h:
  • inspector/InspectorDOMAgent.h: (WebCore::InspectorDOMAgent::create):
11:35 AM Changeset in webkit [49949] by Dimitri Glazkov
  • 19 edits in trunk/WebCore

2009-10-22 Dimitri Glazkov <Dimitri Glazkov>

Reviewed by Adam Barth.

[V8] Rework event listeners to not hold references to frame or V8 context.

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

Covered by existing layout tests: fast/events/attribute-listener*

  • bindings/scripts/CodeGeneratorV8.pm: Added passing ScriptExecutionContext*

to event listener handling code.

  • bindings/v8/DateExtension.cpp: (WebCore::DateExtension::setAllowSleep): Changed to use currentContext().
  • bindings/v8/ScriptEventListener.cpp: (WebCore::createAttributeEventListener): Reworked to match JSC logic. (WebCore::getEventListenerHandlerBody): Added ScriptExecutionContext* param.
  • bindings/v8/V8AbstractEventListener.cpp: (WebCore::V8AbstractEventListener::V8AbstractEventListener): Removed Frame* param

and usage.

(WebCore::V8AbstractEventListener::handleEvent): Chaged to use ScriptExecutionContext*.
(WebCore::V8AbstractEventListener::invokeEventHandler): Ditto.

  • bindings/v8/V8AbstractEventListener.h: (WebCore::V8AbstractEventListener::getListenerObject): Ditto. (WebCore::V8AbstractEventListener::prepareListenerObject): Ditto.
  • bindings/v8/V8DOMWrapper.cpp: (WebCore::V8DOMWrapper::convertEventListenerToV8Object): Added ScriptExecutionContext* param. (WebCore::V8DOMWrapper::getEventListener): Ditto.
  • bindings/v8/V8DOMWrapper.h: (WebCore::V8DOMWrapper::convertEventListenerToV8Object): Ditto.
  • bindings/v8/V8EventListenerList.h: (WebCore::V8EventListenerList::findOrCreateWrapper): Removed ContextType* template param,

because it's no longer needed.

  • bindings/v8/V8IsolatedWorld.h: (WebCore::V8IsolatedWorld::sharedContext): Renamed from shared_context.
  • bindings/v8/V8LazyEventListener.cpp: (WebCore::V8LazyEventListener::V8LazyEventListener): Removed Frame* param and usage. (WebCore::V8LazyEventListener::callListenerFunction): Added ScriptExecutionContext* param. (WebCore::V8LazyEventListener::prepareListenerObject): Ditto.
  • bindings/v8/V8LazyEventListener.h: (WebCore::V8LazyEventListener::create): Reordered params to match JSC impl.
  • bindings/v8/V8Proxy.cpp: (WebCore::V8Proxy::V8Proxy): Adjusted formatting to match WebKit style. (WebCore::V8Proxy::evaluateInNewContext): Changed to use m_context directly. (WebCore::V8Proxy::setInjectedScriptContextDebugId): Ditto. (WebCore::V8Proxy::createWrapperFromCacheSlowCase): Ditto. (WebCore::V8Proxy::isContextInitialized): Ditto. (WebCore::V8Proxy::updateDocumentWrapperCache): Ditto. (WebCore::V8Proxy::clearDocumentWrapperCache): Ditto. (WebCore::V8Proxy::disposeContextHandles): Added explicit disposing of m_context. (WebCore::V8Proxy::clearForClose): Changed to use m_context directly. (WebCore::V8Proxy::clearForNavigation): Ditto. (WebCore::V8Proxy::setSecurityToken): Ditto. (WebCore::V8Proxy::updateDocument): Ditto. (WebCore::V8Proxy::initContextIfNeeded): Ditto. (WebCore::V8Proxy::context): Changed to use v8::Local. (WebCore::V8Proxy::mainWorldContext): Changed to use m_context directly. (WebCore::V8Proxy::setContextDebugId): Ditto. (WebCore::toV8Context):
  • bindings/v8/V8Proxy.h: Removed shared_context decl, changed to use straight

v8::Persistent for m_context.

  • bindings/v8/V8WorkerContextEventListener.cpp: (WebCore::workerProxy): Added. (WebCore::V8WorkerContextEventListener::V8WorkerContextEventListener): Removed

WorkerContextExecutionProxy* param.

(WebCore::V8WorkerContextEventListener::handleEvent): Started using ScriptExecutionContext*.
(WebCore::V8WorkerContextEventListener::reportError): Ditto.
(WebCore::V8WorkerContextEventListener::callListenerFunction): Ditto.
(WebCore::V8WorkerContextEventListener::getReceiverObject): Ditto.

  • bindings/v8/V8WorkerContextEventListener.h: (WebCore::V8WorkerContextEventListener::create): Removed

WorkerContextExecutionProxy* param.

  • bindings/v8/WorkerContextExecutionProxy.cpp: (WebCore::WorkerContextExecutionProxy::findOrCreateEventListener): Removed ContextType*

template param.

  • bindings/v8/custom/V8CustomEventListener.cpp: (WebCore::V8EventListener::V8EventListener): Removed Frame* param. (WebCore::V8EventListener::getListenerFunction): Started using ScriptExecutionContext*. (WebCore::V8EventListener::callListenerFunction): Ditto.
  • bindings/v8/custom/V8CustomEventListener.h: (WebCore::V8EventListener::create): Removed Frame* param.
11:10 AM Changeset in webkit [49948] by eric.carlson@apple.com
  • 4 edits in trunk/WebKit/mac

2009-10-22 Eric Carlson <eric.carlson@apple.com>

Reviewed by Dan Bernstein & Simon Fraser.

<rdar://problem/7297717>
https://bugs.webkit.org/show_bug.cgi?id=30323
Play/pause button in full-screen video controller does not reflect current state

  • WebView/WebVideoFullscreenController.mm: (-[WebVideoFullscreenController setMediaElement:WebCore::]): Register for QTMovieRateDidChangeNotification (-[WebVideoFullscreenController rateChanged:]): Call HUD controller when rate changes.
  • WebView/WebVideoFullscreenHUDWindowController.h:
  • WebView/WebVideoFullscreenHUDWindowController.mm: (-[WebVideoFullscreenHUDWindowController scheduleTimeUpdate]): Call updateRate so play button reflects playback state. (-[WebVideoFullscreenHUDWindowController updateRate]): New, set play button state according to movie state. (-[WebVideoFullscreenHUDWindowController playingChanged:]): Only call [self playing] once.
10:49 AM Changeset in webkit [49947] by darin@chromium.org
  • 5 edits in trunk/WebCore

2009-10-14 Gaurav Shah <gauravsh@google.com>

Reviewed by Darin Fisher.

Replaces temporary link stub for <keygen> tag handler for the Chromium
browser with a call via the Chromium Bridge.

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

  • platform/SSLKeyGenerator.h:
  • platform/chromium/ChromiumBridge.h:
  • platform/chromium/SSLKeyGeneratorChromium.cpp: (WebCore::getSupportedKeySizes): (WebCore::signedPublicKeyAndChallengeString):
  • platform/chromium/TemporaryLinkStubs.cpp: (WebCore::KURL::fileSystemPath): (WebCore::SharedBuffer::createWithContentsOfFile):
10:48 AM Changeset in webkit [49946] by atwilson@chromium.org
  • 3 edits in trunk/WebCore

Fixes Chromium Mac pasteboard handling to flow through the same code paths as other platforms.

Patch by Avi Drissman <avi@chromium.org> on 2009-10-22
Reviewed by Eric Seidel.

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

  • platform/chromium/ChromiumBridge.h:
  • platform/chromium/PasteboardChromium.cpp:

(WebCore::Pasteboard::writeImage):

10:33 AM Changeset in webkit [49945] by Simon Fraser
  • 6 edits in trunk/WebCore

2009-10-20 Simon Fraser <Simon Fraser>

Reviewed by Darin Adler.

Make accelerated compositing debug indicators work in release builds
https://bugs.webkit.org/show_bug.cgi?id=30588

Allow some debug indicators, that show which page elements go into compositing layers,
to work in release builds, when the runtime prefs are set.

  • platform/graphics/GraphicsLayer.cpp: (WebCore::GraphicsLayer::GraphicsLayer): (WebCore::GraphicsLayer::setZPosition):
  • platform/graphics/GraphicsLayer.h:
  • platform/graphics/mac/GraphicsLayerCA.h:
  • platform/graphics/mac/GraphicsLayerCA.mm: (WebCore::clearBorderColor): (WebCore::GraphicsLayer::showRepaintCounter): (WebCore::GraphicsLayerCA::GraphicsLayerCA): (WebCore::GraphicsLayerCA::updateMasksToBounds): (WebCore::GraphicsLayerCA::updateLayerDrawsContent): (WebCore::GraphicsLayerCA::setDebugBorder): (WebCore::GraphicsLayerCA::swapFromOrToTiledLayer): (WebCore::GraphicsLayerCA::setupContentsLayer):
  • platform/graphics/mac/WebLayer.mm:
10:30 AM Changeset in webkit [49944] by xan@webkit.org
  • 2 edits in trunk/LayoutTests

2009-10-22 Xan Lopez <xlopez@igalia.com>

Skip failing tests that seems to be somewhat tied to the Mac
platform. See https://bugs.webkit.org/show_bug.cgi?id=30680

  • platform/gtk/Skipped:
9:42 AM Changeset in webkit [49943] by tonikitoo@webkit.org
  • 2 edits in trunk/WebKit/qt

Code standarlization for QGVLauncher.

Patch by Antonio Gomes <tonikitoo@webkit.org> on 2009-10-22
RubberStamped by Tor Arne Vestbø.

1) Made member initilization lists in constructors

to be per line.

2) Made applyProxy method inline as all other methods in

WebPage class.

  • QGVLauncher/main.cpp:

(WebPage::WebPage):
(WebPage::applyProxy):
(MainView::MainView):
(MainWindow::MainWindow):
(MainWindow::init):

9:34 AM Changeset in webkit [49942] by tonikitoo@webkit.org
  • 2 edits in trunk/WebKit/qt

Add a Y-Axis rotation to QGVLauncher.

Patch by Antonio Gomes <tonikitoo@webkit.org> on 2009-10-22
Reviewed by Tor Arne Vestbø.

It uses the QStateMachine API from Qt 4.6.

  • QGVLauncher/main.cpp:

(WebView::WebView):
(WebView::setYRotation):
(WebView::yRotation):
(MainView::flip):
(MainView::animatedYFlip):
(SharedScene::SharedScene):
(SharedScene::webView):
(MainWindow::init):
(MainWindow::animatedYFlip):
(MainWindow::buildUI):

8:53 AM Changeset in webkit [49941] by eric@webkit.org
  • 2 edits in trunk/LayoutTests

2009-10-22 Andrew Scherkus <scherkus@chromium.org>

Reviewed by Eric Carlson.

Fix video-played.js to use an error timeout based on time rather than function call count.

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

  • media/video-played.js: (nowInSecs): Returns the current time in seconds. (playForMillisecs.callPauseIfTimeIsReached): Change from call counts to using elapsed time. (playForMillisecs): Ditto.
8:45 AM Changeset in webkit [49940] by eric@webkit.org
  • 3 edits in trunk/WebCore

2009-10-22 Keishi Hattori <casey.hattori@gmail.com>

Reviewed by Pavel Feldman.

Inspector should support monitorEvents/un monitorEvents() in the command line
https://bugs.webkit.org/show_bug.cgi?id=19879

  • inspector/front-end/EventListenersSidebarPane.js: (WebInspector.EventListenersSidebarPane.prototype.update.callback): Ignores event listeners generated by monitorEvent
  • inspector/front-end/InjectedScript.js: (InjectedScript._ensureCommandLineAPIInstalled): Added _inspectorCommandLineAPI._logEvent, _allEventTypes, _normalizeEventTypes, monitorEvent, unmonitorEvent.
8:28 AM Changeset in webkit [49939] by Joseph Pecoraro
  • 2 edits in trunk/WebCore

2009-10-21 Joseph Pecoraro <Joseph Pecoraro>

Reviewed by Pavel Feldman.

Web Inspector: windowFocused and windowBlurred Fail to Clear/Mark Inspector as "inactive"
https://bugs.webkit.org/show_bug.cgi?id=30663

8:23 AM Changeset in webkit [49938] by Joseph Pecoraro
  • 2 edits in trunk/WebCore

2009-10-22 Joseph Pecoraro <Joseph Pecoraro>

Reviewed by Pavel Feldman.

Web Inspector: Error - requestContentType [undefined] is not an object
https://bugs.webkit.org/show_bug.cgi?id=30666

  • inspector/front-end/ResourceView.js: (WebInspector.ResourceView.prototype._refreshFormData):
4:52 AM Changeset in webkit [49937] by eric@webkit.org
  • 2 edits in trunk/WebCore

2009-10-22 Maxime Simon <Maxime Simon>

Reviewed by Holger Freyther.

[Haiku] Correction of the accented letter width (they were 2 characters long).
https://bugs.webkit.org/show_bug.cgi?id=30629

  • platform/graphics/haiku/SimpleFontDataHaiku.cpp: (WebCore::SimpleFontData::platformWidthForGlyph):
3:49 AM Changeset in webkit [49936] by eric@webkit.org
  • 2 edits in trunk/WebKit/gtk

2009-10-22 Alejandro G. Castro <alex@igalia.com>

Reviewed by Xan Lopez.

m_webView should be private attribute of the EditorClientGtk.
https://bugs.webkit.org/show_bug.cgi?id=30664

  • WebKit/gtk/WebCoreSupport/EditorClientGtk.h:
3:39 AM Changeset in webkit [49935] by xan@webkit.org
  • 2 edits in trunk/LayoutTests

2009-10-22 Xan Lopez <xlopez@igalia.com>

libsoup can reconstruct URIs with empty querys now, so unskip
another bunch of tests.

  • platform/gtk/Skipped:
3:24 AM Changeset in webkit [49934] by xan@webkit.org
  • 2 edits in trunk/LayoutTests

2009-10-22 Xan Lopez <xlopez@igalia.com>

Unskip a couple of working tests.

  • platform/gtk/Skipped:
12:15 AM Changeset in webkit [49933] by jhoneycutt@apple.com
  • 6 edits in trunk

<rdar://problem/7270320> Screenshots of off-screen plug-ins are blank
<rdar://problem/7270314> After halting a transparent PluginView on
Windows, the transparency is applied twice

JavaScriptCore:

Reviewed by Dan Bernstein.

Export WTF::deleteOwnedPtr(HDC).

Ditto.

WebCore:

Replace use of Frame::nodeImage() with a function that takes a snapshot
of a PluginView.

Reviewed by Dan Bernstein.

  • plugins/PluginView.h:

Made paintWindowedPluginIntoContext() non-const, as it now calls
paintIntoTransformedContext(). Declare paintIntoTransformedContext()
and snapshot() for Windows platforms.

  • plugins/win/PluginViewWin.cpp:

(WebCore::PluginView::paintIntoTransformedContext):
Paints into the passed HDC without applying any coordinate translations.
Code moved from paintWindowedPluginIntoContext() and paint(). Removed
the memset() of windowpos in lieu of assignment.
(WebCore::PluginView::paintWindowedPluginIntoContext):
Code moved to paintIntoTransformedContext().
(WebCore::PluginView::paint):
Ditto.
(WebCore::PluginView::snapshot):
Create a context, and if the plug-in is windowless, translate it so the
plug-in will draw at the correct location. Create a bitmap, and select
it into the context. Paint the plug-in, and create a BitmapImage from
the bitmap.
(WebCore::PluginView::halt):
Use snapshot().

12:11 AM Changeset in webkit [49932] by eric@webkit.org
  • 2 edits in trunk/WebCore

2009-10-22 Fumitoshi Ukai <ukai@chromium.org>

Reviewed by Alexey Proskuryakov.

Fix calculation of length in WebSocketChannel.cpp
https://bugs.webkit.org/show_bug.cgi?id=30656

  • websockets/WebSocketChannel.cpp: (WebCore::WebSocketChannel::didReceiveData):

Oct 21, 2009:

11:20 PM Changeset in webkit [49931] by eric@webkit.org
  • 3 edits in trunk/WebKitTools

2009-10-21 Eric Seidel <eric@webkit.org>

Reviewed by Adam Barth.

bugzilla-tool's "patch failed to download an apply" error should give more information
https://bugs.webkit.org/show_bug.cgi?id=30632

  • Scripts/modules/scm.py:
    • Use the common run_command method instead of custom POpen code.
    • Make run_command know how to take pipes as input.
  • Scripts/modules/scm_unittest.py:
    • Add new tests to cover change.
    • Also move test_error_handlers into new SCMClassTests so we don't run it 3 times.
9:32 PM Changeset in webkit [49930] by tkent@chromium.org
  • 2 edits in trunk/WebKitTools

committers

4:57 PM Changeset in webkit [49929] by dumi@chromium.org
  • 2 edits in trunk/WebCore

Adding the ability to enable/disable functions in V8 at runtime.

Patch by Dumitru Daniliuc <dumi@chromium.org> on 2009-10-21
Reviewed by Dimitri Glazkov.

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

  • bindings/scripts/CodeGeneratorV8.pm:
4:47 PM Changeset in webkit [49928] by jorlow@chromium.org
  • 6 edits
    11 adds
    21 deletes in trunk/LayoutTests

Combine local and session storage tests into one and use script-tests properly
https://bugs.webkit.org/show_bug.cgi?id=30640

Patch by Jeremy Orlow <jorlow@chromium.org> on 2009-10-21
Reviewed by Darin Adler.

I was apparently misusing the script-tests stuff. I changed all the tests to
test both local storage and session storage so that we can use the
make-test-script-wrappers stuff as is.

  • storage/domstorage/clear-expected.txt: Added.
  • storage/domstorage/clear.html: Added.
  • storage/domstorage/complex-keys-expected.txt: Added.
  • storage/domstorage/complex-keys.html: Added.
  • storage/domstorage/complex-values-expected.txt: Added.
  • storage/domstorage/complex-values.html: Added.
  • storage/domstorage/localstorage/clear-expected.txt: Removed.
  • storage/domstorage/localstorage/clear.html: Removed.
  • storage/domstorage/localstorage/complex-keys-expected.txt: Removed.
  • storage/domstorage/localstorage/complex-keys.html: Removed.
  • storage/domstorage/localstorage/complex-values-expected.txt: Removed.
  • storage/domstorage/localstorage/complex-values.html: Removed.
  • storage/domstorage/localstorage/quota-expected.txt: Removed.
  • storage/domstorage/localstorage/quota.html: Removed.
  • storage/domstorage/localstorage/remove-item-expected.txt: Removed.
  • storage/domstorage/localstorage/remove-item.html: Removed.
  • storage/domstorage/quota-expected.txt: Added.
  • storage/domstorage/quota.html: Added.
  • storage/domstorage/remove-item-expected.txt: Added.
  • storage/domstorage/remove-item.html: Added.
  • storage/domstorage/script-tests/TEMPLATE.html: Added.
  • storage/domstorage/script-tests/clear.js:

(test):
(runTest):

  • storage/domstorage/script-tests/complex-keys.js:

(test):
(runTest):

  • storage/domstorage/script-tests/complex-values.js:

(test):
(runTest):

  • storage/domstorage/script-tests/no-quota.js: Removed.
  • storage/domstorage/script-tests/quota.js:

(testQuota):
(testNoQuota):
(runTest):

  • storage/domstorage/script-tests/remove-item.js:

(test):
(runTest):

  • storage/domstorage/sessionstorage/clear-expected.txt: Removed.
  • storage/domstorage/sessionstorage/clear.html: Removed.
  • storage/domstorage/sessionstorage/complex-keys-expected.txt: Removed.
  • storage/domstorage/sessionstorage/complex-keys.html: Removed.
  • storage/domstorage/sessionstorage/complex-values-expected.txt: Removed.
  • storage/domstorage/sessionstorage/complex-values.html: Removed.
  • storage/domstorage/sessionstorage/no-quota-expected.txt: Removed.
  • storage/domstorage/sessionstorage/no-quota.html: Removed.
  • storage/domstorage/sessionstorage/remove-item-expected.txt: Removed.
  • storage/domstorage/sessionstorage/remove-item.html: Removed.
4:39 PM Changeset in webkit [49927] by kevino@webkit.org
  • 5 edits in trunk/WebKitTools

Reviewed by Kevin Ollivier.

Update the Windows installer builder to work with Vista / Win 7 and with git.

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

4:10 PM Changeset in webkit [49926] by Darin Adler
  • 4 edits in trunk

WebCore: Swedish search (and other languages as well) is broken while fixing Japanese search
https://bugs.webkit.org/show_bug.cgi?id=30646

Patch by Darin Adler <Darin Adler> on 2009-10-21
Rolled Japanese tailoring out that was done to fix
https://bugs.webkit.org/show_bug.cgi?id=30437 earlier.
It was overriding the locale-specific tailoring.
We'll land a fix once we figure out how to add the
Japanese tailoring without removing anything.

  • editing/TextIterator.cpp: Rolled out r49924.

LayoutTests: Update since we rolled Japanese tailoring out that was done to fix
https://bugs.webkit.org/show_bug.cgi?id=30437 earlier.

Patch by Darin Adler <Darin Adler> on 2009-10-21

  • fast/text/international/japanese-kana-letters-expected.txt: Update

test results to expect failure. We'll land results that expect
success once we re-fix the bug.

4:06 PM Changeset in webkit [49925] by eric@webkit.org
  • 3 edits in trunk/LayoutTests

2009-10-21 Andrew Scherkus <scherkus@chromium.org>

Reviewed by Eric Seidel.

Update more media layout tests to use findMediaFile().

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

  • media/controls-drag-timebar.html: Switch to using findMediaFile().
  • media/video-currentTime-set.html: Remove src attribute and rely on findMediaFile() instead.
3:16 PM Changeset in webkit [49924] by bweinstein@apple.com
  • 6 edits
    3 deletes in trunk

Roll out r49922 because it broke Tiger and SL in strange ways.

2:59 PM Changeset in webkit [49923] by pfeldman@chromium.org
  • 2 edits in trunk/WebCore

2009-10-21 Pavel Feldman <pfeldman@chromium.org>

Not reviewed: chromium build fix - add missing image to gypi.

  • WebCore.gypi:
2:51 PM Changeset in webkit [49922] by bweinstein@apple.com
  • 8 edits
    3 adds in trunk

WebCore: Fixes part of <http://webkit.org/b/30522>.
Web Inspector: DOM Exceptions throughout the Inspector should be more human readable.

Reviewed by Eric Seidel.

Expose the description attribute that is now a member of
ExceptionBase as of r49723, so when a user logs an exception, they
can see the description in the Web Inspector.

Test: fast/dom/dom-exception-description.html

  • dom/DOMCoreException.idl:
  • dom/EventException.idl:
  • dom/RangeException.idl:
  • svg/SVGException.idl:
  • xml/XMLHttpRequestException.idl:
  • xml/XPathException.idl:

LayoutTests: Fixes part of <http://webkit.org/b/30522>.
Web Inspector: DOM Exceptions throughout the Inspector should be more human readable.

Reviewed by Eric Seidel.

Adds a test to check the name and description of some DOM Exceptions.

  • fast/dom/dom-exception-description-expected.txt: Added.
  • fast/dom/dom-exception-description.html: Added.
  • fast/dom/script-tests/dom-exception-description.js: Added.
1:47 PM Changeset in webkit [49921] by eric@webkit.org
  • 2 edits in trunk/WebKitTools

2009-10-21 Alejandro G. Castro <alex@igalia.com>

Reviewed by Gustavo Noronha.

[GTK] Added conditional code to avoid using
gdk_window_get_root_coords if we do not have a gtk+ release newer
than 2.17.3.
https://bugs.webkit.org/show_bug.cgi?id=30636

  • WebKitTools/DumpRenderTree/gtk/EventSender.cpp:
1:28 PM Writing Layout Tests for DumpRenderTree edited by jorlow@chromium.org
(diff)
1:22 PM WebKit Team edited by tkent@chromium.org
(diff)
1:02 PM Changeset in webkit [49920] by sullivan@apple.com
  • 2 edits in trunk/WebKit/mac

Rubber-stamped by Dan Bernstein.

  • WebView/WebDynamicScrollBarsView.h:

Corrected a couple of long-obsolete comments regarding Safari's use of this class.

12:53 PM Changeset in webkit [49919] by bweinstein@apple.com
  • 2 edits in trunk/WebCore

Fixes <https://bugs.webkit.org/show_bug.cgi?id=30616>.
REGRESSION (r49036): Web Inspector: Summary graph no longer switching to size summary.

Reviewed by Timothy Hatcher.

Add a check to make sure a function we need to calculate percentages is
defined before we use it.

  • inspector/front-end/ResourcesPanel.js:

(WebInspector.ResourcesPanel.prototype._updateGraphDividersIfNeeded):

12:43 PM Changeset in webkit [49918] by eric@webkit.org
  • 4 edits in trunk/LayoutTests

2009-10-21 Alejandro G. Castro <alex@igalia.com>

Reviewed by Xan Lopez.

[GTK] Removed 5333725.html from Skipped and modified the expected
result of 5232159-expected.txt and fake-drag-expected.txt, this
modifications come from the changes in EventSender (bug 30633 and
bug 30636).
https://bugs.webkit.org/show_bug.cgi?id=30593

  • LayoutTests/platform/gtk/Skipped:
  • LayoutTests/platform/gtk/editing/selection/5232159-expected.txt:
  • LayoutTests/platform/gtk/editing/selection/fake-drag-expected.tx :
12:36 PM Changeset in webkit [49917] by eric@webkit.org
  • 3 edits in trunk/WebCore

2009-10-21 Keishi Hattori <casey.hattori@gmail.com>

Reviewed by Timothy Hatcher.

REGRESSION: Clicking on nodes in the console should take you to the element in the DOM
https://bugs.webkit.org/show_bug.cgi?id=27231

  • inspector/front-end/ConsoleView.js: (WebInspector.ConsoleView.prototype._formatnode): Turns on ElementsTreeOutline.showInElementsPanelEnabled
  • inspector/front-end/ElementsTreeOutline.js: (WebInspector.ElementsTreeOutline): Added ElementsTreeOutline.showInElementsPanelEnabled (WebInspector.ElementsTreeElement.prototype.onmousedown): Reveals the node in the Elements panel if treeOutline.showInElementsPanelEnabled is true.
12:22 PM Changeset in webkit [49916] by jorlow@chromium.org
  • 2 edits in trunk/WebCore

https://bugs.webkit.org/show_bug.cgi?id=30637
Fix a compiler warning in windows.

Patch by Jeremy Orlow <jorlow@chromium.org> on 2009-10-21
Reviewed by Eric Seidel.

  • loader/ResourceLoadNotifier.h:
12:18 PM Changeset in webkit [49915] by eric@webkit.org
  • 3 edits in trunk/WebKitTools

2009-10-21 Shu Chang <Chang.Shu@nokia.com>

Reviewed by Eric Seidel.

[Qt] Added dummy implementation for keepWebHistory()
https://bugs.webkit.org/show_bug.cgi?id=30592

  • DumpRenderTree/qt/LayoutTestControllerQt.cpp: (LayoutTestController::keepWebHistory):
  • DumpRenderTree/qt/LayoutTestControllerQt.h:
12:10 PM Changeset in webkit [49914] by eric@webkit.org
  • 2 edits in trunk/LayoutTests

2009-10-21 Victor Wang <victorw@chromium.org>

Reviewed by Eric Seidel.

Fix click-focus-anchor layout test to mimic click event.

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

  • fast/events/click-focus-anchor.html:
12:03 PM Changeset in webkit [49913] by eric@webkit.org
  • 2 edits in trunk/WebKitTools

2009-10-21 Alejandro G. Castro <alex@igalia.com>

Reviewed by Xan Lopez.

[GTK] Fixed the double click condition, it is not double click if
we move in just in one direction.
https://bugs.webkit.org/show_bug.cgi?id=30636

  • WebKitTools/DumpRenderTree/gtk/EventSender.cpp:
11:55 AM Changeset in webkit [49912] by eric@webkit.org
  • 2 edits in trunk/WebKitTools

2009-10-21 Alejandro G. Castro <alex@igalia.com>

Reviewed by Xan Lopez.

[GTK] Initialize the events completly before emitting them.
https://bugs.webkit.org/show_bug.cgi?id=30633

  • WebKitTools/DumpRenderTree/gtk/EventSender.cpp:
11:32 AM Changeset in webkit [49911] by dimich@chromium.org
  • 3 edits
    2 adds in trunk

Fix the crash when a node not in a document receives dispatchEvent.
https://bugs.webkit.org/show_bug.cgi?id=30611

Reviewed by Darin Adler.

WebCore:

Test: fast/events/dispatch-event-no-document.html

  • dom/EventTarget.cpp: Check for scriptExecutionContext() at the moment of dispatchEvent; do nothing if no context.

(WebCore::EventTarget::dispatchEvent):

LayoutTests:

  • fast/events/dispatch-event-no-document-expected.txt: Added.
  • fast/events/dispatch-event-no-document.html: Added.
11:25 AM Changeset in webkit [49910] by kevino@webkit.org
  • 2 edits in trunk/WebKitTools

wx build fix. Fix for when linking using --as-needed with gcc.

11:23 AM Changeset in webkit [49909] by agl@chromium.org
  • 2 edits in trunk/WebCore

2009-10-21 Adam Langley <agl@google.com>

Reviewed by Eric Seidel.

Chromium Linux: disable subpixel text on layers.

https://bugs.webkit.org/show_bug.cgi?id=30635
http://code.google.com/p/chromium/issues/detail?id=25365

With the addition of layers for drawing rounded corners in r49641,
subpixel text on rounded rectangles broke. This is because the layer
only contains a single alpha channel and this is insufficient to
compose subpixel text correctly.

On Windows, a large body of code in TransparencyWin.cpp exists to try
to deal with this. Even then, in some cases, it downgrades to
anti-aliased text. We need a fix for the grevious effects quickly thus
this patch disables subpixel text when rendering into a layer.

This would be covered by existing tests except that subpixel text is
disabled for pixel tests on Chromium Linux.

  • platform/graphics/chromium/FontLinux.cpp: (WebCore::isCanvasMultiLayered): (WebCore::adjustTextRenderMode): (WebCore::Font::drawGlyphs): (WebCore::Font::drawComplexText):
11:13 AM Changeset in webkit [49908] by kevino@webkit.org
  • 2 edits in trunk/WebKit/wx

Reviewed by Kevin Ollivier.

Include 'WebFrame.h' declared classes in wxPython bindings.

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

11:06 AM Changeset in webkit [49907] by kevino@webkit.org
  • 5 edits in trunk

wxMac build fixes. Fix issues with linking against wrong versions of sqlite3 and curl on Mac 10.4.

9:47 AM Changeset in webkit [49906] by mitz@apple.com
  • 4 edits
    1 copy
    4 adds in trunk

WebCore: Fixed background-clip parsing regressions introduced in r46240.

Reviewed by Sam Weinig.

Test: fast/css/background-clip-text.html

  • css/CSSParser.cpp:

(WebCore::parseBackgroundClip): Added this helper method.
(WebCore::CSSParser::parseFillShorthand): Use parseBackgroundClip() to
reparse the value. The old code called parseFillProperty() again, which
didn’t work, because parseFillProperty() advances the value list.
(WebCore::CSSParser::parseFillProperty): Changed to use
parseBackgroundClip().

  • css/CSSPrimitiveValueMappings.h:

(WebCore::CSSPrimitiveValue::operator EFillBox): Added a case for
CSSValueWebkitText.

LayoutTests: Added a background-clip parsing test that would have caught some
regressions that were introduced in r46240.

Reviewed by Sam Weinig.

  • fast/css/background-clip-text-expected.txt: Added.
  • fast/css/background-clip-text.html: Added.
  • fast/css/script-tests: Added.
  • fast/css/script-tests/TEMPLATE.html: Copied from LayoutTests/fast/backgrounds/repeat/script-tests/TEMPLATE.html.
  • fast/css/script-tests/background-clip-text.js: Added.

(test):

6:57 AM Changeset in webkit [49905] by eric@webkit.org
  • 3 edits in trunk/WebCore

2009-10-21 Alexander Pavlov <apavlov@chromium.org>

Reviewed by Timothy Hatcher.

Enable creation of custom SidebarTreeElements for different ProfileTypes

ProfileTypes can now create sidebar tree elements of custom types.
https://bugs.webkit.org/show_bug.cgi?id=30520

  • inspector/front-end/ProfileView.js: (WebInspector.CPUProfileType.prototype.setRecordingProfile): (WebInspector.CPUProfileType.prototype.createSidebarTreeElementForProfile): (WebInspector.CPUProfileType.prototype.createView): (WebInspector.CPUProfile.prototype.get head):
  • inspector/front-end/ProfilesPanel.js: (WebInspector.ProfileType.prototype.buttonClicked): (WebInspector.ProfileType.prototype.viewForProfile): (WebInspector.ProfileType.prototype.createView): (WebInspector.ProfileType.prototype.createSidebarTreeElementForProfile): (WebInspector.ProfilesPanel.prototype.addProfileHeader): (WebInspector.ProfilesPanel.prototype.showProfile):
6:45 AM Changeset in webkit [49904] by Joseph Pecoraro
  • 4 edits in trunk/WebCore

2009-10-20 Joseph Pecoraro <Joseph Pecoraro>

Reviewed by Timothy Hatcher.

Web Inspector: Selectors in the Styles pane should trigger a search for that selector when clicked
https://bugs.webkit.org/show_bug.cgi?id=17126

  • inspector/front-end/ElementsPanel.js: (WebInspector.ElementsPanel.prototype.addNodesToSearchResult): only change focusedDOMNode on a manual search
  • inspector/front-end/StylesSidebarPane.js: (WebInspector.StylePropertiesSection.prototype._clickSelector): trigger a search
  • inspector/front-end/ElementsTreeOutline.js: (WebInspector.ElementsTreeElement.prototype._startEditing): correctly start editing attribute value instead of name
6:33 AM Changeset in webkit [49903] by pfeldman@chromium.org
  • 2 edits in trunk/WebCore

2009-10-20 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Timothy Hatcher.

Web Inspector: Fix DOM Agent leak.

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

  • inspector/InspectorController.cpp: (WebCore::InspectorController::~InspectorController):
6:08 AM Changeset in webkit [49902] by eric@webkit.org
  • 1 edit
    3 adds in trunk/WebCore

2009-10-21 Girish Ramakrishnan <Girish Ramakrishnan>

Reviewed by Holger Freyther.

Plugins: Create manual tests for testing plugins

Create two test files:

  1. windowed.html - Test plugin in windowed mode.
  2. windowless.html - Test plugin in windowless mode.

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

  • manual-tests/plugins/windowed.html: Added.
  • manual-tests/plugins/windowless.html: Added.
2:10 AM Changeset in webkit [49901] by xan@webkit.org
  • 2 edits in trunk/LayoutTests

2009-10-21 Xan Lopez <xlopez@igalia.com>

Skip another failing test.

  • platform/gtk/Skipped:
12:52 AM Changeset in webkit [49900] by xan@webkit.org
  • 2 edits in trunk/LayoutTests

2009-10-21 Xan Lopez <xlopez@igalia.com>

Skip a few more libxml2 related tests.

  • platform/gtk/Skipped:
12:11 AM Changeset in webkit [49899] by eric@webkit.org
  • 3 edits
    2 adds in trunk

2009-10-21 Satoshi Nakagawa <psychs@limechat.net>

Reviewed by Darin Adler.

Fixed Japanese text search problems.
Treat small kana letters and kana letters as different characters in search.
Do not ignore diacritic marks in search for Japanese texts.

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

  • fast/text/international/japanese-kana-letters-expected.txt: Added.
  • fast/text/international/japanese-kana-letters.html: Added.

2009-10-21 Satoshi Nakagawa <psychs@limechat.net>

Reviewed by Darin Adler.

Fixed Japanese text search problems.
Treat small kana letters and kana letters as different characters in search.
Do not ignore diacritic marks in search for Japanese texts.

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

Test: fast/text/international/japanese-kana-letters.html

  • editing/TextIterator.cpp: (WebCore::createCollator): (WebCore::collator): (WebCore::createSearcher):

Oct 20, 2009:

9:56 PM Changeset in webkit [49898] by eric@webkit.org
  • 3 edits in trunk/WebCore

2009-10-20 Eric Z. Ayers <zundel@google.com>

Reviewed by Timothy Hatcher.

Changes TimelineItems to be created with raw timestamps as opposed
to a time relative to a start of session. Normalized timestamps cause
problems when monitoring a browsing session across multiple page
transitions.

  • inspector/InspectorTimelineAgent.cpp: (WebCore::InspectorTimelineAgent::InspectorTimelineAgent): (WebCore::InspectorTimelineAgent::willDispatchDOMEvent): (WebCore::InspectorTimelineAgent::willLayout): (WebCore::InspectorTimelineAgent::willRecalculateStyle): (WebCore::InspectorTimelineAgent::willPaint): (WebCore::InspectorTimelineAgent::willWriteHTML): (WebCore::InspectorTimelineAgent::reset): (WebCore::InspectorTimelineAgent::didCompleteCurrentRecord):
  • inspector/InspectorTimelineAgent.h:
8:37 PM Changeset in webkit [49897] by pfeldman@chromium.org
  • 7 edits in trunk/LayoutTests

2009-10-20 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Timothy Hatcher.

Web Inspector: simple console tests do not require testing harness.

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

  • inspector/uncaught-dom1-exception-expected.txt:
  • inspector/uncaught-dom1-exception.html:
  • inspector/uncaught-dom3-exception-expected.txt:
  • inspector/uncaught-dom3-exception.html:
  • inspector/uncaught-dom8-exception-expected.txt:
  • inspector/uncaught-dom8-exception.html:
7:31 PM Changeset in webkit [49896] by mrowe@apple.com
  • 2 edits in trunk/WebCore

Fix leak of WebSocketChannel by adopting the newly-created reference.

Reviewed by Oliver Hunt.

  • websockets/WebSocketChannel.h:

(WebCore::WebSocketChannel::create):

5:42 PM Changeset in webkit [49895] by mrowe@apple.com
  • 3 edits
    2 deletes in trunk

Roll out r49876 as it causes many assertion failures during regression tests on SnowLeopard.

5:40 PM Changeset in webkit [49894] by jorlow@chromium.org
  • 6 edits in trunk/WebCore

DOM Storage runtime flag changes
https://bugs.webkit.org/show_bug.cgi?id=30602

Patch by Jeremy Orlow <jorlow@chromium.org> on 2009-10-20
Reviewed by Adam Barth.

Part 1/2. Removing sessionStorageEnabled in next patch after the
Chromium side of the changes lands.

Revert my changes to Settings and instead implement DOM Storage enabling via
the methods agreed upon in https://bugs.webkit.org/show_bug.cgi?id=30240

This stuff was (intentionally) never exposed to web pages or DRT, so there's no
LayoutTest visible changes and thus no tests.

  • bindings/v8/custom/V8CustomBinding.h:
  • bindings/v8/custom/V8DOMWindowCustom.cpp:

(WebCore::ACCESSOR_RUNTIME_ENABLER):

  • page/DOMWindow.idl:
  • storage/Storage.cpp:

(WebCore::Storage::setLocalStorageAvailable):
(WebCore::Storage::localStorageAvailable):
(WebCore::Storage::setSessionStorageAvailable):
(WebCore::Storage::sessionStorageAvailable):

  • storage/Storage.h:
5:22 PM Changeset in webkit [49893] by atwilson@chromium.org
  • 8 edits in trunk/WebCore

Need to turn off notifications properly at runtime
https://bugs.webkit.org/show_bug.cgi?id=30409

Patch by John Gregg <johnnyg@google.com> on 2009-10-20
Reviewed by David Levin.

This code only affects chromium, and is all behind a compile time
flag current turned off, so no new tests.

  • bindings/v8/custom/V8CustomBinding.h:
  • bindings/v8/custom/V8DOMWindowCustom.cpp:

(WebCore::ACCESSOR_RUNTIME_ENABLER):

  • bindings/v8/custom/V8WorkerContextCustom.cpp:

(WebCore::ACCESSOR_RUNTIME_ENABLER):

  • notifications/NotificationCenter.cpp:

(WebCore::NotificationCenter::setIsAvailable):
(WebCore::NotificationCenter::isAvailable):

  • notifications/NotificationCenter.h:
  • page/DOMWindow.idl:
  • workers/WorkerContext.idl:
5:08 PM Changeset in webkit [49892] by eric@webkit.org
  • 3 edits in trunk/WebCore

2009-10-20 James Robinson <jamesr@chromium.org>

Reviewed by Adam Barth.

Fixes RefPtr initialization in the V8 implementation of WebCore::ScriptString to use the ::create() idiom and
use adoptRef() properly. I failed to read the RefPtr docs the first time through :(

No new tests, error was caught by valgrind on the Chromium builders.

  • bindings/v8/ScriptString.h: (WebCore::ScriptString::ScriptString): (WebCore::ScriptString::operator=):
  • bindings/v8/ScriptStringImpl.h: (WebCore::ScriptStringImpl::create):
4:10 PM Changeset in webkit [49891] by eric@webkit.org
  • 4 edits in trunk/WebCore

2009-10-20 Mikhail Naganov <mnaganov@chromium.org>

Reviewed by Pavel Feldman.

Web Inspector: populate child nodes before sorting them.

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

  • inspector/front-end/BottomUpProfileDataGridTree.js: (WebInspector.BottomUpProfileDataGridNode): Swapped with BottomUpProfileDataGridTree to be consistent with TopDownProfileDataGridNode. (WebInspector.BottomUpProfileDataGridNode.prototype._takePropertiesFromProfileDataGridNode): (WebInspector.BottomUpProfileDataGridNode.prototype._keepOnlyChild): (WebInspector.BottomUpProfileDataGridNode.prototype._exclude): (WebInspector.BottomUpProfileDataGridNode.prototype._merge): (WebInspector.BottomUpProfileDataGridNode.prototype._sharedPopulate): (WebInspector.BottomUpProfileDataGridTree.prototype.exclude):
  • inspector/front-end/ProfileDataGridTree.js: (WebInspector.ProfileDataGridNode.prototype.sort): Added missing parentheses. (WebInspector.ProfileDataGridNode.prototype.get _parent): (WebInspector.ProfileDataGridNode.prototype._populate):
  • inspector/front-end/TopDownProfileDataGridTree.js: (WebInspector.TopDownProfileDataGridNode.prototype._sharedPopulate):
2:56 PM Changeset in webkit [49890] by jorlow@chromium.org
  • 2 edits in trunk/WebCore

Remove redundant String ref/deref calls in generated V8 bindings.
https://bugs.webkit.org/show_bug.cgi?id=30579

Patch by Jens Alfke <snej@chromium.org> on 2009-10-20
Reviewed by Eric Seidel.

  • bindings/v8/V8Binding.h:

(WebCore::toString): Fix return type of 'toString' to make it truly a no-op

instead of constructing/destructing a String.

2:56 PM Changeset in webkit [49889] by ggaren@apple.com
  • 2 edits in trunk/JavaScriptCore

Windows build fix: updated variable name.

Patch by Geoffrey Garen <ggaren@apple.com> on 2009-10-20

  • runtime/DatePrototype.cpp:

(JSC::formatLocaleDate):

2:55 PM Changeset in webkit [49888] by ggaren@apple.com
  • 2 edits in trunk/JavaScriptCore
  • jit/JITOpcodes.cpp:

(JSC::JIT::emit_op_next_pname): Slightly tweaked this #ifdef to match the
size of a JSValue because m_jsStrings is an array of JSValues.

Reviewed by Mark Rowe.

2:49 PM Changeset in webkit [49887] by ggaren@apple.com
  • 2 edits in trunk/JavaScriptCore

Fixed a 64-bit regression caused by the fix for
https://bugs.webkit.org/show_bug.cgi?id=30570.

Patch by Geoffrey Garen <ggaren@apple.com> on 2009-10-20
Reviewed by Mark Rowe.

  • jit/JITOpcodes.cpp:

(JSC::JIT::emit_op_next_pname): Use TimesEight stepping on 64-bit, since
64-bit pointers are eight bytes long.

2:39 PM Changeset in webkit [49886] by ggaren@apple.com
  • 4 edits in trunk/JavaScriptCore

Refactored DateInstance::msToGregorianDateTime so that a DateInstance's
caller doesn't need to supply the DateInstance's own internal value to
the DateInstance.

Patch by Geoffrey Garen <ggaren@apple.com> on 2009-10-20
Reviewed by Sam Weinig.

  • runtime/DateInstance.cpp:

(JSC::DateInstance::getGregorianDateTime): Renamed from "msToGregorianDateTime".

  • runtime/DateInstance.h:
  • runtime/DatePrototype.cpp:

(JSC::formatLocaleDate):
(JSC::dateProtoFuncToString):
(JSC::dateProtoFuncToUTCString):
(JSC::dateProtoFuncToISOString):
(JSC::dateProtoFuncToDateString):
(JSC::dateProtoFuncToTimeString):
(JSC::dateProtoFuncToLocaleString):
(JSC::dateProtoFuncToLocaleDateString):
(JSC::dateProtoFuncToLocaleTimeString):
(JSC::dateProtoFuncGetTime):
(JSC::dateProtoFuncGetFullYear):
(JSC::dateProtoFuncGetUTCFullYear):
(JSC::dateProtoFuncToGMTString):
(JSC::dateProtoFuncGetMonth):
(JSC::dateProtoFuncGetUTCMonth):
(JSC::dateProtoFuncGetDate):
(JSC::dateProtoFuncGetUTCDate):
(JSC::dateProtoFuncGetDay):
(JSC::dateProtoFuncGetUTCDay):
(JSC::dateProtoFuncGetHours):
(JSC::dateProtoFuncGetUTCHours):
(JSC::dateProtoFuncGetMinutes):
(JSC::dateProtoFuncGetUTCMinutes):
(JSC::dateProtoFuncGetSeconds):
(JSC::dateProtoFuncGetUTCSeconds):
(JSC::dateProtoFuncGetTimezoneOffset):
(JSC::setNewValueFromTimeArgs):
(JSC::setNewValueFromDateArgs):
(JSC::dateProtoFuncSetYear):
(JSC::dateProtoFuncGetYear): Also renamed "utc" to "outputIsUTC", for clarity.

2:20 PM Changeset in webkit [49885] by eric@webkit.org
  • 2 edits in trunk/WebCore

2009-10-20 Joanmarie Diggs <joanmarie.diggs@gmail.com>

Reviewed by Xan Lopez.

https://bugs.webkit.org/show_bug.cgi?id=25411
[GTK] ATK accessible ancestry broken

Work around for the problem of bogus additional objects in the ancestry.
We now set the parent when we ref the child, then ask the Atk child if
it knows its parent. This solves the bulk of the cases. For those it
doesn't, fall back to the existing logic.

  • accessibility/gtk/AccessibilityObjectWrapperAtk.cpp: (webkit_accessible_get_parent): (webkit_accessible_ref_child):
1:56 PM Changeset in webkit [49884] by mitz@apple.com
  • 2 edits in trunk/WebKit/mac

Rubber-stamped by Mark Rowe.

<rdar://problem/7319749> REGRESSION(r49806): Contextual Menu items missing
https://bugs.webkit.org/show_bug.cgi?id=30568

Revert accidental change from r49806.

  • WebCoreSupport/WebContextMenuClient.mm:

(WebContextMenuClient::getCustomMenuFromDefaultItems):

1:50 PM Changeset in webkit [49883] by bweinstein@apple.com
  • 2 edits in trunk/WebCore

Fixes <http://webkit.org/b/30421>.
Web Inpsector: Local Files can show up on Cookies List with Cookies from other Domain.

Reviewed by Alice Liu.

When a page like Google generates an about:blank, it shows up as a local file on
the list of Cookie domains, but has all of the cookies of Google. When we are adding
cookie domains to the inspector, we should only add the domain is the URL of the resource
is in the HTTP protocol family or a file URL.

  • inspector/InspectorController.cpp:

(WebCore::InspectorController::populateScriptObjects):
(WebCore::InspectorController::didFinishLoading):

1:38 PM Changeset in webkit [49882] by Nate Chapin
  • 2 edits
    3 adds in trunk/WebCore

2009-10-20 Mark Mentovai <mark@chromium.org>

Reviewed by Darin Fisher.

Use a version of libWebKitSystemInterface with global symbols marked
private_extern for Chromium Mac.

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

  • WebCore.gyp/WebCore.gyp:
  • WebCore.gyp/mac: Added.
  • WebCore.gyp/mac/Empty.cpp: Added.
  • WebCore.gyp/mac/adjust_visibility.sh: Added.
1:12 PM Changeset in webkit [49881] by zoltan@webkit.org
  • 2 edits in trunk/WebCore

2009-10-20 Zoltan Horvath <zoltan@webkit.org>

Reviewed by Eric Seidel.

Allow custom memory allocation control for WebCore's CSSProperty
https://bugs.webkit.org/show_bug.cgi?id=30564

Inherits CSSProperty class from FastAllocBase because it has been
instantiated by 'new' in WebCore/css/CSSParser.cpp:367.

  • css/CSSProperty.h:
1:03 PM Changeset in webkit [49880] by xan@webkit.org
  • 2 edits in trunk/LayoutTests

2009-10-20 Xan Lopez <xlopez@igalia.com>

Another batch of skipped tests with links to their bugs.

  • platform/gtk/Skipped:
12:52 PM Changeset in webkit [49879] by kenneth@webkit.org
  • 2 edits in trunk/WebKit/qt

Reviewed By Adam Barth.

Patch by Kenneth Rohde Christiansen <kenneth@webkit.org> on 2009-10-20
Add some actions to the menu for cursor debugging.

GraphicsView based launcher only.

  • QGVLauncher/main.cpp:

(MainView::setWaitCursor):
(MainView::resetCursor):
(MainView::flip):
(MainWindow::setWaitCursor):
(MainWindow::resetCursor):
(MainWindow::buildUI):

12:08 PM Changeset in webkit [49878] by eric@webkit.org
  • 2 edits in trunk/WebCore

2009-10-20 Andrew Scherkus <scherkus@chromium.org>

Reviewed by Eric Seidel.

Hide Chromium's media slider thumb if no source has been loaded.

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

Covered by existing layout tests, notably:

LayoutTests/media/controls-rendering.html
LayoutTests/media/unsupported-tracks.html
LayoutTests/media/video-src-none.html

  • rendering/RenderMediaControlsChromium.cpp: (WebCore::hasSource): Returns true if an HTMLMediaElement has a valid source set. (WebCore::paintMediaMuteButton): Refactored to use hasSource(). (WebCore::paintMediaPlayButton): Ditto. (WebCore::paintMediaSliderThumb): Add call to hasSource() to determine if we should paint the thumb.
12:02 PM Changeset in webkit [49877] by xan@webkit.org
  • 2 edits in trunk/LayoutTests

2009-10-20 Xan Lopez <xlopez@igalia.com>

Another batch of skipped tests with links to their bugs.

  • platform/gtk/Skipped:
12:00 PM Changeset in webkit [49876] by eric@webkit.org
  • 3 edits
    2 adds in trunk

2009-10-20 Satoshi Nakagawa <psychs@limechat.net>

Reviewed by Darin Adler.

Fixed Japanese text search problems.
Treat small kana letters and kana letters as different characters in search.
Do not ignore diacritic marks in search for Japanese texts.

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

  • fast/text/international/japanese-kana-letters-expected.txt: Added.
  • fast/text/international/japanese-kana-letters.html: Added.

2009-10-20 Satoshi Nakagawa <psychs@limechat.net>

Reviewed by Darin Adler.

Fixed Japanese text search problems.
Treat small kana letters and kana letters as different characters in search.
Do not ignore diacritic marks in search for Japanese texts.

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

Test: fast/text/international/japanese-kana-letters.html

  • editing/TextIterator.cpp: (WebCore::createCollator): (WebCore::collator): (WebCore::createSearcher):
11:46 AM Changeset in webkit [49875] by hamaji@chromium.org
  • 1 edit
    2 adds
    4 deletes in trunk/LayoutTests

2009-10-20 Shinichiro Hamaji <hamaji@chromium.org>

Reviewed by Darin Adler.

fast/css/getComputedStyle/marginComputedStyle is missing
https://bugs.webkit.org/show_bug.cgi?id=30566

  • fast/css/getComputedStyle/marginComputedStyle-expected.txt: Added.
  • fast/css/getComputedStyle/marginComputedStyle.html: Added.
  • platform/mac/fast/css/getComputedStyle/marginComputedStyle-expected.checksum: Removed.
  • platform/mac/fast/css/getComputedStyle/marginComputedStyle-expected.png: Removed.
  • platform/mac/fast/css/getComputedStyle/marginComputedStyle-expected.txt: Removed.
  • platform/qt/fast/css/marginComputedStyle-expected.txt: Removed.
11:41 AM Changeset in webkit [49874] by eric@webkit.org
  • 2 edits in trunk/JavaScriptCore

2009-10-20 Gabor Loki <loki@inf.u-szeged.hu>

Reviewed by Geoffrey Garen.

The op_next_pname should use 4 bytes addressing mode in case of JSValue32
https://bugs.webkit.org/show_bug.cgi?id=30570

  • jit/JITOpcodes.cpp: (JSC::JIT::emit_op_next_pname):
11:20 AM Changeset in webkit [49873] by xan@webkit.org
  • 2 edits in trunk/LayoutTests

2009-10-20 Xan Lopez <xlopez@igalia.com>

Another batch of skipped tests with links to their bugs.

  • platform/gtk/Skipped:
11:17 AM Changeset in webkit [49872] by Nate Chapin
  • 2 edits in trunk/WebCore

2009-10-20 Nate Chapin <Nate Chapin>

Reviewed by Adam Barth.

Added getter for FrameLoader:m_suppressOpenerInNewFrame.
This will allow Chromium to more intelligently detect
noreferrer links and therefore open them in a new process.

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

  • loader/FrameLoader.h: (WebCore::FrameLoader::suppressOpenerInNewFrame): Added.
10:46 AM Changeset in webkit [49871] by abarth@webkit.org
  • 2 edits in trunk/WebKitTools

2009-10-20 Anton Muhin <antonm@chromium.org>

Reviewed by Adam Barth.

Add {ager,antonm,yurys}@chromium.org into committers.py
https://bugs.webkit.org/show_bug.cgi?id=30560

  • Scripts/modules/committers.py:
10:45 AM Changeset in webkit [49870] by kenneth@webkit.org
  • 4 edits in trunk/WebKit/qt

Remove clipRenderToViewport as agreed upon in
https://bugs.webkit.org/show_bug.cgi?id=29843

Patch by Kenneth Rohde Christiansen <kenneth@webkit.org> on 2009-10-20
Rubberstamped by Adam Barth.

  • Api/qwebframe.cpp:
  • Api/qwebframe.h:
  • Api/qwebframe_p.h:

(QWebFramePrivate::QWebFramePrivate):

10:37 AM Changeset in webkit [49869] by xan@webkit.org
  • 2 edits in trunk/LayoutTests

2009-10-20 Xan Lopez <xlopez@igalia.com>

Skip 3 failing xmlhttprequest tests.

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

  • platform/gtk/Skipped:
10:22 AM Changeset in webkit [49868] by xan@webkit.org
  • 2 edits in trunk/LayoutTests

2009-10-20 Xan Lopez <xlopez@igalia.com>

Skip two failing security canvas tests.

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

  • platform/gtk/Skipped:
10:10 AM Changeset in webkit [49867] by xan@webkit.org
  • 2 edits in trunk/LayoutTests

2009-10-20 Xan Lopez <xlopez@igalia.com>

Skip failing editing test.

  • platform/gtk/Skipped:
10:03 AM Changeset in webkit [49866] by kenneth@webkit.org
  • 2 edits in trunk/WebKit/qt

Update the tests to test the new render functionality, and take
into consideration that render() clips to the frame itself as well
as the viewport.

Patch by Kenneth Rohde Christiansen <kenneth@webkit.org> on 2009-10-20
Reviewed by Adam Barth.

QWebFrame::render() now always clips, so the old tests were bogus.

Rendering pure contents (no scrollbars etc) without clipping can now
be accomplished using QWebFrame::documentElement()->render(...)

  • Api/qwebframe.cpp:
  • Api/qwebframe.h:
  • Api/qwebframe_p.h:

(QWebFramePrivate::QWebFramePrivate):

  • tests/qwebframe/tst_qwebframe.cpp:
10:03 AM Changeset in webkit [49865] by kenneth@webkit.org
  • 2 edits in trunk/WebKit/qt

As we do not support rendering a QWebFrame without it being clipped
the the frame as well as the viewport, we now set the viewport size
to the size of the contents.

Patch by Kenneth Rohde Christiansen <kenneth@webkit.org> on 2009-10-20
Reviewed by Adam Barth.

Rendering pure contents (no scrollbars etc) without clipping can be
acomplished using QWebFrame::documentElement()->render(...)

  • tests/qwebelement/tst_qwebelement.cpp:

(tst_QWebElement::render):

9:43 AM Changeset in webkit [49864] by xan@webkit.org
  • 2 edits in trunk/LayoutTests

2009-10-20 Xan Lopez <xlopez@igalia.com>

Skip a bunch of failing DnD tests.

  • platform/gtk/Skipped:
9:40 AM Changeset in webkit [49863] by zoltan@webkit.org
  • 2 edits in trunk/WebCore

2009-10-20 Zoltan Horvath <zoltan@webkit.org>

Reviewed by Darin Adler.

Allow custom memory allocation control for WebCore's CSSParserFunction
https://bugs.webkit.org/show_bug.cgi?id=30563

Inherits CSSParserFunction struct from FastAllocBase because it has been
instantiated by 'new' in WebCore/css/CSSParser.cpp:4827.

  • css/CSSProperty.h:
9:29 AM Changeset in webkit [49862] by xan@webkit.org
  • 2 edits in trunk/LayoutTests

2009-10-20 Xan Lopez <xlopez@igalia.com>

Skip two zoom related tests.

  • platform/gtk/Skipped:
8:55 AM Changeset in webkit [49861] by xan@webkit.org
  • 2 edits in trunk/LayoutTests

2009-10-20 Xan Lopez <xlopez@igalia.com>

Skip a couple of failing media tests.

  • platform/gtk/Skipped:
8:47 AM Changeset in webkit [49860] by xan@webkit.org
  • 2 edits in trunk/LayoutTests

2009-10-20 Xan Lopez <xlopez@igalia.com>

Skip failing a11y test while we investigate what's going on.

  • platform/gtk/Skipped:
8:41 AM Changeset in webkit [49859] by xan@webkit.org
  • 2 edits in trunk/LayoutTests

2009-10-20 Xan Lopez <xlopez@igalia.com>

Skip tests expecting new results.

  • platform/gtk/Skipped:
8:30 AM Changeset in webkit [49858] by Joseph Pecoraro
  • 2 edits in trunk/WebCore

2009-10-20 Joseph Pecoraro <Joseph Pecoraro>

Reviewed by Timothy Hatcher.

Web Inspector: Watched Expressions Buttons Do Not Match Inspector Styles
https://bugs.webkit.org/show_bug.cgi?id=30554

  • inspector/front-end/inspector.css: added style to pane buttons
8:29 AM Changeset in webkit [49857] by xan@webkit.org
  • 2 edits in trunk/LayoutTests

2009-10-20 Xan Lopez <xlopez@igalia.com>

Skip media/restore-from-page-cache.html since we are missing some
of the needed API for the test.

  • platform/gtk/Skipped:
8:06 AM Changeset in webkit [49856] by yurys@chromium.org
  • 3 edits in trunk/WebCore

=fix for 30559

8:00 AM Changeset in webkit [49855] by xan@webkit.org
  • 2 edits in trunk/LayoutTests

2009-10-20 Xan Lopez <xlopez@igalia.com>

Skip another failing test because we don't currently support
changing the locale after gtk_init has been called.

Also, add another test to the list of libxml2 issues.

  • platform/gtk/Skipped:
7:25 AM Changeset in webkit [49854] by Csaba Osztrogonác
  • 2 edits in trunk/LayoutTests

2009-10-20 Csaba Osztrogonác <Csaba Osztrogonác>

Reviewed by Tor Arne Vestbø.

[Qt] Remove http/tests/xmlhttprequest tests
from skiplist, and add only now failing tests.

  • platform/qt/Skipped:
7:18 AM Changeset in webkit [49853] by xan@webkit.org
  • 2 edits in trunk/LayoutTests

2009-10-20 Xan Lopez <xlopez@igalia.com>

Skip a bunch of tests failing in the debug bots because WebKit
does not work correctly with very recent versions of libxml2. The
tracker bug for the issue is
https://bugs.webkit.org/show_bug.cgi?id=30508

  • platform/gtk/Skipped:
7:15 AM Changeset in webkit [49852] by kenneth@webkit.org
  • 2 edits in trunk/WebKitTools

Make the Netscape Test plugin available to the Qt launcher.

Patch by Kenneth Rohde Christiansen <kenneth@webkit.org> on 2009-10-20
Reviewed by Tor Arne Vestbø

  • Scripts/run-launcher:
7:14 AM Changeset in webkit [49851] by kenneth@webkit.org
  • 2 edits in trunk/WebKit/qt

Add menu item to dump the plugin list to the console,
which can be handy for debugging.

Patch by Kenneth Rohde Christiansen <kenneth@webkit.org> on 2009-10-20
Reviewed by Tor Arne Vestbø.

  • QtLauncher/main.cpp:

(MainWindow::dumpPlugins):
(MainWindow::setupUI):

7:14 AM Changeset in webkit [49850] by kenneth@webkit.org
  • 4 edits in trunk/WebKit/qt

Introduce new render method on QWebFrame, which supports specifying
which layers to render (scrollbars, contents, pan-icon).

Patch by Kenneth Rohde Christiansen <kenneth@webkit.org> on 2009-10-19
Reviewed by Tor Arne Vestbø.

Deprecate render().

  • Api/qwebframe.cpp:

(QWebFramePrivate::renderPrivate):
(QWebFrame::render):

  • Api/qwebframe.h:
  • Api/qwebframe_p.h:
5:16 AM Changeset in webkit [49849] by Csaba Osztrogonác
  • 2 edits in trunk/LayoutTests

[Qt] Enable previously disabled fast/css tests after r48873 landed which
fixed https://bugs.webkit.org/show_bug.cgi?id=28288

Patch by Andras Becsi <becsi.andras@stud.u-szeged.hu> on 2009-10-20
Reviewed by Ariya Hidayat.

  • platform/qt/Skipped:
4:18 AM Changeset in webkit [49848] by xan@webkit.org
  • 2 edits in trunk/LayoutTests

2009-10-20 Xan Lopez <xlopez@igalia.com>

Skip failing test, with comment pointing to the bug to track it.

  • platform/gtk/Skipped:
3:44 AM Changeset in webkit [49847] by xan@webkit.org
  • 2 edits in trunk/LayoutTests

2009-10-20 Xan Lopez <xlopez@igalia.com>

Skip plugins/private-browsing-mode.html, since our plugin
implementation lacks that feature.

  • platform/gtk/Skipped:
2:54 AM Changeset in webkit [49846] by tonikitoo@webkit.org
  • 3 edits in trunk/WebKit/qt

[Qt] Infinite loop (leading to crash) when setting cursor in QGraphicsWebView
https://bugs.webkit.org/show_bug.cgi?id=30549

Patch by Antonio Gomes <tonikitoo@webkit.org> on 2009-10-19
Reviewed by Ariya Hidayat.

Patch reimplements QGraphicsItem's itemChange method, and make
CursorChange event to be emitted after cursor has already been
set.

QWidget::setCursor send the event just after it sets the cursor,
then patch makes both behaviors compatible.

  • Api/qgraphicswebview.cpp:

(QGraphicsWebView::itemChange):

  • Api/qgraphicswebview.h:
2:42 AM Changeset in webkit [49845] by Csaba Osztrogonác
  • 3 edits in trunk/JavaScriptCore

Move OverridesMarkChildren flag from DatePrototype to its parent class
https://bugs.webkit.org/show_bug.cgi?id=30372

Patch by Gabor Loki <loki@inf.u-szeged.hu> on 2009-10-20
Reviewed by Oliver Hunt.

  • runtime/DateInstance.h:

(JSC::DateInstance::createStructure):

  • runtime/DatePrototype.h:
1:52 AM Changeset in webkit [49844] by Csaba Osztrogonác
  • 2 edits in trunk/WebCore

Unreviewed buildfix.
[Qt] Add FEGaussianBlur.h and FEGaussianBlur.cpp which were refactored in r49778 to WebCore.pro.

Patch by Andras Becsi <becsi.andras@stud.u-szeged.hu> on 2009-10-20

  • WebCore.pro:
12:36 AM Changeset in webkit [49843] by ukai@chromium.org
  • 27 edits in trunk

2009-10-20 Fumitoshi Ukai <ukai@chromium.org>

Reviewed by David Levin.

Set EnabledAtRuntime for WebSocket in DOMWindow
https://bugs.webkit.org/show_bug.cgi?id=29896

Supported by chromium/v8 only.
Add WebSocket::isAvailable()/setIsAvailable(bool) to control v8
bindings.
Remove Settings::experimentalWebSocketsEnabled() and
setExperimentalWebSocketsEnabled(bool).

  • WebCore.base.exp:
  • bindings/js/JSDOMWindowCustom.cpp: (WebCore::JSDOMWindow::webSocket):
  • bindings/v8/V8DOMWrapper.cpp: (WebCore::V8DOMWrapper::getConstructor):
  • bindings/v8/custom/V8CustomBinding.h:
  • bindings/v8/custom/V8DOMWindowCustom.cpp: (WebCore::ACCESSOR_RUNTIME_ENABLER):
  • page/DOMWindow.idl:
  • page/Settings.cpp: (WebCore::Settings::Settings):
  • page/Settings.h:
  • websockets/WebSocket.cpp: (WebCore::WebSocket::setIsAvailable): (WebCore::WebSocket::isAvailable):
  • websockets/WebSocket.h:

2009-10-20 Fumitoshi Ukai <ukai@chromium.org>

Reviewed by David Levin.

Removed WebSocket runtime settings.
https://bugs.webkit.org/show_bug.cgi?id=29896

WebSocket runtime configuration is supported by chromium/v8 only.

  • webkit/webkitwebview.cpp: (webkit_web_view_update_settings): (webkit_web_view_settings_notify):

2009-10-20 Fumitoshi Ukai <ukai@chromium.org>

Reviewed by David Levin.

Removed WebSocket runtime settings.
https://bugs.webkit.org/show_bug.cgi?id=29896

WebSocket runtime configuration is supported by chromium/v8 only.

  • WebView/WebPreferenceKeysPrivate.h:
  • WebView/WebPreferences.mm: (+[WebPreferences initialize]):
  • WebView/WebPreferencesPrivate.h:
  • WebView/WebView.mm: (-[WebView _preferencesChangedNotification:]):

2009-10-20 Fumitoshi Ukai <ukai@chromium.org>

Reviewed by David Levin.

Removed WebSocket runtime settings.
https://bugs.webkit.org/show_bug.cgi?id=29896

WebSocket runtime configuration is supported by chromium/v8 only.

  • Interfaces/IWebPreferencesPrivate.idl:
  • WebPreferenceKeysPrivate.h:
  • WebPreferences.cpp: (WebPreferences::initializeDefaultSettings): (WebPreferences::experimentalNotificationsEnabled):
  • WebPreferences.h:
  • WebView.cpp: (WebView::notifyPreferencesChanged):

2009-10-20 Fumitoshi Ukai <ukai@chromium.org>

Reviewed by David Levin.

Removed WebSocket runtime settings.
https://bugs.webkit.org/show_bug.cgi?id=29896

WebSocket runtime configuration is supported by chromium/v8 only.

  • DumpRenderTree/mac/DumpRenderTree.mm: (resetDefaultsToConsistentValues):
  • DumpRenderTree/win/DumpRenderTree.cpp: (resetDefaultsToConsistentValues):
12:23 AM Changeset in webkit [49842] by Csaba Osztrogonác
  • 2 edits in trunk/LayoutTests

Rubber-stamped by Tor Arne Vestbø.

Patch by Andras Becsi <becsi.andras@stud.u-szeged.hu> on 2009-10-20
[Qt] Skip failing tests introduced by disabling Phonon support in Buildbot's Qt version
until a proper solution for the Phonon related craches and timeouts is found.

  • platform/qt/Skipped:
Note: See TracTimeline for information about the timeline view.