Timeline
Aug 14, 2011:
- 11:57 PM Changeset in webkit [93033] by
-
- 2 edits in trunk/LayoutTests
[QT]REGRESSION after r93011
https://bugs.webkit.org/show_bug.cgi?id=66212
Unreviewed gardening.
- platform/qt-4.8/Skipped:
- 4:08 PM Changeset in webkit [93032] by
-
- 3 edits2 adds in trunk
Source/WebCore: Crash in HTMLTreeBuilder::processAnyOtherEndTagForInBody
https://bugs.webkit.org/show_bug.cgi?id=66187
Reviewed by Adam Barth.
RefPtr a few ContainerNodes to prevent premature deletion.
Test: fast/html/process-end-tag-for-inbody-crash.html
- html/parser/HTMLTreeBuilder.cpp:
(WebCore::HTMLTreeBuilder::processCloseWhenNestedTag):
(WebCore::HTMLTreeBuilder::processAnyOtherEndTagForInBody):
(WebCore::HTMLTreeBuilder::callTheAdoptionAgency):
LayoutTests: Crash in HTMLTreeBuilder::processAnyOtherEndTagForInBody.
https://bugs.webkit.org/show_bug.cgi?id=66187
Reviewed by Adam Barth.
- fast/html/process-end-tag-for-inbody-crash-expected.txt: Added.
- fast/html/process-end-tag-for-inbody-crash.html: Added.
- 10:54 AM Changeset in webkit [93031] by
-
- 1 copy in tags/Safari-534.51.15
New tag.
- 5:21 AM Changeset in webkit [93030] by
-
- 2 edits in trunk/Source/WebCore
Fix GTK Windows build after r92308.
https://bugs.webkit.org/show_bug.cgi?id=66140
Patch by Kalev Lember <kalevlember@gmail.com> on 2011-08-14
Reviewed by Xan Lopez.
- config.h: Don't try to include WebCoreHeaderDetection.h for GTK.
- 12:02 AM Changeset in webkit [93029] by
-
- 7 edits2 adds in trunk/Tools
Switch results detail view over to new-style object-oriented UI widgets
https://bugs.webkit.org/show_bug.cgi?id=66200
Reviewed by Dimitri Glazkov.
This patch replaces my goofy template-based UI for the results
comparison screen with new object-oriented UI widgets.
- BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/garden-o-matic.html:
- BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/main.css:
- BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/main.js:
- BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/run-unittests.html:
- BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/ui.js:
- BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/ui/results.js: Added.
- BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/ui_unittests.js:
Aug 13, 2011:
- 11:55 PM Changeset in webkit [93028] by
-
- 30 edits in trunk/LayoutTests/platform/mac
Set the svn:mime-type property of some PNG files to image/png
- 1:38 PM Changeset in webkit [93027] by
-
- 4 edits in trunk/Source/WebKit
Changed the value of WebKitShowsToolTipOverTruncatedTextPreferenceKey to match the naming everywhere else.
Reviewed by Joseph Pecoraro.
../WebKit/mac:
- WebView/WebPreferenceKeysPrivate.h:
../WebKit/win:
- WebPreferenceKeysPrivate.h:
- 12:27 PM Changeset in webkit [93026] by
-
- 2 edits in trunk/Source/WebCore
Remove forward declaration of -[WebUndefined dealloc] to appease the bot gods.
- bindings/objc/WebScriptObject.mm:
- 12:25 PM Changeset in webkit [93025] by
-
- 2 edits in trunk/Source/WebCore
Remove assertion from -[WebUndefined dealloc] to try to mollify the bots.
- bindings/objc/WebScriptObject.mm:
(-[WebUndefined dealloc]):
- 12:12 PM Changeset in webkit [93024] by
-
- 2 edits in trunk/Source/WebKit/mac
Remove unused variables from WebView/WebFullScreenController.h
https://bugs.webkit.org/show_bug.cgi?id=66194
Reviewed by Dan Bernstein.
- WebView/WebFullScreenController.h:
Remove a few variables which were not used.
- 12:03 PM Changeset in webkit [93023] by
-
- 2 edits in trunk/Source/WebKit2
Remove unused variables from WKFullScreenWindowController.h
https://bugs.webkit.org/show_bug.cgi?id=66193
Reviewed by Dan Bernstein.
- UIProcess/mac/WKFullScreenWindowController.h:
Remove _layerViewPlaceholder, _isWindowLoaded and _initialFrame which were not used.
- 11:57 AM Changeset in webkit [93022] by
-
- 2 edits in trunk/Source/WebCore
Remove unused variables from WebVideoFullscreenController.h
https://bugs.webkit.org/show_bug.cgi?id=66192
Reviewed by Dan Bernstein.
- platform/mac/WebVideoFullscreenController.h:
Remove _isWindowLoaded, _savedUIMode and _savedUIOptions which were not used.
- 11:51 AM Changeset in webkit [93021] by
-
- 3 edits in trunk/Source/WebCore
Fix incorrect objective-c initialize in WebCore
https://bugs.webkit.org/show_bug.cgi?id=66191
Reviewed by David Kilzer.
- accessibility/mac/AccessibilityObjectWrapper.mm:
(-[AccessibilityObjectWrapper initWithAccessibilityObject:]):
- rendering/RenderThemeMac.mm:
(-[WebCoreRenderThemeNotificationObserver initWithTheme:WebCore::]):
Correctly initialize by assigning to self and nil checking the result.
- 11:31 AM Changeset in webkit [93020] by
-
- 2 edits in trunk/Source/WebCore
Fix clang build.
- platform/chromium/PopupMenuChromium.h:
- 11:06 AM Changeset in webkit [93019] by
-
- 2 edits in trunk/Source/WebCore
<http://webkit.org/b/66188> WebCore.xcodeproj has duplicate entries again
Reviewed by Dan Bernstein.
- WebCore.xcodeproj/project.pbxproj: Remove duplicate entries
by using uniq. Originally noticed by Xcode 4.
- 10:54 AM Changeset in webkit [93018] by
-
- 2 edits in trunk/Source/JavaScriptCore
Add back 0xbbadbeef to CRASH to allow for old habits
https://bugs.webkit.org/show_bug.cgi?id=66190
Reviewed by David Kilzer.
- wtf/Assertions.h:
Add back the assignment to the memory address 0xbbadbeef in the CRASH
macro, as it does not cause issue in the clang static analyzer and many
people use its presence in crash reports to easily identify ASSERTs.
- 10:48 AM Changeset in webkit [93017] by
-
- 4 edits in trunk/Source/JavaScriptCore
Fix a bunch of minor bugs caught by the clang static analyzer in JavaScriptCore
https://bugs.webkit.org/show_bug.cgi?id=66182
Reviewed by Dan Bernstein.
Fixes 10 warnings in JavaScriptCore and 2 in testapi.
- API/tests/testapi.c:
(main):
Remove dead variables.
- dfg/DFGGraph.cpp:
(JSC::DFG::Graph::dump):
Initialize hasPrinted and silence an unused warning by casting to void (Ok here
since it is debug code and I want to keep it clear that if other cases are added,
the hasPrinted flag would be needed).
- wtf/dtoa.cpp:
(WTF::d2b):
The variable "de" in the else block is always zero, so there is no reason to
use it.
- 8:39 AM Changeset in webkit [93016] by
-
- 5 edits in trunk/Tools
garden-o-matic's analyzeUnexpectedFailures needs a completion callback.
https://bugs.webkit.org/show_bug.cgi?id=66166
Also changed base.RequestTracker to:
a) fire callback immediately if requestsInFlight is 0;
b) not barf if callback is not supplied.
Reviewed by Adam Barth.
- BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/main.js: Started using completion callback.
- BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/model.js: Added completion callback.
- BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/base.js: Changed RequestTracker.
- BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/base_unittest.js: Added unit tests.
- 12:00 AM Changeset in webkit [93015] by
-
- 2 edits in trunk/Source/WebKit/win
Build fix.
- WebView.cpp:
(WebView::notifyPreferencesChanged):
Aug 12, 2011:
- 11:25 PM Changeset in webkit [93014] by
-
- 25 edits in trunk/Source
Source/WebCore: <rdar://problem/7337717> Add an option to automatically show tooltips (with the full text) over truncated text
https://bugs.webkit.org/show_bug.cgi?id=66178
Reviewed by Simon Fraser.
- WebCore.exp.in: Export setShowsToolTipOverTruncatedText().
- page/Chrome.cpp:
(WebCore::Chrome::setToolTip): If no title is found, and the page is set to show tooltips over
truncated text, try to set the tooltip to the full text of the truncated text, if any.
- page/Settings.cpp:
(WebCore::Settings::Settings): Initialize new member variable.
(WebCore::Settings::setShowsToolTipOverTruncatedText): Added this setter.
- page/Settings.h:
(WebCore::Settings::showsToolTipOverTruncatedText): Added this getter.
- rendering/HitTestResult.cpp:
(WebCore::HitTestResult::innerTextIfTruncated): Added. If the inner node or its nearest enclosing
block has text-overflow: ellipsis and has truncated lines, return the node’s (full) inner text.
- rendering/HitTestResult.h:
- rendering/RootInlineBox.h:
(WebCore::RootInlineBox::hasEllipsisBox): Made this public.
Source/WebKit/mac: Add an option to automatically show tooltips (with the full text) over truncated text
https://bugs.webkit.org/show_bug.cgi?id=66178
Reviewed by Simon Fraser.
- WebView/WebPreferenceKeysPrivate.h: Defined a preference key.
- WebView/WebPreferences.mm:
(+[WebPreferences initialize]): Added a default value of NO for the
ShowsToolTipOverTruncatedText preference.
(-[WebPreferences showsToolTipOverTruncatedText]): Added this getter.
(-[WebPreferences setShowsToolTipOverTruncatedText:]): Added this setter.
- WebView/WebPreferencesPrivate.h:
- WebView/WebView.mm:
(-[WebView _preferencesChanged:]): Added call to Settings::setShowsToolTipOverTruncatedText().
Source/WebKit/win: Add an option to automatically show tooltips (with the full text) over truncated text
https://bugs.webkit.org/show_bug.cgi?id=66178
Reviewed by Simon Fraser.
- Interfaces/IWebPreferencesPrivate.idl: Added accessors.
- WebPreferenceKeysPrivate.h: Defined a preference key.
- WebPreferences.cpp:
(WebPreferences::initializeDefaultSettings): Added a default value of
false for the ShowsToolTipOverTruncatedText preference.
(WebPreferences::showsToolTipOverTruncatedText): Added this getter.
(WebPreferences::setShowsToolTipOverTruncatedText): Added this setter.
- WebPreferences.h:
- WebView.cpp:
(WebView::notifyPreferencesChanged): Added call to Settings::setShowsToolTipOverTruncatedText().
Source/WebKit2: <rdar://problem/7337717> Add an option to automatically show tooltips (with the full text) over truncated text
https://bugs.webkit.org/show_bug.cgi?id=66178
Reviewed by Simon Fraser.
- Shared/WebPreferencesStore.h:
- UIProcess/API/C/WKPreferences.cpp:
(WKPreferencesSetShowsToolTipOverTruncatedText): Added this setter.
(WKPreferencesGetShowsToolTipOverTruncatedText): Added this getter.
- UIProcess/API/C/WKPreferencesPrivate.h:
- WebProcess/WebPage/WebPage.cpp:
(WebKit::WebPage::updatePreferences): Handle the showsToolTipOverTruncatedText preference.
- 8:26 PM Changeset in webkit [93013] by
-
- 11 edits in trunk/Source/WebCore
Ownership of canvas's GraphicsContext3D should be moved to PlatformContextSkia
https://bugs.webkit.org/show_bug.cgi?id=66154
Reviewed by Kenneth Russell.
Covered by existing tests in fast/canvas and canvas/philip.
- html/canvas/CanvasRenderingContext2D.cpp:
(WebCore::CanvasRenderingContext2D::isAccelerated):
Plumb this call through GraphicsContext::isAcceleratedContext().
(WebCore::CanvasRenderingContext2D::paintsIntoCanvasBuffer):
For the ACCELERATED_2D_CANVAS path, plumb this call through
(the new) GraphicsContext::paintsIntoCanvasBuffer().
(WebCore::CanvasRenderingContext2D::clearAcceleration):
(WebCore::CanvasRenderingContext2D::resetAcceleration):
Remove the use of the m_context3D member; use a temporary instead.
- html/canvas/CanvasRenderingContext2D.h:
Remove the m_context3D member.
- platform/graphics/GraphicsContext.cpp:
(WebCore::GraphicsContext::isAcceleratedContext):
(WebCore::GraphicsContext::paintsIntoImageBuffer):
Implement stub versions of these functions for other platforms.
- platform/graphics/GraphicsContext.h:
Expose isAcceleratedContext() to all platforms. Add
paintsIntoImageBuffer() member function.
- platform/graphics/gpu/SharedGraphicsContext3D.cpp:
(WebCore::SharedGraphicsContext3D::create):
- platform/graphics/gpu/SharedGraphicsContext3D.h:
- platform/graphics/skia/GraphicsContextSkia.cpp:
(WebCore::GraphicsContext::setGraphicsContext3D):
(WebCore::GraphicsContext::isAcceleratedContext):
(WebCore::GraphicsContext::paintsIntoImageBuffer):
Basically gut this class leaving only a static creation function.
- platform/graphics/skia/ImageSkia.cpp:
(WebCore::paintSkBitmap):
(WebCore::Image::drawPattern):
Use isAccelerated() in place of useSkiaGpu().
- platform/graphics/skia/PlatformContextSkia.cpp:
(WebCore::PlatformContextSkia::isNativeFontRenderingAllowed):
(WebCore::PlatformContextSkia::paintsIntoImageBuffer):
- platform/graphics/skia/PlatformContextSkia.h:
(WebCore::PlatformContextSkia::isAccelerated):
Rename useSkiaGpu() to isAccelerated(). Plumb through
paintsIntoImageBuffer() to GraphicsContext3D.
- 7:40 PM Changeset in webkit [93012] by
-
- 13 edits in trunk/Source
Use builtin_trap() for CRASH when building with clang
https://bugs.webkit.org/show_bug.cgi?id=66152
Reviewed by Anders Carlsson.
- wtf/Assertions.h:
Add Clang specific CRASH macro that calls builtin_trap() instead
of silly techniques to crash. This allows the static analyzer to understand
that we are intentionally crashing. As a result, we need to mark some functions
as not returning.
Also adds a macros that annotates a function as never returning due to ASSERT or CRASH.
- wtf/Compiler.h:
Add COMPILIER(CLANG) and fix some formatting and spelling mistakes.
- wtf/FastMalloc.cpp:
(WTF::Internal::fastMallocMatchFailed):
Add NO_RETURN_DUE_TO_CRASH.
- yarr/YarrParser.h:
(JSC::Yarr::Parser::CharacterClassParserDelegate::assertionWordBoundary):
(JSC::Yarr::Parser::CharacterClassParserDelegate::atomBackReference):
Add NO_RETURN_DUE_TO_ASSERT.
../WebCore:
- bindings/js/SerializedScriptValue.cpp:
(WebCore::CloneBase::fail):
- bindings/objc/WebScriptObject.mm:
- platform/mac/BlockExceptions.h:
- platform/text/cf/StringImplCF.cpp:
Add NO_RETURN_DUE_TO_ASSERT.
- bridge/IdentifierRep.h:
Don't define the destructor since it is never called,
../WebKit/mac:
- Plugins/WebBaseNetscapePluginView.mm:
(-[WebBaseNetscapePluginView setAttributeKeys:andValues:]):
(-[WebBaseNetscapePluginView handleMouseMoved:]):
(-[WebBaseNetscapePluginView handleMouseEntered:]):
(-[WebBaseNetscapePluginView handleMouseExited:]):
(-[WebBaseNetscapePluginView focusChanged]):
(-[WebBaseNetscapePluginView windowFocusChanged:]):
(-[WebBaseNetscapePluginView createPlugin]):
(-[WebBaseNetscapePluginView loadStream]):
(-[WebBaseNetscapePluginView shouldStop]):
(-[WebBaseNetscapePluginView destroyPlugin]):
(-[WebBaseNetscapePluginView updateAndSetWindow]):
(-[WebBaseNetscapePluginView sendModifierEventWithKeyCode:character:]):
(-[WebBaseNetscapePluginView pluginLayer]):
(-[WebBaseNetscapePluginView getFormValue:]):
Remove the ASSERT_NOT_REACHED from the base class methods. They were not
adding much value and were getting in the way of making the static analyzer
work. A better way to do this would be to use a formal protocol.
- 6:44 PM Changeset in webkit [93011] by
-
- 2 edits in trunk/Source/WebCore
Abandoned Memory: Temporary CSS Fonts May Never Be Purged
https://bugs.webkit.org/show_bug.cgi?id=66153
Reviewed by Dan Bernstein.
No new tests, this is not a functional change.
- css/CSSFontFaceSource.cpp:
(WebCore::CSSFontFaceSource::getFontData):
If the CSS font-face is loading from a URL then immediately
fallback to a system font matching the description, since we
cannot determine a reasonable family name from the m_string
URL. The URL could be a dataURI.
- 4:27 PM Changeset in webkit [93010] by
-
- 8 edits in trunk/Source/JavaScriptCore
DFG JIT has inconsistent use of boxDouble and unboxDouble,
inconsistent use of assertions regarding doubles, and those
assertions are not turned on in debug builds
https://bugs.webkit.org/show_bug.cgi?id=66160
Reviewed by Gavin Barraclough.
JIT assertions are now turned on in debug builds. JIT
assertions are now used for boxing and unboxing doubles, and boxing
and unboxing no longer involves code duplication.
- dfg/DFGJITCodeGenerator.cpp:
(JSC::DFG::JITCodeGenerator::fillDouble):
- dfg/DFGJITCodeGenerator.h:
(JSC::DFG::JITCodeGenerator::boxDouble):
(JSC::DFG::JITCodeGenerator::unboxDouble):
- dfg/DFGJITCompiler.cpp:
(JSC::DFG::JITCompiler::fillNumericToDouble):
(JSC::DFG::GeneralizedRegister::moveTo):
(JSC::DFG::GeneralizedRegister::swapWith):
- dfg/DFGJITCompiler.h:
(JSC::DFG::JITCompiler::boxDouble):
(JSC::DFG::JITCompiler::unboxDouble):
- dfg/DFGNode.h:
- dfg/DFGNonSpeculativeJIT.cpp:
(JSC::DFG::NonSpeculativeJIT::knownConstantArithOp):
(JSC::DFG::NonSpeculativeJIT::compile):
- dfg/DFGSpeculativeJIT.cpp:
(JSC::DFG::SpeculativeJIT::fillSpeculateDouble):
(JSC::DFG::SpeculativeJIT::convertToDouble):
- 4:18 PM Changeset in webkit [93009] by
-
- 9 edits4 adds in trunk/Source
Refactoring of PopupMenuChromium For Readability and Maintainability
https://bugs.webkit.org/show_bug.cgi?id=66009
Reviewed by Darin Fisher.
Source/WebCore:
Split PopupMenuChromium into three sets of files: PopupMenuChromium.{h|cpp}, PopupContainer.{h|cpp},
and PopupLisBox.{h|cpp} for readability and maintainability.
No new tests as there's no change in functionality.
- WebCore.gypi:
- platform/chromium/PopupContainer.cpp: Added.
(WebCore::constructRelativeMouseEvent):
(WebCore::constructRelativeWheelEvent):
(WebCore::PopupContainer::create):
(WebCore::PopupContainer::PopupContainer):
(WebCore::PopupContainer::~PopupContainer):
(WebCore::PopupContainer::layoutAndCalculateWidgetRect):
(WebCore::PopupContainer::showPopup):
(WebCore::PopupContainer::hidePopup):
(WebCore::PopupContainer::notifyPopupHidden):
(WebCore::PopupContainer::layoutAndGetRTLOffset):
(WebCore::PopupContainer::handleMouseDownEvent):
(WebCore::PopupContainer::handleMouseMoveEvent):
(WebCore::PopupContainer::handleMouseReleaseEvent):
(WebCore::PopupContainer::handleWheelEvent):
(WebCore::PopupContainer::handleTouchEvent):
(WebCore::PopupContainer::handleGestureEvent):
(WebCore::PopupContainer::handleKeyEvent):
(WebCore::PopupContainer::hide):
(WebCore::PopupContainer::paint):
(WebCore::PopupContainer::paintBorder):
(WebCore::PopupContainer::isInterestedInEventForKey):
(WebCore::PopupContainer::chromeClientChromium):
(WebCore::PopupContainer::showInRect):
(WebCore::PopupContainer::refresh):
(WebCore::PopupContainer::isRTL):
(WebCore::PopupContainer::selectedIndex):
(WebCore::PopupContainer::menuItemHeight):
(WebCore::PopupContainer::menuItemFontSize):
(WebCore::PopupContainer::menuStyle):
(WebCore::popupData):
(WebCore::PopupContainer::getSelectedItemToolTip):
- platform/chromium/PopupContainer.h: Added.
(WebCore::PopupContainer::listBox):
(WebCore::PopupContainer::popupType):
- platform/chromium/PopupListBox.cpp: Added.
(WebCore::PopupListBox::PopupListBox):
(WebCore::PopupListBox::handleMouseDownEvent):
(WebCore::PopupListBox::handleMouseMoveEvent):
(WebCore::PopupListBox::handleMouseReleaseEvent):
(WebCore::PopupListBox::handleWheelEvent):
(WebCore::PopupListBox::isInterestedInEventForKey):
(WebCore::PopupListBox::handleTouchEvent):
(WebCore::PopupListBox::handleGestureEvent):
(WebCore::isCharacterTypeEvent):
(WebCore::PopupListBox::handleKeyEvent):
(WebCore::PopupListBox::hostWindow):
(WebCore::stripLeadingWhiteSpace):
(WebCore::PopupListBox::typeAheadFind):
(WebCore::PopupListBox::paint):
(WebCore::PopupListBox::paintRow):
(WebCore::PopupListBox::getRowFont):
(WebCore::PopupListBox::abandon):
(WebCore::PopupListBox::pointToRowIndex):
(WebCore::PopupListBox::acceptIndex):
(WebCore::PopupListBox::selectIndex):
(WebCore::PopupListBox::setOriginalIndex):
(WebCore::PopupListBox::getRowHeight):
(WebCore::PopupListBox::getRowBounds):
(WebCore::PopupListBox::invalidateRow):
(WebCore::PopupListBox::scrollToRevealRow):
(WebCore::PopupListBox::isSelectableItem):
(WebCore::PopupListBox::clearSelection):
(WebCore::PopupListBox::selectNextRow):
(WebCore::PopupListBox::selectPreviousRow):
(WebCore::PopupListBox::adjustSelectedIndex):
(WebCore::PopupListBox::hidePopup):
(WebCore::PopupListBox::updateFromElement):
(WebCore::PopupListBox::setMaxWidthAndLayout):
(WebCore::PopupListBox::layout):
(WebCore::PopupListBox::clear):
(WebCore::PopupListBox::isPointInBounds):
- platform/chromium/PopupListBox.h: Added.
(WebCore::PopupItem::PopupItem):
(WebCore::PopupListBox::create):
(WebCore::PopupListBox::selectedIndex):
(WebCore::PopupListBox::numItems):
(WebCore::PopupListBox::setBaseWidth):
(WebCore::PopupListBox::setMaxHeight):
(WebCore::PopupListBox::setMaxWidth):
(WebCore::PopupListBox::disconnectClient):
(WebCore::PopupListBox::items):
(WebCore::PopupListBox::~PopupListBox):
(WebCore::PopupListBox::scrollToRevealSelection):
- platform/chromium/PopupMenuChromium.cpp:
- platform/chromium/PopupMenuChromium.h:
Source/WebKit/chromium:
- src/ChromeClientImpl.cpp:
- src/WebPopupMenuImpl.cpp:
- src/WebViewImpl.cpp:
- tests/PopupMenuTest.cpp:
- 3:59 PM Changeset in webkit [93008] by
-
- 8 edits in branches/chromium/835/Source
Need a way to selectively use hixie-76 for websocket connections depending on destination and/or origin
https://bugs.webkit.org/show_bug.cgi?id=65835
Patch by Denis Lagno <dilmah@chromium.org> on 2011-08-12
Reviewed by Alexey Proskuryakov and Yuta Kitamura.
WebCore:
No new tests. (because it is temporary solution for transient period between hixie-76 and hybi-NN).
- page/Settings.cpp:
(WebCore::Settings::addHixie76WebSocketProtocolException):
(WebCore::Settings::useHixie76WebSocketProtocol):
- page/Settings.h:
(WebCore::Settings::setUseHixie76WebSocketProtocol):
- websockets/WebSocketChannel.cpp:
(WebCore::WebSocketChannel::WebSocketChannel):
WebKit/chromium:
- public/WebSettings.h:
- src/WebSettingsImpl.cpp:
(WebKit::WebSettingsImpl::addHixie76WebSocketProtocolException):
- src/WebSettingsImpl.h:
- 3:56 PM Changeset in webkit [93007] by
-
- 4 edits in trunk/Source/WebKit/chromium
Expose pageScaleFactor() to Chromium's WebViewImpl
https://bugs.webkit.org/show_bug.cgi?id=66067
Reviewed by Darin Fisher.
- public/WebView.h:
- src/WebViewImpl.cpp:
(WebKit::WebViewImpl::pageScaleFactor):
- src/WebViewImpl.h:
- 3:48 PM Changeset in webkit [93006] by
-
- 2 edits in trunk/LayoutTests
Skip the test added by r92997 on Qt port because it uses a method on EventSender that hasn't been
implemented by Qt port's DRT. The failure is tracked by the bug 66173.
- platform/qt/Skipped:
- 3:35 PM Changeset in webkit [93005] by
-
- 17 edits in trunk
Be more forward-looking in the choice of compiler.
Rubber-stamped by Jon Honeycutt.
Source/JavaScriptCore:
- Configurations/CompilerVersion.xcconfig:
Source/JavaScriptGlue:
- Configurations/CompilerVersion.xcconfig:
Source/ThirdParty/ANGLE:
- Configurations/CompilerVersion.xcconfig:
Source/WebCore:
- Configurations/CompilerVersion.xcconfig:
Source/WebKit/mac:
- Configurations/CompilerVersion.xcconfig:
Source/WebKit2:
- Configurations/CompilerVersion.xcconfig:
Tools:
- DumpRenderTree/mac/Configurations/CompilerVersion.xcconfig:
- MiniBrowser/Configurations/CompilerVersion.xcconfig:
- TestWebKitAPI/Configurations/CompilerVersion.xcconfig:
- WebKitTestRunner/Configurations/CompilerVersion.xcconfig:
- 3:32 PM Changeset in webkit [93004] by
-
- 2 edits in trunk/Source/JavaScriptCore
[GTK] Fix non-pthreads build after r91906.
https://bugs.webkit.org/show_bug.cgi?id=66151
Patch by Kalev Lember <kalevlember@gmail.com> on 2011-08-12
Reviewed by David Levin.
r91906 broke the non-pthreads GTK+ build by including a header which
doesn't exist. Fix it by including DateMath.h instead of DateMap.h.
- wtf/gtk/ThreadingGtk.cpp:
- 3:00 PM Changeset in webkit [93003] by
-
- 2 edits in trunk/Source/WebCore
[chromium] Fix comile warning on CCLayerTreeHost
- platform/graphics/chromium/cc/CCLayerTreeHost.cpp:
(WebCore::CCLayerTreeHost::CCLayerTreeHost):
- 2:44 PM Changeset in webkit [93002] by
-
- 2 edits in trunk/LayoutTests
Update expectations to note flakiness.
- platform/chromium/test_expectations.txt:
- 2:42 PM Changeset in webkit [93001] by
-
- 50 edits in trunk
Apple-style-span class seems unnecessary
https://bugs.webkit.org/show_bug.cgi?id=12248
Reviewed by Justin Garcia.
Source/WebCore:
Stop generating span or font elements with class="Apple-style-span" given WebKit's editing component
no longer depends on Apple-style-span since r92823 removed the dependency of copy and paste code on
style spans. WebKit continues to recognize Apple style spans to remove them.
Also renamed isSpanWithoutAttributesOrUnstyleStyleSpan to isSpanWithoutAttributesOrUnstyle*d*StyleSpan.
- editing/ApplyStyleCommand.cpp:
(WebCore::isLegacyAppleStyleSpan): Renamed from isStyleSpan.
(WebCore::isSpanWithoutAttributesOrUnstyledStyleSpan): Renamed from isSpanWithoutAttributesOr*Unstyle*StyleSpan.
(WebCore::createFontElement): No longer adds class="Apple-style-span".
(WebCore::createStyleSpanElement): Ditto.
(WebCore::ApplyStyleCommand::applyRelativeFontStyleChange): Calls isSpanWithoutAttributesOrUnstyleStyleSpan
instead of isUnstyledStyleSpan since there won't be any Apple style spans.
(WebCore::dummySpanAncestorForNode):
(WebCore::ApplyStyleCommand::cleanupUnstyledAppleStyleSpans): Ditto.
(WebCore::ApplyStyleCommand::removeEmbeddingUpToEnclosingBlock): Ditto.
(WebCore::ApplyStyleCommand::removeImplicitlyStyledElement):
(WebCore::ApplyStyleCommand::removeCSSStyle):
- editing/ApplyStyleCommand.h:
- editing/EditingStyle.cpp:
(WebCore::EditingStyle::removeStyleFromRulesAndContext): Ditto.
- editing/ReplaceSelectionCommand.cpp:
(WebCore::ReplaceSelectionCommand::removeRedundantStylesAndKeepStyleSpanInline): Calls isLegacyAppleStyleSpan instead
of isStyleSpan.
(WebCore::handleStyleSpansBeforeInsertion): Ditto.
(WebCore::ReplaceSelectionCommand::handleStyleSpans): Ditto.
(WebCore::ReplaceSelectionCommand::doApply): Ditto.
- editing/markup.cpp:
(WebCore::StyledMarkupAccumulator::appendStyleNodeOpenTag): No longer adds class="Apple-style-span".
LayoutTests:
Rebaselined the tests because spans in these tests lost no longer have class="Apple-style-span".
- editing/deleting/delete-br-012-expected.txt:
- editing/deleting/delete-line-break-between-paragraphs-with-same-style-expected.txt:
- editing/deleting/delete-select-all-001-expected.txt:
- editing/execCommand/16049-expected.txt:
- editing/execCommand/5685604-1.html:
- editing/execCommand/delete-image-in-anchor-expected.txt:
- editing/execCommand/hilitecolor-expected.txt:
- editing/execCommand/modifyForeColorByCharacter-expected.txt:
- editing/execCommand/query-font-size-expected.txt:
- editing/execCommand/remove-format-multiple-elements-expected.txt:
- editing/execCommand/script-tests/remove-format-multiple-elements.js:
- editing/execCommand/script-tests/toggle-text-decorations.js:
- editing/execCommand/script-tests/toggle-unlink.js:
- editing/execCommand/toggle-style-2-expected.txt:
- editing/execCommand/toggle-text-decorations-expected.txt:
- editing/execCommand/toggle-unlink-expected.txt:
- editing/inserting/5994480-2-expected.txt:
- editing/pasteboard/5065605-expected.txt:
- editing/pasteboard/copy-null-characters-expected.txt:
- editing/pasteboard/copy-text-with-backgroundcolor-expected.txt:
- editing/pasteboard/data-transfer-items-expected.txt:
- editing/pasteboard/do-not-copy-unnecessary-styles-2-expected.txt:
- editing/pasteboard/do-not-copy-unnecessary-styles-expected.txt:
- editing/pasteboard/onpaste-text-html-expected.txt:
- editing/pasteboard/paste-text-with-style-2-expected.txt:
- editing/style/apply-font-size-to-multiple-nodes-expected.txt:
- editing/style/block-style-004-expected.txt:
- editing/style/block-style-005-expected.txt:
- editing/style/block-style-006-expected.txt:
- editing/style/fontsize-1-expected.txt:
- editing/style/fore-color-by-name-expected.txt:
- editing/style/highlight-insert-paragraph-expected.txt:
- editing/style/inline-style-container-expected.txt:
- editing/style/inline-style-extend-run-expected.txt:
- editing/style/invalid-font-size-expected.txt:
- editing/style/push-down-font-styles-expected.txt:
- editing/style/script-tests/inline-style-container.js:
- editing/style/script-tests/inline-style-extend-run.js:
- editing/style/script-tests/push-down-font-styles.js:
- editing/style/style-text-node-without-editable-parent-expected.txt:
- fast/events/ondrop-text-html-expected.txt:
- platform/chromium-win/editing/inserting/5994480-2-expected.txt:
- platform/gtk/editing/inserting/5994480-2-expected.txt:
- 2:42 PM Changeset in webkit [93000] by
-
- 2 edits in trunk/Source/JavaScriptCore
Update some configuration settings that were missed back in r92432.
- Configurations/CompilerVersion.xcconfig:
- 2:41 PM Changeset in webkit [92999] by
-
- 2 edits in trunk/Source/WebCore
[chromium] Fix compositor breakage due to duplicate context creation AND by context-lost
https://bugs.webkit.org/show_bug.cgi?id=66168
- platform/graphics/chromium/cc/CCLayerTreeHost.cpp:
(WebCore::CCLayerTreeHost::CCLayerTreeHost):
(WebCore::CCLayerTreeHost::initialize):
- 2:35 PM Changeset in webkit [92998] by
-
- 8 edits in trunk/Source/WebCore
Switch RenderMenuList, RenderListBox, and RenderFieldSet to new layout units
https://bugs.webkit.org/show_bug.cgi?id=66149
Reviewed by Eric Seidel.
Changing RenderMenuList, RenderListBox, and RenderFieldSet to the LayoutUnit
abstraction from ints.
No tests as no change in functionality.
- rendering/RenderFieldset.cpp:
(WebCore::RenderFieldset::layoutSpecialExcludedChild):
- rendering/RenderListBox.cpp:
(WebCore::RenderListBox::numVisibleItems):
(WebCore::RenderListBox::listHeight):
(WebCore::RenderListBox::baselinePosition):
(WebCore::RenderListBox::itemBoundingBoxRect):
(WebCore::itemOffsetForAlignment):
(WebCore::RenderListBox::panScroll):
(WebCore::RenderListBox::scrollToward):
(WebCore::RenderListBox::autoscroll):
(WebCore::RenderListBox::scrollSize):
(WebCore::RenderListBox::scrollPosition):
(WebCore::RenderListBox::setScrollOffset):
(WebCore::RenderListBox::itemHeight):
(WebCore::RenderListBox::verticalScrollbarWidth):
(WebCore::RenderListBox::scrollWidth):
(WebCore::RenderListBox::scrollHeight):
(WebCore::RenderListBox::scrollLeft):
(WebCore::RenderListBox::setScrollLeft):
(WebCore::RenderListBox::scrollTop):
(WebCore::RenderListBox::setScrollTop):
(WebCore::RenderListBox::controlClipRect):
(WebCore::RenderListBox::invalidateScrollbarRect):
(WebCore::RenderListBox::convertFromScrollbarToContainingView):
(WebCore::RenderListBox::convertFromContainingViewToScrollbar):
(WebCore::RenderListBox::contentsSize):
(WebCore::RenderListBox::currentMousePosition):
- rendering/RenderListBox.h:
(WebCore::RenderListBox::scrollCornerRect):
(WebCore::RenderListBox::invalidateScrollCornerRect):
- rendering/RenderMenuList.cpp:
(WebCore::RenderMenuList::controlClipRect):
(WebCore::RenderMenuList::showPopup):
- rendering/RenderMenuList.h:
- 2:28 PM Changeset in webkit [92997] by
-
- 9 edits5 adds in trunk
Source/WebKit/chromium: Wire gesture tap events, add support for gesture events in the DRT and add a layout test for it.
https://bugs.webkit.org/show_bug.cgi?id=66105
Patch by Sadrul Habib Chowdhury <sadrul@chromium.org> on 2011-08-12
Reviewed by Darin Fisher.
- public/WebInputEvent.h:
- src/WebInputEventConversion.cpp:
(WebKit::PlatformGestureEventBuilder::PlatformGestureEventBuilder):
- src/WebPopupMenuImpl.cpp:
(WebKit::WebPopupMenuImpl::handleInputEvent):
- src/WebViewImpl.cpp:
(WebKit::WebViewImpl::handleInputEvent):
Tools: DRT: Add support for gesture events.
https://bugs.webkit.org/show_bug.cgi?id=66105
Patch by Sadrul Habib Chowdhury <sadrul@chromium.org> on 2011-08-12
Reviewed by Darin Fisher.
- DumpRenderTree/chromium/EventSender.cpp:
(EventSender::EventSender):
(EventSender::gestureScrollBegin):
(EventSender::gestureScrollEnd):
(EventSender::gestureTap):
(EventSender::gestureEvent):
- DumpRenderTree/chromium/EventSender.h:
LayoutTests: DRT: Add support for gesture events.
https://bugs.webkit.org/show_bug.cgi?id=66105
Patch by Sadrul Habib Chowdhury <sadrul@chromium.org> on 2011-08-12
Reviewed by Darin Fisher.
- fast/events/touch/gesture/gesture-click-expected.txt: Added.
- fast/events/touch/gesture/gesture-click.html: Added.
- platform/chromium/fast/events/touch/gesture/gesture-click-expected.txt: Added.
- 2:10 PM Changeset in webkit [92996] by
-
- 4 edits in branches/safari-534.51-branch/Source/JavaScriptCore
Merge r92986.
- 2:08 PM Changeset in webkit [92995] by
-
- 4 edits in branches/safari-534.51-branch/Source/JavaScriptCore
Merge r92911.
- 2:06 PM Changeset in webkit [92994] by
-
- 2 edits in branches/safari-534.51-branch/Source/JavaScriptCore
Merge r92909.
- 2:02 PM Changeset in webkit [92993] by
-
- 7 edits2 adds in trunk
https://bugs.webkit.org/show_bug.cgi?id=66133
Make hit testing work on RenderRegions. Pass off the hit testing to the RenderFlowThread
layer tree (just as we did with painting).
Reviewed by Sam Weinig.
Added hit-test-float.html to demonstrate basic hit testing of content flowed into regions.
Source/WebCore:
- rendering/HitTestRequest.h:
(WebCore::HitTestRequest::type):
- rendering/RenderFlowThread.cpp:
(WebCore::RenderFlowThread::hitTestRegion):
- rendering/RenderFlowThread.h:
- rendering/RenderRegion.cpp:
(WebCore::RenderRegion::paintReplaced):
(WebCore::RenderRegion::nodeAtPoint):
- rendering/RenderRegion.h:
LayoutTests:
- fast/regions/hit-test-float-expected.txt: Added.
- fast/regions/hit-test-float.html: Added.
- 2:01 PM Changeset in webkit [92992] by
-
- 8 edits in trunk
Switch RenderTable* to new layout types
https://bugs.webkit.org/show_bug.cgi?id=66146
Reviewed by Eric Seidel.
Converting RenderTable* classes to new LayoutUnits from ints.
No new tests as no new functionality.
- rendering/RenderTable.cpp:
(WebCore::RenderTable::firstLineBoxBaseline):
(WebCore::RenderTable::overflowClipRect):
- rendering/RenderTable.h:
(WebCore::RenderTable::columnPositions):
- rendering/RenderTableCol.cpp:
(WebCore::RenderTableCol::clippedOverflowRectForRepaint):
- rendering/RenderTableRow.cpp:
(WebCore::RenderTableRow::clippedOverflowRectForRepaint):
- rendering/RenderTableSection.cpp:
(WebCore::RenderTableSection::setCellLogicalWidths):
(WebCore::RenderTableSection::calcRowLogicalHeight):
(WebCore::RenderTableSection::layoutRows):
(WebCore::RenderTableSection::calcOuterBorderBefore):
(WebCore::RenderTableSection::calcOuterBorderAfter):
(WebCore::RenderTableSection::calcOuterBorderStart):
(WebCore::RenderTableSection::calcOuterBorderEnd):
(WebCore::RenderTableSection::firstLineBoxBaseline):
(WebCore::RenderTableSection::paintObject):
(WebCore::RenderTableSection::nodeAtPoint):
- rendering/RenderTableSection.h:
(WebCore::RenderTableSection::outerBorderBefore):
(WebCore::RenderTableSection::outerBorderAfter):
(WebCore::RenderTableSection::outerBorderStart):
(WebCore::RenderTableSection::outerBorderEnd):
(WebCore::RenderTableSection::getBaseline):
- 1:58 PM Changeset in webkit [92991] by
-
- 2 edits3 adds in trunk/Source/WebCore
Patch by Adam Bergkvist <adam.bergkvist@ericsson.com> on 2011-08-12
Reviewed by Alexey Proskuryakov.
EventSource loader should not buffer data
https://bugs.webkit.org/show_bug.cgi?id=61863
Disabled buffering in the EventSource loader.
Added a manual test.
- manual-tests/eventsource/eventsource-loader-buffering.html: Added.
- manual-tests/eventsource/eventsource-loader-buffering.php: Added.
- page/EventSource.cpp:
(WebCore::EventSource::connect):
- 1:56 PM Changeset in webkit [92990] by
-
- 2 edits in trunk/LayoutTests
Skip the test added by r92982 on Qt since it uses a new layoutTestController method
that hasn't been implemented by Qt DRT. The failure is tracked by the bug 66162.
- platform/qt/Skipped:
- 1:46 PM Changeset in webkit [92989] by
-
- 2 edits4 adds in trunk/Source/WebKit/chromium
[Chromium] Implement the embedders for the HTML5 Track List objects
https://bugs.webkit.org/show_bug.cgi?id=60260
Patch by Tommy Widenflycht <tommyw@google.com> on 2011-08-12
Reviewed by Darin Fisher.
- WebKit.gyp:
- public/WebMediaStreamTrack.h: Added.
(WebKit::WebMediaStreamTrack::WebMediaStreamTrack):
(WebKit::WebMediaStreamTrack::~WebMediaStreamTrack):
(WebKit::WebMediaStreamTrack::isNull):
- public/WebMediaStreamTrackList.h: Added.
(WebKit::WebMediaStreamTrackList::WebMediaStreamTrackList):
(WebKit::WebMediaStreamTrackList::~WebMediaStreamTrackList):
(WebKit::WebMediaStreamTrackList::isNull):
- src/WebMediaStreamTrack.cpp: Added.
(WebKit::WebMediaStreamTrack::initialize):
(WebKit::WebMediaStreamTrack::reset):
(WebKit::WebMediaStreamTrack::WebMediaStreamTrack):
(WebKit::WebMediaStreamTrack::operator WTF::PassRefPtr<WebCore::MediaStreamTrack>):
- src/WebMediaStreamTrackList.cpp: Added.
(WebKit::WebMediaStreamTrackList::initialize):
(WebKit::WebMediaStreamTrackList::reset):
(WebKit::WebMediaStreamTrackList::WebMediaStreamTrackList):
(WebKit::WebMediaStreamTrackList::operator WTF::PassRefPtr<MediaStreamTrackList>):
- 1:41 PM Changeset in webkit [92988] by
-
- 5 edits in trunk/LayoutTests
GTK rebaselines after r92890.
- platform/gtk/fast/dom/Window/window-properties-expected.txt:
- platform/gtk/fast/dom/Window/window-property-descriptors-expected.txt:
- platform/gtk/fast/dom/prototype-inheritance-2-expected.txt:
- platform/gtk/fast/js/global-constructors-expected.txt:
- 1:34 PM Changeset in webkit [92987] by
-
- 5 edits in trunk/LayoutTests
Mac rebaselines after r92890.
- fast/dom/Window/window-properties-expected.txt:
- fast/dom/Window/window-property-descriptors-expected.txt:
- fast/dom/prototype-inheritance-2-expected.txt:
- platform/mac/fast/js/global-constructors-expected.txt:
- 1:33 PM Changeset in webkit [92986] by
-
- 4 edits in trunk/Source/JavaScriptCore
REGRESSION (r91610?): Bing Maps fail to initialize (InvalidOperation:
Matrix3D.invert)
https://bugs.webkit.org/show_bug.cgi?id=66038
Patch by Filip Pizlo <fpizlo@apple.com> on 2011-08-12
Reviewed by Gavin Barraclough.
Simplest and lowest-impact fix for the case where the spilled format
of a DFG node differs from the register format: if the format is
converted then indicate that the spilled value is no longer valid
("kill the spill").
- dfg/DFGGenerationInfo.h:
(JSC::DFG::GenerationInfo::killSpilled):
- dfg/DFGJITCodeGenerator.cpp:
(JSC::DFG::JITCodeGenerator::fillDouble):
- dfg/DFGSpeculativeJIT.cpp:
(JSC::DFG::SpeculativeJIT::fillSpeculateDouble):
- 1:33 PM Changeset in webkit [92985] by
-
- 2 edits in trunk/Source/WebCore
ResourceLoader::didReceiveDataArray() does not handle m_shouldBufferData correctly
https://bugs.webkit.org/show_bug.cgi?id=65926
Reviewed by Alexey Proskuryakov.
- loader/mac/ResourceLoaderMac.mm:
(WebCore::ResourceLoader::didReceiveDataArray): Make sure we call the client callbacks when
m_shouldBufferData is set to false.
- 1:18 PM Changeset in webkit [92984] by
-
- 2 edits in trunk/Tools
allowRoundingHacksCallback uses the wrong argument for 'thisObject'.
https://bugs.webkit.org/show_bug.cgi?id=66158
Reviewed by Dan Bernstein.
- DumpRenderTree/LayoutTestController.cpp:
(allowRoundingHacksCallback): The third argument is the 'this'
JSObjectRef, not the second.
- 1:16 PM Changeset in webkit [92983] by
-
- 8 edits4 deletes in trunk/Source
Unreviewed, rolling out r92976.
http://trac.webkit.org/changeset/92976
https://bugs.webkit.org/show_bug.cgi?id=66159
Does not compile on chromium-win (Requested by abarth|gardener
on #webkit).
Patch by Sheriff Bot <webkit.review.bot@gmail.com> on 2011-08-12
Source/WebCore:
- WebCore.gypi:
- platform/chromium/PopupContainer.cpp: Removed.
- platform/chromium/PopupContainer.h: Removed.
- platform/chromium/PopupListBox.cpp: Removed.
- platform/chromium/PopupListBox.h: Removed.
- platform/chromium/PopupMenuChromium.cpp:
(WebCore::PopupListBox::create):
(WebCore::PopupListBox::selectedIndex):
(WebCore::PopupListBox::numItems):
(WebCore::PopupListBox::setBaseWidth):
(WebCore::PopupListBox::setMaxHeight):
(WebCore::PopupListBox::setMaxWidth):
(WebCore::PopupListBox::disconnectClient):
(WebCore::PopupListBox::items):
(WebCore::PopupListBox::PopupListBox):
(WebCore::PopupListBox::~PopupListBox):
(WebCore::PopupListBox::scrollToRevealSelection):
(WebCore::constructRelativeMouseEvent):
(WebCore::constructRelativeWheelEvent):
(WebCore::PopupContainer::create):
(WebCore::PopupContainer::PopupContainer):
(WebCore::PopupContainer::~PopupContainer):
(WebCore::PopupContainer::layoutAndCalculateWidgetRect):
(WebCore::PopupContainer::showPopup):
(WebCore::PopupContainer::hidePopup):
(WebCore::PopupContainer::notifyPopupHidden):
(WebCore::PopupContainer::layoutAndGetRTLOffset):
(WebCore::PopupContainer::handleMouseDownEvent):
(WebCore::PopupContainer::handleMouseMoveEvent):
(WebCore::PopupContainer::handleMouseReleaseEvent):
(WebCore::PopupContainer::handleWheelEvent):
(WebCore::PopupContainer::handleTouchEvent):
(WebCore::PopupContainer::handleGestureEvent):
(WebCore::PopupContainer::handleKeyEvent):
(WebCore::PopupContainer::hide):
(WebCore::PopupContainer::paint):
(WebCore::PopupContainer::paintBorder):
(WebCore::PopupContainer::isInterestedInEventForKey):
(WebCore::PopupContainer::chromeClientChromium):
(WebCore::PopupContainer::showInRect):
(WebCore::PopupContainer::refresh):
(WebCore::PopupContainer::isRTL):
(WebCore::PopupContainer::selectedIndex):
(WebCore::PopupContainer::menuItemHeight):
(WebCore::PopupContainer::menuItemFontSize):
(WebCore::PopupContainer::menuStyle):
(WebCore::popupData):
(WebCore::PopupContainer::getSelectedItemToolTip):
(WebCore::PopupListBox::handleMouseDownEvent):
(WebCore::PopupListBox::handleMouseMoveEvent):
(WebCore::PopupListBox::handleMouseReleaseEvent):
(WebCore::PopupListBox::handleWheelEvent):
(WebCore::PopupListBox::isInterestedInEventForKey):
(WebCore::PopupListBox::handleTouchEvent):
(WebCore::PopupListBox::handleGestureEvent):
(WebCore::isCharacterTypeEvent):
(WebCore::PopupListBox::handleKeyEvent):
(WebCore::PopupListBox::hostWindow):
(WebCore::stripLeadingWhiteSpace):
(WebCore::PopupListBox::typeAheadFind):
(WebCore::PopupListBox::paint):
(WebCore::PopupListBox::paintRow):
(WebCore::PopupListBox::getRowFont):
(WebCore::PopupListBox::abandon):
(WebCore::PopupListBox::pointToRowIndex):
(WebCore::PopupListBox::acceptIndex):
(WebCore::PopupListBox::selectIndex):
(WebCore::PopupListBox::setOriginalIndex):
(WebCore::PopupListBox::getRowHeight):
(WebCore::PopupListBox::getRowBounds):
(WebCore::PopupListBox::invalidateRow):
(WebCore::PopupListBox::scrollToRevealRow):
(WebCore::PopupListBox::isSelectableItem):
(WebCore::PopupListBox::clearSelection):
(WebCore::PopupListBox::selectNextRow):
(WebCore::PopupListBox::selectPreviousRow):
(WebCore::PopupListBox::adjustSelectedIndex):
(WebCore::PopupListBox::hidePopup):
(WebCore::PopupListBox::updateFromElement):
(WebCore::PopupListBox::setMaxWidthAndLayout):
(WebCore::PopupListBox::layout):
(WebCore::PopupListBox::clear):
(WebCore::PopupListBox::isPointInBounds):
- platform/chromium/PopupMenuChromium.h:
(WebCore::PopupItem::PopupItem):
(WebCore::PopupContainer::listBox):
(WebCore::PopupContainer::popupType):
Source/WebKit/chromium:
- src/ChromeClientImpl.cpp:
- src/WebPopupMenuImpl.cpp:
- src/WebViewImpl.cpp:
- 12:56 PM Changeset in webkit [92982] by
-
- 19 edits2 adds in trunk
Cancel in onbeforeunload dialog sometime causes a button to stop working.
https://bugs.webkit.org/show_bug.cgi?id=26211
Reviewed by Alexey Proskuryakov.
Source/WebCore:
Test: fast/loader/form-submission-after-beforeunload-cancel.html
If an onbeforeunload handler cancels a navigation that was triggered by
a form submission, WebCore's multiple form submission protection
prevents the form from being submitted a second time even though no
first submission actually took place. Fix this by clearing
m_submittedFormURL if the onbeforeunload handler cancels the load. This
allows the submission to be retried.
- loader/FrameLoader.cpp:
(WebCore::FrameLoader::shouldClose): Set m_submittedFormURL to KURL()
if shouldClose() will return false.
Tools:
Implement a new LayoutTestController method that a test can call to set
the return value of DumpRenderTree's beforeunload UI delegate.
- DumpRenderTree/LayoutTestController.cpp:
(LayoutTestController::LayoutTestController): Initialize
m_shouldStayOnPageAfterHandlingBeforeUnload.
(setShouldStayOnPageAfterHandlingBeforeUnloadCallback): Call
LayoutTestController::setShouldStayOnPageAfterHandlingBeforeUnload().
(LayoutTestController::staticFunctions): Register
'setShouldStayOnPageAfterHandlingBeforeUnload' as a static function.
- DumpRenderTree/LayoutTestController.h:
(LayoutTestController::shouldStayOnPageAfterHandlingBeforeUnload):
(LayoutTestController::setShouldStayOnPageAfterHandlingBeforeUnload):
- DumpRenderTree/chromium/LayoutTestController.cpp:
(LayoutTestController::LayoutTestController): Initialize
m_shouldStayOnPageAfterHandlingBeforeUnload and bind
'setShouldStayOnPageAfterHandlingBeforeUnload' to its c++ setter.
(LayoutTestController::reset): Reset
m_shouldStayOnPageAfterHandlingBeforeUnload to false.
(LayoutTestController::setShouldStayOnPageAfterHandlingBeforeUnload):
Set m_shouldStayOnPageAfterHandlingBeforeUnload to the value passed in
from JavaScript.
- DumpRenderTree/chromium/LayoutTestController.h:
(LayoutTestController::shouldStayOnPageAfterHandlingBeforeUnload):
- DumpRenderTree/chromium/WebViewHost.cpp:
(WebViewHost::runModalBeforeUnloadDialog): Add the correct logging and
return the inverse of
LayoutTestController::shouldStayOnPageAfterHandlingBeforeUnload().
- DumpRenderTree/mac/UIDelegate.mm:
(-[UIDelegate webView:runBeforeUnloadConfirmPanelWithMessage:initiatedByFrame:]): Ditto.
- DumpRenderTree/win/UIDelegate.cpp:
(UIDelegate::runBeforeUnloadConfirmPanelWithMessage): Ditto.
- WebKitTestRunner/InjectedBundle/Bindings/LayoutTestController.idl:
Define setShouldStayOnPageAfterHandlingBeforeUnload().
- WebKitTestRunner/InjectedBundle/InjectedBundle.cpp:
(WTR::InjectedBundle::postNewBeforeUnloadReturnValue): Post a message
to the Test Controller telling it what value it should return in its
onbeforeunload UI delegate.
- WebKitTestRunner/InjectedBundle/InjectedBundle.h:
- WebKitTestRunner/InjectedBundle/LayoutTestController.cpp:
(WTR::LayoutTestController::setShouldStayOnPageAfterHandlingBeforeUnload):
Call InjectedBundle::postNewBeforeUnloadReturnValue().
- WebKitTestRunner/InjectedBundle/LayoutTestController.h:
- WebKitTestRunner/TestController.cpp:
(WTR::TestController::TestController): Initialize
m_beforeUnloadReturnValue to true.
(WTR::runBeforeUnloadConfirmPanel): Add the correct logging and return
the value of TestController::beforeUnloadReturnValue().
(WTR::TestController::resetStateToConsistentValues): Reset
m_beforeUnloadReturnValue to true.
- WebKitTestRunner/TestController.h:
(WTR::TestController::beforeUnloadReturnValue):
(WTR::TestController::setBeforeUnloadReturnValue):
- WebKitTestRunner/TestInvocation.cpp:
(WTR::TestInvocation::didReceiveMessageFromInjectedBundle): Handle the
message posted from the injected bundle by retrieving the message body
as a WKBoolean and calling TestController::setBeforeUnloadReturnValue().
LayoutTests:
- fast/loader/form-submission-after-beforeunload-cancel-expected.txt: Added.
- fast/loader/form-submission-after-beforeunload-cancel.html: Added.
- 12:53 PM Changeset in webkit [92981] by
-
- 12 edits4 adds6 deletes in trunk
https://bugs.webkit.org/show_bug.cgi?id=66130
RenderRegions need to paint the entire RenderFlowThread layer tree. Instead of just calling
paintBlock, make sure RenderRegions paint the flow thread's layer tree instead with the appropriate
offset.
Fix RenderRegions to derive from RenderReplaced instead of RenderBox, since it simplifies the code.
They no longer have to subclass their own layout method or worry about all of the painting logic
for anything other than the content area.
Reviewed by Anders Carlsson.
Existing tests have a layer tree example, and so those results are updated to show the tree now.
Source/WebCore:
- rendering/RenderFlowThread.cpp:
(WebCore::RenderFlowThread::createFlowThreadStyle):
(WebCore::RenderFlowThread::paintIntoRegion):
- rendering/RenderLayer.cpp:
(WebCore::RenderLayer::collectLayers):
- rendering/RenderRegion.cpp:
(WebCore::RenderRegion::RenderRegion):
(WebCore::RenderRegion::paintReplaced):
(WebCore::RenderRegion::styleDidChange):
- rendering/RenderRegion.h:
LayoutTests:
- fast/regions/content-flowed-into-regions-dynamically-added-expected.png:
- fast/regions/content-flowed-into-regions-dynamically-removed-expected.png:
- fast/regions/content-flowed-into-regions-expected.png:
- fast/regions/content-flowed-into-regions-with-dyn-index-expected.png:
- fast/regions/content-flowed-into-regions-with-index-dom-expected.png:
- fast/regions/content-flowed-into-regions-with-index-expected.png:
- fast/regions/flow-content-basic-vertical.html:
- fast/regions/webkit-flow-renderer-layer-expected.txt:
- platform/mac/fast/regions/flow-content-basic-expected.png:
- platform/mac/fast/regions/flow-content-basic-vertical-expected.png:
- platform/mac/fast/regions/flow-content-basic-vertical-expected.txt:
- platform/mac/fast/regions/flow-content-basic-vertical-rl-expected.png:
- 12:38 PM Changeset in webkit [92980] by
-
- 4 edits in trunk/Source/WebKit/chromium
Unreviewed, rolling out r92978.
http://trac.webkit.org/changeset/92978
https://bugs.webkit.org/show_bug.cgi?id=66155
This patch does not compile (Requested by abarth|gardener on
#webkit).
Patch by Sheriff Bot <webkit.review.bot@gmail.com> on 2011-08-12
- public/WebView.h:
- src/WebViewImpl.cpp:
(WebKit::WebViewImpl::zoomLevel):
(WebKit::WebViewImpl::setZoomLevel):
(WebKit::WebViewImpl::zoomLimitsChanged):
(WebKit::WebViewImpl::fullFramePluginZoomLevelChanged):
(WebKit::WebView::zoomLevelToZoomFactor):
(WebKit::WebView::zoomFactorToZoomLevel):
- src/WebViewImpl.h:
- 11:56 AM Changeset in webkit [92979] by
-
- 2 edits in trunk/Source/WebCore
Fix mac build when web audio is enabled
https://bugs.webkit.org/show_bug.cgi?id=66150
Unreviewed build fix.
- platform/audio/mac/AudioFileReaderMac.cpp:
(WebCore::AudioFileReader::createBus):
- 11:43 AM Changeset in webkit [92978] by
-
- 4 edits in trunk/Source/WebKit/chromium
Chromium's WebKit API should use floats instead of doubles for zoom to match WebCore
https://bugs.webkit.org/show_bug.cgi?id=66089
Reviewed by Darin Fisher.
- public/WebView.h:
- src/WebViewImpl.cpp:
(WebKit::WebViewImpl::zoomLevel):
(WebKit::WebViewImpl::setZoomLevel):
(WebKit::WebViewImpl::zoomLimitsChanged):
(WebKit::WebViewImpl::fullFramePluginZoomLevelChanged):
(WebKit::WebView::zoomLevelToZoomFactor):
(WebKit::WebView::zoomFactorToZoomLevel):
- src/WebViewImpl.h:
- 11:42 AM Changeset in webkit [92977] by
-
- 2 edits in trunk/Source/WebCore
Need to handle kCACFContextNeedsFlushNotification notifications that arrive after the AVFWrapper has been disposed
https://bugs.webkit.org/show_bug.cgi?id=65724
Instead of using a pointer to the AVFWrapper object as the context for various callbacks, assign each object an
ID and use that instead. Keep track of the mapping between object IDs and AVFWrapper objects in a HashMap, and manage
access to this map using a Mutex since it can be accessed from multiple threads. This allows us to actually delete
AVFWrapper objects instead of leaking them (which we were doing before to prevent crashes).
Reviewed by Eric Carlson.
No new tests, uses existing media tests.
- platform/graphics/avfoundation/MediaPlayerPrivateAVFoundationCF.cpp:
(WebCore::AVFWrapper::callbackContext): Added.
(WebCore::AVFWrapper::AVFWrapper): Initialize m_objectID and add it to the HashMap.
(WebCore::AVFWrapper::~AVFWrapper): Log object ID and remove obsolete assert.
(WebCore::AVFWrapper::mapLock): Added.
(WebCore::AVFWrapper::map): Added.
(WebCore::AVFWrapper::addToMap): Added.
(WebCore::AVFWrapper::removeFromMap): Added.
(WebCore::AVFWrapper::avfWrapperForCallbackContext): Added.
(WebCore::AVFWrapper::scheduleDisconnectAndDelete): Remove AVFWrapper from HashMap instead of zeroing m_owner.
(WebCore::AVFWrapper::disconnectAndDeleteAVFWrapper): Use callbackContext(), delete the AVFWrapper here.
(WebCore::AVFWrapper::createPlayer): Use callbackContext().
(WebCore::AVFWrapper::createPlayerItem): Use callbackContext().
(WebCore::AVFWrapper::periodicTimeObserverCallback): Retrieve AVFWrapper using the HashMap.
(WebCore::AVFWrapper::notificationCallback): Retrieve AVFWrapper using the HashMap.
(WebCore::AVFWrapper::loadPlayableCompletionCallback): Retrieve AVFWrapper using the HashMap.
(WebCore::AVFWrapper::checkPlayability): Use callbackContext().
(WebCore::AVFWrapper::loadMetadataCompletionCallback): Retrieve AVFWrapper using the HashMap.
(WebCore::AVFWrapper::beginLoadingMetadata): Use callbackContext().
(WebCore::AVFWrapper::seekCompletedCallback): Retrieve AVFWrapper using the HashMap.
(WebCore::AVFWrapper::seekToTime): Use callbackContext().
(WebCore::AVFWrapper::platformLayer): Remove overly noisy LOG().
- 11:36 AM Changeset in webkit [92976] by
-
- 8 edits4 adds in trunk/Source
Refactoring of PopupMenuChromium
https://bugs.webkit.org/show_bug.cgi?id=66009
Reviewed by Darin Fisher.
Source/WebCore:
Split PopupMenuChromium into three sets of files: PopupMenuChromium.{h|cpp}, PopupContainer.{h|cpp}, and PopupLisBox.{h|cpp}
for readability and maintainability.
No new tests as no functionality has changed.
- WebCore.gypi:
- platform/chromium/PopupContainer.cpp: Added.
(WebCore::constructRelativeMouseEvent):
(WebCore::constructRelativeWheelEvent):
(WebCore::PopupContainer::create):
(WebCore::PopupContainer::PopupContainer):
(WebCore::PopupContainer::~PopupContainer):
(WebCore::PopupContainer::layoutAndCalculateWidgetRect):
(WebCore::PopupContainer::showPopup):
(WebCore::PopupContainer::hidePopup):
(WebCore::PopupContainer::notifyPopupHidden):
(WebCore::PopupContainer::layoutAndGetRTLOffset):
(WebCore::PopupContainer::handleMouseDownEvent):
(WebCore::PopupContainer::handleMouseMoveEvent):
(WebCore::PopupContainer::handleMouseReleaseEvent):
(WebCore::PopupContainer::handleWheelEvent):
(WebCore::PopupContainer::handleTouchEvent):
(WebCore::PopupContainer::handleGestureEvent):
(WebCore::PopupContainer::handleKeyEvent):
(WebCore::PopupContainer::hide):
(WebCore::PopupContainer::paint):
(WebCore::PopupContainer::paintBorder):
(WebCore::PopupContainer::isInterestedInEventForKey):
(WebCore::PopupContainer::chromeClientChromium):
(WebCore::PopupContainer::showInRect):
(WebCore::PopupContainer::refresh):
(WebCore::PopupContainer::isRTL):
(WebCore::PopupContainer::selectedIndex):
(WebCore::PopupContainer::menuItemHeight):
(WebCore::PopupContainer::menuItemFontSize):
(WebCore::PopupContainer::menuStyle):
(WebCore::popupData):
(WebCore::PopupContainer::getSelectedItemToolTip):
- platform/chromium/PopupContainer.h: Added.
(WebCore::PopupContainer::listBox):
(WebCore::PopupContainer::popupType):
- platform/chromium/PopupListBox.cpp: Added.
(WebCore::PopupListBox::PopupListBox):
(WebCore::PopupListBox::handleMouseDownEvent):
(WebCore::PopupListBox::handleMouseMoveEvent):
(WebCore::PopupListBox::handleMouseReleaseEvent):
(WebCore::PopupListBox::handleWheelEvent):
(WebCore::PopupListBox::isInterestedInEventForKey):
(WebCore::PopupListBox::handleTouchEvent):
(WebCore::PopupListBox::handleGestureEvent):
(WebCore::isCharacterTypeEvent):
(WebCore::PopupListBox::handleKeyEvent):
(WebCore::PopupListBox::hostWindow):
(WebCore::stripLeadingWhiteSpace):
(WebCore::PopupListBox::typeAheadFind):
(WebCore::PopupListBox::paint):
(WebCore::PopupListBox::paintRow):
(WebCore::PopupListBox::getRowFont):
(WebCore::PopupListBox::abandon):
(WebCore::PopupListBox::pointToRowIndex):
(WebCore::PopupListBox::acceptIndex):
(WebCore::PopupListBox::selectIndex):
(WebCore::PopupListBox::setOriginalIndex):
(WebCore::PopupListBox::getRowHeight):
(WebCore::PopupListBox::getRowBounds):
(WebCore::PopupListBox::invalidateRow):
(WebCore::PopupListBox::scrollToRevealRow):
(WebCore::PopupListBox::isSelectableItem):
(WebCore::PopupListBox::clearSelection):
(WebCore::PopupListBox::selectNextRow):
(WebCore::PopupListBox::selectPreviousRow):
(WebCore::PopupListBox::adjustSelectedIndex):
(WebCore::PopupListBox::hidePopup):
(WebCore::PopupListBox::updateFromElement):
(WebCore::PopupListBox::setMaxWidthAndLayout):
(WebCore::PopupListBox::layout):
(WebCore::PopupListBox::clear):
(WebCore::PopupListBox::isPointInBounds):
- platform/chromium/PopupListBox.h: Added.
(WebCore::PopupItem::PopupItem):
(WebCore::PopupListBox::create):
(WebCore::PopupListBox::selectedIndex):
(WebCore::PopupListBox::numItems):
(WebCore::PopupListBox::setBaseWidth):
(WebCore::PopupListBox::setMaxHeight):
(WebCore::PopupListBox::setMaxWidth):
(WebCore::PopupListBox::disconnectClient):
(WebCore::PopupListBox::items):
(WebCore::PopupListBox::~PopupListBox):
(WebCore::PopupListBox::scrollToRevealSelection):
- platform/chromium/PopupMenuChromium.cpp:
- platform/chromium/PopupMenuChromium.h:
Source/WebKit/chromium:
- src/ChromeClientImpl.cpp:
- src/WebPopupMenuImpl.cpp:
- src/WebViewImpl.cpp:
- 11:06 AM Changeset in webkit [92975] by
-
- 1 edit2 copies in branches/chromium/835
Merge 92798
BUG=91972
Review URL: http://codereview.chromium.org/7634029
- 10:51 AM Changeset in webkit [92974] by
-
- 10 edits4 adds in trunk
Move compiler specific macros to their own header
https://bugs.webkit.org/show_bug.cgi?id=66119
Reviewed by Anders Carlsson.
Source/JavaScriptCore:
- JavaScriptCore.gypi:
- JavaScriptCore.vcproj/WTF/WTF.vcproj:
- JavaScriptCore.xcodeproj/project.pbxproj:
- wtf/CMakeLists.txt:
Add Compiler.h
- wtf/AlwaysInline.h:
Move the contents of this file (which no longer was just about ALWAYS_INLINE) to
Compiler.h. We can remove this file in a later commit.
- wtf/Compiler.h: Added.
Put all compiler specific checks and features in this file.
- wtf/Platform.h:
Move COMPILER macro and definitions (and the odd WARN_UNUSED_RETURN compiler feature)
to Compiler.h. Include Compiler.h since it is necessary.
Source/JavaScriptGlue:
- ForwardingHeaders/wtf/Compiler.h: Added.
Source/WebCore:
- ForwardingHeaders/wtf/Compiler.h: Added.
Tools:
- DumpRenderTree/ForwardingHeaders/wtf/Compiler.h: Added.
- 10:36 AM Changeset in webkit [92973] by
-
- 5 edits in trunk/Source/WebCore
[Qt][REGRESSION] composited content doesn't render since r92651
https://bugs.webkit.org/show_bug.cgi?id=66108
Reviewed by Benjamin Poulain.
Changed the signature for GraphicsLayerTextureMapper::syncCompositingState to match
the new signature in GraphicsLayer.h.
No new tests. This is a regression that's covered by existing pixel tests.
- platform/graphics/texmap/GraphicsLayerTextureMapper.cpp:
(WebCore::GraphicsLayerTextureMapper::syncCompositingState):
- platform/graphics/texmap/GraphicsLayerTextureMapper.h:
- 10:34 AM Changeset in webkit [92972] by
-
- 3 edits in trunk/Tools
Add a way to extend DOM objects in garden-o-matic.
https://bugs.webkit.org/show_bug.cgi?id=66096
Reviewed by Adam Barth.
- BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/base.js: Added base.extends.
- BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/base_unittests.js: Added tests.
- 10:33 AM Changeset in webkit [92971] by
-
- 1 edit2 copies in branches/chromium/835
Merge 92791
BUG=91362
Review URL: http://codereview.chromium.org/7634026
- 10:28 AM Changeset in webkit [92970] by
-
- 3 edits in trunk/Source/WebKit/chromium
Expose pageScaleFactor in Chromium's WebHistoryItem
https://bugs.webkit.org/show_bug.cgi?id=66139
Reviewed by Darin Fisher.
- public/WebHistoryItem.h:
- src/WebHistoryItem.cpp:
(WebKit::WebHistoryItem::pageScaleFactor):
(WebKit::WebHistoryItem::setPageScaleFactor):
- 10:22 AM Changeset in webkit [92969] by
-
- 19 edits2 copies27 adds in trunk
Original patch by Mihnea Ovidenie <mihnea@adobe.com>.
Also contains some improvements done by Dave Hyatt <hyatt@apple.com>.
[CSSRegions]RenderFlowThread should display its content using RenderRegion
https://bugs.webkit.org/show_bug.cgi?id=65627
Source/WebCore:
RenderFlowThread collects RenderObjects from a flow. These objects are displayed by means
of RenderRegion objects that get the content from the same flow.
Patch by Alexandru Chiculita <Alexandru Chiculita> on 2011-08-12
Reviewed by David Hyatt.
Tests: fast/regions/content-flowed-into-regions-dynamically-added.html
fast/regions/content-flowed-into-regions-dynamically-removed.html
fast/regions/content-flowed-into-regions-with-dyn-index.html
fast/regions/content-flowed-into-regions-with-index-dom.html
fast/regions/content-flowed-into-regions-with-index.html
fast/regions/content-flowed-into-regions.html
fast/regions/flow-content-basic-vertical-rl.html
fast/regions/flow-content-basic-vertical.html
- dom/Node.cpp:
(WebCore::Node::diff):
- rendering/RenderFlowThread.cpp:
(WebCore::RenderFlowThread::RenderFlowThread):
(WebCore::RenderFlowThread::createFlowThreadStyle):
(WebCore::RenderFlowThread::styleDidChange):
(WebCore::compareRenderRegions):
(WebCore::RenderFlowThread::addRegionToThread):
(WebCore::RenderFlowThread::removeRegionFromThread):
(WebCore::RenderFlowThread::layout):
(WebCore::RenderFlowThread::computeLogicalWidth):
(WebCore::RenderFlowThread::computeLogicalHeight):
(WebCore::RenderFlowThread::paintIntoRegion):
- rendering/RenderFlowThread.h:
- rendering/RenderObject.cpp:
(WebCore::RenderObject::createObject):
- rendering/RenderRegion.cpp:
(WebCore::RenderRegion::RenderRegion):
(WebCore::RenderRegion::~RenderRegion):
(WebCore::RenderRegion::paint):
(WebCore::RenderRegion::styleDidChange):
- rendering/RenderRegion.h:
(WebCore::RenderRegion::setRegionRect):
(WebCore::RenderRegion::regionRect):
- rendering/RenderTreeAsText.cpp:
(WebCore::writeLayers):
- rendering/RenderView.cpp:
(WebCore::RenderView::styleDidChange):
(WebCore::RenderView::renderFlowThreadWithName):
- rendering/RenderView.h:
LayoutTests:
Also updated the old tests that expected an 100% by 100% RenderFlowThread. Those tests
require at least a region attached, otherwise the RenderFlowThread would have been 0/0 in size,
and no output would be generated for them in the render tree.
The attached regions are now printed in the render tree, so some tests required updated results.
Patch by Alexandru Chiculita <Alexandru Chiculita> on 2011-08-12
Reviewed by David Hyatt.
- fast/regions/content-flowed-into-regions-dynamically-added-expected.png: Added.
- fast/regions/content-flowed-into-regions-dynamically-added-expected.txt: Added.
- fast/regions/content-flowed-into-regions-dynamically-added.html: Added.
- fast/regions/content-flowed-into-regions-dynamically-removed-expected.png: Added.
- fast/regions/content-flowed-into-regions-dynamically-removed-expected.txt: Added.
- fast/regions/content-flowed-into-regions-dynamically-removed.html: Added.
- fast/regions/content-flowed-into-regions-expected.png: Added.
- fast/regions/content-flowed-into-regions-expected.txt: Added.
- fast/regions/content-flowed-into-regions-with-dyn-index-expected.png: Added.
- fast/regions/content-flowed-into-regions-with-dyn-index-expected.txt: Added.
- fast/regions/content-flowed-into-regions-with-dyn-index.html: Added.
- fast/regions/content-flowed-into-regions-with-index-dom-expected.png: Added.
- fast/regions/content-flowed-into-regions-with-index-dom-expected.txt: Added.
- fast/regions/content-flowed-into-regions-with-index-dom.html: Added.
- fast/regions/content-flowed-into-regions-with-index-expected.png: Added.
- fast/regions/content-flowed-into-regions-with-index-expected.txt: Added.
- fast/regions/content-flowed-into-regions-with-index.html: Added.
- fast/regions/content-flowed-into-regions.html: Added.
- fast/regions/flow-content-basic-vertical-rl.html: Copied from LayoutTests/fast/regions/flow-content-basic.html.
- fast/regions/flow-content-basic-vertical.html: Copied from LayoutTests/fast/regions/flow-content-basic.html.
- fast/regions/flow-content-basic.html:
- fast/regions/render-region-renderer-expected.txt:
- fast/regions/resources/region-style.css: Added.
(.greenBox):
(.blueBox):
(.orangeBox):
(.flowArticle):
(.regionArticle):
(.borderBox):
- fast/regions/webkit-flow-renderer-expected.txt:
- fast/regions/webkit-flow-renderer-layer-expected.txt:
- fast/regions/webkit-flow-renderer-layer.html:
- fast/regions/webkit-flow-renderer-nested-expected.txt:
- fast/regions/webkit-flow-renderer-nested.html:
- fast/regions/webkit-flow-renderer.html:
- platform/mac/fast/regions/flow-content-basic-expected.png: Added.
- platform/mac/fast/regions/flow-content-basic-expected.txt: Added.
- platform/mac/fast/regions/flow-content-basic-vertical-expected.png: Added.
- platform/mac/fast/regions/flow-content-basic-vertical-expected.txt: Added.
- platform/mac/fast/regions/flow-content-basic-vertical-rl-expected.png: Added.
- platform/mac/fast/regions/flow-content-basic-vertical-rl-expected.txt: Added.
- 10:17 AM Changeset in webkit [92968] by
-
- 2 edits2 copies in branches/chromium/782
Merge 92966
BUG=92621
Review URL: http://codereview.chromium.org/7640019
- 10:15 AM Changeset in webkit [92967] by
-
- 2 edits2 copies in branches/chromium/835
Merge 92966
BUG=92621
Review URL: http://codereview.chromium.org/7635022
- 9:27 AM Changeset in webkit [92966] by
-
- 4 edits2 adds in trunk
Crash in WebCore::editingIgnoresContent
https://bugs.webkit.org/show_bug.cgi?id=66125
Reviewed by Ryosuke Niwa.
Source/WebCore:
RefPtr a few nodes in case they get blown away in
dispatchEvent calls.
Test: editing/selection/select-start-remove-root-crash.html
- editing/FrameSelection.cpp:
(WebCore::FrameSelection::selectAll):
- editing/ReplaceSelectionCommand.cpp:
(WebCore::ReplacementFragment::ReplacementFragment):
LayoutTests:
Tests that we do not crash when we blow away the root
during the firing of selectstart event in selection.
- editing/selection/select-start-remove-root-crash-expected.txt: Added.
- editing/selection/select-start-remove-root-crash.html: Added.
- 9:26 AM Changeset in webkit [92965] by
-
- 2 edits1 add in trunk/Source/WebKit/chromium
[chromium] Add WebThread to WebKitClient
https://bugs.webkit.org/show_bug.cgi?id=66013
Reviewed by Darin Fisher.
- public/WebKitClient.h:
(WebKit::WebKitClient::createThread):
- public/WebThread.h: Added.
(WebKit::WebThread::~WebThread):
- 8:34 AM Changeset in webkit [92964] by
-
- 3 edits in trunk/Source/WebCore
Web Inspector: properly update console message count in source frames and resources panel.
https://bugs.webkit.org/show_bug.cgi?id=57009
Reviewed by Yury Semikhatsky.
- inspector/front-end/ConsoleView.js:
(WebInspector.ConsoleView.prototype._registerConsoleDomainDispatcher.dispatcher.messageRepeatCountUpdated):
- inspector/front-end/SourceFrame.js:
(WebInspector.SourceFrame.prototype.show):
(WebInspector.SourceFrame.prototype.addMessageToSource):
(WebInspector.SourceFrame.prototype._updateMessageRepeatCount):
- 8:15 AM Changeset in webkit [92963] by
-
- 2 edits in trunk/Source/WebKit/qt
[Qt] Add test for correct order of load signals in QWebPage
https://bugs.webkit.org/show_bug.cgi?id=66016
Reviewed by Benjamin Poulain.
Add API test to ensure the order of load signals: loadStarted() needs to be emitted
first, then loadProgress(100), followed by loadFinished().
The test is skipped since this right now is broken, the bug
https://bugs.webkit.org/show_bug.cgi?id=28851 tracks one possible way to fix.
- tests/qwebpage/tst_qwebpage.cpp:
(SpyForLoadSignalsOrder::SpyForLoadSignalsOrder):
(SpyForLoadSignalsOrder::isFinished):
(SpyForLoadSignalsOrder::onLoadProgress):
(tst_QWebPage::loadSignalsOrder_data):
(tst_QWebPage::loadSignalsOrder):
- 7:56 AM Changeset in webkit [92962] by
-
- 2 edits in trunk/Source/WebCore
2011-08-12 Pavel Feldman <pfeldman@google.com>
Not reviewed: follow up to inspector test breakage.
- inspector/front-end/ScriptsPanel.js: (WebInspector.ScriptsPanel.prototype._showScriptFoldersSettingChanged):
- 7:50 AM Changeset in webkit [92961] by
-
- 12 edits in trunk
Web Inspector: expand exception properties when wrapping it as object.
https://bugs.webkit.org/show_bug.cgi?id=66035
Source/WebCore:
Use toString() value as a description for value thrown during eval.
Reviewed by Pavel Feldman.
- inspector/InjectedScriptSource.js:
(.):
LayoutTests:
Reviewed by Pavel Feldman.
- inspector/console/console-eval-global-expected.txt:
- inspector/console/console-eval-syntax-error-expected.txt:
- inspector/console/console-eval-undefined-override-expected.txt:
- inspector/extensions/extensions-eval-expected.txt:
- inspector/runtime/runtime-setPropertyValue-expected.txt:
- platform/chromium/inspector/console/console-eval-global-expected.txt:
- platform/chromium/inspector/console/console-eval-syntax-error-expected.txt:
- platform/chromium/inspector/console/console-eval-undefined-override-expected.txt:
- platform/chromium/inspector/debugger/debugger-activation-crash2-expected.txt:
- 7:33 AM Changeset in webkit [92960] by
-
- 2 edits in trunk/Source/WebKit/qt
[Qt] Make sure QtWebKit correctly compiles when building WebKit2 with Qt5.
In order to build correctly the QtQuick1 plugin we need to explicitly
link to the new library added in Qt5 "qtquick1".
Reviewed by Simon Hausmann.
- declarative/declarative.pro:
- 7:05 AM Changeset in webkit [92959] by
-
- 1 edit38 adds in trunk/LayoutTests
2011-08-12 Andrei Popescu <andreip@google.com>
[Chromium] Rebaseline these tests on Leopard.
Unreviewed gardening.
- LayoutTests/platform/chromium-linux/platform/mac-snowleopard/platform/mac/fast/text/international/Geeza-Pro-vertical-metrics-adjustment-expected.png: Added.
- LayoutTests/platform/chromium-mac-leopard/platform/mac-snowleopard/platform/mac/fast/text/international/Geeza-Pro-vertical-metrics-adjustment-expected.png: Added.
- LayoutTests/platform/chromium-mac-leopard/platform/mac-snowleopard/platform/mac/fast/text/international/Geeza-Pro-vertical-metrics-adjustment-expected.txt: Added.
- LayoutTests/platform/chromium-mac/platform/mac-snowleopard/platform/mac/fast/text/international/Geeza-Pro-vertical-metrics-adjustment-expected.png: Added.
- LayoutTests/platform/chromium-mac/platform/mac-snowleopard/platform/mac/fast/text/international/Geeza-Pro-vertical-metrics-adjustment-expected.txt: Added.
- LayoutTests/platform/chromium-win-xp/platform/mac-snowleopard/platform/mac/fast/text/international/Geeza-Pro-vertical-metrics-adjustment-expected.png: Added.
- LayoutTests/platform/chromium-win/platform/mac-snowleopard/platform/mac/fast/text/international/Geeza-Pro-vertical-metrics-adjustment-expected.png: Added.
- LayoutTests/platform/chromium-win/platform/mac-snowleopard/platform/mac/fast/text/international/Geeza-Pro-vertical-metrics-adjustment-expected.txt: Added.
- 6:54 AM Changeset in webkit [92958] by
-
- 10 edits in trunk
Web Inspector: make folders optional in the Scripts' panel file selector.
https://bugs.webkit.org/show_bug.cgi?id=66100
Reviewed by Yury Semikhatsky.
Source/WebCore:
- English.lproj/localizedStrings.js:
- inspector/front-end/ScriptsPanel.js:
(WebInspector.ScriptsPanel.prototype._showScriptFoldersSettingChanged):
(WebInspector.ScriptsPanel.prototype._addOptionToFilesSelect.insertOrdered.optionCompare):
(WebInspector.ScriptsPanel.prototype._addOptionToFilesSelect.insertOrdered):
(WebInspector.ScriptsPanel.prototype._addOptionToFilesSelect):
(WebInspector.ScriptsPanel.prototype.reset):
(WebInspector.ScriptsPanel.prototype._resetFilesSelect):
- inspector/front-end/Settings.js:
(WebInspector.Settings):
- inspector/front-end/SettingsScreen.js:
(WebInspector.SettingsScreen):
LayoutTests:
- http/tests/inspector/debugger-test.js:
(initialize_DebuggerTest):
- inspector/debugger/scripts-sorting-expected.txt:
- inspector/debugger/scripts-sorting.html:
- platform/chromium/inspector/debugger/scripts-sorting-expected.txt:
- 6:03 AM Changeset in webkit [92957] by
-
- 5 edits in trunk
Web Inspector: add space between error info and error message in source frame message bubble.
https://bugs.webkit.org/show_bug.cgi?id=65069
Reviewed by Pavel Feldman.
Source/WebCore:
- inspector/front-end/ConsoleView.js:
(WebInspector.ConsoleMessage.prototype._formatMessage):
LayoutTests:
- http/tests/inspector-enabled/console-log-before-frame-navigation-expected.txt:
- inspector/debugger/source-frame-expected.txt:
- 5:56 AM Changeset in webkit [92956] by
-
- 16 edits in trunk
Web Inspector: console loses focus upon reloading the page from the inspector.
https://bugs.webkit.org/show_bug.cgi?id=66068
Reviewed by Yury Semikhatsky.
Source/WebCore:
- inspector/front-end/AuditsPanel.js:
(WebInspector.AuditsPanel.prototype._auditFinishedCallback):
(WebInspector.AuditsPanel.prototype._clearButtonClicked):
- inspector/front-end/DataGrid.js:
(WebInspector.DataGrid.prototype.revealAndSelect):
(WebInspector.DataGridNode.prototype.revealAndSelect):
- inspector/front-end/DetailedHeapshotView.js:
(WebInspector.DetailedHeapshotView.prototype._jumpToSearchResult):
- inspector/front-end/ElementsPanel.js:
(WebInspector.ElementsPanel.this.treeOutline.selectedNodeChanged):
(WebInspector.ElementsPanel.get this):
(WebInspector.ElementsPanel):
(WebInspector.ElementsPanel.prototype._reset):
(WebInspector.ElementsPanel.prototype._setDocument.selectNode):
(WebInspector.ElementsPanel.prototype._setDocument.selectLastSelectedNode):
(WebInspector.ElementsPanel.prototype._setDocument):
(WebInspector.ElementsPanel.prototype._domWordWrapSettingChanged):
(WebInspector.ElementsPanel.prototype.populateHrefContextMenu):
(WebInspector.ElementsPanel.prototype.switchToAndFocus):
(WebInspector.ElementsPanel.prototype.selectedDOMNode):
(WebInspector.ElementsPanel.prototype.selectDOMNode):
(WebInspector.ElementsPanel.prototype.updateModifiedNodes):
(WebInspector.ElementsPanel.prototype.updateBreadcrumb.selectCrumbFunction):
(WebInspector.ElementsPanel.prototype.updateBreadcrumb):
(WebInspector.ElementsPanel.prototype.updateStyles):
(WebInspector.ElementsPanel.prototype.updateMetrics):
(WebInspector.ElementsPanel.prototype.updateProperties):
(WebInspector.ElementsPanel.prototype.updateEventListeners):
(WebInspector.ElementsPanel.prototype.handleCopyEvent):
(WebInspector.ElementsPanel.prototype.updateFocusedNode):
- inspector/front-end/ElementsTreeOutline.js:
(WebInspector.ElementsTreeOutline):
(WebInspector.ElementsTreeOutline.prototype.selectedDOMNode):
(WebInspector.ElementsTreeOutline.prototype.selectDOMNode):
(WebInspector.ElementsTreeOutline.prototype.update):
(WebInspector.ElementsTreeOutline.prototype._revealAndSelectNode):
(WebInspector.ElementsTreeOutline.prototype._ondrop.callback):
(WebInspector.ElementsTreeOutline.prototype._ondrop):
(WebInspector.ElementsTreeElement.prototype._updateChildren.updateChildrenOfNode):
(WebInspector.ElementsTreeElement.prototype._updateChildren):
(WebInspector.ElementsTreeElement.prototype.onselect):
(WebInspector.ElementsTreeElement.prototype.selectOnMouseDown):
(WebInspector.ElementsTreeElement.prototype._startEditingTarget):
(WebInspector.ElementsTreeElement.prototype._startEditing):
- inspector/front-end/Panel.js:
(WebInspector.Panel.prototype.reset):
- inspector/front-end/ProfileView.js:
(WebInspector.CPUProfileView.prototype._jumpToSearchResult):
- inspector/front-end/ProfilesPanel.js:
(WebInspector.ProfilesPanel.prototype._addProfileHeader):
(WebInspector.ProfilesPanel.prototype.showProfile):
(WebInspector.ProfileSidebarTreeElement.prototype.onselect):
- inspector/front-end/ResourcesPanel.js:
(WebInspector.ResourcesPanel.prototype._initDefaultSelection.get if):
(WebInspector.ResourcesPanel.prototype._initDefaultSelection):
(WebInspector.ResourcesPanel.prototype.reset):
(WebInspector.ResourcesPanel.prototype.showResource):
- inspector/front-end/StylesSidebarPane.js:
(WebInspector.StylePropertiesSection.prototype.editingSelectorCommitted):
(WebInspector.StylePropertyTreeElement.prototype.updateTitle.linkifyURL):
- inspector/front-end/TimelinePanel.js:
(WebInspector.TimelinePanel.prototype._createTopPane):
- inspector/front-end/inspector.js:
- inspector/front-end/inspectorCommon.css:
(body):
- inspector/front-end/treeoutline.js:
(TreeOutline.prototype.revealAndSelect):
(TreeElement.prototype.selectOnMouseDown):
(TreeElement.prototype.revealAndSelect):
(TreeElement.prototype.select):
LayoutTests:
- inspector/console/command-line-api-inspect.html:
- 5:17 AM Changeset in webkit [92955] by
-
- 2 edits in trunk/LayoutTests
Source/WebCore: Add HTMLUnknownElement interface as defined in
http://www.w3.org/TR/html5/elements.html#elements-in-the-dom.
https://bugs.webkit.org/show_bug.cgi?id=41841
Unreviewed gardening after r92890.
- platform/qt-wk2/fast/dom/Window/window-properties-expected.txt: Updated.
- 5:06 AM Changeset in webkit [92954] by
-
- 2 edits in trunk/Source/WebCore
2011-08-12 Yury Semikhatsky <yurys@chromium.org>
Unreviewed. Inspector clean-up: remove unused method declaration from InspectorAgent.h
- inspector/InspectorAgent.h:
- 4:44 AM Changeset in webkit [92953] by
-
- 29 edits in trunk
Web Inspector: Network resource identifier should have RequestId type and requestId name in protocol.
https://bugs.webkit.org/show_bug.cgi?id=66061
Reviewed by Pavel Feldman.
Source/WebCore:
- inspector/ConsoleMessage.cpp:
(WebCore::ConsoleMessage::ConsoleMessage):
(WebCore::ConsoleMessage::addToFrontend):
- inspector/ConsoleMessage.h:
- inspector/IdentifiersFactory.cpp:
(WebCore::IdentifiersFactory::requestId):
- inspector/IdentifiersFactory.h:
- inspector/Inspector.json:
- inspector/InspectorConsoleAgent.cpp:
(WebCore::InspectorConsoleAgent::didReceiveResponse):
(WebCore::InspectorConsoleAgent::didFailLoading):
- inspector/InspectorResourceAgent.cpp:
(WebCore::InspectorResourceAgent::willSendRequest):
(WebCore::InspectorResourceAgent::markResourceAsCached):
(WebCore::InspectorResourceAgent::didReceiveResponse):
(WebCore::InspectorResourceAgent::didReceiveData):
(WebCore::InspectorResourceAgent::didFinishLoading):
(WebCore::InspectorResourceAgent::didFailLoading):
(WebCore::InspectorResourceAgent::didLoadResourceFromMemoryCache):
(WebCore::InspectorResourceAgent::setInitialScriptContent):
(WebCore::InspectorResourceAgent::didReceiveScriptResponse):
(WebCore::InspectorResourceAgent::setInitialXHRContent):
(WebCore::InspectorResourceAgent::didReceiveXHRResponse):
(WebCore::InspectorResourceAgent::didCreateWebSocket):
(WebCore::InspectorResourceAgent::willSendWebSocketHandshakeRequest):
(WebCore::InspectorResourceAgent::didReceiveWebSocketHandshakeResponse):
(WebCore::InspectorResourceAgent::didCloseWebSocket):
(WebCore::InspectorResourceAgent::getResourceContent):
- inspector/InspectorResourceAgent.h:
- inspector/InspectorTimelineAgent.cpp:
(WebCore::InspectorTimelineAgent::willSendResourceRequest):
(WebCore::InspectorTimelineAgent::willReceiveResourceData):
(WebCore::InspectorTimelineAgent::willReceiveResourceResponse):
(WebCore::InspectorTimelineAgent::didFinishLoadingResource):
- inspector/NetworkResourcesData.cpp:
(WebCore::NetworkResourcesData::ResourceData::ResourceData):
(WebCore::NetworkResourcesData::resourceCreated):
(WebCore::NetworkResourcesData::responseReceived):
(WebCore::NetworkResourcesData::setResourceType):
(WebCore::NetworkResourcesData::resourceType):
(WebCore::NetworkResourcesData::setResourceContent):
(WebCore::NetworkResourcesData::maybeAddResourceData):
(WebCore::NetworkResourcesData::maybeDecodeDataToContent):
(WebCore::NetworkResourcesData::addCachedResource):
(WebCore::NetworkResourcesData::addResourceSharedBuffer):
(WebCore::NetworkResourcesData::data):
(WebCore::NetworkResourcesData::clear):
(WebCore::NetworkResourcesData::ensureNoDataForRequestId):
(WebCore::NetworkResourcesData::ensureFreeSpace):
- inspector/NetworkResourcesData.h:
(WebCore::NetworkResourcesData::ResourceData::requestId):
- inspector/TimelineRecordFactory.cpp:
(WebCore::TimelineRecordFactory::createResourceSendRequestData):
(WebCore::TimelineRecordFactory::createResourceReceiveResponseData):
(WebCore::TimelineRecordFactory::createResourceFinishData):
(WebCore::TimelineRecordFactory::createReceiveResourceData):
- inspector/TimelineRecordFactory.h:
- inspector/front-end/ConsoleView.js:
(WebInspector.ConsoleView.prototype._registerConsoleDomainDispatcher.dispatcher.messageAdded):
- inspector/front-end/ExtensionAPI.js:
(WebInspector.injectedExtensionAPI.Resources.prototype.getHAR):
- inspector/front-end/ExtensionServer.js:
(WebInspector.ExtensionServer):
(WebInspector.ExtensionServer.prototype._notifyResourceFinished):
(WebInspector.ExtensionServer.prototype._onGetHAR):
(WebInspector.ExtensionServer.prototype._requestId):
- inspector/front-end/NetworkManager.js:
(WebInspector.NetworkManager.prototype.requestContent):
(WebInspector.NetworkDispatcher.prototype.requestWillBeSent):
(WebInspector.NetworkDispatcher.prototype.resourceMarkedAsCached):
(WebInspector.NetworkDispatcher.prototype.responseReceived):
(WebInspector.NetworkDispatcher.prototype.dataReceived):
(WebInspector.NetworkDispatcher.prototype.loadingFinished):
(WebInspector.NetworkDispatcher.prototype.loadingFailed):
(WebInspector.NetworkDispatcher.prototype.resourceLoadedFromMemoryCache):
(WebInspector.NetworkDispatcher.prototype.webSocketCreated):
(WebInspector.NetworkDispatcher.prototype.webSocketWillSendHandshakeRequest):
(WebInspector.NetworkDispatcher.prototype.webSocketHandshakeResponseReceived):
(WebInspector.NetworkDispatcher.prototype.webSocketClosed):
(WebInspector.NetworkDispatcher.prototype._appendRedirect):
(WebInspector.NetworkDispatcher.prototype._startResource):
(WebInspector.NetworkDispatcher.prototype._finishResource):
(WebInspector.NetworkDispatcher.prototype._createResource):
- inspector/front-end/NetworkPanel.js:
(WebInspector.NetworkLogView.prototype._appendResource):
(WebInspector.NetworkLogView.prototype._matchResource):
(WebInspector.NetworkLogView.prototype._updateSearchMatchedListAfterRequestIdChanged):
(WebInspector.NetworkLogView.prototype.performSearch):
- inspector/front-end/Resource.js:
(WebInspector.Resource):
- inspector/front-end/TimelinePanel.js:
(WebInspector.TimelinePanel.prototype._addRecordToTimeline):
(WebInspector.TimelinePanel.prototype._findParentRecord):
(WebInspector.TimelinePanel.FormattedRecord):
LayoutTests:
- http/tests/inspector/network/download.html:
- http/tests/inspector/network/x-frame-options-deny.html:
- inspector/timeline/timeline-load.html:
- inspector/timeline/timeline-network-resource-expected.txt:
- inspector/timeline/timeline-network-resource.html:
- inspector/timeline/timeline-test.js:
- platform/chromium/inspector/timeline/timeline-network-resource-expected.txt:
- 3:57 AM Changeset in webkit [92952] by
-
- 6 edits1 add in trunk/LayoutTests
Source/WebCore: Add HTMLUnknownElement interface as defined in
http://www.w3.org/TR/html5/elements.html#elements-in-the-dom.
https://bugs.webkit.org/show_bug.cgi?id=41841
Unreviewed gardening after r92890.
- fast/html/unknown-tag-expected.txt: Added.
- platform/qt/fast/dom/Window/window-properties-expected.txt: Updated.
- platform/qt/fast/dom/Window/window-property-descriptors-expected.txt: Updated.
- platform/qt/fast/dom/prototype-inheritance-2-expected.png: Updated.
- platform/qt/fast/dom/prototype-inheritance-2-expected.txt: Updated.
- platform/qt/fast/js/global-constructors-expected.txt: Updated.
- 2:32 AM Changeset in webkit [92951] by
-
- 5 edits in trunk
IndexedDB: Object store records don't need to have keys in all indexes
https://bugs.webkit.org/show_bug.cgi?id=66049
Reviewed by Tony Chang.
Source/WebCore:
Allow inserting records in an object store even though they don't
yield keys in some index. The spec has changed in this regard.
- storage/IDBObjectStoreBackendImpl.cpp:
(WebCore::IDBObjectStoreBackendImpl::putInternal):
LayoutTests:
Update test to make sure it's possible to add an object store record
even though it doesn't yield keys in every index.
- storage/indexeddb/index-basics-expected.txt:
- storage/indexeddb/index-basics.html:
- 1:51 AM Changeset in webkit [92950] by
-
- 12 edits in trunk
IndexedDB: Overwriting key in unique index should be possible
https://bugs.webkit.org/show_bug.cgi?id=65993
Reviewed by Tony Chang.
Source/WebCore:
It should be possible to overwrite an object store record even if
there is a derived key for that record in an index with the unique flag set.
- storage/IDBBackingStore.h:
- storage/IDBIndexBackendImpl.cpp:
(WebCore::IDBIndexBackendImpl::addingKeyAllowed):
- storage/IDBIndexBackendImpl.h:
- storage/IDBLevelDBBackingStore.cpp:
(WebCore::IDBLevelDBBackingStore::keyExistsInIndex):
- storage/IDBLevelDBBackingStore.h:
- storage/IDBObjectStoreBackendImpl.cpp:
(WebCore::IDBObjectStoreBackendImpl::putInternal):
- storage/IDBSQLiteBackingStore.cpp:
(WebCore::IDBSQLiteBackingStore::keyExistsInIndex):
- storage/IDBSQLiteBackingStore.h:
LayoutTests:
Test that it's possible to overwrite an object store record even if there
is a derived key for that record in an index with the unique flag set.
- storage/indexeddb/index-unique-expected.txt:
- storage/indexeddb/index-unique.html:
- 12:52 AM Changeset in webkit [92949] by
-
- 2 edits in trunk/Source/WebKit/efl
[EFL] Fix after r91815 - restore changes from r91659.
https://bugs.webkit.org/show_bug.cgi?id=64682
Patch by Grzegorz Czajkowski <g.czajkowski@samsung.com> on 2011-08-12
Reviewed by Kenneth Rohde Christiansen.
Restore patch for fix ewk_view_context_menu_forward_event which
has been overwrititen by mistake.
- ewk/ewk_view.cpp:
(ewk_view_context_menu_forward_event):
- 12:46 AM Changeset in webkit [92948] by
-
- 3 edits in trunk/Source/WebKit/efl
[EFL] HTML Saving feature.
https://bugs.webkit.org/show_bug.cgi?id=55455
Patch by Grzegorz Czajkowski <g.czajkowski@samsung.com> on 2011-08-12
Reviewed by Antonio Gomes.
Gets the source and location of resources for document.
TODO:
- Support others resources (css, plugins, media files).
- Currently only HTML documents are supported.
- ewk/ewk_frame.cpp:
(ewk_frame_source_get):
(ewk_frame_resources_location_get):
- ewk/ewk_frame.h:
- 12:08 AM Changeset in webkit [92947] by
-
- 1 copy in tags/Safari-534.51.14
New tag.
Aug 11, 2011:
- 11:44 PM Changeset in webkit [92946] by
-
- 18 edits2 copies5 adds in trunk
WebSocket: Implement "protocol" attribute
https://bugs.webkit.org/show_bug.cgi?id=65248
Reviewed by Kent Tamura.
Source/WebCore:
Tests: http/tests/websocket/tests/hybi/no-subprotocol.html (added)
http/tests/websocket/tests/hybi/set-protocol.html (added)
http/tests/websocket/tests/hybi/workers/no-subprotocol.html (added)
http/tests/websocket/tests/hixie76/undefined-attributes.html (updated)
http/tests/websocket/tests/hybi/multiple-subprotocols.html (updated)
http/tests/websocket/tests/hybi/workers/multiple-subprotocols.html (updated)
- websockets/ThreadableWebSocketChannel.h:
Added subprotocol() function. This function is named differently from the counterpart of
WebSocket class, because the name "protocol" can be confused with the WebSocket protocol.
Added m_useHixie76Protocol and m_subprotocol, because these value may be used after
m_channel has been released. Using bool should be fine, because boolean literals do not
appear in ambiguous context.
- websockets/ThreadableWebSocketChannelClientWrapper.cpp:
(WebCore::ThreadableWebSocketChannelClientWrapper::ThreadableWebSocketChannelClientWrapper):
(WebCore::ThreadableWebSocketChannelClientWrapper::subprotocol):
(WebCore::ThreadableWebSocketChannelClientWrapper::setSubprotocol):
- websockets/ThreadableWebSocketChannelClientWrapper.h:
- websockets/WebSocket.cpp:
(WebCore::WebSocket::WebSocket):
(WebCore::WebSocket::connect):
(WebCore::WebSocket::protocol):
The "protocol" attribute is available only when the hybi-10 protocol is chosen.
(WebCore::WebSocket::binaryType):
(WebCore::WebSocket::setBinaryType):
(WebCore::WebSocket::didConnect):
- websockets/WebSocket.h:
- websockets/WebSocket.idl:
- websockets/WebSocketChannel.cpp:
(WebCore::WebSocketChannel::subprotocol):
- websockets/WebSocketChannel.h:
- websockets/WorkerThreadableWebSocketChannel.cpp:
(WebCore::WorkerThreadableWebSocketChannel::subprotocol):
(WebCore::workerContextDidConnect):
Subprotocol value is saved in the client wrapper object after the WebSocket connection is
established.
(WebCore::WorkerThreadableWebSocketChannel::Peer::didConnect):
- websockets/WorkerThreadableWebSocketChannel.h:
LayoutTests:
- http/tests/websocket/tests/hixie76/undefined-attributes-expected.txt:
- http/tests/websocket/tests/hixie76/undefined-attributes.html:
Added a test to check "protocol" attribute.
- http/tests/websocket/tests/hybi/multiple-subprotocols-expected.txt:
- http/tests/websocket/tests/hybi/multiple-subprotocols.html:
The attribute should return an empty string at first. After the connection is established,
it should return the value of Sec-WebSocket-Protocol header in the server's handshake response.
- http/tests/websocket/tests/hybi/no-subprotocol-expected.txt: Added.
- http/tests/websocket/tests/hybi/no-subprotocol.html: Added.
- http/tests/websocket/tests/hybi/set-protocol-expected.txt: Added.
- http/tests/websocket/tests/hybi/set-protocol.html: Added.
- http/tests/websocket/tests/hybi/workers/multiple-subprotocols-expected.txt:
- http/tests/websocket/tests/hybi/workers/no-subprotocol-expected.txt: Added.
- http/tests/websocket/tests/hybi/workers/no-subprotocol.html: Added.
- http/tests/websocket/tests/hybi/workers/resources/multiple-subprotocols.js:
(ws.onopen):
(ws.onclose):
(checkAfterOnClose):
- http/tests/websocket/tests/hybi/workers/resources/no-subprotocol.js: Added.
(ws.onopen):
(ws.onclose):
(checkAfterOnClose):
- 11:29 PM Changeset in webkit [92945] by
-
- 5 edits2 copies in branches/safari-534.51-branch
Merge r92692.
- 11:27 PM Changeset in webkit [92944] by
-
- 3 edits2 copies in branches/safari-534.51-branch
Merge r92389.
- 11:25 PM Changeset in webkit [92943] by
-
- 20 edits in branches/safari-534.51-branch/Source
Merge r92384.
- 11:21 PM Changeset in webkit [92942] by
-
- 5 edits in branches/safari-534.51-branch/Source
Merge r92231.
- 11:17 PM Changeset in webkit [92941] by
-
- 3 edits2 copies in branches/safari-534.51-branch
Merge r92132.
- 11:14 PM Changeset in webkit [92940] by
-
- 6 edits4 deletes in branches/safari-534.51-branch
Merge r91965.
- 11:10 PM Changeset in webkit [92939] by
-
- 3 edits in branches/safari-534.51-branch/Source/WebKit2
Merge r91961.
- 11:08 PM Changeset in webkit [92938] by
-
- 6 edits in branches/safari-534.51-branch/Source/WebKit2
Merge r91939.
- 11:06 PM Changeset in webkit [92937] by
-
- 3 edits2 copies in branches/safari-534.51-branch
Merge r91908.
- 11:04 PM Changeset in webkit [92936] by
-
- 4 edits in branches/safari-534.51-branch/Source/WebKit2
Merge r91793.
- 11:01 PM Changeset in webkit [92935] by
-
- 3 edits2 copies in branches/safari-534.51-branch
Merge r91781.
- 10:58 PM Changeset in webkit [92934] by
-
- 1 edit in branches/safari-534.51-branch/Source/JavaScriptCore/ChangeLog
Merge r91728.
- 10:57 PM Changeset in webkit [92933] by
-
- 3 edits2 copies in branches/safari-534.51-branch
Merge r91689.
- 10:47 PM Changeset in webkit [92932] by
-
- 3 edits2 copies in branches/safari-534.51-branch
Merge r91611.
- 10:45 PM Changeset in webkit [92931] by
-
- 2 edits in branches/safari-534.51-branch/Source/WebCore
Merge r91559.
- 10:44 PM Changeset in webkit [92930] by
-
- 3 edits2 copies in branches/safari-534.51-branch
Merge r91531.
- 10:41 PM Changeset in webkit [92929] by
-
- 19 edits in branches/safari-534.51-branch
Merge r91474.
- 10:38 PM Changeset in webkit [92928] by
-
- 3 edits2 adds in trunk
[Chromium][DRT] editing/spelling/spelling-backspace-between-lines.html fails
https://bugs.webkit.org/show_bug.cgi?id=45438
Reviewed by Kent Tamura.
Tools:
MockSpellCheck was assuming the input of spellCheckWord() is a word.
But it can be a series of word. This change made spellCheckWord()
understand simple word boundary which appears in tests.
- DumpRenderTree/chromium/MockSpellCheck.cpp:
(MockSpellCheck::spellCheckWord):
LayoutTests:
Added a chromium specific expectation because chromium will use window's (wrong) expectation
unless chromium has its own.
- platform/chromium/editing/spelling/spelling-backspace-between-lines-expected.txt: Added
- 10:27 PM Changeset in webkit [92927] by
-
- 3 edits2 copies in branches/safari-534.51-branch
Merge r91270.
- 10:25 PM Changeset in webkit [92926] by
-
- 3 edits3 copies in branches/safari-534.51-branch
Merge r91152.
- 10:23 PM Changeset in webkit [92925] by
-
- 2 edits in branches/safari-534.51-branch/Source/WebKit2
Merge r91100.
- 10:22 PM Changeset in webkit [92924] by
-
- 2 edits in branches/safari-534.51-branch/Source/WebKit2
Merge r91098.
- 10:21 PM Changeset in webkit [92923] by
-
- 44 edits in branches/safari-534.51-branch/Source
Merge r91097.
- 10:18 PM Changeset in webkit [92922] by
-
- 3 edits2 moves2 adds in trunk
Implement proper handling of events with a related target in regard to shadow DOM boundaries.
https://bugs.webkit.org/show_bug.cgi?id=65899
Reviewed by Dimitri Glazkov.
Fixes issues in the following corner cases:
- When both a target node and a relatedTarget node are immediate children of
the same shadow root, an event is not dispatched.
- If a target node is an ancestor of a relatedTarget node, crossing
shadow boundaries, or vice verse, an event is not dispatched or wrongly
dispatched.
Source/WebCore:
Test: fast/dom/shadow/shadow-boundary-events.html
- dom/EventDispatcher.cpp:
(WebCore::EventDispatcher::adjustToShadowBoundaries):
LayoutTests:
- fast/dom/shadow/shadow-boundary-crossing-expected.txt: Renamed from LayoutTests/fast/events/shadow-boundary-crossing-expected.txt.
- fast/dom/shadow/shadow-boundary-crossing.html: Renamed from LayoutTests/fast/events/shadow-boundary-crossing.html.
- fast/dom/shadow/shadow-boundary-events-expected.txt: Added.
- fast/dom/shadow/shadow-boundary-events.html: Added.
- 10:13 PM Changeset in webkit [92921] by
-
- 1 edit in branches/safari-534.51-branch/Source/WebKit2/ChangeLog
Merge r91085.
- 10:12 PM Changeset in webkit [92920] by
-
- 3 edits6 copies in branches/safari-534.51-branch
Merge r91044.
- 10:10 PM Changeset in webkit [92919] by
-
- 2 edits in branches/safari-534.51-branch/Source/WebCore
Merge r91040.
- 9:57 PM Changeset in webkit [92918] by
-
- 7 edits2 copies in branches/safari-534.51-branch
Merge r90595.
- 9:54 PM Changeset in webkit [92917] by
-
- 3 edits6 copies in branches/safari-534.51-branch
Merge r89987.
- 9:39 PM Changeset in webkit [92916] by
-
- 2 edits in branches/safari-534.51-branch/Source/WebCore
Merge r89214.
- 9:32 PM Changeset in webkit [92915] by
-
- 16 edits in branches/safari-534.51-branch/Source/WebKit2
Merge r88669.
- 9:16 PM Changeset in webkit [92914] by
-
- 2 edits in trunk/Source/WebKit2
VectorArgumentCoder doesn't encode/decode enough data for more than one byte vector elements
https://bugs.webkit.org/show_bug.cgi?id=66109
Patch by Viatcheslav Ostapenko <ostapenko.viatcheslav@nokia.com> on 2011-08-11
Reviewed by Anders Carlsson.
Multiply vector size by element size in encode/decode to get correct byte size of vector data.
- Platform/CoreIPC/ArgumentCoders.h:
- 9:11 PM Changeset in webkit [92913] by
-
- 3 edits2 copies in branches/safari-534.51-branch
Merge r88434.
- 9:10 PM Changeset in webkit [92912] by
-
- 2 edits in branches/safari-534.51-branch/Source/WebCore
Merge r87875.
- 9:09 PM Changeset in webkit [92911] by
-
- 4 edits in trunk/Source/JavaScriptCore
DFG JIT-specific structure stub info code offset fields are signed
8-bit, but it is possible for the offsets to be greater than 127
https://bugs.webkit.org/show_bug.cgi?id=66122
Reviewed by Gavin Barraclough.
- bytecode/StructureStubInfo.h:
- dfg/DFGJITCodeGenerator.cpp:
(JSC::DFG::JITCodeGenerator::cachedGetById):
(JSC::DFG::JITCodeGenerator::cachedPutById):
- 9:08 PM Changeset in webkit [92910] by
-
- 3 edits2 copies in branches/safari-534.51-branch
Merge r87863.
- 8:41 PM Changeset in webkit [92909] by
-
- 2 edits in trunk/Source/JavaScriptCore
DFG JIT speculation failure code sometimes picks the wrong register
as a scratch register.
https://bugs.webkit.org/show_bug.cgi?id=66104
Reviewed by Gavin Barraclough.
Hardened the code with more assertions and fixed the bug. Now a
spilled register is only used for scratch if it also isn't being
used for shuffling.
- dfg/DFGJITCompiler.cpp:
(JSC::DFG::ShuffledRegister::handleNonCyclingPermutation):
(JSC::DFG::JITCompiler::jumpFromSpeculativeToNonSpeculative):
- 8:32 PM Changeset in webkit [92908] by
-
- 12 edits in trunk/Source
Readback composited webgl results for printing
https://bugs.webkit.org/show_bug.cgi?id=65658
Patch by John Bauman <jbauman@chromium.org> on 2011-08-11
Reviewed by James Robinson.
Source/WebCore:
The real composited results may be locked inside the compositor
context's version of a texture because the drawing buffer was
automatically cleared, so read from there to get the actual presented
version to draw.
- html/canvas/WebGLRenderingContext.cpp:
(WebCore::WebGLRenderingContext::paintRenderingResultsToCanvas):
- platform/graphics/GraphicsContext3D.h:
- platform/graphics/chromium/Extensions3DChromium.h:
- platform/graphics/chromium/WebGLLayerChromium.cpp:
(WebCore::WebGLLayerChromium::paintRenderedResultsToCanvas):
- platform/graphics/chromium/WebGLLayerChromium.h:
- platform/graphics/opengl/GraphicsContext3DOpenGL.cpp:
(WebCore::GraphicsContext3D::paintCompositedResultsToCanvas):
Source/WebKit/chromium:
Add support for reading from the composited version of a canvas.
- public/WebGraphicsContext3D.h:
- src/Extensions3DChromium.cpp:
(WebCore::Extensions3DChromium::paintFramebufferToCanvas):
- src/GraphicsContext3DChromium.cpp:
(WebCore::GraphicsContext3DInternal::GraphicsContext3DInternal):
(WebCore::GraphicsContext3DInternal::paintFramebufferToCanvas):
(WebCore::GraphicsContext3DInternal::paintRenderingResultsToCanvas):
(WebCore::GraphicsContext3DInternal::paintCompositedResultsToCanvas):
(WebCore::GraphicsContext3DInternal::paintRenderingResultsToImageData):
(WebCore::GraphicsContext3DInternal::reshape):
- src/GraphicsContext3DInternal.h:
- 8:14 PM Changeset in webkit [92907] by
-
- 4 edits in trunk/Source
REGRESSION: Qt JavaScript bridge signal connection fails
https://bugs.webkit.org/show_bug.cgi?id=66097
Patch by Andrew Wason <rectalogic@rectalogic.com> on 2011-08-11
Reviewed by Gavin Barraclough.
Source/WebCore:
Tests in WebKit/qt/tests/qwebframe
Call toThisObject() on exec->lexicalGlobalObject() so the correct
object is passed to the JS signal handler.
- bridge/qt/qt_runtime.cpp:
(JSC::Bindings::QtRuntimeConnectionMethod::call):
Source/WebKit/qt:
Unmark tests as expected failures, they pass now.
- tests/qwebframe/tst_qwebframe.cpp:
(tst_QWebFrame::connectAndDisconnect):
- 8:00 PM Changeset in webkit [92906] by
-
- 4 edits in trunk/Source
Remove incorrect comment about m_wheelEventHandlerCount
https://bugs.webkit.org/show_bug.cgi?id=66117
Reviewed by Simon Fraser.
From what I can tell, this number is changed only in
Frame::notifyChromeClientWheelEventHandlerCountChanged(),
and there it just reads the number modified in
Document::didAdd/RemoveWheelEventHandler(). So this is just the number
of event handlers (and not related to horizontal scrollbars), and the
variable name expresses that already.
Source/WebCore:
- dom/Document.h:
Source/WebKit2:
- UIProcess/WebPageProxy.h:
- 7:54 PM Changeset in webkit [92905] by
-
- 5 edits in trunk/Source
Versioning.
- 7:52 PM Changeset in webkit [92904] by
-
- 1 copy in tags/Safari-535.1
New tag.
- 7:48 PM Changeset in webkit [92903] by
-
- 2 edits in trunk/Source/JavaScriptCore
Unreviewed, rolling out r92880.
http://trac.webkit.org/changeset/92880
https://bugs.webkit.org/show_bug.cgi?id=66123
Breaks compile in VS2010 (Requested by jamesr_ on #webkit).
Patch by Sheriff Bot <webkit.review.bot@gmail.com> on 2011-08-11
- wtf/PassRefPtr.h:
- 7:11 PM Changeset in webkit [92902] by
-
- 2 edits in trunk/PerformanceTests/SunSpider
Fix sunspider-compare-results to default to the same suite version as the sunspider script.
Rubber-stamped by Maciej Stachowiak.
- sunspider-compare-results:
- 7:11 PM Changeset in webkit [92901] by
-
- 2 edits in trunk/Source/JavaScriptCore
Don't conditionalize the use of -fomit-frame-pointer on compiler version as
all of our supported compilers are now new enough to have the same, sane behavior.
Rubber-stamped by Sam Weinig.
- Configurations/JavaScriptCore.xcconfig:
- 7:07 PM Changeset in webkit [92900] by
-
- 20 edits2 moves in trunk/Source/WebCore
[chromium] Defer managed texture creation and destruction until updateCompositorResources
https://bugs.webkit.org/show_bug.cgi?id=64772
Patch by James Robinson <jamesr@chromium.org> on 2011-08-11
Reviewed by Kenneth Russell.
This adds support for deferring GraphicsContext3D calls for texture creation and destruction for managed
textures and defers these calls until updateCompositorResources() for contents textures. The primary benefit of
this change is that it decouples managing the texture budget for the next frame (which has to happen before
painting layer contents) from the actual GL calls that create/destroy textures. That way, in the threaded
compositing world we can continue to use textures from the previous frame while software painting contents for
the next frame into software buffers or SkPictures.
Also renames LayerTexture to ManagedTexture to better reflect what it is. Not all ManagedTextures we create are
necessarily associated with layers, but they are all managed by a TextureManager.
Covered by compositing/ layout tests.
- WebCore.gypi:
- platform/graphics/chromium/ContentLayerChromium.h:
- platform/graphics/chromium/ImageLayerChromium.cpp:
(WebCore::ImageLayerTextureUpdater::updateTextureRect):
- platform/graphics/chromium/LayerRendererChromium.cpp:
(WebCore::LayerRendererChromium::releaseTextures):
(WebCore::LayerRendererChromium::drawLayers):
(WebCore::LayerRendererChromium::updateLayers):
(WebCore::LayerRendererChromium::getOffscreenLayerTexture):
(WebCore::LayerRendererChromium::useRenderSurface):
(WebCore::LayerRendererChromium::initializeSharedObjects):
- platform/graphics/chromium/LayerRendererChromium.h:
- platform/graphics/chromium/LayerTextureUpdater.h:
- platform/graphics/chromium/LayerTextureUpdaterCanvas.cpp:
(WebCore::LayerTextureUpdaterBitmap::updateTextureRect):
(WebCore::LayerTextureUpdaterSkPicture::updateTextureRect):
- platform/graphics/chromium/LayerTextureUpdaterCanvas.h:
- platform/graphics/chromium/LayerTilerChromium.cpp:
(WebCore::LayerTilerChromium::getSingleTexture):
(WebCore::LayerTilerChromium::createTile):
(WebCore::LayerTilerChromium::updateRect):
(WebCore::LayerTilerChromium::drawTiles):
- platform/graphics/chromium/LayerTilerChromium.h:
(WebCore::LayerTilerChromium::Tile::Tile):
(WebCore::LayerTilerChromium::Tile::texture):
- platform/graphics/chromium/ManagedTexture.cpp: Renamed from Source/WebCore/platform/graphics/chromium/LayerTexture.cpp.
(WebCore::ManagedTexture::ManagedTexture):
(WebCore::ManagedTexture::~ManagedTexture):
(WebCore::ManagedTexture::isValid):
(WebCore::ManagedTexture::reserve):
(WebCore::ManagedTexture::unreserve):
(WebCore::ManagedTexture::bindTexture):
(WebCore::ManagedTexture::framebufferTexture2D):
- platform/graphics/chromium/ManagedTexture.h: Renamed from Source/WebCore/platform/graphics/chromium/LayerTexture.h.
(WebCore::ManagedTexture::create):
(WebCore::ManagedTexture::format):
(WebCore::ManagedTexture::isReserved):
- platform/graphics/chromium/RenderSurfaceChromium.h:
- platform/graphics/chromium/TextureManager.cpp:
(WebCore::TextureManager::TextureManager):
(WebCore::TextureManager::deleteEvictedTextures):
(WebCore::TextureManager::removeTexture):
(WebCore::TextureManager::allocateTexture):
(WebCore::TextureManager::requestTexture):
- platform/graphics/chromium/TextureManager.h:
(WebCore::TextureManager::create):
(WebCore::TextureManager::setAssociatedContextDebugOnly):
(WebCore::TextureManager::associatedContextDebugOnly):
- platform/graphics/chromium/TiledLayerChromium.h:
- platform/graphics/chromium/cc/CCHeadsUpDisplay.cpp:
(WebCore::CCHeadsUpDisplay::draw):
- platform/graphics/chromium/cc/CCHeadsUpDisplay.h:
- platform/graphics/chromium/cc/CCRenderSurface.cpp:
(WebCore::CCRenderSurface::prepareContentsTexture):
(WebCore::CCRenderSurface::drawSurface):
- platform/graphics/chromium/cc/CCRenderSurface.h:
(WebCore::CCRenderSurface::contentsTexture):
- platform/graphics/chromium/cc/CCTiledLayerImpl.cpp:
(WebCore::CCTiledLayerImpl::bindContentsTexture):
- 6:12 PM Changeset in webkit [92899] by
-
- 1 edit in branches/chromium/782/Source/WebCore/rendering/CounterNode.cpp
Merge 92630
BUG=88216
Review URL: http://codereview.chromium.org/7637006
- 5:54 PM Changeset in webkit [92898] by
-
- 3 edits in branches/chromium/782
Merge 92744
Review URL: http://codereview.chromium.org/7637005
- 5:52 PM Changeset in webkit [92897] by
-
- 2 edits in trunk/Source/WebCore
<http://webkit.org/b/66113> [CFNetwork] willCacheResponse() leaks CFCachedURLResponseRef if delegate changes CacheStoragePolicy
Reviewed by Joseph Pecoraro.
This first appeared in ToT WebKit r23462 (Windows merge).
- platform/network/cf/ResourceHandleCFNet.cpp:
(WebCore::willCacheResponse): Only retain cachedResponse if we
are returning the same object passed into the method, otherwise
we end up double-retaining the new object created.
- 5:45 PM Changeset in webkit [92896] by
-
- 2 edits in trunk/Tools
Rework some Makefile logic to remove a double-negative.
Reviewed by Jon Honeycutt.
- DumpRenderTree/mac/PerlSupport/Makefile:
- 5:22 PM Changeset in webkit [92895] by
-
- 16 edits in trunk/Source
[chromium] Make WebViewImpl point at CCLayerTreeHost and related separation
https://bugs.webkit.org/show_bug.cgi?id=65791
With this patch, LayerRendererChromium becomes increasingly responsible
for rendering, while render scheduling and tree hosting moves into the
CCLayerTreeHost.
Reviewed by James Robinson.
Source/WebCore:
- platform/graphics/chromium/ContentLayerChromium.cpp:
(WebCore::ContentLayerChromium::createTextureUpdaterIfNeeded):
- platform/graphics/chromium/LayerChromium.h:
- platform/graphics/chromium/LayerRendererChromium.cpp:
(WebCore::LayerRendererChromium::create):
(WebCore::LayerRendererChromium::LayerRendererChromium):
(WebCore::LayerRendererChromium::initialize):
(WebCore::LayerRendererChromium::releaseTextures):
(WebCore::LayerRendererChromium::updateRootLayerContents):
(WebCore::LayerRendererChromium::drawRootLayer):
(WebCore::LayerRendererChromium::invalidateRootLayerRect):
(WebCore::LayerRendererChromium::rootLayerChanged):
(WebCore::LayerRendererChromium::viewportChanged):
(WebCore::LayerRendererChromium::updateLayers):
(WebCore::LayerRendererChromium::drawLayers):
(WebCore::LayerRendererChromium::drawLayersInternal):
(WebCore::LayerRendererChromium::getFramebufferPixels):
(WebCore::LayerRendererChromium::getOffscreenLayerTexture):
(WebCore::LayerRendererChromium::copyOffscreenTextureToDisplay):
(WebCore::LayerRendererChromium::useRenderSurface):
(WebCore::LayerRendererChromium::setScissorToRect):
(WebCore::LayerRendererChromium::layerTreeAsText):
(WebCore::LayerRendererChromium::dumpRenderSurfaces):
- platform/graphics/chromium/LayerRendererChromium.h:
(WebCore::LayerRendererChromium::settings):
(WebCore::LayerRendererChromium::owner):
(WebCore::LayerRendererChromium::rootLayer):
(WebCore::LayerRendererChromium::skiaContext):
- platform/graphics/chromium/WebGLLayerChromium.cpp:
(WebCore::WebGLLayerChromium::setTextureUpdated):
- platform/graphics/chromium/cc/CCHeadsUpDisplay.cpp:
(WebCore::CCHeadsUpDisplay::draw):
(WebCore::CCHeadsUpDisplay::enabled):
- platform/graphics/chromium/cc/CCHeadsUpDisplay.h:
- platform/graphics/chromium/cc/CCLayerTreeHost.cpp:
(WebCore::CCLayerTreeHost::create):
(WebCore::CCLayerTreeHost::CCLayerTreeHost):
(WebCore::CCLayerTreeHost::initialize):
(WebCore::CCLayerTreeHost::animateAndLayout):
(WebCore::CCLayerTreeHost::createLayerTreeHostCommitter):
(WebCore::CCLayerTreeHost::createLayerTreeHostImpl):
(WebCore::CCLayerTreeHost::context):
(WebCore::CCLayerTreeHost::compositeAndReadback):
(WebCore::CCLayerTreeHost::createRootLayerPainter):
(WebCore::CCLayerTreeHost::finishAllRendering):
(WebCore::CCLayerTreeHost::invalidateRootLayerRect):
(WebCore::CCLayerTreeHost::setNeedsCommitAndRedraw):
(WebCore::CCLayerTreeHost::setNeedsRedraw):
(WebCore::CCLayerTreeHost::setRootLayer):
(WebCore::CCLayerTreeHost::setViewport):
(WebCore::CCLayerTreeHost::setVisible):
(WebCore::CCLayerTreeHost::doComposite):
(WebCore::CCLayerTreeHost::composite):
(WebCore::CCLayerTreeHost::reallocateRenderer):
- platform/graphics/chromium/cc/CCLayerTreeHost.h:
(WebCore::CCLayerTreeHost::animating):
(WebCore::CCLayerTreeHost::setAnimating):
(WebCore::CCLayerTreeHost::rootLayer):
(WebCore::CCLayerTreeHost::settings):
(WebCore::CCLayerTreeHost::viewportContentRect):
(WebCore::CCLayerTreeHost::viewportScrollPosition):
(WebCore::CCLayerTreeHost::viewportVisibleRect):
- platform/graphics/chromium/cc/CCLayerTreeHostImpl.cpp:
(WebCore::CCLayerTreeHostImpl::create):
(WebCore::CCLayerTreeHostImpl::CCLayerTreeHostImpl):
(WebCore::CCLayerTreeHostImpl::drawLayers):
(WebCore::CCLayerTreeHostImpl::drawLayersOnMainThread):
- platform/graphics/chromium/cc/CCLayerTreeHostImpl.h:
- platform/graphics/chromium/cc/CCLayerTreeHostImplProxy.cpp:
(WebCore::CCLayerTreeHostImplProxy::initImplOnCCThread):
Source/WebKit/chromium:
- src/WebViewImpl.cpp:
(WebKit::WebViewImpl::WebViewImpl):
(WebKit::WebViewImpl::resize):
(WebKit::WebViewImpl::animate):
(WebKit::WebViewImpl::doPixelReadbackToCanvas):
(WebKit::WebViewImpl::paint):
(WebKit::WebViewImpl::composite):
(WebKit::WebViewImpl::setRootPlatformLayer):
(WebKit::WebViewImpl::setRootLayerNeedsDisplay):
(WebKit::WebViewImpl::scrollRootLayerRect):
(WebKit::WebViewImpl::invalidateRootLayerRect):
(WebKit::getCompositorSettings):
(WebKit::WebViewImpl::setIsAcceleratedCompositingActive):
(WebKit::WebViewImpl::createRootLayerPainter):
(WebKit::WebViewImpl::animateAndLayout):
(WebKit::WebViewImpl::didRecreateGraphicsContext):
(WebKit::WebViewImpl::scheduleComposite):
(WebKit::WebViewImpl::updateLayerTreeViewport):
(WebKit::WebViewImpl::graphicsContext3D):
(WebKit::WebViewImpl::setVisibilityState):
- src/WebViewImpl.h:
- 5:20 PM Changeset in webkit [92894] by
-
- 2 edits in trunk/Source/WebCore
WebCore::AVFWrapper fails to remove observer for kCACFContextNeedsFlushNotification
https://bugs.webkit.org/show_bug.cgi?id=66116
We need to pass 0 as the object parameter to CFNotificationCenterRemoveObserver() when removing
kCACFContextNeedsFlushNotification to match what we do when we registered for the same
notification with CFNotificationCenterAddObserver().
Reviewed by John Sullivan.
No new tests, uses existing media tests.
- platform/graphics/avfoundation/MediaPlayerPrivateAVFoundationCF.cpp:
(WebCore::AVFWrapper::disconnectAndDeleteAVFWrapper): Pass 0 as the object parameter to CFNotificationCenterRemoveObserver().
(WebCore::AVFWrapper::notificationCallback): Remove obsolete FIXME comment.
- 5:18 PM Changeset in webkit [92893] by
-
- 7 edits in trunk/Source/WebKit/chromium
[chromium] Expose "min/max scroll size", "has horizontal/vertical scrollbar", "number of wheel handlers" to clients
https://bugs.webkit.org/show_bug.cgi?id=66094
Reviewed by James Robinson.
We want to implement two-finger-swipe for history like safari does on
os x lion. The way this works is:
- If a page has no horizontal scrollbars, or the page is all the way to the left, scrolling to the left will go back in history instead.
- Same for right/forward.
This is modelled after the WebKit2 functions
WKPageIsPinnedToLeftSide(), WKPageIsPinnedToRightSide(),
WKPageHasHorizontalScrollbar(), and
webPageProxy::willHandleHorizontalScrollEvents().
- public/WebFrame.h:
- public/WebViewClient.h:
(WebKit::WebViewClient::numWheelEventHandlersChanged):
- src/ChromeClientImpl.cpp:
(WebKit::ChromeClientImpl::numWheelEventHandlersChanged):
- src/ChromeClientImpl.h:
- src/WebFrameImpl.cpp:
(WebKit::WebFrameImpl::hasHorizontalScrollbar):
(WebKit::WebFrameImpl::hasVerticalScrollbar):
(WebKit::WebFrameImpl::isPinnedToLeft):
(WebKit::WebFrameImpl::isPinnedToRight):
- src/WebFrameImpl.h:
- 5:11 PM Changeset in webkit [92892] by
-
- 4 edits in trunk/Source/JavaScriptCore
DFG JIT verbose mode does not report the generated types of nodes
https://bugs.webkit.org/show_bug.cgi?id=65830
Reviewed by Sam Weinig.
Added code that prints the type selected for each node's result.
- dfg/DFGGenerationInfo.h:
(JSC::DFG::dataFormatToString):
- dfg/DFGNonSpeculativeJIT.cpp:
(JSC::DFG::NonSpeculativeJIT::compile):
- dfg/DFGSpeculativeJIT.cpp:
(JSC::DFG::SpeculativeJIT::compile):
- 4:43 PM Changeset in webkit [92891] by
-
- 3 edits2 adds in trunk
Share code between isStyleSpanOrSpanWithOnlyStyleAttribute, isUnstyledStyleSpan,
isSpanWithoutAttributesOrUnstyleStyleSpan and replaceWithSpanOrRemoveIfWithoutAttributes
https://bugs.webkit.org/show_bug.cgi?id=66091
Reviewed by Tony Chang.
Source/WebCore:
Extracted common code as hasNoAttributeOrOnlyStyleAttribute. The only behavioral difference is that
replaceWithSpanOrRemoveIfWithoutAttributes will now remove elements with class="Apple-style-span",
for which I'm adding a test.
Test: editing/style/remove-styled-element-with-style-span.html
- editing/ApplyStyleCommand.cpp:
(WebCore::hasNoAttributeOrOnlyStyleAttribute):
(WebCore::isStyleSpanOrSpanWithOnlyStyleAttribute):
(WebCore::isUnstyledStyleSpan):
(WebCore::isSpanWithoutAttributesOrUnstyleStyleSpan):
(WebCore::ApplyStyleCommand::replaceWithSpanOrRemoveIfWithoutAttributes):
LayoutTests:
Added a test to ensure WebKit removes implicitly styled elements such as em, b, etc...
with class="Apple-style-span" when they're not necessary.
- editing/style/remove-styled-element-with-style-span-expected.txt: Added.
- editing/style/remove-styled-element-with-style-span.html: Added.
- 4:35 PM Changeset in webkit [92890] by
-
- 21 edits3 adds in trunk
Source/WebCore: Add HTMLUnknownElement interface as defined in
http://www.w3.org/TR/html5/elements.html#elements-in-the-dom.
https://bugs.webkit.org/show_bug.cgi?id=41841
Patch by Tom Zakrajsek <tomz@codeaurora.org> on 2011-08-11
Reviewed by Adam Barth.
Test: fast/html/unknown-tag.html
- CMakeLists.txt:
- CodeGenerators.pri:
- DerivedSources.cpp:
- DerivedSources.make:
- GNUmakefile.list.am:
- WebCore.gypi:
- WebCore.vcproj/WebCore.vcproj:
- WebCore.xcodeproj/project.pbxproj:
- bindings/scripts/CodeGeneratorV8.pm:
(IsDOMNodeType):
- dom/make_names.pl:
(defaultParametersHash):
(buildConstructorMap):
(printJSElementIncludes):
(printElementIncludes):
(printWrapperFunctions):
(printWrapperFactoryCppFile):
- html/HTMLTagNames.in:
- html/HTMLUnknownElement.h: Added.
(WebCore::HTMLUnknownElement::create):
(WebCore::HTMLUnknownElement::HTMLUnknownElement):
- html/HTMLUnknownElement.idl: Added.
- mathml/mathtags.in:
- page/DOMWindow.idl:
- svg/svgtags.in:
LayoutTests: Created tests for HTMLUnknownElement interface.
https://bugs.webkit.org/show_bug.cgi?id=41841.
Patch by Tom Zakrajsek <tomz@codeaurora.org> on 2011-08-11
Reviewed by Adam Barth.
- fast/dom/prototype-inheritance-expected.txt:
- fast/html/unknown-tag.html: Added.
- platform/chromium/fast/dom/prototype-inheritance-expected.txt:
- platform/gtk/fast/dom/prototype-inheritance-expected.txt:
- platform/qt/fast/dom/prototype-inheritance-expected.txt:
- platform/win/fast/dom/prototype-inheritance-expected.txt:
- 4:20 PM Changeset in webkit [92889] by
-
- 2 edits in trunk/Tools
Add a FIXME.
- TestWebKitAPI/TestsController.cpp:
(TestWebKitAPI::TestsController::TestsController):
- 4:19 PM Changeset in webkit [92888] by
-
- 3 edits in trunk/Source/WebCore
Remove dead code: borderInnerRect
https://bugs.webkit.org/show_bug.cgi?id=66087
Reviewed by Adam Barth.
Removing an old and currently unused function: RenderObject::borderInnerRect.
No new tests since I'm just pruning dead code.
- rendering/RenderObject.cpp:
- rendering/RenderObject.h:
- 4:18 PM Changeset in webkit [92887] by
-
- 2 edits in trunk/Tools
Initialize threading before running any tests
https://bugs.webkit.org/show_bug.cgi?id=66112
Reviewed by Adam Barth.
- TestWebKitAPI/TestsController.cpp:
(TestWebKitAPI::TestsController::TestsController):
- 3:56 PM Changeset in webkit [92886] by
-
- 8 edits1 copy1 add in trunk/Tools
Add unexpected-passes.html to TestFailures for marking tests as passing
https://bugs.webkit.org/show_bug.cgi?id=66102
Reviewed by Dimitri Glazkov.
- BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/garden-o-matic.html:
- BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/main.js:
- Move some code into the library so it can be shared.
- BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/model.js:
- BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/ui.js:
- BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/ui_unittests.js:
- Add round-trip unit tests.
- BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/unexpected-passes.html: Added.
- BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/unexpected-passes.js: Added.
- I'm not entirely sure this feature should remain a separate HTML file, but it seemed better than cluttering up the main HTML file with too much extra stuff.
- Scripts/webkitpy/tool/servers/gardeningserver.py:
- Scripts/webkitpy/tool/servers/gardeningserver_unittest.py:
- The gardening server couldn't handle adding PASS expectations. Now it can.
- 3:48 PM Changeset in webkit [92885] by
-
- 1 edit2 adds in trunk/LayoutTests
pfeldman says we need to rebaseline this test. The massively complex
baseline fallback graph confused him when he created a platform/mac
fallback.
- platform/chromium-win/inspector/debugger: Added.
- platform/chromium-win/inspector/debugger/scripts-panel-expected.txt: Added.
- 3:29 PM Changeset in webkit [92884] by
-
- 2 edits in trunk/Source/WebCore
Avoid adding the visible wash layer twice
https://bugs.webkit.org/show_bug.cgi?id=66098
Reviewed by Chris Marrin.
Fix two issues; only create a m_visibleTileWashLayer if we don't have one
already, and avoid adding multiple m_visibleTileWashLayers to the sublayers
array when flipping between tiled and non-tiled.
- platform/graphics/ca/GraphicsLayerCA.cpp:
(WebCore::GraphicsLayerCA::swapFromOrToTiledLayer):
- 3:02 PM Changeset in webkit [92883] by
-
- 2 edits in trunk/Source/WebCore
The PageSerializer should always use absolute URL
https://bugs.webkit.org/show_bug.cgi?id=66006
Patch by Benjamin Poulain <benjamin@webkit.org> on 2011-08-11
Reviewed by Adam Barth.
Page serializer resolves every URL for storing the resources.
The MarkupAccumulator used there should use the same rules in order to
create a valid file.
No new tests because the test infrastructure for this
does not exist with layout tests and the Chromium unit test for
the serializer are disabled.
- page/PageSerializer.cpp:
(WebCore::SerializerMarkupAccumulator::SerializerMarkupAccumulator):
- 2:59 PM Changeset in webkit [92882] by
-
- 6 edits in trunk/Tools
NRWT has wrong fallback order for Mac now that Lion exists
https://bugs.webkit.org/show_bug.cgi?id=66093
Reviewed by Adam Barth.
ORWT used a different system for fallback orders than NRWT did.
I moved win.py to match ORWT fallback orders in bug 64486.
This bug moves mac.py to match ORWT fallback order
and adds 'lion' as a supported OS version.
Because ChromiumMac shares OS version detection code with AppleMac
this also added support for 'lion' to chromium mac. Hopefully that's a good thing.
- Scripts/webkitpy/layout_tests/port/chromium_mac.py:
- Scripts/webkitpy/layout_tests/port/chromium_mac_unittest.py:
- Scripts/webkitpy/layout_tests/port/mac.py:
- Scripts/webkitpy/layout_tests/port/mac_unittest.py:
- Scripts/webkitpy/layout_tests/port/win.py:
- 2:56 PM Changeset in webkit [92881] by
-
- 1 edit3 moves1 add1 delete in trunk/LayoutTests
Move some Snow Leopard test results to the right place.
- platform/mac-snowleopard/fast/loader/file-url-mimetypes-2-expected.txt: Removed.
- platform/mac-snowleopard/fast/loader/file-url-mimetypes-expected.txt: Removed.
- platform/mac-snowleopard/fast/text/international/Geeza-Pro-vertical-metrics-adjustment-expected.txt: Removed.
- platform/mac-snowleopard/fast/text/international/Geeza-Pro-vertical-metrics-adjustment.html: Removed.
- platform/mac-snowleopard/platform/mac/fast/loader/file-url-mimetypes-expected.txt: Copied from platform/mac-snowleopard/fast/loader/file-url-mimetypes-expected.txt.
- platform/mac-snowleopard/platform/mac/fast/text/international: Added.
- platform/mac-snowleopard/platform/mac/fast/text/international/Geeza-Pro-vertical-metrics-adjustment-expected.txt: Copied from platform/mac-snowleopard/fast/text/international/Geeza-Pro-vertical-metrics-adjustment-expected.txt.
- platform/mac-snowleopard/platform/mac/fast/text/international/Geeza-Pro-vertical-metrics-adjustment.html: Copied from platform/mac-snowleopard/fast/text/international/Geeza-Pro-vertical-metrics-adjustment.html.
- 2:39 PM Changeset in webkit [92880] by
-
- 2 edits in trunk/Source/JavaScriptCore
nullptr can't be used for PassRefPtr
https://bugs.webkit.org/show_bug.cgi?id=66024
Patch by James Robinson <jamesr@chromium.org> on 2011-08-11
Reviewed by Anders Carlsson.
- wtf/PassRefPtr.h:
(WTF::PassRefPtr::PassRefPtr):
- 2:29 PM Changeset in webkit [92879] by
-
- 2 edits in trunk/LayoutTests
Record that these tests are now passing in these configurations.
- platform/chromium/test_expectations.txt:
- 2:20 PM Changeset in webkit [92878] by
-
- 1 edit2 adds in trunk/LayoutTests
2011-08-11 Pavel Feldman <pfeldman@chromium.org>
Not reviewed: inspector's scripts-panel test baseline for mac added.
- platform/mac/inspector/debugger/scripts-panel-expected.txt: Added.
- 2:08 PM Changeset in webkit [92877] by
-
- 2 edits in branches/chromium/835/Source/WebCore
Merge 92867 - [Chromium] Temporarily disable layer anti-aliasing on ChromeOS.
https://bugs.webkit.org/show_bug.cgi?id=65922
Patch by David Reveman <reveman@chromium.org> on 2011-08-11
Reviewed by James Robinson.
No new tests.
- platform/graphics/chromium/LayerTilerChromium.cpp:
(WebCore::LayerTilerChromium::draw):
TBR=commit-queue@webkit.org
Review URL: http://codereview.chromium.org/7629002
- 2:08 PM Changeset in webkit [92876] by
-
- 2 edits in trunk/Tools
Update test results now that we use absolute URLs.
- BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/checkout_unittests.js:
- 1:59 PM Changeset in webkit [92875] by
-
- 3 edits in trunk/Tools
garden-o-matic should be able to determine when tests unexpectedly pass
https://bugs.webkit.org/show_bug.cgi?id=66092
Reviewed by Dimitri Glazkov.
This patch just contains the library methods for extracting this
information. I'll add some UI in a subsequent patch.
- BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/results.js:
- BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/results_unittests.js:
- 1:43 PM Changeset in webkit [92874] by
-
- 7 edits4 adds in trunk
Enable fast path scrolling for composited iframes
https://bugs.webkit.org/show_bug.cgi?id=65056
Reviewed by James Robinson.
Source/WebCore:
iframes with composited layers for content should be able to do fast
path scrolling because they draw into their own backing. To enable
this, useSlowRepaints is modified to return true only for the cases
that apply to composited layers. The fast path scrolling function is
modified to send the invalidation to the backing itself rather than to
the window.
Renamed a number of "slow repaint" functions to be the more accurate
"cannot blit to window" as it will not always force slow path
scrolling.
- page/FrameView.cpp:
(WebCore::FrameView::reset):
(WebCore::FrameView::useSlowRepaints):
(WebCore::FrameView::useSlowRepaintsIfNotOverlapped):
(WebCore::FrameView::contentsInCompositedLayer):
(WebCore::FrameView::setCannotBlitToWindow):
(WebCore::FrameView::scrollContentsFastPath):
(WebCore::FrameView::scrollContentsSlowPath):
- page/FrameView.h:
- rendering/RenderLayer.cpp:
(WebCore::RenderLayer::cannotBlitToWindow):
- rendering/RenderLayer.h:
- rendering/RenderView.cpp:
(WebCore::RenderView::paintBoxDecorations):
LayoutTests:
Added test of iframe overlapping another iframe. Mac gets its own
incorrect expectations due to bug 63282.
- compositing/iframes/overlapped-iframe-iframe-expected.txt: Added.
- compositing/iframes/overlapped-iframe-iframe.html: Added.
- platform/chromium-gpu/compositing/iframes/overlapped-iframe-iframe-expected.txt: Added.
- platform/mac/compositing/iframes/overlapped-iframe-iframe-expected.txt: Added.
- 12:44 PM Changeset in webkit [92873] by
-
- 4 edits13 adds in trunk
Support creating compositing layers for scrollable frames and iframes
https://bugs.webkit.org/show_bug.cgi?id=55257
Reviewed by James Robinson.
Source/WebCore:
When force compositing mode is turned on, iframes with overflow will
become composited (in order to enable faster scrolling).
This patch was originally by Daniel Sievers <sievers@chromium.org>.
Tests: platform/chromium/compositing/force-compositing-mode/no-overflow-iframe-layer.html
platform/chromium/compositing/force-compositing-mode/overflow-hidden-iframe-layer.html
platform/chromium/compositing/force-compositing-mode/overflow-iframe-enter-compositing.html
platform/chromium/compositing/force-compositing-mode/overflow-iframe-layer.html
platform/chromium/compositing/force-compositing-mode/overflow-iframe-leave-compositing.html
- rendering/RenderLayerCompositor.cpp:
(WebCore::RenderLayerCompositor::cacheAcceleratedCompositingFlags):
(WebCore::RenderLayerCompositor::requiresCompositingForScrollableFrame):
- rendering/RenderLayerCompositor.h:
LayoutTests:
- platform/chromium/compositing/force-compositing-mode/no-overflow-iframe-layer-expected.txt: Added.
- platform/chromium/compositing/force-compositing-mode/no-overflow-iframe-layer.html: Added.
- platform/chromium/compositing/force-compositing-mode/overflow-hidden-iframe-layer-expected.txt: Added.
- platform/chromium/compositing/force-compositing-mode/overflow-hidden-iframe-layer.html: Added.
- platform/chromium/compositing/force-compositing-mode/overflow-iframe-enter-compositing-expected.txt: Added.
- platform/chromium/compositing/force-compositing-mode/overflow-iframe-enter-compositing.html: Added.
- platform/chromium/compositing/force-compositing-mode/overflow-iframe-layer-expected.txt: Added.
- platform/chromium/compositing/force-compositing-mode/overflow-iframe-layer.html: Added.
- platform/chromium/compositing/force-compositing-mode/overflow-iframe-leave-compositing-expected.txt: Added.
- platform/chromium/compositing/force-compositing-mode/overflow-iframe-leave-compositing.html: Added.
- platform/chromium/compositing/force-compositing-mode/resources/subframe-big.html: Added.
- platform/chromium/compositing/force-compositing-mode/resources/subframe-small.html: Added.
- 12:24 PM Changeset in webkit [92872] by
-
- 2 edits in trunk/Source/WebKit/chromium
Remove the temporary workaround added by http://trac.webkit.org/changeset/51338
https://bugs.webkit.org/show_bug.cgi?id=65908
Patch by Marja Hölttä <marja@chromium.org> on 2011-08-11
Reviewed by Adam Barth.
SecurityOrigin for a file URL returns "file://", and
SecurityOrigin::createFromString("file://") creates a unique (null)
security origin (because "file://" is first canonicalized to "file:///"
and that is a directory). This means the conversion SecurityOrigin ->
string -> SecurityOrigin doens't give back the same SecurityOrigin in
case of file URLs.
WebStorageNamespaceImpl::createStorageArea contains a workaround which
changes the string "file://" into "file:///a" before giving it to
SecurityOrigin::createFromString.
If SecurityOrigin::m_enforceFilePathSeparation is true, then
SecurityOrigin::toString() returns "null" for file URLs, and this
problem does not exist, and the workaround code is not ran.
This patch removes the workaround.
- src/WebStorageNamespaceImpl.cpp:
(WebKit::WebStorageNamespaceImpl::createStorageArea):
- 12:22 PM Changeset in webkit [92871] by
-
- 6 edits in trunk/Source/WebCore
https://bugs.webkit.org/show_bug.cgi?id=65944
Replace QuotesData::operator==() with QuotesData::equal(const QuotesData*
, const QuotesData*) because QuotesData::operator==() were using non-standard
C++ code that is not supported by all compilers.
Patch by Yong Li <yoli@rim.com> on 2011-08-11
Reviewed by Alexey Proskuryakov.
No new tests because some existing test cases can show the issue,
for example, fast/css/content/content-quotes-01.html.
- rendering/RenderQuote.cpp:
(WebCore::RenderQuote::styleDidChange):
- rendering/style/QuotesData.cpp:
(WebCore::QuotesData::equal):
- rendering/style/QuotesData.h:
- rendering/style/RenderStyle.cpp:
(WebCore::RenderStyle::setQuotes):
- rendering/style/StyleRareInheritedData.cpp:
(WebCore::StyleRareInheritedData::operator==):
- 11:51 AM WebKit Team edited by
- (diff)
- 11:08 AM Changeset in webkit [92870] by
-
- 5 edits in trunk
https://bugs.webkit.org/show_bug.cgi?id=66075
Source/WebCore:
Turn on regions and exclusions by default. Fix one stray ifdef that I missed.
Reviewed by James Robinson.
- css/CSSComputedStyleDeclaration.cpp:
(WebCore::contentToCSSValue):
LayoutTests:
Fix the broken test result for layer dumping, since at the moment flow threads don't collect
layers (this will be fixed eventually).
Unskip the regions and exclusions tests now that they are turned on by default.
Reviewed by James Robinson.
- fast/regions/webkit-flow-renderer-layer-expected.txt:
- platform/mac/Skipped:
- 10:57 AM Changeset in webkit [92869] by
-
- 2 edits in trunk/LayoutTests
[Qt] fast/dom/HTMLDocument/hasFocus.html fails
https://bugs.webkit.org/show_bug.cgi?id=66076
Unreviewed gardening.
- platform/qt/Skipped:
- 10:29 AM Changeset in webkit [92868] by
-
- 2 edits in trunk/Source/JavaScriptCore
Removed unused variable in StackBounds::initialize() to resolve
compiler warning when building on QNX.
https://bugs.webkit.org/show_bug.cgi?id=66072
Patch by Daniel Bates <dbates@rim.com> on 2011-08-11
Reviewed by Antonio Gomes.
- wtf/StackBounds.cpp:
(WTF::StackBounds::initialize):
- 10:23 AM Changeset in webkit [92867] by
-
- 2 edits in trunk/Source/WebCore
[Chromium] Temporarily disable layer anti-aliasing on ChromeOS.
https://bugs.webkit.org/show_bug.cgi?id=65922
Patch by David Reveman <reveman@chromium.org> on 2011-08-11
Reviewed by James Robinson.
No new tests.
- platform/graphics/chromium/LayerTilerChromium.cpp:
(WebCore::LayerTilerChromium::draw):
- 10:19 AM Changeset in webkit [92866] by
-
- 2 edits in trunk/LayoutTests
Merge expectations to mute flakiness.
- platform/chromium/test_expectations.txt:
- 10:11 AM Changeset in webkit [92865] by
-
- 5 edits in branches/safari-534.51-branch/Source
Versioning.
- 10:08 AM Changeset in webkit [92864] by
-
- 1 copy in tags/Safari-534.51.13
New tag.
- 10:07 AM Changeset in webkit [92863] by
-
- 5 edits in branches/safari-534.51-branch/Source
Versioning.
- 10:05 AM Changeset in webkit [92862] by
-
- 1 delete in tags/Safari-534.51.13
delete tag.
- 9:56 AM Changeset in webkit [92861] by
-
- 3 edits in trunk/Tools
New expectation entries should be placed next to existing ones.
https://bugs.webkit.org/show_bug.cgi?id=66066
Reviewed by Adam Barth.
- Scripts/webkitpy/layout_tests/models/test_expectations.py: Added the notion of insertion point,
which is computed as the last found reference to the same test.
- Scripts/webkitpy/layout_tests/models/test_expectations_unittest.py: Added and adjusted unit tests.
- 9:50 AM Changeset in webkit [92860] by
-
- 1 copy in tags/Safari-534.51.13
New tag.
- 9:48 AM Changeset in webkit [92859] by
-
- 2 edits in trunk/Source/JavaScriptCore
Implementation of monotonically increasing clock on GTK
https://bugs.webkit.org/show_bug.cgi?id=62175
Patch by Devdatta Deshpande <pwjd73@motorola.com> on 2011-08-11
Reviewed by Martin Robinson.
- wtf/CurrentTime.cpp:
(WTF::monotonicallyIncreasingTime):
The default implementation of monotonicallyIncreasingTime only
guarantees the result to be non-decreasing.
If the system time is changed to past then default implementation will
still fail and WebCore timers will not fire.
- 9:47 AM Changeset in webkit [92858] by
-
- 2 edits in trunk/LayoutTests
[Qt] fast/dom/HTMLDocument/hasFocus.html failed
https://bugs.webkit.org/show_bug.cgi?id=62126
This test case passed under the latest revision, remove it from skip list.
Reviewed by Chang Shu.
- platform/qt/Skipped:
- 9:44 AM Changeset in webkit [92857] by
-
- 3 edits in trunk/Tools
Unreviewed, rolling out r92847.
http://trac.webkit.org/changeset/92847
https://bugs.webkit.org/show_bug.cgi?id=65986
Broke check-webkit-style
- Scripts/webkitpy/style/checker.py:
- Scripts/webkitpy/style/checker_unittest.py:
- 9:37 AM Changeset in webkit [92856] by
-
- 2 edits in trunk/Source/WebCore
[Qt] PageSerializer is specific to MHTML, we should not compile it
https://bugs.webkit.org/show_bug.cgi?id=66054
Patch by Benjamin Poulain <benjamin@webkit.org> on 2011-08-11
Reviewed by Noam Rosenthal.
PageSerializer is specific to MHTML. Do not build PageSerializer unless MHTML is enabled.
- WebCore.pro:
- 9:36 AM Changeset in webkit [92855] by
-
- 4 edits in trunk/Source/WebCore
Web Inspector: [refactoring] make tabbed pane a view
https://bugs.webkit.org/show_bug.cgi?id=66060
Reviewed by Pavel Feldman.
- inspector/front-end/NetworkItemView.js:
(WebInspector.NetworkItemView):
(WebInspector.NetworkItemView.prototype.show):
(WebInspector.NetworkItemView.prototype._selectTab.get if):
(WebInspector.NetworkItemView.prototype._selectTab):
- inspector/front-end/TabbedPane.js:
(WebInspector.TabbedPane):
- inspector/front-end/inspector.html:
- 8:57 AM Changeset in webkit [92854] by
-
- 2 edits in trunk/Source/WebKit/gtk/po
[GTK] create pot files in builddir, not (read only) srcdir
https://bugs.webkit.org/show_bug.cgi?id=66059
Reviewed by Gustavo Noronha Silva.
- GNUmakefile.am: write and read pot files from the build dir.
- 8:53 AM Changeset in webkit [92853] by
-
- 2 edits in trunk/Source/WebKit/gtk
Add the javascript core dynamic library for gtk-doc's introspection helper.
Patch by Gustavo Noronha Silva <Gustavo Noronha Silva> on 2011-08-11
Reviewed by Xan Lopez.
- docs/GNUmakefile.am:
- 8:46 AM Changeset in webkit [92852] by
-
- 4 edits in trunk
2011-08-11 Xan Lopez <xlopez@igalia.com>
[GTK] Update NEWS and configure.ac for 1.5.2 release
Reviewed by Gustavo Noronha.
- configure.ac: update for release.
Source/WebKit/gtk:
2011-08-11 Xan Lopez <xlopez@igalia.com>
[GTK] Update NEWS and configure.ac for 1.5.2 release
Reviewed by Gustavo Noronha.
- NEWS: update for release.
- 8:33 AM Changeset in webkit [92851] by
-
- 12 edits in trunk/Source/WebKit2
[Qt/WK2] Add initial support for viewport meta tag
https://bugs.webkit.org/show_bug.cgi?id=65975
Reviewed by Benjamin Poulain.
Redirect the viewport arguments through to the view.
- UIProcess/API/qt/qdesktopwebview.cpp:
(QDesktopWebViewPrivate::didReceiveViewportArguments):
- UIProcess/API/qt/qdesktopwebview_p.h:
- UIProcess/API/qt/qtouchwebview.cpp:
(QTouchWebView::geometryChanged):
- UIProcess/qt/QtWebPageProxy.cpp:
(QtWebPageProxy::setViewportArguments):
- UIProcess/qt/QtWebPageProxy.h:
- UIProcess/qt/TouchViewInterface.cpp:
(WebKit::TouchViewInterface::updateViewportState):
(WebKit::TouchViewInterface::didReceiveViewportArguments):
- UIProcess/qt/TouchViewInterface.h:
(WebKit::TouchViewInterface::ViewportState::ViewportState):
- UIProcess/qt/ViewInterface.h:
- UIProcess/qt/qtouchwebpageproxy.cpp:
(QTouchWebPageProxy::QTouchWebPageProxy):
- UIProcess/qt/qtouchwebpageproxy.h:
- 8:24 AM Changeset in webkit [92850] by
-
- 2 edits in trunk/Source/WebCore
Web Inspector: source frame popover simetimes pops up after user switches from Scripts panel
https://bugs.webkit.org/show_bug.cgi?id=66057
Reviewed by Pavel Feldman.
- inspector/front-end/SourceFrame.js: remove popover timer when removing popover.
(WebInspector.SourceFrame.prototype._mouseDown):
(WebInspector.SourceFrame.prototype._hidePopup):
- 7:59 AM Changeset in webkit [92849] by
-
- 5 edits in trunk/Source/WebCore
Web Inspector: Scripts panel: display the current search match index in the toolbar.
https://bugs.webkit.org/show_bug.cgi?id=66048
Reviewed by Pavel Feldman.
- English.lproj/localizedStrings.js:
- inspector/front-end/ScriptsPanel.js:
(WebInspector.ScriptsPanel.prototype.performSearch.finishedCallback):
(WebInspector.ScriptsPanel.prototype.performSearch):
(WebInspector.ScriptsPanel.prototype.jumpToNextSearchResult):
(WebInspector.ScriptsPanel.prototype.jumpToPreviousSearchResult):
- inspector/front-end/SearchController.js:
(WebInspector.SearchController.prototype.updateSearchMatchesCount):
(WebInspector.SearchController.prototype.updateCurrentMatchIndex):
(WebInspector.SearchController.prototype.activePanelChanged.performPanelSearch):
(WebInspector.SearchController.prototype.activePanelChanged):
(WebInspector.SearchController.prototype._updateSearchMatchesCountAndCurrentMatchIndex):
(WebInspector.SearchController.prototype._performSearch):
- inspector/front-end/SourceFrame.js:
(WebInspector.SourceFrame.prototype.get currentSearchResultIndex):
- 7:43 AM Changeset in webkit [92848] by
-
- 2 edits in trunk/Source/WebCore
2011-08-11 Xan Lopez <xlopez@igalia.com>
[GTK] Add another missing file to GNUmakefile
Reviewed by Gustavo Noronha.
- GNUmakefile.list.am:
- 7:40 AM Changeset in webkit [92847] by
-
- 3 edits in trunk/Tools
[Qt] autotests shouldn't require config.h in the style check
https://bugs.webkit.org/show_bug.cgi?id=65986
Remove the include rules from the style-check for Qt tests.
Since for the most part it's just a matter of using the
API provided by QtWebKit, enforcing the WebCore set of rules
only seems to make sense in the case of MIMESniffing.
Patch by Pierre Rossi <pierre.rossi@gmail.com> on 2011-08-11
Reviewed by Benjamin Poulain.
- Scripts/webkitpy/style/checker.py:
- Scripts/webkitpy/style/checker_unittest.py:
- 7:34 AM Changeset in webkit [92846] by
-
- 3 edits in trunk/Source/WebCore
2011-08-11 Xan Lopez <xlopez@igalia.com>
[GTK] Add missing files to GNUMakefile
Reviewed by Gustavo Noronha.
- GNUmakefile.am:
- GNUmakefile.list.am:
- 7:15 AM Changeset in webkit [92845] by
-
- 1 edit2 adds in trunk/LayoutTests
2011-08-11 Andrei Popescu <andreip@google.com>
[Chromium] Rebaseline these tests on Leopard.
Unreviewed gardening.
- platform/chromium-mac-leopard/editing/pasteboard/paste-text-016-expected.png: Added
- platform/chromium-mac-leopard/editing/spelling/inline_spelling_markers-expected.png: Added
- 7:02 AM Changeset in webkit [92844] by
-
- 3 edits in trunk/LayoutTests
WebKitTestRunner needs a more-complete implementation of eventSender
https://bugs.webkit.org/show_bug.cgi?id=42194
fast/encoding/mailto-always-utf-8.html needs eventSender
Move it to the common wk2 Skipped list from the qt-wk2.
Unreviewed gardening.
Patch by Zsolt Fehér <h490267@stud.u-szeged.hu> on 2011-08-11
- platform/qt-wk2/Skipped:
- platform/wk2/Skipped:
- 7:00 AM Changeset in webkit [92843] by
-
- 2 edits in trunk/Source/WebCore
WebSocket: Use PassRefPtr<> in function arguments in WorkerThreadableWebSocketChannel.cpp
https://bugs.webkit.org/show_bug.cgi?id=66047
Reviewed by Kent Tamura.
It's safe to use PassRefPtr<> in these functions because:
- CrossThreadTask knows how to pass a RefPtr<> across threads, and
- Call sites do not pass the pointers to anywhere else.
No change in functionality, thus no new tests.
- websockets/WorkerThreadableWebSocketChannel.cpp:
(WebCore::workerContextDidSend):
(WebCore::workerContextDidGetBufferedAmount):
(WebCore::workerContextDidConnect):
(WebCore::workerContextDidReceiveMessage):
(WebCore::workerContextDidStartClosingHandshake):
(WebCore::workerContextDidClose):
- 6:59 AM Changeset in webkit [92842] by
-
- 6 edits in trunk
Web Inspector: [Mac] search as you type does not work with the new indent in the script selector.
https://bugs.webkit.org/show_bug.cgi?id=66039
Reviewed by Yury Semikhatsky.
Source/WebCore:
- inspector/front-end/ScriptsPanel.js:
(WebInspector.ScriptsPanel.prototype._addOptionToFilesSelect):
- inspector/front-end/inspector.css:
(select.status-bar-item):
LayoutTests:
- inspector/debugger/scripts-panel-expected.txt:
- platform/chromium/inspector/debugger/scripts-sorting-expected.txt:
- 5:45 AM Changeset in webkit [92841] by
-
- 2 edits in trunk/Source/WebCore
Unreviewed build fix for Qt.
Make sure we build when Qt is namespaced.
- platform/graphics/gstreamer/PlatformVideoWindowPrivate.h:
- 5:44 AM Changeset in webkit [92840] by
-
- 5 edits in branches/chromium/835/Source
Merge 92269 - [Chromium] Fix OOP font loading to work on 10.6.6 and above.
https://bugs.webkit.org/show_bug.cgi?id=65543
In 10.6.6 the function used to get the unique ID for an NSFont in the
renderer was changed so it fails in the sandbox (it now tries to access
the on-disk font file). In order to work around this, we get the font
ID from the browser process.
To speed things up, we introduce 2 levels of caching in WebKit. A font
name cache where we can perform a quick lookup without the need for the
font id and a font id cache which we can only lookup in after getting
the unique ID from the browser process.
Reviewed by Kenneth Russell.
No new tests since this is not readily testable.
Source/WebCore:
- platform/chromium/PlatformBridge.h:
- platform/graphics/chromium/CrossProcessFontLoading.h:
- platform/graphics/chromium/CrossProcessFontLoading.mm:
(WebCore::MemoryActivatedFont::create):
(WebCore::MemoryActivatedFont::MemoryActivatedFont):
(WebCore::MemoryActivatedFont::~MemoryActivatedFont):
Source/WebKit/chromium:
- public/mac/WebSandboxSupport.h: Plumb font ID parameter through.
- src/PlatformBridge.cpp:
(WebCore::PlatformBridge::loadFont): ditto.
TBR=jeremy@chromium.org
Review URL: http://codereview.chromium.org/7618010
- 5:40 AM Changeset in webkit [92839] by
-
- 8 edits in trunk
[WK2] Implement waitForPolicyDelegate
https://bugs.webkit.org/show_bug.cgi?id=42330
Patch by Zsolt Fehér <h490267@stud.u-szeged.hu> on 2011-08-11
Reviewed by Csaba Osztrogonác.
Tools:
- WebKitTestRunner/InjectedBundle/Bindings/LayoutTestController.idl:
- WebKitTestRunner/InjectedBundle/InjectedBundlePage.cpp:
(WTR::navigationTypeToString):
(WTR::InjectedBundlePage::decidePolicyForNavigationAction):
- WebKitTestRunner/InjectedBundle/LayoutTestController.cpp:
(WTR::LayoutTestController::LayoutTestController):
(WTR::LayoutTestController::setCustomPolicyDelegate):
(WTR::LayoutTestController::waitForPolicyDelegate):
- WebKitTestRunner/InjectedBundle/LayoutTestController.h:
(WTR::LayoutTestController::isPolicyDelegateEnabled):
(WTR::LayoutTestController::isPolicyDelegatePermissive):
LayoutTests:
Unskip passing tests.
- platform/qt-wk2/Skipped:
- platform/wk2/Skipped:
- 5:20 AM Changeset in webkit [92838] by
-
- 11 edits2 adds in trunk
Web Inspector: use source-file-replaced instead of source-file-changed event.
https://bugs.webkit.org/show_bug.cgi?id=65551
Reviewed by Pavel Feldman.
Source/WebCore:
Test: inspector/debugger/source-frame-count.html
- inspector/front-end/DebuggerPresentationModel.js:
(WebInspector.DebuggerPresentationModel.prototype._addScript):
(WebInspector.DebuggerPresentationModel.prototype._uiSourceCodeReplaced):
- inspector/front-end/ScriptsPanel.js:
(WebInspector.ScriptsPanel.prototype._removeSourceFrame):
(WebInspector.ScriptsPanel.prototype._sourceFileReplaced):
- inspector/front-end/SourceFile.js:
(WebInspector.RawSourceCode):
(WebInspector.RawSourceCode.prototype.get uiSourceCode):
(WebInspector.RawSourceCode.prototype.reload):
- inspector/front-end/inspector.js:
(WebInspector._createPanels):
LayoutTests:
- inspector/debugger/scripts-panel-expected.txt:
- inspector/debugger/scripts-panel.html:
- inspector/debugger/source-file-expected.txt:
- inspector/debugger/source-file.html:
- inspector/debugger/source-frame-count-expected.txt: Added.
- inspector/debugger/source-frame-count.html: Added.
- platform/chromium/test_expectations.txt:
- 4:30 AM Changeset in webkit [92837] by
-
- 2 edits in trunk/LayoutTests
2011-08-11 Andrei Popescu <andreip@google.com>
Master: Support CSS3 font feature properties
https://bugs.webkit.org/show_bug.cgi?id=63796
Unreviewed gardening.
css3/font-feature-settings-rendering.html is missing expectations. Looking at the
actual test output, it appears clearly wrong so we cannot rebaseline it either.
We'll therefore update the expectations file to mark this test as missing expected
results and update the bug to say it needs investigation and fixing.
- platform/chromium/test_expectations.txt:
- 3:04 AM Changeset in webkit [92836] by
-
- 2 edits in trunk/Source/ThirdParty/ANGLE
[Qt]Fix warnings after r92805.
Reviewed by Csaba Osztrogonác.
Control reached the end non-void functions.
- src/compiler/ossource_posix.cpp:
(OS_AllocTLSIndex):
(OS_SetTLSValue):
- 2:34 AM Changeset in webkit [92835] by
-
- 2 edits in trunk/Source/WebCore
Web Inspector: [REGRESSION] Resources panel search fails when search result is found in a resource used in several frames.
https://bugs.webkit.org/show_bug.cgi?id=66007
Reviewed by Pavel Feldman.
- inspector/front-end/ResourcesPanel.js:
(WebInspector.ResourcesPanel.prototype.performSearch.callback):
(WebInspector.ResourcesPanel.prototype.performSearch):
(WebInspector.FrameTreeElement.prototype.resourceByURL):
- 2:24 AM Changeset in webkit [92834] by
-
- 2 edits in trunk/Source/WebCore
[GTK] Fix the build after r92792.
Rubber-stamped by Csaba Osztrogonác.
No new tests needed.
- GNUmakefile.list.am: add missing
RenderFlowThread.{cpp|h}
RenderRegion.{cpp|h}
- 1:59 AM Changeset in webkit [92833] by
-
- 1 edit3 adds in trunk/LayoutTests
Rebaseline these tests, as requested by morrita.
- platform/chromium-mac/editing/pasteboard/paste-text-016-expected.png: Added.
- platform/chromium-mac/editing/selection/move-by-character-005-expected.png: Added.
- platform/chromium-mac/editing/spelling/inline_spelling_markers-expected.png: Added.
- 1:54 AM Changeset in webkit [92832] by
-
- 4 edits in trunk/Tools
rebaseline-expectations should use the release bots, not the debug
bots. The debug bots don't all run all the tests. Also, delay
updating test_expectations a long as possible to avoid clearing out
lines that we'll need for other ports.
- Scripts/webkitpy/layout_tests/port/builders.py:
- Scripts/webkitpy/tool/commands/rebaseline.py:
- Scripts/webkitpy/tool/commands/rebaseline_unittest.py:
- 1:48 AM Changeset in webkit [92831] by
-
- 2 edits in trunk/LayoutTests
[Qt] Enable WebGL by default for QtWebKit
https://bugs.webkit.org/show_bug.cgi?id=65998
disable failing tests after https://trac.webkit.org/changeset/92805
- platform/qt/Skipped: Skip fast/canvas/webgl directory
- 1:45 AM Changeset in webkit [92830] by
-
- 2 edits in trunk/Source/WebCore
[Qt] Fix WebGL on Windows after r92805
https://bugs.webkit.org/show_bug.cgi?id=66042
WEBGL is temporarly disabled, because it broked the bots.
- features.pri:
- 1:18 AM Changeset in webkit [92829] by
-
- 2 edits in trunk/LayoutTests
Fix Snow Leopard expectation after r92518.
- platform/mac-snowleopard/fast/loader/file-url-mimetypes-expected.txt:
- 1:10 AM Changeset in webkit [92828] by
-
- 1 edit2 adds in trunk/LayoutTests
Test how SpeechInputResultList behaves with not enough arguments
https://bugs.webkit.org/show_bug.cgi?id=65812
Reviewed by Sam Weinig.
SpeechInputResultList is only exposed via the x-webkit-speech
attribute, which means there is little compatibility risk in aligning
our too-few-arguments behavior with the specification. There's a
question about whether all these "fake array" objects should work
consistently, but we'll probably want the "fake array" objects we
create in the future to match the spec, so even if we're consistent
now, we'll end up not being consistent in the future.
- fast/speech/speech-input-result-list-not-enough-arguments-expected.txt: Added.
- fast/speech/speech-input-result-list-not-enough-arguments.html: Added.
- 12:56 AM Changeset in webkit [92827] by
-
- 3 edits in trunk/Source/WebCore
Web Inspector: do not evaluate watch expressions on load.
https://bugs.webkit.org/show_bug.cgi?id=66002
Patch by Pavel Feldman <pfeldman@google.com> on 2011-08-11
Reviewed by Yury Semikhatsky.
- inspector/front-end/ScriptsPanel.js:
(WebInspector.ScriptsPanel.prototype.show):
(WebInspector.ScriptsPanel.prototype.hide):
(WebInspector.ScriptsPanel.prototype.reset):
- inspector/front-end/WatchExpressionsSidebarPane.js:
(WebInspector.WatchExpressionsSidebarPane):
(WebInspector.WatchExpressionsSidebarPane.prototype.hide):
(WebInspector.WatchExpressionsSidebarPane.prototype.reset):
(WebInspector.WatchExpressionsSidebarPane.prototype.refreshExpressions):
(WebInspector.WatchExpressionsSidebarPane.prototype._refreshExpressionsIfNeeded):
(WebInspector.WatchExpressionsSidebarPane.prototype._refreshButtonClicked):
- 12:25 AM Changeset in webkit [92826] by
-
- 2 edits in trunk/Source/ThirdParty/ANGLE
Build fix on Qt Windows 32-bit Release/Debug after r92805.
Unreviewed.
- src/compiler/ossource_posix.cpp:
(OS_FreeTLSIndex):
Aug 10, 2011:
- 11:57 PM Changeset in webkit [92825] by
-
- 2 edits in trunk/LayoutTests
Skip tests added by r92769 on WebKit2 because TestRunner doesn't support
drag and drop by eventSender yet.
- platform/wk2/Skipped:
- 11:40 PM Changeset in webkit [92824] by
-
- 4 edits in trunk/LayoutTests
Unreviewed pixel expectations update. New expectations now have
bad grammar markers. (The settings has been enabled since r82159.)
- platform/mac/editing/pasteboard/paste-text-016-expected.png:
- platform/mac/editing/selection/move-by-character-005-expected.png:
- platform/mac/editing/spelling/inline_spelling_markers-expected.png:
- 11:29 PM Changeset in webkit [92823] by
-
- 16 edits4 deletes in trunk
Copying can result in span around block elements on the clipboard
https://bugs.webkit.org/show_bug.cgi?id=34564
Reviewed by Tony Chang.
Source/WebCore:
Completely overhauled the way WebKit preserves style in copy and paste. Instead of wrapping the entire
serialized contents by a Apple style span, WebKit now adds inline style to the top level elements,
wrap top level text nodes by a style span.
- editing/EditingStyle.cpp:
(WebCore::EditingStyle::collapseTextDecorationProperties): Remove text-decoration property when the value
of -webkit-text-decorations-in-effect is none.
(WebCore::EditingStyle::removeStyleFromRulesAndContext): Since display: inline and float: none are now
added on copy, remove these properties on paste.
(WebCore::EditingStyle::removePropertiesInElementDefaultStyle): Takes Element* instead of StyledElement*.
(WebCore::EditingStyle::forceInline): Added.
(WebCore::getPropertiesNotIn): Remove properties only when the base style has them.
- editing/EditingStyle.h:
- editing/markup.cpp:
(WebCore::StyledMarkupAccumulator::shouldApplyWrappingStyle): Added.
(WebCore::StyledMarkupAccumulator::StyledMarkupAccumulator): Takes highestNodeToBeSerialized.
(WebCore::StyledMarkupAccumulator::wrapWithStyleNode): Calls appendStyleNodeOpenTag and styleNodeCloseTag.
(WebCore::StyledMarkupAccumulator::appendStyleNodeOpenTag): Extracted from wrapWithStyleNode.
(WebCore::StyledMarkupAccumulator::styleNodeCloseTag): Ditto.
(WebCore::StyledMarkupAccumulator::appendText): Wraps text node with a style span if needed.
Set display: inline and float: none so that it won't be converted to a block on paste side.
(WebCore::StyledMarkupAccumulator::appendElement): Add wrapping style if appropriate; Remove any properties
that are overridden by default style and any style that may conflict with the computed style of node to
avoid modifying the appearance of the serialized nodes.
(WebCore::StyledMarkupAccumulator::serializeNodes): Compute wrapping style; copies of this style are
modified as needed when serializing top-level elements or text nodes. We call traverseNodesForSerialization
with NodeTraversalMode set to DoNotEmitString first to compute the highest node to be serialized. The second
call to the function actually serialize the nodes.
(WebCore::StyledMarkupAccumulator::traverseNodesForSerialization): Extracted from serializeNodes.
Outputs string only if NodeTraversalMode is set to EmitString.
(WebCore::createMarkup): No longer adds wrapping spans.
LayoutTests:
This patch overhauled the way we preserve styles when copying and pasting HTML contents. Many tests progressed
and lost wrapping spans while others are observing different markup on the clipboard as expected.
- editing/deleting/deleting-line-break-preserves-underline-color-expected.txt: span's style attribute now has
a trailing space.
- editing/execCommand/insert-list-with-noneditable-content-expected.txt: LightGray is now rgb(211, 211, 211).
- editing/pasteboard/data-transfer-items-expected.txt: Different serialization.
- editing/pasteboard/onpaste-text-html-expected.txt: Ditto.
- editing/pasteboard/paste-4039777-fix-expected.txt: No longer nests ul erroneously or aligns the inner list
to the right.
- editing/pasteboard/paste-code-in-pre-expected.txt: No longer produces erroneous wrapping span.
- editing/pasteboard/paste-pre-001-expected.txt: Ditto.
- editing/pasteboard/paste-pre-002-expected.txt: Ditto.
- editing/pasteboard/paste-text-012-expected.txt: Ditto.
- editing/pasteboard/paste-text-with-style-4-expected.txt: Ditto.
- fast/events/ondrop-text-html-expected.txt: Different serialization.
- platform/chromium-win/editing/pasteboard/paste-code-in-pre-expected.txt: Removed.
- platform/chromium-win/editing/pasteboard/paste-pre-001-expected.txt: Removed.
- platform/chromium-win/editing/pasteboard/paste-pre-002-expected.txt: Removed.
- platform/qt-mac/editing/pasteboard/paste-code-in-pre-expected.txt: Removed.
- 11:16 PM Changeset in webkit [92822] by
-
- 2 edits in trunk/LayoutTests
Skip a test added by r92769 on Windows port since drag and drop won't work well on Windows port.
- platform/win/Skipped:
- 11:00 PM Changeset in webkit [92821] by
-
- 3 edits in trunk/LayoutTests
GTK and Qt rebaselines after r92792.
- platform/gtk/fast/css/getComputedStyle/computed-style-without-renderer-expected.txt:
- platform/qt/fast/css/getComputedStyle/computed-style-without-renderer-expected.txt:
- 10:15 PM Changeset in webkit [92820] by
-
- 2 edits in trunk/Tools
r92792 inadvertently removed PlatformSpecificScheduler for all Snow Leopard bots.
Revert it partially so that the remaining Snow Leopard bots can fulfill their duty.
- BuildSlaveSupport/build.webkit.org-config/config.json:
- 10:01 PM Changeset in webkit [92819] by
-
- 13 edits14 copies6 moves1 add3 deletes in trunk/LayoutTests
Move <input type=date> tests to fast/forms/date/
https://bugs.webkit.org/show_bug.cgi?id=66029
Reviewed by Hajime Morita.
- fast/forms/ValidityState-rangeOverflow-expected.txt: Removed "date" part.
- fast/forms/ValidityState-rangeUnderflow-expected.txt: ditto.
- fast/forms/ValidityState-stepMismatch-expected.txt: ditto.
- fast/forms/ValidityState-typeMismatch-date.html: Removed.
- fast/forms/date: Added.
- fast/forms/date-input-visible-strings-expected.txt: Removed.
- fast/forms/date-input-visible-strings.html: Removed.
- fast/forms/date/ValidityState-rangeOverflow-date-expected.txt: Copied from LayoutTests/fast/forms/ValidityState-rangeOverflow-expected.txt.
- fast/forms/date/ValidityState-rangeOverflow-date.html: Copied from LayoutTests/fast/forms/script-tests/ValidityState-rangeOverflow.js.
- fast/forms/date/ValidityState-rangeUnderflow-date-expected.txt: Copied from LayoutTests/fast/forms/ValidityState-rangeUnderflow-expected.txt.
- fast/forms/date/ValidityState-rangeUnderflow-date.html: Copied from LayoutTests/fast/forms/script-tests/ValidityState-rangeUnderflow.js.
- fast/forms/date/ValidityState-stepMismatch-date-expected.txt: Copied from LayoutTests/fast/forms/ValidityState-stepMismatch-expected.txt.
- fast/forms/date/ValidityState-stepMismatch-date.html: Copied from LayoutTests/fast/forms/script-tests/ValidityState-stepMismatch.js.
- fast/forms/date/ValidityState-typeMismatch-date-expected.txt: Copied from LayoutTests/fast/forms/ValidityState-typeMismatch-date-expected.txt.
- fast/forms/date/ValidityState-typeMismatch-date.html: Copied from LayoutTests/fast/forms/script-tests/ValidityState-typeMismatch-date.js.
- fast/forms/date/date-input-type-expected.txt: Copied from LayoutTests/fast/forms/week/week-input-type-expected.txt.
- fast/forms/date/date-input-type.html: Copied from LayoutTests/fast/forms/week/week-input-type.html.
- fast/forms/date/date-input-visible-strings-expected.txt: Copied from LayoutTests/fast/forms/date-input-visible-strings-expected.txt.
- fast/forms/date/date-input-visible-strings.html: Copied from LayoutTests/fast/forms/date-input-visible-strings.html.
- fast/forms/date/date-stepup-stepdown-expected.txt: Copied from LayoutTests/fast/forms/input-stepup-stepdown-expected.txt.
- fast/forms/date/date-stepup-stepdown-from-renderer-expected.txt: Copied from LayoutTests/fast/forms/input-stepup-stepdown-from-renderer-expected.txt.
- fast/forms/date/date-stepup-stepdown-from-renderer.html: Copied from LayoutTests/fast/forms/script-tests/input-stepup-stepdown-from-renderer.js.
- fast/forms/date/date-stepup-stepdown.html: Copied from LayoutTests/fast/forms/script-tests/input-stepup-stepdown.js.
- fast/forms/date/input-valueasdate-date-expected.txt: Copied from LayoutTests/fast/forms/input-valueasdate-date-expected.txt.
- fast/forms/date/input-valueasdate-date.html: Copied from LayoutTests/fast/forms/script-tests/input-valueasdate-date.js.
- fast/forms/date/input-valueasnumber-date-expected.txt: Copied from LayoutTests/fast/forms/input-valueasnumber-date-expected.txt.
- fast/forms/date/input-valueasnumber-date.html: Copied from LayoutTests/fast/forms/script-tests/input-valueasnumber-date.js.
- fast/forms/input-stepup-stepdown-expected.txt: Removed "date" part.
- fast/forms/input-stepup-stepdown-from-renderer-expected.txt: ditto.
- fast/forms/input-type-change3-expected.txt: ditto.
- fast/forms/input-valueasdate-date-expected.txt: Removed.
- fast/forms/input-valueasdate-date.html: Removed.
- fast/forms/input-valueasnumber-date-expected.txt: Removed.
- fast/forms/input-valueasnumber-date.html: Removed.
- fast/forms/script-tests/ValidityState-rangeOverflow.js: Removed "date" part.
- fast/forms/script-tests/ValidityState-rangeUnderflow.js: ditto.
- fast/forms/script-tests/ValidityState-stepMismatch.js: ditto.
- fast/forms/script-tests/input-stepup-stepdown-from-renderer.js: ditto.
- fast/forms/script-tests/input-stepup-stepdown.js: ditto.
- fast/forms/script-tests/input-type-change3.js: ditto.
- fast/forms/script-tests/input-valueasdate-date.js: Removed.
- fast/forms/script-tests/input-valueasnumber-date.js: Removed.
- 9:02 PM Changeset in webkit [92818] by
-
- 3 edits in trunk/LayoutTests
Test how TouchList behaves with too few arguments
https://bugs.webkit.org/show_bug.cgi?id=65815
Reviewed by Sam Weinig.
This API is very new and the compatibility cost of making this API
match the specification is minimal. We should align with the
specification and (future) implementations in other browsers by making
this argument required.
- fast/events/touch/document-create-touch-list-expected.txt:
- fast/events/touch/script-tests/document-create-touch-list.js:
- 9:00 PM Changeset in webkit [92817] by
-
- 3 edits in trunk/LayoutTests
Test how AudioBufferSourceNode reacts when given too few arguments
https://bugs.webkit.org/show_bug.cgi?id=65816
Reviewed by Sam Weinig.
The WebAudio API is extremely new. It has only been available as an
experimental API in a single browser. The compatibility cost of
tightening up these APIs is tiny and these APIs make very little sense
to call with too-few arguments.
- webaudio/audionode-expected.txt:
- webaudio/audionode.html:
- 8:58 PM Changeset in webkit [92816] by
-
- 3 edits2 adds in trunk
Add tests of optional arguments for Geolocation
https://bugs.webkit.org/show_bug.cgi?id=65810
Reviewed by Sam Weinig.
Source/WebCore:
Geolocation's treatment of not-enough-arguments almost matches the
spec. getCurrentPosition and watchPosition work properly, but, because
they are custom, we should use the prettier form of the [Optional]
attribute.
Before this series of patches, clearWatch argument was optional, but
that doesn't match the spec or other browsers. Calling this function
with zero arguments is pretty non-sensical, so there shouldn't be much
compat risk to tightening up our behavior here. Matching other
browsers and the spec seems like the bigger win. If we run into compat
problems, we can re-evaluate this decision.
Test: fast/dom/Geolocation/not-enough-arguments.html
- page/Geolocation.idl:
LayoutTests:
Test what happens when calling geolocation APIs with too few arguments.
- fast/dom/Geolocation/not-enough-arguments-expected.txt: Added.
- fast/dom/Geolocation/not-enough-arguments.html: Added.
- 7:27 PM Changeset in webkit [92815] by
-
- 2 edits in branches/chromium/782/Source/WebCore
2011-08-10 Julien Chaffraix <jchaffraix@webkit.org>
Unreviewed (4 liners - Chromium only).
Hack to remove the crash from: (properly solved on trunk)
https://bugs.webkit.org/show_bug.cgi?id=62884
This is one of Chromium's top most crashers and merging the fix from the bug (r89445)
did not help as it relies on other SVG changes being in. This change just
mitigates the crash until it naturely gets resolved when we roll WebKit revision
in Chromium.
- rendering/svg/SVGTextRunRenderingContext.cpp: (WebCore::floatWidthOfSubStringUsingSVGFont): (WebCore::SVGTextRunRenderingContext::drawTextUsingSVGFont): Added 2 null-checks here.
- 6:56 PM Changeset in webkit [92814] by
-
- 100 edits in branches/safari-534.51-branch/Source
Merged JavaScriptCore & JavaScriptGlue from 92405 to 92804.
Reviewed by nobody.
EXCEPT, Did not merge patches:
92437, 92523.
../JavaScriptGlue:
EXCEPT, Did not merge patches:
92437, 92523.
- 6:54 PM Changeset in webkit [92813] by
-
- 2 edits in trunk/LayoutTests
Skip some more tests on Lion.
- platform/mac-lion/Skipped:
- 6:29 PM Changeset in webkit [92812] by
-
- 2 edits in trunk/Source/WebCore
SVGAElement check for internal non-view anchors is inside ENABLE(SVG_ANIMATION)
https://bugs.webkit.org/show_bug.cgi?id=66026
Reviewed by Rob Buis.
Move only the SMIL-related code inside the ENABLE(SVG_ANIMATION) block.
No new tests, as it depends on the enabled features.
- svg/SVGAElement.cpp:
(WebCore::SVGAElement::defaultEventHandler):
- 6:17 PM Changeset in webkit [92811] by
-
- 10 edits2 adds in trunk/Source/WebKit/chromium
[chromium] Update WebScrollbar so that it works with overlay scrollbars on Lion
https://bugs.webkit.org/show_bug.cgi?id=66008
Reviewed by James Robinson.
- WebKit.gyp:
- public/WebScrollbar.h:
- public/WebScrollbarClient.h:
- src/ScrollbarGroup.cpp: Added.
(WebKit::ScrollbarGroup::ScrollbarGroup):
(WebKit::ScrollbarGroup::~ScrollbarGroup):
(WebKit::ScrollbarGroup::scrollbarCreated):
(WebKit::ScrollbarGroup::scrollbarDestroyed):
(WebKit::ScrollbarGroup::setLastMousePosition):
(WebKit::ScrollbarGroup::scrollSize):
(WebKit::ScrollbarGroup::scrollPosition):
(WebKit::ScrollbarGroup::setScrollOffset):
(WebKit::ScrollbarGroup::invalidateScrollbarRect):
(WebKit::ScrollbarGroup::invalidateScrollCornerRect):
(WebKit::ScrollbarGroup::isActive):
(WebKit::ScrollbarGroup::enclosingScrollableArea):
(WebKit::ScrollbarGroup::isScrollCornerVisible):
(WebKit::ScrollbarGroup::getTickmarks):
(WebKit::ScrollbarGroup::convertFromContainingViewToScrollbar):
(WebKit::ScrollbarGroup::horizontalScrollbar):
(WebKit::ScrollbarGroup::verticalScrollbar):
(WebKit::ScrollbarGroup::minimumScrollPosition):
(WebKit::ScrollbarGroup::maximumScrollPosition):
(WebKit::ScrollbarGroup::visibleHeight):
(WebKit::ScrollbarGroup::visibleWidth):
(WebKit::ScrollbarGroup::contentsSize):
(WebKit::ScrollbarGroup::overhangAmount):
(WebKit::ScrollbarGroup::currentMousePosition):
(WebKit::ScrollbarGroup::shouldSuspendScrollAnimations):
(WebKit::ScrollbarGroup::scrollbarStyleChanged):
(WebKit::ScrollbarGroup::isOnActivePage):
(WebKit::ScrollbarGroup::disconnectFromPage):
- src/ScrollbarGroup.h: Added.
(WebKit::ScrollbarGroup::scrollCornerRect):
- src/WebPluginContainerImpl.cpp:
(WebKit::WebPluginContainerImpl::paint):
(WebKit::WebPluginContainerImpl::reportGeometry):
(WebKit::WebPluginContainerImpl::scrollbarGroup):
(WebKit::WebPluginContainerImpl::willStartLiveResize):
(WebKit::WebPluginContainerImpl::willEndLiveResize):
(WebKit::WebPluginContainerImpl::handleMouseEvent):
- src/WebPluginContainerImpl.h:
- src/WebScrollbarImpl.cpp:
(WebKit::WebScrollbar::createForPlugin):
(WebKit::WebScrollbarImpl::WebScrollbarImpl):
(WebKit::WebScrollbarImpl::~WebScrollbarImpl):
(WebKit::WebScrollbarImpl::setScrollOffset):
(WebKit::WebScrollbarImpl::invalidateScrollbarRect):
(WebKit::WebScrollbarImpl::getTickmarks):
(WebKit::WebScrollbarImpl::convertFromContainingViewToScrollbar):
(WebKit::WebScrollbarImpl::scrollbarStyleChanged):
(WebKit::WebScrollbarImpl::isOverlay):
(WebKit::WebScrollbarImpl::setValue):
(WebKit::WebScrollbarImpl::scroll):
(WebKit::WebScrollbarImpl::onMouseMove):
(WebKit::WebScrollbarImpl::onMouseLeave):
(WebKit::WebScrollbarImpl::onMouseWheel):
(WebKit::WebScrollbarImpl::onKeyDown):
- src/WebScrollbarImpl.h:
(WebKit::WebScrollbarImpl::scrollOffset):
(WebKit::WebScrollbarImpl::scrollbar):
- src/WebViewImpl.cpp:
(WebKit::WebViewImpl::willStartLiveResize):
(WebKit::WebViewImpl::willEndLiveResize):
- 6:08 PM Changeset in webkit [92810] by
-
- 3 edits in trunk/Source/JavaScriptCore
Removed some incorrect code that was dead.
Reviewed by Oliver Hunt.
clearSingleTransition() wasn't resetting m_data. Luckily,
no one cares, because its caller was unused. Removed both.
- runtime/Structure.cpp:
- runtime/StructureTransitionTable.h:
(JSC::StructureTransitionTable::~StructureTransitionTable):
- 6:02 PM Changeset in webkit [92809] by
-
- 3 edits2 adds in trunk
Crash when clicking an SVG <a> link to the local document
https://bugs.webkit.org/show_bug.cgi?id=66019
<rdar://problem/9933311>
Reviewed by Rob Buis.
Source/WebCore:
Check if the target element of an internal link is valid before
making use of it.
Test: svg/custom/click-internal-anchor-with-use-crash.xhtml
- svg/SVGAElement.cpp:
(WebCore::SVGAElement::defaultEventHandler):
LayoutTests:
Add a test which programmatically clicks a link referring to "#",
and should crash between r89745 and this commit.
- svg/custom/click-internal-anchor-with-use-crash-expected.txt: Added.
- svg/custom/click-internal-anchor-with-use-crash.xhtml: Added.
- 5:54 PM Changeset in webkit [92808] by
-
- 4 edits in trunk/Source/WebKit/efl
[EFL] Return from _ewk_view_smart_add if smart data can not be allocated
https://bugs.webkit.org/show_bug.cgi?id=65853
Patch by Grzegorz Czajkowski <g.czajkowski@samsung.com> on 2011-08-10
Reviewed by Kenneth Rohde Christiansen.
It prevents the crash while allocating a memory for view's smart data.
Others checking are needed for support single and tiled backing store.
As a result an empty window will be displayed.
- ewk/ewk_view.cpp:
(_ewk_view_smart_add):
(_ewk_view_smart_resize):
(_ewk_view_smart_move):
(_ewk_view_smart_show):
(_ewk_view_smart_hide):
- ewk/ewk_view_single.c:
(_ewk_view_single_smart_add):
(_ewk_view_single_smart_resize):
- ewk/ewk_view_tiled.c:
(_ewk_view_tiled_smart_add):
- 5:50 PM Changeset in webkit [92807] by
-
- 3 edits in trunk/LayoutTests
JSC should always throw when function arg list is too long
https://bugs.webkit.org/show_bug.cgi?id=65869
Patch by Mark Hahnenberg <mhahnenberg@apple.com> on 2011-08-10
Reviewed by Oliver Hunt.
Previous change forgot to update some of the tests in fast/js/function-apply.html so we're doing it now.
- fast/js/function-apply-expected.txt:
- fast/js/script-tests/function-apply.js:
- 5:43 PM Changeset in webkit [92806] by
-
- 2 edits in trunk
WebViewImpl::selectionRange should return false if there is no selection
https://bugs.webkit.org/show_bug.cgi?id=66012
Patch by Varun Jain <varunjain@google.com> on 2011-08-10
Reviewed by Darin Fisher.
- Source/WebKit/chromium/src/WebViewImpl.cpp:
- 5:27 PM Changeset in webkit [92805] by
-
- 2 edits in trunk/Source/WebCore
[Qt] Enable WebGL by default for QtWebKit
https://bugs.webkit.org/show_bug.cgi?id=65998
Reviewed by Benjamin Poulain.
Make ENABLE_WEBGL true by default when Qt is configured with OpenGL.
No new tests. Build change only.
- features.pri:
- 5:17 PM Changeset in webkit [92804] by
-
- 2 edits in trunk/Source/JavaScriptCore
REGRESSION(r92670-r92744): WebKit crashes when opening Gmail
https://bugs.webkit.org/show_bug.cgi?id=66010
Reviewed by Oliver Hunt.
Made sure that Construct calls use() on the this argument.
- dfg/DFGJITCodeGenerator.cpp:
(JSC::DFG::JITCodeGenerator::emitCall):
- 5:00 PM Changeset in webkit [92803] by
-
- 3 edits in trunk/Tools
new-run-webkit-test's WinPort has no fallback logic
https://bugs.webkit.org/show_bug.cgi?id=64486
Unreviewed.
Turns out that we've long had a subtle bug whereby if you didn't
already have DumpRenderTree built, we might end up running unsupported tests
because the DumpRenderTree --print-supported-features check would silently fail.
When I moved the call from using popen to using Executive.run_command, this silent
failure became noisy. (Breaking some bots.)
This re-orders our steps so that we always build DRT before collecting expectations,
as well as adds a warning message when DumpRenderTree can't be found.
- Scripts/webkitpy/layout_tests/port/webkit.py:
- Scripts/webkitpy/layout_tests/run_webkit_tests.py:
- 4:51 PM Changeset in webkit [92802] by
-
- 7 edits in trunk/Source/WebKit/chromium
[Chromium] Decouple implementation of allowFileSystem, openFileSystem and allowDatabase from WebWorkerBase.
https://bugs.webkit.org/show_bug.cgi?id=65997.
This patch moves implementation of allowFileSystem, openFileSystem and allowDatabase from
WebWorkerBase to respectively LocalFileSystemChromium and DatabaseObserver,
parameterizing them with relevant data from WebWorker.
Reviewed by Jian Li.
- src/DatabaseObserver.cpp: Move allowDatabase from WebWorkerBase and update the caller.
(WebKit::AllowDatabaseMainThreadBridge::create):
(WebKit::AllowDatabaseMainThreadBridge::cancel):
(WebKit::AllowDatabaseMainThreadBridge::result):
(WebKit::AllowDatabaseMainThreadBridge::signalCompleted):
(WebKit::AllowDatabaseMainThreadBridge::AllowDatabaseMainThreadBridge):
(WebKit::AllowDatabaseMainThreadBridge::allowDatabaseTask):
(WebKit::AllowDatabaseMainThreadBridge::didComplete):
(WebKit::allowDatabaseForWorker):
(WebCore::DatabaseObserver::canEstablishDatabase):
- src/LocalFileSystemChromium.cpp: Move allowFileSystem and openFileSystem from WebWorkerBase and update the caller.
(WebCore::openFileSystemHelper):
- src/WebWorkerBase.cpp: Move allowFileSystem, openFileSystem and allowDatabase to LocalFileSystemChromium and DatabaseObserver resp.
- src/WebWorkerBase.h:
(WebKit::WebWorkerBase::webView):
- src/WorkerFileSystemCallbacksBridge.cpp: Generalized WorkerFileSystemCallbacksBridge to work on WorkerLoaderProxy, not on WebWorkerBase.
(WebKit::WorkerFileSystemCallbacksBridge::stop):
(WebKit::WorkerFileSystemCallbacksBridge::WorkerFileSystemCallbacksBridge):
(WebKit::WorkerFileSystemCallbacksBridge::dispatchTaskToMainThread):
(WebKit::WorkerFileSystemCallbacksBridge::mayPostTaskToWorker):
- src/WorkerFileSystemCallbacksBridge.h:
(WebKit::WorkerFileSystemCallbacksBridge::create):
- 4:19 PM Changeset in webkit [92801] by
-
- 2 edits in trunk
[wx] Unreviewed build fix. Do not build LocaleToScriptMappingICU.cpp for now as ICU on Leopard is too old to build this.
- 4:16 PM Changeset in webkit [92800] by
-
- 4 edits in trunk/Tools
new-run-webkit-test's WinPort has no fallback logic
https://bugs.webkit.org/show_bug.cgi?id=64486
Unreviewed. Fix test-webkitpy under Python 2.5.
- Scripts/webkitpy/layout_tests/port/win.py:
- 2:51 PM Changeset in webkit [92799] by
-
- 2 edits in trunk/Tools
[Qt][WK2] MiniBrowser is firing twice the QDesktopWebView::mousePressEvent method
https://bugs.webkit.org/show_bug.cgi?id=65875
Patch by Benjamin Poulain <benjamin@webkit.org> on 2011-08-10
Reviewed by Kenneth Rohde Christiansen.
For some reason, QWidget send fake mouse event on some platform, under certains conditions.
This causes mouse events to be sent twice in some cases. We work around the problem
by defining the first touch point to be a primary touch point (case which hit some
weird condition in QWidget, skipping the generation of fake mouse events).
- MiniBrowser/qt/MiniBrowserApplication.cpp:
(MiniBrowserApplication::notify):
- 2:48 PM Changeset in webkit [92798] by
-
- 3 edits2 adds in trunk
Source/WebCore: Check that we do not need layout before trying to dirty
m_originatingLine for our floats.
https://bugs.webkit.org/show_bug.cgi?id=65938
Reviewed by Dave Hyatt.
Test: fast/block/float/float-originating-line-deleted-crash.html
- rendering/RenderBlock.cpp:
(WebCore::RenderBlock::removeFloatingObject):
(WebCore::RenderBlock::clearFloats):
LayoutTests: Tests that we do not crash when trying to access a removed
m_originatingLine for a float.
https://bugs.webkit.org/show_bug.cgi?id=65938
Reviewed by Dave Hyatt.
- fast/block/float/float-originating-line-deleted-crash-expected.txt: Added.
- fast/block/float/float-originating-line-deleted-crash.html: Added.
- 2:42 PM Changeset in webkit [92797] by
-
- 6 edits in trunk
JSC should always throw when function arg list is too long
https://bugs.webkit.org/show_bug.cgi?id=65869
Patch by Mark Hahnenberg <mhahnenberg@apple.com> on 2011-08-10
Reviewed by Oliver Hunt.
Source/JavaScriptCore:
Changed the behavior of the interpreter and JIT to throw an exception
when too many arguments are passed rather than truncating the list. Added
a new method to create a "Too many arguments." exception used by this
new functionality.
- interpreter/Interpreter.cpp:
(JSC::Interpreter::privateExecute):
- jit/JITStubs.cpp:
(JSC::DEFINE_STUB_FUNCTION):
- runtime/ExceptionHelpers.cpp:
(JSC::createTooManyParamsError):
- runtime/ExceptionHelpers.h:
LayoutTests:
Changed the tests to make sure passing too many function arguments throws an exception
rather than truncating the arguments.
- fast/js/function-apply-many-args-expected.txt:
- fast/js/script-tests/function-apply-many-args.js:
- 2:03 PM Changeset in webkit [92796] by
-
- 2 edits in trunk/Source/WebCore
Add missing regions/exclusions-related files to EFL.
- CMakeLists.txt:
- 1:59 PM Changeset in webkit [92795] by
-
- 2 edits in trunk/Source/WebCore
Add missing regions/exclusions-related files to Chromium.
- WebCore.gypi:
- 1:54 PM Changeset in webkit [92794] by
-
- 2 edits in trunk/Source/WebCore
Fix Qt build bustage. Add missing regions/exclusions-related files to their project.
- WebCore.pro:
- 1:39 PM Changeset in webkit [92793] by
-
- 25 edits in trunk/LayoutTests
Update chromium gpu baselines for r92255
- platform/chromium-gpu-mac/compositing/transitions/scale-transition-no-start-expected.png:
- platform/chromium-gpu-mac/media/video-transformed-expected.png:
- platform/chromium-gpu-mac/media/video-zoom-controls-expected.png:
- platform/chromium-gpu-mac/platform/chromium/compositing/backface-visibility-transformed-expected.png:
- platform/chromium-gpu-mac/platform/chromium/compositing/huge-layer-rotated-expected.png:
- platform/chromium-gpu-mac/platform/chromium/compositing/layout-width-change-expected.png:
- platform/chromium-gpu-mac/platform/chromium/compositing/perpendicular-layer-sorting-expected.png:
- platform/chromium-gpu-mac/platform/chromium/compositing/tiny-layer-rotated-expected.png:
- platform/chromium-gpu-win/compositing/color-matching/image-color-matching-expected.png:
- platform/chromium-gpu-win/compositing/direct-image-compositing-expected.png:
- platform/chromium-gpu-win/compositing/geometry/ancestor-overflow-change-expected.png:
- platform/chromium-gpu-win/compositing/geometry/fixed-position-expected.png:
- platform/chromium-gpu-win/compositing/geometry/horizontal-scroll-composited-expected.png:
- platform/chromium-gpu-win/compositing/geometry/layer-due-to-layer-children-deep-expected.png:
- platform/chromium-gpu-win/compositing/geometry/layer-due-to-layer-children-expected.png:
- platform/chromium-gpu-win/compositing/geometry/vertical-scroll-composited-expected.png:
- platform/chromium-gpu-win/compositing/iframes/composited-iframe-alignment-expected.png:
- platform/chromium-gpu-win/compositing/masks/masked-ancestor-expected.png:
- platform/chromium-gpu-win/compositing/overflow/fixed-position-ancestor-clip-expected.png:
- platform/chromium-gpu-win/compositing/reflections/nested-reflection-transition-expected.png:
- platform/chromium-gpu-win/compositing/reflections/transform-inside-reflection-expected.png:
- platform/chromium-gpu-win/compositing/shadows/shadow-drawing-expected.png:
- platform/chromium-gpu-win/compositing/transitions/scale-transition-no-start-expected.png:
- platform/chromium/test_expectations.txt:
- 1:36 PM Changeset in webkit [92792] by
-
- 43 edits in trunk
https://bugs.webkit.org/show_bug.cgi?id=66004
Source/WebCore:
Remove ifdefs for CSS Exclusions (since the feature doesn't exist any longer and has morphed into positioned
floats instead). Go ahead and yank the ifdefs for CSS Regions also and just turn them on.
Reviewed by Adam Roben.
- Configurations/FeatureDefines.xcconfig:
- css/CSSComputedStyleDeclaration.cpp:
(WebCore::CSSComputedStyleDeclaration::getPropertyCSSValue):
- css/CSSParser.cpp:
(WebCore::CSSParser::parseValue):
(WebCore::CSSParser::parseContent):
(WebCore::CSSParser::parseWrapShape):
(WebCore::CSSParser::parseFromFlowContent):
- css/CSSParser.h:
- css/CSSPrimitiveValue.cpp:
(WebCore::CSSPrimitiveValue::init):
(WebCore::CSSPrimitiveValue::cleanup):
(WebCore::CSSPrimitiveValue::getStringValue):
(WebCore::CSSPrimitiveValue::cssText):
- css/CSSPrimitiveValue.h:
(WebCore::CSSPrimitiveValue::getShapeValue):
- css/CSSPrimitiveValueMappings.h:
(WebCore::CSSPrimitiveValue::operator RegionOverflow):
- css/CSSPropertyNames.in:
- css/CSSStyleSelector.cpp:
(WebCore::CSSStyleSelector::applyProperty):
- css/CSSValueKeywords.in:
- css/CSSWrapShapes.cpp:
- css/CSSWrapShapes.h:
- css/SVGCSSValueKeywords.in:
- dom/Node.cpp:
(WebCore::Node::diff):
- dom/NodeRenderingContext.cpp:
(WebCore::NodeRenderingContext::NodeRenderingContext):
(WebCore::NodeRenderingContext::setStyle):
(WebCore::NodeRenderingContext::nextRenderer):
(WebCore::NodeRenderingContext::previousRenderer):
(WebCore::NodeRenderingContext::parentRenderer):
(WebCore::NodeRenderingContext::moveToFlowThreadIfNeeded):
(WebCore::NodeRendererFactory::createRendererIfNeeded):
- dom/NodeRenderingContext.h:
- rendering/RenderFlowThread.cpp:
- rendering/RenderFlowThread.h:
- rendering/RenderLayer.cpp:
(WebCore::RenderLayer::collectLayers):
- rendering/RenderObject.cpp:
(WebCore::RenderObject::createObject):
- rendering/RenderObject.h:
(WebCore::RenderObject::isRenderRegion):
(WebCore::RenderObject::isRenderFlowThread):
- rendering/RenderRegion.cpp:
- rendering/RenderRegion.h:
- rendering/RenderTreeAsText.cpp:
(WebCore::writeLayers):
- rendering/RenderView.cpp:
(WebCore::RenderView::renderFlowThreadWithName):
- rendering/RenderView.h:
- rendering/style/RenderStyle.cpp:
(WebCore::RenderStyle::diff):
- rendering/style/RenderStyle.h:
(WebCore::InheritedFlags::regionOverflow):
(WebCore::InheritedFlags::setRegionOverflow):
(WebCore::InheritedFlags::initialWrapShape):
(WebCore::InheritedFlags::initialRegionOverflow):
- rendering/style/RenderStyleConstants.h:
- rendering/style/StyleRareNonInheritedData.cpp:
(WebCore::StyleRareNonInheritedData::StyleRareNonInheritedData):
(WebCore::StyleRareNonInheritedData::operator==):
- rendering/style/StyleRareNonInheritedData.h:
Source/WebKit/mac:
Removing the CSS Regions and Exclusions ifdefs.
Reviewed by Adam Roben.
- Configurations/FeatureDefines.xcconfig:
Source/WebKit2:
Removing the CSS Regions and CSS Exclusions ifdefs.
Reviewed by Adam Roben.
- Configurations/FeatureDefines.xcconfig:
Tools:
Remove the regions and exclusions bots and also the flags in build-webkit, since I'm just turning
the code on by default.
Reviewed by Adam Roben.
- BuildSlaveSupport/build.webkit.org-config/config.json:
- Scripts/build-webkit:
LayoutTests:
Removing ifdefs for CSS Exclusions and CSS Regions. The code is now on by default, so the new CSS
properties are showing up in the computed style results.
Reviewed by Adam Roben.
- fast/css/getComputedStyle/computed-style-expected.txt:
- platform/mac/fast/css/getComputedStyle/computed-style-without-renderer-expected.txt:
- svg/css/getComputedStyle-basic-expected.txt:
- 1:12 PM Changeset in webkit [92791] by
-
- 3 edits2 adds in trunk
Source/WebCore: Don't create html renderer for SVG <glyphRef>
https://bugs.webkit.org/show_bug.cgi?id=66000
Reviewed by David Hyatt.
Test: svg/text/glyphref-renderer-create-crash.html
- svg/SVGGlyphRefElement.h:
LayoutTests: ASSERTION FAILED: curr->isRenderInline()
https://bugs.webkit.org/show_bug.cgi?id=66000
Reviewed by David Hyatt.
- svg/text/glyphref-renderer-create-crash-expected.txt: Added.
- svg/text/glyphref-renderer-create-crash.html: Added.
- 12:57 PM Changeset in webkit [92790] by
-
- 3 edits3 adds in trunk/LayoutTests
[Chromium] plugins/mouse-events-fixedpos.html should pass on win/mac/linux
https://bugs.webkit.org/show_bug.cgi?id=65841
Patch by Noel Gordon <noel.gordon@gmail.com> on 2011-08-10
Reviewed by Tony Chang.
Add attribute windowPlugin="false" to the <embed> in this test to force the Chromium
Windows and Linux ports to load the test plugin in windowless mode. The plugin will
then receive events and can log them to the console as required by this test.
Also note, the Apple Mac/Win ports report 0 as the line number for all console.log()
messages, whereas, the Chromium ports report the actual line numbers, hence the need
for separate test results for Chromium.
- platform/chromium-linux/plugins/mouse-events-fixedpos-expected.txt:
- platform/chromium-mac/plugins/mouse-events-fixedpos-expected.txt:
- platform/chromium-win/plugins/mouse-events-fixedpos-expected.txt:
- platform/chromium/test_expectations.txt: New expectations.
- plugins/mouse-events-fixedpos.html: Add <embed> attribute windowPlugin="false".
- 12:53 PM Changeset in webkit [92789] by
-
- 2 edits in trunk/Source/WebCore
OpenTypeSanitizer in chromium gyp files is linked twice
https://bugs.webkit.org/show_bug.cgi?id=65927
Patch by Scott Graham <scottmg@google.com> on 2011-08-10
Reviewed by Adam Barth.
No new tests, should just build and link properly on all platforms.
- WebCore.gyp/WebCore.gyp:
- 12:52 PM Changeset in webkit [92788] by
-
- 12 edits in trunk/Source
Make GC checks more aggressive in release builds
https://bugs.webkit.org/show_bug.cgi?id=66001
Reviewed by Gavin Barraclough.
../../../../Volumes/Data/git/WebKit/OpenSource/Source/JavaScriptCore:
- heap/HandleHeap.cpp:
(JSC::HandleHeap::visitStrongHandles):
(JSC::HandleHeap::visitWeakHandles):
(JSC::HandleHeap::finalizeWeakHandles):
(JSC::HandleHeap::writeBarrier):
(JSC::HandleHeap::isLiveNode):
(JSC::HandleHeap::isValidWeakNode):
Increase handle heap validation logic, and make some of
the crashes trigger in release builds as well as debug.
- heap/HandleHeap.h:
(JSC::HandleHeap::allocate):
(JSC::HandleHeap::makeWeak):
Ditto
- runtime/JSGlobalData.cpp:
(WTF::Recompiler::operator()):
- runtime/JSGlobalObject.cpp:
(JSC::JSGlobalObject::visitChildren):
Fix GC bugs found while testing this patch
../../../../Volumes/Data/git/WebKit/OpenSource/Source/WebCore:
Fix GC bugs found while testing increased validation logic
- bindings/js/JSDOMWindowShell.cpp:
(WebCore::JSDOMWindowShell::JSDOMWindowShell):
- bindings/js/JSDOMWindowShell.h:
- bindings/js/ScriptController.cpp:
(WebCore::ScriptController::createWindowShell):
- bridge/objc/ObjCRuntimeObject.h:
(JSC::Bindings::ObjCRuntimeObject::create):
- bridge/objc/ObjCRuntimeObject.mm:
(JSC::Bindings::ObjCRuntimeObject::ObjCRuntimeObject):
- bridge/objc/objc_instance.mm:
- 12:47 PM Changeset in webkit [92787] by
-
- 3 edits in trunk/Source/WebCore
[skia] Move calls to makeGrContextCurrent into clipPathAntiAliased from callers
https://bugs.webkit.org/show_bug.cgi?id=65733
This protects against callers of clipPathAntiAliased forgetting to call makeGrContextCurrent.
Patch by Ben Wells <benwells@chromium.org> on 2011-08-10
Reviewed by James Robinson.
No change in functionality, no new tests.
- platform/graphics/skia/GraphicsContextSkia.cpp:
(WebCore::GraphicsContext::addInnerRoundedRectClip):
(WebCore::GraphicsContext::clip):
- platform/graphics/skia/PlatformContextSkia.cpp:
(WebCore::PlatformContextSkia::clipPathAntiAliased):
- 12:45 PM Changeset in webkit [92786] by
-
- 3 edits in trunk/Source/WebCore
MarkupAccumulator: make resolution of URLs implicit to appendQuotedURLAttributeValue()
https://bugs.webkit.org/show_bug.cgi?id=65990
Patch by Benjamin Poulain <ikipou@gmail.com> on 2011-08-10
Reviewed by Ryosuke Niwa.
Instead of resolving the URLs in the call site of appendQuotedURLAttributeValue(),
the URL is now resolved if necessary in appendQuotedURLAttributeValue().
- editing/MarkupAccumulator.cpp:
(WebCore::MarkupAccumulator::appendQuotedURLAttributeValue):
(WebCore::MarkupAccumulator::appendAttribute):
- editing/MarkupAccumulator.h:
- 12:42 PM Changeset in webkit [92785] by
-
- 4 edits in trunk
JSEvaluteScript does not return the correct object when given JSONP data
https://bugs.webkit.org/show_bug.cgi?id=66003
Reviewed by Gavin Barraclough.
Source/JavaScriptCore:
Make sure we propagate the result of the function call rather than the
argument.
- interpreter/Interpreter.cpp:
(JSC::Interpreter::execute):
Tools:
Add a test case.
- DumpRenderTree/mac/DumpRenderTree.mm:
(testStringByEvaluatingJavaScriptFromString):
- 12:35 PM Changeset in webkit [92784] by
-
- 3 edits in trunk/Tools
WIN: NRWT runs compositing tests on configurations that don't support compositing
https://bugs.webkit.org/show_bug.cgi?id=64472
Reviewed by Adam Barth.
I think the code was just wrong. It was never splitting the string
into list pieces before.
I've now tested the supported_features code and theoretically it
should now work with Windows DRT.
I also made the list-lookup functions always return lists, to make
it possible to clean up the list-transform code in the future
(I opted not to do that cleanup in this patch, but at least now
both types of feature lookup functions match return types.)
- Scripts/webkitpy/layout_tests/port/webkit.py:
- Scripts/webkitpy/layout_tests/port/webkit_unittest.py:
- 12:24 PM Changeset in webkit [92783] by
-
- 9 edits5 adds in trunk
[Chromium] Implement text shaping with font-feature-settings on Linux
https://bugs.webkit.org/show_bug.cgi?id=65903
Source/WebCore:
- Modifies Font::codePath() to return Complex when there is font feature settings CSS property.
- Implements OpenType advanced typography feature using Harfbuzz on Chromium Linux port.
Reviewed by Adam Barth.
Test: css3/font-feature-settings-rendering.html
- platform/graphics/Font.cpp:
(WebCore::Font::codePath): Returns Complex when fontFeatureSettings exists.
- platform/graphics/chromium/ComplexTextControllerLinux.cpp:
(WebCore::setupFontFeatures): Added.
(WebCore::ComplexTextController::setupFontForScriptRun): Calls setupFontFeatures().
LayoutTests:
Adds a rendering test for -webkit-font-feature-settings.
This tests disabled for now expect for chromium-linux port.
FeatureTest.ttf is created from scratch by using FontForge.
Reviewed by Adam Barth.
- css3/font-feature-settings-rendering.html: Added.
- css3/resources/FeatureTest.ttf: Added.
- platform/chromium-linux/css3/font-feature-settings-rendering-expected.png: Added.
- platform/chromium-linux/css3/font-feature-settings-rendering-expected.txt: Added.
- platform/chromium/test_expectations.txt: Skips css3/font-feature-settings-rendering.html.
- platform/gtk/Skipped: Ditto.
- platform/mac/Skipped: Ditto.
- platform/qt/Skipped: Ditto.
- platform/win/Skipped: Ditto.
- 12:13 PM Changeset in webkit [92782] by
-
- 2 edits in trunk/Source/WebCore
[chromium] Notify MemoryCache of decoded data access
https://bugs.webkit.org/show_bug.cgi?id=65859
Reviewed by James Robinson.
This causes us to call CachedImage::didDraw() which calls CachedResource::didAccessDecodedData()
which updates the LRU for decoded image data and potentially prunes the cache. This is already done
by almost all of the other ports.
The decoded bitmaps obviously use quite a bit of memory, so pruning them when necessary is important.
In my tests, this saves several MB on many popular web pages.
No new tests because no observable difference in functionality.
- platform/graphics/skia/ImageSkia.cpp:
(WebCore::BitmapImage::draw):
(WebCore::BitmapImageSingleFrameSkia::draw):
- 11:45 AM Changeset in webkit [92781] by
-
- 17 edits6 adds in trunk
Web Inspector: Remove Network.initialContentSet from protocol, store workers content on backend.
https://bugs.webkit.org/show_bug.cgi?id=65929
Reviewed by Pavel Feldman.
Source/WebCore:
Tests: http/tests/inspector/network/network-shared-worker.html
http/tests/inspector/network/network-worker.html
- inspector/Inspector.json:
- inspector/InspectorInstrumentation.cpp:
(WebCore::InspectorInstrumentation::didReceiveScriptResponseImpl):
- inspector/InspectorInstrumentation.h:
(WebCore::InspectorInstrumentation::didReceiveScriptResponse):
- inspector/InspectorResourceAgent.cpp:
(WebCore::InspectorResourceAgent::didReceiveResponse):
(WebCore::InspectorResourceAgent::setInitialScriptContent):
(WebCore::InspectorResourceAgent::didReceiveScriptResponse):
- inspector/InspectorResourceAgent.h:
- inspector/front-end/NetworkManager.js:
- inspector/front-end/Resource.js:
- workers/DefaultSharedWorkerRepository.cpp:
(WebCore::SharedWorkerScriptLoader::didReceiveResponse):
- workers/Worker.cpp:
(WebCore::Worker::didReceiveResponse):
- workers/Worker.h:
- workers/WorkerScriptLoader.cpp:
(WebCore::WorkerScriptLoader::didReceiveResponse):
- workers/WorkerScriptLoaderClient.h:
(WebCore::WorkerScriptLoaderClient::didReceiveResponse):
Source/WebKit/chromium:
- src/SharedWorkerRepository.cpp:
(WebCore::SharedWorkerScriptLoader::didReceiveResponse):
LayoutTests:
- http/tests/inspector/network/network-shared-worker-expected.txt: Added.
- http/tests/inspector/network/network-shared-worker.html: Added.
- http/tests/inspector/network/network-worker-expected.txt: Added.
- http/tests/inspector/network/network-worker.html: Added.
- http/tests/inspector/network/resources/shared-worker.js: Added.
(onconnect):
- http/tests/inspector/network/resources/worker.js: Added.
- platform/chromium/test_expectations.txt:
- 11:41 AM Changeset in webkit [92780] by
-
- 3 edits in trunk/Source/WebCore
Web Inspector: [Network] Filtering by stylesheet does not update the # reqs, filesizes.
https://bugs.webkit.org/show_bug.cgi?id=65515
Reviewed by Pavel Feldman.
- English.lproj/localizedStrings.js:
- inspector/front-end/NetworkPanel.js:
(WebInspector.NetworkLogView.prototype._updateSummaryBar):
(WebInspector.NetworkLogView.prototype._updateFilter):
- 11:37 AM Changeset in webkit [92779] by
-
- 2 edits in trunk/Tools
Clean up ChromiumDriver a little
https://bugs.webkit.org/show_bug.cgi?id=65995
Unreviewed.
Fix NRWT for chromium. Silly python.
- Scripts/webkitpy/layout_tests/port/chromium.py:
- 11:30 AM Changeset in webkit [92778] by
-
- 2 edits in trunk/Source/WebKit2
Accidentally commented out an assertion in BackingStore::incorporateUpdate
https://bugs.webkit.org/show_bug.cgi?id=65989
Reviewed by Anders Carlsson.
I couldn't get this assertion to fire in quick testing, let's bring it back.
- UIProcess/BackingStore.cpp: (WebKit::BackingStore::incorporateUpdate):
- 10:58 AM Changeset in webkit [92777] by
-
- 2 edits in trunk/Tools
Clean up ChromiumDriver a little
https://bugs.webkit.org/show_bug.cgi?id=65995
Reviewed by Adam Barth.
- We no longer support threading, so remove threading specific code.
- Add FIXMEs about using ServerProcess instead.
- Condense option-mapping if-cascade into a for loop.
- Unindent long if blocks by using early return.
- Unwrap lines which are needlessly wrapped.
There should be no functional changes here, just code cleanup/dead-code removal.
- Scripts/webkitpy/layout_tests/port/chromium.py:
- 10:56 AM Changeset in webkit [92776] by
-
- 3 edits2 adds in trunk
Web Inspector: console messages markers are lost in Resources panel after page reload
https://bugs.webkit.org/show_bug.cgi?id=60701
Reviewed by Pavel Feldman.
Source/WebCore:
Test: http/tests/inspector/resource-tree/resource-tree-errors-reload.html
- inspector/front-end/ResourceTreeModel.js:
(WebInspector.ResourceTreeModel):
(WebInspector.ResourceTreeModel.prototype._consoleMessageAdded):
(WebInspector.ResourceTreeModel.prototype._addPendingConsoleMessage):
(WebInspector.ResourceTreeModel.prototype._addPendingConsoleMessagesToResource):
(WebInspector.ResourceTreeModel.prototype._addConsoleMessageToResource):
(WebInspector.ResourceTreeModel.prototype._consoleCleared):
(WebInspector.ResourceTreeModel.prototype._bindResourceURL):
LayoutTests:
- http/tests/inspector/resource-tree/resource-tree-errors-reload-expected.txt: Added.
- http/tests/inspector/resource-tree/resource-tree-errors-reload.html: Added.
- 10:08 AM Changeset in webkit [92775] by
-
- 6 edits in trunk/Tools
Unmuddle construction options for TestConfiguration.
https://bugs.webkit.org/show_bug.cgi?id=65988
The TestConfiguration instance is now constructed only using actual values.
Port-based construction is split into a "from_port" class method.
Reviewed by Eric Seidel.
- Scripts/webkitpy/layout_tests/models/test_configuration.py: Added TestConfiguration.from_port
- Scripts/webkitpy/layout_tests/models/test_configuration_unittest.py: Adjusted code to accommodate the change.
- Scripts/webkitpy/layout_tests/models/test_expectations_unittest.py: Ditto.
- Scripts/webkitpy/layout_tests/port/base.py: Ditto.
- Scripts/webkitpy/layout_tests/port/webkit_unittest.py: Ditto.
- 9:35 AM Changeset in webkit [92774] by
-
- 2 edits in trunk/LayoutTests
Update expectations for this test. It's just a minor flake.
- platform/chromium/test_expectations.txt:
- 9:27 AM Changeset in webkit [92773] by
-
- 6 edits in trunk/Source
Add device specific settings to support viewport computing
https://bugs.webkit.org/show_bug.cgi?id=65984
Patch by Lars Knudsen <lars.knudsen@nokia.com> on 2011-08-10
Reviewed by Kenneth Rohde Christiansen.
Source/WebCore:
Based on patch by Zalan Bujtas.
Added settings for device size, dpi and layout fallback width
to support viewport computing.
- page/Settings.cpp:
(WebCore::Settings::Settings):
- page/Settings.h:
(WebCore::Settings::setLayoutFallbackWidth):
(WebCore::Settings::layoutFallbackWidth):
(WebCore::Settings::setDeviceWidth):
(WebCore::Settings::deviceWidth):
(WebCore::Settings::setDeviceHeight):
(WebCore::Settings::deviceHeight):
(WebCore::Settings::setDeviceDPI):
(WebCore::Settings::deviceDPI):
Source/WebKit2:
Expose new viewport related WebCore settings to WebKit2.
- Shared/WebPreferencesStore.h:
- WebProcess/WebPage/WebPage.cpp:
(WebKit::WebPage::updatePreferences):
- 8:49 AM Changeset in webkit [92772] by
-
- 3 edits1 add in trunk/Tools
Leaks bot shows python logging prefixes as part of summary display
https://bugs.webkit.org/show_bug.cgi?id=65931
Reviewed by Adam Barth.
The big part of this change is adding a unittesting system for
master.cfg, so mere-mortals can make changes to build.webkit.org
with greatly-reduced risk of breaking the world.
Turns out that in typing up my trival logging fix I made 2 typos,
so hopefully the hour spent getting master.cfg to load was worth it. :)
I also tried to make mastercfg_unittest run as part of test-webkitpy
but right now test-webkitpy requires that unittest files be located in
a module-name-compatible directory. 'build.webkit.org-config' is not
a valid module name.
- BuildSlaveSupport/build.webkit.org-config/master.cfg:
- Fixed _parseNewRunWebKitTestsOutput to strip NRWT's python formatter prefix by using a new _strip_python_logging_prefix function.
- Also fixed this function to file to work with a more modern simplejson version (which returns unicode objects instead of str objects) and filed a related bug with buildbot.net due to their handling of unicode builder names.
- BuildSlaveSupport/build.webkit.org-config/mastercfg_unittest.py: Added.
- BuildBotConfigLoader is a huge pile of code just to get master.cfg to load.
- The actual tiny unittest of the new _parseNewRunWebKitTestsOutput logic.
- Scripts/webkitpy/thirdparty/init.py:
- Add autoinstallation of buildbot (and jinja2) for use by mastercfg_unittest.py
- 8:43 AM Changeset in webkit [92771] by
-
- 2 edits in trunk/Source/WebKit/gtk
[GTK] wrong annotation for the return value of webkit_web_view_get_hit_test_results()
https://bugs.webkit.org/show_bug.cgi?id=65985
Patch by Gustavo Noronha Silva <Gustavo Noronha Silva> on 2011-08-10
Reviewed by Xan Lopez.
- webkit/webkitwebview.cpp:
- 8:34 AM Changeset in webkit [92770] by
-
- 5 edits in trunk/Tools
garden-o-matic should extrapolate failures given a set of known results
https://bugs.webkit.org/show_bug.cgi?id=65882
Reviewed by Adam Barth.
- Scripts/webkitpy/layout_tests/models/test_configuration.py: Spuriously covered my old sins.
- Scripts/webkitpy/layout_tests/port/builders.py: Augmented builders dictionary to contain build coverage specifiers,
adjusted all call sites.
- Scripts/webkitpy/tool/servers/gardeningserver.py: Added BuildCoverageExtrapolator, a thing that
makes wider net for specific failures.
- Scripts/webkitpy/tool/servers/gardeningserver_unittest.py: Added unit tests.
- 8:27 AM Changeset in webkit [92769] by
-
- 17 edits6 adds in trunk
Unify the way we generate HTML for an image in the Clipboard
https://bugs.webkit.org/show_bug.cgi?id=58043
Patch by Benjamin Poulain <benjamin@webkit.org> on 2011-08-10
Reviewed by Ryosuke Niwa.
Source/WebCore:
Unify the way we generate the markup when dragging and image.
Previously, the code was using the special function imageToMarkup() that was
spread from the Chromium port.
That function ignores some edge cases so this patch replace it by the
more generic MarkupAccumulator (which is also used by the other pasteboard
actions).
The MarkupAccumulator was modified because following r61178, the local filepath
are treated differently (the enum was changed to make this behavior
explicit).
Tests: fast/drag-and-drop/drag-and-drop-image-contenteditable.html
fast/drag-and-drop/drag-and-drop-inputimage-contenteditable.html
fast/drag-and-drop/drag-and-drop-objectimage-contenteditable.html
- editing/MarkupAccumulator.cpp:
(WebCore::MarkupAccumulator::MarkupAccumulator):
(WebCore::MarkupAccumulator::resolveURLIfNeeded):
(WebCore::MarkupAccumulator::appendAttribute):
- editing/MarkupAccumulator.h:
- editing/markup.cpp:
- editing/markup.h:
- page/PageSerializer.cpp:
(WebCore::SerializerMarkupAccumulator::SerializerMarkupAccumulator):
- platform/chromium/ClipboardChromium.cpp:
(WebCore::ClipboardChromium::declareAndWriteDragImage):
(WebCore::ClipboardChromium::writeRange):
- platform/chromium/PasteboardChromium.cpp:
(WebCore::Pasteboard::writeSelection):
- platform/gtk/ClipboardGtk.cpp:
(WebCore::ClipboardGtk::declareAndWriteDragImage):
(WebCore::ClipboardGtk::writeRange):
- platform/gtk/DataObjectGtk.cpp:
(WebCore::DataObjectGtk::markup):
- platform/gtk/PasteboardGtk.cpp:
(WebCore::Pasteboard::writeSelection):
(WebCore::Pasteboard::writeImage):
- platform/haiku/PasteboardHaiku.cpp:
(WebCore::Pasteboard::writeSelection):
- platform/qt/ClipboardQt.cpp:
(WebCore::ClipboardQt::declareAndWriteDragImage):
(WebCore::ClipboardQt::writeRange):
- platform/qt/PasteboardQt.cpp:
(WebCore::Pasteboard::writeSelection):
- platform/win/ClipboardWin.cpp:
(WebCore::ClipboardWin::declareAndWriteDragImage):
LayoutTests:
Add 3 tests for dragging an image-like element to an editable area.
- editing/pasteboard/drag-and-drop-image-contenteditable-expected.txt: Added.
- editing/pasteboard/drag-and-drop-image-contenteditable.html: Added.
- editing/pasteboard/drag-and-drop-inputimage-contenteditable-expected.txt: Added.
- editing/pasteboard/drag-and-drop-inputimage-contenteditable.html: Added.
- editing/pasteboard/drag-and-drop-objectimage-contenteditable-expected.txt: Added.
- editing/pasteboard/drag-and-drop-objectimage-contenteditable.html: Added.
- platform/qt/Skipped: Qt does not have support for testing drag and drop.
- 7:37 AM Changeset in webkit [92768] by
-
- 15 edits in trunk
Web Inspector: auto-completion for string literals doesn't work
https://bugs.webkit.org/show_bug.cgi?id=65701
Patch by Pavel Feldman <pfeldman@google.com> on 2011-08-10
Reviewed by Yury Semikhatsky.
Source/WebCore:
- inspector/InjectedScript.cpp:
(WebCore::InjectedScript::evaluateOnCallFrame):
- inspector/InjectedScript.h:
- inspector/InjectedScriptSource.js:
(.):
():
- inspector/Inspector.json:
- inspector/InspectorDebuggerAgent.cpp:
(WebCore::InspectorDebuggerAgent::evaluateOnCallFrame):
- inspector/InspectorDebuggerAgent.h:
- inspector/front-end/ConsoleView.js:
(WebInspector.ConsoleView.prototype._completions.evaluated.getCompletions):
(WebInspector.ConsoleView.prototype._completions.evaluated):
(WebInspector.ConsoleView.prototype._completions.receivedPropertyNamesFromEval):
(WebInspector.ConsoleView.prototype._completions.receivedPropertyNames):
(WebInspector.ConsoleView.prototype._completions):
(WebInspector.ConsoleView.prototype.evalInInspectedWindow):
- inspector/front-end/DebuggerPresentationModel.js:
(WebInspector.PresenationCallFrame.prototype.evaluate):
- inspector/front-end/RemoteObject.js:
(WebInspector.RemoteObject):
- inspector/front-end/ScriptsPanel.js:
(WebInspector.ScriptsPanel.prototype.evaluateInSelectedCallFrame):
(WebInspector.SourceFrameDelegateForScriptsPanel.prototype.evaluateInSelectedCallFrame):
LayoutTests:
- inspector/debugger/debugger-completions-on-call-frame-expected.txt:
- inspector/debugger/debugger-completions-on-call-frame.html:
- 7:06 AM Changeset in webkit [92767] by
-
- 2 edits in trunk/Source/WebKit/chromium
2011-08-10 Andrey Kosyakov <caseq@chromium.org>
Unreviewed. Rolled DEPS 95813:96009 to pick data url timing fix.
- DEPS:
- 7:01 AM Changeset in webkit [92766] by
-
- 4 edits2 deletes in trunk
Web Inspector: CSS editing file containing tabs leads to the whole file change in revision history.
https://bugs.webkit.org/show_bug.cgi?id=65959
Reviewed by Yury Semikhatsky.
Source/WebCore:
- inspector/front-end/SourceFrame.js:
(WebInspector.SourceFrame):
- inspector/front-end/TextEditorModel.js:
(WebInspector.TextEditorModel.prototype._innerSetText):
LayoutTests:
- inspector/editor/text-editor-model-replace-tabs-expected.txt: Removed.
- inspector/editor/text-editor-model-replace-tabs.html: Removed.
- 6:41 AM Changeset in webkit [92765] by
-
- 15 edits1 delete in trunk
Web Inspector: there should be a way to tell what properties are non-enumerable when expanding objects.
https://bugs.webkit.org/show_bug.cgi?id=65518
Reviewed by Yury Semikhatsky.
Source/WebCore:
- inspector/InjectedScript.cpp:
(WebCore::InjectedScript::getProperties):
- inspector/InjectedScript.h:
- inspector/InjectedScriptSource.js:
(.):
- inspector/Inspector.json:
- inspector/InspectorRuntimeAgent.cpp:
(WebCore::InspectorRuntimeAgent::getProperties):
- inspector/InspectorRuntimeAgent.h:
- inspector/front-end/ObjectPropertiesSection.js:
(WebInspector.ObjectPropertyTreeElement.prototype.ondblclick):
(WebInspector.ObjectPropertyTreeElement.prototype.update):
- inspector/front-end/RemoteObject.js:
(WebInspector.RemoteObject.prototype.getOwnProperties):
(WebInspector.RemoteObject.prototype.getAllProperties):
(WebInspector.RemoteObjectProperty):
- inspector/front-end/inspector.css:
(.section .properties .dimmed):
LayoutTests:
- inspector/debugger/debugger-expand-scope-expected.txt:
- inspector/protocol/runtime-agent-expected.txt:
- inspector/runtime/runtime-getProperties-expected.txt:
- inspector/runtime/runtime-getProperties.html:
- platform/chromium/inspector/runtime/runtime-getProperties-expected.txt: Removed.
- 6:36 AM Changeset in webkit [92764] by
-
- 4 edits in trunk
Web Inspector: [REGRESSION] Editor lost after committing a CSS property value for inline style
https://bugs.webkit.org/show_bug.cgi?id=65918
Reviewed by Pavel Feldman.
Source/WebCore:
- inspector/front-end/StylesSidebarPane.js:
(WebInspector.StylesSidebarPane.prototype._innerUpdate.stylesCallback):
(WebInspector.StylesSidebarPane.prototype._innerUpdate.computedStyleCallback):
(WebInspector.StylesSidebarPane.prototype._innerUpdate):
(WebInspector.StylePropertyTreeElement.prototype):
(WebInspector.StylePropertyTreeElement.prototype.event):
(WebInspector.StylePropertyTreeElement.prototype.styleText.updateInterface.majorChange.isRevert.parentPane):
(WebInspector.StylePropertyTreeElement.prototype.styleText.updateInterface.majorChange.isRevert):
LayoutTests:
- inspector/styles/styles-commit-editing.html:
- 4:53 AM Changeset in webkit [92763] by
-
- 3 edits in trunk/Source/WebKit/efl
[EFL] API methods check NULL and return proper value.
https://bugs.webkit.org/show_bug.cgi?id=65844
Patch by Grzegorz Czajkowski <g.czajkowski@samsung.com> on 2011-08-10
Reviewed by Kenneth Rohde Christiansen.
Check NULL before access to the smart and private data of the view object
and return the proper value if error occurred.
- ewk/ewk_view.cpp:
(ewk_view_zoom_range_set):
(ewk_view_zoom_range_min_get):
(ewk_view_zoom_range_max_get):
(ewk_view_user_scalable_set):
(ewk_view_user_scalable_get):
(ewk_view_device_pixel_ratio_get):
- ewk/ewk_view.h:
- 3:24 AM Changeset in webkit [92762] by
-
- 15 edits14 copies6 moves1 add3 deletes in trunk/LayoutTests
Move <input type=datetime> tests to fast/forms/datetime/
https://bugs.webkit.org/show_bug.cgi?id=65970
Reviewed by Hajime Morita.
- fast/forms/ValidityState-rangeOverflow-expected.txt: Removed "datetime" part.
- fast/forms/ValidityState-rangeUnderflow-expected.txt: ditto.
- fast/forms/ValidityState-stepMismatch-expected.txt: ditto.
- fast/forms/ValidityState-typeMismatch-datetime-expected.txt: Removed.
- fast/forms/ValidityState-typeMismatch-datetime.html: Removed.
- fast/forms/date-input-visible-strings-expected.txt: Removed "datetime" part.
- fast/forms/date-input-visible-strings.html: ditto.
- fast/forms/datetime: Added.
- fast/forms/datetime/ValidityState-rangeOverflow-datetime-expected.txt: Copied from LayoutTests/fast/forms/ValidityState-rangeOverflow-expected.txt.
- fast/forms/datetime/ValidityState-rangeOverflow-datetime.html: Copied from LayoutTests/fast/forms/script-tests/ValidityState-rangeOverflow.js.
- fast/forms/datetime/ValidityState-rangeUnderflow-datetime-expected.txt: Copied from LayoutTests/fast/forms/ValidityState-rangeUnderflow-expected.txt.
- fast/forms/datetime/ValidityState-rangeUnderflow-datetime.html: Copied from LayoutTests/fast/forms/script-tests/ValidityState-rangeUnderflow.js.
- fast/forms/datetime/ValidityState-stepMismatch-datetime-expected.txt: Copied from LayoutTests/fast/forms/ValidityState-stepMismatch-expected.txt.
- fast/forms/datetime/ValidityState-stepMismatch-datetime.html: Copied from LayoutTests/fast/forms/script-tests/ValidityState-stepMismatch.js.
- fast/forms/datetime/ValidityState-typeMismatch-datetime-expected.txt: Copied from LayoutTests/fast/forms/ValidityState-typeMismatch-datetime-expected.txt.
- fast/forms/datetime/ValidityState-typeMismatch-datetime.html: Copied from LayoutTests/fast/forms/script-tests/ValidityState-typeMismatch-datetime.js.
- fast/forms/datetime/datetime-input-type-expected.txt: Copied from LayoutTests/fast/forms/week/week-input-type-expected.txt.
- fast/forms/datetime/datetime-input-type.html: Copied from LayoutTests/fast/forms/week/week-input-type.html.
- fast/forms/datetime/datetime-input-visible-string-expected.txt: Copied from LayoutTests/fast/forms/week/week-input-visible-string-expected.txt.
- fast/forms/datetime/datetime-input-visible-string.html: Copied from LayoutTests/fast/forms/week/week-input-visible-string.html.
- fast/forms/datetime/datetime-stepup-stepdown-expected.txt: Copied from LayoutTests/fast/forms/input-stepup-stepdown-expected.txt.
- fast/forms/datetime/datetime-stepup-stepdown-from-renderer-expected.txt: Copied from LayoutTests/fast/forms/input-stepup-stepdown-from-renderer-expected.txt.
- fast/forms/datetime/datetime-stepup-stepdown-from-renderer.html: Copied from LayoutTests/fast/forms/script-tests/input-stepup-stepdown-from-renderer.js.
- fast/forms/datetime/datetime-stepup-stepdown.html: Copied from LayoutTests/fast/forms/script-tests/input-stepup-stepdown.js.
- fast/forms/datetime/input-valueasdate-datetime-expected.txt: Copied from LayoutTests/fast/forms/input-valueasdate-datetime-expected.txt.
- fast/forms/datetime/input-valueasdate-datetime.html: Copied from LayoutTests/fast/forms/script-tests/input-valueasdate-datetime.js.
- fast/forms/datetime/input-valueasnumber-datetime-expected.txt: Copied from LayoutTests/fast/forms/input-valueasnumber-datetime-expected.txt.
- fast/forms/datetime/input-valueasnumber-datetime.html: Copied from LayoutTests/fast/forms/script-tests/input-valueasnumber-datetime.js.
- fast/forms/input-stepup-stepdown-expected.txt: Removed "datetime" part.
- fast/forms/input-stepup-stepdown-from-renderer-expected.txt: ditto.
- fast/forms/input-type-change3-expected.txt: ditto.
- fast/forms/input-valueasdate-datetime-expected.txt: Removed.
- fast/forms/input-valueasdate-datetime.html: Removed.
- fast/forms/input-valueasnumber-datetime-expected.txt: Removed.
- fast/forms/input-valueasnumber-datetime.html: Removed.
- fast/forms/script-tests/ValidityState-rangeOverflow.js: Removed "datetime" part.
- fast/forms/script-tests/ValidityState-rangeUnderflow.js: ditto.
- fast/forms/script-tests/ValidityState-stepMismatch.js: ditto.
- fast/forms/script-tests/ValidityState-typeMismatch-datetime.js: Removed.
- fast/forms/script-tests/input-stepup-stepdown-from-renderer.js: Removed "datetime" part.
- fast/forms/script-tests/input-stepup-stepdown.js: ditto.
- fast/forms/script-tests/input-type-change3.js: ditto.
- fast/forms/script-tests/input-valueasdate-datetime.js: Removed.
- fast/forms/script-tests/input-valueasnumber-datetime.js: Removed.
- 2:59 AM Changeset in webkit [92761] by
-
- 5 edits in trunk/LayoutTests
[Qt] http/tests/misc/object-embedding-svg-delayed-size-negotiation-2.htm fails intermittently
https://bugs.webkit.org/show_bug.cgi?id=65969
Unreviewed gardening.
- platform/qt-4.8/Skipped: Remove http/tests/misc/object-embedding-svg-delayed-size-negotiation-2.htm, because it isn't Qt 4.8 specific bug.
- platform/qt/Skipped: Skip http/tests/misc/object-embedding-svg-delayed-size-negotiation-2.htm.
- platform/qt/http/tests/misc/object-embedding-svg-delayed-size-negotiation-2-expected.txt: Update to the correct results.
- platform/qt/test_expectations.txt: Remove http/tests/misc/object-embedding-svg-delayed-size-negotiation-2.htm.
- 2:23 AM Changeset in webkit [92760] by
-
- 3 edits in trunk/LayoutTests
2011-08-10 Andrey Kosyakov <caseq@chromium.org>
Unreviewed test fix.
- inspector/network-status-non-http-expected.txt:
- inspector/network-status-non-http.html:
- 2:19 AM Changeset in webkit [92759] by
-
- 5 edits4 adds in trunk
WebSocket: Add binaryType attribute
https://bugs.webkit.org/show_bug.cgi?id=65967
Reviewed by Kent Tamura.
Source/WebCore:
Add a new attribute "binaryType" to WebSocket. It controls the data type of MessageEvent
for WebSocket's binary frames.
Tests: http/tests/websocket/tests/hixie76/undefined-attributes.html
http/tests/websocket/tests/hybi/binary-type.html
- websockets/WebSocket.cpp:
(WebCore::WebSocket::WebSocket):
(WebCore::WebSocket::binaryType):
(WebCore::WebSocket::setBinaryType):
- websockets/WebSocket.h:
- websockets/WebSocket.idl:
LayoutTests:
- http/tests/websocket/tests/hixie76/undefined-attributes-expected.txt: Added.
- http/tests/websocket/tests/hixie76/undefined-attributes.html: Added.
- http/tests/websocket/tests/hybi/binary-type-expected.txt: Added.
- http/tests/websocket/tests/hybi/binary-type.html: Added.
- 2:09 AM Changeset in webkit [92758] by
-
- 2 edits in trunk/Source/WebKit/efl
[EFL] Remove unused local variable from _ewk_tiled_backing_store_item_process_idler_cb.
https://bugs.webkit.org/show_bug.cgi?id=65846
As Ewk_Tiled_Backing_Store_Item *it is declared as local variable, but it's never referenced anywhere,
it is better to be removed from _ewk_tiled_backing_store_item_process_idler_cb.
Patch by KwangHyuk Kim <hyuki.kim@samsung.com> on 2011-08-10
Reviewed by Kenneth Rohde Christiansen.
- ewk/ewk_tiled_backing_store.c:
(_ewk_tiled_backing_store_item_process_idler_cb):
- 2:07 AM Changeset in webkit [92757] by
-
- 7 edits1 move10 adds1 delete in trunk
WebSocket: Accept multiple subprotocols
https://bugs.webkit.org/show_bug.cgi?id=65247
Reviewed by Kent Tamura.
Source/WebCore:
WebSocket constructor should accept an array of subprotocols if hybi-10 protocol is chosen.
Tests: http/tests/websocket/tests/hybi/invalid-subprotocol-characters.html
http/tests/websocket/tests/hybi/invalid-subprotocols.html
http/tests/websocket/tests/hybi/multiple-subprotocols.html
http/tests/websocket/tests/hybi/workers/multiple-subprotocols.html
- bindings/js/JSWebSocketCustom.cpp:
(WebCore::JSWebSocketConstructor::constructJSWebSocket):
If the second argument of WebSocket constructor is an array, build a Vector<String> and
pass it to WebSocket::connect().
- bindings/v8/custom/V8WebSocketCustom.cpp:
(WebCore::V8WebSocket::constructorCallback):
Ditto.
- websockets/WebSocket.cpp:
(WebCore::isValidProtocolCharacter): Added.
(WebCore::isValidProtocolString): Added.
(WebCore::isValidProtocolStringHixie76): Renamed from isValidProtocolString.
(WebCore::joinStrings):
(WebCore::WebSocket::connect):
If an array is provided as a subprotocol, and if hixie-76 is used for this connection, it
must be converted to a string in the same manner as JavaScript's Array.toString() in order
to keep compatibility.
- websockets/WebSocket.h:
Removed m_protocol because it was not used at all.
- websockets/WebSocketChannel.cpp:
(WebCore::WebSocketChannel::disconnect):
WebSocketChannel may be released before connect() is called, thus m_handshake may be null.
LayoutTests:
Add tests to check whether WebSocket accepts multiple subprotocols in constructor.
- http/tests/websocket/tests/hybi/accept-first-subprotocol_wsh.py: Added.
- http/tests/websocket/tests/hybi/invalid-subprotocol-characters-expected.txt: Added.
- http/tests/websocket/tests/hybi/invalid-subprotocol-characters.html: Added.
- http/tests/websocket/tests/hybi/invalid-subprotocols-expected.txt: Added.
- http/tests/websocket/tests/hybi/invalid-subprotocols.html: Added.
- http/tests/websocket/tests/hybi/multiple-subprotocols-expected.txt: Renamed from LayoutTests/http/tests/websocket/tests/hybi/sub-protocol-with-space-expected.txt.
- http/tests/websocket/tests/hybi/multiple-subprotocols.html: Added.
- http/tests/websocket/tests/hybi/sub-protocol-with-space-expected.txt: Removed.
- http/tests/websocket/tests/hybi/sub-protocol-with-space.html:
Removed because hybi-10 does not allow subprotocols containing a space.
- http/tests/websocket/tests/hybi/workers/multiple-subprotocols-expected.txt: Added.
- http/tests/websocket/tests/hybi/workers/multiple-subprotocols.html: Added.
- http/tests/websocket/tests/hybi/workers/resources/multiple-subprotocols.js: Added.
(ws.onopen):
(ws.onmessage):
(ws.onclose):
- http/tests/websocket/tests/hybi/workers/resources/protocol-test_wsh.py: Copied from http/tests/websocket/tests/hybi/protocol-test_wsh.py.
- 1:44 AM Changeset in webkit [92756] by
-
- 3 edits in trunk/LayoutTests
[Chromium] plugins/mouse-events.html should pass on the mac.
https://bugs.webkit.org/show_bug.cgi?id=65957
Patch by Noel Gordon <noel.gordon@gmail.com> on 2011-08-10
Reviewed by Kent Tamura.
Cocoa plugins do not generate adjustCursorEvents. Rebaseline this test result for the
chromium Mac port. Note the WebKit Mac/Win ports report 0 for the line number for all
console.log() events. The Chromium port reports the actual line number, and hence the
need for separate test results for Chromium. Following bug 61721, this test now runs,
and passes on all the Chromium ports. The test confirms user-plugin interactivity, so
we don't regress sites like youtube.com. Refer to bug 11517.
- platform/chromium-mac/plugins/mouse-events-expected.txt:
- platform/chromium/test_expectations.txt:
- 1:25 AM Changeset in webkit [92755] by
-
- 5 edits2 adds in trunk
2011-08-10 Andrey Kosyakov <caseq@chromium.org>
Web Inspector: Network panel always displays file:// resources as "(pending)"
https://bugs.webkit.org/show_bug.cgi?id=65784
Reviewed by Pavel Feldman.
Test: inspector/network-status-non-http.html
- English.lproj/localizedStrings.js:
- inspector/front-end/NetworkPanel.js: (WebInspector.NetworkDataGridNode.prototype._refreshStatusCell):
- inspector/front-end/Resource.js: (WebInspector.Resource.prototype.isHttpFamily):
2011-08-10 Andrey Kosyakov <caseq@chromium.org>
Web Inspector: Network panel always displays file:// resources as "(pending)"
https://bugs.webkit.org/show_bug.cgi?id=65784
Reviewed by Pavel Feldman.
- inspector/network-status-non-http-expected.txt: Added.
- inspector/network-status-non-http.html: Added.
- 1:06 AM Changeset in webkit [92754] by
-
- 15 edits14 copies6 moves1 add3 deletes in trunk/LayoutTests
Move <input type=datetime-local> tests to fast/forms/datetimelocal/
https://bugs.webkit.org/show_bug.cgi?id=65958
Reviewed by Hajime Morita.
- fast/forms/ValidityState-rangeOverflow-expected.txt: Removed datetime-local part.
- fast/forms/ValidityState-rangeUnderflow-expected.txt: ditto.
- fast/forms/ValidityState-stepMismatch-expected.txt: ditto.
- fast/forms/ValidityState-typeMismatch-datetimelocal-expected.txt: Removed.
- fast/forms/ValidityState-typeMismatch-datetimelocal.html: Removed.
- fast/forms/date-input-visible-strings-expected.txt: Removed datetime-local and time parts.
- fast/forms/date-input-visible-strings.html: ditto.
- fast/forms/datetimelocal: Added.
- fast/forms/datetimelocal/ValidityState-rangeOverflow-datetimelocal-expected.txt: Copied from LayoutTests/fast/forms/ValidityState-rangeOverflow-expected.txt.
- fast/forms/datetimelocal/ValidityState-rangeOverflow-datetimelocal.html: Copied from LayoutTests/fast/forms/script-tests/ValidityState-rangeOverflow.js.
- fast/forms/datetimelocal/ValidityState-rangeUnderflow-datetimelocal-expected.txt: Copied from LayoutTests/fast/forms/ValidityState-rangeUnderflow-expected.txt.
- fast/forms/datetimelocal/ValidityState-rangeUnderflow-datetimelocal.html: Copied from LayoutTests/fast/forms/script-tests/ValidityState-rangeUnderflow.js.
- fast/forms/datetimelocal/ValidityState-stepMismatch-datetimelocal-expected.txt: Copied from LayoutTests/fast/forms/ValidityState-stepMismatch-expected.txt.
- fast/forms/datetimelocal/ValidityState-stepMismatch-datetimelocal.html: Copied from LayoutTests/fast/forms/script-tests/ValidityState-stepMismatch.js.
- fast/forms/datetimelocal/ValidityState-typeMismatch-datetimelocal-expected.txt: Copied from LayoutTests/fast/forms/ValidityState-typeMismatch-datetimelocal-expected.txt.
- fast/forms/datetimelocal/ValidityState-typeMismatch-datetimelocal.html: Copied from LayoutTests/fast/forms/script-tests/ValidityState-typeMismatch-datetimelocal.js.
- fast/forms/datetimelocal/datetimelocal-input-type-expected.txt: Copied from LayoutTests/fast/forms/week/week-input-type-expected.txt.
- fast/forms/datetimelocal/datetimelocal-input-type.html: Copied from LayoutTests/fast/forms/week/week-input-type.html.
- fast/forms/datetimelocal/datetimelocal-input-visible-string-expected.txt: Copied from LayoutTests/fast/forms/week/week-input-visible-string-expected.txt.
- fast/forms/datetimelocal/datetimelocal-input-visible-string.html: Copied from LayoutTests/fast/forms/week/week-input-visible-string.html.
- fast/forms/datetimelocal/datetimelocal-stepup-stepdown-expected.txt: Copied from LayoutTests/fast/forms/input-stepup-stepdown-expected.txt.
- fast/forms/datetimelocal/datetimelocal-stepup-stepdown-from-renderer-expected.txt: Copied from LayoutTests/fast/forms/input-stepup-stepdown-from-renderer-expected.txt.
- fast/forms/datetimelocal/datetimelocal-stepup-stepdown-from-renderer.html: Copied from LayoutTests/fast/forms/script-tests/input-stepup-stepdown-from-renderer.js.
- fast/forms/datetimelocal/datetimelocal-stepup-stepdown.html: Copied from LayoutTests/fast/forms/script-tests/input-stepup-stepdown.js.
- fast/forms/datetimelocal/input-valueasdate-datetimelocal-expected.txt: Copied from LayoutTests/fast/forms/input-valueasdate-datetimelocal-expected.txt.
- fast/forms/datetimelocal/input-valueasdate-datetimelocal.html: Copied from LayoutTests/fast/forms/script-tests/input-valueasdate-datetimelocal.js.
- fast/forms/datetimelocal/input-valueasnumber-datetimelocal-expected.txt: Copied from LayoutTests/fast/forms/input-valueasnumber-datetimelocal-expected.txt.
- fast/forms/datetimelocal/input-valueasnumber-datetimelocal.html: Copied from LayoutTests/fast/forms/script-tests/input-valueasnumber-datetimelocal.js.
- fast/forms/input-stepup-stepdown-expected.txt: Removed datetime-local part.
- fast/forms/input-stepup-stepdown-from-renderer-expected.txt: ditto.
- fast/forms/input-type-change3-expected.txt: ditto.
- fast/forms/input-valueasdate-datetimelocal-expected.txt: Removed.
- fast/forms/input-valueasdate-datetimelocal.html: Removed.
- fast/forms/input-valueasnumber-datetimelocal-expected.txt: Removed.
- fast/forms/input-valueasnumber-datetimelocal.html: Removed.
- fast/forms/script-tests/ValidityState-rangeOverflow.js: Removed datetime-local part.
- fast/forms/script-tests/ValidityState-rangeUnderflow.js: ditto.
- fast/forms/script-tests/ValidityState-stepMismatch.js: ditto.
- fast/forms/script-tests/ValidityState-typeMismatch-datetimelocal.js: Removed.
- fast/forms/script-tests/input-stepup-stepdown-from-renderer.js: Removed datetime-local part.
- fast/forms/script-tests/input-stepup-stepdown.js: Removed datetime-local part.
- fast/forms/script-tests/input-type-change3.js: Removed datetime-local part.
- fast/forms/script-tests/input-valueasdate-datetimelocal.js: Removed.
- fast/forms/script-tests/input-valueasnumber-datetimelocal.js: Removed.
- 12:39 AM Changeset in webkit [92753] by
-
- 9 edits in trunk/LayoutTests
Updated results after http://trac.webkit.org/changeset/92744.
- platform/chromium-linux/fast/css-generated-content/table-row-before-after-child-add-expected.png:
- platform/chromium-linux/fast/css-generated-content/table-table-before-after-child-add-expected.png:
- platform/chromium-mac-leopard/fast/css-generated-content/table-row-before-after-child-add-expected.png:
- platform/chromium-mac-leopard/fast/css-generated-content/table-table-before-after-child-add-expected.png:
- platform/chromium-mac/fast/css-generated-content/table-row-before-after-child-add-expected.png:
- platform/chromium-mac/fast/css-generated-content/table-table-before-after-child-add-expected.png:
- platform/chromium-win/fast/css-generated-content/table-row-before-after-child-add-expected.png:
- platform/chromium-win/fast/css-generated-content/table-table-before-after-child-add-expected.png:
- 12:36 AM Changeset in webkit [92752] by
-
- 4 edits in trunk/Tools
The URL we were computing for retrieving baselines was missing a slash
character because our mocks had an extra slash character. I've
corrected both the code and the mock.
- Scripts/webkitpy/tool/commands/rebaseline.py:
- Scripts/webkitpy/tool/commands/rebaseline_unittest.py:
- Scripts/webkitpy/tool/mocktool.py:
- 12:35 AM Changeset in webkit [92751] by
-
- 2 edits in trunk/Source/WebKit/chromium
[Chromium] Name an enum used in some template arguments.
https://bugs.webkit.org/show_bug.cgi?id=65953
Patch by Peter Kasting <pkasting@google.com> on 2011-08-10
Reviewed by Darin Fisher.
- public/WebHTTPBody.h:
- 12:06 AM Changeset in webkit [92750] by
-
- 2 edits in trunk/Source/JavaScriptCore
DFG JIT heap prediction causes regressions when combined with
aggressive integer prediction
https://bugs.webkit.org/show_bug.cgi?id=65954
Reviewed by Gavin Barraclough.
Disabled heap prediction, but did not remove the capability.
This improves V8 crypto performance by 20%.
- dfg/DFGGraph.h:
(JSC::DFG::Graph::predict):