Timeline



Sep 1, 2009:

10:03 PM Changeset in webkit [47954] by bweinstein@apple.com
  • 5 edits in trunk

2009-09-01 Brian Weinstein <bweinstein@apple.com>

Reviewed by John Sullivan.

Added a new case to aria-readonly.html to account for textareas instead of just input fields.

  • accessibility/aria-readonly-expected.txt:
  • accessibility/aria-readonly.html:

2009-09-01 Brian Weinstein <bweinstein@apple.com>

Reviewed by John Sullivan.

Fixed a bug in handling of isReadOnly, where textareas were casted to HTMLInputElements.

  • accessibility/AccessibilityRenderObject.cpp: (WebCore::AccessibilityRenderObject::isReadOnly):
7:26 PM Changeset in webkit [47953] by ddkilzer@apple.com
  • 3 edits in trunk/WebKitTools

<http://webkit.org/b/28601> bugzilla-tool post-commits posts commits backwards

Reviewed by Adam Barth.

  • Scripts/modules/scm.py: (Git.commit_ids_from_commitish_arguments): Reverse the list of commits returned from git-rev-list since we always want to post the oldest patches first to bugs.webkit.org.
  • Scripts/modules/scm_unittest.py: (run): Added return statement to return the output now that we want it sometimes. (SVNTestRepository._setup_test_commits): Added a fourth commit so the GitTest.test_commitish_order() test has more commits to work with. (GitTest.test_commitish_order): Added unit test for change to Git.commit_ids_from_commitish_arguments() in scm.py.
5:38 PM Changeset in webkit [47952] by eric@webkit.org
  • 6 edits
    1 add in trunk/WebCore

2009-09-01 Patrick Mueller <Patrick_Mueller@us.ibm.com>

Reviewed by Timothy Hatcher.

https://bugs.webkit.org/show_bug.cgi?id=28047
add utility method to enable logging messages for inspector development

manual test case added (see below)

  • inspector/front-end/ConsoleView.js: (WebInspector.ConsoleMessage.prototype.toMessageElement): (WebInspector.ConsoleMessage.prototype.toString):
  • inspector/front-end/inspector.css:
  • inspector/front-end/inspector.js: (WebInspector.log):
  • manual-tests/inspector/webinspector-log.html: Added.
  • page/Console.cpp: (WebCore::printMessageSourceAndLevelPrefix):
  • page/Console.h: (WebCore::):
5:15 PM Changeset in webkit [47951] by eric@webkit.org
  • 1 edit
    1 move in trunk/LayoutTests

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

Rubber stamped by Simon Fraser.

REGRESSION: media/video-source-add-src.html (and other media tests?) crashing/timing-out intermittently
https://bugs.webkit.org/show_bug.cgi?id=28845

Disabling this test on all platforms until
a media expert can have a chance to look at it.
See bug 28845 for further discussion.

  • media/video-source-add-src.html-disabled: Renamed from LayoutTests/media/video-source-add-src.html.
5:06 PM Changeset in webkit [47950] by ap@apple.com
  • 3 edits
    2 adds in trunk

Reviewed by Darin Adler.

https://bugs.webkit.org/show_bug.cgi?id=21990
When a rare EUC-JP character is present, explicitly (and correctly) labelled EUC-JP document
is mistreated as Shift_JIS

<rdar://problem/5828506> Safari should not try auto-detecting if charset information exists

WebKit used to unconditionally auto-detect document encoding if it was one of Japanese ones.
This didn't really match any browser, and did cause problems on some sites.

Our detector is very quick to label an EUC-JP page as Shift_JIS if it notices a byte that
cannot be part of EUC-JP source. However, such bytes do appear in Web pages sometimes,
especially in comments. Also, some valid EUC-JP sequences are not known to it.

Some pages that are fixed by this change:

<rdar://problem/5934750> Encoding in incorrect at http://cocodeparis.blog45.fc2.com/. This
page has incorrect encoding in title, so no browser displays it 100% fine. We now match
Firefox and IE.

<rdar://problem/6007713> The web page http://www.sf.us.emb-japan.go.jp/top.htm is garbled
in Safari. There is a stray byte in comments that was makes charset detector think it can't
be EUC-JP.

<rdar://problem/6965711> A Japanese web page www.nippontv.studiolegend.tv is garbled in
Safari. Many comments on this page are encoded as Shift_JIS - IE magically displays that
fine in View Source mode (it doesn't seem to have an ability to switch between EUC-JP and
Shift_JIS in normal HTML view though).

Test: fast/encoding/japanese-encoding-mix.html

  • loader/TextResourceDecoder.cpp: (WebCore::TextResourceDecoder::decode): Only invoke auto- detection if there is no solid charset information yet.
4:07 PM Changeset in webkit [47949] by ddkilzer@apple.com
  • 4 edits in trunk/WebKitTools

<http://webkit.org/b/28877> Implement bugzilla-tool mark-fixed

Reviewed by David Levin.

The mark-fixed subcommand is for those times when you don't use
bugzilla-tool to commit a patch, but you want to use it to close
the bug with a committed-revision message.

  • Scripts/bugzilla-tool: (bug_comment_from_svn_revision): Added. Extracted from bug_comment_from_commit_text(). (bug_comment_from_commit_text): Extracted bug_comment_from_svn_revision() from this method. (MarkBugFixed.init): Added. (MarkBugFixed._fetch_commit_log): Added. Retrieves the commit log from the last commit if no svn revision is specified, else the commit log for the specified svn revision. (MarkBugFixed._determine_bug_id_and_svn_revision): Added. Attempts to determine the bug id and svn revision if one or both were not defined on the command line. (MarkBugFixed.execute): Added. Adds a comment about the revision that fixed the bug and closes the bug. (BugzillaTool.init): Added mark-fixed subcommand.
  • Scripts/modules/bugzilla.py: (Bugzilla.fetch_title_from_bug): Added. Returns the title of a bug given a bug id.
  • Scripts/modules/scm.py: (SCM.strip_r_from_svn_revision): Added. Utility method to strip the leading 'r' from an svn revision. (SCM.svn_commit_log): Added. Subclasses must override. (SCM.last_svn_commit_log): Added. Subclasses must override. (SVN.svn_commit_log): Added. Returns svn log for a given revision. (SVN.last_svn_commit_log): Added. Uses svnversion to find the last commit in an svn working directory and then runs svn log. (Git.svn_commit_log): Added. Returns svn log for a given revision. (Git.last_svn_commit_log): Added. Runs git-svn-log with a limit of one log message.
1:18 PM Changeset in webkit [47948] by Dimitri Glazkov
  • 3 edits in trunk/LayoutTests

2009-09-01 Victor Wang <victorw@chromium.org>

Reviewed by Darin Adler.

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

Fix fast/dom/StyleSheet/ownerNode-lifetime-2.html
The problem is WebKit <link> nodes will only have a
style sheet created if they have href attribute.
The src attribute to non-existant css doesn't work.
Change src to href to make this test works in WebKit.

Also update the expectation result.

  • fast/dom/StyleSheet/ownerNode-lifetime-2-expected.txt:
  • fast/dom/StyleSheet/ownerNode-lifetime-2.html:
1:16 PM Changeset in webkit [47947] by jianli@chromium.org
  • 4 edits
    1 add in trunk/WebCore

[V8] FileList cannot be accessed via index in Chromium.
https://bugs.webkit.org/show_bug.cgi?id=28883

Reviewed by Dimitri Glazkov.

Tested by clipboard-file-access.html.

  • WebCore.gypi:
  • bindings/v8/V8DOMWrapper.cpp:

(WebCore::V8DOMWrapper::getTemplate):

  • bindings/v8/custom/V8CustomBinding.h:
  • bindings/v8/custom/V8FileListCustom.cpp: Added.
12:52 PM Changeset in webkit [47946] by jhoneycutt@apple.com
  • 9 edits in trunk/WebCore

Remove the workaround added in r47316.

Reviewed by Sam Weinig.

  • accessibility/AXObjectCache.cpp:

(WebCore::AXObjectCache::AXObjectCache):

  • accessibility/AXObjectCache.h:

Remove the Document argument to the AXObjectCache constructor and the
m_document member variable, and replace the no-parameter
handleFocusedUIElementChanged() with the two-parameter GTK function.
(WebCore::AXObjectCache::handleFocusedUIElementChanged):

  • accessibility/chromium/AXObjectCacheChromium.cpp:

(WebCore::AXObjectCache::handleFocusedUIElementChanged):

  • accessibility/gtk/AXObjectCacheAtk.cpp:

(WebCore::AXObjectCache::handleFocusedUIElementChanged):

  • accessibility/gtk/AccessibilityObjectWrapperAtk.cpp:

(fallbackObject):
Don't pass a null Document when constructing the AXObjectCache.

  • accessibility/mac/AXObjectCacheMac.mm:

(WebCore::AXObjectCache::handleFocusedUIElementChanged):

  • accessibility/win/AXObjectCacheWin.cpp:

(WebCore::AXObjectCache::handleFocusedUIElementChanged):
Get the document from the RenderObject; m_document was removed.

  • dom/Document.cpp:

(WebCore::Document::axObjectCache):
Don't pass the Document when constructing the AXObjectCache.
(WebCore::Document::setFocusedNode):
Pass the old and new focused RenderObjects when calling
handleFocusedUIElementChanged().

11:42 AM Changeset in webkit [47945] by ddkilzer@apple.com
  • 2 edits in trunk/WebKitTools

<http://webkit.org/b/28880> svn-apply --force doesn't actually work

Reviewed by Brady Eidson.

  • Scripts/svn-apply: (applyPatch): Add "--force" to $options arrayref if $force is set.
11:17 AM Changeset in webkit [47944] by pfeldman@chromium.org
  • 3 edits in trunk/WebCore

2009-09-01 Yury Semikhatsky <yurys@chromium.org>

Reviewed by Timothy Hatcher.

Check that InspectorDOMAgent and InspectorFrontend are not null
before accessing them in InspectorBackend. They may be null if
InspectorController is closed before an asynchronous operation is
started.

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

  • inspector/InspectorBackend.cpp: (WebCore::InspectorBackend::getCookies): (WebCore::InspectorBackend::getChildNodes): (WebCore::InspectorBackend::setAttribute): (WebCore::InspectorBackend::removeAttribute): (WebCore::InspectorBackend::setTextNodeValue): (WebCore::InspectorBackend::highlight): (WebCore::InspectorBackend::nodeForId): (WebCore::InspectorBackend::pushNodePathToFrontend): (WebCore::InspectorBackend::addNodesToSearchResult): (WebCore::InspectorBackend::selectDatabase): (WebCore::InspectorBackend::selectDOMStorage): (WebCore::InspectorBackend::inspectorDOMAgent): (WebCore::InspectorBackend::inspectorFrontend):
  • inspector/InspectorBackend.h:
11:13 AM Changeset in webkit [47943] by beidson@apple.com
  • 8 edits in trunk/WebCore

Page Cache should support pages with frames
https://bugs.webkit.org/show_bug.cgi?id=13631

Reviewed by Sam Weinig.

Things learned in the full task that this patch addresses:

  • The notions of clearing a CachedFrame and destroying a CachedFrame were two concepts that needed to be managed separately.
  • Once we start restoring pages with multiple CachedFrames, we'll have to be calling FrameLoader::open(CachedFrame&) multiple times with different CachedFrames, and this process will be guided by the CachedFrames themselves.

No new tests. (No change in behavior, current layout tests pass)

  • history/CachedFrame.cpp:

(WebCore::CachedFrame::CachedFrame):
(WebCore::CachedFrame::~CachedFrame):
(WebCore::CachedFrame::clear): Only clear pointers out.
(WebCore::CachedFrame::destroy): Perform the destructive cleanup work here - Things that shouldn't

occur simply by navigating back to a page with a CachedFrame.

  • history/CachedFrame.h:
  • history/CachedPage.cpp:

(WebCore::CachedPage::~CachedPage):
(WebCore::CachedPage::restore): Call clear() after restoring.
(WebCore::CachedPage::clear):
(WebCore::CachedPage::destroy):

  • history/CachedPage.h:
  • history/PageCache.cpp:

(WebCore::PageCache::releaseAutoreleasedPagesNow): destroy() CachedPages that are being pruned,

instead of clearing them.

  • loader/FrameLoader.cpp:

(WebCore::FrameLoader::commitProvisionalLoad): Let the CachedPage clear() itself after restoring.
(WebCore::FrameLoader::open): Don't ::open() the CachedFrame, as the CachedPage does that now.

  • loader/FrameLoader.h: Add the friending so CachedPage and CachedFrame can better guide the process. Even though CachedFrame::restore() isn't used for now, it will be soon.
11:06 AM Coding Style Guidelines edited by berkeley@ptc.com
(diff)
11:00 AM Changeset in webkit [47942] by benm@google.com
  • 3 edits in trunk/WebCore

Add a "lowMemoryNotification" method to the V8 ScriptController class.
This is needed in order to allow the host application (Android Web browser in this case)
to call the v8::V8::LowMemoryNotification() method added to V8 in
http://code.google.com/p/v8/source/detail?spec=svn2777&r=2725

Patch by Andrei Popescu <andreip@google.com> on 2009-09-01
Reviewed by David Levin.

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

  • bindings/v8/ScriptController.cpp:

(WebCore::ScriptController::lowMemoryNotification):

  • bindings/v8/ScriptController.h:
9:39 AM Changeset in webkit [47941] by abarth@webkit.org
  • 9 edits
    1 add in trunk/WebCore

2009-09-01 Fumitoshi Ukai <ukai@chromium.org>

Reviewed by Adam Barth.

V8 Bindings for WebSocket API.
https://bugs.webkit.org/show_bug.cgi?id=28844

  • bindings/v8/DOMObjectsInclude.h:
  • bindings/v8/DerivedSourcesAllInOne.cpp:
  • bindings/v8/V8DOMWrapper.cpp: (WebCore::V8DOMWrapper::getTemplate): (WebCore::V8DOMWrapper::convertToV8Object): (WebCore::V8DOMWrapper::convertEventTargetToV8Object):
  • bindings/v8/V8DOMWrapper.h:
  • bindings/v8/V8Index.cpp:
  • bindings/v8/V8Index.h:
  • bindings/v8/WorkerContextExecutionProxy.cpp:
  • bindings/v8/custom/V8CustomBinding.h:
  • bindings/v8/custom/V8WebSocketCustom.cpp: Added.
9:35 AM Changeset in webkit [47940] by Adam Roben
  • 4 edits in trunk/WebKit/win

Post a notification when a WebView is closing

This is useful for situations where some code wants to react to the
WebView closing but isn't in charge of the WebView and can't otherwise
be notified. This notification could replace
IWebUIDelegatePrivate::webViewClosing if we decide that the
notification system isn't too burdensome.

<http://webkit.org/b/28875>

Reviewed by John Sullivan.

  • Interfaces/IWebViewPrivate.idl: Added the

WebViewWillCloseNotification string constant.

  • WebKit.vcproj/WebKit.vcproj: Link against comsuppw.lib in all

configurations so that we can use _bstr_t.

  • WebView.cpp:

(WebView::close): Post the WebViewWillCloseNotification.

9:17 AM Changeset in webkit [47939] by pkasting@chromium.org
  • 2 edits in trunk/WebCore

Unreviewed (build fix).

Missed a "." -> "->" conversion.

  • platform/graphics/qt/ImageDecoderQt.cpp:

(WebCore::ImageDecoderQt::ReadContext::ReadContext):

9:12 AM Changeset in webkit [47938] by pkasting@chromium.org
  • 3 edits in trunk/WebCore

Unreviewed (build fix).

Try to fix Qt build, take 2. The Qt code had what looks like a
broken virtual function override.

  • platform/graphics/qt/ImageDecoderQt.cpp:

(WebCore::ImageDecoderQt::ReadContext::ReadContext):
(WebCore::ImageDecoderQt::setData):

  • platform/graphics/qt/ImageDecoderQt.h:
9:09 AM Changeset in webkit [47937] by abarth@webkit.org
  • 2 edits in trunk/WebCore

2009-09-01 Adam Barth <abarth@webkit.org>

Reviewed by Sam Weinig.

Uninitialized variable in XSSAuditor
https://bugs.webkit.org/show_bug.cgi?id=28874

One ought to initialize variables before use even when it doesn't
matter. No behavior change.

  • page/XSSAuditor.h: (WebCore::XSSAuditor::CachingURLCanonicalizer::CachingURLCanonicalizer):
8:57 AM Changeset in webkit [47936] by pkasting@chromium.org
  • 2 edits in trunk/WebCore

Unreviewed (build fix).

Try to fix Qt build.

  • platform/graphics/ImageSource.cpp:
8:50 AM Changeset in webkit [47935] by pkasting@chromium.org
  • 4 edits in trunk/WebCore

https://bugs.webkit.org/show_bug.cgi?id=27965
Eliminate most of ImageSourceQt.cpp in favor of ImageSource.cpp.

Reviewed by Eric Seidel.

  • WebCore.pro:
  • platform/graphics/ImageSource.cpp:

(WebCore::ImageSource::frameHasAlphaAtIndex):

  • platform/graphics/qt/ImageSourceQt.cpp:

(WebCore::ImageSource::createFrameAtIndex):
(WebCore::ImageSource::frameHasAlphaAtIndex):
(WebCore::ImageSource::frameIsCompleteAtIndex):

7:58 AM Changeset in webkit [47934] by pfeldman@chromium.org
  • 4 edits in trunk/WebCore

2009-09-01 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Timothy Hatcher.

WebInspector: Make InjectedScript self-contained (move necessary
methods from utilities.js into InjectedScript).

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

  • inspector/front-end/ElementsPanel.js: (WebInspector.ElementsPanel.prototype.generateStylesheet):
  • inspector/front-end/InjectedScript.js: (InjectedScript._ensureCommandLineAPIInstalled.inspectObject): (InjectedScript._ensureCommandLineAPIInstalled): (InjectedScript.pushNodeToFrontend): (InjectedScript.createProxyObject): (Object.type): (Object.hasProperties): (Object.className): (String.prototype.escapeCharacters):
  • inspector/front-end/utilities.js:
7:41 AM Changeset in webkit [47933] by pfeldman@chromium.org
  • 2 edits in trunk/WebCore

2009-09-01 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Timothy Hatcher.

WebInspector: console log message repeat count double.

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

  • inspector/front-end/InjectedScript.js: (InjectedScript._evaluateOn):
7:11 AM Changeset in webkit [47932] by ddkilzer@apple.com
  • 2 edits in trunk/WebKitTools

<http://webkit.org/b/28725> resolve-ChangeLogs: determineVCSRoot() returns incorrect repository root during git filter-branch

Reviewed by Adam Roben.

When git-filter-branch has been invoked to rewrite ChangeLog
files on series of git commits, it changes directories into
.git-rewrite/t before re-running resolve-ChangeLogs. This
causes determineVCSRoot() in VCSUtils.pm to return
".git-rewrite/t", which causes that path to be prepended to all
ChangeLog paths, which results in an error like this:

error: pathspec '.git-rewrite/t/ChangeLog' did not match any file(s) known to git.
Died at WebKitTools/Scripts/resolve-ChangeLogs line 376.

The correct way to fix this is not to try to find the repository
root when invoked by git-filter-branch.

  • Scripts/resolve-ChangeLogs: If isInGitFilterBranch() is true, set $relativePath to '.' instead of calling chdirReturningRelativePath(determineVCSRoot()). (isInGitFilterBranch): Added. Checks for the existence of the MAPPED_PREVIOUS_COMMIT environment variable.
6:06 AM Changeset in webkit [47931] by kov@webkit.org
  • 4 edits in trunk/WebKit/gtk

2009-09-01 Gustavo Noronha Silva <gustavo.noronha@collabora.co.uk>

Reviewed by Xan Lopez.

Avoid causing unexpected issues with window sizing when sites
disable scrollbars completely.

  • WebCoreSupport/ChromeClientGtk.cpp: (WebKit::ChromeClient::scrollbarsModeDidChange):
  • tests/testwindow.c: (test_webkit_window_scrollbar_policy):
  • webkit/webkitwebframe.cpp: (webkit_web_frame_class_init):
5:50 AM Changeset in webkit [47930] by xan@webkit.org
  • 5 edits
    1 add in trunk

LayoutTests:

2009-09-01 Xan Lopez <xlopez@igalia.com>

Reviewed by Gustavo Noronha.

Add support for Fedora distros in the http tests
https://bugs.webkit.org/show_bug.cgi?id=28263

Add httpd conf file for Fedora.

  • http/conf/fedora-httpd.conf: Added.

WebKitTools:

2009-09-01 Xan Lopez <xlopez@igalia.com>

Reviewed by Gustavo Noronha.

Add support for Fedora distros in the http tests
https://bugs.webkit.org/show_bug.cgi?id=28263

Add detection code for Fedora distribution, and use the proper
httpd conf file when needed.

  • Scripts/run-webkit-httpd:
  • Scripts/run-webkit-tests:
  • Scripts/webkitdirs.pm:
5:19 AM Changeset in webkit [47929] by jmalonzo@webkit.org
  • 6 edits
    3 adds in trunk

2009-09-01 Jan Michael Alonzo <jmalonzo@webkit.org>

Reviewed by Xan Lopez and Gustavo Noronha.

[Gtk] Implement a WebDataSource for the gtk port
https://bugs.webkit.org/show_bug.cgi?id=24758

Implement WebKitWebResource for the resource-related API for
WebKitWebDataSource.

  • GNUmakefile.am:

Add WebKitWebResource to the build script.

  • GNUmakefile.am:
5:05 AM Changeset in webkit [47928] by jmalonzo@webkit.org
  • 5 edits in trunk

2009-09-01 Jan Michael Alonzo <jmalonzo@webkit.org>

Reviewed by Eric Seidel.

[Gtk] DRT needs implementation of overridePreference
https://bugs.webkit.org/show_bug.cgi?id=28830

Unskip tests that are now working.

  • platform/gtk/Skipped:

Implement overridePreference.

  • DumpRenderTree/gtk/DumpRenderTree.cpp: (resetDefaultsToConsistentValues): (setDefaultsToConsistentStateValuesForTesting): (runTest):
  • DumpRenderTree/gtk/LayoutTestControllerGtk.cpp: (toWebSettingKey): (LayoutTestController::overridePreference):
4:55 AM Changeset in webkit [47927] by jmalonzo@webkit.org
  • 4 edits in trunk

2009-09-01 Jan Michael Alonzo <jmalonzo@webkit.org>

Reviewed by Mark Rowe.

[Gtk] Fix DATALIST build
https://bugs.webkit.org/show_bug.cgi?id=28826

Unskip now working fast/forms/input-list.html.

  • platform/gtk/Skipped:

Add ENABLE_DATALIST to FEATURE_DEFINES.

  • GNUmakefile.am:
4:11 AM Changeset in webkit [47926] by eric@webkit.org
  • 3 edits in trunk/WebCore

2009-09-01 Cameron McCormack <cam@mcc.id.au>

Reviewed by Eric Seidel.

SVGSVGElement suspend methods argument/return types incorrect
https://bugs.webkit.org/show_bug.cgi?id=28860

Also fix the style of the argument names.

  • svg/SVGSVGElement.cpp: (WebCore::SVGSVGElement::suspendRedraw): (WebCore::SVGSVGElement::unsuspendRedraw):
  • svg/SVGSVGElement.h:
4:01 AM Changeset in webkit [47925] by eric@webkit.org
  • 3 edits in trunk/WebCore

2009-09-01 Roland Steiner <rolandsteiner@google.com>

Reviewed by Eric Seidel.

Fix bug 28808: [Skia] Fix flaky layout test svg/dynamic-updates/SVGClipPathElement-dom-clipPathUnits-attr.html [DEBUG]
(https://bugs.webkit.org/show_bug.cgi?id=28808)

The bug fix adds an explicit check for the validity of the path when it is
transformed into local coordinates.
This was the most suitable place I could find that didn't cause an inordinate
amount of refactoring. Also, it seems pertinent and may also catch other cases
that are not limited to clipping paths.

TEST: existing svg/dynamic-updates/SVGClipPathElement-dom-clipPathUnits-attr.html

  • platform/graphics/skia/GraphicsContextSkia.cpp: make isPathSkiaSafe accessible (WebCore::):
  • platform/graphics/skia/PlatformContextSkia.cpp: (PlatformContextSkia::currentPathInLocalCoordinates): check Skia safety of path
3:54 AM Changeset in webkit [47924] by xan@webkit.org
  • 4 edits in trunk/WebKit/gtk

2009-09-01 Xan Lopez <xlopez@igalia.com>

Reviewed by Jan Alonzo.

https://bugs.webkit.org/show_bug.cgi?id=28842
[GTK] Do not emit extra FINISHED load-status signals

Do not emit FINISHED load-status signals on our own, just rely on
WebCore for that. Avoids emitting FINISHED several times when we
fail to load a URI.

  • WebCoreSupport/FrameLoaderClientGtk.cpp: (WebKit::FrameLoaderClient::dispatchDidFailProvisionalLoad): (WebKit::FrameLoaderClient::dispatchDidFailLoad):
  • tests/testloading.c: (web_loading_fixture_setup): (web_loading_fixture_teardown): (load_finished_cb): (status_changed_cb): (test_loading_status): (load_error_status_changed_cb): (load_error_cb): (test_loading_error): (main):
  • webkit/webkitwebframe.h:
3:24 AM Changeset in webkit [47923] by eric@webkit.org
  • 3 edits in trunk/WebKitTools

2009-09-01 Joseph Pecoraro <Joseph Pecoraro>

<http://webkit.org/b/28623> svn-[un]apply should change directories to the repository root before [un]applying

Reviewed by Eric Seidel.

Jump back and forth between the repository root directory (to apply) and the
directory the script was run from (to find the patch).

  • Scripts/svn-apply:
  • Scripts/svn-unapply:
3:14 AM Changeset in webkit [47922] by eric@webkit.org
  • 2 edits in trunk/WebCore

2009-09-01 Yusuke Sato <yusukes@chromium.org>

Reviewed by Eric Seidel.

[Chromium] Combining Diacritical Marks (U+0300..) are not handled correctly.
https://bugs.webkit.org/show_bug.cgi?id=28742

Normalize (NFC) a TextRun when the run contains combining diacritical marks
so that Skia can pick a correct glyph without relying on GSUB table in a font.
This change is for Chromium Linux.

  • platform/graphics/chromium/FontLinux.cpp: (WebCore::TextRunWalker::getTextRun): Added. (WebCore::TextRunWalker::getNormalizedTextRun): Added.

This function is called when the run contains U+0300..U+036F and converts characters in
the run to the combined form (NFC) using ICU.

3:05 AM Changeset in webkit [47921] by eric@webkit.org
  • 5 edits in trunk/WebCore

2009-09-01 Vitaly Repeshko <vitalyr@chromium.org>

Reviewed by Dimitri Glazkov.

[V8] Inlined fast paths of V8DOMWrapper::convertNodeToV8Object and
V8Proxy::createWrapperFromCache.
https://bugs.webkit.org/show_bug.cgi?id=28848

  • bindings/v8/V8DOMWrapper.cpp: (WebCore::V8DOMWrapper::convertDocumentToV8Object): Special function for document wrapping. (WebCore::V8DOMWrapper::convertNewNodeToV8Object): Slow case of wrapping a new node.
  • bindings/v8/V8DOMWrapper.h: (WebCore::V8DOMWrapper::convertNodeToV8Object): Extracted fast path from V8DOMWrapper.cpp.
  • bindings/v8/V8Proxy.cpp: (WebCore::V8Proxy::createWrapperFromCacheSlowCase):
  • bindings/v8/V8Proxy.h: (WebCore::V8Proxy::createWrapperFromCache): Extracted fast path from V8Proxy.cpp.
2:34 AM Changeset in webkit [47920] by eric@webkit.org
  • 2 edits in trunk/WebCore

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

No review, windows build fix only. Clearly I should have read the patch closer. :(

Allow excluding certain plugins from loading
https://bugs.webkit.org/show_bug.cgi?id=28677

  • plugins/PluginDatabase.cpp: (WebCore::PluginDatabase::refresh):
2:31 AM Changeset in webkit [47919] by pfeldman@chromium.org
  • 9 edits in trunk/WebCore

2009-09-01 Alexander Pavlov <apavlov@chromium.org>

Reviewed by Timothy Hatcher.

Implement conditional breakpoints in the Web Inspector backend
and add frontend JavaScript stubs.
https://bugs.webkit.org/show_bug.cgi?id=28846

  • inspector/InspectorBackend.cpp: (WebCore::InspectorBackend::profiles): (WebCore::InspectorBackend::addBreakpoint): (WebCore::InspectorBackend::updateBreakpoint):
  • inspector/InspectorBackend.h:
  • inspector/InspectorBackend.idl:
  • inspector/JavaScriptDebugServer.cpp: (WebCore::JavaScriptDebugServer::BreakpointInfo::condition): (WebCore::JavaScriptDebugServer::BreakpointInfo::setCondition): (WebCore::JavaScriptDebugServer::addBreakpoint): (WebCore::JavaScriptDebugServer::breakpointInfo): (WebCore::JavaScriptDebugServer::updateBreakpoint): (WebCore::JavaScriptDebugServer::updateBreakpointInfo): (WebCore::JavaScriptDebugServer::removeBreakpoint): (WebCore::JavaScriptDebugServer::hasBreakpoint): (WebCore::JavaScriptDebugServer::clearBreakpoints):
  • inspector/JavaScriptDebugServer.h: (WebCore::JavaScriptDebugServer::BreakpointInfo::BreakpointInfo):
  • inspector/front-end/Breakpoint.js: (WebInspector.Breakpoint): (WebInspector.Breakpoint.prototype.get id): (WebInspector.Breakpoint.prototype.get condition): (WebInspector.Breakpoint.prototype.set condition):
  • inspector/front-end/BreakpointsSidebarPane.js: (WebInspector.BreakpointsSidebarPane.prototype.addBreakpoint): (WebInspector.BreakpointsSidebarPane.prototype._breakpointEnableChanged):
  • inspector/front-end/ScriptsPanel.js: (WebInspector.ScriptsPanel.prototype.addScript):
2:13 AM Changeset in webkit [47918] by eric@webkit.org
  • 3 edits
    1 add in trunk/WebCore

2009-09-01 Marius Renn <damarvy@gmail.com>

Reviewed by Eric Seidel.

Added delegate to PluginDatabase to disable loading certain plugins.
This is useful when you want to load from the standard plugin
directory, but want to exclude certain plugins. Plugins may be
excluded early on by name, so that no plugin code is executed (which
may have caused a crash).

  • plugins/PluginDatabase.cpp: (WebCore::PluginDatabase::PluginDatabase): (WebCore::PluginDatabase::refresh):
  • plugins/PluginDatabase.h: (WebCore::PluginDatabase::setClient):
  • plugins/PluginDatabaseClient.h: Added. (WebCore::PluginDatabaseClient::~PluginDatabaseClient):

Aug 31, 2009:

8:45 PM Changeset in webkit [47917] by beidson@apple.com
  • 2 edits in trunk/WebCore

Rubberstamped by Sam Weinig

  • WebCore.base.exp: Removed an unneeded symbol export.
6:34 PM Changeset in webkit [47916] by kov@webkit.org
  • 7 edits in trunk

WebCore

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

Reviewed by Alexey Proskuryakov.

https://bugs.webkit.org/show_bug.cgi?id=28466
When downloading a file, two GET requests are sent to the HTTP server

Can't think of a way to test this.

  • loader/ResourceLoader.cpp: (WebCore::ResourceLoader::releaseResources): only nullify the handle's client when it is the ResourceLoader, to support the case in which a download API uses a different client

WebKit/gtk

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

Reviewed by Alexey Proskuryakov.

https://bugs.webkit.org/show_bug.cgi?id=28466
When downloading a file, two GET requests are sent to the HTTP server

Support reusing a connection that is already ongoing, instead of
cancelling the connection and creating another.

  • WebCoreSupport/FrameLoaderClientGtk.cpp: (WebKit::FrameLoaderClient::download):
  • webkit/webkitdownload.cpp: (webkit_download_new_with_handle): (webkit_download_start):
  • webkit/webkitprivate.h:
  • webkit/webkitwebview.cpp: (webkit_web_view_request_download):
4:44 PM Changeset in webkit [47915] by bweinstein@apple.com
  • 5 edits in trunk

2009-08-31 Brian Weinstein <bweinstein@apple.com>

Reviewed by Darin Adler.

Text Fields and Text Areas are reported as read-only by inspect32.exe.
https://bugs.webkit.org/show_bug.cgi?id=28854

Added another case in AcccessibiltyRenderObject::isReadOnly to test if text fields
and texts areas are read-only.

  • accessibility/AccessibilityRenderObject.cpp: (WebCore::AccessibilityRenderObject::isReadOnly): (WebCore::AccessibilityRenderObject::canSetValueAttribute):

2009-08-31 Brian Weinstein <bweinstein@apple.com>

Reviewed by Darin Adler.

Added a case to check for readonly and non-readonly set through the readonly attribute
instead of through aria-readonly. This tests our new case in AccessibilityRenderObject::isReadOnly.

  • accessibility/aria-readonly-expected.txt:
  • accessibility/aria-readonly.html:
4:40 PM Changeset in webkit [47914] by atwilson@chromium.org
  • 12 edits in trunk

WebCore: fast/workers/worker-lifecycle.html crashes intermittently on snowleopard
https://bugs.webkit.org/show_bug.cgi?id=28795

Reviewed by Alexey Proskuryakov.

Changed WorkerContext::close() to no longer stop the worker thread.
It is now the parent's responsibility to shut down the worker thread when it is notified that the context is closed.

  • workers/AbstractWorker.cpp:

Updated dispatchScriptErrorEvent to pass-through the "handled" value from dispatchEvent(), to allow event handlers added with addEventListener() to mark events as handled.

  • workers/DefaultSharedWorkerRepository.cpp:

(WebCore::SharedWorkerProxy::workerContextClosed):
Now shuts down the worker thread when notified that the context is closed.
(WebCore::SharedWorkerProxy::close):
Now handles being invoked when the context is already in the process of shutting down.

  • workers/WorkerContext.cpp:

(WebCore::WorkerContext::close):
No longer calls WorkerThread::stop() to avoid race conditions with shutting down the thread while the parent is still interacting with it.

  • workers/WorkerContext.h:

(WebCore::WorkerContext::isClosing):
isClosing() is now public so WorkerRunLoop can call it to determine whether to process tasks.

  • workers/WorkerMessagingProxy.cpp:

(WebCore::WorkerExceptionTask::performTask):
Error events should still be delivered even if the worker thread is closing.
Also fixed problem where error events were not dispatched if error handlers were added via addEventListener().
(WebCore::WorkerTerminateTask::create):
New task that invokes terminateWorkerContext() on the parent thread when the context is closed.
(WebCore::WorkerTerminateTask::WorkerTerminateTask):
(WebCore::WorkerTerminateTask::performTask):
(WebCore::WorkerMessagingProxy::workerContextClosed):
Now fires off a WorkerTerminateTask to shut down the thread when the context is closed.

  • workers/WorkerMessagingProxy.h:

WorkerMessagingProxy now overrides workerContextClosed().

  • workers/WorkerRunLoop.cpp:

(WebCore::WorkerRunLoop::runInMode):
Now drops tasks on the floor if the WorkerContext is closing.

LayoutTests: fast/workers/worker-lifecycle.html crashes intermittently on snowleopard
https://bugs.webkit.org/show_bug.cgi?id=28795

Reviewed by Alexey Proskuryakov.

Removed worker-lifecycle.html from the list of Skipped tests since it no longer crashes.
Added test for case where we capture script errors via an event listener added via addEventListener().

  • fast/workers/worker-script-error.html
  • fast/workers/worker-script-error-expected.txt

Added testScriptErrorAddEventListener to test catching script errors via addEventListener().

  • platform/mac-snowleopard/Skipped:
4:36 PM Changeset in webkit [47913] by Beth Dakin
  • 2 edits in trunk/LayoutTests

Rubber-stamped by Mark Rowe.

Meant to update these results with my earlier check-in, but I
forgot!

  • platform/mac/fast/inspector/style-expected.txt:
4:30 PM Changeset in webkit [47912] by ojan@chromium.org
  • 4 edits in trunk/WebCore

Not reviewed (build fix)

Build fix for Chromium to match r49707.

  • bindings/v8/ScriptCallFrame.cpp:

(WebCore::ScriptCallFrame::ScriptCallFrame):

  • bindings/v8/ScriptSourceCode.h:

(WebCore::ScriptSourceCode::ScriptSourceCode):

  • platform/KURLGoogle.cpp:

(WebCore::KURL::KURL):
(WebCore::blankURL):

4:01 PM Changeset in webkit [47911] by ap@apple.com
  • 2 edits in trunk/WebKit/gtk

Gtk build fix.

This is also a behavior fix, at least on debug builds - a caller of
webkit_web_history_item_new_with_data() could pass an URL that could cause an assertion
failure in KURL.

  • webkit/webkitwebhistoryitem.cpp: (webkit_web_history_item_new_with_data): Pass a null KURL as base to parse properly.
3:49 PM Changeset in webkit [47910] by ap@apple.com
  • 3 edits
    2 adds in trunk

Reviewed by Darin Adler.

https://bugs.webkit.org/show_bug.cgi?id=28858
Element.baseURI parses xml:base attribute incorrectly

Test: fast/dom/base-attribute-parsing.xhtml

  • dom/Element.cpp: (WebCore::Element::baseURI): Avoid assertion failure (no change in release mode).
3:27 PM Changeset in webkit [47909] by Dimitri Glazkov
  • 4 edits in trunk

WebCore:

2009-08-31 Dimitri Glazkov <Dimitri Glazkov>

Reverting http://trac.webkit.org/changeset/47904, because it caused
layout test failure.

Test: fast/dom/Window/new-window-opener.html

WebKit/mac:

2009-08-31 Dimitri Glazkov <Dimitri Glazkov>

Reverting http://trac.webkit.org/changeset/47904, because it caused
layout test failure.

3:09 PM Changeset in webkit [47908] by mrowe@apple.com
  • 2 edits
    2 deletes in trunk/WebKit

Remove WebViewEditingContextMenu.nib and WebViewEditingContextMenuOld.nib.

Rubber-stamped by Beth Dakin.

We stopped using these back in r18592 when we switched over to context menus
driven by WebCore.

  • English.lproj/WebViewEditingContextMenu.nib/classes.nib: Removed.
  • English.lproj/WebViewEditingContextMenu.nib/info.nib: Removed.
  • English.lproj/WebViewEditingContextMenu.nib/objects.nib: Removed.
  • English.lproj/WebViewEditingContextMenuOld.nib/classes.nib: Removed.
  • English.lproj/WebViewEditingContextMenuOld.nib/info.nib: Removed.
  • English.lproj/WebViewEditingContextMenuOld.nib/objects.nib: Removed.
  • WebKit.xcodeproj/project.pbxproj:
3:07 PM Changeset in webkit [47907] by ap@apple.com
  • 50 edits in trunk

Reviewed by Darin Adler.

https://bugs.webkit.org/show_bug.cgi?id=28852
Rename KURL single argument constructor to avoid confusion

  • platform/KURL.h: The constructor that used to be single argument should now be invoked as KURL(ParsedURLString, myString).
3:05 PM Changeset in webkit [47906] by Beth Dakin
  • 22 edits
    2 deletes in trunk

WebCore: Fix for https://bugs.webkit.org/show_bug.cgi?id=28635 [CSS3
Backgrounds and Borders] Add support for 2-keyword values for
background-repeat

Reviewed by Darin Adler.

This patch allows background-repeat to take two values by making
background-repeat just like background-position internally. There
is a little extra legwork for background-repeat because the spec
indicates that its computed value should be equivalent to how it
was specified. I keep track of the specified thing by setting the
implicit flag whenever background-repeat is defined with only one
value (since internally, we now store this as 2 values.)

Here we can't access the implicit flag, so for backwards-
compatibility's sake, we always return one value when that makes
sense.

  • css/CSSComputedStyleDeclaration.cpp:

(WebCore::fillRepeatToCSSValue):
(WebCore::CSSComputedStyleDeclaration::getPropertyCSSValue):

Here we check for the implicit flag and return one value when it is
set.

  • css/CSSMutableStyleDeclaration.cpp:

(WebCore::CSSMutableStyleDeclaration::getPropertyValue):
(WebCore::CSSMutableStyleDeclaration::getLayeredShorthandValue):
(WebCore::CSSMutableStyleDeclaration::cssText):

Add support for CSSPropertyBackgroundRepeatX and
CSSPropertyBackgroundRepeatY

  • css/CSSParser.cpp:

(WebCore::CSSParser::parseValue):
(WebCore::CSSParser::parseFillShorthand):
(WebCore::CSSParser::parseFillRepeat):
(WebCore::CSSParser::parseFillProperty):

  • css/CSSParser.h:

Get rid of mappings to RepeatXFill and RepeatYFill since we don't
need those parts of the EFillRepeat enum anymore.

  • css/CSSPrimitiveValueMappings.h:

(WebCore::CSSPrimitiveValue::CSSPrimitiveValue):
(WebCore::CSSPrimitiveValue::operator EFillRepeat):

Add CSSPropertyBackgroundRepeatX and CSSPropertyBackgroundRepeatY

  • css/CSSPropertyLonghand.cpp:

(WebCore::initShorthandMap):

Add background-repeat-x and -y.

  • css/CSSPropertyNames.in:

Break repeat into x and y.

  • css/CSSStyleSelector.cpp:

(WebCore::CSSStyleSelector::applyProperty):
(WebCore::CSSStyleSelector::mapFillRepeatX):
(WebCore::CSSStyleSelector::mapFillRepeatY):

  • css/CSSStyleSelector.h:

There is no more RepeatXFill. Instead, look for
fillLayer->repeatX() == RepeatFill

  • rendering/RenderBoxModelObject.cpp:

(WebCore::RenderBoxModelObject::calculateBackgroundImageGeometry):

Break m_repeat into m_repeatX and m_repeatY

  • rendering/style/FillLayer.cpp:

(WebCore::FillLayer::FillLayer):
(WebCore::FillLayer::operator=):
(WebCore::FillLayer::operator==):
(WebCore::FillLayer::fillUnsetProperties):
(WebCore::FillLayer::cullEmptyLayers):

  • rendering/style/FillLayer.h:

(WebCore::FillLayer::repeatX):
(WebCore::FillLayer::repeatY):
(WebCore::FillLayer::isRepeatXSet):
(WebCore::FillLayer::isRepeatYSet):
(WebCore::FillLayer::setRepeatX):
(WebCore::FillLayer::setRepeatY):
(WebCore::FillLayer::clearRepeatX):
(WebCore::FillLayer::clearRepeatY):
(WebCore::FillLayer::initialFillRepeatX):
(WebCore::FillLayer::initialFillRepeatY):

Again, break backgroundRepeat into backgroundRepeatX and
backgroundRepeatY

  • rendering/style/RenderStyle.h:

(WebCore::InheritedFlags::backgroundRepeatX):
(WebCore::InheritedFlags::backgroundRepeatY):
(WebCore::InheritedFlags::maskRepeatX):
(WebCore::InheritedFlags::maskRepeatY):

We don't need RepeatXFill or RepeatYFill. Now that we store two
values, they are represented by Repeat-NoRepeat and NoRepeat-
Repeat, respectively.

  • rendering/style/RenderStyleConstants.h:

(WebCore::):

LayoutTests: Tests for https://bugs.webkit.org/show_bug.cgi?id=28635 [CSS3
Backgrounds and Borders] Add support for 2-keyword values for
background-repeat

Reviewed by Darin Adler.

This test is updated to test for two values.

  • fast/backgrounds/repeat/parsing-background-repeat-expected.txt:
  • fast/backgrounds/repeat/resources/parsing-background-repeat.js:

These tests are updated to reflect new order.

  • fast/css/background-position-serialize-expected.txt:
  • fast/css/remove-shorthand-expected.txt:

Reflect new order in results, and make the test dump as text.

  • fast/inspector/style.html:
  • platform/mac/fast/inspector/style-expected.checksum: Removed.
  • platform/mac/fast/inspector/style-expected.png: Removed.
  • platform/mac/fast/inspector/style-expected.txt:
2:00 PM Changeset in webkit [47905] by Simon Fraser
  • 3 edits
    4 adds in trunk

2009-08-31 Simon Fraser <Simon Fraser>

Reviewed by Dan Bernstein.

Compositing layer size needs to take into account software reflections on children.
https://bugs.webkit.org/show_bug.cgi?id=28837

When computing the bounds of a composited layer, take software-rendered
reflections into account.

Test: compositing/reflections/reflection-in-composited.html

  • rendering/RenderLayerCompositor.cpp: (WebCore::RenderLayerCompositor::calculateCompositedBounds):
1:37 PM Changeset in webkit [47904] by Dimitri Glazkov
  • 4 edits in trunk

WebCore:

2009-08-31 Mark Mentovai <mark@chromium.org>

Reviewed by Dave Hyatt.

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

Perform a layout prior to checking whether the scrollbar modes are
off, on, or automatic. The modes may change during layout.

  • platform/ScrollView.cpp: (WebCore::ScrollView::updateScrollbars):

WebKit/mac:

2009-08-31 Mark Mentovai <mark@chromium.org>

Reviewed by Dave Hyatt.

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

Perform a layout prior to checking whether the scrollbar modes are
off, on, or automatic. The modes may change during layout.

  • WebView/WebDynamicScrollBarsView.mm: (-[WebDynamicScrollBarsView updateScrollers]):
1:15 PM Changeset in webkit [47903] by eric@webkit.org
  • 8 edits
    1 add in trunk/WebCore

2009-08-31 Patrick Mueller <Patrick_Mueller@us.ibm.com>

Reviewed by Timothy Hatcher.

Inspector Request Headers Should Show Data Sent With Request
https://bugs.webkit.org/show_bug.cgi?id=22920

Manual test added - see below.

  • English.lproj/localizedStrings.js:
  • inspector/InspectorResource.cpp: (WebCore::InspectorResource::updateRequest): (WebCore::InspectorResource::createScriptObject): (WebCore::InspectorResource::updateScriptObject):
  • inspector/InspectorResource.h:
  • inspector/front-end/Resource.js: (WebInspector.Resource):
  • inspector/front-end/ResourceView.js: (WebInspector.ResourceView): (WebInspector.ResourceView.prototype._refreshURL): (WebInspector.ResourceView.prototype._refreshQueryString): (WebInspector.ResourceView.prototype._refreshFormData): (WebInspector.ResourceView.prototype._refreshRequestPayload): (WebInspector.ResourceView.prototype._refreshParms): (WebInspector.ResourceView.prototype._toggleURLdecoding): (WebInspector.ResourceView.prototype._getHeaderValue): (WebInspector.ResourceView.prototype._refreshRequestHeaders):
  • inspector/front-end/inspector.css:
  • inspector/front-end/inspector.js: (WebInspector.addResource): (WebInspector.updateResource):
  • manual-tests/inspector/display-form-data.html: Added.
12:31 PM Changeset in webkit [47902] by barraclough@apple.com
  • 2 edits in trunk/JavaScriptCore

Fixed typo.
https://bugs.webkit.org/show_bug.cgi?id=28691

Patch by Robert Agoston <Agoston.Robert@stud.u-szeged.hu> on 2009-08-31
Reviewed by Gavin Barraclough.

  • parser/Parser.h:

(JSC::Parser::parse):

12:01 PM Changeset in webkit [47901] by Simon Fraser
  • 3 edits in trunk/LayoutTests

2009-08-31 Simon Fraser <Simon Fraser>

Followup from r47877; add test for the Obj-C wrapper class for WebkitCSSTransformValue
https://bugs.webkit.org/show_bug.cgi?id=27727

  • platform/mac/fast/dom/wrapper-classes-objc-expected.txt:
  • platform/mac/fast/dom/wrapper-classes-objc.html:
11:37 AM Changeset in webkit [47900] by Simon Fraser
  • 3 edits
    8 adds in trunk

2009-08-28 Simon Fraser <Simon Fraser>

Reviewed by Dan Bernstein.

Fix positioning error when a compositing, absolutely-positioned element has an ancestor with opacity.
https://bugs.webkit.org/show_bug.cgi?id=28754

For absolutely-positioned layers, convertToLayerCoords() jumps to the positioned
ancestor, since the layer x and y are relative to that ancestor. However, this could
skip over the ancestorLayer, thus giving the wrong result.

Fix by duplicating the enclosingPositionedAncestor() logic in convertToLayerCoords(),
and checking for ancestorLayer along the way. If found, compute offset of both to
the enclosingPositionedAncestor() and subtract.

This also fixes a positioning bug with abs. positioned elements in reflections,
so there is a new reflection test with a pixel result.

Tests: compositing/geometry/abs-position-inside-opacity.html

fast/reflections/abs-position-in-reflection.html

  • rendering/RenderLayer.cpp: (WebCore::isPositionedContainer): (WebCore::RenderLayer::enclosingPositionedAncestor): (WebCore::RenderLayer::enclosingTransformedAncestor): (WebCore::RenderLayer::convertToLayerCoords):
11:12 AM Changeset in webkit [47899] by Adam Roben
  • 2 edits in trunk/WebKitTools

Fall back to a Release version of Safari if a Debug one doesn't exist

<http://webkit.org/b/28849>

Reviewed by Sam Weinig.

  • Scripts/webkitdirs.pm:

(safariPath): If the user is working with a Debug build, but there's
no Debug version of Safari present, fall back to using a Release
version of Safari.

10:54 AM Changeset in webkit [47898] by Adam Roben
  • 2 edits in trunk/WebKitTools

Make safariPath() work for Debug builds of Safari on Windows

<http://webkit.org/b/28849>

Reviewed by Sam Weinig.

  • Scripts/webkitdirs.pm:

(safariPath): If the user is working with a Debug build, add the
_debug suffix to Safari.exe.

10:20 AM Changeset in webkit [47897] by pfeldman@chromium.org
  • 7 edits in trunk/WebCore

2009-08-31 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Timothy Hatcher.

WebInspector: pass call frames into the frontend as a part of
pausedScript event.

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

  • inspector/InspectorController.cpp: (WebCore::InspectorController::didPause):
  • inspector/InspectorFrontend.cpp: (WebCore::InspectorFrontend::pausedScript):
  • inspector/InspectorFrontend.h:
  • inspector/front-end/ScriptsPanel.js: (WebInspector.ScriptsPanel.prototype.debuggerPaused):
  • inspector/front-end/inspector.js: (WebInspector.pausedScript):
8:06 AM Changeset in webkit [47896] by Adam Roben
  • 4 edits in trunk/WebKit/win

Give WebKit clients access to the window features specified in window.open

Reviewed by Steve Falkenburg.

  • COMVariantSetter.h: Added COMVariantSetters for bool and float.
  • Interfaces/IWebUIDelegatePrivate.idl: Added IWebUIDelegatePrivate2,

which contains a new version of createWebViewWithRequest that also
passes the window features as an IPropertyBag. This is analagous to
the -createWebViewWithRequest:windowFeatures: method added to
WebUIDelegate on Mac in r27452. Also added key strings for use with
the window features property bag.

  • WebCoreSupport/WebChromeClient.cpp:

(createWindowFeaturesPropertyBag): Added this helper function to turn
a WebCore::WindowFeatures into an IPropertyBag for passing to WebKit
clients.
(WebChromeClient::createWindow): If the UI delegate implements
IWebUIDelegatePrivate2, call the new createWebViewWithRequest function
that takes a window features property bag. If the delegate doesn't
implement the new function, fall back to the old functions.

8:05 AM Changeset in webkit [47895] by Adam Roben
  • 2 edits in trunk/WebKit/win

A little clean-up in WebChromeClient::createWindow

Reviewed by Steve Falkenburg.

  • WebCoreSupport/WebChromeClient.cpp:

(WebChromeClient::createWindow): Refactored to share more code between
the dialog and non-dialog cases.

5:50 AM Changeset in webkit [47894] by zoltan@webkit.org
  • 14 edits in trunk/LayoutTests

2009-08-31 Andras Becsi <becsi.andras@stud.u-szeged.hu>

Reviewed by Ariya Hidayat.

[Qt] Update outdated font metrics in the remaining css1 tests. All tests pass.

  • platform/qt/Skipped:
  • platform/qt/css1/text_properties/letter_spacing-expected.txt:
  • platform/qt/css1/text_properties/line_height-expected.txt:
  • platform/qt/css1/text_properties/text_align-expected.txt:
  • platform/qt/css1/text_properties/text_decoration-expected.txt:
  • platform/qt/css1/text_properties/text_indent-expected.txt:
  • platform/qt/css1/text_properties/text_transform-expected.txt:
  • platform/qt/css1/text_properties/vertical_align-expected.txt:
  • platform/qt/css1/text_properties/word_spacing-expected.txt:
  • platform/qt/css1/units/color_units-expected.txt:
  • platform/qt/css1/units/length_units-expected.txt:
  • platform/qt/css1/units/percentage_units-expected.txt:
  • platform/qt/css1/units/urls-expected.txt:
5:15 AM Changeset in webkit [47893] by zoltan@webkit.org
  • 14 edits in trunk/LayoutTests

2009-08-31 Andras Becsi <becsi.andras@stud.u-szeged.hu>

Reviewed by Ariya Hidayat.

[Qt] Update font metrics for these css1 tests.

  • platform/qt/Skipped:
  • platform/qt/css1/formatting_model/canvas-expected.txt:
  • platform/qt/css1/formatting_model/floating_elements-expected.txt:
  • platform/qt/css1/formatting_model/height_of_lines-expected.txt:
  • platform/qt/css1/formatting_model/horizontal_formatting-expected.txt:
  • platform/qt/css1/formatting_model/inline_elements-expected.txt:
  • platform/qt/css1/formatting_model/replaced_elements-expected.txt:
  • platform/qt/css1/formatting_model/vertical_formatting-expected.txt:
  • platform/qt/css1/pseudo/anchor-expected.txt:
  • platform/qt/css1/pseudo/firstletter-expected.txt:
  • platform/qt/css1/pseudo/firstline-expected.txt:
  • platform/qt/css1/pseudo/multiple_pseudo_elements-expected.txt:
  • platform/qt/css1/pseudo/pseudo_elements_in_selectors-expected.txt:
5:09 AM Changeset in webkit [47892] by vestbo@webkit.org
  • 2 edits in trunk/WebCore

[Qt] Un-blacklist Silverlight on Mac

Reviewed by Ariya Hidayat.

Does not seem to crash anymore, probably due to how we now handle
drawing and event model handshake better.

  • plugins/mac/PluginPackageMac.cpp:

(WebCore::PluginPackage::isPluginBlacklisted):

4:17 AM Changeset in webkit [47891] by zoltan@webkit.org
  • 10 edits in trunk/LayoutTests

2009-08-31 Andras Becsi <becsi.andras@stud.u-szeged.hu>

Reviewed by Ariya Hidayat.

[Qt] Update outdated font metrics for these css1 tests.

  • platform/qt/Skipped:
  • platform/qt/css1/color_and_background/background-expected.txt:
  • platform/qt/css1/color_and_background/background_attachment-expected.txt:
  • platform/qt/css1/color_and_background/background_color-expected.txt:
  • platform/qt/css1/color_and_background/background_image-expected.txt:
  • platform/qt/css1/color_and_background/background_position-expected.txt:
  • platform/qt/css1/color_and_background/background_repeat-expected.txt:
  • platform/qt/css1/color_and_background/color-expected.txt:
  • platform/qt/css1/conformance/forward_compatible_parsing-expected.txt:
4:06 AM Changeset in webkit [47890] by eric@webkit.org
  • 4 edits in trunk/WebCore

2009-08-31 Cameron McCormack <cam@mcc.id.au>

Reviewed by Eric Seidel.

https://bugs.webkit.org/show_bug.cgi?id=28827
SVGSVGElement.unsuspendRedraw() shouldn't throw

  • svg/SVGSVGElement.cpp: (WebCore::SVGSVGElement::unsuspendRedraw):
  • svg/SVGSVGElement.h:
  • svg/SVGSVGElement.idl:
3:55 AM Changeset in webkit [47889] by eric@webkit.org
  • 4 edits in trunk/WebCore

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

Reviewed by Eric Seidel.

  • html/HTMLInputElement.cpp: (WebCore::HTMLInputElement::list): (WebCore::HTMLInputElement::dataList): (WebCore::HTMLInputElement::selectedOption):
  • html/HTMLInputElement.h:
  • html/HTMLInputElement.idl:
3:45 AM Changeset in webkit [47888] by eric@webkit.org
  • 2 edits in trunk/WebCore

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

Reviewed by Eric Seidel.

Remove ASSERT(frame) in ResourceHandle::start (ResourceHandleCurl.cpp)
https://bugs.webkit.org/show_bug.cgi?id=28802

Remove ASSERT because the frame could be null if the
ResourceHandle is not associated with any frame, e.g. if we are
downloading a file.
If the frame is not null but the page is null this must be an
attempted load from an onUnload handler, so let's just block it.

  • platform/network/curl/ResourceHandleCurl.cpp: (WebCore::ResourceHandle::start):
3:18 AM Changeset in webkit [47887] by eric@webkit.org
  • 3 edits in trunk/WebCore

2009-08-31 Maxime Simon <Maxime Simon>

Reviewed by Eric Seidel.

Build fix for platforms which don't enable DOM_STORAGE.
https://bugs.webkit.org/show_bug.cgi?id=28834

  • bindings/js/ScriptController.cpp: (WebCore::ScriptController::evaluate):
  • bindings/v8/ScriptController.cpp: (WebCore::ScriptController::evaluate):
2:59 AM Changeset in webkit [47886] by vestbo@webkit.org
  • 1 edit in trunk/WebKitTools/DumpRenderTree/qt/TestNetscapePlugin/TestNetscapePlugin.pro

Fix the Qt/Mac build of TestNetscapePlugin after Qt change 3940f25

2:53 AM Changeset in webkit [47885] by vestbo@webkit.org
  • 1 edit in trunk/WebCore/WebCore.pro

Fix the Qt/Mac build after Qt change 3940f25 (LIBS_PRIVATE)

We no longer pick up the libraries/frameworks that Qt was built
against, so we have to add them manually for our targets.

12:49 AM Changeset in webkit [47884] by zoltan@webkit.org
  • 23 edits in trunk/LayoutTests

2009-08-31 Renata Hodovan <hodovan.renata@stud.u-szeged.hu>

Reviewed by Ariya Hidayat.

[Qt] Update outdated metrics in css2.1/t040*. (19 files)
Fix relative path in .css files.

  • css2.1/support/css1test64a.css:
  • css2.1/support/css1test64b.css:
  • platform/qt/Skipped:
  • platform/qt/css2.1/t0402-c71-fwd-parsing-00-f-expected.txt:
  • platform/qt/css2.1/t0402-c71-fwd-parsing-01-f-expected.txt:
  • platform/qt/css2.1/t0402-c71-fwd-parsing-02-f-expected.txt:
  • platform/qt/css2.1/t0402-c71-fwd-parsing-03-f-expected.txt:
  • platform/qt/css2.1/t0402-c71-fwd-parsing-04-f-expected.txt:
  • platform/qt/css2.1/t0402-syntax-01-f-expected.txt:
  • platform/qt/css2.1/t0402-syntax-02-f-expected.txt:
  • platform/qt/css2.1/t0402-syntax-03-f-expected.txt:
  • platform/qt/css2.1/t0402-syntax-04-f-expected.txt:
  • platform/qt/css2.1/t0402-syntax-05-f-expected.txt:
  • platform/qt/css2.1/t0402-syntax-06-f-expected.txt:
  • platform/qt/css2.1/t040302-c61-ex-len-00-b-a-expected.txt:
  • platform/qt/css2.1/t040302-c61-phys-len-00-b-expected.txt:
  • platform/qt/css2.1/t040302-c61-rel-len-00-b-ag-expected.txt:
  • platform/qt/css2.1/t040303-c62-percent-00-b-ag-expected.txt:
  • platform/qt/css2.1/t040304-c64-uri-00-a-g-expected.txt:
  • platform/qt/css2.1/t040306-c63-color-00-b-ag-expected.txt:
  • platform/qt/css2.1/t040306-syntax-01-f-expected.txt:
  • platform/qt/css2.1/t040307-syntax-01-b-expected.txt:

Aug 29, 2009:

4:59 PM Changeset in webkit [47883] by adele@apple.com
  • 3 edits
    2 adds in trunk

WebCore: Fix for https://bugs.webkit.org/show_bug.cgi?id=28829
Crash in AccessibilityRenderObject::activeDescendant when trying to set aria-activedescendant to something without a renderer

Reviewed by Dan Bernstein.

Test: accessibility/aria-activedescendant-crash.html

  • accessibility/AccessibilityRenderObject.cpp: (WebCore::AccessibilityRenderObject::activeDescendant): Nil check.

LayoutTests: Test for https://bugs.webkit.org/show_bug.cgi?id=28829
Crash in AccessibilityRenderObject::activeDescendant when trying to set aria-activedescendant to something without a renderer

Reviewed by Dan Bernstein.

  • accessibility/aria-activedescendant-crash-expected.txt: Added.
  • accessibility/aria-activedescendant-crash.html: Added.

Aug 28, 2009:

11:56 PM Changeset in webkit [47882] by jmalonzo@webkit.org
  • 6 edits in trunk

2009-08-28 Jan Michael Alonzo <jmalonzo@webkit.org>

Reviewed by Oliver Hunt.

Gtk Build broken for OSX Quartz
https://bugs.webkit.org/show_bug.cgi?id=28727

Define XP_UNIX for non-Win OS builds.

  • GNUmakefile.am:

Check for XP_UNIX instead of GTK or X11 to ease building of the Gtk
port in Mac and Unix/Linux.

  • plugins/PluginView.cpp: (WebCore::PluginView::PluginView):
  • plugins/PluginView.h:
  • plugins/gtk/PluginViewGtk.cpp: (WebCore::PluginView::setNPWindowIfNeeded): (WebCore::PluginView::getValueStatic): (WebCore::PluginView::getValue): (WebCore::PluginView::init):
6:51 PM Changeset in webkit [47881] by eric@webkit.org
  • 4 edits in trunk/WebKitTools

2009-08-28 Eric Seidel <eric@webkit.org>

Reviewed by Adam Barth.

commit-queue needs a master process
https://bugs.webkit.org/show_bug.cgi?id=28040

Add a bugzilla-tool commit-queue command
Keeps per-bug logs, but doesn't yet upload them anywhere.

  • Scripts/bugzilla-tool: Add LandPatchesFromCommitQueue to handle 'commit-queue'
  • Scripts/modules/buildbot.py: remove noisy log message
  • Scripts/modules/logging.py: add a 'tee()' call for splitting outputs in python
4:37 PM Changeset in webkit [47880] by sfalken@apple.com
  • 2 edits in trunk/WebKitLibraries

Define _CRT_SECURE_CPP_OVERLOAD_STANDARD_NAMES=1
to automatically use secure versions of C runtime lib calls on Windows.
https://bugs.webkit.org/show_bug.cgi?id=28824

Reviewed by Adam Roben.

  • win/tools/vsprops/common.vsprops:
4:31 PM Changeset in webkit [47879] by kov@webkit.org
  • 2 edits in trunk/LayoutTests

2009-08-28 Gustavo Noronha Silva <Gustavo Noronha Silva>

Re-skipping tests that still fail on the bot.

  • platform/gtk/Skipped:
4:13 PM Changeset in webkit [47878] by kov@webkit.org
  • 2 edits in trunk/LayoutTests

2009-08-28 Gustavo Noronha Silva <Gustavo Noronha Silva>

Unskip some passing tests, skip new tests that are failing.

  • platform/gtk/Skipped:
4:08 PM Changeset in webkit [47877] by Simon Fraser
  • 2 edits in trunk/WebCore

2009-08-28 Simon Fraser <Simon Fraser>

Reviewed by Mark Rowe

Wrap WebkitCSSTransformValues in the correct class of DOM wrapper.
https://bugs.webkit.org/show_bug.cgi?id=27727

When fetching the DOM wrapper for a WebkitCSSTransformValue (which is a kind of
CSSValueList), we need to ask the value list whether it's a WebkitCSSTransformValue,
because there is not a unique enum value for WebkitCSSTransformValue.

  • bindings/objc/DOMCSS.mm: (kitClass):
3:09 PM Changeset in webkit [47876] by pkasting@chromium.org
  • 2 edits in trunk/WebCore

https://bugs.webkit.org/show_bug.cgi?id=28308
Fix some warnings introduced by r47381.

Reviewed by Jan Michael Alonzo.

  • platform/image-decoders/jpeg/JPEGImageDecoder.cpp:

(WebCore::convertCMYKToRGBA):
(WebCore::convertRGBToRGBA):

3:03 PM Changeset in webkit [47875] by adachan@apple.com
  • 2 edits in trunk/WebCore

Reviewed by Adam Roben.

https://bugs.webkit.org/show_bug.cgi?id=21442
Make sure filesystemPathFromUrlOrTitle() returns a string with length
<= MAX_PATH - 1.

  • platform/win/ClipboardWin.cpp: (WebCore::filesystemPathFromUrlOrTitle):
2:44 PM Changeset in webkit [47874] by ojan@chromium.org
  • 5 edits
    1 copy
    2 deletes in trunk/WebCore

2009-08-28 Dumitru Daniliuc <dumi@chromium.org>

Reviewed by Dimitri Glazkov.

Adding Chromium's VFS for Linux and Mac.

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

  • WebCore.gypi:
  • platform/chromium/ChromiumBridge.h:
  • platform/sql/chromium/SQLiteFileSystemChromium.cpp: (WebCore::SQLiteFileSystem::openDatabase): (WebCore::SQLiteFileSystem::deleteDatabaseFile):
  • platform/sql/chromium/SQLiteFileSystemChromiumLinux.cpp: Removed.
  • platform/sql/chromium/SQLiteFileSystemChromiumMac.cpp: Removed.
  • platform/sql/chromium/SQLiteFileSystemChromiumPosix.cpp: Added.
  • platform/sql/chromium/SQLiteFileSystemChromiumWin.cpp:
2:16 PM Changeset in webkit [47873] by jparent@chromium.org
  • 2 edits in trunk/LayoutTests

set-dangerous-headers.html shouldn't set allowed header to same value
as forbidden headers. AUTHORIZATION is an allowed header, and the
value is being set to "foobar" like the forbidden headers, which causes
the test to fail when run using http server that doesn't strip the
AUTHORIZATION header.
https://bugs.webkit.org/show_bug.cgi?id=28818

Patch by Julie Parent <jparent@chromium.org> on 2009-08-28
Reviewed by Alexey Proskuryakov.

  • http/tests/xmlhttprequest/set-dangerous-headers.html:
11:59 AM Changeset in webkit [47872] by Dimitri Glazkov
  • 8 edits in trunk/WebCore

2009-08-28 Vitaly Repeshko <vitalyr@chromium.org>

Reviewed by Dimitri Glazkov.

[V8] Made bindings use Node::setOnEvent functions for Node descendants
instead of Node::setAttributeEventListener.

Node::setAttributeEventListener is an internal function and
doesn't do the forwarding to DOM window that is required for some
HTML elements.
https://bugs.webkit.org/show_bug.cgi?id=28709

Tests added in http://trac.webkit.org/changeset/47326 now pass with V8.

  • bindings/scripts/CodeGeneratorV8.pm: Now generates EventListener accessors for Node descendants instead of using custom callbacks.
  • bindings/v8/V8DOMWrapper.cpp: (WebCore::V8DOMWrapper::getEventListener): Moved from V8NodeCustom.cpp.
  • bindings/v8/V8DOMWrapper.h: (WebCore::V8DOMWrapper::convertEventListenerToV8Object): Fixed a bug in signature.
  • bindings/v8/V8Utilities.cpp: (WebCore::createHiddenDependency): (WebCore::removeHiddenDependency): Changed signature to accept Handle instead of Local.
  • bindings/v8/V8Utilities.h:
  • bindings/v8/custom/V8CustomBinding.h:
  • bindings/v8/custom/V8NodeCustom.cpp: (WebCore::CALLBACK_FUNC_DECL): Removed custom callbacks for EventListener accessors.
11:40 AM Changeset in webkit [47871] by eric@webkit.org
  • 2 edits in trunk/WebCore

2009-08-28 Ben Murdoch <benm@google.com>

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


Implements V8CustomVoidCallback reporting any exceptions that were thrown from the callback function.

  • bindings/v8/custom/V8CustomVoidCallback.cpp: (WebCore::invokeCallback): Checks if the callback raised an exception and reports it to WebCore if so.
11:34 AM Changeset in webkit [47870] by Chris Fleizach
  • 4 edits in trunk/WebKit

WebKit: update-webkit-localizable-strings script can no longer complete
https://bugs.webkit.org/show_bug.cgi?id=28792

Reviewed by John Sullivan.

  • English.lproj/Localizable.strings:

WebKit/mac: update-webkit-localizable-strings script can no longer complete
https://bugs.webkit.org/show_bug.cgi?id=28792

Reviewed by John Sullivan.

  • WebCoreSupport/WebViewFactory.mm:

(-[WebViewFactory AXARIAContentGroupText:]):

11:29 AM Changeset in webkit [47869] by pkasting@chromium.org
  • 3 edits in trunk/WebCore

Not reviewed (build fix).

  • platform/graphics/qt/ImageDecoderQt.cpp: Accidentally added some spaces.

(WebCore::ImageDecoder::create):
(WebCore::ImageDecoderQt::ImageDecoderQt):

  • platform/graphics/qt/ImageDecoderQt.h: Make constructor public so factory function can access it (this seemed better than making ImageDecoder a friend).
11:08 AM Changeset in webkit [47868] by pkasting@chromium.org
  • 7 edits in trunk/WebCore

https://bugs.webkit.org/show_bug.cgi?id=27965
Move ImageDecoder creation function to a factory function on
ImageDecoder. This is arguably where it makes the most sense anyway,
and it will (soon) allow ImageSourceQt.cpp to have one less dedicated
function.

Reviewed by Eric Seidel.

  • platform/graphics/ImageSource.cpp:
  • platform/graphics/qt/ImageDecoderQt.cpp:

(WebCore::ImageDecoder::create):

  • platform/graphics/qt/ImageDecoderQt.h:
  • platform/graphics/qt/ImageSourceQt.cpp:

(WebCore::ImageSource::setData):

  • platform/image-decoders/ImageDecoder.cpp:
  • platform/image-decoders/ImageDecoder.h:
11:00 AM Changeset in webkit [47867] by pkasting@chromium.org
  • 5 edits in trunk/WebCore

https://bugs.webkit.org/show_bug.cgi?id=28785
Combine duplicated code from ImageDecoder*.cpp into the main
ImageDecoder.cpp.

Reviewed by Eric Seidel.

  • platform/image-decoders/ImageDecoder.cpp:

(WebCore::RGBA32Buffer::RGBA32Buffer):
(WebCore::RGBA32Buffer::clear):
(WebCore::RGBA32Buffer::zeroFill):
(WebCore::RGBA32Buffer::copyBitmapData):
(WebCore::RGBA32Buffer::setSize):
(WebCore::RGBA32Buffer::hasAlpha):
(WebCore::RGBA32Buffer::setHasAlpha):
(WebCore::RGBA32Buffer::setStatus):
(WebCore::RGBA32Buffer::operator=):
(WebCore::RGBA32Buffer::width):
(WebCore::RGBA32Buffer::height):

  • platform/image-decoders/cairo/ImageDecoderCairo.cpp:
  • platform/image-decoders/haiku/ImageDecoderHaiku.cpp:
  • platform/image-decoders/wx/ImageDecoderWx.cpp:
8:24 AM Changeset in webkit [47866] by kov@webkit.org
  • 25 edits
    1 add in trunk

2009-08-28 Gustavo Noronha Silva <gustavo.noronha@collabora.co.uk>

Reviewed by Holger Freyther.

https://bugs.webkit.org/show_bug.cgi?id=25889
[GTK] scrollbar policy for main frame is not implementable

Adding files for the new test for window-related issues (starting
with scrollbar policy).

  • GNUmakefile.am:

WebCore

2009-08-28 Gustavo Noronha Silva <gustavo.noronha@collabora.co.uk>

Reviewed by Holger Freyther.

https://bugs.webkit.org/show_bug.cgi?id=25889
[GTK] scrollbar policy for main frame is not implementable

Override setScrollbarModes to correctly forward scrollbar modes
policy changes to client code, via a new platform-specific method,
scrollbarsPolicyDidChange.

  • page/ChromeClient.h: (WebCore::ChromeClient::scrollbarsPolicyDidChange):
  • platform/ScrollView.cpp: (WebCore::ScrollView::createScrollbar):
  • platform/ScrollView.h:
  • platform/gtk/ScrollViewGtk.cpp: (WebCore::ScrollView::setScrollbarModes):

WebKit/gtk

2009-08-28 Gustavo Noronha Silva <gustavo.noronha@collabora.co.uk>

Reviewed by Holger Freyther.

https://bugs.webkit.org/show_bug.cgi?id=25889
[GTK] scrollbar policy for main frame is not implementable

Add properties and a signal to report scrollbar policy changes to
client code, and a default handler that does the right thing when
using GtkScrolledWindow.

  • WebCoreSupport/ChromeClientGtk.cpp: (WebKit::ChromeClient::scrollbarsModeDidChange):
  • WebCoreSupport/ChromeClientGtk.h:
  • webkit/webkitwebframe.cpp: (webkit_web_frame_get_property): (webkit_web_frame_class_init): (webkit_web_frame_get_horizontal_scrollbar_policy): (webkit_web_frame_get_vertical_scrollbar_policy):
  • webkit/webkitwebframe.h:
  • tests/testwindow.c: Added. (load_finished_cb): (test_webkit_window_scrollbar_policy): (main):

WebKit/*

Added empty implementation of new ChromeClient method.

8:12 AM Changeset in webkit [47865] by jmalonzo@webkit.org
  • 3 edits in trunk/WebKit/gtk

2009-08-28 Jan Michael Alonzo <jmalonzo@webkit.org>

Reviewed by Gustavo Noronha and Xan Lopez.

[Gtk] Add view source mode API
https://bugs.webkit.org/show_bug.cgi?id=28805

Implement setter and getter for "view source" mode.

  • webkit/webkitwebview.cpp: (webkit_web_view_set_view_source_mode): (webkit_web_frame_get_view_source_mode):
  • webkit/webkitwebview.h:
7:27 AM Changeset in webkit [47864] by jmalonzo@webkit.org
  • 4 edits
    5 adds in trunk

2009-08-28 Jan Michael Alonzo <jmalonzo@webkit.org>

Reviewed by Gustavo Noronha.

[GTK] Geolocation needs permission API before being enabled by default

Unskip the Geolocation tests and rebaseline three DOM tests.

  • platform/gtk/Skipped:
  • platform/gtk/fast/dom/Window/window-properties-expected.txt: Added.
  • platform/gtk/fast/dom/navigator-detached-no-crash-expected.txt: Added.
  • platform/gtk/fast/dom/prototype-inheritance-2-expected.txt: Added.

Build the Gtk port with geolocation enabled. This option is only
enabled for the buildbot. The autotools option will be enabled by
default once the permissions API is implemented.

  • Scripts/build-webkit:
6:58 AM Changeset in webkit [47863] by vestbo@webkit.org
  • 3 edits in trunk/WebCore

[Qt/Mac] Un-blacklist the QuickTime plugin

Patch by Cédric Luthi <cedric.luthi@gmail.com> on 2009-08-28
Reviewed by Tor Arne Vestbø

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

Now that the drawing model is properly negotiated, there is no need the
blacklist the QuickTime plugin anymore.

This revealed a bug in PluginView::performRequest: it should not initiate a
request if the plugin has been stopped, which is what happens when a plugin
only supports the QuickDraw drawing model.

  • plugins/PluginView.cpp:
  • plugins/mac/PluginPackageMac.cpp:
6:42 AM Changeset in webkit [47862] by pfeldman@chromium.org
  • 3 edits in trunk/WebCore

2009-08-28 Yury Semikhatsky <yurys@chromium.org>

Reviewed by Timothy Hatcher.

Set DOM agent document to 0 in InspectorController::close to
make DOM agent remove DOM listeners from the inspected document.

Replace windowVisible checks with m_frontend checks to as inspector
window lives in a different process in Chromium while frontend proxy
is in the inspected page process.

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

  • inspector/InspectorController.cpp: (WebCore::InspectorController::inspect): (WebCore::InspectorController::addConsoleMessage): (WebCore::InspectorController::close): (WebCore::InspectorController::pruneResources): (WebCore::InspectorController::didCommitLoad): (WebCore::InspectorController::didLoadResourceFromMemoryCache): (WebCore::InspectorController::identifierForInitialRequest): (WebCore::InspectorController::willSendRequest): (WebCore::InspectorController::didReceiveResponse): (WebCore::InspectorController::didReceiveContentLength): (WebCore::InspectorController::didFinishLoading): (WebCore::InspectorController::didFailLoading): (WebCore::InspectorController::resourceRetrievedByXMLHttpRequest): (WebCore::InspectorController::scriptImported): (WebCore::InspectorController::didOpenDatabase): (WebCore::InspectorController::didUseDOMStorage): (WebCore::InspectorController::addProfile):
  • inspector/InspectorController.h:
6:23 AM Changeset in webkit [47861] by pfeldman@chromium.org
  • 3 edits in trunk/WebCore

2009-08-28 Mikhail Naganov <mnaganov@chromium.org>

Reviewed by Timothy Hatcher.

Remove dependency of SummaryBar on WebInspector.resourceCategories.

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

  • inspector/front-end/ResourcesPanel.js: (WebInspector.ResourcesPanel): (WebInspector.ResourcesPanel.prototype.toolbarItemClass.get categories):
  • inspector/front-end/SummaryBar.js: (WebInspector.SummaryBar): (WebInspector.SummaryBar.prototype.update):
5:28 AM Changeset in webkit [47860] by ariya@webkit.org
  • 41 edits in trunk/LayoutTests

2009-08-28 Renata Hodovan <hodovan.renata@stud.u-szeged.hu>

Reviewed by Ariya Hidayat.

[Qt] Adapt expecteds of css2.1/t0401* to new metrics. Remove these files from Skipped. (39 files)

  • platform/qt/Skipped:
  • platform/qt/css2.1/t040102-keywords-00-b-expected.txt:
  • platform/qt/css2.1/t040102-keywords-01-b-expected.txt:
  • platform/qt/css2.1/t040103-case-00-b-expected.txt:
  • platform/qt/css2.1/t040103-case-01-c-expected.txt:
  • platform/qt/css2.1/t040103-escapes-00-b-expected.txt:
  • platform/qt/css2.1/t040103-escapes-01-b-expected.txt:
  • platform/qt/css2.1/t040103-escapes-02-d-expected.txt:
  • platform/qt/css2.1/t040103-escapes-03-b-expected.txt:
  • platform/qt/css2.1/t040103-escapes-04-b-expected.txt:
  • platform/qt/css2.1/t040103-escapes-05-c-expected.txt:
  • platform/qt/css2.1/t040103-escapes-06-b-expected.txt:
  • platform/qt/css2.1/t040103-escapes-07-b-expected.txt:
  • platform/qt/css2.1/t040103-escapes-08-b-expected.txt:
  • platform/qt/css2.1/t040103-ident-00-c-expected.txt:
  • platform/qt/css2.1/t040103-ident-01-c-expected.txt:
  • platform/qt/css2.1/t040103-ident-02-c-expected.txt:
  • platform/qt/css2.1/t040103-ident-03-c-expected.txt:
  • platform/qt/css2.1/t040103-ident-04-c-expected.txt:
  • platform/qt/css2.1/t040103-ident-05-c-expected.txt:
  • platform/qt/css2.1/t040103-ident-06-c-expected.txt:
  • platform/qt/css2.1/t040103-ident-07-c-expected.txt:
  • platform/qt/css2.1/t040103-ident-08-c-expected.txt:
  • platform/qt/css2.1/t040103-ident-09-c-expected.txt:
  • platform/qt/css2.1/t040103-ident-10-c-expected.txt:
  • platform/qt/css2.1/t040103-ident-11-c-expected.txt:
  • platform/qt/css2.1/t040103-ident-12-c-expected.txt:
  • platform/qt/css2.1/t040103-ident-13-c-expected.txt:
  • platform/qt/css2.1/t040105-atkeyw-00-b-expected.txt:
  • platform/qt/css2.1/t040105-atkeyw-01-b-expected.txt:
  • platform/qt/css2.1/t040105-atkeyw-02-b-expected.txt:
  • platform/qt/css2.1/t040105-atrule-00-b-expected.txt:
  • platform/qt/css2.1/t040105-atrule-01-b-expected.txt:
  • platform/qt/css2.1/t040105-atrule-02-b-expected.txt:
  • platform/qt/css2.1/t040105-atrule-03-b-expected.txt:
  • platform/qt/css2.1/t040105-atrule-04-b-expected.txt:
  • platform/qt/css2.1/t040105-import-00-b-expected.txt:
  • platform/qt/css2.1/t040105-import-01-b-expected.txt:
  • platform/qt/css2.1/t040109-c17-comments-00-b-expected.txt:
  • platform/qt/css2.1/t040109-c17-comments-01-b-expected.txt:
2:00 AM Changeset in webkit [47859] by pfeldman@chromium.org
  • 2 edits in trunk/WebCore

2009-08-28 Pavel Feldman <pfeldman@chromium.org>

Not reviewed (patching in l18n change that was missing due to
the git binary diff format).

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

  • English.lproj/localizedStrings.js:
1:08 AM Changeset in webkit [47858] by abarth@webkit.org
  • 3 edits in trunk/WebCore

2009-08-28 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.

Make XSSAuditor go fast
https://bugs.webkit.org/show_bug.cgi?id=28667

Make the XSSAuditor go faster by implementing two optimizations:

1) We avoid canonicalizing scripts that are larger than the page's URL.

This saves a bunch of time for large inline scripts.

2) We memoize canonicalizing the page's URL because it's silly to

canonicalize repeated for each inline event listener.

These optimizations have a measurable affect on the intl1 page cycler.

No behavior change.

  • page/XSSAuditor.cpp: (WebCore::XSSAuditor::MemoizingURLCanonicalizer::canonicalizeURL): (WebCore::XSSAuditor::canEvaluate): (WebCore::XSSAuditor::canEvaluateJavaScriptURL): (WebCore::XSSAuditor::canCreateInlineEventListener): (WebCore::XSSAuditor::canLoadExternalScriptFromSrc): (WebCore::XSSAuditor::canLoadObject): (WebCore::XSSAuditor::canSetBaseElementURL): (WebCore::XSSAuditor::decodeURL): (WebCore::XSSAuditor::findInRequest):
  • page/XSSAuditor.h:
Note: See TracTimeline for information about the timeline view.