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

Timeline



Mar 24, 2011:

11:56 PM Changeset in webkit [81944] by gyuyoung.kim@samsung.com
  • 5 edits
    4 adds in trunk/Source

2011-03-24 Gyuyoung Kim <gyuyoung.kim@samsung.com>

Reviewed by Antonio Gomes.

[EFL] Add sound / mute button to MediaControl UI
https://bugs.webkit.org/show_bug.cgi?id=56726

Add sound / mute button to MediaControl UI.

  • platform/efl/RenderThemeEfl.cpp: (WebCore::RenderThemeEfl::edjeGroupFromFormType): (WebCore::RenderThemeEfl::emitMediaButtonSignal): (WebCore::RenderThemeEfl::paintMediaMuteButton):
  • platform/efl/RenderThemeEfl.h:

2011-03-24 Gyuyoung Kim <gyuyoung.kim@samsung.com>

Reviewed by Antonio Gomes.

[EFL] Add sound / mute button to MediaControl UI
https://bugs.webkit.org/show_bug.cgi?id=56726

Add sound / mute button to MediaControl UI.

  • DefaultTheme/default.edc:
  • DefaultTheme/widget/mediacontrol/mutebutton/mute_button.edc: Added.
  • DefaultTheme/widget/mediacontrol/mutebutton/mutebutton.png: Added.
  • DefaultTheme/widget/mediacontrol/mutebutton/unmutebutton.png: Added.
11:35 PM Changeset in webkit [81943] by Nikolas Zimmermann
  • 18 edits
    2 adds in trunk/Source

2011-03-24 Nikolas Zimmermann <nzimmermann@rim.com>

Reviewed by Darin Adler.

Introduce WTF HexNumber.h
https://bugs.webkit.org/show_bug.cgi?id=56099

Introduce a set of functions that ease converting from a bye or a number to a hex string,
replacing several of these conversions and String::format("%x") usages all over WebCore.

  • GNUmakefile.am: Add HexNumber.h to build.
  • JavaScriptCore.exp: Export StringBuilder::reserveCapacity.
  • JavaScriptCore.gypi: Add HexNumber.h to build.
  • JavaScriptCore.vcproj/JavaScriptCore/JavaScriptCore.def: Export StringBuilder::reserveCapacity.
  • JavaScriptCore.vcproj/WTF/WTF.vcproj: Add HexNumber.h to build.
  • JavaScriptCore.xcodeproj/project.pbxproj: Ditto.
  • wtf/CMakeLists.txt: Ditto.
  • wtf/HexNumber.h: Added. (WTF::Internal::hexDigitsForMode): Internal helper. (WTF::appendByteAsHex): Free function, that appends a byte as hex string into a destination. (WTF::placeByteAsHex): Ditto, but places the result using *foo++ = '..' or foo[index++] = '..' (WTF::appendUnsignedAsHex): Free function, that appends a number as hex string into a destination.

2011-03-24 Nikolas Zimmermann <nzimmermann@rim.com>

Reviewed by Darin Adler.

Introduce WTF HexNumber.h
https://bugs.webkit.org/show_bug.cgi?id=56099

Introduce a set of functions that ease converting from a bye or a number to a hex string,
replacing several of these conversions and String::format("%x") usages all over WebCore.

  • ForwardingHeaders/wtf/HexNumber.h: Added.
  • css/CSSOMUtils.cpp: (WebCore::serializeCharacterAsCodePoint):
  • css/CSSParser.cpp: (WebCore::quoteCSSString):
  • inspector/InspectorResourceAgent.cpp: (WebCore::createReadableStringFromBinary):
  • platform/FileSystem.cpp: (WebCore::encodeForFileName):
  • platform/KURL.cpp: (WebCore::appendEscapedChar):
  • platform/UUID.cpp: (WebCore::createCanonicalUUIDString):
  • platform/graphics/Color.cpp: (WebCore::Color::serialized):
  • platform/network/FormDataBuilder.cpp: (WebCore::FormDataBuilder::encodeStringAsFormData):
  • rendering/RenderTreeAsText.cpp: (WebCore::quoteAndEscapeNonPrintables):
11:31 PM Changeset in webkit [81942] by yutak@chromium.org
  • 1 edit
    4 adds in trunk/LayoutTests

2011-03-24 Yuta Kitamura <yutak@chromium.org>

Unreviewed, rebaseline Chromium test results for Mac Leopard.

  • platform/chromium-mac-leopard/fast/blockflow/vertical-baseline-alignment-expected.checksum: Added.
  • platform/chromium-mac-leopard/fast/blockflow/vertical-baseline-alignment-expected.png: Added.
  • platform/chromium-mac-leopard/fast/dynamic/text-combine-expected.checksum: Added.
  • platform/chromium-mac-leopard/fast/dynamic/text-combine-expected.png: Added.
11:27 PM Changeset in webkit [81941] by jer.noble@apple.com
  • 2 edits in trunk/Source/WebKit/mac

2011-03-24 Sheriff Bot <webkit.review.bot@gmail.com>

Unreviewed, rolling out r81939.
http://trac.webkit.org/changeset/81939
https://bugs.webkit.org/show_bug.cgi?id=57084

Rolling out 81939, as it's causing a number of app cache test
failures. (Requested by jernoble on #webkit).

  • WebView/WebView.mm: (-[WebView _preferencesChangedNotification:]):
11:05 PM Changeset in webkit [81940] by kevino@webkit.org
  • 4 edits in trunk

[wx] A couple build fixes after recent changes.

10:33 PM Changeset in webkit [81939] by jer.noble@apple.com
  • 2 edits in trunk/Source/WebKit/mac

2011-03-23 Jer Noble <jer.noble@apple.com>

Reviewed by Maciej Stachowiak.

Application Cache should save audio/ and video/ mime types as flat files
https://bugs.webkit.org/show_bug.cgi?id=53784
<rdar://problem/8932473>

No new tests.

Initialize the AppCache Total size quotas with the values for WebKitPreferences
upon startup.

  • WebView/WebView.mm: (-[WebView _preferencesChangedNotification:]):
10:06 PM Changeset in webkit [81938] by commit-queue@webkit.org
  • 3 edits
    4 deletes in trunk

2011-03-24 Ilya Sherman <isherman@chromium.org>

Reviewed by Adam Roben.

Implement layoutTestController.setAutoFilled in DRT on Windows
https://bugs.webkit.org/show_bug.cgi?id=56828

  • platform/chromium/fast/forms/input-autofilled-expected.txt: Removed.
  • platform/chromium/fast/forms/reset-autofilled-expected.txt: Removed.
  • platform/win/fast/forms/input-autofilled-expected.txt: Removed.
  • platform/win/fast/forms/reset-autofilled-expected.txt: Removed.

2011-03-24 Ilya Sherman <isherman@chromium.org>

Reviewed by Adam Roben.

Implement layoutTestController.setAutoFilled in DRT on Windows
https://bugs.webkit.org/show_bug.cgi?id=56828

  • DumpRenderTree/win/LayoutTestControllerWin.cpp: (LayoutTestController::setAutofilled): Implemented.
9:10 PM Changeset in webkit [81937] by Stephanie Lewis
  • 2 edits in trunk/Source/WebCore

https://bugs.webkit.org/show_bug.cgi?id=57073
Rework the AtomicHTMLConstructor to reserve space for attributes based on the size of the attribute list.
Saves 1.5 MB on Membuster.

Reviewed by Geoff Garen.

No new tests because there was no change in behavior. Current tests pass.

  • html/parser/HTMLToken.h:

(WebCore::AtomicHTMLToken::AtomicHTMLToken):
(WebCore::AtomicHTMLToken::initializeAttributes):

8:58 PM Changeset in webkit [81936] by Lucas Forschler
  • 8 edits in branches/safari-534.26-branch

Versioning and Merge 81681, 34859, 81683, 34860, 81718.

8:50 PM Changeset in webkit [81935] by Lucas Forschler
  • 1 copy in tags/Safari-534.26.2

New Tag.

8:23 PM Changeset in webkit [81934] by yutak@chromium.org
  • 4 edits in trunk/LayoutTests

2011-03-24 Yuta Kitamura <yutak@chromium.org>

Unreviewed. Rebaseline Chromium test results for fast/css/percentage-non-integer.html.

  • platform/chromium-win/fast/css/percentage-non-integer-expected.checksum:
  • platform/chromium-win/fast/css/percentage-non-integer-expected.png:
  • platform/chromium-win/fast/css/percentage-non-integer-expected.txt:
7:54 PM Changeset in webkit [81933] by commit-queue@webkit.org
  • 45 edits
    4 deletes in trunk

2011-03-24 Sheriff Bot <webkit.review.bot@gmail.com>

Unreviewed, rolling out r81916 and r81917.
http://trac.webkit.org/changeset/81916
http://trac.webkit.org/changeset/81917
https://bugs.webkit.org/show_bug.cgi?id=57071

broke a test on platforms that do not have QuickTime installed
(Requested by estes on #webkit).

  • fast/images/embed-image-plugins-disabled.html: Removed.
  • platform/mac/fast/images/embed-image-expected.checksum:
  • platform/mac/fast/images/embed-image-expected.png:
  • platform/mac/fast/images/embed-image-expected.txt:
  • platform/mac/fast/images/embed-image-plugins-disabled-expected.checksum: Removed.
  • platform/mac/fast/images/embed-image-plugins-disabled-expected.png: Removed.
  • platform/mac/fast/images/embed-image-plugins-disabled-expected.txt: Removed.

2011-03-24 Sheriff Bot <webkit.review.bot@gmail.com>

Unreviewed, rolling out r81916 and r81917.
http://trac.webkit.org/changeset/81916
http://trac.webkit.org/changeset/81917
https://bugs.webkit.org/show_bug.cgi?id=57071

broke a test on platforms that do not have QuickTime installed
(Requested by estes on #webkit).

  • html/HTMLEmbedElement.cpp: (WebCore::HTMLEmbedElement::HTMLEmbedElement):
  • html/HTMLObjectElement.cpp: (WebCore::HTMLObjectElement::HTMLObjectElement): (WebCore::HTMLObjectElement::parametersForPlugin):
  • html/HTMLPlugInImageElement.cpp: (WebCore::HTMLPlugInImageElement::HTMLPlugInImageElement): (WebCore::HTMLPlugInImageElement::isImageType): (WebCore::HTMLPlugInImageElement::wouldLoadAsNetscapePlugin):
  • html/HTMLPlugInImageElement.h:
  • loader/EmptyClients.h: (WebCore::EmptyFrameLoaderClient::objectContentType):
  • loader/FrameLoader.cpp: (WebCore::FrameLoader::defaultObjectContentType):
  • loader/FrameLoader.h:
  • loader/FrameLoaderClient.h:
  • loader/SubframeLoader.cpp: (WebCore::SubframeLoader::resourceWillUsePlugin): (WebCore::SubframeLoader::requestObject): (WebCore::SubframeLoader::shouldUsePlugin):
  • loader/SubframeLoader.h:

2011-03-24 Sheriff Bot <webkit.review.bot@gmail.com>

Unreviewed, rolling out r81916 and r81917.
http://trac.webkit.org/changeset/81916
http://trac.webkit.org/changeset/81917
https://bugs.webkit.org/show_bug.cgi?id=57071

broke a test on platforms that do not have QuickTime installed
(Requested by estes on #webkit).

  • src/FrameLoaderClientImpl.cpp: (WebKit::FrameLoaderClientImpl::objectContentType):
  • src/FrameLoaderClientImpl.h:

2011-03-24 Sheriff Bot <webkit.review.bot@gmail.com>

Unreviewed, rolling out r81916 and r81917.
http://trac.webkit.org/changeset/81916
http://trac.webkit.org/changeset/81917
https://bugs.webkit.org/show_bug.cgi?id=57071

broke a test on platforms that do not have QuickTime installed
(Requested by estes on #webkit).

  • WebCoreSupport/FrameLoaderClientEfl.cpp: (WebCore::FrameLoaderClientEfl::objectContentType):
  • WebCoreSupport/FrameLoaderClientEfl.h:

2011-03-24 Sheriff Bot <webkit.review.bot@gmail.com>

Unreviewed, rolling out r81916 and r81917.
http://trac.webkit.org/changeset/81916
http://trac.webkit.org/changeset/81917
https://bugs.webkit.org/show_bug.cgi?id=57071

broke a test on platforms that do not have QuickTime installed
(Requested by estes on #webkit).

  • WebCoreSupport/FrameLoaderClientGtk.cpp: (WebKit::FrameLoaderClient::objectContentType):
  • WebCoreSupport/FrameLoaderClientGtk.h:

2011-03-24 Sheriff Bot <webkit.review.bot@gmail.com>

Unreviewed, rolling out r81916 and r81917.
http://trac.webkit.org/changeset/81916
http://trac.webkit.org/changeset/81917
https://bugs.webkit.org/show_bug.cgi?id=57071

broke a test on platforms that do not have QuickTime installed
(Requested by estes on #webkit).

  • WebCoreSupport/FrameLoaderClientHaiku.cpp: (WebCore::FrameLoaderClientHaiku::objectContentType):
  • WebCoreSupport/FrameLoaderClientHaiku.h:

2011-03-24 Sheriff Bot <webkit.review.bot@gmail.com>

Unreviewed, rolling out r81916 and r81917.
http://trac.webkit.org/changeset/81916
http://trac.webkit.org/changeset/81917
https://bugs.webkit.org/show_bug.cgi?id=57071

broke a test on platforms that do not have QuickTime installed
(Requested by estes on #webkit).

  • WebCoreSupport/WebFrameLoaderClient.h:
  • WebCoreSupport/WebFrameLoaderClient.mm: (WebFrameLoaderClient::objectContentType):

2011-03-24 Sheriff Bot <webkit.review.bot@gmail.com>

Unreviewed, rolling out r81916 and r81917.
http://trac.webkit.org/changeset/81916
http://trac.webkit.org/changeset/81917
https://bugs.webkit.org/show_bug.cgi?id=57071

broke a test on platforms that do not have QuickTime installed
(Requested by estes on #webkit).

  • WebCoreSupport/FrameLoaderClientQt.cpp: (WebCore::FrameLoaderClientQt::objectContentType):
  • WebCoreSupport/FrameLoaderClientQt.h:

2011-03-24 Sheriff Bot <webkit.review.bot@gmail.com>

Unreviewed, rolling out r81916 and r81917.
http://trac.webkit.org/changeset/81916
http://trac.webkit.org/changeset/81917
https://bugs.webkit.org/show_bug.cgi?id=57071

broke a test on platforms that do not have QuickTime installed
(Requested by estes on #webkit).

  • WebFrame.cpp: (WebFrame::objectContentType):
  • WebFrame.h:

2011-03-24 Sheriff Bot <webkit.review.bot@gmail.com>

Unreviewed, rolling out r81916 and r81917.
http://trac.webkit.org/changeset/81916
http://trac.webkit.org/changeset/81917
https://bugs.webkit.org/show_bug.cgi?id=57071

broke a test on platforms that do not have QuickTime installed
(Requested by estes on #webkit).

  • WebCoreSupport/FrameLoaderClientWinCE.cpp: (WebKit::FrameLoaderClientWinCE::objectContentType):
  • WebCoreSupport/FrameLoaderClientWinCE.h:

2011-03-24 Sheriff Bot <webkit.review.bot@gmail.com>

Unreviewed, rolling out r81916 and r81917.
http://trac.webkit.org/changeset/81916
http://trac.webkit.org/changeset/81917
https://bugs.webkit.org/show_bug.cgi?id=57071

broke a test on platforms that do not have QuickTime installed
(Requested by estes on #webkit).

  • WebKitSupport/FrameLoaderClientWx.cpp: (WebCore::FrameLoaderClientWx::objectContentType):
  • WebKitSupport/FrameLoaderClientWx.h:

2011-03-24 Sheriff Bot <webkit.review.bot@gmail.com>

Unreviewed, rolling out r81916 and r81917.
http://trac.webkit.org/changeset/81916
http://trac.webkit.org/changeset/81917
https://bugs.webkit.org/show_bug.cgi?id=57071

broke a test on platforms that do not have QuickTime installed
(Requested by estes on #webkit).

  • WebProcess/WebCoreSupport/WebFrameLoaderClient.cpp: (WebKit::WebFrameLoaderClient::objectContentType):
  • WebProcess/WebCoreSupport/WebFrameLoaderClient.h:
7:34 PM Changeset in webkit [81932] by Dimitri Glazkov
  • 6 edits in trunk/Source/WebCore

2011-03-24 Dimitri Glazkov <Dimitri Glazkov>

Reviewed by Darin Adler.

Eliminate Node::dispatchGenericEvent.
https://bugs.webkit.org/show_bug.cgi?id=57045

No functional change, covered by existing tests.

  • dom/Node.cpp: (WebCore::Node::dispatchEvent): Combined with the body of dispatchGenericEvent,

removed unnecessary extra refs and a stale comment.

  • dom/Node.h:
  • page/DOMWindow.cpp: (WebCore::DOMWindow::dispatchLoadEvent): Changed to use dispatchEvent.
  • svg/SVGElement.cpp: (WebCore::SVGElement::sendSVGLoadEventIfPossible): Ditto.
  • svg/SVGElementInstance.cpp: (WebCore::SVGElementInstance::dispatchEvent): Ditto. The code here still

works thanks to SVG-aware retargeting in Node.

7:13 PM Changeset in webkit [81931] by crogers@google.com
  • 7 edits in trunk/Source/WebCore

2011-03-24 Chris Rogers <crogers@google.com>

Reviewed by Kenneth Russell.

web audio: Properly sample-rate convert audio assets in chromium port
https://bugs.webkit.org/show_bug.cgi?id=56980

No new tests since audio API is not yet implemented.

  • WebCore.gypi:
  • platform/audio/AudioBus.cpp: (WebCore::AudioBus::createBySampleRateConverting): (WebCore::AudioBus::createByMixingToMono):
  • platform/audio/AudioBus.h:
  • platform/audio/HRTFKernel.cpp: (WebCore::extractAverageGroupDelay): (WebCore::HRTFKernel::HRTFKernel):
  • platform/audio/SincResampler.cpp:
  • platform/audio/chromium/AudioBusChromium.cpp: (WebCore::AudioBus::loadPlatformResource): (WebCore::createBusFromInMemoryAudioFile):
7:09 PM Changeset in webkit [81930] by commit-queue@webkit.org
  • 4 edits in trunk/Source/WebKit/chromium

2011-03-24 Min Qin <qinmin@google.com>

Reviewed by James Robinson.

Allow WebMediaPlayerClient to notify MediaPlayer that the playback state has changed

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

  • public/WebMediaPlayerClient.h:
  • src/WebMediaPlayerClientImpl.cpp: (WebKit::WebMediaPlayerClientImpl::playbackStateChanged):
  • src/WebMediaPlayerClientImpl.h:
6:57 PM Changeset in webkit [81929] by scheib@chromium.org
  • 6 edits in trunk/LayoutTests

2011-03-24 Vincent Scheib <scheib@chromium.org>

Update Chromium baselines for 81887

  • platform/chromium-win/editing/pasteboard/5065605-expected.txt:
  • platform/chromium-win/editing/pasteboard/display-block-on-spans-expected.txt:
  • platform/chromium-win/editing/pasteboard/paste-text-011-expected.txt:
  • platform/chromium-win/editing/pasteboard/paste-text-at-tabspan-001-expected.txt:
  • platform/chromium-win/editing/pasteboard/paste-text-at-tabspan-002-expected.txt:
6:34 PM Changeset in webkit [81928] by beidson@apple.com
  • 9 edits
    2 adds in trunk/Source/WebKit2

https://bugs.webkit.org/show_bug.cgi?id=57069
Add WKIconDatabase api to the project files, and expose accessor on WKContext.

Reviewed by Darin Adler.

Project file stuffs:

  • GNUmakefile.am:
  • WebKit2.xcodeproj/project.pbxproj:
  • win/WebKit2.vcproj:
  • Shared/API/c/WKBase.h:
  • UIProcess/API/C/WKAPICast.h:
  • UIProcess/API/C/WKContext.cpp:

(WKContextGetIconDatabase):

  • UIProcess/API/C/WKContext.h:
  • UIProcess/API/C/WKIconDatabase.cpp: Added.

(WKIconDatabaseGetTypeID):

  • UIProcess/API/C/WKIconDatabase.h: Added.
  • UIProcess/WebContext.h:

(WebKit::WebContext::iconDatabase):

6:29 PM Changeset in webkit [81927] by jam@chromium.org
  • 2 edits in trunk/Source/WebKit/chromium

2011-03-23 John Abd-El-Malek <jam@chromium.org>

Reviewed by Tony Chang.

[chromium] Get rid of WEB_TEXT_CHECKING_RESULT_IS_A_STRUCT since it&apos;s not needed anymore
https://bugs.webkit.org/show_bug.cgi?id=56955

  • DEPS:
  • public/WebTextCheckingResult.h:
6:10 PM Changeset in webkit [81926] by dimich@chromium.org
  • 2 edits in trunk/Source/WebKit/chromium

2011-03-24 Bill Budge <bbudge@chromium.org>

Reviewed by Eric Seidel.

Change a forward declaration of WebURLLoaderOptions to struct to prevent a build break in Chromium.
https://bugs.webkit.org/show_bug.cgi?id=57064

No tests. Exposes no new functionality.

  • public/WebFrame.h:
6:09 PM Changeset in webkit [81925] by commit-queue@webkit.org
  • 6 edits in trunk

2011-03-24 Rik Cabanier <cabanier@adobe.com>

Reviewed by David Hyatt.

REGRESSION (r81625): fast/css/percentage-non-integer.html fails on Windows
https://bugs.webkit.org/show_bug.cgi?id=56829

  • platform/win/fast/css/percentage-non-integer-expected.checksum:
  • platform/win/fast/css/percentage-non-integer-expected.png:
  • platform/win/fast/css/percentage-non-integer-expected.txt:

2011-03-24 Rik Cabanier <cabanier@adobe.com>

Reviewed by David Hyatt.

REGRESSION (r81625): fast/css/percentage-non-integer.html fails on Windows
https://bugs.webkit.org/show_bug.cgi?id=56829

Test: fast/css/percentage-non-integer.html

  • platform/Length.h: (WebCore::Length::calcValue): (WebCore::Length::calcMinValue): (WebCore::Length::calcFloatValue):
6:08 PM Changeset in webkit [81924] by bfulgham@webkit.org
  • 2 edits in trunk/Source/WebKit2

Build correction.

Disable the CA Layer stuff when building with Cairo.

  • win/WebKit2.vcproj:
5:22 PM Changeset in webkit [81923] by mihaip@chromium.org
  • 3 edits
    2 copies in trunk/LayoutTests

2011-03-24 Mihai Parparita <mihaip@chromium.org>

Add Chromium Snow Leopard baselines for fast/css/bidi-override-in-anonymous-block.html.

  • platform/chromium-mac-leopard/fast/css/bidi-override-in-anonymous-block-expected.checksum: Copied from LayoutTests/platform/chromium-mac/fast/css/bidi-override-in-anonymous-block-expected.checksum.
  • platform/chromium-mac-leopard/fast/css/bidi-override-in-anonymous-block-expected.png: Copied from LayoutTests/platform/chromium-mac/fast/css/bidi-override-in-anonymous-block-expected.png.
  • platform/chromium-mac/fast/css/bidi-override-in-anonymous-block-expected.checksum:
  • platform/chromium-mac/fast/css/bidi-override-in-anonymous-block-expected.png:
5:18 PM Changeset in webkit [81922] by bfulgham@webkit.org
  • 3 edits in trunk/Tools

Unreviewed build correction.

Correct vsprops file for Debug variant of the CFLite build.
Disable a CFNETWORK-specific test when building for CFLite.

  • TestWebKitAPI/Configurations/TestWebKitAPIDebugCairoCFLite.vsprops:
  • TestWebKitAPI/win/TestWebKitAPI.vcproj:
5:17 PM Changeset in webkit [81921] by mihaip@chromium.org
  • 63 edits
    1 copy
    7 adds
    4 deletes in trunk

2011-03-24 Mihai Parparita <mihaip@chromium.org>

Reviewed by James Robinson.

[Chromium] Vertical Japanese text is not displayed on Snow Leopard
https://bugs.webkit.org/show_bug.cgi?id=56962

Pixel rebaselines after r80582 (vertical text was shifted by one pixel
vertical).

Make Snow Leopard check added by r80740 into a runtime check for
Chromium, since it uses the same binary on both Leopard and Snow Leopard.

  • platform/chromium-mac/fast/blockflow/Kusa-Makura-background-canvas-expected.checksum:
  • platform/chromium-mac/fast/blockflow/Kusa-Makura-background-canvas-expected.png:
  • platform/chromium-mac/fast/blockflow/border-vertical-lr-expected.checksum:
  • platform/chromium-mac/fast/blockflow/border-vertical-lr-expected.png:
  • platform/chromium-mac/fast/blockflow/broken-ideograph-small-caps-expected.checksum: Added.
  • platform/chromium-mac/fast/blockflow/broken-ideograph-small-caps-expected.png: Added.
  • platform/chromium-mac/fast/blockflow/broken-ideograph-small-caps-expected.txt: Added.
  • platform/chromium-mac/fast/blockflow/japanese-lr-selection-expected.checksum: Added.
  • platform/chromium-mac/fast/blockflow/japanese-lr-selection-expected.png: Added.
  • platform/chromium-mac/fast/blockflow/japanese-lr-text-expected.checksum:
  • platform/chromium-mac/fast/blockflow/japanese-lr-text-expected.png:
  • platform/chromium-mac/fast/blockflow/japanese-rl-selection-expected.checksum:
  • platform/chromium-mac/fast/blockflow/japanese-rl-selection-expected.png:
  • platform/chromium-mac/fast/blockflow/japanese-rl-text-expected.checksum:
  • platform/chromium-mac/fast/blockflow/japanese-rl-text-expected.png:
  • platform/chromium-mac/fast/blockflow/japanese-rl-text-with-broken-font-expected.checksum: Removed.
  • platform/chromium-mac/fast/blockflow/japanese-rl-text-with-broken-font-expected.png: Removed.
  • platform/chromium-mac/fast/blockflow/vertical-align-table-baseline-expected.checksum: Removed.
  • platform/chromium-mac/fast/blockflow/vertical-align-table-baseline-expected.png: Removed.
  • platform/chromium-mac/fast/blockflow/vertical-baseline-alignment-expected.checksum: Added.
  • platform/chromium-mac/fast/blockflow/vertical-baseline-alignment-expected.png: Added.
  • platform/chromium-mac/fast/dynamic/text-combine-expected.checksum:
  • platform/chromium-mac/fast/dynamic/text-combine-expected.png:
  • platform/chromium-mac/fast/repaint/japanese-rl-selection-clear-expected.checksum:
  • platform/chromium-mac/fast/repaint/japanese-rl-selection-clear-expected.png:
  • platform/chromium-mac/fast/repaint/japanese-rl-selection-repaint-expected.checksum:
  • platform/chromium-mac/fast/repaint/japanese-rl-selection-repaint-expected.png:
  • platform/chromium-mac/fast/repaint/repaint-across-writing-mode-boundary-expected.checksum:
  • platform/chromium-mac/fast/repaint/repaint-across-writing-mode-boundary-expected.png:
  • platform/chromium-mac/fast/ruby/base-shorter-than-text-expected.checksum:
  • platform/chromium-mac/fast/ruby/base-shorter-than-text-expected.png:
  • platform/chromium-mac/fast/text/justify-ideograph-vertical-expected.checksum:
  • platform/chromium-mac/fast/text/justify-ideograph-vertical-expected.png:
  • platform/chromium/test_expectations.txt:
  • platform/mac/fast/blockflow/Kusa-Makura-background-canvas-expected.checksum:
  • platform/mac/fast/blockflow/Kusa-Makura-background-canvas-expected.png:
  • platform/mac/fast/blockflow/border-vertical-lr-expected.checksum:
  • platform/mac/fast/blockflow/border-vertical-lr-expected.png:
  • platform/mac/fast/blockflow/broken-ideographic-font-expected.checksum:
  • platform/mac/fast/blockflow/broken-ideographic-font-expected.png:
  • platform/mac/fast/blockflow/japanese-lr-selection-expected.checksum:
  • platform/mac/fast/blockflow/japanese-lr-selection-expected.png:
  • platform/mac/fast/blockflow/japanese-lr-text-expected.checksum:
  • platform/mac/fast/blockflow/japanese-lr-text-expected.png:
  • platform/mac/fast/blockflow/japanese-rl-selection-expected.checksum:
  • platform/mac/fast/blockflow/japanese-rl-selection-expected.png:
  • platform/mac/fast/blockflow/japanese-rl-text-expected.checksum:
  • platform/mac/fast/blockflow/japanese-rl-text-expected.png:
  • platform/mac/fast/blockflow/japanese-rl-text-with-broken-font-expected.checksum:
  • platform/mac/fast/blockflow/japanese-rl-text-with-broken-font-expected.png:
  • platform/mac/fast/blockflow/japanese-ruby-vertical-lr-expected.checksum:
  • platform/mac/fast/blockflow/japanese-ruby-vertical-lr-expected.png:
  • platform/mac/fast/blockflow/japanese-ruby-vertical-rl-expected.checksum:
  • platform/mac/fast/blockflow/japanese-ruby-vertical-rl-expected.png:
  • platform/mac/fast/blockflow/vertical-align-table-baseline-expected.checksum:
  • platform/mac/fast/blockflow/vertical-align-table-baseline-expected.png:
  • platform/mac/fast/blockflow/vertical-baseline-alignment-expected.checksum:
  • platform/mac/fast/blockflow/vertical-baseline-alignment-expected.png:
  • platform/mac/fast/blockflow/vertical-font-fallback-expected.checksum:
  • platform/mac/fast/blockflow/vertical-font-fallback-expected.png:
  • platform/mac/fast/repaint/japanese-rl-selection-clear-expected.checksum:
  • platform/mac/fast/repaint/japanese-rl-selection-clear-expected.png:
  • platform/mac/fast/repaint/japanese-rl-selection-repaint-expected.checksum:
  • platform/mac/fast/repaint/japanese-rl-selection-repaint-expected.png:
  • platform/mac/fast/repaint/repaint-across-writing-mode-boundary-expected.checksum:
  • platform/mac/fast/repaint/repaint-across-writing-mode-boundary-expected.png:
  • platform/mac/fast/ruby/base-shorter-than-text-expected.checksum:
  • platform/mac/fast/ruby/base-shorter-than-text-expected.png:
  • platform/mac/fast/text/justify-ideograph-vertical-expected.checksum:
  • platform/mac/fast/text/justify-ideograph-vertical-expected.png:

2011-03-24 Mihai Parparita <mihaip@chromium.org>

Reviewed by James Robinson.

[Chromium] Vertical Japanese text is not displayed on Snow Leopard
https://bugs.webkit.org/show_bug.cgi?id=56962

Make Snow Leopard check added by r80740 into a runtime check for
Chromium, since it uses the same binary on both Leopard and Snow Leopard.

  • platform/graphics/mac/FontMac.mm: (WebCore::hasBrokenCTFontGetVerticalTranslationsForGlyphs): (WebCore::showGlyphsWithAdvances):
4:58 PM Changeset in webkit [81920] by beidson@apple.com
  • 11 edits in trunk/Source

https://bugs.webkit.org/show_bug.cgi?id=57058
Hookup the UIProcess WebIconDatabase to a WebCore::IconDatabase as its implementation

Reviewed by Sam Weinig.

../WebCore:

  • loader/icon/IconDatabase.cpp:

(WebCore::IconDatabase::synchronousLoadDecisionForIconURL): Only add the DocumentLoader to the set if it's non-zero.
(WebCore::IconDatabase::IconDatabase): Add more logging.
(WebCore::IconDatabase::performURLImport): Dispatch the "didFinishURLImport" client callback using the following 3 methods.
(WebCore::FinishedURLImport::FinishedURLImport):
(WebCore::FinishedURLImport::performWork):
(WebCore::IconDatabase::dispatchDidFinishURLImportOnMainThread):

  • loader/icon/IconDatabase.h:

(WebCore::IconDatabase::create): Expose a PassOwnPtr<> creator.

  • WebCore.exp.in:

../WebKit2:

  • UIProcess/WebIconDatabase.cpp:

(WebKit::WebIconDatabase::WebIconDatabase):
(WebKit::WebIconDatabase::setDatabasePath): Create a new WebCore::IconDatabase and open it to this path.
(WebKit::WebIconDatabase::retainIconForPageURL): Actually retain the url, using the impl.
(WebKit::WebIconDatabase::releaseIconForPageURL): Actually release the url, using the impl.
(WebKit::WebIconDatabase::setIconURLForPageURL): Actually set the url, using the impl.
(WebKit::WebIconDatabase::setIconDataForIconURL): Actually set the data, using the impl.
(WebKit::WebIconDatabase::getLoadDecisionForIconURL): Actually ask for a load decision to call back with.
(WebKit::WebIconDatabase::performImport):
(WebKit::WebIconDatabase::didImportIconURLForPageURL):
(WebKit::WebIconDatabase::didImportIconDataForPageURL):
(WebKit::WebIconDatabase::didChangeIconForPageURL):
(WebKit::WebIconDatabase::didRemoveAllIcons):
(WebKit::WebIconDatabase::didFinishURLImport): Notify all the pending callbacks what their load decisions

are now that they're available.

  • UIProcess/WebIconDatabase.h:

Add an IconDatabase logging channel:

  • Platform/Logging.cpp:

(WebKit::initializeLogChannelsIfNecessary):

  • Platform/Logging.h:
  • UIProcess/WebContext.cpp:

(WebKit::WebContext::setIconDatabasePath): Pass the new path on to the WebIconDatabase, possibly

opening it.

  • UIProcess/WebContext.h:
4:57 PM Changeset in webkit [81919] by jer.noble@apple.com
  • 5 edits in trunk/Source/WebCore

2011-03-23 Jer Noble <jer.noble@apple.com>

Reviewed by Maciej Stachowiak.

Application Cache should save audio/ and video/ mime types as flat files
https://bugs.webkit.org/show_bug.cgi?id=53784
<rdar://problem/8932473>

No new tests.

ApplicationCacheResource::create() now takes an additional "path" parameter. To facilitate
extracting this path information, two functions in ApplicationCacheHost have been made public.

  • loader/appcache/ApplicationCacheGroup.cpp: (WebCore::ApplicationCacheGroup::didReceiveResponse): Pass along new "path" parameter. (WebCore::ApplicationCacheGroup::didFail): Ditto.
  • loader/appcache/ApplicationCacheHost.h: (WebCore::ApplicationCacheHost::shouldLoadResourceFromApplicationCache): Made public. (WebCore::ApplicationCacheHost::getApplicationCacheFallbackResource): Made public.
  • loader/appcache/ApplicationCacheResource.cpp: (WebCore::ApplicationCacheResource::ApplicationCacheResource): Add new "path" parameter.
  • loader/appcache/ApplicationCacheResource.h: (WebCore::ApplicationCacheResource::create): Ditto. (WebCore::ApplicationCacheResource::path): New accessor.
4:44 PM Changeset in webkit [81918] by dpranke@chromium.org
  • 5 edits in trunk/Tools

2011-03-24 Dirk Pranke <dpranke@chromium.org>

Reviewed by Adam Barth.

fix regressions introduced in 81908 - there were a couple of
code paths that only fired under python 2.5 that I missed.

Also change a couple of 'python' references to sys.executable
in order to handle runing test-webkitpy with a binary other than
something called 'python'.

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

  • Scripts/webkitpy/layout_tests/port/mac.py:
  • Scripts/webkitpy/layout_tests/port/port_testcase.py:
  • Scripts/webkitpy/test/cat.py:
  • Scripts/webkitpy/test/echo.py:
4:37 PM Changeset in webkit [81917] by aestes@apple.com
  • 2 edits in trunk/Source/WebKit/chromium

Fix the chromium build.

  • src/FrameLoaderClientImpl.cpp:

(WebKit::FrameLoaderClientImpl::objectContentType):

4:34 PM Changeset in webkit [81916] by aestes@apple.com
  • 45 edits
    4 adds in trunk

2011-03-24 Andy Estes <aestes@apple.com>

Reviewed by Darin Adler.

REGRESSION (r70748): latest nightly builds kills AC_QuickTime.js
https://bugs.webkit.org/show_bug.cgi?id=49016

  • fast/images/embed-image-plugins-disabled.html: Added.
  • platform/mac/fast/images/embed-image-expected.checksum:
  • platform/mac/fast/images/embed-image-expected.png:
  • platform/mac/fast/images/embed-image-expected.txt:
  • platform/mac/fast/images/embed-image-plugins-disabled-expected.checksum: Added.
  • platform/mac/fast/images/embed-image-plugins-disabled-expected.png: Added.
  • platform/mac/fast/images/embed-image-plugins-disabled-expected.txt: Added.

2011-03-24 Andy Estes <aestes@apple.com>

Reviewed by Darin Adler.

REGRESSION (r70748): latest nightly builds kills AC_QuickTime.js
https://bugs.webkit.org/show_bug.cgi?id=49016

Update objectContentType() implementation to handle the
shouldPreferPlugInsForImages flag.

  • src/FrameLoaderClientImpl.cpp: (WebKit::FrameLoaderClientImpl::objectContentType):
  • src/FrameLoaderClientImpl.h:

2011-03-24 Andy Estes <aestes@apple.com>

Reviewed by Darin Adler.

REGRESSION (r70748): latest nightly builds kills AC_QuickTime.js
https://bugs.webkit.org/show_bug.cgi?id=49016

Update objectContentType() implementation to handle the
shouldPreferPlugInsForImages flag.

  • WebCoreSupport/FrameLoaderClientQt.cpp: (WebCore::FrameLoaderClientQt::objectContentType):
  • WebCoreSupport/FrameLoaderClientQt.h:

2011-03-24 Andy Estes <aestes@apple.com>

Reviewed by Darin Adler.

REGRESSION (r70748): latest nightly builds kills AC_QuickTime.js
https://bugs.webkit.org/show_bug.cgi?id=49016

Update objectContentType() implementation to handle the
shouldPreferPlugInsForImages flag.

  • WebCoreSupport/FrameLoaderClientGtk.cpp: (WebKit::FrameLoaderClient::objectContentType):
  • WebCoreSupport/FrameLoaderClientGtk.h:

2011-03-24 Andy Estes <aestes@apple.com>

Reviewed by Darin Adler.

REGRESSION (r70748): latest nightly builds kills AC_QuickTime.js
https://bugs.webkit.org/show_bug.cgi?id=49016

Update objectContentType() implementation to handle the
shouldPreferPlugInsForImages flag.

  • WebFrame.cpp: (WebFrame::objectContentType):
  • WebFrame.h:

2011-03-24 Andy Estes <aestes@apple.com>

Reviewed by Darin Adler.

REGRESSION (r70748): latest nightly builds kills AC_QuickTime.js
https://bugs.webkit.org/show_bug.cgi?id=49016

Update objectContentType() implementation to handle the
shouldPreferPlugInsForImages flag.

  • WebKitSupport/FrameLoaderClientWx.cpp: (WebCore::FrameLoaderClientWx::objectContentType):
  • WebKitSupport/FrameLoaderClientWx.h:

2011-03-24 Andy Estes <aestes@apple.com>

Reviewed by Darin Adler.

REGRESSION (r70748): latest nightly builds kills AC_QuickTime.js
https://bugs.webkit.org/show_bug.cgi?id=49016

Update objectContentType() implementation to handle the
shouldPreferPlugInsForImages flag.

  • WebCoreSupport/WebFrameLoaderClient.h:
  • WebCoreSupport/WebFrameLoaderClient.mm: (WebFrameLoaderClient::objectContentType):

2011-03-24 Andy Estes <aestes@apple.com>

Reviewed by Darin Adler.

REGRESSION (r70748): latest nightly builds kills AC_QuickTime.js
https://bugs.webkit.org/show_bug.cgi?id=49016

Update objectContentType() implementation to handle the
shouldPreferPlugInsForImages flag.

  • WebCoreSupport/FrameLoaderClientHaiku.cpp: (WebCore::FrameLoaderClientHaiku::objectContentType):
  • WebCoreSupport/FrameLoaderClientHaiku.h:

2011-03-24 Andy Estes <aestes@apple.com>

Reviewed by Darin Adler.

REGRESSION (r70748): latest nightly builds kills AC_QuickTime.js
https://bugs.webkit.org/show_bug.cgi?id=49016

Update objectContentType() implementation to handle the
shouldPreferPlugInsForImages flag.

  • WebCoreSupport/FrameLoaderClientWinCE.cpp: (WebKit::FrameLoaderClientWinCE::objectContentType):
  • WebCoreSupport/FrameLoaderClientWinCE.h:

2011-03-24 Andy Estes <aestes@apple.com>

Reviewed by Darin Adler.

REGRESSION (r70748): latest nightly builds kills AC_QuickTime.js
https://bugs.webkit.org/show_bug.cgi?id=49016

Update objectContentType() implementation to handle the
shouldPreferPlugInsForImages flag.

  • WebCoreSupport/FrameLoaderClientEfl.cpp: (WebCore::FrameLoaderClientEfl::objectContentType):
  • WebCoreSupport/FrameLoaderClientEfl.h:

2011-03-24 Andy Estes <aestes@apple.com>

Reviewed by Darin Adler.

REGRESSION (r70748): latest nightly builds kills AC_QuickTime.js
https://bugs.webkit.org/show_bug.cgi?id=49016

AC_QuickTime.js uses the common <object>/<embed> paradigm to embed the
QuickTime plug-in in web pages. r70748 removed our mapping of classids
to MIME types, which causes WebKit to fall back from the object to the
embed tag when QuickTime is embedded by this script. The script emits
the following embed tag to embed a QuickTime movie with a poster frame:

<embed src="poster-frame.png" target="quicktimeplayer" href="movie.mov">

The expectation is that a QuickTime plug-in is instantiated to display the
poster frame, since QuickTime registers for many common image MIME
types. This is how Gecko behaves for embed. However, WebKit prefers to
use its native image rendering for image embeds, in which case no movie
is played when the poster frame is clicked.

Fix this by changing embed to check for a plug-in that can handle the
image type before rendering the image natively. This matches Gecko.

Test: fast/images/embed-image-plugins-disabled.html

  • html/HTMLObjectElement.cpp: (WebCore::HTMLObjectElement::parametersForPlugin):
  • html/HTMLPlugInImageElement.cpp: (WebCore::HTMLPlugInImageElement::HTMLPlugInImageElement): (WebCore::HTMLPlugInImageElement::isImageType): (WebCore::HTMLPlugInImageElement::wouldLoadAsNetscapePlugin):
  • html/HTMLPlugInImageElement.h: (WebCore::HTMLPlugInImageElement::preferPluginsForImages):
  • loader/EmptyClients.h: (WebCore::EmptyFrameLoaderClient::objectContentType):
  • loader/FrameLoader.cpp: (WebCore::FrameLoader::defaultObjectContentType):
  • loader/FrameLoader.h:
  • loader/FrameLoaderClient.h:
  • loader/SubframeLoader.cpp: (WebCore::SubframeLoader::resourceWillUsePlugin): (WebCore::SubframeLoader::requestPlugin): (WebCore::SubframeLoader::requestObject): (WebCore::SubframeLoader::shouldUsePlugin):
  • loader/SubframeLoader.h:

2011-03-24 Andy Estes <aestes@apple.com>

Reviewed by Darin Adler.

REGRESSION (r70748): latest nightly builds kills AC_QuickTime.js
https://bugs.webkit.org/show_bug.cgi?id=49016

Update objectContentType() implementation to handle the
shouldPreferPlugInsForImages flag.

  • WebProcess/WebCoreSupport/WebFrameLoaderClient.cpp: (WebKit::WebFrameLoaderClient::objectContentType):
  • WebProcess/WebCoreSupport/WebFrameLoaderClient.h:
4:12 PM Changeset in webkit [81915] by commit-queue@webkit.org
  • 6 edits in trunk/Source/WebCore

2011-03-24 Mike Reed <reed@google.com>

Reviewed by James Robinson.

Move lifetime management of grContext from a global to being
per-SharedGraphicsContext3D, which correctly is 1:1 with the
underlying opengl context.
https://bugs.webkit.org/show_bug.cgi?id=54330

No new tests. Existing <canvas> tests exercise this:

  • platform/graphics/chromium/DrawingBufferChromium.cpp: (WebCore::DrawingBuffer::DrawingBuffer): (WebCore::DrawingBuffer::~DrawingBuffer): (WebCore::DrawingBuffer::publishToPlatformLayer): (WebCore::DrawingBuffer::setGrContext):
  • platform/graphics/gpu/DrawingBuffer.h:
  • platform/graphics/gpu/SharedGraphicsContext3D.cpp: (WebCore::SharedGraphicsContext3D::SharedGraphicsContext3D): (WebCore::SharedGraphicsContext3D::~SharedGraphicsContext3D): (WebCore::SharedGraphicsContext3D::getGrContext):
  • platform/graphics/gpu/SharedGraphicsContext3D.h:
  • platform/graphics/skia/PlatformContextSkia.cpp: (WebCore::PlatformContextSkia::~PlatformContextSkia): (WebCore::PlatformContextSkia::setSharedGraphicsContext3D):
3:59 PM Changeset in webkit [81914] by bfulgham@webkit.org
  • 2 edits
    2 adds in trunk/Tools

Unreviewed build correction.

Add a stub implementation of the TestInvocation dumping logic.
Update project files to reflect new file.

  • WebKitTestRunner/cairo: Added.
  • WebKitTestRunner/cairo/TestInvocationCairo.cpp: Added.

(WTR::TestInvocation::dumpPixelsAndCompareWithExpected):

  • WebKitTestRunner/win/WebKitTestRunner.vcproj:
3:54 PM Changeset in webkit [81913] by commit-queue@webkit.org
  • 4 edits in trunk/Source/WebKit/chromium

2011-03-24 Bill Budge <bbudge@chromium.org>

Reviewed by Dmitry Titov.

Add an overloaded WebFrame::createAssociatedURLLoader method that takes WebURLLoaderOptions, so clients can specify CORS options to allow cross-origin requests.
https://bugs.webkit.org/show_bug.cgi?id=56885

No new tests. Exposes no new functionality.

  • public/WebFrame.h:
  • src/WebFrameImpl.cpp: (WebKit::WebFrameImpl::createAssociatedURLLoader):
  • src/WebFrameImpl.h:
3:51 PM Changeset in webkit [81912] by Adam Roben
  • 3 edits
    2 adds in trunk/Source/WebKit2

Start compiling LayerTreeHostCA on Windows

Fixes <http://webkit.org/b/57060> WebKit2.vcproj should compile LayerTreeHostCA

Reviewed by Anders Carlsson.

  • WebProcess/WebPage/ca/win/LayerTreeHostCAWin.cpp: Added.

(WebKit::LayerTreeHostCA::platformInitialize):
(WebKit::LayerTreeHostCA::scheduleLayerFlush):
(WebKit::LayerTreeHostCA::platformInvalidate):
(WebKit::LayerTreeHostCA::platformSizeDidChange):
(WebKit::LayerTreeHostCA::platformForceRepaint):
(WebKit::LayerTreeHostCA::platformDidPerformScheduledLayerFlush):
Stubbed out.

  • win/WebKit2.vcproj: Added new files. Let VS reorder existing files.
  • win/WebKit2Apple.vsprops: Added WebProcess\WebPage\ca to the include path.
3:42 PM Changeset in webkit [81911] by jer.noble@apple.com
  • 2 edits in trunk/Source/WebKit2

Unreviewed build fix.

Remove reference to LayerTreeHostMac.h from WebFullScreenManagerMac.mm.

  • WebProcess/FullScreen/mac/WebFullScreenManagerMac.mm:
3:24 PM Changeset in webkit [81910] by ggaren@apple.com
  • 2 edits in trunk/Source/JavaScriptCore

2011-03-24 Geoffrey Garen <ggaren@apple.com>

Windows build fix take 2: Add new symobl.


(I should have used the EWS bots for this!)

3:20 PM Changeset in webkit [81909] by Dimitri Glazkov
  • 4 edits in trunk/Source/WebCore

2011-03-24 Dimitri Glazkov <Dimitri Glazkov>

Reviewed by Darin Adler.

Untangle dependency between event ancestor chain computation and InspectorDOMAgent.
https://bugs.webkit.org/show_bug.cgi?id=57050

Inspector's list of event listeners does not need to invoke Node::getEventListeners,
because it simply wants to collect all ancestors and never uses EventContext bits.

No functional change, covered by existing tests.

  • dom/Node.cpp: (WebCore::getEventAncestors): Converted into a static function. (WebCore::Node::dispatchGenericEvent): Changed to pass node to getEventAncestors.
  • dom/Node.h: Removed decl, moved EventDispatchBehavior enum inside.
  • inspector/InspectorDOMAgent.cpp: (WebCore::InspectorDOMAgent::getEventListenersForNode): Replaced the call

to getEventAncestors with a simple ancestor traversal loop.

3:14 PM Changeset in webkit [81908] by dpranke@chromium.org
  • 7 edits in trunk/Tools

2011-03-24 Dirk Pranke <dpranke@chromium.org>

Reviewed by Tony Chang.

new-run-webkit-tests: clean up worker model defaults. This
changes the default behavior to 'threads' instead of
'old-threads', and clarifies that chromium-win-* and
chromium-mac-leopard are exceptions.

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

  • Scripts/webkitpy/layout_tests/port/base.py:
  • Scripts/webkitpy/layout_tests/port/chromium_linux.py:
  • Scripts/webkitpy/layout_tests/port/chromium_mac.py:
  • Scripts/webkitpy/layout_tests/port/chromium_win.py:
  • Scripts/webkitpy/layout_tests/port/test.py:
  • Scripts/webkitpy/layout_tests/run_webkit_tests_unittest.py:
3:13 PM Changeset in webkit [81907] by ggaren@apple.com
  • 2 edits in trunk/Source/JavaScriptCore

2011-03-24 Geoffrey Garen <ggaren@apple.com>

Windows build fix take 1: Removed old symobl.

3:13 PM Changeset in webkit [81906] by commit-queue@webkit.org
  • 4 edits in trunk

2011-03-24 Adam Klein <adamk@chromium.org>

Reviewed by David Levin.

[fileapi] Make FileError and FileException accessible from WorkerContext
https://bugs.webkit.org/show_bug.cgi?id=57041

Fixed some test expectations related to FileError/FileExcetion and
exposed while testing this change under Chromium.

  • http/tests/filesystem/workers/resolve-url-sync-expected.txt:

2011-03-24 Adam Klein <adamk@chromium.org>

Reviewed by David Levin.

[fileapi] Make FileError and FileException accessible from WorkerContext
https://bugs.webkit.org/show_bug.cgi?id=57041

  • workers/WorkerContext.idl:
3:11 PM Changeset in webkit [81905] by weinig@apple.com
  • 5 edits in trunk/Source

2011-03-24 Sam Weinig <sam@webkit.org>

Reviewed by Anders Carlsson.

Remove legacy version of findPlainText.
https://bugs.webkit.org/show_bug.cgi?id=57056

  • src/WebFrameImpl.cpp: (WebKit::WebFrameImpl::scopeStringMatches): Replace use of legacy findPlainText with a version that takes an options parameter.

2011-03-24 Sam Weinig <sam@webkit.org>

Reviewed by Anders Carlsson.

Remove legacy version of findPlainText.
https://bugs.webkit.org/show_bug.cgi?id=57056

  • editing/TextIterator.cpp:
  • editing/TextIterator.h: Remove legacy overload of findPlainText that doesn't take an options parameter.
3:02 PM Changeset in webkit [81904] by ggaren@apple.com
  • 15 edits in trunk/Source

2011-03-24 Geoffrey Garen <ggaren@apple.com>

Reviewed by Oliver Hunt.

Ensure that all compilation takes place within a dynamic global object scope
https://bugs.webkit.org/show_bug.cgi?id=57054
<rdar://problem/9083011>

Otherwise, entry to the global object scope might throw away the code
we just compiled, causing a crash.

  • debugger/Debugger.cpp: (JSC::evaluateInGlobalCallFrame):
  • debugger/DebuggerCallFrame.cpp: (JSC::DebuggerCallFrame::evaluate): Removed explicit compilation calls here because (a) they took place outside a dynamic global object scope and (b) they were redundant.
  • interpreter/CachedCall.h: (JSC::CachedCall::CachedCall): Updated for signature change.
  • interpreter/Interpreter.cpp: (JSC::Interpreter::execute): (JSC::Interpreter::executeCall): (JSC::Interpreter::executeConstruct): Declare our dynamic global object scope earlier, to ensure that compilation takes place within it.
  • runtime/Completion.cpp: (JSC::evaluate): Removed explicit compilation calls here because (a) they took place outside a dynamic global object scope and (b) they were redundant.
  • runtime/Executable.h: (JSC::EvalExecutable::compile): (JSC::ProgramExecutable::compile): (JSC::FunctionExecutable::compileForCall): (JSC::FunctionExecutable::compileForConstruct): Added an ASSERT to verify our new invariant that all compilation takes place within a dynamic global object scope.
  • runtime/JSGlobalObject.cpp: (JSC::DynamicGlobalObjectScope::DynamicGlobalObjectScope):
  • runtime/JSGlobalObject.h: Changed the signature of DynamicGlobalObjectScope to require a JSGlobalData instead of an ExecState* since it is often easier to provide the former, and the latter was not necessary.

2011-03-24 Geoffrey Garen <ggaren@apple.com>

Reviewed by Oliver Hunt.

Ensure that all compilation takes place within a dynamic global object scope
https://bugs.webkit.org/show_bug.cgi?id=57054

  • WebView/WebScriptDebugDelegate.mm: (-[WebScriptCallFrame evaluateWebScript:]): Updated for signature change.

2011-03-24 Geoffrey Garen <ggaren@apple.com>

Reviewed by Oliver Hunt.

Ensure that all compilation takes place within a dynamic global object scope
https://bugs.webkit.org/show_bug.cgi?id=57054

  • bindings/js/JSErrorHandler.cpp: (WebCore::JSErrorHandler::handleEvent):
  • bindings/js/JSEventListener.cpp: (WebCore::JSEventListener::handleEvent): Updated for signature change.
2:50 PM Changeset in webkit [81903] by Adam Roben
  • 3 edits
    1 move
    2 adds in trunk/Source/WebKit2

Split Mac-specific parts of LayerTreeHostCA into LayerTreeHostCAMac.mm

Fixes <http://webkit.org/b/57046> LayerTreeHostMac's code should be shareable with Windows

Reviewed by Anders Carlsson.

  • WebKit2.xcodeproj/project.pbxproj:
  • WebProcess/WebPage/ca/LayerTreeHostCA.cpp: Renamed from Source/WebKit2/WebProcess/WebPage/ca/LayerTreeHostCA.mm.

(WebKit::LayerTreeHostCA::LayerTreeHostCA): Moved some Mac-specific code to
LayerTreeHostCAMac.mm.
(WebKit::LayerTreeHostCA::~LayerTreeHostCA): Wrapped some Mac-specific code in
PLATFORM(MAC).

(WebKit::LayerTreeHostCA::invalidate):
(WebKit::LayerTreeHostCA::sizeDidChange):
(WebKit::LayerTreeHostCA::forceRepaint):
Moved some Mac-specific code to LayerTreeHostCAMac.mm.

(WebKit::LayerTreeHostCA::performScheduledLayerFlush): Renamed from
flushPendingLayerChangesRunLoopObserverCallback. Moved some code from here...
(WebKit::LayerTreeHostCA::didPerformScheduledLayerFlush): ...to here. Moved some
Mac-specific code to LayerTreeHostCAMac.mm.

  • WebProcess/WebPage/ca/LayerTreeHostCA.h: Added new functions, wrapped some Mac-specific

declarations in PLATFORM(MAC).

  • WebProcess/WebPage/ca/mac/LayerTreeHostCAMac.mm: Added.

(WebKit::LayerTreeHostCA::platformInitialize):
(WebKit::LayerTreeHostCA::scheduleLayerFlush):
(WebKit::LayerTreeHostCA::platformInvalidate):
(WebKit::LayerTreeHostCA::platformSizeDidChange):
(WebKit::LayerTreeHostCA::platformForceRepaint):
(WebKit::LayerTreeHostCA::flushPendingLayerChangesRunLoopObserverCallback):
(WebKit::LayerTreeHostCA::platformDidPerformScheduledLayerFlush):
Code was extracted from LayerTreeHostCA.cpp.

2:49 PM Changeset in webkit [81902] by Adam Roben
  • 3 edits
    2 moves
    1 add in trunk/Source/WebKit2

Rename LayerTreeHostMac to LayerTreeHostCA

This is the first step toward sharing code with Windows.

Fixes <http://webkit.org/b/57051>.

Reviewed by Anders Carlsson.

  • WebKit2.xcodeproj/project.pbxproj:
  • WebProcess/WebPage/LayerTreeHost.cpp:

(WebKit::LayerTreeHost::create):
Updated for rename.

  • WebProcess/WebPage/ca/LayerTreeHostCA.h: Renamed from Source/WebKit2/WebProcess/WebPage/mac/LayerTreeHostMac.h.
  • WebProcess/WebPage/ca/LayerTreeHostCA.mm: Renamed from Source/WebKit2/WebProcess/WebPage/mac/LayerTreeHostMac.mm.
2:45 PM Changeset in webkit [81901] by jer.noble@apple.com
  • 3 edits in trunk/Source/WebKit2

2011-03-17 Jer Noble <jer.noble@apple.com>

Reviewed by Maciej Stachowiak.

WebKit2: Cancelling full screen early leaves full screen window up.
https://bugs.webkit.org/show_bug.cgi?id=56589

No new tests, as WebKitTestRunner does not currently support the new Full Screen API.

Notify the UIProcess when it needs to tear down its layer hosting view, turn off
the background layer when not in accelerated rendering mode and don't swap out
the web view unnecessarily

  • UIProcess/mac/WKFullScreenWindowController.mm: (-[WKFullScreenWindowController beganExitFullScreenAnimation]): Check before swapping _webView. (-[WKFullScreenWindowController enterAcceleratedCompositingMode:]): Add the _layerHostingView

to the full screen window's animationView, not its contentsView.

(-[WKFullScreenWindowController exitAcceleratedCompositingMode]): Hide the background layer.

  • WebProcess/FullScreen/mac/WebFullScreenManagerMac.mm: (WebKit::WebFullScreenManagerMac::setRootFullScreenLayer): If given a null rootLayer, tell

the client to exit accelerated mode.

2:44 PM Changeset in webkit [81900] by oliver@apple.com
  • 3 edits in trunk/Source/JavaScriptCore

2011-03-24 Oliver Hunt <oliver@apple.com>

Reviewed by Geoffrey Garen.

REGRESSION (r79987-r80210): Crash in JSWeakObjectMapClear
https://bugs.webkit.org/show_bug.cgi?id=55671

This is no longer necessary, and it seems that with the new weakmap
model it's simply unsafe, so this reduces it to a no-op.

  • API/JSWeakObjectMapRefPrivate.cpp:
2:40 PM Changeset in webkit [81899] by commit-queue@webkit.org
  • 4 edits in trunk/Source/WebCore

2011-03-24 John Bauman <jbauman@chromium.org>

Reviewed by Kenneth Russell.

preserveDrawingBuffer=true is ignored
https://bugs.webkit.org/show_bug.cgi?id=56987

Add code to the V8 and JSC bindings to support grabbing the value of
preserveDrawingBuffer from the input context attributes. Also, in
WebGLRenderingContext use the WebGLContextAttributes that were input
directly, not those from the GraphicsContext3D which could have been
changed.

No new tests, as this can't be tested with DRT. However, this works
when tested manually.

  • bindings/js/JSHTMLCanvasElementCustom.cpp: (WebCore::JSHTMLCanvasElement::getContext):
  • bindings/v8/custom/V8HTMLCanvasElementCustom.cpp: (WebCore::V8HTMLCanvasElement::getContextCallback):
  • html/canvas/WebGLRenderingContext.cpp: (WebCore::WebGLRenderingContext::clearIfComposited): (WebCore::WebGLRenderingContext::paintRenderingResultsToCanvas):
2:22 PM Changeset in webkit [81898] by enrica@apple.com
  • 11 edits in trunk/Source

WebKit2:Services menu item to convert selected Simplified/Traditional Chinese Text is not working.
https://bugs.webkit.org/show_bug.cgi?id=56975
<rdar://problem/8915066>

Reviewed by Alexey Proskuryakov.

Source/WebCore:

Adding support in WebCore to implement readSelectionFromPasteboard
to support Mac OS X services from WebKit2.

  • WebCore.exp.in:
  • editing/Editor.h:
  • editing/mac/EditorMac.mm:

(WebCore::Editor::readSelectionFromPasteboard): Added entry point
to call the paste functions with the specified pasteboard.

Source/WebKit2:

Adding missing entry point to support Mac OS X services in WebKit2.

  • UIProcess/API/mac/WKView.mm:

(-[WKView readSelectionFromPasteboard:]): Added.

  • UIProcess/WebPageProxy.h:
  • UIProcess/mac/WebPageProxyMac.mm:

(WebKit::WebPageProxy::readSelectionFromPasteboard): Added.

  • WebProcess/WebPage/WebPage.h:
  • WebProcess/WebPage/WebPage.messages.in: Added synchronous

message.

  • WebProcess/WebPage/mac/WebPageMac.mm:

(WebKit::WebPage::readSelectionFromPasteboard): Added.

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

2011-03-24 Vsevolod Vlasov <vsevik@chromium.org>

Reviewed by Pavel Feldman.

Web Inspector: Inspector does not show correct transfer size for synchronous requests
https://bugs.webkit.org/show_bug.cgi?id=56951

Fixed transfer size for synchronous load.

  • http/tests/inspector/network/network-size-sync-expected.txt: Added.
  • http/tests/inspector/network/network-size-sync.html: Added.
  • platform/chromium/test_expectations.txt:
  • platform/gtk/Skipped:
  • platform/mac/Skipped:
  • platform/qt/Skipped:
  • platform/win/Skipped:

2011-03-24 Vsevolod Vlasov <vsevik@chromium.org>

Reviewed by Pavel Feldman.

Web Inspector: Inspector does not show correct transfer size for synchronous requests
https://bugs.webkit.org/show_bug.cgi?id=56951

Fixed transfer size for synchronous load.

Test: http/tests/inspector/network/network-size-sync.html

  • loader/FrameLoader.cpp: (WebCore::FrameLoader::willLoadMediaElementURL): (WebCore::FrameLoader::commitProvisionalLoad): (WebCore::FrameLoader::loadResourceSynchronously): (WebCore::FrameLoader::loadedResourceFromMemoryCache):
  • loader/ResourceLoadNotifier.cpp: (WebCore::ResourceLoadNotifier::sendRemainingDelegateMessages):
  • loader/ResourceLoadNotifier.h:
2:14 PM Changeset in webkit [81896] by commit-queue@webkit.org
  • 4 edits in trunk/Source

2011-03-24 Jia Pu <jpu@apple.com>

Reviewed by Darin Adler.

Dismissed reversion suggestion is incorrectly learned.
https://bugs.webkit.org/show_bug.cgi?id=57039

CorrectionPanel should always use [NSSpellChecker dismissCorrectionBubbleForView].
[NSSpellChecker cancelCorrectionBubbleForView] is reserved for situation where correction panel
is dismissed explicitly by ESC key or clicking the dimiss button. Misusing these causes
incorrect automatic learning.

  • WebCoreSupport/CorrectionPanel.mm: (CorrectionPanel::dismissInternal):

2011-03-24 Jia Pu <jpu@apple.com>

Reviewed by Darin Adler.

Dismissed reversion suggestion is incorrectly learned.
https://bugs.webkit.org/show_bug.cgi?id=57039

CorrectionPanel should always use [NSSpellChecker dismissCorrectionBubbleForView].
[NSSpellChecker cancelCorrectionBubbleForView] is reserved for situation where correction panel
is dismissed explicitly by ESC key or clicking the dimiss button. Misusing these causes
incorrect automatic learning.

  • UIProcess/mac/CorrectionPanel.mm: (WebKit::CorrectionPanel::dismissInternal):
1:47 PM Changeset in webkit [81895] by bweinstein@apple.com
  • 10 edits in trunk/Source/WebKit2

WebKit2: Add Trackpoint driver hack to support IBM trackpads
https://bugs.webkit.org/show_bug.cgi?id=49830
<rdar://problem/8705951>

Reviewed by Adam Roben.

Copy code from WebKit1 to WebKit2 to handle initializing fake scrollbars so
IBM machines with a trackpad send us WM_VSCROLL and WM_HSCROLL messages.

Listen for the WM_VSCROLL and WM_HSCROLL messages, and turn the values into
ScrollDirection and ScrollGranularity, and send a scroll command to the
WebProcess.

  • UIProcess/WebPageProxy.cpp:

(WebKit::WebPageProxy::scrollBy): Send a message to the WebProcess.

  • UIProcess/WebPageProxy.h:
  • UIProcess/win/WebView.cpp:

(WebKit::WebView::wndProc): Add WM_VSCROLL and WM_HSCROLL message handling.
(WebKit::WebView::initialize): Call shouldInitializeTrackPointHack.
(WebKit::WebView::onHorizontalScroll): Turn wParam into a ScrollDirection and ScrollGranularity.
(WebKit::WebView::onVerticalScroll): Ditto.
(WebKit::WebView::shouldInitializeTrackPointHack): Check the registry for keys that indicate

the machine has a IBM Trackpoint driver.

  • UIProcess/win/WebView.h:
  • WebProcess/WebPage/WebPage.cpp:

(WebKit::WebPage::scrollBy): Call scroll method.
(WebKit::WebPage::scroll): Moved from WebPageMac and WebPageWin.
(WebKit::WebPage::logicalScroll): Ditto.

  • WebProcess/WebPage/WebPage.h:
  • WebProcess/WebPage/WebPage.messages.in: Add a new scrollBy message.
  • WebProcess/WebPage/mac/WebPageMac.mm: Remove scroll and logicalScroll, they are now in WebPage.cpp.
  • WebProcess/WebPage/win/WebPageWin.cpp: Ditto.
1:38 PM Changeset in webkit [81894] by cevans@google.com
  • 4 edits
    2 copies in branches/chromium/696

Merge 81794
BUG=76195

1:26 PM Changeset in webkit [81893] by enrica@apple.com
  • 5 edits in trunk/LayoutTests

Updating tests results for QT after changeset 81887.

  • platform/qt/editing/pasteboard/5065605-expected.txt:
  • platform/qt/editing/pasteboard/display-block-on-spans-expected.txt:
  • platform/qt/editing/pasteboard/paste-text-011-expected.txt:
  • platform/qt/editing/pasteboard/paste-text-at-tabspan-002-expected.txt:
1:22 PM Changeset in webkit [81892] by beidson@apple.com
  • 4 edits in trunk

Source/WebCore: https://bugs.webkit.org/show_bug.cgi?id=57030
REGRESSION (r81782): http/tests/inspector/extensions-resources-redirect.html sometimes crashes WebKit2's
web process while handling a WebIconDatabaseProxy::ReceivedIconLoadDecision message

Reviewed by Adam Roben.

The callback objects had some bogus ASSERTs and missed a relevant null check. If a callback was waiting on
a message back from the UIProcess, but was invalidated from within the WebProcess in the meantime, it's
perfectly valid to attempt to performCallback() after the callback function pointer has been cleared.

  • loader/icon/IconDatabaseBase.h:

(WebCore::EnumCallback::performCallback):
(WebCore::EnumCallback::invalidate):
(WebCore::EnumCallback::EnumCallback):
(WebCore::ObjectCallback::performCallback):
(WebCore::ObjectCallback::invalidate):
(WebCore::ObjectCallback::ObjectCallback):

LayoutTests: Resolving https://bugs.webkit.org/show_bug.cgi?id=57030, unskip the busted test.

Reviewed by Adam Roben.

  • platform/mac-wk2/Skipped:
1:14 PM Changeset in webkit [81891] by cevans@google.com
  • 2 edits
    2 copies in branches/chromium/696

Merge 81795
BUG=76474

1:06 PM Changeset in webkit [81890] by weinig@apple.com
  • 9 edits in trunk/Source

Dictionary text extraction is not correctly detecting word boundaries on bing.com
<rdar://problem/9078569>
https://bugs.webkit.org/show_bug.cgi?id=56995

Reviewed by Darin Adler.

../WebCore:

  • WebCore.exp.in:

Add some editing related exports needed by WebKit2.

../WebKit2:

  • Shared/DictionaryPopupInfo.cpp:

(WebKit::DictionaryPopupInfo::encode):
(WebKit::DictionaryPopupInfo::decode):

  • Shared/DictionaryPopupInfo.h:

Add options dictionary.

  • UIProcess/API/mac/PageClientImpl.mm:

(WebKit::PageClientImpl::didPerformDictionaryLookup):
Add path that can pass options through.

  • WebProcess/WebCoreSupport/mac/WebContextMenuClientMac.mm:

(WebKit::WebContextMenuClient::lookUpInDictionary):
Use the new performDictionaryLookupForSelection which can extract context
if supported.

  • WebProcess/WebPage/WebPage.h:
  • WebProcess/WebPage/mac/WebPageMac.mm:

(WebKit::characterRangeAtPositionForPoint):
(WebKit::characterRangeAtPoint):
Split functionality out of characterRangeAtPoint and into
characterRangeAtPositionForPoint to avoid doing duplicate work
if you already have the position.

(WebKit::isPositionInRange):
(WebKit::shouldUseSelection):
Add predicate to determine if we should use the selection instead
of expanding to find the word we are over.

(WebKit::WebPage::performDictionaryLookupAtLocation):
If available, use the surrounding paragraph as context to get better extraction
and to get lexicographical information about the word. Also, clean up and use
editing APIs to make the code more concise and understandable.

(WebKit::WebPage::performDictionaryLookupForSelection):
Use similar logic as in performDictionaryLookupAtLocation to extract additional
details from a selection for use in the dictionary popup.

(WebKit::WebPage::performDictionaryLookupForRange):
Pass options to WebProcess if available.

12:58 PM Changeset in webkit [81889] by aestes@apple.com
  • 3 edits
    2 adds in trunk

2011-03-24 Andy Estes <aestes@apple.com>

Reviewed by Eric Seidel.

REGRESSION (r80231): Bad cast in HTMLTreeBuilder with closed </form> tags
https://bugs.webkit.org/show_bug.cgi?id=56836

During fragment parsing, HTMLConstructionSite holds a reference to the
fragment context's closest form ancestor. If a misnested form end tag is
then encountered as the first node of the fragment, we will check to see
if a corresponding form start tag is in scope even though no such tag
exists. This led to isScope() walking the HTMLElementStack all the way
to the root DocumentFragment and attempting to cast it to Element*.

Fix this by ensuring that the inScope() family of functions operate in
terms of ContainerNodes to account for the fragment case.

Test: fast/parser/fragment-closest-form-ancestor.html

  • html/parser/HTMLElementStack.cpp: (WebCore::HTMLNames::isRootNode): (WebCore::HTMLNames::isScopeMarker): (WebCore::HTMLNames::isTableScopeMarker): (WebCore::HTMLNames::isTableBodyScopeMarker): (WebCore::HTMLNames::isTableRowScopeMarker): (WebCore::HTMLElementStack::hasOnlyHTMLElementsInScope): (WebCore::HTMLElementStack::inScope):

2011-03-24 Andy Estes <aestes@apple.com>

Reviewed by Eric Seidel.

REGRESSION (r80231): Bad cast in HTMLTreeBuilder with closed </form> tags
https://bugs.webkit.org/show_bug.cgi?id=56836

  • fast/parser/fragment-closest-form-ancestor-expected.txt: Added.
  • fast/parser/fragment-closest-form-ancestor.html: Added.
12:58 PM Changeset in webkit [81888] by cevans@google.com
  • 6 edits
    2 copies in branches/chromium/696

Merge 81851
BUG=76966

12:46 PM Changeset in webkit [81887] by enrica@apple.com
  • 8 edits
    2 adds in trunk

Source/WebCore: Repeated copy and paste-in-place operation results in increasingly verbose HTML.
<rdar://problem/8690506>
https://bugs.webkit.org/show_bug.cgi?id=56874

Reviewed by Darin Adler.

When we calculate the style to apply at the insertion point we compare the initial
style at the insertion point against the style calculated at the span we wrap the
copied markup fragment with. We could end up with a series of unnecessary spans
to remove the initial style that simply grow our markup.
The consists in moving the insertion point outside any inline element that could
affect the fragment being inserted when we are not pasting and matching the style.

Test: editing/pasteboard/paste-text-with-style.html

  • editing/ReplaceSelectionCommand.cpp:

(WebCore::isInlineNodeWithStyle): Added.
(WebCore::ReplaceSelectionCommand::doApply): Added logic to change the insertion
point according to the new rules.

LayoutTests: Repeated copy and paste-in-place operation results in increasingly verbose HTML.
<rdar://problem/8690506>
https://bugs.webkit.org/show_bug.cgi?id=56874

Reviewed by Darin Adler.

  • editing/pasteboard/paste-text-with-style-expected.txt: Added.
  • editing/pasteboard/paste-text-with-style.html: Added.

The following are new results for existing tests that now produce
a different markup.

  • platform/mac/editing/pasteboard/5065605-expected.txt:
  • platform/mac/editing/pasteboard/display-block-on-spans-expected.txt:
  • platform/mac/editing/pasteboard/paste-text-011-expected.txt:
  • platform/mac/editing/pasteboard/paste-text-at-tabspan-001-expected.txt:
  • platform/mac/editing/pasteboard/paste-text-at-tabspan-002-expected.txt:
12:34 PM Changeset in webkit [81886] by benjamin.poulain@nokia.com
  • 7 edits
    7 adds in trunk/Source

2011-03-24 Benjamin Poulain <benjamin.poulain@nokia.com>

Reviewed by Kenneth Rohde Christiansen.

[Qt] When we render WebGL offscreen, color conversion cost a lot of CPU cycles
https://bugs.webkit.org/show_bug.cgi?id=40884

The software fallback is now only needed for corner cases like a manual rendering
of the page to QImage.

Keeping the image with the last pixel values is no longer needed. Removing it reduce the
performance for real-time rendering on software surface, but this case should no longer be
supported.

The conversion from OpenGL color space and coordinates is done manually for performance. This
also fix the bug of the inverted X axis due to the transformation.

The tests and benchmarks are done through Qt API tests.

  • platform/graphics/qt/GraphicsContext3DQt.cpp: (WebCore::swapBgrToRgb): (WebCore::GraphicsContext3DInternal::paint): (WebCore::GraphicsContext3D::reshape):

2011-03-24 Benjamin Poulain <benjamin.poulain@nokia.com>

Reviewed by Kenneth Rohde Christiansen.

[Qt] When we render WebGL offscreen, color conversion cost a lot of CPU cycles
https://bugs.webkit.org/show_bug.cgi?id=40884

Add tests and benchmarks for the software fallback of WebGL.

  • tests/benchmarks/webgl/10000_triangles.html: Added.
  • tests/benchmarks/webgl/tst_webgl.cpp: Added. (GraphicsView::GraphicsView): (GraphicsView::resizeEvent): (tst_WebGlPerformance::init): (tst_WebGlPerformance::cleanup): (tst_WebGlPerformance::benchSoftwareFallbackRgb16): (tst_WebGlPerformance::benchSoftwareFallbackRgb32): (tst_WebGlPerformance::benchSoftwareFallbackArgb32): (tst_WebGlPerformance::benchSoftwareFallbackArgb32Premultiplied): (tst_WebGlPerformance::benchmarkFrameRenderingOnImage):
  • tests/benchmarks/webgl/tst_webgl.qrc: Added.
  • tests/benchmarks/webgl/webgl.pro: Added.
  • tests/qgraphicswebview/qgraphicswebview.pro:
  • tests/qgraphicswebview/resources/pointing_right.html: Added.
  • tests/qgraphicswebview/resources/pointing_up.html: Added.
  • tests/qgraphicswebview/tst_qgraphicswebview.cpp: (compareImagesFuzzyPixelCount): (GraphicsView::GraphicsView): (tst_QGraphicsWebView::webglSoftwareFallbackVerticalOrientation): (tst_QGraphicsWebView::webglSoftwareFallbackHorizontalOrientation): (tst_QGraphicsWebView::compareCanvasToImage):
  • tests/qgraphicswebview/tst_qgraphicswebview.qrc:
  • tests/tests.pro:
12:30 PM Changeset in webkit [81885] by cevans@google.com
  • 2 edits
    2 copies in branches/chromium/696

Merge 81786
BUG=77130

12:17 PM Changeset in webkit [81884] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebCore

2011-03-24 Nat Duca <nduca@chromium.org>

Reviewed by James Robinson.

[chromium] Remove bool that forces compositor HUD to always be enabled
https://bugs.webkit.org/show_bug.cgi?id=57034

  • platform/graphics/chromium/cc/CCHeadsUpDisplay.h: (WebCore::CCHeadsUpDisplay::enabled):
12:14 PM Changeset in webkit [81883] by ddkilzer@apple.com
  • 2 edits in trunk/Source/WebKit/mac

<http://webkit.org/b/56993> ENABLE_PLUGIN_PROXY_FOR_VIDEO should be defined to 0 on Mac OS X

Reviewed by Joseph Pecoraro.

  • Plugins/WebPluginContainerPrivate.h: Changed value of

ENABLE_PLUGIN_PROXY_FOR_VIDEO from 1 to 0.

12:14 PM Changeset in webkit [81882] by commit-queue@webkit.org
  • 2 edits in trunk/Source/JavaScriptCore

2011-03-24 Ben Taylor <bentaylor.solx86@gmail.com>

Reviewed by Darin Adler.

https://bugs.webkit.org/show_bug.cgi?id=20302
Correct implementation of signbit on Solaris

  • wtf/MathExtras.h: (signbit):
12:02 PM Changeset in webkit [81881] by commit-queue@webkit.org
  • 6 edits
    1 add in trunk/Source

2011-03-24 Nat Duca <nduca@chromium.org>

Reviewed by Kenneth Russell.

[chromium] Add traceEvents to compositor
https://bugs.webkit.org/show_bug.cgi?id=56965

  • WebCore.gypi:
  • platform/chromium/TraceEvent.h: Added. (WebCore::internal::ScopeTracer::ScopeTracer): (WebCore::internal::ScopeTracer::~ScopeTracer):
  • platform/graphics/chromium/LayerRendererChromium.cpp: (WebCore::LayerRendererChromium::updateRootLayerContents): (WebCore::LayerRendererChromium::updateRootLayerScrollbars): (WebCore::LayerRendererChromium::updateLayers): (WebCore::LayerRendererChromium::drawLayers): (WebCore::LayerRendererChromium::finish): (WebCore::LayerRendererChromium::present):
  • platform/graphics/chromium/LayerTilerChromium.cpp: (WebCore::LayerTilerChromium::update):

2011-03-24 Nat Duca <nduca@chromium.org>

Reviewed by Kenneth Russell.

[chromium] Add traceEvents to compositor
https://bugs.webkit.org/show_bug.cgi?id=56965

  • src/WebViewImpl.cpp: (WebKit::WebViewImpl::composite):
11:57 AM Changeset in webkit [81880] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebKit2

2011-03-24 Jon Lee <jonlee@apple.com>

Reviewed by Darin Adler.

WebKit2: Tabbing from the last focused field to a non-webpage element leaves the selection in a weird state
<rdar://problem/8553962>

  • WebProcess/WebPage/WebPage.cpp: (WebKit::WebPage::setFocused): When the page loses focus, clear out any selection in the frame
11:11 AM Changeset in webkit [81879] by bweinstein@apple.com
  • 2 edits in trunk/Source/WebKit2

Fix a typo in the VK_LEFT case of performDefaultBehaviorForKeyEvent.

Rubber-stamped by Sam Weinig.

  • WebProcess/WebPage/mac/WebPageMac.mm:

(WebKit::WebPage::performDefaultBehaviorForKeyEvent): It should be a logical or,

not a bitwise or.

11:11 AM Changeset in webkit [81878] by pfeldman@chromium.org
  • 10 edits
    1 add in trunk

2011-03-24 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Yury Semikhatsky.

Web Inspector: render XHRs matching JSON regex as JSON.
https://bugs.webkit.org/show_bug.cgi?id=57035

  • English.lproj/localizedStrings.js:
  • WebCore.gypi:
  • WebCore.vcproj/WebCore.vcproj:
  • inspector/front-end/NetworkItemView.js: (WebInspector.NetworkItemView):
  • inspector/front-end/RemoteObject.js: (WebInspector.LocalJSONObject.prototype.get description.switch.case): (WebInspector.LocalJSONObject.prototype.get description): (WebInspector.LocalJSONObject.prototype._concatenate): (WebInspector.LocalJSONObject.prototype.getProperties): (WebInspector.LocalJSONObject.prototype._children):
  • inspector/front-end/ResourceJSONView.js: Added. (WebInspector.ResourceJSONView): (WebInspector.ResourceJSONView.parseJSON.WebInspector.ResourceJSONView.prototype.hasContent): (WebInspector.ResourceJSONView.parseJSON.WebInspector.ResourceJSONView.prototype.show): (WebInspector.ResourceJSONView.parseJSON.WebInspector.ResourceJSONView.prototype._initialize):
  • inspector/front-end/WebKit.qrc:
  • inspector/front-end/inspector.css: (.resource-view.json):
  • inspector/front-end/inspector.html:
10:51 AM Changeset in webkit [81877] by jeffm@apple.com
  • 2 edits in trunk/Source/WebKit2

Fix typo - USE(CF_NETWORK) should be USE(CFNETWORK).

  • WebProcess/InjectedBundle/win/InjectedBundleWin.cpp:

(WebKit::InjectedBundle::setHostAllowsAnyHTTPSCertificate): USE(CF_NETWORK) should be USE(CFNETWORK)

10:47 AM Changeset in webkit [81876] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebKit/qt

2011-03-24 Kristian Amlie <kristian.amlie@nokia.com>

Reviewed by Benjamin Poulain.

Avoided ASCII-cast warnings for WebKit.

Normally they won't be enabled anyway, but if you build webkit from
within the Qt mother repository it will pick up Qt's default build
settings, which do enable it. We need to disable them because
warnings are treated as errors and there are way too many of them in
the WebKit code.

[Qt] Avoid ASCII-cast warnings for WebKit.
https://bugs.webkit.org/show_bug.cgi?id=57016

10:21 AM Changeset in webkit [81875] by commit-queue@webkit.org
  • 3 edits in trunk/Source/WebKit/chromium

2011-03-24 Dominic Mazzoni <dmazzoni@google.com>

Reviewed by Dimitri Glazkov.

Add accessibilityObject accessor to WebDocument (needed for
building an accessibility tree that includes iframes).
https://bugs.webkit.org/show_bug.cgi?id=56984

  • public/WebDocument.h:
  • src/WebDocument.cpp: (WebKit::WebDocument::accessibilityObject):
10:03 AM Changeset in webkit [81874] by Adam Roben
  • 3 edits in trunk/LayoutTests

Add some flaky tests to the mac-wk2 and win-wk2 Skipped files

The failures are tracked by <http://webkit.org/b/57027> and <http://webkit.org/b/57030>.

  • platform/mac-wk2/Skipped: Added http/tests/inspector/extensions-resources-redirect.html.
  • platform/win-wk2/Skipped: Added inspector/debugger/source-frame.html.
10:03 AM Changeset in webkit [81873] by commit-queue@webkit.org
  • 3 edits in trunk/Source/WebKit/chromium

2011-03-24 Ilya Sherman <isherman@chromium.org>

Reviewed by Dimitri Glazkov.

Expose Node::isFocusable() in the Chromium WebKit API
In service of https://code.google.com/p/chromium/issues/detail?id=72918
https://bugs.webkit.org/show_bug.cgi?id=56809

  • public/WebNode.h:
  • src/WebNode.cpp: (WebKit::WebNode::isFocusable):
9:58 AM Changeset in webkit [81872] by Dimitri Glazkov
  • 4 edits in trunk/Source/WebCore

2011-03-24 Dimitri Glazkov <Dimitri Glazkov>

Reviewed by Darin Adler.

Move media controls subtree creation into one method.
https://bugs.webkit.org/show_bug.cgi?id=56969

Mechanical move, no changes in functionality.

The purpose of this patch is to align existing code closer with its
future version, when MediaControls is an element whose tree is created
at the instantiation.

  • html/shadow/MediaControls.cpp: (WebCore::MediaControls::create): Added, consolidating all subtree generation logic into one place. (WebCore::MediaControls::update): Replaced methods with one call.
  • html/shadow/MediaControls.h: Updated decls.
  • rendering/MediaControlElements.cpp: (WebCore::MediaControlTimelineElement::create): Moved setting of attributes here. (WebCore::MediaControlVolumeSliderElement::create): Ditto.
9:53 AM Changeset in webkit [81871] by sfalken@apple.com
  • 2 edits in trunk/Source/WebCore

Use proper string method to generate webloc string.
https://bugs.webkit.org/show_bug.cgi?id=57028
<rdar://problem/9181955>

Reviewed by Darin Adler.

  • platform/win/ClipboardUtilitiesWin.cpp:

(WebCore::getWebLocData):

9:27 AM Changeset in webkit [81870] by Adam Roben
  • 1 edit
    3 adds in trunk/LayoutTests

Add Windows WebKit2 results for fast/css/pseudo-any.html

This test passes on Windows but fails on Mac. <http://webkit.org/b/56988> tracks the Mac
failure.

  • platform/win-wk2/fast/css/pseudo-any-expected.txt: Added.
9:08 AM Changeset in webkit [81869] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebCore

2011-03-24 Ben Taylor <bentaylor.solx86@gmail.com>

Reviewed by Alexey Proskuryakov.

https://bugs.webkit.org/show_bug.cgi?id=32821
Fix conditionals which had an int for one case and a pointer for another.
Fix is similar to https://bugs.webkit.org/show_bug.cgi?id=56198

No new tests. Fix compilation on Solaris 10 with SunStudio 12 C++

  • css/CSSComputedStyleDeclaration.cpp: (WebCore::CSSComputedStyleDeclaration::valueForShadow):
8:30 AM Changeset in webkit [81868] by commit-queue@webkit.org
  • 4 edits in trunk/Source/WebKit/qt

2011-03-24 Caio Marcelo de Oliveira Filho <caio.oliveira@openbossa.org>

Reviewed by Benjamin Poulain.

[Qt] Resetting the URL property of a QWebView results in the current directory being set as file::-type URL
https://bugs.webkit.org/show_bug.cgi?id=29595

Qt Designer resets the URL by setting it to QUrl(). The bug was caused by
ensureAbsoluteUrl() helper function treating the empty URL as a relative URL, and
prepending the current directory.

By fixing this, now we can pass QUrl() invalid and empty URLs to WebCore layer, which
will end up loading "about:blank", but keeping it as a requested URL.

This patch also simplifies the logic for requestedUrl(), since m_lastRequestedUrl
is filled for the loaded URLs as well, we can use it in every case.

Three new autotests were added, to better cover the expected behavior of setting
the QUrl() in a QWebFrame.

  • Api/qwebframe.cpp: (ensureAbsoluteUrl): do not treat invalid URLs (empty included) as relative. (QWebFrame::requestedUrl): always use m_lastRequestedUrl.
  • WebCoreSupport/FrameLoaderClientQt.cpp: (WebCore::FrameLoaderClientQt::dispatchDidFinishLoad): do not clear m_lastRequestedUrl anymore, since we always rely on it even for loaded frames.
  • tests/qwebframe/tst_qwebframe.cpp: (tst_QWebFrame::setUrlToEmpty): verify the behavior of setting empty URLs. This includes the reduction of the bug report. (tst_QWebFrame::setUrlToInvalid): setting invalid, but not necessarily empty, URLs. (tst_QWebFrame::setUrlHistory): to verify how setting empty URLs affect history.
7:57 AM Changeset in webkit [81867] by podivilov@chromium.org
  • 6 edits in trunk

2011-03-24 Pavel Podivilov <podivilov@chromium.org>

Reviewed by Yury Semikhatsky.

Web Inspector: provide live edit callback to source frame delegate.
https://bugs.webkit.org/show_bug.cgi?id=57003

  • inspector/debugger/live-edit.html:

2011-03-24 Pavel Podivilov <podivilov@chromium.org>

Reviewed by Yury Semikhatsky.

Web Inspector: provide live edit callback to source frame delegate.
https://bugs.webkit.org/show_bug.cgi?id=57003

  • inspector/front-end/DebuggerModel.js: (WebInspector.DebuggerModel.prototype.editScriptSource): (WebInspector.DebuggerModel.prototype._didEditScriptSource):
  • inspector/front-end/DebuggerPresentationModel.js: (WebInspector.DebuggerPresentationModel): (WebInspector.DebuggerPresentationModel.prototype.canEditScriptSource): (WebInspector.DebuggerPresentationModel.prototype.editScriptSource.didEditScriptSource): (WebInspector.DebuggerPresentationModel.prototype.editScriptSource): (WebInspector.DebuggerPresentationModel.prototype._updateBreakpointsAfterLiveEdit): (WebInspector.DebuggerPresentationModel.prototype._scriptForSourceFileId):
  • inspector/front-end/ScriptsPanel.js: (WebInspector.ScriptsPanel.prototype._createSourceFrame): (WebInspector.SourceFrameDelegateForScriptsPanel): (WebInspector.SourceFrameDelegateForScriptsPanel.prototype.canEditScriptSource): (WebInspector.SourceFrameDelegateForScriptsPanel.prototype.editScriptSource):
7:23 AM Changeset in webkit [81866] by Csaba Osztrogonác
  • 2 edits
    8 adds in trunk/LayoutTests

[Qt] Some Unicode tests fail with Qt version >= 4.7.0
https://bugs.webkit.org/show_bug.cgi?id=46251

Patch by Joe Wild <joseph.wild@nokia.com> on 2011-03-24
Reviewed by Csaba Osztrogonác.

Updated tests the print out character 0xFFFF for Qt Platform.
QString.toUtf8() converts non-defined Unicode char values, like
0xFFFF to a replacement character ('?') as documented. Other
platforms leave the character as 0xFFFF (but utf8 encoded).
Converting to a real printable char seems reasonable and may
be more reliable.

  • platform/qt/Skipped:
  • platform/qt/fast/js/regexp-range-bound-ffff-expected.txt: Added.
  • platform/qt/fast/js/switch-behaviour-expected.txt: Added.
  • platform/qt/fast/url: Added.
  • platform/qt/fast/url/anchor-expected.txt: Added.
  • platform/qt/fast/url/path-expected.txt: Added.
  • platform/qt/http/tests/websocket: Added.
  • platform/qt/http/tests/websocket/tests: Added.
  • platform/qt/http/tests/websocket/tests/bad-sub-protocol-non-ascii-expected.txt: Added.
7:07 AM Changeset in webkit [81865] by pfeldman@chromium.org
  • 2 edits
    2 deletes in trunk/LayoutTests

2011-03-24 Pavel Feldman <pfeldman@chromium.org>

Not reviewed: removing newly added inspector test that fails on release bots.

  • http/tests/inspector/network/network-redirect-expected.txt: Removed.
  • http/tests/inspector/network/network-redirect.html: Removed.
  • platform/chromium/test_expectations.txt:
6:59 AM Changeset in webkit [81864] by pfeldman@chromium.org
  • 2 edits in trunk/LayoutTests

2011-03-24 Pavel Feldman <pfeldman@chromium.org>

Not reviewed: updated chromium expectations.

  • platform/chromium/test_expectations.txt:
6:48 AM Changeset in webkit [81863] by benjamin.poulain@nokia.com
  • 2 edits in trunk/Source/WebCore

2011-03-24 Benjamin Poulain <benjamin.poulain@nokia.com>

Reviewed by Andreas Kling.

Regression: WebKit does not build with Python 3 following 56807
https://bugs.webkit.org/show_bug.cgi?id=56923

The module string does not have the function replace in Python 3. The str.replace function
can do the same operation in this case and works with the versions 2 and 3.

  • inspector/generate-inspector-idl:
6:38 AM Changeset in webkit [81862] by pfeldman@chromium.org
  • 2 edits in trunk/Source/WebKit/chromium

2011-03-24 Pavel Feldman <pfeldman@chromium.org>

Not reviewed: fix chromium compilation.

  • public/WebDevToolsAgent.h:
6:27 AM Changeset in webkit [81861] by pfeldman@chromium.org
  • 20 edits
    2 adds in trunk

2011-03-24 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Yury Semikhatsky.

Web Inspector: brush up Network agent API.
https://bugs.webkit.org/show_bug.cgi?id=57001

  • http/tests/inspector/network/network-redirect-expected.txt: Added.
  • http/tests/inspector/network/network-redirect.html: Added.
  • inspector/timeline/timeline-network-resource-expected.txt:
  • platform/chromium/inspector/timeline/timeline-network-resource-expected.txt:

2011-03-24 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Yury Semikhatsky.

Web Inspector: brush up Network agent API.
https://bugs.webkit.org/show_bug.cgi?id=57001

  • inspector/Inspector.json:
  • inspector/InspectorInstrumentation.cpp: (WebCore::InspectorInstrumentation::willSendRequestImpl):
  • inspector/InspectorInstrumentation.h: (WebCore::InspectorInstrumentation::willSendRequest):
  • inspector/InspectorResourceAgent.cpp: (WebCore::buildObjectForResourceRequest): (WebCore::buildObjectForResourceResponse): (WebCore::buildObjectForCachedResource): (WebCore::InspectorResourceAgent::willSendRequest): (WebCore::InspectorResourceAgent::willSendWebSocketHandshakeRequest): (WebCore::InspectorResourceAgent::didReceiveWebSocketHandshakeResponse): (WebCore::InspectorResourceAgent::setExtraHeaders):
  • inspector/InspectorResourceAgent.h:
  • inspector/TimelineRecordFactory.cpp: (WebCore::TimelineRecordFactory::createResourceReceiveResponseData):
  • inspector/front-end/NetworkManager.js: (WebInspector.NetworkDispatcher.prototype._updateResourceWithRequest): (WebInspector.NetworkDispatcher.prototype._updateResourceWithResponse): (WebInspector.NetworkDispatcher.prototype._updateResourceWithCachedResource): (WebInspector.NetworkDispatcher.prototype.willSendRequest): (WebInspector.NetworkDispatcher.prototype.didReceiveWebSocketHandshakeResponse): (WebInspector.NetworkDispatcher.prototype.didCloseWebSocket):
  • inspector/front-end/Resource.js:
  • inspector/front-end/TimelinePanel.js: (WebInspector.TimelinePanel.FormattedRecord.prototype._generatePopupContent):
  • loader/ResourceLoadNotifier.cpp: (WebCore::ResourceLoadNotifier::assignIdentifierToInitialRequest): (WebCore::ResourceLoadNotifier::dispatchWillSendRequest):
  • loader/appcache/ApplicationCacheGroup.cpp: (WebCore::ApplicationCacheGroup::createResourceHandle):

2011-03-24 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Yury Semikhatsky.

Web Inspector: brush up Network agent API.
https://bugs.webkit.org/show_bug.cgi?id=57001

  • public/WebDevToolsAgent.h:
  • src/WebDevToolsAgentImpl.cpp: (WebKit::WebDevToolsAgentImpl::identifierForInitialRequest): (WebKit::WebDevToolsAgentImpl::willSendRequest):
  • src/WebDevToolsAgentImpl.h:
4:19 AM Changeset in webkit [81860] by steveblock@google.com
  • 23 edits
    1 copy in trunk/Source/WebCore

2011-03-11 Steve Block <steveblock@google.com>

Reviewed by Jeremy Orlow.

JNIType is not specific to JNI so should be renamed
https://bugs.webkit.org/show_bug.cgi?id=56197

This patch renames JNIType to JavaType, renames the values of
the enum, and moves it out of JNIUtility.h to its own file.
Also renames the corresponding JavaField and JavaMethod getters.

No new tests, refactoring only.

  • GNUmakefile.am:
  • WebCore.gypi:
  • WebCore.xcodeproj/project.pbxproj:
  • bridge/jni/JNIUtility.cpp: (JSC::Bindings::javaTypeFromClassName): (JSC::Bindings::signatureFromJavaType): (JSC::Bindings::javaTypeFromPrimitiveType): (JSC::Bindings::getJNIField): (JSC::Bindings::callJNIMethod):
  • bridge/jni/JNIUtility.h:
  • bridge/jni/JavaMethod.cpp: (JavaMethod::JavaMethod): (JavaMethod::signature):
  • bridge/jni/JavaMethod.h: (JSC::Bindings::JavaMethod::returnTypeClassName): (JSC::Bindings::JavaMethod::returnType):
  • bridge/jni/JavaType.h: Copied from Source/WebCore/bridge/jni/JavaMethod.h.
  • bridge/jni/jni_jsobject.mm: (JavaJSObject::toString):
  • bridge/jni/jni_objc.mm: (JSC::Bindings::dispatchJNICall):
  • bridge/jni/jsc/JNIUtilityPrivate.cpp: (JSC::Bindings::convertArrayInstanceToJavaArray): (JSC::Bindings::convertValueToJValue):
  • bridge/jni/jsc/JNIUtilityPrivate.h:
  • bridge/jni/jsc/JavaArrayJSC.cpp: (JavaArray::setValueAt): (JavaArray::valueAt):
  • bridge/jni/jsc/JavaClassJSC.cpp:
  • bridge/jni/jsc/JavaFieldJSC.cpp: (JavaField::JavaField): (JavaField::dispatchValueFromInstance): (JavaField::valueFromInstance): (JavaField::dispatchSetValueToInstance): (JavaField::setValueToInstance):
  • bridge/jni/jsc/JavaFieldJSC.h: (JSC::Bindings::JavaField::typeClassName): (JSC::Bindings::JavaField::type):
  • bridge/jni/jsc/JavaInstanceJSC.cpp: (JavaInstance::invokeMethod):
  • bridge/jni/v8/JNIUtilityPrivate.cpp: (JSC::Bindings::convertNPVariantToJValue): (JSC::Bindings::convertJValueToNPVariant):
  • bridge/jni/v8/JNIUtilityPrivate.h:
  • bridge/jni/v8/JavaFieldV8.cpp: (JavaField::JavaField):
  • bridge/jni/v8/JavaFieldV8.h: (JSC::Bindings::JavaField::typeClassName): (JSC::Bindings::JavaField::type):
  • bridge/jni/v8/JavaInstanceV8.cpp: (JavaInstance::invokeMethod): (JavaInstance::getField):
  • bridge/jni/v8/JavaNPObjectV8.cpp: (JSC::Bindings::JavaNPObjectInvoke): (JSC::Bindings::JavaNPObjectGetProperty):
3:55 AM Changeset in webkit [81859] by Philippe Normand
  • 2 edits in trunk/LayoutTests

2011-03-24 Philippe Normand <pnormand@igalia.com>

Unreviewed, skipping 3 xmlhttprequest tests timing out on GTK, see
also bug 33026.

  • platform/gtk/Skipped:
3:23 AM Changeset in webkit [81858] by Philippe Normand
  • 1 edit
    50 adds in trunk/LayoutTests

2011-03-24 Philippe Normand <pnormand@igalia.com>

Unreviewed, new GTK fast/block/lineboxcontain baselines after
r81684 and r81608.

  • editing/selection/fake-drag-expected.txt: Added.
  • platform/gtk/fast/block/lineboxcontain/block-expected.checksum: Added.
  • platform/gtk/fast/block/lineboxcontain/block-expected.png: Added.
  • platform/gtk/fast/block/lineboxcontain/block-expected.txt: Added.
  • platform/gtk/fast/block/lineboxcontain/block-font-expected.checksum: Added.
  • platform/gtk/fast/block/lineboxcontain/block-font-expected.png: Added.
  • platform/gtk/fast/block/lineboxcontain/block-font-expected.txt: Added.
  • platform/gtk/fast/block/lineboxcontain/block-glyphs-expected.checksum: Added.
  • platform/gtk/fast/block/lineboxcontain/block-glyphs-expected.png: Added.
  • platform/gtk/fast/block/lineboxcontain/block-glyphs-expected.txt: Added.
  • platform/gtk/fast/block/lineboxcontain/block-glyphs-replaced-expected.checksum: Added.
  • platform/gtk/fast/block/lineboxcontain/block-glyphs-replaced-expected.png: Added.
  • platform/gtk/fast/block/lineboxcontain/block-glyphs-replaced-expected.txt: Added.
  • platform/gtk/fast/block/lineboxcontain/block-replaced-expected.checksum: Added.
  • platform/gtk/fast/block/lineboxcontain/block-replaced-expected.png: Added.
  • platform/gtk/fast/block/lineboxcontain/block-replaced-expected.txt: Added.
  • platform/gtk/fast/block/lineboxcontain/font-expected.checksum: Added.
  • platform/gtk/fast/block/lineboxcontain/font-expected.png: Added.
  • platform/gtk/fast/block/lineboxcontain/font-expected.txt: Added.
  • platform/gtk/fast/block/lineboxcontain/font-replaced-expected.checksum: Added.
  • platform/gtk/fast/block/lineboxcontain/font-replaced-expected.png: Added.
  • platform/gtk/fast/block/lineboxcontain/font-replaced-expected.txt: Added.
  • platform/gtk/fast/block/lineboxcontain/glyphs-expected.checksum: Added.
  • platform/gtk/fast/block/lineboxcontain/glyphs-expected.png: Added.
  • platform/gtk/fast/block/lineboxcontain/glyphs-expected.txt: Added.
  • platform/gtk/fast/block/lineboxcontain/inline-box-expected.checksum: Added.
  • platform/gtk/fast/block/lineboxcontain/inline-box-expected.png: Added.
  • platform/gtk/fast/block/lineboxcontain/inline-box-expected.txt: Added.
  • platform/gtk/fast/block/lineboxcontain/inline-box-replaced-expected.checksum: Added.
  • platform/gtk/fast/block/lineboxcontain/inline-box-replaced-expected.png: Added.
  • platform/gtk/fast/block/lineboxcontain/inline-box-replaced-expected.txt: Added.
  • platform/gtk/fast/block/lineboxcontain/inline-box-vertical-expected.checksum: Added.
  • platform/gtk/fast/block/lineboxcontain/inline-box-vertical-expected.png: Added.
  • platform/gtk/fast/block/lineboxcontain/inline-box-vertical-expected.txt: Added.
  • platform/gtk/fast/block/lineboxcontain/inline-expected.checksum: Added.
  • platform/gtk/fast/block/lineboxcontain/inline-expected.png: Added.
  • platform/gtk/fast/block/lineboxcontain/inline-expected.txt: Added.
  • platform/gtk/fast/block/lineboxcontain/inline-replaced-expected.checksum: Added.
  • platform/gtk/fast/block/lineboxcontain/inline-replaced-expected.png: Added.
  • platform/gtk/fast/block/lineboxcontain/inline-replaced-expected.txt: Added.
  • platform/gtk/fast/block/lineboxcontain/none-expected.checksum: Added.
  • platform/gtk/fast/block/lineboxcontain/none-expected.png: Added.
  • platform/gtk/fast/block/lineboxcontain/none-expected.txt: Added.
  • platform/gtk/fast/block/lineboxcontain/parsing-invalid-expected.checksum: Added.
  • platform/gtk/fast/block/lineboxcontain/parsing-invalid-expected.png: Added.
  • platform/gtk/fast/block/lineboxcontain/parsing-invalid-expected.txt: Added.
  • platform/gtk/fast/block/lineboxcontain/replaced-expected.checksum: Added.
  • platform/gtk/fast/block/lineboxcontain/replaced-expected.png: Added.
  • platform/gtk/fast/block/lineboxcontain/replaced-expected.txt: Added.
12:55 AM Changeset in webkit [81857] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebKit/chromium

2011-03-24 Noel Gordon <noel.gordon@gmail.com>

Reviewed by Ojan Vafai.

[chromium] WebBindings::getRangeImpl() should NULL check its NPObject argument
https://bugs.webkit.org/show_bug.cgi?id=56996

  • src/WebBindings.cpp: (WebKit::getRangeImpl):
12:31 AM Changeset in webkit [81856] by ojan@chromium.org
  • 2 edits in trunk/Source/WebCore

2011-03-24 Ojan Vafai <ojan@chromium.org>

Was being a little braindead when I committed this.
Only simple selectors are allowed, but sibling selectors
are sibling selectors.

  • css/CSSStyleSelector.cpp: (WebCore::collectFeaturesFromList):
12:00 AM Changeset in webkit [81855] by morrita@google.com
  • 10 edits
    1 add in trunk/Source/WebCore

2011-03-23 MORITA Hajime <morrita@google.com>

Reviewed by Kent Tamura.

Spellcheck feature specific symbols should be defined.
https://bugs.webkit.org/show_bug.cgi?id=56818

  • Introduced USE(UNIFIED_TEXT_CHECKING), USE(GRAMMAR_CHECKING) and USE(AUTOMATIC_TEXT_REPLACEMENT)
  • Replaced a part of of BUILDING_ON_* conditionals with them.

Note that small amount of code path on Editor.cpp is now compiled under USE(GRAMMAR_CHECKING)
because these code path is already built under non-Mac ports and
Keeping them buildable for such platforms reduces the build breakage risk.
These path is guarded by Settings thus should never get reached.

No new tests. No behavioral change.

  • WebCore.gypi:
  • WebCore.pro:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • editing/Editor.cpp: (WebCore::Editor::advanceToNextMisspelling): (WebCore::Editor::isSelectionUngrammatical): (WebCore::Editor::guessesForUngrammaticalSelection): (WebCore::Editor::guessesForMisspelledOrUngrammaticalSelection): (WebCore::Editor::markMisspellingsAfterTypingToWord): (WebCore::Editor::markMisspellingsOrBadGrammar): (WebCore::Editor::markBadGrammar): (WebCore::Editor::markAllMisspellingsAndBadGrammarInRanges): (WebCore::Editor::changeBackToReplacedString): (WebCore::Editor::markMisspellingsAndBadGrammar):
  • editing/Editor.h:
  • editing/TextCheckingHelper.cpp: (WebCore::TextCheckingHelper::findFirstMisspellingOrBadGrammar): (WebCore::TextCheckingHelper::findFirstGrammarDetail): (WebCore::TextCheckingHelper::findFirstBadGrammar): (WebCore::TextCheckingHelper::isUngrammatical): (WebCore::TextCheckingHelper::guessesForMisspelledOrUngrammaticalRange): (WebCore::TextCheckingHelper::markAllBadGrammar):
  • loader/EmptyClients.h:
  • platform/text/TextCheckerClient.h:
  • platform/text/TextChecking.h: Added.

Mar 23, 2011:

11:48 PM Changeset in webkit [81854] by commit-queue@webkit.org
  • 3 edits in trunk/Source/WebKit/chromium

2011-03-23 Noel Gordon <noel.gordon@gmail.com>

Reviewed by Ojan Vafai.

[chromium] WebBindings{.h,.cpp}: should use consistent argument naming
https://bugs.webkit.org/show_bug.cgi?id=56898

Address FIXME I noted/added in r81676. Wanted to just copy/paste the checking
code from elsewhere in the file, but inconsistent argument naming made it more
difficult than it need be, so first step: use consistent naming.

  • public/WebBindings.h:
  • src/WebBindings.cpp: (WebKit::WebBindings::construct): (WebKit::WebBindings::enumerate): (WebKit::WebBindings::evaluate): (WebKit::WebBindings::evaluateHelper): (WebKit::WebBindings::getProperty): (WebKit::WebBindings::hasMethod): (WebKit::WebBindings::hasProperty): (WebKit::WebBindings::invoke): (WebKit::WebBindings::invokeDefault): (WebKit::WebBindings::releaseObject): (WebKit::WebBindings::removeProperty): (WebKit::WebBindings::retainObject): (WebKit::WebBindings::setException): (WebKit::WebBindings::setProperty): (WebKit::WebBindings::unregisterObject): (WebKit::WebBindings::extractIdentifierData): (WebKit::getRangeImpl): (WebKit::getElementImpl): (WebKit::makeIntArrayImpl): (WebKit::WebBindings::getRange): (WebKit::WebBindings::makeIntArray):
9:02 PM Changeset in webkit [81853] by Lucas Forschler
  • 5 edits in branches/safari-534.26-branch/Source

Versioning.

8:55 PM Changeset in webkit [81852] by Lucas Forschler
  • 1 copy in tags/Safari-534.26.1

New tag.

8:48 PM Changeset in webkit [81851] by tkent@chromium.org
  • 8 edits
    2 adds in trunk

[Chromium] Force to make validation bubble DIV position:absolute
https://bugs.webkit.org/show_bug.cgi?id=56901

Reviewed by Dimitri Glazkov.

Source/WebCore:

Test: fast/forms/interactive-validation-crash-by-style-override.html

  • html/ValidationMessage.cpp:

(WebCore::ValidationMessage::buildBubbleTree): Add position:absolute

because we need to move the validation message to a good position.

  • rendering/RenderMenuList.cpp:

(WebCore::RenderMenuList::addChild):

LayoutTests:

  • fast/forms/interactive-validation-crash-by-style-override-expected.txt: Added.
  • fast/forms/interactive-validation-crash-by-style-override.html: Added.
  • platform/gtk/Skipped: Skip the new test because the interactive validation feature is not enabled.
  • platform/mac/Skipped: ditto.
  • platform/qt/Skipped: ditto.
  • platform/win/Skipped: ditto.
8:22 PM Changeset in webkit [81850] by ojan@chromium.org
  • 1 edit
    2 adds in trunk/LayoutTests

2011-03-23 Ojan Vafai <ojan@chromium.org>

Add failing result for WebKit2. All tests that require
focus fail on WebKit2. See https://bugs.webkit.org/show_bug.cgi?id=56988.

  • platform/mac-wk2/fast/css/pseudo-any-expected.txt: Added.
8:18 PM Changeset in webkit [81849] by commit-queue@webkit.org
  • 6 edits
    9 adds in trunk

2011-03-23 Mike Lawther <mikelawther@chromium.org>

Reviewed by Ojan Vafai.

flex/bison tokens and grammar for CSS calc
https://bugs.webkit.org/show_bug.cgi?id=54412

Only the 'errors' tests pass as ultimately intended right now. As of this
patch, none of the actual calculations are expected to work, since only
the lexing/grammar stage is present.

  • css3/calc/calc-errors-expected.txt: Added.
  • css3/calc/calc-errors.html: Added.
  • css3/calc/minmax-errors-expected.txt: Added.
  • css3/calc/minmax-errors.html: Added.
  • css3/calc/simple-calcs-expected.txt: Added.
  • css3/calc/simple-calcs.html: Added.
  • css3/calc/simple-minmax-expected.txt: Added.
  • css3/calc/simple-minmax.html: Added.

2011-03-23 Mike Lawther <mikelawther@chromium.org>

Reviewed by Ojan Vafai.

flex/bison tokens and grammar for CSS calc
https://bugs.webkit.org/show_bug.cgi?id=54412

Tests: css3/calc/calc-errors.html

css3/calc/minmax-errors.html
css3/calc/simple-calcs.html
css3/calc/simple-minmax.html

  • css/CSSGrammar.y:
  • css/CSSParserValues.cpp: (WebCore::CSSParserValueList::insertValueAt): (WebCore::CSSParserValueList::extend):
  • css/CSSParserValues.h:
  • css/tokenizer.flex:
8:10 PM Changeset in webkit [81848] by commit-queue@webkit.org
  • 9 edits in trunk

2011-03-23 Adam Klein <adamk@chromium.org>

Reviewed by David Levin.

Fix resolveLocalFileSystemURL (and sync version) error codes to match the spec
https://bugs.webkit.org/show_bug.cgi?id=56961

  • http/tests/filesystem/resolve-uri-expected.txt:
  • http/tests/filesystem/script-tests/resolve-uri.js: Added three tests to exercise SECURITY_ERR: (runWrongDomain): (runWrongPort): (runWrongScheme): And updated four tests to test error codes: (runBogusURL): (runWrongProtocol): (runNotEnoughSlashes): (runNotEnoughSlashes2):
  • http/tests/filesystem/workers/resolve-url-expected.txt:
  • http/tests/filesystem/workers/resolve-url-sync-expected.txt:
  • http/tests/filesystem/workers/script-tests/resolve-url-sync.js: Added three tests to exercise SECURITY_ERR: (runWrongDomain): (runWrongPort): (runWrongScheme): And updated four tests to test error codes: (runBogusURL): (runWrongProtocol): (runNotEnoughSlashes): (runNotEnoughSlashes2):

2011-03-23 Adam Klein <adamk@chromium.org>

Reviewed by David Levin.

Fix resolveLocalFileSystemURL (and sync version) error codes to match the spec
https://bugs.webkit.org/show_bug.cgi?id=56961

See error code listing in the Files & Directories spec:
http://dev.w3.org/2009/dap/file-system/file-dir-sys.html#widl-LocalFileSystemSync-resolveLocalFileSystemSyncURL

Note that the spec currently only specifies errors for the sync
version; I've used the same codes for the async version.

  • page/DOMWindow.cpp: (WebCore::DOMWindow::resolveLocalFileSystemURL):
  • workers/WorkerContext.cpp: (WebCore::WorkerContext::resolveLocalFileSystemURL): (WebCore::WorkerContext::resolveLocalFileSystemSyncURL):
7:53 PM Changeset in webkit [81847] by commit-queue@webkit.org
  • 30 edits
    5 adds in trunk

2011-03-23 Jia Pu <jpu@apple.com>

Reviewed by Darin Adler.

Hook up new AppKit autocorrection UI with WK2.
https://bugs.webkit.org/show_bug.cgi?id=56055
<rdar://problem/8947463>

Please see WebCore/ChangeLog for detail.

  • platform/mac-wk2/Skipped: Enable autocorrection tests for WK2.

2011-03-23 Jia Pu <jpu@apple.com>

Reviewed by Darin Adler.

Hook up new AppKit autocorrection UI with WK2.
https://bugs.webkit.org/show_bug.cgi?id=56055
<rdar://problem/8947463>

This patch is to enable WK2 to utilize autocorrection UI on Mac OS X. It contains following
major changes:

  1. All but one autocorrection related message is synchronous. Since dismissing autocorrection panel can potentially cause editing to occur, a synchronous dismissCorrectionPanelSoon() function is introduced to ensure all editing commands occur in correct order.
  1. Additional condition variable is needed to implement dismissCorrectionPanelSoon(). To improve maintainability, CorrectionPanel class is introduced to manage the internal state related to correction panel. This change is applied to both WK1 and WK2.
  1. EditorClient::isShowingCorrectionPanel() has been removed. The original purpose is to allow editor to know when to handle ESC key event. Now this is handled internally in AppKit, so EditorClient::isShowingCorrectionPanel() isn't necessary anymore.
  1. The Editor* argument in EditorClient::showCorrectionPanel() has been removed, since we can access object via WebView or WKView.
  • editing/Editor.cpp: (WebCore::Editor::markMisspellingsAfterTypingToWord): (WebCore::Editor::markAllMisspellingsAndBadGrammarInRanges): (WebCore::Editor::correctionPanelTimerFired): (WebCore::Editor::dismissCorrectionPanel): (WebCore::Editor::dismissCorrectionPanelSoon): (WebCore::Editor::applyAutocorrectionBeforeTypingIfAppropriate):
  • editing/Editor.h:
  • editing/EditorCommand.cpp: (WebCore::createCommandMap):
  • loader/EmptyClients.h: (WebCore::EmptyEditorClient::showCorrectionPanel): (WebCore::EmptyEditorClient::dismissCorrectionPanelSoon):
  • manual-tests/autocorrection/close-window-when-correction-is-shown.html: Added.
  • page/EditorClient.h:

2011-03-23 Jia Pu <jpu@apple.com>

Reviewed by Darin Adler.

Hook up new AppKit autocorrection UI with WK2.
https://bugs.webkit.org/show_bug.cgi?id=56055
<rdar://problem/8947463>

Please see WebCore/ChangeLog for detail.

  • WebKit.xcodeproj/project.pbxproj:

2011-03-23 Jia Pu <jpu@apple.com>

Reviewed by Darin Adler.

Hook up new AppKit autocorrection UI with WK2.
https://bugs.webkit.org/show_bug.cgi?id=56055
<rdar://problem/8947463>

Please see WebCore/ChangeLog for detail.

  • WebCoreSupport/CorrectionPanel.h: Added. (CorrectionPanel::isShowing):
  • WebCoreSupport/CorrectionPanel.mm: Added. (correctionBubbleType): (CorrectionPanel::CorrectionPanel): (CorrectionPanel::~CorrectionPanel): (CorrectionPanel::show): (CorrectionPanel::dismiss): (CorrectionPanel::dismissSoon): (CorrectionPanel::dismissInternal): (CorrectionPanel::recordAutocorrectionResponse):
  • WebCoreSupport/WebEditorClient.h:
  • WebCoreSupport/WebEditorClient.mm: (WebEditorClient::WebEditorClient): (WebEditorClient::showCorrectionPanel): (WebEditorClient::dismissCorrectionPanel): (WebEditorClient::dismissCorrectionPanelSoon): (WebEditorClient::recordAutocorrectionResponse):
  • WebView/WebView.mm: (-[WebView handleCorrectionPanelResult:]):
  • WebView/WebViewPrivate.h:

2011-03-23 Jia Pu <jpu@apple.com>

Reviewed by Darin Adler.

Hook up new AppKit autocorrection UI with WK2.
https://bugs.webkit.org/show_bug.cgi?id=56055
<rdar://problem/8947463>

Please see WebCore/ChangeLog for detail.

The calls to AppKit are implemented in PageClientImpl. Other changes are necessary for the
plumbing work.

  • UIProcess/API/mac/PageClientImpl.h:
  • UIProcess/API/mac/PageClientImpl.mm: (WebKit::PageClientImpl::showCorrectionPanel): (WebKit::PageClientImpl::dismissCorrectionPanel): (WebKit::PageClientImpl::dismissCorrectionPanelSoon): (WebKit::PageClientImpl::recordAutocorrectionResponse):
  • UIProcess/API/mac/WKView.mm: (-[WKView spellCheckerDocumentTag]): (-[WKView handleCorrectionPanelResult:]):
  • UIProcess/API/mac/WKViewPrivate.h:
  • UIProcess/PageClient.h:
  • UIProcess/WebPageProxy.cpp: (WebKit::WebPageProxy::didCommitLoadForFrame): (WebKit::WebPageProxy::showCorrectionPanel): (WebKit::WebPageProxy::dismissCorrectionPanel): (WebKit::WebPageProxy::dismissCorrectionPanelSoon): (WebKit::WebPageProxy::recordAutocorrectionResponse): (WebKit::WebPageProxy::handleCorrectionPanelResult):
  • UIProcess/WebPageProxy.h:
  • UIProcess/WebPageProxy.messages.in:
  • UIProcess/mac/CorrectionPanel.h: Added. (WebKit::CorrectionPanel::isShowing):
  • UIProcess/mac/CorrectionPanel.mm: Added. (correctionBubbleType): (WebKit::CorrectionPanel::CorrectionPanel): (WebKit::CorrectionPanel::~CorrectionPanel): (WebKit::CorrectionPanel::show): (WebKit::CorrectionPanel::dismiss): (WebKit::CorrectionPanel::dismissSoon): (WebKit::CorrectionPanel::dismissInternal): (WebKit::CorrectionPanel::recordAutocorrectionResponse):
  • WebKit2.xcodeproj/project.pbxproj:
  • WebProcess/WebCoreSupport/WebEditorClient.h:
  • WebProcess/WebCoreSupport/mac/WebEditorClientMac.mm: (WebKit::WebEditorClient::showCorrectionPanel): (WebKit::WebEditorClient::dismissCorrectionPanel): (WebKit::WebEditorClient::dismissCorrectionPanelSoon): (WebKit::WebEditorClient::recordAutocorrectionResponse):
  • WebProcess/WebPage/WebPage.cpp: (WebKit::WebPage::handleCorrectionPanelResult):
  • WebProcess/WebPage/WebPage.h:
  • WebProcess/WebPage/WebPage.messages.in:
7:43 PM Changeset in webkit [81846] by jcivelli@chromium.org
  • 8 edits
    7 adds in trunk/Source/WebKit/chromium

2011-03-23 Jay Civelli <jcivelli@chromium.org>

Reviewed by David Levin.

Relanding:
Moving the method that is used to retrieve all the resources in
a page from the Chromium code (dom_operations.cc) to WebKit.
https://bugs.webkit.org/show_bug.cgi?id=55859

  • WebKit.gyp:
  • public/WebPageSerializer.h:
  • public/WebURL.h: (WebKit::operator==): (WebKit::operator!=):
  • public/WebVector.h: (WebKit::WebVector::contains):
  • src/WebDataSourceImpl.cpp: (WebKit::WebDataSourceImpl::WebDataSourceImpl):
  • src/WebPageSerializer.cpp: (WebCore::getSubResourceURLFromElement): (WebCore::retrieveResourcesForElement): (WebCore::retrieveResourcesForFrame): (WebKit::WebPageSerializer::retrieveAllResources):
  • tests/WebFrameTest.cpp:
  • tests/WebPageSerializerTest.cpp: Added.
  • tests/data/pageserialization/awesome.png: Added.
  • tests/data/pageserialization/embed_iframe.html: Added.
  • tests/data/pageserialization/object_iframe.html: Added.
  • tests/data/pageserialization/simple_iframe.html: Added.
  • tests/data/pageserialization/simple_page.html: Added.
  • tests/data/pageserialization/top_frame.html: Added.
7:34 PM Changeset in webkit [81845] by ojan@chromium.org
  • 7 edits in trunk/Source/WebCore

2011-03-22 Ojan Vafai <ojan@chromium.org>

Reviewed by Antti Koivisto.

move :not over to using selectorList instead of simpleSelector
https://bugs.webkit.org/show_bug.cgi?id=56894

Saves memory and simplifies code.

No new tests since existing tests cover this code.

  • css/CSSGrammar.y: Also removed extranenous calls to updateLastSelectorLineAndPosition in simple_selector_list. These happened to work, but are only actually needed in selector_list.
  • css/CSSParserValues.h:
  • css/CSSSelector.cpp: (WebCore::CSSSelector::specificityForOneSelector): Removed null-check. The parser null-checks, I don't see why we need to here. (WebCore::CSSSelector::selectorText): Ditto. (WebCore::CSSSelector::setArgument): (WebCore::CSSSelector::isSimple):
  • css/CSSSelector.h:
  • css/CSSSelectorList.cpp: (WebCore::forEachTagSelector):
  • css/CSSStyleSelector.cpp: (WebCore::CSSStyleSelector::SelectorChecker::checkOneSelector): (WebCore::collectFeaturesFromSelector): (WebCore::collectFeaturesFromList):
7:25 PM Changeset in webkit [81844] by carol.szabo@nokia.com
  • 5 edits
    2 adds in trunk

2011-03-23 Carol Szabo <carol.szabo@nokia.com>

Reviewed by David Hyatt.

This test verifies the correct update of a second renderer that
displays the same CounterNode.

Assertion and incorrect rendering of counters.
https://bugs.webkit.org/show_bug.cgi?id=56896

  • fast/css/counters/2displays-expected.txt: Added.
  • fast/css/counters/2displays.html: Added.

2011-03-23 Carol Szabo <carol.szabo@nokia.com>

Reviewed by David Hyatt.

Made sure that renderers displaying counters are invalidated upon
counter destruction.

Assertion and incorrect rendering of counters.
https://bugs.webkit.org/show_bug.cgi?id=56896

Test: fast/css/counters/2displays.html

  • rendering/CounterNode.cpp: (WebCore::CounterNode::~CounterNode): Added to make sure that its display renderers are always reset when the node is deleted. (WebCore::CounterNode::resetRenderers): Fixed bug that would prevent reset of second and subsequent display renderers.
  • rendering/CounterNode.h:
  • rendering/RenderCounter.cpp: (WebCore::destroyCounterNodeWithoutMapRemoval): removed unnecessary calls to resetRenderers() as the CounterNode destructor takes care of that now.
7:03 PM Changeset in webkit [81843] by Lucas Forschler
  • 2 edits in branches/safari-534.26-branch/Source/WebCore

Merge r81832.

7:02 PM Changeset in webkit [81842] by Lucas Forschler
  • 2 edits in branches/safari-534.26-branch/Source/WebCore

Merge r81831.

6:59 PM Changeset in webkit [81841] by Lucas Forschler
  • 2 edits in branches/safari-534.26-branch/Source/WebCore

Merge r81745.

6:57 PM Changeset in webkit [81840] by Lucas Forschler
  • 2 edits in branches/safari-534.26-branch/Source/WebKit2

Merge r81680.

6:54 PM Changeset in webkit [81839] by Lucas Forschler
  • 4 edits in branches/safari-534.26-branch/Source/WebKit2

Merge r81622.

6:54 PM Changeset in webkit [81838] by commit-queue@webkit.org
  • 2 edits in trunk/Tools

2011-03-23 Leandro Gracia Gil <leandrogracia@chromium.org>

Reviewed by Darin Adler.

Media Stream API: fix the order of the options in build-webkit.
https://bugs.webkit.org/show_bug.cgi?id=56949

Fix the order of the options in build-webkit.

  • Scripts/build-webkit:
6:52 PM Changeset in webkit [81837] by Lucas Forschler
  • 18 edits in branches/safari-534.26-branch

Merge r81600.

6:44 PM Changeset in webkit [81836] by bweinstein@apple.com
  • 4 edits in trunk/Source

WebKit2: Need API to manage the Media Cache
https://bugs.webkit.org/show_bug.cgi?id=56878
<rdar://problem/9082503>

Reviewed by Maciej Stachowiak.

Source/WebCore:

Add functions that need to be exported.

  • WebCore.exp.in:

Source/WebKit2:

Call through to HTMLMediaElement functions to manage the WebCore media cache.

  • WebProcess/MediaCache/WebMediaCacheManager.cpp:

(WebKit::WebMediaCacheManager::getHostnamesWithMediaCache):
(WebKit::WebMediaCacheManager::clearCacheForHostname):
(WebKit::WebMediaCacheManager::clearCacheForAllHostnames):

6:33 PM Changeset in webkit [81835] by commit-queue@webkit.org
  • 3 edits in trunk/Source/WebCore

2011-03-23 Robert Kroeger <rjkroege@chromium.org>

Reviewed by James Robinson.

Correct use of ENABLE() Macro

In http://trac.webkit.org/changeset/81618, I mis-used the ENABLE()
macro. This patch corrects.

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

  • page/EventHandler.cpp: (WebCore::EventHandler::EventHandler): (WebCore::EventHandler::handleTouchEvent):
  • page/EventHandler.h:
6:13 PM Changeset in webkit [81834] by commit-queue@webkit.org
  • 3 edits in trunk/Tools

2011-03-23 Adam Klein <adamk@chromium.org>

Reviewed by Mihai Parparita.

Add an option to new-run-webkit-httpd to allow overriding the location of LayoutTests directory
https://bugs.webkit.org/show_bug.cgi?id=56884

This option will be utilized by Chromium's ui_tests to allow
referencing js-test-resources from a layout test over HTTP.

  • Scripts/new-run-webkit-httpd: Added --layout_tests_dir option.
  • Scripts/webkitpy/layout_tests/port/http_server.py: Plumbed through as layout_tests_dir param.
5:58 PM Changeset in webkit [81833] by jer.noble@apple.com
  • 3 edits in trunk/Source/WebCore

2011-03-23 Jer Noble <jer.noble@apple.com>

Reviewed by Simon Fraser.

Scrubbing <video> with HTTP Live Stream resizes element to 0x0
https://bugs.webkit.org/show_bug.cgi?id=55702

QTKit will occasionally set the natural size of a QTMovie to 0x0 while scrubbing
an HTTP Live Stream. So we will cache the last valid value returned by QTKit
and use that as our naturalSize until a new valid value is returned.

Unfortunately, QTKit will also fail to generate a notification when the natural size
changes, so we are forced to cache the natural size from within naturalSize(), which
is a const function, necessitating a const_cast to set m_cachedNaturalSize.

  • platform/graphics/mac/MediaPlayerPrivateQTKit.h: Added m_cachedNaturalSize.
  • platform/graphics/mac/MediaPlayerPrivateQTKit.mm: (WebCore::MediaPlayerPrivateQTKit::MediaPlayerPrivateQTKit): Initialize m_cachedNaturalSize. (WebCore::MediaPlayerPrivateQTKit::naturalSize):
5:50 PM Changeset in webkit [81832] by ap@apple.com
  • 2 edits in trunk/Source/WebCore

Build fix.

  • loader/FrameLoader.cpp: (WebCore::FrameLoader::checkLoadCompleteForThisFrame): Removed a stray parenthesis.
5:48 PM Changeset in webkit [81831] by ap@apple.com
  • 2 edits in trunk/Source/WebCore

Reviewed by Maciej Stachowiak.

REGRESSION: Infinite recursion in recursiveCheckLoadComplete()/checkLoadCompleteForThisFrame()/stopLoading()
https://bugs.webkit.org/show_bug.cgi?id=56978
<rdar://problem/9041670>

No new tests, because there is no known way to reproduce.

Removed strange code that was trying to stop a non-loading provisional document loader and
even its subframes (?!). Added assertions to catch it if we can actually be loading here in
some sense.
Rewrote a condition for clarity - starting with r43148, it was acting as an opposite of a
recursion guard (but that didn't cause immediate problems then).

  • loader/FrameLoader.cpp: (WebCore::FrameLoader::checkLoadCompleteForThisFrame):
5:38 PM Changeset in webkit [81830] by beidson@apple.com
  • 2 edits in trunk/Source/WebCore

Attempt to fix the build for platforms that have the icon database
disabled but still provide settings API for it...?

  • loader/icon/IconDatabase.h:

(WebCore::IconDatabase::defaultDatabaseFilename):

5:21 PM Changeset in webkit [81829] by kbr@google.com
  • 2 edits in trunk/Source/WebKit/chromium

2011-03-23 Kenneth Russell <kbr@google.com>

Rolled forward Chromium DEPS again to fix DumpRenderTree build
failure caused by r81821.

  • DEPS:
5:21 PM Changeset in webkit [81828] by Lucas Forschler
  • 5 edits in branches/safari-534.26-branch/Source

Versioning.

5:21 PM Changeset in webkit [81827] by enrica@apple.com
  • 2 edits in trunk/Source/WebKit/mac

Services menu item to convert selected Simplified/Traditional Chinese Text is not working.
https://bugs.webkit.org/show_bug.cgi?id=56974
<rdar://problem/8284642>

Reviewed by Alexey Proskuryakov.

This fixes a regression introduced by http://trac.webkit.org/changeset/61794
where readSelectionFromPasteboard was accidentally removed.
This method and writeSelectionToPasteboard are needed to support Mac OS X
services.

  • WebView/WebHTMLView.mm:

(-[WebHTMLView _pasteAsPlainTextWithPasteboard:]): Added.
(-[WebHTMLView readSelectionFromPasteboard:]): Added.

5:20 PM Changeset in webkit [81826] by mrowe@apple.com
  • 2 edits in trunk/Source/WebCore

Fix the build.

  • WebCore.exp.in: Export the new version of IconDatabase::open.
5:06 PM Changeset in webkit [81825] by mrowe@apple.com
  • 2 edits in trunk/Source/JavaScriptCore

<rdar://problem/7959320> Threads that use APIs above the BSD layer must be registered with the Obj-C GC.

Reviewed by Darin Adler.

  • wtf/ThreadingPthreads.cpp:

(WTF::initializeCurrentThreadInternal):

4:56 PM Changeset in webkit [81824] by beidson@apple.com
  • 17 edits in trunk/Source

Change IconDatabase opening to allow for arbitrary filenames
https://bugs.webkit.org/show_bug.cgi?id=56977

Reviewed by Sam Weinig.

../WebCore:

  • WebCore.exp.in:
  • loader/icon/IconDatabase.cpp:

(WebCore::IconDatabase::open):

  • loader/icon/IconDatabase.h:
  • loader/icon/IconDatabaseBase.cpp:

(WebCore::IconDatabaseBase::open):

  • loader/icon/IconDatabaseBase.h:
  • loader/icon/IconDatabaseClient.h:

../WebKit/efl:

  • ewk/ewk_settings.cpp:

(ewk_settings_icon_database_path_set):

../WebKit/gtk:

  • webkit/webkiticondatabase.cpp:

(webkit_icon_database_set_path):

../WebKit/mac:

  • Misc/WebIconDatabase.mm:

(-[WebIconDatabase _startUpIconDatabase]):

../WebKit/qt:

  • Api/qwebsettings.cpp:

(QWebSettings::setIconDatabasePath):

../WebKit/win:

  • WebIconDatabase.cpp:

(WebIconDatabase::startUpIconDatabase):

4:56 PM Changeset in webkit [81823] by kbr@google.com
  • 2 edits in trunk/Source/WebKit/chromium

2011-03-23 Kenneth Russell <kbr@google.com>

Rolled forward Chromium DEPS to fix build breakage after r81821.

  • DEPS:
4:48 PM Changeset in webkit [81822] by xji@chromium.org
  • 2 edits
    2 adds in trunk/LayoutTests

2011-03-23 Xiaomei Ji <xji@chromium.org>

Rebaseline chromium-mac after r81807.

  • platform/chromium-mac/fast/css/bidi-override-in-anonymous-block-expected.checksum: Added.
  • platform/chromium-mac/fast/css/bidi-override-in-anonymous-block-expected.png: Added.
  • platform/chromium/test_expectations.txt:
4:24 PM Changeset in webkit [81821] by kbr@google.com
  • 7 edits in trunk/Source

2011-03-23 Kenneth Russell <kbr@google.com>

Reviewed by James Robinson.

[chromium] WebGLRenderingContext and other DOMWindow properties should remain if WebGL is disabled
https://bugs.webkit.org/show_bug.cgi?id=56947

Removed the WebRuntimeFeatures setting for WebGL. WebGL support is
now controlled exclusively through the page's Settings.

  • public/WebRuntimeFeatures.h:
  • src/WebRuntimeFeatures.cpp:

2011-03-23 Kenneth Russell <kbr@google.com>

Reviewed by James Robinson.

[chromium] WebGLRenderingContext and other DOMWindow properties should remain if WebGL is disabled
https://bugs.webkit.org/show_bug.cgi?id=56947

Removed the RuntimeEnabledFeatures setting for WebGL and the
EnabledAtRuntime extended attribute from the WebGL-related
properties on the DOMWindow. WebGL support is now controlled
exclusively through the page's Settings.

No new tests; tested manually in Chromium with --disable-webgl
command line argument and blacklist.

  • bindings/generic/RuntimeEnabledFeatures.cpp:
  • bindings/generic/RuntimeEnabledFeatures.h:
  • page/DOMWindow.idl:
4:23 PM Changeset in webkit [81820] by xji@chromium.org
  • 2 edits
    7 adds in trunk/LayoutTests

2011-03-23 Xiaomei Ji <xji@chromium.org>

Rebaseline after r81807.

  • platform/chromium-linux/fast/css/bidi-override-in-anonymous-block-expected.checksum: Added.
  • platform/chromium-linux/fast/css/bidi-override-in-anonymous-block-expected.png: Added.
  • platform/chromium-win/fast/css/bidi-override-in-anonymous-block-expected.checksum: Added.
  • platform/chromium-win/fast/css/bidi-override-in-anonymous-block-expected.png: Added.
  • platform/chromium-win/fast/css/bidi-override-in-anonymous-block-expected.txt: Added.
  • platform/chromium/test_expectations.txt:
  • platform/gtk/fast/css/bidi-override-in-anonymous-block-expected.txt: Added.
  • platform/qt/fast/css/bidi-override-in-anonymous-block-expected.txt: Added.
4:19 PM Changeset in webkit [81819] by jeffm@apple.com
  • 8 edits in trunk/Source/WebKit2

2011-03-23 Jeff Miller <jeffm@apple.com>

Reviewed by Darin Adler.

Add WKBundleSetHostAllowsAnyHTTPSCertificate() for Windows
https://bugs.webkit.org/show_bug.cgi?id=56972


This is the WebKit2 equivalent to the WebKit1 API IWebMutableURLRequest::setAllowsAnyHTTPSCertificate().

  • WebProcess/InjectedBundle/API/c/WKBundle.cpp: (WKBundleSetHostAllowsAnyHTTPSCertificate): Added.
  • WebProcess/InjectedBundle/API/c/WKBundlePrivate.h: Added WKBundleSetHostAllowsAnyHTTPSCertificate().
  • WebProcess/InjectedBundle/InjectedBundle.h: Added setHostAllowsAnyHTTPSCertificate().
  • WebProcess/InjectedBundle/gtk/InjectedBundleGtk.cpp: (WebKit::InjectedBundle::setHostAllowsAnyHTTPSCertificate): Added, should never be called.
  • WebProcess/InjectedBundle/mac/InjectedBundleMac.cpp: (WebKit::InjectedBundle::setHostAllowsAnyHTTPSCertificate): Added, should never be called.
  • WebProcess/InjectedBundle/qt/InjectedBundleQt.cpp: (WebKit::InjectedBundle::setHostAllowsAnyHTTPSCertificate): Added, should never be called.
  • WebProcess/InjectedBundle/win/InjectedBundleWin.cpp: (WebKit::InjectedBundle::setHostAllowsAnyHTTPSCertificate): Added.
4:01 PM Changeset in webkit [81818] by mihaip@chromium.org
  • 4 edits
    2 copies
    1 add in trunk/LayoutTests

2011-03-23 Mihai Parparita <mihaip@chromium.org>

Rebaseline transitions/default-timing-function.html for Chromium Snow
Leopard.

  • platform/chromium-mac-leopard/transitions/default-timing-function-expected.checksum: Copied from LayoutTests/platform/chromium-mac/transitions/default-timing-function-expected.checksum.
  • platform/chromium-mac-leopard/transitions/default-timing-function-expected.png: Copied from LayoutTests/platform/chromium-mac/transitions/default-timing-function-expected.png.
  • platform/chromium-mac/transitions/default-timing-function-expected.checksum:
  • platform/chromium-mac/transitions/default-timing-function-expected.png:
  • platform/chromium/test_expectations.txt:
3:41 PM Changeset in webkit [81817] by Martin Robinson
  • 2 edits in trunk/LayoutTests

2011-03-23 Martin Robinson <mrobinson@igalia.com>

[GTK] fast/overflow/horizontal-scroll-after-back.html fails
https://bugs.webkit.org/show_bug.cgi?id=56966

  • platform/gtk/Skipped: Skip this test on GTK+ until the fix is in.
3:39 PM Changeset in webkit [81816] by hyatt@apple.com
  • 12 edits in trunk/Source/WebCore

Unreviewed, rolling out r81802.
http://trac.webkit.org/changeset/81802
https://bugs.webkit.org/show_bug.cgi?id=56963

Patch by Sheriff Bot <webkit.review.bot@gmail.com> on 2011-03-23
Broke a layout test, and this fix is not trivial. (Requested
by dhyatt on #webkit).

  • rendering/RenderBlock.cpp:

(WebCore::RenderBlock::layoutBlock):
(WebCore::RenderBlock::layoutOnlyPositionedObjects):

  • rendering/RenderBlock.h:
  • rendering/RenderBox.cpp:

(WebCore::RenderBox::styleDidChange):

  • rendering/RenderBoxModelObject.cpp:

(WebCore::RenderBoxModelObject::styleWillChange):

  • rendering/RenderFlexibleBox.cpp:

(WebCore::RenderFlexibleBox::layoutBlock):

  • rendering/RenderObject.cpp:

(WebCore::RenderObject::RenderObject):
(WebCore::RenderObject::adjustStyleDifference):
(WebCore::RenderObject::setStyle):
(WebCore::RenderObject::styleDidChange):

  • rendering/RenderObject.h:

(WebCore::RenderObject::needsLayout):
(WebCore::RenderObject::needsPositionedMovementLayoutOnly):
(WebCore::RenderObject::setNeedsLayout):
(WebCore::RenderObject::setChildNeedsLayout):
(WebCore::RenderObject::markContainingBlocksForLayout):

  • rendering/RenderTable.cpp:

(WebCore::RenderTable::layout):

  • rendering/style/RenderStyle.cpp:

(WebCore::RenderStyle::diff):

  • rendering/style/RenderStyleConstants.h:
  • rendering/svg/RenderSVGText.cpp:

(WebCore::RenderSVGText::layout):

3:34 PM Changeset in webkit [81815] by dpranke@chromium.org
  • 5 edits in trunk/Tools

2011-03-23 Dirk Pranke <dpranke@chromium.org>

Reviewed by Mihai Parparita.

new-run-webkit-tests: run tests in ascending alphabetical order per
dir. This is a second attempt at the fix; the first attempt, in
r81597, broke the behavior on the old-inline and old-threads
worker models.

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

  • Scripts/webkitpy/layout_tests/layout_package/dump_render_tree_thread.py:
  • Scripts/webkitpy/layout_tests/layout_package/test_runner.py:
  • Scripts/webkitpy/layout_tests/port/test.py:
  • Scripts/webkitpy/layout_tests/run_webkit_tests_unittest.py:
3:29 PM Changeset in webkit [81814] by weinig@apple.com
  • 10 edits in trunk/Source/WebKit2

Roll out r81593 Need WebKit2 API for creating a page with a specific main frame name

Reviewed by Anders Carlsson.

This API is not necessary.

  • Shared/WebPageCreationParameters.cpp:

(WebKit::WebPageCreationParameters::encode):
(WebKit::WebPageCreationParameters::decode):

  • Shared/WebPageCreationParameters.h:
  • UIProcess/API/mac/WKView.h:
  • UIProcess/API/mac/WKView.mm:

(-[WKView initWithFrame:]):
(-[WKView initWithFrame:contextRef:]):
(-[WKView initWithFrame:contextRef:pageGroupRef:]):

  • UIProcess/WebPageProxy.cpp:

(WebKit::WebPageProxy::creationParameters):

  • UIProcess/WebPageProxy.h:
  • WebProcess/WebPage/WebFrame.cpp:

(WebKit::WebFrame::createMainFrame):

  • WebProcess/WebPage/WebFrame.h:
  • WebProcess/WebPage/WebPage.cpp:

(WebKit::WebPage::WebPage):

3:23 PM Changeset in webkit [81813] by mihaip@chromium.org
  • 7 edits
    2 copies in trunk/LayoutTests

2011-03-23 Mihai Parparita <mihaip@chromium.org>

Update expectations for one SVG and two media tests for Chromium Snow
Leopard.

  • platform/chromium-mac-leopard/svg/W3C-SVG-1.1-SE/types-dom-01-b-expected.checksum: Copied from LayoutTests/platform/chromium-mac/svg/W3C-SVG-1.1-SE/types-dom-01-b-expected.checksum.
  • platform/chromium-mac-leopard/svg/W3C-SVG-1.1-SE/types-dom-01-b-expected.png: Copied from LayoutTests/platform/chromium-mac/svg/W3C-SVG-1.1-SE/types-dom-01-b-expected.png.
  • platform/chromium-mac/media/controls-after-reload-expected.checksum:
  • platform/chromium-mac/media/controls-after-reload-expected.png:
  • platform/chromium-mac/media/controls-without-preload-expected.checksum:
  • platform/chromium-mac/media/controls-without-preload-expected.png:
  • platform/chromium-mac/svg/W3C-SVG-1.1-SE/types-dom-01-b-expected.checksum:
  • platform/chromium-mac/svg/W3C-SVG-1.1-SE/types-dom-01-b-expected.png:
3:04 PM Changeset in webkit [81812] by commit-queue@webkit.org
  • 4 edits
    2 adds in trunk

2011-03-23 Luiz Agostini <luiz.agostini@openbossa.org>

Reviewed by David Hyatt.

REGRESSION(81035): crash in RenderDetails::removeChild
https://bugs.webkit.org/show_bug.cgi?id=56773

  • fast/html/details-children-merge-crash-expected.txt: Added.
  • fast/html/details-children-merge-crash.html: Added.

2011-03-23 Luiz Agostini <luiz.agostini@openbossa.org>

Reviewed by David Hyatt.

REGRESSION(81035): crash in RenderDetails::removeChild
https://bugs.webkit.org/show_bug.cgi?id=56773

Test: fast/html/details-children-merge-crash.html

Preventing merging of RenderDetails's anonymous children.

  • rendering/RenderBlock.cpp: (WebCore::canMergeContiguousAnonymousBlocks):

Fixing RenderDetails::removeChild and cleaning m_marker in RenderDetails::destroy.

  • rendering/RenderDetails.cpp: (WebCore::RenderDetails::destroy): (WebCore::RenderDetails::removeChild):
3:03 PM Changeset in webkit [81811] by kbalazs@webkit.org
  • 2 edits in trunk/Source/WebKit2

2011-03-23 Balazs Kelemen <kbalazs@webkit.org>

Reviewed by Andreas Kling.

[WK2] Handle keyboard and mouse events on X11
https://bugs.webkit.org/show_bug.cgi?id=56103

Propagate keyboard and mouse events to the plugin
as XEvent's.

  • WebProcess/Plugins/Netscape/x11/NetscapePluginX11.cpp: (WebKit::initializeXEvent): (WebKit::xTimeStamp): (WebKit::xKeyModifiers): (WebKit::setCommonMouseEventFields): (WebKit::setXMotionEventFields): (WebKit::setXButtonEventFields): (WebKit::setXCrossingEventFields): (WebKit::NetscapePlugin::platformHandleMouseEvent): (WebKit::NetscapePlugin::platformHandleMouseEnterEvent): (WebKit::NetscapePlugin::platformHandleMouseLeaveEvent): (WebKit::setXKeyEventFields): (WebKit::NetscapePlugin::platformHandleKeyboardEvent):
3:02 PM Changeset in webkit [81810] by mario@webkit.org
  • 2 edits in trunk/Tools

2011-03-23 Mario Sanchez Prada <msanchez@igalia.com>

Reviewed by Martin Robinson.

[GTK] GTK's DRT to ouput detail's string for the 'property-change' signal
https://bugs.webkit.org/show_bug.cgi?id=56953

Print the string for the detail of 'property-change' signal.

  • DumpRenderTree/gtk/AccessibilityCallbacks.cpp: (axObjectEventListener): Print the detail string.
2:46 PM Changeset in webkit [81809] by mrowe@apple.com
  • 1 copy in branches/safari-534.26-branch

New branch.

2:36 PM Changeset in webkit [81808] by xji@chromium.org
  • 2 edits in trunk/LayoutTests

2011-03-23 Xiaomei Ji <xji@chromium.org>

Add fast/css/bidi-override-in-anonymous-block.html as expected to fail after r81807.

  • platform/chromium/test_expectations.txt:
2:27 PM Changeset in webkit [81807] by xji@chromium.org
  • 7 edits
    2 adds in trunk

2011-03-23 Xiaomei Ji <xji@chromium.org>

Reviewed by David Hyatt.

Preserve unicode-bidi:bidi-override in anonymous block.
https://bugs.webkit.org/show_bug.cgi?id=21440

unicode-bidi:bidi-override should be preserved in anonymous block
according to http://www.w3.org/TR/CSS21/visuren.html#propdef-unicode-bidi.

Introduce a helper function RenderStyle()::createAnonymousStyle() which
creates a default RenderStyle, inherits the inherited style from parent,
and inherit unicode-bidi:bidi-override from parent.
And replace the style creation when anonymous block is created.

Note: the TABLE releated anonymous blocks are untouched, including:
TABLE, TABLE_CELL, TABLE_ROW, TABLE_ROW_GROUP anonymous blocks created in
RenderObject, RenderTable, RenderTableRow, and RenderTableSection,
due to the expected behavior is not clear.
Please refer to https://bugs.webkit.org/show_bug.cgi?id=56594.


Test: fast/css/bidi-override-in-anonymous-block.html

  • rendering/RenderBlock.cpp: (WebCore::RenderBlock::styleDidChange): (WebCore::RenderBlock::removeChild): (WebCore::RenderBlock::createAnonymousBlock): (WebCore::RenderBlock::createAnonymousColumnsBlock): (WebCore::RenderBlock::createAnonymousColumnSpanBlock):
  • rendering/RenderInline.cpp: (WebCore::RenderInline::addChildIgnoringContinuation):
  • rendering/RenderRubyRun.cpp: (WebCore::RenderRubyRun::createRubyBase): (WebCore::RenderRubyRun::staticCreateRubyRun):
  • rendering/style/RenderStyle.cpp: (WebCore::RenderStyle::createAnonymousStyle):
  • rendering/style/RenderStyle.h:

2011-03-23 Xiaomei Ji <xji@chromium.org>

Reviewed by David Hyatt.

Preserve unicode-bidi:bidi-override in anonymous block.
https://bugs.webkit.org/show_bug.cgi?id=21440

  • fast/css/bidi-override-in-anonymous-block.html: Added.
  • platform/mac/fast/css/bidi-override-in-anonymous-block-expected.txt: Added.
2:24 PM Changeset in webkit [81806] by mihaip@chromium.org
  • 2 edits in trunk/LayoutTests

2011-03-23 Mihai Parparita <mihaip@chromium.org>

Group Snow Leopard failures for bug 56962.

  • platform/chromium/test_expectations.txt:
2:21 PM Changeset in webkit [81805] by mrowe@apple.com
  • 2 edits in trunk/Source/JavaScriptCore

Stop setting OTHER_OPTIONS in JavaScriptCore's Makefile.

It's not necessary to pass "-target All" as xcodebuild always builds the
first target in the project unless otherwise specified. The presence of
that option also breaks "make clean" since that results in both the
-target and -alltargets options being passed to xcodebuild.

  • Makefile:
2:06 PM Changeset in webkit [81804] by inferno@chromium.org
  • 2 edits in trunk/LayoutTests

2011-03-23 Abhishek Arya <inferno@chromium.org>

Unreviewed. Replace shockwave plugin with test netscape in recent
timing out test plugins/change-widget-and-click-crash.html.

  • plugins/change-widget-and-click-crash.html:
2:05 PM Changeset in webkit [81803] by commit-queue@webkit.org
  • 2 edits in trunk

2011-03-23 George Guo <George.Guo@nokia.com>

Reviewed by Darin Adler.

Webkit warning: converting to 'unsigned int' from 'double'
https://bugs.webkit.org/show_bug.cgi?id=56865

With -Werr enabled, this cause the build failure on RedHat Linux.
Add a cast helped.

  • Source/JavaScriptCore/runtime/TimeoutChecker.cpp: (JSC::getCPUTime):
1:39 PM Changeset in webkit [81802] by hyatt@apple.com
  • 12 edits in trunk/Source/WebCore

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

Reviewed by Dan Bernstein.

Add a simplified normal flow layout path optimization for overflow recomputation
and for positioned objects inside relative positioned containers.

Currently there is an optimized code path for positioned objects, but as soon as
we encounter a normal flow object in the containing block chain, we lose the
optimization.

This patch adds a new type of style difference called SimplifiedLayout that is
returned when only overflow needs to be recomputed. Whenever opacity changes or
a transform changes, this is the hint returned now instead of a full layout.

In addition, when positioned objects need layout and start marking up the
containing block chain, we now propagate the fact that the layout is simplified
all the way up to the root, even when we encounter normal flow containing
blocks.

The layoutOnlyPositionedObjects function has been renamed to simplifiedLayout()
and is now used for all of these cases (in addition to what it handled before).

No new tests, since existing tests covered this very well (especially the opacity
and transforms repaint tests in fast/repaint).

  • rendering/RenderBlock.cpp:

(WebCore::RenderBlock::layoutBlock):
(WebCore::RenderBlock::simplifiedNormalFlowLayout):
(WebCore::RenderBlock::simplifiedLayout):

  • rendering/RenderBlock.h:
  • rendering/RenderBox.cpp:

(WebCore::RenderBox::styleDidChange):

  • rendering/RenderBoxModelObject.cpp:

(WebCore::RenderBoxModelObject::styleWillChange):

  • rendering/RenderFlexibleBox.cpp:

(WebCore::RenderFlexibleBox::layoutBlock):

  • rendering/RenderObject.cpp:

(WebCore::RenderObject::RenderObject):
(WebCore::RenderObject::adjustStyleDifference):
(WebCore::RenderObject::setStyle):
(WebCore::RenderObject::styleDidChange):

  • rendering/RenderObject.h:

(WebCore::RenderObject::needsLayout):
(WebCore::RenderObject::needsPositionedMovementLayoutOnly):
(WebCore::RenderObject::needsSimplifiedNormalFlowLayout):
(WebCore::RenderObject::setNeedsLayout):
(WebCore::RenderObject::setChildNeedsLayout):
(WebCore::RenderObject::setNeedsSimplifiedNormalFlowLayout):
(WebCore::RenderObject::markContainingBlocksForLayout):

  • rendering/RenderTable.cpp:

(WebCore::RenderTable::layout):

  • rendering/style/RenderStyle.cpp:

(WebCore::RenderStyle::diff):

  • rendering/style/RenderStyleConstants.h:
  • rendering/svg/RenderSVGText.cpp:

(WebCore::RenderSVGText::layout):

1:34 PM Changeset in webkit [81801] by mihaip@chromium.org
  • 19 edits
    4 copies
    4 moves
    1 add
    4 deletes in trunk/LayoutTests

2011-03-23 Mihai Parparita <mihaip@chromium.org>

Fix more tests for Chromium Snow Leopard by promoting chromium-mac/
results to mac-leopard/ where possible.

  • platform/chromium-mac-leopard/fast/backgrounds/background-position-parsing-expected.checksum: Copied from LayoutTests/platform/chromium-mac/fast/backgrounds/background-position-parsing-expected.checksum.
  • platform/chromium-mac-leopard/fast/backgrounds/background-position-parsing-expected.png: Copied from LayoutTests/platform/chromium-mac/fast/backgrounds/background-position-parsing-expected.png.
  • platform/chromium-mac-leopard/fast/box-shadow/box-shadow-transformed-expected.checksum: Copied from LayoutTests/platform/chromium-mac/fast/box-shadow/box-shadow-transformed-expected.checksum.
  • platform/chromium-mac-leopard/fast/box-shadow/box-shadow-transformed-expected.png: Copied from LayoutTests/platform/chromium-mac/fast/box-shadow/box-shadow-transformed-expected.png.
  • platform/chromium-mac/css1/text_properties/text_indent-expected.checksum: Removed.
  • platform/chromium-mac/css1/text_properties/text_indent-expected.png: Removed.
  • platform/chromium-mac/css1/units/length_units-expected.checksum: Removed.
  • platform/chromium-mac/css1/units/length_units-expected.png: Removed.
  • platform/chromium-mac/fast/backgrounds/background-position-parsing-expected.checksum:
  • platform/chromium-mac/fast/backgrounds/background-position-parsing-expected.png:
  • platform/chromium-mac/fast/box-shadow/box-shadow-transformed-expected.checksum:
  • platform/chromium-mac/fast/box-shadow/box-shadow-transformed-expected.png:
  • platform/chromium-mac/media/media-document-audio-repaint-expected.checksum:
  • platform/chromium-mac/media/media-document-audio-repaint-expected.png:
  • platform/mac-leopard/css1/text_properties/text_indent-expected.checksum:
  • platform/mac-leopard/css1/text_properties/text_indent-expected.png:
  • platform/mac-leopard/css1/units/length_units-expected.checksum:
  • platform/mac-leopard/css1/units/length_units-expected.png:
  • platform/mac-leopard/fast/css-generated-content/table-row-before-after-expected.checksum: Renamed from LayoutTests/platform/chromium-mac/fast/css-generated-content/table-row-before-after-expected.checksum.
  • platform/mac-leopard/fast/css-generated-content/table-row-before-after-expected.png: Renamed from LayoutTests/platform/chromium-mac/fast/css-generated-content/table-row-before-after-expected.png.
  • platform/mac-leopard/fast/multicol/scrolling-overflow-expected.checksum: Renamed from LayoutTests/platform/chromium-mac/fast/multicol/scrolling-overflow-expected.checksum.
  • platform/mac-leopard/fast/multicol/scrolling-overflow-expected.png: Renamed from LayoutTests/platform/chromium-mac/fast/multicol/scrolling-overflow-expected.png.
  • platform/mac/css1/text_properties/text_indent-expected.checksum:
  • platform/mac/css1/text_properties/text_indent-expected.png:
  • platform/mac/css1/units/length_units-expected.checksum:
  • platform/mac/css1/units/length_units-expected.png:
  • platform/mac/editing/selection/inline-closest-leaf-child-expected.checksum:
  • platform/mac/editing/selection/inline-closest-leaf-child-expected.png:
  • platform/mac/fast/box-shadow/box-shadow-transformed-expected.checksum:
  • platform/mac/fast/box-shadow/box-shadow-transformed-expected.png:
1:17 PM Changeset in webkit [81800] by bweinstein@apple.com
  • 2 edits in trunk/LayoutTests

Skip plugins/change-widget-and-click-crash.html on WebKit2, because
WebKit2 doesn't have an implementation of eventSender yet.

  • platform/mac-wk2/Skipped:
1:13 PM Changeset in webkit [81799] by commit-queue@webkit.org
  • 13 edits in trunk/Source/WebCore

2011-03-23 Tyler Close <tjclose@chromium.org>

Reviewed by Jeremy Orlow.

Fix ambiguous method call in V8 IDL generated code for DOMStringList callback parameter
https://bugs.webkit.org/show_bug.cgi?id=56950

  • bindings/scripts/CodeGeneratorV8.pm:
  • bindings/scripts/test/CPP/WebDOMTestCallback.cpp: (WebDOMTestCallback::callbackWithStringList):
  • bindings/scripts/test/CPP/WebDOMTestCallback.h:
  • bindings/scripts/test/GObject/WebKitDOMTestCallback.cpp: (webkit_dom_test_callback_callback_with_string_list):
  • bindings/scripts/test/GObject/WebKitDOMTestCallback.h:
  • bindings/scripts/test/JS/JSTestCallback.cpp: (WebCore::JSTestCallback::callbackWithStringList):
  • bindings/scripts/test/JS/JSTestCallback.h:
  • bindings/scripts/test/ObjC/DOMTestCallback.h:
  • bindings/scripts/test/ObjC/DOMTestCallback.mm: (-[DOMTestCallback callbackWithStringList:]):
  • bindings/scripts/test/TestCallback.idl:
  • bindings/scripts/test/V8/V8TestCallback.cpp: (WebCore::V8TestCallback::callbackWithStringList):
  • bindings/scripts/test/V8/V8TestCallback.h:
1:10 PM Changeset in webkit [81798] by andersca@apple.com
  • 3 edits in trunk/Source/WebKit2

2011-03-23 Anders Carlsson <andersca@apple.com>

Reviewed by Sam Weinig.

Connection::SyncMessageState::processIncomingMessage should ignore whether the message is sync or not
https://bugs.webkit.org/show_bug.cgi?id=56954

This is so we'll eventually be able to send sync messages that aren't processed immediately if the
destination is waiting for a reply to another synchronous message.

  • Platform/CoreIPC/Connection.cpp: (CoreIPC::Connection::SyncMessageState::processIncomingMessage): Only check for the DispatchMessageWhenWaitingForSyncReply flag.

(CoreIPC::Connection::sendSyncMessage):
Add both DispatchMessageWhenWaitingForSyncReply and SyncMessage to the message ID.

  • Platform/CoreIPC/Connection.h: (CoreIPC::Connection::sendSync): (CoreIPC::Connection::deprecatedSendSync): No need to add SyncMessage here anymore.
1:04 PM Changeset in webkit [81797] by crogers@google.com
  • 4 edits in trunk/Source/WebKit/chromium

2011-03-23 Chris Rogers <crogers@google.com>

Reviewed by Kenneth Russell.

Add API to retrieve the audio hardware sample-rate
https://bugs.webkit.org/show_bug.cgi?id=56882

  • public/WebAudioDevice.h:
  • public/WebKitClient.h: (WebKit::WebKitClient::audioHardwareSampleRate):
  • src/AudioDestinationChromium.cpp: (WebCore::AudioDestination::hardwareSampleRate):
12:52 PM Changeset in webkit [81796] by mihaip@chromium.org
  • 1 edit
    8 moves in trunk/LayoutTests

2011-03-23 Mihai Parparita <mihaip@chromium.org>

Fix fast/table for Chromium Snow Leopard by promoting the chromium-mac/
baselines to mac-leopard/.

  • platform/mac-leopard/fast/table/dynamic-descendant-percentage-height-expected.checksum: Renamed from LayoutTests/platform/chromium-mac/fast/table/dynamic-descendant-percentage-height-expected.checksum.
  • platform/mac-leopard/fast/table/dynamic-descendant-percentage-height-expected.png: Renamed from LayoutTests/platform/chromium-mac/fast/table/dynamic-descendant-percentage-height-expected.png.
  • platform/mac-leopard/fast/table/relative-position-containment-expected.checksum: Renamed from LayoutTests/platform/chromium-mac/fast/table/relative-position-containment-expected.checksum.
  • platform/mac-leopard/fast/table/relative-position-containment-expected.png: Renamed from LayoutTests/platform/chromium-mac/fast/table/relative-position-containment-expected.png.
  • platform/mac-leopard/fast/table/relative-position-offsets-expected.checksum: Renamed from LayoutTests/platform/chromium-mac/fast/table/relative-position-offsets-expected.checksum.
  • platform/mac-leopard/fast/table/relative-position-offsets-expected.png: Renamed from LayoutTests/platform/chromium-mac/fast/table/relative-position-offsets-expected.png.
  • platform/mac-leopard/fast/table/relative-position-stacking-expected.checksum: Renamed from LayoutTests/platform/chromium-mac/fast/table/relative-position-stacking-expected.checksum.
  • platform/mac-leopard/fast/table/relative-position-stacking-expected.png: Renamed from LayoutTests/platform/chromium-mac/fast/table/relative-position-stacking-expected.png.
12:35 PM Changeset in webkit [81795] by inferno@chromium.org
  • 5 edits
    2 adds in trunk

2011-03-23 Cris Neckar <cdn@chromium.org>

Reviewed by Eric Seidel.

Add refptr for widget. Mutations can happen within the event handler.
https://bugs.webkit.org/show_bug.cgi?id=56774

  • src/WebPluginContainerImpl.cpp: (WebKit::WebPluginContainerImpl::handleEvent):

2011-03-23 Cris Neckar <cdn@chromium.org>

Reviewed by Eric Seidel.

Add refptr for widget. Mutations can happen within the event handler.
https://bugs.webkit.org/show_bug.cgi?id=56774

Test: plugins/change-widget-and-click-crash.html

  • html/HTMLPlugInElement.cpp: (WebCore::HTMLPlugInElement::defaultEventHandler):

2011-03-23 Cris Neckar <cdn@chromium.org>

Reviewed by Eric Seidel.

Add test for crash when modifying a widget object within an event handler.
https://bugs.webkit.org/show_bug.cgi?id=56774

  • plugins/change-widget-and-click-crash-expected.txt: Added.
  • plugins/change-widget-and-click-crash.html: Added.
12:32 PM Changeset in webkit [81794] by inferno@chromium.org
  • 6 edits
    2 adds in trunk

2011-03-23 Abhishek Arya <inferno@chromium.org>

Reviewed by Dave Hyatt.

Tests that we do not crash when casting a text fragment of an input box
to combine text.
https://bugs.webkit.org/show_bug.cgi?id=56358

  • fast/text/input-box-text-fragment-combine-text-crash-expected.txt: Added.
  • fast/text/input-box-text-fragment-combine-text-crash.html: Added.

2011-03-23 Abhishek Arya <inferno@chromium.org>

Reviewed by Dave Hyatt.

Add combine text cast checks since style property is insufficient
in telling object types.
https://bugs.webkit.org/show_bug.cgi?id=56358

Test: fast/text/input-box-text-fragment-combine-text-crash.html

  • rendering/InlineTextBox.cpp: (WebCore::InlineTextBox::paint):
  • rendering/RenderBlock.cpp: (WebCore::RenderBlock::computeInlinePreferredLogicalWidths):
  • rendering/RenderBlockLineLayout.cpp: (WebCore::RenderBlock::findNextLineBreak):
  • rendering/RenderText.cpp: (WebCore::RenderText::widthFromCache):
12:29 PM Changeset in webkit [81793] by andreas.kling@nokia.com
  • 3 edits in trunk/Source/WebCore

2011-03-23 Andreas Kling <kling@webkit.org>

Reviewed by Kenneth Rohde Christiansen.

[Qt] "Unwavering" HTML5 game freezes the web page.
https://bugs.webkit.org/show_bug.cgi?id=56944

For canvas's getImageData() API, we don't want to make a deep-copy of
the pixels, which is the case on Qt's "raster" graphics system.

To work around this, we trick QPixmap::toImage() into giving us the
QPixmap's backing QImage by temporarily pointing the paint engine to
a null paint device.

  • platform/graphics/qt/ImageBufferData.h:
  • platform/graphics/qt/ImageBufferQt.cpp: (WebCore::ImageBufferData::toQImage): Added, returns the ImageBufferData as a QImage, avoiding a backend deep-copy if possible. (WebCore::ImageBuffer::platformTransformColorSpace): (WebCore::getImageData):
12:13 PM Changeset in webkit [81792] by commit-queue@webkit.org
  • 5 edits in trunk/Source

2011-03-23 Viet-Trung Luu <viettrungluu@chromium.org>

Reviewed by Tony Chang.

[chromium] Add FormatPlainText to WebClipboard::Format enumeration.
https://bugs.webkit.org/show_bug.cgi?id=56868

Chromium's WebClipboard::isFormatAvailable() will be correspondingly
extended.

  • platform/chromium/PasteboardPrivate.h: Add corresponding enum entry.

This part of the change is inert and should have no effect.

2011-03-23 Viet-Trung Luu <viettrungluu@chromium.org>

Reviewed by Tony Chang.

[chromium] Add FormatPlainText to WebClipboard::Format enumeration.
https://bugs.webkit.org/show_bug.cgi?id=56868

Chromium's WebClipboard::isFormatAvailable() will be correspondingly
extended.

  • public/WebClipboard.h:
  • src/AssertMatchingEnums.cpp:
11:39 AM Changeset in webkit [81791] by mihaip@chromium.org
  • 47 edits
    30 copies
    23 moves in trunk/LayoutTests

2011-03-23 Mihai Parparita <mihaip@chromium.org>

Update Chromium Snow Leopard baselines for SVG tests after r81168.

  • platform/chromium-mac-leopard/svg/W3C-SVG-1.1-SE/coords-units-03-b-expected.checksum: Copied from LayoutTests/platform/chromium-mac/svg/W3C-SVG-1.1-SE/coords-units-03-b-expected.checksum.
  • platform/chromium-mac-leopard/svg/W3C-SVG-1.1-SE/coords-units-03-b-expected.png: Copied from LayoutTests/platform/chromium-mac/svg/W3C-SVG-1.1-SE/coords-units-03-b-expected.png.
  • platform/chromium-mac-leopard/svg/W3C-SVG-1.1-SE/pservers-grad-20-b-expected.checksum: Copied from LayoutTests/platform/chromium-mac/svg/W3C-SVG-1.1-SE/pservers-grad-20-b-expected.checksum.
  • platform/chromium-mac-leopard/svg/W3C-SVG-1.1-SE/pservers-grad-20-b-expected.png: Copied from LayoutTests/platform/chromium-mac/svg/W3C-SVG-1.1-SE/pservers-grad-20-b-expected.png.
  • platform/chromium-mac-leopard/svg/W3C-SVG-1.1-SE/text-intro-02-b-expected.checksum: Renamed from LayoutTests/platform/chromium-mac/svg/W3C-SVG-1.1-SE/text-intro-02-b-expected.checksum.
  • platform/chromium-mac-leopard/svg/W3C-SVG-1.1-SE/text-intro-02-b-expected.png: Renamed from LayoutTests/platform/chromium-mac/svg/W3C-SVG-1.1-SE/text-intro-02-b-expected.png.
  • platform/chromium-mac-leopard/svg/W3C-SVG-1.1-SE/text-intro-02-b-expected.txt: Renamed from LayoutTests/platform/chromium-mac/svg/W3C-SVG-1.1-SE/text-intro-02-b-expected.txt.
  • platform/chromium-mac-leopard/svg/W3C-SVG-1.1-SE/text-intro-05-t-expected.checksum: Renamed from LayoutTests/platform/chromium-mac/svg/W3C-SVG-1.1-SE/text-intro-05-t-expected.checksum.
  • platform/chromium-mac-leopard/svg/W3C-SVG-1.1-SE/text-intro-05-t-expected.png: Renamed from LayoutTests/platform/chromium-mac/svg/W3C-SVG-1.1-SE/text-intro-05-t-expected.png.
  • platform/chromium-mac-leopard/svg/W3C-SVG-1.1-SE/text-intro-05-t-expected.txt: Renamed from LayoutTests/platform/chromium-mac/svg/W3C-SVG-1.1-SE/text-intro-05-t-expected.txt.
  • platform/chromium-mac-leopard/svg/W3C-SVG-1.1-SE/text-intro-09-b-expected.checksum: Renamed from LayoutTests/platform/chromium-mac/svg/W3C-SVG-1.1-SE/text-intro-09-b-expected.checksum.
  • platform/chromium-mac-leopard/svg/W3C-SVG-1.1-SE/text-intro-09-b-expected.png: Renamed from LayoutTests/platform/chromium-mac/svg/W3C-SVG-1.1-SE/text-intro-09-b-expected.png.
  • platform/chromium-mac-leopard/svg/W3C-SVG-1.1-SE/types-dom-05-b-expected.checksum: Copied from LayoutTests/platform/chromium-mac/svg/W3C-SVG-1.1-SE/types-dom-05-b-expected.checksum.
  • platform/chromium-mac-leopard/svg/W3C-SVG-1.1-SE/types-dom-05-b-expected.png: Copied from LayoutTests/platform/chromium-mac/svg/W3C-SVG-1.1-SE/types-dom-05-b-expected.png.
  • platform/chromium-mac-leopard/svg/W3C-SVG-1.1/filters-blend-01-b-expected.checksum: Copied from LayoutTests/platform/chromium-mac/svg/W3C-SVG-1.1/filters-blend-01-b-expected.checksum.
  • platform/chromium-mac-leopard/svg/W3C-SVG-1.1/filters-blend-01-b-expected.png: Copied from LayoutTests/platform/chromium-mac/svg/W3C-SVG-1.1/filters-blend-01-b-expected.png.
  • platform/chromium-mac-leopard/svg/W3C-SVG-1.1/filters-turb-02-f-expected.checksum: Copied from LayoutTests/platform/chromium-mac/svg/W3C-SVG-1.1/filters-turb-02-f-expected.checksum.
  • platform/chromium-mac-leopard/svg/W3C-SVG-1.1/filters-turb-02-f-expected.png: Copied from LayoutTests/platform/chromium-mac/svg/W3C-SVG-1.1/filters-turb-02-f-expected.png.
  • platform/chromium-mac-leopard/svg/W3C-SVG-1.1/text-fonts-03-t-expected.checksum: Renamed from LayoutTests/platform/chromium-mac/svg/W3C-SVG-1.1/text-fonts-03-t-expected.checksum.
  • platform/chromium-mac-leopard/svg/W3C-SVG-1.1/text-fonts-03-t-expected.png: Renamed from LayoutTests/platform/chromium-mac/svg/W3C-SVG-1.1/text-fonts-03-t-expected.png.
  • platform/chromium-mac-leopard/svg/W3C-SVG-1.1/text-intro-01-t-expected.checksum: Copied from LayoutTests/platform/chromium-mac/svg/W3C-SVG-1.1/text-intro-01-t-expected.checksum.
  • platform/chromium-mac-leopard/svg/W3C-SVG-1.1/text-intro-01-t-expected.png: Copied from LayoutTests/platform/chromium-mac/svg/W3C-SVG-1.1/text-intro-01-t-expected.png.
  • platform/chromium-mac-leopard/svg/carto.net/scrollbar-expected.checksum: Copied from LayoutTests/platform/chromium-mac/svg/carto.net/scrollbar-expected.checksum.
  • platform/chromium-mac-leopard/svg/carto.net/scrollbar-expected.png: Copied from LayoutTests/platform/chromium-mac/svg/carto.net/scrollbar-expected.png.
  • platform/chromium-mac-leopard/svg/carto.net/selectionlist-expected.checksum: Copied from LayoutTests/platform/chromium-mac/svg/carto.net/selectionlist-expected.checksum.
  • platform/chromium-mac-leopard/svg/carto.net/selectionlist-expected.png: Copied from LayoutTests/platform/chromium-mac/svg/carto.net/selectionlist-expected.png.
  • platform/chromium-mac-leopard/svg/custom/image-small-width-height-expected.checksum: Copied from LayoutTests/platform/chromium-mac/svg/custom/image-small-width-height-expected.checksum.
  • platform/chromium-mac-leopard/svg/custom/image-small-width-height-expected.png: Copied from LayoutTests/platform/chromium-mac/svg/custom/image-small-width-height-expected.png.
  • platform/chromium-mac-leopard/svg/custom/mouse-move-on-svg-container-expected.checksum: Renamed from LayoutTests/platform/chromium-mac/svg/custom/mouse-move-on-svg-container-expected.checksum.
  • platform/chromium-mac-leopard/svg/custom/mouse-move-on-svg-container-expected.png: Renamed from LayoutTests/platform/chromium-mac/svg/custom/mouse-move-on-svg-container-expected.png.
  • platform/chromium-mac-leopard/svg/custom/mouse-move-on-svg-container-standalone-expected.checksum: Renamed from LayoutTests/platform/chromium-mac/svg/custom/mouse-move-on-svg-container-standalone-expected.checksum.
  • platform/chromium-mac-leopard/svg/custom/mouse-move-on-svg-container-standalone-expected.png: Renamed from LayoutTests/platform/chromium-mac/svg/custom/mouse-move-on-svg-container-standalone-expected.png.
  • platform/chromium-mac-leopard/svg/custom/mouse-move-on-svg-root-expected.checksum: Copied from LayoutTests/platform/chromium-mac/svg/custom/mouse-move-on-svg-root-expected.checksum.
  • platform/chromium-mac-leopard/svg/custom/mouse-move-on-svg-root-expected.png: Copied from LayoutTests/platform/chromium-mac/svg/custom/mouse-move-on-svg-root-expected.png.
  • platform/chromium-mac-leopard/svg/custom/mouse-move-on-svg-root-standalone-expected.checksum: Copied from LayoutTests/platform/chromium-mac/svg/custom/mouse-move-on-svg-root-standalone-expected.checksum.
  • platform/chromium-mac-leopard/svg/custom/mouse-move-on-svg-root-standalone-expected.png: Copied from LayoutTests/platform/chromium-mac/svg/custom/mouse-move-on-svg-root-standalone-expected.png.
  • platform/chromium-mac-leopard/svg/text/bidi-text-anchor-direction-expected.checksum: Renamed from LayoutTests/platform/chromium-mac/svg/text/bidi-text-anchor-direction-expected.checksum.
  • platform/chromium-mac-leopard/svg/text/bidi-text-anchor-direction-expected.png: Renamed from LayoutTests/platform/chromium-mac/svg/text/bidi-text-anchor-direction-expected.png.
  • platform/chromium-mac-leopard/svg/text/bidi-text-anchor-direction-expected.txt: Renamed from LayoutTests/platform/chromium-mac/svg/text/bidi-text-anchor-direction-expected.txt.
  • platform/chromium-mac-leopard/svg/text/bidi-text-query-expected.checksum: Renamed from LayoutTests/platform/chromium-mac/svg/text/bidi-text-query-expected.checksum.
  • platform/chromium-mac-leopard/svg/text/bidi-text-query-expected.png: Renamed from LayoutTests/platform/chromium-mac/svg/text/bidi-text-query-expected.png.
  • platform/chromium-mac-leopard/svg/text/bidi-text-query-expected.txt: Renamed from LayoutTests/platform/chromium-mac/svg/text/bidi-text-query-expected.txt.
  • platform/chromium-mac-leopard/svg/text/bidi-tspans-expected.checksum: Renamed from LayoutTests/platform/chromium-mac/svg/text/bidi-tspans-expected.checksum.
  • platform/chromium-mac-leopard/svg/text/bidi-tspans-expected.png: Renamed from LayoutTests/platform/chromium-mac/svg/text/bidi-tspans-expected.png.
  • platform/chromium-mac-leopard/svg/text/bidi-tspans-expected.txt: Renamed from LayoutTests/platform/chromium-mac/svg/text/bidi-tspans-expected.txt.
  • platform/chromium-mac-leopard/svg/text/text-align-02-b-expected.checksum: Copied from LayoutTests/platform/chromium-mac/svg/text/text-align-02-b-expected.checksum.
  • platform/chromium-mac-leopard/svg/text/text-align-02-b-expected.png: Copied from LayoutTests/platform/chromium-mac/svg/text/text-align-02-b-expected.png.
  • platform/chromium-mac-leopard/svg/text/text-fonts-02-t-expected.checksum: Copied from LayoutTests/platform/chromium-mac/svg/text/text-fonts-02-t-expected.checksum.
  • platform/chromium-mac-leopard/svg/text/text-fonts-02-t-expected.png: Copied from LayoutTests/platform/chromium-mac/svg/text/text-fonts-02-t-expected.png.
  • platform/chromium-mac-leopard/svg/text/text-text-01-b-expected.checksum: Copied from LayoutTests/platform/chromium-mac/svg/text/text-text-01-b-expected.checksum.
  • platform/chromium-mac-leopard/svg/text/text-text-01-b-expected.png: Copied from LayoutTests/platform/chromium-mac/svg/text/text-text-01-b-expected.png.
  • platform/chromium-mac-leopard/svg/text/text-tspan-01-b-expected.checksum: Copied from LayoutTests/platform/chromium-mac/svg/text/text-tspan-01-b-expected.checksum.
  • platform/chromium-mac-leopard/svg/text/text-tspan-01-b-expected.png: Copied from LayoutTests/platform/chromium-mac/svg/text/text-tspan-01-b-expected.png.
  • platform/chromium-mac/svg/W3C-SVG-1.1-SE/coords-units-03-b-expected.checksum:
  • platform/chromium-mac/svg/W3C-SVG-1.1-SE/coords-units-03-b-expected.png:
  • platform/chromium-mac/svg/W3C-SVG-1.1-SE/pservers-grad-17-b-expected.checksum:
  • platform/chromium-mac/svg/W3C-SVG-1.1-SE/pservers-grad-17-b-expected.png:
  • platform/chromium-mac/svg/W3C-SVG-1.1-SE/pservers-grad-20-b-expected.checksum:
  • platform/chromium-mac/svg/W3C-SVG-1.1-SE/pservers-grad-20-b-expected.png:
  • platform/chromium-mac/svg/W3C-SVG-1.1-SE/types-dom-05-b-expected.checksum:
  • platform/chromium-mac/svg/W3C-SVG-1.1-SE/types-dom-05-b-expected.png:
  • platform/chromium-mac/svg/W3C-SVG-1.1/filters-blend-01-b-expected.checksum:
  • platform/chromium-mac/svg/W3C-SVG-1.1/filters-blend-01-b-expected.png:
  • platform/chromium-mac/svg/W3C-SVG-1.1/filters-turb-02-f-expected.checksum:
  • platform/chromium-mac/svg/W3C-SVG-1.1/filters-turb-02-f-expected.png:
  • platform/chromium-mac/svg/W3C-SVG-1.1/text-intro-01-t-expected.checksum:
  • platform/chromium-mac/svg/W3C-SVG-1.1/text-intro-01-t-expected.png:
  • platform/chromium-mac/svg/carto.net/scrollbar-expected.checksum:
  • platform/chromium-mac/svg/carto.net/scrollbar-expected.png:
  • platform/chromium-mac/svg/carto.net/selectionlist-expected.checksum:
  • platform/chromium-mac/svg/carto.net/selectionlist-expected.png:
  • platform/chromium-mac/svg/custom/image-small-width-height-expected.checksum:
  • platform/chromium-mac/svg/custom/image-small-width-height-expected.png:
  • platform/chromium-mac/svg/custom/mouse-move-on-svg-root-expected.checksum:
  • platform/chromium-mac/svg/custom/mouse-move-on-svg-root-expected.png:
  • platform/chromium-mac/svg/custom/mouse-move-on-svg-root-standalone-expected.checksum:
  • platform/chromium-mac/svg/custom/mouse-move-on-svg-root-standalone-expected.png:
  • platform/chromium-mac/svg/text/text-align-02-b-expected.checksum:
  • platform/chromium-mac/svg/text/text-align-02-b-expected.png:
  • platform/chromium-mac/svg/text/text-fonts-02-t-expected.checksum:
  • platform/chromium-mac/svg/text/text-fonts-02-t-expected.png:
  • platform/chromium-mac/svg/text/text-text-01-b-expected.checksum:
  • platform/chromium-mac/svg/text/text-text-01-b-expected.png:
  • platform/chromium-mac/svg/text/text-tspan-01-b-expected.checksum:
  • platform/chromium-mac/svg/text/text-tspan-01-b-expected.png:
  • platform/mac/svg/W3C-SVG-1.1-SE/coords-units-03-b-expected.checksum:
  • platform/mac/svg/W3C-SVG-1.1-SE/coords-units-03-b-expected.png:
  • platform/mac/svg/W3C-SVG-1.1-SE/pservers-grad-17-b-expected.checksum:
  • platform/mac/svg/W3C-SVG-1.1-SE/pservers-grad-17-b-expected.png:
  • platform/mac/svg/W3C-SVG-1.1-SE/pservers-grad-20-b-expected.checksum:
  • platform/mac/svg/W3C-SVG-1.1-SE/pservers-grad-20-b-expected.png:
  • platform/mac/svg/W3C-SVG-1.1-SE/types-dom-05-b-expected.checksum:
  • platform/mac/svg/W3C-SVG-1.1-SE/types-dom-05-b-expected.png:
  • platform/mac/svg/W3C-SVG-1.1/filters-turb-02-f-expected.checksum:
  • platform/mac/svg/W3C-SVG-1.1/filters-turb-02-f-expected.png:
  • platform/mac/svg/carto.net/scrollbar-expected.checksum:
  • platform/mac/svg/carto.net/scrollbar-expected.png:
  • platform/mac/svg/carto.net/selectionlist-expected.checksum:
  • platform/mac/svg/carto.net/selectionlist-expected.png:
11:31 AM Changeset in webkit [81790] by bweinstein@apple.com
  • 2 edits in trunk/Source/WebKit2

Qt Build Fix.

  • DerivedSources.pro:
11:26 AM Changeset in webkit [81789] by commit-queue@webkit.org
  • 16 edits in trunk/Source/WebCore

2011-03-23 Tyler Close <tjclose@chromium.org>

Reviewed by Jeremy Orlow.

run-bindings-tests reference files are out of sync with CodeGenerator*.pm
https://bugs.webkit.org/show_bug.cgi?id=56934

  • bindings/scripts/test/GObject/WebKitDOMTestObj.cpp:
  • bindings/scripts/test/GObject/WebKitDOMTestObj.h:
  • bindings/scripts/test/JS/JSTestInterface.cpp: (WebCore::JSTestInterfaceConstructor::createStructure): (WebCore::JSTestInterfaceConstructor::JSTestInterfaceConstructor): (WebCore::JSTestInterface::createPrototype):
  • bindings/scripts/test/JS/JSTestInterface.h: (WebCore::JSTestInterface::createStructure): (WebCore::JSTestInterfacePrototype::createStructure):
  • bindings/scripts/test/JS/JSTestMediaQueryListListener.cpp: (WebCore::JSTestMediaQueryListListenerConstructor::createStructure): (WebCore::JSTestMediaQueryListListenerConstructor::JSTestMediaQueryListListenerConstructor): (WebCore::JSTestMediaQueryListListener::createPrototype):
  • bindings/scripts/test/JS/JSTestMediaQueryListListener.h: (WebCore::JSTestMediaQueryListListener::createStructure): (WebCore::JSTestMediaQueryListListenerPrototype::createStructure):
  • bindings/scripts/test/JS/JSTestObj.cpp: (WebCore::JSTestObjConstructor::createStructure): (WebCore::JSTestObjConstructor::JSTestObjConstructor): (WebCore::JSTestObj::createPrototype):
  • bindings/scripts/test/JS/JSTestObj.h: (WebCore::JSTestObj::createStructure): (WebCore::JSTestObjPrototype::createStructure):
  • bindings/scripts/test/JS/JSTestSerializedScriptValueInterface.cpp: (WebCore::JSTestSerializedScriptValueInterfaceConstructor::createStructure): (WebCore::JSTestSerializedScriptValueInterfaceConstructor::JSTestSerializedScriptValueInterfaceConstructor): (WebCore::JSTestSerializedScriptValueInterface::createPrototype):
  • bindings/scripts/test/JS/JSTestSerializedScriptValueInterface.h: (WebCore::JSTestSerializedScriptValueInterface::createStructure): (WebCore::JSTestSerializedScriptValueInterfacePrototype::createStructure):
  • bindings/scripts/test/V8/V8TestCallback.cpp:
  • bindings/scripts/test/V8/V8TestInterface.cpp: (WebCore::V8TestInterface::wrapSlow):
  • bindings/scripts/test/V8/V8TestMediaQueryListListener.cpp: (WebCore::V8TestMediaQueryListListener::wrapSlow):
  • bindings/scripts/test/V8/V8TestObj.cpp: (WebCore::V8TestObj::wrapSlow):
  • bindings/scripts/test/V8/V8TestSerializedScriptValueInterface.cpp: (WebCore::V8TestSerializedScriptValueInterface::wrapSlow):
11:17 AM Changeset in webkit [81788] by bweinstein@apple.com
  • 20 edits
    9 adds in trunk/Source/WebKit2

WebKit2: Need API to manage the Media Cache
https://bugs.webkit.org/show_bug.cgi?id=56878
<rdar://problem/9082503>

Reviewed by Darin Adler.

Project File Fun:

  • GNUmakefile.am:
  • WebKit2.pri:
  • WebKit2.pro:
  • WebKit2.xcodeproj/project.pbxproj:
  • win/WebKit2.vcproj:
  • win/WebKit2Common.vsprops:
  • win/WebKit2Generated.make:


Derived Sources and API fun:

  • DerivedSources.make:
  • DerivedSources.pro:
  • Platform/CoreIPC/MessageID.h:
  • Shared/API/c/WKBase.h:
  • Shared/APIObject.h:
  • UIProcess/API/C/WKAPICast.h:


  • UIProcess/WebProcessProxy.cpp:

(WebKit::WebProcessProxy::didReceiveMessage): Add a case for MediaCache messages.

  • WebProcess/WebProcess.cpp:

(WebKit::WebProcess::didReceiveMessage): Ditto.

  • UIProcess/WebContext.cpp:

(WebKit::WebContext::WebContext): Initialize the WebMediaCacheManagerProxy member variable.
(WebKit::WebContext::~WebContext): Invalidate the WebMediaCacheManagerProxy.
(WebKit::WebContext::disconnectProcess): Ditto.
(WebKit::WebContext::didReceiveMessage): Add a case for MediaCache messages.

  • UIProcess/WebContext.h:

(WebKit::WebContext::mediaCacheManagerProxy):

  • UIProcess/API/C/WKContext.cpp:

(WKContextGetMediaCacheManager): Returns the context's WebMediaCacheManager.

  • UIProcess/API/C/WKContext.h:
  • UIProcess/API/C/WKMediaCacheManager.cpp: Added.

(WKMediaCacheManagerGetTypeID): Returns the WebMediaCacheManagerProxy type.
(WKMediaCacheManagerGetHostnamesWithMediaCache): Calls down to the WebMediaCacheManagerProxy.
(WKMediaCacheManagerClearCacheForHostname): Ditto.
(WKMediaCacheManagerClearCacheForAllHostnames): Ditto.

  • UIProcess/API/C/WKMediaCacheManager.h: Added.
  • UIProcess/WebMediaCacheManagerProxy.cpp: Added.

(WebKit::WebMediaCacheManagerProxy::create):
(WebKit::WebMediaCacheManagerProxy::WebMediaCacheManagerProxy):
(WebKit::WebMediaCacheManagerProxy::~WebMediaCacheManagerProxy):
(WebKit::WebMediaCacheManagerProxy::invalidate):
(WebKit::WebMediaCacheManagerProxy::didReceiveMessage):
(WebKit::WebMediaCacheManagerProxy::getHostnamesWithMediaCache): Call through to the WebProcess.
(WebKit::WebMediaCacheManagerProxy::didGetHostnamesWithMediaCache): Process the WebProcess response,

and call the callback.

(WebKit::WebMediaCacheManagerProxy::clearCacheForHostname): Call through to the WebProcess.
(WebKit::WebMediaCacheManagerProxy::clearCacheForAllHostnames): Call through to the WebProcess.

  • UIProcess/WebMediaCacheManagerProxy.h: Added.

(WebKit::WebMediaCacheManagerProxy::clearContext):
(WebKit::WebMediaCacheManagerProxy::type):

  • UIProcess/WebMediaCacheManagerProxy.messages.in: Added.
  • WebProcess/MediaCache: Added.
  • WebProcess/MediaCache/WebMediaCacheManager.cpp: Added.

(WebKit::WebMediaCacheManager::shared):
(WebKit::WebMediaCacheManager::WebMediaCacheManager):
(WebKit::WebMediaCacheManager::didReceiveMessage):
(WebKit::WebMediaCacheManager::getHostnamesWithMediaCache): Added a FIXME to call through to WebCore.
(WebKit::WebMediaCacheManager::clearCacheForHostname): Ditto.
(WebKit::WebMediaCacheManager::clearCacheForAllHostnames): Ditto.

  • WebProcess/MediaCache/WebMediaCacheManager.h: Added.
  • WebProcess/MediaCache/WebMediaCacheManager.messages.in: Added.
10:58 AM Changeset in webkit [81787] by dbates@webkit.org
  • 3 edits in trunk/Source/WebCore

2011-03-23 Daniel Bates <dbates@rim.com>

Reviewed by Antonio Gomes.

Clean up: Rename HTMLFrameSetElement::{noresize, frameborder, and frameBorderSet}
to conform to the WebKit Code style guidelines
https://bugs.webkit.org/show_bug.cgi?id=56871

No functionality changed. So, no new tests.

  • html/HTMLFrameSetElement.cpp: (WebCore::HTMLFrameSetElement::HTMLFrameSetElement): (WebCore::HTMLFrameSetElement::parseMappedAttribute): (WebCore::HTMLFrameSetElement::attach): (WebCore::HTMLFrameSetElement::defaultEventHandler):
  • html/HTMLFrameSetElement.h: (WebCore::HTMLFrameSetElement::hasFrameBorder): (WebCore::HTMLFrameSetElement::noResize):
10:43 AM Changeset in webkit [81786] by inferno@chromium.org
  • 4 edits
    2 adds in trunk

2011-03-23 Abhishek Arya <inferno@chromium.org>

Reviewed by Dan Bernstein.

Tests that we do not crash when accessing entries in the percent
height descendant map.
https://bugs.webkit.org/show_bug.cgi?id=56902

  • fast/block/percent-height-descendant-not-removed-crash-expected.txt: Added.
  • fast/block/percent-height-descendant-not-removed-crash.html: Added.

2011-03-23 Abhishek Arya <inferno@chromium.org>

Reviewed by Dan Bernstein.

Replace height with logicalHeight when removing items from
gPercentHeightDescendantMap so that it is writing mode aware.
https://bugs.webkit.org/show_bug.cgi?id=56902

Test: fast/block/percent-height-descendant-not-removed-crash.html

  • rendering/RenderBox.cpp: (WebCore::RenderBox::destroy): (WebCore::RenderBox::styleDidChange):
  • rendering/RenderWidget.cpp: (WebCore::RenderWidget::destroy):
10:39 AM Changeset in webkit [81785] by Martin Robinson
  • 2 edits in trunk/LayoutTests

2011-03-23 Martin Robinson <mrobinson@igalia.com>

Rebaseline a GTK+ result after r81684.

  • platform/gtk/fast/css/getComputedStyle/computed-style-without-renderer-expected.txt:
10:36 AM Changeset in webkit [81784] by andersca@apple.com
  • 2 edits in trunk/Source/WebCore

Fix clang build.

  • inspector/DOMNodeHighlighter.cpp:

Remove an unused function.

10:32 AM Changeset in webkit [81783] by mihaip@chromium.org
  • 109 edits
    6 copies
    8 moves
    44 deletes in trunk/LayoutTests

2011-03-23 Mihai Parparita <mihaip@chromium.org>

Update pixel results for Snow Leopard after r80755. Where possible,
chromium-mac results were promoted to mac-leopard, except for cases
where Chromium behavior due to bug 52335 or form control rendering.

  • platform/chromium-mac-leopard/fast/block/margin-collapse/103-expected.checksum: Copied from LayoutTests/platform/chromium-mac/fast/block/margin-collapse/103-expected.checksum.
  • platform/chromium-mac-leopard/fast/block/margin-collapse/103-expected.png: Copied from LayoutTests/platform/chromium-mac/fast/block/margin-collapse/103-expected.png.
  • platform/chromium-mac-leopard/tables/mozilla/bugs/bug1318-expected.checksum: Copied from LayoutTests/platform/chromium-mac/tables/mozilla/bugs/bug1318-expected.checksum.
  • platform/chromium-mac-leopard/tables/mozilla/bugs/bug1318-expected.png: Copied from LayoutTests/platform/chromium-mac/tables/mozilla/bugs/bug1318-expected.png.
  • platform/chromium-mac-leopard/tables/mozilla/bugs/bug4527-expected.checksum: Copied from LayoutTests/platform/chromium-mac/tables/mozilla/bugs/bug4527-expected.checksum.
  • platform/chromium-mac-leopard/tables/mozilla/bugs/bug4527-expected.png: Copied from LayoutTests/platform/chromium-mac/tables/mozilla/bugs/bug4527-expected.png.
  • platform/chromium-mac-snowleopard/css2.1/t1008-c44-ln-box-02-d-ag-expected.checksum: Removed.
  • platform/chromium-mac-snowleopard/css2.1/t1008-c44-ln-box-02-d-ag-expected.png: Removed.
  • platform/chromium-mac-snowleopard/css2.1/t100801-c544-valgn-03-d-agi-expected.checksum: Removed.
  • platform/chromium-mac-snowleopard/css2.1/t100801-c544-valgn-03-d-agi-expected.png: Removed.
  • platform/chromium-mac/css2.1/t040302-c61-ex-len-00-b-a-expected.checksum: Removed.
  • platform/chromium-mac/css2.1/t040302-c61-ex-len-00-b-a-expected.png: Removed.
  • platform/chromium-mac/css2.1/t040302-c61-rel-len-00-b-ag-expected.checksum: Removed.
  • platform/chromium-mac/css2.1/t040302-c61-rel-len-00-b-ag-expected.png: Removed.
  • platform/chromium-mac/css2.1/t1008-c44-ln-box-02-d-ag-expected.checksum:
  • platform/chromium-mac/css2.1/t1008-c44-ln-box-02-d-ag-expected.png:
  • platform/chromium-mac/css2.1/t100801-c544-valgn-01-d-ag-expected.checksum: Removed.
  • platform/chromium-mac/css2.1/t100801-c544-valgn-01-d-ag-expected.png: Removed.
  • platform/chromium-mac/css2.1/t100801-c544-valgn-03-d-agi-expected.checksum:
  • platform/chromium-mac/css2.1/t100801-c544-valgn-03-d-agi-expected.png:
  • platform/chromium-mac/css2.1/t100801-c548-ln-ht-02-b-ag-expected.checksum: Removed.
  • platform/chromium-mac/css2.1/t100801-c548-ln-ht-02-b-ag-expected.png: Removed.
  • platform/chromium-mac/css2.1/t1507-c526-font-sz-02-b-a-expected.checksum: Removed.
  • platform/chromium-mac/css2.1/t1507-c526-font-sz-02-b-a-expected.png: Removed.
  • platform/chromium-mac/css2.1/t1601-c547-indent-00-b-a-expected.checksum: Removed.
  • platform/chromium-mac/css2.1/t1601-c547-indent-00-b-a-expected.png: Removed.
  • platform/chromium-mac/css2.1/t1604-c541-word-sp-00-b-a-expected.checksum: Removed.
  • platform/chromium-mac/css2.1/t1604-c541-word-sp-00-b-a-expected.png: Removed.
  • platform/chromium-mac/css2.1/t1604-c542-letter-sp-00-b-a-expected.checksum: Removed.
  • platform/chromium-mac/css2.1/t1604-c542-letter-sp-00-b-a-expected.png: Removed.
  • platform/chromium-mac/editing/selection/caret-rtl-2-expected.checksum: Removed.
  • platform/chromium-mac/editing/selection/caret-rtl-2-expected.png: Removed.
  • platform/chromium-mac/editing/selection/caret-rtl-2-left-expected.checksum: Removed.
  • platform/chromium-mac/editing/selection/caret-rtl-2-left-expected.png: Removed.
  • platform/chromium-mac/editing/selection/caret-rtl-expected.checksum: Removed.
  • platform/chromium-mac/editing/selection/caret-rtl-expected.png: Removed.
  • platform/chromium-mac/editing/selection/caret-rtl-right-expected.checksum: Removed.
  • platform/chromium-mac/editing/selection/caret-rtl-right-expected.png: Removed.
  • platform/chromium-mac/fast/block/margin-collapse/103-expected.checksum:
  • platform/chromium-mac/fast/block/margin-collapse/103-expected.png:
  • platform/chromium-mac/fast/css/ex-after-font-variant-expected.checksum: Removed.
  • platform/chromium-mac/fast/css/ex-after-font-variant-expected.png: Removed.
  • platform/chromium-mac/fast/css/non-standard-checkbox-size-expected.checksum: Removed.
  • platform/chromium-mac/fast/css/non-standard-checkbox-size-expected.png: Removed.
  • platform/chromium-mac/fast/lists/008-expected.checksum: Removed.
  • platform/chromium-mac/fast/lists/008-expected.png: Removed.
  • platform/chromium-mac/fast/media/mq-relative-constraints-08-expected.checksum: Removed.
  • platform/chromium-mac/fast/media/mq-relative-constraints-08-expected.png: Removed.
  • platform/chromium-mac/fast/overflow/overflow-rtl-expected.checksum: Removed.
  • platform/chromium-mac/fast/overflow/overflow-rtl-expected.png: Removed.
  • platform/chromium-mac/fast/replaced/width100percent-checkbox-expected.checksum: Removed.
  • platform/chromium-mac/fast/replaced/width100percent-checkbox-expected.png: Removed.
  • platform/chromium-mac/fast/replaced/width100percent-radio-expected.checksum: Removed.
  • platform/chromium-mac/fast/replaced/width100percent-radio-expected.png: Removed.
  • platform/chromium-mac/fast/text/whitespace/pre-wrap-spaces-after-newline-expected.checksum: Removed.
  • platform/chromium-mac/fast/text/whitespace/pre-wrap-spaces-after-newline-expected.png: Removed.
  • platform/chromium-mac/tables/mozilla/bugs/bug1318-expected.checksum:
  • platform/chromium-mac/tables/mozilla/bugs/bug1318-expected.png:
  • platform/chromium-mac/tables/mozilla/bugs/bug4527-expected.checksum:
  • platform/chromium-mac/tables/mozilla/bugs/bug4527-expected.png:
  • platform/mac-leopard/css2.1/t040302-c61-ex-len-00-b-a-expected.checksum:
  • platform/mac-leopard/css2.1/t040302-c61-ex-len-00-b-a-expected.png:
  • platform/mac-leopard/css2.1/t040302-c61-rel-len-00-b-ag-expected.checksum:
  • platform/mac-leopard/css2.1/t040302-c61-rel-len-00-b-ag-expected.png:
  • platform/mac-leopard/css2.1/t100801-c544-valgn-01-d-ag-expected.checksum:
  • platform/mac-leopard/css2.1/t100801-c544-valgn-01-d-ag-expected.png:
  • platform/mac-leopard/css2.1/t100801-c548-ln-ht-02-b-ag-expected.checksum:
  • platform/mac-leopard/css2.1/t100801-c548-ln-ht-02-b-ag-expected.png:
  • platform/mac-leopard/css2.1/t1507-c526-font-sz-02-b-a-expected.checksum:
  • platform/mac-leopard/css2.1/t1507-c526-font-sz-02-b-a-expected.png:
  • platform/mac-leopard/css2.1/t1601-c547-indent-00-b-a-expected.checksum:
  • platform/mac-leopard/css2.1/t1601-c547-indent-00-b-a-expected.png:
  • platform/mac-leopard/css2.1/t1604-c541-word-sp-00-b-a-expected.checksum:
  • platform/mac-leopard/css2.1/t1604-c541-word-sp-00-b-a-expected.png:
  • platform/mac-leopard/css2.1/t1604-c542-letter-sp-00-b-a-expected.checksum:
  • platform/mac-leopard/css2.1/t1604-c542-letter-sp-00-b-a-expected.png:
  • platform/mac-leopard/editing/selection/caret-rtl-2-expected.checksum:
  • platform/mac-leopard/editing/selection/caret-rtl-2-expected.png:
  • platform/mac-leopard/editing/selection/caret-rtl-2-left-expected.checksum:
  • platform/mac-leopard/editing/selection/caret-rtl-2-left-expected.png:
  • platform/mac-leopard/editing/selection/caret-rtl-expected.checksum:
  • platform/mac-leopard/editing/selection/caret-rtl-expected.png:
  • platform/mac-leopard/editing/selection/caret-rtl-right-expected.checksum:
  • platform/mac-leopard/editing/selection/caret-rtl-right-expected.png:
  • platform/mac-leopard/fast/css/ex-after-font-variant-expected.checksum:
  • platform/mac-leopard/fast/css/ex-after-font-variant-expected.png:
  • platform/mac-leopard/fast/css/non-standard-checkbox-size-expected.checksum:
  • platform/mac-leopard/fast/css/non-standard-checkbox-size-expected.png:
  • platform/mac-leopard/fast/lists/008-expected.checksum:
  • platform/mac-leopard/fast/lists/008-expected.png:
  • platform/mac-leopard/fast/media/mq-relative-constraints-08-expected.checksum:
  • platform/mac-leopard/fast/media/mq-relative-constraints-08-expected.png:
  • platform/mac-leopard/fast/overflow/overflow-rtl-expected.checksum:
  • platform/mac-leopard/fast/overflow/overflow-rtl-expected.png:
  • platform/mac-leopard/fast/replaced/width100percent-checkbox-expected.checksum:
  • platform/mac-leopard/fast/replaced/width100percent-checkbox-expected.png:
  • platform/mac-leopard/fast/replaced/width100percent-radio-expected.checksum:
  • platform/mac-leopard/fast/replaced/width100percent-radio-expected.png:
  • platform/mac-leopard/fast/text/whitespace/pre-wrap-spaces-after-newline-expected.checksum:
  • platform/mac-leopard/fast/text/whitespace/pre-wrap-spaces-after-newline-expected.png:
  • platform/mac/css2.1/t040302-c61-ex-len-00-b-a-expected.checksum:
  • platform/mac/css2.1/t040302-c61-ex-len-00-b-a-expected.png:
  • platform/mac/css2.1/t040302-c61-rel-len-00-b-ag-expected.checksum:
  • platform/mac/css2.1/t040302-c61-rel-len-00-b-ag-expected.png:
  • platform/mac/css2.1/t1008-c44-ln-box-02-d-ag-expected.checksum:
  • platform/mac/css2.1/t1008-c44-ln-box-02-d-ag-expected.png:
  • platform/mac/css2.1/t100801-c544-valgn-01-d-ag-expected.checksum:
  • platform/mac/css2.1/t100801-c544-valgn-01-d-ag-expected.png:
  • platform/mac/css2.1/t100801-c544-valgn-03-d-agi-expected.checksum:
  • platform/mac/css2.1/t100801-c544-valgn-03-d-agi-expected.png:
  • platform/mac/css2.1/t100801-c548-ln-ht-02-b-ag-expected.checksum:
  • platform/mac/css2.1/t100801-c548-ln-ht-02-b-ag-expected.png:
  • platform/mac/css2.1/t1507-c526-font-sz-02-b-a-expected.checksum:
  • platform/mac/css2.1/t1507-c526-font-sz-02-b-a-expected.png:
  • platform/mac/css2.1/t1601-c547-indent-00-b-a-expected.checksum:
  • platform/mac/css2.1/t1601-c547-indent-00-b-a-expected.png:
  • platform/mac/css2.1/t1604-c541-word-sp-00-b-a-expected.checksum:
  • platform/mac/css2.1/t1604-c541-word-sp-00-b-a-expected.png:
  • platform/mac/css2.1/t1604-c542-letter-sp-00-b-a-expected.checksum:
  • platform/mac/css2.1/t1604-c542-letter-sp-00-b-a-expected.png:
  • platform/mac/editing/selection/caret-ltr-2-expected.checksum:
  • platform/mac/editing/selection/caret-ltr-2-expected.png:
  • platform/mac/editing/selection/caret-ltr-2-left-expected.checksum:
  • platform/mac/editing/selection/caret-ltr-2-left-expected.png:
  • platform/mac/editing/selection/caret-ltr-expected.checksum:
  • platform/mac/editing/selection/caret-ltr-expected.png:
  • platform/mac/editing/selection/caret-ltr-right-expected.checksum:
  • platform/mac/editing/selection/caret-ltr-right-expected.png:
  • platform/mac/editing/selection/caret-rtl-2-expected.checksum:
  • platform/mac/editing/selection/caret-rtl-2-expected.png:
  • platform/mac/editing/selection/caret-rtl-2-left-expected.checksum:
  • platform/mac/editing/selection/caret-rtl-2-left-expected.png:
  • platform/mac/editing/selection/caret-rtl-expected.checksum:
  • platform/mac/editing/selection/caret-rtl-expected.png:
  • platform/mac/editing/selection/caret-rtl-right-expected.checksum:
  • platform/mac/editing/selection/caret-rtl-right-expected.png:
  • platform/mac/fast/block/margin-collapse/103-expected.checksum:
  • platform/mac/fast/block/margin-collapse/103-expected.png:
  • platform/mac/fast/css/ex-after-font-variant-expected.checksum:
  • platform/mac/fast/css/ex-after-font-variant-expected.png:
  • platform/mac/fast/css/non-standard-checkbox-size-expected.checksum:
  • platform/mac/fast/css/non-standard-checkbox-size-expected.png:
  • platform/mac/fast/lists/008-expected.checksum:
  • platform/mac/fast/lists/008-expected.png:
  • platform/mac/fast/media/mq-relative-constraints-08-expected.checksum:
  • platform/mac/fast/media/mq-relative-constraints-08-expected.png:
  • platform/mac/fast/overflow/overflow-rtl-expected.checksum:
  • platform/mac/fast/overflow/overflow-rtl-expected.png:
  • platform/mac/fast/replaced/width100percent-checkbox-expected.checksum:
  • platform/mac/fast/replaced/width100percent-checkbox-expected.png:
  • platform/mac/fast/replaced/width100percent-radio-expected.checksum:
  • platform/mac/fast/replaced/width100percent-radio-expected.png:
  • platform/mac/fast/text/whitespace/pre-wrap-spaces-after-newline-expected.checksum:
  • platform/mac/fast/text/whitespace/pre-wrap-spaces-after-newline-expected.png:
  • platform/mac/tables/mozilla/bugs/bug1318-expected.checksum:
  • platform/mac/tables/mozilla/bugs/bug1318-expected.png:
  • platform/mac/tables/mozilla/bugs/bug4527-expected.checksum:
  • platform/mac/tables/mozilla/bugs/bug4527-expected.png:
10:26 AM Changeset in webkit [81782] by beidson@apple.com
  • 11 edits in trunk/Source

2011-03-22 Brady Eidson <beidson@apple.com>

Reviewed by Sam Weinig.

Add asynchronous load decision call to WebKit2 IconDatabase
https://bugs.webkit.org/show_bug.cgi?id=56887

Clear the callback when it's made:

  • loader/DocumentLoader.cpp: (WebCore::DocumentLoader::continueIconLoadWithDecision):

Don't load icons when using new-style icon database if in private browsing:

  • loader/FrameLoader.cpp: (WebCore::FrameLoader::continueIconLoadWithDecision):


Enhance the IconDatabase callbacks to have an ID and inherit from a common base:

  • loader/icon/IconDatabaseBase.h: (WebCore::CallbackBase::~CallbackBase): (WebCore::CallbackBase::callbackID): (WebCore::CallbackBase::CallbackBase): (WebCore::CallbackBase::context): (WebCore::CallbackBase::generateCallbackID): (WebCore::EnumCallback::performCallback): (WebCore::EnumCallback::EnumCallback): (WebCore::ObjectCallback::performCallback): (WebCore::ObjectCallback::ObjectCallback):

2011-03-22 Brady Eidson <beidson@apple.com>

Reviewed by Sam Weinig.

Add asynchronous load decision call to WebKit2 IconDatabase
https://bugs.webkit.org/show_bug.cgi?id=56887

  • UIProcess/WebIconDatabase.cpp: (WebKit::WebIconDatabase::getLoadDecisionForIconURL): In the future, get the actual load decision from the database. For now, always message "IconLoadNo" back to the WebProcess.
  • UIProcess/WebIconDatabase.h:
  • UIProcess/WebIconDatabase.messages.in:


  • WebProcess/IconDatabase/WebIconDatabaseProxy.cpp: (WebKit::WebIconDatabaseProxy::loadDecisionForIconURL): Message the UIProcess for the load decision. (WebKit::WebIconDatabaseProxy::receivedIconLoadDecision): Callback into WebCore with the received load decision.
  • WebProcess/IconDatabase/WebIconDatabaseProxy.h:
  • WebProcess/IconDatabase/WebIconDatabaseProxy.messages.in:
10:02 AM Changeset in webkit [81781] by yurys@chromium.org
  • 3 edits in trunk/Source/WebKit/chromium

2011-03-23 Yury Semikhatsky <yurys@chromium.org>

Reviewed by Pavel Feldman.

[Chromium] Web Inspector: Remove DebuggerScript.js from Chromium resource bundle.
https://bugs.webkit.org/show_bug.cgi?id=56916

  • DEPS: bump up Chromium dependency to the revision where no references to the DebuggerScript.js resource are left.
  • WebKit.grd:
9:49 AM Changeset in webkit [81780] by podivilov@chromium.org
  • 3 edits in trunk/Source/WebCore

2011-03-21 Pavel Podivilov <podivilov@chromium.org>

Reviewed by Yury Semikhatsky.

Web Inspector: move scripts concatenation logic to SourceFile.
https://bugs.webkit.org/show_bug.cgi?id=56756

  • inspector/front-end/ScriptsPanel.js:
  • inspector/front-end/SourceFile.js: (WebInspector.SourceFile.prototype._requestContent): (WebInspector.SourceFile.prototype._loadResourceContent): (WebInspector.SourceFile.prototype._loadAndConcatenateScriptsContent): (WebInspector.SourceFile.prototype._concatenateScriptsContent):
9:47 AM Changeset in webkit [81779] by commit-queue@webkit.org
  • 3 edits in trunk/Source/WebCore

2011-03-23 Leandro Gracia Gil <leandrogracia@chromium.org>

Reviewed by Steve Block.

Media Stream API: add a flag to RuntimeEnabledFeatures.
https://bugs.webkit.org/show_bug.cgi?id=56921

Add a flag to RuntimeEnabledFeatures to check if the Media Stream API is enabled at runtime.

Tests for the Media Stream API will be provided by the bug 56587.

  • bindings/generic/RuntimeEnabledFeatures.cpp:
  • bindings/generic/RuntimeEnabledFeatures.h: (WebCore::RuntimeEnabledFeatures::mediaStreamEnabled): (WebCore::RuntimeEnabledFeatures::setMediaStreamEnabled): (WebCore::RuntimeEnabledFeatures::webkitGetUserMediaEnabled):
9:38 AM Changeset in webkit [81778] by carol.szabo@nokia.com
  • 3 edits
    2 adds in trunk

2011-03-23 Carol Szabo <carol.szabo@nokia.com>

Reviewed by David Hyatt.

Added new test to expose failure to correctly find the
before/after container when creating a CSS counter.

CSS 2.1 failure: various before-after-* tests fail
https://bugs.webkit.org/show_bug.cgi?id=47207

  • fast/css/counters/complex-before-expected.txt: Added.
  • fast/css/counters/complex-before.html: Added.

2011-03-23 Carol Szabo <carol.szabo@nokia.com>

Reviewed by David Hyatt.

Modified RenderCounter::originalText() to correctly attach
the created counter to the before/after container even when
that is not the RenderCounter's direct parent.

CSS 2.1 failure: various before-after-* tests fail
https://bugs.webkit.org/show_bug.cgi?id=47207

Test: fast/css/counters/complex-before.html

  • rendering/RenderCounter.cpp: (WebCore::RenderCounter::originalText):
9:21 AM Changeset in webkit [81777] by gyuyoung.kim@samsung.com
  • 5 edits
    5 adds in trunk/Source

2011-03-23 Gyuyoung Kim <gyuyoung.kim@samsung.com>

Reviewed by Antonio Gomes.

[EFL] Add play / pause button to media control
https://bugs.webkit.org/show_bug.cgi?id=55463

Add play|pause button to media control.

  • platform/efl/RenderThemeEfl.cpp: (WebCore::RenderThemeEfl::edjeGroupFromFormType): (WebCore::RenderThemeEfl::emitMediaButtonSignal): Added. (WebCore::RenderThemeEfl::paintMediaPlayButton):
  • platform/efl/RenderThemeEfl.h:

2011-03-23 Gyuyoung Kim <gyuyoung.kim@samsung.com>

Reviewed by Antonio Gomes.

[EFL] Add play / pause button to media control
https://bugs.webkit.org/show_bug.cgi?id=55463

Add play|pause button to media control.

  • DefaultTheme/default.edc:
  • DefaultTheme/widget/mediacontrol/playpausebutton/pausebutton.png: Added.
  • DefaultTheme/widget/mediacontrol/playpausebutton/playbutton.png: Added.
  • DefaultTheme/widget/mediacontrol/playpausebutton/playpause_button.edc: Added.
9:03 AM Changeset in webkit [81776] by yurys@chromium.org
  • 4 edits in trunk/Source/WebKit/chromium

2011-03-23 Yury Semikhatsky <yurys@chromium.org>

Reviewed by Pavel Feldman.

[Chromium] Web Inspector: remove inspectNode method from WebDevToolsAgent
https://bugs.webkit.org/show_bug.cgi?id=56926

  • public/WebDevToolsAgent.h:
  • src/WebDevToolsAgentImpl.cpp:
  • src/WebDevToolsAgentImpl.h:
8:54 AM Changeset in webkit [81775] by commit-queue@webkit.org
  • 4 edits in trunk/Source

2011-03-23 Aparna Nandyal <aparna.nand@wipro.com>

Reviewed by Andreas Kling.

[Qt] QtWebKit rendering problem when maximizing and doing a back
https://bugs.webkit.org/show_bug.cgi?id=56669

Contents of the page are not re-sized on going back after
maximizing. ContentsResized() is not getting called. Hence old
document width and height are used. Corrected this behaviour by
calling setBoundsSize() which calls contentsSize().

  • loader/FrameLoader.cpp: (WebCore::FrameLoader::open):

2011-03-23 Aparna Nandyal <aparna.nand@wipro.com>

Reviewed by Andreas Kling.

[Qt] QtWebKit rendering problem when maximizing and doing a back
https://bugs.webkit.org/show_bug.cgi?id=56669

Added an auto test.
Patch by Alexis Menard < alexis.menard@nokia.com> on 2011-03-21

  • tests/qwebview/tst_qwebview.cpp: (tst_QWebView::rendering):
8:04 AM Changeset in webkit [81774] by podivilov@chromium.org
  • 6 edits in trunk/Source/WebCore

2011-03-21 Pavel Podivilov <podivilov@chromium.org>

Reviewed by Yury Semikhatsky.

Web Inspector: add support for formating source files in debugger presentation model.
https://bugs.webkit.org/show_bug.cgi?id=56558

  • inspector/front-end/DebuggerModel.js: (WebInspector.DebuggerModel.prototype.reset): (WebInspector.DebuggerModel.prototype.get scripts):
  • inspector/front-end/DebuggerPresentationModel.js: (WebInspector.DebuggerPresentationModel): (WebInspector.DebuggerPresentationModel.prototype._addScript): (WebInspector.DebuggerPresentationModel.prototype._refreshBreakpoints): (WebInspector.DebuggerPresentationModel.prototype.toggleFormatSourceFiles): (WebInspector.DebuggerPresentationModel.prototype.continueToLine): (WebInspector.DebuggerPresentationModel.prototype.setBreakpoint): (WebInspector.DebuggerPresentationModel.prototype._breakpointAdded): (WebInspector.DebuggerPresentationModel.prototype.set selectedCallFrame): (WebInspector.DebuggerPresentationModel.prototype._sourceFileForScript): (WebInspector.DebuggerPresentationModel.prototype._reset):
  • inspector/front-end/ScriptFormatter.js: (WebInspector.ScriptFormatter.prototype.formatContent.didFormatChunks): (WebInspector.ScriptFormatter.prototype.formatContent):
  • inspector/front-end/ScriptsPanel.js: (WebInspector.ScriptsPanel): (WebInspector.ScriptsPanel.prototype.reset): (WebInspector.ScriptsPanel.prototype._toggleFormatSourceFiles):
  • inspector/front-end/Settings.js:
  • inspector/front-end/SourceFile.js: (WebInspector.SourceFile.prototype.scriptLocationToSourceLocation): (WebInspector.SourceFile.prototype.sourceLocationToScriptLocation): (WebInspector.FormattedSourceFile): (WebInspector.FormattedSourceFile.prototype.scriptLocationToSourceLocation): (WebInspector.FormattedSourceFile.prototype.sourceLocationToScriptLocation): (WebInspector.FormattedSourceFile.prototype._didRequestContent):
7:04 AM Changeset in webkit [81773] by yurys@chromium.org
  • 7 edits in trunk/Source/WebCore

2011-03-23 Yury Semikhatsky <yurys@chromium.org>

Reviewed by Pavel Feldman.

Web Inspector: use unsigned char instead of char when storing script source
https://bugs.webkit.org/show_bug.cgi?id=56920

  • bindings/v8/DebuggerScript.js:
  • bindings/v8/ScriptDebugServer.cpp: (WebCore::ScriptDebugServer::ensureDebuggerScriptCompiled):
  • inspector/InjectedScriptManager.cpp: (WebCore::InjectedScriptManager::injectedScriptSource):
  • inspector/InjectedScriptSource.js: whitespace change to trigger compilation
  • inspector/xxd.pl:
  • xml/XMLViewer.xsl:
6:56 AM Changeset in webkit [81772] by kov@webkit.org
  • 2 edits in trunk/Source/WebKit/gtk/po

2011-03-23 Luca Ferretti <lferrett@gnome.org>

Rubber-stamped by Gustavo Noronha.

Italian translation update.

  • it.po: Updated.
6:22 AM Changeset in webkit [81771] by loislo@chromium.org
  • 18 edits
    2 adds in trunk/Source/WebCore

2011-03-23 Ilya Tikhonovsky <loislo@chromium.org>

Reviewed by Yury Semikhatsky.

Web Inspector: move node searching and node highlight related methods from InspectorAgent to InspectorDOMAgent.
https://bugs.webkit.org/show_bug.cgi?id=56912

The next functions were moved:

setSearchingForNode
highlightDOMNode
hideDOMNodeHighlight
highlightFrame
hideFrameHighlight
mouseDidMoveOverElement
handleMousePress

The code which does real highlight in Graphic context was moved to new files DOMNodeHighlighter.*

  • CMakeLists.txt:
  • GNUmakefile.am:
  • WebCore.gypi:
  • WebCore.pro:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • inspector/DOMNodeHighlighter.cpp: Added. (WebCore::DOMNodeHighlighter::DrawNodeHighlight):
  • inspector/DOMNodeHighlighter.h: Added.
  • inspector/Inspector.json:
  • inspector/InspectorAgent.cpp: (WebCore::InspectorAgent::InspectorAgent): (WebCore::InspectorAgent::~InspectorAgent): (WebCore::InspectorAgent::inspectedPageDestroyed): (WebCore::InspectorAgent::disconnectFrontend):
  • inspector/InspectorAgent.h:
  • inspector/InspectorController.cpp: (WebCore::InspectorController::drawNodeHighlight): (WebCore::InspectorController::hideHighlight):
  • inspector/InspectorDOMAgent.cpp: (WebCore::InspectorDOMAgent::InspectorDOMAgent): (WebCore::InspectorDOMAgent::~InspectorDOMAgent): (WebCore::InspectorDOMAgent::clearFrontend): (WebCore::InspectorDOMAgent::handleMousePress): (WebCore::InspectorDOMAgent::mouseDidMoveOverElement): (WebCore::InspectorDOMAgent::searchingForNodeInPage): (WebCore::InspectorDOMAgent::setSearchingForNode): (WebCore::InspectorDOMAgent::highlight): (WebCore::InspectorDOMAgent::highlightDOMNode): (WebCore::InspectorDOMAgent::highlightFrame): (WebCore::InspectorDOMAgent::hideHighlight): (WebCore::InspectorDOMAgent::drawNodeHighlight):
  • inspector/InspectorDOMAgent.h: (WebCore::InspectorDOMAgent::hideDOMNodeHighlight): (WebCore::InspectorDOMAgent::hideFrameHighlight):
  • inspector/InspectorInstrumentation.cpp: (WebCore::InspectorInstrumentation::mouseDidMoveOverElementImpl): (WebCore::InspectorInstrumentation::handleMousePressImpl):
  • inspector/InstrumentingAgents.h: (WebCore::InstrumentingAgents::InstrumentingAgents): (WebCore::InstrumentingAgents::inspectorAgent): (WebCore::InstrumentingAgents::setInspectorAgent):
  • inspector/front-end/ElementsPanel.js: (WebInspector.ElementsPanel.prototype.setSearchingForNode):
  • inspector/front-end/ResourcesPanel.js: (WebInspector.FrameTreeElement.prototype.onselect): (WebInspector.FrameTreeElement.prototype.set hovered):
  • inspector/front-end/inspector.js: (WebInspector.highlightDOMNode):
6:19 AM Changeset in webkit [81770] by yurys@chromium.org
  • 3 edits in trunk/Source/WebCore

2011-03-23 Yury Semikhatsky <yurys@chromium.org>

Unreviewed. Rollout r81768 which broke compilation on Win.

  • xml/XMLTreeViewer.cpp: (WebCore::XMLTreeViewer::transformDocumentToTreeView):
  • xml/XMLViewer.xsl:
6:11 AM Changeset in webkit [81769] by Csaba Osztrogonác
  • 3 edits in trunk/LayoutTests

[Qt] Update expected files after r81684.

  • platform/qt/fast/css/getComputedStyle/computed-style-expected.txt:
  • platform/qt/fast/css/getComputedStyle/computed-style-without-renderer-expected.txt:
5:56 AM Changeset in webkit [81768] by yurys@chromium.org
  • 3 edits in trunk/Source/WebCore

2011-03-23 Yury Semikhatsky <yurys@chromium.org>

Reviewed by Pavel Feldman.

Web Inspector: remove unnecessary reinterpret_cast in XMLTreeViewer
https://bugs.webkit.org/show_bug.cgi?id=56919

  • xml/XMLTreeViewer.cpp: (WebCore::XMLTreeViewer::transformDocumentToTreeView):
  • xml/XMLViewer.xsl: whitespace change to trigger compilation.
5:52 AM Changeset in webkit [81767] by podivilov@chromium.org
  • 7 edits
    1 add in trunk

2011-03-21 Pavel Podivilov <podivilov@chromium.org>

Reviewed by Yury Semikhatsky.

Web Inspector: move content loading logic to a new SourceFile class.
https://bugs.webkit.org/show_bug.cgi?id=56748

  • WebCore.gypi:
  • WebCore.vcproj/WebCore.vcproj:
  • inspector/front-end/DebuggerPresentationModel.js: (WebInspector.DebuggerPresentationModel.prototype.requestSourceFileContent): (WebInspector.DebuggerPresentationModel.prototype._addScript.contentChanged): (WebInspector.DebuggerPresentationModel.prototype._addScript): (WebInspector.DebuggerPresentationModel.prototype._scriptSourceChanged): (WebInspector.DebuggerPresentationModel.prototype.set selectedCallFrame):
  • inspector/front-end/SourceFile.js: Added. (WebInspector.SourceFile): (WebInspector.SourceFile.prototype.addScript): (WebInspector.SourceFile.prototype.requestContent): (WebInspector.SourceFile.prototype.forceLoadContent): (WebInspector.SourceFile.prototype.reload): (WebInspector.SourceFile.prototype._requestContent): (WebInspector.SourceFile.prototype._loadResourceContent): (WebInspector.SourceFile.prototype._loadScriptContent): (WebInspector.SourceFile.prototype._loadAndConcatenateScriptsContent): (WebInspector.SourceFile.prototype._didRequestContent): (WebInspector.SourceFile.prototype._hasPendingResource):
  • inspector/front-end/WebKit.qrc:
  • inspector/front-end/inspector.html:
5:41 AM Changeset in webkit [81766] by podivilov@chromium.org
  • 2 edits in trunk/Source/WebCore

2011-03-23 Pavel Podivilov <podivilov@chromium.org>

Unreviewed, fix compilation broken by r81758.

  • xml/XMLTreeViewer.cpp: (WebCore::XMLTreeViewer::transformDocumentToTreeView):
5:35 AM Changeset in webkit [81765] by pfeldman@chromium.org
  • 3 edits in trunk/Source/WebCore

2011-03-23 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Yury Semikhatsky.

Web Inspector: define array types properly in the Inspector.json
https://bugs.webkit.org/show_bug.cgi?id=56915

  • inspector/Inspector.json:
  • inspector/generate-inspector-idl:
4:34 AM Changeset in webkit [81764] by pfeldman@chromium.org
  • 2 edits in trunk/Source/JavaScriptCore

2011-03-23 Pavel Feldman <pfeldman@chromium.org>

Not reviewed: bring back Vector::contains that was removed as a part of roll back.

  • wtf/Vector.h: (WTF::::contains):
4:09 AM Changeset in webkit [81763] by pfeldman@chromium.org
  • 10 edits
    7 deletes in trunk/Source

2011-03-23 Sheriff Bot <webkit.review.bot@gmail.com>

Unreviewed, rolling out r81686.
http://trac.webkit.org/changeset/81686
https://bugs.webkit.org/show_bug.cgi?id=56914

Breaks webkit_tests in Chromium again. (Requested by pfeldman
on #webkit).

  • wtf/Vector.h:

2011-03-23 Sheriff Bot <webkit.review.bot@gmail.com>

Unreviewed, rolling out r81686.
http://trac.webkit.org/changeset/81686
https://bugs.webkit.org/show_bug.cgi?id=56914

Breaks webkit_tests in Chromium again. (Requested by pfeldman
on #webkit).

  • WebKit.gyp:
  • public/WebPageSerializer.h:
  • public/WebURL.h:
  • public/WebVector.h: (WebKit::WebVector::operator[]):
  • src/WebDataSourceImpl.cpp: (WebKit::WebDataSourceImpl::WebDataSourceImpl):
  • src/WebPageSerializer.cpp:
  • tests/WebFrameTest.cpp:
  • tests/WebPageSerializerTest.cpp: Removed.
  • tests/data/pageserialization/awesome.png: Removed.
  • tests/data/pageserialization/embed_iframe.html: Removed.
  • tests/data/pageserialization/object_iframe.html: Removed.
  • tests/data/pageserialization/simple_iframe.html: Removed.
  • tests/data/pageserialization/simple_page.html: Removed.
  • tests/data/pageserialization/top_frame.html: Removed.
4:09 AM Changeset in webkit [81762] by yurys@chromium.org
  • 7 edits in trunk

2011-03-23 Yury Semikhatsky <yurys@chromium.org>

Reviewed by Pavel Feldman.

[V8] Web Inspector: compile DebuggerScript.js into DebuggerScriptSource.h
https://bugs.webkit.org/show_bug.cgi?id=56843

  • inspector/front-end/WebKit.qrc:

2011-03-23 Yury Semikhatsky <yurys@chromium.org>

Reviewed by Pavel Feldman.

[V8] Web Inspector: compile DebuggerScript.js into DebuggerScriptSource.h
https://bugs.webkit.org/show_bug.cgi?id=56843

  • DumpRenderTree/chromium/DRTDevToolsAgent.cpp:
  • DumpRenderTree/chromium/DRTDevToolsAgent.h:

2011-03-23 Yury Semikhatsky <yurys@chromium.org>

Reviewed by Pavel Feldman.

[V8] Web Inspector: compile DebuggerScript.js into DebuggerScriptSource.h
https://bugs.webkit.org/show_bug.cgi?id=56843

  • WebCoreSupport/InspectorClientQt.cpp: (WebCore::InspectorClientQt::openInspectorFrontend):
3:53 AM BuildingQtOnLinux edited by benjamin.poulain@nokia.com
(diff)
3:33 AM Changeset in webkit [81761] by podivilov@chromium.org
  • 3 edits in trunk/Source/WebCore

2011-03-23 Andrey Adaikin <aandrey@google.com>

Reviewed by Yury Semikhatsky.

Web Inspector: Add a star while editing a source code
https://bugs.webkit.org/show_bug.cgi?id=56743

  • inspector/front-end/ScriptsPanel.js: (WebInspector.ScriptsPanel.prototype.setScriptSourceIsBeingEdited): (WebInspector.SourceFrameDelegateForScriptsPanel.prototype.setScriptSourceIsBeingEdited):
  • inspector/front-end/SourceFrame.js: (WebInspector.SourceFrame.prototype._startEditing): (WebInspector.SourceFrame.prototype._registerShortcuts): (WebInspector.SourceFrame.prototype._handleSave): (WebInspector.SourceFrame.prototype._handleRevertEditing): (WebInspector.SourceFrameDelegate.prototype.setScriptSourceIsBeingEdited):
3:30 AM Changeset in webkit [81760] by podivilov@chromium.org
  • 2 edits in trunk/Source/WebCore

2011-03-23 Andrey Adaikin <aandrey@google.com>

Reviewed by Yury Semikhatsky.

Web Inspector: Scrolling and navigation is not smooth on a script with many long lines
https://bugs.webkit.org/show_bug.cgi?id=56559

  • inspector/front-end/TextViewer.js: (WebInspector.TextEditorChunkedPanel.prototype._expandChunks): (WebInspector.TextEditorMainPanel.prototype._expandChunks): (WebInspector.TextEditorMainPanel.prototype._highlightDataReady): (WebInspector.TextEditorMainPanel.prototype._schedulePaintLines): (WebInspector.TextEditorMainPanel.prototype._paintScheduledLines): (WebInspector.TextEditorMainPanel.prototype._restorePaintLinesOperationsCredit): (WebInspector.TextEditorMainPanel.prototype._paintLines): (WebInspector.TextEditorMainPanel.prototype._paintLine): (WebInspector.TextEditorMainPanel.prototype._applyDomUpdates):
3:17 AM Changeset in webkit [81759] by abarth@webkit.org
  • 2 edits in trunk/Source/JavaScriptCore

2011-03-23 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.

JavaScriptCore GYP build should work on a case-sensitive file system
https://bugs.webkit.org/show_bug.cgi?id=56911

The issue is that there are two UString.h headers, one named UString.h
and one named ustring.h. This patch excludes ustring.h from the header
map to avoid confusion. While I was editing this part of the GYP file,
I cleaned up the exclude rules to be more modern.

  • gyp/JavaScriptCore.gyp:
3:09 AM Changeset in webkit [81758] by yurys@chromium.org
  • 11 edits in trunk/Source

2011-03-22 Yury Semikhatsky <yurys@chromium.org>

Reviewed by Pavel Feldman.

[V8] Web Inspector: compile DebuggerScript.js into DebuggerScriptSource.h
https://bugs.webkit.org/show_bug.cgi?id=56843

DebuggerScript.js is now translated into a char[] array before inspector compilation.

  • WebCore.gyp/WebCore.gyp:
  • WebCore.gypi:
  • bindings/v8/ScriptDebugServer.cpp: (WebCore::ScriptDebugServer::ensureDebuggerScriptCompiled):
  • bindings/v8/ScriptDebugServer.h:
  • inspector/InjectedScriptManager.cpp: (WebCore::InjectedScriptManager::injectedScriptSource):
  • inspector/InjectedScriptSource.js: whitespace change to trigger InjectedScriptSource.h generation.
  • inspector/xxd.pl:

2011-03-22 Yury Semikhatsky <yurys@chromium.org>

Reviewed by Pavel Feldman.

[V8] Web Inspector: compile DebuggerScript.js into DebuggerScriptSource.h
https://bugs.webkit.org/show_bug.cgi?id=56843

  • public/WebDevToolsAgentClient.h: the script source is now compiled along with the inspector code.
  • src/WebDevToolsAgentImpl.cpp: (WebKit::WebDevToolsAgentImpl::attach):
2:49 AM Changeset in webkit [81757] by yurys@chromium.org
  • 2 edits in trunk/Source/WebKit/chromium

2011-03-22 Yury Semikhatsky <yurys@chromium.org>

Reviewed by Pavel Feldman.

[Chromium] Web Inspector: remove obsolete methods from WebDevToolsAgentClient.h
https://bugs.webkit.org/show_bug.cgi?id=56897

  • public/WebDevToolsAgentClient.h:
12:34 AM Changeset in webkit [81756] by mrowe@apple.com
  • 2 edits in trunk/Source/WebCore

Fix the 32-bit build.

  • html/canvas/WebGLRenderingContext.cpp:

(WebCore::WebGLRenderingContext::initializeNewContext):

12:21 AM Changeset in webkit [81755] by Chris Fleizach
  • 4 edits in trunk/Source/WebKit2

2011-03-23 Chris Fleizach <Chris Fleizach>

Reviewed by Darin Adler.

REGRESSION: WK2: AX: PDF in Safari no longer accessible.
https://bugs.webkit.org/show_bug.cgi?id=56849

The WKView needs to know when to return the WKPDFView and when to return
the remote web process connection.

  • UIProcess/API/mac/PDFViewController.h: (WebKit::PDFViewController::pdfView):
  • UIProcess/API/mac/WKView.mm: (-[WKView accessibilityFocusedUIElement]): (-[WKView accessibilityHitTest:]): (-[WKView accessibilityAttributeValue:]):
12:16 AM Changeset in webkit [81754] by abarth@webkit.org
  • 2 edits in trunk

2011-03-23 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.

GYP build should default to Production configuration
https://bugs.webkit.org/show_bug.cgi?id=56899

  • Source/gyp/common.gypi:
Note: See TracTimeline for information about the timeline view.