Timeline



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:

Oct 19, 2009:

11:56 PM Changeset in webkit [49841] by oliver@apple.com
  • 2 edits in trunk/WebCore

REGRESSION: Dromaeo DOM test is 14% slower
https://bugs.webkit.org/show_bug.cgi?id=30273

Reviewed by Sam Weinig.

Whoops, make prototype bindings actually use the StructureFlags.

10:01 PM Changeset in webkit [49840] by abarth@webkit.org
  • 4 edits
    2 copies in trunk/WebCore

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

Reviewed by Adam Barth.

Better implementation of WebCore::ScriptString for the V8 bindings.

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

WebCore::ScriptString is used for XMLHttpRequest's responseText attribute which is
shared with JavaScript. Thus, simply using a WebCore::String and copying the value
is pretty inefficient, especially since responseText is built up with a sequence of
operator+= calls. JSC builds use a JSC::UString to share the buffer when possible,
this patch adopts a similar approach for V8.

No new tests, behavior is unchanged and covered by LayoutTests/http/tests/xmlhttprequest

  • WebCore.gypi:
  • bindings/v8/ScriptString.h: (WebCore::ScriptString::ScriptString): (WebCore::ScriptString::operator String): (WebCore::ScriptString::isNull): (WebCore::ScriptString::size): (WebCore::ScriptString::operator=): (WebCore::ScriptString::operator+=): (WebCore::ScriptString::v8StringOrNull):
  • bindings/v8/ScriptStringImpl.cpp: Added. (WebCore::ScriptStringImpl::ScriptStringImpl): (WebCore::ScriptStringImpl::~ScriptStringImpl): (WebCore::ScriptStringImpl::toString): (WebCore::ScriptStringImpl::isNull): (WebCore::ScriptStringImpl::size): (WebCore::ScriptStringImpl::append):
  • bindings/v8/ScriptStringImpl.h: Added. (WebCore::ScriptStringImpl::ScriptStringImpl): (WebCore::ScriptStringImpl::v8StringHandle):
  • bindings/v8/custom/V8XMLHttpRequestCustom.cpp: (WebCore::ACCESSOR_GETTER):
9:46 PM Changeset in webkit [49839] by abarth@webkit.org
  • 3 edits
    1 delete in trunk/WebCore

2009-10-19 Adam Barth <abarth@webkit.org>

No review, rolling out r49837.
http://trac.webkit.org/changeset/49837

  • page/Settings.cpp: (WebCore::Settings::setStandardFontFamily): (WebCore::Settings::setFixedFontFamily): (WebCore::Settings::setSerifFontFamily): (WebCore::Settings::setSansSerifFontFamily): (WebCore::Settings::setCursiveFontFamily): (WebCore::Settings::setFantasyFontFamily):
  • page/Settings.h: (WebCore::Settings::standardFontFamily): (WebCore::Settings::fixedFontFamily): (WebCore::Settings::serifFontFamily): (WebCore::Settings::sansSerifFontFamily): (WebCore::Settings::cursiveFontFamily): (WebCore::Settings::fantasyFontFamily):
  • platform/text/UScriptCode.h: Removed.
9:39 PM Changeset in webkit [49838] by abarth@webkit.org
  • 2 edits in trunk/WebCore

2009-10-19 Kwang Yul Seo <skyul@company100.net>

Reviewed by Eric Seidel.

Check COMPILER(MSVC) instead of PLATFORM(WIN_OS) to avoid the use of vasprintf.
https://bugs.webkit.org/show_bug.cgi?id=30473

vasprintf is missing in MSVC. Use COMPILER(MSVC) guards instead of
PLALTFORM(WIN_OS) guards.

  • dom/XMLTokenizerLibxml2.cpp: (WebCore::XMLTokenizer::error):
9:37 PM Changeset in webkit [49837] by abarth@webkit.org
  • 3 edits
    1 add in trunk/WebCore

2009-10-19 Jungshik Shin <jshin@chromium.org>

Reviewed by Eric Seidel.

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

Make generic font family getters/setters accept an additional
argument (script code). It has a default value so that if an embedder
does not have/want a per-script font family setting, call-sites
don't have to be changed.
This is to prepare for fixing bug 10874 (font selection is not
language-dependent) and bug 18085.

There should be no change in layout and no new layout test
is added.

  • WebCore.base.exp:
  • page/Settings.cpp:
  • page/Settings.h:
  • platform/text/UScriptCode.h: Added. This is for ports that do not use ICU. the part of ICU's common/unicode/uscript.h that defines script code enum was copied. To keep enums compatible with those in ICU, we don't generate the list out of Scripts.txt of the Unicode Data base or CLDR's data.
9:28 PM Changeset in webkit [49836] by abarth@webkit.org
  • 2 edits in trunk/WebCore

2009-10-19 Evan Stade <estade@chromium.org>

Reviewed by Darin Adler.

Clarify usage of SuddenTermination API. No code change.

  • platform/SuddenTermination.h:
9:05 PM Changeset in webkit [49835] by oliver@apple.com
  • 10 edits in trunk/WebCore

REGRESSION: Dromaeo DOM test is 14% slower
https://bugs.webkit.org/show_bug.cgi?id=30273

Reviewed by Gavin Barraclough.

Make DOM bindings automatically inherit correct structure
flags rather than being needlessly conservative. This is
done by making the bindings generator use the same model
for TypeInfo flags that we now use in JSC.

This gains us about 1% of this regression back.

9:01 PM Changeset in webkit [49834] by abarth@webkit.org
  • 2 edits in trunk/WebKit

2009-10-19 John Gregg <johnnyg@google.com>

Reviewed by Dimitri Glazkov.

set ENABLE_NOTIFICATIONS=1 for Chromium
https://bugs.webkit.org/show_bug.cgi?id=29949

  • chromium/features.gypi:
8:19 PM Changeset in webkit [49833] by eric@webkit.org
  • 3 edits
    2 adds in trunk

2009-10-19 Robin Qiu <robin.qiu@torchmobile.com.cn>

Reviewed by Nikolas Zimmermann.

Add a layout test for nested SVG <use> element which has a child.
https://bugs.webkit.org/show_bug.cgi?id=26117

  • svg/custom/use-on-use-with-child-expected.txt: Added.
  • svg/custom/use-on-use-with-child.svg: Added.

2009-10-19 Robin Qiu <robin.qiu@torchmobile.com.cn>

Reviewed by Nikolas Zimmermann.

Fixed a bug on nested SVG <use> elements.
https://bugs.webkit.org/show_bug.cgi?id=26117
When a <use> element refer to another <use> element which has
child/children, the instance tree built for this <use> element
is incorrect (more nodes than expected).

Test: svg/dom/use-on-use-with-child.svg

  • svg/SVGUseElement.cpp: (WebCore::SVGUseElement::buildInstanceTree):
7:59 PM Changeset in webkit [49832] by timothy@apple.com
  • 2 edits in trunk/WebCore

Small changes to fully invalidate and update the JavaScriptCallFrame.

Patch by Oliver Hunt <oliver@apple.com> on 2009-10-19
<rdar://problem/7020755> JSDebugger crashes after reloading from a breakpoint
https://bugs.webkit.org/show_bug.cgi?id=27146

Reviewed by Timothy Hatcher.

  • inspector/JavaScriptCallFrame.h:

(WebCore::JavaScriptCallFrame::invalidate):
(WebCore::JavaScriptCallFrame::update):

7:51 PM Changeset in webkit [49831] by jorlow@chromium.org
  • 3 edits in trunk/LayoutTests

2009-10-19 Jeremy Orlow <jorlow@chromium.org>

Reviewed by Adam Barth.

Fix fallout from last change.
https://bugs.webkit.org/show_bug.cgi?id=30550

Update init-events to match reality. I only ran the domstorage layout tests
and thus missed one in fast/events/. This addresses fallout from
https://bugs.webkit.org/show_bug.cgi?id=30536

  • fast/events/init-events-expected.txt:
  • fast/events/script-tests/init-events.js:
6:32 PM Changeset in webkit [49830] by jorlow@chromium.org
  • 17 edits in trunk

2009-10-19 Jeremy Orlow <jorlow@chromium.org>

Reviewed by Dmitri Titov.

Remove "source" from storage events
https://bugs.webkit.org/show_bug.cgi?id=30536

Remove "source" from storage events per
http://lists.whatwg.org/htdig.cgi/whatwg-whatwg.org/2009-October/023703.html

This was removed because it makes it introduces synchronous access that can
cross the event loop boundry (since a storage event can fire from one process
and be handled in another).

  • inspector/InspectorDOMStorageResource.cpp: (WebCore::InspectorDOMStorageResource::handleEvent):
  • storage/StorageEvent.cpp: (WebCore::StorageEvent::create): (WebCore::StorageEvent::StorageEvent): (WebCore::StorageEvent::initStorageEvent):
  • storage/StorageEvent.h: (WebCore::StorageEvent::uri):
  • storage/StorageEvent.idl:
  • storage/StorageEventDispatcher.cpp: (WebCore::StorageEventDispatcher::dispatch):

2009-10-19 Jeremy Orlow <jorlow@chromium.org>

Reviewed by Dmitri Titov.

Remove "source" from storage events
https://bugs.webkit.org/show_bug.cgi?id=30536

Remove "source" from storage events per
http://lists.whatwg.org/htdig.cgi/whatwg-whatwg.org/2009-October/023703.html

  • storage/domstorage/localstorage/iframe-events-expected.txt:
  • storage/domstorage/localstorage/iframe-events.html:
  • storage/domstorage/localstorage/resources/iframe-events-second.html:
  • storage/domstorage/localstorage/simple-events-expected.txt:
  • storage/domstorage/localstorage/simple-events.html:
  • storage/domstorage/sessionstorage/iframe-events-expected.txt:
  • storage/domstorage/sessionstorage/iframe-events.html:
  • storage/domstorage/sessionstorage/resources/iframe-events-second.html:
  • storage/domstorage/sessionstorage/simple-events-expected.txt:
  • storage/domstorage/sessionstorage/simple-events.html:
6:29 PM Changeset in webkit [49829] by abarth@webkit.org
  • 5 edits in trunk/WebCore

2009-10-19 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.

Remove ResourceLoadNotifier::activeDocumentLoader
https://bugs.webkit.org/show_bug.cgi?id=30533

Removing this method from ResourceLoadNotifier better decouples the
notifier from FrameLoader.

  • loader/FrameLoader.cpp: (WebCore::FrameLoader::continueLoadAfterWillSubmitForm): (WebCore::FrameLoader::requestFromDelegate):
  • loader/ResourceLoadNotifier.cpp: (WebCore::ResourceLoadNotifier::didReceiveResponse):
  • loader/ResourceLoadNotifier.h:
  • loader/ResourceLoader.cpp: (WebCore::ResourceLoader::willSendRequest):
6:28 PM Changeset in webkit [49828] by abarth@webkit.org
  • 5 edits in trunk/WebCore

2009-10-19 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.

Move sendRemainingDelegateMessages to ResourceLoadNotifier
https://bugs.webkit.org/show_bug.cgi?id=30531

This method is about notifying folks and doesn't interact with the rest
of FrameLoader.

  • loader/FrameLoader.cpp: (WebCore::FrameLoader::willLoadMediaElementURL): (WebCore::FrameLoader::commitProvisionalLoad): (WebCore::FrameLoader::loadResourceSynchronously): (WebCore::FrameLoader::loadedResourceFromMemoryCache):
  • loader/FrameLoader.h:
  • loader/ResourceLoadNotifier.cpp: (WebCore::ResourceLoadNotifier::sendRemainingDelegateMessages):
  • loader/ResourceLoadNotifier.h:
6:19 PM Changeset in webkit [49827] by abarth@webkit.org
  • 7 edits
    2 adds in trunk

2009-10-19 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.

Bypass popup blocker using click event
https://bugs.webkit.org/show_bug.cgi?id=21501

Test that a fake event can't get around the popup blocker.

  • http/tests/security/popup-blocked-from-fake-event-expected.txt: Added.
  • http/tests/security/popup-blocked-from-fake-event.html: Added.

2009-10-19 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.

Bypass popup blocker using click event
https://bugs.webkit.org/show_bug.cgi?id=21501

Keep track of which events were generated by JavaScript and use that
inforation when figuring out if we're processing a user gesture.

Test: http/tests/security/popup-blocked-from-fake-event.html

  • bindings/js/ScriptController.cpp: (WebCore::ScriptController::processingUserGestureEvent):
  • bindings/v8/ScriptController.cpp: (WebCore::ScriptController::processingUserGesture):
  • dom/Document.cpp: (WebCore::Document::createEvent):
  • dom/Event.cpp: (WebCore::Event::Event):
  • dom/Event.h: (WebCore::Event::createdByDOM): (WebCore::Event::setCreatedByDOM):
6:09 PM Changeset in webkit [49826] by dumi@chromium.org
  • 2 edits in trunk/WebCore

Fixing a typo.

Patch by Dumitru Daniliuc <dumi@chromium.org> on 2009-10-19
Reviewed by Darin Adler.

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

  • platform/sql/SQLiteDatabase.h:
6:05 PM Changeset in webkit [49825] by dumi@chromium.org
  • 1 edit in trunk/WebKitTools/Scripts/modules/committers.py

Adding myself to committers.py.

5:41 PM Changeset in webkit [49824] by dimich@chromium.org
  • 1 edit
    1 add in trunk/WebCore

Manual Test for crash caused by JS accessing DOMWindow which is disconnected from the Frame.
https://bugs.webkit.org/show_bug.cgi?id=30544

Reviewed by Darin Adler.

  • manual-tests/crash-on-accessing-domwindow-without-frame.html: Added.
5:38 PM Changeset in webkit [49823] by Simon Fraser
  • 3 edits in trunk/WebCore

2009-10-19 Simon Fraser <Simon Fraser>

Reviewed by Darin Adler.

Flash at end of opacity/transform transition sometimes
https://bugs.webkit.org/show_bug.cgi?id=30501

When a transition finishes, there window of time between when the animation is
removed, and the final style set on the GraphicsLayer. This caused the layer to revert
to its old appearance for one or two frames. To avoid this, we set the final
transform or opacity at the start of the transition; we know that the animation
will override the final value for as long as its running.

No test because this is a very transient effect that can't be captured
in a test.

  • rendering/RenderLayerBacking.cpp: (WebCore::RenderLayerBacking::createGraphicsLayer): (WebCore::RenderLayerBacking::updateLayerOpacity): (WebCore::RenderLayerBacking::updateLayerTransform): (WebCore::RenderLayerBacking::updateGraphicsLayerGeometry): (WebCore::RenderLayerBacking::startTransition):
  • rendering/RenderLayerBacking.h:
5:12 PM Changeset in webkit [49822] by cmarrin@apple.com
  • 1 edit in trunk/WebKitSite/blog-files/webgl/resources/blogpost.css

Added one more style rule

5:03 PM Changeset in webkit [49821] by cmarrin@apple.com
  • 1 add in trunk/WebKitSite/blog-files/webgl/resources/blogpost.css

Moved WebGL blog CSS to a separate file

3:59 PM Changeset in webkit [49820] by ggaren@apple.com
  • 3 edits in trunk/JavaScriptCore

Tightened up some put_by_id_transition code generation.
https://bugs.webkit.org/show_bug.cgi?id=30539

Patch by Geoffrey Garen <ggaren@apple.com> on 2009-10-19
Reviewed by Oliver Hunt.

  • jit/JIT.h:
  • jit/JITPropertyAccess.cpp:

(JSC::JIT::testPrototype):
(JSC::JIT::privateCompilePutByIdTransition): No need to do object type
checks or read Structures and prototypes from objects: they're all known
constants at compile time.

3:44 PM Changeset in webkit [49819] by eric@webkit.org
  • 3 edits in trunk/LayoutTests

2009-10-19 Jason Yan <tailofthesun@gmail.com>

Reviewed by Eric Seidel.

Fixed issue with LayoutTests/fast/cookies/local-file-can-set-cookies.html
which can fail when the system under test has local file cookies set already.
https://bugs.webkit.org/show_bug.cgi?id=30281

  • fast/cookies/local-file-can-set-cookies-expected.txt: Modified. Updated expected results.
  • fast/cookies/local-file-can-set-cookies.html: Modified. Check for existing cookie and set non-persistent cookie.
3:36 PM Changeset in webkit [49818] by eric@webkit.org
  • 3 edits in trunk/LayoutTests

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

Reviewed by Eric Seidel.

Language tag should include both language and country sub-tags,
case insensitive.
https://bugs.webkit.org/show_bug.cgi?id=30440

  • fast/js/navigator-language-expected.txt:
  • fast/js/navigator-language.html:
3:28 PM Changeset in webkit [49817] by eric@webkit.org
  • 2 edits in trunk/WebKitLibraries

2009-10-19 Marshall Culpepper <mculpepper@appcelerator.com>

Reviewed by Eric Seidel.

added cairo include and lib directories to debug_wincairo.vsprops
https://bugs.webkit.org/show_bug.cgi?id=29831

  • win/tools/vsprops/debug_wincairo.vsprops:
3:10 PM Changeset in webkit [49816] by eric@webkit.org
  • 2 edits in trunk/WebKitSite

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

Reviewed by Eric Seidel.

Replace boolean operator indentation example with an if statement.

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

  • coding/coding-style.html: Replace boolean indenting example to use "if"
3:00 PM Changeset in webkit [49815] by pfeldman@chromium.org
  • 2 edits in trunk/WebCore

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

Reviewed by Timothy Hatcher.

Web Inspector: Add more stop characters into console completions.

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

  • inspector/front-end/ConsoleView.js: (WebInspector.ConsoleView.prototype.completions): (WebInspector.ConsoleView.prototype._reportCompletions):
3:00 PM Changeset in webkit [49814] by Adam Roben
  • 2 edits in trunk/WebKit/win

Fix crashes/assertions when calling WebLocalizedString from multiple threads concurrently

Fixes <http://webkit.org/b/30534> WebLocalizedString asserts if called
from multiple threads concurrently

Reviewed by John Sullivan.

  • WebLocalizableStrings.cpp:

(mainBundleLocStringsMutex):
(frameworkLocStringsMutex):
Added these new getters.

(findCachedString):
(cacheString):
Lock the relevant mutex before accessing each string map. Otherwise
bad things could happen if two threads end up here at the same time.

3:00 PM Changeset in webkit [49813] by Adam Roben
  • 2 edits in trunk/WebKit/win

Get rid of a few static initializers/exit-time destructors in WebLocalizableStrings

Reviewed by Dan Bernstein.

  • WebLocalizableStrings.cpp:

(mainBundleLocStrings):
(frameworkLocStrings):
Added these functions instead of file-scope variables.

(findCachedString):
(cacheString):
Updated to use the new functions.

2:41 PM Changeset in webkit [49812] by Nate Chapin
  • 4 edits in trunk/WebKit

2009-10-19 Nate Chapin <Nate Chapin>

Unreviewed, build fix.

Update call to FrameLoader::loadFrameRequest() in qt and
FrameLoader::urlSelected() in win.

2:39 PM Changeset in webkit [49811] by dbates@webkit.org
  • 2 edits in trunk/LayoutTests

2009-10-19 Daniel Bates <dbates@webkit.org>

Reviewed by Eric Seidel.

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


Added the test case for bug #30266 to the Windows skip list because
event.dataTransfer.types does not behave correctly. See bug #30527.

  • platform/win/Skipped: Added test case to Windows skipped list.
2:31 PM Changeset in webkit [49810] by eric@webkit.org
  • 2 edits in trunk/WebCore

2009-10-19 Marshall Culpepper <mculpepper@appcelerator.com>

Reviewed by Adam Roben.

implemented ClipboardWin::files()
https://bugs.webkit.org/show_bug.cgi?id=29666

  • platform/win/ClipboardWin.cpp: (WebCore::ClipboardWin::files):
2:15 PM Changeset in webkit [49809] by Nate Chapin
  • 13 edits
    11 adds in trunk

2009-10-19 Nate Chapin <Nate Chapin>

Reviewed by Alexey Proskuryakov.

Add support for link relations in HTMLAnchorElement and implement rel="noreferrer".

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

2:08 PM Changeset in webkit [49808] by ddkilzer@apple.com
  • 2 edits in trunk/WebKit/mac

Documentation fix for -[NSObject(WebResourceLoadDelegate) webView:resource:willSendRequest:redirectResponse:fromDataSource:]

Reviewed by Dan Bernstein.

  • WebView/WebResourceLoadDelegate.h: Prepended "webView:" to the

@method declaration of
-webView:resource:willSendRequest:redirectResponse:fromDataSource:
which was absent before r3953 and was missed during the great
documentation clean up of r4269.

2:01 PM Changeset in webkit [49807] by andersca@apple.com
  • 2 edits in trunk/WebKit/mac

Fix build.

  • Plugins/WebNetscapePluginView.mm:

(-[WebNetscapePluginView saveAndSetNewPortStateForUpdate:]):

1:30 PM Changeset in webkit [49806] by mitz@apple.com
  • 8 edits in trunk/WebKit/mac

Split assertions of conjunctions into separate statements or remove
redundant assertions.

Reviewed by Anders Carlsson.

  • Plugins/Hosted/NetscapePluginHostProxy.mm:

(WebKit::NetscapePluginHostProxy::deadNameNotificationCallback):

  • Plugins/Hosted/WebHostedNetscapePluginView.mm:

(-[WebHostedNetscapePluginView setAttributeKeys:andValues:]):
(-[WebHostedNetscapePluginView _webPluginContainerCancelCheckIfAllowedToLoadRequest:]):

  • Plugins/WebBaseNetscapePluginView.mm:

(-[WebBaseNetscapePluginView halt]):
(-[WebBaseNetscapePluginView resumeFromHalt]):

  • Plugins/WebNetscapePluginView.mm:

(-[WebNetscapePluginView saveAndSetNewPortStateForUpdate:]):
(-[WebNetscapePluginView _webPluginContainerCancelCheckIfAllowedToLoadRequest:]):

  • WebCoreSupport/WebContextMenuClient.mm:

(WebContextMenuClient::getCustomMenuFromDefaultItems):

  • WebCoreSupport/WebEditorClient.mm:

(WebEditorClient::checkGrammarOfString):
(WebEditorClient::checkTextOfParagraph):

  • WebView/WebTextCompletionController.mm:

(-[WebTextCompletionController _reflectSelection]):

1:25 PM Changeset in webkit [49805] by Nate Chapin
  • 2 edits in trunk/LayoutTests

2009-10-19 Nate Chapin <Nate Chapin>

Reviewed by Geoffrey Garen.

Add plugins/netscape-invoke-failures.html to Snow Leopard skipped list.

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

  • platform/mac-snowleopard/Skipped: Add plugins/netscape-invoke-failures.html.
1:25 PM Changeset in webkit [49804] by kenneth@webkit.org
  • 3 edits in trunk/WebCore

Refactor out some of the code in paint() into paintPanScrollIcon()
and paintScrollbars() in preparation of some Qt API changes.

Patch by Kenneth Rohde Christiansen <kenneth@webkit.org> on 2009-10-19
Reviewed by David Hyatt.

  • platform/ScrollView.cpp:

(WebCore::ScrollView::paint):

  • platform/ScrollView.h:
1:18 PM Changeset in webkit [49803] by hclam@chromium.org
  • 4 edits in trunk/WebCore

[v8] typeof(HTMLMediaElement) should return undefined if media
engine is not available
https://bugs.webkit.org/show_bug.cgi?id=30343

Patch by Alpha Lam <hclam@chromium.org> on 2009-10-19
Reviewed by Dimitri Glazkov (Dimitri Glazkov).

Check for availability of the media engine to disable
HTMLMediaElement, HTMLAudioElement, HTMLVideoElement, MediaError
in runtime.

Try runs and review in Chromium:
http://codereview.chromium.org/276011

Test: manual-tests/chromium/media-player-not-available.html

The above test can work work in Chromium. Testing procedures:

  1. Remove all media support libraries in Chromium (e.g. ffmpeg libraries)
  2. Open Chromium with above test page
  3. Verify the test results according to the test page
  • bindings/v8/custom/V8CustomBinding.h:

Declare enabler methods.

  • bindings/v8/custom/V8DOMWindowCustom.cpp:

Report enabled only if media player is available.

  • manual-tests/chromium/media-player-not-available.html: Added.

Manual test, following instruction in this test for procedures.

  • page/DOMWindow.idl:

Mark HTMLMediaElement, HTMLAudioElement, HTMLVideElement, MediaError
as enabled at runtime.

12:59 PM Changeset in webkit [49802] by ggaren@apple.com
  • 9 edits
    1 add in trunk/JavaScriptCore

Added a private API for getting a global context from a context, for
clients who want to preserve a context for a later callback.

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

  • API/APICast.h:

(toGlobalRef): Added an ASSERT, since this function is used more often
than before.

  • API/JSContextRef.cpp:
  • API/JSContextRefPrivate.h: Added. The new API.
  • API/tests/testapi.c:

(print_callAsFunction):
(main): Test the new API.

12:38 PM Changeset in webkit [49801] by jmalonzo@webkit.org
  • 2 edits in trunk/WebKit/gtk

2009-10-19 Jan Michael Alonzo <jmalonzo@webkit.org>

Reviewed by Alexey Proskuryakov.

[Gtk] ApplicationCacheStorage error while compiling WebKit/gtk/webkit/webkitapplicationcache.cpp
https://bugs.webkit.org/show_bug.cgi?id=30358

Guard calls to cacheStorage() with ENABLE(OFFLINE_WEB_APPLICATIONS).

  • webkit/webkitapplicationcache.cpp:
12:24 PM Changeset in webkit [49800] by kenneth@webkit.org
  • 2 edits in trunk/WebCore

Make the local static panScrollIcon into a Image* instead
of a RefPtr<Image>.

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

  • platform/ScrollView.cpp:

(WebCore::ScrollView::wheelEvent):

12:19 PM Changeset in webkit [49799] by ajwong@chromium.org
  • 2 edits in trunk/WebCore

Forcefully set antialiasing for Chromium media controls.

Patch by Andrew Scherkus <scherkus@chromium.org> on 2009-10-19
Reviewed by NOBODY (OOPS!).

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

Before it was nondeterministiacally being enabled based on the previous state of GraphicsContext.

Covered by existing layout tests.

  • rendering/RenderMediaControlsChromium.cpp:

(WebCore::paintMediaSlider): Added setShouldAntialias(true).

12:11 PM Changeset in webkit [49798] by eric@webkit.org
  • 11 edits in trunk/WebCore

2009-10-19 Jens Alfke <jens@mooseyard.com>

Reviewed by Darin Adler.

Optimize string upper/lowercasing
https://bugs.webkit.org/show_bug.cgi?id=30261

  • Added AtomicString::upper() and lower()
  • Further optimized StringImpl::lower()
  • Removed StringImpl::isLower()
  • Added QualifiedName::localNameUpper(), which is cached, thereby saving thousands of upper() calls and string allocations.
  • dom/Element.cpp: (WebCore::Element::setAttribute): Call AtomicString::lower()
  • dom/QualifiedName.cpp: (WebCore::QualifiedName::localNameUpper): New method
  • dom/QualifiedName.h: Added localNameUpper() method
  • dom/StyledElement.cpp: (WebCore::StyledElement::parseMappedAttribute): Call AtomicString::lower()
  • html/HTMLDocument.cpp: (WebCore::HTMLDocument::createElement): Call AtomicString::lower()
  • html/HTMLElement.cpp: (WebCore::HTMLElement::nodeName): Call localNameUpper()
  • platform/text/AtomicString.cpp: (WebCore::AtomicString::lower): New method (WebCore::AtomicString::upper): New method
  • platform/text/AtomicString.h: Added lower() and upper()
  • platform/text/StringImpl.cpp: Removed isLower() (WebCore::StringImpl::lower): Further optimization of initial loop
  • platform/text/StringImpl.h: Removed isLower()
11:59 AM Changeset in webkit [49797] by eric@webkit.org
  • 19 edits
    1 delete in trunk/LayoutTests

2009-10-19 Zan Dobersek <zandobersek@gmail.com>

Reviewed by Gustavo Noronha.

Update test results and Skipped list after changes in r49791.

Some test results were accidentally committed in r49192. At that time
results were incorrect, but testing still succeeded because of
unimplemented features. After r49791 tests, though generating semantically
equal output to Mac's render trees, started to fail and need to have
generated results updated. This patch does just that.

  • platform/gtk/Skipped: Skip a test that does not generate semantically equal render trees.
  • platform/gtk/editing/inserting/insert-3907422-fix-expected.txt:
  • platform/gtk/editing/pasteboard/4076267-2-expected.txt:
  • platform/gtk/editing/pasteboard/4076267-3-expected.txt:
  • platform/gtk/editing/pasteboard/4076267-expected.txt:
  • platform/gtk/editing/pasteboard/paste-table-001-expected.txt:
  • platform/gtk/editing/pasteboard/paste-text-001-expected.txt:
  • platform/gtk/editing/pasteboard/paste-text-002-expected.txt:
  • platform/gtk/editing/pasteboard/paste-text-003-expected.txt:
  • platform/gtk/editing/pasteboard/paste-text-004-expected.txt:
  • platform/gtk/editing/pasteboard/paste-text-005-expected.txt:
  • platform/gtk/editing/pasteboard/paste-text-006-expected.txt:
  • platform/gtk/editing/pasteboard/paste-text-007-expected.txt:
  • platform/gtk/editing/pasteboard/paste-text-008-expected.txt:
  • platform/gtk/editing/pasteboard/paste-text-009-expected.txt:
  • platform/gtk/editing/pasteboard/paste-text-010-expected.txt:
  • platform/gtk/editing/pasteboard/paste-text-015-expected.txt:
  • platform/gtk/editing/pasteboard/paste-xml-expected.txt:
  • platform/gtk/editing/pasteboard/select-element-1-expected.txt: Removed.
11:50 AM Changeset in webkit [49796] by eric@webkit.org
  • 6 edits
    2 adds in trunk

2009-10-19 Nate Chapin <Nate Chapin>

Reviewed by Adam Barth.

Test that the proper exception is thrown into JS when a plugin
invoke call returns false.

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

  • plugins/netscape-invoke-failure-expected.txt: Added.
  • plugins/netscape-invoke-failure.html: Added.

2009-10-19 Nate Chapin <Nate Chapin>

Reviewed by Adam Barth.

If a call to a plugin's invokeMethod, invokeDefault or construct
returns false, throw an exception into JS.

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

Test: plugins/netscape-invoke-failure.html

  • bindings/v8/V8NPObject.cpp: Check return values of invokeMethod, invokeDefault and construct. (npObjectInvokeImpl):
  • bridge/c/c_instance.cpp: (JSC::Bindings::CInstance::invokeMethod): Check return value (JSC::Bindings::CInstance::invokeDefaultMethod): Check return value (JSC::Bindings::CInstance::invokeConstruct): Check return value

2009-10-19 Nate Chapin <Nate Chapin>

Reviewed by Adam Barth.

Add testFail() to test plugin so we can test our handling of a
plugin invoke call returning false.

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

  • DumpRenderTree/TestNetscapePlugIn.subproj/PluginObject.cpp: Add testFail(). (pluginInvoke): (testIdentifierToString): Always return true, since returning false will now cause an exception to be thrown.
11:47 AM Changeset in webkit [49795] by ggaren@apple.com
  • 2 edits in trunk/JavaScriptCore

Tightened up some instanceof code generation.
https://bugs.webkit.org/show_bug.cgi?id=30488

Patch by Geoffrey Garen <ggaren@apple.com> on 2009-10-17
Reviewed by Oliver Hunt.

  • jit/JITOpcodes.cpp:

(JSC::JIT::emit_op_instanceof):
(JSC::JIT::emitSlow_op_instanceof): No need to do object type checks -
cell type checks and ImplementsDefaultHasIntance checks implicitly
supersede object type checks.

11:38 AM Changeset in webkit [49794] by senorblanco@chromium.org
  • 3 edits
    4 adds in trunk

WebCore: Fix for chromium/skia canvas arc start/end positions, when the arc is

360 degrees.

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

Reviewed by Adam Barth.

Covered by new test:
LayoutTests/fast/canvas/arc360.html

  • platform/graphics/skia/PathSkia.cpp:

(WebCore::Path::addArc):

LayoutTests: New test for 360 degree arc endpoints, and expected results.
http://bugs.webkit.org/show_bug.cgi?id=30449

Reviewed by Adam Barth.

  • fast/canvas/arc360.html: Added.
  • platform/mac/fast/canvas/arc360-expected.checksum: Added.
  • platform/mac/fast/canvas/arc360-expected.png: Added.
  • platform/mac/fast/canvas/arc360-expected.txt: Added.
11:14 AM Changeset in webkit [49793] by bweinstein@apple.com
  • 2 edits in trunk/LayoutTests

Rubber-stamped by Adam Roben.

Add tests that have never passed on Windows to the Skipped list, and reopen
bugs.

  • platform/win/Skipped:
10:35 AM Changeset in webkit [49792] by eric@webkit.org
  • 2 edits in trunk/WebCore

2009-10-19 Kent Tamura <tkent@chromium.org>

Reviewed by Alexey Proskuryakov.

Fix uninitialized variable reference in Element::removeAttribute().
https://bugs.webkit.org/show_bug.cgi?id=30502

  • dom/Element.cpp: (WebCore::Element::removeAttribute):
10:25 AM Changeset in webkit [49791] by eric@webkit.org
  • 7 edits in trunk

2009-10-19 Zan Dobersek <zandobersek@gmail.com>

Reviewed by Gustavo Noronha.

Enable tests that pass when DOM pasting is allowed.

  • platform/gtk/Skipped:

2009-10-19 Zan Dobersek <zandobersek@gmail.com>

Reviewed by Gustavo Noronha.

Add a settings to allow DOM pastes to be performed.

  • webkit/webkitwebsettings.cpp: (webkit_web_settings_class_init): (webkit_web_settings_set_property): (webkit_web_settings_get_property): (webkit_web_settings_copy):
  • webkit/webkitwebview.cpp: (webkit_web_view_update_settings): (webkit_web_view_settings_notify):

2009-10-19 Zan Dobersek <zandobersek@gmail.com>

Reviewed by Gustavo Noronha.

Enable DOM pasting when running layout tests.

  • DumpRenderTree/gtk/DumpRenderTree.cpp: (resetDefaultsToConsistentValues):
10:13 AM Changeset in webkit [49790] by eric.carlson@apple.com
  • 11 edits in trunk

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

Reviewed by Simon Fraser.

https://bugs.webkit.org/show_bug.cgi?id=30463
Remove HTML5 media element 'loadend' event

  • dom/EventNames.h:
  • html/HTMLMediaElement.cpp: (WebCore::HTMLMediaElement::parseMappedAttribute): Remove 'loadend'. (WebCore::HTMLMediaElement::loadInternal): Ditto. (WebCore::HTMLMediaElement::noneSupported): Ditto. (WebCore::HTMLMediaElement::mediaEngineError): Ditto. (WebCore::HTMLMediaElement::setNetworkState): Ditto. (WebCore::HTMLMediaElement::userCancelledLoad): Ditto.

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

Reviewed by Simon Fraser.

https://bugs.webkit.org/show_bug.cgi?id=30463
Remove HTML5 media element 'loadend' event

  • media/event-attributes-expected.txt:
  • media/event-attributes.html: Switch from 'loadend' to 'canplaythrough'.
  • media/media-load-event-expected.txt:
  • media/media-load-event.html: Ditto.
  • media/progress-event-at-least-one-expected.txt:
  • media/progress-event-at-least-one.html: Ditto.
9:26 AM Changeset in webkit [49789] by eric@webkit.org
  • 3 edits in trunk/WebCore

2009-10-19 Jocelyn Turcotte <jocelyn.turcotte@nokia.com>

Reviewed by Adam Barth.

[Qt] Remove deletion of outgoing FormData object when reply is finished.
The buffers will be destroyed a bit later, when the QNetworkReply is deleted.
Bug: https://bugs.webkit.org/show_bug.cgi?id=29551

  • platform/network/qt/QNetworkReplyHandler.cpp:
  • platform/network/qt/QNetworkReplyHandler.h:
9:18 AM Changeset in webkit [49788] by eric@webkit.org
  • 2 edits in trunk/WebCore

2009-10-19 Dimitri Glazkov <Dimitri Glazkov>

Reviewed by Darin Adler.

Fix hard-to-reproduce crash in HTMLTokenizer by avoiding a rare
fastRealloc edge case.
https://bugs.webkit.org/show_bug.cgi?id=29313

No test, the crash shows up occasionally in crash dumps, we weren't able
to reproduce it locally.

  • html/HTMLTokenizer.cpp: (WebCore::HTMLTokenizer::enlargeScriptBuffer): Added an early exit to

avoid calling fastRealloc with the size of 0.

9:08 AM Changeset in webkit [49787] by eric@webkit.org
  • 2 edits in trunk/WebCore

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

Reviewed by Eric Seidel.

Use fillRect() instead of drawRect() to fix Chromium media controls rendering.

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

Chromium Mac layout tests will need to be rebaselined. Existing layout tests for Linux/Win still pass.

  • rendering/RenderMediaControlsChromium.cpp: (WebCore::paintMediaSlider): Use fillRect() instead of drawRect().
9:00 AM Changeset in webkit [49786] by eric@webkit.org
  • 2 edits in trunk/WebCore

2009-10-19 Dmitry Titov <dimich@chromium.org>

Reviewed by Adam Barth.

Crash in DOMWindow::clearTimeout etc when DOMWindow is not connected to a frame.
https://bugs.webkit.org/show_bug.cgi?id=29832

Need to make sure the script caches are reset when frame gets disconnected from still-alive DOMWindow.
This will prevent JS from calling DOMWindow methods that can not be completed w/o the frame.

I am not sure it's possible to test this since the only file that repros the problem need ~10 seconds to cause crash.

  • page/Frame.cpp: (WebCore::Frame::~Frame): Right after frame disconnects from DOMWindow, clear WindowShell.
8:53 AM Changeset in webkit [49785] by Laszlo Gombos
  • 5 edits
    1 add in trunk/WebKit/qt

2009-10-19 Viatcheslav Ostapenko <ostapenko.viatcheslav@nokia.com>

Reviewed by Ariya Hidayat.

Add QWebElement::render API which allows rendering of single
element.

  • Api/qwebelement.cpp: (QWebElement::render):
  • Api/qwebelement.h:
  • tests/qwebelement/tst_qwebelement.cpp: (tst_QWebElement::render):
  • tests/qwebelement/qwebelement.qrc:
  • tests/qwebelement/image.png: Added.
8:16 AM Changeset in webkit [49784] by ariya@webkit.org
  • 2 edits in trunk/WebKit/qt

QWebPage: Doc: setNetworkAccessManager should only be called once.

Patch by Markus Goetz <Markus.Goetz@nokia.com> on 2009-10-19
Reviewed by Ariya Hidayat.

8:02 AM Changeset in webkit [49783] by Csaba Osztrogonác
  • 2 edits in trunk/WebCore

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

Unreviewed build warning fix.
[Qt] Apply changes to WebCore.pro introduced in r49778

  • WebCore.pro: Removed svg/graphics/filters/SVGFEGaussianBlur.h from HEADERS
8:01 AM Changeset in webkit [49782] by tonikitoo@webkit.org
  • 2 edits in trunk/WebKit/qt

Wrong ifdef combination in QGraphicsWebView's event method.

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

  • Api/qgraphicswebview.cpp:

(QGraphicsWebView::event):

7:42 AM Changeset in webkit [49781] by Csaba Osztrogonác
  • 2 edits in trunk/LayoutTests

[Qt] Skip failing tests for now and add test-specific information about the cause to the Skipped list.

Patch by Andras Becsi <becsi.andras@stud.u-szeged.hu> on 2009-10-19
Reviewed by Tor Arne Vestbø.

  • platform/qt/Skipped:
7:16 AM Changeset in webkit [49780] by kov@webkit.org
  • 2 edits in trunk/WebKit/gtk

Make WebKitHitTestResult actually appear in the index

7:00 AM Changeset in webkit [49779] by Csaba Osztrogonác
  • 1 edit
    1 add in trunk/LayoutTests

[Qt] Add platform expected file for canvas test.

Patch by Andras Becsi <becsi.andras@stud.u-szeged.hu> on 2009-10-19
Reviewed by Tor Arne Vestbø.

  • platform/qt/fast/canvas/canvas-empty-image-pattern-expected.txt: Added.
5:57 AM Changeset in webkit [49778] by krit@webkit.org
  • 7 edits
    2 moves in trunk/WebCore

2009-10-19 Dirk Schulze <krit@webkit.org>

Reviewed by Darin Adler.

Move feGaussianBlur from WebCore/svg to WebCore/platform
https://bugs.webkit.org/show_bug.cgi?id=30495

This patch moves SVGFRGaussianBlur from svg/graphics/filters
to platform/graphics/filters/FEGaussianBlur. This is needed
for shadow support on Qt and Cairo.
No change in functionality, therfore no new test case needed.

  • GNUmakefile.am:
  • WebCore.gypi:
  • WebCore.pro:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • platform/graphics/filters/FEGaussianBlur.cpp: Added.
  • platform/graphics/filters/FEGaussianBlur.h: Added.
  • svg/SVGFEGaussianBlurElement.h:
  • svg/graphics/filters/SVGFEGaussianBlur.cpp: Removed.
  • svg/graphics/filters/SVGFEGaussianBlur.h: Removed.
5:49 AM Changeset in webkit [49777] by yael.aharon@nokia.com
  • 1 edit in trunk/WebCore/ChangeLog

Unreviewed.

Patch by Yael Aharon <yael.aharon@nokia.com> on 2009-10-19
Added reviewer name to r49574.

5:48 AM Changeset in webkit [49776] by Csaba Osztrogonác
  • 3 edits in trunk/LayoutTests

2009-10-19 Andras Becsi <becsi.andras@stud.u-szeged.hu>

Reviewed by Tor Arne Vestbø.

[Qt] Update outdated metrics for these passing tests.

  • platform/qt/css1/font_properties/font-expected.txt:
  • platform/qt/fast/xsl/document-function-expected.txt:
4:58 AM Changeset in webkit [49775] by eric@webkit.org
  • 2 edits in trunk/WebCore

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

Reviewed by Xan Lopez.

https://bugs.webkit.org/show_bug.cgi?id=30489
[Gtk] The document frame/html container claims to be parentless

Work around for the parentless claim made by the accessible associated
with the embedded WebView. When we identify this condition, get the
widget's parent, and then get the AtkObject from that. Admittedly an ugly
hack, but we'll need it for Yelp and other Gtk+/GNOME applications which
plan to switch over to WebKit.

  • accessibility/gtk/AccessibilityObjectWrapperAtk.cpp: (webkit_accessible_get_parent):
4:51 AM Changeset in webkit [49774] by jmalonzo@webkit.org
  • 2 edits in trunk/WebCore

2009-10-19 Benjamin Otte <otte@gnome.org>

Reviewed by Jan Alonzo.

Handle duration queries properly
https://bugs.webkit.org/show_bug.cgi?id=29999

Previously duration queries failed to handle unknown duration
and didn't treat the returned duration as an unsigned.

  • platform/graphics/gtk/MediaPlayerPrivateGStreamer.cpp: (WebCore::MediaPlayerPrivate::duration):
4:50 AM Changeset in webkit [49773] by eric@webkit.org
  • 2 edits in trunk/WebKit/gtk/po

2009-10-19 Daniel Macks <dmacks@netspace.org>

Reviewed by Adam Barth.

Inconsistent handling of gettext domain can cause compile failure
https://bugs.webkit.org/show_bug.cgi?id=30292

  • GNUmakefile.am: Use DOMAIN variable instead of hard-coded value
4:28 AM Changeset in webkit [49772] by eric@webkit.org
  • 2 edits in trunk/WebCore

2009-10-19 Jocelyn Turcotte <jocelyn.turcotte@nokia.com>

Reviewed by Tor Arne Vestbø.

[Qt] Fix build on Windows.

  • plugins/win/PluginViewWin.cpp: (windowHandleForPageClient):
4:17 AM Changeset in webkit [49771] by zecke@webkit.org
  • 5 edits
    2 adds in trunk

Windowed Plugins: Don't crash when client is 0.

Client is 0 when we use QWebPage without a QWebView or QGraphicsWebView.
In addition, setFrameRect()/updatePluginWidget() is called even if the
plugin was not succesfully loaded. updatePluginWidget() updates the
window rect which is, in theory, useful to draw something that indicates
that we didn't load successfully.

So, a status check is added to setNPWindowIfNeeded.

3:44 AM Changeset in webkit [49770] by eric@webkit.org
  • 7 edits in trunk

2009-10-19 Girish Ramakrishnan <Girish Ramakrishnan>

Reviewed by Holger Freyther.

[Qt] Windowed Plugins: Fix crash when QWebPage is deleted after QWebView.

Fixes various sources of crashes:

  1. The PluginContainer is a child of QWebView. When the view gets deleted, the PluginView is not notified about the deletion of PluginContainer.
  2. QWebView destructor does not set client to 0.
  3. Sometimes pending paint events are sent after the plugin has died, so add a check in PluginView::setNPWindowIfNeeded.

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

  • plugins/qt/PluginContainerQt.cpp: (PluginContainerQt::~PluginContainerQt):
  • plugins/qt/PluginViewQt.cpp: (WebCore::PluginView::setNPWindowIfNeeded):

2009-10-19 Girish Ramakrishnan <Girish Ramakrishnan>

Reviewed by Holger Freyther.

[Qt] Windowed Plugins: Fix crash when QWebPage is deleted after QWebView.

Fixes various sources of crashes:

  1. The PluginContainer is a child of QWebView. When the view gets deleted, the PluginView is not notified about the deletion of PluginContainer.
  2. QWebView destructor does not set client to 0.
  3. Sometimes pending paint events are sent after the plugin has died, so add a check in PluginView::setNPWindowIfNeeded.

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

  • Api/qwebview.cpp: (QWebView::~QWebView):
  • tests/qwebview/qwebview.pro:
  • tests/qwebview/tst_qwebview.cpp: (tst_QWebView::reusePage_data): (tst_QWebView::reusePage):
12:17 AM Changeset in webkit [49769] by eric@webkit.org
  • 7 edits in trunk

2009-10-19 Jakob Truelsen <antialize@gmail.com>

Reviewed by Adam Barth.

https://bugs.webkit.org/show_bug.cgi?id=29042
Allow one to customize the minimal and maximal shrink factors used when printing.

  • page/PrintContext.cpp: (WebCore::PrintContext::begin):
  • page/Settings.cpp: (WebCore::Settings::Settings): (WebCore::Settings::setPrintingMinimumShrinkFactor): (WebCore::Settings::setPrintingMaximumShrinkFactor):
  • page/Settings.h: (WebCore::Settings::printingMinimumShrinkFactor): (WebCore::Settings::printingMaximumShrinkFactor):

2009-10-19 Jakob Truelsen <antialize@gmail.com>

Reviewed by Adam Barth.

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

Allow one to costumize the minimal and maximal shrink factors,
Added methods setPrintingMinimumShrinkFactor, printingMinimumShrinkFactor,
setPrintingMaximumShrinkFactor, printingMaximumShrinkFactor to QWebSettings.

  • Api/qwebsettings.cpp: (QWebSettingsPrivate::apply): (QWebSettings::QWebSettings): (QWebSettings::setPrintingMinimumShrinkFactor): (QWebSettings::printingMinimumShrinkFactor): (QWebSettings::setPrintingMaximumShrinkFactor): (QWebSettings::printingMaximumShrinkFactor):
  • Api/qwebsettings.h:
12:04 AM Changeset in webkit [49768] by abarth@webkit.org
  • 1 edit in trunk/WebKit/qt/ChangeLog

Fix ChangeLog spelling.

Note: See TracTimeline for information about the timeline view.