⚠ Archived content — this site is no longer maintained.   Current WebKit documentation is at docs.webkit.org.

Timeline



Aug 16, 2010:

11:48 PM Changeset in webkit [65486] by Philippe Normand
  • 4 edits in trunk

2010-08-17 Philippe Normand <​pnormand@igalia.com>

Reviewed by Martin Robinson.

[GStreamer] media/invalid-media-url-crash.html fails
​https://bugs.webkit.org/show_bug.cgi?id=42960

Implemented two ResourceHandleClient callbacks that need to notify
downstream GStreamer elements of errors while accessing the
resource.

  • platform/graphics/gstreamer/WebKitWebSourceGStreamer.cpp: (StreamingClient::wasBlocked): (StreamingClient::cannotShowURL):

LayoutTests:

2010-08-17 Philippe Normand <​pnormand@igalia.com>

Reviewed by Martin Robinson.

[GStreamer] media/invalid-media-url-crash.html fails
​https://bugs.webkit.org/show_bug.cgi?id=42960

  • platform/gtk/Skipped: Unskip fixed test
9:27 PM Changeset in webkit [65485] by kinuko@chromium.org
  • 2 edits in trunk/WebKit/chromium

2010-08-16 Kinuko Yasuda <​kinuko@chromium.org>

Unreviewed; attempt to fix Chromium Windows build.

  • public/WebFileUtilities.h:
9:07 PM Changeset in webkit [65484] by jhoneycutt@apple.com
  • 3 edits in trunk/WebKit2

Allow some tests to pass in WebKitTestRunner before SharedMemory and
PluginInfoStore are implemented on Windows.

Reviewed by Sam Weinig.

  • Platform/win/SharedMemoryWin.cpp:

Disable not implemented warnings.

  • UIProcess/Plugins/win/PluginInfoStoreWin.cpp:

Ditto.

8:59 PM Changeset in webkit [65483] by morrita@google.com
  • 2 edits in trunk/LayoutTests

2010-08-16 Yuta Kitamura <​yutak@chromium.org>

Unreviewed test expectation update.

Update Chromium test expectations.
​https://bugs.webkit.org/show_bug.cgi?id=44090

  • platform/chromium/test_expectations.txt:
8:53 PM Changeset in webkit [65482] by kinuko@chromium.org
  • 5 edits
    1 add in trunk/WebKit/chromium

2010-08-16 Kinuko Yasuda <​kinuko@chromium.org>

Reviewed by Jian Li.

[chromium] Prepare to rename WebFileSystem interface to WebFileUtilities
​https://bugs.webkit.org/show_bug.cgi?id=44077

We are going to have an implementation for HTML5 FileSystem API and
it would probably be less confusing to use the name "WebFileSystem"
for an WebKit API for the HTML5 API.

This patch adds WebFileUtilities interface that is meant to replace
the existing WebFileSystem.

  • WebKit.gyp:
  • public/WebFileSystem.h:
  • public/WebFileUtilities.h: Added.
  • public/WebKitClient.h: (WebKit::WebKitClient::fileUtilities): Added.
  • src/ChromiumBridge.cpp: (WebCore::ChromiumBridge::fileExists): (WebCore::ChromiumBridge::deleteFile): (WebCore::ChromiumBridge::deleteEmptyDirectory): (WebCore::ChromiumBridge::getFileSize): (WebCore::ChromiumBridge::getFileModificationTime): (WebCore::ChromiumBridge::directoryName): (WebCore::ChromiumBridge::pathByAppendingComponent): (WebCore::ChromiumBridge::makeAllDirectories): (WebCore::ChromiumBridge::getAbsolutePath): (WebCore::ChromiumBridge::isDirectory): (WebCore::ChromiumBridge::filePathToURL): (WebCore::ChromiumBridge::openFile): (WebCore::ChromiumBridge::closeFile): (WebCore::ChromiumBridge::seekFile): (WebCore::ChromiumBridge::truncateFile): (WebCore::ChromiumBridge::readFromFile): (WebCore::ChromiumBridge::writeToFile):
8:42 PM Changeset in webkit [65481] by kinuko@chromium.org
  • 2 edits in trunk/WebKit/win

2010-08-16 Kinuko Yasuda <​kinuko@chromium.org>

Unreviewed; build fix attempt for Windows.

  • DefaultDownloadDelegate.cpp:
8:39 PM Changeset in webkit [65480] by jhoneycutt@apple.com
  • 3 edits in trunk/LayoutTests

Add some tests that fail in WebKitTestRunner to the Skipped lists.

Rubber-stamped by Sam Weinig.

  • platform/mac-wk2/Skipped:
  • platform/win-wk2/Skipped:
7:21 PM Changeset in webkit [65479] by barraclough@apple.com
  • 3 edits in trunk/JavaScriptCore

Add VectorTraits to String & DefaultHash traits to UString to unify behaviour.

Rubber stamped by Sam Weinig

  • runtime/UString.h:

(JSC::UStringHash::hash):
(JSC::UStringHash::equal):
(WTF::):

  • wtf/text/WTFString.h:

(WTF::):

7:13 PM Changeset in webkit [65478] by barraclough@apple.com
  • 15 edits in trunk

Remove unnecessary includes from UString.h, add new includes as necessary.

Rubber stamped by Sam Weinig

JavaScriptCore:

  • profiler/CallIdentifier.h:
  • profiler/ProfileNode.h:
  • runtime/DateConversion.cpp:
  • runtime/Identifier.h:

(JSC::IdentifierRepHash::hash):

  • runtime/RegExpCache.h:
  • runtime/RegExpKey.h:
  • runtime/UString.cpp:

(JSC::UString::substr):

  • runtime/UString.h:
  • wtf/text/WTFString.h:

WebCore:

  • dom/ScriptExecutionContext.h:
  • loader/appcache/ApplicationCache.cpp:

WebKit2:

  • WebProcess/Plugins/Netscape/NetscapePlugin.h:
6:54 PM Changeset in webkit [65477] by commit-queue@webkit.org
  • 2 edits in trunk/LayoutTests

2010-08-16 Ademar de Souza Reis Jr <Ademar Reis>

Reviewed by Ariya Hidayat.

[Qt] REGRESSION: editing/selection/shrink-selection-after-shift-pagedown.html fails after 56567
​https://bugs.webkit.org/show_bug.cgi?id=36628

Just enabling the test, it's not failing anymore.

  • platform/qt/Skipped:
6:19 PM Changeset in webkit [65476] by commit-queue@webkit.org
  • 2 edits in trunk/LayoutTests

2010-08-16 Ademar de Souza Reis Jr <Ademar Reis>

Reviewed by Ariya Hidayat.

[Qt] qt/Skipped needs cleanup on outdaded comments
​https://bugs.webkit.org/show_bug.cgi?id=44066

Minor improvements/fixes to the list of Skipped Qt tests

  • move bug reference to the right place (#35145)
  • remove outdated bug reference (#32922)
  • move inspector testcase to the right place
  • add reference to bug closed on qt-4.7 (QTBUG-9618)
  • platform/qt/Skipped: cleanups only, no logical changes
6:09 PM Changeset in webkit [65475] by andreas.kling@nokia.com
  • 3 edits in trunk

2010-08-16 Andreas Kling <​andreas.kling@nokia.com>

Reviewed by Darin Adler.

CSS: Add fast-path for rgba() color parsing
​https://bugs.webkit.org/show_bug.cgi?id=42965

Test: fast/canvas/rgba-parsing.html

  • css/CSSParser.cpp: (WebCore::CSSParser::parseColor): Extended with support for rgba(). (WebCore::parseAlphaValue): Added, parses an alpha value using WTF::strtod() (if necessary) and clamps between 0 and 1. (WebCore::isTenthAlpha):

2010-08-16 Andreas Kling <​andreas.kling@nokia.com>

Reviewed by Darin Adler.

CSS: Add fast-path for rgba() color parsing
​https://bugs.webkit.org/show_bug.cgi?id=42965

Add a test of rgba() color parsing.

  • fast/canvas/rgba-parsing-expected.txt: Added.
  • fast/canvas/rgba-parsing.html: Added.
  • fast/canvas/script-tests/rgba-parsing.js: Added.
6:07 PM Changeset in webkit [65474] by kinuko@chromium.org
  • 16 edits
    14 adds in trunk/WebCore

2010-08-16 Kinuko Yasuda <​kinuko@chromium.org>

Reviewed by Dumitru Daniliuc.

Add idl and mock classes for FileSystem API's {File,Directory}Entry and related interfaces
​https://bugs.webkit.org/show_bug.cgi?id=43993

Adding DirectoryEntry, DirectoryReader, EntriesCallback and FileEntry as defined in HTML5 FileSystem API.
​http://dev.w3.org/2009/dap/file-system/file-dir-sys.html

Also adding Makefile entries for idl and mock classes added in changeset 64414 (except for Android platforms).

Tests will be added when we add underlying implementation.

  • CMakeLists.txt:
  • DerivedSources.cpp:
  • DerivedSources.make:
  • GNUmakefile.am:
  • WebCore.gypi:
  • WebCore.pri:
  • WebCore.pro:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • storage/DOMFileSystem.cpp: (WebCore::DOMFileSystem::root):
  • storage/DOMFileSystem.h:
  • storage/DOMFileSystem.idl:
  • storage/DirectoryEntry.cpp: Added.
  • storage/DirectoryEntry.h: Added.
  • storage/DirectoryEntry.idl: Added.
  • storage/DirectoryReader.cpp: Added.
  • storage/DirectoryReader.h: Added.
  • storage/DirectoryReader.idl: Added.
  • storage/EntriesCallback.h: Added.
  • storage/EntriesCallback.idl: Added.
  • storage/EntryArray.cpp: Added.
  • storage/EntryArray.h: Added.
  • storage/EntryArray.idl: Added.
  • storage/Entry.cpp: (WebCore::Entry::Entry): (WebCore::Entry::getMetadata): (WebCore::Entry::moveTo): (WebCore::Entry::copyTo): (WebCore::Entry::remove): (WebCore::Entry::getParent):
  • storage/Entry.h: (WebCore::Entry::isFile): (WebCore::Entry::isDirectory):
  • storage/Entry.idl:
  • storage/FileEntry.cpp: Added.
  • storage/FileEntry.h: Added.
  • storage/FileEntry.idl: Added.
5:46 PM Changeset in webkit [65473] by weinig@apple.com
  • 3 edits in trunk/WebKitTools

Add WebKit2 Mac tester to buildbot.

Reviewed by Mark Rowe.

  • BuildSlaveSupport/build.webkit.org-config/config.json:
  • BuildSlaveSupport/build.webkit.org-config/master.cfg:
5:41 PM Changeset in webkit [65472] by andreas.kling@nokia.com
  • 4 edits
    3 adds in trunk

2010-08-16 Andreas Kling <​andreas.kling@nokia.com>

Reviewed by Ariya Hidayat.

[Qt] Path::closeSubpath() should only close the last subpath if it has >1 point
​https://bugs.webkit.org/show_bug.cgi?id=44061

Spec link:
​http://www.whatwg.org/specs/web-apps/current-work/#dom-context-2d-closepath

Test: fast/canvas/canvas-closePath-single-point.html

  • platform/graphics/Path.h: Add a Qt-only member to track the last subpath.
  • platform/graphics/qt/PathQt.cpp: (WebCore::Path::closeSubpath): Only close the last subpath if it has more than 1 point. Otherwise behave as moveTo(first point in last subpath) (WebCore::Path::Path): (WebCore::Path::operator=): (WebCore::Path::moveTo): (WebCore::Path::transform):

2010-08-16 Andreas Kling <​andreas.kling@nokia.com>

Reviewed by Ariya Hidayat.

[Qt] Path::closeSubpath() should only close the last subpath if it has >1 point
​https://bugs.webkit.org/show_bug.cgi?id=44061

Add a test to verify behavior of closePath() when path has only 1 point.

  • fast/canvas/canvas-closePath-single-point-expected.txt: Added.
  • fast/canvas/canvas-closePath-single-point.html: Added.
  • fast/canvas/script-tests/canvas-closePath-single-point.js: Added.
5:20 PM Changeset in webkit [65471] by Nate Chapin
  • 2 edits in trunk/WebKit/chromium

2010-08-16 Nate Chapin <Nate Chapin>

Unreviewed, build fix.

Don't include TilingDataTest.cpp, since it doesn't
compile on mac.

  • WebKit.gyp:
4:57 PM Changeset in webkit [65470] by Nate Chapin
  • 3 edits in trunk

2010-08-16 Nate Chapin <Nate Chapin>

Unreviewed, Chromium expectations update.

Mark 3 tests failing on Chromium Win/Linux after r65449.
They appear to have same root cause as
bugs.webkit.org/show_bug.cgi?id=43912.

  • platform/chromium/test_expectations.txt:
4:37 PM Changeset in webkit [65469] by Nate Chapin
  • 2 edits in trunk/WebCore

2010-08-16 Nate Chapin <Nate Chapin>

Unreviewed, Chromium mac build fix (with help from jamesr).

  • platform/chromium/ScrollbarThemeChromiumMac.mm: (WebCore::ScrollbarThemeChromiumMac::paint):
4:31 PM Changeset in webkit [65468] by barraclough@apple.com
  • 58 edits in trunk

Bug 44080 - String find/reverseFind methods need tidying up
These methods have a couple of problems with their interface, and implementation.

Reviewed by Sam Weinig

These methods take and int index, and return an int - however this is problematic
since on 64-bit string indices may have a full 32-bit range. This spills out into
surrounding code, which unsafely casts string indices from unsigned to int. Code
checking the result of these methods check for a mix of "== -1", "< 0", and
"== notFound". Clean this up by changing these methods to take an unsigned
starting index, and return a size_t. with a failed match indicated by notFound.
reverseFind also has a special meaning for the starting index argument, in that a
negative index is interpreted as an offset back from the end of the string. Remove
this functionality, in the (1!) case where it is used we should just calculate the
offset by subtracting from the string's length.

The implementation has a few problems too. The code is not in webkit style, in
using assorted abbreviations in variable names, and implementations of similar
find methods with differing argument types were unnecessarily inconsistent. When
find is passed const char* data the string would be handled as latin1 (zero
extended to UTF-16) for all characters but the first; this is sign extended.
Case-insensitive find is broken for unicode strings; the hashing optimization is
not unicode safe, and could result in false negatives.

Unify UString find methods to match String.

JavaScriptCore:

(JSC::escapeQuotes):

  • bytecompiler/NodesCodegen.cpp:

(JSC::substitute):

  • runtime/JSString.cpp:

(JSC::JSString::replaceCharacter):

  • runtime/RegExp.cpp:

(JSC::RegExp::RegExp):

  • runtime/RegExpKey.h:

(JSC::RegExpKey::getFlagsValue):

  • runtime/StringPrototype.cpp:

(JSC::substituteBackreferencesSlow):
(JSC::substituteBackreferences):
(JSC::stringProtoFuncReplace):
(JSC::stringProtoFuncIndexOf):
(JSC::stringProtoFuncLastIndexOf):
(JSC::stringProtoFuncSplit):

  • runtime/UString.cpp:
  • runtime/UString.h:

(JSC::UString::find):
(JSC::UString::reverseFind):

  • wtf/text/AtomicString.h:

(WTF::AtomicString::find):

  • wtf/text/StringImpl.cpp:

(WTF::StringImpl::find):
(WTF::StringImpl::findCaseInsensitive):
(WTF::StringImpl::reverseFind):
(WTF::StringImpl::reverseFindCaseInsensitive):
(WTF::StringImpl::endsWith):
(WTF::StringImpl::replace):

  • wtf/text/StringImpl.h:

(WTF::StringImpl::startsWith):

  • wtf/text/WTFString.cpp:

(WTF::String::split):

  • wtf/text/WTFString.h:

(WTF::String::find):
(WTF::String::reverseFind):
(WTF::String::findCaseInsensitive):
(WTF::String::reverseFindCaseInsensitive):
(WTF::String::contains):
(WTF::find):
(WTF::reverseFind):

WebCore:

  • css/CSSSelector.cpp:

(WebCore::CSSSelector::RareData::parseNth):

  • css/CSSStyleDeclaration.cpp:

(WebCore::CSSStyleDeclaration::setProperty):

  • css/CSSStyleSelector.cpp:

(WebCore::CSSStyleSelector::SelectorChecker::checkOneSelector):

  • dom/Document.cpp:

(WebCore::Document::getImageMap):

  • editing/CompositeEditCommand.cpp:

(WebCore::CompositeEditCommand::inputText):

  • editing/InsertTextCommand.cpp:

(WebCore::InsertTextCommand::input):

  • editing/TextIterator.cpp:

(WebCore::TextIterator::handleTextBox):

  • editing/TypingCommand.cpp:

(WebCore::TypingCommand::insertText):

  • editing/markup.cpp:

(WebCore::fillContainerFromString):
(WebCore::createFragmentFromText):

  • html/File.cpp:

(WebCore::File::Init):

  • html/HTMLAnchorElement.cpp:

(WebCore::HTMLAnchorElement::setHost):

  • html/HTMLEmbedElement.cpp:

(WebCore::HTMLEmbedElement::parseMappedAttribute):

  • html/HTMLFormControlElement.cpp:

(WebCore::HTMLTextFormControlElement::isPlaceholderEmpty):

  • html/HTMLObjectElement.cpp:

(WebCore::HTMLObjectElement::parseMappedAttribute):

  • inspector/InspectorDOMAgent.cpp:

(WebCore::InspectorDOMAgent::performSearch):

  • loader/CrossOriginPreflightResultCache.cpp:

(WebCore::parseAccessControlAllowList):

  • loader/MainResourceLoader.cpp:

(WebCore::MainResourceLoader::substituteMIMETypeFromPluginDatabase):

  • loader/appcache/ApplicationCacheStorage.cpp:

(WebCore::parseHeader):
(WebCore::parseHeaders):

  • loader/icon/IconFetcher.cpp:

(WebCore::parseIconLink):

  • page/DOMWindow.cpp:

(WebCore::DOMWindow::parseModalDialogFeatures):

  • page/SecurityOrigin.cpp:

(WebCore::SecurityOrigin::createFromDatabaseIdentifier):

  • page/UserContentURLPattern.cpp:

(WebCore::UserContentURLPattern::parse):

  • page/XSSAuditor.cpp:

(WebCore::XSSAuditor::findInRequest):

  • platform/ContentType.cpp:

(WebCore::ContentType::parameter):
(WebCore::ContentType::type):

  • platform/KURL.cpp:

(WebCore::KURL::lastPathComponent):
(WebCore::KURL::setProtocol):
(WebCore::decodeURLEscapeSequences):
(WebCore::substituteBackslashes):
(WebCore::mimeTypeFromDataURL):

  • platform/Length.cpp:

(WebCore::newCoordsArray):
(WebCore::newLengthArray):

  • platform/LinkHash.cpp:

(WebCore::findSlashDotDotSlash):
(WebCore::findSlashSlash):
(WebCore::findSlashDotSlash):
(WebCore::cleanPath):

  • platform/MIMETypeRegistry.cpp:

(WebCore::MIMETypeRegistry::getMIMETypeForPath):

  • platform/SchemeRegistry.cpp:

(WebCore::SchemeRegistry::shouldTreatURLAsLocal):

  • platform/graphics/MediaPlayer.cpp:

(WebCore::MediaPlayer::load):

  • platform/mac/DragImageMac.mm:

(WebCore::createDragImageIconForCachedImage):

  • platform/network/CredentialStorage.cpp:

(WebCore::protectionSpaceMapKeyFromURL):
(WebCore::findDefaultProtectionSpaceForURL):

  • platform/network/HTTPParsers.cpp:

(WebCore::skipWhiteSpace):
(WebCore::skipToken):
(WebCore::parseHTTPRefresh):
(WebCore::filenameFromHTTPContentDisposition):
(WebCore::findCharsetInMediaType):
(WebCore::parseXSSProtectionHeader):
(WebCore::extractReasonPhraseFromHTTPStatusLine):

  • platform/network/ResourceResponseBase.cpp:

(WebCore::ResourceResponseBase::isAttachment):
(WebCore::parseCacheHeader):

  • rendering/RenderEmbeddedObject.cpp:

(WebCore::RenderEmbeddedObject::updateWidget):

  • storage/Entry.cpp:

(WebCore::Entry::Entry):

  • svg/SVGFont.cpp:

(WebCore::isCompatibleGlyph):

  • svg/SVGURIReference.cpp:

(WebCore::SVGURIReference::getTarget):

  • svg/animation/SVGSMILElement.cpp:

(WebCore::SVGSMILElement::parseClockValue):
(WebCore::SVGSMILElement::parseCondition):

  • xml/XPathFunctions.cpp:

(WebCore::XPath::FunSubstringBefore::evaluate):
(WebCore::XPath::FunSubstringAfter::evaluate):
(WebCore::XPath::FunTranslate::evaluate):
(WebCore::XPath::FunLang::evaluate):

  • xml/XPathParser.cpp:

(WebCore::XPath::Parser::expandQName):

4:23 PM Changeset in webkit [65467] by rniwa@webkit.org
  • 4 edits
    2 adds in trunk

2010-08-16 Ryosuke Niwa <​rniwa@webkit.org>

Reviewed by Tony Chang.

REGRESSION(r65208): editing/pasteboard/smart-paste-003.html and smart-paste-004.html crash on Windows Chromium
​https://bugs.webkit.org/show_bug.cgi?id=44070

The crash was caused by start or end node being removed in pushDownInlineStyleAroundNode.
It made start and end orphaned, and caused s.node() to be detached from the document.
Fixed the crash by using nodes passed to pushDownInlineStyleAroundNode for start and end if the original
start and end are orphaned because pushDownInlineStyleAroundNode won't prune targetNode.

Test: editing/pasteboard/smart-paste-003-trailing-whitespace.html

  • editing/ApplyStyleCommand.cpp: (WebCore::ApplyStyleCommand::removeInlineStyle):

2010-08-16 Ryosuke Niwa <​rniwa@webkit.org>

Reviewed by Tony Chang.

REGRESSION(r65208): editing/pasteboard/smart-paste-003.html and smart-paste-004.html crash on Windows Chromium
​https://bugs.webkit.org/show_bug.cgi?id=44070

The bug only reproduced on Windows when trailing space is selected only on Windows.
Added a crash test to copy & paste a word with trailing space.

  • editing/pasteboard/smart-paste-003-trailing-whitespace-expected.txt: Added.
  • editing/pasteboard/smart-paste-003-trailing-whitespace.html: Added.
  • platform/chromium/test_expectations.txt: editing/pasteboard/smart-paste-003.html and smart-paste-004.html should not crash.
4:06 PM Changeset in webkit [65466] by weinig@apple.com
  • 2 edits in trunk/WebKit2

Fix windows build.

  • Shared/win/CommandLineWin.cpp:
4:00 PM Changeset in webkit [65465] by weinig@apple.com
  • 46 edits in trunk/WebKit2

Remove #includes of <WebCore/PlatformString.h> in favor of <wtf/text/WTFString.h>

Reviewed by Gavin Barraclough.

  • Platform/Module.cpp:
  • Platform/Module.h:
  • Shared/CommandLine.h:
  • Shared/WebCoreArgumentCoders.h:
  • Shared/WebEvent.h:
  • Shared/WebNavigationDataStore.h:
  • Shared/WebPreferencesStore.h:
  • Shared/WebString.h:
  • Shared/WebURL.h:
  • Shared/mac/CommandLineMac.cpp:
  • Shared/qt/UpdateChunk.cpp:
  • UIProcess/API/C/WKContext.cpp:
  • UIProcess/API/C/cf/WKStringCF.cpp:
  • UIProcess/API/C/cf/WKURLCF.cpp:
  • UIProcess/API/cpp/qt/WKStringQt.cpp:
  • UIProcess/API/cpp/qt/WKURLQt.cpp:
  • UIProcess/API/mac/PageClientImpl.mm:
  • UIProcess/API/qt/qgraphicswkview.cpp:
  • UIProcess/Launcher/mac/ProcessLauncherMac.mm:
  • UIProcess/Launcher/qt/ProcessLauncherQt.cpp:
  • UIProcess/Launcher/win/ProcessLauncherWin.cpp:
  • UIProcess/WebBackForwardListItem.cpp:
  • UIProcess/WebBackForwardListItem.h:
  • UIProcess/WebContext.h:
  • UIProcess/WebContextInjectedBundleClient.cpp:
  • UIProcess/WebFrameProxy.cpp:
  • UIProcess/WebFrameProxy.h:
  • UIProcess/WebPageProxy.h:
  • UIProcess/WebPolicyClient.cpp:
  • UIProcess/WebProcessProxy.cpp:
  • UIProcess/WebUIClient.cpp:
  • UIProcess/win/WebView.cpp:
  • WebProcess/InjectedBundle/API/c/WKBundlePage.cpp:
  • WebProcess/InjectedBundle/InjectedBundle.h:
  • WebProcess/InjectedBundle/InjectedBundlePageEditorClient.cpp:
  • WebProcess/InjectedBundle/InjectedBundlePageLoaderClient.cpp:
  • WebProcess/InjectedBundle/InjectedBundlePageUIClient.cpp:
  • WebProcess/InjectedBundle/mac/InjectedBundleMac.cpp:
  • WebProcess/Plugins/JSNPObject.cpp:
  • WebProcess/Plugins/NPJSObject.cpp:
  • WebProcess/Plugins/Netscape/NetscapePluginModule.cpp:
  • WebProcess/Plugins/Netscape/NetscapePluginModule.h:
  • WebProcess/WebCoreSupport/WebChromeClient.h:
  • WebProcess/WebKitMain.cpp:
  • WebProcess/win/WebProcessMainWin.cpp:
3:54 PM Changeset in webkit [65464] by Nate Chapin
  • 2 edits
    1 add in trunk/LayoutTests

2010-08-16 Nate Chapin <Nate Chapin>

Chromium win rebaseline + test expectations tweaks.

  • platform/chromium-win/svg/custom/absolute-sized-content-with-resources-expected.txt: Added.
  • platform/chromium/test_expectations.txt:
3:48 PM Changeset in webkit [65463] by jamesr@google.com
  • 4 edits in trunk

fix build

3:35 PM Changeset in webkit [65462] by hyatt@apple.com
  • 1 edit in trunk/WebCore/platform/graphics/GraphicsContext3D.h

Fix Chromium/Qt bustage.

3:32 PM Changeset in webkit [65461] by hyatt@apple.com
  • 1 edit in trunk/WebKit/chromium/src/GraphicsContext3D.cpp

Fix Chromium Bustage.

3:30 PM Changeset in webkit [65460] by hyatt@apple.com
  • 2 edits in trunk

Fix Chromium Bustage.

3:22 PM Changeset in webkit [65459] by weinig@apple.com
  • 2 edits in trunk/WebKit2

Null CFURLRef returned from provisionalURL after searching on zillow.com
<rdar://problem/8261812>
​https://bugs.webkit.org/show_bug.cgi?id=44072

Reviewed by Gavin Barraclough.

Conversion from WKURLRef to CFRURLRef was breaking on URLs using characters
such as '{'.

  • UIProcess/API/C/cf/WKURLCF.cpp:

(WKURLCopyCFURL): Use code similar to that found in KURLCFNet.cpp for conversion.

2:48 PM Changeset in webkit [65458] by hyatt@apple.com
  • 3 edits in trunk/WebCore/platform/graphics

Fix Qt bustage.

2:43 PM Changeset in webkit [65457] by weinig@apple.com
  • 3 edits in trunk/WebKit2

Remove unnecessary copy constructor, assignment operator and swap function.
The defaults (for all except swap) should do just fine.

Reviewed by Gavin Barraclough.

  • Shared/WebPreferencesStore.cpp:
  • Shared/WebPreferencesStore.h:
2:37 PM Changeset in webkit [65456] by hyatt@apple.com
  • 1 edit in trunk/WebCore/platform/graphics/qt/ImageBufferQt.cpp

Fix Qt build bustage.

2:36 PM Changeset in webkit [65455] by commit-queue@webkit.org
  • 10 edits
    3 adds in trunk

2010-08-16 Vincent Scheib <​scheib@chromium.org>

Reviewed by Dimitri Glazkov.

Canvas2D does not support images larger than system's GPU max texture size
​https://bugs.webkit.org/show_bug.cgi?id=43864

A new class "TilingData" has been created that provides logic for
splitting a large image into a series of smaller tiles.

GLES2Texture and CLES2Canvas are modified to use the TilingData to
split images on texture upload and render one logical image as many smaller
images.

TilingData is tested via unit tests in WebKit/chromium/tests/TilingDataTest.cpp

  • WebCore.gypi:
  • platform/graphics/chromium/GLES2Canvas.cpp: (WebCore::GLES2Canvas::drawTexturedRect): (WebCore::GLES2Canvas::drawTexturedRectTile):
  • platform/graphics/chromium/GLES2Canvas.h:
  • platform/graphics/chromium/GLES2Texture.cpp: (WebCore::GLES2Texture::GLES2Texture): (WebCore::GLES2Texture::~GLES2Texture): (WebCore::GLES2Texture::create): (WebCore::copySubRect): (WebCore::GLES2Texture::load): (WebCore::GLES2Texture::bindTile):
  • platform/graphics/chromium/GLES2Texture.h: (WebCore::GLES2Texture::tiles):
  • platform/graphics/chromium/TilingData.cpp: Added. (WebCore::computeNumTiles): (WebCore::TilingData::TilingData): (WebCore::TilingData::tileXIndexFromSrcCoord): (WebCore::TilingData::tileYIndexFromSrcCoord): (WebCore::TilingData::tileBounds): (WebCore::TilingData::tileBoundsWithBorder): (WebCore::TilingData::tileBoundsNormalized): (WebCore::TilingData::tilePositionX): (WebCore::TilingData::tilePositionY): (WebCore::TilingData::tileSizeX): (WebCore::TilingData::tileSizeY): (WebCore::TilingData::overlappedTileIndices): (WebCore::TilingData::intersectDrawQuad):
  • platform/graphics/chromium/TilingData.h: Added. (WebCore::TilingData::maxTextureSize): (WebCore::TilingData::totalSizeX): (WebCore::TilingData::totalSizeY): (WebCore::TilingData::numTiles): (WebCore::TilingData::numTilesX): (WebCore::TilingData::numTilesY): (WebCore::TilingData::tileIndex): (WebCore::TilingData::tileXIndex): (WebCore::TilingData::tileYIndex): (WebCore::TilingData::TilingData): (WebCore::TilingData::assertTile):
  • platform/graphics/skia/ImageSkia.cpp: (WebCore::BitmapImage::draw):
  • platform/graphics/skia/PlatformContextSkia.cpp: (WebCore::PlatformContextSkia::uploadSoftwareToHardware):

2010-08-16 Vincent Scheib <​scheib@chromium.org>

Reviewed by Dimitri Glazkov.

Canvas2D does not support images larger than system's GPU max texture size
​https://bugs.webkit.org/show_bug.cgi?id=43864

Unit tests for TilingData class.

  • WebKit.gyp:
  • tests/TilingDataTest.cpp: Added. (WebCore::TEST):
2:34 PM Changeset in webkit [65454] by hyatt@apple.com
  • 1 edit in trunk/WebCore/platform/graphics/cairo/ImageCairo.cpp

Fix GTK build bustage.

2:31 PM Changeset in webkit [65453] by hyatt@apple.com
  • 1 edit in trunk/WebCore/platform/graphics/cairo/ImageBufferCairo.cpp

Fix GTK build bustage.

2:28 PM Changeset in webkit [65452] by hyatt@apple.com
  • 1 edit in trunk/WebCore/platform/graphics/qt/ImageBufferQt.cpp

Fix Qt build bustage.

2:26 PM Changeset in webkit [65451] by hyatt@apple.com
  • 1 edit in trunk/WebCore/platform/graphics/cairo/GraphicsContextCairo.cpp

Fix GTK build bustage.

2:21 PM Changeset in webkit [65450] by cmarrin@apple.com
  • 1 edit in trunk/WebCore/platform/graphics/GraphicsContext3D.h

Put ANGLE include inside PLATFORM(MAC) to fix Chrome and Qt builds

1:44 PM Changeset in webkit [65449] by hyatt@apple.com
  • 49 edits in trunk/WebCore

​https://bugs.webkit.org/show_bug.cgi?id=43507, stop ImageBuffer from copying its data when rendering after canvas changes happen.

Reviewed by Anders Carlsson.

This patch renames ImageBuffer::image() to ImageBuffer::copyImage(). The new method always returns a new copy that is a current
snapshot of the ImageBuffer.

To draw an ImageBuffer, you now use new GraphicsContext calls: drawImageBuffer. Platforms can then optimize these calls to draw
efficiently without having to copy the bits of the ImageBuffer into an Image.

  • WebCore.xcodeproj/project.pbxproj:
  • css/CSSCanvasValue.cpp:

(WebCore::CSSCanvasValue::image):

  • html/HTMLCanvasElement.cpp:

(WebCore::HTMLCanvasElement::willDraw):
(WebCore::HTMLCanvasElement::reset):
(WebCore::HTMLCanvasElement::paint):
(WebCore::HTMLCanvasElement::setSurfaceSize):
(WebCore::HTMLCanvasElement::copiedImage):
(WebCore::HTMLCanvasElement::clearCopiedImage):

  • html/HTMLCanvasElement.h:
  • html/canvas/CanvasPattern.cpp:

(WebCore::CanvasPattern::CanvasPattern):

  • html/canvas/CanvasPattern.h:

(WebCore::CanvasPattern::create):

  • html/canvas/CanvasRenderingContext2D.cpp:

(WebCore::CanvasRenderingContext2D::drawImage):
(WebCore::CanvasRenderingContext2D::createPattern):
(WebCore::CanvasRenderingContext2D::drawTextInternal):

  • html/canvas/WebGLRenderingContext.cpp:

(WebCore::WebGLRenderingContext::paintRenderingResultsToCanvas):
(WebCore::WebGLRenderingContext::reshape):
(WebCore::WebGLRenderingContext::texImage2D):
(WebCore::WebGLRenderingContext::texSubImage2D):

  • html/canvas/WebGLRenderingContext.h:

(WebCore::WebGLRenderingContext::paintsIntoCanvasBuffer):

  • platform/graphics/GeneratedImage.cpp:

(WebCore::GeneratedImage::drawPattern):

  • platform/graphics/GraphicsContext.cpp:

(WebCore::GraphicsContext::drawImageBuffer):
(WebCore::GraphicsContext::clipToImageBuffer):

  • platform/graphics/GraphicsContext.h:
  • platform/graphics/GraphicsContext3D.h:

(WebCore::GraphicsContext3D::paintsIntoCanvasBuffer):

  • platform/graphics/Image.h:
  • platform/graphics/ImageBuffer.h:

(WebCore::ImageBuffer::width):
(WebCore::ImageBuffer::height):

  • platform/graphics/Pattern.cpp:

(WebCore::Pattern::Pattern):

  • platform/graphics/Pattern.h:

(WebCore::Pattern::create):

  • platform/graphics/cairo/GraphicsContextCairo.cpp:
  • platform/graphics/cairo/ImageBufferCairo.cpp:

(WebCore::ImageBuffer::drawsUsingCopy):
(WebCore::ImageBuffer::copyImage):
(WebCore::ImageBuffer::clip):
(WebCore::ImageBuffer::draw):
(WebCore::ImageBuffer::drawPattern):

  • platform/graphics/cg/GraphicsContextCG.cpp:
  • platform/graphics/cg/ImageBufferCG.cpp:

(WebCore::ImageBuffer::ImageBuffer):
(WebCore::ImageBuffer::drawsUsingCopy):
(WebCore::ImageBuffer::copyImage):
(WebCore::ImageBuffer::draw):
(WebCore::ImageBuffer::drawPattern):
(WebCore::ImageBuffer::clip):

  • platform/graphics/cg/ImageBufferData.h:
  • platform/graphics/filters/FEColorMatrix.cpp:

(WebCore::FEColorMatrix::apply):

  • platform/graphics/filters/FEComposite.cpp:

(WebCore::FEComposite::apply):

  • platform/graphics/filters/SourceAlpha.cpp:

(WebCore::SourceAlpha::apply):

  • platform/graphics/filters/SourceGraphic.cpp:

(WebCore::SourceGraphic::apply):

  • platform/graphics/mac/GraphicsContext3DMac.mm:
  • platform/graphics/qt/GraphicsContext3DQt.cpp:
  • platform/graphics/qt/GraphicsContextQt.cpp:
  • platform/graphics/qt/ImageBufferData.h:
  • platform/graphics/qt/ImageBufferQt.cpp:

(WebCore::ImageBufferData::ImageBufferData):
(WebCore::ImageBuffer::drawsUsingCopy):
(WebCore::ImageBuffer::copyImage):
(WebCore::ImageBuffer::draw):
(WebCore::ImageBuffer::drawPattern):
(WebCore::ImageBuffer::clip):

  • platform/graphics/skia/GraphicsContextSkia.cpp:
  • platform/graphics/skia/ImageBufferSkia.cpp:

(WebCore::ImageBuffer::drawsUsingCopy):
(WebCore::ImageBuffer::copyImage):
(WebCore::ImageBuffer::clip):
(WebCore::):

  • platform/graphics/wx/GraphicsContextWx.cpp:
  • platform/graphics/wx/ImageBufferWx.cpp:

(WebCore::ImageBuffer::drawsUsingCopy):
(WebCore::ImageBuffer::copyImage):
(WebCore::ImageBuffer::clip):
(WebCore::ImageBuffer::draw):
(WebCore::ImageBuffer::drawPattern):

  • platform/mac/ScrollbarThemeMac.mm:

(WebCore::ScrollbarThemeMac::paint):

  • rendering/RenderBoxModelObject.cpp:

(WebCore::RenderBoxModelObject::paintFillLayerExtended):

  • rendering/RenderSVGResourceClipper.cpp:

(WebCore::RenderSVGResourceClipper::applyClippingToContext):

  • rendering/RenderSVGResourceFilter.cpp:

(WebCore::RenderSVGResourceFilter::postApplyResource):

  • rendering/RenderSVGResourceGradient.cpp:

(WebCore::clipToTextMask):

  • rendering/RenderSVGResourceMasker.cpp:

(WebCore::RenderSVGResourceMasker::applyResource):

  • rendering/RenderSVGResourcePattern.cpp:

(WebCore::RenderSVGResourcePattern::buildPattern):

  • rendering/RenderThemeMac.mm:

(WebCore::RenderThemeMac::paintProgressBar):

  • svg/SVGFEImageElement.cpp:

(WebCore::SVGFEImageElement::build):

  • svg/graphics/SVGImage.cpp:

(WebCore::SVGImage::nativeImageForCurrentFrame):

  • svg/graphics/SVGImage.h:
  • svg/graphics/filters/SVGFEMerge.cpp:

(WebCore::FEMerge::apply):

  • svg/graphics/filters/SVGFEOffset.cpp:

(WebCore::FEOffset::apply):

  • svg/graphics/filters/SVGFETile.cpp:

(WebCore::FETile::apply):

1:19 PM Changeset in webkit [65448] by cmarrin@apple.com
  • 2 edits in trunk/WebCore

Fixed a warning in ANGLEWebKitBridge.cpp

1:15 PM Changeset in webkit [65447] by cmarrin@apple.com
  • 1 edit in trunk/WebKitTools/Scripts/build-webkit

Build ANGLE for all versions of Mac to fix Leopard build

12:30 PM BuildingQtOnWindows edited by jocelyn.turcotte@nokia.com
python 2.6 -> python 2.x (diff)
12:18 PM Changeset in webkit [65446] by cmarrin@apple.com
  • 19 edits
    2 adds in trunk

2010-08-16 Paul Sawaya <​psawaya@apple.com>

Reviewed by Chris Marrin.

Added shader validation via ANGLE
​https://bugs.webkit.org/show_bug.cgi?id=42405

12:12 PM Changeset in webkit [65445] by kevino@webkit.org
  • 13 edits
    1 move in trunk

[wx] Build fix, do not build WebCore as a convenience library as this leads to
errors in the Win build w/export symbols and causes problems with DOM bindings
debugging in gdb.

12:00 PM Changeset in webkit [65444] by commit-queue@webkit.org
  • 2 edits in trunk/LayoutTests

2010-08-16 Mihai Parparita <​mihaip@chromium.org>

Reviewed by Nate Chapin.

Some http/tests/history tests now pass on Chromium after r65340
​https://bugs.webkit.org/show_bug.cgi?id=44033

Remove expected failures.

  • platform/chromium/test_expectations.txt:
11:24 AM Changeset in webkit [65443] by loislo@chromium.org
  • 2 edits
    2 adds in trunk/WebCore

2010-08-16 Ilya Tikhonovsky <​loislo@chromium.org>

Reviewed by Adam Roben.

Current implementation of WebCore post-build step on windows platform is error prone.
It would be better to extract the script into external file.
​https://bugs.webkit.org/show_bug.cgi?id=44058

  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.vcproj/copyForwardingHeaders.cmd: Added.
  • WebCore.vcproj/copyInspectorFiles.cmd: Added.
11:09 AM Changeset in webkit [65442] by kevino@webkit.org
  • 2 edits in trunk/WebCore

[wx] Fix typo in previous build fix.

11:04 AM Changeset in webkit [65441] by commit-queue@webkit.org
  • 5 edits in trunk

2010-08-16 Alex Milowski <​alex@milowski.com>

Reviewed by David Levin.

Added the configuration of the MathML related files.

  • WebCore.gyp/WebCore.gyp:
  • WebCore.gypi:

2010-08-16 Alex Milowski <​alex@milowski.com>

Reviewed by David Levin.

Added the ENABLE_MATHML=1 to the feature_defines.

  • features.gypi:
10:39 AM Changeset in webkit [65440] by pfeldman@chromium.org
  • 2 edits in trunk/WebCore

2010-08-16 Pavel Feldman <​pfeldman@chromium.org>

Not reviewed: fix Leopard Release inspector tests.

  • inspector/front-end/inspector.js:
10:30 AM Changeset in webkit [65439] by bulach@chromium.org
  • 7 edits
    3 copies
    1 add in trunk

2010-08-16 Marcus Bulach <​bulach@chromium.org>

Reviewed by Jeremy Orlow.

Implements IDBKeyPath extractor.
​https://bugs.webkit.org/show_bug.cgi?id=43276

Unit-test IDBKeyPathExtractorTest.cpp.
LayoutTests will arrive as IndexedDB infrastructure is fleshed out.

  • bindings/v8/IDBBindingUtilities.cpp: (WebCore::getValueFrom): (WebCore::createIDBKeyFromSerializedValueAndKeyPath):
  • bindings/v8/IDBBindingUtilities.h:

2010-08-16 Marcus Bulach <​bulach@chromium.org>

Reviewed by Jeremy Orlow.

Implements IDBKeyPath extractor.
​https://bugs.webkit.org/show_bug.cgi?id=43276

Unit-test IDBKeyPathExtractorTest.cpp.
LayoutTests will arrive as IndexedDB infrastructure is fleshed out.

  • WebKit.gyp:
  • public/WebIDBKey.h:
  • public/WebIDBKeyPath.h: Added. (WebKit::WebIDBKeyPath::WebIDBKeyPath): (WebKit::WebIDBKeyPath::~WebIDBKeyPath):
  • public/WebPrivateOwnPtr.h: Added. (WebKit::WebPrivateOwnPtr::WebPrivateOwnPtr): (WebKit::WebPrivateOwnPtr::~WebPrivateOwnPtr): (WebKit::WebPrivateOwnPtr::reset): (WebKit::WebPrivateOwnPtr::get): (WebKit::WebPrivateOwnPtr::operator->):
  • src/WebIDBKey.cpp: (WebKit::WebIDBKey::createFromValueAndKeyPath):
  • src/WebIDBKeyPath.cpp: Added. (WebKit::WebIDBKeyPath::create): (WebKit::WebIDBKeyPath::WebIDBKeyPath): (WebKit::WebIDBKeyPath::parseError): (WebKit::WebIDBKeyPath::assign): (WebKit::WebIDBKeyPath::reset): (WebKit::WebIDBKeyPath::operator const WTF::Vector<WebCore::IDBKeyPathElement, 0>&):
  • tests/IDBBindingUtilitiesTest.cpp: Added. (WebCore::LocalContext::LocalContext): (WebCore::LocalContext::~LocalContext): (WebCore::checkKeyFromValueAndKeyPathInternal): (WebCore::checkKeyPathNullValue): (WebCore::checkKeyPathStringValue): (WebCore::checkKeyPathNumberValue): (WebCore::TEST):
10:10 AM Changeset in webkit [65438] by kov@webkit.org
  • 2 edits in trunk/WebCore

2010-08-13 Gustavo Noronha Silva <​gustavo.noronha@collabora.co.uk>

Reviewed by Kenneth Rohde Christiansen and Martin Robinson.

[GTK] GTK+ drawing idle starved
​https://bugs.webkit.org/show_bug.cgi?id=43994

Tune down the priority of the shared timers to guarantee GTK+ has
an opportunity to redraw.

  • platform/gtk/SharedTimerGtk.cpp: (WebCore::setSharedTimerFireTime):
9:44 AM Changeset in webkit [65437] by leandro@webkit.org
  • 8 edits in trunk

2010-08-16 Leandro Pereira <​leandro@profusion.mobi>

[EFL] Build fix after r65366.

  • CMakeLists.txt: Use if (VAR) instead of if (${VAR}) to check if they're empty.
  • jsc/CMakeLists.txt: Ditto.
  • wtf/CMakeLists.txt: Ditto.

2010-08-16 Leandro Pereira <​leandro@profusion.mobi>

[EFL] Build fix after r65366.

  • CMakeLists.txt: Use if (VAR) instead of if (${VAR}) to check if they're empty.

2010-08-16 Leandro Pereira <​leandro@profusion.mobi>

[EFL] Build fix after r65366.

EFL port does not support automated tests yet.

  • CMakeLists.txt: Use if (VAR) instead of if (${VAR}) to check if they're empty.
9:40 AM Changeset in webkit [65436] by steveblock@google.com
  • 2 edits in trunk/LayoutTests

Unreviewed build fix.

Add fast/dom/Geolocation/disconnected-frame-permission-denied.html to failing tests for Chromium
​https://bugs.webkit.org/show_bug.cgi?id=44059

  • platform/chromium/test_expectations.txt:
9:35 AM Changeset in webkit [65435] by mitz@apple.com
  • 2 edits in trunk/WebKitTools

Build fix.

  • WebKitTestRunner/TestController.cpp:

(WTR::createOtherPage): Initialize the contentsSizeChanged member.
(WTR::TestController::initialize): Ditto.

9:33 AM Changeset in webkit [65434] by pfeldman@chromium.org
  • 6 edits
    1 delete in trunk

2010-08-16 Pavel Feldman <​pfeldman@chromium.org>

Reviewed by Yury Semikhatsky.

Web Inspector: Make InjectedScript proto-based.
​https://bugs.webkit.org/show_bug.cgi?id=44028

  • inspector/front-end/InjectedScript.js: (injectedScriptConstructor): (injectedScriptConstructor.):

2010-08-16 Pavel Feldman <​pfeldman@chromium.org>

Reviewed by Yury Semikhatsky.

Web Inspector: Make InjectedScript proto-based.
​https://bugs.webkit.org/show_bug.cgi?id=44028
Drive-by test fix.

  • http/tests/inspector-enabled/console-log-before-frame-navigation.html:
  • http/tests/inspector-enabled/resources/console-log-before-frame-navigation.js: Removed.
  • http/tests/inspector-enabled/resources/console-log-frame-before-navigation.html:
9:31 AM Changeset in webkit [65433] by kevino@webkit.org
  • 2 edits in trunk/WebKit/wx

wxMSW build fix. Don't build the Python DOM bindings until we get the export issues
sorted out.

9:29 AM Changeset in webkit [65432] by Csaba Osztrogonác
  • 2 edits in trunk/WebCore

Fix warning in WebCore/plugins/win/PluginPackageWin.cpp
​https://bugs.webkit.org/show_bug.cgi?id=43920

Reviewed by Adam Roben.

  • plugins/win/PluginPackageWin.cpp:

(WebCore::PluginPackage::fetchInfo):

9:19 AM Changeset in webkit [65431] by Csaba Osztrogonác
  • 2 edits in trunk/LayoutTests

Unreviewed.

[Qt] fast/dom/Geolocation/disconnected-frame-permission-denied.html fails
​https://bugs.webkit.org/show_bug.cgi?id=44059

  • platform/qt/Skipped: fast/dom/Geolocation/disconnected-frame-permission-denied.html skipped until fix.
9:15 AM Changeset in webkit [65430] by pfeldman@chromium.org
  • 2 edits in trunk/WebCore

2010-08-16 Pavel Feldman <​pfeldman@chromium.org>

Not reviewed. Touch inspector file to force frontend deployment.

  • inspector/InspectorController.cpp:
9:06 AM Changeset in webkit [65429] by kevino@webkit.org
  • 2 edits in trunk/WebCore

[wx] Build fix for non-Win ports on Windows.

8:16 AM Changeset in webkit [65428] by agl@chromium.org
  • 5 edits in trunk

2010-08-16 Adam Langley <​agl@chromium.org>

Reviewed by Tony Chang.

[chromium] Improve spacing support for complex text on Linux

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

Previously, our complex text support ignored word-spacing,
justification and letter-spacing. This fixes the first two issues and
allows us to render Scribd's HTML5 documents much better.

Test: fast/text/atsui-spacing-features.html

  • platform/graphics/chromium/FontLinux.cpp: (WebCore::TextRunWalker::TextRunWalker): (WebCore::TextRunWalker::setWordSpacingAdjustment): (WebCore::TextRunWalker::setLetterSpacingAdjustment): (WebCore::TextRunWalker::setPadding): (WebCore::TextRunWalker::setGlyphXPositions): (WebCore::TextRunWalker::isCodepointSpace): (WebCore::Font::drawComplexText): (WebCore::Font::floatWidthForComplexText): (WebCore::Font::offsetForPositionForComplexText): (WebCore::Font::selectionRectForComplexText):
8:14 AM Changeset in webkit [65427] by Csaba Osztrogonác
  • 5 edits in trunk/WebCore

Fix warnings: unknown conversion type character 'l' in format
​https://bugs.webkit.org/show_bug.cgi?id=43359

Reviewed by Kenneth Rohde Christiansen.

  • loader/icon/IconDatabase.cpp:

(WebCore::IconDatabase::performURLImport): Replace %zu with %lu, because Windows doesn't understand z modifier.
(WebCore::IconDatabase::pruneUnretainedIcons): Use %I64i intsead of %lli on Windows.

  • platform/graphics/qt/MediaPlayerPrivatePhonon.cpp:

(WebCore::MediaPlayerPrivate::totalTimeChanged): Use %I64d intsead of %lld on Windows.

  • platform/sql/SQLiteDatabase.cpp:

(WebCore::SQLiteDatabase::setMaximumSize): Use %I64i intsead of %lli on Windows.

  • storage/DatabaseTracker.cpp:

(WebCore::DatabaseTracker::setQuota): Use %I64u intsead of %llu on Windows.

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

Fix warnings: unknown conversion type character 'l' in format
​https://bugs.webkit.org/show_bug.cgi?id=43359

Reviewed by Kenneth Rohde Christiansen.

  • loader/FTPDirectoryParser.cpp: Use %I64u format specifier instead of %llu on Windows.

(WebCore::parseOneFTPLine):

7:32 AM Changeset in webkit [65425] by ariya@webkit.org
  • 6 edits
    2 adds in trunk

[Qt] Shadow blur for rectangle fill
​https://bugs.webkit.org/show_bug.cgi?id=44025

Patch by Ariya Hidayat <​ariya@sencha.com> on 2010-08-16
Reviewed by Kenneth Rohde Christiansen.

WebCore:

The blur implementation follows the approximation of Gaussian blur
with three successive box blurs, working on the alpha channel only.
(see ​http://www.w3.org/TR/SVG/filters.html#feGaussianBlur).

  • WebCore.pro: Refer to ContexShadow.* files.
  • platform/graphics/qt/ContextShadow.cpp: Added.

(WebCore::ContextShadow::ContextShadow):
(WebCore::ContextShadow::clear):
(WebCore::blurHorizontal): Box blurs in horizontal direction, working
on QImage (in-place).
(WebCore::shadowBlur): Main blur function which just calls the
above blurHorizontal function twice, once with transposed image
(equivalent as it would have done vertical box blurs).
(WebCore::ContextShadow::drawShadowRect):

  • platform/graphics/qt/ContextShadow.h: Added.

(WebCore::ContextShadow::):

  • platform/graphics/qt/GraphicsContextQt.cpp:

(WebCore::GraphicsContextPlatformPrivate::hasShadow):
(WebCore::GraphicsContext::restorePlatformState):
(WebCore::GraphicsContext::fillRect): Use ContextShadow for the shadow.
(WebCore::GraphicsContext::setPlatformShadow):

LayoutTests:

Update pixel test reference after the blur support.

  • platform/qt/fast/box-shadow/basic-shadows-expected.checksum:
  • platform/qt/fast/box-shadow/basic-shadows-expected.png:
7:09 AM Changeset in webkit [65424] by abecsi@webkit.org
  • 2 edits in trunk/WebCore

2010-08-16 Andras Becsi <​abecsi@webkit.org>

Rubber-stamped by Gustavo Noronha Silva.

[GTK] Build fix for MathML code generation.

No new tests needed.

  • GNUmakefile.am:
6:58 AM Changeset in webkit [65423] by commit-queue@webkit.org
  • 2 edits in trunk/WebCore

2010-08-16 Sheriff Bot <​webkit.review.bot@gmail.com>

Unreviewed, rolling out r65414.
​http://trac.webkit.org/changeset/65414
​https://bugs.webkit.org/show_bug.cgi?id=44055

GTL & QT tests failing. (Requested by pfeldman on #webkit).

  • inspector/front-end/InjectedScript.js: (injectedScriptConstructor): (injectedScriptConstructor.):
6:39 AM Changeset in webkit [65422] by abecsi@webkit.org
  • 12 edits in trunk

2010-08-16 Balazs Kelemen <​kb@inf.u-szeged.hu>

Reviewed by Kenneth Rohde Christiansen.

Handle content size change in WebKit2
Re-landing after fix.

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

  • MiniBrowser/mac/BrowserWindowController.m: (-[BrowserWindowController awakeFromNib]): Initialize WKPageUICallback::contetsSizeChanged to 0.
  • MiniBrowser/win/BrowserView.cpp: (BrowserView::create): Initialize WKPageUICallback::contetsSizeChanged to 0.

2010-08-16 Balazs Kelemen <​kb@inf.u-szeged.hu>

Reviewed by Kenneth Rohde Christiansen.

Handle content size change in WebKit2
Re-landing after fix.

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

Based on the work of Antti Koivisto.
Send message to the UI client when the contents size has changed through the WebChromeClient
and propagate it to the WKPageUIClient.

  • Shared/CoreIPCSupport/WebPageProxyMessageKinds.h: (WebPageProxyMessage::): Added ContentsSizeChanged message kind.
  • UIProcess/API/C/WKPage.h: Added WKPageContentsSizeChangedCallback callback to the WKPageUIClient.
  • UIProcess/API/qt/qwkpage.cpp: (QWKPage::QWKPage): Initialize the new callback to 0.
  • UIProcess/WebPageProxy.cpp: (WebKit::WebPageProxy::didReceiveMessage): Handle ContentsSizeChanged message. Calls contetsSizeChanged. (WebKit::WebPageProxy::contentsSizeChanged): Added. Propagate the event to the the UI client.
  • UIProcess/WebPageProxy.h:
  • UIProcess/WebUIClient.cpp: (WebKit::WebUIClient::contentsSizeChanged): Added. Propagate the event forward to the WKPageUIClient.
  • UIProcess/WebUIClient.h:
  • WebProcess/WebCoreSupport/WebChromeClient.cpp: (WebKit::WebChromeClient::contentsSizeChanged): Implemented.
6:28 AM Changeset in webkit [65421] by commit-queue@webkit.org
  • 12 edits in trunk

2010-08-16 Sheriff Bot <​webkit.review.bot@gmail.com>

Unreviewed, rolling out r65419.
​http://trac.webkit.org/changeset/65419
​https://bugs.webkit.org/show_bug.cgi?id=44053

Broke the Windows build (Requested by bbandix on #webkit).

  • Shared/CoreIPCSupport/WebPageProxyMessageKinds.h: (WebPageProxyMessage::):
  • UIProcess/API/C/WKPage.h:
  • UIProcess/API/qt/qwkpage.cpp: (QWKPage::QWKPage):
  • UIProcess/WebPageProxy.cpp: (WebKit::WebPageProxy::didReceiveMessage):
  • UIProcess/WebPageProxy.h:
  • UIProcess/WebUIClient.cpp:
  • UIProcess/WebUIClient.h:
  • WebProcess/WebCoreSupport/WebChromeClient.cpp: (WebKit::WebChromeClient::contentsSizeChanged):

2010-08-16 Sheriff Bot <​webkit.review.bot@gmail.com>

Unreviewed, rolling out r65419.
​http://trac.webkit.org/changeset/65419
​https://bugs.webkit.org/show_bug.cgi?id=44053

Broke the Windows build (Requested by bbandix on #webkit).

  • MiniBrowser/mac/BrowserWindowController.m: (-[BrowserWindowController awakeFromNib]):
  • MiniBrowser/win/BrowserView.cpp: (BrowserView::create):
5:58 AM Changeset in webkit [65420] by ariya@webkit.org
  • 2 edits in trunk/WebCore

2010-08-16 Ariya Hidayat <​ariya@sencha.com>

Reviewed by Antonio Gomes.

[Qt] Save and restore shadow state in GraphicsContextQt
​https://bugs.webkit.org/show_bug.cgi?id=44031

This is needed after r65362.
Use a new class ContextShadowParameter to track and keep the shadow
state. We use this rather than using GraphicsContextState to allow
possible optimizations (right now only to determine the shadow type,
but in future it might cover things like cached scratch image,
persistent shader, etc).

  • platform/graphics/qt/GraphicsContextQt.cpp: (WebCore::ContextShadowParameter::): (WebCore::ContextShadowParameter::ContextShadowParameter): (WebCore::ContextShadowParameter::clear): (WebCore::GraphicsContextPlatformPrivate::hasShadow): (WebCore::GraphicsContextPlatformPrivate::GraphicsContextPlatformPrivate): (WebCore::GraphicsContext::savePlatformState): (WebCore::GraphicsContext::restorePlatformState): (WebCore::GraphicsContext::drawRect): (WebCore::GraphicsContext::drawLine): (WebCore::GraphicsContext::strokeArc): (WebCore::GraphicsContext::drawConvexPolygon): (WebCore::GraphicsContext::fillPath): (WebCore::GraphicsContext::strokePath): (WebCore::GraphicsContext::fillRect): (WebCore::GraphicsContext::fillRoundedRect): (WebCore::GraphicsContext::setPlatformShadow): (WebCore::GraphicsContext::clearPlatformShadow):
5:52 AM Changeset in webkit [65419] by abecsi@webkit.org
  • 12 edits in trunk

2010-08-16 Balazs Kelemen <​kb@inf.u-szeged.hu>

Reviewed by Kenneth Rohde Christiansen.

Handle content size change in WebKit2

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

  • MiniBrowser/mac/BrowserWindowController.m: (-[BrowserWindowController awakeFromNib]): Initialize WKPageUICallback::contetsSizeChanged to 0.
  • MiniBrowser/win/BrowserView.cpp: (BrowserView::create): Initialize WKPageUICallback::contetsSizeChanged to 0.

2010-08-16 Balazs Kelemen <​kb@inf.u-szeged.hu>

Reviewed by Kenneth Rohde Christiansen.

Handle content size change in WebKit2

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

Based on the work of Antti Koivisto.
Send message to the UI client when the contents size has changed through the WebChromeClient
and propagate it to the WKPageUIClient.

  • Shared/CoreIPCSupport/WebPageProxyMessageKinds.h: (WebPageProxyMessage::): Added ContentsSizeChanged message kind.
  • UIProcess/API/C/WKPage.h: Added WKPageContentsSizeChangedCallback callback to the WKPageUIClient.
  • UIProcess/API/qt/qwkpage.cpp: (QWKPage::QWKPage): Initialize the new callback to 0.
  • UIProcess/WebPageProxy.cpp: (WebKit::WebPageProxy::didReceiveMessage): Handle ContentsSizeChanged message. Calls contetsSizeChanged. (WebKit::WebPageProxy::contentsSizeChanged): Added. Propagate the event to the the UI client.
  • UIProcess/WebPageProxy.h:
  • UIProcess/WebUIClient.cpp: (WebKit::WebUIClient::contentsSizeChanged): Added. Propagate the event forward to the WKPageUIClient.
  • UIProcess/WebUIClient.h:
  • WebProcess/WebCoreSupport/WebChromeClient.cpp: (WebKit::WebChromeClient::contentsSizeChanged): Implemented.
5:44 AM Changeset in webkit [65418] by ariya@webkit.org
  • 2 edits in trunk/WebKitTools

Add my new email address to committers.py.

  • Scripts/webkitpy/common/config/committers.py:
5:43 AM Changeset in webkit [65417] by bulach@chromium.org
  • 4 edits
    3 adds in trunk

2010-07-28 Marcus Bulach <​bulach@chromium.org>

Reviewed by Adam Barth.

Implement IDBKeyPath parser.
​https://bugs.webkit.org/show_bug.cgi?id=42976

IDBKeyPath is an internal class, exposed only for unit tests for now.
It'll allow indexed database implementation to traverse a given key path and obtain the corresponding properties
from a SerializedScriptValue.

  • WebCore.gypi:
  • storage/IDBKeyPath.cpp: Added. (WebCore::IDBKeyPath::create): (WebCore::IDBKeyPath::IDBKeyPath): (WebCore::IDBKeyPath::~IDBKeyPath): (WebCore::IDBKeyPath::hasNext): (WebCore::IDBKeyPath::next): (WebCore::IDBKeyPath::identifier): (WebCore::IDBKeyPath::hasIndex): (WebCore::IDBKeyPath::indexValue): (WebCore::IDBKeyPath::parserError): (WebCore::IDBKeyPath::parse): (WebCore::IDBKeyPath::Lexer::lex): (WebCore::isSafeIdentifierStartCharacter): (WebCore::isSafeIdentifierCharacter): (WebCore::IDBKeyPath::Lexer::lexIdentifier): (WebCore::IDBKeyPath::Lexer::lexNumber):
  • storage/IDBKeyPath.h: Added. (WebCore::IDBKeyPath::Token::Token): (WebCore::IDBKeyPath::): (WebCore::IDBKeyPath::Lexer::Lexer): (WebCore::IDBKeyPath::Lexer::next): (WebCore::IDBKeyPath::Lexer::currentToken):

2010-07-28 Marcus Bulach <​bulach@chromium.org>

Reviewed by Adam Barth.

Implement IDBKeyPath parser.
​https://bugs.webkit.org/show_bug.cgi?id=42976

IDBKeyPath is an internal class, exposed only for unit tests for now.
It'll allow indexed database implementation to traverse a given key path and obtain the corresponding properties
from a SerializedScriptValue.

  • WebKit.gyp:
  • tests/IDBKeyPathTest.cpp: Added. (WebCore::ExpectedToken::ExpectedToken): (WebCore::checkKeyPath): (WebCore::TEST):
5:30 AM Changeset in webkit [65416] by steveblock@google.com
  • 5 edits
    7 adds in trunk

2010-08-13 Steve Block <​steveblock@google.com>

Reviewed by Alexey Proskuryakov.

Geolocation requests in progress when the frame is disconnected should invoke the error callback
​https://bugs.webkit.org/show_bug.cgi?id=43974

If requests are ongoing when the Frame is disconnected, we abort them with a fatal error.
To do this, when Geolocation::disconnectFrame() is called we call cancelAllRequests() to
set a fatal error on all ongoing requests. Once the requests have made their error
callbacks, they are deleted. Note that we check at callback time that the script
execution context for the callback is still valid, so it's safe to attempt these
callbacks even after the Geolocation object's Frame has gone.

This requires a change to allow multiple calls to GeoNotifier::setFatalError().
For repeated calls, we do not replace the existing error. This ensures that when
permission has been denied and the frame is then disconnected, the error callback
reports the permission error, as required by the spec.

Tests: fast/dom/Geolocation/disconnected-frame.html

fast/dom/Geolocation/disconnected-frame-permission-denied.html

  • page/Geolocation.cpp: (WebCore::Geolocation::GeoNotifier::setFatalError): (WebCore::Geolocation::disconnectFrame): (WebCore::Geolocation::startRequest): (WebCore::Geolocation::cancelAllRequests):
  • page/Geolocation.h:

2010-08-13 Steve Block <​steveblock@google.com>

Reviewed by Alexey Proskuryakov.

Geolocation requests in progress when the frame is disconnected should invoke the error callback
​https://bugs.webkit.org/show_bug.cgi?id=43974

Added new tests to GTK skipped list.

  • fast/dom/Geolocation/disconnected-frame.html: Added.
  • fast/dom/Geolocation/disconnected-frame-expected.txt: Added.
  • fast/dom/Geolocation/script-tests/disconnected-frame.js: Added.
  • fast/dom/Geolocation/resources/disconnected-frame-inner.html: Added.
  • fast/dom/Geolocation/disconnected-frame-permission-denied.html: Added.
  • fast/dom/Geolocation/disconnected-frame-permission-denied-expected.txt: Added.
  • fast/dom/Geolocation/script-tests/disconnected-frame-permission-denied.js: Added.
  • platform/gtk/Skipped:
5:23 AM Changeset in webkit [65415] by pfeldman@chromium.org
  • 5 edits in trunk

2010-08-16 Pavel Feldman <​pfeldman@chromium.org>

Reviewed by Joseph Pecoraro.

Web Inspector: upstream frontend-side WebSocket transport.
​https://bugs.webkit.org/show_bug.cgi?id=43970

Chromium already has an alternate WebSocket-based communication channel with
the backend. Upstreaming it in this change. We will agree on the URI
of the remote service as the protocol matures.

2010-08-16 Pavel Feldman <​pfeldman@chromium.org>

Reviewed by Joseph Pecoraro.

Web Inspector: upstream frontend-side WebSocket transport.
​https://bugs.webkit.org/show_bug.cgi?id=43970

Chromium already has an alternate WebSocket-based communication channel with
the backend. Upstreaming it in this change. We will agree on the URI
of the remote service as the protocol matures.

  • src/WebDevToolsFrontendImpl.cpp: (WebKit::WebDevToolsFrontendImpl::dispatchOnInspectorFrontend):
  • src/js/DevTools.js: (WebInspector.loaded): (devtools.domContentLoaded):
5:19 AM Changeset in webkit [65414] by pfeldman@chromium.org
  • 2 edits in trunk/WebCore

2010-08-16 Pavel Feldman <​pfeldman@chromium.org>

Reviewed by Yury Semikhatsky.

Web Inspector: Make InjectedScript proto-based.
​https://bugs.webkit.org/show_bug.cgi?id=44028

  • inspector/front-end/InjectedScript.js: (injectedScriptConstructor): (injectedScriptConstructor.):
  • inspector/front-end/inspector.js: (WebInspector.loaded):
5:16 AM Changeset in webkit [65413] by yurys@chromium.org
  • 3 edits in trunk/WebKit/chromium

2010-08-16 Yury Semikhatsky <​yurys@chromium.org>

Reviewed by Pavel Feldman.

Web Inspector: remove utility context related code from debugger agent
​https://bugs.webkit.org/show_bug.cgi?id=44049

  • src/DebuggerAgentManager.cpp: (WebKit::DebuggerAgentManager::onV8DebugMessage): (WebKit::DebuggerAgentManager::pauseScript):
  • src/DebuggerAgentManager.h:
5:16 AM Changeset in webkit [65412] by pfeldman@chromium.org
  • 8 edits
    2 adds in trunk

2010-08-16 Pavel Feldman <​pfeldman@chromium.org>

Reviewed by Joseph Pecoraro.

Web Inspector: [REGRESSION] console's clear command does not work.
​https://bugs.webkit.org/show_bug.cgi?id=44027

  • inspector/console-clear.html:
  • inspector/console-command-clear-expected.txt: Added.
  • inspector/console-command-clear.html: Copied from LayoutTests/inspector/console-clear.html.

2010-08-16 Pavel Feldman <​pfeldman@chromium.org>

Reviewed by Joseph Pecoraro.

Web Inspector: [REGRESSION] console's clear command does not work.
​https://bugs.webkit.org/show_bug.cgi?id=44027

Test: inspector/console-command-clear.html

  • inspector/Inspector.idl:
  • inspector/InspectorBackend.cpp:
  • inspector/InspectorBackend.h:
  • inspector/InspectorController.cpp: (WebCore::InspectorController::clearConsoleMessages):
  • inspector/front-end/ConsoleView.js: (WebInspector.ConsoleView.prototype.requestClearMessages):
  • inspector/front-end/inspector.js: (WebInspector.consoleMessagesCleared):
4:57 AM Changeset in webkit [65411] by hamaji@chromium.org
  • 2 edits in trunk/WebKit/chromium

2010-08-16 Yuta Kitamura <​yutak@chromium.org>

Reviewed by Shinichiro Hamaji.

Attempt to fix Chromium's test_shell_tests.

Port Chromium's WebEntities to HTMLEntityTable
​https://bugs.webkit.org/show_bug.cgi?id=44036

  • src/WebEntities.cpp: (WebKit::):
4:26 AM Changeset in webkit [65410] by jorlow@chromium.org
  • 5 edits in trunk/WebCore

2010-08-16 Jeremy Orlow <​jorlow@chromium.org>

Ugh, the #if ENABLE() was supposed to be added to Settings not GroupSettings.

  • page/GroupSettings.cpp: (WebCore::GroupSettings::GroupSettings): (WebCore::GroupSettings::setLocalStorageQuotaBytes):
  • page/GroupSettings.h:
  • page/Settings.cpp: (WebCore::Settings::Settings):
  • page/Settings.h:
4:09 AM Changeset in webkit [65409] by jorlow@chromium.org
  • 2 edits in trunk/WebCore

2010-08-16 Jeremy Orlow <​jorlow@chromium.org>

Last build fix?

  • page/PageGroup.cpp: (WebCore::PageGroup::PageGroup):
3:53 AM Changeset in webkit [65408] by jorlow@chromium.org
  • 3 edits in trunk/WebCore

2010-08-16 Jeremy Orlow <​jorlow@chromium.org>

Ugh...another build fix.

  • page/GroupSettings.cpp: (WebCore::GroupSettings::GroupSettings):
  • page/GroupSettings.h:
3:40 AM Changeset in webkit [65407] by jorlow@chromium.org
  • 2 edits in trunk/WebCore

2010-08-16 Jeremy Orlow <​jorlow@chromium.org>

Build fix.

  • GNUmakefile.am:
3:20 AM Changeset in webkit [65406] by jorlow@chromium.org
  • 2 edits in trunk/WebCore

2010-08-16 Jeremy Orlow <​jorlow@chromium.org>

Build fix.

  • page/PageGroup.cpp: (WebCore::PageGroup::localStorage):
3:02 AM Changeset in webkit [65405] by jorlow@chromium.org
  • 12 edits
    2 adds in trunk/WebCore

2010-08-13 Jeremy Orlow <​jorlow@chromium.org>

Some settings are linked to the PageGroup not the Page. Create a new class for those.
​https://bugs.webkit.org/show_bug.cgi?id=43794

Since persistient storage is shared between pages, there's no way to modify
some settings related to it on a per page basis. As such, it's not technically
correct for these settings to be on the Page's settings. Create a new class
called GroupSettings, move group-wide local storage setting there (the other we
can't), and add a new setting for IndexedDB's path (which is prompting this change).

No behavior has changed, so no tests.

  • Android.mk:
  • CMakeLists.txt:
  • GNUmakefile.am:
  • WebCore.gypi:
  • WebCore.pro:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • page/GroupSettings.cpp: Added. (WebCore::GroupSettings::GroupSettings): (WebCore::GroupSettings::setLocalStorageQuotaBytes): (WebCore::GroupSettings::setLocalStorageDatabasePath): (WebCore::GroupSettings::setIndexedDBDatabasePath):
  • page/GroupSettings.h: Added. (WebCore::GroupSettings::create): (WebCore::GroupSettings::localStorageQuotaBytes): (WebCore::GroupSettings::localStorageDatabasePath): (WebCore::GroupSettings::indexedDBDatabasePath):
  • page/PageGroup.cpp: (WebCore::PageGroup::PageGroup): (WebCore::PageGroup::localStorage):
  • page/PageGroup.h: (WebCore::PageGroup::groupSettings):
  • page/Settings.cpp: (WebCore::Settings::Settings): (WebCore::Settings::setSessionStorageQuota):
  • page/Settings.h:
2:44 AM Changeset in webkit [65404] by morrita@google.com
  • 2 edits in trunk/LayoutTests

2010-08-16 MORITA Hajime <​morrita@google.com>

Unreviewed; test expectations fix for Chromium.

  • platform/chromium/test_expectations.txt:
2:21 AM Changeset in webkit [65403] by zoltan@webkit.org
  • 3 edits in trunk/WebKit2

Unreviewed Qt WebKit2 build fix.

Patch by Balazs Kelemen <​kb@inf.u-szeged.hu> on 2010-08-16

  • UIProcess/qt/WebContextQt.cpp:

(WebKit::WebContext::platformSetUpWebProcess): Added empty stub implementation.

  • WebKit2.pro: Enumerate new files, include WebKit.pri for correct define macro set.
2:09 AM Changeset in webkit [65402] by abarth@webkit.org
  • 1 edit
    2 adds in trunk/LayoutTests

2010-08-16 Adam Barth <​abarth@webkit.org>

Reviewed by Alexey Proskuryakov.

document.write causes the links to appear twice (noscript content is not ignored)
​https://bugs.webkit.org/show_bug.cgi?id=16230

Add a test for this case.

  • fast/parser/document-write-noscript-expected.txt: Added.
  • fast/parser/document-write-noscript.html: Added.
1:51 AM Changeset in webkit [65401] by abarth@webkit.org
  • 1 edit
    2 adds in trunk/LayoutTests

2010-08-16 Adam Barth <​abarth@webkit.org>

Reviewed by Alexey Proskuryakov.

document.write() isn't processed immediately before DOM access
​https://bugs.webkit.org/show_bug.cgi?id=36881

Test that document.write gets flushed before DOM access takes place.

  • fast/tokenizer/document-write-racing-dom-access-expected.txt: Added.
  • fast/tokenizer/document-write-racing-dom-access.html: Added.
1:45 AM Changeset in webkit [65400] by hamaji@chromium.org
  • 15 edits
    12 adds in trunk/LayoutTests

2010-08-16 Yuta Kitamura <​yutak@chromium.org>

Not reviewed. Chromium test expectations updates and rebaselines.

  • platform/chromium-linux/fast/canvas/setWidthResetAfterForcedRender-expected.checksum: Added.
  • platform/chromium-linux/fast/canvas/setWidthResetAfterForcedRender-expected.png: Added.
  • platform/chromium-linux/fast/text/word-space-expected.checksum:
  • platform/chromium-linux/fast/text/word-space-expected.png:
  • platform/chromium-linux/tables/mozilla/other/wa_table_thtd_rowspan-expected.checksum:
  • platform/chromium-linux/tables/mozilla/other/wa_table_thtd_rowspan-expected.png:
  • platform/chromium-mac/fast/canvas/setWidthResetAfterForcedRender-expected.checksum: Added.
  • platform/chromium-mac/fast/canvas/setWidthResetAfterForcedRender-expected.png: Added.
  • platform/chromium-mac/fast/text/word-space-expected.checksum: Added.
  • platform/chromium-mac/fast/text/word-space-expected.png: Added.
  • platform/chromium-mac/tables/mozilla/other/wa_table_thtd_rowspan-expected.checksum: Added.
  • platform/chromium-mac/tables/mozilla/other/wa_table_thtd_rowspan-expected.png: Added.
  • platform/chromium-win/fast/canvas/setWidthResetAfterForcedRender-expected.checksum: Added.
  • platform/chromium-win/fast/canvas/setWidthResetAfterForcedRender-expected.png: Added.
  • platform/chromium-win/fast/canvas/setWidthResetAfterForcedRender-expected.txt: Added.
  • platform/chromium-win/fast/text/word-space-expected.checksum:
  • platform/chromium-win/fast/text/word-space-expected.png:
  • platform/chromium-win/fast/text/word-space-expected.txt:
  • platform/chromium-win/svg/W3C-SVG-1.1/animate-elem-32-t-expected.txt:
  • platform/chromium-win/svg/W3C-SVG-1.1/shapes-intro-01-t-expected.txt:
  • platform/chromium-win/tables/mozilla/other/wa_table_thtd_rowspan-expected.checksum:
  • platform/chromium-win/tables/mozilla/other/wa_table_thtd_rowspan-expected.png:
  • platform/chromium-win/tables/mozilla/other/wa_table_thtd_rowspan-expected.txt:
  • platform/chromium-win/tables/mozilla/other/wa_table_tr_align-expected.txt:
  • platform/chromium/test_expectations.txt:
1:45 AM Changeset in webkit [65399] by zoltan@webkit.org
  • 2 edits in trunk/WebKitTools

2010-08-16 Zoltan Horvath <​zoltan@webkit.org>

Add my old e-mail addresses to committers.py.

  • Scripts/webkitpy/common/config/committers.py:
1:22 AM Changeset in webkit [65398] by morrita@google.com
  • 2 edits in trunk/LayoutTests

2010-08-16 MORITA Hajime <​morrita@google.com>

Unreviewd, added skip list entries.

This is follow up for r65395. QT port ignores tests under
editing/pasteboard.

  • platform/qt/Skipped:
1:14 AM Changeset in webkit [65397] by morrita@google.com
  • 2 edits in trunk/LayoutTests

2010-08-16 MORITA Hajime <​morrita@google.com>

Unreviewd, added skip list entries.

This is follow up for r65395. GTK port ignores tests under
editing/pasteboard.

  • platform/gtk/Skipped:
1:04 AM Changeset in webkit [65396] by abarth@webkit.org
  • 3 edits
    2 adds in trunk

2010-08-16 Adam Barth <​abarth@webkit.org>

Reviewed by Alexey Proskuryakov.

HTML5 fragment parsing doesn't load iframes
​https://bugs.webkit.org/show_bug.cgi?id=44038

Test that iframes created in detached documents actually get loaded
when they're adopted into attached documents (or, rather, when they're
inserted into the DOM).

  • fast/frames/adopt-from-created-document-expected.txt: Added.
  • fast/frames/adopt-from-created-document.html: Added.

2010-08-16 Adam Barth <​abarth@webkit.org>

Reviewed by Alexey Proskuryakov.

HTML5 fragment parsing doesn't load iframes
​https://bugs.webkit.org/show_bug.cgi?id=44038

Previously, this code assumed that every source iframe was actually
attached to a render tree. It tried to "keep the iframe alive" as it
moved documents. However, if the iframe wasn't attached to begin with,
it was never "alive," so trying to keep a dead iframe alive lead to a
... zombie iframe.

Test: fast/frames/adopt-from-created-document.html

  • dom/Document.cpp: (WebCore::Document::adoptNode):
12:30 AM Changeset in webkit [65395] by morrita@google.com
  • 8 edits
    6 adds in trunk

2010-08-13 MORITA Hajime <​morrita@google.com>

Reviewed by Tony Chang.

​https://bugs.webkit.org/show_bug.cgi?id=43778
Dropping should fire textInput event.

  • editing/pasteboard/drop-text-events-expected.txt: Added.
  • editing/pasteboard/drop-text-events-sideeffect-expected.txt: Added.
  • editing/pasteboard/drop-text-events-sideeffect.html: Added.
  • editing/pasteboard/drop-text-events.html: Added.
  • editing/pasteboard/script-tests/drop-text-events-sideeffect.js: Added.
  • editing/pasteboard/script-tests/drop-text-events.js: Added.

2010-08-13 MORITA Hajime <​morrita@google.com>

Reviewed by Tony Chang.

​https://bugs.webkit.org/show_bug.cgi?id=43778
Dropping should fire textInput event

This change:

  • added TextEvent::m_isDrop to distinguish drop-initiated events.
  • added to dispatch TextEvent from DragController before making a side effect and cancel the side effect if the event prevents default

Tests: editing/pasteboard/drop-text-events-sideeffect.html

editing/pasteboard/drop-text-events.html

  • dom/TextEvent.cpp: (WebCore::TextEvent::createForDrop): Added. (WebCore::TextEvent::TextEvent):
  • dom/TextEvent.h: (WebCore::TextEvent::isDrop): Added.
  • editing/Editor.cpp: (WebCore::Editor::handleTextEvent): (WebCore::Editor::findEventTargetFrom): Extracted from findEventTargetFromSelection(). (WebCore::Editor::findEventTargetFromSelection):
  • editing/Editor.h:
  • page/DragController.cpp: (WebCore::DragController::dispatchTextInputEventFor): Added. (WebCore::DragController::concludeEditDrag): Added an event dispatch path.
  • page/DragController.h:

Aug 15, 2010:

11:38 PM Changeset in webkit [65394] by ariya@webkit.org
  • 2 edits in trunk/WebCore

2010-08-15 Ariya Hidayat <​ariya@sencha.com>

Unreviewed, rolling out r65393.
​http://trac.webkit.org/changeset/65393
​https://bugs.webkit.org/show_bug.cgi?id=44031

Breaks some canvas tests.

  • platform/graphics/qt/GraphicsContextQt.cpp: (WebCore::GraphicsContextPlatformPrivate::): (WebCore::GraphicsContextPlatformPrivate::hasShadow): (WebCore::GraphicsContextPlatformPrivate::GraphicsContextPlatformPrivate): (WebCore::GraphicsContext::savePlatformState): (WebCore::GraphicsContext::restorePlatformState): (WebCore::GraphicsContext::strokeArc): (WebCore::GraphicsContext::drawConvexPolygon): (WebCore::GraphicsContext::fillPath): (WebCore::GraphicsContext::strokePath): (WebCore::GraphicsContext::fillRect): (WebCore::GraphicsContext::fillRoundedRect): (WebCore::GraphicsContext::setPlatformShadow): (WebCore::GraphicsContext::clearPlatformShadow):
11:19 PM Changeset in webkit [65393] by ariya@webkit.org
  • 2 edits in trunk/WebCore

[Qt] Save and restore shadow state in GraphicsContextQt
​https://bugs.webkit.org/show_bug.cgi?id=44031

Patch by Ariya Hidayat <​ariya@sencha.com> on 2010-08-15
Reviewed by Antonio Gomes.

This is needed after r65362.
Use a new class ContextShadowParameter to track and keep the shadow
state. We use this rather than using GraphicsContextState to allow
possible optimizations (right now only to determine the shadow type,
but in future it might cover things like cached scratch image,
persistent shader, etc).

  • platform/graphics/qt/GraphicsContextQt.cpp:

(WebCore::ContextShadowParameter::):
(WebCore::ContextShadowParameter::ContextShadowParameter):
(WebCore::ContextShadowParameter::clear):
(WebCore::GraphicsContextPlatformPrivate::hasShadow):
(WebCore::GraphicsContextPlatformPrivate::GraphicsContextPlatformPrivate):
(WebCore::GraphicsContext::savePlatformState):
(WebCore::GraphicsContext::restorePlatformState):
(WebCore::GraphicsContext::drawRect):
(WebCore::GraphicsContext::drawLine):
(WebCore::GraphicsContext::strokeArc):
(WebCore::GraphicsContext::drawConvexPolygon):
(WebCore::GraphicsContext::fillPath):
(WebCore::GraphicsContext::strokePath):
(WebCore::GraphicsContext::fillRect):
(WebCore::GraphicsContext::fillRoundedRect):
(WebCore::GraphicsContext::setPlatformShadow):
(WebCore::GraphicsContext::clearPlatformShadow):

10:27 PM Changeset in webkit [65392] by ariya@webkit.org
  • 5 edits in trunk

2010-08-15 Ariya Hidayat <​ariya@sencha.com>

Reviewed by Antonio Gomes.

[Qt] Border should not cast shadows
​https://bugs.webkit.org/show_bug.cgi?id=44015

Update pixel test reference.

  • platform/qt/fast/box-shadow/basic-shadows-expected.checksum:
  • platform/qt/fast/box-shadow/basic-shadows-expected.png:

2010-08-15 Ariya Hidayat <​ariya@sencha.com>

Reviewed by Antonio Gomes.

[Qt] Border should not cast shadows
​https://bugs.webkit.org/show_bug.cgi?id=44015

Shadows should be not casted except for brush fill (which is not what
drawLine and drawRect are supposed to do).

  • platform/graphics/qt/GraphicsContextQt.cpp: (WebCore::GraphicsContext::drawRect): Remove shadow painting, (WebCore::GraphicsContext::drawLine): ditto.
8:28 PM Changeset in webkit [65391] by commit-queue@webkit.org
  • 4 edits in trunk

2010-08-15 Ryuan Choi <​ryuan.choi@samsung.com>

Reviewed by Antonio Gomes.

[EFL] Build error on r65378
​https://bugs.webkit.org/show_bug.cgi?id=44019

Add GENERATE_JSON macro for HTMLEntityNames.json instead of HTMLEntityNames.gperf

  • cmake/WebKitMacros.cmake:

2010-08-15 Ryuan Choi <​ryuan.choi@samsung.com>

Reviewed by Antonio Gomes.

[EFL] Build error on r65378
​https://bugs.webkit.org/show_bug.cgi?id=44019

Change build script for HTMLEntityNames.json instead of HTMLEntityNames.gperf

  • CMakeLists.txt:
6:58 PM Changeset in webkit [65390] by jhoneycutt@apple.com
  • 5 edits in trunk/WebKitTools

WebEditorClient::didBeginEditing is never called in WebKit2
​https://bugs.webkit.org/show_bug.cgi?id=42939

Reviewed by Sam Weinig.

  • WebKitTestRunner/PlatformWebView.h:

Declare focus().

  • WebKitTestRunner/TestController.cpp:

(WTR::TestController::resetStateToConsistentValues):
Focus the PlatformWebView.

  • WebKitTestRunner/mac/PlatformWebViewMac.mm:

(WTR::PlatformWebView::focus):
Stubbed.

  • WebKitTestRunner/win/PlatformWebViewWin.cpp:

(WTR::PlatformWebView::focus):
Focus the view.

6:57 PM Changeset in webkit [65389] by jhoneycutt@apple.com
  • 2 edits in trunk/WebKitTools

run-webkit-tests should not strip editing callbacks when using
WebKitTestRunner on Windows
​https://bugs.webkit.org/show_bug.cgi?id=44000

Reviewed by Mark Rowe.

  • Scripts/old-run-webkit-tests:

Leave $stripEditingCallbacks undefined until we look for command-line
arguments. If using WebKit2, set it to 0 if not explicity set on the
command line. Later, set it to isCygwin() to match old behavior if it is
not yet defined.

6:48 PM Changeset in webkit [65388] by abarth@webkit.org
  • 3 edits in trunk/WebKit/chromium

2010-08-15 Adam Barth <​abarth@webkit.org>

Reviewed by Eric Seidel.

Port Chromium's WebEntities to HTMLEntityTable
​https://bugs.webkit.org/show_bug.cgi?id=44036

This code is wrong and needs to be removed. However, at least after
this patch, it will compile.

  • src/WebEntities.cpp: (WebKit::): (WebKit::WebEntities::WebEntities):
  • src/WebEntities.h:
5:12 PM Changeset in webkit [65387] by kevino@webkit.org
  • 2 edits in trunk/WebCore

[wx] Build fix, add missing header.

5:05 PM Changeset in webkit [65386] by kevino@webkit.org
  • 2 edits in trunk/WebCore

[wx] Build fix, don't build PluginDataNone.cpp even on GTK as PluginDataWx.cpp compiles.

5:02 PM Changeset in webkit [65385] by kevino@webkit.org
  • 2 edits in trunk/JavaScriptCore

[wx] Build fix, don't build intermediate source in DerivedSources dir.

4:47 PM Changeset in webkit [65384] by kevino@webkit.org
  • 2 edits in trunk/WebKitTools

[wx] Build fix, remove define always set to the correct value by wtf/Platform.h.

1:15 PM Changeset in webkit [65383] by Joseph Pecoraro
  • 7 edits
    2 adds in trunk/WebKit

2010-08-14 Joseph Pecoraro <Joseph Pecoraro>

Reviewed by Pavel Feldman.

Web Inspector: -attach and detach should work
​https://bugs.webkit.org/show_bug.cgi?id=43924

  • WebKit.xcodeproj/project.pbxproj: Added WebInspectorFrontend.{h,mm}.

2010-08-14 Joseph Pecoraro <Joseph Pecoraro>

Reviewed by Pavel Feldman.

Web Inspector: -attach and detach should work
​https://bugs.webkit.org/show_bug.cgi?id=43924

For actions that relate only to the Web Inspector's Frontend
window (actions on the InspectorFrontendClient) the new Obj-C
wrapper class WebInspectorFrontend can be used. For now, this
is just attach and detach.

  • WebCoreSupport/WebInspectorClient.mm: (WebInspectorClient::openInspectorFrontend): setup and attach the WebInspectorFrontend to the WebInspector.
  • WebInspector/WebInspector.h:
  • WebInspector/WebInspector.mm: (-dealloc): (-attach:): delegate to the frontend. (-detach:): delegate to the frontend. (-setFrontend:):
  • WebInspector/WebInspectorFrontend.h: Added.
  • WebInspector/WebInspectorFrontend.mm: Added. (-[WebInspectorFrontend initWithFrontendClient:]): (-[WebInspectorFrontend attach]): (-[WebInspectorFrontend detach]):
  • WebInspector/WebInspectorPrivate.h:
12:43 PM Changeset in webkit [65382] by abarth@webkit.org
  • 4 edits
    4 adds in trunk

2010-08-15 Adam Barth <​abarth@webkit.org>

Reviewed by Eric Seidel.

document.write() doesn't flush plain text
​https://bugs.webkit.org/show_bug.cgi?id=8961

Add test cases for flushing character tokens.

  • fast/tokenizer/flush-characters-in-document-write-evil-expected.txt: Added.
  • fast/tokenizer/flush-characters-in-document-write-evil.html: Added.
  • fast/tokenizer/flush-characters-in-document-write-expected.txt: Added.
  • fast/tokenizer/flush-characters-in-document-write.html: Added.

2010-08-15 Adam Barth <​abarth@webkit.org>

Reviewed by Eric Seidel.

document.write() doesn't flush plain text
​https://bugs.webkit.org/show_bug.cgi?id=8961

Originally I thought we should buffer the character tokens until we've
reached the end of the input stream, but that's not how the spec
handles things (it emits the character tokens one-by-one).

Tests: fast/tokenizer/flush-characters-in-document-write-evil.html

fast/tokenizer/flush-characters-in-document-write.html

  • html/HTMLTokenizer.cpp: (WebCore::HTMLTokenizer::emitEndOfFile): (WebCore::HTMLTokenizer::nextToken): (WebCore::HTMLTokenizer::haveBufferedCharacterToken):
    • Renamed this function now that it's simplier.
  • html/HTMLTokenizer.h:
8:34 AM Changeset in webkit [65381] by abarth@webkit.org
  • 3 edits
    2 adds in trunk

2010-08-15 Adam Barth <​abarth@webkit.org>

Reviewed by Eric Seidel.

Don't try to replace a non-existent document after executing JavaScript URLs
​https://bugs.webkit.org/show_bug.cgi?id=44024

Test what happens if a JavaScript URL returns a value after deleting
the frame it was supposed to operate on.

  • fast/frames/javascript-url-for-deleted-frame-expected.txt: Added.
  • fast/frames/javascript-url-for-deleted-frame.html: Added.

2010-08-15 Adam Barth <​abarth@webkit.org>

Reviewed by Eric Seidel.

Don't try to replace a non-existent document after executing JavaScript URLs
​https://bugs.webkit.org/show_bug.cgi?id=44024

Synchronous JavaScript execution is evil. Previously, the frame was
deleted after executing the JavaScript URL, so we'd get confused when
we tried to replace its document.

Test: fast/frames/javascript-url-for-deleted-frame.html

  • bindings/ScriptControllerBase.cpp: (WebCore::ScriptController::executeIfJavaScriptURL):
7:10 AM Changeset in webkit [65380] by commit-queue@webkit.org
  • 2 edits in trunk/WebKit/efl

2010-08-15 Gyuyoung Kim <​gyuyoung.kim@samsung.com>

Reviewed by Antonio Gomes.

[EFL] Enable localStorage of HTML5 for WebKit EFL
​https://bugs.webkit.org/show_bug.cgi?id=44020

  • ewk/ewk_view.cpp: (_ewk_view_priv_new):
12:24 AM Changeset in webkit [65379] by mitz@apple.com
  • 5 edits in trunk/WebKit2

Revised build fix from r65363 based on suggestion from Darin Adler.

  • UIProcess/API/C/WKArray.h: Include stddef.h.
  • UIProcess/API/C/WKBase.h: Uninclude ctype.h.
  • UIProcess/API/C/WKData.h: Include stddef.h.
  • UIProcess/API/C/WKDictionary.h: Ditto.
Note: See TracTimeline for information about the timeline view.