Timeline
Jan 15, 2011:
- 9:11 PM Changeset in webkit [75891] by
-
- 2 edits in trunk/Source/JavaScriptCore
2011-01-15 Geoffrey Garen <ggaren@apple.com>
Reviewed by Oliver Hunt.
Rolled back in r75886.
https://bugs.webkit.org/show_bug.cgi?id=52527
r75886 broke the GTK Linux bot because Linux was -- quite surprisingly --
set up to use the constants for embedded devices.
- jit/ExecutableAllocatorFixedVMPool.cpp: (JSC::maybeModifyVMPoolSize): Separated Linux constants from embedded constants.
- 8:22 PM Changeset in webkit [75890] by
-
- 2 edits in trunk/Source/JavaScriptCore
2011-01-15 Sheriff Bot <webkit.review.bot@gmail.com>
Unreviewed, rolling out r75886.
http://trac.webkit.org/changeset/75886
https://bugs.webkit.org/show_bug.cgi?id=52526
"Broke GTK+ 64bit" (Requested by xan_ on #webkit).
- jit/ExecutableAllocatorFixedVMPool.cpp:
- 6:14 PM Changeset in webkit [75889] by
-
- 2 edits in trunk/LayoutTests
Skip fast/dom/Range/range-clone-contents.html on Windows
Not reviewed.
Filed: <https://bugs.webkit.org/show_bug.cgi?id=52524>
- platform/win/Skipped: Skip test on Windows. The test passes
on all other bots.
- 4:57 PM Changeset in webkit [75888] by
-
- 5 edits in tags/Safari-534.15.9/WebKit/mac
Merge r75873.
- 4:57 PM Changeset in webkit [75887] by
-
- 5 edits in tags/Safari-534.15.9
Versioning.
- 3:48 PM Changeset in webkit [75886] by
-
- 2 edits in trunk/Source/JavaScriptCore
<rdar://problem/8870429> Shrink the executable pool on embedded devices
Reviewed by Sam Weinig.
- jit/ExecutableAllocatorFixedVMPool.cpp: Dropped the pool size from 32MB
to 16MB.
- 3:41 PM Changeset in webkit [75885] by
-
- 14 edits2 deletes in trunk
2011-01-15 Sheriff Bot <webkit.review.bot@gmail.com>
Unreviewed, rolling out r75708.
http://trac.webkit.org/changeset/75708
https://bugs.webkit.org/show_bug.cgi?id=52521
Breaks Qt build if mobility is not installed. (Requested by
benjaminp on #webkit).
- WebCore.pro:
- platform/graphics/qt/MediaPlayerPrivateQt.cpp: (WebCore::MediaPlayerPrivateQt::MediaPlayerPrivateQt):
- platform/graphics/qt/MediaPlayerPrivateQt.h: (WebCore::MediaPlayerPrivateQt::supportsFullscreen):
2011-01-15 Sheriff Bot <webkit.review.bot@gmail.com>
Unreviewed, rolling out r75708.
http://trac.webkit.org/changeset/75708
https://bugs.webkit.org/show_bug.cgi?id=52521
Breaks Qt build if mobility is not installed. (Requested by
benjaminp on #webkit).
- Api/qwebkitplatformplugin.h:
- WebCoreSupport/ChromeClientQt.cpp: (WebCore::ChromeClientQt::ChromeClientQt): (WebCore::ChromeClientQt::~ChromeClientQt):
- WebCoreSupport/ChromeClientQt.h:
- WebCoreSupport/FullScreenVideoQt.cpp: Removed.
- WebCoreSupport/FullScreenVideoQt.h: Removed.
- WebCoreSupport/QtPlatformPlugin.cpp:
- WebCoreSupport/QtPlatformPlugin.h:
- examples/platformplugin/WebPlugin.cpp: (WebPlugin::supportsExtension): (WebPlugin::createExtension):
- examples/platformplugin/WebPlugin.h:
- examples/platformplugin/platformplugin.pro:
- examples/platformplugin/qwebkitplatformplugin.h:
- 3:39 PM Changeset in webkit [75884] by
-
- 5 edits in trunk
2011-01-15 Oliver Hunt <oliver@apple.com>
Reviewed by Maciej Stachowiak.
Incorrect behavior changing attributes of an accessor
https://bugs.webkit.org/show_bug.cgi?id=52515
defineProperty doesn't correctly handle changing attributes of an accessor
property. This is because we don't pass the full descriptor to the
putDescriptor helper function, which means we have insufficient information
to do the right thing. Once that's passed the correct behavior is relatively
simple to implement.
- runtime/JSObject.cpp: (JSC::putDescriptor): (JSC::JSObject::defineOwnProperty):
- 3:33 PM Changeset in webkit [75883] by
-
- 1 copy in tags/Safari-534.15.9
New tag.
- 12:44 PM Changeset in webkit [75882] by
-
- 3 edits2 adds in trunk
<http://webkit.org/b/52512> REGRESSION(r73818): range.cloneContents() ignores end offset
Reviewed by Adele Peterson.
WebCore:
The fix for Bug 50710 in r73799 introduced an off-by-one error
when copying nodes to a local NodeVector for processing. A fix
was attempted for Bug 50854 in r73818, but instead of stopping
at the end offset, it iterates through all the sibling nodes
because the loop variable (i) is never incremented. To clean
this up, revert back to the code in r73799 and fix the
off-by-one error.
Test: fast/dom/Range/range-clone-contents.html
- dom/Range.cpp:
(WebCore::Range::processContents): Fix the loop that copies
nodes to a local NodeVector by restoring the code from r73799
and fixing the off-by-one error.
LayoutTests:
- fast/dom/Range/range-clone-contents-expected.txt: Added.
- fast/dom/Range/range-clone-contents.html: Added.
- 12:04 PM Changeset in webkit [75881] by
-
- 2 edits in trunk/Tools
2011-01-15 Adam Barth <abarth@webkit.org>
Teach svn-apply about WebKit2 move.
- Scripts/VCSUtils.pm:
- 12:02 PM Changeset in webkit [75880] by
-
- 32 edits1 move in trunk
Move WebKit2 into Source
https://bugs.webkit.org/show_bug.cgi?id=52438
Rubber-stamped by Eric Seidel.
.:
Update reference to WebKit2.
- DerivedSources.pro:
- GNUmakefile.am:
- Makefile:
- Source/cmake/WebKitPackaging.cmake:
- WebKit.pro:
- configure.ac:
Source/WebCore:
- WebCore.pro:
Source/WebKit2:
- DerivedSources.pro:
- GNUmakefile.am:
- WebKit2.pro:
- WebProcess.pro:
- win/WebKit2.vcproj:
- win/WebKit2WebProcess.vcproj:
Tools:
- MiniBrowser/qt/MiniBrowser.pro:
- Scripts/build-webkit:
- Scripts/do-file-rename:
- Scripts/do-webcore-rename:
- Scripts/test-webkitpy:
- Scripts/webkitdirs.pm:
- Scripts/webkitpy/style/checker.py:
- WebKitTestRunner/DerivedSources.pro:
- WebKitTestRunner/InjectedBundle/qt/InjectedBundle.pro:
- WebKitTestRunner/qt/WebKitTestRunner.pro:
WebKit/win:
Update reference to WebKit2.
- WebKit.vcproj/WebKit.sln:
Websites/bugs.webkit.org:
Remove WebKit2 from the list of top-level source directories.
- PrettyPatch/PrettyPatch.rb:
- 11:26 AM Changeset in webkit [75879] by
-
- 3 edits in trunk/Source/WebCore
2011-01-15 Joone Hur <joone.hur@collabora.co.uk>
Reviewed by Martin Robinson.
[GTK] Linux build with FileSystem API enabled fails
https://bugs.webkit.org/show_bug.cgi?id=43878
This patch allows WebKitGtk+ to build with FileSystem API option.
No new tests because no new functionality.
- GNUmakefile.am: Included AsyncFileSystem.h,cpp and Excluded duplicated JSFileException.h,cpp.
- bindings/js/JSDirectoryEntryCustom.cpp: Included ExceptionCode.h.
- 6:51 AM Changeset in webkit [75878] by
-
- 4 edits in trunk
2011-01-15 Robert Hogan <robert@webkit.org>
Reviewed by Kenneth Rohde Christiansen.
[Qt] plugins/no-mime-with-valid-extension.html fails
Check extension of resource against installed
plugins when determining mime-type.
https://bugs.webkit.org/show_bug.cgi?id=52288
https://bugs.webkit.org/show_bug.cgi?id=50657
- platform/qt/Skipped:
2011-01-15 Robert Hogan <robert@webkit.org>
Reviewed by Kenneth Rohde Christiansen.
[Qt] plugins/no-mime-with-valid-extension.html fails
Check extension of resource against installed
plugins when determining mime-type.
https://bugs.webkit.org/show_bug.cgi?id=52288
https://bugs.webkit.org/show_bug.cgi?id=50657
- WebCoreSupport/FrameLoaderClientQt.cpp: (WebCore::FrameLoaderClientQt::objectContentType):
- 6:17 AM Changeset in webkit [75877] by
-
- 2 edits in trunk/LayoutTests
2011-01-15 Stephen White <senorblanco@chromium.org>
Unreviewed; updated chromium test expectations.
- platform/chromium-gpu/test_expectations.txt:
- 6:11 AM Changeset in webkit [75876] by
-
- 3 edits in trunk/LayoutTests
2011-01-15 Stephen White <senorblanco@chromium.org>
Unreviewed; updated chromium test expectations.
- platform/chromium-gpu/test_expectations.txt:
- platform/chromium/test_expectations.txt:
- 1:33 AM Changeset in webkit [75875] by
-
- 5 edits in trunk
2011-01-15 Jochen Eisinger <jochen@chromium.org>
Reviewed by David Levin.
[chromium] remove obsolete createView method
https://bugs.webkit.org/show_bug.cgi?id=52437
- DumpRenderTree/chromium/WebViewHost.cpp: (WebViewHost::createView):
- DumpRenderTree/chromium/WebViewHost.h:
2011-01-15 Jochen Eisinger <jochen@chromium.org>
Reviewed by David Levin.
[chromium] remove obsolete createView method
https://bugs.webkit.org/show_bug.cgi?id=52437
- public/WebViewClient.h: (WebKit::WebViewClient::createView):
- 12:58 AM Changeset in webkit [75874] by
-
- 2 edits in trunk/LayoutTests
2011-01-15 Tony Chang <tony@chromium.org>
Unreviewed, add new test to GTK+ Skipped list.
https://bugs.webkit.org/show_bug.cgi?id=52508
- platform/gtk/Skipped:
Jan 14, 2011:
- 10:48 PM Changeset in webkit [75873] by
-
- 5 edits in trunk/WebKit/mac
2011-01-14 Simon Fraser <Simon Fraser>
Reviewed by Dan Bernstein.
Plugins render incorrectly with transformed ancestors
https://bugs.webkit.org/show_bug.cgi?id=52507
-[NSView visibleRect] gives the wrong answer if there are CSS transforms
in the ancestor chain of a plugin.
So use of this method with calls to -actualVisibleRectInWindow, which
maps rects through the render tree to compute the correct rect,
clipping via windowClipRect() if necessary.
Not testable, because doing so relies on the behavior of some
plugins, which stop rendering if setWindow passes an empty rect.
- Plugins/Hosted/WebHostedNetscapePluginView.mm: (-[WebHostedNetscapePluginView updateAndSetWindow]):
- Plugins/WebBaseNetscapePluginView.h:
- Plugins/WebBaseNetscapePluginView.mm: (-[WebBaseNetscapePluginView actualVisibleRectInWindow]):
- Plugins/WebNetscapePluginView.mm: (-[WebNetscapePluginView saveAndSetNewPortStateForUpdate:]):
- 10:35 PM Changeset in webkit [75872] by
-
- 5 edits in trunk
2011-01-14 Oliver Hunt <oliver@apple.com>
Reviewed by Maciej Stachowiak.
[jsfunfuzz] Incorrect handling of consecutive duplicate labels
https://bugs.webkit.org/show_bug.cgi?id=52505
Add a few tests of consecutive labels containing duplicates to the parser check.
- fast/js/parser-syntax-check-expected.txt:
- fast/js/script-tests/parser-syntax-check.js:
2011-01-14 Oliver Hunt <oliver@apple.com>
Reviewed by Maciej Stachowiak.
[jsfunfuzz] Incorrect handling of consecutive duplicate labels
https://bugs.webkit.org/show_bug.cgi?id=52505
Compare StringImpl*'s instead of Identifier*'s when looking for duplicate
labels.
- parser/JSParser.cpp: (JSC::JSParser::parseExpressionOrLabelStatement):
- 10:01 PM Changeset in webkit [75871] by
-
- 2 edits in trunk/WebKit/chromium
2011-01-14 Xiyuan Xia <xiyuan@chromium.org>
Reviewed by Darin Fisher.
Prepare Linux WebThemeEngine interface to support new theme parts.
https://bugs.webkit.org/show_bug.cgi?id=52413
- public/linux/WebThemeEngine.h:
- 9:38 PM Changeset in webkit [75870] by
-
- 12 edits in trunk
2011-01-14 Sam Magnuson <smagnuso@gmail.com>
Reviewed by Kenneth Rohde Christiansen.
[Qt] Compile with QT_NO_GRAPHICSVIEW
https://bugs.webkit.org/show_bug.cgi?id=49750
- platform/graphics/qt/GraphicsLayerQt.cpp:
- platform/graphics/qt/GraphicsLayerQt.h:
- platform/qt/PlatformMouseEventQt.cpp:
2011-01-14 Sam Magnuson <smagnuso@gmail.com>
Reviewed by Kenneth Rohde Christiansen.
[Qt] Compile with QT_NO_GRAPHICSVIEW
https://bugs.webkit.org/show_bug.cgi?id=49750
- Api/qgraphicswebview.cpp:
- Api/qgraphicswebview.h:
- Api/qwebpage.cpp: (QWebPage::event):
- Api/qwebpage_p.h:
- WebCoreSupport/FrameLoaderClientQt.cpp: (WebCore::FrameLoaderClientQt::createPlugin):
- WebCoreSupport/PageClientQt.cpp:
- WebCoreSupport/PageClientQt.h:
- 8:20 PM Changeset in webkit [75869] by
-
- 5 edits1 delete in trunk/LayoutTests
Not reviewed, update of Chromium test expectations.
Update of files affected by mass rebaseline in r75828 and r75838
- platform/chromium-mac/animations/3d/state-at-end-event-transform-expected.checksum:
- platform/chromium-mac/animations/3d/state-at-end-event-transform-expected.png:
- platform/chromium-mac/plugins/embed-attributes-style-expected.txt: Removed.
- platform/chromium-mac/svg/custom/svg-curve-with-relative-cordinates-expected.checksum:
- platform/chromium-mac/svg/custom/svg-curve-with-relative-cordinates-expected.png:
- 7:53 PM Changeset in webkit [75868] by
-
- 1 edit in trunk/Source/JavaScriptCore/ChangeLog
2011-01-14 Simon Fraser <Simon Fraser>
No review.
Initialize m_operationInProgress after r75855.
- runtime/Heap.cpp: (JSC::Heap::Heap):
- 7:32 PM Changeset in webkit [75867] by
-
- 2 edits in trunk/Source/JavaScriptCore
Reverted accidentally committed code from my last checkin.
- runtime/Heap.cpp:
(JSC::Heap::markRoots):
- 7:28 PM Changeset in webkit [75866] by
-
- 2 edits in trunk/Source/JavaScriptCore
Try to fix the GTK bot.
Reviewed by Sam Weinig.
- runtime/Heap.cpp:
(JSC::Heap::Heap):
(JSC::Heap::markRoots): Kids, remember to initialize your data members.
Knowing is half the battle.
- 6:33 PM Changeset in webkit [75865] by
-
- 2 edits in trunk/LayoutTests
Not reviewed, update of test expectations.
Ignore new test fast/js/js-continue-break-restrictions.html, it fails for V8.
- platform/chromium/test_expectations.txt:
- 6:26 PM Changeset in webkit [75864] by
-
- 2 edits6 adds in trunk/LayoutTests
2011-01-14 Eric Uhrhane <ericu@chromium.org>
Reviewed by David Levin.
Add tests for FileWriterSync
https://bugs.webkit.org/show_bug.cgi?id=50720
- fast/filesystem/resources/file-writer-sync-truncate-extend.js: Added.
- fast/filesystem/workers/file-writer-sync-truncate-extend.html: Added.
- fast/filesystem/workers/file-writer-sync-truncate-extend-expected.txt: Added.
- fast/filesystem/resources/file-writer-sync-write-overlapped.js: Added.
- fast/filesystem/workers/file-writer-sync-write-overlapped.html: Added.
- fast/filesystem/workers/file-writer-sync-write-overlapped-expected.txt: Added.
- fast/filesystem/resources/file-writer-utils.js: (assert): Make assert throw, rather than continue. (cleanUp): Always call finishJSTest, even without fileEntryForCleanup.
- 6:22 PM Changeset in webkit [75863] by
-
- 3 edits in trunk/WebKit/efl
2011-01-14 Patrick Gansterer <Patrick Gansterer>
Unreviewed. Try to fix EFL build.
- ewk/ewk_frame.cpp: (ewk_frame_scroll_add): Added WebCore namespace to IntSize.
- ewk/ewk_view.cpp: (_ewk_view_viewport_attributes_compute): Ditto.
- 6:14 PM Changeset in webkit [75862] by
-
- 5 edits in trunk
2011-01-14 Oliver Hunt <oliver@apple.com>
Reviewed by Stephanie Lewis.
[jsfunfuzz] We should be clearing the lexers temporary character buffers when switching to strict mode
https://bugs.webkit.org/show_bug.cgi?id=52501
Add checks for correct behaviour when the strict mode token is immediately followed
by a value that needs a temporary buffer in the lexer.
- fast/js/basic-strict-mode-expected.txt:
- fast/js/script-tests/basic-strict-mode.js:
2011-01-14 Oliver Hunt <oliver@apple.com>
Reviewed by Stephanie Lewis.
[jsfunfuzz] We should be clearing the lexers temporary character buffers when switching to strict mode
https://bugs.webkit.org/show_bug.cgi?id=52501
Clear the temporary character buffers used for reading escaped characters and
numbers.
- parser/Lexer.h: (JSC::Lexer::setOffset):
- 6:06 PM Changeset in webkit [75861] by
-
- 4 edits2 adds in trunk
2011-01-14 Tony Chang <tony@chromium.org>
Reviewed by Alexey Proskuryakov.
Strip NUL character when copying text on Windows
https://bugs.webkit.org/show_bug.cgi?id=52236
- editing/pasteboard/copy-null-characters-expected.txt: Added.
- editing/pasteboard/copy-null-characters.html: Added.
2011-01-14 Tony Chang <tony@chromium.org>
Reviewed by Alexey Proskuryakov.
Strip NUL character when copying text on Windows
https://bugs.webkit.org/show_bug.cgi?id=52236
Test: editing/pasteboard/copy-null-characters.html
- editing/Editor.cpp: (WebCore::Editor::selectedText):
- platform/mac/PasteboardMac.mm: (WebCore::Pasteboard::writeSelection): Use editor()->selectedText() which matches the other platforms.
- 6:00 PM Changeset in webkit [75860] by
-
- 4 edits in tags/Safari-534.15.8/WebCore
Merge r75758.
- 6:00 PM Changeset in webkit [75859] by
-
- 3 edits in tags/Safari-534.15.8/WebKit/mac
Merge r75829.
- 6:00 PM Changeset in webkit [75858] by
-
- 5 edits in tags/Safari-534.15.8
Versioning.
- 5:59 PM Changeset in webkit [75857] by
-
- 32 edits in trunk
2011-01-14 Yuzo Fujishima <yuzo@google.com>
Reviewed by Antti Koivisto.
Rename cache() to memoryCache()
https://bugs.webkit.org/show_bug.cgi?id=52433
No new tests because the behavior remains the same.
- WebCore.exp.in:
- WebCore.order:
- history/PageCache.cpp: (WebCore::PageCache::releaseAutoreleasedPagesNow):
- inspector/InspectorResourceAgent.cpp: (WebCore::InspectorResourceAgent::cachedResource):
- loader/FrameLoader.cpp: (WebCore::FrameLoader::tellClientAboutPastMemoryCacheLoads):
- loader/archive/cf/LegacyWebArchive.cpp: (WebCore::LegacyWebArchive::create):
- loader/cache/CachedImage.cpp: (WebCore::CachedImage::allClientsRemoved): (WebCore::CachedImage::data):
- loader/cache/CachedResource.cpp: (WebCore::CachedResource::~CachedResource): (WebCore::CachedResource::addClientToSet): (WebCore::CachedResource::removeClient): (WebCore::CachedResource::setDecodedSize): (WebCore::CachedResource::setEncodedSize): (WebCore::CachedResource::didAccessDecodedData):
- loader/cache/CachedResourceLoader.cpp: (WebCore::CachedResourceLoader::CachedResourceLoader): (WebCore::CachedResourceLoader::requestUserCSSStyleSheet): (WebCore::CachedResourceLoader::requestResource): (WebCore::CachedResourceLoader::revalidateResource): (WebCore::CachedResourceLoader::loadResource): (WebCore::CachedResourceLoader::clearPreloads): (WebCore::CachedResourceLoader::printPreloadStats):
- loader/cache/CachedResourceRequest.cpp: (WebCore::CachedResourceRequest::load): (WebCore::CachedResourceRequest::didFail): (WebCore::CachedResourceRequest::didReceiveResponse):
- loader/cache/MemoryCache.cpp: (WebCore::memoryCache):
- loader/cache/MemoryCache.h:
2011-01-14 Yuzo Fujishima <yuzo@google.com>
Reviewed by Antti Koivisto.
Rename cache() to memoryCache()
https://bugs.webkit.org/show_bug.cgi?id=52433
- src/WebCache.cpp: (WebKit::WebCache::setCapacities): (WebKit::WebCache::clear): (WebKit::WebCache::getUsageStats): (WebKit::WebCache::getResourceTypeStats):
2011-01-14 Yuzo Fujishima <yuzo@google.com>
Reviewed by Antti Koivisto.
Rename cache() to memoryCache()
https://bugs.webkit.org/show_bug.cgi?id=52433
- webkit/webkitglobals.cpp: (webkit_set_cache_model):
2011-01-14 Yuzo Fujishima <yuzo@google.com>
Reviewed by Antti Koivisto.
Rename cache() to memoryCache()
https://bugs.webkit.org/show_bug.cgi?id=52433
- Misc/WebCache.mm: (+[WebCache statistics]): (+[WebCache setDisabled:]): (+[WebCache isDisabled]):
- WebView/WebView.mm: (+[WebView _setCacheModel:]):
2011-01-14 Yuzo Fujishima <yuzo@google.com>
Reviewed by Antti Koivisto.
Rename cache() to memoryCache()
https://bugs.webkit.org/show_bug.cgi?id=52433
- Api/qwebpage.cpp: (QWebPagePrivate::_q_cleanupLeakMessages):
- Api/qwebsettings.cpp: (QWebSettings::clearMemoryCaches): (QWebSettings::setObjectCacheCapacities):
2011-01-14 Yuzo Fujishima <yuzo@google.com>
Reviewed by Antti Koivisto.
Rename cache() to memoryCache()
https://bugs.webkit.org/show_bug.cgi?id=52433
- WebCache.cpp: (WebCache::statistics): (WebCache::empty): (WebCache::setDisabled): (WebCache::disabled):
- WebView.cpp: (WebView::setCacheModel):
2011-01-14 Yuzo Fujishima <yuzo@google.com>
Reviewed by Antti Koivisto.
Rename cache() to memoryCache()
https://bugs.webkit.org/show_bug.cgi?id=52433
- WebView.cpp: (wxWebView::SetCachePolicy):
2011-01-14 Yuzo Fujishima <yuzo@google.com>
Reviewed by Antti Koivisto.
Rename cache() to memoryCache()
https://bugs.webkit.org/show_bug.cgi?id=52433
- WebProcess/WebProcess.cpp: (WebKit::WebProcess::shutdownIfPossible): (WebKit::WebProcess::didClose):
- WebProcess/mac/WebProcessMac.mm: (WebKit::WebProcess::platformSetCacheModel):
- WebProcess/win/WebProcessWin.cpp: (WebKit::WebProcess::platformSetCacheModel):
- 5:52 PM Changeset in webkit [75856] by
-
- 2 edits in trunk/Source/JavaScriptCore
Try to fix non-Dtrace builds: #include Tracing.h instead of TracingDtrace.h.
- runtime/Heap.cpp:
- 5:43 PM Changeset in webkit [75855] by
-
- 18 edits2 copies in trunk
2011-01-13 Geoffrey Garen <ggaren@apple.com>
Reviewed by Oliver Hunt.
Split out a MarkedSpace strategy object from Heap.
https://bugs.webkit.org/show_bug.cgi?id=52421
- Misc/WebCoreStatistics.mm: (+[WebCoreStatistics memoryStatistics]): Updated for class move.
2011-01-13 Geoffrey Garen <ggaren@apple.com>
Reviewed by Oliver Hunt.
Split out a MarkedSpace strategy object from Heap.
https://bugs.webkit.org/show_bug.cgi?id=52421
SunSpider reports no change.
- Android.mk:
- CMakeLists.txt:
- GNUmakefile.am:
- JavaScriptCore.gypi:
- JavaScriptCore.pro:
- JavaScriptCore.vcproj/JavaScriptCore/JavaScriptCore.vcproj:
- JavaScriptCore.xcodeproj/project.pbxproj: Updated our 7 build systems. Which is cool.
- runtime/Heap.cpp: (JSC::Heap::Heap): (JSC::Heap::destroy): (JSC::Heap::recordExtraCost): (JSC::Heap::allocate): (JSC::Heap::markConservatively): (JSC::Heap::markRoots): (JSC::Heap::objectCount): (JSC::Heap::statistics): (JSC::Heap::size): (JSC::Heap::isBusy): (JSC::Heap::collectAllGarbage): (JSC::Heap::primaryHeapBegin): (JSC::Heap::primaryHeapEnd):
- runtime/Heap.h: (JSC::Heap::globalData): (JSC::Heap::markedSpace): (JSC::Heap::isCellMarked): (JSC::Heap::checkMarkCell): (JSC::Heap::markCell): Moved all code pertaining to managing chunks of collector memory out of this class. Heap now just delegates to MarkedSpace.
- runtime/JSCell.h: (JSC::JSCell::Heap::heap): Updated for MarkedSpace delegation.
- runtime/JSValue.h: Moved the ValueStringPair typedef to help with #includes.
- runtime/MarkedSpace.cpp: Copied from runtime/Heap.cpp. (JSC::MarkedSpace::MarkedSpace): (JSC::MarkedSpace::destroy): (JSC::MarkedSpace::allocateBlock): (JSC::MarkedSpace::freeBlock): (JSC::MarkedSpace::allocate): (JSC::MarkedSpace::resizeBlocks): (JSC::MarkedSpace::growBlocks): (JSC::MarkedSpace::shrinkBlocks): (JSC::MarkedSpace::markConservatively): (JSC::MarkedSpace::clearMarkBits): (JSC::MarkedSpace::markedCells): (JSC::MarkedSpace::sweep): (JSC::MarkedSpace::objectCount): (JSC::MarkedSpace::addToStatistics): (JSC::MarkedSpace::statistics): (JSC::MarkedSpace::size): (JSC::MarkedSpace::reset): (JSC::MarkedSpace::primaryHeapBegin): (JSC::MarkedSpace::primaryHeapEnd):
- runtime/MarkedSpace.h: Copied from runtime/Heap.h. (JSC::MarkedSpace::globalData): (JSC::MarkedSpace::didShrink): (JSC::MarkedSpace::cellBlock): (JSC::MarkedSpace::cellOffset): (JSC::MarkedSpace::isCellMarked): (JSC::MarkedSpace::checkMarkCell): (JSC::MarkedSpace::markCell): Moved all code pertaining to managing chunks of collector memory into this class.
- runtime/MemoryStatistics.cpp: (JSC::heapStatistics):
- runtime/MemoryStatistics.h: Updated for MarkedSpace delegation.
2011-01-13 Geoffrey Garen <ggaren@apple.com>
Reviewed by Oliver Hunt.
Split out a MarkedSpace strategy object from Heap.
https://bugs.webkit.org/show_bug.cgi?id=52421
- Shared/mac/WebMemorySampler.mac.mm: (WebKit::WebMemorySampler::sampleWebKit): Updated for class move.
- 5:40 PM Changeset in webkit [75854] by
-
- 5 edits in trunk/LayoutTests
2011-01-14 Nico Weber <thakis@chromium.org>
Unreviewed, test expectations.
Rebaseline gradient test due to updated skia. reed checked the test and
says the new look is right and is an intentional change.
https://bugs.webkit.org/show_bug.cgi?id=52105
- platform/chromium-linux/svg/batik/paints/gradientLimit-expected.checksum:
- platform/chromium-linux/svg/batik/paints/gradientLimit-expected.png:
- platform/chromium-win/svg/batik/paints/gradientLimit-expected.checksum:
- platform/chromium-win/svg/batik/paints/gradientLimit-expected.png:
- 5:31 PM Changeset in webkit [75853] by
-
- 10 edits in trunk/WebKit2
2011-01-14 Anders Carlsson <andersca@apple.com>
Reviewed by Dan Bernstein.
Add a new PageClient::displayView function and expose it on WebPageProxy
https://bugs.webkit.org/show_bug.cgi?id=52500
- UIProcess/API/mac/PageClientImpl.h:
- UIProcess/API/mac/PageClientImpl.mm: (WebKit::PageClientImpl::displayView):
- UIProcess/API/qt/qwkpage.cpp: (QWKPagePrivate::displayView):
- UIProcess/API/qt/qwkpage_p.h:
- UIProcess/PageClient.h:
- UIProcess/WebPageProxy.cpp: (WebKit::WebPageProxy::displayView):
- UIProcess/WebPageProxy.h:
- UIProcess/win/WebView.cpp: (WebKit::WebView::displayView):
- UIProcess/win/WebView.h:
- 5:22 PM Changeset in webkit [75852] by
-
- 5 edits3 adds in trunk
2011-01-14 Oliver Hunt <oliver@apple.com>
Reviewed by Gavin Barraclough.
[jsfunfuzz] parser doesn't enforce continue restrictions correctly.
https://bugs.webkit.org/show_bug.cgi?id=52493
Add a few tests for continue to cover the cases where continue
isn't syntactically valid.
- fast/js/js-continue-break-restrictions-expected.txt: Added.
- fast/js/js-continue-break-restrictions.html: Added.
- fast/js/script-tests/js-continue-break-restrictions.js: Added.
2011-01-14 Oliver Hunt <oliver@apple.com>
Reviewed by Gavin Barraclough.
[jsfunfuzz] parser doesn't enforce continue restrictions correctly.
https://bugs.webkit.org/show_bug.cgi?id=52493
This patch reworks handling of break, continue and label statements
to correctly handle all the valid and invalid cases. Previously certain
errors would be missed by the parser in strict mode, but the bytecode
generator needed to handle those cases for non-strict code so nothing
failed, it simply became non-standard behaviour.
Now that we treat break and continue errors as early faults in non-strict
mode as well that safety net has been removed so the parser bugs result in
crashes at codegen time.
- parser/JSParser.cpp: (JSC::JSParser::ScopeLabelInfo::ScopeLabelInfo): (JSC::JSParser::next): (JSC::JSParser::nextTokenIsColon): (JSC::JSParser::continueIsValid):
Continue is only valid in loops so we can't use breakIsValid()
(JSC::JSParser::pushLabel):
We now track whether the label is for a loop (and is therefore a
valid target for continue.
(JSC::JSParser::popLabel):
(JSC::JSParser::getLabel):
Replace hasLabel with getLabel so that we can validate the target
when parsing continue statements.
(JSC::JSParser::Scope::continueIsValid):
(JSC::JSParser::Scope::pushLabel):
(JSC::JSParser::Scope::getLabel):
(JSC::JSParser::JSParser):
(JSC::JSParser::parseBreakStatement):
(JSC::JSParser::parseContinueStatement):
(JSC::LabelInfo::LabelInfo):
(JSC::JSParser::parseExpressionOrLabelStatement):
Consecutive labels now get handled iteratively so that we can determine
whether they're valid targets for continue.
- parser/Lexer.cpp: (JSC::Lexer::nextTokenIsColon):
- parser/Lexer.h: (JSC::Lexer::setOffset):
- 5:20 PM Changeset in webkit [75851] by
-
- 5 edits1 add in trunk
Source/WebCore: WebCore part of <rdar://problem/8441312> Crash in -[NSView _invalidateGStatesForTree]
Reviewed by Simon Fraser.
- WebCore.exp.in: Export RenderWidget::suspendWidgetHierarchyUpdates() and
RenderWidget::resumeWidgetHierarchyUpdates().
- manual-tests/plug-in-mutates-NSView-hierarchy-during-resize.html: Added.
- rendering/RenderWidget.cpp:
(WebCore::RenderWidget::setWidgetGeometry): Removed the assertion that widget hierarchy updates
are disabled. When this assertion was added, this condition was a subset of the “calling out to
plug-in code is forbidden” condition, hence the assertion was valid. The WebKit part of this
change now suspends widget hierarchy updates even at times where plug-in code is expected to be
called, which invalidates the assertion.
WebKit/mac: WebKit/mac part of <rdar://problem/8441312> Crash in -[NSView _invalidateGStatesForTree]
Reviewed by Simon Fraser.
- WebView/WebHTMLView.mm:
(-[WebHTMLView _invalidateGStatesForTree]): Override this NSView method, and bracket the call
to the superclass implementation with suspending WebCore Widget hierarchy updates. This ensures
that the NSView tree doesn’t change from under AppKit as it traverses it.
- 5:19 PM Changeset in webkit [75850] by
-
- 2 edits in trunk
2011-01-14 Ahmad Sharif <asharif.tools@gmail.com>
Reviewed by Darin Fisher.
When inlining is performed, the progress variable is
declared as uninitialized.
https://bugs.webkit.org/show_bug.cgi?id=51084
- WebCore: Copied from WebCore.
- WebCore/page/animation/KeyframeAnimation.cpp: (WebCore::KeyframeAnimation::animate): (WebCore::KeyframeAnimation::getAnimatedStyle):
- 5:18 PM Changeset in webkit [75849] by
-
- 2 edits in trunk/WebKit2
2011-01-14 Anders Carlsson <andersca@apple.com>
Reviewed by Sam Weinig.
Implement the "should paint bounds instead of indiviual rects" algorithm from WebKit1
https://bugs.webkit.org/show_bug.cgi?id=52499
- WebProcess/WebPage/DrawingAreaImpl.cpp: (WebKit::shouldPaintBoundsRect): Port code from -[WebView _mustDrawUnionedRect:singleRects:count:].
(WebKit::DrawingAreaImpl::display):
If shouldPaintBoundsRect returns true, clear the rects vector and append the bounds rect.
- 5:17 PM Changeset in webkit [75848] by
-
- 4 edits in trunk
2011-01-14 Tony Gentilcore <tonyg@chromium.org>
Unreviewed build fix.
Fix Qt build after r75837
https://bugs.webkit.org/show_bug.cgi?id=52494
- rendering/style/StyleRareInheritedData.cpp:
- rendering/style/StyleRareNonInheritedData.cpp:
- 5:15 PM Changeset in webkit [75847] by
-
- 2 edits in trunk/Websites/bugs.webkit.org
2011-01-14 Ojan Vafai <ojan@chromium.org>
Reviewed by Adam Barth.
fix expanded lines in the code review tool
https://bugs.webkit.org/show_bug.cgi?id=52498
Fixes them to work now that we have wrapper divs.
Also fixes the long-standing bug that the line numbers
were incorrect sometimes.
- code-review.js:
- 5:08 PM Changeset in webkit [75846] by
-
- 4 edits in trunk
2011-01-14 Tony Gentilcore <tonyg@chromium.org>
Unreviewed build fix.
Fix Qt build after r75837
https://bugs.webkit.org/show_bug.cgi?id=52494
- rendering/style/RenderStyle.cpp:
2011-01-14 Tony Gentilcore <tonyg@chromium.org>
Unreviewed build fix.
Fix Qt build after r75837
https://bugs.webkit.org/show_bug.cgi?id=52494
- ewk/ewk_view.cpp:
- 5:02 PM Changeset in webkit [75845] by
-
- 3 edits in trunk/Websites/bugs.webkit.org
2011-01-14 Ojan Vafai <ojan@chromium.org>
Reviewed by Adam Barth.
put remove lines to the left of add lines in sidebyside view
https://bugs.webkit.org/show_bug.cgi?id=52458
Also, sneaking in a fix to make long lines with no whitespace wrap.
- PrettyPatch/PrettyPatch.rb:
- code-review.js:
- 5:00 PM Changeset in webkit [75844] by
-
- 2 edits in trunk/Source/WebCore
2011-01-14 Tony Gentilcore <tonyg@chromium.org>
Unreviewed build fix.
Fix Qt build after r75837
https://bugs.webkit.org/show_bug.cgi?id=52494
- editing/EditorCommand.cpp:
- 4:55 PM Changeset in webkit [75843] by
-
- 3 edits in trunk/Source/WebCore
2011-01-14 Tony Gentilcore <tonyg@chromium.org>
Unreviewed build fix.
Fix Qt build after r75837
https://bugs.webkit.org/show_bug.cgi?id=52494
- editing/ApplyStyleCommand.cpp:
- 4:44 PM Changeset in webkit [75842] by
-
- 2 edits in trunk/Source/WebCore
2011-01-14 Tony Gentilcore <tonyg@chromium.org>
Unreviewed build fix.
Fix Qt build after r75837
https://bugs.webkit.org/show_bug.cgi?id=52494
- css/CSSStyleSelector.cpp:
- 4:42 PM Changeset in webkit [75841] by
-
- 3 edits in trunk/LayoutTests
2011-01-14 Maciej Stachowiak <mjs@apple.com>
Reviewed by Anders Carlsson.
Fix a few things in the WebKit2 skipped list
https://bugs.webkit.org/show_bug.cgi?id=52484
- platform/mac-wk2/Skipped: Add/remove some tests to the WK2 skipped list.
- editing/style/stylewithcss-without-selection.html: Fix this test to avoid referencing an undefined variable in the failure case.
- 4:35 PM Changeset in webkit [75840] by
-
- 3 edits in trunk/Source/WebCore
2011-01-14 Tony Gentilcore <tonyg@chromium.org>
Unreviewed build fix.
Fix Qt build after r75837
https://bugs.webkit.org/show_bug.cgi?id=52494
- rendering/InlineFlowBox.h:
- rendering/RenderBoxModelObject.h:
- 4:29 PM Changeset in webkit [75839] by
-
- 2 edits in trunk/Source/JavaScriptCore
2011-01-14 Patrick Gansterer <Patrick Gansterer>
Reviewed by Adam Roben.
Use the Windows thread pool instead of an extra thread for FastMalloc scavenging
https://bugs.webkit.org/show_bug.cgi?id=45186
- wtf/FastMalloc.cpp: (WTF::TCMalloc_PageHeap::scheduleScavenger): Added missing this pointer to CreateTimerQueueTimer().
- 4:28 PM Changeset in webkit [75838] by
-
- 231 edits230 copies16 adds in trunk/LayoutTests
2011-01-14 Mihai Parparita <mihaip@chromium.org>
Update the svg/ pixel expectations for the chromium-mac port.
For the pixel tests that currently fail on Snow Leopard, this moves the
current pixel results to platform/chromium-mac-leopard and adds new
results to platform/chromium-mac.
List of files omitted due to length but composed entirely of moves from
platform/chromium-mac to platform/chromium-mac-leopard and newly added
files in platform/chromium-mac.
- 4:27 PM Changeset in webkit [75837] by
-
- 56 edits in trunk
2011-01-14 Tony Gentilcore <tonyg@chromium.org>
Reviewed by David Levin.
Do some forward declaration in RenderStyle.h
https://bugs.webkit.org/show_bug.cgi?id=52453
No new tests because no new functionality.
- css/CSSComputedStyleDeclaration.cpp:
- css/CSSStyleSelector.cpp:
- editing/Editor.cpp:
- page/EventHandler.cpp:
- rendering/EllipsisBox.cpp:
- rendering/InlineBox.cpp:
- rendering/InlineTextBox.cpp:
- rendering/RenderBlock.cpp:
- rendering/RenderBlock.h:
- rendering/RenderBox.cpp:
- rendering/RenderBox.h:
- rendering/RenderEmbeddedObject.cpp:
- rendering/RenderFieldset.cpp:
- rendering/RenderFileUploadControl.cpp:
- rendering/RenderFrameSet.cpp:
- rendering/RenderHTMLCanvas.cpp:
- rendering/RenderImageResource.cpp:
- rendering/RenderInputSpeech.cpp:
- rendering/RenderLayer.h:
- rendering/RenderLineBoxList.cpp:
- rendering/RenderListBox.cpp:
- rendering/RenderMediaControlsChromium.cpp:
- rendering/RenderObject.cpp:
- rendering/RenderObject.h:
- rendering/RenderObjectChildList.cpp:
- rendering/RenderProgress.cpp:
- rendering/RenderScrollbarPart.cpp:
- rendering/RenderTable.cpp:
- rendering/RenderTable.h:
- rendering/RenderTableCell.cpp:
- rendering/RenderTableRow.cpp:
- rendering/RenderTableSection.cpp:
- rendering/RenderTheme.cpp:
- rendering/RenderThemeChromiumSkia.cpp:
- rendering/RenderThemeChromiumWin.cpp:
- rendering/RootInlineBox.cpp:
- rendering/style/RenderStyle.cpp:
- rendering/style/RenderStyle.h:
- rendering/style/StyleRareInheritedData.cpp:
- 4:19 PM Changeset in webkit [75836] by
-
- 2 edits in trunk/WebKit/chromium
2011-01-14 Jochen Eisinger <jochen@chromium.org>
Unreviewed, roll chromium deps to r71432.
- DEPS:
- 4:16 PM Changeset in webkit [75835] by
-
- 7 edits in trunk/Source/WebCore
2011-01-14 Ryosuke Niwa <rniwa@webkit.org>
Reviewed by Eric Seidel.
Stop instantiating legacy editing positions in AccessibilityRenderObject.cpp, Element.cpp,
BreakBlockquoteCommand.cpp, CompositeEditCommand.cpp, and DeleteButtonController.cpp
https://bugs.webkit.org/show_bug.cgi?id=52481
Removed instantiation of legacy editing positions.
Calls to Position::Position are replaced by calls to Position's convenience functions.
Also fixed firstPositionInOrBeforeNode and lastPositionInOrAfterNode so that
they instantiate right positions for text nodes.
- accessibility/AccessibilityRenderObject.cpp: (WebCore::AccessibilityRenderObject::setSelectedTextRange):
- dom/Element.cpp: (WebCore::Element::updateFocusAppearance):
- editing/BreakBlockquoteCommand.cpp: (WebCore::BreakBlockquoteCommand::doApply):
- editing/CompositeEditCommand.cpp: (WebCore::CompositeEditCommand::inputText): (WebCore::CompositeEditCommand::rebalanceWhitespaceAt): (WebCore::CompositeEditCommand::moveParagraphContentsToNewBlockIfNecessary): (WebCore::CompositeEditCommand::moveParagraphs): (WebCore::CompositeEditCommand::breakOutOfEmptyListItem): (WebCore::CompositeEditCommand::breakOutOfEmptyMailBlockquotedParagraph): (WebCore::CompositeEditCommand::splitTreeToNode):
- editing/DeleteButtonController.cpp: (WebCore::enclosingDeletableElement):
- editing/htmlediting.h: (WebCore::firstPositionInOrBeforeNode): Instantiates firstPositionInNode instead of positionBeforeNode for a text node. (WebCore::lastPositionInOrAfterNode): Ditto.
- 4:10 PM Changeset in webkit [75834] by
-
- 1 copy in tags/Safari-534.15.8
New tag.
- 3:46 PM Changeset in webkit [75833] by
-
- 5 edits3 adds in trunk
2011-01-14 Helder Correia <helder@sencha.com>
Reviewed by Simon Fraser.
Shadow is not drawn when filling a path with a gradient
https://bugs.webkit.org/show_bug.cgi?id=51982
This happens in CG and is related to bug 51869, this time to be fixed
in GraphicsContext::fillPath(const Path& path). We need to draw the
gradient clipped to the path on a CGLayer first, and then draw the
layer on the GraphicsContext.
- fast/canvas/canvas-fillPath-gradient-shadow-expected.txt: Added.
- fast/canvas/canvas-fillPath-gradient-shadow.html: Added.
- fast/canvas/script-tests/canvas-fillPath-gradient-shadow.js: Added.
- platform/chromium/test_expectations.txt: Skipping new test since it fails.
- platform/qt/Skipped: Ditto.
2011-01-14 Helder Correia <helder@sencha.com>
Reviewed by Simon Fraser.
Shadow is not drawn when filling a path with a gradient
https://bugs.webkit.org/show_bug.cgi?id=51982
This happens in CG and is related to bug 51869, this time to be fixed
in GraphicsContext::fillPath(const Path& path). We need to draw the
gradient clipped to the path on a CGLayer first, and then draw the
layer on the GraphicsContext.
Test: fast/canvas/canvas-fillPath-gradient-shadow.html
- platform/graphics/cg/GraphicsContextCG.cpp: (WebCore::GraphicsContext::fillPath):
- 3:45 PM Changeset in webkit [75832] by
-
- 5 edits in trunk/Source/WebCore
2011-01-14 Simon Fraser <Simon Fraser>
Reviewed by Adam Roben.
Layer syncing should go through the compositor
https://bugs.webkit.org/show_bug.cgi?id=52486
Rather than have FrameView go directly to GraphicsLayer to
sync pending changes, route the call through RenderLayerCompositor.
Add a FIXME about an existing issue with flushing and subframes.
No behavior change, so no tests.
- page/FrameView.cpp: (WebCore::FrameView::syncCompositingStateForThisFrame):
- rendering/RenderLayerBacking.cpp: (WebCore::RenderLayerBacking::notifySyncRequired):
- rendering/RenderLayerCompositor.cpp: (WebCore::RenderLayerCompositor::scheduleLayerFlush): (WebCore::RenderLayerCompositor::flushPendingLayerChanges):
- rendering/RenderLayerCompositor.h: (WebCore::RenderLayerCompositor::notifySyncRequired):
- 3:37 PM Changeset in webkit [75831] by
-
- 2 edits in trunk/Websites/bugs.webkit.org
2011-01-14 Ojan Vafai <ojan@chromium.org>
Fix bad merge in r75816.
- code-review.js:
- 3:36 PM Changeset in webkit [75830] by
-
- 2 edits in trunk/Source/JavaScriptCore
2011-01-14 Patrick Gansterer <Patrick Gansterer>
Reviewed by Adam Roben.
Use the Windows thread pool instead of an extra thread for FastMalloc scavenging
https://bugs.webkit.org/show_bug.cgi?id=45186
r75819 accidentally changed the initial state of the scavenge timer.
- wtf/FastMalloc.cpp: (WTF::TCMalloc_PageHeap::initializeScavenger): Changed initial state of m_scavengingSuspended to true.
- 3:35 PM Changeset in webkit [75829] by
-
- 3 edits in trunk/WebKit/mac
Fix for <rdar://problem/7793902> Artifacts when scrolling
page in Safari while page is loading
Reviewed by Mark Rowe.
Call setDrawsBackground:YES on the scrollView in
dispatchDidFirstLayout() rather than waiting until
frameLoadCompleted(). Also suspend scrolling elasticity
between provisionalLoadStarted() and dispatchDidFirstLayout()
- WebCoreSupport/WebFrameLoaderClient.h:
- WebCoreSupport/WebFrameLoaderClient.mm:
(WebFrameLoaderClient::WebFrameLoaderClient):
(WebFrameLoaderClient::dispatchDidFirstLayout):
(WebFrameLoaderClient::provisionalLoadStarted):
- 3:04 PM Changeset in webkit [75828] by
-
- 248 edits247 copies28 adds in trunk/LayoutTests
2011-01-14 Mihai Parparita <mihaip@chromium.org>
Update the pixel expectations for the chromium-mac port.
For the pixel tests that currently fail on Snow Leopard, this moves the
current pixel results to platform/chromium-mac-leopard and adds new
results to platform/chromium-mac.
List of files omitted due to length but composed entirely of moves from
platform/chromium-mac to platform/chromium-mac-leopard and newly added
files in platform/chromium-mac.
- 2:57 PM Changeset in webkit [75827] by
-
- 2 edits in trunk/Source/JavaScriptCore
2011-01-14 Patrick Gansterer <Patrick Gansterer>
Unreviewed Windows Release build fix.
- wtf/FastMalloc.cpp: (WTF::TCMalloc_PageHeap::scavengerTimerFired):
- 2:57 PM Changeset in webkit [75826] by
-
- 6 edits in trunk
ShouldLoadResourceForFrame should use strings, not URLs.
https://bugs.webkit.org/show_bug.cgi?id=52476
Reviewed by Geoff Garen.
Tools:
- WebKitTestRunner/InjectedBundle/InjectedBundlePage.cpp:
(WTR::InjectedBundlePage::shouldLoadResourceForFrame): Update to use a WKStringRef.
- WebKitTestRunner/InjectedBundle/InjectedBundlePage.h:
WebKit2:
- WebProcess/InjectedBundle/API/c/WKBundlePage.h:
- WebProcess/InjectedBundle/InjectedBundlePageLoaderClient.cpp:
(WebKit::InjectedBundlePageLoaderClient::shouldLoadResourceForFrame): Pass resourceURL as a string instead of
a URL, and rename from subresourceURL to resourceURL.
- 2:49 PM Changeset in webkit [75825] by
-
- 2 edits in trunk/Source/JavaScriptCore
2011-01-14 Patrick Gansterer <Patrick Gansterer>
Unreviewed Windows Release build fix.
- wtf/FastMalloc.cpp: (WTF::TCMalloc_PageHeap::scavengerTimerFired):
- 2:45 PM Changeset in webkit [75824] by
-
- 2 edits in trunk/LayoutTests
2011-01-14 Steve Lacey <sjl@chromium.org>
Reviewed by Simon Fraser.
Update media/video-zoom-controls.html to set the video source before initializing the test.
https://bugs.webkit.org/show_bug.cgi?id=52461
- media/video-zoom-controls.html:
- 2:40 PM Changeset in webkit [75823] by
-
- 3 edits2 adds in trunk
2011-01-14 Abhishek Arya <inferno@chromium.org>
Reviewed by David Hyatt.
Fix parent block calculation when trying to find top most node
containing "this" float.
https://bugs.webkit.org/show_bug.cgi?id=51711
Replace use of containingBlock and traverse the parents directly
to check for float existence. containingBlock can skip parents and
jump to the RenderView directly which will cause floats to not get
cleared from intermediate parents.
Test: fast/block/float/floats-not-cleared-crash.html
- rendering/RenderBox.cpp: (WebCore::RenderBox::removeFloatingOrPositionedChildFromBlockLists):
2011-01-14 Abhishek Arya <inferno@chromium.org>
Reviewed by Dave Hyatt.
Tests that we do not crash due to uncleared floats in parent nodes.
https://bugs.webkit.org/show_bug.cgi?id=51711
- fast/block/float/floats-not-cleared-crash-expected.txt: Added.
- fast/block/float/floats-not-cleared-crash.html: Added.
- 2:39 PM Changeset in webkit [75822] by
-
- 6 edits in trunk/WebKit2
Fix for <rdar://problem/8711576> The web process should be able to read resources from the UI process.
Reviewed by Sam Weinig.
Based on a patch by Ivan Krstić. This also makes the NSURLCache path a sandbox parameter.
- Shared/WebProcessCreationParameters.cpp:
(WebKit::WebProcessCreationParameters::encode):
(WebKit::WebProcessCreationParameters::decode):
Encode and decode the uiProcessBundleResourcePath member variable.
- Shared/WebProcessCreationParameters.h:
Add uiProcessBundleResourcePath member variable.
- UIProcess/mac/WebContextMac.mm:
(WebKit::WebContext::platformInitializeWebProcess):
Set the uiProcessBundleResourcePath member variable.
- WebProcess/com.apple.WebProcess.sb:
Allow read-only access to UI_PROCESS_BUNDLE_RESOURCE_DIR and all subdirectories.
- WebProcess/mac/WebProcessMac.mm:
(WebKit::initializeSandbox):
When initializing the sandbox, pass UI_PROCESS_BUNDLE_RESOURCE_DIR.
(WebKit::WebProcess::platformInitializeWebProcess):
Remove the code that would consume a sandbox extension for the url cache path.
- 2:32 PM Changeset in webkit [75821] by
-
- 2 edits in trunk/LayoutTests
REGRESSION (r75660): compositing/reflections/load-video-in-reflection.html crashes on Windows 7
https://bugs.webkit.org/show_bug.cgi?id=52397
Unskipping tests that no longer fail on Windows 7 Release. Unreviewed.
- platform/win/Skipped:
Un-skip compositing/reflections/load-video-in-reflection.html, which appears to be fixed by
http://trac.webkit.org/changeset/75807
- 2:27 PM Changeset in webkit [75820] by
-
- 1 edit in trunk/Tools/Scripts/webkitpy/common/config/committers.py
Add ericu@chromium.org [me] to the committers file.
- 2:21 PM Changeset in webkit [75819] by
-
- 2 edits in trunk/Source/JavaScriptCore
2011-01-14 Patrick Gansterer <Patrick Gansterer>
Reviewed by Adam Roben.
Use the Windows thread pool instead of an extra thread for FastMalloc scavenging
https://bugs.webkit.org/show_bug.cgi?id=45186
Use CreateTimerQueueTimer() to start periodicScavenge() and stop it with DeleteTimerQueueTimer().
- wtf/FastMalloc.cpp: (WTF::TCMalloc_PageHeap::initializeScavenger): (WTF::TCMalloc_PageHeap::isScavengerSuspended): (WTF::TCMalloc_PageHeap::scheduleScavenger): (WTF::TCMalloc_PageHeap::rescheduleScavenger): (WTF::TCMalloc_PageHeap::suspendScavenger): (WTF::scavengerTimerFired): (WTF::TCMalloc_PageHeap::periodicScavenge): (WTF::TCMalloc_PageHeap::signalScavenger):
- 2:15 PM Changeset in webkit [75818] by
-
- 2 edits in trunk/Tools
2011-01-14 Tony Gentilcore <tonyg@chromium.org>
Reviewed by David Levin.
Add Tony Gentilcore as a reviewer
https://bugs.webkit.org/show_bug.cgi?id=52480
- Scripts/webkitpy/common/config/committers.py:
- 2:13 PM Changeset in webkit [75817] by
-
- 2 edits in trunk/Websites/bugs.webkit.org
2011-01-14 Ojan Vafai <ojan@chromium.org>
Reviewed by Adam Barth.
improve line selection with fast drags
https://bugs.webkit.org/show_bug.cgi?id=52477
- code-review.js:
- 2:08 PM WebKit Team edited by
- (diff)
- 2:00 PM Changeset in webkit [75816] by
-
- 2 edits in trunk/Websites/bugs.webkit.org
2011-01-14 Ojan Vafai <ojan@chromium.org>
Reviewed by Adam Barth.
tweak line selection in code review tool
https://bugs.webkit.org/show_bug.cgi?id=52462
-Improve handling of data-comment-base-line code to deal with
lines that have multiple values.
-Make it so that if you click on the line immediately above
a line that has a comment it will add a new comment instead of
adding lines to the following comment. If the last selected line
overlaps existing comment lines though they will still get
added to the existing comment.
- code-review.js:
- 1:59 PM Changeset in webkit [75815] by
-
- 10 edits2 moves1 delete in trunk
2011-01-14 Adam Klein <adamk@chromium.org>
Reviewed by Darin Fisher.
[Chromium] Replace BackForwardListClient with BackForwardControllerClient/BackForwardList
https://bugs.webkit.org/show_bug.cgi?id=42237
Remove Chromium-specific implementation from WebCore.
No tests added; this refactor should be covered by existing history-exercising tests.
- WebCore.gyp/WebCore.gyp:
- WebCore.gypi:
- history/BackForwardListChromium.cpp: Removed.
- history/BackForwardListImpl.h:
2011-01-14 Adam Klein <adamk@chromium.org>
Reviewed by Darin Fisher.
[Chromium] Replace BackForwardListClient with BackForwardControllerClient/BackForwardList
https://bugs.webkit.org/show_bug.cgi?id=42237
Replace Chromium-specific BackForwardListClient in WebCore with a Chromium impl
of the BackForwardList interface.
- WebKit.gyp:
- src/BackForwardListChromium.cpp: Copied from WebKit/chromium/src/BackForwardListClientImpl.cpp. (WebKit::BackForwardListChromium::create): (WebKit::BackForwardListChromium::BackForwardListChromium): (WebKit::BackForwardListChromium::~BackForwardListChromium): (WebKit::BackForwardListChromium::addItem): (WebKit::BackForwardListChromium::goToItem): (WebKit::BackForwardListChromium::itemAtIndex): (WebKit::BackForwardListChromium::backListCount): (WebKit::BackForwardListChromium::forwardListCount): (WebKit::BackForwardListChromium::isActive): (WebKit::BackForwardListChromium::close):
- src/BackForwardListChromium.h: Copied from WebKit/chromium/src/BackForwardListClientImpl.h.
- src/BackForwardListClientImpl.cpp: Removed.
- src/BackForwardListClientImpl.h: Removed.
- src/FrameLoaderClientImpl.cpp:
- src/WebFrameImpl.cpp: (WebKit::WebFrameImpl::loadHistoryItem): (WebKit::WebFrameImpl::currentHistoryItem):
- src/WebViewImpl.cpp: (WebKit::WebViewImpl::WebViewImpl):
- src/WebViewImpl.h:
- 1:51 PM Changeset in webkit [75814] by
-
- 3 edits in trunk/WebKit2
Delay initializing the sandbox until we get the InitializeWebProcess message.
Reviewed by Sam Weinig.
- WebProcess/mac/WebProcessMac.mm:
(WebKit::initializeSandbox):
(WebKit::WebProcess::platformInitializeWebProcess):
- WebProcess/mac/WebProcessMainMac.mm:
(WebKit::WebProcessMain):
- 1:30 PM Changeset in webkit [75813] by
-
- 3 edits in trunk/Tools
2011-01-14 David Levin <levin@chromium.org>
Reviewed by Eric Seidel.
check-webkit-style: _FunctionState should keep track of where the function name starts.
https://bugs.webkit.org/show_bug.cgi?id=52454
- Scripts/webkitpy/style/checkers/cpp.py: Added function_name_start_position to _FunctionState and logic for setting it correctly.
- Scripts/webkitpy/style/checkers/cpp_unittest.py: Added tests.
- 12:43 PM Changeset in webkit [75812] by
-
- 2 edits in trunk/WebKit/qt
2011-01-14 Konstantin Tokarev <Konstantin Tokarev>
Reviewed by Laszlo Gombos.
[Qt] Fixed compilation when QT_NO_SHORTCUT is defined
https://bugs.webkit.org/show_bug.cgi?id=52460
This patch fixes compilation when QT_NO_SHORTCUT is defined in Qt
configuration.
- WebCoreSupport/EditorClientQt.cpp: (WebCore::EditorClientQt::handleKeyboardEvent): Added QT_NO_SHORTCUT guards
- 12:39 PM Changeset in webkit [75811] by
-
- 7 edits2 adds in trunk
2011-01-12 Satish Sampath <satish@chromium.org>
Reviewed by Dimitri Glazkov.
Fix a crash when accessing speech input from script.
https://bugs.webkit.org/show_bug.cgi?id=52325
Added a layout test to check enabling/disabling speech input from script.
- fast/speech/speech-input-scripting-expected.txt: Added.
- fast/speech/speech-input-scripting.html: Added.
2011-01-12 Satish Sampath <satish@chromium.org>
Reviewed by Dimitri Glazkov.
Fix a crash when accessing speech input from script.
https://bugs.webkit.org/show_bug.cgi?id=52325
Test: fast/speech/speech-input-scripting.html
- html/HTMLInputElement.cpp: (WebCore::HTMLInputElement::parseMappedAttribute): Recreate renderer when speech input is enabled/disabled.
- rendering/RenderTextControlSingleLine.cpp: Remove unused code.
- rendering/RenderTextControlSingleLine.h:
- rendering/TextControlInnerElements.cpp: Take self references before firing events and check for renderer validity after. (WebCore::InputFieldSpeechButtonElement::defaultEventHandler): (WebCore::InputFieldSpeechButtonElement::setRecognitionResult): (WebCore::InputFieldSpeechButtonElement::detach):
- 12:35 PM Changeset in webkit [75810] by
-
- 4 edits2 adds in trunk
2011-01-14 Abhishek Arya <inferno@chromium.org>
Reviewed by David Hyatt.
Prevent merging of anonymous blocks if one of them is already getting
destroyed.
https://bugs.webkit.org/show_bug.cgi?id=52402
Test: fast/block/merge-anonymous-block-remove-child-crash2.html
- rendering/RenderBlock.cpp: (WebCore::RenderBlock::RenderBlock): initialize m_beingDestroyed to false. (WebCore::RenderBlock::destroy): set m_beingDestroyed to true. (WebCore::canMergeContiguousAnonymousBlocks): do not merge if any or prev or next is being destroyed. (WebCore::RenderBlock::removeChild): remove the hack previously done for preventing oldChild merging with nextBlock's next sibling.
- rendering/RenderBlock.h: (WebCore::RenderBlock::beingDestroyed): public function for m_beingDestroyed.
2011-01-14 Abhishek Arya <inferno@chromium.org>
Reviewed by David Hyatt.
Tests that we do not crash when trying to merge anonymous blocks, one of which
is already getting destroyed.
https://bugs.webkit.org/show_bug.cgi?id=52402
- fast/block/merge-anonymous-block-remove-child-crash2-expected.txt: Added.
- fast/block/merge-anonymous-block-remove-child-crash2.html: Added.
- 12:30 PM Changeset in webkit [75809] by
-
- 2 edits in trunk/Websites/bugs.webkit.org
2011-01-14 Ojan Vafai <ojan@chromium.org>
Reviewed by Adam Roben.
fix bugs going back and forth between unified and sidebyside
https://bugs.webkit.org/show_bug.cgi?id=52470
Now that we're appending DOM nodes directly, we only want to
get the first ".text" element in the subtree. In the cases
where there are multiple, their contents are the same.
Also, when making side-by-side expansion lines, we need to
clone the contents so we have a second copy to append to the DOM.
- code-review.js:
- 12:27 PM Changeset in webkit [75808] by
-
- 2 edits in trunk/Source/WebCore
2011-01-14 Pavel Feldman <pfeldman@chromium.org>
Not reviewed. Follow up to r75791: fix missing dispatch.
https://bugs.webkit.org/show_bug.cgi?id=52442
- inspector/front-end/DebuggerModel.js: (WebInspector.DebuggerDispatcher.prototype.breakpointResolved):
- 12:01 PM Changeset in webkit [75807] by
-
- 2 edits in trunk/Source/WebCore
2011-01-14 Simon Fraser <Simon Fraser>
Reviewed by Adam Roben.
Issues with contentsScale in GraphicsLayerCA
https://bugs.webkit.org/show_bug.cgi?id=52463
Fix various issues with the recent contentsScale changes:
- platform/graphics/ca/GraphicsLayerCA.cpp: (WebCore::GraphicsLayerCA::GraphicsLayerCA): Initialize m_contentsScale to 1. (WebCore::GraphicsLayerCA::updateContentsScale): Only do a setNeedsDisplay() if the layer draws content, otherwise we'll create backing store for empty layers. (WebCore::GraphicsLayerCA::swapFromOrToTiledLayer): Be sure to set contentsScale on the new (tiled or untiled) layer. (WebCore::GraphicsLayerCA::cloneLayer): Clones need contentsScale too.
- 11:50 AM Changeset in webkit [75806] by
-
- 4 edits in trunk
2011-01-14 Laszlo Gombos <Laszlo Gombos>
Reviewed by Kenneth Rohde Christiansen.
Align import/export directives
https://bugs.webkit.org/show_bug.cgi?id=52208
- API/JSBase.h: Align import/export directives with WebKit2/Shared/API/c/WKBase.h
2011-01-14 Laszlo Gombos <Laszlo Gombos>
Reviewed by Kenneth Rohde Christiansen.
Align import/export directives
https://bugs.webkit.org/show_bug.cgi?id=52208
- Shared/API/c/WKBase.h: Align import/export directives with JavaScriptCore/API/JSBase.h.
- 11:16 AM Changeset in webkit [75805] by
-
- 4 edits in trunk/WebKit/chromium
2011-01-14 Kent Tamura <tkent@chromium.org>
Reviewed by Darin Fisher.
[Chromium] WebKit API: Add a function for form interactive validation setting
https://bugs.webkit.org/show_bug.cgi?id=52430
- public/WebSettings.h: Add a pure setInteractiveFormValidationEnabled().
- src/WebSettingsImpl.cpp: (WebKit::WebSettingsImpl::setInteractiveFormValidationEnabled): Implemented.
- src/WebSettingsImpl.h: Add a declaration.
- 10:54 AM Changeset in webkit [75804] by
-
- 2 edits in trunk/Source/WebCore
2011-01-14 Simon Fraser <Simon Fraser>
Let Xcode have it's way with the project file.
- WebCore.xcodeproj/project.pbxproj:
- 10:53 AM Changeset in webkit [75803] by
-
- 3 edits in trunk/Source/WebCore
2011-01-14 Simon Fraser <Simon Fraser>
Reviewed by Anders Carlsson.
Refactor some FrameView::syncCompositingState code
https://bugs.webkit.org/show_bug.cgi?id=52459
Refactor some code in FrameView related to synchronizing
compositing layer state.
- page/FrameView.cpp: (WebCore::FrameView::syncCompositingStateForThisFrame): (WebCore::FrameView::syncCompositingStateRecursive): (WebCore::FrameView::paintContents):
- page/FrameView.h:
- 10:38 AM Changeset in webkit [75802] by
-
- 12 edits in trunk/LayoutTests
2011-01-14 Tony Chang <tony@chromium.org>
Unreviewed, updating baselines.
[chromium] rebaseline vertical tests on linux
https://bugs.webkit.org/show_bug.cgi?id=52457
After updating fonts.conf, these tests are now passing.
- platform/chromium-linux/fast/blockflow/japanese-lr-text-expected.checksum:
- platform/chromium-linux/fast/blockflow/japanese-lr-text-expected.png:
- platform/chromium-linux/fast/blockflow/japanese-rl-text-expected.checksum:
- platform/chromium-linux/fast/blockflow/japanese-rl-text-expected.png:
- platform/chromium-linux/fast/repaint/japanese-rl-selection-clear-expected.checksum:
- platform/chromium-linux/fast/repaint/japanese-rl-selection-clear-expected.png:
- platform/chromium-linux/fast/repaint/japanese-rl-selection-repaint-expected.checksum:
- platform/chromium-linux/fast/repaint/japanese-rl-selection-repaint-expected.png:
- platform/chromium-linux/svg/batik/text/verticalText-expected.checksum:
- platform/chromium-linux/svg/batik/text/verticalText-expected.png:
- platform/chromium/test_expectations.txt:
- 10:30 AM Changeset in webkit [75801] by
-
- 2 edits in trunk/Source/WebCore
2011-01-14 Viatcheslav Ostapenko <ostapenko.viatcheslav@nokia.com>
Reviewed by Eric Seidel.
[Qt] Fix build problem introduced by http://trac.webkit.org/changeset/75713
https://bugs.webkit.org/show_bug.cgi?id=30179
- plugins/symbian/PluginViewSymbian.cpp:
- 10:09 AM Changeset in webkit [75800] by
-
- 7 edits in trunk
2011-01-14 Vincent Scheib <scheib@chromium.org>
Reviewed by David Levin.
Add support to DumpRenderTree to use the GPU rather than software renderer
https://bugs.webkit.org/show_bug.cgi?id=52333
- DumpRenderTree/chromium/DumpRenderTree.cpp: (main):
- Scripts/webkitpy/layout_tests/port/chromium.py:
- Scripts/webkitpy/layout_tests/run_webkit_tests.py:
2011-01-14 Vincent Scheib <scheib@chromium.org>
Reviewed by David Levin.
Add support to DumpRenderTree to use the GPU rather than software renderer
https://bugs.webkit.org/show_bug.cgi?id=52333
- DEPS: Moved forward to take required support from webkit_support.h
- WebKit.gyp: DumpRenderTree depends on Angle on windows
- 9:35 AM Changeset in webkit [75799] by
-
- 4 edits in trunk
2011-01-14 Tony Chang <tony@chromium.org>
Reviewed by Kent Tamura.
[chromium] update fonts.conf to fix 6 linux tests in drt
https://bugs.webkit.org/show_bug.cgi?id=52411
- platform/chromium/test_expectations.txt:
2011-01-14 Tony Chang <tony@chromium.org>
Reviewed by Kent Tamura.
[chromium] update fonts.conf to fix 6 linux tests in drt
https://bugs.webkit.org/show_bug.cgi?id=52411
- DumpRenderTree/chromium/fonts.conf: Update to match the copy in
chromium's src/webkit/tools/test_shell/resources/fonts.conf
- 9:27 AM Changeset in webkit [75798] by
-
- 16 edits1 add in trunk
2011-01-14 Philippe Normand <pnormand@igalia.com>
Reviewed by Martin Robinson.
[GTK] Add volume slider to media player
https://bugs.webkit.org/show_bug.cgi?id=51532
- platform/gtk/Skipped:
- platform/gtk/media/audio-controls-rendering-expected.txt:
- platform/gtk/media/controls-after-reload-expected.txt:
- platform/gtk/media/controls-strict-expected.txt:
- platform/gtk/media/controls-styling-expected.txt:
- platform/gtk/media/video-controls-rendering-expected.txt:
- platform/gtk/media/video-display-toggle-expected.txt:
- platform/gtk/media/video-empty-source-expected.txt:
- platform/gtk/media/video-no-audio-expected.txt:
- platform/gtk/media/video-volume-slider-expected.txt: Added.
2011-01-14 Carlos Garcia Campos <cgarcia@igalia.com>
Reviewed by Martin Robinson.
[GTK] Add volume slider to media player
https://bugs.webkit.org/show_bug.cgi?id=51532
- css/mediaControlsGtk.css: (audio::-webkit-media-controls-panel, video::-webkit-media-controls-panel): (audio::-webkit-media-controls-volume-slider-container, video::-webkit-media-controls-volume-slider-container): (audio::-webkit-media-controls-volume-slider, video::-webkit-media-controls-volume-slider):
- platform/gtk/RenderThemeGtk.cpp: (WebCore::RenderThemeGtk::adjustMediaSliderThumbSize): (WebCore::RenderThemeGtk::paintMediaVolumeSliderContainer): (WebCore::RenderThemeGtk::paintMediaVolumeSliderTrack): (WebCore::RenderThemeGtk::paintMediaVolumeSliderThumb):
- platform/gtk/RenderThemeGtk.h:
- platform/gtk/RenderThemeGtk2.cpp: (WebCore::RenderThemeGtk::paintSliderTrack): (WebCore::RenderThemeGtk::paintSliderThumb): (WebCore::RenderThemeGtk::adjustSliderThumbSize):
- platform/gtk/RenderThemeGtk3.cpp: (WebCore::RenderThemeGtk::paintSliderTrack): (WebCore::RenderThemeGtk::paintSliderThumb): (WebCore::RenderThemeGtk::adjustSliderThumbSize):
- 9:03 AM Changeset in webkit [75797] by
-
- 2 edits in trunk/Source/WebCore
2011-01-14 Joone Hur <joone.hur@collabora.co.uk>
Reviewed by David Levin.
[GTK] Convert use of raw pointers to GOwnPtr in FileSystemGtk.cpp
https://bugs.webkit.org/show_bug.cgi?id=52434
No new tests. This code will be tested when an implementation
of beginDragWithFiles is complete.
- platform/gtk/FileSystemGtk.cpp: (WebCore::filenameToString): Use GOwnPtr instead of gchar pointer. (WebCore::fileSystemRepresentation): Ditto. (WebCore::filenameForDisplay): Ditto. (WebCore::pathGetFileName): Ditto.
- 8:02 AM Changeset in webkit [75796] by
-
- 5 edits in trunk
2011-01-14 Michael Saboff <msaboff@apple.com>
Reviewed by Oliver Hunt.
Incorrect backtracking for nested alternatives
https://bugs.webkit.org/show_bug.cgi?id=52387
Added tests to verfiy backtracking of alternatives following
parenthesis.
- fast/regex/parentheses-expected.txt:
- fast/regex/script-tests/parentheses.js:
2011-01-14 Michael Saboff <msaboff@apple.com>
Reviewed by Oliver Hunt.
Incorrect backtracking for nested alternatives
https://bugs.webkit.org/show_bug.cgi?id=52387
In the process of propigating a datalabel it wasn't getting connected
to a destination when the destination was an indirect jump. Added
code to recognize a direct backtrack destination that was an indirect
jump and added mechanism to associate DataLabelPtrs with indirect
jump entries.
Removed dead method
BacktrackDestination::linkDataLabelToHereIfExists()
- yarr/YarrJIT.cpp: (JSC::Yarr::YarrGenerator::IndirectJumpEntry::IndirectJumpEntry): (JSC::Yarr::YarrGenerator::IndirectJumpEntry::addDataLabel): (JSC::Yarr::YarrGenerator::GenerationState::addIndirectJumpEntry): (JSC::Yarr::YarrGenerator::GenerationState::emitIndirectJumpTable): Changes to link indirect jumps with DataLabelPtr's. (JSC::Yarr::YarrGenerator::BacktrackDestination::clearSubDataLabelPtr): (JSC::Yarr::YarrGenerator::TermGenerationState::linkDataLabelToBacktrackIfExists): Updated to handle immediate linking of indirect jumps to DataLabelPtr. (JSC::Yarr::YarrGenerator::generateParenthesesDisjunction): Changed to reflect updated linkDataLabelToBacktrackIfExists().
- 7:58 AM Changeset in webkit [75795] by
-
- 6 edits in trunk/Source/WebCore
2011-01-14 Pavel Podivilov <podivilov@chromium.org>
Reviewed by Yury Semikhatsky.
Web Inspector: breakpoint text snippet in breakpoints sidebar pane disappears after reload.
https://bugs.webkit.org/show_bug.cgi?id=52215
- inspector/front-end/Breakpoint.js: (WebInspector.Breakpoint): (WebInspector.Breakpoint.prototype.populateLabelElement):
- inspector/front-end/Script.js: (WebInspector.Script.prototype.get linesCount): (WebInspector.Script.prototype.sourceLine): (WebInspector.Script.prototype.sourceLine.didRequestSource): (WebInspector.Script.prototype.set source): (WebInspector.Script.prototype.requestSource.didGetScriptSource): (WebInspector.Script.prototype.requestSource):
- inspector/front-end/ScriptView.js: (WebInspector.ScriptView.prototype.setupSourceFrameIfNeeded.didRequestSource): (WebInspector.ScriptView.prototype.setupSourceFrameIfNeeded):
- inspector/front-end/SourceFrame.js: (WebInspector.SourceFrame.prototype._addBreakpoint):
- inspector/front-end/utilities.js: (String.prototype.findAll):
- 7:49 AM Changeset in webkit [75794] by
-
- 13 edits in trunk/Source
2011-01-14 Pavel Podivilov <podivilov@chromium.org>
Reviewed by Yury Semikhatsky.
Web Inspector: provide script column offset to frontend.
https://bugs.webkit.org/show_bug.cgi?id=52377
- parser/SourceCode.h: (JSC::SourceCode::SourceCode): (JSC::SourceCode::firstColumn):
2011-01-14 Pavel Podivilov <podivilov@chromium.org>
Reviewed by Yury Semikhatsky.
Web Inspector: provide script column offset to frontend.
https://bugs.webkit.org/show_bug.cgi?id=52377
- bindings/js/ScriptDebugServer.cpp: (WebCore::ScriptDebugServer::dispatchDidParseSource):
- bindings/js/ScriptSourceCode.h: (WebCore::ScriptSourceCode::ScriptSourceCode):
- bindings/v8/DebuggerScript.js: ():
- bindings/v8/ScriptDebugServer.cpp: (WebCore::ScriptDebugServer::dispatchDidParseSource):
- inspector/Inspector.idl:
- inspector/InspectorDebuggerAgent.cpp: (WebCore::InspectorDebuggerAgent::didParseSource):
- inspector/InspectorDebuggerAgent.h:
- inspector/ScriptDebugListener.h:
- inspector/front-end/DebuggerModel.js: (WebInspector.DebuggerModel.prototype.parsedScriptSource):
- inspector/front-end/Script.js: (WebInspector.Script):
- 7:28 AM Changeset in webkit [75793] by
-
- 2 edits in trunk/Source/WebCore
2011-01-14 Ilya Tikhonovsky <loislo@chromium.org>
Unreviewed one line fix for console-xhr-logging test.
The problem was introduced at r75788.
- inspector/InspectorBrowserDebuggerAgent.cpp: (WebCore::InspectorBrowserDebuggerAgent::willSendXMLHttpRequest):
- 6:57 AM Changeset in webkit [75792] by
-
- 22 edits2 adds in trunk/Source/WebCore
2011-01-13 Yury Semikhatsky <yurys@chromium.org>
Reviewed by Pavel Feldman.
Web Inspector: extract console related functionality into InspectorConsoleAgent
https://bugs.webkit.org/show_bug.cgi?id=52282
- CMakeLists.txt:
- GNUmakefile.am:
- WebCore.gypi:
- WebCore.pro:
- inspector/CodeGeneratorInspector.pm:
- inspector/ConsoleMessage.cpp: (WebCore::ConsoleMessage::~ConsoleMessage):
- inspector/ConsoleMessage.h:
- inspector/Inspector.idl:
- inspector/InspectorConsoleAgent.cpp: Added. (WebCore::InspectorConsoleAgent::InspectorConsoleAgent): (WebCore::InspectorConsoleAgent::~InspectorConsoleAgent): (WebCore::InspectorConsoleAgent::setConsoleMessagesEnabled): (WebCore::InspectorConsoleAgent::clearConsoleMessages): (WebCore::InspectorConsoleAgent::reset): (WebCore::InspectorConsoleAgent::setFrontend): (WebCore::InspectorConsoleAgent::addMessageToConsole): (WebCore::InspectorConsoleAgent::startTiming): (WebCore::InspectorConsoleAgent::stopTiming): (WebCore::InspectorConsoleAgent::count): (WebCore::InspectorConsoleAgent::resourceRetrievedByXMLHttpRequest): (WebCore::InspectorConsoleAgent::didReceiveResponse): (WebCore::InspectorConsoleAgent::didFailLoading): (WebCore::InspectorConsoleAgent::addConsoleMessage):
- inspector/InspectorConsoleAgent.h: Added.
- inspector/InspectorController.cpp: (WebCore::InspectorController::InspectorController): (WebCore::InspectorController::clearConsoleMessages): (WebCore::InspectorController::connectFrontend): (WebCore::InspectorController::disconnectFrontend): (WebCore::InspectorController::didCommitLoad):
- inspector/InspectorController.h: (WebCore::InspectorController::consoleAgent):
- inspector/InspectorInstrumentation.cpp: (WebCore::InspectorInstrumentation::didReceiveResourceResponseImpl): (WebCore::InspectorInstrumentation::didFailLoadingImpl): (WebCore::InspectorInstrumentation::resourceRetrievedByXMLHttpRequestImpl): (WebCore::InspectorInstrumentation::addMessageToConsole): (WebCore::InspectorInstrumentation::count): (WebCore::InspectorInstrumentation::startTiming): (WebCore::InspectorInstrumentation::stopTiming): (WebCore::InspectorInstrumentation::addStartProfilingMessageToConsole):
- inspector/InspectorInstrumentation.h:
- inspector/InspectorProfilerAgent.cpp: (WebCore::InspectorProfilerAgent::addProfileFinishedMessageToConsole): (WebCore::InspectorProfilerAgent::addStartProfilingMessageToConsole):
- inspector/front-end/ConsoleView.js: (WebInspector.ConsoleView): (WebInspector.ConsoleView.prototype.addConsoleMessage): (WebInspector.ConsoleView.prototype.updateConsoleMessageExpiredCount): (WebInspector.ConsoleView.prototype.consoleMessagesCleared):
- inspector/front-end/inspector.js:
- page/Console.cpp: (WebCore::Console::addMessage): (WebCore::Console::count): (WebCore::Console::profile): (WebCore::Console::time): (WebCore::Console::timeEnd): (WebCore::Console::group): (WebCore::Console::groupCollapsed): (WebCore::Console::groupEnd):
- page/Console.h:
- page/Console.idl:
- 6:50 AM Changeset in webkit [75791] by
-
- 11 edits in trunk
2011-01-14 Pavel Feldman <pfeldman@chromium.org>
Web Inspector: do not use this as protocol message
dispatcher in models / agents.
https://bugs.webkit.org/show_bug.cgi?id=52442
Models should never for InspectorBackend.registerDomainDispatcher("foo", this).
There should be a clear way to tell whether method is being called
from within front-end or by the backend.
- http/tests/inspector/debugger-test2.js: (initialize_DebuggerTest.InspectorTest.startDebuggerTest.startTest): (initialize_DebuggerTest.InspectorTest.startDebuggerTest): (initialize_DebuggerTest.InspectorTest.completeDebuggerTest.disableDebugger): (initialize_DebuggerTest.InspectorTest.showScriptSource):
2011-01-14 Pavel Feldman <pfeldman@chromium.org>
Reviewed by Yury Semikhatsky.
Web Inspector: do not use this as protocol message
dispatcher in models / agents.
https://bugs.webkit.org/show_bug.cgi?id=52442
Models should never for InspectorBackend.registerDomainDispatcher("foo", this).
There should be a clear way to tell whether method is being called
from within front-end or by the backend.
- inspector/front-end/DOMAgent.js: (WebInspector.DOMAgent): (WebInspector.DOMAgent.prototype.nodeForId): (WebInspector.DOMAgent.prototype._bindNodes): (WebInspector.DOMAgent.prototype._removeBreakpoints): (WebInspector.DOMDispatcher): (WebInspector.DOMDispatcher.prototype.setDocument): (WebInspector.DOMDispatcher.prototype.attributesUpdated): (WebInspector.DOMDispatcher.prototype.characterDataModified): (WebInspector.DOMDispatcher.prototype.setChildNodes): (WebInspector.DOMDispatcher.prototype.setDetachedRoot): (WebInspector.DOMDispatcher.prototype.childNodeCountUpdated): (WebInspector.DOMDispatcher.prototype.childNodeInserted): (WebInspector.DOMDispatcher.prototype.childNodeRemoved): (WebInspector.DOMDispatcher.prototype.didCommitLoad): (WebInspector.ApplicationCacheDispatcher): (WebInspector.ApplicationCacheDispatcher.prototype.getApplicationCachesAsync): (WebInspector.ApplicationCacheDispatcher.prototype.updateApplicationCacheStatus): (WebInspector.ApplicationCacheDispatcher.prototype.updateNetworkState):
- inspector/front-end/DOMStorage.js: (WebInspector.DOMStorageDispatcher): (WebInspector.DOMStorageDispatcher.prototype.addDOMStorage): (WebInspector.DOMStorageDispatcher.prototype.selectDOMStorage): (WebInspector.DOMStorageDispatcher.prototype.updateDOMStorage):
- inspector/front-end/Database.js: (WebInspector.Database.prototype.executeSql): (WebInspector.DatabaseDispatcher): (WebInspector.DatabaseDispatcher.prototype.addDatabase): (WebInspector.DatabaseDispatcher.prototype.selectDatabase): (WebInspector.DatabaseDispatcher.prototype.sqlTransactionSucceeded): (WebInspector.DatabaseDispatcher.prototype.sqlTransactionFailed):
- inspector/front-end/DebuggerModel.js: (WebInspector.DebuggerModel): (WebInspector.DebuggerModel.prototype._pausedScript): (WebInspector.DebuggerModel.prototype._resumedScript): (WebInspector.DebuggerModel.prototype._parsedScriptSource): (WebInspector.DebuggerModel.prototype._failedToParseScriptSource): (WebInspector.DebuggerDispatcher): (WebInspector.DebuggerDispatcher.prototype.pausedScript): (WebInspector.DebuggerDispatcher.prototype.resumedScript): (WebInspector.DebuggerDispatcher.prototype.parsedScriptSource): (WebInspector.DebuggerDispatcher.prototype.failedToParseScriptSource):
- inspector/front-end/FileSystemView.js: (WebInspector.FileSystemDispatcher): (WebInspector.FileSystemDispatcher.prototype.getFileSystemPathsAsync): (WebInspector.FileSystemDispatcher.prototype.didGetFileSystemPath): (WebInspector.FileSystemDispatcher.prototype.didGetFileSystemError): (WebInspector.FileSystemDispatcher.prototype.didGetFileSystemDisabled):
- inspector/front-end/ProfilesPanel.js: (WebInspector.ProfilesPanel): (WebInspector.ProfilesPanel.prototype._addProfileHeader): (WebInspector.ProfilesPanel.prototype._removeProfileHeader): (WebInspector.ProfilesPanel.prototype._addHeapSnapshotChunk): (WebInspector.ProfilesPanel.prototype._finishHeapSnapshot): (WebInspector.ProfilesPanel.prototype._setRecordingProfile): (WebInspector.ProfilerDispatcher): (WebInspector.ProfilerDispatcher.prototype.profilerWasEnabled): (WebInspector.ProfilerDispatcher.prototype.profilerWasDisabled): (WebInspector.ProfilerDispatcher.prototype.resetProfiles): (WebInspector.ProfilerDispatcher.prototype.addProfileHeader): (WebInspector.ProfilerDispatcher.prototype.addHeapSnapshotChunk): (WebInspector.ProfilerDispatcher.prototype.finishHeapSnapshot): (WebInspector.ProfilerDispatcher.prototype.setRecordingProfile): (WebInspector.ProfileSidebarTreeElement.prototype.ondelete):
- inspector/front-end/ResourceTreeModel.js: (WebInspector.ResourceTreeModel):
- inspector/front-end/TimelinePanel.js: (WebInspector.TimelinePanel): (WebInspector.TimelinePanel.prototype._timelineProfilerWasStarted): (WebInspector.TimelinePanel.prototype._timelineProfilerWasStopped): (WebInspector.TimelinePanel.prototype._addRecordToTimeline): (WebInspector.TimelineDispatcher): (WebInspector.TimelineDispatcher.prototype.timelineProfilerWasStarted): (WebInspector.TimelineDispatcher.prototype.timelineProfilerWasStopped): (WebInspector.TimelineDispatcher.prototype.addRecordToTimeline):
- 6:25 AM Changeset in webkit [75790] by
-
- 2 edits in trunk/Source/WebCore
[Qt][V8] Unreviewed buildfix after r75788.
- inspector/InspectorBrowserDebuggerAgent.cpp:
- 5:31 AM Changeset in webkit [75789] by
-
- 2 edits in trunk/LayoutTests
2011-01-14 Andrey Kosyakov <caseq@chromium.org>
Unreviewed test fix.
Filter out /favicon.ico from page resources list to avoid test expectations mismatch on GTK.
- http/tests/inspector/extensions-resources-redirect.html:
- 4:54 AM Changeset in webkit [75788] by
-
- 14 edits2 adds in trunk/Source/WebCore
2011-01-12 Ilya Tikhonovsky <loislo@chromium.org>
Reviewed by Yury Semikhatsky.
Web Inspector: Extract BrowserDebuggerAgent from InspectorController, InspectorDOMAgent and InspectorDebugger agent.
We have some methods of Debugger which are related to DOM.
Lets extract these methods to BrowserDebugger agent.
- CMakeLists.txt:
- WebCore.gypi:
- WebCore.pro:
- WebCore.vcproj/WebCore.vcproj:
- WebCore.xcodeproj/project.pbxproj:
- inspector/CodeGeneratorInspector.pm:
- inspector/Inspector.idl:
- inspector/InspectorBrowserDebuggerAgent.cpp: Added. (WebCore::InspectorBrowserDebuggerAgent::InspectorBrowserDebuggerAgent): (WebCore::InspectorBrowserDebuggerAgent::~InspectorBrowserDebuggerAgent): (WebCore::InspectorBrowserDebuggerAgent::discardBindings): (WebCore::InspectorBrowserDebuggerAgent::setEventListenerBreakpoint): (WebCore::InspectorBrowserDebuggerAgent::removeEventListenerBreakpoint): (WebCore::InspectorBrowserDebuggerAgent::didInsertDOMNode): (WebCore::InspectorBrowserDebuggerAgent::didRemoveDOMNode): (WebCore::InspectorBrowserDebuggerAgent::setDOMBreakpoint): (WebCore::InspectorBrowserDebuggerAgent::removeDOMBreakpoint): (WebCore::InspectorBrowserDebuggerAgent::willInsertDOMNode): (WebCore::InspectorBrowserDebuggerAgent::willRemoveDOMNode): (WebCore::InspectorBrowserDebuggerAgent::willModifyDOMAttr): (WebCore::InspectorBrowserDebuggerAgent::descriptionForDOMEvent): (WebCore::InspectorBrowserDebuggerAgent::hasBreakpoint): (WebCore::InspectorBrowserDebuggerAgent::updateSubtreeBreakpoints): (WebCore::InspectorBrowserDebuggerAgent::pauseOnNativeEventIfNeeded): (WebCore::InspectorBrowserDebuggerAgent::setXHRBreakpoint): (WebCore::InspectorBrowserDebuggerAgent::removeXHRBreakpoint): (WebCore::InspectorBrowserDebuggerAgent::willSendXMLHttpRequest): (WebCore::InspectorBrowserDebuggerAgent::clearForPageNavigation):
- inspector/InspectorBrowserDebuggerAgent.h: Added. (WebCore::InspectorBrowserDebuggerAgent::create):
- inspector/InspectorController.cpp: (WebCore::InspectorController::InspectorController): (WebCore::InspectorController::inspectedPageDestroyed): (WebCore::InspectorController::didCommitLoad): (WebCore::InspectorController::enableDebuggerFromFrontend): (WebCore::InspectorController::disableDebugger): (WebCore::InspectorController::restoreStickyBreakpoints): (WebCore::InspectorController::restoreStickyBreakpoint):
- inspector/InspectorController.h:
- inspector/InspectorDOMAgent.cpp: (WebCore::InspectorDOMAgent::discardBindings): (WebCore::InspectorDOMAgent::didInsertDOMNode): (WebCore::InspectorDOMAgent::didRemoveDOMNode):
- inspector/InspectorDOMAgent.h:
- inspector/InspectorInstrumentation.cpp: (WebCore::InspectorInstrumentation::willInsertDOMNodeImpl): (WebCore::InspectorInstrumentation::didInsertDOMNodeImpl): (WebCore::InspectorInstrumentation::willRemoveDOMNodeImpl): (WebCore::InspectorInstrumentation::didRemoveDOMNodeImpl): (WebCore::InspectorInstrumentation::willModifyDOMAttrImpl): (WebCore::InspectorInstrumentation::willSendXMLHttpRequestImpl): (WebCore::InspectorInstrumentation::pauseOnNativeEventIfNeeded):
- 4:33 AM Changeset in webkit [75787] by
-
- 4 edits in trunk/Tools
2011-01-14 Eric Seidel <eric@webkit.org>
Original patch from Peter Gal <galpater@inf.u-szeged.hu>
Reviewed by Adam Barth.
Fix when running Tools/Scripts/check-webkit-style without arguments.
https://bugs.webkit.org/show_bug.cgi?id=52261#c16
- Scripts/webkitpy/common/checkout/scm.py: Handle None for changed_files argument better in Git.create_patch method.
- Scripts/webkitpy/common/checkout/scm_unittest.py:
- 4:25 AM Changeset in webkit [75786] by
-
- 18 edits in trunk
2011-01-14 Andrey Kosyakov <caseq@chromium.org>
Reviewed by Pavel Feldman.
Web Inspector: redirected resources not handled properly in Network panel
https://bugs.webkit.org/show_bug.cgi?id=52292
- inspector/Inspector.idl: Do not pass isMainResource to identifierForInitialRequest() (it's useless, as we may hit provisional load)
- inspector/InspectorInstrumentation.cpp: Ditto. (WebCore::InspectorInstrumentation::identifierForInitialRequestImpl):
- inspector/InspectorResourceAgent.cpp: Ditto. (WebCore::InspectorResourceAgent::identifierForInitialRequest):
- inspector/InspectorResourceAgent.h: Ditto.
- inspector/InspectorInstrumentation.cpp: Ditto. (WebCore::InspectorInstrumentation::identifierForInitialRequestImpl): Fix main resource detection.
- inspector/front-end/AuditsPanel.js: WebInspector.networkResources now returns array, not map.
- inspector/front-end/ConsoleView.js: User WebInspector.resourceById() to get resource (WebInspector.ConsoleMessage.prototype._formatMessage):
- inspector/front-end/ExtensionServer.js: Ditto. (WebInspector.ExtensionServer.prototype._onRevealAndSelectResource): (WebInspector.ExtensionServer.prototype._onGetResourceContent):
- inspector/front-end/HAREntry.js: WebInspector.networkResources now returns array, not map (WebInspector.HARLog.prototype.build): (WebInspector.HARLog.prototype._convertResource):
- inspector/front-end/NetworkManager.js: (WebInspector.NetworkManager): Use appendResource, not refreshResource, when adding a new resource. (WebInspector.NetworkManager.prototype.identifierForInitialRequest): (WebInspector.NetworkManager.prototype.willSendRequest): (WebInspector.NetworkManager.prototype.didLoadResourceFromMemoryCache): (WebInspector.NetworkManager.prototype.setInitialContent): (WebInspector.NetworkManager.prototype.didCommitLoadForFrame): (WebInspector.NetworkManager.prototype.didCreateWebSocket): (WebInspector.NetworkManager.prototype._createResource): always register resource URL with ResourceTreeModel (WebInspector.NetworkManager.prototype._appendRedirect):
- inspector/front-end/NetworkPanel.js: (WebInspector.NetworkPanel): (WebInspector.NetworkPanel.prototype.get resources): (WebInspector.NetworkPanel.prototype.resourceById): (WebInspector.NetworkPanel.prototype.appendResource): Add resource as new iff appendResource was added. (WebInspector.NetworkPanel.prototype.refreshResource): ditto. (WebInspector.NetworkPanel.prototype.mainResourceChanged): Clear console upon arrival of new main resource.
- inspector/front-end/ResourceTreeModel.js: Expose unbindResourceURL, bind resources automatically upon creation. (WebInspector.ResourceTreeModel.prototype._clearResources): (WebInspector.ResourceTreeModel.prototype.unbindResourceURL): (WebInspector.ResourceTreeModel.prototype._addFramesRecursively): (WebInspector.ResourceTreeModel.prototype.createResource):
- inspector/front-end/inspector.js: (WebInspector.resourceById):
2011-01-14 Andrey Kosyakov <caseq@chromium.org>
Reviewed by Pavel Feldman.
Web Inspector: redirected resources not handled properly in Network panel
Added test for redirected resource presence in HAR
Changed test to request entire HAR log, not multiple HAR entries.
https://bugs.webkit.org/show_bug.cgi?id=52292
- http/tests/inspector/extensions-resources-redirect-expected.txt:
- http/tests/inspector/extensions-resources-redirect.html:
- http/tests/inspector/resource-har-conversion.html:
- inspector/extensions-resources.html:
- 2:48 AM Changeset in webkit [75785] by
-
- 14 edits2 deletes in trunk/Source/WebCore
2011-01-14 Sheriff Bot <webkit.review.bot@gmail.com>
Unreviewed, rolling out r75783.
http://trac.webkit.org/changeset/75783
https://bugs.webkit.org/show_bug.cgi?id=52439
a bit strange but the same patch have broken mac builds at
linking stage. (Requested by loislo2 on #webkit).
- CMakeLists.txt:
- GNUmakefile.am:
- WebCore.gypi:
- WebCore.pro:
- WebCore.vcproj/WebCore.vcproj:
- WebCore.xcodeproj/project.pbxproj:
- inspector/CodeGeneratorInspector.pm:
- inspector/Inspector.idl:
- inspector/InspectorBrowserDebuggerAgent.cpp: Removed.
- inspector/InspectorBrowserDebuggerAgent.h: Removed.
- inspector/InspectorController.cpp: (WebCore::InspectorController::InspectorController): (WebCore::InspectorController::inspectedPageDestroyed): (WebCore::InspectorController::didCommitLoad): (WebCore::InspectorController::enableDebuggerFromFrontend): (WebCore::InspectorController::disableDebugger): (WebCore::InspectorController::restoreStickyBreakpoints): (WebCore::InspectorController::restoreStickyBreakpoint): (WebCore::InspectorController::setEventListenerBreakpoint): (WebCore::InspectorController::removeEventListenerBreakpoint): (WebCore::InspectorController::hasEventListenerBreakpoint): (WebCore::InspectorController::setXHRBreakpoint): (WebCore::InspectorController::removeXHRBreakpoint): (WebCore::InspectorController::hasXHRBreakpoint):
- inspector/InspectorController.h:
- inspector/InspectorDOMAgent.cpp: (WebCore::InspectorDOMAgent::discardBindings): (WebCore::InspectorDOMAgent::setDOMBreakpoint): (WebCore::InspectorDOMAgent::removeDOMBreakpoint): (WebCore::InspectorDOMAgent::shouldBreakOnNodeInsertion): (WebCore::InspectorDOMAgent::shouldBreakOnNodeRemoval): (WebCore::InspectorDOMAgent::shouldBreakOnAttributeModification): (WebCore::InspectorDOMAgent::descriptionForDOMEvent): (WebCore::InspectorDOMAgent::didInsertDOMNode): (WebCore::InspectorDOMAgent::didRemoveDOMNode): (WebCore::InspectorDOMAgent::hasBreakpoint): (WebCore::InspectorDOMAgent::updateSubtreeBreakpoints):
- inspector/InspectorDOMAgent.h:
- inspector/InspectorInstrumentation.cpp: (WebCore::InspectorInstrumentation::willInsertDOMNodeImpl): (WebCore::InspectorInstrumentation::didInsertDOMNodeImpl): (WebCore::InspectorInstrumentation::willRemoveDOMNodeImpl): (WebCore::InspectorInstrumentation::didRemoveDOMNodeImpl): (WebCore::InspectorInstrumentation::willModifyDOMAttrImpl): (WebCore::InspectorInstrumentation::willSendXMLHttpRequestImpl): (WebCore::InspectorInstrumentation::pauseOnNativeEventIfNeeded):
- 2:24 AM Changeset in webkit [75784] by
-
- 2 edits in trunk/WebKit/chromium
2011-01-14 Finnur Thorarinsson <finnur.webkit@gmail.com>
Reviewed by Ojan Vafai.
Address some user-select-none issues better in our Find in page scoping
function.
The scoping function is in part based on Editor::countMatchesForText
(formerly Frame::markAllMatchesForText), but was rewritten with
asychronous search and interruptability in mind. At the time the
function was written, countMatchesForText didn't work well with
user-select-none style but that has now changed. While
investigating http://crbug.com/68494 I noticed WebKit had fixed this
very problem in the countMatchesForText and I believe we should make
the same changes in our platform code. Therefore, this changelist
adopts the same approach by integrating:
https://bugs.webkit.org/show_bug.cgi?id=33508
(r53142: handling user-select-none better) and
https://bugs.webkit.org/show_bug.cgi?id=51623
(r74886: handling match within textfield better).
For more details see those changelists.
This changelist, however, is tracked here:
https://bugs.webkit.org/show_bug.cgi?id=52367
The original changelists did not come with layout tests, but I'm adding
a test on the Chromium side for this.
- src/WebFrameImpl.cpp: (WebKit::WebFrameImpl::scopeStringMatches):
- 2:22 AM Changeset in webkit [75783] by
-
- 14 edits2 adds in trunk/Source/WebCore
2011-01-12 Ilya Tikhonovsky <loislo@chromium.org>
Reviewed by Yury Semikhatsky.
Web Inspector: Extract BrowserDebuggerAgent from InspectorController, InspectorDOMAgent and InspectorDebugger agent.
We have some methods of Debugger which are related to DOM.
Lets extract these methods to BrowserDebugger agent.
- CMakeLists.txt:
- WebCore.gypi:
- WebCore.pro:
- WebCore.vcproj/WebCore.vcproj:
- WebCore.xcodeproj/project.pbxproj:
- inspector/CodeGeneratorInspector.pm:
- inspector/Inspector.idl:
- inspector/InspectorBrowserDebuggerAgent.cpp: Added. (WebCore::InspectorBrowserDebuggerAgent::InspectorBrowserDebuggerAgent): (WebCore::InspectorBrowserDebuggerAgent::~InspectorBrowserDebuggerAgent): (WebCore::InspectorBrowserDebuggerAgent::discardBindings): (WebCore::InspectorBrowserDebuggerAgent::setEventListenerBreakpoint): (WebCore::InspectorBrowserDebuggerAgent::removeEventListenerBreakpoint): (WebCore::InspectorBrowserDebuggerAgent::didInsertDOMNode): (WebCore::InspectorBrowserDebuggerAgent::didRemoveDOMNode): (WebCore::InspectorBrowserDebuggerAgent::setDOMBreakpoint): (WebCore::InspectorBrowserDebuggerAgent::removeDOMBreakpoint): (WebCore::InspectorBrowserDebuggerAgent::willInsertDOMNode): (WebCore::InspectorBrowserDebuggerAgent::willRemoveDOMNode): (WebCore::InspectorBrowserDebuggerAgent::willModifyDOMAttr): (WebCore::InspectorBrowserDebuggerAgent::descriptionForDOMEvent): (WebCore::InspectorBrowserDebuggerAgent::hasBreakpoint): (WebCore::InspectorBrowserDebuggerAgent::updateSubtreeBreakpoints): (WebCore::InspectorBrowserDebuggerAgent::pauseOnNativeEventIfNeeded): (WebCore::InspectorBrowserDebuggerAgent::setXHRBreakpoint): (WebCore::InspectorBrowserDebuggerAgent::removeXHRBreakpoint): (WebCore::InspectorBrowserDebuggerAgent::willSendXMLHttpRequest): (WebCore::InspectorBrowserDebuggerAgent::clearForPageNavigation):
- inspector/InspectorBrowserDebuggerAgent.h: Added. (WebCore::InspectorBrowserDebuggerAgent::create):
- inspector/InspectorController.cpp: (WebCore::InspectorController::InspectorController): (WebCore::InspectorController::inspectedPageDestroyed): (WebCore::InspectorController::didCommitLoad): (WebCore::InspectorController::enableDebuggerFromFrontend): (WebCore::InspectorController::disableDebugger): (WebCore::InspectorController::restoreStickyBreakpoints): (WebCore::InspectorController::restoreStickyBreakpoint):
- inspector/InspectorController.h:
- inspector/InspectorDOMAgent.cpp: (WebCore::InspectorDOMAgent::discardBindings): (WebCore::InspectorDOMAgent::didInsertDOMNode): (WebCore::InspectorDOMAgent::didRemoveDOMNode):
- inspector/InspectorDOMAgent.h:
- inspector/InspectorInstrumentation.cpp: (WebCore::InspectorInstrumentation::willInsertDOMNodeImpl): (WebCore::InspectorInstrumentation::didInsertDOMNodeImpl): (WebCore::InspectorInstrumentation::willRemoveDOMNodeImpl): (WebCore::InspectorInstrumentation::didRemoveDOMNodeImpl): (WebCore::InspectorInstrumentation::willModifyDOMAttrImpl): (WebCore::InspectorInstrumentation::willSendXMLHttpRequestImpl): (WebCore::InspectorInstrumentation::pauseOnNativeEventIfNeeded):
- 2:05 AM Changeset in webkit [75782] by
-
- 2 edits in trunk/WebKit/gtk
2011-01-14 Pavel Feldman <pfeldman@chromium.org>
Not reviewed: gtk build fix (2).
- webkit/webkitwebview.cpp:
- 2:02 AM Changeset in webkit [75781] by
-
- 3 edits in trunk/Tools
2011-01-04 Jochen Eisinger <jochen@chromium.org>
Reviewed by David Levin.
Store the tempdir object in a variable so it doesn't get deleted too early.
https://bugs.webkit.org/show_bug.cgi?id=51875
- Scripts/update-webkit-auxiliary-libs:
- Scripts/update-webkit-support-libs:
- 1:59 AM Changeset in webkit [75780] by
-
- 2 edits in trunk/WebKit/gtk
2011-01-14 Pavel Feldman <pfeldman@chromium.org>
Not reviewed: gtk build fix.
- webkit/webkitwebinspector.cpp:
- 1:52 AM Changeset in webkit [75779] by
-
- 2 edits in trunk/Tools
2011-01-14 Eric Seidel <eric@webkit.org>
Unreviewed. Make it work in python 2.5.
webkit-patch suggest-reviewers dies when ChangeLogs are missing
https://bugs.webkit.org/show_bug.cgi?id=49158
- Scripts/webkitpy/common/checkout/api_unittest.py:
- 1:47 AM Changeset in webkit [75778] by
-
- 4 edits in trunk
2011-01-14 Pavel Feldman <pfeldman@chromium.org>
Not reviewed: build fix.
- WebCore.vcproj/WebCore.vcproj:
- 1:28 AM Changeset in webkit [75777] by
-
- 13 edits2 adds in trunk
2011-01-14 Pavel Feldman <pfeldman@chromium.org>
Reviewed by Yury Semikhatsky.
Web Inspector: extract InspectorSettings from InspectorState,
simplify it.
https://bugs.webkit.org/show_bug.cgi?id=52429
- CMakeLists.txt:
- GNUmakefile.am:
- WebCore.gypi:
- WebCore.pro:
- WebCore.xcodeproj/project.pbxproj:
- inspector/InspectorClient.h:
- inspector/InspectorController.cpp: (WebCore::InspectorController::InspectorController): (WebCore::InspectorController::inspectorStartsAttached): (WebCore::InspectorController::setInspectorStartsAttached): (WebCore::InspectorController::setInspectorAttachedHeight): (WebCore::InspectorController::inspectorAttachedHeight): (WebCore::InspectorController::setMonitoringXHREnabled): (WebCore::InspectorController::restoreDebugger): (WebCore::InspectorController::restoreProfiler): (WebCore::InspectorController::ensureSettingsLoaded): (WebCore::InspectorController::enableProfiler): (WebCore::InspectorController::disableProfiler): (WebCore::InspectorController::enableDebuggerFromFrontend): (WebCore::InspectorController::disableDebugger):
- inspector/InspectorController.h:
- inspector/InspectorSettings.cpp: Added. (WebCore::InspectorSettings::InspectorSettings): (WebCore::InspectorSettings::getBoolean): (WebCore::InspectorSettings::setBoolean): (WebCore::InspectorSettings::getLong): (WebCore::InspectorSettings::setLong): (WebCore::InspectorSettings::registerBoolean): (WebCore::InspectorSettings::registerLong):
- inspector/InspectorSettings.h: Added.
- inspector/InspectorState.cpp: (WebCore::InspectorState::InspectorState): (WebCore::InspectorState::setValue): (WebCore::InspectorState::setObject): (WebCore::InspectorState::registerBoolean): (WebCore::InspectorState::registerString): (WebCore::InspectorState::registerLong): (WebCore::InspectorState::registerObject): (WebCore::InspectorState::Property::create):
- inspector/InspectorState.h: (WebCore::InspectorState::setBoolean): (WebCore::InspectorState::setString): (WebCore::InspectorState::setLong):
2011-01-14 Pavel Feldman <pfeldman@chromium.org>
Reviewed by Yury Semikhatsky.
Web Inspector: extract InspectorSettings from InspectorState,
simplify it.
https://bugs.webkit.org/show_bug.cgi?id=52429
- WebProcess/WebCoreSupport/WebInspectorClient.cpp:
- 1:21 AM Changeset in webkit [75776] by
-
- 15 edits2 deletes in trunk
2011-01-14 Sheriff Bot <webkit.review.bot@gmail.com>
Unreviewed, rolling out r75774.
http://trac.webkit.org/changeset/75774
https://bugs.webkit.org/show_bug.cgi?id=52431
gtk builds were broken (Requested by loislo2 on #webkit).
- CMakeLists.txt:
- WebCore.gypi:
- WebCore.pro:
- WebCore.vcproj/WebCore.vcproj:
- WebCore.xcodeproj/project.pbxproj:
- inspector/CodeGeneratorInspector.pm:
- inspector/Inspector.idl:
- inspector/InspectorBrowserDebuggerAgent.cpp: Removed.
- inspector/InspectorBrowserDebuggerAgent.h: Removed.
- inspector/InspectorController.cpp: (WebCore::InspectorController::InspectorController): (WebCore::InspectorController::inspectedPageDestroyed): (WebCore::InspectorController::didCommitLoad): (WebCore::InspectorController::enableDebuggerFromFrontend): (WebCore::InspectorController::disableDebugger): (WebCore::InspectorController::restoreStickyBreakpoints): (WebCore::InspectorController::restoreStickyBreakpoint): (WebCore::InspectorController::setEventListenerBreakpoint): (WebCore::InspectorController::removeEventListenerBreakpoint): (WebCore::InspectorController::hasEventListenerBreakpoint): (WebCore::InspectorController::setXHRBreakpoint): (WebCore::InspectorController::removeXHRBreakpoint): (WebCore::InspectorController::hasXHRBreakpoint):
- inspector/InspectorController.h:
- inspector/InspectorDOMAgent.cpp: (WebCore::InspectorDOMAgent::discardBindings): (WebCore::InspectorDOMAgent::setDOMBreakpoint): (WebCore::InspectorDOMAgent::removeDOMBreakpoint): (WebCore::InspectorDOMAgent::shouldBreakOnNodeInsertion): (WebCore::InspectorDOMAgent::shouldBreakOnNodeRemoval): (WebCore::InspectorDOMAgent::shouldBreakOnAttributeModification): (WebCore::InspectorDOMAgent::descriptionForDOMEvent): (WebCore::InspectorDOMAgent::didInsertDOMNode): (WebCore::InspectorDOMAgent::didRemoveDOMNode): (WebCore::InspectorDOMAgent::hasBreakpoint): (WebCore::InspectorDOMAgent::updateSubtreeBreakpoints):
- inspector/InspectorDOMAgent.h:
- inspector/InspectorInstrumentation.cpp: (WebCore::InspectorInstrumentation::willInsertDOMNodeImpl): (WebCore::InspectorInstrumentation::didInsertDOMNodeImpl): (WebCore::InspectorInstrumentation::willRemoveDOMNodeImpl): (WebCore::InspectorInstrumentation::didRemoveDOMNodeImpl): (WebCore::InspectorInstrumentation::willModifyDOMAttrImpl): (WebCore::InspectorInstrumentation::willSendXMLHttpRequestImpl): (WebCore::InspectorInstrumentation::pauseOnNativeEventIfNeeded):
2011-01-14 Sheriff Bot <webkit.review.bot@gmail.com>
Unreviewed, rolling out r75774.
http://trac.webkit.org/changeset/75774
https://bugs.webkit.org/show_bug.cgi?id=52431
gtk builds were broken (Requested by loislo2 on #webkit).
- Scripts/webkitpy/common/checkout/scm.py:
- 12:56 AM Changeset in webkit [75775] by
-
- 3 edits in trunk/Tools
check-webkit-style: _FunctionState should detect pure functions.
https://bugs.webkit.org/show_bug.cgi?id=52428
Reviewed by Shinichiro Hamaji.
- Scripts/webkitpy/style/checkers/cpp.py: Added detection for if a function declaration
is a pure virtual function.
- Scripts/webkitpy/style/checkers/cpp_unittest.py: Added tests for verifying that pure
virtual functions are correctly detected.
- 12:21 AM Changeset in webkit [75774] by
-
- 15 edits2 adds in trunk
2011-01-12 Ilya Tikhonovsky <loislo@chromium.org>
Reviewed by Yury Semikhatsky.
Web Inspector: Extract BrowserDebuggerAgent from InspectorController, InspectorDOMAgent and InspectorDebugger agent.
We have some methods of Debugger which are related to DOM.
Lets extract these methods to BrowserDebugger agent.
- CMakeLists.txt:
- WebCore.gypi:
- WebCore.pro:
- WebCore.vcproj/WebCore.vcproj:
- WebCore.xcodeproj/project.pbxproj:
- inspector/CodeGeneratorInspector.pm:
- inspector/Inspector.idl:
- inspector/InspectorBrowserDebuggerAgent.cpp: Added. (WebCore::InspectorBrowserDebuggerAgent::InspectorBrowserDebuggerAgent): (WebCore::InspectorBrowserDebuggerAgent::~InspectorBrowserDebuggerAgent): (WebCore::InspectorBrowserDebuggerAgent::discardBindings): (WebCore::InspectorBrowserDebuggerAgent::setEventListenerBreakpoint): (WebCore::InspectorBrowserDebuggerAgent::removeEventListenerBreakpoint): (WebCore::InspectorBrowserDebuggerAgent::didInsertDOMNode): (WebCore::InspectorBrowserDebuggerAgent::didRemoveDOMNode): (WebCore::InspectorBrowserDebuggerAgent::setDOMBreakpoint): (WebCore::InspectorBrowserDebuggerAgent::removeDOMBreakpoint): (WebCore::InspectorBrowserDebuggerAgent::willInsertDOMNode): (WebCore::InspectorBrowserDebuggerAgent::willRemoveDOMNode): (WebCore::InspectorBrowserDebuggerAgent::willModifyDOMAttr): (WebCore::InspectorBrowserDebuggerAgent::descriptionForDOMEvent): (WebCore::InspectorBrowserDebuggerAgent::hasBreakpoint): (WebCore::InspectorBrowserDebuggerAgent::updateSubtreeBreakpoints): (WebCore::InspectorBrowserDebuggerAgent::pauseOnNativeEventIfNeeded): (WebCore::InspectorBrowserDebuggerAgent::setXHRBreakpoint): (WebCore::InspectorBrowserDebuggerAgent::removeXHRBreakpoint): (WebCore::InspectorBrowserDebuggerAgent::willSendXMLHttpRequest): (WebCore::InspectorBrowserDebuggerAgent::clearForPageNavigation):
- inspector/InspectorBrowserDebuggerAgent.h: Added. (WebCore::InspectorBrowserDebuggerAgent::create):
- inspector/InspectorController.cpp: (WebCore::InspectorController::InspectorController): (WebCore::InspectorController::inspectedPageDestroyed): (WebCore::InspectorController::didCommitLoad): (WebCore::InspectorController::enableDebuggerFromFrontend): (WebCore::InspectorController::disableDebugger): (WebCore::InspectorController::restoreStickyBreakpoints): (WebCore::InspectorController::restoreStickyBreakpoint):
- inspector/InspectorController.h:
- inspector/InspectorDOMAgent.cpp: (WebCore::InspectorDOMAgent::discardBindings): (WebCore::InspectorDOMAgent::didInsertDOMNode): (WebCore::InspectorDOMAgent::didRemoveDOMNode):
- inspector/InspectorDOMAgent.h:
- inspector/InspectorInstrumentation.cpp: (WebCore::InspectorInstrumentation::willInsertDOMNodeImpl): (WebCore::InspectorInstrumentation::didInsertDOMNodeImpl): (WebCore::InspectorInstrumentation::willRemoveDOMNodeImpl): (WebCore::InspectorInstrumentation::didRemoveDOMNodeImpl): (WebCore::InspectorInstrumentation::willModifyDOMAttrImpl): (WebCore::InspectorInstrumentation::willSendXMLHttpRequestImpl): (WebCore::InspectorInstrumentation::pauseOnNativeEventIfNeeded):
2011-01-13 Ilya Tikhonovsky <loislo@chromium.org>
Reviewed by Yury Semikhatsky.
Just small fix for problem with check-webkit-style.
It just didn't work for this patch.
Old code can't concatenate a list of strings with None.
- Scripts/webkitpy/common/checkout/scm.py:
- 12:14 AM Changeset in webkit [75773] by
-
- 2 edits in trunk/LayoutTests
2011-01-14 Hayato Ito <hayato@chromium.org>
Unreviewed chromium expectations update.
Exclude http/tests/loading/cross-origin-XHR-willLoadRequest.html (added in r75768)
- platform/chromium/test_expectations.txt: