Timeline



Apr 29, 2013:

11:39 PM Changeset in webkit [149353] by zandobersek@gmail.com
  • 2 edits in trunk/LayoutTests

Unreviewed GTK gardening.

  • platform/gtk/TestExpectations: Skipping the Shadow DOM layout tests after the feature was disabled in r149352.
11:33 PM Changeset in webkit [149352] by zandobersek@gmail.com
  • 4 edits in trunk

[GTK] Disable Shadow DOM feature
https://bugs.webkit.org/show_bug.cgi?id=115374

Reviewed by Martin Robinson.

Disable the Shadow DOM feature on the GTK port, the feature is planned for removal.

.:

  • Source/autotools/SetupWebKitFeatures.m4:

Tools:

  • Scripts/webkitperl/FeatureList.pm:
9:50 PM Changeset in webkit [149351] by fpizlo@apple.com
  • 12 edits
    1 add in branches/dfgFourthTier/Source

fourthTier: String::utf8() should also be available as StringImpl::utf8() so that you don't have to ref() a StringImpl just to get its utf8()
https://bugs.webkit.org/show_bug.cgi?id=115393

Reviewed by Geoffrey Garen.

Source/JavaScriptCore:

  • runtime/JSGlobalObjectFunctions.cpp:

(JSC::encode):

Source/WebCore:

No new tests because no new behavior.

  • Modules/websockets/WebSocket.cpp:

(WebCore::WebSocket::close):

  • Modules/websockets/WebSocketChannel.cpp:

(WebCore::WebSocketChannel::send):

  • html/MediaFragmentURIParser.cpp:

(WebCore::MediaFragmentURIParser::parseFragments):

Source/WTF:

  • WTF.xcodeproj/project.pbxproj:
  • wtf/text/ConversionMode.h: Added.

(WTF):

  • wtf/text/StringImpl.cpp:

(WTF):
(WTF::putUTF8Triple):
(WTF::StringImpl::utf8):

  • wtf/text/StringImpl.h:

(StringImpl):

  • wtf/text/WTFString.cpp:

(WTF):
(WTF::String::utf8):

  • wtf/text/WTFString.h:

(String):

8:20 PM Changeset in webkit [149350] by glenn@skynav.com
  • 2 edits in trunk/Tools

[webkitpy] Add missing contributor email addresses
https://bugs.webkit.org/show_bug.cgi?id=115394

Reviewed by Ryosuke Niwa.

  • Scripts/webkitpy/common/config/contributors.json:
7:55 PM Changeset in webkit [149349] by commit-queue@webkit.org
  • 15 edits
    1 add in trunk/Source

[ARM] Expand the use of integer division
https://bugs.webkit.org/show_bug.cgi?id=115138

Patch by Cosmin Truta <ctruta@blackberry.com> on 2013-04-29
Reviewed by Benjamin Poulain.

Source/JavaScriptCore:

If availability of hardware integer division isn't known at compile
time, check the CPU flags and decide at runtime whether to fall back
to software. Currently, this OS-specific check is implemented on QNX.

Moreover, use operator % instead of fmod() in the calculation of the
software modulo. Even when it's software-emulated, operator % is faster
than fmod(): on ARM v7 QNX, without hardware division, we noticed

3% speedup on SunSpider.

  • CMakeLists.txt:
  • GNUmakefile.list.am:
  • JavaScriptCore.xcodeproj/project.pbxproj:
  • assembler/ARMv7Assembler.h:

(JSC::ARMv7Assembler::sdiv): Did not compile conditionally.
(JSC::ARMv7Assembler::udiv): Ditto.

  • assembler/AbstractMacroAssembler.h:

(JSC::isARMv7s): Removed.

  • assembler/MacroAssemblerARMv7.cpp: Added.

(JSC::isIntegerDivSupported): Added.

  • assembler/MacroAssemblerARMv7.h:

(JSC::MacroAssemblerARMv7::supportsIntegerDiv): Added.

  • dfg/DFGFixupPhase.cpp:

(JSC::DFG::FixupPhase::fixupNode): Checked MacroAssembler::supportsIntegerDiv() in ArithDiv case.

  • dfg/DFGOperations.cpp:

(JSC::DFG::operationModOnInts): Added.

  • dfg/DFGOperations.h:

(JSC::DFG::Z_DFGOperation_ZZ): Added.

  • dfg/DFGSpeculativeJIT.cpp:

(JSC::DFG::SpeculativeJIT::compileSoftModulo): Separated the X86-specific and ARM-specific codegen
from the common implementation; used operationModOnInts on ARM.
(JSC::DFG::SpeculativeJIT::compileIntegerArithDivForARM): Renamed from compileIntegerArithDivForARMv7.
(JSC::DFG::SpeculativeJIT::compileArithMod): Allowed run-time detection of integer div on ARM.

  • dfg/DFGSpeculativeJIT.h:

(JSC::DFG::SpeculativeJIT::callOperation): Added overloads with Z_DFGOperation_ZZ arguments.

  • dfg/DFGSpeculativeJIT32_64.cpp:

(JSC::DFG::SpeculativeJIT::compile): Used compileIntegerArithDivForARM.

Source/WTF:

  • wtf/Platform.h: Added ENABLE_ARM_INTEGER_DIV.
7:04 PM Changeset in webkit [149348] by roger_fong@apple.com
  • 22 edits
    3 adds in trunk/LayoutTests

Unreviewed gardening. Rebaseline some Windows tests after disabling subpixel layout.

  • platform/win/editing/selection/caret-ltr-right-expected.txt:
  • platform/win/editing/selection/caret-rtl-2-left-expected.txt:
  • platform/win/editing/selection/caret-rtl-right-expected.txt:
  • platform/win/editing/selection/extend-selection-word-expected.txt: Added.
  • platform/win/editing/selection/mixed-editability-10-expected.txt:
  • platform/win/editing/selection/vertical-lr-ltr-extend-line-backward-br-expected.txt:
  • platform/win/editing/selection/vertical-lr-ltr-extend-line-forward-br-expected.txt:
  • platform/win/editing/selection/vertical-rl-ltr-extend-line-backward-br-expected.txt:
  • platform/win/editing/selection/vertical-rl-ltr-extend-line-backward-p-expected.txt:
  • platform/win/editing/selection/vertical-rl-ltr-extend-line-forward-br-expected.txt:
  • platform/win/editing/selection/vertical-rl-ltr-extend-line-forward-p-expected.txt:
  • platform/win/editing/selection/vertical-rl-rtl-extend-line-backward-br-expected.txt:
  • platform/win/editing/selection/vertical-rl-rtl-extend-line-backward-p-expected.txt:
  • platform/win/editing/selection/vertical-rl-rtl-extend-line-forward-br-expected.txt:
  • platform/win/editing/selection/vertical-rl-rtl-extend-line-forward-p-expected.txt:
  • platform/win/fast/borders/bidi-012-expected.txt:
  • platform/win/fast/forms/fieldset-legend-padding-unclipped-fieldset-border-expected.txt:
  • platform/win/fast/forms/file/file-input-disabled-expected.txt:
  • platform/win/fast/forms/range: Added.
  • platform/win/fast/forms/range/slider-mouse-events-expected.txt: Added.
  • platform/win/fast/inline/inline-box-background-expected.txt:
  • platform/win/fast/inline/inline-box-background-long-image-expected.txt:
  • platform/win/fast/inline/inline-box-background-repeat-x-expected.txt:
  • platform/win/fast/inline/inline-box-background-repeat-y-expected.txt:
6:59 PM Changeset in webkit [149347] by commit-queue@webkit.org
  • 6 edits
    4 deletes in trunk

Remove all remaining Skia code
https://bugs.webkit.org/show_bug.cgi?id=115263

Patch by Alberto Garcia <agarcia@igalia.com> on 2013-04-29
Reviewed by Benjamin Poulain.

Source/WebCore:

  • html/HTMLCanvasElement.cpp:

(WebCore): Remove the MaxSkiaDim constant, which was a leftover
from r149193.

  • platform/graphics/harfbuzz/FontHarfBuzz.cpp: Removed.
  • platform/graphics/harfbuzz/FontPlatformDataHarfBuzz.cpp: Removed.
  • platform/graphics/harfbuzz/FontPlatformDataHarfBuzz.h: Removed.
  • platform/graphics/harfbuzz/HarfBuzzFaceSkia.cpp: Removed.

Tools:

Remove all remaining references to Skia.

  • Scripts/webkitdirs.pm:

(blackberryCMakeArguments):

  • Scripts/webkitpy/common/config/contributionareas.py:
  • Scripts/webkitpy/common/config/watchlist:
6:47 PM Changeset in webkit [149346] by roger_fong@apple.com
  • 1 edit
    22 adds in trunk/LayoutTests

Unreviewed. Windows gardening after r148987.

  • platform/win/editing/deleting/delete-by-word-002-expected.txt: Added.
  • platform/win/editing/deleting/delete-cell-contents-expected.txt: Added.
  • platform/win/editing/execCommand: Added.
  • platform/win/editing/execCommand/button-expected.txt: Added.
  • platform/win/editing/execCommand/createLink-expected.txt: Added.
  • platform/win/editing/execCommand/indent-pre-expected.txt: Added.
  • platform/win/editing/execCommand/unlink-expected.txt: Added.
  • platform/win/editing/inserting/insert-before-link-1-expected.txt: Added.
  • platform/win/editing/pasteboard/display-block-on-spans-expected.txt: Added.
  • platform/win/editing/pasteboard/merge-end-list-2-expected.txt: Added.
  • platform/win/editing/pasteboard/merge-end-table-2-expected.txt: Added.
  • platform/win/editing/pasteboard/page-zoom-expected.txt: Added.
  • platform/win/editing/style: Added.
  • platform/win/editing/style/apply-through-end-of-document-expected.txt: Added.
  • platform/win/editing/style/remove-underline-across-paragraph-expected.txt: Added.
  • platform/win/editing/style/remove-underline-across-paragraph-in-bold-expected.txt: Added.
  • platform/win/editing/style/remove-underline-expected.txt: Added.
  • platform/win/editing/style/remove-underline-from-stylesheet-expected.txt: Added.
  • platform/win/editing/style/remove-underline-in-bold-expected.txt: Added.
  • platform/win/editing/style/unbold-in-bold-expected.txt: Added.
  • platform/win/editing/undo: Added.
  • platform/win/editing/undo/replace-text-in-node-preserving-markers-crash-expected.txt: Added.
6:44 PM Changeset in webkit [149345] by andersca@apple.com
  • 3 edits in trunk/Source/WebCore

Fix build.

  • platform/graphics/SimpleFontData.cpp:

(WebCore::SimpleFontData::DerivedFontData::~DerivedFontData):

  • platform/graphics/mac/SimpleFontDataMac.mm:

(WebCore::SimpleFontData::getCompositeFontReferenceFontData):

6:22 PM Changeset in webkit [149344] by benjamin@webkit.org
  • 6 edits in trunk/Source/JavaScriptCore

Unify the data access of StringImpl members from JavaScriptCore
https://bugs.webkit.org/show_bug.cgi?id=115320

Reviewed by Andreas Kling.

DFG accesses the member infos by directly calling the methods on StringImpl,
while the baseline JIT was using helper methods on ThunkHelpers.

Cut the middle man, and use StringImpl directly everywhere.

  • jit/JITInlines.h:

(JSC::JIT::emitLoadCharacterString):

  • jit/JITPropertyAccess.cpp:

(JSC::JIT::stringGetByValStubGenerator):

  • jit/JITPropertyAccess32_64.cpp:

(JSC::JIT::stringGetByValStubGenerator):

  • jit/JSInterfaceJIT.h:
  • jit/ThunkGenerators.cpp:

(JSC::stringCharLoad):

6:03 PM Changeset in webkit [149343] by Simon Fraser
  • 2 edits in trunk/Source/WebCore

Don't do CGContext flipping when painting subframes in WebKit1 on post-MountainLion OSes
https://bugs.webkit.org/show_bug.cgi?id=115392
<rdar://problem/13421519>

Reviewed by Beth Dakin.

Sync up with AppKit behavior changes that could cause subviews to be flipped when
painted.

  • platform/mac/WidgetMac.mm:

(WebCore::Widget::paint):

5:46 PM Changeset in webkit [149342] by Chris Fleizach
  • 9 edits in trunk/Source

WEB SPEECH: need to identify the default voice per language
https://bugs.webkit.org/show_bug.cgi?id=115366

Reviewed by Tim Horton.

Source/WebCore:

Use a WKSystemInterface method to determine which voice per language is the default voice.

When looking for a voice, there may not be a default at all, so we need to search until we
find a default.

  • WebCore.exp.in:
  • platform/mac/PlatformSpeechSynthesizerMac.mm:

(-[WebSpeechSynthesisWrapper speakUtterance:WebCore::]):
(WebCore::PlatformSpeechSynthesizer::initializeVoiceList):

  • platform/mac/WebCoreSystemInterface.h:
  • platform/mac/WebCoreSystemInterface.mm:

Source/WebKit/mac:

  • WebCoreSupport/WebSystemInterface.mm:

(InitWebCoreSystemInterface):

Source/WebKit2:

  • WebProcess/WebCoreSupport/mac/WebSystemInterface.mm:

(InitWebCoreSystemInterface):

5:46 PM Changeset in webkit [149341] by andersca@apple.com
  • 2 edits in trunk/Source/WTF

It should be an error to use adoptPtr with RefCounted subclasses
https://bugs.webkit.org/show_bug.cgi?id=115389

Reviewed by Benjamin Poulain.

  • wtf/PassOwnPtr.h:

(WTF::adoptPtr):
Add COMPILE_ASSERTs.

5:34 PM Changeset in webkit [149340] by commit-queue@webkit.org
  • 4 edits in trunk

Settings.in selectionIncludesAltImageText should default to true, due to update in HTML spec.
https://bugs.webkit.org/show_bug.cgi?id=115283

Patch by James Craig <james@cookiecrook.com> on 2013-04-29
Reviewed by Chris Fleizach.

Source/WebCore:

Config change due to this spec update:
https://www.w3.org/Bugs/Public/show_bug.cgi?id=21403
http://html5.org/tools/web-apps-tracker?from=7854&to=7855

Updated existing test coverage.

  • page/Settings.in:

LayoutTests:

Update test to account for the initial setting change.

  • editing/pasteboard/copy-image-with-alt-text.html:
5:29 PM Changeset in webkit [149339] by commit-queue@webkit.org
  • 2 edits in trunk/Tools

Get rid of unused options in Tools/Scripts/webkit-build-directory
https://bugs.webkit.org/show_bug.cgi?id=115383

Patch by Seokju Kwon <Seokju Kwon> on 2013-04-29
Reviewed by Darin Adler.

  • Scripts/webkit-build-directory:
5:24 PM Changeset in webkit [149338] by andersca@apple.com
  • 7 edits in trunk/Source/WebCore

Make RunLoop ref-counted
https://bugs.webkit.org/show_bug.cgi?id=115384

Reviewed by Sam Weinig.

This is a step towards creating a shared base class that both RunLoop and WorkQueue will derive from.

  • platform/RunLoop.cpp:

(WebCore::RunLoop::current):
(WebCore::RunLoop::~RunLoop):
(WebCore):

  • platform/RunLoop.h:

(RunLoop):

  • platform/cf/RunLoopCF.cpp:

(WebCore::RunLoop::initializeMainRunLoop):
(WebCore::RunLoop::current):
(WebCore::RunLoop::~RunLoop):

5:14 PM Changeset in webkit [149337] by bfulgham@webkit.org
  • 13 edits
    2 deletes in trunk/Source/WebCore

[Windows, WinCairo] Remove link and include directives for pthread.
https://bugs.webkit.org/show_bug.cgi?id=115381

Reviewed by Benjamin Poulain.

Update various VS2005 and VS2010 property sheets to remove
reference to the pthreads Include path and instructions to
link to the pthreads library.

  • WebCore.vcproj/QTMovieWinCommon.vsprops:
  • WebCore.vcproj/WebCoreCommon.vsprops:
  • WebCore.vcproj/WebCoreDebug.vsprops:
  • WebCore.vcproj/WebCoreDebugAll.vsprops:
  • WebCore.vcproj/WebCoreProduction.vsprops:
  • WebCore.vcproj/WebCorePthreads.vsprops: Removed.
  • WebCore.vcproj/WebCoreRelease.vsprops:
  • WebCore.vcxproj/WebCoreCommon.props:
  • WebCore.vcxproj/WebCoreDebug.props:
  • WebCore.vcxproj/WebCoreDebugWinCairo.props:
  • WebCore.vcxproj/WebCoreProduction.props:
  • WebCore.vcxproj/WebCorePthreads.props: Removed.
  • WebCore.vcxproj/WebCoreRelease.props:
  • WebCore.vcxproj/WebCoreReleaseWinCairo.props:
5:12 PM Changeset in webkit [149336] by commit-queue@webkit.org
  • 3 edits in trunk/Source/WebKit/blackberry

[BlackBerry] Replace disappearing fillBuffer() API with graphics context drawing
https://bugs.webkit.org/show_bug.cgi?id=115360
Internal PR 303048.

Patch by Jakob Petsovits <jpetsovits@blackberry.com> on 2013-04-29
Reviewed by Rob Buis.

Instead of using fillBuffer() to draw directly to the
target buffer, we now lock a Drawable on it and fill it
with PlatformGraphicsContext::addPredefinedPattern().

As a bonus, this also includes related clean-ups -
simpler checkerboard painting code, removal of
fillWindow(), clearWindow() and paintDefaultBackground(),
as well as getting rid of the DEBUG_CHECKERBOARD define
which has been useless for performance tracing purposes
for a while now.

  • Api/BackingStore.cpp:

(BlackBerry::WebKit::BackingStorePrivate::blitVisibleContents):

  • Api/BackingStore_p.h:
5:02 PM Changeset in webkit [149335] by ap@apple.com
  • 2 edits in trunk/Tools

[WK2][Mac] Test with NetworkProcess enabled.

Fix the build for non-Mac.

  • WebKitTestRunner/TestController.cpp: (WTR::TestController::initialize):
4:59 PM Changeset in webkit [149334] by benjamin@webkit.org
  • 3 edits in trunk/Source/WebCore

Do not allocate Static Strings for Notification::permissionString
https://bugs.webkit.org/show_bug.cgi?id=115316

Reviewed by Geoffrey Garen.

The code is not nearly hot enough to justify keeping this memory around.

  • Modules/notifications/Notification.cpp:

(WebCore::Notification::permission):
(WebCore::Notification::permissionString):

  • Modules/notifications/Notification.h:

(Notification):

4:58 PM Changeset in webkit [149333] by ap@apple.com
  • 3 edits in trunk/LayoutTests

REGRESSION (r149287?): Assertion failure in fast/frames/flattening/iframe-flattening-crash.html
https://bugs.webkit.org/show_bug.cgi?id=115386

  • platform/mac/TestExpectations: Skipping the test.
  • platform/mac-wk2/TestExpectations: Removed it from here, as the assertion failure is not limited to wk2.
4:56 PM Changeset in webkit [149332] by benjamin@webkit.org
  • 2 edits in trunk/Source/WebKit2

[WK2] Streamline the use of strings in WebProcess::getWebCoreStatistics
https://bugs.webkit.org/show_bug.cgi?id=115315

Reviewed by Geoffrey Garen.

Remove a couple of unjustified DEFINE_STATIC_LOCAL.
Use ASCIILiteral when allocating new String.

  • WebProcess/WebProcess.cpp:

(WebKit::getWebCoreMemoryCacheStatistics):
(WebKit::WebProcess::getWebCoreStatistics):

4:54 PM Changeset in webkit [149331] by benjamin@webkit.org
  • 2 edits in trunk/Source/JavaScriptCore

Use push and pop for iOS math function thunks
https://bugs.webkit.org/show_bug.cgi?id=115215

Patch by Benjamin Poulain <bpoulain@apple.com> on 2013-04-29
Reviewed by Filip Pizlo.

The iOS ABI is a little different than regular ARM ABI regarding stack alignment.
The requirement is 4 bytes:
"The ARM environment uses a stack that—at the point of function calls—is 4-byte aligned,

grows downward, and contains local variables and a function’s parameters."

Subsequently, we can just use push and pop to preserve the link register.

  • jit/ThunkGenerators.cpp:
4:52 PM Changeset in webkit [149330] by ap@apple.com
  • 2 edits in trunk/LayoutTests

REGRESSION (r149287?): Assertion failure in fast/frames/flattening/iframe-flattening-crash.html
https://bugs.webkit.org/show_bug.cgi?id=115386

  • platform/mac-wk2/TestExpectations: Marked the test as crashing in debug.
4:26 PM Changeset in webkit [149329] by andersca@apple.com
  • 6 edits in trunk/Source

Remove StorageTracker::originsLoaded
https://bugs.webkit.org/show_bug.cgi?id=115382

Reviewed by Sam Weinig.

Source/WebCore:

Remove m_finishedImportingOriginIdentifiers.

  • storage/StorageTracker.cpp:

(WebCore::StorageTracker::StorageTracker):
(WebCore::StorageTracker::finishedImportingOriginIdentifiers):

  • storage/StorageTracker.h:

(StorageTracker):

Source/WebKit2:

Add m_didFinishLoadingOrigins.

  • WebProcess/Storage/WebKeyValueStorageManager.cpp:

(WebKit::WebKeyValueStorageManager::WebKeyValueStorageManager):
(WebKit::WebKeyValueStorageManager::getKeyValueStorageOrigins):
(WebKit::WebKeyValueStorageManager::didFinishLoadingOrigins):

  • WebProcess/Storage/WebKeyValueStorageManager.h:

(WebKeyValueStorageManager):

4:03 PM Changeset in webkit [149328] by andersca@apple.com
  • 4 edits in trunk/Source/WebCore

Get rid of StorageTracker::setClient
https://bugs.webkit.org/show_bug.cgi?id=115376

Reviewed by Sam Weinig.

  • WebCore.exp.in:
  • storage/StorageTracker.cpp:
  • storage/StorageTracker.h:

(StorageTracker):

3:54 PM Changeset in webkit [149327] by Chris Fleizach
  • 2 edits in trunk/Source/WebKit2

WEB SPEECH Update seatbelt profile to work with speech synthesis
https://bugs.webkit.org/show_bug.cgi?id=115368

Reviewed by Alexey Proskuryakov.

  • WebProcess/com.apple.WebProcess.sb.in:
3:44 PM Changeset in webkit [149326] by jpfau@apple.com
  • 15 edits
    1 add in trunk

Change approach to third-party blocking for LocalStorage
https://bugs.webkit.org/show_bug.cgi?id=115004

Reviewed by Maciej Stachowiak.

Source/WebCore:

Instead of outright blocking LocalStorage that's used in a third-party
context, silently partition it based on the origin of the top context.
These partitions are then discarded when the page group is unloaded.

No new tests; modified existing tests to account for changes.

  • page/DOMWindow.cpp:

(WebCore::DOMWindow::localStorage):

  • page/PageGroup.cpp:

(WebCore::PageGroup::localStorage):
(WebCore):
(WebCore::PageGroup::transientLocalStorage):

  • page/PageGroup.h:

(PageGroup):

  • page/SecurityOrigin.cpp:

(WebCore::SecurityOrigin::canAccessStorage):

LayoutTests:

  • http/tests/security/cross-origin-local-storage-allowed-expected.txt:
  • http/tests/security/cross-origin-local-storage-allowed.html:
  • http/tests/security/cross-origin-local-storage-expected.txt:
  • http/tests/security/cross-origin-local-storage-wk1.html:
  • http/tests/security/cross-origin-local-storage-wk1-expected.txt:
  • http/tests/security/cross-origin-local-storage.html:
  • http/tests/security/resources/cross-origin-iframe-for-local-storage.html:
  • http/tests/security/resources/load-local-storage.html: Added.
  • platform/mac-wk2/http/tests/security/cross-origin-local-storage-wk1-expected.txt:
  • platform/mac/http/tests/security/cross-origin-local-storage-wk1-expected.txt:
3:40 PM Changeset in webkit [149325] by Chris Fleizach
  • 4 edits in trunk/WebKitLibraries

WEB SPEECH: need to identify the default voice per language
https://bugs.webkit.org/show_bug.cgi?id=115366

Reviewed by Simon Fraser.

  • WebKitSystemInterface.h:
  • libWebKitSystemInterfaceLion.a:
  • libWebKitSystemInterfaceMountainLion.a:
3:31 PM Changeset in webkit [149324] by mrowe@apple.com
  • 2 edits in tags/Safari-537.39.2/Source/WebKit2

Merge r149302.

3:31 PM Changeset in webkit [149323] by mrowe@apple.com
  • 4 edits in tags/Safari-537.39.2/Source

Versioning.

3:25 PM Changeset in webkit [149322] by commit-queue@webkit.org
  • 5 edits in trunk

HTML Editing: Insertion point jumps when using link detection
https://bugs.webkit.org/show_bug.cgi?id=82198

Source/WebCore:

Patch by Yi Shen <max.hong.shen@gmail.com> on 2013-04-29
Reviewed by Ryosuke Niwa.

Add links should be done only when typing a word separator (e.g. space or dot) which
breaks the text into two parts - the link and the rest of the text. Reflected in
code, we should only add link when the caret position is at the right end location of
TextChecking result. Otherwise, the text on the right of a word separator could be
highlighted as a link and the insertion point jumps unexpectedly.

Tests: update editing/inserting/typing-space-to-trigger-smart-link.html

  • editing/Editor.cpp:

(WebCore::Editor::markAndReplaceFor):

LayoutTests:

Patch by Yi Shen <max.hong.shen@gmail.com> on 2013-04-29
Reviewed by Ryosuke Niwa.

Update typing-space-to-trigger-smart-link.html to verify link detection behavior
when typing a link before existing text.

  • editing/inserting/typing-space-to-trigger-smart-link-expected.txt:
  • editing/inserting/typing-space-to-trigger-smart-link.html:
3:19 PM Changeset in webkit [149321] by mrowe@apple.com
  • 1 copy in tags/Safari-537.39.2

New tag.

3:06 PM Changeset in webkit [149320] by bfulgham@webkit.org
  • 7 edits in trunk/Source/JavaScriptCore

[Windows, WinCairo] Get rid of last few pthread include/link references.
https://bugs.webkit.org/show_bug.cgi?id=115375

Reviewed by Tim Horton.

3:05 PM Changeset in webkit [149319] by roger_fong@apple.com
  • 2 edits in trunk/Source/JavaScriptCore

Unreviewed. AppleWin VS2010 build fix.

  • JavaScriptCore.vcxproj/JavaScriptCoreExportGenerator/JavaScriptCoreExports.def.in:
2:53 PM Changeset in webkit [149318] by roger_fong@apple.com
  • 3 edits in trunk/LayoutTests

Unreviewed. Tests mistakenly added to general TestExpectations file moved to mac TestExpectations file.

2:53 PM Changeset in webkit [149317] by Beth Dakin
  • 11 edits in trunk/Source

Need a LayoutMilestone to fire when we have done our first paint after suppressing
incremental layout
https://bugs.webkit.org/show_bug.cgi?id=115330
-and corresponding-
<rdar://problem/12722365>

Reviewed by Simon Fraser.

Source/WebCore:

To meet the needs of all of our clients, we really need two milestones. One
indicating that a layout has happened after setVisualUpdatesAllowed(true), and
another indicating that painting has happened.

If layout is needed when setVisualUpdatesAllowed(true) is called, we need to
update it so that we can guarantee the first paint is really happening at this
time. Also fire the DidFirstLayoutAfterSuppressedIncrementalRendering milestone,
and call add DidFirstPaintAfterSuppressedIncrementalRendering to the FrameView's
pending paint milestones.
up the painting milestone.

  • dom/Document.cpp:

(WebCore::Document::setVisualUpdatesAllowed):

FrameView now stores m_milestonesPendingPaint. We'll send and clear them once we
have painted.

  • page/FrameView.cpp:

(WebCore::FrameView::FrameView):
(WebCore::FrameView::paintContents):
(WebCore::FrameView::addPaintPendingMilestones):
(WebCore::FrameView::firePaintRelatedMilestones):

  • page/FrameView.h:

(WebCore::FrameView::milestonesPendingPaint):

Two new milestones.

  • page/LayoutMilestones.h:

(WebCore):

We don't need m_headerLayerAwaitingFirstFlush anymore since we can use FrameView's
pending paint milestones instead.

  • rendering/RenderLayerCompositor.cpp:

(WebCore::RenderLayerCompositor::RenderLayerCompositor):
(WebCore::RenderLayerCompositor::flushPendingLayerChanges):
(WebCore::RenderLayerCompositor::updateLayerForHeader):

  • rendering/RenderLayerCompositor.h:

(RenderLayerCompositor):

Source/WebKit2:

Two new millstones.

  • Shared/API/c/WKPageLoadTypes.h:
  • Shared/API/c/WKSharedAPICast.h:

(WebKit::toWKLayoutMilestones):
(WebKit::toLayoutMilestones):

This null-check is necessary now since this code ends up running at
WebFrame::init() time while we're setting up the Document.
setVisualUpdatesAllowed(true) has always been called as a part of that process,
and now the updateLayout(), ends up calling this code too, but we don't actually
have a mainFrame yet since it's still being created.

  • WebProcess/WebCoreSupport/WebChromeClient.cpp:

(WebKit::findLargestFrameInFrameSet):

2:40 PM Changeset in webkit [149316] by jpfau@apple.com
  • 2 edits in trunk/LayoutTests

Unreviewed, added missing new line in baseline.

  • fast/canvas/2d.imageDataHD-expected.txt:
2:36 PM Changeset in webkit [149315] by hyatt@apple.com
  • 2 edits in trunk/Source/WebCore

[Mac] Links can't be hovered or clicked with overlay scrollbars hidden.
https://bugs.webkit.org/show_bug.cgi?id=99379

This is also <rdar://problem/10929565> Overlay scrollbars prevent hit-testing on
content underneath them, even when hidden

Reviewed by Beth Dakin.

  • rendering/RenderLayer.cpp:

(WebCore::RenderLayer::verticalScrollbarWidth):
(WebCore::RenderLayer::horizontalScrollbarHeight):

2:07 PM Changeset in webkit [149314] by andersca@apple.com
  • 3 edits in trunk/Source/WebCore

Begin cleaning up StorageTracker
https://bugs.webkit.org/show_bug.cgi?id=115373

Reviewed by Andreas Kling.

Rename Mutexes to use "Mutex" and not "Guard".
Don't use static locals for ".localstorage" and "*.localstorage" strings.
Put end iterator declarations together with begin declarations.

  • storage/StorageTracker.cpp:

(WebCore::StorageTracker::setDatabaseDirectoryPath):
(WebCore::StorageTracker::trackerDatabasePath):
(WebCore::StorageTracker::openTrackerDatabase):
(WebCore::StorageTracker::finishedImportingOriginIdentifiers):
(WebCore::StorageTracker::syncImportOriginIdentifiers):
(WebCore::StorageTracker::syncFileSystemAndTrackerDatabase):
(WebCore::StorageTracker::setOriginDetails):
(WebCore::StorageTracker::syncSetOriginDetails):
(WebCore::StorageTracker::origins):
(WebCore::StorageTracker::deleteAllOrigins):
(WebCore::StorageTracker::syncDeleteAllOrigins):
(WebCore::StorageTracker::deleteOrigin):
(WebCore::StorageTracker::syncDeleteOrigin):
(WebCore::StorageTracker::willDeleteAllOrigins):
(WebCore::StorageTracker::willDeleteOrigin):
(WebCore::StorageTracker::canDeleteOrigin):
(WebCore::StorageTracker::cancelDeletingOrigin):
(WebCore::StorageTracker::setClient):
(WebCore::StorageTracker::databasePathForOrigin):
(WebCore::StorageTracker::diskUsageForOrigin):

  • storage/StorageTracker.h:

(StorageTracker):

2:06 PM Changeset in webkit [149313] by akling@apple.com
  • 2 edits in trunk/Source/WebCore

Don't check for @media rules affected by viewport changes in every layout.
<http://webkit.org/b/115370>
<rdar://problem/13765739>

Reviewed by Antti Koivisto.

Move the logic that checks if any @media rules are affected by a change in the viewport
size to FrameView::setFrameRect(). This way it's no longer necessary to do the style
recalc synchronously, which allows us to coalesce multiple updates in some cases.

  • page/FrameView.cpp:

(WebCore::FrameView::setFrameRect):
(WebCore::FrameView::layout):

2:00 PM Changeset in webkit [149312] by roger_fong@apple.com
  • 1 edit
    1 copy in trunk/WebKitLibraries

Need a tools32 folder for VS2010 builds.
tools folder will be nuked when we nuke VS2005.

  • win/tools32: Copied from WebKitLibraries/win/tools.
1:55 PM Changeset in webkit [149311] by andersca@apple.com
  • 4 edits in trunk/Source/WebCore

Remove the last StorageSyncManager function
https://bugs.webkit.org/show_bug.cgi?id=115371

Reviewed by Sam Weinig.

Move the import scheduling to the StorageAreaSync constructor. Remove code to set m_importComplete to true
since that could only happen if the StorageSyncManager's thread is null which doesn't make any sense.

  • storage/StorageAreaSync.cpp:

(WebCore::StorageAreaSync::StorageAreaSync):
(WebCore::StorageAreaSync::create):

  • storage/StorageSyncManager.cpp:
  • storage/StorageSyncManager.h:

(StorageSyncManager):

1:28 PM Changeset in webkit [149310] by ap@apple.com
  • 2 edits in trunk/Tools

[WK2][Mac] Test with NetworkProcess enabled.

Reviewed by Sam Weinig.

  • WebKitTestRunner/TestController.cpp: (WTR::TestController::initialize):
1:17 PM Changeset in webkit [149309] by zandobersek@gmail.com
  • 2 edits in trunk/Tools

[GTK] Set up the TestWebCore in TestWebKitAPI
https://bugs.webkit.org/show_bug.cgi?id=115237

Reviewed by Gustavo Noronha Silva.

Set up the WebCore unit test program, consisting of the unit tests under the TestWebKitAPI/Tests/WebCore directory.
Again the approach of specifying libtool libraries multiple times in the TestWebCore program's LDFLAGS is used as
a workaround for the layer violations and circular dependencies between various static libraries. Linking against
either libwebkitgtk or libwebkit2gtk shared libraries is avoided due to the unit tests covering WebCore, which
shouldn't rely on neither of the two distributable libraries (hence the dirty workaround).

  • TestWebKitAPI/GNUmakefile.am:
1:16 PM Changeset in webkit [149308] by commit-queue@webkit.org
  • 3 edits in trunk/Source/WebCore

Remove unused method RenderMeter::valueRatio()
https://bugs.webkit.org/show_bug.cgi?id=115266

Patch by Daker Fernandes Pinheiro <daker.pinheiro@openbossa.org> on 2013-04-29
Reviewed by Darin Adler.

The removed method is not used anywhere.
No new tests are required.

  • rendering/RenderMeter.cpp:
  • rendering/RenderMeter.h:

(RenderMeter):

1:10 PM Changeset in webkit [149307] by andersca@apple.com
  • 4 edits in trunk/Source/WebCore

Remove two more StorageSyncManager wrapper functions
https://bugs.webkit.org/show_bug.cgi?id=115367

Reviewed by Andreas Kling.

  • storage/StorageAreaSync.cpp:

(WebCore::StorageAreaSync::scheduleFinalSync):
(WebCore::StorageAreaSync::syncTimerFired):

  • storage/StorageSyncManager.cpp:
  • storage/StorageSyncManager.h:
12:56 PM Changeset in webkit [149306] by schenney@chromium.org
  • 3 edits
    2 adds in trunk

SVGElement destructor can use invalid iterator
https://bugs.webkit.org/show_bug.cgi?id=115361

Reviewed by Philip Rogers.

Source/WebCore:

When an SVGElement object has rare data, its destructor gets a
hash map iterator for the rare data, uses it to clear resources,
then uses the iterator to delete the rare data. However, the resource
cleanup can delete other SVG elements, thus modifying the hash map
from which the iterator came and hence invalidating the iterator
itself.

The fix is to re-get the iterator before deleting the rare data.

Test: svg/custom/svg-element-destructor-iteration-crash.html

  • svg/SVGElement.cpp:

(WebCore::SVGElement::~SVGElement): Get a new iterator after clearing rare data.

LayoutTests:

Test asserts in Debug and should also crash in memory checking builds.

  • svg/custom/svg-element-destructor-iteration-crash-expected.txt: Added.
  • svg/custom/svg-element-destructor-iteration-crash.html: Added.
12:52 PM Changeset in webkit [149305] by mhahnenberg@apple.com
  • 3 edits in trunk/Source/JavaScriptCore

~BlockAllocator should ASSERT that it has no more Regions left
https://bugs.webkit.org/show_bug.cgi?id=115287

Reviewed by Andreas Kling.

  • heap/BlockAllocator.cpp:

(JSC::BlockAllocator::~BlockAllocator):
(JSC::BlockAllocator::allRegionSetsAreEmpty):

  • heap/BlockAllocator.h:

(RegionSet):
(JSC::BlockAllocator::RegionSet::isEmpty):
(BlockAllocator):

12:08 PM Changeset in webkit [149304] by mhahnenberg@apple.com
  • 2 edits in trunk/Source/JavaScriptCore

IndexingTypes should use hex
https://bugs.webkit.org/show_bug.cgi?id=115286

Decimal is kind of confusing/hard to read because they're used as bit masks. Hex seems more appropriate.

Reviewed by Geoffrey Garen.

  • runtime/IndexingType.h:
12:07 PM Changeset in webkit [149303] by beidson@apple.com
  • 30 edits in trunk/Source

REGRESSION: We see authentication challenge sheets for favicon requests.
<rdar://problem/13753470> and https://bugs.webkit.org/show_bug.cgi?id=115288

Reviewed by Alexey Proskuryakov.

Source/WebCore:

No new tests (Not in a tested config, might fix existing test).

Rename ClientCrossOriginCredentialPolicy to ClientCredentialPolicy, make it be three options.
and move it to ResourceHandleTypes where it belongs:

  • loader/ResourceLoaderOptions.h:
  • platform/network/ResourceHandleTypes.h:

Expose the ClientCredentialPolicy the ResourceLoader was created with:

  • loader/ResourceLoader.h:

(WebCore::ResourceLoader::clientCredentialPolicy):

Rework the “should ask client” clause for the new values of ClientCredentialPolicy:

  • loader/ResourceLoader.cpp:

(WebCore::ResourceLoader::didReceiveAuthenticationChallenge):

Rework loadResourceSynchronously() to include a ClientCredentialPolicy argument:

  • loader/FrameLoader.cpp:

(WebCore::FrameLoader::loadResourceSynchronously):

  • loader/FrameLoader.h:
  • loader/LoaderStrategy.cpp:

(WebCore::LoaderStrategy::loadResourceSynchronously):

  • loader/LoaderStrategy.h:

Never ask the client for credentials for icon loads:

  • loader/icon/IconLoader.cpp:

(WebCore::IconLoader::startLoading):

Update all other users of ClientCredentialPolicy to the appropriate new value,
and update all users of loadResourceSynchronously to the new function signature:

  • inspector/InspectorFrontendHost.cpp:

(WebCore::InspectorFrontendHost::loadResourceSynchronously):

  • loader/DocumentLoader.cpp:

(WebCore::DocumentLoader::startLoadingMainResource):

  • loader/DocumentThreadableLoader.cpp:

(WebCore::DocumentThreadableLoader::loadRequest):

  • loader/FrameLoader.cpp:

(WebCore::FrameLoader::loadResourceSynchronously):

  • loader/FrameLoader.h:

(FrameLoader):

  • loader/LoaderStrategy.cpp:

(WebCore::LoaderStrategy::loadResourceSynchronously):

  • loader/LoaderStrategy.h:

(LoaderStrategy):

  • loader/NetscapePlugInStreamLoader.cpp:

(WebCore::NetscapePlugInStreamLoader::NetscapePlugInStreamLoader):

  • loader/cache/CachedResourceLoader.cpp:

(WebCore::CachedResourceLoader::requestUserCSSStyleSheet):
(WebCore::CachedResourceLoader::defaultCachedResourceOptions):

  • xml/XSLTProcessorLibxslt.cpp:

(WebCore::docLoaderFunc):

  • xml/parser/XMLDocumentParserLibxml2.cpp:

(WebCore::openFunc):

  • WebCore.exp.in:

Source/WebKit2:

Have NetworkResourceLoadParameters remember the ClientCredentialPolicy:

  • Shared/Network/NetworkResourceLoadParameters.cpp:

(WebKit::NetworkResourceLoadParameters::NetworkResourceLoadParameters):
(WebKit::NetworkResourceLoadParameters::encode):
(WebKit::NetworkResourceLoadParameters::decode):

  • Shared/Network/NetworkResourceLoadParameters.h:

Pass along the ClientCredentialPolicy to the NetworkProcess:

  • WebProcess/Network/WebResourceLoadScheduler.cpp:

(WebKit::WebResourceLoadScheduler::scheduleLoad):

  • WebProcess/WebCoreSupport/WebPlatformStrategies.cpp:

(WebKit::WebPlatformStrategies::loadResourceSynchronously):

  • WebProcess/WebCoreSupport/WebPlatformStrategies.h:

Have the SchedulableLoader remember the ClientCredentialPolicy:

  • NetworkProcess/SchedulableLoader.cpp:

(WebKit::SchedulableLoader::SchedulableLoader):

  • NetworkProcess/SchedulableLoader.h:

(WebKit::SchedulableLoader::clientCredentialPolicy):

Don’t message for credentials if the ClientCredentialPolicy forbids it:

  • NetworkProcess/NetworkResourceLoader.cpp:

(WebKit::NetworkResourceLoader::didReceiveAuthenticationChallenge):

  • NetworkProcess/NetworkResourceLoader.h:

Update for the new loadResourceSynchronously signature (which is still a no-op on the NetworkProcess side):

  • NetworkProcess/NetworkProcessPlatformStrategies.cpp:

(WebKit::NetworkProcessPlatformStrategies::loadResourceSynchronously):

  • NetworkProcess/NetworkProcessPlatformStrategies.h:
11:53 AM Changeset in webkit [149302] by jer.noble@apple.com
  • 2 edits in trunk/Source/WebKit2

Unreviewed build fix. Point the Network process towards its own version of SecItemShim.dyld.

  • NetworkProcess/EntryPoint/mac/XPCService/NetworkService/Info.plist:
11:31 AM Changeset in webkit [149301] by fpizlo@apple.com
  • 16 edits
    4 adds in branches/dfgFourthTier/Source

fourthTier: ASSERT that commonly used not-thread-safe methods in the runtime are not being called during compilation
https://bugs.webkit.org/show_bug.cgi?id=115297

Source/JavaScriptCore:

Reviewed by Geoffrey Garen.

Put in assertions that we're not doing bad things in compilation threads. Also
factored compilation into compile+link so that even though we don't yet have
concurrent compilation, we can be explicit about which parts of DFG work are
meant to be concurrent, and which aren't.

Also fix a handful of bugs found by these assertions.

  • JavaScriptCore.xcodeproj/project.pbxproj:
  • bytecode/ResolveGlobalStatus.cpp:

(JSC::computeForStructure):

  • bytecode/Watchpoint.cpp:

(JSC::WatchpointSet::add):
(JSC::InlineWatchpointSet::inflateSlow):

  • dfg/DFGDriver.cpp:

(JSC::DFG::compile):

  • dfg/DFGJITCompiler.cpp:

(JSC::DFG::JITCompiler::~JITCompiler):
(DFG):
(JSC::DFG::JITCompiler::compileBody):
(JSC::DFG::JITCompiler::compile):
(JSC::DFG::JITCompiler::link):
(JSC::DFG::JITCompiler::compileFunction):
(JSC::DFG::JITCompiler::linkFunction):

  • dfg/DFGJITCompiler.h:

(JITCompiler):

  • ftl/FTLCompile.cpp:

(JSC::FTL::compile):

  • ftl/FTLCompile.h:

(FTL):

  • ftl/FTLLink.cpp: Added.

(FTL):
(JSC::FTL::compileEntry):
(JSC::FTL::link):

  • ftl/FTLLink.h: Added.

(FTL):

  • ftl/FTLState.cpp:

(JSC::FTL::State::State):

  • ftl/FTLState.h:

(FTL):
(State):

  • runtime/Structure.cpp:

(JSC::Structure::get):
(JSC::Structure::prototypeChainMayInterceptStoreTo):

  • runtime/Structure.h:

(JSC::Structure::materializePropertyMapIfNecessary):

  • runtime/StructureInlines.h:

(JSC::Structure::get):

Source/WTF:

Reviewed by Geoffrey Garen.

Taught WTF the notion of compilation threads. This allows all parts of our stack
to assert that we're not being called from a JSC compilation thread. This is in
WTF because it will probably end up being used in StringImpl and WTFString.

  • WTF.xcodeproj/project.pbxproj:
  • wtf/CompilationThread.cpp: Added.

(WTF):
(WTF::initializeCompilationThreadsOnce):
(WTF::initializeCompilationThreads):
(WTF::isCompilationThread):
(WTF::exchangeIsCompilationThread):

  • wtf/CompilationThread.h: Added.

(WTF):
(CompilationScope):
(WTF::CompilationScope::CompilationScope):
(WTF::CompilationScope::~CompilationScope):
(WTF::CompilationScope::leaveEarly):

11:26 AM Changeset in webkit [149300] by andersca@apple.com
  • 3 edits in trunk/Source/WebCore

Add a StorageSyncManager::dispatch helper function
https://bugs.webkit.org/show_bug.cgi?id=115365

Reviewed by Andreas Kling.

  • storage/StorageSyncManager.cpp:

(WebCore::StorageSyncManager::dispatch):
(WebCore):
(WebCore::StorageSyncManager::close):
(WebCore::StorageSyncManager::scheduleImport):
(WebCore::StorageSyncManager::scheduleSync):
(WebCore::StorageSyncManager::scheduleDeleteEmptyDatabase):

  • storage/StorageSyncManager.h:

(StorageSyncManager):

11:19 AM Changeset in webkit [149299] by andersca@apple.com
  • 2 edits in trunk/Source/WebCore

Re-indent StorageSyncManager and remove SecurityOrigin forward declaration.

Rubber-stamped by Andreas Kling.

  • storage/StorageSyncManager.h:
11:08 AM Changeset in webkit [149298] by ap@apple.com
  • 2 edits in trunk/Source/WebKit2

Threads get stuck in NetworkProcess when canceling loads
https://bugs.webkit.org/show_bug.cgi?id=115319

Backing out part of the fix, as it's causing crashes.

  • WebProcess/Network/WebResourceLoader.cpp: (WebKit::WebResourceLoader::didReceiveResponseWithCertificateInfo):
11:06 AM Changeset in webkit [149297] by andersca@apple.com
  • 12 edits
    2 deletes in trunk/Source/WebCore

Get rid of StorageTask
https://bugs.webkit.org/show_bug.cgi?id=115313

Reviewed by Andreas Kling.

  • CMakeLists.txt:
  • GNUmakefile.list.am:
  • Target.pri:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.vcxproj/WebCore.vcxproj:
  • WebCore.vcxproj/WebCore.vcxproj.filters:
  • WebCore.xcodeproj/project.pbxproj:
  • storage/StorageSyncManager.cpp:
  • storage/StorageTask.cpp: Removed.
  • storage/StorageTask.h: Removed.
  • storage/StorageThread.cpp:

(WebCore::StorageThread::threadEntryPoint):
(WebCore::StorageThread::dispatch):
(WebCore::StorageThread::terminate):

  • storage/StorageThread.h:

(StorageThread):

  • storage/StorageTracker.cpp:
10:38 AM Changeset in webkit [149296] by zandobersek@gmail.com
  • 2 edits in trunk/Tools

[GTK] Set up the TestJavaScriptCore in TestWebKitAPI
https://bugs.webkit.org/show_bug.cgi?id=115236

Reviewed by Gustavo Noronha Silva.

Set up the TestJavaScriptCore unit test program, consisting of the unit tests
under the TestWebKitAPI/Tests/JavaScriptCore directory.

  • TestWebKitAPI/GNUmakefile.am:
10:30 AM May 2013 Meeting edited by Simon Fraser
(diff)
10:13 AM Changeset in webkit [149295] by Patrick Gansterer
  • 2 edits in trunk/Source/WTF

[WIN] Add workaround for UNUSED_PARAM()
https://bugs.webkit.org/show_bug.cgi?id=115350

Reviewed by Andreas Kling.

r147882 added an workaround for the MSVC compiler on Windows CE.
Extend this workaround to whole MSVC compiler, since the same
problem occurs with the Visual Studio 10 compiler too.

  • wtf/UnusedParam.h:
10:12 AM Changeset in webkit [149294] by akling@apple.com
  • 2 edits in trunk/Source/WebCore

Add branch-prediction hint to FAST_RETURN_IF_NO_FRONTENDS.
<http://webkit.org/b/115357>

Reviewed by Anders Carlsson.

Since this macro is used in some very hot code paths, give the compiler an extra hint
to try and minimize the impact it has when the inspector is not active.

  • inspector/InspectorInstrumentation.h:
9:33 AM Changeset in webkit [149293] by Patrick Gansterer
  • 6 edits in trunk/Source

[WIN] Add WTF_USE_IMLANG_FONT_LINK2
https://bugs.webkit.org/show_bug.cgi?id=115198

Reviewed by Andreas Kling.

Source/WebCore:

Add this new define to have a central place for switching
the IMLangFontLinkType and adop all usages of the type.
This makes the usage of the code easier on desktop pcs.

  • platform/graphics/FontCache.h:

(WebCore):

  • platform/graphics/wince/FontCacheWinCE.cpp:

(WebCore::currentFontContainsCharacter):
(WebCore::createMLangFont):
(WebCore::FontCache::getFontDataForCharacters):

  • platform/graphics/wince/SimpleFontDataWinCE.cpp:

(WebCore::SimpleFontData::containsCharacters):

Source/WTF:

  • wtf/Platform.h:
9:24 AM Changeset in webkit [149292] by commit-queue@webkit.org
  • 7 edits in trunk/Source

Get rid of "non-composited contents" in CoordinatedLayerTreeHost
https://bugs.webkit.org/show_bug.cgi?id=110355

Patch by Noam Rosenthal <Noam Rosenthal> on 2013-04-29
Reviewed by Jocelyn Turcotte.

Source/WebCore:

When in force compositing mode, always assume that the main layer needs
a backing store. Make setVisibleContentRectTrajectoryVector and accumulatedCoverRect
recursive so that they don't rely on a specialized layer.

No new testable behavior, changes to coverRect and trajectory vector only
affect tiling latency in rare cases that are not trivial to test.

  • platform/graphics/texmap/coordinated/CoordinatedGraphicsLayer.cpp:

(WebCore::CoordinatedGraphicsLayer:setVisibleContentRectTrajectoryVector):

Make the trajectory vector recursive, though limited only to layers with
translate/identity. This allows us to keep the trajectory vector while removing
the non-composited contents specialization.

(WebCore::CoordinatedGraphicsLayer::accumulatedCoverRect):

Added accumulatedCoverRect, so that the coverRect calculations for the UI process
are not bound to the non-composited contents layer.

(WebCore::CoordinatedGraphicsLayer::findDescendantWithContentsRecursively):

Instead of saving a reference to the non-composited contents layer, we assume that
the first layer we found recursively which has contents is the one to be used for
coverRect/trajectory calculations.

(WebCore):

  • platform/graphics/texmap/coordinated/CoordinatedGraphicsLayer.h:

(CoordinatedGraphicsLayer):

  • rendering/RenderLayerBacking.cpp:

(WebCore::RenderLayerBacking::paintsIntoWindow):

Always create a layer for the non-composited contents when in forceCompositing mode.

Source/WebKit2:

Instead of using a special non-composited contents layer, we let RenderLayerCompositor create
a proper GraphicsLayer for that content.
CoordinatedLayerTreeHost now needs to find the main contents layer for the purpose of setting
the trajectory vector and applying the cover rect.

  • WebProcess/WebPage/CoordinatedGraphics/CoordinatedLayerTreeHost.cpp:

(WebKit::CoordinatedLayerTreeHost::CoordinatedLayerTreeHost):
(WebKit::CoordinatedLayerTreeHost::setRootCompositingLayer):

Don't create the non-composited layer, instead keep a raw pointer to the root
compositing layer created by the WebCore compositor.

(WebKit::CoordinatedLayerTreeHost::setNonCompositedContentsNeedDisplay):
(WebKit::CoordinatedLayerTreeHost::setNonCompositedContentsNeedDisplayInRect):
(WebKit::CoordinatedLayerTreeHost::scrollNonCompositedContents):
(WebKit::CoordinatedLayerTreeHost::sizeDidChange):
(WebKit::CoordinatedLayerTreeHost::paintContents):
(WebKit::CoordinatedLayerTreeHost::deviceOrPageScaleFactorChanged):
(WebKit::CoordinatedLayerTreeHost::setVisibleContentsRect):

Remove non-composited layer specialization.

9:07 AM Changeset in webkit [149291] by zarvai@inf.u-szeged.hu
  • 2 edits in trunk/Source/WebKit2

[Qt] Build fix. Enabling c++11 for Qt WK2 after r149259.
https://bugs.webkit.org/show_bug.cgi?id=115345

Reviewed by Anders Carlsson.

  • WebKit2.pri:
8:59 AM Changeset in webkit [149290] by Patrick Gansterer
  • 2 edits in trunk/Source/WTF

Define USE(ICU_UNICODE) if USE(WCHAR_UNICODE) is false
https://bugs.webkit.org/show_bug.cgi?id=115349

Reviewed by Andreas Kling.

Since we support only ICU and WCHAR as Unicode backend (ICU is the default)
automatically use ICU if WCHAR was not selected via compiler defines.

  • wtf/Platform.h:
8:57 AM Changeset in webkit [149289] by Bruno de Oliveira Abinader
  • 10 edits in trunk/Source/WebKit2

[WK2][EFL] Skip cancel fullscreen request if not made from FullScreen API
https://bugs.webkit.org/show_bug.cgi?id=108201

Reviewed by Andreas Kling.

Adds a check on WebFullScreenManagerProxyEfl whether fullscreen mode was
set from FullScreen API or not, so it can skip cancelling fullscreen
request if fullscreen mode was set by application, preventing
WebProcess' FullScreenManager from crashing.

EFL unit test available: ewk_view_cancel_full_screen_request.

  • UIProcess/API/C/CoordinatedGraphics/WKView.cpp:

(WKViewExitFullScreen):

  • UIProcess/API/C/CoordinatedGraphics/WKView.h:

WKViewExitFullScreen now returns a bool, indicating whether the
operation has succeeded or not.

  • UIProcess/API/efl/ewk_view.cpp:

(ewk_view_fullscreen_exit):
Skip request if not in fullscreen mode.

  • UIProcess/API/efl/tests/test_ewk2_view.cpp:

(TEST_F):
Added ewk_view_cancel_full_screen_request unit test.

  • UIProcess/CoordinatedGraphics/WebView.cpp:

(WebKit::WebView::exitFullScreen):

  • UIProcess/CoordinatedGraphics/WebView.h:

(WebView):
exitFullScreen now returns a bool, indicating whether the operation has
succeeded or not.

  • UIProcess/WebFullScreenManagerProxy.cpp:

(WebKit::WebFullScreenManagerProxy::WebFullScreenManagerProxy):

  • UIProcess/WebFullScreenManagerProxy.h:

(WebFullScreenManagerProxy):

  • UIProcess/efl/WebFullScreenManagerProxyEfl.cpp:

(WebKit::WebFullScreenManagerProxy::isFullScreen):
(WebKit::WebFullScreenManagerProxy::enterFullScreen):
(WebKit::WebFullScreenManagerProxy::exitFullScreen):
Added m_hasRequestedFullScreen boolean flag to EFL port. Used to
indicate if fullscreen mode is enabled, as well as sanity checks on both
{enter,exit}FullScreen functions.

8:46 AM Changeset in webkit [149288] by krit@webkit.org
  • 4 edits in trunk

Animate clip rect() between different Length types
https://bugs.webkit.org/show_bug.cgi?id=115336

Reviewed by Andreas Kling.

Source/WebCore:

At the moment the animation for the 'clip' property
has an early return if the type of two Length values
does not match. This patch removes this early return.

Modified existing tests to cover the changes.

  • page/animation/CSSPropertyAnimation.cpp:

(WebCore::blendFunc):

LayoutTests:

Modify test for clip animation to cover animations between
different Length types.

  • transitions/clip-transition.html:
8:33 AM Changeset in webkit [149287] by akling@apple.com
  • 5 edits in trunk/Source/WebCore

When updating geometry, send JavaScript resize before before layout/paint.
<http://webkit.org/b/115199>
<rdar://problem/13735535>

Reviewed by Geoffrey Garen.

Make the "resize" event synchronous and dispatch it before doing any automatic
resize-induced layout. This avoids unnecessary work in cases where an event handler
does something that dirties style.

It also avoids a kind of JS-side FOUC that happens if we drop out to the runloop
and paint before the resize event handler runs; for example, when viewing a popup
on <http://lokeshdhakar.com/projects/lightbox2/> and resizing the window, the dark
overlay sometimes resizes out of sync with the web view, since a paint may happen
between automatic layout and the resize event.

  • page/EventHandler.h:
  • page/EventHandler.cpp:

(WebCore::EventHandler::dispatchResizeEvent):

  • page/FrameView.h:
  • page/FrameView.cpp:

(WebCore::FrameView::dispatchResizeEvent):

Renamed sendResizeEvent() to dispatchResizeEvent() and made it dispatch the event
synchronously instead of adding it to the DocumentEventQueue.

(WebCore::FrameView::performPostLayoutTasks):
(WebCore::FrameView::setFrameRect):

Dispatch the "resize" event right when the new frame rect is set.

8:30 AM Changeset in webkit [149286] by andersca@apple.com
  • 2 edits in trunk/Source/WebKit2

Stop using range-based for syntax
https://bugs.webkit.org/show_bug.cgi?id=115356

Reviewed by Andreas Kling.

  • PluginProcess/WebProcessConnection.cpp:

(WebKit::WebProcessConnection::didClose):

8:22 AM Changeset in webkit [149285] by Patrick Gansterer
  • 2 edits in trunk/Source/WTF

Build fix for WinCE after r148888 and r149097
https://bugs.webkit.org/show_bug.cgi?id=115168

Reviewed by Anders Carlsson.

When using compiler intrinsics on Windows CE the compiler complains
about wrong linkage specification of replaced CRT functions defined
in math.h. This is because the compiler has intrinsics for this
functions defined, and requires them to have extern "C" linkage.

  • wtf/MathExtras.h:

(wtf_atan2): Added extern "C" to function signature.
(wtf_fmod): Ditto.
(wtf_pow): Ditto.

8:04 AM Changeset in webkit [149284] by Patrick Gansterer
  • 5 edits in trunk/Source/WebCore

Remove OS(WINCE) from (Render|Scrollbar)ThemeWin.cpp
https://bugs.webkit.org/show_bug.cgi?id=115348

Reviewed by Andreas Kling.

Compile DIBPixelData.cpp on Windows CE too, which allows us
to get rid of a few #if OS(WINCE) in other code parts.

  • PlatformWinCE.cmake:
  • platform/graphics/win/DIBPixelData.cpp:

(WebCore::DIBPixelData::setRGBABitmapAlpha):

  • platform/win/ScrollbarThemeWin.cpp:

(WebCore::ScrollbarThemeWin::paintTrackPiece):
(WebCore::ScrollbarThemeWin::paintButton):
(WebCore::ScrollbarThemeWin::paintThumb):

  • rendering/RenderThemeWin.cpp:

(WebCore::drawControl):

7:55 AM Changeset in webkit [149283] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebKit2

[EFL][WK2]: WebProcessMain: hide call to ecore_x_shutdown() under guard
https://bugs.webkit.org/show_bug.cgi?id=115295

Patch by Sergio Correia <Sergio Correia> on 2013-04-29
Reviewed by Andreas Kling.

Since the mentioned function is declared in a header that is included
conditionally depending on HAVE_ECORE_X being defined.

  • WebProcess/efl/WebProcessMainEfl.cpp:

(WebKit::WebProcessMainEfl):

7:46 AM Changeset in webkit [149282] by commit-queue@webkit.org
  • 4 edits in trunk/Source/WebCore

[BlackBerry] Fake-bold monospace fonts need to use non-fake-bold advances
https://bugs.webkit.org/show_bug.cgi?id=115280

Patch by Eli Fidler <efidler@blackberry.com> on 2013-04-29
Reviewed by George Staikos.

BlackBerry PR 330864

Tested by fast/css/css3-ch-unit.html

  • platform/graphics/blackberry/FontPlatformDataBlackBerry.cpp:

(WebCore::FontPlatformData::applyState):
(WebCore::FontPlatformData::isFixedPitch):

  • platform/graphics/blackberry/ITypeUtils.h:
  • platform/graphics/blackberry/SimpleFontDataBlackBerry.cpp:

(WebCore::SimpleFontData::platformWidthForGlyph):

7:36 AM Changeset in webkit [149281] by commit-queue@webkit.org
  • 4 edits in trunk/Source/WebCore

[GTK][AC] Clean up code related with the children management stuff for clutter based AC
https://bugs.webkit.org/show_bug.cgi?id=114260

Patch by ChangSeok Oh <ChangSeok Oh> on 2013-04-29
Reviewed by Gustavo Noronha Silva.

Remove unused or usless code for children management stuff.

No new tests since no functionality changed.

  • platform/graphics/clutter/GraphicsLayerActor.cpp:

(_GraphicsLayerActorPrivate):
(graphics_layer_actor_class_init):
(graphicsLayerActorAllocate):
(graphicsLayerActorDraw):
(graphicsLayerActorNew):
(graphicsLayerActorSetSublayers):

  • platform/graphics/clutter/GraphicsLayerActor.h:
  • platform/graphics/clutter/GraphicsLayerClutter.cpp:

(WebCore::idleDestroy):
(WebCore::GraphicsLayerClutter::ensureStructuralLayer):
(WebCore::GraphicsLayerClutter::updateLayerDrawsContent):

7:26 AM Changeset in webkit [149280] by Patrick Gansterer
  • 3 edits in trunk/Source/WebCore

[WIN] Guard cfHDropFormat() with USE(CF) instead of OS(WINCE)
https://bugs.webkit.org/show_bug.cgi?id=115195

Reviewed by Andreas Kling.

USE(CF) is the correct guard, which works on desktop pcs too.

  • platform/win/ClipboardWin.cpp:

(WebCore::ClipboardWin::files):

  • platform/win/DragDataWin.cpp:

(WebCore::DragData::containsFiles):
(WebCore::DragData::numberOfFiles):
(WebCore::DragData::asFilenames):

7:16 AM Changeset in webkit [149279] by commit-queue@webkit.org
  • 4 edits in trunk/Source/WebKit2

[WK2] Remove build warnings by -Wunused-parameter
https://bugs.webkit.org/show_bug.cgi?id=115344

Patch by Jinwoo Song <jinwoo7.song@samsung.com> on 2013-04-29
Reviewed by Noam Rosenthal.

Fix build warnigs -Wunused-parameter.

  • UIProcess/CoordinatedGraphics/WebView.cpp:

(WebKit::WebView::createContextMenuProxy):

  • UIProcess/DrawingAreaProxy.h:

(WebKit::DrawingAreaProxy::waitForPossibleGeometryUpdate):

  • UIProcess/WebPageProxy.cpp:

(WebKit::WebPageProxy::connectionWillClose):

7:06 AM Changeset in webkit [149278] by zarvai@inf.u-szeged.hu
  • 1 edit
    67 moves
    37 adds in trunk/LayoutTests

[Qt] Unreviewed gardening.

Move Qt Mac specific results to qt-mac after r149210.

  • platform/qt-mac/compositing/visibility/visibility-image-layers-dynamic-expected.txt: Renamed from LayoutTests/platform/qt-5.0/compositing/visibility/visibility-image-layers-dynamic-expected.txt.
  • platform/qt-mac/editing/selection/collapse-selection-in-bidi-expected.txt: Renamed from LayoutTests/platform/qt-5.0/editing/selection/collapse-selection-in-bidi-expected.txt.
  • platform/qt-mac/editing/selection/drag-start-event-client-x-y-expected.txt: Renamed from LayoutTests/platform/qt-5.0/editing/selection/drag-start-event-client-x-y-expected.txt.
  • platform/qt-mac/fast/backgrounds/size/contain-and-cover-zoomed-expected.txt: Renamed from LayoutTests/platform/qt-5.0/fast/backgrounds/size/contain-and-cover-zoomed-expected.txt.
  • platform/qt-mac/fast/block/basic/fieldset-stretch-to-legend-expected.txt: Renamed from LayoutTests/platform/qt-5.0/fast/block/basic/fieldset-stretch-to-legend-expected.txt.
  • platform/qt-mac/fast/block/positioning/replaced-inside-fixed-top-bottom-expected.txt: Renamed from LayoutTests/platform/qt-5.0/fast/block/positioning/replaced-inside-fixed-top-bottom-expected.txt.
  • platform/qt-mac/fast/borders/fieldsetBorderRadius-expected.txt: Renamed from LayoutTests/platform/qt-5.0/fast/borders/fieldsetBorderRadius-expected.txt.
  • platform/qt-mac/fast/css/absolute-child-with-percent-height-inside-relative-parent-expected.txt: Renamed from LayoutTests/platform/qt-5.0/fast/css/absolute-child-with-percent-height-inside-relative-parent-expected.txt.
  • platform/qt-mac/fast/css/bug4860-absolute-block-child-does-not-inherit-alignment-expected.txt: Renamed from LayoutTests/platform/qt-5.0/fast/css/bug4860-absolute-block-child-does-not-inherit-alignment-expected.txt.
  • platform/qt-mac/fast/css/getComputedStyle/getComputedStyle-margin-percentage-expected.txt: Renamed from LayoutTests/platform/qt-5.0/fast/css/getComputedStyle/getComputedStyle-margin-percentage-expected.txt.
  • platform/qt-mac/fast/events/document-elementFromPoint-expected.txt: Renamed from LayoutTests/platform/qt-5.0/fast/events/document-elementFromPoint-expected.txt.
  • platform/qt-mac/fast/flexbox/flex-hang-expected.txt: Renamed from LayoutTests/platform/qt-5.0/fast/flexbox/flex-hang-expected.txt.
  • platform/qt-mac/fast/images/repaint-subrect-grid-expected.txt: Renamed from LayoutTests/platform/qt-5.0/fast/images/repaint-subrect-grid-expected.txt.
  • platform/qt-mac/fast/images/zoomed-img-size-expected.txt: Renamed from LayoutTests/platform/qt-5.0/fast/images/zoomed-img-size-expected.txt.
  • platform/qt-mac/fast/multicol/break-properties-expected.txt: Renamed from LayoutTests/platform/qt-5.0/fast/multicol/break-properties-expected.txt.
  • platform/qt-mac/fast/multicol/vertical-lr/break-properties-expected.txt: Renamed from LayoutTests/platform/qt-5.0/fast/multicol/vertical-lr/break-properties-expected.txt.
  • platform/qt-mac/fast/multicol/vertical-rl/break-properties-expected.txt: Renamed from LayoutTests/platform/qt-5.0/fast/multicol/vertical-rl/break-properties-expected.txt.
  • platform/qt-mac/fast/overflow/infiniteRecursionGuard-expected.txt: Renamed from LayoutTests/platform/qt-5.0/fast/overflow/infiniteRecursionGuard-expected.txt.
  • platform/qt-mac/fast/parser/style-script-head-test-expected.txt: Renamed from LayoutTests/platform/qt-5.0/fast/parser/style-script-head-test-expected.txt.
  • platform/qt-mac/fast/reflections/reflection-with-zoom-expected.txt: Renamed from LayoutTests/platform/qt-5.0/fast/reflections/reflection-with-zoom-expected.txt.
  • platform/qt-mac/fast/transforms/rotated-transform-affects-scrolling-1-expected.txt: Renamed from LayoutTests/platform/qt-5.0/fast/transforms/rotated-transform-affects-scrolling-1-expected.txt.
  • platform/qt-mac/fast/transforms/rotated-transform-affects-scrolling-2-expected.txt: Renamed from LayoutTests/platform/qt-5.0/fast/transforms/rotated-transform-affects-scrolling-2-expected.txt.
  • platform/qt-mac/http/tests/misc/acid3-expected.txt: Renamed from LayoutTests/platform/qt-5.0/http/tests/misc/acid3-expected.txt.
  • platform/qt-mac/svg/W3C-SVG-1.1-SE/coords-dom-01-f-expected.txt: Renamed from LayoutTests/platform/qt-5.0/svg/W3C-SVG-1.1-SE/coords-dom-01-f-expected.txt.
  • platform/qt-mac/svg/W3C-SVG-1.1-SE/coords-dom-02-f-expected.txt: Renamed from LayoutTests/platform/qt-5.0/svg/W3C-SVG-1.1-SE/coords-dom-02-f-expected.txt.
  • platform/qt-mac/svg/W3C-SVG-1.1-SE/coords-dom-03-f-expected.txt: Renamed from LayoutTests/platform/qt-5.0/svg/W3C-SVG-1.1-SE/coords-dom-03-f-expected.txt.
  • platform/qt-mac/svg/W3C-SVG-1.1-SE/coords-dom-04-f-expected.txt: Renamed from LayoutTests/platform/qt-5.0/svg/W3C-SVG-1.1-SE/coords-dom-04-f-expected.txt.
  • platform/qt-mac/svg/W3C-SVG-1.1-SE/filters-felem-01-b-expected.txt: Renamed from LayoutTests/platform/qt-5.0/svg/W3C-SVG-1.1-SE/filters-felem-01-b-expected.txt.
  • platform/qt-mac/svg/W3C-SVG-1.1-SE/interact-pointer-03-t-expected.txt: Renamed from LayoutTests/platform/qt-5.0/svg/W3C-SVG-1.1-SE/interact-pointer-03-t-expected.txt.
  • platform/qt-mac/svg/W3C-SVG-1.1-SE/linking-uri-01-b-expected.txt: Renamed from LayoutTests/platform/qt-5.0/svg/W3C-SVG-1.1-SE/linking-uri-01-b-expected.txt.
  • platform/qt-mac/svg/W3C-SVG-1.1-SE/painting-marker-07-f-expected.txt: Renamed from LayoutTests/platform/qt-5.0/svg/W3C-SVG-1.1-SE/painting-marker-07-f-expected.txt.
  • platform/qt-mac/svg/W3C-SVG-1.1-SE/paths-dom-02-f-expected.txt: Renamed from LayoutTests/platform/qt-5.0/svg/W3C-SVG-1.1-SE/paths-dom-02-f-expected.txt.
  • platform/qt-mac/svg/W3C-SVG-1.1-SE/pservers-grad-17-b-expected.txt: Renamed from LayoutTests/platform/qt-5.0/svg/W3C-SVG-1.1-SE/pservers-grad-17-b-expected.txt.
  • platform/qt-mac/svg/W3C-SVG-1.1-SE/pservers-grad-20-b-expected.txt: Renamed from LayoutTests/platform/qt-5.0/svg/W3C-SVG-1.1-SE/pservers-grad-20-b-expected.txt.
  • platform/qt-mac/svg/W3C-SVG-1.1-SE/pservers-pattern-04-f-expected.txt: Renamed from LayoutTests/platform/qt-5.0/svg/W3C-SVG-1.1-SE/pservers-pattern-04-f-expected.txt.
  • platform/qt-mac/svg/W3C-SVG-1.1-SE/styling-pres-02-f-expected.txt: Renamed from LayoutTests/platform/qt-5.0/svg/W3C-SVG-1.1-SE/styling-pres-02-f-expected.txt.
  • platform/qt-mac/svg/W3C-SVG-1.1-SE/text-tref-03-b-expected.txt: Renamed from LayoutTests/platform/qt-5.0/svg/W3C-SVG-1.1-SE/text-tref-03-b-expected.txt.
  • platform/qt-mac/svg/W3C-SVG-1.1-SE/text-tspan-02-b-expected.txt: Renamed from LayoutTests/platform/qt-5.0/svg/W3C-SVG-1.1-SE/text-tspan-02-b-expected.txt.
  • platform/qt-mac/svg/W3C-SVG-1.1-SE/types-dom-01-b-expected.txt: Renamed from LayoutTests/platform/qt-5.0/svg/W3C-SVG-1.1-SE/types-dom-01-b-expected.txt.
  • platform/qt-mac/svg/W3C-SVG-1.1-SE/types-dom-02-f-expected.txt: Renamed from LayoutTests/platform/qt-5.0/svg/W3C-SVG-1.1-SE/types-dom-02-f-expected.txt.
  • platform/qt-mac/svg/W3C-SVG-1.1-SE/types-dom-03-b-expected.txt: Renamed from LayoutTests/platform/qt-5.0/svg/W3C-SVG-1.1-SE/types-dom-03-b-expected.txt.
  • platform/qt-mac/svg/W3C-SVG-1.1-SE/types-dom-04-b-expected.txt: Renamed from LayoutTests/platform/qt-5.0/svg/W3C-SVG-1.1-SE/types-dom-04-b-expected.txt.
  • platform/qt-mac/svg/W3C-SVG-1.1-SE/types-dom-06-f-expected.txt: Renamed from LayoutTests/platform/qt-5.0/svg/W3C-SVG-1.1-SE/types-dom-06-f-expected.txt.
  • platform/qt-mac/svg/W3C-SVG-1.1-SE/types-dom-07-f-expected.txt: Renamed from LayoutTests/platform/qt-5.0/svg/W3C-SVG-1.1-SE/types-dom-07-f-expected.txt.
  • platform/qt-mac/svg/W3C-SVG-1.1/filters-turb-02-f-expected.txt: Renamed from LayoutTests/platform/qt-5.0/svg/W3C-SVG-1.1/filters-turb-02-f-expected.txt.
  • platform/qt-mac/svg/as-image/svg-non-integer-scaled-image-expected.txt: Renamed from LayoutTests/platform/qt-5.0/svg/as-image/svg-non-integer-scaled-image-expected.txt.
  • platform/qt-mac/svg/custom/fractional-rects-expected.txt: Renamed from LayoutTests/platform/qt-5.0/svg/custom/fractional-rects-expected.txt.
  • platform/qt-mac/svg/custom/js-update-container-expected.txt: Renamed from LayoutTests/platform/qt-5.0/svg/custom/js-update-container-expected.txt.
  • platform/qt-mac/svg/custom/object-sizing-width-50p-height-75p-on-target-svg-absolute-expected.txt: Renamed from LayoutTests/platform/qt-5.0/svg/custom/object-sizing-width-50p-height-75p-on-target-svg-absolute-expected.txt.
  • platform/qt-mac/svg/custom/object-sizing-width-50p-height-75p-on-target-svg-expected.txt: Renamed from LayoutTests/platform/qt-5.0/svg/custom/object-sizing-width-50p-height-75p-on-target-svg-expected.txt.
  • platform/qt-mac/svg/custom/object-sizing-width-75p-height-50p-on-target-svg-expected.txt: Renamed from LayoutTests/platform/qt-5.0/svg/custom/object-sizing-width-75p-height-50p-on-target-svg-expected.txt.
  • platform/qt-mac/svg/custom/svg-fonts-with-no-element-reference-expected.txt: Renamed from LayoutTests/platform/qt-5.0/svg/custom/svg-fonts-with-no-element-reference-expected.txt.
  • platform/qt-mac/svg/custom/use-css-no-effect-on-shadow-tree-expected.txt: Renamed from LayoutTests/platform/qt-5.0/svg/custom/use-css-no-effect-on-shadow-tree-expected.txt.
  • platform/qt-mac/svg/custom/viewBox-hit-expected.txt: Renamed from LayoutTests/platform/qt-5.0/svg/custom/viewBox-hit-expected.txt.
  • platform/qt-mac/svg/hixie/text/001-expected.txt: Renamed from LayoutTests/platform/qt-5.0/svg/hixie/text/001-expected.txt.
  • platform/qt-mac/svg/hixie/text/003a-expected.txt: Renamed from LayoutTests/platform/qt-5.0/svg/hixie/text/003a-expected.txt.
  • platform/qt-mac/svg/text/small-fonts-2-expected.txt: Renamed from LayoutTests/platform/qt-5.0/svg/text/small-fonts-2-expected.txt.
  • platform/qt-mac/svg/text/text-hkern-expected.txt: Renamed from LayoutTests/platform/qt-5.0/svg/text/text-hkern-expected.txt.
  • platform/qt-mac/svg/text/text-viewbox-rescale-expected.txt: Renamed from LayoutTests/platform/qt-5.0/svg/text/text-viewbox-rescale-expected.txt.
  • platform/qt-mac/svg/wicd/sizing-flakiness-expected.txt: Renamed from LayoutTests/platform/qt-5.0/svg/wicd/sizing-flakiness-expected.txt.
  • platform/qt-mac/svg/zoom/page/zoom-background-image-tiled-expected.txt: Renamed from LayoutTests/platform/qt-5.0/svg/zoom/page/zoom-background-image-tiled-expected.txt.
  • platform/qt-mac/svg/zoom/page/zoom-svg-as-background-with-relative-size-and-viewBox-expected.txt: Renamed from LayoutTests/platform/qt-5.0/svg/zoom/page/zoom-svg-as-background-with-relative-size-and-viewBox-expected.txt.
  • platform/qt-mac/svg/zoom/page/zoom-svg-as-image-expected.txt: Renamed from LayoutTests/platform/qt-5.0/svg/zoom/page/zoom-svg-as-image-expected.txt.
  • platform/qt-mac/svg/zoom/page/zoom-svg-as-object-expected.txt: Renamed from LayoutTests/platform/qt-5.0/svg/zoom/page/zoom-svg-as-object-expected.txt.
  • platform/qt-mac/svg/zoom/page/zoom-svg-as-relative-image-expected.txt: Renamed from LayoutTests/platform/qt-5.0/svg/zoom/page/zoom-svg-as-relative-image-expected.txt.
  • platform/qt-mac/svg/zoom/page/zoom-svg-through-object-with-auto-size-expected.txt: Renamed from LayoutTests/platform/qt-5.0/svg/zoom/page/zoom-svg-through-object-with-auto-size-expected.txt.
  • platform/qt-mac/tables/mozilla/bugs/bug83786-expected.txt: Renamed from LayoutTests/platform/qt-5.0/tables/mozilla/bugs/bug83786-expected.txt.
4:28 AM Changeset in webkit [149277] by Carlos Garcia Campos
  • 9 edits in trunk/Source

Unreviewed. Fix make distcheck.

Source/JavaScriptCore:

  • GNUmakefile.list.am: Add missing headers files to compilation

and offlineasm/sh4.rb script.

Source/WebCore:

  • GNUmakefile.am: Add missing scripts to EXTRA_DIST.
  • GNUmakefile.list.am: Add missing files to compilation.

Source/WebKit2:

  • GNUmakefile.list.am: Add missing header files to compilation.

Source/WTF:

  • GNUmakefile.list.am: Add missing header file to compilation.
2:52 AM Changeset in webkit [149276] by commit-queue@webkit.org
  • 4 edits in trunk/Source/WebCore

[Qt] fast/canvas/webgl/webgl-composite-modes.html broken after r148433.
https://bugs.webkit.org/show_bug.cgi?id=115243

Patch by Noam Rosenthal <Noam Rosenthal> on 2013-04-29
Reviewed by Jocelyn Turcotte.

When compositing a layer with a canvas or directly composited image, it is
wrong to use TextureMapperLayer::setBackingStore. Instead, setContentsLayer
should be used, which allows for the layer to have both a backing store and
a contents layer.

  • platform/graphics/texmap/TextureMapperLayer.cpp:

(WebCore::TextureMapperLayer::paintSelf):

  • platform/graphics/texmap/TextureMapperLayer.h:

(State):
(WebCore::TextureMapperLayer::State::State):

  • platform/graphics/texmap/coordinated/CoordinatedGraphicsScene.cpp:

(WebCore::CoordinatedGraphicsScene::createCanvasIfNeeded):
(WebCore::CoordinatedGraphicsScene::destroyCanvasIfNeeded):
(WebCore::CoordinatedGraphicsScene::assignImageBackingToLayer):

2:44 AM Changeset in webkit [149275] by Patrick Gansterer
  • 3 edits in trunk/Source/WebKit/wince

Unreviewed build fix.

  • WebCoreSupport/FrameLoaderClientWinCE.cpp: Added missing header.
  • WebView.h: Ditto.
2:03 AM Changeset in webkit [149274] by Patrick Gansterer
  • 2 edits in trunk/Source/WebCore

Unreviewed build fix.

  • bindings/js/WorkerScriptDebugServer.cpp: Added missing header.
1:56 AM Changeset in webkit [149273] by Patrick Gansterer
  • 2 edits in trunk/Source/WebCore

Unreviewed build fix.

  • platform/graphics/wince/FontWinCE.cpp:

(WebCore::generateComponents):

1:47 AM Changeset in webkit [149272] by Patrick Gansterer
  • 3 edits in trunk/Source/WebCore

Remove OS(WINCE) from HTMLSelectElement.cpp
https://bugs.webkit.org/show_bug.cgi?id=115194

Reviewed by Andreas Kling.

Use the same files like the the other PLATFORM(WIN) ports use.

  • PlatformWinCE.cmake:
  • html/HTMLSelectElement.cpp:

(WebCore):

1:05 AM Changeset in webkit [149271] by zandobersek@gmail.com
  • 2 edits in trunk/Tools

[GTK] Export the COLLECT_NO_DEMANGLE variable on the builders
https://bugs.webkit.org/show_bug.cgi?id=115323

Reviewed by Philippe Normand.

Export the COLLECT_NO_DEMANGLE environment variable on the GTK BuildBot builders.
Its presence enforces linkers to output mangled symbols when reporting linking failures
due to unresolved references. People trying to fix the build can then simply copy
the printed mangled symbol into the symbols.filter file instead of doing a manual build
or connecting to the builders to discover the mangled version of the missing symbol.

  • BuildSlaveSupport/gtk/daemontools-buildbot.conf:

Apr 28, 2013:

11:51 PM Changeset in webkit [149270] by zandobersek@gmail.com
  • 3 edits
    1 delete in trunk/Tools

Remove Chromium code from Tools/BuildSlaveSupport
https://bugs.webkit.org/show_bug.cgi?id=115325

Reviewed by Benjamin Poulain.

Remove the two Chromium-specific scripts and the Chromium-specific parts from the code
covering archiving test results and built products.

  • BuildSlaveSupport/built-product-archive:

(archiveBuiltProduct):
(extractBuiltProduct):

  • BuildSlaveSupport/chromium/output-android-device-status: Removed.
  • BuildSlaveSupport/chromium/remove-crash-logs: Removed.
  • BuildSlaveSupport/test-result-archive:

(archiveTestResults):

11:43 PM Changeset in webkit [149269] by commit-queue@webkit.org
  • 2 edits in trunk/Tools

Remove support for Chromium from webkitdirs.pm
https://bugs.webkit.org/show_bug.cgi?id=115338

Patch by Seokju Kwon <Seokju Kwon> on 2013-04-28
Reviewed by Darin Adler.

  • Scripts/webkitdirs.pm:

(determineBaseProductDir):
(argumentsForConfiguration):
(builtDylibPathForName):
(determineIsQt):
(isAppleWebKit):

11:17 PM Changeset in webkit [149268] by zandobersek@gmail.com
  • 11 edits in trunk

Rename TestRunner's setAudioData to setAudioResult
https://bugs.webkit.org/show_bug.cgi?id=115326

Reviewed by Darin Adler.

Tools:

Rename the TestRunner's setAudioData method to setAudioResult. The new name better conveys the method's purpose
than the previous one. The change is a simple rename, there's no change in the method's functionality. The
affected tests are adjusted accordingly.

  • DumpRenderTree/qt/TestRunnerQt.cpp:

(TestRunnerQt::setAudioResult):

  • DumpRenderTree/qt/TestRunnerQt.h:

(TestRunnerQt):

  • WebKitTestRunner/InjectedBundle/Bindings/TestRunner.idl:
  • WebKitTestRunner/InjectedBundle/TestRunner.cpp:

(WTR::TestRunner::setAudioResult):

  • WebKitTestRunner/InjectedBundle/TestRunner.h:

(TestRunner):

LayoutTests:

Adjust the testRunner.setAudioData references/callsites, the method is now named setAudioResult.

  • platform/efl-wk1/TestExpectations:
  • webaudio/resources/audio-codec-test.js:

(finishedLoading):

  • webaudio/resources/audio-testing.js:

(finishAudioTest):

  • webaudio/test-basic.html:
10:04 PM Changeset in webkit [149267] by gyuyoung.kim@samsung.com
  • 2 edits in trunk/Source/WebKit2

Unreviewed fix build break on EFL port since r149259.

In EFL port, there is a non-compatible code when compiling based on C++11. This is a trivial build fix.

I got a confirm from Sam about landing a trivial build fix patch without sign-off from
WK2 owner.

  • UIProcess/API/efl/EwkViewCallbacks.h:
9:13 PM Changeset in webkit [149266] by ryuan.choi@samsung.com
  • 2 edits in trunk

[EFL][CMAKE] Build break after r149259
https://bugs.webkit.org/show_bug.cgi?id=115339

Reviewed by Gyuyoung Kim.

r149259 used c++11 features(Right angle bracket, Range-based for-loop).
This patch enforces c++0x when cmake based ports build WebKit2 using gcc.

  • Source/cmake/WebKitHelpers.cmake:
5:57 PM Changeset in webkit [149265] by commit-queue@webkit.org
  • 2 edits in trunk/LayoutTests

Get rid of a duplicate file from LayoutTests/TestExpectations
https://bugs.webkit.org/show_bug.cgi?id=115337

Unreviewed gardening.

Patch by Seokju Kwon <Seokju Kwon> on 2013-04-28

  • TestExpectations: Remove one of two svg/zoom/page/zoom-replaced-intrinsic-ratio-001.htm.
5:40 PM Changeset in webkit [149264] by fpizlo@apple.com
  • 8 edits in branches/dfgFourthTier/Source/JavaScriptCore

FTL should support double variables
https://bugs.webkit.org/show_bug.cgi?id=113624

Reviewed by Geoffrey Garen.

Made all of the operations that the FTL already supports, also support doubles.
OSR exit already basically had everything it needed, so no changes there. This
mostly just glues together bits of DFG IR to LLVM IR, in a straight-forward way.

  • ftl/FTLAbbreviations.h:

(FTL):
(JSC::FTL::doubleType):
(JSC::FTL::constReal):
(JSC::FTL::buildPhi):
(JSC::FTL::addIncoming):
(JSC::FTL::buildFAdd):
(JSC::FTL::buildFSub):
(JSC::FTL::buildFMul):
(JSC::FTL::buildFNeg):
(JSC::FTL::buildSIToFP):
(JSC::FTL::buildUIToFP):
(JSC::FTL::buildBitCast):
(JSC::FTL::buildFCmp):

  • ftl/FTLCapabilities.cpp:

(JSC::FTL::canCompile):

  • ftl/FTLCommonValues.cpp:

(JSC::FTL::CommonValues::CommonValues):

  • ftl/FTLCommonValues.h:

(CommonValues):

  • ftl/FTLLowerDFGToLLVM.cpp:

(JSC::FTL::LowerDFGToLLVM::LowerDFGToLLVM):
(JSC::FTL::LowerDFGToLLVM::lower):
(JSC::FTL::LowerDFGToLLVM::compileGetLocal):
(JSC::FTL::LowerDFGToLLVM::compileSetLocal):
(JSC::FTL::LowerDFGToLLVM::compileAdd):
(JSC::FTL::LowerDFGToLLVM::compileArithSub):
(JSC::FTL::LowerDFGToLLVM::compileArithMul):
(JSC::FTL::LowerDFGToLLVM::compileArithNegate):
(JSC::FTL::LowerDFGToLLVM::compileUInt32ToNumber):
(JSC::FTL::LowerDFGToLLVM::compileGetByVal):
(JSC::FTL::LowerDFGToLLVM::compileCompareEq):
(JSC::FTL::LowerDFGToLLVM::compileCompareLess):
(JSC::FTL::LowerDFGToLLVM::lowDouble):
(LowerDFGToLLVM):
(JSC::FTL::LowerDFGToLLVM::lowJSValue):
(JSC::FTL::LowerDFGToLLVM::isCellOrMisc):
(JSC::FTL::LowerDFGToLLVM::unboxDouble):
(JSC::FTL::LowerDFGToLLVM::boxDouble):
(JSC::FTL::LowerDFGToLLVM::speculate):
(JSC::FTL::LowerDFGToLLVM::speculateNumber):
(JSC::FTL::LowerDFGToLLVM::speculateRealNumber):
(JSC::FTL::LowerDFGToLLVM::appendOSRExit):
(JSC::FTL::LowerDFGToLLVM::addExitArgumentForNode):

  • ftl/FTLOutput.h:

(JSC::FTL::Output::constDouble):
(Output):
(JSC::FTL::Output::phi):
(JSC::FTL::Output::doubleAdd):
(JSC::FTL::Output::doubleSub):
(JSC::FTL::Output::doubleMul):
(JSC::FTL::Output::doubleNeg):
(JSC::FTL::Output::intToFP):
(JSC::FTL::Output::intToDouble):
(JSC::FTL::Output::unsignedToFP):
(JSC::FTL::Output::unsignedToDouble):
(JSC::FTL::Output::bitCast):
(JSC::FTL::Output::loadDouble):
(JSC::FTL::Output::storeDouble):
(JSC::FTL::Output::doubleEqual):
(JSC::FTL::Output::doubleNotEqualOrUnordered):
(JSC::FTL::Output::doubleLessThan):
(JSC::FTL::Output::doubleLessThanOrEqual):
(JSC::FTL::Output::doubleGreaterThan):
(JSC::FTL::Output::doubleGreaterThanOrEqual):
(JSC::FTL::Output::doubleEqualOrUnordered):
(JSC::FTL::Output::doubleNotEqual):
(JSC::FTL::Output::doubleLessThanOrUnordered):
(JSC::FTL::Output::doubleLessThanOrEqualOrUnordered):
(JSC::FTL::Output::doubleGreaterThanOrUnordered):
(JSC::FTL::Output::doubleGreaterThanOrEqualOrUnordered):
(JSC::FTL::Output::testIsZero64):

3:45 PM Changeset in webkit [149263] by rniwa@webkit.org
  • 4 edits in trunk/Source/WebKit

Windows build fix.

Source/WebKit:

  • WebKit.vcxproj/WebKitExportGenerator/WebKitExports.def.in:

Source/WebKit/win:

  • WebKit.vcproj/WebKitExports.def.in:
3:21 PM Changeset in webkit [149262] by rniwa@webkit.org
  • 18 edits
    2 deletes in trunk/Source

StaticHashSetNodeList is unnecessary
https://bugs.webkit.org/show_bug.cgi?id=115219

Reviewed by Antonio Gomes.

Source/WebCore:

Moved Document::nodesFromRect into Internals::nodesFromRect per
https://chromium.googlesource.com/chromium/blink/+/78207c5c34383fbb807bb9347deb67168f09bb04.

Also made TouchAdjustment functions use HashListSet directly instead of wrapping it inside
a StaticHashSetNodeList since there was no need to use the latter.

With these two changes, I deleted StaticHashSetNodeList.

  • GNUmakefile.list.am:
  • Target.pri:
  • WebCore.exp.in:
  • WebCore.xcodeproj/project.pbxproj:
  • dom/Document.cpp:

(WebCore):

  • dom/Document.h:

(Document):

  • dom/StaticHashSetNodeList.cpp: Removed.
  • dom/StaticHashSetNodeList.h: Removed.
  • page/EventHandler.cpp:

(WebCore::EventHandler::bestClickableNodeForTouchPoint):
(WebCore::EventHandler::bestContextMenuNodeForTouchPoint):
(WebCore::EventHandler::bestZoomableAreaForTouchPoint):

  • page/TouchAdjustment.cpp:

(WebCore::TouchAdjustment::compileSubtargetList):
(WebCore::TouchAdjustment::compileZoomableSubtargets):
(WebCore::findBestClickableCandidate):
(WebCore::findBestContextMenuCandidate):
(WebCore::findBestZoomableArea):

  • page/TouchAdjustment.h:

(WebCore):

  • testing/Internals.cpp:

(WebCore::Internals::nodesFromRect):

Source/WebKit:

  • WebKit.vcxproj/WebKitExportGenerator/WebKitExports.def.in:

Source/WebKit/win:

  • WebKit.vcproj/WebKitExports.def.in:
2:52 PM Changeset in webkit [149261] by dino@apple.com
  • 12 edits
    2 deletes in trunk

[Mac] Disable canvas backing store scaling (HIGH_DPI_CANVAS)
https://bugs.webkit.org/show_bug.cgi?id=115310

Reviewed by Simon Fraser.

Source/JavaScriptCore:

Remove ENABLE_HIGH_DPI_CANVAS_macosx.

  • Configurations/FeatureDefines.xcconfig:

Source/WebCore:

Disable the automatic pixel doubling in canvas backing stores on high
dpi devices. This will potentially make canvases look worse of high dpi
displays, but only if they were ignoring devicePixelRatio. Since the
majority of high dpi displays *do not* auto-double (including mobile
devices and other desktop browsers), this provides consistency and less
chance of unnecessary memory use.

  • Configurations/FeatureDefines.xcconfig: Remove ENABLE_HIGH_DPI_CANVAS_macosx.

Source/WebKit/mac:

Remove ENABLE_HIGH_DPI_CANVAS_macosx.

  • Configurations/FeatureDefines.xcconfig:

Source/WebKit2:

Remove ENABLE_HIGH_DPI_CANVAS_macosx.

  • Configurations/FeatureDefines.xcconfig:

LayoutTests:

No need for platform-specific results any more.

  • fast/canvas/2d.backingStorePixelRatio-expected.txt:
  • fast/canvas/2d.imageDataHD-expected.txt:
  • fast/canvas/canvas-resize-reset-pixelRatio-expected.txt:
  • platform/win/fast/canvas/2d.backingStorePixelRatio-expected.txt: Removed.
  • platform/win/fast/canvas/canvas-resize-reset-pixelRatio-expected.txt: Removed.
1:31 PM Changeset in webkit [149260] by weinig@apple.com
  • 2 edits in trunk/Source/WebKit2

[WebKit2] Adopt new API for detecting window occlusion
https://bugs.webkit.org/show_bug.cgi?id=115334

Reviewed by Simon Fraser.

Switch off of WKSI for window occlusion and use the new NSWindowDidChangeOcclusionState notification
instead.

  • UIProcess/API/mac/WKView.mm:

(-[WKView dealloc]):
(-[WKView addWindowObserversForWindow:]):
(-[WKView removeWindowObservers]):
(-[WKView viewWillMoveToWindow:]):
(-[WKView viewDidMoveToWindow]):
(-[WKView _windowDidOrderOnScreen:]):
(-[WKView _windowDidChangeOcclusionState:]):
(-[WKView _setIsWindowOccluded:]):
(-[WKView initWithFrame:contextRef:pageGroupRef:relatedToPage:]):
(-[WKView windowOcclusionDetectionEnabled]):
(-[WKView setWindowOcclusionDetectionEnabled:]):
Only compile occlusion detection code where supported and remove calls to register/deregister
for detection callbacks, now that NSWindow will handle that for us. We also no longer need to
track all the WKViews, as that was only required to lookup the view that matched the window number
returned from the occlusion callback.

11:17 AM Changeset in webkit [149259] by andersca@apple.com
  • 3 edits in trunk/Source/WebKit2

Use OwnPtr<PluginControllerProxy> for storing plug-in controllers in WebProcessConnection
https://bugs.webkit.org/show_bug.cgi?id=115259

Reviewed by Andreas Kling.

  • PluginProcess/WebProcessConnection.cpp:

(WebKit::WebProcessConnection::addPluginControllerProxy):
(WebKit::WebProcessConnection::removePluginControllerProxy):
(WebKit::WebProcessConnection::didClose):

  • PluginProcess/WebProcessConnection.h:

(WebProcessConnection):

5:39 AM Changeset in webkit [149258] by zandobersek@gmail.com
  • 3 edits
    17 deletes in trunk

[WKTR] Tests dumping audio data shouldn't dump pixels
https://bugs.webkit.org/show_bug.cgi?id=115270

Reviewed by Benjamin Poulain.

Tools:

Prevent dumping the pixel output when Web Audio tests are dumping output data
as the testing output. Pixel output in these tests has no real meaning and is
as such unnecessary.

  • WebKitTestRunner/InjectedBundle/TestRunner.cpp:

(WTR::TestRunner::setAudioData):

LayoutTests:

Remove the now-unnecessary pixel baselines for audio tests that were used on the EFL WK2 platform.

  • platform/efl-wk2/webaudio/audiobuffersource-expected.png: Removed.
  • platform/efl-wk2/webaudio/audiobuffersource-loop-points-expected.png: Removed.
  • platform/efl-wk2/webaudio/audiobuffersource-multi-channels-expected.png: Removed.
  • platform/efl-wk2/webaudio/audiobuffersource-playbackrate-expected.png: Removed.
  • platform/efl-wk2/webaudio/codec-tests/aac/vbr-128kbps-44khz-expected.png: Removed.
  • platform/efl-wk2/webaudio/codec-tests/wav/24bit-22khz-resample-expected.png: Removed.
  • platform/efl-wk2/webaudio/codec-tests/wav/24bit-44khz-expected.png: Removed.
  • platform/efl-wk2/webaudio/gain-expected.png: Removed.
  • platform/efl-wk2/webaudio/mixing-expected.png: Removed.
  • platform/efl-wk2/webaudio/oscillator-custom-expected.png: Removed.
  • platform/efl-wk2/webaudio/oscillator-sawtooth-expected.png: Removed.
  • platform/efl-wk2/webaudio/oscillator-sine-expected.png: Removed.
  • platform/efl-wk2/webaudio/oscillator-square-expected.png: Removed.
  • platform/efl-wk2/webaudio/oscillator-triangle-expected.png: Removed.
  • platform/efl-wk2/webaudio/up-mixing-mono-51-expected.png: Removed.
  • platform/efl-wk2/webaudio/up-mixing-mono-stereo-expected.png: Removed.
  • platform/efl-wk2/webaudio/up-mixing-stereo-51-expected.png: Removed.
2:38 AM Changeset in webkit [149257] by Christophe Dumez
  • 61 edits in trunk/Source/WebCore

Get rid of [Callback] IDL extended attribute for parameters
https://bugs.webkit.org/show_bug.cgi?id=115250

Reviewed by Kentaro Hara.

Remove the WebKit-specific [Callback] extended attribute and make the bindings
generator smart enough to know when a parameter is of callback type on its
own.

No new tests, no behavior change.

  • DerivedSources.make: Use WebCore/Modules as IDL include path instead of using

subdirs. The generator will search recursively anyway and the list of subdirs was
incomplete.

  • Modules/filesystem/DOMWindowFileSystem.idl:
  • Modules/filesystem/DirectoryEntry.idl:
  • Modules/filesystem/DirectoryReader.idl:
  • Modules/filesystem/Entry.idl:
  • Modules/filesystem/FileEntry.idl:
  • Modules/filesystem/WorkerContextFileSystem.idl:
  • Modules/mediastream/NavigatorMediaStream.idl:
  • Modules/mediastream/RTCPeerConnection.idl:
  • Modules/notifications/Notification.idl:
  • Modules/notifications/NotificationCenter.idl:
  • Modules/quota/StorageInfo.idl:
  • Modules/quota/StorageQuota.idl:
  • Modules/webaudio/AudioContext.idl:
  • Modules/webdatabase/DOMWindowWebDatabase.idl:
  • Modules/webdatabase/Database.idl:
  • Modules/webdatabase/DatabaseSync.idl:
  • Modules/webdatabase/SQLTransaction.idl:
  • Modules/webdatabase/WorkerContextWebDatabase.idl:
  • bindings/gobject/GNUmakefile.am: Add missing IDL include dirs.
  • bindings/scripts/CodeGenerator.pm:

(IsWrapperType):
(IsCallbackInterface):

  • bindings/scripts/CodeGeneratorGObject.pm:

(SkipFunction):

  • bindings/scripts/CodeGeneratorJS.pm:

(GenerateParametersCheckExpression):
(GenerateParametersCheck):

  • bindings/scripts/IDLAttributes.txt: Remove "Callback" extended attribute.
  • bindings/scripts/test/CPP/WebDOMTestCallback.cpp:

(WebDOMTestCallback::callbackWithArrayParam):
(WebDOMTestCallback::callbackWithSerializedScriptValueParam):
(WebDOMTestCallback::callbackWithNonBoolReturnType):
(WebDOMTestCallback::callbackRequiresThisToPass):

  • bindings/scripts/test/CPP/WebDOMTestCallback.h:

(WebDOMTestCallback):

  • bindings/scripts/test/CPP/WebDOMTestObj.cpp:

(WebDOMTestObj::convert1):
(WebDOMTestObj::convert2):
(WebDOMTestObj::convert4):
(WebDOMTestObj::convert5):

  • bindings/scripts/test/CPP/WebDOMTestObj.h:
  • bindings/scripts/test/CPP/WebDOMTestSerializedScriptValueInterface.cpp:

(WebDOMTestSerializedScriptValueInterface::acceptTransferList):
(WebDOMTestSerializedScriptValueInterface::multiTransferList):

  • bindings/scripts/test/CPP/WebDOMTestSerializedScriptValueInterface.h:

(WebDOMTestSerializedScriptValueInterface):

  • bindings/scripts/test/CPP/WebDOMTestTypedefs.cpp:

(WebDOMTestTypedefs::multiTransferList):

  • bindings/scripts/test/CPP/WebDOMTestTypedefs.h:

(WebDOMTestTypedefs):

  • bindings/scripts/test/GObject/WebKitDOMTestCallback.cpp:

(webkit_dom_test_callback_callback_with_array_param):
(webkit_dom_test_callback_callback_with_serialized_script_value_param):
(webkit_dom_test_callback_callback_with_non_bool_return_type):
(webkit_dom_test_callback_callback_requires_this_to_pass):

  • bindings/scripts/test/GObject/WebKitDOMTestCallback.h:
  • bindings/scripts/test/GObject/WebKitDOMTestObj.cpp:

(webkit_dom_test_obj_convert1):
(webkit_dom_test_obj_convert2):
(webkit_dom_test_obj_convert4):
(webkit_dom_test_obj_convert5):

  • bindings/scripts/test/GObject/WebKitDOMTestObj.h:
  • bindings/scripts/test/GObject/WebKitDOMTestSerializedScriptValueInterface.cpp:

(webkit_dom_test_serialized_script_value_interface_accept_transfer_list):
(webkit_dom_test_serialized_script_value_interface_multi_transfer_list):

  • bindings/scripts/test/GObject/WebKitDOMTestSerializedScriptValueInterface.h:
  • bindings/scripts/test/GObject/WebKitDOMTestTypedefs.cpp:

(webkit_dom_test_typedefs_multi_transfer_list):

  • bindings/scripts/test/GObject/WebKitDOMTestTypedefs.h:
  • bindings/scripts/test/JS/JSTestCallback.cpp:

(WebCore::JSTestCallback::callbackWithArrayParam):
(WebCore::JSTestCallback::callbackWithSerializedScriptValueParam):
(WebCore::JSTestCallback::callbackRequiresThisToPass):

  • bindings/scripts/test/JS/JSTestCallback.h:

(JSTestCallback):

  • bindings/scripts/test/JS/JSTestObj.cpp:

(WebCore::jsTestObjPrototypeFunctionConvert1):
(WebCore::jsTestObjPrototypeFunctionConvert2):
(WebCore::jsTestObjPrototypeFunctionConvert4):
(WebCore::jsTestObjPrototypeFunctionConvert5):

  • bindings/scripts/test/JS/JSTestSerializedScriptValueInterface.cpp:

(WebCore::JSTestSerializedScriptValueInterfaceConstructor::constructJSTestSerializedScriptValueInterface):
(WebCore::jsTestSerializedScriptValueInterfacePrototypeFunctionAcceptTransferList):
(WebCore::jsTestSerializedScriptValueInterfacePrototypeFunctionMultiTransferList):

  • bindings/scripts/test/JS/JSTestTypedefs.cpp:

(WebCore::jsTestTypedefsPrototypeFunctionMultiTransferList):

  • bindings/scripts/test/ObjC/DOMTestCallback.h:
  • bindings/scripts/test/ObjC/DOMTestCallback.mm:

(-[DOMTestCallback callbackWithArrayParam:]):
(-[DOMTestCallback callbackWithSerializedScriptValueParam:strArg:]):
(-[DOMTestCallback callbackWithNonBoolReturnType:]):
(-[DOMTestCallback callbackRequiresThisToPass:testNodeParam:]):

  • bindings/scripts/test/ObjC/DOMTestObj.h:
  • bindings/scripts/test/ObjC/DOMTestObj.mm:

(-[DOMTestObj convert1:]):
(-[DOMTestObj convert2:]):
(-[DOMTestObj convert4:]):
(-[DOMTestObj convert5:]):

  • bindings/scripts/test/ObjC/DOMTestSerializedScriptValueInterface.h:
  • bindings/scripts/test/ObjC/DOMTestSerializedScriptValueInterface.mm:

(-[DOMTestSerializedScriptValueInterface acceptTransferList:transferList:]):
(-[DOMTestSerializedScriptValueInterface multiTransferList:tx:second:txx:]):

  • bindings/scripts/test/ObjC/DOMTestTypedefs.h:
  • bindings/scripts/test/ObjC/DOMTestTypedefs.mm:

(-[DOMTestTypedefs multiTransferList:tx:second:txx:]):

  • bindings/scripts/test/TestCallback.idl: Use valid types instead of Class1, Class2, ...

so that the bindings generator does not complain about not being able to resolve
them.

  • bindings/scripts/test/TestObj.idl: Use valid parameter types instead of 'a', 'b', 'd', 'e'

so that the bindings generator does not complain about them.

  • bindings/scripts/test/TestSerializedScriptValueInterface.idl:
  • bindings/scripts/test/TestTypedefs.idl:
  • css/FontLoader.idl:
  • dom/DataTransferItem.idl:
  • page/DOMWindow.idl:

Apr 27, 2013:

9:53 PM Changeset in webkit [149256] by jberlin@webkit.org
  • 2 edits in trunk/Source/WebKit2

Build fix.

  • NetworkProcess/NetworkResourceLoader.cpp:

(WebKit::NetworkResourceLoader::didReceiveBuffer):
Do not declare another variable with the same name.

8:51 PM Changeset in webkit [149255] by Darin Adler
  • 171 edits in trunk/Source

Move from constructor and member function adoptCF/NS to free function adoptCF/NS.
https://bugs.webkit.org/show_bug.cgi?id=115307

Reviewed by Geoffrey Garen.

Source/JavaScriptCore:

  • heap/HeapTimer.cpp:

(JSC::HeapTimer::HeapTimer):

  • runtime/VM.cpp:

(JSC::enableAssembler):
Use adoptCF free function.

Source/WebCore:

This change was done by a global replace followed by compiling and inspecting for
cases that needed to be tweaked by hand.

  • accessibility/ios/AXObjectCacheIOS.mm:

(WebCore::AXObjectCache::attachWrapper):

  • accessibility/mac/AXObjectCacheMac.mm:

(WebCore::AXObjectCache::attachWrapper):

  • bridge/objc/objc_class.mm:

(JSC::Bindings::ObjcClass::methodNamed):
(JSC::Bindings::ObjcClass::fieldNamed):

  • bridge/objc/objc_runtime.mm:

(JSC::Bindings::ObjcField::ObjcField):

  • editing/mac/EditorMac.mm:

(WebCore::Editor::fontAttributesForSelectionStart):

  • history/cf/HistoryPropertyList.cpp:

(WebCore::HistoryPropertyListWriter::releaseData):

  • html/HTMLMediaElement.cpp:

(WebCore::createFileURLForApplicationCacheResource):

  • loader/archive/cf/LegacyWebArchive.cpp:

(WebCore::LegacyWebArchive::createPropertyListRepresentation):
(WebCore::LegacyWebArchive::create):
(WebCore::LegacyWebArchive::rawDataRepresentation):

  • loader/archive/cf/LegacyWebArchiveMac.mm:

(WebCore::LegacyWebArchive::createPropertyListRepresentation):

  • loader/mac/ResourceLoaderMac.mm:

(WebCore::ResourceLoader::willCacheResponse):

  • page/CaptionUserPreferencesMac.mm:

(WebCore::CaptionUserPreferencesMac::userPrefersCaptions):
(WebCore::CaptionUserPreferencesMac::userPrefersSubtitles):
(WebCore::CaptionUserPreferencesMac::captionsWindowCSS):
(WebCore::CaptionUserPreferencesMac::captionsBackgroundCSS):
(WebCore::CaptionUserPreferencesMac::captionsTextColor):
(WebCore::CaptionUserPreferencesMac::captionsDefaultFontCSS):
(WebCore::CaptionUserPreferencesMac::preferredLanguages):
(WebCore::trackDisplayName):

  • platform/Language.cpp:

(WebCore::displayNameForLanguageLocale):

  • platform/LocalizedStrings.cpp:

(WebCore::contextMenuItemTagSearchWeb):
(WebCore::imageTitle):

  • platform/MIMETypeRegistry.cpp:

(WebCore::initializeSupportedImageMIMETypes):
(WebCore::initializeSupportedImageMIMETypesForEncoding):

  • platform/audio/mac/AudioFileReaderMac.cpp:

(WebCore::AudioFileReader::AudioFileReader):

  • platform/cf/KURLCFNet.cpp:

(WebCore::KURL::fileSystemPath):

  • platform/cf/SharedBufferCF.cpp:

(WebCore::SharedBuffer::maybeTransferPlatformData):

  • platform/graphics/avfoundation/objc/MediaPlayerPrivateAVFoundationObjC.mm:

(WebCore::MediaPlayerPrivateAVFoundationObjC::MediaPlayerPrivateAVFoundationObjC):
(WebCore::MediaPlayerPrivateAVFoundationObjC::createVideoLayer):
(WebCore::MediaPlayerPrivateAVFoundationObjC::createAVAssetForURL):
(WebCore::MediaPlayerPrivateAVFoundationObjC::createAVPlayer):
(WebCore::MediaPlayerPrivateAVFoundationObjC::createAVPlayerItem):
(WebCore::MediaPlayerPrivateAVFoundationObjC::createVideoOutput):
(WebCore::MediaPlayerPrivateAVFoundationObjC::paintWithVideoOutput):

  • platform/graphics/ca/GraphicsLayerCA.cpp:

(WebCore::GraphicsLayerCA::setContentsToImage):

  • platform/graphics/ca/mac/LayerFlushSchedulerMac.cpp:

(WebCore::LayerFlushScheduler::schedule):

  • platform/graphics/ca/mac/PlatformCAAnimationMac.mm:

(PlatformCAAnimation::PlatformCAAnimation):

  • platform/graphics/ca/mac/PlatformCAFiltersMac.mm:

(PlatformCAFilters::filterValueForOperation):

  • platform/graphics/ca/mac/PlatformCALayerMac.mm:

(PlatformCALayer::PlatformCALayer):
(PlatformCALayer::addAnimationForKey):
(PlatformCALayer::setBackgroundColor):
(PlatformCALayer::setBorderColor):

  • platform/graphics/cg/BitmapImageCG.cpp:

(WebCore::BitmapImage::checkForSolidColor):
(WebCore::BitmapImage::getCGImageArray):

  • platform/graphics/cg/ColorCG.cpp:

(WebCore::cachedCGColor):

  • platform/graphics/cg/GraphicsContext3DCG.cpp:

(WebCore::GraphicsContext3D::ImageExtractor::extractImage):

  • platform/graphics/cg/GraphicsContextCG.cpp:

(WebCore::GraphicsContext::applyStrokePattern):
(WebCore::GraphicsContext::applyFillPattern):
(WebCore::GraphicsContext::setURLForRect):

  • platform/graphics/cg/ImageBufferCG.cpp:

(WebCore::createIOSurface):
(WebCore::ImageBuffer::ImageBuffer):
(WebCore::ImageBuffer::copyImage):
(WebCore::ImageBuffer::draw):
(WebCore::ImageBuffer::clip):
(WebCore::ImageBuffer::putByteArray):
(WebCore::CGImageEncodeToData):
(WebCore::CGImageToDataURL):
(WebCore::ImageBuffer::toDataURL):
(WebCore::ImageDataToDataURL):

  • platform/graphics/cg/ImageBufferDataCG.cpp:

(WebCore::ImageBufferData::getData):
(WebCore::ImageBufferData::putData):

  • platform/graphics/cg/ImageCG.cpp:

(WebCore::Image::imageWithColorSpace):
(WebCore::Image::drawPattern):

  • platform/graphics/cg/ImageSourceCG.cpp:

(WebCore::ImageSource::setData):
(WebCore::ImageSource::isSizeAvailable):
(WebCore::ImageSource::frameSizeAtIndex):
(WebCore::ImageSource::orientationAtIndex):
(WebCore::ImageSource::getHotSpot):
(WebCore::ImageSource::repetitionCount):
(WebCore::ImageSource::createFrameAtIndex):
(WebCore::ImageSource::frameDurationAtIndex):

  • platform/graphics/cg/PDFDocumentImage.cpp:

(WebCore::PDFDocumentImage::dataChanged):

  • platform/graphics/cg/PathCG.cpp:

(WebCore::createScratchContext):
(WebCore::Path::contains):

  • platform/graphics/cocoa/FontPlatformDataCocoa.mm:

(WebCore::FontPlatformData::FontPlatformData):
(WebCore::FontPlatformData::setFont):
(WebCore::createFeatureSettingDictionary):
(WebCore::cascadeToLastResortFontDescriptor):
(WebCore::cascadeToLastResortAndDisableSwashesFontDescriptor):
(WebCore::FontPlatformData::ctFont):
(WebCore::FontPlatformData::description):

  • platform/graphics/gpu/mac/DrawingBufferMac.mm:

(WebCore::DrawingBuffer::DrawingBuffer):

  • platform/graphics/mac/ComplexTextControllerCoreText.mm:

(-[WebCascadeList objectAtIndex:]):
(WebCore::ComplexTextController::collectComplexTextRunsForCharacters):

  • platform/graphics/mac/FontCustomPlatformData.cpp:

(WebCore::createFontCustomPlatformData):

  • platform/graphics/mac/GlyphPageTreeNodeMac.cpp:

(WebCore::GlyphPage::fill):

  • platform/graphics/mac/GraphicsContext3DMac.mm:

(WebCore::GraphicsContext3D::GraphicsContext3D):

  • platform/graphics/mac/GraphicsContextMac.mm:

(WebCore::GraphicsContext::drawFocusRing):
(WebCore::linearRGBColorSpaceRef):

  • platform/graphics/mac/ImageMac.mm:

(WebCore::BitmapImage::getTIFFRepresentation):
(WebCore::BitmapImage::getNSImage):

  • platform/graphics/mac/MediaPlayerPrivateQTKit.mm:

(WebCore::MediaPlayerPrivateQTKit::MediaPlayerPrivateQTKit):
(WebCore::MediaPlayerPrivateQTKit::createQTMovie):
(WebCore::MediaPlayerPrivateQTKit::createQTMovieView):
(WebCore::MediaPlayerPrivateQTKit::createQTVideoRenderer):
(WebCore::MediaPlayerPrivateQTKit::createQTMovieLayer):
(WebCore::MediaPlayerPrivateQTKit::setPreservesPitch):
(WebCore::addFileTypesToCache):

  • platform/graphics/mac/SimpleFontDataCoreText.cpp:

(WebCore::SimpleFontData::getCFStringAttributes):

  • platform/graphics/mac/SimpleFontDataMac.mm:

(WebCore::fontHasVerticalGlyphs):
(WebCore::SimpleFontData::getCompositeFontReferenceFontData):
(WebCore::SimpleFontData::platformCharWidthInit):
(WebCore::SimpleFontData::canRenderCombiningCharacterSequence):

  • platform/graphics/mac/WebGLLayer.mm:

(-[WebGLLayer copyImageSnapshotWithColorSpace:]):

  • platform/mac/ContentFilterMac.mm:

(WebCore::ContentFilter::ContentFilter):

  • platform/mac/ContextMenuItemMac.mm:

(WebCore::ContextMenuItem::ContextMenuItem):

  • platform/mac/CursorMac.mm:

(WebCore::createCustomCursor):
(WebCore::createNamedCursor):

  • platform/mac/DisplaySleepDisabler.cpp:

(WebCore::DisplaySleepDisabler::DisplaySleepDisabler):

  • platform/mac/DragImageMac.mm:

(WebCore::dissolveDragImageToFraction):
(WebCore::createDragImageFromImage):

  • platform/mac/HTMLConverter.mm:

(+[WebHTMLConverter editingAttributedStringFromRange:]):

  • platform/mac/Language.mm:

(WebCore::httpStyleLanguageCode):
(WebCore::platformUserPreferredLanguages):

  • platform/mac/LocalizedStringsMac.cpp:

(WebCore::localizedString):

  • platform/mac/PasteboardMac.mm:

(WebCore::Pasteboard::getDataSelection):
(WebCore::Pasteboard::writeSelectionForTypes):
(WebCore::fragmentFromWebArchive):

  • platform/mac/PlatformClockCM.mm:

(PlatformClockCM::PlatformClockCM):
(PlatformClockCM::initializeWithTimingSource):

  • platform/mac/PlatformPasteboardMac.mm:

(WebCore::PlatformPasteboard::addTypes):
(WebCore::PlatformPasteboard::setTypes):
(WebCore::PlatformPasteboard::setPathnamesForType):

  • platform/mac/PlatformSpeechSynthesizerMac.mm:

(WebCore::PlatformSpeechSynthesizer::speak):

  • platform/mac/PopupMenuMac.mm:

(WebCore::PopupMenuMac::populate):
(WebCore::PopupMenuMac::show):

  • platform/mac/ScrollAnimatorMac.mm:

(-[WebScrollbarPainterDelegate setUpAlphaAnimation:scrollerPainter:part:WebCore::animateAlphaTo:duration:]):
(-[WebScrollbarPainterDelegate scrollerImp:animateUIStateTransitionWithDuration:]):
(-[WebScrollbarPainterDelegate scrollerImp:animateExpansionTransitionWithDuration:]):
(WebCore::ScrollAnimatorMac::ScrollAnimatorMac):
(WebCore::ScrollAnimatorMac::didAddVerticalScrollbar):
(WebCore::ScrollAnimatorMac::didAddHorizontalScrollbar):

  • platform/mac/SharedBufferMac.mm:

(WebCore::SharedBuffer::createCFData):

  • platform/mac/WebCoreNSStringExtras.mm:

(stringEncodingForResource):

  • platform/network/ProtectionSpace.cpp:

(WebCore::ProtectionSpace::receivesCredentialSecurely):

  • platform/network/SynchronousLoaderClient.cpp:

(WebCore::SynchronousLoaderClient::platformBadResponseError):

  • platform/network/cf/CredentialStorageCFNet.cpp:

(WebCore::CredentialStorage::getFromPersistentStorage):

  • platform/network/cf/DNSCFNet.cpp:

(WebCore::DNSResolveQueue::platformProxyIsEnabledInSystemPreferences):

  • platform/network/cf/FormDataStreamCFNet.cpp:

(WebCore::advanceCurrentStream):

  • platform/network/cf/NetworkStorageSessionCFNet.cpp:

(WebCore::overrideCookieStorage):

  • platform/network/cf/ProxyServerCFNet.cpp:

(WebCore::proxyAutoConfigurationResultCallback):
(WebCore::processProxyServers):
(WebCore::addProxyServersForURL):

  • platform/network/cf/ResourceErrorCF.cpp:

(WebCore::ResourceError::platformLazyInit):
(WebCore::ResourceError::cfError):

  • platform/network/cf/ResourceHandleCFNet.cpp:

(WebCore::willSendRequest):
(WebCore::ResourceHandle::createCFURLConnection):
(WebCore::ResourceHandle::didReceiveAuthenticationChallenge):
(WebCore::ResourceHandle::receivedCredential):
(WebCore::ResourceHandle::handleDataArray):

  • platform/network/cf/ResourceRequestCFNet.cpp:

(WebCore::setHeaderFields):
(WebCore::ResourceRequest::doUpdatePlatformRequest):
(WebCore::ResourceRequest::doUpdatePlatformHTTPBody):
(WebCore::ResourceRequest::doUpdateResourceRequest):
(WebCore::ResourceRequest::setStorageSession):

  • platform/network/cf/ResourceResponseCFNet.cpp:

(WebCore::ResourceResponse::cfURLResponse):
(WebCore::ResourceResponse::platformLazyInit):

  • platform/network/cf/SocketStreamHandleCFNet.cpp:

(WebCore::SocketStreamHandle::SocketStreamHandle):
(WebCore::SocketStreamHandle::executePACFileURL):
(WebCore::SocketStreamHandle::chooseProxy):
(WebCore::SocketStreamHandle::createStreams):
(WebCore::SocketStreamHandle::addCONNECTCredentials):
(WebCore::SocketStreamHandle::readStreamCallback):
(WebCore::SocketStreamHandle::writeStreamCallback):
(WebCore::SocketStreamHandle::reportErrorToClient):

  • platform/network/mac/AuthenticationMac.mm:

(WebCore::mac):
(WebCore::AuthenticationChallenge::setAuthenticationClient):

  • platform/network/mac/CookieJarMac.mm:

(WebCore::filterCookies):

  • platform/network/mac/NetworkStateNotifierMac.cpp:

(WebCore::NetworkStateNotifier::updateState):
(WebCore::NetworkStateNotifier::NetworkStateNotifier):

  • platform/network/mac/ResourceErrorMac.mm:

(WebCore::createNSErrorFromResourceErrorBase):
(WebCore::ResourceError::nsError):

  • platform/network/mac/ResourceHandleMac.mm:

(WebCore::ResourceHandle::createNSURLConnection):

  • platform/network/mac/ResourceRequestMac.mm:

(WebCore::ResourceRequest::updateNSURLRequest):
(WebCore::ResourceRequest::doUpdatePlatformRequest):
(WebCore::ResourceRequest::doUpdatePlatformHTTPBody):
(WebCore::ResourceRequest::setStorageSession):

  • platform/network/mac/ResourceResponseMac.mm:

(WebCore::ResourceResponse::initNSURLResponse):
(WebCore::ResourceResponse::nsURLResponse):
(WebCore::ResourceResponse::platformLazyInit):

  • platform/network/mac/UTIUtilities.mm:

(WebCore::mimeTypeFromUTITree):

  • platform/network/mac/WebCoreURLResponse.mm:

(WebCore::adjustMIMETypeIfNecessary):

  • platform/text/cf/HyphenationCF.cpp:

(WebCore::::createValueForNullKey):
(WebCore::lastHyphenLocation):

  • platform/text/mac/HyphenationMac.mm:

(WebCore::::createValueForKey):
(WebCore::lastHyphenLocation):

  • platform/text/mac/LocaleMac.mm:

(WebCore::determineLocale):
(WebCore::LocaleMac::LocaleMac):
(WebCore::LocaleMac::initializeLocaleData):

  • platform/text/mac/TextBreakIteratorInternalICUMac.mm:

(WebCore::textBreakLocalePreference):
(WebCore::canonicalLanguageIdentifier):

  • rendering/RenderThemeMac.mm:

(WebCore::RenderThemeMac::RenderThemeMac):
(WebCore::RenderThemeMac::levelIndicatorFor):
(WebCore::RenderThemeMac::paintMenuListButtonGradients):
(WebCore::RenderThemeMac::paintSliderTrack):
(WebCore::RenderThemeMac::popupButton):
(WebCore::RenderThemeMac::search):
(WebCore::RenderThemeMac::searchMenuTemplate):
(WebCore::RenderThemeMac::sliderThumbHorizontal):
(WebCore::RenderThemeMac::sliderThumbVertical):
(WebCore::RenderThemeMac::textField):
Use adoptNS and adoptCF free functions.

Source/WebKit/cf:

  • WebCoreSupport/WebInspectorClientCF.cpp:

(populateSetting): Use adoptCF.

Source/WebKit/mac:

This change was done by a global replace followed by compiling and inspecting for
cases that needed to be tweaked by hand.

  • DefaultDelegates/WebDefaultContextMenuDelegate.mm:

(-[WebDefaultUIDelegate menuItemWithTag:target:representedObject:]):

  • Misc/WebNSFileManagerExtras.mm:

(-[NSFileManager _webkit_startupVolumeName]):

  • Plugins/Hosted/HostedNetscapePluginStream.mm:

(WebKit::HostedNetscapePluginStream::HostedNetscapePluginStream):

  • Plugins/Hosted/NetscapePluginHostManager.mm:

(WebKit::NetscapePluginHostManager::spawnPluginHost):
(WebKit::NetscapePluginHostManager::instantiatePlugin):

  • Plugins/Hosted/NetscapePluginHostProxy.mm:

(WebKit::NetscapePluginHostProxy::NetscapePluginHostProxy):
(WebKit::NetscapePluginHostProxy::beginModal):
(WKPCBooleanAndDataReply):

  • Plugins/Hosted/NetscapePluginInstanceProxy.mm:

(WebKit::NetscapePluginInstanceProxy::print):
(WebKit::NetscapePluginInstanceProxy::snapshot):
(WebKit::NetscapePluginInstanceProxy::status):
(WebKit::NetscapePluginInstanceProxy::loadURL):
(WebKit::NetscapePluginInstanceProxy::evaluateJavaScript):
(WebKit::NetscapePluginInstanceProxy::enumerate):
(WebKit::NetscapePluginInstanceProxy::marshalValue):
(WebKit::NetscapePluginInstanceProxy::marshalValues):
(WebKit::NetscapePluginInstanceProxy::demarshalValue):
(WebKit::NetscapePluginInstanceProxy::demarshalValues):

  • Plugins/Hosted/WebHostedNetscapePluginView.mm:

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

  • Plugins/WebBaseNetscapePluginView.mm:

(-[WebBaseNetscapePluginView initWithFrame:pluginPackage:URL:baseURL:MIMEType:attributeKeys:attributeValues:loadManually:element:WebCore::]):
(-[WebBaseNetscapePluginView cacheSnapshot]):
(WebKit::getAuthenticationInfo):

  • Plugins/WebBasePluginPackage.mm:

(-[WebBasePluginPackage initWithPath:]):

  • Plugins/WebNetscapePluginEventHandlerCarbon.mm:

(WebNetscapePluginEventHandlerCarbon::startTimers):

  • Plugins/WebNetscapePluginEventHandlerCocoa.mm:

(WebNetscapePluginEventHandlerCocoa::syntheticKeyDownWithCommandModifier):
(WebNetscapePluginEventHandlerCocoa::handleTSMEvent):

  • Plugins/WebNetscapePluginPackage.mm:

(-[WebNetscapePluginPackage _initWithPath:]):

  • Plugins/WebNetscapePluginStream.mm:

(WebNetscapePluginStream::WebNetscapePluginStream):
(WebNetscapePluginStream::deliverData):
(WebNetscapePluginStream::deliverDataToFile):
(WebNetscapePluginStream::didReceiveData):

  • Plugins/WebNetscapePluginView.mm:

(-[WebNetscapePluginView createPlugin]):
(-[WebNetscapePluginView initWithFrame:pluginPackage:URL:baseURL:MIMEType:attributeKeys:attributeValues:loadManually:element:WebCore::]):
(-[WebNetscapePluginView evaluateJavaScriptPluginRequest:]):

  • Storage/WebDatabaseManagerClient.mm:

(WebDatabaseManagerClient::dispatchDidModifyOrigin):
(WebDatabaseManagerClient::dispatchDidModifyDatabase):

  • Storage/WebStorageTrackerClient.mm:

(WebStorageTrackerClient::dispatchDidModifyOrigin):

  • WebCoreSupport/CorrectionPanel.mm:

(CorrectionPanel::handleAcceptedReplacement):

  • WebCoreSupport/WebApplicationCache.mm:

(+[WebApplicationCache originsWithCache]):

  • WebCoreSupport/WebFrameLoaderClient.mm:

(WebFrameLoaderClient::didRunInsecureContent):

  • WebCoreSupport/WebInspectorClient.mm:

(WebInspectorClient::WebInspectorClient):
(WebInspectorClient::openInspectorFrontend):

  • WebView/WebFullScreenController.mm:

(-[WebFullScreenController enterFullScreen:]):

  • WebView/WebPDFDocumentExtras.mm:

(allScriptsInPDFDocument):

  • WebView/WebPreferences.mm:

(-[WebPreferences initWithIdentifier:]):
(-[WebPreferences initWithCoder:]):

  • WebView/WebScriptDebugger.mm:

(WebScriptDebugger::initGlobalCallFrame):
(WebScriptDebugger::callEvent):

  • WebView/WebView.mm:

(-[WebView previousValidKeyView]):
Use adoptNS and adoptCF free functions.

Source/WebKit2:

This change was done by a global replace followed by compiling and inspecting for
cases that needed to be tweaked by hand.

  • Platform/cg/CGUtilities.cpp:

(WebKit::paintBitmapContext):

  • Platform/mac/ModuleMac.mm:

(WebKit::Module::load):
(WebKit::Module::platformFunctionPointer):

  • PluginProcess/mac/PluginProcessMac.mm:

(WebKit::PluginProcess::initializeSandbox):

  • Shared/API/c/mac/WKURLRequestNS.mm:

(WKURLRequestCreateWithNSURLRequest):

  • Shared/API/c/mac/WKURLResponseNS.mm:

(WKURLResponseCreateWithNSURLResponse):

  • Shared/Downloads/mac/DownloadMac.mm:

(WebKit::Download::start):
(WebKit::Download::startWithHandle):

  • Shared/EntryPointUtilities/mac/LegacyProcess/ChildProcessEntryPoint.h:

(WebKit::ChildProcessMain):

  • Shared/Plugins/Netscape/mac/NetscapePluginModuleMac.mm:

(WebKit::getPluginArchitecture):
(WebKit::contentsOfPropertyListAtURL):
(WebKit::getMIMETypesFromPluginBundle):
(WebKit::getStringListResource):
(WebKit::NetscapePluginModule::getPluginInfo):

  • Shared/ShareableResource.cpp:

(WebKit::ShareableResource::Handle::tryWrapInSharedBuffer):

  • Shared/cf/ArgumentCodersCF.cpp:

(CoreIPC::decode):
(CoreIPC::encode):

  • Shared/cg/ShareableBitmapCG.cpp:

(WebKit::ShareableBitmap::createGraphicsContext):
(WebKit::ShareableBitmap::makeCGImageCopy):
(WebKit::ShareableBitmap::makeCGImage):
(WebKit::ShareableBitmap::createCGImage):

  • Shared/mac/ArgumentCodersMac.mm:

(CoreIPC::decode):

  • Shared/mac/PlatformCertificateInfo.mm:

(WebKit::PlatformCertificateInfo::dump):

  • Shared/mac/WebCoreArgumentCodersMac.mm:

(CoreIPC::::encodePlatformData):
(CoreIPC::::decodePlatformData):

  • UIProcess/API/mac/FindIndicatorWindow.mm:

(WebKit::FindIndicatorWindow::setFindIndicator):
(WebKit::FindIndicatorWindow::startFadeOutTimerFired):

  • UIProcess/API/mac/PDFViewController.mm:

(-[WKPDFView initWithFrame:PDFViewController:]):
(WebKit::PDFViewController::PDFViewController):
(WebKit::convertPostScriptDataSourceToPDF):
(WebKit::PDFViewController::setPDFDocumentData):

  • UIProcess/API/mac/PageClientImpl.mm:

(WebKit::PageClientImpl::PageClientImpl):
(WebKit::PageClientImpl::registerEditCommand):
(WebKit::PageClientImpl::setDragImage):

  • UIProcess/API/mac/WKBrowsingContextController.mm:

(didFailProvisionalLoadWithErrorForFrame):
(didFailLoadWithErrorForFrame):

  • UIProcess/API/mac/WKConnection.mm:

(didReceiveMessage):

  • UIProcess/API/mac/WKPrintingView.mm:

(-[WKPrintingView drawRect:]):

  • UIProcess/API/mac/WKView.mm:

(-[WKView browsingContextController]):
(-[WKView _setAcceleratedCompositingModeRootLayer:]):
(-[WKView _setPromisedData:WebCore::withFileName:withExtension:withTitle:withURL:withVisibleURL:withArchive:WebCore::forPasteboard:]):
(-[WKView namesOfPromisedFilesDroppedAtDestination:]):
(-[WKView fullScreenWindowController]):
(-[WKView _executeSavedCommandBySelector:]):
(-[WKView printOperationWithPrintInfo:forFrame:]):

  • UIProcess/Launcher/mac/ProcessLauncherMac.mm:

(WebKit::createProcess):

  • UIProcess/Plugins/mac/PluginInfoStoreMac.mm:

(WebKit::PluginInfoStore::pluginPathsInDirectory):
(WebKit::PluginInfoStore::getMIMETypeForExtension):

  • UIProcess/Plugins/mac/PluginProcessProxyMac.mm:

(WebKit::PluginProcessProxy::beginModal):

  • UIProcess/cf/WebBackForwardListCF.cpp:

(WebKit::WebBackForwardList::createCFDictionaryRepresentation):

  • UIProcess/cf/WebPageProxyCF.cpp:

(WebKit::WebPageProxy::sessionStateData):
(WebKit::WebPageProxy::restoreFromSessionStateData):
(WebKit::WebPageProxy::saveRecentSearches):
(WebKit::WebPageProxy::loadRecentSearches):

  • UIProcess/cf/WebPreferencesCF.cpp:

(WebKit::cfStringFromWebCoreString):
(WebKit::setStringValueIfInUserDefaults):
(WebKit::setBoolValueIfInUserDefaults):
(WebKit::setUInt32ValueIfInUserDefaults):
(WebKit::setDoubleValueIfInUserDefaults):
(WebKit::WebPreferences::platformUpdateUInt32ValueForKey):
(WebKit::WebPreferences::platformUpdateDoubleValueForKey):
(WebKit::WebPreferences::platformUpdateFloatValueForKey):

  • UIProcess/mac/BackingStoreMac.mm:

(WebKit::BackingStore::resetScrolledRect):
(WebKit::BackingStore::backingStoreContext):

  • UIProcess/mac/CorrectionPanel.mm:

(WebKit::CorrectionPanel::handleAcceptedReplacement):

  • UIProcess/mac/TextCheckerMac.mm:

(WebKit::TextChecker::checkTextOfParagraph):
(WebKit::TextChecker::updateSpellingUIWithGrammarString):

  • UIProcess/mac/WKFullScreenWindowController.mm:

(-[WKFullScreenWindowController enterFullScreen:]):
(-[WKFullScreenWindowController _startEnterFullScreenAnimationWithDuration:]):
(-[WKFullScreenWindowController _startExitFullScreenAnimationWithDuration:]):

  • UIProcess/mac/WebContextMac.mm:

(WebKit::WebContext::platformDefaultDiskCacheDirectory):

  • UIProcess/mac/WebContextMenuProxyMac.mm:

(WebKit::nsMenuItemVector):
(WebKit::WebContextMenuProxyMac::populate):

  • UIProcess/mac/WebInspectorProxyMac.mm:

(WebKit::WebInspectorProxy::createInspectorWindow):
(WebKit::WebInspectorProxy::platformCreateInspectorPage):

  • UIProcess/mac/WebPopupMenuProxyMac.mm:

(WebKit::WebPopupMenuProxyMac::populate):
(WebKit::WebPopupMenuProxyMac::showPopupMenu):

  • WebProcess/InjectedBundle/mac/InjectedBundleMac.mm:

(WebKit::InjectedBundle::load):

  • WebProcess/Plugins/Netscape/mac/NetscapePluginMac.mm:

(WebKit::NetscapePlugin::updatePluginLayer):

  • WebProcess/Plugins/Netscape/mac/PluginProxyMac.mm:

(WebKit::PluginProxy::pluginLayer):

  • WebProcess/Plugins/PDF/PDFPlugin.mm:

(WebKit::PDFPlugin::PDFPlugin):
(WebKit::PDFPlugin::createScrollbar):
(WebKit::PDFPlugin::pdfDocumentDidLoad):
(WebKit::PDFPlugin::writeItemsToPasteboard):

  • WebProcess/Plugins/PDF/SimplePDFPlugin.mm:

(getAllScriptsInPDFDocument):
(WebKit::SimplePDFPlugin::pdfDocumentDidLoad):
(WebKit::SimplePDFPlugin::streamDidReceiveData):
(WebKit::SimplePDFPlugin::manualStreamDidReceiveData):

  • WebProcess/ResourceCache/WebResourceCacheManager.cpp:

(WebKit::WebResourceCacheManager::clearCacheForOrigin):

  • WebProcess/ResourceCache/cf/WebResourceCacheManagerCFNet.cpp:

(WebKit::WebResourceCacheManager::cfURLCacheHostNames):
(WebKit::WebResourceCacheManager::clearCFURLCacheForHostNames):

  • WebProcess/WebCoreSupport/mac/WebErrorsMac.mm:

(WebKit::createNSError):

  • WebProcess/WebPage/WebPage.cpp:

(WebKit::WebPage::drawPagesToPDF):

  • WebProcess/WebPage/mac/WebPageMac.mm:

(WebKit::WebPage::performDictionaryLookupForRange):
(WebKit::cachedResponseForURL):
(WebKit::WebPage::drawPagesToPDFFromPDFDocument):

  • WebProcess/mac/WebProcessMac.mm:

(WebKit::WebProcess::platformSetCacheModel):
Use adoptCF and adoptNS free functions.

Source/WTF:

  • wtf/RunLoopTimerCF.cpp:

(WTF::RunLoopTimerBase::start):

  • wtf/SchedulePair.h:

(WTF::SchedulePair::SchedulePair):

  • wtf/SchedulePairMac.mm:

(WTF::SchedulePair::SchedulePair):

  • wtf/unicode/icu/CollatorICU.cpp:

(WTF::Collator::userDefault):
Use adoptCF free function.

8:43 PM Changeset in webkit [149254] by rniwa@webkit.org
  • 3 edits
    2 adds in trunk

Pressing mouse button inside a dragstart event causes a crash
https://bugs.webkit.org/show_bug.cgi?id=115296

Reviewed by Darin Adler.

Source/WebCore:

Add a missing null pointer check. We should better encapsulate the states in DragState in the long term
but this is good enough for now.

Test: fast/events/mousedown-inside-dragstart-should-not-cause-crash.html

  • page/EventHandler.cpp:

(WebCore::EventHandler::handleDrag):

LayoutTests:

Added a regression test. While the bug report involves opening inspector and setting a breakpoint,
a simpler reduction that uses eventSender significantly reduces the complexity.

  • fast/events/mousedown-inside-dragstart-should-not-cause-crash-expected.txt: Added.
  • fast/events/mousedown-inside-dragstart-should-not-cause-crash.html: Added.
8:38 PM Changeset in webkit [149253] by ap@apple.com
  • 2 edits in trunk/Source/WebKit2

<rdar://problem/13757007> Crashes in NetworkResourceLoader::didReceiveResponseAsync
https://bugs.webkit.org/show_bug.cgi?id=115318

Reviewed by Darin Adler.

sendAbortingOnFailure may actually fail, and abort the request, making m_handle null.

  • NetworkProcess/NetworkResourceLoader.cpp: (WebKit::NetworkResourceLoader::didReceiveResponseAsync): Null check m_handle after sending a message, because the request will cancelled when connection is invalid. (WebKit::NetworkResourceLoader::didReceiveBuffer): Assert that m_handle matches the handle that we are called with. (WebKit::NetworkResourceLoader::didFinishLoading): Ditto. (WebKit::NetworkResourceLoader::didFail): Ditto. (WebKit::NetworkResourceLoader::willSendRequestAsync): Ditto. (WebKit::NetworkResourceLoader::didSendData): Ditto. (WebKit::NetworkResourceLoader::shouldUseCredentialStorage): Ditto. (WebKit::NetworkResourceLoader::shouldUseCredentialStorageAsync): Ditto. (WebKit::NetworkResourceLoader::didReceiveAuthenticationChallenge): Ditto. (WebKit::NetworkResourceLoader::didCancelAuthenticationChallenge): Ditto. (WebKit::NetworkResourceLoader::canAuthenticateAgainstProtectionSpaceAsync): Ditto.
8:37 PM Changeset in webkit [149252] by andersca@apple.com
  • 2 edits in trunk/Source/JavaScriptCore

Try to fix the Windows build.

8:22 PM Changeset in webkit [149251] by ap@apple.com
  • 5 edits in trunk/Source/WebKit2

<rdar://problem/13757687> Threads get stuck in NetworkProcess when canceling loads
https://bugs.webkit.org/show_bug.cgi?id=115319

Reviewed by Darin Adler.

WebResourceLoader was second-guessing NetworkProcess as to whether it needs a response
to DidReceiveResponseWithCertificateInfo, and it was guessing wrong if the load was
alrady canceled on WebProcess side.

  • NetworkProcess/NetworkResourceLoader.cpp:

(WebKit::NetworkResourceLoader::didReceiveResponseAsync):

  • WebProcess/Network/WebResourceLoader.cpp:

(WebKit::WebResourceLoader::didReceiveResponseWithCertificateInfo):

  • WebProcess/Network/WebResourceLoader.h:
  • WebProcess/Network/WebResourceLoader.messages.in:
6:17 PM Changeset in webkit [149250] by gyuyoung.kim@samsung.com
  • 2 edits in trunk/Source/WebCore

Unreviewed build fix when disabling video and video-track.

  • platform/graphics/gstreamer/VideoSinkGStreamer.cpp:
5:09 PM Changeset in webkit [149249] by ggaren@apple.com
  • 3 edits in trunk/Source/WebCore

WebGL shouldn't allocate a "length" Identifier just to move some numbers around
https://bugs.webkit.org/show_bug.cgi?id=115317

Reviewed by Dean Jackson.

Saw this while debugging an ammo.js bug.

  • bindings/js/JSArrayBufferViewHelper.h:

(WebCore::setWebGLArrayWithTypedArrayArgument):
(WebCore::setWebGLArrayHelper):
(WebCore::constructArrayBufferViewWithTypedArrayArgument):
(WebCore::constructArrayBufferView):

  • bindings/js/JSWebGLRenderingContextCustom.cpp:

(WebCore::toVector): Use the pre-computed "length" identifier instead of
allocating a new one for each vector operation. There are lots more
optimizations we could do here. This is a start.

4:14 PM Changeset in webkit [149248] by fpizlo@apple.com
  • 9 edits in branches/dfgFourthTier/Source/JavaScriptCore

fourthTier: SymbolTable should be thread-safe
https://bugs.webkit.org/show_bug.cgi?id=115301

Reviewed by Geoffrey Garen.

Makes SymbolTable thread-safe. Relies on SymbolTableEntry already being immutable,
other than the WatchpointSet; but the WatchpointSet already has a righteous
concurrency protocol. So, this patch just protects the SymbolTable's HashMap.

  • bytecode/CodeBlock.cpp:

(JSC::CodeBlock::nameForRegister):

  • bytecompiler/BytecodeGenerator.cpp:

(JSC::BytecodeGenerator::addVar):

  • runtime/Executable.cpp:

(JSC::ProgramExecutable::addGlobalVar):

  • runtime/JSActivation.cpp:

(JSC::JSActivation::getOwnNonIndexPropertyNames):
(JSC::JSActivation::symbolTablePutWithAttributes):

  • runtime/JSSymbolTableObject.cpp:

(JSC::JSSymbolTableObject::getOwnNonIndexPropertyNames):

  • runtime/JSSymbolTableObject.h:

(JSC::symbolTableGet):
(JSC::symbolTablePut):
(JSC::symbolTablePutWithAttributes):

  • runtime/SymbolTable.cpp:

(JSC::SymbolTable::SymbolTable):
(JSC::SymbolTable::~SymbolTable):

  • runtime/SymbolTable.h:

(JSC::SymbolTable::find):
(JSC::SymbolTable::get):
(JSC::SymbolTable::inlineGet):
(JSC::SymbolTable::begin):
(JSC::SymbolTable::end):
(JSC::SymbolTable::size):
(JSC::SymbolTable::add):
(JSC::SymbolTable::set):
(JSC::SymbolTable::contains):

4:14 PM Changeset in webkit [149247] by ggaren@apple.com
  • 26 edits in trunk

Cleaned up pre/post inc/dec in bytecode
https://bugs.webkit.org/show_bug.cgi?id=115222

Reviewed by Filip Pizlo.

Source/JavaScriptCore:

A few related changes here:

(*) Removed post_inc and post_dec. The two-result form was awkward to
reason about. Being explicit about the intermediate mov and to_number
reduces DFG overhead, removes some fragile ASSERTs from the DFG, and
fixes a const bug. Plus, we get to blow away 262 lines of code.

(*) Renamed pre_inc and pre_dec to inc and dec, since there's only one
version now.

(*) Renamed to_jsnumber to to_number, to match the ECMA name.

(*) Tightened up the codegen and runtime support for to_number.

  • bytecode/CodeBlock.cpp:

(JSC::CodeBlock::dumpBytecode):

  • bytecode/Opcode.h:

(JSC::padOpcodeName):

  • bytecompiler/BytecodeGenerator.cpp:

(JSC::BytecodeGenerator::emitInc):
(JSC::BytecodeGenerator::emitDec):

  • bytecompiler/BytecodeGenerator.h:

(JSC::BytecodeGenerator::emitToNumber):
(BytecodeGenerator): Removed post_inc and post_dec.

  • bytecompiler/NodesCodegen.cpp:

(JSC::emitPreIncOrDec): Updated for rename.

(JSC::emitPostIncOrDec): Issue an explicit mov and to_number when needed.
These are rare, and they boil away in the DFG.

(JSC::PostfixNode::emitResolve):
(JSC::PrefixNode::emitResolve): For const, use an explicit mov instead
of any special forms. This fixes a bug where we would do string
add/subtract instead of number.

  • dfg/DFGByteCodeParser.cpp:

(JSC::DFG::ByteCodeParser::parseBlock):

  • dfg/DFGCapabilities.h:

(JSC::DFG::canCompileOpcode):

  • jit/JIT.cpp:

(JSC::JIT::privateCompileMainPass):
(JSC::JIT::privateCompileSlowCases):

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

(JSC::JIT::emit_op_inc):
(JSC::JIT::emitSlow_op_inc):
(JSC::JIT::emit_op_dec):
(JSC::JIT::emitSlow_op_dec):

  • jit/JITArithmetic32_64.cpp:

(JSC::JIT::emit_op_inc):
(JSC::JIT::emitSlow_op_inc):
(JSC::JIT::emit_op_dec):
(JSC::JIT::emitSlow_op_dec): Removed post_inc/dec, and updated for renames.

  • jit/JITOpcodes.cpp:

(JSC::JIT::emit_op_to_number):
(JSC::JIT::emitSlow_op_to_number): Removed a test for number cells. There's
no such thing!

  • jit/JITOpcodes32_64.cpp:

(JSC::JIT::emit_op_to_number): Use LowestTag to avoid making assumptions
about the lowest valued tag.

(JSC::JIT::emitSlow_op_to_number): Updated for renames.

  • jit/JITStubs.cpp:

(JSC::DEFINE_STUB_FUNCTION):

  • jit/JITStubs.h:
  • llint/LLIntSlowPaths.cpp:

(JSC::LLInt::LLINT_SLOW_PATH_DECL):

  • llint/LLIntSlowPaths.h:
  • llint/LowLevelInterpreter32_64.asm:
  • llint/LowLevelInterpreter64.asm:
  • parser/NodeConstructors.h:

(JSC::UnaryPlusNode::UnaryPlusNode): Removed post_inc/dec, and updated for renames.

  • runtime/Operations.cpp:

(JSC::jsIsObjectType): Removed a test for number cells. There's
no such thing!

LayoutTests:

  • fast/js/const-expected.txt:
  • fast/js/resources/const.js: Added tests for some const cases we used

to get wrong.

4:02 PM Changeset in webkit [149246] by ap@apple.com
  • 2 edits in trunk/Source/WebKit2

Regression tests that reference a non-local resource crash with NetworkProcess enabled
https://bugs.webkit.org/show_bug.cgi?id=115290

Reviewed by Sam Weinig.

Covered by fast/dom/icon-url-change.html and many other existing tests.

  • WebProcess/Network/WebResourceLoadScheduler.cpp:

(WebKit::WebResourceLoadScheduler::remove): Handle this case. Architecturally, it
would be better if SubresourceLoad::create just created an object an didn't do
further checks before returning a pointer, but changing that is beyond the scope
of the fix.

1:55 PM Changeset in webkit [149245] by andersca@apple.com
  • 5 edits in trunk/Source/WebCore

Remove two more StorageTask types
https://bugs.webkit.org/show_bug.cgi?id=115312

Reviewed by Dean Jackson.

  • storage/StorageTask.cpp:

(WebCore::StorageTask::performTask):

  • storage/StorageTask.h:
  • storage/StorageTracker.cpp:

(WebCore::StorageTracker::setOriginDetails):
(WebCore::StorageTracker::deleteOrigin):

  • storage/StorageTracker.h:
1:44 PM Changeset in webkit [149244] by andersca@apple.com
  • 5 edits in trunk/Source/WebCore

Get rid of more uses of StorageTask
https://bugs.webkit.org/show_bug.cgi?id=115311

Reviewed by Dean Jackson.

Make StorageAreaSync derive from ThreadSafeRefCounted as well, since it's passed between threads.

  • storage/StorageAreaSync.h:
  • storage/StorageSyncManager.cpp:

(WebCore::StorageSyncManager::scheduleImport):
(WebCore::StorageSyncManager::scheduleSync):
(WebCore::StorageSyncManager::scheduleDeleteEmptyDatabase):

  • storage/StorageTask.cpp:

(WebCore::StorageTask::StorageTask):
(WebCore::StorageTask::performTask):

  • storage/StorageTask.h:

(StorageTask):

1:32 PM Changeset in webkit [149243] by andersca@apple.com
  • 2 edits in trunk/Source/WebCore

Indent StorageAreaSync to match the style guidelines.

Rubber-stamped by Andreas Kling.

  • storage/StorageAreaSync.h:
1:28 PM Changeset in webkit [149242] by andersca@apple.com
  • 6 edits in trunk/Source/WebCore

Replace uses of StorageTask with StorageThread::dispatch and WTF::bind
https://bugs.webkit.org/show_bug.cgi?id=115309

Reviewed by Andreas Kling.

  • storage/StorageTask.cpp:

(WebCore::StorageTask::StorageTask):
(WebCore::StorageTask::performTask):

  • storage/StorageTask.h:
  • storage/StorageThread.cpp:

(WebCore::StorageThread::terminate):
(WebCore::StorageThread::releaseFastMallocFreeMemoryInAllThreads):

  • storage/StorageThread.h:
  • storage/StorageTracker.cpp:

(WebCore::StorageTracker::importOriginIdentifiers):
(WebCore::StorageTracker::deleteAllOrigins):

12:59 PM Changeset in webkit [149241] by andersca@apple.com
  • 5 edits in trunk/Source/WebCore

Add StorageThread::dispatch
https://bugs.webkit.org/show_bug.cgi?id=115308

Reviewed by Andreas Kling.

StorageThread::dispatch takes a Function<void ()> and runs it on the storage thread. This will be used to eventually get rid of StorageTask.

  • storage/StorageTask.cpp:

(WebCore::StorageTask::StorageTask):
(WebCore::StorageTask::performTask):

  • storage/StorageTask.h:

(WebCore::StorageTask::createDispatch):

  • storage/StorageThread.cpp:

(WebCore::StorageThread::dispatch):

  • storage/StorageThread.h:

(StorageThread):

11:25 AM Changeset in webkit [149240] by commit-queue@webkit.org
  • 2 edits in trunk/Source/JavaScriptCore

REGRESSION(r149114): cache flush for SH4 arch may flush an extra page.
https://bugs.webkit.org/show_bug.cgi?id=115305

Patch by Julien Brianceau <jbrianceau@nds.com> on 2013-04-27
Reviewed by Andreas Kling.

  • assembler/SH4Assembler.h:

(JSC::SH4Assembler::cacheFlush):

11:23 AM Changeset in webkit [149239] by fpizlo@apple.com
  • 5 edits in branches/dfgFourthTier/Source/JavaScriptCore

fourthTier: WatchpointSet should make racy uses easier to reason about
https://bugs.webkit.org/show_bug.cgi?id=115299

Reviewed by Anders Carlsson.

The compiler often does things like:

1c) Observe something that would imply that a WatchpointSet ought to be invalid

2c) Check that it is invalid

The main thread often does things like:

1m) Fire the watchpoint set

2m) Do some other thing that would cause the compiler to assume that the WatchpointSet
ought to be invalid

An example is structure transitions, where (1c) is the compiler noticing that a
put_by_id inline cache is in a transition state, with the source structure being S;
(2c) is the compiler asserting that S's watchpoint set is invalid; (1m) is the main
thread firing S's watchpoint set before it does the first transition away from S; and
(2m) is the main thread caching the put_by_id transition away from S.

This is totally fine, except that (1c) and (2c), and (1m) and (2m) could be reordered.
Probably, in most cases, this ought to do enough things that the main thread probably
already has some fencing. But the compiler thread definitely doesn't have fencing. In
any case, we should play it safe and just have additional fencing in all of the
relevant places.

We already have some idioms to put load-load and store-store fences in the right
places. But this change just makes WatchpointSet take care of this for us, thus
reducing the chances of us getting this wrong.

  • bytecode/Watchpoint.cpp:

(JSC::WatchpointSet::notifyWriteSlow):

  • bytecode/Watchpoint.h:

(WatchpointSet):
(JSC::WatchpointSet::isStillValid):
(JSC::WatchpointSet::hasBeenInvalidated):
(JSC::InlineWatchpointSet::hasBeenInvalidated):
(JSC::InlineWatchpointSet::notifyWrite):

  • dfg/DFGByteCodeParser.cpp:

(JSC::DFG::ByteCodeParser::parseBlock):

  • dfg/DFGDesiredWatchpoints.h:

(JSC::DFG::GenericDesiredWatchpoints::shouldAssumeMixedState):

11:18 AM Changeset in webkit [149238] by rakuco@webkit.org
  • 2 edits in trunk/Tools

[EFL] Unreviewed Intel bots fix after r149231.

  • efl/jhbuild.modules: Disable the libxml Python bindings since the

bots don't have Python's development files installed. libxml should
detect this kind of thing better.

10:08 AM Changeset in webkit [149237] by jer.noble@apple.com
  • 2 edits in trunk/Source/WebCore

REGRESSION (r147261): Audio controls background not displayed after loading audio file
https://bugs.webkit.org/show_bug.cgi?id=115221

Reviewed by Darin Adler.

As in r147261, set a flex-shrink: 0 to avoid shrinking the media controls panel
below the specified height.

No new tests; fixes the media/media-document-audio-repaint.html test.

  • css/mediaControlsQuickTime.css:

(audio::-webkit-media-controls-panel, video::-webkit-media-controls-panel):

9:51 AM WebKitGTK/2.0.x edited by zandobersek@gmail.com
Adding a merge proposal for 2.0.2. (diff)

Apr 26, 2013:

11:43 PM Changeset in webkit [149236] by ggaren@apple.com
  • 16 edits
    2 adds in trunk

Source/JavaScriptCore: Re-landing <http://trac.webkit.org/changeset/148999>

Filled out more cases of branch folding in bytecode when emitting
expressions into a branching context
https://bugs.webkit.org/show_bug.cgi?id=115057

Reviewed by Phil Pizlo.

We can't fold the number == 1 case to boolean because all non-zero numbers
down-cast to true, but only 1 is == to true.

Source/WTF: Filled out more cases of branch folding in bytecode when emitting expressions into a branching context
https://bugs.webkit.org/show_bug.cgi?id=115057

Reviewed by Filip Pizlo.

Added a helper constructor for TriState so clients can make one without
branching or making assumptions about the integer values of TriStates.

  • wtf/TriState.h:

(WTF::triState):
(WTF):

LayoutTests: Re-landing <http://trac.webkit.org/changeset/148999>

Filled out more cases of branch folding in bytecode when emitting
expressions into a branching context
https://bugs.webkit.org/show_bug.cgi?id=115057

Reviewed by Phil Pizlo.

Added a more exhaustive correctness test.

  • fast/js/branch-fold-correctness-expected.txt: Added.
  • fast/js/branch-fold-correctness.html: Added.
11:33 PM Changeset in webkit [149235] by fpizlo@apple.com
  • 2 edits in branches/dfgFourthTier/Source/JavaScriptCore

Fix indentation of SymbolTable.h

Rubber stamped by Mark Hahnenberg.

  • runtime/SymbolTable.h:
11:28 PM Changeset in webkit [149234] by fpizlo@apple.com
  • 2 edits in trunk/Source/JavaScriptCore

Correct indentation of SymbolTable.h

Rubber stamped by Mark Hahnenberg.

  • runtime/SymbolTable.h:
10:44 PM Changeset in webkit [149233] by fpizlo@apple.com
  • 16 edits in branches/dfgFourthTier/Source/JavaScriptCore

fourthTier: CFA should defend against results seeming inconsistent due to a watchpoint firing during compilation
https://bugs.webkit.org/show_bug.cgi?id=115083

Reviewed by Geoffrey Garen.

This ruggedizes our racyness with respect to watchpoints. We want to be able to assert,
in some places, that a watchpoint-based optimization has only occurred if the
watchpoint set was still valid. But currently we *can* soundly do watchpoint-based
optimizations even for invalid watchpoints, so long as we recorded in the IR that we
had done so; this will then lead to the code being insta-jettisoned after compilation
completes. Obviously, we don't want this to happen often - but we do want to allow it
precisely in the case of watchpoint races.

This adds the ability to assert that we hadn't over-watchpointed ourselves, with and
exemption for races.

  • dfg/DFGAbstractState.cpp:

(JSC::DFG::AbstractState::executeEffects):

  • dfg/DFGAbstractValue.cpp:

(JSC::DFG::AbstractValue::setFuturePossibleStructure):
(JSC::DFG::AbstractValue::filterFuturePossibleStructure):

  • dfg/DFGByteCodeParser.cpp:

(JSC::DFG::ByteCodeParser::addStructureTransitionCheck):
(JSC::DFG::ByteCodeParser::parseResolveOperations):
(JSC::DFG::ByteCodeParser::parseBlock):

  • dfg/DFGConstantFoldingPhase.cpp:

(JSC::DFG::ConstantFoldingPhase::addStructureTransitionCheck):

  • dfg/DFGDesiredWatchpoints.h:

(GenericDesiredWatchpoints):
(JSC::DFG::GenericDesiredWatchpoints::isStillValid):
(JSC::DFG::GenericDesiredWatchpoints::shouldAssumeMixedState):
(JSC::DFG::GenericDesiredWatchpoints::isValidOrMixed):
(JSC::DFG::DesiredWatchpoints::isStillValid):
(JSC::DFG::DesiredWatchpoints::shouldAssumeMixedState):
(JSC::DFG::DesiredWatchpoints::isValidOrMixed):
(DesiredWatchpoints):

  • dfg/DFGFixupPhase.cpp:

(JSC::DFG::FixupPhase::canOptimizeStringObjectAccess):

  • dfg/DFGGraph.h:

(JSC::DFG::Graph::masqueradesAsUndefinedWatchpointIsStillValid):
(Graph):

  • dfg/DFGJITCompiler.cpp:

(JSC::DFG::JITCompiler::link):
(JSC::DFG::JITCompiler::compile):
(JSC::DFG::JITCompiler::compileFunction):

  • dfg/DFGJITCompiler.h:

(JSC::DFG::JITCompiler::addLazily):
(JITCompiler):

  • dfg/DFGSpeculativeJIT.cpp:

(JSC::DFG::SpeculativeJIT::compilePeepHoleObjectEquality):

  • dfg/DFGSpeculativeJIT.h:

(SpeculativeJIT):
(JSC::DFG::SpeculativeJIT::masqueradesAsUndefinedWatchpointIsStillValid):
(JSC::DFG::SpeculativeJIT::speculationWatchpointForMasqueradesAsUndefined):
(JSC::DFG::SpeculativeJIT::speculateStringObjectForStructure):

  • dfg/DFGSpeculativeJIT32_64.cpp:

(JSC::DFG::SpeculativeJIT::nonSpeculativeNonPeepholeCompareNull):
(JSC::DFG::SpeculativeJIT::nonSpeculativePeepholeBranchNull):
(JSC::DFG::SpeculativeJIT::compileObjectEquality):
(JSC::DFG::SpeculativeJIT::compileObjectToObjectOrOtherEquality):
(JSC::DFG::SpeculativeJIT::compilePeepHoleObjectToObjectOrOtherEquality):
(JSC::DFG::SpeculativeJIT::compileObjectOrOtherLogicalNot):
(JSC::DFG::SpeculativeJIT::emitObjectOrOtherBranch):
(JSC::DFG::SpeculativeJIT::compile):

  • dfg/DFGSpeculativeJIT64.cpp:

(JSC::DFG::SpeculativeJIT::nonSpeculativeNonPeepholeCompareNull):
(JSC::DFG::SpeculativeJIT::nonSpeculativePeepholeBranchNull):
(JSC::DFG::SpeculativeJIT::compileObjectEquality):
(JSC::DFG::SpeculativeJIT::compileObjectToObjectOrOtherEquality):
(JSC::DFG::SpeculativeJIT::compilePeepHoleObjectToObjectOrOtherEquality):
(JSC::DFG::SpeculativeJIT::compileObjectOrOtherLogicalNot):
(JSC::DFG::SpeculativeJIT::emitObjectOrOtherBranch):
(JSC::DFG::SpeculativeJIT::compile):

  • ftl/FTLCompile.cpp:

(JSC::FTL::compile):

  • ftl/FTLState.h:

(State):

10:30 PM Changeset in webkit [149232] by jer.noble@apple.com
  • 5 edits
    3 adds in trunk

Video playback has corruption on the edges of the video
https://bugs.webkit.org/show_bug.cgi?id=115216

Reviewed by Simon Fraser.

Source/WebCore:

Test: media/video-poster-background.html

RenderImage correctly answers foregroundIsKnownToBeOpaqueInRect() method
when a poster image is displayed, but once the video begins playing and
the poster image is no longer displayed, RenderImage will continue to
answer 'true' even when the video does not occupy the entire render box.
Override foregroundIsKnownToBeOpaqueInRect() in order to more correctly
answer the question for the video layer.

  • rendering/RenderImage.h: Make foregroundIsKnownToBeOpaqueInRect() protected

(vs. private).

  • rendering/RenderVideo.cpp:

(WebCore::RenderVideo::foregroundIsKnownToBeOpaqueInRect): Added. Return

true if the rect parameter is entirely contained by the video box.

  • rendering/RenderVideo.h:

LayoutTests:

  • media/content/test.jpg: Added.
  • media/video-poster-background-expected.html: Added.
  • media/video-poster-background.html: Added.
9:41 PM Changeset in webkit [149231] by ryuan.choi@samsung.com
  • 2 edits in trunk/Tools

[EFL] Build break using jhbuild on ubuntu 13.04
https://bugs.webkit.org/show_bug.cgi?id=115225

Reviewed by Gyuyoung Kim.

libxslt in Ubuntu 13.04 requires libxml 2.9.0 or higher and
libxml 2.9.0 has a bug to make regression which r138422 mentioned.
So this patch bumped libxml version to 2.9.1.

  • efl/jhbuild.modules:
6:31 PM Changeset in webkit [149230] by commit-queue@webkit.org
  • 3 edits in trunk/Source/WebKit2

[WK2][EFL] Build break after r149212
https://bugs.webkit.org/show_bug.cgi?id=115294

Patch by Yael Aharon <yael.aharon@intel.com> on 2013-04-26
Reviewed by Simon Fraser.
r144672 bumped up the revision of WKContextClient API, and introduced a version array
APIClientTraits<WKContextClient>::interfaceSizesByVersion .
In r149212, the version was bumped back down and the version array was emptied but not removed.
This causes a warning "array subscript is below array bounds", and in EFL the warning is
treated as error.

  • Shared/APIClientTraits.cpp:

(WebKit):

  • Shared/APIClientTraits.h:
5:51 PM Changeset in webkit [149229] by roger_fong@apple.com
  • 2 edits in trunk/LayoutTests

Unreviewed. Skipping some more tests that still need rebaselining after disabling subpixel layout on mac.
http://bugs.webkit.org/show_bug.cgi?id=115292

5:51 PM Changeset in webkit [149228] by jpfau@apple.com
  • 2 edits in trunk/Tools

delete-stale-build-files does not clear .d files
http://bugs.webkit.org/show_bug.cgi?id=115293

Reviewed by Ryosuke Niwa.

  • BuildSlaveSupport/delete-stale-build-files:

(main): Add .d to the file type list

5:38 PM Changeset in webkit [149227] by roger_fong@apple.com
  • 2 edits in trunk/WebKitLibraries

Disable sub-pixel-layout on Apple Windows port, VS2005 edition.

  • win/tools/vsprops/FeatureDefines.vsprops:
5:35 PM Changeset in webkit [149226] by hmuller@adobe.com
  • 12 edits in trunk/Source/WebCore

[CSS Exclusions] ExclusionShape bounding box methods should return LayoutRects
https://bugs.webkit.org/show_bug.cgi?id=115117

Reviewed by Dirk Schulze.

Redefined the ExclusionShape API in terms of LayoutUnits, instead of floats: all of the
ExclusionShape methods now have LayoutUnit parameters and return LayoutUnit values.
This is more natural, since the callers work exclusively in LayoutUnits.

This is strictly a refactoring, no new tests were needed.

  • rendering/ExclusionPolygon.cpp:

(WebCore::ExclusionPolygon::getExcludedIntervals):
(WebCore::ExclusionPolygon::getIncludedIntervals):
(WebCore::ExclusionPolygon::firstIncludedIntervalLogicalTop):

  • rendering/ExclusionPolygon.h:
  • rendering/ExclusionRectangle.cpp:

(WebCore::ExclusionRectangle::getExcludedIntervals):
(WebCore::ExclusionRectangle::getIncludedIntervals):
(WebCore::ExclusionRectangle::firstIncludedIntervalLogicalTop):

  • rendering/ExclusionRectangle.h:
  • rendering/ExclusionShape.cpp:

(WebCore::ExclusionShape::createExclusionShape):

  • rendering/ExclusionShape.h:

(LineSegment):
(ExclusionShape):

  • rendering/ExclusionShapeInfo.cpp:

(WebCore):
(WebCore::::computedShape):

  • rendering/ExclusionShapeInfo.h:

(WebCore):
(WebCore::ExclusionShapeInfo::shapeLogicalTop):
(WebCore::ExclusionShapeInfo::shapeLogicalBottom):
(ExclusionShapeInfo):

  • rendering/ExclusionShapeInsideInfo.cpp:

(WebCore::ExclusionShapeInsideInfo::adjustLogicalLineTop):

  • rendering/ExclusionShapeInsideInfo.h:
  • rendering/ExclusionShapeOutsideInfo.h:
5:17 PM Changeset in webkit [149225] by jer.noble@apple.com
  • 2 edits in trunk/Source/WebCore

Nil-check the results of -[AVPlayerItemVideoOutput copyPixelBufferForItemTime:itemTimeForDisplay:].
https://bugs.webkit.org/show_bug.cgi?id=115265

Reviewed by Eric Carlson.

copyPixelBufferForItemTime:itemTimeForDisplay: can return nil, even
when hasNewPixelBufferForItemTime: returns YES. Check the results
before passing the buffer on to VTPixelTransferSessionTransferImage()
which does not NULL-check its parameters.

  • platform/graphics/avfoundation/objc/MediaPlayerPrivateAVFoundationObjC.mm:

(WebCore::MediaPlayerPrivateAVFoundationObjC::createPixelBuffer):

5:09 PM Changeset in webkit [149224] by roger_fong@apple.com
  • 2 edits in trunk/LayoutTests

Unreviewed. Skipping some tests that still need rebaselining after disabling subpixel layout on mac.
http://bugs.webkit.org/show_bug.cgi?id=115292

4:45 PM Changeset in webkit [149223] by roger_fong@apple.com
  • 2 edits in trunk/Source/WebCore

Uninflate caret rect.
http://bugs.webkit.org/show_bug.cgi?id=114997.
<rdar://problem/12629007>.

Reviewed by Timothy Horton.

Inflation was causing painting errors with focus rings.
It is no longer needed after disabling subpixel layout.

  • editing/FrameSelection.cpp:

(WebCore::repaintCaretForLocalRect):

4:40 PM Changeset in webkit [149222] by roger_fong@apple.com
  • 3 edits
    66 adds in trunk/LayoutTests

Unreviewed. More rebaselines for mac after disabling sub-pixel layout.

4:40 PM Changeset in webkit [149221] by roger_fong@apple.com
  • 2 edits in trunk/WebKitLibraries

Disable sub-pixel-layout on Apple Windows port.

  • win/tools/vsprops/FeatureDefines.props:
4:17 PM Changeset in webkit [149220] by roger_fong@apple.com
  • 216 edits
    1 delete in trunk

Make Apple Windows VS2010 build results into and get dependencies from 32 suffixed folders.
Make the DebugSuffix configuration use _debug dependencies.

4:06 PM Changeset in webkit [149219] by Martin Robinson
  • 5 edits in trunk/Source/WebKit2

[GTK] Add methods to add a user style sheet to the WebKit2 GTK+ API
https://bugs.webkit.org/show_bug.cgi?id=99081

Reviewed by Carlos Garcia Campos, Gustavo Noronha Silva, and Benjamin Poulain.

Add methods to WebKitWebViewGroup to add and remove user style sheets.
This allows clients to inject style sheets into pages with a set of
rules for when those style sheets apply.

  • UIProcess/API/gtk/WebKitWebViewGroup.cpp:

(toImmutableArray): Added this helper which converts the GList* parameters into
ImmutableArrays for use with the WebKit2 internal API.
(webkit_web_view_group_add_user_style_sheet): Added new API for adding a style sheet.
(webkit_web_view_group_remove_all_user_style_sheets): Add new API for clearing out all style sheets.

  • UIProcess/API/gtk/WebKitWebViewGroup.h: Added new method declarations.
  • UIProcess/API/gtk/docs/webkit2gtk-sections.txt: Added new API to the documentation.
  • UIProcess/API/gtk/tests/TestWebKitWebViewGroup.cpp: Added a test for the new API.

(isStyleSheetInjectedForURLAtPath): Function to check whether the style sheet has been injected for a given URL.
(fillURLListFromPaths): Helper which converts paths passed via varargs into a whitelist or blacklist.
(removeOldInjectedStyleSheetsAndResetLists): Function to start afresh.
(testWebViewGroupInjectedStyleSheet): The actual test.
(serverCallback): Server callback for use with the test. We cannot use loadHTML or
loadAlternateHTML, because that checks the whitelist and blacklist against about:blank.
(beforeAll): Initialize the server and new test.
(afterAll): Clean up the server.

3:29 PM Changeset in webkit [149218] by roger_fong@apple.com
  • 2 edits in trunk/Source/WebCore

Unreviewed build fix.

  • platform/LayoutUnit.h:

(WebCore::LayoutUnit::LayoutUnit):

3:22 PM Changeset in webkit [149217] by ap@apple.com
  • 8 edits in trunk/Source/WebKit2

2013-04-26 Simon Cooper <scooper@apple.com>

Permit ~/Library or ~/Library/Preferences to be symlinks
https://bugs.webkit.org/show_bug.cgi?id=115142
<rdar://problem/12953603>

Reviewed by Alexey Proskuryakov.

Pass in two new parameters which are the resolved paths to ~/Library
and ~/Library/Preferences. Added new expansion functions and
substituted the new functions for home-regex, home-subpath and
home-literal for places referencing resources in "~/Library" or
"~/Library/Preferences".

  • Resources/PlugInSandboxProfiles/com.apple.QuickTime Plugin.plugin.sb:
  • Resources/PlugInSandboxProfiles/com.apple.WebKit.plugin-common.sb:
  • Resources/PlugInSandboxProfiles/com.apple.ist.ds.appleconnect.webplugin.sb:
  • Resources/PlugInSandboxProfiles/com.macromedia.Flash Player.plugin.sb:
  • Resources/PlugInSandboxProfiles/com.microsoft.SilverlightPlugin.sb:
  • Resources/PlugInSandboxProfiles/com.oracle.java.JavaAppletPlugin.sb:
  • Shared/mac/ChildProcessMac.mm: (WebKit::ChildProcess::initializeSandbox):
3:20 PM Changeset in webkit [149216] by msaboff@apple.com
  • 4 edits in trunk/Source/WebCore

WebCore ObjC bridge is missing support for bool type
https://bugs.webkit.org/show_bug.cgi?id=115276

Reviewed by Geoffrey Garen.

Added code to handle conversion between ObjC and JS booleans.

  • bridge/objc/objc_instance.mm:

(ObjcInstance::invokeObjcMethod):

  • bridge/objc/objc_utility.h:
  • bridge/objc/objc_utility.mm:

(JSC::Bindings::convertValueToObjcValue):
(JSC::Bindings::convertObjcValueToValue):
(JSC::Bindings::objcValueTypeForType):

3:15 PM Changeset in webkit [149215] by mrowe@apple.com
  • 1 edit in tags/Safari-537.39.1/Source/WebKit/mac/WebCoreSupport/WebEditorClient.mm

Build fix.

3:15 PM Changeset in webkit [149214] by mrowe@apple.com
  • 4 edits in tags/Safari-537.39.1/Source

Versioning.

3:09 PM Changeset in webkit [149213] by mrowe@apple.com
  • 1 copy in tags/Safari-537.39.1

New tag.

2:54 PM Changeset in webkit [149212] by ap@apple.com
  • 4 edits in trunk/Source/WebKit2

[WK2] WKContextClient doesn't need to be versioned yet
https://bugs.webkit.org/show_bug.cgi?id=115279

Reviewed by Anders Carlsson.

  • Shared/APIClientTraits.cpp:
  • Shared/APIClientTraits.h:
  • UIProcess/API/C/WKContext.h:
2:39 PM Changeset in webkit [149211] by andersca@apple.com
  • 8 edits in trunk/Source/WebKit2

Downloads need to be converted from the NSURLConnection, not canceled and restarted, when using the NetworkProcess
https://bugs.webkit.org/show_bug.cgi?id=115277
<rdar://problem/12890184>

Reviewed by Sam Weinig.

  • NetworkProcess/NetworkConnectionToWebProcess.cpp:

(WebKit::NetworkConnectionToWebProcess::convertMainResourceLoadToDownload):
Ask the download manager to convert the main resource load to a download.

  • NetworkProcess/NetworkConnectionToWebProcess.messages.in:

Add new message.

  • NetworkProcess/NetworkResourceLoader.h:

(WebKit::NetworkResourceLoader::handle):
Add getter.

  • Shared/Downloads/mac/DownloadMac.mm:

(dispatchOnMainThread):
Helper function that ensures that a given block is called on the main thread.

(-[WKDownloadAsDelegate downloadDidBegin:]):
(-[WKDownloadAsDelegate download:didReceiveAuthenticationChallenge:]):
(-[WKDownloadAsDelegate download:didReceiveResponse:]):
(-[WKDownloadAsDelegate download:didReceiveDataOfLength:]):
(-[WKDownloadAsDelegate download:shouldDecodeSourceDataOfMIMEType:]):
(-[WKDownloadAsDelegate download:decideDestinationWithSuggestedFilename:]):
(-[WKDownloadAsDelegate download:didCreateDestination:]):
(-[WKDownloadAsDelegate downloadDidFinish:]):
(-[WKDownloadAsDelegate download:didFailWithError:]):
Use dispatchOnMainThread.

  • UIProcess/WebPageProxy.cpp:

(WebKit::WebPageProxy::receivedPolicyDecision):
Remove code that cancels the current load and starts a new download.

  • WebProcess/WebPage/WebFrame.cpp:

(WebKit::WebFrame::convertMainResourceLoadToDownload):
Send the ConvertMainResourceLoadToDownload message to the network process.

2:25 PM Changeset in webkit [149210] by roger_fong@apple.com
  • 865 edits
    359 adds
    262 deletes in trunk/LayoutTests

Unreviewed. Rebaseline a bunch of tests and run optimize-baselines after disabling sub-pixel layout on mac.
https://bugs.webkit.org/show_bug.cgi?id=114999.

2:19 PM Changeset in webkit [149209] by roger_fong@apple.com
  • 9 edits in trunk/Source

Disable sub-pixel layout on mac.
https://bugs.webkit.org/show_bug.cgi?id=114999.

Reviewed by Simon Fraser.

  • Configurations/FeatureDefines.xcconfig:
  • WebCore.exp.in:

Export symbol needed to be added for IntRect(const LayoutRect&) constructor,
which is now being used in pixelSnappedIntRect(const LayoutRect&) in LayoutRect.h after disabling sub-pixel layout.

2:13 PM Changeset in webkit [149208] by Simon Fraser
  • 2 edits in trunk/Source/WebKit2

Inspector window is often blank when opened
https://bugs.webkit.org/show_bug.cgi?id=115220
<rdar://problem/13735496>

Reviewed by Tim Horton.

After disabling tiled scrolling and accelerated drawing in r149006, the
Inspector window was often missing its content after opening. This only
happened when the user has a preference on disk to use accelerated drawing
in the Inspector.

The missing content was caused by a failure to connect the correct contextID
for remote layer hosting. On window creation, we hook up a context using
WindowServer layer hosting (the default), but then switch back to app-hosted
layers for the Inspector window. This switching happens in didUpdateBackingStoreState(),
but that can be re-entered via its call to sendUpdateBackingStoreState(). In
that cause, on unwinding the stack it would call enterAcceleratedCompositingMode()
with a stale layerTreeContext.

Fix by moving the code that exits/enters accelerated compositing mode to before
the call to sendUpdateBackingStoreState() that can cause re-entrancy. That avoids
use of a possibly stale layerTreeContext.

  • UIProcess/DrawingAreaProxyImpl.cpp:

(WebKit::DrawingAreaProxyImpl::didUpdateBackingStoreState):

2:13 PM Changeset in webkit [149207] by Simon Fraser
  • 4 edits in trunk/Source

Layer-backed WebViews don't repaint content outside the visible area
https://bugs.webkit.org/show_bug.cgi?id=115275

Reviewed by Beth Dakin.

Source/WebCore:

Export ScrollView::setPaintsEntireContents(bool).

  • WebCore.exp.in:

Source/WebKit/mac:

When a client of WebView makes the WebView be layer-backed, then
AppKit propagaes the layer-backing down to the WebHTMLView (but
no further).

In this situation, we had a bug where content that was not
in the visible content rect would not get repainted, because
WebCore clips repaints to the visible rect.

Fix by calling ScrollView::setPaintsEntireContents(bool) from
-[WebHTMLView setLayer:], saying that we need
to paint the entire contents if we have a layer.

  • WebView/WebHTMLView.mm:

(-[WebHTMLView setLayer:]):

1:34 PM Changeset in webkit [149206] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebKit/blackberry

[BlackBerry] Check for valid element in elementTouched
https://bugs.webkit.org/show_bug.cgi?id=115205

Patch by Nima Ghanavatian <nghanavatian@blackberry.com> on 2013-04-26
Reviewed by Rob Buis.

Internally reviewed by Genevieve Mak.

PR 331546
We might receive a null ptr from nodeAsElementIfApplicable which
is passed in here. Check to make sure it's valid before using.

  • WebKitSupport/InputHandler.cpp:

(BlackBerry::WebKit::InputHandler::elementTouched):

1:12 PM Changeset in webkit [149205] by oliver@apple.com
  • 2 edits in trunk/Source/JavaScriptCore

Make stack tracing more robust
https://bugs.webkit.org/show_bug.cgi?id=115272

Reviewed by Geoffrey Garen.

CallFrame already handles stack walking confusion robustly,
so we should make sure that the actual walk handles that as well.

  • interpreter/Interpreter.cpp:

(JSC::getCallerInfo):

12:42 PM Changeset in webkit [149204] by ap@apple.com
  • 2 edits in trunk/Source/WebKit2

Crashes under ResourceHandle::continueDidReceiveResponse when loading blobs
https://bugs.webkit.org/show_bug.cgi?id=115273

Reviewed by Anders Carlsson.

Covered by multiple existing tests (when using NetworkProcess), including:

  • http/tests/fileapi/blob-url-in-subframe.html
  • http/tests/fileapi/create-blob-url-from-data-url.html
  • NetworkProcess/NetworkResourceLoader.cpp: (WebKit::NetworkResourceLoader::continueDidReceiveResponse): Work around BlobResourceHandle badness.
12:09 PM Changeset in webkit [149203] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebCore

Change RenderMeter::valueRatio() visibility to public
https://bugs.webkit.org/show_bug.cgi?id=115266

This function is currently unused.
However, it is necessary to implement a RenderTheme capable of rendering Meter elements.

Patch by Daker Fernandes Pinheiro <daker.pinheiro@openbossa.org> on 2013-04-26
Reviewed by Alexis Menard.

No new tests, because no behaviour is changed.

  • rendering/RenderMeter.h:

(RenderMeter):

12:08 PM Changeset in webkit [149202] by timothy@apple.com
  • 6 edits
    1 add in trunk/Source

Add Runtime.parse to the Inspector protocol.

This will be used to parse console expressions for errors
before evaluating them fully.

https://webkit.org/b/115242

Reviewed by Oliver Hunt.

  • ForwardingHeaders/parser/ParserError.h: Added.
  • inspector/Inspector.json:
  • inspector/InspectorRuntimeAgent.cpp:

(WebCore::InspectorRuntimeAgent::parse):

  • inspector/InspectorRuntimeAgent.h:

(InspectorRuntimeAgent):

12:04 PM Changeset in webkit [149201] by benjamin@webkit.org
  • 2 edits in trunk/Source/WebCore

Fix the copyright years after r149057

Patch by Benjamin Poulain <bpoulain@apple.com> on 2013-04-26

  • page/DiagnosticLoggingKeys.cpp: I accidentally removed a year from

the copyright in r149057.

11:57 AM Changeset in webkit [149200] by commit-queue@webkit.org
  • 3 edits
    2 adds in trunk

Web Inspector: Crash due to null items from getDOMStorageItems
https://bugs.webkit.org/show_bug.cgi?id=115176

Patch by Konrad Piascik <kpiascik@blackberry.com> on 2013-04-26
Reviewed by Joseph Pecoraro.

Source/WebCore:

findStorageArea was returning a null storageArea causing the items
input paramater to not be set. This was happening without any error
being set at all. Set an error to prevent a crash when we try to
convert the result to a JSON string.

Added tests to check if session and local storage are empty that
they are still functional.

  • inspector/InspectorDOMStorageAgent.cpp:

(WebCore::InspectorDOMStorageAgent::getDOMStorageItems):

LayoutTests:

Tests to check that if local and session storage are empty
that they still work corerctly.

  • inspector/storage-panel-dom-storage-empty-expected.txt: Added.
  • inspector/storage-panel-dom-storage-empty.html: Added.
11:40 AM Changeset in webkit [149199] by ap@apple.com
  • 3 edits in trunk/Source/WebKit2

[WK2] Remove NetworkResourceLoader::cleanupOnMainThread
https://bugs.webkit.org/show_bug.cgi?id=115268

Reviewed by Brady Eidson.

All cleanup() callers are now on main thread, and this indirection only complicates debugging.

  • NetworkProcess/NetworkResourceLoader.cpp: (WebKit::NetworkResourceLoader::connectionToWebProcessDidClose): (WebKit::NetworkResourceLoader::abortInProgressLoad): (WebKit::NetworkResourceLoader::didFinishLoading): (WebKit::NetworkResourceLoader::didFail):
  • NetworkProcess/NetworkResourceLoader.h:
11:24 AM Changeset in webkit [149198] by aestes@apple.com
  • 3 edits in trunk/Source/WebKit2

[WebKit2] CustomProtocolManager's m_registeredSchemes HashSet can be accessed by multiple threads
https://bugs.webkit.org/show_bug.cgi?id=115267

Reviewed by Alexey Proskuryakov.

  • Shared/Network/CustomProtocols/CustomProtocolManager.h: Added a Mutex to protect m_registeredSchemes.
  • Shared/Network/CustomProtocols/mac/CustomProtocolManagerMac.mm:

(WebKit::CustomProtocolManager::registerScheme): Acquired the mutex before accessing m_registeredSchemes.
(WebKit::CustomProtocolManager::unregisterScheme): Ditto.
(WebKit::CustomProtocolManager::supportsScheme): Ditto.

11:10 AM Changeset in webkit [149197] by eric.carlson@apple.com
  • 2 edits in trunk/LayoutTests

[Mac] media/track/track-mode.html sometimes trigger InvalidStateError
https://bugs.webkit.org/show_bug.cgi?id=115045

Reviewed by Alexey Proskuryakov.

  • media/track/track-mode.html: Don't assume that the video element is ready for seeking

after a 100ms timeout.

10:51 AM Changeset in webkit [149196] by ap@apple.com
  • 2 edits in trunk/Source/WebKit2

Blobs are not associated with connection when there are no sandbox extensions
https://bugs.webkit.org/show_bug.cgi?id=115264

Reviewed by Brady Eidson.

Covered by multiple fast/files tests when using NetworkProcess.

  • NetworkProcess/FileAPI/NetworkBlobRegistry.cpp: (WebKit::NetworkBlobRegistry::registerBlobURL): Fixed a logic error.
10:41 AM Changeset in webkit [149195] by zandobersek@gmail.com
  • 2 edits in trunk/Tools

[GTK] Clean up the TestWebKitAPI GNUmakefile.am
https://bugs.webkit.org/show_bug.cgi?id=115233

Reviewed by Martin Robinson.

Specify unconditional, non-installable program targets in one declaration. Only conditionally specify
WebKit2-specific targets, not the complete target setups (i.e. targets' sources, cppflags listings etc.).
Add missing unit test source files to the TestWTF program, namely HashSet.cpp and MetaAllocator.cpp.
Specify the TestWTF program's CXXFLAGS to be the same as the global cxxflags as the source files need to
be compiled with the -fno-rtti flag that's present in the global_cxxflags variable.

  • TestWebKitAPI/GNUmakefile.am:
9:55 AM Changeset in webkit [149194] by aestes@apple.com
  • 11 edits
    3 copies
    2 adds in trunk

[WebKit2] Loading a resource from a custom protocol in a synchronous XHR times out
https://bugs.webkit.org/show_bug.cgi?id=115223

Reviewed by Darin Adler.

Source/WebKit2:

When WebKit performs a synchronous load on the Mac, it spins a nested
run loop in a mode that only accepts input from the NSURLConnection
performing the load. When the load is for a custom protocol in WebKit2,
we proxy it to the UI process via CoreIPC messages, but the response
messages from the UI process are never processed as long as the run
loop is in a non-common mode (and we wouldn't want to process messages
that could re-enter WebCore in the middle of loading, anyway). Since
these messages never make it back to the NSURLConnection handling the
request, the connection eventually times out.

Fix this by using a work queue to handle custom protocol messages in
the networking process. The work queue can process incoming custom
protocol messages while the main thread is blocked.

  • NetworkProcess/NetworkProcess.cpp:

(WebKit::NetworkProcess::initializeConnection): Called
initializeConnection() on all the NetworkProcess's supplements.

  • Shared/ChildProcessSupplement.h: Added a base class for

NetworkProcessSupplement and WebProcessSupplement which defines
initializeConnection and provides an empty default implementation.
(WebKit::ChildProcessSupplement::~ChildProcessSupplement):
(WebKit::ChildProcessSupplement::initializeConnection):

  • Shared/Network/CustomProtocols/CustomProtocolManager.h:
  • Shared/Network/CustomProtocols/mac/CustomProtocolManagerMac.mm:

(WebKit::CustomProtocolManager::CustomProtocolManager): Instantiated a
work queue for message processing.
(WebKit::CustomProtocolManager::initializeConnection): Added the work
queue as a message receiver on the CoreIPC connection.
(WebKit::CustomProtocolManager::initialize): If we're in the web
process and the network process is being used, unregister and destroy
the work queue we previously created. It'd be better to not create it
in the first place, but we have to register our work queue with the
CoreIPC connection before it is established, which is before the UI
process has told us whether the network process is in use.

  • Shared/Network/NetworkProcessSupplement.h: Inherited from

ChildProcessSupplement.

  • WebKit2.xcodeproj/project.pbxproj: Added ChildProcessSupplement.h.
  • WebProcess/WebProcess.cpp:

(WebKit::WebProcess::initializeConnection): Called
initializeConnection() on all the WebProcess's supplements.

  • WebProcess/WebProcessSupplement.h: Inherited from

ChildProcessSupplement.

Tools:

Added an API test.

  • TestWebKitAPI/TestWebKitAPI.xcodeproj/project.pbxproj: Added new files.
  • TestWebKitAPI/Tests/CustomProtocolsSyncXHRTest.mm: Added.

(TestWebKitAPI::TEST): Tested that a synchronous XHR does not time out
when it loads a request with a custom protocol.

  • TestWebKitAPI/Tests/WebKit2/custom-protocol-sync-xhr.html: Added.
  • TestWebKitAPI/Tests/WebKit2ObjC/CustomProtocolsTest.mm: Moved the

NSURLProtocol subclass to TestProtocol.{h, mm} and did some
miscellaneous cleanup.

  • TestWebKitAPI/mac/TestProtocol.h: Copied from Source/WebKit2/WebProcess/WebProcessSupplement.h.
  • TestWebKitAPI/mac/TestProtocol.mm: Copied from Tools/TestWebKitAPI/Tests/WebKit2ObjC/CustomProtocolsTest.mm.

(+[TestProtocol canInitWithRequest:]):
(+[TestProtocol canonicalRequestForRequest:]):
(+[TestProtocol requestIsCacheEquivalent:toRequest:]):
(+[TestProtocol scheme]):
(-[TestProtocol startLoading]):
(-[TestProtocol stopLoading]):

9:24 AM Changeset in webkit [149193] by Martin Robinson
  • 49 edits in trunk

Remove the remaining Skia #ifdefs
https://bugs.webkit.org/show_bug.cgi?id=114886

Reviewed by Benjamin Poulain.

Source/WebCore:

  • html/HTMLCanvasElement.cpp: Remove Skia #ifdef references.
  • platform/graphics/BitmapImage.cpp: Ditto.
  • platform/graphics/FloatPoint.h: Ditto.
  • platform/graphics/FloatRect.h: Ditto.
  • platform/graphics/FontCache.h: Ditto.
  • platform/graphics/Gradient.cpp: Ditto.
  • platform/graphics/Gradient.h: Ditto.
  • platform/graphics/GraphicsContext.cpp: Ditto.
  • platform/graphics/GraphicsContext.h: Ditto.
  • platform/graphics/GraphicsContext3D.h: Ditto.
  • platform/graphics/ImageBuffer.cpp: Ditto.
  • platform/graphics/ImageBuffer.h: Ditto.
  • platform/graphics/ImageBufferData.h: Ditto.
  • platform/graphics/IntPoint.h: Ditto.
  • platform/graphics/IntRect.h: Ditto.
  • platform/graphics/NativeImagePtr.h: Ditto.
  • platform/graphics/Path.h: Ditto.
  • platform/graphics/Pattern.cpp: Ditto.
  • platform/graphics/Pattern.h: Ditto.
  • platform/graphics/filters/FEBlend.h: Ditto.
  • platform/graphics/filters/FEColorMatrix.h: Ditto.
  • platform/graphics/filters/FEComponentTransfer.h: Ditto.
  • platform/graphics/filters/FEComposite.h: Ditto.
  • platform/graphics/filters/FEConvolveMatrix.h: Ditto.
  • platform/graphics/filters/FEDisplacementMap.h: Ditto.
  • platform/graphics/filters/FEGaussianBlur.h: Ditto.
  • platform/graphics/filters/FELighting.h: Ditto.
  • platform/graphics/filters/FEMorphology.h: Ditto.
  • platform/graphics/filters/FEOffset.h: Ditto.
  • platform/graphics/filters/FilterEffect.cpp: Ditto.
  • platform/graphics/filters/FilterEffect.h: Ditto.
  • platform/graphics/transforms/AffineTransform.h: Ditto.
  • platform/graphics/transforms/TransformationMatrix.h: Ditto.
  • platform/image-decoders/ImageDecoder.cpp: Ditto.
  • platform/image-decoders/ImageDecoder.h: Ditto.
  • platform/image-decoders/jpeg/JPEGImageDecoder.cpp: Ditto.
  • platform/image-decoders/webp/WEBPImageDecoder.cpp: Ditto.
  • platform/mac/LocalCurrentGraphicsContext.h: Ditto.
  • platform/mac/LocalCurrentGraphicsContext.mm: Ditto.
  • rendering/svg/RenderSVGResourceSolidColor.cpp: Ditto.
  • svg/graphics/SVGImage.cpp: Ditto.
  • svg/graphics/SVGImage.h: Ditto.

Source/WebKit/blackberry:

  • Api/WebPage.cpp: Remove Skia #ifdef references.
  • WebCoreSupport/AboutDataUseFeatures.in: Ditto.

Source/WTF:

  • wtf/Platform.h:

Tools:

  • DumpRenderTree/blackberry/PixelDumpSupportBlackBerry.cpp:

(createBitmapContextFromWebView): Remove Skia #ifdef references.

9:16 AM Changeset in webkit [149192] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebKit2

[GTK] Compilation of ProcessLauncherGtk.cpp fails due to unresolved symbols
https://bugs.webkit.org/show_bug.cgi?id=115240

Patch by Eduardo Lima Mitev <elima@igalia.com> on 2013-04-26
Reviewed by Martin Robinson.

  • UIProcess/Launcher/gtk/ProcessLauncherGtk.cpp: Includes sys/socket.h if OS is Linux
9:15 AM Changeset in webkit [149191] by ddkilzer@apple.com
  • 2 edits in trunk/LayoutTests

Sort jsc-test-list

$ sort -f -o LayoutTests/fast/js/jsc-test-list LayoutTests/fast/js/jsc-test-list

  • fast/js/jsc-test-list: Sort case-insensitively.
9:15 AM Changeset in webkit [149190] by ddkilzer@apple.com
  • 2 edits in trunk/LayoutTests

Upstream iOS changes to jsc-test-list

Reviewed by Michael Saboff.

  • fast/js/jsc-test-list: Add tests.
9:12 AM Changeset in webkit [149189] by kadam@inf.u-szeged.hu
  • 2 edits in trunk/LayoutTests

[Qt] Unreviewed gardening. Skip two failing tests.

  • platform/qt/TestExpectations:
9:08 AM Changeset in webkit [149188] by allan.jensen@digia.com
  • 2 edits in trunk/Source/WebCore

Assert in JSC::Heap::unprotect when closing facebook.com web site
https://bugs.webkit.org/show_bug.cgi?id=115058

Reviewed by Darin Adler.

Grab a JSLock before calling RootObject::invalidate().

  • bindings/js/ScriptController.cpp:

(WebCore::ScriptController::~ScriptController):

8:17 AM Changeset in webkit [149187] by eric.carlson@apple.com
  • 6 edits in trunk/Source/WebCore

[Mac] in-band cues sometimes have incorrect duration
https://bugs.webkit.org/show_bug.cgi?id=115200

Reviewed by Jer Noble.

No new tests, this is not possible to test in DRT.

  • html/track/InbandTextTrack.cpp:

(WebCore::InbandTextTrack::addGenericCue): Don't add completed cues to the map.
(WebCore::InbandTextTrack::removeGenericCue): Log when a cue is removed from the track.

  • platform/graphics/avfoundation/InbandTextTrackPrivateAVF.cpp:

(WebCore::InbandTextTrackPrivateAVF::InbandTextTrackPrivateAVF): Initialize m_pendingCueStatus.
(WebCore::InbandTextTrackPrivateAVF::processCue): Never call update() on a cue that is delivered

while seeking.

(WebCore::InbandTextTrackPrivateAVF::beginSeeking): Flush all incomplete cues, remember that

we are seeking.

(WebCore::InbandTextTrackPrivateAVF::resetCueValues):

  • platform/graphics/avfoundation/InbandTextTrackPrivateAVF.h:
  • platform/graphics/avfoundation/MediaPlayerPrivateAVFoundation.cpp:

(WebCore::MediaPlayerPrivateAVFoundation::seek): Call track->beginSeeking() instead of resetCueValues().
(WebCore::MediaPlayerPrivateAVFoundation::seekCompleted): Call track->endSeeking().

  • platform/graphics/avfoundation/objc/InbandTextTrackPrivateAVFObjC.mm:

(WebCore::InbandTextTrackPrivateAVFObjC::kind): Include class name in Kind enum values to

avoid compile error.

8:14 AM Changeset in webkit [149186] by akling@apple.com
  • 21 edits in trunk/Source/WebCore

Remove wxWebKit from WebCore.
<http://webkit.org/b/115255>

Reviewed by Anders Carlsson.

  • DerivedSources.make:
  • WebCorePrefix.h:
  • bindings/js/JSInspectorFrontendHostCustom.cpp:

(WebCore::JSInspectorFrontendHost::port):

  • config.h:
  • html/HTMLFormElement.cpp:
  • platform/ContextMenu.h:

(ContextMenu):

  • platform/ContextMenuItem.h:
  • platform/Cursor.h:
  • platform/DragData.h:
  • platform/DragImage.h:
  • platform/FileSystem.h:
  • platform/PlatformExportMacros.h:
  • platform/PlatformKeyboardEvent.h:

(PlatformKeyboardEvent):

  • platform/PlatformMenuDescription.h:
  • platform/PlatformMouseEvent.h:

(PlatformMouseEvent):

  • platform/PlatformWheelEvent.h:

(PlatformWheelEvent):

  • platform/ScrollView.cpp:
  • platform/ScrollView.h:

(ScrollView):

  • platform/Widget.h:
  • platform/win/SharedTimerWin.cpp:
8:13 AM Changeset in webkit [149185] by commit-queue@webkit.org
  • 6 edits
    3 adds in trunk

use-after-free removing a frame from its parent in a beforeload event of an OBJECT element
https://bugs.webkit.org/show_bug.cgi?id=113964

Source/WebCore:

Object elements have the tendecny to modify or even fully remove
the containing Document inside beforeload callback. While Document is removed,
RenderArena gets destroyed. Retained RenderWidgets fails to function with NULL arena.

Protect RendereArena from getting wiped out, when Document is removed
during FrameView::updateWidget().

Patch by Zalan Bujtas <Alan Bujtas> on 2013-04-26
Reviewed by Antti Koivisto.

Test: fast/frames/crash-remove-iframe-during-object-beforeload.html

  • dom/Document.cpp:

(WebCore::Document::attach):

  • dom/Document.h:

(Document):

  • page/FrameView.cpp:

(WebCore::FrameView::updateWidgets):

  • rendering/RenderArena.h:

(RenderArena):
(WebCore::RenderArena::create):

LayoutTests:

Patch by Zalan Bujtas <Alan Bujtas> on 2013-04-26
Reviewed by Antti Koivisto.

  • fast/frames/crash-remove-iframe-during-object-beforeload-expected.txt: Added.
  • fast/frames/crash-remove-iframe-during-object-beforeload.html: Added.
  • fast/frames/resources/remove-this-during-object-beforeload.html: Added.
7:56 AM Changeset in webkit [149184] by mikhail.pozdnyakov@intel.com
  • 2 edits in trunk/Source/WTF

Add move semantics to RefPtr
https://bugs.webkit.org/show_bug.cgi?id=115033

Reviewed by Anders Carlsson.

Add move constructors and move assignment operators to RefPtr when
COMPILER_SUPPORTS(CXX_RVALUE_REFERENCES).
This obviates unnecessary reffing/ureffing when RefPtr is created
or assigned from rvalue references.

  • wtf/RefPtr.h:

(RefPtr):
(WTF::RefPtr::RefPtr):
(WTF::RefPtr::operator=):

7:32 AM Changeset in webkit [149183] by mhahnenberg@apple.com
  • 2 edits in trunk/Source/JavaScriptCore

REGRESSION(r149165): It made many tests crash on 32 bit
https://bugs.webkit.org/show_bug.cgi?id=115227

Reviewed by Csaba Osztrogonác.

m_reservation is uninitialized when ENABLE(SUPER_REGION) is false.

  • heap/SuperRegion.cpp:

(JSC::SuperRegion::~SuperRegion):

6:59 AM Changeset in webkit [149182] by mary.wu@torchmobile.com.cn
  • 2 edits in trunk/Source/WebKit/blackberry

[BlackBerry] Should check if it's cached resource before download
https://bugs.webkit.org/show_bug.cgi?id=115101

Reviewed by Rob Buis.

Since main resource maybe cached, if user want to save the resource, we first check
if it's cached. If yes, don't need to initiate a fresh load again, but get the
cached resource data out to save.

RIM bug# 324003, internally reviewed by Charles Wei.

  • WebCoreSupport/FrameLoaderClientBlackBerry.cpp:

(WebCore::FrameLoaderClientBlackBerry::convertMainResourceLoadToDownload):

6:20 AM Changeset in webkit [149181] by zandobersek@gmail.com
  • 3 edits
    20 adds in trunk/LayoutTests

[GTK] Enable Web Audio layout tests on WK2, provide platform-specific expected output
https://bugs.webkit.org/show_bug.cgi?id=114513

Reviewed by Philippe Normand.

Unskip the Web Audio layout tests for GTK WK2 and generate platform-specific baselines for the tests.
The audio baselines are not audibly different from the generic baselines, but it can be expected that
different backends won't produce the same binary output.

The remaining failures are marked as such in the TestExpectations, each failure having the appropriate
bug handle covering the issue. The tests are still skipped for GTK WK1 as DumpRenderTree is still missing
infrastructure support for running these tests.

  • platform/gtk-wk1/TestExpectations:
  • platform/gtk/TestExpectations:
  • platform/gtk/webaudio/audiobuffersource-loop-points-expected.wav: Added.
  • platform/gtk/webaudio/audiobuffersource-playbackrate-expected.wav: Added.
  • platform/gtk/webaudio/codec-tests/aac/vbr-128kbps-44khz-expected.wav: Added.
  • platform/gtk/webaudio/codec-tests/mp3/128kbps-44khz-expected.wav: Added.
  • platform/gtk/webaudio/codec-tests/vorbis/vbr-128kbps-44khz-expected.wav: Added.
  • platform/gtk/webaudio/codec-tests/vorbis/vbr-70kbps-44khz-expected.wav: Added.
  • platform/gtk/webaudio/codec-tests/vorbis/vbr-96kbps-44khz-expected.wav: Added.
  • platform/gtk/webaudio/codec-tests/wav/24bit-22khz-resample-expected.wav: Added.
  • platform/gtk/webaudio/codec-tests/wav/24bit-44khz-expected.wav: Added.
  • platform/gtk/webaudio/oscillator-custom-expected.wav: Added.
  • platform/gtk/webaudio/oscillator-sawtooth-expected.wav: Added.
  • platform/gtk/webaudio/oscillator-sine-expected.wav: Added.
  • platform/gtk/webaudio/oscillator-square-expected.wav: Added.
  • platform/gtk/webaudio/oscillator-triangle-expected.wav: Added.
6:16 AM Changeset in webkit [149180] by mary.wu@torchmobile.com.cn
  • 5 edits in trunk

[BlackBerry] Clean up load interface in WebPage
https://bugs.webkit.org/show_bug.cgi?id=113267

Reviewed by Rob Buis.

Source/WebKit/blackberry:

Remove unused loadExtended(), combine load() and download() api in WebPage.
RIM Bug# 315535, internally reviewed by Joe Mason.

  • Api/WebPage.cpp:

(BlackBerry::WebKit::WebPage::load):

  • Api/WebPage.h:

Tools:

Update DumpRenderTree with change of load() interface in WebPage.
RIM Bug# 315535, internally reviewed by Joe Mason.

  • DumpRenderTree/blackberry/DumpRenderTree.cpp:

(BlackBerry::WebKit::DumpRenderTree::runTest):

6:09 AM Changeset in webkit [149179] by Claudio Saavedra
  • 2 edits in trunk/LayoutTests

[GTK] Unreviewed gardnering. Unskip passing test.

  • platform/gtk/TestExpectations: Unskip http/tests/inspector/compiler-source-mapping-debug.html
6:07 AM Changeset in webkit [149178] by akling@apple.com
  • 8 edits
    3 deletes in trunk/Source/WTF

Remove wxWebKit from WTF.
<http://webkit.org/b/115249>

Reviewed by Antti Koivisto.

  • wscript: Removed.
  • wtf/ArrayBufferView.h:

(ArrayBufferView):

  • wtf/CurrentTime.cpp:
  • wtf/DisallowCType.h:
  • wtf/ExportMacros.h:
  • wtf/FeatureDefines.h:
  • wtf/Platform.h:
  • wtf/text/WTFString.h:

(String):

  • wtf/wx/MainThreadWx.cpp: Removed.
  • wtf/wx/StringWx.cpp: Removed.
6:03 AM Changeset in webkit [149177] by Christophe Dumez
  • 2 edits in trunk/Source/WebCore

Optimize function and interface object length computation in bindings generator
https://bugs.webkit.org/show_bug.cgi?id=115247

Reviewed by Kentaro Hara.

Introduce new GetFunctionLength() function that efficiently compute the length
of a function (i.e. its number of mandatory parameters).

We now call GetFunctionLength() instead of GenerateFunctionParametersCheck()
whenever we care only interested in the function length and not the actual
expression for checking the parameters. This is much more efficient as
GenerateFunctionParametersCheck() does a lot more processing than we need in
this case.

No new tests, no behavior change.

  • bindings/scripts/CodeGeneratorJS.pm:

(GetFunctionLength):
(GenerateImplementation):
(GenerateConstructorHelperMethods):

6:02 AM Changeset in webkit [149176] by zandobersek@gmail.com
  • 1 edit
    1 delete in trunk/Tools

Remove the HeapGraphSerializer unit test
https://bugs.webkit.org/show_bug.cgi?id=115231

Reviewed by Andreas Kling.

Remove the HeapGraphSerializer unit test. The code it was testing was removed in r148921,
specifically the Source/WebCore/inspector/HeapGraphSerializer.(cpp|h) files.

  • TestWebKitAPI/Tests/WebCore/HeapGraphSerializerTest.cpp: Removed.
5:09 AM FeatureFlags edited by akling@apple.com
(diff)
4:48 AM Changeset in webkit [149175] by akling@apple.com
  • 5 edits in trunk/Source/WebCore

Web Audio: Remove reduplicative addInput() in AnalyserNode.
<http://webkit.org/b/115244>

From Blink r149155 by <james.wei@intel.com>:

The parent class BasicInspectorNode already called addInput() and addOutput().
So it is reduplicative to call them in AnalyserNode.

  • Modules/webaudio/AnalyserNode.cpp:

(WebCore::AnalyserNode::AnalyserNode):

  • Modules/webaudio/AudioBasicInspectorNode.cpp:

(WebCore::AudioBasicInspectorNode::AudioBasicInspectorNode):

  • Modules/webaudio/AudioBasicInspectorNode.h:

(AudioBasicInspectorNode):

  • Modules/webaudio/MediaStreamAudioDestinationNode.cpp:

(WebCore::MediaStreamAudioDestinationNode::MediaStreamAudioDestinationNode):

3:30 AM Changeset in webkit [149174] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebKit/efl

[EFL] Add unit test cases for ewk_settings APIs
https://bugs.webkit.org/show_bug.cgi?id=114897

Patch by Jose Lejin PJ <jose.lejin@gmail.com> on 2013-04-26
Reviewed by Gyuyoung Kim.

Added unit tests for ewk_settings APIs.

  • tests/test_ewk_setting.cpp:

(TEST_F):

2:42 AM Changeset in webkit [149173] by allan.jensen@digia.com
  • 20 edits
    4 adds in trunk

Mouseenter and mouseleave events not supported
https://bugs.webkit.org/show_bug.cgi?id=18930

Reviewed by David Hyatt.

Source/WebCore:

Implements mouseenter and mouseleave events from W3C DOM Level 3 Events.
These event are already supported by all other major browsers.

To avoid performance regressions the new events are only dispatched when
there are event listeners for them.

Tests: fast/events/mouseenter-mouseleave-capture.html

fast/events/mouseenter-mouseleave.html

  • bindings/scripts/CodeGenerator.pm:
  • dom/Document.cpp:

(WebCore::Document::prepareMouseEvent):
(WebCore::Document::updateHoverActiveState):

  • dom/Document.h:

(Document):

  • dom/Document.idl:
  • dom/Element.h:

(Element):

  • dom/Element.idl:
  • dom/EventListenerMap.cpp:

(WebCore::EventListenerMap::containsCapturing):

  • dom/EventListenerMap.h:

(EventListenerMap):

  • dom/EventNames.h:
  • dom/EventTarget.h:

(EventTarget):
(WebCore::EventTarget::hasCapturingEventListeners):

  • dom/MouseEvent.cpp:

(WebCore::MouseEvent::create):
(WebCore::MouseEvent::toElement):
(WebCore::MouseEvent::fromElement):

  • html/HTMLAttributeNames.in:
  • html/HTMLElement.cpp:

(WebCore::HTMLElement::eventNameForAttributeName):

  • page/DOMWindow.h:

(DOMWindow):

  • page/DOMWindow.idl:
  • svg/SVGElement.cpp:

(WebCore::SVGElement::parseAttribute):

  • svg/SVGElementInstance.h:

(SVGElementInstance):

  • svg/SVGElementInstance.idl:

LayoutTests:

To new tests that mouseenter and mouseleave works in both bubbling and capture phase.

  • fast/events/mouseenter-mouseleave-capture-expected.txt: Added.
  • fast/events/mouseenter-mouseleave-capture.html: Added.
  • fast/events/mouseenter-mouseleave-expected.txt: Added.
  • fast/events/mouseenter-mouseleave.html: Added.
2:36 AM Changeset in webkit [149172] by commit-queue@webkit.org
  • 2 edits in trunk/Source/JavaScriptCore

Fix SH4 build broken since r149159.
https://bugs.webkit.org/show_bug.cgi?id=115229

Add BranchTruncateType enum in SH4 port and handle it in branchTruncateDoubleToInt32.

Patch by Julien Brianceau <jbrianceau@nds.com> on 2013-04-26
Reviewed by Allan Sandfeld Jensen.

  • assembler/MacroAssemblerSH4.h:

(JSC::MacroAssemblerSH4::branchTruncateDoubleToInt32):

2:16 AM Changeset in webkit [149171] by Christophe Dumez
  • 5 edits in trunk/LayoutTests

[Qt] REGRESSION(r149001): It made two fast/dom/DeviceMotion tests fail.
https://bugs.webkit.org/show_bug.cgi?id=115235

Reviewed by Kentaro Hara.

Update window-property.html test cases for DeviceMotion and DeviceOrientation.
The global constructors are no longer enumerable after r149001.

  • fast/dom/DeviceMotion/script-tests/window-property.js:
  • fast/dom/DeviceMotion/window-property-expected.txt:
  • fast/dom/DeviceOrientation/script-tests/window-property.js:
  • fast/dom/DeviceOrientation/window-property-expected.txt:
12:38 AM WebKitIDL edited by Christophe Dumez
Drop Supplemental in favor of partial interfaces due to r149170 (diff)
12:35 AM WebKitIDL edited by Christophe Dumez
[Callback] only applies to parameters, not interfaces (diff)
12:29 AM Changeset in webkit [149170] by Christophe Dumez
  • 33 edits in trunk/Source/WebCore

Add support for Web IDL partial interfaces to the bindings generator
https://bugs.webkit.org/show_bug.cgi?id=115228

Reviewed by Kentaro Hara.

Add support for Web IDL partial interfaces to the bindings generator:
http://dev.w3.org/2006/webapi/WebIDL/#dfn-partial-interface

Also drop support for the the WebKit-specific [Supplemental] extended
attribute and use partial interfaces instead in existing IDL files.

No new tests, no behavior change.

  • Modules/battery/NavigatorBattery.idl:
  • Modules/filesystem/DOMWindowFileSystem.idl:
  • Modules/filesystem/DataTransferItemFileSystem.idl:
  • Modules/filesystem/HTMLInputElementFileSystem.idl:
  • Modules/filesystem/WorkerContextFileSystem.idl:
  • Modules/gamepad/NavigatorGamepad.idl:
  • Modules/geolocation/NavigatorGeolocation.idl:
  • Modules/indexeddb/DOMWindowIndexedDatabase.idl:
  • Modules/indexeddb/WorkerContextIndexedDatabase.idl:
  • Modules/mediastream/DOMWindowMediaStream.idl:
  • Modules/mediastream/NavigatorMediaStream.idl:
  • Modules/navigatorcontentutils/NavigatorContentUtils.idl:
  • Modules/networkinfo/NavigatorNetworkInfoConnection.idl:
  • Modules/notifications/DOMWindowNotifications.idl:
  • Modules/notifications/WorkerContextNotifications.idl:
  • Modules/quota/DOMWindowQuota.idl:
  • Modules/quota/NavigatorStorageQuota.idl:
  • Modules/quota/WorkerNavigatorStorageQuota.idl:
  • Modules/speech/DOMWindowSpeech.idl:
  • Modules/speech/DOMWindowSpeechSynthesis.idl:
  • Modules/vibration/NavigatorVibration.idl:
  • Modules/webaudio/DOMWindowWebAudio.idl:
  • Modules/webdatabase/DOMWindowWebDatabase.idl:
  • Modules/webdatabase/WorkerContextWebDatabase.idl:
  • Modules/websockets/DOMWindowWebSocket.idl:
  • Modules/websockets/WorkerContextWebSocket.idl:
  • bindings/scripts/IDLAttributes.txt:
  • bindings/scripts/IDLParser.pm:

(parsePartialDefinition):

  • bindings/scripts/generate-bindings.pl:
  • bindings/scripts/preprocess-idls.pl:

(getPartialInterfaceNameFromIDLFile):

  • bindings/scripts/test/TestSupplemental.idl:
  • page/DOMWindowPagePopup.idl:

Apr 25, 2013:

11:50 PM Changeset in webkit [149169] by zandobersek@gmail.com
  • 3 edits in trunk/Source/WebKit2

[GTK] Plugin process broken due to a missing symbol
https://bugs.webkit.org/show_bug.cgi?id=114901

Reviewed by Gustavo Noronha Silva.

Work around the missing symbols in the WebKitPluginProcess by specifying the required libraries
multiple times in the program's LDADD list. Libtool fights fiercely to avoid duplicate static library
references among the linker arguments (even if using the --preserve-dup-deps flag) so the required libraries
are specified in various ways.

The unresolved symbols in object files are no longer ignored as this can lead to further trouble (for instance
this patch also adds a few missing source files to the list of WebKitPluginProcess sources). The Freetype and
gamepad dependencies' libraries are also added to the binary's LDADD list.

  • GNUmakefile.am:
  • GNUmakefile.list.am: Add a few missing files to the build.
11:23 PM Changeset in webkit [149168] by mihnea@adobe.com
  • 4 edits
    2 adds in trunk

[CSS Regions] Hit testing is broken for absolutely positioned regions that have overflow: hidden
https://bugs.webkit.org/show_bug.cgi?id=113874

Reviewed by David Hyatt.

Source/WebCore:

Test: fast/regions/hit-test-abspos-overflow-region.html

When a region is an out-of-flow positioned object with an overflow clip, we need
to make sure that hit testing works also for cases other than foreground (content)
hit testing. This patch moves the previous hit testing code, that delegated foreground
hit testing to the region's flow thread hit testing, into the hitTestContent method,
now that RenderRegion is RenderBlock based.

  • rendering/RenderRegion.cpp:

(WebCore::RenderRegion::hitTestContents):

  • rendering/RenderRegion.h:

(RenderRegion):

LayoutTests:

Add test for hit-testing of border of an out-of-flow region with overflow: hidden.

  • fast/regions/hit-test-abspos-overflow-region-expected.txt: Added.
  • fast/regions/hit-test-abspos-overflow-region.html: Added.
9:51 PM Changeset in webkit [149167] by rniwa@webkit.org
  • 4 edits
    2 adds in trunk

Copy and paste can strip !important CSS rules due to a bug in mergeStyleFromRules
https://bugs.webkit.org/show_bug.cgi?id=115217

Reviewed by Darin Adler.

Source/WebCore:

The bug was caused by mergeStyleFromRules overriding "important" style rules with "unimportant" inline styles.
Fixed the bug by using addParsedProperty, which respects !important, in MutableStylePropertySet's
mergeAndOverrideOnConflict, which was only used in editing code. Now that we've fixed this function, we can use
it in ViewportStyleResolver::addViewportRule as well.

Test: editing/pasteboard/copy-paste-with-important-rules.html

  • css/StylePropertySet.cpp:

(WebCore::MutableStylePropertySet::mergeAndOverrideOnConflict): Fixed to respect !important.

  • css/ViewportStyleResolver.cpp:

(WebCore::ViewportStyleResolver::addViewportRule): Use mergeAndOverrideOnConflict now that the code is identical.

LayoutTests:

Added a regression test.

  • editing/pasteboard/copy-paste-with-important-rules-expected.txt: Added.
  • editing/pasteboard/copy-paste-with-important-rules.html: Added.
9:50 PM Changeset in webkit [149166] by commit-queue@webkit.org
  • 3 edits in trunk/Tools

Remove support for Chromium from build/update scripts
https://bugs.webkit.org/show_bug.cgi?id=115218

Patch by Seokju Kwon <Seokju Kwon> on 2013-04-25
Reviewed by Darin Adler.

  • Scripts/build-webkit:

(writeCongrats):

  • Scripts/update-webkit:
7:47 PM Changeset in webkit [149165] by mhahnenberg@apple.com
  • 4 edits in trunk/Source/JavaScriptCore

SuperRegion doesn't call deallocate() on its PageReservation
https://bugs.webkit.org/show_bug.cgi?id=115208

Reviewed by Geoffrey Garen.

It should. This doesn't cause us to leak physical memory, but it does cause us to leak virtual
address space (and probably mach ports), which is also bad :-( FixedVMPoolExecutableAllocator
also has this bug, but it doesn't matter much because there's only one instance of that class
throughout the entire lifetime of the process, whereas each VM has its own SuperRegion.

  • heap/SuperRegion.cpp:

(JSC::SuperRegion::~SuperRegion):

  • heap/SuperRegion.h:

(SuperRegion):

  • jit/ExecutableAllocatorFixedVMPool.cpp:

(FixedVMPoolExecutableAllocator):
(JSC::FixedVMPoolExecutableAllocator::~FixedVMPoolExecutableAllocator):

7:30 PM Changeset in webkit [149164] by commit-queue@webkit.org
  • 3 edits in trunk/Tools

Remove support for Chromium from api-test
https://bugs.webkit.org/show_bug.cgi?id=115211

Patch by Seokju Kwon <Seokju Kwon> on 2013-04-25
Reviewed by Dirk Pranke.

  • Scripts/build-api-tests:
  • Scripts/run-api-tests:

(isSupportedPlatform):

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

Get rid of Chromium in build-dumprendertree
https://bugs.webkit.org/show_bug.cgi?id=115212

Patch by Seokju Kwon <Seokju Kwon> on 2013-04-25
Reviewed by Dirk Pranke.

  • Scripts/build-dumprendertree:
6:18 PM Changeset in webkit [149162] by fpizlo@apple.com
  • 7 edits in trunk/Source/JavaScriptCore

DFG doesn't support to_jsnumber
https://bugs.webkit.org/show_bug.cgi?id=115129

Reviewed by Geoffrey Garen.

Based on Oliver's patch. Implements to_jsnumber as Identity(Number:@thingy), and then does
an optimization in Fixup to turn Identity(Number:) into Identity(Int32:) if the predictions
tell us to. Identity is later turned into Phantom.

Also fixed BackPropMask, which appeared to have NodeDoesNotExit included in it. That's
wrong; NodeDoesNotExit is not a backward propagation property.

Also fixed Identity to be marked as CanExit (i.e. not NodeDoesNotExit).

This more than doubles the FPS on ammo.

  • dfg/DFGByteCodeParser.cpp:

(JSC::DFG::ByteCodeParser::parseBlock):

  • dfg/DFGCapabilities.h:

(JSC::DFG::canCompileOpcode):

  • dfg/DFGFixupPhase.cpp:

(JSC::DFG::FixupPhase::fixupNode):
(FixupPhase):
(JSC::DFG::FixupPhase::observeUseKindOnNode):
(JSC::DFG::FixupPhase::observeUseKindOnEdge):

  • dfg/DFGNodeFlags.h:

(DFG):

  • dfg/DFGNodeType.h:

(DFG):

  • dfg/DFGPredictionPropagationPhase.cpp:

(JSC::DFG::PredictionPropagationPhase::propagate):

5:50 PM Changeset in webkit [149161] by mrowe@apple.com
  • 4 edits in trunk/Source

Versioning.

5:49 PM Changeset in webkit [149160] by mrowe@apple.com
  • 1 copy in tags/Safari-537.39

New tag.

5:41 PM Changeset in webkit [149159] by oliver@apple.com
  • 21 edits
    12 adds in trunk

Add support for Math.imul
https://bugs.webkit.org/show_bug.cgi?id=115143

Reviewed by Filip Pizlo.

Source/JavaScriptCore:

Add support for Math.imul, a thunk generator for Math.imul,
and an intrinsic.

Fairly self explanatory set of changes, DFG intrinsics simply
leverages the existing ValueToInt32 nodes.

  • create_hash_table:
  • dfg/DFGAbstractState.cpp:

(JSC::DFG::AbstractState::executeEffects):

  • dfg/DFGBackwardsPropagationPhase.cpp:

(JSC::DFG::BackwardsPropagationPhase::propagate):

  • dfg/DFGByteCodeParser.cpp:

(JSC::DFG::ByteCodeParser::handleIntrinsic):

  • dfg/DFGCSEPhase.cpp:

(JSC::DFG::CSEPhase::performNodeCSE):

  • dfg/DFGFixupPhase.cpp:

(JSC::DFG::FixupPhase::fixupNode):

  • dfg/DFGNodeType.h:

(DFG):

  • dfg/DFGPredictionPropagationPhase.cpp:

(JSC::DFG::PredictionPropagationPhase::propagate):

  • dfg/DFGSpeculativeJIT.cpp:

(JSC::DFG::SpeculativeJIT::compileArithIMul):

  • dfg/DFGSpeculativeJIT.h:

(SpeculativeJIT):

  • dfg/DFGSpeculativeJIT32_64.cpp:

(JSC::DFG::SpeculativeJIT::compile):

  • dfg/DFGSpeculativeJIT64.cpp:

(JSC::DFG::SpeculativeJIT::compile):

  • jit/ThunkGenerators.cpp:

(JSC::imulThunkGenerator):
(JSC):

  • jit/ThunkGenerators.h:

(JSC):

  • runtime/Intrinsic.h:
  • runtime/MathObject.cpp:

(JSC):
(JSC::mathProtoFuncIMul):

  • runtime/VM.cpp:

(JSC::thunkGeneratorForIntrinsic):

LayoutTests:

Add a bunch of tests for Math.imul

  • fast/js/Object-getOwnPropertyNames-expected.txt:
  • fast/js/imul-expected.txt: Added.
  • fast/js/imul.html: Added.
  • fast/js/regress/imul-double-only-expected.txt: Added.
  • fast/js/regress/imul-double-only.html: Added.
  • fast/js/regress/imul-int-only-expected.txt: Added.
  • fast/js/regress/imul-int-only.html: Added.
  • fast/js/regress/imul-mixed-expected.txt: Added.
  • fast/js/regress/imul-mixed.html: Added.
  • fast/js/regress/script-tests/imul-double-only.js: Added.

(f):

  • fast/js/regress/script-tests/imul-int-only.js: Added.

(f):

  • fast/js/regress/script-tests/imul-mixed.js: Added.

(f):

  • fast/js/script-tests/Object-getOwnPropertyNames.js:
  • fast/js/script-tests/imul.js: Added.

(testIMul):

5:40 PM Changeset in webkit [149158] by fpizlo@apple.com
  • 15 edits in trunk/Source

Unreviewed, roll out http://trac.webkit.org/changeset/148999
It broke http://kripken.github.io/ammo.js/examples/new/ammo.html

Source/JavaScriptCore:

(JSC::BytecodeGenerator::emitNewArray):
(JSC::BytecodeGenerator::emitThrowReferenceError):
(JSC::BytecodeGenerator::emitReadOnlyExceptionIfNeeded):

  • bytecompiler/BytecodeGenerator.h:

(JSC::BytecodeGenerator::shouldEmitProfileHooks):
(BytecodeGenerator):

  • bytecompiler/NodesCodegen.cpp:

(JSC):
(JSC::NullNode::emitBytecode):
(JSC::BooleanNode::emitBytecode):
(JSC::NumberNode::emitBytecode):
(JSC::StringNode::emitBytecode):
(JSC::IfNode::emitBytecode):
(JSC::IfElseNode::emitBytecode):

  • parser/ASTBuilder.h:

(JSC::ASTBuilder::createIfStatement):
(ASTBuilder):

  • parser/NodeConstructors.h:

(JSC):
(JSC::NullNode::NullNode):
(JSC::BooleanNode::BooleanNode):
(JSC::NumberNode::NumberNode):
(JSC::StringNode::StringNode):
(JSC::IfNode::IfNode):
(JSC::IfElseNode::IfElseNode):

  • parser/Nodes.h:

(JSC::ExpressionNode::isPure):
(JSC::ExpressionNode::isSubtract):
(StatementNode):
(NullNode):
(JSC::NullNode::isNull):
(BooleanNode):
(JSC::BooleanNode::isPure):
(NumberNode):
(JSC::NumberNode::value):
(JSC::NumberNode::isPure):
(StringNode):
(JSC::StringNode::isPure):
(JSC::StringNode::isString):
(BinaryOpNode):
(IfNode):
(JSC):
(IfElseNode):
(ContinueNode):
(BreakNode):

  • parser/Parser.cpp:

(JSC::::parseIfStatement):

  • parser/ResultType.h:

(ResultType):

  • runtime/JSCJSValueInlines.h:

(JSC::JSValue::pureToBoolean):

  • runtime/JSCell.h:

(JSCell):

  • runtime/JSCellInlines.h:

(JSC):

Source/WTF:

  • wtf/TriState.h:
5:33 PM Changeset in webkit [149157] by akling@apple.com
  • 4 edits in trunk/Source/WebCore

StylePropertySet::getPropertyShorthand() should return a String.
<http://webkit.org/b/115213>

Reviewed by Anders Carlsson.

Return a String directly from getPropertyShorthand() instead of forcing clients
to call getPropertyNameString().

  • css/PropertySetCSSStyleDeclaration.cpp:

(WebCore::PropertySetCSSStyleDeclaration::getPropertyShorthand):

  • css/StylePropertySet.cpp:

(WebCore::StylePropertySet::getPropertyShorthand):

  • css/StylePropertySet.h:

(StylePropertySet):

5:27 PM Changeset in webkit [149156] by Chris Fleizach
  • 13 edits in trunk

WEB SPEECH: language support does not work as expected
https://bugs.webkit.org/show_bug.cgi?id=115119

Reviewed by Alexey Proskuryakov.

Source/WebCore:

Make the Mac platform synthesizer have access to all the voices installed on the system.

  • WebCore.exp.in:
  • platform/mac/PlatformSpeechSynthesizerMac.mm:

(WebCore::PlatformSpeechSynthesizer::initializeVoiceList):

  • platform/mac/WebCoreSystemInterface.h:
  • platform/mac/WebCoreSystemInterface.mm:

Source/WebKit/mac:

  • WebCoreSupport/WebSystemInterface.mm:

(InitWebCoreSystemInterface):

Source/WebKit2:

  • WebProcess/WebCoreSupport/mac/WebSystemInterface.mm:

(InitWebCoreSystemInterface):

WebKitLibraries:

  • WebKitSystemInterface.h:
4:33 PM Changeset in webkit [149155] by Chris Fleizach
  • 11 edits
    2 adds in trunk

Source/WebCore: <meter> element not exposed to accessibility
https://bugs.webkit.org/show_bug.cgi?id=109023
rdar://problem/13658964

Reviewed by Tim Horton.

Makes the <meter> element appear in the AX tree by reusing the
AccessibilityProgressIndicator element to handle either progress or meter
elements.

Test: accessibility/meter-element.html

  • accessibility/AXObjectCache.cpp:

(WebCore::createFromRenderer):
(WebCore::AXObjectCache::getOrCreate):

  • accessibility/AccessibilityNodeObject.cpp:

(WebCore::AccessibilityNodeObject::canHaveChildren):
(WebCore::AccessibilityNodeObject::visibleText):

  • accessibility/AccessibilityProgressIndicator.cpp:

(WebCore::AccessibilityProgressIndicator::AccessibilityProgressIndicator):
(WebCore::AccessibilityProgressIndicator::create):
(WebCore::AccessibilityProgressIndicator::valueForRange):
(WebCore::AccessibilityProgressIndicator::maxValueForRange):
(WebCore::AccessibilityProgressIndicator::minValueForRange):
(WebCore::AccessibilityProgressIndicator::progressElement):
(WebCore::AccessibilityProgressIndicator::meterElement):

  • accessibility/AccessibilityProgressIndicator.h:

(AccessibilityProgressIndicator):

  • accessibility/AccessibilityRenderObject.cpp:

(WebCore::AccessibilityRenderObject::firstChild):
(WebCore::AccessibilityRenderObject::lastChild):

These methods are modified to handle when there is no rendered children, but there are Node
children. The node children are desired when calculating text within a node, for example.

LayoutTests: <meter> element not exposed to accessibility
https://bugs.webkit.org/show_bug.cgi?id=109023

Reviewed by Tim Horton.

  • accessibility/meter-element-expected.txt: Added.
  • accessibility/meter-element.html: Added.
  • platform/mac/accessibility/role-subrole-roledescription-expected.txt:
  • platform/mac/accessibility/role-subrole-roledescription.html:
4:10 PM Changeset in webkit [149154] by fpizlo@apple.com
  • 3 edits in trunk/Source/JavaScriptCore

PreciseJumpTargets should treat loop_hint as a jump target
https://bugs.webkit.org/show_bug.cgi?id=115209

Reviewed by Mark Hahnenberg.

I didn't add a test but I turned this into a release assertion. Running Octane is enough
to trigger it.

  • bytecode/PreciseJumpTargets.cpp:

(JSC::computePreciseJumpTargets):

  • dfg/DFGByteCodeParser.cpp:

(JSC::DFG::ByteCodeParser::parseBlock):

4:06 PM Changeset in webkit [149153] by commit-queue@webkit.org
  • 5 edits in trunk/Source/WebCore

[CSS Shaders] Remove the meshType from the CustomFilterOperation
https://bugs.webkit.org/show_bug.cgi?id=102529

Patch by Qiankun Miao <qiankun.miao@intel.com> on 2013-04-25
Reviewed by Dean Jackson.

No new tests, no new functionality.

MeshType can be accessed from program, so remove the m_meshType
related code in CustomFilterOperation.

  • css/StyleResolver.cpp:

(WebCore::StyleResolver::createCustomFilterOperationWithInlineSyntax):

  • platform/graphics/filters/CustomFilterOperation.cpp:

(WebCore::CustomFilterOperation::CustomFilterOperation):
(WebCore::CustomFilterOperation::blend):

  • platform/graphics/filters/CustomFilterOperation.h:

(WebCore::CustomFilterOperation::create):
(WebCore::CustomFilterOperation::meshType):
(CustomFilterOperation):
(WebCore::CustomFilterOperation::operator==):

  • platform/graphics/texmap/coordinated/CoordinatedCustomFilterOperation.h:

(WebCore::CoordinatedCustomFilterOperation::CoordinatedCustomFilterOperation):

3:52 PM Changeset in webkit [149152] by commit-queue@webkit.org
  • 10 edits
    9 adds in trunk

Source/JavaScriptCore: Fix problems with processing negative zero on DFG.
https://bugs.webkit.org/show_bug.cgi?id=113862

Patch by Roman Zhuykov <zhroma@ispras.ru> on 2013-04-25
Reviewed by Filip Pizlo.

Fix NodeNeedsNegZero flag propagation in BackwardPropagationPhase.
Function arithNodeFlags should not mask NodeNeedsNegZero flag for ArithNegate and DoubleAsInt32
nodes and this flag should be always used to decide where we need to generate nezative-zero checks.
Remove unnecessary negative-zero checks from integer ArithDiv on ARM.
Also remove such checks from integer ArithMod on ARM and X86, and make them always to
check not only "modulo_result == 0" but also "dividend < 0".
Generate faster code for case when ArithMod operation divisor is constant power of 2 on ARMv7
in the same way as on ARMv7s, and add negative-zero checks into this code when needed.
Change speculationCheck ExitKind from Overflow to NegativeZero where applicable.

This shows 30% speedup of math-spectral-norm, and 5% speedup
on SunSpider overall on ARMv7 Linux.

  • assembler/MacroAssemblerARM.h:

(JSC::MacroAssemblerARM::branchConvertDoubleToInt32):

  • assembler/MacroAssemblerARMv7.h:

(JSC::MacroAssemblerARMv7::branchConvertDoubleToInt32):

  • assembler/MacroAssemblerMIPS.h:

(JSC::MacroAssemblerMIPS::branchConvertDoubleToInt32):

  • assembler/MacroAssemblerSH4.h:

(JSC::MacroAssemblerSH4::branchConvertDoubleToInt32):

  • assembler/MacroAssemblerX86Common.h:

(JSC::MacroAssemblerX86Common::branchConvertDoubleToInt32):

  • dfg/DFGBackwardsPropagationPhase.cpp:

(JSC::DFG::BackwardsPropagationPhase::isNotNegZero):
(JSC::DFG::BackwardsPropagationPhase::isNotPosZero):
(JSC::DFG::BackwardsPropagationPhase::propagate):

  • dfg/DFGNode.h:

(JSC::DFG::Node::arithNodeFlags):

  • dfg/DFGSpeculativeJIT.cpp:

(JSC::DFG::SpeculativeJIT::compileDoubleAsInt32):
(JSC::DFG::SpeculativeJIT::compileSoftModulo):
(JSC::DFG::SpeculativeJIT::compileArithNegate):

LayoutTests: Arithmetic operations with negative zero should be optimized correclty.
https://bugs.webkit.org/show_bug.cgi?id=113862

Patch by Roman Zhuykov <zhroma@ispras.ru> on 2013-04-25
Reviewed by Filip Pizlo.

  • fast/js/regress/negative-zero-divide-expected.txt: Added.
  • fast/js/regress/negative-zero-divide.html: Added.
  • fast/js/regress/negative-zero-modulo-expected.txt: Added.
  • fast/js/regress/negative-zero-modulo.html: Added.
  • fast/js/regress/negative-zero-negate-expected.txt: Added.
  • fast/js/regress/negative-zero-negate.html: Added.
  • fast/js/regress/script-tests/negative-zero-divide.js: Added.

(foo):

  • fast/js/regress/script-tests/negative-zero-modulo.js: Added.

(foo):

  • fast/js/regress/script-tests/negative-zero-negate.js: Added.

(foo):

3:51 PM Changeset in webkit [149151] by tkent@chromium.org
  • 7 edits in trunk

Ignore invalid regular expressions for input[pattern].
https://bugs.webkit.org/show_bug.cgi?id=115204

Reviewed by Darin Adler.

Source/WebCore:

According to the specification, we should not proceed regular expression
matching if a pattern attribute value is an invalid regular
expression. We had a bug that invalid expressions such as
pattern=")foo(" made RegularExpression objects successfully.

http://www.whatwg.org/specs/web-apps/current-work/multipage/common-input-element-attributes.html#the-pattern-attribute

If an input element has a pattern attribute specified, and the
attribute's value, when compiled as a JavaScript regular expression with
the global, ignoreCase, and multiline flags disabled (see ECMA262
Edition 5, sections 15.10.7.2 through 15.10.7.4), compiles successfully,
then the resulting regular expression is the element's compiled pattern
regular expression. If the element has no such attribute, or if the
value doesn't compile successfully, then the element has no compiled
pattern regular expression.

This imports a part of Blink r148951.

Tests: Update fast/forms/ValidityState-patternMismatch.html

  • html/BaseTextInputType.cpp:

(WebCore::BaseTextInputType::patternMismatch):
Check correctness of pattern attribute value before wrapping with parentheses.

  • platform/text/RegularExpression.cpp:

(WebCore::RegularExpression::isValid): Added.

  • platform/text/RegularExpression.h:

(RegularExpression): Declare isValid.

LayoutTests:

  • fast/forms/ValidityState-patternMismatch-expected.txt:
  • fast/forms/ValidityState-patternMismatch.html:
3:50 PM Changeset in webkit [149150] by oliver@apple.com
  • 2 edits in trunk/Source/WTF

Fix 32bit build

3:37 PM Changeset in webkit [149149] by Simon Fraser
  • 2 edits in trunk/Source/WebKit/mac

Build fix for smart compliers.

Reviewed by Tim Horton.

[NSDictionary dictionaryWithObjectsAndKeys:] returns an NSDictionary,
not a NSMutableDictionary.

  • WebCoreSupport/WebEditorClient.mm:

(WebEditorClient::updateSpellingUIWithGrammarString):

3:25 PM Changeset in webkit [149148] by ap@apple.com
  • 8 edits in trunk/Tools

Pass relatedPage when creating a page in WebKitTestRunner
https://bugs.webkit.org/show_bug.cgi?id=115206

Reviewed by Geoffrey Garen.

To make sure that window.open() opens in the same process even when using multiple processes.

  • WebKitTestRunner/PlatformWebView.h:
  • WebKitTestRunner/TestController.cpp: (WTR::TestController::createOtherPage): (WTR::TestController::createWebViewWithOptions):
  • WebKitTestRunner/efl/PlatformWebViewEfl.cpp: (WTR::PlatformWebView::PlatformWebView):
  • WebKitTestRunner/gtk/PlatformWebViewGtk.cpp: (WTR::PlatformWebView::PlatformWebView):
  • WebKitTestRunner/mac/PlatformWebViewMac.mm: (-[TestRunnerWKView initWithFrame:contextRef:pageGroupRef:relatedToPage:useTiledDrawing:]): (WTR::PlatformWebView::PlatformWebView):
  • WebKitTestRunner/qt/PlatformWebViewQt.cpp: (WTR::PlatformWebView::PlatformWebView):
  • WebKitTestRunner/win/PlatformWebViewWin.cpp: (WTR::PlatformWebView::PlatformWebView):
3:23 PM Changeset in webkit [149147] by andersca@apple.com
  • 5 edits in trunk/Source/WebKit2

Rename DecidePolicyForResponse message to DecidePolicyForResponseSync
https://bugs.webkit.org/show_bug.cgi?id=115207

Reviewed by Beth Dakin.

In preparation for adding a new async DecidePolicyForResponse message,
rename the current one to DecidePolicyForResponseSync. Also split out the async part
into a separate WebPageProxy::decidePolicyForResponse member function.

  • UIProcess/WebPageProxy.cpp:

(WebKit::WebPageProxy::WebPageProxy):
(WebKit::WebPageProxy::receivedPolicyDecision):
(WebKit::WebPageProxy::decidePolicyForResponse):
(WebKit::WebPageProxy::decidePolicyForResponseSync):

  • UIProcess/WebPageProxy.h:

(WebPageProxy):

  • UIProcess/WebPageProxy.messages.in:
  • WebProcess/WebCoreSupport/WebFrameLoaderClient.cpp:

(WebKit::WebFrameLoaderClient::dispatchDecidePolicyForResponse):

3:17 PM Changeset in webkit [149146] by oliver@apple.com
  • 4 edits in trunk/Source

Stack guards are too conservative
https://bugs.webkit.org/show_bug.cgi?id=115147

Reviewed by Mark Hahnenberg.

Source/JavaScriptCore:

Increase stack guard to closer to old size.

  • interpreter/Interpreter.cpp:

(JSC::Interpreter::StackPolicy::StackPolicy):

Source/WTF:

Use getrlimit on darwin to get the stack size for the main thread.

  • wtf/StackBounds.cpp:

(WTF::StackBounds::initialize):

2:39 PM Changeset in webkit [149145] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebKit/blackberry

[BlackBerry] Selection overlay on non-composited iframes are incorrectly positioned.
https://bugs.webkit.org/show_bug.cgi?id=115197

Patch by Andrew Lo <anlo@rim.com> on 2013-04-25
Reviewed by Rob Buis.

When drawing the selection overlay, the rects to
paint when selecting text on non-composited sub-frames
need to be adjusted by the frame position.

  • WebKitSupport/SelectionOverlay.cpp:

(BlackBerry::WebKit::SelectionOverlay::paintContents):

2:29 PM Changeset in webkit [149144] by Antoine Quint
  • 2 edits in trunk/Source/WebCore

Glyphs may fail to render when using SVG font
https://bugs.webkit.org/show_bug.cgi?id=115193

Reviewed by Simon Fraser.

Calling SimpleFontData::applyTransforms() when the font used is
an SVG font makes little sense since Core Text doesn’t know or
understand SVG fonts and would be passed some other, unrelated
platform font.

  • platform/graphics/SimpleFontData.h:

(WebCore::SimpleFontData::applyTransforms):

2:21 PM Changeset in webkit [149143] by rniwa@webkit.org
  • 2 edits in trunk/Tools

Fix autocompletion for Benjamin. The Bugzilla email address must come first.

  • Scripts/webkitpy/common/config/contributors.json:
2:02 PM Changeset in webkit [149142] by jer.noble@apple.com
  • 3 edits in trunk/Source/WebCore

Media elements shouldn't resume playback when a page is restored from the back/forward cache if the WKView isn't in a window
https://bugs.webkit.org/show_bug.cgi?id=115191

Reviewed by Eric Carlson.

If a page is suspended, then resumed when its WebView or WKView has been removed from a window,
the page's media elements will unpause. Check whether media is allowed to start during resume()
via Page::canMediaStart() and if not, register for mediaCanStart notifications. Then, in
mediaCanStart() if the media is force-paused, unpause it.

  • html/HTMLMediaElement.cpp:

(WebCore::HTMLMediaElement::suspend):
(WebCore::HTMLMediaElement::resume):
(WebCore::HTMLMediaElement::mediaCanStart):

  • html/HTMLMediaElement.h:

(WebCore::HTMLMediaElement::pageConsentRequiredForResume):

2:01 PM Changeset in webkit [149141] by ap@apple.com
  • 2 edits in trunk/LayoutTests

Update a comment, mentioning a newly filed bug.

  • platform/wk2/TestExpectations:
1:37 PM Changeset in webkit [149140] by akling@apple.com
  • 7 edits in trunk/Source

Remove ENABLE(PARSED_STYLE_SHEET_CACHING) and make it always-on.

Rubber-stamped by Anders Koivisto.

Source/WebCore:

  • PlatformWinCE.cmake:
  • accessibility/AccessibilityObject.h:
  • html/HTMLLinkElement.cpp:

(WebCore::HTMLLinkElement::setCSSStyleSheet):

Source/WebKit:

  • PlatformWinCE.cmake:

Source/WebKit/blackberry:

  • WebCoreSupport/AboutDataEnableFeatures.in:

Source/WTF:

  • wtf/FeatureDefines.h:
1:26 PM Changeset in webkit [149139] by Patrick Gansterer
  • 5 edits in trunk/Source

Remove OS(WINCE) from AccessibilityObject.h
https://bugs.webkit.org/show_bug.cgi?id=115192

Reviewed by Andreas Kling.

Use the same files like the the other PLATFORM(WIN) ports use.

Source/WebCore:

  • PlatformWinCE.cmake:
  • accessibility/AccessibilityObject.h:

(AccessibilityObject):

Source/WebKit:

  • PlatformWinCE.cmake:
1:11 PM Changeset in webkit [149138] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebCore

Add definition of GlyphBufferGlyph for BlackBerry
https://bugs.webkit.org/show_bug.cgi?id=115183

Patch by Alberto Garcia <agarcia@igalia.com> on 2013-04-25
Reviewed by Xan Lopez.

BlackBerry uses an unsigned int.

  • platform/graphics/GlyphBuffer.h:

(WebCore):

1:02 PM Changeset in webkit [149137] by commit-queue@webkit.org
  • 5 edits in trunk/Source/WebKit/blackberry

[BlackBerry] Make scroll position adjustment work with pages with fixed position elements.
https://bugs.webkit.org/show_bug.cgi?id=115178

Patch by Iris Wu <shuwu@blackberry.com> on 2013-04-25
Reviewed by Rob Buis.

PR 308796

Currently the position WebPage::adjustDocumentScrollPosition adjusts is the top
left point of the viewport.
On the page with fixed position elements, we want it to adjust the position beneath
the fixed elements so it can be always visible.

The basic idea is:

  1. Detect if there are fixed position elements before going through ProximityDetector.
  2. If the fixed element exists, calculate its the size and the actual visible position

beneath it.

  1. Pass the position to ProximityDetector. Then according to the new position we get,

calculate the top left position of the viewport (final scroll position).

  • Api/WebPage.cpp:

(BlackBerry::WebKit::WebPage::fixedElementSizeDelta):
(WebKit):

  • Api/WebPage.h:
  • Api/WebPageCompositor.cpp:

(BlackBerry::WebKit::WebPageCompositorPrivate::findFixedElementRect):
(WebKit):

  • Api/WebPageCompositor_p.h:

(WebPageCompositorPrivate):

1:00 PM Changeset in webkit [149136] by oliver@apple.com
  • 2 edits in trunk/Source/JavaScriptCore

Stack guards are too conservative
https://bugs.webkit.org/show_bug.cgi?id=115147

Reviewed by Geoffrey Garen.

Reduce the limits and simplify the decision making.

  • interpreter/Interpreter.cpp:

(JSC::Interpreter::StackPolicy::StackPolicy):

12:56 PM Changeset in webkit [149135] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebKit/blackberry

[BlackBerry] Enable balanced page group load deferrer behaviour.
https://bugs.webkit.org/show_bug.cgi?id=115189

Patch by Mike Lattanzio <mlattanzio@blackberry.com> on 2013-04-25
Reviewed by Rob Buis.

Prevent a possible deadlock by enabling balanced deferrers.
Internally reviewed by: Joe Mason
PR 329986

  • Api/WebPage.cpp:

(BlackBerry::WebKit::WebPagePrivate::init):

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

JSC: Fix interpreter misbehavior in builds with JIT disabled
https://bugs.webkit.org/show_bug.cgi?id=115190

Patch by Nick Diego Yamane <nick.yamane@openbossa.org> on 2013-04-25
Reviewed by Oliver Hunt.

Commit http://trac.webkit.org/changeset/147858 modified
some details on how JS stack traces are built. The method
"getLineNumberForCallFrame", renamed in that changeset to
"getBytecodeOffsetForCallFrame" is always returning `0' when
JIT is disabled

How to reproduce:

  • Build webkit with JIT disabled
  • Open MiniBrowser, for example, with http://google.com
  • In a debug build, WebProcess will hit the following ASSERT: Source/JavaScriptCore/bytecode/UnlinkedCodeBlock.cpp:279 ASSERT(low);
  • interpreter/Interpreter.cpp:

(JSC::getBytecodeOffsetForCallFrame):

12:50 PM Changeset in webkit [149133] by rniwa@webkit.org
  • 2 edits in trunk/Tools

Build fix. Forgot to merge args.

  • Scripts/webkitpy/tool/bot/irc_command.py:

(Hi.execute):

12:44 PM Changeset in webkit [149132] by rniwa@webkit.org
  • 2 edits in trunk/Tools

webkitbot should recognize its own nickname in greetings.
https://bugs.webkit.org/show_bug.cgi?id=115196

Reviewed by Andreas Kling.

Make webkitbot recognize other forms of its nick. Also be tolerant of spaces between the nick and !.

  • Scripts/webkitpy/tool/bot/irc_command.py:

(Hi.execute):

12:31 PM Changeset in webkit [149131] by Joseph Pecoraro
  • 21 edits
    2 adds
    2 deletes in trunk

ScriptExecutionContext log exception should include a column number
https://bugs.webkit.org/show_bug.cgi?id=114315

Reviewed by Oliver Hunt.

Source/WebCore:

Test: inspector/console/console-exception-stack-traces.html

  • bindings/js/ScriptCallStackFactory.h:
  • bindings/js/ScriptCallStackFactory.cpp:

(WebCore::createScriptCallStackFromException):
Generate a ScriptCallStack from an exception. Use the vm.exceptionStack
if available, and fallback to the exception object where needed.

  • bindings/js/JSDOMBinding.cpp:

(WebCore::reportException):
Always include a non-empty call stack with exceptions.
Where not provided, fallback to the exception object.

  • dom/Document.h:
  • dom/Document.cpp:

(WebCore::Document::logExceptionToConsole):

  • dom/ScriptExecutionContext.h:
  • dom/ScriptExecutionContext.cpp:

(WebCore::ScriptExecutionContext::PendingException::PendingException):
(ScriptExecutionContext::PendingException):
(WebCore::ScriptExecutionContext::reportException):

  • workers/DefaultSharedWorkerRepository.cpp:
  • workers/SharedWorkerContext.cpp:

(WebCore::SharedWorkerContext::logExceptionToConsole):

  • workers/SharedWorkerContext.h:
  • workers/WorkerContext.cpp:

(WebCore::WorkerContext::logExceptionToConsole):

  • workers/WorkerContext.h:
  • workers/WorkerMessagingProxy.cpp:

(WebCore::WorkerExceptionTask::performTask):
Plumb columnNumber through as needed.

LayoutTests:

  • inspector/console/console-exception-stack-traces-expected.txt: Added.
  • inspector/console/console-exception-stack-traces.html: Added.

Verifies that exceptions have stack traces and match a console.trace stack.

  • http/tests/inspector-enabled/console-exception-while-no-inspector-expected.txt: Removed.
  • http/tests/inspector-enabled/console-exception-while-no-inspector.html: Removed.

We now save exception backtraces even if the inspector is not open.

  • fast/dom/javascript-url-exception-isolation-expected.txt
  • fast/events/remove-target-with-shadow-in-drag-expected.txt
  • fast/events/set-attribute-listener-window-onerror-crash-expected.txt
  • inspector-protocol/media-query-listener-exception-expected.txt:
  • inspector/console/console-uncaught-exception-expected.txt:
  • inspector/console/console-uncaught-exception-in-eval-expected.txt:

Update results that now include backtraces or different data. Most better, some worse.

12:02 PM WebKitIDL edited by Christophe Dumez
[Callback] only applies to parameters, not interfaces (diff)
11:59 AM Changeset in webkit [149130] by oliver@apple.com
  • 4 edits in trunk/Source/JavaScriptCore

Make checkSyntax take a VM instead of an ExecState

RS=Tim

11:57 AM WebKitIDL edited by Christophe Dumez
Update [Callback] extended attribute doc due to r149113 (diff)
11:50 AM Changeset in webkit [149129] by Gregg Tavares
  • 5 edits in trunk/Source/WebCore

Fix build files so WebGLDebugXXX extensions can be used if enabled.
https://bugs.webkit.org/show_bug.cgi?id=113976

Reviewed by Dean Jackson.

No new tests as no new functionality.

  • DerivedSources.make:
  • GNUmakefile.list.am:
  • WebCore.xcodeproj/project.pbxproj:
  • bindings/js/JSWebGLRenderingContextCustom.cpp:
11:35 AM Changeset in webkit [149128] by msaboff@apple.com
  • 2 edits in trunk/Source/JavaScriptCore

32 Bit: Crash due to RegExpTest nodes not setting result type to Boolean
https://bugs.webkit.org/show_bug.cgi?id=115188

Reviewed by Geoffrey Garen.

Changed the RegExpTest node to set the AbstractValue to boolean, since that
what it is.

  • dfg/DFGAbstractState.cpp:

(JSC::DFG::AbstractState::executeEffects):

11:33 AM Changeset in webkit [149127] by rniwa@webkit.org
  • 2 edits in trunk/Source/WebCore

cloneChildNodes looks for deleteButtonController in each level of recursion
https://bugs.webkit.org/show_bug.cgi?id=115146

Reviewed by Andreas Kling.

Obtain the delete button controller upfront, and shallow copy descendents of each child
so that we don't look for the delete button controller inside cloneNode called on each child.

Performance Tests: DOM/CloneNodes.html

  • dom/ContainerNode.cpp:

(WebCore::cloneChildNodesAvoidingDeleteButon): Extracted.
(WebCore::ContainerNode::cloneChildNodes):

11:32 AM Changeset in webkit [149126] by rniwa@webkit.org
  • 8 edits in trunk

HTMLOptionsCollection's namedItem and name getter should return the first item
https://bugs.webkit.org/show_bug.cgi?id=115150

Reviewed by Andreas Kling.

Source/WebCore:

Following the resolution in http://lists.whatwg.org/pipermail/whatwg-whatwg.org/2012-December/038355.html,
the spefication has been updated to only return the first item when there are multiple items of the same name
in HTMLOptionsCollection; this new behavior matches that of Firefox and Opera (Presto).

Implement this new behavior to remove the custom binding code and use the fast path in namedItem and name
getter of HTMLOptionsCollection. (Obtaining all items for a given name is expensive!).

Tests: fast/dom/HTMLSelectElement/named-options.html

fast/dom/html-collections-named-getter.html

  • bindings/js/JSHTMLOptionsCollectionCustom.cpp:

(WebCore): Removed the custom bindings for name getter and namedItem.

  • html/HTMLOptionsCollection.idl:

LayoutTests:

Changed the expectations of the tests.

  • fast/dom/HTMLSelectElement/named-options-expected.txt:
  • fast/dom/HTMLSelectElement/script-tests/named-options.js:
  • fast/dom/html-collections-named-getter-expected.txt:
  • fast/dom/html-collections-named-getter.html:
11:31 AM Changeset in webkit [149125] by Joseph Pecoraro
  • 59 edits
    1 move
    3 adds
    1 delete in trunk

Web Inspector: ConsoleMessage should include line and column number where possible
https://bugs.webkit.org/show_bug.cgi?id=114929

Source/WebCore:

  • adds "m_column" to WebCore::ConsoleMessage
  • adds "column" to Console.ConsoleMessage in the inspector protocol
  • set the column number for console.* functions (Console.cpp)
  • set the column number for XSLT errors (XSLTProcessor)
  • plumb columnNumber everywhere else it is needed, set it to 0 and file bugs for all cases missing columnNumber that could provide it.

Reviewed by Timothy Hatcher.

Test: inspector/console/console-url-line-column.html

inspector/console/console-messages-stack-traces.html

  • inspector/ConsoleMessage.h:
  • inspector/ConsoleMessage.cpp:

(WebCore::ConsoleMessage::ConsoleMessage):
(WebCore::ConsoleMessage::autogenerateMetadata):
(WebCore::ConsoleMessage::addToFrontend):
(WebCore::ConsoleMessage::isEqual):
Add m_column and set it where appropriate.

  • inspector/Inspector.json:

Add column property to Console.ConsoleMessage.

  • page/Console.cpp:

(WebCore::internalAddMessage):
(WebCore::Console::profile):
Set columnNumber like lineNumber from the last stack frame.
(WebCore::Console::groupEnd):
Line and column are unused in this message type, set both to 0.

  • xml/XSLTProcessorLibxslt.cpp:

(WebCore::XSLTProcessor::parseErrorFunc):

  • xml/XSLTProcessorQt.cpp:

(WebCore::XSLTMessageHandler::handleMessage):
Add real column numbers, the XSLT handlers already had it available.

  • bindings/js/JSCustomXPathNSResolver.cpp:

(WebCore::JSCustomXPathNSResolver::lookupNamespaceURI):

  • css/CSSParser.cpp:

(WebCore::CSSParser::logError):

  • dom/ScriptExecutionContext.h:
  • dom/ScriptExecutionContext.cpp:

(WebCore::ScriptExecutionContext::addConsoleMessage):

  • dom/Document.cpp:

(WebCore::Document::logExceptionToConsole):
(WebCore::Document::addMessage):

  • dom/Document.h:
  • inspector/InspectorConsoleAgent.cpp:

(WebCore::InspectorConsoleAgent::addMessageToConsole):
(WebCore::InspectorConsoleAgent::stopTiming):
(WebCore::InspectorConsoleAgent::didFinishXHRLoading):
(WebCore::InspectorConsoleAgent::didReceiveResponse):
(WebCore::InspectorConsoleAgent::didFailLoading):

  • inspector/InspectorConsoleAgent.h:
  • inspector/InspectorConsoleInstrumentation.h:

(WebCore::InspectorInstrumentation::addMessageToConsole):
(WebCore::InspectorInstrumentation::addStartProfilingMessageToConsole):

  • inspector/InspectorInstrumentation.cpp:

(WebCore::InspectorInstrumentation::addMessageToConsoleImpl):
(WebCore::InspectorInstrumentation::addStartProfilingMessageToConsoleImpl):
(WebCore::InspectorInstrumentation::addProfileImpl):

  • inspector/InspectorInstrumentation.h:

(InspectorInstrumentation):

  • inspector/InspectorProfilerAgent.cpp:

(WebCore::InspectorProfilerAgent::addProfile):
(WebCore::InspectorProfilerAgent::addProfileFinishedMessageToConsole):
(WebCore::InspectorProfilerAgent::addStartProfilingMessageToConsole):
(WebCore::InspectorProfilerAgent::start):
(WebCore::InspectorProfilerAgent::stop):

  • inspector/InspectorProfilerAgent.h:

(InspectorProfilerAgent):

  • loader/EmptyClients.h:

(WebCore::EmptyChromeClient::addMessageToConsole):

  • page/ChromeClient.h:

(WebCore::ChromeClient::addMessageToConsole):

  • page/ContentSecurityPolicy.cpp:

(WebCore::ContentSecurityPolicy::logToConsole):

  • page/PageConsole.cpp:

(WebCore::PageConsole::addMessage):

  • page/PageConsole.h:
  • workers/DefaultSharedWorkerRepository.cpp:

(SharedWorkerProxy):
(WebCore::postExceptionTask):
(WebCore::SharedWorkerProxy::postExceptionToWorkerObject):
(WebCore::postConsoleMessageTask):
(WebCore::SharedWorkerProxy::postConsoleMessageToWorkerObject):

  • workers/SharedWorkerContext.cpp:

(WebCore::SharedWorkerContext::logExceptionToConsole):

  • workers/WorkerContext.cpp:

(WebCore::WorkerContext::logExceptionToConsole):
(WebCore::WorkerContext::addConsoleMessage):
(WebCore::WorkerContext::addMessage):
(WebCore::WorkerContext::addMessageToWorkerConsole):

  • workers/WorkerContext.h:
  • workers/WorkerMessagingProxy.cpp:

(WebCore::WorkerExceptionTask::create):
(WebCore::WorkerExceptionTask::WorkerExceptionTask):
(WebCore::WorkerExceptionTask::performTask):
(WorkerExceptionTask):
(WebCore::WorkerMessagingProxy::postExceptionToWorkerObject):
(WebCore::postConsoleMessageTask):
(WebCore::WorkerMessagingProxy::postConsoleMessageToWorkerObject):

  • workers/WorkerMessagingProxy.h:

(WorkerMessagingProxy):

  • workers/WorkerReportingProxy.h:

(WorkerReportingProxy):
Plumb columnNumber through where appropriate. File bugs where missing.

Source/WebKit/blackberry:

Reviewed by Timothy Hatcher.

  • Api/DumpRenderTreeClient.h:
  • Api/WebPageClient.h:
  • WebCoreSupport/ChromeClientBlackBerry.cpp:

(WebCore::ChromeClientBlackBerry::addMessageToConsole):

  • WebCoreSupport/ChromeClientBlackBerry.h:

(ChromeClientBlackBerry):

Source/WebKit/efl:

Reviewed by Timothy Hatcher.

  • WebCoreSupport/ChromeClientEfl.cpp:

(WebCore::ChromeClientEfl::addMessageToConsole):

  • WebCoreSupport/ChromeClientEfl.h:

(ChromeClientEfl):

Source/WebKit/gtk:

Reviewed by Timothy Hatcher.

  • WebCoreSupport/ChromeClientGtk.cpp:

(WebKit::ChromeClient::addMessageToConsole):

  • WebCoreSupport/ChromeClientGtk.h:

(ChromeClient):

Source/WebKit/mac:

Reviewed by Timothy Hatcher.

  • WebCoreSupport/WebChromeClient.h:
  • WebCoreSupport/WebChromeClient.mm:

(WebChromeClient::addMessageToConsole):

Source/WebKit/qt:

Reviewed by Timothy Hatcher.

  • WebCoreSupport/ChromeClientQt.cpp:

(WebCore::ChromeClientQt::addMessageToConsole):

  • WebCoreSupport/ChromeClientQt.h:

Source/WebKit/win:

Reviewed by Timothy Hatcher.

  • WebCoreSupport/WebChromeClient.cpp:

(WebChromeClient::addMessageToConsole):

  • WebCoreSupport/WebChromeClient.h:

Source/WebKit/wince:

Reviewed by Timothy Hatcher.

  • WebCoreSupport/ChromeClientWinCE.cpp:

(WebKit::ChromeClientWinCE::addMessageToConsole):

  • WebCoreSupport/ChromeClientWinCE.h:

Source/WebKit2:

Reviewed by Timothy Hatcher.

  • WebProcess/WebCoreSupport/WebChromeClient.cpp:

(WebKit::WebChromeClient::addMessageToConsole):

  • WebProcess/WebCoreSupport/WebChromeClient.h:

LayoutTests:

Update a test that was outputting the url and line number of console
method calls, to output the column number as well. Added a test that
verifies that console API messages have stack traces if appropriate.

Reviewed by Timothy Hatcher.

  • inspector/console/console-url-and-line-expected.txt: Removed.
  • inspector/console/console-url-line-column-expected.txt: Added.
  • inspector/console/console-url-line-column.html: Renamed from LayoutTests/inspector/console/console-url-and-line.html.
  • inspector/console/console-messages-stack-traces-expected.txt: Added.
  • inspector/console/console-messages-stack-traces.html: Added.
11:29 AM Changeset in webkit [149124] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebKit/blackberry

[BlackBerry] Get rid of return in void method
https://bugs.webkit.org/show_bug.cgi?id=115186

Patch by Konrad Piascik <kpiascik@blackberry.com> on 2013-04-25
Reviewed by Rob Buis.

  • Api/WebPage.cpp:

(BlackBerry::WebKit::WebPage::setForcedTextEncoding):

11:19 AM Changeset in webkit [149123] by allan.jensen@digia.com
  • 5 edits in trunk/Source

[Qt] Animated opacity does not trigger accelerated compositing
https://bugs.webkit.org/show_bug.cgi?id=115107

Reviewed by Simon Fraser.

Source/WebCore:

Add extra option to also trigger compositing on animated opacity.

  • page/ChromeClient.h:
  • rendering/RenderLayerCompositor.cpp:

(WebCore::RenderLayerCompositor::requiresCompositingForAnimation):

Source/WebKit/qt:

We should trigger compositing on animated opacity as it is always faster on Qt.

  • WebCoreSupport/ChromeClientQt.cpp:

VideoTrigger removed because we have already disabled AC for video in qwebsettings.

11:09 AM Changeset in webkit [149122] by Alexandru Chiculita
  • 1 edit in trunk/Tools/Scripts/webkitpy/common/config/contributors.json

Unreviewed. Adding myself to the reviwers list.

10:59 AM Changeset in webkit [149121] by beidson@apple.com
  • 3 edits in trunk/Source/WebKit2

Thread safety issues in WKCustomProtocol.
<rdar://problem/13247304> and https://bugs.webkit.org/show_bug.cgi?id=115185

Reviewed by Alexey Proskuryakov.

  • Shared/Network/CustomProtocols/CustomProtocolManager.h:
  • Shared/Network/CustomProtocols/mac/CustomProtocolManagerMac.mm:

(WebKit::CustomProtocolManager::addCustomProtocol): Protect m_customProtocolMap with its mutex.
(WebKit::CustomProtocolManager::removeCustomProtocol): Ditto.
(WebKit::CustomProtocolManager::didFailWithError):
(WebKit::CustomProtocolManager::didLoadData):
(WebKit::CustomProtocolManager::didReceiveResponse):
(WebKit::CustomProtocolManager::didFinishLoading):
(WebKit::CustomProtocolManager::protocolForID): Protect m_customProtocolMap with its mutex, and return

a RetainPtr instead of a raw pointer.

10:48 AM Changeset in webkit [149120] by commit-queue@webkit.org
  • 4 edits
    3 adds in trunk

Sec-WebSocket-Extensions header field must not appear more than once in an HTTP response.
https://bugs.webkit.org/show_bug.cgi?id=115128

Patch by Lamarque V. Souza <Lamarque.Souza@basyskom.com> on 2013-04-25
Reviewed by Alexey Proskuryakov.

Source/WebCore:

According to WebSocket specification Sec-WebSocket-Extensions header field
must not appear more than once in an HTTP response. It is ok if it appears
more than once in client request. Also add a check for invalid UTF-8
characters when parsing Sec-WebSocket-Extensions quoted string.

Test: http/tests/websocket/tests/hybi/handshake-fail-by-more-extensions-header.html

  • Modules/websockets/WebSocketExtensionParser.cpp:

(WebCore::WebSocketExtensionParser::consumeQuotedString): Return false if there is
invalid character in the quoted string being parsed.

  • Modules/websockets/WebSocketHandshake.cpp:

(WebCore::WebSocketHandshake::readHTTPHeaders): Check if Sec-WebSocket-Extensions
appears more than once in header response. Abort handshake if it does.

LayoutTests:

  • http/tests/websocket/tests/hybi/handshake-fail-by-more-extensions-header-expected.txt: Added.
  • http/tests/websocket/tests/hybi/handshake-fail-by-more-extensions-header.html: Added.
  • http/tests/websocket/tests/hybi/handshake-fail-by-more-extensions-header_wsh.py: Added.
10:31 AM Changeset in webkit [149119] by jer.noble@apple.com
  • 3 edits in trunk/Source/WebKit2

Further build fixes: correct two more misnames of WebProcessShim.dyld.

  • WebProcess/EntryPoint/mac/LegacyProcess/WebContentProcessMain.mm:

(WebKit::WebContentProcessMainDelegate::doPreInitializationWork):

  • WebProcess/EntryPoint/mac/XPCService/WebContentService/Info.plist:
10:12 AM Changeset in webkit [149118] by Martin Robinson
  • 2 edits in trunk/Source/WebKit2

Make gobject-introspection warnings non-fatal
https://bugs.webkit.org/show_bug.cgi?id=115184

Reviewed by Carlos Garcia Campos.

  • GNUmakefile.am: Make it so that gobject-introspection errors do not fail the buld

until we get to a point where gobject-introspection gives consistent warnings.

9:46 AM Changeset in webkit [149117] by Carlos Garcia Campos
  • 14 edits
    4 adds in trunk/Source/WebKit2

[GTK] Add WebKitWebViewGroup to WebKit2 GTK+ API
https://bugs.webkit.org/show_bug.cgi?id=111265

Reviewed by Anders Carlsson.

In the current API the settings are always shared by all web
views, because they are always created on the default page
group. This makes impossible to have different settings on
different web views. Expose the page group in the API,
so that users can create web views on a group different than the
default one to have their own settings.

  • GNUmakefile.list.am: Add new files to compilation.
  • UIProcess/API/gtk/WebKitSettings.cpp:

(webkitSettingsGetPreferences): Helper private method to get the
WebPreferences wrapped by a WebKitSettings object.

  • UIProcess/API/gtk/WebKitSettingsPrivate.h:
  • UIProcess/API/gtk/WebKitWebContext.cpp:

(_WebKitWebContextPrivate): Add a default WebKitWebViewGroup.
(webkitWebContextCreatePageForWebView): Create the WebPageProxy
with the WebPageGroup of the given WebKitWebViewGroup.
(webkitWebContextGetDefaultWebViewGroup): Helper private method to
get the default WebKitWebViewGroup.

  • UIProcess/API/gtk/WebKitWebContextPrivate.h:
  • UIProcess/API/gtk/WebKitWebView.cpp:

(_WebKitWebViewPrivate): Add a WebKitWebViewGroup in case the view
is created with a group.
(webkitWebViewUpdateSettings): Keep a reference of the group
settings to be able to disconnect the signals when the settings
object is replaced in the view group and connect to the signals
for the custom settings.
(webkitWebViewSettingsChanged): Disconnect the signals for the
previous settings and set the new settings.
(webkitWebViewDisconnectSettingsChangedSignalHandler): Disconnect
notify::settings signal of WebKitWebViewGroup.
(webkitWebViewConstructed): Pass the current WebKitWebViewGroup to
webkitWebContextCreatePageForWebView().
(webkitWebViewSetProperty): Add setter for group property.
(webkitWebViewGetProperty): Add getter for group property.
(webkitWebViewDispose): Call
webkitWebViewDisconnectSettingsChangedSignalHandler().
(webkit_web_view_class_init): Add WebKitWebView:group property.
(webkitWebViewHandleAuthenticationChallenge): Use
webkit_web_view_get_settings().
(webkit_web_view_new_with_group): Create a new web view with the
given WebKitWebViewGroup.
(webkit_web_view_get_group): Return the current WebKitWebViewGroup
or the default one if the view was not created with
webkit_web_view_new_with_group().
(webkit_web_view_set_settings): Set the settings of the current
WebKitWebViewGroup.
(webkit_web_view_get_settings): Get the settings of the current
WebKitWebViewGroup.
(webkit_web_view_set_zoom_level): Use
webkit_web_view_get_settings().
(webkit_web_view_get_zoom_level): Ditto.

  • UIProcess/API/gtk/WebKitWebView.h:
  • UIProcess/API/gtk/WebKitWebViewGroup.cpp: Added.

(_WebKitWebViewGroupPrivate):
(webkitWebViewGroupSetProperty):
(webkitWebViewGroupGetProperty):
(webkitWebViewGroupConstructed):
(webkit_web_view_group_class_init):
(webkitWebViewGroupAttachSettingsToPageGroup): Set the
WebPreferences of the current WebKitSettings to the WebPageGroup.
(webkitWebViewGroupCreate): Create a new WebKitWebViewGroup for
the given WebPageGroup.
(webkitWebViewGroupGetPageGroup): Helper private method to get the
WebPageGroup wrapped by the WebKitWebViewGroup.
(webkit_web_view_group_new): Create a new WebKitWebViewGroup.
(webkit_web_view_group_get_name): Return the name of a
WebKitWebViewGroup.
(webkit_web_view_group_get_settings): Get the settings of a
WebKitWebViewGroup.
(webkit_web_view_group_set_settings): Set new settings of a
WebKitWebViewGroup.

  • UIProcess/API/gtk/WebKitWebViewGroup.h: Added.

(_WebKitWebViewGroup):
(_WebKitWebViewGroupClass):

  • UIProcess/API/gtk/WebKitWebViewGroupPrivate.h: Added.
  • UIProcess/API/gtk/docs/webkit2gtk-docs.sgml: Add

WebKitWebViewGroup section.

  • UIProcess/API/gtk/docs/webkit2gtk-sections.txt: Add new symbols.
  • UIProcess/API/gtk/docs/webkit2gtk.types: Add

webkit_web_view_group_get_type.

  • UIProcess/API/gtk/tests/GNUmakefile.am: Add new files to

compilation.

  • UIProcess/API/gtk/tests/TestWebKitWebView.cpp:

(testWebViewSettings): We can't watch settings in the default view
group anymore, because they are not released with the web view but
with the web context that is freed after the Test destructor.

  • UIProcess/API/gtk/tests/TestWebKitWebViewGroup.cpp: Added.

(testWebViewGroupDefault):
(testWebViewGroupNewGroup):
(testWebViewNewWithGroup):
(testWebViewGroupSettings):
(beforeAll):
(afterAll):

  • UIProcess/API/gtk/webkit2.h: Include WebKitWebViewGroup.h.
9:39 AM Changeset in webkit [149116] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WTF

[Windows] Compile fix.
https://bugs.webkit.org/show_bug.cgi?id=115170

Patch by peavo@outlook.com <peavo@outlook.com> on 2013-04-25
Reviewed by Filip Pizlo.

Forward declare function, instead of including intrin.h.

  • wtf/Atomics.h:
9:32 AM Changeset in webkit [149115] by rakuco@webkit.org
  • 2 edits in trunk/Source/WebCore

REGRESSION(r148758): Remove WTFLogAlways call from IconLoader.
https://bugs.webkit.org/show_bug.cgi?id=115182

Reviewed by Dan Bernstein.

The call added in r148758 was apparently a leftover from some debugging
session, and it makes the Interactive/window-resize.html performance
test randomly fail on Qt and EFL due to WTFLogAlways writing to stderr.

  • loader/icon/IconLoader.cpp:

(WebCore::IconLoader::notifyFinished):

9:18 AM Changeset in webkit [149114] by commit-queue@webkit.org
  • 2 edits in trunk/Source/JavaScriptCore

REGRESSION(r137994): Random crashes occur with SH4 JSC.
https://bugs.webkit.org/show_bug.cgi?id=115167.

Patch by Julien Brianceau <jbrianceau@nds.com> on 2013-04-25
Reviewed by Oliver Hunt.

Since r137994, uncommited pages could be inside the area of memory in
parameter of the cacheFlush function. That's why we have to flush each
page separately to avoid a fail of the whole flush, if an uncommited page
is in the area.

This patch is very similar to changeset 145194 made for ARMv7 architecture,
see https://bugs.webkit.org/show_bug.cgi?id=111441 for further information.

  • assembler/SH4Assembler.h:

(JSC::SH4Assembler::cacheFlush):

8:55 AM Changeset in webkit [149113] by Christophe Dumez
  • 32 edits in trunk/Source/WebCore

Add support for Web IDL callback interfaces to the bindings generator
https://bugs.webkit.org/show_bug.cgi?id=115179

Reviewed by Kentaro Hara.

Add support in the bindings generator for Web IDL callback interfaces:
http://dev.w3.org/2006/webapi/WebIDL/#dfn-callback-interface

Drop support for the WebKit-specific [Callback] extended attributes
on Web IDL interfaces and update existing IDL files to use proper
callback interfaces instead.

No new tests, no behavior change.

  • Modules/filesystem/EntriesCallback.idl:
  • Modules/filesystem/EntryCallback.idl:
  • Modules/filesystem/ErrorCallback.idl:
  • Modules/filesystem/FileCallback.idl:
  • Modules/filesystem/FileSystemCallback.idl:
  • Modules/filesystem/FileWriterCallback.idl:
  • Modules/filesystem/MetadataCallback.idl:
  • Modules/geolocation/PositionCallback.idl:
  • Modules/geolocation/PositionErrorCallback.idl:
  • Modules/mediastream/NavigatorUserMediaErrorCallback.idl:
  • Modules/mediastream/NavigatorUserMediaSuccessCallback.idl:
  • Modules/mediastream/RTCErrorCallback.idl:
  • Modules/mediastream/RTCSessionDescriptionCallback.idl:
  • Modules/mediastream/RTCStatsCallback.idl:
  • Modules/notifications/NotificationPermissionCallback.idl:
  • Modules/quota/StorageErrorCallback.idl:
  • Modules/quota/StorageQuotaCallback.idl:
  • Modules/quota/StorageUsageCallback.idl:
  • Modules/webaudio/AudioBufferCallback.idl:
  • Modules/webdatabase/DatabaseCallback.idl:
  • Modules/webdatabase/SQLStatementCallback.idl:
  • Modules/webdatabase/SQLStatementErrorCallback.idl:
  • Modules/webdatabase/SQLTransactionCallback.idl:
  • Modules/webdatabase/SQLTransactionErrorCallback.idl:
  • Modules/webdatabase/SQLTransactionSyncCallback.idl:
  • bindings/scripts/CodeGeneratorJS.pm:

(GenerateInterface):

  • bindings/scripts/IDLParser.pm:

(parseCallbackRestOrInterface):

  • bindings/scripts/test/TestCallback.idl:
  • dom/RequestAnimationFrameCallback.idl:
  • dom/StringCallback.idl:
  • html/VoidCallback.idl:
8:17 AM Changeset in webkit [149112] by mikhail.pozdnyakov@intel.com
  • 2 edits in trunk/Source/WTF

Use std::move in AtomicString
https://bugs.webkit.org/show_bug.cgi?id=115102

Reviewed by Anders Carlsson.

Use std::move in AtomicString instead of direct casting to rvalue references.

  • wtf/text/AtomicString.h:

(AtomicString):
(WTF::AtomicString::AtomicString):
(WTF::AtomicString::operator=):

7:52 AM Changeset in webkit [149111] by commit-queue@webkit.org
  • 8 edits
    1 copy in trunk

[GStreamer] Add audio/speex MIME type as supported
https://bugs.webkit.org/show_bug.cgi?id=115032

Patch by Adrian Perez de Castro <Adrian Perez de Castro> on 2013-04-25
Reviewed by Philippe Normand.

Source/WebCore:

GStreamer has support for the Speex codec (http://www.speex.org).
Speex streams may be embedded in a Ogg container, or standalone.
The case of the Ogg container is already covered by the "*/ogg"
MIME types declared as supported by the GStreamer media playing
code, but for standalone streams to work, "audio/speex" has to
be added.

With this, and the needed GStreamer plugin installed, the GTK+
launcher is able to correctly play Speex streams served with
"Content-type: audio/speex".

  • platform/graphics/gstreamer/MediaPlayerPrivateGStreamer.cpp:

LayoutTests:

Speex (http://www.speex.org) streams can be embedded into Ogg
Ogg containers, so add a check for it in the test case.

  • media/media-can-play-ogg-expected.txt:
  • media/media-can-play-ogg.html:
  • platform/blackberry/media/media-can-play-ogg-expected.txt:
  • platform/mac/media/media-can-play-ogg-expected.txt:
  • platform/qt-linux/media/media-can-play-ogg-expected.txt:
  • platform/win/media/media-can-play-ogg-expected.txt:
7:33 AM Changeset in webkit [149110] by akling@apple.com
  • 9 edits in trunk/Source/WebCore

Add FINAL decorators to the InlineBox class hierarchy.
<http://webkit.org/b/115177>

Reviewed by Antti Koivisto.

From Blink r148628 by <cevans@chromium.org>:

FINAL is a macro in wtf/Compiler.h that does the correct thing if the compiler does not support "final")
The approach used is as simple as possible whilst being thorough.
So, leaf classes have FINAL applied to the whole class whereas intermediary classes have FINAL applied to relevant methods.

FINAL allows a compiler to devirtualize call sites and turn them into direct calls. As you might expect, this is perf positive:
(clang on Linux):

  • line_layout.html goes from 120 runs/s -> 123 runs/2, +2.5%
  • html5-full-render.html goes from 3176ms -> 3162ms, +0.4%

I have confidence that the former result is statistically significant (as the numbers are very very stable) but not the latter.

  • rendering/EllipsisBox.h:
  • rendering/InlineFlowBox.h:
  • rendering/InlineTextBox.h:
  • rendering/RootInlineBox.h:
  • rendering/TrailingFloatsRootInlineBox.h:
  • rendering/svg/SVGInlineFlowBox.h:
  • rendering/svg/SVGInlineTextBox.h:
  • rendering/svg/SVGRootInlineBox.h:
7:19 AM Changeset in webkit [149109] by commit-queue@webkit.org
  • 2 edits in trunk/Tools

[GTK] Tools/gtk/install-dependencies added ragel dependency
https://bugs.webkit.org/show_bug.cgi?id=115163

Patch by Xavier Castano <xcastanho@igalia.com> on 2013-04-25
Reviewed by Xan Lopez.

The install-dependencies script was missing a dependency on ragel
which is expected when building HarfBuzz.

  • gtk/install-dependencies: Added a dependency on ragel for both

apt and yum.

7:05 AM Changeset in webkit [149108] by commit-queue@webkit.org
  • 2 edits in trunk/Tools

[GTK] Remove pango from optional jhbuild dependencies
https://bugs.webkit.org/show_bug.cgi?id=115162

Patch by Eduardo Lima Mitev <elima@igalia.com> on 2013-04-25
Reviewed by Martin Robinson.

Pango is not needed since revision r148293 that lowers the pango
dependency to 1.30, which is available in most distributions.

  • gtk/jhbuild-optional.modules: Removes description of pango module.
6:46 AM Changeset in webkit [149107] by jer.noble@apple.com
  • 5 edits in trunk/Source/WebKit2

Unreviewed build fix: Fix the name of DyldInterpose.h in
SecItemShimLibrary.mm, fix the name of WebProcessShim.dyld in
WebContentServiceEntryPoint.mm, and stop linking against libWTF.a
directly (which means removing ASSERTs from
CookieStorageShimLibrary.cpp).

  • Shared/mac/CookieStorageShimLibrary.cpp:

(WebKit::shimCFHTTPCookieStorageCopyRequestHeaderFieldsForURL):
(WebKit::WebKitCookieStorageShimInitialize):

  • WebKit2.xcodeproj/project.pbxproj:
  • WebProcess/EntryPoint/mac/XPCService/WebContentServiceEntryPoint.mm:

(WebContentServiceInitializer):

  • WebProcess/mac/SecItemShimLibrary.mm:
6:43 AM Changeset in webkit [149106] by akling@apple.com
  • 4 edits
    4 adds in trunk

CSS parser: Add error recovery while parsing @-webkit-keyframes key values.
<http://webkit.org/b/115175>

Source/WebCore:

From Blink r148714 by <apavlov@chromium.org>:

If not a percentage, "from", or "to" value is used in a key list, the rule is erroneous,
and due to the absense of recovery, the parser skips the following, valid CSS rule.

On a related note, keyframes, whose selectors contain invalid keys, should be discarded
altogether, according to <http://www.w3.org/TR/css3-animations/#keyframes>

Tests: animations/keyframes-invalid-keys.html

fast/css/webkit-keyframes-errors.html

  • css/CSSGrammar.y.in:
  • css/CSSParser.cpp:

(WebCore::CSSParser::rewriteSpecifiers):

LayoutTests:

From Blink r148714 by <apavlov@chromium.org>.

  • animations/keyframes-invalid-keys-expected.txt: Added.
  • animations/keyframes-invalid-keys.html: Added.
  • fast/css/webkit-keyframes-errors-expected.html: Added.
  • fast/css/webkit-keyframes-errors.html: Added.
6:23 AM Changeset in webkit [149105] by Antti Koivisto
  • 7 edits in trunk/Source/WebCore

REGRESSION (r147797): Animations slideshows of images on www.thesuperficial.com are slow
https://bugs.webkit.org/show_bug.cgi?id=115172

Reviewed by Andreas Kling.

On this page ads dynamically loaded to subframes on slideshow navigation switch us to state where we throttle layer flushes.

Fix by ignoring any subframe-originated loads when determining throttling.

  • loader/FrameLoader.cpp:

(WebCore::FrameLoader::loadProgressingStatusChanged):

  • loader/ProgressTracker.cpp:

(WebCore::ProgressTracker::isMainLoadProgressing):

Rename, ignore subframe originated loads. Subframe loads that are initiated during the main load still count.

  • loader/ProgressTracker.h:

(ProgressTracker):

  • page/FrameView.cpp:

(WebCore::FrameView::updateLayerFlushThrottlingInAllFrames):

  • page/FrameView.h:

(FrameView):

  • rendering/RenderLayerBacking.cpp:

(WebCore::computeTileCoverage):

6:10 AM Changeset in webkit [149104] by kadam@inf.u-szeged.hu
  • 2 edits in trunk/LayoutTests

[Qt] Unreviewed gardening. Skip some failing tests after r148996.

  • platform/qt/TestExpectations:
6:04 AM Changeset in webkit [149103] by commit-queue@webkit.org
  • 2 edits in trunk/Tools

Remove Chromium-specific cases from FeatureList.pm
https://bugs.webkit.org/show_bug.cgi?id=115169

Patch by Seokju Kwon <Seokju Kwon> on 2013-04-25
Reviewed by Andreas Kling.

  • Scripts/webkitperl/FeatureList.pm:
5:56 AM Changeset in webkit [149102] by akling@apple.com
  • 4 edits
    2 adds in trunk

SVG: Fix viewBox animations on shapes with non-scaling-stroke.
<http://webkit.org/b/115173>

Source/WebCore:

From Blink r149058 by <pdr@chromium.org>:

Previously hasAttribute() to determine if a viewBox was set in the transform
code for non-scaling-strokes. hasAttribute() should not be used in this case,
as it will return false if the attribute is not set but the value is animating.

This patch switches to checking if the viewBox is empty instead of checking for
the presence of the attribute.

Test: svg/stroke/animated-non-scaling-stroke.html

  • svg/SVGSVGElement.cpp:

(WebCore::SVGSVGElement::localCoordinateSpaceTransform):

  • svg/SVGSVGElement.h:

(WebCore::SVGSVGElement::hasEmptyViewBox):

LayoutTests:

From Blink r149058 by <pdr@chromium.org>.

  • svg/stroke/animated-non-scaling-stroke-expected.html: Added.
  • svg/stroke/animated-non-scaling-stroke.html: Added.
5:32 AM Changeset in webkit [149101] by akling@apple.com
  • 2 edits in trunk/Source/WebCore

Remove isPluginElement hack in Document::setFocusedNode()
<http://webkit.org/b/115171>

From Blink r148800 by <tkent@chromium.org>:

This hack is no longer necessary because r147591 made isFocusable for plugin
elements work correctly.

  • dom/Document.cpp:

(WebCore::Document::setFocusedNode):

5:13 AM Changeset in webkit [149100] by kadam@inf.u-szeged.hu
  • 4 edits in trunk/LayoutTests

[Qt] Unreviewed gardening. Unskip now passing tests.

  • platform/qt-5.0-wk1/TestExpectations:
  • platform/qt-5.0-wk2/TestExpectations:
  • platform/qt/TestExpectations:
3:49 AM Changeset in webkit [149099] by kadam@inf.u-szeged.hu
  • 23 edits
    22 copies
    8 adds in trunk/LayoutTests

[Qt] Unreviewed gardening. Organize expected files after r149095.

  • platform/qt-5.0-wk1/editing/deleting/delete-by-word-002-expected.txt: Copied from LayoutTests/platform/qt/editing/deleting/delete-by-word-002-expected.txt.
  • platform/qt-5.0-wk1/editing/deleting/delete-cell-contents-expected.txt: Copied from LayoutTests/platform/qt/editing/deleting/delete-cell-contents-expected.txt.
  • platform/qt-5.0-wk1/editing/execCommand/button-expected.txt: Copied from LayoutTests/platform/qt/editing/execCommand/button-expected.txt.
  • platform/qt-5.0-wk1/editing/execCommand/createLink-expected.txt: Copied from LayoutTests/platform/qt/editing/execCommand/createLink-expected.txt.
  • platform/qt-5.0-wk1/editing/execCommand/indent-pre-expected.txt: Copied from LayoutTests/platform/qt/editing/execCommand/indent-pre-expected.txt.
  • platform/qt-5.0-wk1/editing/execCommand/unlink-expected.txt: Copied from LayoutTests/platform/qt/editing/execCommand/unlink-expected.txt.
  • platform/qt-5.0-wk1/editing/input/emacs-ctrl-o-expected.txt: Copied from LayoutTests/platform/qt/editing/input/emacs-ctrl-o-expected.txt.
  • platform/qt-5.0-wk1/editing/inserting/insert-before-link-1-expected.txt: Copied from LayoutTests/platform/qt/editing/inserting/insert-before-link-1-expected.txt.
  • platform/qt-5.0-wk1/editing/pasteboard/display-block-on-spans-expected.txt: Copied from LayoutTests/platform/qt/editing/pasteboard/display-block-on-spans-expected.txt.
  • platform/qt-5.0-wk1/editing/pasteboard/emacs-cntl-y-001-expected.txt: Copied from LayoutTests/platform/qt/editing/pasteboard/emacs-cntl-y-001-expected.txt.
  • platform/qt-5.0-wk1/editing/pasteboard/merge-end-list-2-expected.txt: Copied from LayoutTests/platform/qt/editing/pasteboard/merge-end-list-2-expected.txt.
  • platform/qt-5.0-wk1/editing/pasteboard/merge-end-table-2-expected.txt: Copied from LayoutTests/platform/qt/editing/pasteboard/merge-end-table-2-expected.txt.
  • platform/qt-5.0-wk1/editing/pasteboard/page-zoom-expected.txt: Copied from LayoutTests/platform/qt/editing/pasteboard/page-zoom-expected.txt.
  • platform/qt-5.0-wk1/editing/selection/extend-selection-word-expected.txt: Copied from LayoutTests/platform/qt/editing/selection/extend-selection-word-expected.txt.
  • platform/qt-5.0-wk1/editing/style/apply-through-end-of-document-expected.txt: Copied from LayoutTests/platform/qt/editing/style/apply-through-end-of-document-expected.txt.
  • platform/qt-5.0-wk1/editing/style/remove-underline-across-paragraph-expected.txt: Copied from LayoutTests/platform/qt/editing/style/remove-underline-across-paragraph-expected.txt.
  • platform/qt-5.0-wk1/editing/style/remove-underline-across-paragraph-in-bold-expected.txt: Copied from LayoutTests/platform/qt/editing/style/remove-underline-across-paragraph-in-bold-expected.txt.
  • platform/qt-5.0-wk1/editing/style/remove-underline-expected.txt: Copied from LayoutTests/platform/qt/editing/style/remove-underline-expected.txt.
  • platform/qt-5.0-wk1/editing/style/remove-underline-from-stylesheet-expected.txt: Copied from LayoutTests/platform/qt/editing/style/remove-underline-from-stylesheet-expected.txt.
  • platform/qt-5.0-wk1/editing/style/remove-underline-in-bold-expected.txt: Copied from LayoutTests/platform/qt/editing/style/remove-underline-in-bold-expected.txt.
  • platform/qt-5.0-wk1/editing/style/unbold-in-bold-expected.txt: Copied from LayoutTests/platform/qt/editing/style/unbold-in-bold-expected.txt.
  • platform/qt-5.0-wk1/editing/undo/replace-text-in-node-preserving-markers-crash-expected.txt: Copied from LayoutTests/platform/qt/editing/undo/replace-text-in-node-preserving-markers-crash-expected.txt.
  • platform/qt/editing/deleting/delete-by-word-002-expected.txt:
  • platform/qt/editing/deleting/delete-cell-contents-expected.txt:
  • platform/qt/editing/execCommand/button-expected.txt:
  • platform/qt/editing/execCommand/createLink-expected.txt:
  • platform/qt/editing/execCommand/indent-pre-expected.txt:
  • platform/qt/editing/execCommand/unlink-expected.txt:
  • platform/qt/editing/input/emacs-ctrl-o-expected.txt:
  • platform/qt/editing/inserting/insert-before-link-1-expected.txt:
  • platform/qt/editing/pasteboard/display-block-on-spans-expected.txt:
  • platform/qt/editing/pasteboard/emacs-cntl-y-001-expected.txt:
  • platform/qt/editing/pasteboard/merge-end-list-2-expected.txt:
  • platform/qt/editing/pasteboard/merge-end-table-2-expected.txt:
  • platform/qt/editing/pasteboard/page-zoom-expected.txt:
  • platform/qt/editing/selection/extend-selection-word-expected.txt:
  • platform/qt/editing/style/apply-through-end-of-document-expected.txt:
  • platform/qt/editing/style/remove-underline-across-paragraph-expected.txt:
  • platform/qt/editing/style/remove-underline-across-paragraph-in-bold-expected.txt:
  • platform/qt/editing/style/remove-underline-expected.txt:
  • platform/qt/editing/style/remove-underline-from-stylesheet-expected.txt:
  • platform/qt/editing/style/remove-underline-in-bold-expected.txt:
  • platform/qt/editing/style/unbold-in-bold-expected.txt:
  • platform/qt/editing/undo/replace-text-in-node-preserving-markers-crash-expected.txt:
2:51 AM Changeset in webkit [149098] by commit-queue@webkit.org
  • 4 edits
    2 adds in trunk

EventSource: Synchronous loader callback not handled properly
https://bugs.webkit.org/show_bug.cgi?id=115104

Source/WebCore:

When attempting a cross-origin request towards a non-HTTP URL, an early verification
of the protocol scheme will cause didFailAccessControlCheck to be called synchronously
before the loader has even finished being created. This special case was not handled
properly, since we tried to cancel a non-existing loader, which resulted in a crash.
In addition to checking whether a request is in flight before trying to cancel it,
this change also schedules the initial creation of the loader to happen asynchronously
when an EventSource is constructed, so that a script can register for the error event
before it is dispatched (as a result of passing a non-HTTP URL to the constructor).

Patch by Per-Erik Brodin <per-erik.brodin@ericsson.com> on 2013-04-25
Reviewed by Alexey Proskuryakov.

Test: http/tests/eventsource/eventsource-cors-non-http.html

  • page/EventSource.cpp:

(WebCore::EventSource::EventSource):
(WebCore::EventSource::create):
(WebCore::EventSource::scheduleInitialConnect):
(WebCore):
(WebCore::EventSource::scheduleReconnect):
(WebCore::EventSource::connectTimerFired):
(WebCore::EventSource::close):
(WebCore::EventSource::abortConnectionAttempt):

  • page/EventSource.h:

(EventSource):

LayoutTests:

Patch by Per-Erik Brodin <per-erik.brodin@ericsson.com> on 2013-04-25
Reviewed by Alexey Proskuryakov.

  • http/tests/eventsource/eventsource-cors-non-http-expected.txt: Added.
  • http/tests/eventsource/eventsource-cors-non-http.html: Added.
2:43 AM Changeset in webkit [149097] by Patrick Gansterer
  • 2 edits in trunk/Source/WTF

Unreviewed WinCE build fix after r148888.

  • wtf/Atomics.h: WinCE version of intrin.h is called cmnintrin.h.
2:42 AM Changeset in webkit [149096] by Patrick Gansterer
  • 2 edits in trunk/Source/WebCore

Unreviewed WinCE build fix after r148545.

  • page/wince/FrameWinCE.cpp:
1:01 AM Changeset in webkit [149095] by kadam@inf.u-szeged.hu
  • 7 edits
    21 adds in trunk/LayoutTests

[Qt] Unreviewed gardening. Added platform specific expected files after r148987.

  • platform/qt/editing/deleting/delete-by-word-002-expected.txt: Added.
  • platform/qt/editing/deleting/delete-cell-contents-expected.txt: Added.
  • platform/qt/editing/execCommand/button-expected.txt: Added.
  • platform/qt/editing/execCommand/createLink-expected.txt: Added.
  • platform/qt/editing/execCommand/indent-pre-expected.txt: Added.
  • platform/qt/editing/execCommand/unlink-expected.txt: Added.
  • platform/qt/editing/input/emacs-ctrl-o-expected.png:
  • platform/qt/editing/input/emacs-ctrl-o-expected.txt:
  • platform/qt/editing/inserting/insert-before-link-1-expected.txt: Added.
  • platform/qt/editing/pasteboard/display-block-on-spans-expected.txt: Added.
  • platform/qt/editing/pasteboard/emacs-cntl-y-001-expected.png:
  • platform/qt/editing/pasteboard/emacs-cntl-y-001-expected.txt:
  • platform/qt/editing/pasteboard/merge-end-list-2-expected.txt: Added.
  • platform/qt/editing/pasteboard/merge-end-table-2-expected.txt: Added.
  • platform/qt/editing/pasteboard/page-zoom-expected.txt: Added.
  • platform/qt/editing/selection/extend-selection-word-expected.txt: Added.
  • platform/qt/editing/selection/selection-extend-should-not-move-across-caret-on-mac-expected.txt: Added.
  • platform/qt/editing/style/apply-through-end-of-document-expected.txt: Added.
  • platform/qt/editing/style/make-text-writing-direction-inline-win-expected.txt: Added.
  • platform/qt/editing/style/remove-underline-across-paragraph-expected.txt: Added.
  • platform/qt/editing/style/remove-underline-across-paragraph-in-bold-expected.txt: Added.
  • platform/qt/editing/style/remove-underline-expected.txt: Added.
  • platform/qt/editing/style/remove-underline-from-stylesheet-expected.txt: Added.
  • platform/qt/editing/style/remove-underline-in-bold-expected.txt: Added.
  • platform/qt/editing/style/unbold-in-bold-expected.png:
  • platform/qt/editing/style/unbold-in-bold-expected.txt:
  • platform/qt/editing/undo/replace-text-in-node-preserving-markers-crash-expected.txt: Added.
12:50 AM Changeset in webkit [149094] by rniwa@webkit.org
  • 2 edits in trunk/Source/WebKit/efl

Unreviewed, rolling out r149032.
http://trac.webkit.org/changeset/149032
https://bugs.webkit.org/show_bug.cgi?id=115151

EFL WK1 API test has been broken after r149032 (Requested by
gyuyoung on #webkit).

  • tests/test_ewk_setting.cpp:

(TEST_F):

Note: See TracTimeline for information about the timeline view.