Timeline
Jun 20, 2012:
- 11:08 PM Changeset in webkit [120905] by
-
- 4 edits in trunk/Source/WebCore
[cairo] improve putByteArray speed by avoiding max/min checks at Color construction.
https://bugs.webkit.org/show_bug.cgi?id=89138
Patch by Arnaud Renevier <arno@renevier.net> on 2012-06-20
Reviewed by Adam Barth.
Color constructor accepts integer arguments and checks if they are
between 0 and 255. In some cases, we already known those numbers to be
within those boundaries. For example when using unsigned chars. So
this patch introduces Color::createUnChecked which return a Color
object without checking for boundaries.
No new tests: no behaviour change
- platform/graphics/Color.cpp:
(WebCore::colorFromPremultipliedARGB):
- platform/graphics/Color.h:
(WebCore::Color::createUnCheked):
(Color):
- platform/graphics/cairo/ImageBufferCairo.cpp:
(WebCore::ImageBuffer::putByteArray):
- 10:39 PM Changeset in webkit [120904] by
-
- 4 edits in trunk/Source
[Chromium] Damage tracker is not used without partial swap, causing valid render passes to be removed
https://bugs.webkit.org/show_bug.cgi?id=89589
Patch by Zeev Lieber <zlieber@chromium.org> on 2012-06-20
Reviewed by Adrienne Walker.
Source/WebCore:
When not using partial swap, the CCDamageTracker was not used, and
its m_currentDamageRect was always empty. As a result,
CCLayerTreeHostImpl was thinking no content was changed and was
removing more textures than needed. Fixed this by turning on
CCDamageTracker usage even if not using partial swap, but
overwriting the rootScissorRect with viewport rect if required.
Added unit tests to exercise this scenario.
- platform/graphics/chromium/cc/CCLayerTreeHostImpl.cpp:
(WebCore::CCLayerTreeHostImpl::calculateRenderSurfaceLayerList):
Source/WebKit/chromium:
Added unit tests to check surface texture caching when partial
swap is not used.
- tests/CCLayerTreeHostImplTest.cpp:
- 10:11 PM Changeset in webkit [120903] by
-
- 7 edits in trunk/Source/WebCore
[Forms] Make step action of SpinButtonElement replaceable
https://bugs.webkit.org/show_bug.cgi?id=89439
Reviewed by Kent Tamura.
This patch introduces SpinButtonElement::Callback to specify step
action by control clients for using SpinButtonElement by
TextFieldInputType and future version of TimeInputField.
In addition to this improving flexibility of SpinButtonElement, we
free HTMLInputElement from SpinButtonElement dependency on
stepUpFromRenderer.
No new tests. This patch doesn't change behavior.
- html/HTMLInputElement.cpp: Removed stepUpFromRenderer().
- html/HTMLInputElement.h: Removed stepUpFromRenderer().
- html/TextFieldInputType.cpp:
(WebCore::TextFieldInputType::~TextFieldInputType): Added to call SpinButtonElement::removeStepActionHandler.
(WebCore::TextFieldInputType::handleKeydownEventForSpinButton): Changed to call spinButtonStep{Down,Up}
(WebCore::TextFieldInputType::handleWheelEventForSpinButton): Changed to call spinButtonStep{Down,Up}
(WebCore::TextFieldInputType::createShadowSubtree): Added SpinButtonElement::StepActionHandler parameter.
(WebCore::TextFieldInputType::destroyShadowSubtree): Added to call SpinButtonElement::removeStepActionHandler.
(WebCore::TextFieldInputType::spinButtonStepDown): Added for implementation of SpinButtonElement::StepActionHandler.
(WebCore::TextFieldInputType::spinButtonStepUp): Added for implementation of SpinButtonElement::StepActionHandler.
- html/TextFieldInputType.h:
(TextFieldInputType):
- html/shadow/TextControlInnerElements.cpp:
(WebCore::SpinButtonElement::SpinButtonElement): Added StepActionHandler parameter.
(WebCore::SpinButtonElement::defaultEventHandler): Changed to call doStepAction.
(WebCore::SpinButtonElement::doStepAction): Added to call StepActionHandler if possible.
(WebCore::SpinButtonElement::step): Changed to call doStepAction.
- html/shadow/TextControlInnerElements.h:
(StepActionHandler): Added.
- 9:46 PM Changeset in webkit [120902] by
-
- 4 edits in trunk/Source/WebCore
[chromium] Modify CCDamageTracker hash to allow for layer id 0
https://bugs.webkit.org/show_bug.cgi?id=89631
Reviewed by James Robinson.
HashMap has the bizarre property that 0 is the empty value for integer
keys. Modify the damage tracking HashMap to use negative values for
both the empty and the deleted key traits. Additionally, make sure we
never generate negative layer IDs in practice.
Test: passes webkit_unit_tests with the patch in bug 89589 applied.
- platform/graphics/chromium/LayerChromium.cpp:
(WebCore::LayerChromium::LayerChromium):
- platform/graphics/chromium/cc/CCDamageTracker.h:
(RectMapKeyTraits):
(WebCore::CCDamageTracker::RectMapKeyTraits::emptyValue):
(WebCore::CCDamageTracker::RectMapKeyTraits::constructDeletedValue):
(WebCore::CCDamageTracker::RectMapKeyTraits::isDeletedValue):
(CCDamageTracker):
- platform/graphics/chromium/cc/CCLayerImpl.cpp:
(WebCore::CCLayerImpl::CCLayerImpl):
- 9:27 PM Changeset in webkit [120901] by
-
- 2 edits in trunk/Tools
[WinCairo] Unreviewed build fix.
The wrapper script that launched new- and old-run-webkit-tests
was discarding the --wincairo argument needed to get the proper
test infrastructure to build under WinCairo.
- Scripts/run-webkit-tests: Emulate the Qt, Wx, Chromium, etc.,
behavior to chain the --wincairo flag through to the new build
and test scripts.
- 9:18 PM Changeset in webkit [120900] by
-
- 6 edits in trunk/Source/WebCore
Use Dictionary in MutationObserver.observe to kill custom code
https://bugs.webkit.org/show_bug.cgi?id=89629
Reviewed by Ryosuke Niwa.
Move code for dictionary parsing in MutationObserver.observe
that used to be duplicated (with different implementations)
in JSC and V8 bindings into WebKitMutationObserver.cpp, using
the new Dictionary interface.
No new tests, no change in behavior.
- bindings/js/JSWebKitMutationObserverCustom.cpp:
- bindings/v8/custom/V8WebKitMutationObserverCustom.cpp:
- dom/WebKitMutationObserver.cpp:
(WebCore::WebKitMutationObserver::observe):
- dom/WebKitMutationObserver.h:
(WebCore):
- dom/WebKitMutationObserver.idl:
- 7:03 PM Changeset in webkit [120899] by
-
- 4 edits in trunk/Source
Unreviewed, rolling out r120889.
http://trac.webkit.org/changeset/120889
https://bugs.webkit.org/show_bug.cgi?id=89630
[Chromium] webkit_unit_tests didDrawNotCalledOnHiddenLayer
start failing (Requested by ukai on #webkit).
Patch by Sheriff Bot <webkit.review.bot@gmail.com> on 2012-06-20
Source/WebCore:
- platform/graphics/chromium/cc/CCLayerTreeHostImpl.cpp:
(WebCore::CCLayerTreeHostImpl::calculateRenderSurfaceLayerList):
Source/WebKit/chromium:
- tests/CCLayerTreeHostImplTest.cpp:
- 7:00 PM Changeset in webkit [120898] by
-
- 4 edits in trunk/Source/JavaScriptCore
Reduced (but did not eliminate) use of "berzerker GC"
https://bugs.webkit.org/show_bug.cgi?id=89237
Reviewed by Gavin Barraclough.
(PART 1)
This patch turned out to be crashy, so I'm landing the non-crashy bits
first.
This part is pre-requisite refactoring. I didn't actually turn off
"berzerker GC" or turn on incremental shrinking.
- heap/MarkedAllocator.cpp:
(JSC::MarkedAllocator::removeBlock): Make sure to clear the free list when
we throw away the block we're currently allocating out of. Otherwise, we'll
allocate out of a stale free list.
- heap/MarkedSpace.cpp:
(JSC::Free::Free):
(JSC::Free::operator()):
(JSC::Free::returnValue): Refactored this functor to use a shared helper
function, so we can share our implementation with the incremental sweeper.
Also changed to freeing individual blocks immediately instead of linking
them into a list for later freeing. This makes the programming interface
simpler, and it's slightly more efficient to boot.
(JSC::MarkedSpace::~MarkedSpace): Updated for rename.
(JSC::MarkedSpace::freeBlock):
(JSC::MarkedSpace::freeOrShrinkBlock): New helper functions to share behavior
with the incremental sweeper.
(JSC::MarkedSpace::shrink): Updated for new functor behavior.
- heap/MarkedSpace.h: Statically typed languages are awesome.
- 6:38 PM Changeset in webkit [120897] by
-
- 12 edits2 adds in trunk/Source/JavaScriptCore
DFG should optimize ResolveGlobal
https://bugs.webkit.org/show_bug.cgi?id=89617
Reviewed by Oliver Hunt.
This adds inlining of ResolveGlobal accesses that are known monomorphic. It also
adds the specific function optimization to ResolveGlobal, when it is inlined. And,
it makes internal functions act like specific functions, since that will be the
most common use-case of this optimization.
This is only a slighy speed-up (sub 1%), since we don't yet do the obvious thing
with this optimization, which is to completely inline common "globally resolved"
function and constructor calls, like "new Array()".
- CMakeLists.txt:
- GNUmakefile.list.am:
- JavaScriptCore.vcproj/JavaScriptCore/JavaScriptCore.def:
- JavaScriptCore.xcodeproj/project.pbxproj:
- Target.pri:
- bytecode/CodeBlock.cpp:
(JSC::CodeBlock::globalResolveInfoForBytecodeOffset):
- bytecode/CodeBlock.h:
(CodeBlock):
(JSC::CodeBlock::numberOfGlobalResolveInfos):
- bytecode/GlobalResolveInfo.h:
(JSC::getGlobalResolveInfoBytecodeOffset):
(JSC):
- bytecode/ResolveGlobalStatus.cpp: Added.
(JSC):
(JSC::computeForStructure):
(JSC::computeForLLInt):
(JSC::ResolveGlobalStatus::computeFor):
- bytecode/ResolveGlobalStatus.h: Added.
(JSC):
(ResolveGlobalStatus):
(JSC::ResolveGlobalStatus::ResolveGlobalStatus):
(JSC::ResolveGlobalStatus::state):
(JSC::ResolveGlobalStatus::isSet):
(JSC::ResolveGlobalStatus::operator!):
(JSC::ResolveGlobalStatus::isSimple):
(JSC::ResolveGlobalStatus::takesSlowPath):
(JSC::ResolveGlobalStatus::structure):
(JSC::ResolveGlobalStatus::offset):
(JSC::ResolveGlobalStatus::specificValue):
- dfg/DFGByteCodeParser.cpp:
(ByteCodeParser):
(JSC::DFG::ByteCodeParser::handleGetByOffset):
(DFG):
(JSC::DFG::ByteCodeParser::handleGetById):
(JSC::DFG::ByteCodeParser::parseBlock):
- runtime/JSObject.cpp:
(JSC::getCallableObjectSlow):
(JSC):
(JSC::JSObject::put):
(JSC::JSObject::putDirectVirtual):
(JSC::JSObject::putDirectAccessor):
- runtime/JSObject.h:
(JSC):
(JSC::getCallableObject):
(JSC::JSObject::putOwnDataProperty):
(JSC::JSObject::putDirect):
(JSC::JSObject::putDirectWithoutTransition):
- 6:34 PM Changeset in webkit [120896] by
-
- 8 edits2 adds in trunk
Source/WebCore: [Shadow][Editing] Deleting character in distributed element caused a crash.
https://bugs.webkit.org/show_bug.cgi?id=88484
Reviewed by Ryosuke Niwa.
If we mutate nodes which are children of a shadow host, it causes
ElementShadow::invalidateDistribution(). As a result, shadow host
is detached (and lazyAttached()) and its renderer is gone. That
causes assertion error since isEditablePosition() assumes
anchorNode's style was correctly calculated.
This patch makes isEditablePosition() call
document->updateLayoutIgnorePendingStylesheets() so that it does
not return a bogus result.
There is an exceptional caller of isEditablePosition,
RenderBlock::paintCaret(), from where we can not call
updateLayout() in isEditablePosition because it hits assertion,
ASSERT(!isPainting). So I've added the third parameter to
isEditablePosition to control updating the style.
Test: edition/shadow/delete-characters-in-distributed-node.html
- editing/FrameSelection.h:
(WebCore::FrameSelection::rendererIsEditable):
- editing/VisibleSelection.cpp:
(WebCore::VisibleSelection::rendererIsEditable):
(WebCore):
- editing/VisibleSelection.h:
(VisibleSelection):
- editing/htmlediting.cpp:
(WebCore::isEditablePosition):
- editing/htmlediting.h:
(WebCore):
- rendering/RenderBlock.cpp:
(WebCore::RenderBlock::paintCaret):
LayoutTests: [Shadow][Editing] Deleting character in distributed element caused a crash
https://bugs.webkit.org/show_bug.cgi?id=88484
Reviewed by Ryosuke Niwa.
- editing/shadow/delete-characters-in-distributed-node-crash-expected.txt: Added.
- editing/shadow/delete-characters-in-distributed-node-crash.html: Added.
- 6:31 PM Changeset in webkit [120895] by
-
- 8 edits in trunk
Unmodified form control value are overwritten by another form
control value with the same name after navigating and going back
https://bugs.webkit.org/show_bug.cgi?id=89409
Reviewed by Hajime Morita.
Source/WebCore:
Detail of the bug:
If a page had multiple form controls of which names and types were
identical like the following:
<input type=text name=name1 id=input1>
<input type=text name=name1 id=input2>
and a user updated the value of the second control, then went to
another page and went back to the page again, we restored the updated
value to the first element, and didn't update the second element.
We didn't save unmodified control state, and the form state data
had no ways to represent "this control should be skipped".
How to resovle the bug:
We need to represent "this control should be skipped" in the
seriazlied form state vector.
- A serialized control state had three items:
name, type, value.
Now we change it to:
name, type, flag, optional value
- It is definitely incompatible with serizlized state produced by
older WebCore. So, we need to add the signature string to
represent the version of serialized state format.
- Because the state for a form control is variable-length and we
can't deserialize it in reverse-order, we change the on-memory
representation from Vector<> to Deque<>.
Test: fast/forms/state-restore-to-non-edited-controls.html
- html/FormController.cpp:
(WebCore::FormControlState::serializeTo):
Added. Serialize a state for a form control to a string vector.
(WebCore::FormControlState::deserialize):
Added. Produce a FormControlState object from the specified string vector.
It can produce a FromControlState of the failure type.
(WebCore::formStateSignature): The signature string of the serialized state.
(WebCore::FormController::formElementsState):
- Capacity: The size of seirlized data for one form control is typically 4. +1 for the signature.
- We need to store a FormControlState with no values.
(WebCore::FormController::setStateForNewFormElements):
- We can't iterate over the stateVector in reverse order any more because serialized control state is variable-length.
- We put FormControlState objects to HashMap instead of String objects.
(WebCore::FormController::takeStateForFormElement):
Updated for Deque<>.
- html/FormController.h:
(FormControlState): Declare deserialize() and serializeTo().
(WebCore::FormControlState::isFailure): Added.
(WebCore::FormControlState::FormControlState):
Added to create a FormControlState with failure type.
(FormController):
Change the value type of m_stateForNewFormElements from Vector<String>
to Deque<FormControlState>.
LayoutTests:
- fast/forms/resources/state-restore-broken-state-2.html:
Take care of the signature string at the beginning of the array.
- fast/forms/state-restore-broken-state-expected.txt:
Updated for the serialized format change.
- fast/forms/state-restore-to-non-edited-controls-expected.txt:
- fast/forms/state-restore-to-non-edited-controls.html:
Add a test for a case of duplicated names.
- 6:29 PM Changeset in webkit [120894] by
-
- 4 edits2 adds1 delete in trunk/LayoutTests
Unskip fast/hidpi/device-scale-factor-paint.html
https://bugs.webkit.org/show_bug.cgi?id=89618
Reviewed by Beth Dakin.
Now that this test doesn't cause WebKit2 to crash, we can unskip it.
- fast/hidpi/device-scale-factor-paint-expected.html: Removed.
- Previously, this test was a reftest, but the reference also tried to set the device scale factor, which seems problematic. This patch converts the test to a normal image test. There aren't any scrollbars or text, so many platforms should be able to use the same expected PNG file.
- fast/hidpi/device-scale-factor-paint.html:
- Wrap the call to notifyDone in a thunk because it doesn't work otherwise.
- platform/mac/Skipped:
- platform/mac/fast/hidpi/device-scale-factor-paint-expected.png: Added.
- platform/mac/fast/hidpi/device-scale-factor-paint-expected.txt: Added.
- platform/wk2/Skipped:
- 6:13 PM Changeset in webkit [120893] by
-
- 11 edits2 moves in trunk/Source/WebCore
[CSS Shaders] Rename CustomFilterShader class name to CustomFilterCompiledProgram
https://bugs.webkit.org/show_bug.cgi?id=89578
Reviewed by Dean Jackson.
I've renamed CustomFilterShader to CustomFilterCompiledProgram to make it obvious that it is the result of
calling CustomFilterProgram::compileProgramWithContext.
No new tests, just renamed an existing class.
- CMakeLists.txt:
- GNUmakefile.list.am:
- Target.pri:
- WebCore.gypi:
- WebCore.vcproj/WebCore.vcproj:
- WebCore.xcodeproj/project.pbxproj:
- platform/graphics/filters/CustomFilterCompiledProgram.cpp: Renamed from Source/WebCore/platform/graphics/filters/CustomFilterShader.cpp.
(WebCore):
(WebCore::CustomFilterCompiledProgram::defaultVertexShaderString):
(WebCore::CustomFilterCompiledProgram::defaultFragmentShaderString):
(WebCore::CustomFilterCompiledProgram::CustomFilterCompiledProgram):
(WebCore::CustomFilterCompiledProgram::compileShader):
(WebCore::CustomFilterCompiledProgram::linkProgram):
(WebCore::CustomFilterCompiledProgram::initializeParameterLocations):
(WebCore::CustomFilterCompiledProgram::uniformLocationByName):
(WebCore::CustomFilterCompiledProgram::~CustomFilterCompiledProgram):
- platform/graphics/filters/CustomFilterCompiledProgram.h: Renamed from Source/WebCore/platform/graphics/filters/CustomFilterShader.h.
(WebCore):
- platform/graphics/filters/CustomFilterProgram.cpp:
(WebCore::CustomFilterProgram::compileProgramWithContext): Renamed method from createShaderWithContext to compileProgramWithContext.
- platform/graphics/filters/CustomFilterProgram.h:
(WebCore):
- platform/graphics/filters/FECustomFilter.cpp: Renamed m_shader to m_compiledProgram.
(WebCore::FECustomFilter::platformApplySoftware):
(WebCore::FECustomFilter::initializeContext):
(WebCore::FECustomFilter::bindProgramParameters):
(WebCore::FECustomFilter::bindProgramAndBuffers):
- platform/graphics/filters/FECustomFilter.h:
(WebCore):
(FECustomFilter):
- 6:03 PM Changeset in webkit [120892] by
-
- 1 edit2 adds in trunk/PerformanceTests
Add a performance test for paths in SVG
https://bugs.webkit.org/show_bug.cgi?id=89547
Reviewed by Ryosuke Niwa.
This change adds the first performance test for SVG paths.
In the test we modify complex cubic paths in several ways, testing:
transformations, clipping, d attribute changes, stroke properties,
text on a path, and opacity.
Sample test results on my Linux desktop:
RESULT SVG: SvgCubics= 68.85 ms
median= 69.5 ms, stdev= 4.70398767005 ms, min= 56.0 ms, max= 75.0 ms
RESULT SVG: SvgCubics= 68.4 ms
median= 69.0 ms, stdev= 3.51283361405 ms, min= 59.0 ms, max= 74.0 ms
RESULT SVG: SvgCubics= 66.95 ms
median= 67.0 ms, stdev= 3.4420197559 ms, min= 59.0 ms, max= 74.0 ms
RESULT SVG: SvgCubics= 70.2 ms
median= 71.5 ms, stdev= 3.23419232576 ms, min= 63.0 ms, max= 74.0 ms
- SVG: Added.
- SVG/SvgCubics.html: Added.
- 5:53 PM Changeset in webkit [120891] by
-
- 4 edits in trunk/Source/WebCore
Web Inspector: reduce timeline refresh rate
https://bugs.webkit.org/show_bug.cgi?id=89548
Reviewed by Pavel Feldman.
Minimize resource contention with the inspected page during refresh:
- reduce refresh for both overview and main panes to 300ms
- automatically limit window size to approximately one pageful of events in the lower pane
- do not refresh lower pane for events that are outside of the window
- inspector/front-end/TimelineOverviewPane.js:
(WebInspector.TimelineOverviewPane.prototype._update):
(WebInspector.TimelineOverviewPane.prototype._onWindowChanged):
(WebInspector.TimelineOverviewPane.prototype.setWindowTimes): Set overview window by times.
(WebInspector.TimelineOverviewPane.prototype._updateWindow):
(WebInspector.TimelineOverviewPane.prototype._scheduleRefresh): Refresh once in 300ms (insted of once in 100ms).
(WebInspector.TimelineOverviewWindow.prototype._setWindow):
- inspector/front-end/TimelinePanel.js:
(WebInspector.TimelinePanel.prototype._onTimelineEventRecorded): Do not refresh if the new record is outside of window.
(WebInspector.TimelinePanel.prototype._innerAddRecordToTimeline):
(WebInspector.TimelinePanel.prototype._resetPanel):
(WebInspector.TimelinePanel.prototype._scheduleRefresh): Refresh rate: 100ms -> 300ms.
(WebInspector.TimelinePanel.prototype._refreshRecords): Automatically set overview window.
- inspector/front-end/TimelinePresentationModel.js: Expose filterRecords
(WebInspector.TimelinePresentationModel.prototype.filteredRecords):
(WebInspector.TimelinePresentationModel.prototype.filterRecords):
(WebInspector.TimelinePresentationModel.prototype._innerFilterRecords):
- 5:43 PM Changeset in webkit [120890] by
-
- 24 edits3 adds in trunk/Source/WebKit2
[WK2] Color chooser API missing
https://bugs.webkit.org/show_bug.cgi?id=87495
Patch by Thiago Marcos P. Santos <thiago.santos@intel.com> on 2012-06-20
Reviewed by Andreas Kling.
Added ColorChooser API to WebKit2. This API allows the
embedder to define a custom color picker for <input type="color">.
Only one ColorChooser can be active for a page at a time. Although
the implementation doesn't not assume a modal dialog, no other
color chooser will be created until the active one is closed.
Also added stubs for all platforms, so it wont break the build when
enabling color chooser, even though they don't have the backend
implemented yet.
- CMakeLists.txt:
- GNUmakefile.list.am:
- Target.pri:
- UIProcess/API/efl/PageClientImpl.cpp:
(WebKit):
(WebKit::PageClientImpl::createColorChooserProxy):
- UIProcess/API/efl/PageClientImpl.h:
(PageClientImpl):
- UIProcess/API/gtk/PageClientImpl.cpp:
(WebKit):
(WebKit::PageClientImpl::createColorChooserProxy):
- UIProcess/API/gtk/PageClientImpl.h:
(PageClientImpl):
- UIProcess/API/mac/PageClientImpl.h:
(PageClientImpl):
- UIProcess/API/mac/PageClientImpl.mm:
(WebKit):
(WebKit::PageClientImpl::createColorChooserProxy):
- UIProcess/PageClient.h:
(WebKit):
(PageClient):
- UIProcess/WebColorChooserProxy.h: Added.
(WebCore):
(WebKit):
(WebColorChooserProxy):
(Client):
(WebKit::WebColorChooserProxy::Client::~Client):
(WebKit::WebColorChooserProxy::~WebColorChooserProxy):
(WebKit::WebColorChooserProxy::invalidate):
(WebKit::WebColorChooserProxy::WebColorChooserProxy):
- UIProcess/WebPageProxy.cpp:
(WebKit::WebPageProxy::close):
(WebKit):
(WebKit::WebPageProxy::showColorChooser):
(WebKit::WebPageProxy::setColorChooserColor):
(WebKit::WebPageProxy::endColorChooser):
(WebKit::WebPageProxy::didChooseColor):
(WebKit::WebPageProxy::didEndColorChooser):
(WebKit::WebPageProxy::processDidCrash):
- UIProcess/WebPageProxy.h:
(WebPageProxy):
- UIProcess/WebPageProxy.messages.in:
- UIProcess/qt/QtPageClient.cpp:
(WebKit):
(WebKit::QtPageClient::createColorChooserProxy):
- UIProcess/qt/QtPageClient.h:
(QtPageClient):
- UIProcess/win/WebView.cpp:
(WebKit):
(WebKit::WebView::createColorChooserProxy):
- UIProcess/win/WebView.h:
(WebView):
- WebKit2.xcodeproj/project.pbxproj:
- WebProcess/WebCoreSupport/WebChromeClient.cpp:
(WebKit::WebChromeClient::createColorChooser):
- WebProcess/WebCoreSupport/WebColorChooser.cpp: Added.
(WebKit):
(WebKit::WebColorChooser::WebColorChooser):
(WebKit::WebColorChooser::~WebColorChooser):
(WebKit::WebColorChooser::didChooseColor):
(WebKit::WebColorChooser::didEndChooser):
(WebKit::WebColorChooser::disconnectFromPage):
(WebKit::WebColorChooser::setSelectedColor):
(WebKit::WebColorChooser::endChooser):
- WebProcess/WebCoreSupport/WebColorChooser.h: Added.
(WebCore):
(WebKit):
(WebColorChooser):
- WebProcess/WebPage/WebPage.cpp:
(WebKit::WebPage::WebPage):
(WebKit::WebPage::close):
(WebKit):
(WebKit::WebPage::setActiveColorChooser):
(WebKit::WebPage::didEndColorChooser):
(WebKit::WebPage::didChooseColor):
- WebProcess/WebPage/WebPage.h:
(WebKit):
(WebPage):
(WebKit::WebPage::activeColorChooser):
- WebProcess/WebPage/WebPage.messages.in:
- win/WebKit2.vcproj:
- 5:43 PM Changeset in webkit [120889] by
-
- 4 edits in trunk/Source
[Chromium] Damage tracker is not used without partial swap, causing valid render passes to be removed
https://bugs.webkit.org/show_bug.cgi?id=89589
Patch by Zeev Lieber <zlieber@chromium.org> on 2012-06-20
Reviewed by Adrienne Walker.
Source/WebCore:
When not using partial swap, the CCDamageTracker was not used, and
its m_currentDamageRect was always empty. As a result,
CCLayerTreeHostImpl was thinking no content was changed and was
removing more textures than needed. Fixed this by turning on
CCDamageTracker usage even if not using partial swap, but
overwriting the rootScissorRect with viewport rect if required.
Added unit tests to exercise this scenario.
- platform/graphics/chromium/cc/CCLayerTreeHostImpl.cpp:
(WebCore::CCLayerTreeHostImpl::calculateRenderSurfaceLayerList):
Source/WebKit/chromium:
Added unit tests to check surface texture caching when partial
swap is not used.
- tests/CCLayerTreeHostImplTest.cpp:
- 5:31 PM Changeset in webkit [120888] by
-
- 17 edits in trunk
IndexedDB: Remove redundant IDBObjectStore.delete() overloads
https://bugs.webkit.org/show_bug.cgi?id=89587
Reviewed by Darin Fisher.
Source/WebCore:
The IDBObjectStore.delete(IDBKey) overload can delegate to the delete(IDBKeyRange)
overload. Also cleaned up redundant checks in related overloads.
Test: storage/indexeddb/deleted-objects.html
- Modules/indexeddb/IDBCursorBackendImpl.cpp:
(WebCore::IDBCursorBackendImpl::deleteFunction): Call keyRange variant instead.
- Modules/indexeddb/IDBIndex.cpp:
(WebCore::IDBIndex::get): Remove redundant deleted/invalid IDBKey checks.
(WebCore::IDBIndex::getKey): Remove redundant deleted/invalid IDBKey checks.
- Modules/indexeddb/IDBKeyRange.cpp: Null IDBKeys are not acceptable; currently
not passed in by any callers.
(WebCore::IDBKeyRange::only):
(WebCore::IDBKeyRange::lowerBound):
(WebCore::IDBKeyRange::upperBound):
(WebCore::IDBKeyRange::bound):
- Modules/indexeddb/IDBObjectStore.cpp:
(WebCore::IDBObjectStore::get): Was missing deleted check in IDBKeyRange overload.
Removed redundant checks in IDBKey overload.
(WebCore::IDBObjectStore::deleteFunction): Delegate to IDBKeyRange overload.
- Modules/indexeddb/IDBObjectStoreBackendImpl.cpp: Remove IDBKey overload.
- Modules/indexeddb/IDBObjectStoreBackendImpl.h: Remove IDBKey overload.
(IDBObjectStoreBackendImpl):
- Modules/indexeddb/IDBObjectStoreBackendInterface.h: Remove IDBKey overload.
Source/WebKit/chromium:
- public/WebIDBObjectStore.h: Add note to remove overload when Chromium is updated.
(WebIDBObjectStore):
- src/IDBObjectStoreBackendProxy.cpp: Delete IDBKey overload.
- src/IDBObjectStoreBackendProxy.h: Delete IDBKey overload.
(IDBObjectStoreBackendProxy):
- src/WebIDBObjectStoreImpl.cpp: Delete IDBKey overload.
- src/WebIDBObjectStoreImpl.h: Delete IDBKey overload.
(WebIDBObjectStoreImpl):
LayoutTests:
- storage/indexeddb/deleted-objects-expected.txt:
- storage/indexeddb/resources/deleted-objects.js: Test IDBKeyRange overloads.
(openDatabase.request.onsuccess.request.onsuccess):
(openDatabase.request.onsuccess):
(openDatabase):
- 5:10 PM Changeset in webkit [120887] by
-
- 3 edits4 adds in trunk
[Shadow][Editing] Selection will break editing boundaries in Shadow DOM.
https://bugs.webkit.org/show_bug.cgi?id=89075
Reviewed by Antti Koivisto.
Source/WebCore:
At the shadow boundary, some styles (e.g. user-modify, text-decoration) cannot be inherited
from the shadow host. However, when style property cache is used, such styles are wrongly
inherited from the cache.
So this patch makes not to use cache for inehrited styles at the shadow boundary.
Tests: editing/shadow/breaking-editing-boundary-with-table.html
editing/shadow/contenteditable-propagation-at-shadow-boundary.html
- css/StyleResolver.cpp:
(WebCore::StyleResolver::collectMatchingRulesForList):
LayoutTests:
Test cases to confirm that (1) selection does not break editing boundary,
and (2) content-editable is not propagated from shadow host to shadow DOM.
Actually a patch for Bug 88514 contains a test for (2), but it is not enough.
- editing/shadow/breaking-editing-boundary-with-table.html: Added.
- editing/shadow/contenteditable-propagation-at-shadow-boundary.html: Added.
- 4:55 PM Changeset in webkit [120886] by
-
- 2 edits in trunk/Source/JavaScriptCore
Functions on global objects should be specializable
https://bugs.webkit.org/show_bug.cgi?id=89615
Reviewed by Oliver Hunt.
I tested to see if this brought back the bug in https://bugs.webkit.org/show_bug.cgi?id=33343,
and it didn't. Bug 33343 was the reason why we disabled global object function specialization
to begin with. So I'm guessing this is safe.
- runtime/JSGlobalObject.cpp:
(JSC::JSGlobalObject::init):
- 4:49 PM Changeset in webkit [120885] by
-
- 15 edits in trunk/Source
Source/WebCore: Moving cookieEnabled/setCookieEnabled from Page to Settings.
https://bugs.webkit.org/show_bug.cgi?id=89545
Patch by Garret Kelly <gdk@chromium.org> on 2012-06-20
Reviewed by Adam Barth.
- dom/Document.cpp:
(WebCore::Document::cookie):
(WebCore::Document::setCookie):
- page/Navigator.cpp:
(WebCore::Navigator::cookieEnabled):
- page/Page.h:
(Page):
- page/Settings.cpp:
(WebCore::Settings::Settings):
- page/Settings.h:
(WebCore::Settings::setCookieEnabled):
(WebCore::Settings::cookieEnabled):
(Settings):
- platform/graphics/win/MediaPlayerPrivateQuickTimeVisualContext.cpp:
(WebCore::MediaPlayerPrivateQuickTimeVisualContext::setUpCookiesForQuickTime):
Source/WebKit/chromium: Moving cookieEnabled/setCookieEnabled from Page to Settings, and
exposing through WebSettings.
https://bugs.webkit.org/show_bug.cgi?id=89545
Patch by Garret Kelly <gdk@chromium.org> on 2012-06-20
Reviewed by Adam Barth.
- public/WebSettings.h:
- src/WebSettingsImpl.cpp:
(WebKit::WebSettingsImpl::setCookieEnabled):
- src/WebSettingsImpl.h:
(WebSettingsImpl):
Source/WebKit/mac: Moving cookieEnabled/setCookieEnabled from Page to Settings.
https://bugs.webkit.org/show_bug.cgi?id=89545
Patch by Garret Kelly <gdk@chromium.org> on 2012-06-20
Reviewed by Adam Barth.
- WebView/WebView.mm:
(-[WebView _cookieEnabled]):
(-[WebView _setCookieEnabled:]):
Source/WebKit/win: Moving cookieEnabled/setCookieEnabled from Page to Settings.
https://bugs.webkit.org/show_bug.cgi?id=89545
Patch by Garret Kelly <gdk@chromium.org> on 2012-06-20
Reviewed by Adam Barth.
- WebView.cpp:
(WebView::setCookieEnabled):
(WebView::cookieEnabled):
- 4:45 PM Changeset in webkit [120884] by
-
- 4 edits in branches/safari-536-branch/Source
Versioning.
- 4:37 PM Changeset in webkit [120883] by
-
- 6 edits in trunk
internals.settings.setDeviceScaleFactor doesn't work for WebKit2 and must be removed
https://bugs.webkit.org/show_bug.cgi?id=89274
Reviewed by James Robinson.
Source/WebCore:
Using this window.internals API causes the apple-mac port to ASSERT in
WebKit2 because WebKit2 keeps a copy of this state in the UI process.
When a test uses this internals API, the WebCore state gets out of sync
with the state in the UI process.
Tests should use setBackingScaleFactor instead.
- testing/InternalSettings.cpp:
(WebCore):
- testing/InternalSettings.h:
(InternalSettings):
- testing/InternalSettings.idl:
LayoutTests:
- fast/hidpi/device-scale-factor-paint.html:
- 4:36 PM Changeset in webkit [120882] by
-
- 3 edits in trunk/Source/WebCore
Web Inspector: persist timeline panel overview mode
https://bugs.webkit.org/show_bug.cgi?id=88711
Reviewed by Pavel Feldman.
- replace all internal methods of switching to a new mode of TimelineOverviewPane with setMode();
- make current mode a persistent setting;
- inspector/front-end/TimelineOverviewPane.js:
(WebInspector.TimelineOverviewPane):
(WebInspector.TimelineOverviewPane.prototype.setMode):
- inspector/front-end/TimelinePanel.js:
(WebInspector.TimelinePanel):
- 4:31 PM Changeset in webkit [120881] by
-
- 2 edits in trunk/Source/WebKit/chromium
Perform hit-test from correct frame in isRectTopmost()
https://bugs.webkit.org/show_bug.cgi?id=89492
Patch by Raymes Khoury <raymes@chromium.org> on 2012-06-20
Reviewed by Levi Weintraub.
Hit-testing was being performed from the main frame, which gave the
incorrect result if the plugin was in a different frame.
- src/WebPluginContainerImpl.cpp:
(WebKit::WebPluginContainerImpl::isRectTopmost):
- 4:17 PM Changeset in webkit [120880] by
-
- 1 copy in tags/Safari-536.21
New Tag.
- 4:05 PM Changeset in webkit [120879] by
-
- 8 edits in trunk/Source
<rdar://problem/11653784> and https://bugs.webkit.org/show_bug.cgi?id=89590
showModalDialog message handling is flaky in WebKit2
Because RunLoop::performWork() swaps the function queue to a temporary Vector before calling
the functions an inner run-loop - such as we see with running a modal dialog - does not have
a change to handle any of the functions that were queued after the WebPageProxy::RunModal message.
By servicing the functions in the queue one at a time we can give the RunLoop a chance to pick up
where it left off if RunLoop::performWork is re-entered.
To guarantee RunLoop::performWork is re-entered to handle those functions we also need to signal
its source before entering the modal run loop so our RunLoop is woken up.
Reviewed by Darin Adler.
Source/WebCore:
- WebCore.exp.in:
- platform/RunLoop.cpp:
(WebCore::RunLoop::performWork): Take the first function off the queue one at a time so subsequent
functions remain in the queue and can be handled by an inner modal run loop.
- platform/RunLoop.h:
(RunLoop): Change the function queue to be a Deque to efficiently support "takeFirst"
Source/WebKit2:
- Platform/CoreIPC/Connection.cpp:
(CoreIPC::Connection::wakeUpRunLoop): Added so the WebPageProxy can signal the runloop to be woken up
before it enters the modal dialog run loop.
- Platform/CoreIPC/Connection.h:
- UIProcess/WebPageProxy.cpp:
(WebKit::WebPageProxy::runModal): Call wakeUpRunLoop() before entering the modal dialog run loop.
- 4:01 PM Changeset in webkit [120878] by
-
- 2 edits in trunk/Tools
<http://webkit.org/b/89606> Teach run-safari and debug-safari to work with a Safari.app that has entitlements
Reviewed by Dan Bernstein.
- Scripts/webkitdirs.pm:
(executableHasEntitlements):
(safariPathFromSafariBundle):
- 3:57 PM Changeset in webkit [120877] by
-
- 2 edits in trunk/Source/WebKit/chromium
[chromium] Support mobile device rotation resizing
https://bugs.webkit.org/show_bug.cgi?id=86819
Patch by Alexandre Elias <aelias@google.com> on 2012-06-20
Reviewed by Adam Barth.
Resizes on a mobile device are caused either by rotation or
on-screen-keyboard bringup, and need different treatment to remain
naturally laid out, scaled and scrolled without disorienting the user.
- src/WebViewImpl.cpp:
(WebKit::WebViewImpl::resize):
- 3:49 PM Changeset in webkit [120876] by
-
- 3 edits1 add in trunk
Account for device scale factor when creating image for dragging.
https://bugs.webkit.org/show_bug.cgi?id=89489
Patch by Varun Jain <varunjain@chromium.org> on 2012-06-20
Reviewed by Adam Barth.
.:
- ManualTests/chromium/drag-image-accounts-for-device-scale.html: Added.
Source/WebCore:
Manual Test: ManualTests/chromium/drag-image-accounts-for-device-scale.html
- page/Frame.cpp:
(WebCore::Frame::nodeImage):
(WebCore::Frame::dragImageForSelection):
- 3:36 PM Changeset in webkit [120875] by
-
- 1 edit in trunk/Source/WebCore/ChangeLog
Fixed typos in the change log.
- 3:35 PM Changeset in webkit [120874] by
-
- 5 edits in trunk/Source
Add a != operator to ViewportArguments
https://bugs.webkit.org/show_bug.cgi?id=87505
Reviewed by Antonio Gomes.
Patch by Jacky Jiang <zhajiang@rim.com>
Source/WebCore:
Add a != operator to ViewportArguments for convenience.
- dom/ViewportArguments.h:
(ViewportArguments):
(WebCore::ViewportArguments::operator!=):
Source/WebKit/blackberry:
Use != operator of ViewportArguments.
- Api/WebPage.cpp:
(BlackBerry::WebKit::WebPagePrivate::setLoadState):
(BlackBerry::WebKit::WebPagePrivate::setViewportSize):
- Api/WebViewportArguments.cpp:
(BlackBerry::WebKit::WebViewportArguments::operator!=):
- 3:25 PM Changeset in webkit [120873] by
-
- 2 edits in trunk/LayoutTests
[Qt] Unreviewed gardening, skip new failing tests.
- platform/qt/Skipped:
- 3:22 PM Changeset in webkit [120872] by
-
- 1 edit14 copies in branches/chromium/1180
Merge 120859 - REGRESSION(r113885): Margin not properly applied to elements with align=center
https://bugs.webkit.org/show_bug.cgi?id=89515
Reviewed by Levi Weintraub.
Source/WebCore:
Tests: fast/block/negative-margin-start-positive-margin-end.html
fast/block/negative-start-margin-align-center.html
fast/block/positive-margin-block-child-align-center-rtl.html
fast/block/positive-margin-block-child-align-center.html
fast/block/positive-margin-start-align-center.html
fast/block/positive-margin-start-negative-margin-end-align-center.html
fast/table/table-cell-negative-start-margin-align-center.html
r113885 changed the code-path for elements with auto width to call computeInlineDirectionMargins.
However this uncovered an existing bug in the function when dealing with align="center" (text-align: -webkit-center)
where we would ignore the margin. This goes against what other browsers are doing and our previous behavior.
Note that align="left" and "right" are likely impacted too and will be investigated / fixed in follow-up changes.
- rendering/RenderBox.cpp:
(WebCore::RenderBox::computeInlineDirectionMargins):
To match other browsers' behavior, changed the function to include margin in our computations.
LayoutTests:
- fast/block/negative-margin-start-positive-margin-end-expected.html: Added.
- fast/block/negative-margin-start-positive-margin-end.html: Added.
- fast/block/negative-start-margin-align-center-expected.html: Added.
- fast/block/negative-start-margin-align-center.html: Added.
- fast/block/positive-margin-block-child-align-center-expected.html: Added.
- fast/block/positive-margin-block-child-align-center.html: Added.
- fast/block/positive-margin-start-align-center-expected.html: Added.
- fast/block/positive-margin-start-align-center.html: Added.
- fast/block/positive-margin-start-negative-margin-end-align-center-expected.html: Added.
- fast/block/positive-margin-start-negative-margin-end-align-center.html: Added.
Those checks the combination of margin start / end both positive and negative.
- fast/block/positive-margin-block-child-align-center-rtl-expected.html: Added.
- fast/block/positive-margin-block-child-align-center-rtl.html: Added.
One ltr test as I didn't find any.
- fast/table/table-cell-negative-start-margin-align-center-expected.html: Added.
- fast/table/table-cell-negative-start-margin-align-center.html: Added.
This test is very similar to the one above but involves a table.
TBR=jchaffraix@webkit.org
Review URL: https://chromiumcodereview.appspot.com/10578056
- 3:17 PM Changeset in webkit [120871] by
-
- 1 edit14 copies in branches/chromium/1132
Merge 120859 - REGRESSION(r113885): Margin not properly applied to elements with align=center
https://bugs.webkit.org/show_bug.cgi?id=89515
Reviewed by Levi Weintraub.
Source/WebCore:
Tests: fast/block/negative-margin-start-positive-margin-end.html
fast/block/negative-start-margin-align-center.html
fast/block/positive-margin-block-child-align-center-rtl.html
fast/block/positive-margin-block-child-align-center.html
fast/block/positive-margin-start-align-center.html
fast/block/positive-margin-start-negative-margin-end-align-center.html
fast/table/table-cell-negative-start-margin-align-center.html
r113885 changed the code-path for elements with auto width to call computeInlineDirectionMargins.
However this uncovered an existing bug in the function when dealing with align="center" (text-align: -webkit-center)
where we would ignore the margin. This goes against what other browsers are doing and our previous behavior.
Note that align="left" and "right" are likely impacted too and will be investigated / fixed in follow-up changes.
- rendering/RenderBox.cpp:
(WebCore::RenderBox::computeInlineDirectionMargins):
To match other browsers' behavior, changed the function to include margin in our computations.
LayoutTests:
- fast/block/negative-margin-start-positive-margin-end-expected.html: Added.
- fast/block/negative-margin-start-positive-margin-end.html: Added.
- fast/block/negative-start-margin-align-center-expected.html: Added.
- fast/block/negative-start-margin-align-center.html: Added.
- fast/block/positive-margin-block-child-align-center-expected.html: Added.
- fast/block/positive-margin-block-child-align-center.html: Added.
- fast/block/positive-margin-start-align-center-expected.html: Added.
- fast/block/positive-margin-start-align-center.html: Added.
- fast/block/positive-margin-start-negative-margin-end-align-center-expected.html: Added.
- fast/block/positive-margin-start-negative-margin-end-align-center.html: Added.
Those checks the combination of margin start / end both positive and negative.
- fast/block/positive-margin-block-child-align-center-rtl-expected.html: Added.
- fast/block/positive-margin-block-child-align-center-rtl.html: Added.
One ltr test as I didn't find any.
- fast/table/table-cell-negative-start-margin-align-center-expected.html: Added.
- fast/table/table-cell-negative-start-margin-align-center.html: Added.
This test is very similar to the one above but involves a table.
TBR=jchaffraix@webkit.org
Review URL: https://chromiumcodereview.appspot.com/10536229
- 3:15 PM Changeset in webkit [120870] by
-
- 8 edits21 deletes in trunk/LayoutTests
Computed style tests are a maintenance burden on the project
https://bugs.webkit.org/show_bug.cgi?id=87991
Reviewed by Eric Seidel.
Convert computed-style.html, computed-style-without-renderer.html and getComputedStyle-basic.xhtml
to use a whitelist of properties rather than a blacklist. The whitelist includes all properties
that are supported based on the checked in results so all ports can share the same values. Adding
new properties won't cause these tests to be rebaselined.
- fast/css/getComputedStyle/computed-style-expected.txt:
- fast/css/getComputedStyle/computed-style-without-renderer-expected.txt:
- fast/css/getComputedStyle/computed-style-without-renderer.html: Remove indenting and use a fixed
width for the middle div so we don't get fractional pixel values for ports with fractional pixel
values enabled.
- fast/css/getComputedStyle/computed-style.html:
- fast/css/getComputedStyle/resources/property-names.js: Whitelist of common CSS properties.
- platform/chromium-linux-x86/svg/css/getComputedStyle-basic-expected.txt: Removed.
- platform/chromium-linux/svg/css/getComputedStyle-basic-expected.txt: Removed.
- platform/chromium-mac-leopard/fast/css/getComputedStyle/computed-style-expected.txt: Removed.
- platform/chromium-mac-leopard/svg/css/getComputedStyle-basic-expected.txt: Removed.
- platform/chromium-mac-snowleopard/svg/css/getComputedStyle-basic-expected.txt: Removed.
- platform/chromium-mac/fast/css/getComputedStyle/computed-style-expected.txt: Removed.
- platform/chromium-mac/fast/css/getComputedStyle/computed-style-without-renderer-expected.txt: Removed.
- platform/chromium-mac/svg/css/getComputedStyle-basic-expected.txt: Removed.
- platform/chromium-win-vista/svg/css/getComputedStyle-basic-expected.txt: Removed.
- platform/chromium-win-xp/svg/css/getComputedStyle-basic-expected.txt: Removed.
- platform/chromium-win/fast/css/getComputedStyle/computed-style-expected.txt: Removed.
- platform/chromium-win/fast/css/getComputedStyle/computed-style-without-renderer-expected.txt: Removed.
- platform/chromium-win/svg/css/getComputedStyle-basic-expected.txt: Removed.
- platform/gtk/fast/css/getComputedStyle/computed-style-expected.txt: Removed.
- platform/gtk/svg/css/getComputedStyle-basic-expected.txt: Removed.
- platform/mac/fast/css/getComputedStyle/computed-style-expected.txt: Removed.
- platform/mac/fast/css/getComputedStyle/computed-style-without-renderer-expected.txt: Removed.
- platform/mac/svg/css/getComputedStyle-basic-expected.txt: Removed.
- platform/qt/fast/css/getComputedStyle/computed-style-without-renderer-expected.txt: Removed.
- platform/qt/svg/css/getComputedStyle-basic-expected.png: Removed.
- platform/qt/svg/css/getComputedStyle-basic-expected.txt: Removed.
- platform/win/fast/css/getComputedStyle/computed-style-expected.txt: Removed.
- platform/win/fast/css/getComputedStyle/computed-style-without-renderer-expected.txt: Removed.
- svg/css/getComputedStyle-basic-expected.txt:
- svg/css/getComputedStyle-basic.xhtml:
- 3:08 PM Changeset in webkit [120869] by
-
- 2 edits in trunk/Source/WebCore
Let XCode have its own way.
- WebCore.xcodeproj/project.pbxproj:
- 3:05 PM Changeset in webkit [120868] by
-
- 6 edits in trunk/Source/WebCore
Move m_listsInvalidatedAtDocument from NodeListsNodeData to Document
https://bugs.webkit.org/show_bug.cgi?id=89603
Reviewed by Andreas Kling.
Moved the variable. m_listsInvalidatedAtDocument is never used in non-Document nodes
so it was just wasting memory space.
- dom/Document.cpp:
(WebCore::Document::registerDynamicSubtreeNodeList):
(WebCore::Document::unregisterDynamicSubtreeNodeList):
(WebCore):
(WebCore::Document::clearNodeListCaches):
- dom/Document.h:
(Document):
- dom/DynamicNodeList.h:
(DynamicSubtreeNodeList):
- dom/Node.cpp:
(WebCore::Node::invalidateNodeListsCacheAfterAttributeChanged):
(WebCore::Node::invalidateNodeListsCacheAfterChildrenChanged):
(WebCore):
(WebCore::NodeListsNodeData::invalidateCachesThatDependOnAttributes): RadioNodeList
is invalidated at document level. No need to invalidate it again here.
(WebCore::NodeListsNodeData::isEmpty):
- dom/NodeRareData.h:
(NodeListsNodeData):
- 3:02 PM Changeset in webkit [120867] by
-
- 3 edits in trunk/Tools
Unreviewed, rolling out r120821.
http://trac.webkit.org/changeset/120821
https://bugs.webkit.org/show_bug.cgi?id=89605
It made duplicated reviewer entries (Requested by Ossy on
#webkit).
Patch by Sheriff Bot <webkit.review.bot@gmail.com> on 2012-06-20
- Scripts/webkitpy/common/checkout/changelog.py:
(ChangeLog.set_reviewer):
- Scripts/webkitpy/common/checkout/changelog_unittest.py:
(test_set_reviewer):
(test_set_short_description_and_bug_url):
- 2:58 PM Changeset in webkit [120866] by
-
- 44 edits3 copies in trunk/Source/WebCore
Unreviewed, rolling out r120854.
http://trac.webkit.org/changeset/120854
https://bugs.webkit.org/show_bug.cgi?id=89604
Broke Chromium WebKit Linux (dbg) (Requested by arv on
#webkit).
Patch by Sheriff Bot <webkit.review.bot@gmail.com> on 2012-06-20
- Target.pri:
- UseV8.cmake:
- WebCore.gypi:
- bindings/scripts/CodeGeneratorJS.pm:
(GenerateHeader):
(GenerateImplementation):
- bindings/scripts/CodeGeneratorV8.pm:
(GenerateHeader):
(GenerateNamedConstructorCallback):
(GenerateImplementation):
- bindings/scripts/IDLAttributes.txt:
- bindings/scripts/test/V8/V8Float64Array.cpp:
(WebCore):
- bindings/scripts/test/V8/V8Float64Array.h:
(V8Float64Array):
- bindings/scripts/test/V8/V8TestActiveDOMObject.cpp:
(WebCore):
- bindings/scripts/test/V8/V8TestActiveDOMObject.h:
(V8TestActiveDOMObject):
- bindings/scripts/test/V8/V8TestCustomNamedGetter.cpp:
(WebCore):
- bindings/scripts/test/V8/V8TestCustomNamedGetter.h:
(V8TestCustomNamedGetter):
- bindings/scripts/test/V8/V8TestEventConstructor.cpp:
(WebCore):
- bindings/scripts/test/V8/V8TestEventConstructor.h:
(V8TestEventConstructor):
- bindings/scripts/test/V8/V8TestEventTarget.cpp:
(WebCore):
- bindings/scripts/test/V8/V8TestEventTarget.h:
(V8TestEventTarget):
- bindings/scripts/test/V8/V8TestException.cpp:
(WebCore):
- bindings/scripts/test/V8/V8TestException.h:
(V8TestException):
- bindings/scripts/test/V8/V8TestInterface.cpp:
(WebCore):
- bindings/scripts/test/V8/V8TestInterface.h:
(V8TestInterface):
- bindings/scripts/test/V8/V8TestMediaQueryListListener.cpp:
(WebCore):
- bindings/scripts/test/V8/V8TestMediaQueryListListener.h:
(V8TestMediaQueryListListener):
- bindings/scripts/test/V8/V8TestNamedConstructor.cpp:
(WebCore):
- bindings/scripts/test/V8/V8TestNamedConstructor.h:
(V8TestNamedConstructor):
- bindings/scripts/test/V8/V8TestNode.cpp:
(WebCore):
- bindings/scripts/test/V8/V8TestNode.h:
(V8TestNode):
- bindings/scripts/test/V8/V8TestObj.cpp:
(WebCore):
- bindings/scripts/test/V8/V8TestObj.h:
(V8TestObj):
- bindings/scripts/test/V8/V8TestSerializedScriptValueInterface.cpp:
(WebCore):
- bindings/scripts/test/V8/V8TestSerializedScriptValueInterface.h:
(V8TestSerializedScriptValueInterface):
- bindings/v8/NPV8Object.cpp:
(WebCore::npObjectTypeInfo):
- bindings/v8/V8GCController.cpp:
(WebCore::GrouperVisitor::visitDOMWrapper):
- bindings/v8/WrapperTypeInfo.h:
(WebCore):
(WrapperTypeInfo):
- bindings/v8/custom/V8CSSStyleSheetCustom.cpp: Copied from Source/WebCore/bindings/v8/custom/V8StyleSheetCustom.cpp.
(WebCore):
(WebCore::toV8):
- bindings/v8/custom/V8DOMStringMapCustom.cpp:
(WebCore::toV8):
(WebCore):
- bindings/v8/custom/V8DOMTokenListCustom.cpp: Copied from Source/WebCore/bindings/v8/custom/V8StyleSheetCustom.cpp.
(WebCore):
(WebCore::toV8):
- bindings/v8/custom/V8HTMLImageElementConstructor.cpp:
(WebCore):
- bindings/v8/custom/V8NamedNodeMapCustom.cpp:
(WebCore::toV8):
(WebCore):
- bindings/v8/custom/V8StyleSheetCustom.cpp:
(WebCore::toV8):
- bindings/v8/custom/V8TextTrackListCustom.cpp: Copied from Source/WebCore/bindings/v8/custom/V8StyleSheetCustom.cpp.
(WebCore):
(WebCore::toV8):
- css/CSSStyleSheet.idl:
- css/StyleSheet.idl:
- dom/DOMStringMap.idl:
- dom/NamedNodeMap.idl:
- html/DOMTokenList.idl:
- html/track/TextTrackList.idl:
- 2:30 PM Changeset in webkit [120865] by
-
- 8 edits in trunk/Tools
nrwt outputs empty files for wdiff output if wdiff is not installed
https://bugs.webkit.org/show_bug.cgi?id=88709
Reviewed by Tony Chang.
Reviewed by Tony Chang.
Don't write -wdiff or -pretty.html files if wdiff or prettypatch
aren't available, and clean up the handling for them in the port
code.
- Scripts/webkitpy/run_webkit_tests_integrationtest.py:
(MainTest.test_output_diffs):
- Scripts/webkitpy/layout_tests/controllers/test_result_writer.py:
(TestResultWriter.create_text_diff_and_write_result):
- Scripts/webkitpy/layout_tests/port/base.py:
(Port.check_pretty_patch):
(Port.check_wdiff):
(Port._wdiff_missing_message):
- Scripts/webkitpy/layout_tests/port/chromium_linux.py:
(ChromiumLinuxPort.check_build):
(ChromiumLinuxPort._wdiff_missing_message):
- Scripts/webkitpy/layout_tests/port/chromium_mac.py:
(ChromiumMacPort.check_build):
(ChromiumMacPort):
(ChromiumMacPort._wdiff_missing_message):
- Scripts/webkitpy/layout_tests/port/chromium_mac_unittest.py:
(ChromiumMacPortTest):
- 2:25 PM Changeset in webkit [120864] by
-
- 3 edits in trunk/Tools
nrwt: fix unit tests for ensuring svn revision is correct
https://bugs.webkit.org/show_bug.cgi?id=89498
Reviewed by Eric Seidel.
Reviewed by Eric Seidel.
Fix the unit tests for testing that we are embedding the SVN
revision in the results json for NRWT properly; this was broken
in r120646 but because the MockHost worked differently than the
real one (by always initializing the MockSCM object) we didn't
notice. Unfortunately, just changing the default breaks all
sorts of unit tests ...
- Scripts/webkitpy/common/host_mock.py:
(MockHost.init):
(MockHost._initialize_scm):
- Scripts/webkitpy/layout_tests/controllers/manager_unittest.py:
(ResultSummaryTest.test_no_svn_revision):
(ResultSummaryTest.test_svn_revision):
- 2:22 PM Changeset in webkit [120863] by
-
- 5 edits in trunk/Tools
new-run-webkit-tests appends "/Debug" or "/Release" to $WEBKITOUTPUTDIR
https://bugs.webkit.org/show_bug.cgi?id=69360
Reviewed by Eric Seidel.
Reviewed by Eric Seidel.
Propagate the 'port_implementation' part of the platform (i.e.,
gtk,qt,chromium) to webkit-build-directory so that we can pick
up the gtk-specific handling of WEBKITOUTPUTDIR ...
I didn't write any additional tests for this; testing it
properly is an integration test between the python code and the
perl code, which I verified by hand.
- Scripts/webkitpy/layout_tests/port/base.py:
(Port.init):
- Scripts/webkitpy/layout_tests/port/config.py:
(Config.init):
(Config.build_directory):
- Scripts/webkitpy/layout_tests/port/config_mock.py:
(MockConfig.init):
- Scripts/webkitpy/layout_tests/port/config_unittest.py:
(ConfigTest.test_build_directory_passes_port_implementation):
- 2:20 PM Changeset in webkit [120862] by
-
- 5 edits2 adds in trunk
Crash on accessing a removed renderer from percent height descendant map.
https://bugs.webkit.org/show_bug.cgi?id=88017
Reviewed by Eric Seidel.
Source/WebCore:
Test: fast/block/percent-height-descendant-not-removed-crash2.html
- rendering/RenderBlock.cpp:
(WebCore::RenderBlock::hasPercentHeightContainerMap): helper to tell
if we have a height container map.
(WebCore):
(WebCore::RenderBlock::hasPercentHeightDescendant): change from a debug
only function to a regular function for use. no need to null check
for a percent height container map in this function.
(WebCore::RenderBlock::clearPercentHeightDescendantsFrom): helper to
clear all percent height descendants under us.
(WebCore::RenderBlock::removePercentHeightDescendantIfNeeded): helper to
clear the box if it exists in the percent height descendant map.
- rendering/RenderBlock.h:
(RenderBlock):
- rendering/RenderBox.cpp:
(WebCore::RenderBox::willBeDestroyed): remove the assert and change the
percent height detection check to use removePercentHeightDescendantIfNeeded.
We shouldn't rely on logicalHeight().isPercent() as it can change when our
writing mode changes. Instead, just query the map directly to see if we exist.
(WebCore::RenderBox::styleDidChange): when our writing mode changes from
horizontal to vertical or vice versa, we clear all our descendants from
the percent height descendant map. Cache the value of isHorizontalWritingMode()
before it changes in styleDidChange and compare it with the new value
(can't use oldStyle->isHorizontalWritingMode() since it can be inherited
and already updated).
LayoutTests:
- fast/block/percent-height-descendant-not-removed-crash2-expected.txt: Added.
- fast/block/percent-height-descendant-not-removed-crash2.html: Added.
- 2:15 PM Changeset in webkit [120861] by
-
- 4 edits in trunk/Source/WebKit/blackberry
[BlackBerry] Add an API to immediately enable cross-site XHR
https://bugs.webkit.org/show_bug.cgi?id=89594
Internally Reviewed by Yong Li.
Patch by Christopher Hutten-Czapski <chutten@rim.com> on 2012-06-20
Reviewed by Rob Buis.
There is no API to disable it as clients may rely on legacy behaviour
of not disabling until the next frame load.
- Api/WebPage.cpp:
(BlackBerry::WebKit::enableCrossSiteXHRRecursively):
(WebKit):
(BlackBerry::WebKit::WebPagePrivate::enableCrossSiteXHR):
(BlackBerry::WebKit::WebPage::enableCrossSiteXHR):
- Api/WebPage.h:
- Api/WebPage_p.h:
(WebPagePrivate):
- 2:07 PM Changeset in webkit [120860] by
-
- 2 edits in trunk/Source/JavaScriptCore
build-webkit failure due to illegal 32-bit integer constants in code
generated by offlineasm
https://bugs.webkit.org/show_bug.cgi?id=89347
Reviewed by Geoffrey Garen.
The offending constants are the magic numbers used by offlineasm to find
offsets in the generated machine code. Added code to turn them into what
the C++ compiler will believe to be valid 32-bit values.
- offlineasm/offsets.rb:
- 2:04 PM Changeset in webkit [120859] by
-
- 3 edits14 adds in trunk
REGRESSION(r113885): Margin not properly applied to elements with align=center
https://bugs.webkit.org/show_bug.cgi?id=89515
Reviewed by Levi Weintraub.
Reviewed by Levi Weintraub.
Source/WebCore:
Tests: fast/block/negative-margin-start-positive-margin-end.html
fast/block/negative-start-margin-align-center.html
fast/block/positive-margin-block-child-align-center-rtl.html
fast/block/positive-margin-block-child-align-center.html
fast/block/positive-margin-start-align-center.html
fast/block/positive-margin-start-negative-margin-end-align-center.html
fast/table/table-cell-negative-start-margin-align-center.html
r113885 changed the code-path for elements with auto width to call computeInlineDirectionMargins.
However this uncovered an existing bug in the function when dealing with align="center" (text-align: -webkit-center)
where we would ignore the margin. This goes against what other browsers are doing and our previous behavior.
Note that align="left" and "right" are likely impacted too and will be investigated / fixed in follow-up changes.
- rendering/RenderBox.cpp:
(WebCore::RenderBox::computeInlineDirectionMargins):
To match other browsers' behavior, changed the function to include margin in our computations.
LayoutTests:
- fast/block/negative-margin-start-positive-margin-end-expected.html: Added.
- fast/block/negative-margin-start-positive-margin-end.html: Added.
- fast/block/negative-start-margin-align-center-expected.html: Added.
- fast/block/negative-start-margin-align-center.html: Added.
- fast/block/positive-margin-block-child-align-center-expected.html: Added.
- fast/block/positive-margin-block-child-align-center.html: Added.
- fast/block/positive-margin-start-align-center-expected.html: Added.
- fast/block/positive-margin-start-align-center.html: Added.
- fast/block/positive-margin-start-negative-margin-end-align-center-expected.html: Added.
- fast/block/positive-margin-start-negative-margin-end-align-center.html: Added.
Those checks the combination of margin start / end both positive and negative.
- fast/block/positive-margin-block-child-align-center-rtl-expected.html: Added.
- fast/block/positive-margin-block-child-align-center-rtl.html: Added.
One ltr test as I didn't find any.
- fast/table/table-cell-negative-start-margin-align-center-expected.html: Added.
- fast/table/table-cell-negative-start-margin-align-center.html: Added.
This test is very similar to the one above but involves a table.
- 1:48 PM Changeset in webkit [120858] by
-
- 8 edits in trunk/Source
[chromium] Separate LayerRenderer initialization from updateLayers
https://bugs.webkit.org/show_bug.cgi?id=89525
Reviewed by Adrienne Walker.
Source/WebCore:
This adds an explicit call to initialize the layer renderer of a given CCLayerTreeHost instead of having it be
implicit in updateLayers(). This way the proxies can control the initialization sequence more closely and do
useful work between the two calls.
Refactor, no change in behavior. Covered by existing tests.
- platform/graphics/chromium/cc/CCLayerTreeHost.cpp:
(WebCore::CCLayerTreeHost::compositeAndReadback):
(WebCore::CCLayerTreeHost::initializeLayerRendererIfNeeded):
(WebCore):
(WebCore::CCLayerTreeHost::updateLayers):
- platform/graphics/chromium/cc/CCLayerTreeHost.h:
(CCLayerTreeHost):
- platform/graphics/chromium/cc/CCSingleThreadProxy.cpp:
(WebCore::CCSingleThreadProxy::commitAndComposite):
- platform/graphics/chromium/cc/CCThreadProxy.cpp:
(WebCore::CCThreadProxy::compositeAndReadback):
(WebCore::CCThreadProxy::beginFrame):
Source/WebKit/chromium:
Update tests to call initializeLayerRendererIfNeeded() before calling updateLayers() to reflect what the proxies
do.
- tests/CCLayerTreeHostTest.cpp:
(WTF::CCLayerTreeHostTestLayerOcclusion::beginTest):
(WTF::CCLayerTreeHostTestLayerOcclusionWithFilters::beginTest):
(WTF::CCLayerTreeHostTestManySurfaces::beginTest):
- tests/TiledLayerChromiumTest.cpp:
- 1:33 PM Changeset in webkit [120857] by
-
- 1 edit2 copies in branches/chromium/1180
Merge 120737 - Crash in RenderInline::willBeDestroyed.
BUG=103423
Review URL: https://chromiumcodereview.appspot.com/10575040
- 1:25 PM Changeset in webkit [120856] by
-
- 2 edits in trunk/LayoutTests
r120844: Skip two tests on Qt
The tests fast/block/float/previous-sibling-abspos-001.html and
fast/block/float/previous-sibling-float-001.html have pixel differences
on Qt. Skip them for now - bug 89597 opened.
Unreviewed, gardening.
- platform/qt/Skipped:
- 1:23 PM Changeset in webkit [120855] by
-
- 2 edits in trunk/Source/WebCore
SecurityOrigin::canDisplay() should return true when m_universalAccess is true
https://bugs.webkit.org/show_bug.cgi?id=84865
Patch by Yong Li <yoli@rim.com> on 2012-06-20
Reviewed by Adam Barth.
Add an early return (true) in canDisplay() for if (m_universalAccess),
as we did for canAccess() and canRequest().
No new tests. We should be good as long as this change doesn't break existing
tests, which means the contents supposed to be blocked are still blocked when
m_universalAccess is false.
- page/SecurityOrigin.cpp:
(WebCore::SecurityOrigin::canDisplay):
- 1:16 PM Changeset in webkit [120854] by
-
- 44 edits3 deletes in trunk/Source/WebCore
[V8] Use v8::V8::AddImplicitReferences instead of SetHiddenValue
https://bugs.webkit.org/show_bug.cgi?id=80880
Reviewed by Adam Barth.
We used to add a hidden property in the getter to the returned wrapper.
With this patch we instead handle the liveness of the wrapper in the GC phase by
calling v8::V8::AddHiddenReference.
To reduce the amount of custom code we need, the V8 code generator now supports
GenerateIsReachable (as well as CustomIsReachable) which, even though different
from the JSC attribute, is used in the same cases and takes the same values (even though
at the moment not all JSC values are supported by V8). Interfaces that have *IsReachable
also have a dependent life time (just like if V8DependentLifetime was present).
Retry. Last time we got ASAN issues on some Cromium Linux bots.
No new tests. Covered by existing tests.
- Target.pri:
- UseV8.cmake:
- WebCore.gypi:
- bindings/scripts/CodeGeneratorJS.pm:
(GetGenerateIsReachable): Abstracted GenerateIsReachable and JSGenerateIsReachable.
(GetCustomIsReachable): Ditto.
(GenerateHeader):
(GenerateImplementation):
- bindings/scripts/CodeGeneratorV8.pm:
(NeedsToVisitDOMWrapper):
(GetGenerateIsReachable):
(GetCustomIsReachable):
(GenerateVisitDOMWrapper):
(GenerateHeader):
(GenerateNamedConstructorCallback):
(GenerateImplementation):
- bindings/scripts/IDLAttributes.txt:
- bindings/scripts/test/V8/V8Float64Array.cpp:
(WebCore):
- bindings/scripts/test/V8/V8Float64Array.h:
(V8Float64Array):
- bindings/scripts/test/V8/V8TestActiveDOMObject.cpp:
(WebCore):
- bindings/scripts/test/V8/V8TestActiveDOMObject.h:
(V8TestActiveDOMObject):
- bindings/scripts/test/V8/V8TestCustomNamedGetter.cpp:
(WebCore):
- bindings/scripts/test/V8/V8TestCustomNamedGetter.h:
(V8TestCustomNamedGetter):
- bindings/scripts/test/V8/V8TestEventConstructor.cpp:
(WebCore):
- bindings/scripts/test/V8/V8TestEventConstructor.h:
(V8TestEventConstructor):
- bindings/scripts/test/V8/V8TestEventTarget.cpp:
(WebCore):
- bindings/scripts/test/V8/V8TestEventTarget.h:
(V8TestEventTarget):
- bindings/scripts/test/V8/V8TestException.cpp:
(WebCore):
- bindings/scripts/test/V8/V8TestException.h:
(V8TestException):
- bindings/scripts/test/V8/V8TestInterface.cpp:
(WebCore):
- bindings/scripts/test/V8/V8TestInterface.h:
(V8TestInterface):
- bindings/scripts/test/V8/V8TestMediaQueryListListener.cpp:
(WebCore):
- bindings/scripts/test/V8/V8TestMediaQueryListListener.h:
(V8TestMediaQueryListListener):
- bindings/scripts/test/V8/V8TestNamedConstructor.cpp:
(WebCore):
- bindings/scripts/test/V8/V8TestNamedConstructor.h:
(V8TestNamedConstructor):
- bindings/scripts/test/V8/V8TestNode.cpp:
(WebCore):
- bindings/scripts/test/V8/V8TestNode.h:
(V8TestNode):
- bindings/scripts/test/V8/V8TestObj.cpp:
(WebCore):
- bindings/scripts/test/V8/V8TestObj.h:
(V8TestObj):
- bindings/scripts/test/V8/V8TestSerializedScriptValueInterface.cpp:
(WebCore):
- bindings/scripts/test/V8/V8TestSerializedScriptValueInterface.h:
(V8TestSerializedScriptValueInterface):
- bindings/v8/NPV8Object.cpp:
(WebCore::npObjectTypeInfo):
- bindings/v8/V8GCController.cpp:
(WebCore::GrouperVisitor::visitDOMWrapper):
- bindings/v8/WrapperTypeInfo.h:
(WebCore):
(WrapperTypeInfo):
- bindings/v8/custom/V8CSSStyleSheetCustom.cpp: Removed.
- bindings/v8/custom/V8DOMStringMapCustom.cpp:
- bindings/v8/custom/V8DOMTokenListCustom.cpp: Removed.
- bindings/v8/custom/V8HTMLImageElementConstructor.cpp:
(WebCore):
- bindings/v8/custom/V8NamedNodeMapCustom.cpp:
- bindings/v8/custom/V8StyleSheetCustom.cpp:
(WebCore::toV8):
- bindings/v8/custom/V8TextTrackListCustom.cpp: Removed.
- css/CSSStyleSheet.idl:
- css/StyleSheet.idl:
- dom/DOMStringMap.idl:
- dom/NamedNodeMap.idl:
- html/DOMTokenList.idl:
- html/track/TextTrackList.idl:
- 1:10 PM FeatureFlags edited by
- Add a bug link for CSS_BOX_DECORATION_BREAK (diff)
- 1:00 PM Changeset in webkit [120853] by
-
- 2 edits in trunk/Tools
Fix import sorting missed in r120846
Reviewed by Tony Chang.
- Scripts/webkitpy/layout_tests/servers/apache_http_server_unittest.py:
- 12:53 PM Changeset in webkit [120852] by
-
- 3 edits in trunk/Tools
Fix chromium win http servers after breakage introduced in r120846.
Unreviewed, build fix.
- Scripts/webkitpy/layout_tests/port/base.py:
(Port.to.start_http_server):
- Scripts/webkitpy/layout_tests/servers/http_server.py:
(Lighttpd.init):
- 12:47 PM Changeset in webkit [120851] by
-
- 4 edits in trunk/Tools
tweak output of webkit-patch print-{baselines,expectations}
https://bugs.webkit.org/show_bug.cgi?id=89588
Reviewed by Tony Chang.
Minor tweaks to the output to make it more readable, including
turning off csv by default for print-baselines with multiple ports.
- Scripts/webkitpy/layout_tests/port/test.py:
(TestPort):
- Scripts/webkitpy/tool/commands/queries.py:
(PrintExpectations.execute):
(PrintBaselines.execute):
- Scripts/webkitpy/tool/commands/queries_unittest.py:
(PrintExpectationsTest.run_test):
(PrintExpectationsTest.test_multiple):
(PrintBaselinesTest.setUp):
(PrintBaselinesTest.test_multiple):
- 12:33 PM Changeset in webkit [120850] by
-
- 3 edits in trunk/Source/WebCore
Regression(r116408): Ctrl-A (select all) on large text file hangs the tab with high CPU usage
https://bugs.webkit.org/show_bug.cgi?id=89562
Reviewed by Ryosuke Niwa.
Rather than replace the newlines in-place (in O(n2)), build a new string using StringBuilder, which takes O(n).
No new tests, this is a perf improvement.
- platform/chromium/ClipboardUtilitiesChromium.cpp:
(WebCore::replaceNewlinesWithWindowsStyleNewlines):
- platform/win/ClipboardUtilitiesWin.cpp:
(WebCore::replaceNewlinesWithWindowsStyleNewlines):
- 12:28 PM Changeset in webkit [120849] by
-
- 10 edits17 adds in trunk
Add support for fit-content etc
https://bugs.webkit.org/show_bug.cgi?id=38919
Patch by Elliott Sprehn <Elliott Sprehn> on 2012-06-20
Reviewed by Tony Chang.
Source/WebCore:
Implement the CSS3 intrinsic dimension keywords for width properties and
add most of the plumbing for height properties but don't expose them
yet since this patch doesn't enforce them (matching current Gecko).
http://dev.w3.org/csswg/css3-writing-modes/#intrinsic-sizing
This patch implements -webkit-min-content, -webkit-max-content,
-webkit-fill-available and -webkit-fit-content for all width
properties.
Tests: fast/css-intrinsic-dimensions/height-dynamic-property-value.html
fast/css-intrinsic-dimensions/height-property-value.html
fast/css-intrinsic-dimensions/max-width-constrained.html
fast/css-intrinsic-dimensions/max-width-unconstrained.html
fast/css-intrinsic-dimensions/min-width.html
fast/css-intrinsic-dimensions/width-avoid-floats.html
fast/css-intrinsic-dimensions/width-dynamic-property-value.html
fast/css-intrinsic-dimensions/width-property-value.html
fast/css-intrinsic-dimensions/width.html
- css/CSSParser.cpp: Add parser support for the new keywords.
(WebCore::CSSParser::parseValue):
- css/CSSPrimitiveValue.cpp:
(WebCore):
(WebCore::CSSPrimitiveValue::CSSPrimitiveValue):
- css/CSSValueKeywords.in:
- css/LengthFunctions.cpp: Implement conversion functions.
(WebCore::minimumValueForLength):
(WebCore::valueForLength):
(WebCore::floatValueForLength):
- css/StyleBuilder.cpp:
(WebCore::ApplyPropertyLength::applyValue):
(WebCore::StyleBuilder::StyleBuilder): Clean up the template for old
intrinsic values and add new values. Rename old intrinsic to
LegacyIntrinsic and add support for the new dimension keywords as
Intrinsic.
- platform/Length.h:
(WebCore::Length::isIntrinsicOrAuto):
(WebCore::Length::isLegacyIntrinsic): New method to check for old
min-intrinsic and intrinsic keywords.
(WebCore::Length::isIntrinsic): New method to check for CSS3 intrinsic
dimension keywords like min-content.
- rendering/RenderBox.cpp:
(WebCore::RenderBox::computeLogicalWidthInRegionUsing): Expose the max
and min logical preferred widths as the new keywords.
(WebCore::RenderBox::sizesLogicalWidthToFitContent): Renamed method to
be more consistent with the new MinContent naming in the spec.
- rendering/RenderBox.h:
(RenderBox):
LayoutTests:
Tests for CSS3 intrinsic dimension keywords as defined in the
writing modes spec. This tests the implementation for width and
ensures that height properties don't allow the new keywords yet
since they're not implemented by this patch.
- fast/css-intrinsic-dimensions/height-dynamic-property-value-expected.txt: Added.
- fast/css-intrinsic-dimensions/height-dynamic-property-value.html: Added.
- fast/css-intrinsic-dimensions/height-property-value-expected.txt: Added.
- fast/css-intrinsic-dimensions/height-property-value.html: Added.
- fast/css-intrinsic-dimensions/max-width-constrained-expected.html: Added.
- fast/css-intrinsic-dimensions/max-width-constrained.html: Added.
- fast/css-intrinsic-dimensions/max-width-unconstrained-expected.html: Added.
- fast/css-intrinsic-dimensions/max-width-unconstrained.html: Added.
- fast/css-intrinsic-dimensions/min-width-expected.html: Added.
- fast/css-intrinsic-dimensions/min-width.html: Added.
- fast/css-intrinsic-dimensions/width-avoid-floats-expected.html: Added.
- fast/css-intrinsic-dimensions/width-avoid-floats.html: Added.
- fast/css-intrinsic-dimensions/width-dynamic-property-value-expected.txt: Added.
- fast/css-intrinsic-dimensions/width-dynamic-property-value.html: Added.
- fast/css-intrinsic-dimensions/width-expected.html: Added.
- fast/css-intrinsic-dimensions/width-keyword-classes.css: Added.
(.min-content):
(.max-content):
(.fill-available):
(.fit-content):
(.max-width-min-content):
(.max-width-max-content):
(.max-width-fill-available):
(.max-width-fit-content):
(.min-width-min-content):
(.min-width-max-content):
(.min-width-fill-available):
(.min-width-fit-content):
- fast/css-intrinsic-dimensions/width-property-value-expected.txt: Added.
- fast/css-intrinsic-dimensions/width-property-value.html: Added.
- fast/css-intrinsic-dimensions/width.html: Added.
- 12:25 PM Changeset in webkit [120848] by
-
- 1 edit2 copies in branches/chromium/1180
Merge 120638 - REGRESSION(r118414): some pages on concursolutions.com doesn't show up
https://bugs.webkit.org/show_bug.cgi?id=89399
http://crbug.com/132097
Reviewed by Adam Barth.
Source/WebCore:
The bug was caused by insertAdjacentHTML's not passing context element to createFragmentForInnerOuterHTML.
Test: fast/html/adjacent-html-context-element.html
- html/HTMLElement.cpp:
(WebCore::HTMLElement::insertAdjacentHTML):
LayoutTests:
Add a regression test.
- fast/html/adjacent-html-context-element-expected.txt: Added.
- fast/html/adjacent-html-context-element.html: Added.
Review URL: https://chromiumcodereview.appspot.com/10577039
- 12:17 PM Changeset in webkit [120847] by
-
- 12 edits6 adds in trunk
[chromium] webkit-backface-visibility doesn't work with video
https://bugs.webkit.org/show_bug.cgi?id=88908
When determining a contents layer's backface culling, use the parent
layer's transform and backface-visibility properties. Track which
layers need this special treatment with useParentBackfaceVisibility
and setUseParentBackfaceVisibility functions in WebCore::LayerChromium,
WebKit::WebLayer, and WebCore::CCLayerImpl.
Patch by Christopher Cameron <ccameron@chromium.org> on 2012-06-20
Reviewed by Adrienne Walker.
Source/Platform:
- chromium/public/WebLayer.h:
(WebLayer):
Add an accessor to specify that a layer should use its parent's
backface culling behavior.
Source/WebCore:
Tests: compositing/backface-visibility/backface-visibility-image.html
compositing/backface-visibility/backface-visibility-webgl.html
- platform/graphics/chromium/GraphicsLayerChromium.cpp:
(WebCore::GraphicsLayerChromium::setupContentsLayer):
When a contents layer is added, tag it as inheriting its backface
culling from its parent.
- platform/graphics/chromium/LayerChromium.cpp:
(WebCore::LayerChromium::LayerChromium):
Initialize new m_useParentBackfaceVisibility member variable.
(WebCore::LayerChromium::pushPropertiesTo):
Propagate m_useParentBackfaceVisibility to CCLayerImpl.
- platform/graphics/chromium/LayerChromium.h:
(WebCore::LayerChromium::setUseParentBackfaceVisibility):
(WebCore::LayerChromium::useParentBackfaceVisibility):
(LayerChromium):
Add m_useParentBackfaceVisibility member variable and modify
and query accessors.
- platform/graphics/chromium/cc/CCLayerImpl.cpp:
(WebCore::CCLayerImpl::CCLayerImpl):
Initialize new m_useParentBackfaceVisibility member variable.
- platform/graphics/chromium/cc/CCLayerImpl.h:
(WebCore::CCLayerImpl::setUseParentBackfaceVisibility):
(WebCore::CCLayerImpl::useParentBackfaceVisibility):
(CCLayerImpl):
Add m_useParentBackfaceVisibility member variable and modify
and query accessors.
- platform/graphics/chromium/cc/CCLayerTreeHostCommon.cpp:
(WebCore::layerShouldBeSkipped):
When examining a layer, if the layer has
useParentBackfaceVisibility set then use the layer's parent
layer to determine backface culling.
Source/WebKit/chromium:
- src/WebLayer.cpp:
(WebKit::WebLayer::setUseParentBackfaceVisibility):
Add an accessor to specify that a layer should use its parent's
backface culling behavior.
LayoutTests:
- compositing/backface-visibility/backface-visibility-image-expected.png: Added.
- compositing/backface-visibility/backface-visibility-image-expected.txt: Added.
- compositing/backface-visibility/backface-visibility-image.html: Added.
- compositing/backface-visibility/backface-visibility-webgl-expected.png: Added.
- compositing/backface-visibility/backface-visibility-webgl-expected.txt: Added.
- compositing/backface-visibility/backface-visibility-webgl.html: Added.
- 12:00 PM Changeset in webkit [120846] by
-
- 9 edits1 add in trunk/Tools
new-run-webkit-tests should spin-up enough httpd processes to avoid timeouts
https://bugs.webkit.org/show_bug.cgi?id=88134
Reviewed by Tony Chang.
Change NRWT to spin up 2*min(child_processes, locked_shards)
http servers by default so that we are less likely to get a
bunch of http timeouts at the beginning of a test run.
Note that I had to tweak executive_mock to support mocked stderr
because the apache_http_server code reads stderr when starting a
process to ensure it started okay.
- Scripts/webkitpy/common/system/executive_mock.py:
(MockProcess.init):
- Scripts/webkitpy/common/system/outputcapture.py:
- Scripts/webkitpy/layout_tests/controllers/manager.py:
(Manager._run_tests):
(Manager.start_servers_with_lock):
- Scripts/webkitpy/layout_tests/port/base.py:
(Port.to.start_http_server):
- Scripts/webkitpy/layout_tests/port/test.py:
(TestPort.start_http_server):
(TestPort._path_to_apache):
(TestPort):
(TestPort._path_to_apache_config_file):
- Scripts/webkitpy/layout_tests/servers/apache_http_server.py:
(LayoutTestApacheHttpd.init):
- Scripts/webkitpy/layout_tests/servers/apache_http_server_unittest.py: Added
(LayoutTestApacheHttpd.init):
- Scripts/webkitpy/layout_tests/servers/http_server.py:
(Lighttpd.init):
- Scripts/webkitpy/layout_tests/servers/http_server_base.py:
(HttpServerBase.init):
- 11:42 AM Changeset in webkit [120845] by
-
- 3 edits2 adds in trunk
Source/WebCore: Don't re-enter CachedResource::removeClient() if an XHR
is canceled and restarted multiple times.
https://bugs.webkit.org/show_bug.cgi?id=89378
Reviewed by Eric Seidel.
Test: http/tests/xmlhttprequest/reentrant-cancel.html
- loader/DocumentThreadableLoader.cpp:
(WebCore::DocumentThreadableLoader::cancel):
(WebCore::DocumentThreadableLoader::clearResource): Save off a copy of m_resource
then clear it, so we don't call clearResource() multiple times for the same resource.
LayoutTests: Test for https://bugs.webkit.org/show_bug.cgi?id=89378.
Reviewed by Eric Seidel.
- http/tests/xmlhttprequest/reentrant-cancel-expected.txt: Added.
- http/tests/xmlhttprequest/reentrant-cancel.html: Added.
- 11:40 AM Changeset in webkit [120844] by
-
- 3 edits10 adds in trunk
Negative margin block doesn't properly clear a float enclosed by a previous sibling
https://bugs.webkit.org/show_bug.cgi?id=10900
Reviewed by Eric Seidel.
Source/WebCore:
Tests: fast/block/float/previous-sibling-abspos-001.html
fast/block/float/previous-sibling-abspos-002.html
fast/block/float/previous-sibling-float-001.html
fast/block/float/previous-sibling-float-002.html
fast/css/clear-float-sibling.html
Parent blocks keep a list of child floats that extend out of the parent block and
by implication overhang into the parent's siblings. But this doesn't work if the
sibling has collapsing margins - it will not find the float in the previous block's
list so will ignore the float and fail to clear it.
RenderBlock:collapseMargins() needs to check if a child's collapsing margin has
reduced the height of the parent up past the bottom of its previous sibling's lowest float
and add the now overhanging float to the parent's float list if appropriate. No need to do
this if the previous sibling is a float or is positioned - the child will clear/avoid it anyway
and attempting to avoid floated children of floats causes incorrect layout.
- rendering/RenderBlock.cpp:
(WebCore::RenderBlock::collapseMargins):
LayoutTests:
- fast/block/float/previous-sibling-abspos-001-expected.html: Added.
- fast/block/float/previous-sibling-abspos-001.html: Added.
- fast/block/float/previous-sibling-abspos-002-expected.html: Added.
- fast/block/float/previous-sibling-abspos-002.html: Added.
- fast/block/float/previous-sibling-float-001-expected.html: Added.
- fast/block/float/previous-sibling-float-001.html: Added.
- fast/block/float/previous-sibling-float-002-expected.html: Added.
- fast/block/float/previous-sibling-float-002.html: Added.
- fast/css/clear-float-sibling-expected.html: Added.
- fast/css/clear-float-sibling.html: Added.
- 11:36 AM Changeset in webkit [120843] by
-
- 1 edit3 copies in branches/chromium/1180
Merge 120750 - Fix scrollbar layers being misplaced with a clipped owner layer
https://bugs.webkit.org/show_bug.cgi?id=89486
Reviewed by Simon Fraser.
Source/WebCore:
Scrollbar layers are positioned relative to their parent layer. If
that parent gets clipped, then that positioning becomes incorrect and
scrollbars get positioned relative to the clipped bounds instead of
the full bounds. Fix by not clipping in this case.
Test: compositing/overflow/scrollbars-with-clipped-owner.html
- rendering/RenderLayerBacking.cpp:
(WebCore::RenderLayerBacking::shouldClipCompositedBounds):
LayoutTests:
- compositing/overflow/scrollbars-with-clipped-owner-expected.png: Added.
- compositing/overflow/scrollbars-with-clipped-owner-expected.txt: Added.
- compositing/overflow/scrollbars-with-clipped-owner.html: Added.
TBR=enne@google.com
Review URL: https://chromiumcodereview.appspot.com/10536221
- 11:28 AM Changeset in webkit [120842] by
-
- 17 edits2 copies in trunk/Source/WebCore
Web Inspector: Allow module injections into the InjectedScript
https://bugs.webkit.org/show_bug.cgi?id=89530
Add a new InjectedScriptModule abstract class that inherits InjectedScriptBase
to reuse implementation, and which javascript code is injected via
InjectedScript.
Patch by Andrey Adaikin <aandrey@chromium.org> on 2012-06-20
Reviewed by Pavel Feldman.
- CMakeLists.txt:
- GNUmakefile.list.am:
- Target.pri:
- WebCore.gypi:
- WebCore.vcproj/WebCore.vcproj:
- WebCore.xcodeproj/project.pbxproj:
- bindings/js/ScriptObject.cpp:
(WebCore::ScriptObject::ScriptObject):
(WebCore):
- bindings/js/ScriptObject.h:
(ScriptObject):
(WebCore::ScriptObject::ScriptObject):
- bindings/v8/ScriptObject.cpp:
(WebCore::ScriptObject::ScriptObject):
(WebCore):
- bindings/v8/ScriptObject.h:
(ScriptObject):
(WebCore::ScriptObject::~ScriptObject):
- inspector/InjectedScript.cpp:
- inspector/InjectedScript.h:
(WebCore):
(WebCore::InjectedScript::name):
(InjectedScript):
- inspector/InjectedScriptBase.cpp:
(WebCore::InjectedScriptBase::callFunctionWithEvalEnabled):
- inspector/InjectedScriptBase.h:
(WebCore):
(InjectedScriptBase):
- inspector/InjectedScriptManager.h:
(InjectedScriptManager):
(WebCore::InjectedScriptManager::inspectedStateAccessCheck):
- inspector/InjectedScriptModule.cpp: Copied from Source/WebCore/inspector/InjectedScriptBase.h.
(WebCore):
(WebCore::InjectedScriptModule::InjectedScriptModule):
(WebCore::InjectedScriptModule::ensureInjected):
- inspector/InjectedScriptModule.h: Copied from Source/WebCore/bindings/js/ScriptObject.h.
(WebCore):
(InjectedScriptModule):
- inspector/InjectedScriptSource.js:
(.):
- 11:24 AM Changeset in webkit [120841] by
-
- 3 edits in trunk/Source/JavaScriptCore
Made the incremental sweeper more aggressive
https://bugs.webkit.org/show_bug.cgi?id=89527
Reviewed by Oliver Hunt.
This is a pre-requisite to getting rid of "berzerker GC" because we need
the sweeper to reclaim memory in a timely fashion, or we'll see a memory
footprint regression.
- heap/IncrementalSweeper.h:
- heap/IncrementalSweeper.cpp:
(JSC::IncrementalSweeper::scheduleTimer): Since the time slice is predictable,
no need to use a data member to record it.
(JSC::IncrementalSweeper::doSweep): Sweep as many blocks as we can in a
small time slice. This is better than sweeping only one block per timer
fire because that strategy has a heavy timer overhead, and artificially
delays memory reclamation.
- 11:22 AM Changeset in webkit [120840] by
-
- 4 edits in branches/chromium/1180/Source/WebCore/html/canvas
Merge 120636 - Fix framebuffer-object-attachment.html failures
https://bugs.webkit.org/show_bug.cgi?id=89387
Reviewed by Kenneth Russell.
- html/canvas/WebGLFramebuffer.cpp: fix detachment behavior with depth/stencil/depth_stencil conflicts
(WebCore::WebGLFramebuffer::setAttachmentForBoundFramebuffer):
(WebCore::WebGLFramebuffer::removeAttachmentFromBoundFramebuffer):
- html/canvas/WebGLFramebuffer.h:
(WebGLFramebuffer):
- html/canvas/WebGLRenderingContext.cpp:
(WebCore):
(WebCore::WebGLRenderingContext::framebufferRenderbuffer): move logic to WebGLFramebuffer
(WebCore::WebGLRenderingContext::framebufferTexture2D): Ditto.
(WebCore::WebGLRenderingContext::getParameter): Correct the wrong assumption that it's always checking the drawingbuffer's DEPTH_BITS/STENCIL_BITS
- html/canvas/WebGLRenderingContext.h:
(WebGLRenderingContext):
TBR=zmo@google.com
Review URL: https://chromiumcodereview.appspot.com/10584035
- 11:18 AM Changeset in webkit [120839] by
-
- 2 edits in trunk/Source/WebKit/blackberry
[BlackBerry] Enable setAcceleratedCompositingForFixedPositionEnabled
https://bugs.webkit.org/show_bug.cgi?id=89575
Patch by Konrad Piascik <kpiascik@rim.com> on 2012-06-20
Reviewed by Antonio Gomes.
Enable the flag so that we get position:fixed elemetns to be rendered
using accelerated compositing.
- Api/WebPage.cpp:
(BlackBerry::WebKit::WebPagePrivate::init):
- 11:11 AM Changeset in webkit [120838] by
-
- 6 edits in trunk/Source/WebCore
Web Inspector: Timeline label bar jiggles when scrolling
https://bugs.webkit.org/show_bug.cgi?id=89579
Reviewed by Vsevolod Vlasov.
Adding timeline header into the timeline panel and settings its position
to absolute. Drive-by fix that updates rulers on sidebar resize.
- inspector/front-end/TimelineGrid.js:
(WebInspector.TimelineGrid):
(WebInspector.TimelineGrid.prototype.get gridHeaderElement):
(WebInspector.TimelineGrid.prototype.addEventDividers):
(WebInspector.TimelineGrid.prototype.setScrollAndDividerTop):
- inspector/front-end/TimelineOverviewPane.js:
(WebInspector.TimelineOverviewPane.prototype.sidebarResized):
- inspector/front-end/TimelinePanel.js:
(WebInspector.TimelinePanel):
(WebInspector.TimelinePanel.prototype.sidebarResized):
(WebInspector.TimelinePanel.prototype.onResize):
- inspector/front-end/timelinePanel.css:
(#timeline-grid-header):
- 10:58 AM Changeset in webkit [120837] by
-
- 9 edits in trunk/Source
[chromium] Make sure that render surfaces are not pixel doubled with a device scale factor of 2
https://bugs.webkit.org/show_bug.cgi?id=86882
Patch by Ian Vollick <vollick@chromium.org> on 2012-06-20
Reviewed by Adrienne Walker.
To ensure that render surfaces are not pixel doubled when device scale factor is
two, the render surface's owning layer's draw transform is scaled by the
contents scale (in the same way that the parent matrix is scaled by the device
scale factor). The transformedLayerRect's dimensions also need to be in pixel
space. The surface origin transform should not scale, but needs to offset the
correct number of pixels, and the replica transforms need to be modified to
account for the scaling.
Source/WebCore:
To accomplish this, CCLayerImpl's needed to be made aware of contentsScale.
Thanks to Adrienne Walker for the computation of the replica draw transform.
Unit test: CCLayerTreeHostCommonTest.verifyRenderSurfaceTranformsInHighDPI
- platform/graphics/chromium/LayerChromium.cpp:
(WebCore::LayerChromium::pushPropertiesTo):
- platform/graphics/chromium/LayerChromium.h:
(LayerChromium):
- platform/graphics/chromium/cc/CCLayerImpl.cpp:
(WebCore::CCLayerImpl::CCLayerImpl):
- platform/graphics/chromium/cc/CCLayerImpl.h:
(WebCore::CCLayerImpl::setContentsScale):
(WebCore::CCLayerImpl::contentsScale):
(CCLayerImpl):
- platform/graphics/chromium/cc/CCLayerTreeHostCommon.cpp:
(WebCore::calculateDrawTransformsInternal):
- platform/graphics/chromium/cc/CCScrollbarLayerImpl.cpp:
(WebCore::CCScrollbarLayerImpl::CCScrollbar::totalSize):
Source/WebKit/chromium:
- tests/CCLayerTreeHostCommonTest.cpp:
- 10:51 AM Changeset in webkit [120836] by
-
- 1 edit in trunk/Source/JavaScriptCore/ChangeLog
Fix JSC ChangeLog, forgot to add a comment to the ChangeLog before committing.
- 10:49 AM Changeset in webkit [120835] by
-
- 3 edits2 adds in trunk
[CSS3 Backgrounds and Borders] Implement box-decoration-break rendering.
https://bugs.webkit.org/show_bug.cgi?id=88228
Reviewed by Eric Seidel.
Source/WebCore:
Implement the new CSS property box-decoration-break. It modifies
where we decide whether the borders needs to be sliced or not by
checking if the box-decoration-break is set to clone. If it's the case
then we need to explicitely redraw all edges.
Test: fast/box-decoration-break/box-decoration-break-rendering.html
- rendering/InlineFlowBox.cpp:
(WebCore::InlineFlowBox::determineSpacingForFlowBoxes):
(WebCore::InlineFlowBox::paintFillLayer):
LayoutTests:
Add new tests to cover the feature.
- fast/box-decoration-break/box-decoration-break-rendering-expected.html: Added.
- fast/box-decoration-break/box-decoration-break-rendering.html: Added.
- 10:48 AM Changeset in webkit [120834] by
-
- 20 edits2 adds in trunk/Source/JavaScriptCore
DFG should be able to print disassembly interleaved with the IR
https://bugs.webkit.org/show_bug.cgi?id=89551
Reviewed by Geoffrey Garen.
- CMakeLists.txt:
- GNUmakefile.list.am:
- JavaScriptCore.xcodeproj/project.pbxproj:
- Target.pri:
- assembler/ARMv7Assembler.h:
(JSC::ARMv7Assembler::labelIgnoringWatchpoints):
(ARMv7Assembler):
- assembler/AbstractMacroAssembler.h:
(AbstractMacroAssembler):
(JSC::AbstractMacroAssembler::labelIgnoringWatchpoints):
- assembler/X86Assembler.h:
(X86Assembler):
(JSC::X86Assembler::labelIgnoringWatchpoints):
- dfg/DFGCommon.h:
(JSC::DFG::shouldShowDisassembly):
(DFG):
- dfg/DFGDisassembler.cpp: Added.
(DFG):
(JSC::DFG::Disassembler::Disassembler):
(JSC::DFG::Disassembler::dump):
(JSC::DFG::Disassembler::dumpDisassembly):
- dfg/DFGDisassembler.h: Added.
(DFG):
(Disassembler):
(JSC::DFG::Disassembler::setStartOfCode):
(JSC::DFG::Disassembler::setForBlock):
(JSC::DFG::Disassembler::setForNode):
(JSC::DFG::Disassembler::setEndOfMainPath):
(JSC::DFG::Disassembler::setEndOfCode):
- dfg/DFGDriver.cpp:
(JSC::DFG::compile):
- dfg/DFGGraph.cpp:
(JSC::DFG::Graph::dumpCodeOrigin):
(JSC::DFG::Graph::amountOfNodeWhiteSpace):
(DFG):
(JSC::DFG::Graph::printNodeWhiteSpace):
(JSC::DFG::Graph::dump):
(JSC::DFG::Graph::dumpBlockHeader):
- dfg/DFGGraph.h:
- dfg/DFGJITCompiler.cpp:
(JSC::DFG::JITCompiler::JITCompiler):
(DFG):
(JSC::DFG::JITCompiler::compile):
(JSC::DFG::JITCompiler::compileFunction):
- dfg/DFGJITCompiler.h:
(JITCompiler):
(JSC::DFG::JITCompiler::setStartOfCode):
(JSC::DFG::JITCompiler::setForBlock):
(JSC::DFG::JITCompiler::setForNode):
(JSC::DFG::JITCompiler::setEndOfMainPath):
(JSC::DFG::JITCompiler::setEndOfCode):
- dfg/DFGNode.h:
(Node):
(JSC::DFG::Node::willHaveCodeGen):
- dfg/DFGNodeFlags.cpp:
(JSC::DFG::nodeFlagsAsString):
- dfg/DFGSpeculativeJIT.cpp:
(JSC::DFG::SpeculativeJIT::compile):
- dfg/DFGSpeculativeJIT.h:
(SpeculativeJIT):
- runtime/Options.cpp:
(Options):
(JSC::Options::initializeOptions):
- runtime/Options.h:
(Options):
- 10:27 AM Changeset in webkit [120833] by
-
- 2 edits in trunk/Source/WebKit/chromium
Unreviewed build fix.
- tests/IDBDatabaseBackendTest.cpp:
(WebCore::MockIDBCallbacks::~MockIDBCallbacks):
(WebCore::FakeIDBDatabaseCallbacks::~FakeIDBDatabaseCallbacks):
- 10:24 AM Changeset in webkit [120832] by
-
- 13 edits in trunk/Source
Use IntSize in RenderLayer to represent scroll offsets
https://bugs.webkit.org/show_bug.cgi?id=89154
Reviewed by Eric Seidel.
Source/WebCore:
Refactoring, covered by existing tests.
- accessibility/AccessibilityRenderObject.cpp:
(WebCore::AccessibilityRenderObject::scrollTo):
- html/TextFieldInputType.cpp:
(WebCore::TextFieldInputType::forwardEvent):
- page/SpatialNavigation.cpp:
(WebCore::scrollInDirection):
- rendering/RenderMarquee.cpp:
(WebCore::RenderMarquee::start):
Updated those call-sites to use the IntSize variant of the scrolling function.
- rendering/RenderLayer.cpp:
(WebCore::RenderLayer::panScrollFromPoint):
Updated the signature to use IntPoint to avoid an extra round-trip to LayoutPoint
(furthermore touch points should be in device pixels).
(WebCore::RenderLayer::clampScrollOffset):
Added this function that clamps a scroll offset to the box's size. This enables more
code sharing between the different functions. Also removed a NULL-check that shouldn't
be needed: scrolling is a RenderBox concept so any callers should ensure that we have
a RenderBox or it makes no sense to try to scroll.
(WebCore::RenderLayer::updateLayerPosition):
(WebCore::adjustedScrollDelta):
(WebCore::RenderLayer::scrollByRecursively):
(WebCore::RenderLayer::scrollToOffset):
(WebCore::RenderLayer::scrollRectToVisible):
(WebCore::RenderLayer::updateScrollInfoAfterLayout):
Updated to do IntSize arithmetic. Reuse clampScrollOffset when applicable.
- rendering/RenderLayer.h:
(WebCore::RenderLayer::scrollToXOffset):
(WebCore::RenderLayer::scrollToYOffset):
Updated the functions to take IntSize as much as possible.
Source/WebKit/blackberry:
- Api/WebPage.cpp:
(BlackBerry::WebKit::WebPagePrivate::scrollRenderer):
Updated to pass an IntSize to scrollToOffset.
Source/WebKit/win:
- WebView.cpp:
(WebView::gesture):
Updated to pass an IntSize to scrollByRecursively.
Source/WebKit2:
- WebProcess/WebPage/win/WebPageWin.cpp:
(WebKit::WebPage::gestureDidScroll):
Updated to pass an IntSize to scrollByRecursively.
- 10:22 AM Changeset in webkit [120831] by
-
- 9 edits in trunk/Source/WebKit2
[WK2] Implement Web Intent delivery
https://bugs.webkit.org/show_bug.cgi?id=88989
Patch by Christophe Dumez <Christophe Dumez> on 2012-06-20
Reviewed by Anders Carlsson.
Add a deliverIntent() method to the WebFrame so that
Web intents can be delivered once matched to a
specific service.
- UIProcess/WebFrameProxy.cpp:
(WebKit):
(WebKit::WebFrameProxy::deliverIntent):
- UIProcess/WebFrameProxy.h:
(WebKit):
(WebFrameProxy):
- UIProcess/WebIntentData.h:
(WebKit::WebIntentData::store):
- WebProcess/WebPage/WebFrame.cpp:
(WebKit):
(WebKit::WebFrame::deliverIntent):
- WebProcess/WebPage/WebFrame.h:
(WebKit):
(WebFrame):
- WebProcess/WebPage/WebPage.cpp:
(WebKit):
(WebKit::WebPage::deliverIntentToFrame):
- WebProcess/WebPage/WebPage.h:
(WebKit):
(WebPage):
- WebProcess/WebPage/WebPage.messages.in:
- 10:14 AM Changeset in webkit [120830] by
-
- 7 edits in branches/safari-536-branch/Source/WebKit2
Merged r120376 -> <rdar://problem/11652545>
- 9:33 AM Changeset in webkit [120829] by
-
- 3 edits in trunk/Source/WebCore
Web Inspector: mark used/allocated JS heap on the native memory bar chart.
https://bugs.webkit.org/show_bug.cgi?id=89476
Patch by Alexei Filippov <alexeif@chromium.org> on 2012-06-20
Reviewed by Yury Semikhatsky.
- inspector/front-end/NativeMemorySnapshotView.js:
(WebInspector.NativeMemoryBarChart):
(WebInspector.NativeMemoryBarChart.prototype._updateView):
- inspector/front-end/nativeMemoryProfiler.css:
(.memory-bar-chart-bar):
(.memory-bar-chart-unused):
(.memory-bar-chart-percent):
- 9:17 AM Changeset in webkit [120828] by
-
- 7 edits in trunk/Source
[Chromium] IndexedDB: Don't close database if pending connections are in flight
https://bugs.webkit.org/show_bug.cgi?id=89512
Source/WebCore:
Add a counter tracking connections between the two phases, which is used along with
the completed connection count to determine the total number of connections.
Reviewed by Tony Chang.
Test: webkit_unit_tests --gtest_filter='IDBDatabaseBackendTest.ConnectionLifecycle'
- Modules/indexeddb/IDBDatabaseBackendImpl.cpp:
(WebCore::IDBDatabaseBackendImpl::IDBDatabaseBackendImpl):
(WebCore::IDBDatabaseBackendImpl::setVersion):
(WebCore::IDBDatabaseBackendImpl::connectionCount):
(WebCore):
(WebCore::IDBDatabaseBackendImpl::processPendingCalls):
(WebCore::IDBDatabaseBackendImpl::transaction):
(WebCore::IDBDatabaseBackendImpl::registerFrontendCallbacks):
(WebCore::IDBDatabaseBackendImpl::openConnection):
(WebCore::IDBDatabaseBackendImpl::close):
- Modules/indexeddb/IDBDatabaseBackendImpl.h:
(IDBDatabaseBackendImpl):
- Modules/indexeddb/IDBFactoryBackendImpl.cpp:
(WebCore::IDBFactoryBackendImpl::openInternal):
- inspector/InspectorIndexedDBAgent.cpp:
(WebCore):
Source/WebKit/chromium:
Reviewed by Tony Chang.
- tests/IDBDatabaseBackendTest.cpp:
(MockIDBCallbacks):
(WebCore::MockIDBCallbacks::create):
(WebCore::MockIDBCallbacks::MockIDBCallbacks):
(WebCore):
(FakeIDBDatabaseCallbacks):
(WebCore::FakeIDBDatabaseCallbacks::create):
(WebCore::FakeIDBDatabaseCallbacks::FakeIDBDatabaseCallbacks):
(WebCore::TEST):
- 9:15 AM Changeset in webkit [120827] by
-
- 2 edits in trunk/Source/WebCore
Web Inspector: duplicating selected text when Enter key is pressed
https://bugs.webkit.org/show_bug.cgi?id=89559
Patch by Sergey Rogulenko <rogulenko@google.com> on 2012-06-20
Reviewed by Pavel Feldman.
- inspector/front-end/TextViewer.js:
(WebInspector.TextEditorMainPanel.prototype.handleEnterKey):
The returned result of method normalize() should be used.
- 9:07 AM Changeset in webkit [120826] by
-
- 2 edits in trunk/Source/WebCore
[Cairo] Fix memory leak in GraphicsContext3DCairo.cpp
https://bugs.webkit.org/show_bug.cgi?id=89561
Patch by Sudarsana Nagineni <sudarsana.nagineni@linux.intel.com> on 2012-06-20
Reviewed by Martin Robinson.
Fix a memory leak in GraphicsContext3D::getImageData().
- platform/graphics/cairo/GraphicsContext3DCairo.cpp:
(WebCore::GraphicsContext3D::getImageData): asNewNativeImage() returns a
caller-owned pointer to the native image data. So, using OwnPtr to manage
memory automatically.
- 9:06 AM Changeset in webkit [120825] by
-
- 2 edits in trunk/Tools
Unreviewed: Back out accidentally checked in debug print which broke a test
- Scripts/webkitpy/performance_tests/perftest.py:
(PerfTest.parse_output):
- 9:01 AM WebKitGTK/WebKit2Roadmap edited by
- (diff)
- 8:41 AM Changeset in webkit [120824] by
-
- 60 edits in trunk/Source/WebCore
Use HitTestPoint instead of LayoutPoint for nodeAtPoint.
https://bugs.webkit.org/show_bug.cgi?id=89448
Patch by Allan Sandfeld Jensen <allan.jensen@nokia.com> on 2012-06-20
Reviewed by Eric Seidel.
This patch updates the API of nodeAtPoint without changing any
functionality. This is largest change necessary to support
transforms in area-based hit-testing, but is only boilerplate.
No change in functionality. No new tests.
- rendering/EllipsisBox.cpp:
(WebCore::EllipsisBox::nodeAtPoint):
- rendering/EllipsisBox.h:
(EllipsisBox):
- rendering/HitTestResult.cpp:
(WebCore::HitTestResult::addNodeToRectBasedTestResult):
- rendering/HitTestResult.h:
(WebCore::HitTestPoint::intersects):
(HitTestPoint):
(HitTestResult):
- rendering/InlineBox.cpp:
(WebCore::InlineBox::nodeAtPoint):
- rendering/InlineBox.h:
(InlineBox):
- rendering/InlineFlowBox.cpp:
(WebCore::InlineFlowBox::nodeAtPoint):
- rendering/InlineFlowBox.h:
(InlineFlowBox):
- rendering/InlineTextBox.cpp:
(WebCore::InlineTextBox::nodeAtPoint):
- rendering/InlineTextBox.h:
(InlineTextBox):
- rendering/RenderBlock.cpp:
(WebCore::RenderBlock::nodeAtPoint):
(WebCore::RenderBlock::hitTestFloats):
(WebCore::RenderBlock::hitTestColumns):
(WebCore::RenderBlock::hitTestContents):
- rendering/RenderBlock.h:
(RenderBlock):
- rendering/RenderBox.cpp:
(WebCore::RenderBox::nodeAtPoint):
- rendering/RenderBox.h:
(RenderBox):
- rendering/RenderEmbeddedObject.cpp:
(WebCore::RenderEmbeddedObject::nodeAtPoint):
- rendering/RenderEmbeddedObject.h:
(RenderEmbeddedObject):
- rendering/RenderFlowThread.cpp:
(WebCore::RenderFlowThread::hitTestRegion):
- rendering/RenderFlowThread.h:
- rendering/RenderFrameSet.cpp:
(WebCore::RenderFrameSet::nodeAtPoint):
- rendering/RenderFrameSet.h:
(RenderFrameSet):
- rendering/RenderImage.cpp:
(WebCore::RenderImage::nodeAtPoint):
- rendering/RenderImage.h:
(RenderImage):
- rendering/RenderInline.cpp:
(WebCore::RenderInline::nodeAtPoint):
- rendering/RenderInline.h:
(RenderInline):
- rendering/RenderLayer.cpp:
(WebCore::ClipRect::intersects):
(WebCore::RenderLayer::hitTest):
(WebCore::RenderLayer::createLocalTransformState):
(WebCore::RenderLayer::hitTestLayer):
(WebCore::RenderLayer::hitTestContents):
(WebCore::RenderLayer::hitTestList):
(WebCore::RenderLayer::hitTestPaginatedChildLayer):
(WebCore::RenderLayer::hitTestChildLayerColumns):
- rendering/RenderLayer.h:
(ClipRect):
(RenderLayer):
- rendering/RenderLineBoxList.cpp:
(WebCore::RenderLineBoxList::hitTest):
- rendering/RenderLineBoxList.h:
(RenderLineBoxList):
- rendering/RenderListBox.cpp:
(WebCore::RenderListBox::nodeAtPoint):
- rendering/RenderListBox.h:
(RenderListBox):
- rendering/RenderObject.cpp:
(WebCore::RenderObject::hitTest):
(WebCore::RenderObject::nodeAtPoint):
- rendering/RenderObject.h:
(RenderObject):
- rendering/RenderRegion.cpp:
(WebCore::RenderRegion::nodeAtPoint):
- rendering/RenderRegion.h:
(RenderRegion):
- rendering/RenderTable.cpp:
(WebCore::RenderTable::nodeAtPoint):
- rendering/RenderTable.h:
(RenderTable):
- rendering/RenderTableRow.cpp:
(WebCore::RenderTableRow::nodeAtPoint):
- rendering/RenderTableRow.h:
(RenderTableRow):
- rendering/RenderTableSection.cpp:
(WebCore::RenderTableSection::nodeAtPoint):
- rendering/RenderTableSection.h:
(RenderTableSection):
- rendering/RenderText.h:
- rendering/RenderTextControlMultiLine.cpp:
(WebCore::RenderTextControlMultiLine::nodeAtPoint):
- rendering/RenderTextControlMultiLine.h:
(RenderTextControlMultiLine):
- rendering/RenderTextControlSingleLine.cpp:
(WebCore::RenderTextControlSingleLine::nodeAtPoint):
- rendering/RenderTextControlSingleLine.h:
(RenderTextControlSingleLine):
- rendering/RenderWidget.cpp:
(WebCore::RenderWidget::nodeAtPoint):
- rendering/RenderWidget.h:
(RenderWidget):
- rendering/RootInlineBox.cpp:
(WebCore::RootInlineBox::nodeAtPoint):
- rendering/RootInlineBox.h:
(RootInlineBox):
- rendering/svg/RenderSVGForeignObject.cpp:
(WebCore::RenderSVGForeignObject::nodeAtFloatPoint):
(WebCore::RenderSVGForeignObject::nodeAtPoint):
- rendering/svg/RenderSVGForeignObject.h:
(RenderSVGForeignObject):
- rendering/svg/RenderSVGModelObject.cpp:
(WebCore::RenderSVGModelObject::nodeAtPoint):
- rendering/svg/RenderSVGModelObject.h:
(RenderSVGModelObject):
- rendering/svg/RenderSVGRoot.cpp:
(WebCore::RenderSVGRoot::nodeAtPoint):
- rendering/svg/RenderSVGRoot.h:
(RenderSVGRoot):
- rendering/svg/RenderSVGText.cpp:
(WebCore::RenderSVGText::nodeAtFloatPoint):
(WebCore::RenderSVGText::nodeAtPoint):
- rendering/svg/RenderSVGText.h:
(RenderSVGText):
- rendering/svg/SVGInlineTextBox.cpp:
(WebCore::SVGInlineTextBox::nodeAtPoint):
- rendering/svg/SVGInlineTextBox.h:
(SVGInlineTextBox):
- 8:09 AM Changeset in webkit [120823] by
-
- 3 edits2 adds in trunk
Import themaninblue.com/experiment/AnimationBenchmark/ as performance tests
https://bugs.webkit.org/show_bug.cgi?id=78789
Reviewed by Ryosuke Niwa.
PerformanceTests:
Added PerfTestRunner fixture around it.
- Animation/balls.html: Added.
Tools:
Updated parser to include fps as a valid unit.
- Scripts/webkitpy/performance_tests/perftest.py:
(PerfTest):
(PerfTest.parse_output):
- 8:07 AM Changeset in webkit [120822] by
-
- 4 edits in trunk/Source/WebCore
Web Inspector: don't report context ids before DidCommitLoad
https://bugs.webkit.org/show_bug.cgi?id=89567
Reviewed by Pavel Feldman.
When inspector state is restored only report existing context ids
if "did commit load" even has already been dispatched.
- inspector/InspectorController.cpp:
(WebCore::InspectorController::InspectorController):
- inspector/PageRuntimeAgent.cpp:
(WebCore::PageRuntimeAgent::PageRuntimeAgent):
(WebCore::PageRuntimeAgent::restore):
- inspector/PageRuntimeAgent.h:
(WebCore):
(WebCore::PageRuntimeAgent::create):
(PageRuntimeAgent):
- 8:03 AM Changeset in webkit [120821] by
-
- 3 edits in trunk/Tools
webkit-patch should add reviewer if "Reviewed by NOBODY ..." is missing
https://bugs.webkit.org/show_bug.cgi?id=67935
Patch by Balazs Ankes <bank@inf.u-szeged.hu> on 2012-06-20
Reviewed by Ryosuke Niwa.
- Scripts/webkitpy/common/checkout/changelog.py:
(ChangeLog.set_reviewer):
- Scripts/webkitpy/common/checkout/changelog_unittest.py:
(test_set_reviewer):
(test_set_short_description_and_bug_url):
- 6:52 AM Changeset in webkit [120820] by
-
- 76 edits in trunk/Source
[Chromium] Remove redundant #includes in compositor
https://bugs.webkit.org/show_bug.cgi?id=89503
Patch by Zeev Lieber <zlieber@chromium.org> on 2012-06-20
Reviewed by Adrienne Walker.
Source/WebCore:
Removed some redundant #includes to clarify dependency
picture. Some 90 #includes removed, some 50 less disruptive
#includes added. This brings the number of class dependencies to
902 from 1383.
No new tests are required, as this is only affects compile-time behavior.
- platform/graphics/chromium/AnimationTranslationUtil.h:
(WebCore):
- platform/graphics/chromium/BitmapSkPictureCanvasLayerTextureUpdater.cpp:
- platform/graphics/chromium/BitmapSkPictureCanvasLayerTextureUpdater.h:
- platform/graphics/chromium/Canvas2DLayerBridge.cpp:
- platform/graphics/chromium/Canvas2DLayerBridge.h:
- platform/graphics/chromium/ContentLayerChromium.cpp:
- platform/graphics/chromium/ContentLayerChromium.h:
- platform/graphics/chromium/DrawingBufferChromium.cpp:
- platform/graphics/chromium/FrameBufferSkPictureCanvasLayerTextureUpdater.cpp:
- platform/graphics/chromium/GraphicsLayerChromium.h:
- platform/graphics/chromium/ImageLayerChromium.cpp:
- platform/graphics/chromium/LayerChromium.h:
- platform/graphics/chromium/LayerRendererChromium.cpp:
- platform/graphics/chromium/LayerRendererChromium.h:
(WebCore):
- platform/graphics/chromium/LayerTextureUpdater.h:
- platform/graphics/chromium/ManagedTexture.cpp:
- platform/graphics/chromium/ProgramBinding.h:
- platform/graphics/chromium/RenderSurfaceChromium.cpp:
- platform/graphics/chromium/RenderSurfaceChromium.h:
- platform/graphics/chromium/TextureCopier.cpp:
- platform/graphics/chromium/TextureLayerChromium.cpp:
- platform/graphics/chromium/TextureManager.cpp:
- platform/graphics/chromium/TiledLayerChromium.cpp:
- platform/graphics/chromium/TiledLayerChromium.h:
- platform/graphics/chromium/cc/CCActiveAnimation.h:
(WebCore):
- platform/graphics/chromium/cc/CCDelayBasedTimeSource.cpp:
- platform/graphics/chromium/cc/CCDrawQuad.cpp:
- platform/graphics/chromium/cc/CCFontAtlas.cpp:
- platform/graphics/chromium/cc/CCFrameRateController.cpp:
- platform/graphics/chromium/cc/CCFrameRateController.h:
(WebCore):
- platform/graphics/chromium/cc/CCHeadsUpDisplay.cpp:
- platform/graphics/chromium/cc/CCHeadsUpDisplay.h:
(WebCore):
- platform/graphics/chromium/cc/CCIOSurfaceLayerImpl.cpp:
- platform/graphics/chromium/cc/CCLayerAnimationController.h:
- platform/graphics/chromium/cc/CCLayerImpl.cpp:
- platform/graphics/chromium/cc/CCLayerImpl.h:
(WebCore):
- platform/graphics/chromium/cc/CCLayerTreeHost.cpp:
- platform/graphics/chromium/cc/CCLayerTreeHost.h:
(WebCore):
- platform/graphics/chromium/cc/CCLayerTreeHostCommon.cpp:
- platform/graphics/chromium/cc/CCLayerTreeHostImpl.cpp:
- platform/graphics/chromium/cc/CCLayerTreeHostImpl.h:
- platform/graphics/chromium/cc/CCOcclusionTracker.cpp:
- platform/graphics/chromium/cc/CCOcclusionTracker.h:
(WebCore):
- platform/graphics/chromium/cc/CCProxy.cpp:
- platform/graphics/chromium/cc/CCQuadCuller.cpp:
- platform/graphics/chromium/cc/CCRenderPass.h:
(WebCore):
- platform/graphics/chromium/cc/CCRenderSurface.cpp:
- platform/graphics/chromium/cc/CCRenderSurface.h:
- platform/graphics/chromium/cc/CCRenderer.h:
- platform/graphics/chromium/cc/CCScopedThreadProxy.h:
- platform/graphics/chromium/cc/CCScrollbarLayerImpl.cpp:
- platform/graphics/chromium/cc/CCScrollbarLayerImpl.h:
- platform/graphics/chromium/cc/CCSingleThreadProxy.cpp:
- platform/graphics/chromium/cc/CCSingleThreadProxy.h:
- platform/graphics/chromium/cc/CCSolidColorLayerImpl.cpp:
- platform/graphics/chromium/cc/CCTextureLayerImpl.cpp:
- platform/graphics/chromium/cc/CCThreadProxy.cpp:
- platform/graphics/chromium/cc/CCThreadProxy.h:
- platform/graphics/chromium/cc/CCTiledLayerImpl.cpp:
- platform/graphics/chromium/cc/CCTiledLayerImpl.h:
(WebCore):
- platform/graphics/chromium/cc/CCTimer.cpp:
- platform/graphics/chromium/cc/CCTimer.h:
(WebCore):
- platform/graphics/chromium/cc/CCVideoLayerImpl.cpp:
Source/WebKit/chromium:
Minor adjustments to #include statements to remove
dependencies. No new tests.
- src/WebCompositorImpl.cpp:
- src/WebLayerTreeView.cpp:
- tests/CCLayerTestCommon.cpp:
- tests/CCLayerTreeHostImplTest.cpp:
- tests/CCLayerTreeHostTest.cpp:
- tests/CCOcclusionTrackerTest.cpp:
- tests/CCQuadCullerTest.cpp:
- tests/CCTiledLayerImplTest.cpp:
- tests/ImageLayerChromiumTest.cpp:
- tests/LayerRendererChromiumTest.cpp:
- tests/TiledLayerChromiumTest.cpp:
- 6:32 AM Changeset in webkit [120819] by
-
- 5 edits in trunk/Source/WebCore
Web Inspector: Implement native memory bar diagram
https://bugs.webkit.org/show_bug.cgi?id=89106
Patch by Alexei Filippov <alexeif@chromium.org> on 2012-06-20
Reviewed by Yury Semikhatsky.
- inspector/front-end/NativeMemorySnapshotView.js:
(WebInspector.NativeMemoryBarChart):
(WebInspector.NativeMemoryBarChart.prototype._updateStats):
(WebInspector.NativeMemoryBarChart.prototype.willHide):
(WebInspector.NativeMemoryBarChart.prototype.wasShown):
(WebInspector.NativeMemoryBarChart.prototype._updateView):
- inspector/front-end/ProfileLauncherView.js:
(WebInspector.ProfileLauncherView):
- inspector/front-end/Settings.js:
(WebInspector.ExperimentsSettings):
- inspector/front-end/nativeMemoryProfiler.css:
(.memory-bar-chart-name):
(.memory-bar-chart-bar):
(.memory-bar-chart-size):
(.memory-bar-chart-total):
- 6:26 AM Changeset in webkit [120818] by
-
- 4 edits in trunk/Source/WebKit2
REGRESSION (120705) : LayerTreeHostQt asserts in debug.
https://bugs.webkit.org/show_bug.cgi?id=89487
Reviewed by Noam Rosenthal.
We try to call createHandle twice on the same handle when the
first matching atlas is full. This patch solved the problem by
moving the createHandle call inside UpdateAtlas and call createHandle
only when the atlas is not full. We can also remove the surface()
getter as it is not used anymore.
- WebProcess/WebPage/UpdateAtlas.cpp:
(WebKit::UpdateAtlas::beginPaintingOnAvailableBuffer):
- WebProcess/WebPage/UpdateAtlas.h:
(UpdateAtlas):
- WebProcess/WebPage/qt/LayerTreeHostQt.cpp:
(WebKit::LayerTreeHostQt::beginContentUpdate):
- 5:35 AM Changeset in webkit [120817] by
-
- 13 edits in trunk
Speech JavaScript API: add SpeechRecognition.maxAlternatives attribute
https://bugs.webkit.org/show_bug.cgi?id=89459
Reviewed by Eric Seidel.
Source/WebCore:
Add the SpeechRecognition.maxAlternatives attribute.
Test: fast/speech/scripted/basics.html:
- Modules/speech/SpeechRecognition.cpp:
(WebCore::SpeechRecognition::start):
(WebCore::SpeechRecognition::SpeechRecognition):
- Modules/speech/SpeechRecognition.h:
(WebCore::SpeechRecognition::maxAlternatives):
(WebCore::SpeechRecognition::setMaxAlternatives):
(SpeechRecognition):
- Modules/speech/SpeechRecognition.idl:
- Modules/speech/SpeechRecognitionClient.h:
(SpeechRecognitionClient):
- Modules/speech/SpeechRecognitionController.h:
(WebCore::SpeechRecognitionController::start):
(SpeechRecognitionController):
Source/WebKit/chromium:
Plumbing for the maxAlternatives attribute.
- public/WebSpeechRecognitionParams.h:
(WebKit::WebSpeechRecognitionParams::WebSpeechRecognitionParams):
(WebKit::WebSpeechRecognitionParams::maxAlternatives):
(WebSpeechRecognitionParams):
- src/SpeechRecognitionClientProxy.cpp:
(WebKit::SpeechRecognitionClientProxy::start):
- src/SpeechRecognitionClientProxy.h:
(SpeechRecognitionClientProxy):
LayoutTests:
Update the basic test to check for presence of the maxAlternatives attribute.
- fast/speech/scripted/basics-expected.txt:
- fast/speech/scripted/basics.html:
- 4:48 AM Changeset in webkit [120816] by
-
- 4 edits2 adds in trunk
applyAuthorStyles makes rules declared in all enclosing shadow dom subtrees applicable.
https://bugs.webkit.org/show_bug.cgi?id=89061
Patch by Takashi Sakamoto <tasak@google.com> on 2012-06-20
Reviewed by Dimitri Glazkov.
Source/WebCore:
Fixing apply-author-styles bug. If there exists some shadow subtree A
and A has apply-author-styles flag set, whether some rules declared in
a tree B is applicable in A or not depends on:
- the tree B encloses the tree A, and
- all trees which enclose A and are enclosed by B has apply-author-styles set.
c.f.
http://dvcs.w3.org/hg/webcomponents/raw-file/tip/spec/shadow/index.html#styles
However the second rule was not implemented.
Test: fast/css/style-scoped/style-scoped-apply-author-styles.html
- css/StyleResolver.cpp:
(WebCore::StyleResolver::setupScopeStack):
Modified to add a new member variable, authorStyleBoundsIndex to
ScopeStackFrame used by m_scopeStack. The index means: if shadow dom
subtree A and B has the same value and A encloses B, all shadow dom
subtrees which enclose B and are enclosed by B have
apply-author-styles set. And B also has apply-author-styles set.
This means, if some element X has authorStyleBoundsIndex N, all
rulesets whose scopes have authorStyleBoundsIndex N are applicable in
B's tree scope. If all shadow dom subtrees have apply-author-styles
unset, the index looks almost the same as treeScope(). In this case,
Rules in different tree scopes have different author style bounds
index.
(WebCore::StyleResolver::pushScope):
(WebCore::StyleResolver::popScope):
Added the code for updating m_scopeStackParentBoundsIndex. The member
variable keeps m_scopeStackParent's author style bounds index.
(WebCore::StyleResolver::matchScopedAuthorRules):
Modified to consider author style's bounds index. Firstly applying
scoped styles declared in document if apply-author-styles is set.
Next, applying scoped styles declared in shadow dom subtree with
considering author style bounds index, i.e. considering
apply-author-styles flag.
- css/StyleResolver.h:
(WebCore::StyleResolver::ScopeStackFrame::ScopeStackFrame):
(ScopeStackFrame):
Added m_authorStyleBoundsIndex to keep apply-author-styles flag
information. m_authorStyleBoundsIndex increments by 1 if shadow root is
given and it has apply-author-styles unset.
Now rules in the same authorStyleBoundsIndex should be applied.
(StyleResolver):
Added m_scopeStackParentBoundsIndex.
As m_scopeStackParentAuthorStyleBoundsIndex is a little long,
used m_scopeStackParentBoundsIndex.
LayoutTests:
- fast/css/style-scoped/style-scoped-apply-author-styles-expected.txt: Added.
- fast/css/style-scoped/style-scoped-apply-author-styles.html: Added.
- 4:07 AM Changeset in webkit [120815] by
-
- 2 edits in trunk/Tools
Unreviewed trivial permissions fix.
- qmake/mkspecs/features/features.prf: Don't mark this file as executable.
- 3:36 AM Changeset in webkit [120814] by
-
- 5 edits2 adds in trunk/Source/WebCore
Adds DeviceMotionClientMock
https://bugs.webkit.org/show_bug.cgi?id=89220
Patch by Amy Ousterhout <aousterh@chromium.org> on 2012-06-20
Reviewed by Steve Block.
No new tests because DeviceMotionClientMock is designed to enable future testing and cannot be tested in itself.
Adds DeviceMotionClientMock in WebCore to enable testing of DeviceMotion once it is fully implemented in WebKit.
This addition was originally included in https://bugs.webkit.org/show_bug.cgi?id=89197 but was split into a separate patch for easier review.
- CMakeLists.txt:
- GNUmakefile.list.am:
- Target.pri:
- WebCore.gypi:
- platform/mock/DeviceMotionClientMock.cpp: Added.
(WebCore):
(WebCore::DeviceMotionClientMock::DeviceMotionClientMock):
(WebCore::DeviceMotionClientMock::setController):
(WebCore::DeviceMotionClientMock::startUpdating):
(WebCore::DeviceMotionClientMock::stopUpdating):
(WebCore::DeviceMotionClientMock::setMotion):
(WebCore::DeviceMotionClientMock::timerFired):
- platform/mock/DeviceMotionClientMock.h: Added.
(WebCore):
(DeviceMotionClientMock):
- 2:58 AM Changeset in webkit [120813] by
-
- 2 edits in trunk/Source/WebKit/qt
[Qt] Unreviewed build fix
The platform plugin example uses QGridLayout and thus needs QT += widgets
- examples/platformplugin/platformplugin.pro:
- 1:47 AM Changeset in webkit [120812] by
-
- 2 edits in trunk/LayoutTests
Unreviewed GTK gardening. Removed some tests from TestExpectations
as they were passing in the tree bots.
- platform/gtk/TestExpectations:
- 12:54 AM Changeset in webkit [120811] by
-
- 6 edits1 add in trunk/LayoutTests
Unreviewed rebaselining.
- platform/chromium-linux/platform/chromium/virtual/gpu/fast/canvas/quadraticCurveTo-expected.png:
- platform/chromium-mac-leopard/platform/chromium/virtual/gpu/fast/canvas/quadraticCurveTo-expected.png: Added.
- platform/chromium-mac-snowleopard/platform/chromium/virtual/gpu/fast/canvas/quadraticCurveTo-expected.png:
- platform/chromium-mac/platform/chromium/virtual/gpu/fast/canvas/quadraticCurveTo-expected.png:
- platform/chromium-win/platform/chromium/virtual/gpu/fast/canvas/quadraticCurveTo-expected.png:
- platform/chromium/TestExpectations:
- 12:50 AM QtWebKitBuildBots edited by
- (diff)
- 12:00 AM Changeset in webkit [120810] by
-
- 2 edits in trunk/Source/WebKit/chromium
[chromium] Select the marker range when right-clicking on a marker.
https://bugs.webkit.org/show_bug.cgi?id=89331
Reviewed by Hajime Morita.
Chromium always selects only one word when right-clicking on a spelling marker.
This prevents selecting whole region specified by a marker if the marker
consists of two or more words. This change retrieves a range coverted by a
marker and select the range. This emulates the behavior of Mac Chromium and it
does not need any workarounds for Mac.
- src/ContextMenuClientImpl.cpp:
(WebKit::ContextMenuClientImpl::getCustomMenuFromDefaultItems):
Jun 19, 2012:
- 11:51 PM Changeset in webkit [120809] by
-
- 2 edits in trunk/Source/WebCore
Web Inspector: Clicking "Clear all profiles" while ongoing recording should reset the ProfileLauncherView
https://bugs.webkit.org/show_bug.cgi?id=89464
Patch by Vivek Galatage <vivekgalatage@gmail.com> on 2012-06-19
Reviewed by Yury Semikhatsky.
Resetting the ProfileLauncherView in order to reflect the correct states of buttons
No new tests as UI related minor fix.
- inspector/front-end/ProfilesPanel.js:
(WebInspector.ProfilesPanel.prototype._reset):
- 11:46 PM Changeset in webkit [120808] by
-
- 2 edits in trunk/LayoutTests
[Qt][Mac] Speculative skipping to try to paint the bot green.
- platform/qt-mac/Skipped:
- 11:30 PM Changeset in webkit [120807] by
-
- 4 edits in trunk
[EFL][Regression] Build break after r120786
https://bugs.webkit.org/show_bug.cgi?id=89549
Unreviewed build fix for Efl port.
Source/WebKit2:
- CMakeLists.txt: Added disassembler in includes.
Tools:
- DumpRenderTree/efl/CMakeLists.txt: Added disassembler in includes.
- 11:29 PM Changeset in webkit [120806] by
-
- 3 edits in trunk/Source/WebCore
Assertion for event's target is wrong.
https://bugs.webkit.org/show_bug.cgi?id=89479
Reviewed by Hajime Morita.
Fix an assertion to ensure that event's target is reachable from a current target.
No new tests. Fix an assert condition.
- dom/EventContext.cpp:
(WebCore::EventContext::EventContext):
- dom/EventContext.h:
(EventContext):
(WebCore::EventContext::setRelatedTarget):
(WebCore::EventContext::isUnreachableNode):
(WebCore):
(WebCore::EventContext::isReachable):
- 11:23 PM Changeset in webkit [120805] by
-
- 2 edits in trunk/Source/WebKit
[CMake] Unreviewed speculative buildfix after r120786.
- CMakeLists.txt:
- 11:03 PM Changeset in webkit [120804] by
-
- 3 edits in trunk/Source/WebCore
Refactoring: Geolocation::startRequest() should not create the GeoNotifiers
https://bugs.webkit.org/show_bug.cgi?id=89533
Patch by Benjamin Poulain <bpoulain@apple.com> on 2012-06-19
Reviewed by David Kilzer.
It was odd to create the GeoNotifiers in Geolocation::startRequest() and having
assertions from the call site.
This patch move GeoNotifiers initialization to the exposed methods and pass them to
startRequest() as needed.
This will also help when adding support for dynamic permission reset and suspend/resume.
- Modules/geolocation/Geolocation.cpp:
(WebCore::Geolocation::getCurrentPosition):
(WebCore::Geolocation::watchPosition):
(WebCore::Geolocation::startRequest):
- Modules/geolocation/Geolocation.h:
(Geolocation):
- 10:51 PM Changeset in webkit [120803] by
-
- 2 edits in trunk/Source/WebKit/gtk
Refine syncLayersTimeoutCallback for Accelerated Compositing.
https://bugs.webkit.org/show_bug.cgi?id=89538
Patch by Chang Wan Hong <jourmoon@company100.net> on 2012-06-19
Reviewed by Martin Robinson.
syncLayersTimeout reschedules the timer so that it can render each frame
every 1/60 seconds on animation. However, because it takes to time to execute
renderLayersToWindow, the timer is delayed. To fix this, we must reschedule
the timer before calling renderLayersToWindow.
- WebCoreSupport/AcceleratedCompositingContextGL.cpp:
(WebKit::AcceleratedCompositingContext::syncLayersTimeout):
- 10:40 PM Changeset in webkit [120802] by
-
- 2 edits in trunk/LayoutTests
[Qt][Mac] Skip one more crashing test.
- platform/qt-mac/Skipped:
- 10:36 PM Changeset in webkit [120801] by
-
- 3 edits2 adds in trunk
Source/WebCore: Fixes condition where inserting a CounterNode subtree which could result in incorrect placement.
https://bugs.webkit.org/show_bug.cgi?id=88142
Reviewed by Adam Barth.
Test: fast/css/counters/counter-reset-subtree-insert-crash.html
- rendering/CounterNode.cpp:
(WebCore::CounterNode::insertAfter):
LayoutTests: Add test which inserts a CounterNode subtree which contains reset nodes into another tree.
https://bugs.webkit.org/show_bug.cgi?id=88142
Reviewed by Adam Barth.
- fast/css/counters/counter-reset-subtree-insert-crash-expected.txt: Added.
- fast/css/counters/counter-reset-subtree-insert-crash.html: Added.
- 10:34 PM Changeset in webkit [120800] by
-
- 3 edits in trunk/LayoutTests
Unreviewed expected update after r120735.
- fast/css/getComputedStyle/computed-style-expected.txt:
- fast/css/getComputedStyle/computed-style-without-renderer-expected.txt:
- 10:18 PM Changeset in webkit [120799] by
-
- 2 edits in trunk/Source/WebCore
[Qt] Unreviewed buildfix after r120790. Typo fix.
- platform/graphics/gstreamer/ImageGStreamerQt.cpp:
(ImageGStreamer::ImageGStreamer):
- 10:12 PM Changeset in webkit [120798] by
-
- 31 edits in trunk/LayoutTests
Use testRunner instead of layoutTestController in fast/{files,filesystem}
https://bugs.webkit.org/show_bug.cgi?id=89462
Reviewed by Ryosuke Niwa.
- fast/files/apply-blob-url-to-img.html:
- fast/files/apply-blob-url-to-xhr.html:
- fast/files/blob-builder-crash.html:
- fast/files/blob-slice-overflow.html:
- fast/files/blob-slice-test.html:
- fast/files/create-blob-url-crash.html:
- fast/files/domurl-script-execution-context-crash.html:
- fast/files/file-reader-abort.html:
- fast/files/file-reader-directory-crash.html:
- fast/files/file-reader-event-listener.html:
- fast/files/file-reader-file-url.html:
- fast/files/file-reader-sandbox-iframe.html:
- fast/files/null-origin-string.html:
- fast/files/read-blob-async.html:
- fast/files/read-file-async.html:
- fast/files/resources/file-reader-real.html:
- fast/files/resources/setup-for-read-common.js:
- fast/files/revoke-blob-url.html:
- fast/files/script-tests/xhr-response-blob.js:
- fast/files/workers/inline-worker-via-blob-url.html:
- fast/files/workers/worker-apply-blob-url-to-xhr.html:
- fast/files/workers/worker-read-blob-async.html:
- fast/files/workers/worker-read-blob-sync.html:
- fast/files/workers/worker-read-file-async.html:
- fast/files/workers/worker-read-file-sync.html:
- fast/filesystem/cross-filesystem-op.html:
- fast/filesystem/file-after-reload-crash.html:
- fast/filesystem/file-metadata-after-write.html:
- fast/filesystem/filesystem-no-callback-null-ptr-crash.html:
- fast/filesystem/filesystem-uri-origin.html:
- 9:31 PM Changeset in webkit [120797] by
-
- 8 edits in trunk
[Shadow DOM] ShadowRoot.host should be gone
https://bugs.webkit.org/show_bug.cgi?id=88387
Source/WebCore:
According to the w3c spec change,
i.e. https://www.w3.org/Bugs/Public/show_bug.cgi?id=16764
Removed host attribute from ShadowRoot.idl.
Patch by Takashi Sakamoto <tasak@google.com> on 2012-06-19
Reviewed by Dimitri Glazkov.
No new tests, because this change modified only idl.
- dom/ShadowRoot.idl:
Just removed host attribute.
LayoutTests:
Patch by Takashi Sakamoto <tasak@google.com> on 2012-06-19
Reviewed by Dimitri Glazkov.
- fast/css/style-scoped/style-scoped-in-shadow-expected.txt:
- fast/css/style-scoped/style-scoped-in-shadow.html:
- fast/dom/shadow/resources/shadow-dom.js:
(isShadowRoot):
- fast/dom/shadow/shadow-root-js-api-expected.txt:
- fast/dom/shadow/shadow-root-js-api.html:
Just removed references to shadow root's host attribute.
- 9:26 PM Changeset in webkit [120796] by
-
- 3 edits in trunk/LayoutTests
Fix a form restore test broken by HTML5 parser
https://bugs.webkit.org/show_bug.cgi?id=89539
Reviewed by Kentaro Hara.
Before HTML5 parser, we parsed innerHTML input text and we attached
renderers to parsed nodes at the same time. The parsed nodes had
restored form values because we restore form values only for attached
elements.
After HTML5 parser, we don't attach renderers during innerHTML
parsing. state-restore-to-nonedited-controls.html used innerHTML to
built a form and the test haven't worked since introducing HTML5 parser.
This patch changes the test so that it uses document.write() instead of
innerHTML.
- fast/forms/state-restore-to-non-edited-controls-expected.txt:
Note that the FAIL line is expected. It's due to Bug 88685.
- fast/forms/state-restore-to-non-edited-controls.html:
Use document.write() instead of innerHTML.
- 9:06 PM Changeset in webkit [120795] by
-
- 3 edits in trunk/Tools
NRWT spins up and down the WebSocket server when running a single HTTP test from the command line
https://bugs.webkit.org/show_bug.cgi?id=64489
Reviewed by Tony Chang.
This change changes NRWT so that we don't start the websocket
server if we aren't running websocket tests and don't start the
HTTP server if we aren't running http tests.
- Scripts/webkitpy/layout_tests/controllers/manager.py:
(Manager._websocket_tests):
(Manager.start_servers_with_lock):
(Manager.stop_servers_with_lock):
- Scripts/webkitpy/layout_tests/controllers/manager_unittest.py:
(ManagerTest.test_servers_started):
- 9:01 PM Changeset in webkit [120794] by
-
- 3 edits in trunk/Tools
test-webkitpy: add a -p flag to pass through captured output to enable debugging
https://bugs.webkit.org/show_bug.cgi?id=89158
Reviewed by Eric Seidel.
Normally when outputcapture runs it intercepts stdout/stderr,
which makes running under the debugger difficult and mostly
pointless (since you can't see any output). This change adds a
flag to test-webkitpy (-p) that will cause outputcapture to pass
through the output as well as capture it.
- Scripts/webkitpy/common/system/outputcapture.py:
(OutputCapture.stream_wrapper):
(OutputCapture._capture_output_with_name):
- Scripts/webkitpy/common/system/outputcapture_unittest.py:
(OutputCaptureTest.setUp):
- Scripts/webkitpy/test/main.py:
(Tester._parse_args):
(Tester._run_tests):
(Tester._log_exception):
(_CaptureAndPassThroughStream):
(_CaptureAndPassThroughStream.init):
(_CaptureAndPassThroughStream.write):
(_CaptureAndPassThroughStream._message_is_from_pdb):
(_CaptureAndPassThroughStream.flush):
(_CaptureAndPassThroughStream.getvalue):
- 8:43 PM Changeset in webkit [120793] by
-
- 4 edits in trunk/LayoutTests
Test fix after r120521.
- webarchive/resources/dump-webarchive.js:
- webarchive/test-duplicate-resources-expected.webarchive:
- webarchive/test-script-src-expected.webarchive:
- 8:28 PM Changeset in webkit [120792] by
-
- 1233 edits in trunk/LayoutTests
Use testRunner instead of layoutTestController in fast/doctypes, dom, dynamic, encoding, events, eventsource, and exclusions tests
https://bugs.webkit.org/show_bug.cgi?id=89438
Reviewed by Tony Chang.
- fast/doctypes/005-case-preserving.html:
- fast/doctypes/doctype-after-comment.html:
- fast/doctypes/doctype-at-end.html:
- fast/doctypes/doctype-in-element.html:
- fast/doctypes/doctype-parsing.html:
- fast/doctypes/html-doctype.html:
- fast/doctypes/xhtml-with-xhtmlmp-doctype.xhtml:
- fast/doctypes/xml-doctype.xhtml:
- fast/dom/:
- fast/dynamic/5872671.html:
- fast/dynamic/ancestor-to-absolute.html:
- fast/dynamic/checkbox-selection-crash.html:
- fast/dynamic/containing-block-change.html:
- fast/dynamic/crash-generated-counter.html:
- fast/dynamic/crash-generated-image.html:
- fast/dynamic/crash-generated-quote.html:
- fast/dynamic/crash-generated-text.html:
- fast/dynamic/crash-paint-no-documentElement-renderer.html-disabled:
- fast/dynamic/float-remove-above-line-2.html:
- fast/dynamic/float-remove-above-line.html:
- fast/dynamic/hover-before-position-after-style-change.html:
- fast/dynamic/hover-before-position-after-style-change2.html:
- fast/dynamic/hover-style-recalc-crash.html:
- fast/dynamic/hovered-detach.html:
- fast/dynamic/inline-to-block-crash.html:
- fast/dynamic/insertAdjacentElement.html:
- fast/dynamic/insertAdjacentHTML-allowed-parents.html:
- fast/dynamic/insertAdjacentHTML.html:
- fast/dynamic/insertAdjacentText.html:
- fast/dynamic/jQuery-animation-crash.html:
- fast/dynamic/layer-hit-test-crash.html:
- fast/dynamic/layer-no-longer-paginated.html:
- fast/dynamic/outerHTML-no-element.html:
- fast/dynamic/paused-event-dispatch.html:
- fast/dynamic/position-absolute-to-fixed-crash.html:
- fast/dynamic/position-change-layout.html:
- fast/dynamic/recursive-layout.html:
- fast/dynamic/style-access-late-stylesheet-load.html:
- fast/dynamic/subtree-common-root.html:
- fast/dynamic/subtree-table-cell-height.html:
- fast/dynamic/subtree-unrooted.html:
- fast/encoding/:
- fast/events/:
- platform/chromium/fast/events/drag-downloadURL.html:
- platform/chromium/fast/events/intercept-postmessage.html:
- platform/chromium/fast/events/rtl-scrollbar.html:
- platform/gtk/fast/events/event-sender-metakey.html:
- platform/mac/fast/dom/HTMLImageElement/dashboard-src-quirk.html:
- platform/mac/fast/dom/character-index-for-point.html:
- platform/mac/fast/dom/objc-wrapper-identity.html:
- platform/mac/fast/dom/wrapper-classes-objc.html:
- platform/mac/fast/dom/wrapper-round-tripping.html:
- platform/mac/fast/events/objc-event-api.html:
- platform/mac/fast/events/objc-keyboard-event-creation.html:
- platform/qt/fast/events/event-sender-keydown-frame.html:
- platform/win/fast/events/alt-numpad.html:
- platform/win/fast/events/alt-space-scroll.html:
- platform/win/fast/events/double-dead-char.html:
- platform/win/fast/events/keyLocation-numpad.html:
- platform/win/fast/events/panScroll-correct-direction.html:
- platform/win/fast/events/panScroll-event-fired.html:
- platform/win/fast/events/panScroll-image-no-scroll.html:
- platform/win/fast/events/panScroll-imageMap-href-no-scroll.html:
- platform/win/fast/events/panScroll-imageMap-noHref-scroll.html:
- platform/win/fast/events/panScroll-nested-divs.html:
- platform/win/fast/events/panScroll-no-iframe-jump.html:
- platform/win/fast/events/panScroll-preventDefault.html:
- platform/win/fast/events/script-tests/context-click-events.js:
- 8:27 PM Changeset in webkit [120791] by
-
- 21 edits in trunk/Source/WebCore
Pass ScriptValue by const ref where possible
https://bugs.webkit.org/show_bug.cgi?id=89531
Reviewed by Kentaro Hara.
Previously ScriptValue was sometimes passed by value, sometimes by const ref.
For consistency and optimality, pass by const ref everywhere (except where
ScriptValue is returned from non-accessor methods).
No new tests, no change in behavior.
- bindings/js/ScriptFunctionCall.h:
(ScriptCallback):
- bindings/js/ScriptProfiler.cpp:
(WebCore::ScriptProfiler::getHeapObjectId):
- bindings/js/ScriptProfiler.h:
(ScriptProfiler):
- bindings/js/WorkerScriptController.cpp:
(WebCore::WorkerScriptController::setException):
- bindings/js/WorkerScriptController.h:
(WorkerScriptController):
- bindings/v8/ScriptFunctionCall.cpp:
(WebCore::ScriptCallback::ScriptCallback):
- bindings/v8/ScriptFunctionCall.h:
(ScriptCallback):
- bindings/v8/ScriptProfiler.cpp:
(WebCore::ScriptProfiler::getHeapObjectId):
(WebCore):
- bindings/v8/ScriptProfiler.h:
(ScriptProfiler):
- bindings/v8/ScriptValue.h:
(WebCore::ScriptValue::operator==):
(WebCore::ScriptValue::operator!=):
- bindings/v8/WorkerScriptController.cpp:
(WebCore::WorkerScriptController::setException):
- bindings/v8/WorkerScriptController.h:
(WorkerScriptController):
- css/MediaQueryListListener.h:
(WebCore::MediaQueryListListener::create):
(WebCore::MediaQueryListListener::MediaQueryListListener):
- dom/CustomEvent.cpp:
(WebCore::CustomEvent::initCustomEvent):
- dom/CustomEvent.h:
(CustomEvent):
(WebCore::CustomEvent::detail):
- dom/MessageEvent.h:
(WebCore::MessageEvent::dataAsScriptValue):
- dom/PopStateEvent.h:
(WebCore::PopStateEvent::state):
- inspector/InjectedScript.cpp:
(WebCore::InjectedScript::wrapObject):
- inspector/InjectedScript.h:
(InjectedScript):
- 7:25 PM Changeset in webkit [120790] by
-
- 12 edits in trunk
[GStreamer] 0.11 video-sink
https://bugs.webkit.org/show_bug.cgi?id=77087
Reviewed by Martin Robinson.
- configure.ac: Fix required gstreamer 0.11 version
Source/WebCore:
Port the video sink to GStreamer 0.11 APIs. There is no change in
functionality compared to 0.10, for now.
- platform/graphics/gstreamer/GStreamerVersioning.cpp:
(webkitGetVideoSizeAndFormatFromCaps):
(webkitGstCreateBuffer):
- platform/graphics/gstreamer/GStreamerVersioning.h:
(WebCore):
- platform/graphics/gstreamer/ImageGStreamer.h:
(WebCore::ImageGStreamer::createImage):
(WebCore::ImageGStreamer::setCropRect):
(WebCore::ImageGStreamer::rect):
(ImageGStreamer):
- platform/graphics/gstreamer/ImageGStreamerCairo.cpp:
(ImageGStreamer::ImageGStreamer):
- platform/graphics/gstreamer/ImageGStreamerQt.cpp:
(ImageGStreamer::ImageGStreamer):
- platform/graphics/gstreamer/MediaPlayerPrivateGStreamer.cpp:
(WebCore::MediaPlayerPrivateGStreamer::~MediaPlayerPrivateGStreamer):
(WebCore::MediaPlayerPrivateGStreamer::naturalSize):
- platform/graphics/gstreamer/VideoSinkGStreamer.cpp:
(_WebKitVideoSinkPrivate):
(webkitVideoSinkRender):
(webkitVideoSinkProposeAllocation):
(webkit_video_sink_class_init):
- platform/graphics/gstreamer/VideoSinkGStreamer.h:
- platform/graphics/gstreamer/WebKitWebSourceGStreamer.cpp:
- 7:15 PM Changeset in webkit [120789] by
-
- 12 edits in trunk/Source
Add monitor profile support for Win
https://bugs.webkit.org/show_bug.cgi?id=88565
Patch by Tony Payne <tpayne@chromium.org> on 2012-06-19
Reviewed by Eric Seidel.
Source/Platform:
- chromium/public/Platform.h:
(Platform):
(WebKit::Platform::screenColorProfile):
Source/WebCore:
Tests: fast/images/jpeg-with-color-profile.html
fast/images/png-with-color-profile.html. Also covered by numerous
layout tests.
- WebCore.gyp/WebCore.gyp:
- platform/PlatformScreen.h:
(WebCore):
- platform/blackberry/PlatformScreenBlackBerry.cpp:
(WebCore::screenColorProfile):
(WebCore):
- platform/chromium/PlatformScreenChromium.cpp:
(WebCore::screenColorProfile):
(WebCore):
- platform/efl/PlatformScreenEfl.cpp:
(WebCore::screenColorProfile):
(WebCore):
- platform/gtk/PlatformScreenGtk.cpp:
(WebCore::screenColorProfile):
(WebCore):
- platform/image-decoders/ImageDecoder.h:
(ImageFrame):
(ImageDecoder):
(WebCore::ImageDecoder::qcmsOutputDeviceProfile):
- platform/image-decoders/jpeg/JPEGImageDecoder.cpp:
(turboSwizzled):
(colorSpaceHasAlpha):
- 7:12 PM Changeset in webkit [120788] by
-
- 2 edits in trunk/Tools
garden-o-matic fails to rebaseline tests with MISSING results
https://bugs.webkit.org/show_bug.cgi?id=89521
Reviewed by Kenneth Russell.
If the bots report "MISSING", we should try to find PNG and TXT results on the server.
- BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/scripts/results.js:
- 6:46 PM Changeset in webkit [120787] by
-
- 2 edits in trunk/Tools
svn.delete_list fails to delete empty parent directories
https://bugs.webkit.org/show_bug.cgi?id=89520
Reviewed by Dirk Pranke.
These functions were calling scm.delete and scm.add, which already
delete the parent directory. We'd get an exception when we tried to
delete the parent directory again.
- Scripts/webkitpy/common/checkout/scm/svn.py:
(SVN._add_parent_directories):
(SVN._delete_parent_directories):
- 6:33 PM Changeset in webkit [120786] by
-
- 26 edits1 add in trunk/Source/JavaScriptCore
JSC should be able to show disassembly for all generated JIT code
https://bugs.webkit.org/show_bug.cgi?id=89536
Reviewed by Gavin Barraclough.
Now instead of doing linkBuffer.finalizeCode(), you do
FINALIZE_CODE(linkBuffer, (... explanation ...)). FINALIZE_CODE() then
prints your explanation and the disassembled code, if
Options::showDisassembly is set to true.
- CMakeLists.txt:
- GNUmakefile.list.am:
- JavaScriptCore.vcproj/JavaScriptCore/JavaScriptCore.vcproj:
- JavaScriptCore.xcodeproj/project.pbxproj:
- Target.pri:
- assembler/LinkBuffer.cpp: Added.
(JSC):
(JSC::LinkBuffer::finalizeCodeWithoutDisassembly):
(JSC::LinkBuffer::finalizeCodeWithDisassembly):
(JSC::LinkBuffer::linkCode):
(JSC::LinkBuffer::performFinalization):
(JSC::LinkBuffer::dumpLinkStatistics):
(JSC::LinkBuffer::dumpCode):
- assembler/LinkBuffer.h:
(LinkBuffer):
(JSC):
- assembler/MacroAssemblerCodeRef.h:
(JSC::MacroAssemblerCodeRef::tryToDisassemble):
(MacroAssemblerCodeRef):
- dfg/DFGJITCompiler.cpp:
(JSC::DFG::JITCompiler::compile):
(JSC::DFG::JITCompiler::compileFunction):
- dfg/DFGOSRExitCompiler.cpp:
- dfg/DFGRepatch.cpp:
(JSC::DFG::generateProtoChainAccessStub):
(JSC::DFG::tryCacheGetByID):
(JSC::DFG::tryBuildGetByIDList):
(JSC::DFG::emitPutReplaceStub):
(JSC::DFG::emitPutTransitionStub):
- dfg/DFGThunks.cpp:
(JSC::DFG::osrExitGenerationThunkGenerator):
- disassembler/Disassembler.h:
(JSC):
(JSC::tryToDisassemble):
- disassembler/UDis86Disassembler.cpp:
(JSC::tryToDisassemble):
- jit/JIT.cpp:
(JSC::JIT::privateCompile):
- jit/JITCode.h:
(JSC::JITCode::tryToDisassemble):
- jit/JITOpcodes.cpp:
(JSC::JIT::privateCompileCTIMachineTrampolines):
- jit/JITOpcodes32_64.cpp:
(JSC::JIT::privateCompileCTIMachineTrampolines):
(JSC::JIT::privateCompileCTINativeCall):
- jit/JITPropertyAccess.cpp:
(JSC::JIT::stringGetByValStubGenerator):
(JSC::JIT::privateCompilePutByIdTransition):
(JSC::JIT::privateCompilePatchGetArrayLength):
(JSC::JIT::privateCompileGetByIdProto):
(JSC::JIT::privateCompileGetByIdSelfList):
(JSC::JIT::privateCompileGetByIdProtoList):
(JSC::JIT::privateCompileGetByIdChainList):
(JSC::JIT::privateCompileGetByIdChain):
- jit/JITPropertyAccess32_64.cpp:
(JSC::JIT::stringGetByValStubGenerator):
(JSC::JIT::privateCompilePutByIdTransition):
(JSC::JIT::privateCompilePatchGetArrayLength):
(JSC::JIT::privateCompileGetByIdProto):
(JSC::JIT::privateCompileGetByIdSelfList):
(JSC::JIT::privateCompileGetByIdProtoList):
(JSC::JIT::privateCompileGetByIdChainList):
(JSC::JIT::privateCompileGetByIdChain):
- jit/SpecializedThunkJIT.h:
(JSC::SpecializedThunkJIT::finalize):
- jit/ThunkGenerators.cpp:
(JSC::charCodeAtThunkGenerator):
(JSC::charAtThunkGenerator):
(JSC::fromCharCodeThunkGenerator):
(JSC::sqrtThunkGenerator):
(JSC::floorThunkGenerator):
(JSC::ceilThunkGenerator):
(JSC::roundThunkGenerator):
(JSC::expThunkGenerator):
(JSC::logThunkGenerator):
(JSC::absThunkGenerator):
(JSC::powThunkGenerator):
- llint/LLIntThunks.cpp:
(JSC::LLInt::generateThunkWithJumpTo):
(JSC::LLInt::functionForCallEntryThunkGenerator):
(JSC::LLInt::functionForConstructEntryThunkGenerator):
(JSC::LLInt::functionForCallArityCheckThunkGenerator):
(JSC::LLInt::functionForConstructArityCheckThunkGenerator):
(JSC::LLInt::evalEntryThunkGenerator):
(JSC::LLInt::programEntryThunkGenerator):
- runtime/Options.cpp:
(Options):
(JSC::Options::initializeOptions):
- runtime/Options.h:
(Options):
- yarr/YarrJIT.cpp:
(JSC::Yarr::YarrGenerator::compile):
- 6:31 PM Changeset in webkit [120785] by
-
- 3 edits in trunk/Tools
webkitpy: Simplify fetch_bugs_matching_search()
https://bugs.webkit.org/show_bug.cgi?id=89497
Patch by Thiago Marcos P. Santos <thiago.santos@intel.com> on 2012-06-19
Reviewed by Adam Barth.
Removed parameter with default value. It's not being used
and the handling of the parameter was wrong. Saves a check.
- Scripts/webkitpy/common/net/bugzilla/bugzilla.py:
(BugzillaQueries.fetch_bugs_matching_search):
- Scripts/webkitpy/common/net/bugzilla/bugzilla_mock.py:
(MockBugzillaQueries.fetch_bugs_matching_search):
- 6:29 PM Changeset in webkit [120784] by
-
- 1 edit4 deletes in trunk/LayoutTests
Remove the failing expected results added in
http://trac.webkit.org/changeset/119190 now that these tests pass.
Rubber-stamped by Jessie Berlin.
- platform/win/fast/frames/frame-set-scaling-hit-expected.txt: Removed.
- platform/win/fast/frames/frame-with-noresize-can-be-resized-after-removal-of-noresize-expected.txt: Removed.
- platform/win/fast/frames/frame-with-noresize-can-be-resized-after-setting-noResize-to-false-expected.txt: Removed.
- platform/win/fast/frames/frames-with-frameborder-zero-can-be-resized-expected.txt: Removed.
- 6:04 PM Changeset in webkit [120783] by
-
- 3 edits2 adds in trunk
[V8] Fix issue where a callback got a null calling context
https://bugs.webkit.org/show_bug.cgi?id=89532
Reviewed by Kentaro Hara.
Source/WebCore:
Test: fast/dom/Window/post-message-crash.html
- bindings/v8/custom/V8DOMWindowCustom.cpp:
(WebCore::handlePostMessageCallback): Check the calling context and throw a type error (like JSC) if it is null.
LayoutTests:
- fast/dom/Window/post-message-crash-expected.txt: Added.
- fast/dom/Window/post-message-crash.html: Added.
- 5:54 PM Changeset in webkit [120782] by
-
- 4 edits in branches/safari-536-branch/Source
Versioning.
- 5:49 PM Changeset in webkit [120781] by
-
- 1 copy in tags/Safari-536.20
New Tag.
- 5:40 PM Changeset in webkit [120780] by
-
- 9 edits in trunk/Source/WebCore
Refactor RenderFlexibleBox terminology to use grow/shrink instead of positive/negative
https://bugs.webkit.org/show_bug.cgi?id=89493
Reviewed by Eric Seidel.
The spec stopped referring to positive and negative flex and now
talks about flex grow and flex shrink. Also, the preferred size is
now known as the flex basis. Make the code match the terms used in
the spec. http://dev.w3.org/csswg/css3-flexbox/
This will also make it easier to split flex into long hand notation
(flex-grow, flex-shrink and flex-basis).
No new tests, covered by existing tests.
- css/CSSComputedStyleDeclaration.cpp:
(WebCore::CSSComputedStyleDeclaration::getPropertyCSSValue):
- css/StyleBuilder.cpp:
(WebCore::ApplyPropertyFlex::applyInheritValue):
(WebCore::ApplyPropertyFlex::applyInitialValue):
(WebCore::ApplyPropertyFlex::applyValue):
- page/animation/CSSPropertyAnimation.cpp:
(WebCore::PropertyWrapperFlex::equals):
(WebCore::PropertyWrapperFlex::blend):
- rendering/RenderFlexibleBox.cpp:
(WebCore::RenderFlexibleBox::flexBasisForChild):
(WebCore::RenderFlexibleBox::preferredMainAxisContentExtentForChild):
(WebCore::RenderFlexibleBox::layoutFlexItems):
(WebCore::RenderFlexibleBox::computeMainAxisPreferredSizes):
(WebCore::RenderFlexibleBox::computeNextFlexLine):
(WebCore::RenderFlexibleBox::freezeViolations):
(WebCore::RenderFlexibleBox::resolveFlexibleLengths):
- rendering/RenderFlexibleBox.h:
- rendering/style/RenderStyle.h:
- rendering/style/StyleFlexibleBoxData.cpp:
(WebCore::StyleFlexibleBoxData::StyleFlexibleBoxData):
(WebCore::StyleFlexibleBoxData::operator==):
- rendering/style/StyleFlexibleBoxData.h:
(StyleFlexibleBoxData):
- 5:25 PM Changeset in webkit [120779] by
-
- 2 edits1 delete in trunk/LayoutTests
[Chromium] Unreviewed test expectations update.
- platform/chromium-win-xp/platform/chromium/virtual/gpu/fast/canvas/resize-while-save-active-expected.txt: Removed.
- platform/chromium/TestExpectations:
Referenced bugs 89529 and 89534.
- 5:20 PM Changeset in webkit [120778] by
-
- 4 edits1 delete in trunk/Source/JavaScriptCore
[Qt][Mac] REGRESSION(r120742): It broke the build
https://bugs.webkit.org/show_bug.cgi?id=89516
Reviewed by Geoffrey Garen.
Removing GCActivityCallbackCF.cpp because it doesn't mesh well with cross-platform
code on Darwin (e.g. Qt). We now use plain ol' vanilla ifdefs to handle platforms
without CF support. These if-defs will probably disappear in the future when we
use cross-platform timers in HeapTimer.
- JavaScriptCore.vcproj/JavaScriptCore/JavaScriptCore.vcproj:
- JavaScriptCore.xcodeproj/project.pbxproj:
- runtime/GCActivityCallback.cpp:
(JSC):
(JSC::DefaultGCActivityCallback::DefaultGCActivityCallback):
(JSC::DefaultGCActivityCallback::doWork):
(JSC::DefaultGCActivityCallback::scheduleTimer):
(JSC::DefaultGCActivityCallback::cancelTimer):
(JSC::DefaultGCActivityCallback::didAllocate):
(JSC::DefaultGCActivityCallback::willCollect):
(JSC::DefaultGCActivityCallback::cancel):
- runtime/GCActivityCallbackCF.cpp: Removed.
- 5:15 PM Changeset in webkit [120777] by
-
- 2 edits in trunk/Tools
webkitpy: remove unneeded imports in layout_tests/port/chromium.py
https://bugs.webkit.org/show_bug.cgi?id=89522
Reviewed by Eric Seidel.
They were no longer being used. No tests necessary.
- Scripts/webkitpy/layout_tests/port/chromium.py:
- 5:09 PM Changeset in webkit [120776] by
-
- 5 edits3 adds in trunk
Source/WebCore: Make DOMFileSystem selectively structure clonable so it can be the body of a web intent.
https://bugs.webkit.org/show_bug.cgi?id=88267
Patch by Steve VanDeBogart <vandebo@chromium.org> on 2012-06-19
Reviewed by David Levin.
This functionality will allow FileSystem API objects to be dispatched to
web content in order to enable, for instance, an intent notifying web
content that a particular filesystem has been mounted by the local
system. The goal at this point is not to enable across-the-board
clonability, but just for handling the particular use case.
- bindings/v8/SerializedScriptValue.cpp:
(WebCore::V8ObjectMap::Writer::writeDOMFileSystem):
(WebCore::V8ObjectMap::Serializer::writeDOMFileSystem):
(WebCore::V8ObjectMap::Serializer::doSerialize):
(WebCore::V8ObjectMap::Reader::read):
(WebCore::V8ObjectMap::Reader::readDOMFileSystem):
LayoutTests: Add filesystem regression test to check that the filesystem object is
not serializable as created by javascript.
https://bugs.webkit.org/show_bug.cgi?id=88267
Patch by Greg Billock <gbillock@google.com> on 2012-06-19
Reviewed by David Levin.
- fast/filesystem/filesystem-unserializable-expected.html: Added.
- fast/filesystem/filesystem-unserializable.html: Added.
- fast/filesystem/script-tests/filesystem-unserializable.js: Added.
- 5:07 PM Changeset in webkit [120775] by
-
- 76 edits in trunk/Source
Unreviewed, rolling out r120772.
http://trac.webkit.org/changeset/120772
https://bugs.webkit.org/show_bug.cgi?id=89503
Broke build on Chromium Mac
Source/WebCore:
- platform/graphics/chromium/AnimationTranslationUtil.h:
(WebCore):
- platform/graphics/chromium/BitmapSkPictureCanvasLayerTextureUpdater.cpp:
- platform/graphics/chromium/BitmapSkPictureCanvasLayerTextureUpdater.h:
- platform/graphics/chromium/Canvas2DLayerBridge.cpp:
- platform/graphics/chromium/Canvas2DLayerBridge.h:
- platform/graphics/chromium/ContentLayerChromium.cpp:
- platform/graphics/chromium/ContentLayerChromium.h:
- platform/graphics/chromium/DrawingBufferChromium.cpp:
- platform/graphics/chromium/FrameBufferSkPictureCanvasLayerTextureUpdater.cpp:
- platform/graphics/chromium/GraphicsLayerChromium.h:
(WebCore):
- platform/graphics/chromium/ImageLayerChromium.cpp:
- platform/graphics/chromium/LayerChromium.h:
- platform/graphics/chromium/LayerRendererChromium.cpp:
- platform/graphics/chromium/LayerRendererChromium.h:
(WebCore):
- platform/graphics/chromium/LayerTextureUpdater.h:
- platform/graphics/chromium/ManagedTexture.cpp:
- platform/graphics/chromium/ProgramBinding.h:
- platform/graphics/chromium/RenderSurfaceChromium.cpp:
- platform/graphics/chromium/RenderSurfaceChromium.h:
- platform/graphics/chromium/TextureCopier.cpp:
- platform/graphics/chromium/TextureLayerChromium.cpp:
- platform/graphics/chromium/TextureManager.cpp:
- platform/graphics/chromium/TiledLayerChromium.cpp:
- platform/graphics/chromium/TiledLayerChromium.h:
(WebCore):
- platform/graphics/chromium/cc/CCActiveAnimation.h:
- platform/graphics/chromium/cc/CCDelayBasedTimeSource.cpp:
- platform/graphics/chromium/cc/CCDrawQuad.cpp:
- platform/graphics/chromium/cc/CCFontAtlas.cpp:
- platform/graphics/chromium/cc/CCFrameRateController.cpp:
- platform/graphics/chromium/cc/CCFrameRateController.h:
- platform/graphics/chromium/cc/CCHeadsUpDisplay.cpp:
- platform/graphics/chromium/cc/CCHeadsUpDisplay.h:
(WebCore):
- platform/graphics/chromium/cc/CCIOSurfaceLayerImpl.cpp:
- platform/graphics/chromium/cc/CCLayerAnimationController.h:
- platform/graphics/chromium/cc/CCLayerImpl.cpp:
- platform/graphics/chromium/cc/CCLayerImpl.h:
(WebCore):
- platform/graphics/chromium/cc/CCLayerTreeHost.cpp:
- platform/graphics/chromium/cc/CCLayerTreeHost.h:
(WebCore):
- platform/graphics/chromium/cc/CCLayerTreeHostCommon.cpp:
- platform/graphics/chromium/cc/CCLayerTreeHostImpl.cpp:
- platform/graphics/chromium/cc/CCLayerTreeHostImpl.h:
- platform/graphics/chromium/cc/CCOcclusionTracker.cpp:
- platform/graphics/chromium/cc/CCOcclusionTracker.h:
- platform/graphics/chromium/cc/CCProxy.cpp:
- platform/graphics/chromium/cc/CCQuadCuller.cpp:
- platform/graphics/chromium/cc/CCRenderPass.h:
- platform/graphics/chromium/cc/CCRenderSurface.cpp:
- platform/graphics/chromium/cc/CCRenderSurface.h:
- platform/graphics/chromium/cc/CCRenderer.h:
(WebCore):
- platform/graphics/chromium/cc/CCScopedThreadProxy.h:
- platform/graphics/chromium/cc/CCScrollbarLayerImpl.cpp:
- platform/graphics/chromium/cc/CCScrollbarLayerImpl.h:
- platform/graphics/chromium/cc/CCSingleThreadProxy.cpp:
- platform/graphics/chromium/cc/CCSingleThreadProxy.h:
- platform/graphics/chromium/cc/CCSolidColorLayerImpl.cpp:
- platform/graphics/chromium/cc/CCTextureLayerImpl.cpp:
- platform/graphics/chromium/cc/CCThreadProxy.cpp:
- platform/graphics/chromium/cc/CCThreadProxy.h:
- platform/graphics/chromium/cc/CCTiledLayerImpl.cpp:
- platform/graphics/chromium/cc/CCTiledLayerImpl.h:
- platform/graphics/chromium/cc/CCTimer.cpp:
- platform/graphics/chromium/cc/CCTimer.h:
- platform/graphics/chromium/cc/CCVideoLayerImpl.cpp:
Source/WebKit/chromium:
- src/WebCompositorImpl.cpp:
- src/WebLayerTreeView.cpp:
- tests/CCLayerTestCommon.cpp:
- tests/CCLayerTreeHostImplTest.cpp:
- tests/CCLayerTreeHostTest.cpp:
- tests/CCOcclusionTrackerTest.cpp:
- tests/CCQuadCullerTest.cpp:
- tests/CCTiledLayerImplTest.cpp:
- tests/ImageLayerChromiumTest.cpp:
- tests/LayerRendererChromiumTest.cpp:
- tests/TiledLayerChromiumTest.cpp:
- 4:50 PM Changeset in webkit [120774] by
-
- 2 edits in trunk/Source/WebKit/blackberry
[BlackBerry] Should check the return value of fromUTF8() before executing the script.
https://bugs.webkit.org/show_bug.cgi?id=89500
Patch by Yong Li <yoli@rim.com> on 2012-06-19
Reviewed by Rob Buis.
PR# 166623.
WebPagePrivate::executeJavaScript() should check the return value of fromUTF8()
before executing the script to avoid potential crash on invalid script source.
- Api/WebPage.cpp:
(BlackBerry::WebKit::WebPagePrivate::executeJavaScript):
- 4:47 PM Changeset in webkit [120773] by
-
- 2 edits in trunk/LayoutTests
Skipping failing tests to keep the bots green. Filed
https://bugs.webkit.org/show_bug.cgi?id=89528 to track fixing the
real problem.
- platform/mac/Skipped:
- 4:46 PM Changeset in webkit [120772] by
-
- 76 edits in trunk/Source
[Chromium] Remove redundant #includes in compositor
https://bugs.webkit.org/show_bug.cgi?id=89503
Patch by Zeev Lieber <zlieber@chromium.org> on 2012-06-19
Reviewed by Adrienne Walker.
Source/WebCore:
Removed some redundant #includes to clarify dependency
picture. Some 90 #includes removed, some 50 less disruptive
#includes added. This brings the number of class dependencies to
902 from 1383.
No new tests are required, as this is only affects compile-time behavior.
- platform/graphics/chromium/AnimationTranslationUtil.h:
(WebCore):
- platform/graphics/chromium/BitmapSkPictureCanvasLayerTextureUpdater.cpp:
- platform/graphics/chromium/BitmapSkPictureCanvasLayerTextureUpdater.h:
- platform/graphics/chromium/Canvas2DLayerBridge.cpp:
- platform/graphics/chromium/Canvas2DLayerBridge.h:
- platform/graphics/chromium/ContentLayerChromium.cpp:
- platform/graphics/chromium/ContentLayerChromium.h:
- platform/graphics/chromium/DrawingBufferChromium.cpp:
- platform/graphics/chromium/FrameBufferSkPictureCanvasLayerTextureUpdater.cpp:
- platform/graphics/chromium/GraphicsLayerChromium.h:
- platform/graphics/chromium/ImageLayerChromium.cpp:
- platform/graphics/chromium/LayerChromium.h:
- platform/graphics/chromium/LayerRendererChromium.cpp:
- platform/graphics/chromium/LayerRendererChromium.h:
(WebCore):
- platform/graphics/chromium/LayerTextureUpdater.h:
- platform/graphics/chromium/ManagedTexture.cpp:
- platform/graphics/chromium/ProgramBinding.h:
- platform/graphics/chromium/RenderSurfaceChromium.cpp:
- platform/graphics/chromium/RenderSurfaceChromium.h:
- platform/graphics/chromium/TextureCopier.cpp:
- platform/graphics/chromium/TextureLayerChromium.cpp:
- platform/graphics/chromium/TextureManager.cpp:
- platform/graphics/chromium/TiledLayerChromium.cpp:
- platform/graphics/chromium/TiledLayerChromium.h:
- platform/graphics/chromium/cc/CCActiveAnimation.h:
(WebCore):
- platform/graphics/chromium/cc/CCDelayBasedTimeSource.cpp:
- platform/graphics/chromium/cc/CCDrawQuad.cpp:
- platform/graphics/chromium/cc/CCFontAtlas.cpp:
- platform/graphics/chromium/cc/CCFrameRateController.cpp:
- platform/graphics/chromium/cc/CCFrameRateController.h:
(WebCore):
- platform/graphics/chromium/cc/CCHeadsUpDisplay.cpp:
- platform/graphics/chromium/cc/CCHeadsUpDisplay.h:
(WebCore):
- platform/graphics/chromium/cc/CCIOSurfaceLayerImpl.cpp:
- platform/graphics/chromium/cc/CCLayerAnimationController.h:
- platform/graphics/chromium/cc/CCLayerImpl.cpp:
- platform/graphics/chromium/cc/CCLayerImpl.h:
(WebCore):
- platform/graphics/chromium/cc/CCLayerTreeHost.cpp:
- platform/graphics/chromium/cc/CCLayerTreeHost.h:
(WebCore):
- platform/graphics/chromium/cc/CCLayerTreeHostCommon.cpp:
- platform/graphics/chromium/cc/CCLayerTreeHostImpl.cpp:
- platform/graphics/chromium/cc/CCLayerTreeHostImpl.h:
- platform/graphics/chromium/cc/CCOcclusionTracker.cpp:
- platform/graphics/chromium/cc/CCOcclusionTracker.h:
(WebCore):
- platform/graphics/chromium/cc/CCProxy.cpp:
- platform/graphics/chromium/cc/CCQuadCuller.cpp:
- platform/graphics/chromium/cc/CCRenderPass.h:
(WebCore):
- platform/graphics/chromium/cc/CCRenderSurface.cpp:
- platform/graphics/chromium/cc/CCRenderSurface.h:
- platform/graphics/chromium/cc/CCRenderer.h:
- platform/graphics/chromium/cc/CCScopedThreadProxy.h:
- platform/graphics/chromium/cc/CCScrollbarLayerImpl.cpp:
- platform/graphics/chromium/cc/CCScrollbarLayerImpl.h:
- platform/graphics/chromium/cc/CCSingleThreadProxy.cpp:
- platform/graphics/chromium/cc/CCSingleThreadProxy.h:
- platform/graphics/chromium/cc/CCSolidColorLayerImpl.cpp:
- platform/graphics/chromium/cc/CCTextureLayerImpl.cpp:
- platform/graphics/chromium/cc/CCThreadProxy.cpp:
- platform/graphics/chromium/cc/CCThreadProxy.h:
- platform/graphics/chromium/cc/CCTiledLayerImpl.cpp:
- platform/graphics/chromium/cc/CCTiledLayerImpl.h:
(WebCore):
- platform/graphics/chromium/cc/CCTimer.cpp:
- platform/graphics/chromium/cc/CCTimer.h:
(WebCore):
- platform/graphics/chromium/cc/CCVideoLayerImpl.cpp:
Source/WebKit/chromium:
Minor adjustments to #include statements to remove
dependencies. No new tests.
- src/WebCompositorImpl.cpp:
- src/WebLayerTreeView.cpp:
- tests/CCLayerTestCommon.cpp:
- tests/CCLayerTreeHostImplTest.cpp:
- tests/CCLayerTreeHostTest.cpp:
- tests/CCOcclusionTrackerTest.cpp:
- tests/CCQuadCullerTest.cpp:
- tests/CCTiledLayerImplTest.cpp:
- tests/ImageLayerChromiumTest.cpp:
- tests/LayerRendererChromiumTest.cpp:
- tests/TiledLayerChromiumTest.cpp:
- 4:43 PM Changeset in webkit [120771] by
-
- 4 edits in branches/chromium/1180/Source/WebCore
Merge 120640 - [chromium] REGRESSION(r120346) ContentLayerPainter should clear rect to be painted
https://bugs.webkit.org/show_bug.cgi?id=89376
Reviewed by Adrienne Walker.
In r120346, a refactor moved a clearRect()/clip() call from ContentLayerPainter - which all
ContentLayerChromiums use - out to OpaqueRectTrackingContentLayerDelegate - which all GraphicsLayerChromium
backed layers use, but which callers using WebContentLayer directly do not use. This meant that the canvas
wasn't cleared/clipped as WebContentLayerClient.h claimed it would be. This moves the clear and the clip
into CanvasLayerTextureUpdater which all paths use, including layers that don't use the ContentLayerPainter.
- platform/graphics/chromium/ContentLayerChromium.cpp:
(WebCore::ContentLayerPainter::paint):
- platform/graphics/chromium/OpaqueRectTrackingContentLayerDelegate.cpp:
(WebCore::OpaqueRectTrackingContentLayerDelegate::paintContents):
TBR=jamesr@google.com
BUG=132982
Review URL: https://chromiumcodereview.appspot.com/10574034
- 4:39 PM Changeset in webkit [120770] by
-
- 5 edits in trunk/Source/WebCore
Align RenderLayer's descendant dependent flags semantics
https://bugs.webkit.org/show_bug.cgi?id=89241
Reviewed by Simon Fraser.
Refactoring only, covered by existing tests.
This change aligns the naming and implementation of the different descendant
dependent flags. While aligning, I found some bugs (inefficiencies) with how
the visible descendant flags was computed.
- rendering/RenderLayer.cpp:
(WebCore::RenderLayer::setHasVisibleContent):
Changed this method to not take a boolean as every callers was passing 'true'.
Reworked the logic under this assumption.
(WebCore::RenderLayer::dirtyVisibleContentStatus):
Updated after the following renaming.
(WebCore::RenderLayer::dirtyAncestorChainVisibleDescendantStatus):
Renamed this function from dirtyVisibleDescendantStatus to match the hasSelfPaintingLayer
naming and implementation.
(WebCore::RenderLayer::setAncestorChainHasVisibleDescendant):
Added this function to factor the visible descendant setting out of defunct childVisibilityChanged.
Also improved the efficiency of the function by clearing the dirty flag as it goes up (an unnoticed bug).
(WebCore::RenderLayer::addChild):
(WebCore::RenderLayer::removeChild):
Updated to use the new functions lieu of childVisibilityChanged.
- rendering/RenderLayer.h:
(RenderLayer::childVisibilityChanged):
Removed this function as it wasn't adding much and it's a lot more clear to call
dirtyAncestorChainVisibleDescendantStatus / setAncestorChainHasVisibleDescendant.
- rendering/RenderObject.cpp:
(WebCore::RenderObject::styleWillChange):
- rendering/RenderObjectChildList.cpp:
(WebCore::RenderObjectChildList::appendChildNode):
(WebCore::RenderObjectChildList::insertChildNode):
Updated those callers after removing the boolean parameter from setHasVisibleContent.
- 4:21 PM Changeset in webkit [120769] by
-
- 10 edits2 adds in trunk/Source/WebCore
Web Inspector: Extract InjectedScriptBase class from the InjectedScript
https://bugs.webkit.org/show_bug.cgi?id=89107
Extract methods from the InjectedScript that do not depend on
the concrete InjectedScriptSource.js implementation.
Patch by Andrey Adaikin <aandrey@chromium.org> on 2012-06-19
Reviewed by Pavel Feldman.
- CMakeLists.txt:
- GNUmakefile.list.am:
- Target.pri:
- WebCore.gypi:
- WebCore.vcproj/WebCore.vcproj:
- WebCore.xcodeproj/project.pbxproj:
- inspector/InjectedScript.cpp:
(WebCore::InjectedScript::InjectedScript):
(WebCore::InjectedScript::evaluate):
(WebCore::InjectedScript::callFunctionOn):
(WebCore::InjectedScript::evaluateOnCallFrame):
(WebCore::InjectedScript::getFunctionDetails):
(WebCore::InjectedScript::getProperties):
(WebCore::InjectedScript::nodeForObjectId):
(WebCore::InjectedScript::releaseObject):
(WebCore::InjectedScript::wrapCallFrames):
(WebCore::InjectedScript::wrapObject):
(WebCore::InjectedScript::wrapSerializedObject):
(WebCore::InjectedScript::findObjectById):
(WebCore::InjectedScript::inspectNode):
(WebCore::InjectedScript::releaseObjectGroup):
(WebCore::InjectedScript::nodeAsScriptValue):
- inspector/InjectedScript.h:
(WebCore):
(InjectedScript):
- inspector/InjectedScriptBase.cpp: Added.
(WebCore):
(WebCore::InjectedScriptBase::InjectedScriptBase):
(WebCore::InjectedScriptBase::canAccessInspectedWindow):
(WebCore::InjectedScriptBase::injectedScriptObject):
(WebCore::InjectedScriptBase::callFunctionWithEvalEnabled):
(WebCore::InjectedScriptBase::makeCall):
(WebCore::InjectedScriptBase::makeEvalCall):
- inspector/InjectedScriptBase.h: Copied from Source/WebCore/bindings/js/ScriptObject.h.
(WebCore):
(InjectedScriptBase):
(WebCore::InjectedScriptBase::~InjectedScriptBase):
(WebCore::InjectedScriptBase::hasNoValue):
(WebCore::InjectedScriptBase::scriptState):
- inspector/InspectorAllInOne.cpp:
- 4:19 PM Changeset in webkit [120768] by
-
- 2 edits1 move2 adds2 deletes in trunk/LayoutTests
[Chromium][Mac] Layout Test ietestcenter/css3/bordersbackgrounds/border-radius-clip-002.htm is failing
https://bugs.webkit.org/show_bug.cgi?id=89411
Unreviewed test expectations update generated by garden-o-matic.
- ietestcenter/css3/bordersbackgrounds/border-radius-clip-002-expected.txt: Renamed from LayoutTests/platform/efl/ietestcenter/css3/bordersbackgrounds/border-radius-clip-002-expected.txt.
- platform/chromium-mac-leopard/ietestcenter/css3/bordersbackgrounds/border-radius-clip-002-expected.png: Added.
- platform/chromium-mac/ietestcenter/css3/bordersbackgrounds/border-radius-clip-002-expected.txt: Removed.
- platform/chromium-win/ietestcenter/css3/bordersbackgrounds/border-radius-clip-002-expected.png: Added.
- platform/chromium/TestExpectations:
Removed IMAGE MISSING expectation for this test.
- platform/gtk/ietestcenter/css3/bordersbackgrounds/border-radius-clip-002-expected.txt: Removed.
- 4:10 PM Changeset in webkit [120767] by
-
- 3 edits3 adds in trunk
DFG CFA forgets to notify subsequent phases of found constants if it proves LogicalNot to be a constant
https://bugs.webkit.org/show_bug.cgi?id=89511
<rdar://problem/11700089>
Reviewed by Geoffrey Garen.
Source/JavaScriptCore:
- dfg/DFGAbstractState.cpp:
(JSC::DFG::AbstractState::execute):
LayoutTests:
- fast/js/dfg-constant-fold-logical-not-branch-expected.txt: Added.
- fast/js/dfg-constant-fold-logical-not-branch.html: Added.
- fast/js/script-tests/dfg-constant-fold-logical-not-branch.js: Added.
(foo1):
(foo2):
(Stuff):
(Stuff.prototype.thingy):
(Stuff.prototype.otherThingy):
- 4:09 PM Changeset in webkit [120766] by
-
- 1 edit10 deletes in trunk/LayoutTests
Remove the failing results added in
http://trac.webkit.org/changeset/119144 and
http://trac.webkit.org/changeset/119149 now that these tests pass
without them.
Rubber-stamped by Jessie Berlin.
- platform/win/fast/events/popup-allowed-from-gesture-initiated-event-expected.txt: Removed.
- platform/win/fast/events/popup-allowed-from-gesture-initiated-form-submit-expected.txt: Removed.
- platform/win/fast/events/popup-blocked-from-fake-user-gesture-expected.txt: Removed.
- platform/win/fast/events/popup-blocked-from-untrusted-mouse-click-expected.txt: Removed.
- platform/win/fast/events/popup-blocking-timers-expected.txt: Removed.
- platform/win/fast/events/remove-target-in-mouseup-deep-expected.txt: Removed.
- platform/win/fast/events/remove-target-in-mouseup-expected.txt: Removed.
- platform/win/fast/events/remove-target-in-mouseup-insertback-expected.txt: Removed.
- platform/win/fast/events/remove-target-in-mouseup-twice-expected.txt: Removed.
- platform/win/fast/events/remove-target-with-shadow-in-drag-expected.txt: Removed.
- 3:51 PM Changeset in webkit [120765] by
-
- 4 edits in trunk/Source/WebCore
Unreviewed, rolling out r120621.
http://trac.webkit.org/changeset/120621
https://bugs.webkit.org/show_bug.cgi?id=89518
Memory graph should have been behind the experimental flag.
(Requested by pfeldman on #webkit).
Patch by Sheriff Bot <webkit.review.bot@gmail.com> on 2012-06-19
- inspector/front-end/NativeMemorySnapshotView.js:
- inspector/front-end/ProfileLauncherView.js:
(WebInspector.ProfileLauncherView):
- inspector/front-end/nativeMemoryProfiler.css:
(.memory-blocks-list .item):
- 3:49 PM Changeset in webkit [120764] by
-
- 2 edits in trunk/Source/WebCore
The CSS_WRAP_SHAPE CSSWrapShape::Type is no longer needed
https://bugs.webkit.org/show_bug.cgi?id=89403
Patch by Hans Muller <hmuller@adobe.com> on 2012-06-19
Reviewed by Eric Seidel.
Removed CSS_WRAP_SHAPE_PATH definition from CSSWrapShapes.h.
It's no longer required. This definition was never used or
tested, no new tests are required.
- css/CSSWrapShapes.h:
- 3:48 PM Changeset in webkit [120763] by
-
- 2 edits in trunk/Source/WebKit/blackberry
[BlackBerry] Missing notifyContentRendered() when resuming rendering
https://bugs.webkit.org/show_bug.cgi?id=89507
RIM PR 150049
Reviewed by Antonio Gomes.
All the current calls to notifyContentRendered() are
located in the RenderQueue, but this omits the case
when we force rendering from resumeBackingStore()
after rendering had been suspended for a while.
- Api/WebPage.cpp:
(BlackBerry::WebKit::WebPagePrivate::resumeBackingStore):
- 3:46 PM Changeset in webkit [120762] by
-
- 1 edit2 copies in branches/chromium/1132
Merge 120559
BUG=130356
Review URL: https://chromiumcodereview.appspot.com/10536212
- 3:34 PM Changeset in webkit [120761] by
-
- 3 edits4 adds in trunk
Absolute positioned objects should not be added to anonymous block lists
https://bugs.webkit.org/show_bug.cgi?id=87768
Reviewed by Abhishek Arya.
Source/WebCore:
containingBlock() was returning an anonymous block for absolute
positioned objects under a relative positioned inline in the case
that the inline is split and the object is underneath the block
continuation. Anonymous blocks should never have anything in their
positioned object lists because they can be destroyed at any time
for a different reasons such as anonymous block merging, which is
a problem for layout if they have m_posChildNeedsLayout set.
This patch adds a generic check for anonymous blocks in
containingBlock() to correct this problem.
- rendering/RenderObject.cpp:
(WebCore::RenderObject::containingBlock):
LayoutTests:
Test crashing condition in bug 87768.
- fast/block/positioning/abspositioned-object-under-split-relpositioned-inline-crash-expected.txt: Added
- fast/block/positioning/abspositioned-object-under-split-relpositioned-inline-crash.html: Added
- fast/block/positioning/insert-positioned-in-anonymous-crash-expected.txt: Added
- fast/block/positioning/insert-positioned-in-anonymous-crash.html: Added
- 3:30 PM Changeset in webkit [120760] by
-
- 5 edits in trunk
Source/WebCore: Web Inspector: extensionPanel.onSearch
actionstrings should be
enumerated.
https://bugs.webkit.org/show_bug.cgi?id=89509
Patch by Jan Keromnes <janx@linux.com> on 2012-06-19
Reviewed by Pavel Feldman.
Fixed LayoutTests/inspector/extensions/extensions-api-expected.txt
accordingly.
- inspector/front-end/ExtensionAPI.js:
(defineCommonExtensionSymbols):
(injectedExtensionAPI.Panels.prototype.setOpenResourceHandler):
(injectedExtensionAPI.Panels.prototype.get SearchAction):
- inspector/front-end/ExtensionPanel.js:
(WebInspector.ExtensionPanel.prototype.searchCanceled):
(WebInspector.ExtensionPanel.prototype.performSearch):
(WebInspector.ExtensionPanel.prototype.jumpToNextSearchResult):
(WebInspector.ExtensionPanel.prototype.jumpToPreviousSearchResult):
LayoutTests: Web Inspector: extensionPanel.onSearch
actionstrings should be enumerated
https://bugs.webkit.org/show_bug.cgi?id=89509
Patch by Jan Keromnes <janx@linux.com> on 2012-06-19
Reviewed by Pavel Feldman.
- inspector/extensions/extensions-api-expected.txt:
- 3:18 PM Changeset in webkit [120759] by
-
- 5 edits in trunk/Source/WebCore
[chromium] Add an allocation step for CCRenderer before drawing a frame
https://bugs.webkit.org/show_bug.cgi?id=88924
Reviewed by Adrienne Walker.
This step will be used to decide which RenderPass textures can be kept
alive and do not need to be drawn again.
- platform/graphics/chromium/LayerRendererChromium.cpp:
(WebCore):
(WebCore::LayerRendererChromium::decideRenderPassAllocationsForFrame):
(WebCore::LayerRendererChromium::beginDrawingFrame):
- platform/graphics/chromium/LayerRendererChromium.h:
(LayerRendererChromium):
- platform/graphics/chromium/cc/CCLayerTreeHostImpl.cpp:
(WebCore::CCLayerTreeHostImpl::drawLayers):
- platform/graphics/chromium/cc/CCRenderer.h:
(CCRenderer):
- 3:01 PM Changeset in webkit [120758] by
-
- 2 edits1 add in trunk/LayoutTests
[Chromium] Unreviewed gardening.
- platform/chromium-win-xp/platform/chromium/virtual/gpu/fast/canvas/resize-while-save-active-expected.txt:
Added expectations without trailing newlines; unclear why this only happens in this configuration.
- platform/chromium/TestExpectations:
Added reference to https://bugs.webkit.org/show_bug.cgi?id=89510 .
- 2:44 PM Changeset in webkit [120757] by
-
- 12 edits2 adds in trunk/Source/WebCore
[CSS Shaders] Create a shared object between FECustomFilter objects to store the shared resources
https://bugs.webkit.org/show_bug.cgi?id=87878
Reviewed by Dean Jackson.
An object called CustomFilterGlobalContext is created the first time a new FECustomFilter is needed in a document.
It is used to store the GraphicsContext3D needed to render the CSS Shaders. The context is shared by all the elements
in a document (no cross-domain).
More resources will be shared in following patches, including precompiled shaders and vertices/indices data.
https://bugs.webkit.org/show_bug.cgi?id=88427
https://bugs.webkit.org/show_bug.cgi?id=88429
The plan is to make CustomFilterGlobalContext and FECustomFilter (or a base class extracted out of it) usable from platform
code in the hardware accelerated compositing layer. The only dependency of the code is on the GraphicsContext3D.
No new tests. No functionality is changed yet, just added the new objects.
- CMakeLists.txt:
- GNUmakefile.list.am:
- Target.pri:
- WebCore.gypi:
- WebCore.vcproj/WebCore.vcproj:
- WebCore.xcodeproj/project.pbxproj:
- platform/graphics/filters/CustomFilterGlobalContext.cpp: Added.
(WebCore):
(WebCore::CustomFilterGlobalContext::CustomFilterGlobalContext):
(WebCore::CustomFilterGlobalContext::~CustomFilterGlobalContext):
(WebCore::CustomFilterGlobalContext::prepareContextIfNeeded):
- platform/graphics/filters/CustomFilterGlobalContext.h: Added.
(WebCore):
(CustomFilterGlobalContext):
(WebCore::CustomFilterGlobalContext::context):
- platform/graphics/filters/FECustomFilter.cpp:
(WebCore::FECustomFilter::FECustomFilter): Changed to use the global context instead of the host window.
(WebCore::FECustomFilter::create):
(WebCore::FECustomFilter::initializeContext):
- platform/graphics/filters/FECustomFilter.h:
(WebCore):
(FECustomFilter):
- rendering/FilterEffectRenderer.cpp:
(WebCore::FilterEffectRenderer::build): Changed to initialize the global context if needed and pass it to the FECustomFilter.
- rendering/RenderView.cpp:
(WebCore):
(WebCore::RenderView::customFilterGlobalContext):
- rendering/RenderView.h:
(WebCore):
(RenderView):
- 2:43 PM Changeset in webkit [120756] by
-
- 2 edits in trunk/Tools
Missing results aren't shown in garden-o-matic
Reviewed by Kenneth Russell.
Previously we didn't know to look for text and image results when the
bot told us that results were missing.
- BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/scripts/results.js:
- 2:33 PM Changeset in webkit [120755] by
-
- 7 edits in trunk
[cairo] Fix LayoutTests/fast/canvas/patternfill-repeat.html
https://bugs.webkit.org/show_bug.cgi?id=53085
Patch by Dominik Röttsches <dominik.rottsches@intel.com> on 2012-06-19
Reviewed by Martin Robinson.
Source/WebCore:
Clipping previously unlimited vertical and horizontal pattern repeats with
a clip rectangle similar to the Qt Graphics Context.
No new tests, this patch fixes
canvas/philip/tests/2d.pattern.paint.repeat* tests.
- platform/graphics/Pattern.h:
(WebCore::Pattern::getPatternSpaceTransform): Adding constant getter method to be able to map to pattern space externally, needed by PlatformContextCairo::clipForPatternFilling.
- platform/graphics/cairo/PlatformContextCairo.cpp:
(WebCore::PlatformContextCairo::prepareForFilling):
(WebCore::PlatformContextCairo::clipForPatternFilling): Clip pattern repeats if needed.
- platform/graphics/cairo/PlatformContextCairo.h:
LayoutTests:
canvas/philip/tests/2d.pattern.paint.repeat* tests now passing.
- platform/efl/Skipped:
- platform/gtk/TestExpectations:
- 2:20 PM Changeset in webkit [120754] by
-
- 6 edits in trunk
[Shadow DOM] An upper boundary breaks if a child node of an insertion point is clicked.
https://bugs.webkit.org/show_bug.cgi?id=89407
Reviewed by Dimitri Glazkov.
Source/WebCore:
Update an event re-targeting algorithm so that it can set event's
target correctly when a fallback element in an insertion point is
clicked. To distinguish between distributed nodes and fallback
elements in an ancestor chain of composite shadow tree, an event
dispatcher checks explicitly whether a node is actually
distributed or assigned to the given insertion point.
Test: fast/dom/shadow/shadow-dom-event-dispatching.html
- dom/EventDispatcher.cpp:
(WebCore::isAssignedTo):
(WebCore):
(WebCore::EventDispatcher::ensureEventAncestors):
- html/shadow/InsertionPoint.h:
(WebCore::InsertionPoint::contains):
LayoutTests:
- fast/dom/shadow/shadow-dom-event-dispatching-expected.txt:
- fast/dom/shadow/shadow-dom-event-dispatching.html:
- 2:19 PM Changeset in webkit [120753] by
-
- 5 edits1 copy in trunk/Source/WebKit/chromium
IndexedDB: Add new type (and chromium webkit API) for metadata snapshot
https://bugs.webkit.org/show_bug.cgi?id=88678
Reviewed by Darin Fisher.
- WebKit.gyp:
- public/WebIDBDatabase.h:
(WebKit::WebIDBDatabase::metadata):
- public/WebIDBKeyPath.h:
(WebIDBKeyPath):
(WebKit::WebIDBKeyPath::WebIDBKeyPath):
(WebKit::WebIDBKeyPath::~WebIDBKeyPath):
(WebKit::WebIDBKeyPath::operator=): Used by Chromium backend.
- public/WebIDBMetadata.h: Added new type.
(WebKit):
(WebIDBMetadata):
(WebKit::WebIDBMetadata::WebIDBMetadata):
(ObjectStore):
(WebKit::WebIDBMetadata::ObjectStore::ObjectStore):
(Index):
(WebKit::WebIDBMetadata::Index::Index):
- src/WebIDBKeyPath.cpp: Conform to normal WebXXX patern.
(WebKit::WebIDBKeyPath::assign):
(WebKit):
(WebKit::WebIDBKeyPath::reset):
- 2:12 PM Changeset in webkit [120752] by
-
- 1 edit5 adds in trunk/LayoutTests
[Chromium] Add LayoutTests for find in page.
https://bugs.webkit.org/show_bug.cgi?id=89453
Patch by Martin Kosiba <mkosiba@google.com> on 2012-06-19
Reviewed by Adam Barth.
- platform/chromium/editing/text-iterator/findString-restarts-at-last-position-expected.txt: Added.
- platform/chromium/editing/text-iterator/findString-restarts-at-last-position.html: Added.
- platform/chromium/editing/text-iterator/findString-start-search-after-selection-expected.txt: Added.
- platform/chromium/editing/text-iterator/findString-start-search-after-selection.html: Added.
- 2:06 PM Changeset in webkit [120751] by
-
- 4 edits in trunk/Source/JavaScriptCore
CodeBlock::needsCallReturnIndices() is no longer needed.
https://bugs.webkit.org/show_bug.cgi?id=89490
Patch by Mark Lam <mark.lam@apple.com> on 2012-06-19
Reviewed by Geoffrey Garen.
- bytecode/CodeBlock.h:
(JSC::CodeBlock::needsCallReturnIndices): removed.
- dfg/DFGJITCompiler.cpp:
(JSC::DFG::JITCompiler::link):
- jit/JIT.cpp:
(JSC::JIT::privateCompile):
- 2:00 PM Changeset in webkit [120750] by
-
- 3 edits3 adds in trunk
Fix scrollbar layers being misplaced with a clipped owner layer
https://bugs.webkit.org/show_bug.cgi?id=89486
Reviewed by Simon Fraser.
Source/WebCore:
Scrollbar layers are positioned relative to their parent layer. If
that parent gets clipped, then that positioning becomes incorrect and
scrollbars get positioned relative to the clipped bounds instead of
the full bounds. Fix by not clipping in this case.
Test: compositing/overflow/scrollbars-with-clipped-owner.html
- rendering/RenderLayerBacking.cpp:
(WebCore::RenderLayerBacking::shouldClipCompositedBounds):
LayoutTests:
- compositing/overflow/scrollbars-with-clipped-owner-expected.png: Added.
- compositing/overflow/scrollbars-with-clipped-owner-expected.txt: Added.
- compositing/overflow/scrollbars-with-clipped-owner.html: Added.
- 1:13 PM Changeset in webkit [120749] by
-
- 5 edits2 adds in trunk
[Shadow][Editing] applying document.execCommand('bold') twice to elements having shadow insertion points causes a crash.
https://bugs.webkit.org/show_bug.cgi?id=88502
Reviewed by Ryosuke Niwa.
Source/WebCore:
When inserting or removing insertion points (<shadow> or <content>) into or from Shdaow DOM,
ElementShadow::invalidateDistribution() will be called immediately. It will remove all the renderers
of elements in the Shadow DOM. Since Node::rendererIsEditable() returns false when an renderer does not
exist, all the elements in the Shadow DOM will be considered as non-content-editable until recalculating
layout, though some of them may actually be content-editable.
Actually the current code does not recalculate layout inside editing command, so a disaster happens.
For example, performing an editing command quits before completing all the commands, because the command
thinks it is adding some elements to a non-content-editable element (but actually it's content-editable).
So we have to recalculate layout if necessary when checking an element is content-editable or not.
This can be achieved by using Node::isContentEditable() instead of Node::rendererIsEditable().
Test: editing/shadow/bold-twice-in-shadow.html
- editing/ApplyStyleCommand.cpp:
(WebCore::ApplyStyleCommand::removeInlineStyleFromElement):
- editing/InsertNodeBeforeCommand.cpp:
(WebCore::InsertNodeBeforeCommand::doApply):
(WebCore::InsertNodeBeforeCommand::doUnapply):
- editing/RemoveNodeCommand.cpp:
(WebCore::RemoveNodeCommand::doApply):
LayoutTests:
- editing/shadow/bold-twice-in-shadow-expected.txt: Added.
- editing/shadow/bold-twice-in-shadow.html: Added.
- 1:11 PM Changeset in webkit [120748] by
-
- 5 edits2 deletes in trunk/Source/WebCore
Unreviewed, rolling out r120744.
http://trac.webkit.org/changeset/120744
https://bugs.webkit.org/show_bug.cgi?id=89220
Broke build on Chromium Windows canary bots
- CMakeLists.txt:
- GNUmakefile.list.am:
- Target.pri:
- WebCore.gypi:
- platform/mock/DeviceMotionClientMock.cpp: Removed.
- platform/mock/DeviceMotionClientMock.h: Removed.
- 1:05 PM Changeset in webkit [120747] by
-
- 2 edits in trunk/Source/JavaScriptCore
Unreviewed, try to fix Windows build.
- JavaScriptCore.vcproj/JavaScriptCore/copy-files.cmd:
- 12:48 PM Changeset in webkit [120746] by
-
- 3 edits74 deletes in trunk/LayoutTests
Remove some failing layout tests results added in
http://trac.webkit.org/changeset/119285 and
http://trac.webkit.org/changeset/119295. Most of these tests pass
without the failing results. Only one of them passes with the failing
result; compositing/layer-creation/spanOverlapsCanvas.html. I left that
failing result in place.
Rubber-stamped by Alexey Proskuryakov.
- platform/win/compositing/geometry/flipped-writing-mode-expected.txt: Removed.
- platform/win/compositing/geometry/foreground-layer-expected.txt: Removed.
- platform/win/compositing/geometry/foreground-offset-change-expected.txt: Removed.
- platform/win/compositing/geometry/horizontal-scroll-composited-expected.txt: Removed.
- platform/win/compositing/geometry/layer-due-to-layer-children-deep-expected.txt: Removed.
- platform/win/compositing/geometry/layer-due-to-layer-children-deep-switch-expected.txt: Removed.
- platform/win/compositing/geometry/layer-due-to-layer-children-expected.txt: Removed.
- platform/win/compositing/geometry/layer-due-to-layer-children-switch-expected.txt: Removed.
- platform/win/compositing/geometry/limit-layer-bounds-clipping-ancestor-expected.txt: Removed.
- platform/win/compositing/geometry/limit-layer-bounds-opacity-transition-expected.txt: Removed.
- platform/win/compositing/geometry/limit-layer-bounds-overflow-repaint-expected.txt: Removed.
- platform/win/compositing/geometry/limit-layer-bounds-overflow-root-expected.txt: Removed.
- platform/win/compositing/geometry/limit-layer-bounds-positioned-expected.txt: Removed.
- platform/win/compositing/geometry/limit-layer-bounds-positioned-transition-expected.txt: Removed.
- platform/win/compositing/geometry/limit-layer-bounds-transformed-expected.txt: Removed.
- platform/win/compositing/geometry/limit-layer-bounds-transformed-overflow-expected.txt: Removed.
- platform/win/compositing/geometry/outline-change-expected.txt: Removed.
- platform/win/compositing/geometry/partial-layout-update-expected.txt: Removed.
- platform/win/compositing/geometry/preserve-3d-switching-expected.txt: Removed.
- platform/win/compositing/geometry/root-layer-update-expected.txt: Removed.
- platform/win/compositing/geometry/tall-page-composited-expected.txt: Removed.
- platform/win/compositing/geometry/transfrom-origin-on-zero-size-layer-expected.txt: Removed.
- platform/win/compositing/geometry/vertical-scroll-composited-expected.txt: Removed.
- platform/win/compositing/geometry/video-opacity-overlay-expected.txt: Removed.
- platform/win/compositing/iframes/become-composited-nested-iframes-expected.txt: Removed.
- platform/win/compositing/iframes/become-overlapped-iframe-expected.txt: Removed.
- platform/win/compositing/iframes/composited-iframe-alignment-expected.txt: Removed.
- platform/win/compositing/iframes/composited-iframe-scroll-expected.txt: Removed.
- platform/win/compositing/iframes/composited-parent-iframe-expected.txt: Removed.
- platform/win/compositing/iframes/connect-compositing-iframe-delayed-expected.txt: Removed.
- platform/win/compositing/iframes/connect-compositing-iframe-expected.txt: Removed.
- platform/win/compositing/iframes/connect-compositing-iframe2-expected.txt: Removed.
- platform/win/compositing/iframes/connect-compositing-iframe3-expected.txt: Removed.
- platform/win/compositing/iframes/enter-compositing-iframe-expected.txt: Removed.
- platform/win/compositing/iframes/iframe-content-flipping-expected.txt: Removed.
- platform/win/compositing/iframes/iframe-copy-on-scroll-expected.txt: Removed.
- platform/win/compositing/iframes/iframe-in-composited-layer-expected.txt: Removed.
- platform/win/compositing/iframes/iframe-resize-expected.txt: Removed.
- platform/win/compositing/iframes/iframe-size-from-zero-expected.txt: Removed.
- platform/win/compositing/iframes/invisible-iframe-expected.txt: Removed.
- platform/win/compositing/iframes/invisible-nested-iframe-expected.txt: Removed.
- platform/win/compositing/iframes/invisible-nested-iframe-hide-expected.txt: Removed.
- platform/win/compositing/iframes/invisible-nested-iframe-show-expected.txt:
- platform/win/compositing/iframes/layout-on-compositing-change-expected.txt: Removed.
- platform/win/compositing/iframes/nested-iframe-scrolling-expected.txt: Removed.
- platform/win/compositing/iframes/overlapped-iframe-expected.txt: Removed.
- platform/win/compositing/iframes/overlapped-iframe-iframe-expected.txt:
- platform/win/compositing/iframes/page-cache-layer-tree-expected.txt: Removed.
- platform/win/compositing/iframes/resizer-expected.txt: Removed.
- platform/win/compositing/iframes/scrolling-iframe-expected.txt: Removed.
- platform/win/compositing/images: Removed.
- platform/win/compositing/images/clip-on-directly-composited-image-expected.txt: Removed.
- platform/win/compositing/images/direct-image-background-color-expected.txt: Removed.
- platform/win/compositing/images/direct-pdf-image-expected.txt: Removed.
- platform/win/compositing/images/direct-svg-image-expected.txt: Removed.
- platform/win/compositing/layer-creation/fixed-position-out-of-view-expected.txt: Removed.
- platform/win/compositing/layer-creation/overflow-scroll-overlap-expected.txt: Removed.
- platform/win/compositing/layer-creation/overlap-animation-expected.txt: Removed.
- platform/win/compositing/layer-creation/overlap-child-layer-expected.txt: Removed.
- platform/win/compositing/layer-creation/overlap-clipping-expected.txt: Removed.
- platform/win/compositing/layer-creation/overlap-transformed-and-clipped-expected.txt: Removed.
- platform/win/compositing/layer-creation/overlap-transformed-layer-expected.txt: Removed.
- platform/win/compositing/layer-creation/overlap-transforms-expected.txt: Removed.
- platform/win/compositing/layer-creation/rotate3d-overlap-expected.txt: Removed.
- platform/win/compositing/layer-creation/scroll-partial-update-expected.txt: Removed.
- platform/win/compositing/layer-creation/stacking-context-overlap-expected.txt: Removed.
- platform/win/compositing/layer-creation/stacking-context-overlap-nested-expected.txt: Removed.
- platform/win/compositing/layer-creation/translatez-overlap-expected.txt: Removed.
- platform/win/compositing/masks: Removed.
- platform/win/compositing/masks/direct-image-mask-expected.txt: Removed.
- platform/win/compositing/masks/masked-ancestor-expected.txt: Removed.
- platform/win/compositing/masks/multiple-masks-expected.txt: Removed.
- platform/win/compositing/masks/simple-composited-mask-expected.txt: Removed.
- platform/win/compositing/overflow/ancestor-overflow-expected.txt: Removed.
- platform/win/compositing/overflow/clip-descendents-expected.txt: Removed.
- platform/win/compositing/overflow/content-gains-scrollbars-expected.txt: Removed.
- platform/win/compositing/overflow/content-loses-scrollbars-expected.txt: Removed.
- platform/win/compositing/overflow/overflow-compositing-descendant-expected.txt: Removed.
- platform/win/compositing/overflow/overflow-positioning-expected.txt: Removed.
- platform/win/compositing/overflow/overflow-scrollbar-layers-expected.txt: Removed.
- platform/win/compositing/overflow/resize-painting-expected.txt: Removed.
- platform/win/compositing/rtl: Removed.
- platform/win/compositing/rtl/rtl-absolute-expected.txt: Removed.
- platform/win/compositing/rtl/rtl-absolute-overflow-expected.txt: Removed.
- platform/win/compositing/rtl/rtl-absolute-overflow-scrolled-expected.txt: Removed.
- platform/win/compositing/rtl/rtl-iframe-absolute-expected.txt: Removed.
- platform/win/compositing/rtl/rtl-iframe-absolute-overflow-expected.txt: Removed.
- platform/win/compositing/rtl/rtl-iframe-absolute-overflow-scrolled-expected.txt: Removed.
- platform/win/compositing/rtl/rtl-iframe-fixed-expected.txt: Removed.
- platform/win/compositing/rtl/rtl-iframe-fixed-overflow-expected.txt: Removed.
- platform/win/compositing/rtl/rtl-iframe-fixed-overflow-scrolled-expected.txt: Removed.
- platform/win/compositing/rtl/rtl-iframe-relative-expected.txt: Removed.
- platform/win/compositing/rtl/rtl-relative-expected.txt: Removed.
- platform/win/compositing/tiling/crash-reparent-tiled-layer-expected.txt: Removed.
- platform/win/compositing/visibility: Removed.
- platform/win/compositing/visibility/layer-visible-content-expected.txt: Removed.
- platform/win/compositing/visibility/visibility-image-layers-dynamic-expected.txt: Removed.
- 12:42 PM Changeset in webkit [120745] by
-
- 16 edits22 adds in trunk/Source
It should be possible to look at disassembly
https://bugs.webkit.org/show_bug.cgi?id=89319
Source/JavaScriptCore:
Reviewed by Sam Weinig.
This imports the udis86 disassembler library. The library is placed
behind an abstraction in disassembler/Disassembler.h, so that we can
in the future use other disassemblers (for other platforms) whenever
appropriate. As a first step, the disassembler is being invoked for
DFG verbose dumps.
If we ever want to merge a new version of udis86 in the future, I've
made notes about changes I made to the library in
disassembler/udis86/differences.txt.
- CMakeLists.txt:
- DerivedSources.make:
- GNUmakefile.list.am:
- JavaScriptCore.pri:
- JavaScriptCore.vcproj/JavaScriptCore/JavaScriptCore.vcproj:
- JavaScriptCore.vcproj/JavaScriptCore/JavaScriptCoreCommon.vsprops:
- JavaScriptCore.xcodeproj/project.pbxproj:
- dfg/DFGJITCompiler.cpp:
(JSC::DFG::JITCompiler::compile):
(JSC::DFG::JITCompiler::compileFunction):
- disassembler: Added.
- disassembler/Disassembler.h: Added.
(JSC):
(JSC::tryToDisassemble):
- disassembler/UDis86Disassembler.cpp: Added.
(JSC):
(JSC::tryToDisassemble):
- disassembler/udis86: Added.
- disassembler/udis86/differences.txt: Added.
- disassembler/udis86/itab.py: Added.
(UdItabGenerator):
(UdItabGenerator.init):
(UdItabGenerator.toGroupId):
(UdItabGenerator.genLookupTable):
(UdItabGenerator.genLookupTableList):
(UdItabGenerator.genInsnTable):
(genItabH):
(genItabH.UD_ITAB_H):
(genItabC):
(genItab):
(main):
- disassembler/udis86/optable.xml: Added.
- disassembler/udis86/ud_opcode.py: Added.
(UdOpcodeTables):
(UdOpcodeTables.sizeOfTable):
(UdOpcodeTables.nameOfTable):
(UdOpcodeTables.updateTable):
(UdOpcodeTables.Insn):
(UdOpcodeTables.Insn.init):
(UdOpcodeTables.Insn.init.opcode):
(UdOpcodeTables.parse):
(UdOpcodeTables.addInsnDef):
(UdOpcodeTables.print_table):
(UdOpcodeTables.print_tree):
- disassembler/udis86/ud_optable.py: Added.
(UdOptableXmlParser):
(UdOptableXmlParser.parseDef):
(UdOptableXmlParser.parse):
(printFn):
(parse):
(main):
- disassembler/udis86/udis86.c: Added.
(ud_init):
(ud_disassemble):
(ud_set_mode):
(ud_set_vendor):
(ud_set_pc):
(ud):
(ud_insn_asm):
(ud_insn_off):
(ud_insn_hex):
(ud_insn_ptr):
(ud_insn_len):
- disassembler/udis86/udis86.h: Added.
- disassembler/udis86/udis86_decode.c: Added.
(eff_adr_mode):
(ud_lookup_mnemonic):
(decode_prefixes):
(modrm):
(resolve_operand_size):
(resolve_mnemonic):
(decode_a):
(decode_gpr):
(resolve_gpr64):
(resolve_gpr32):
(resolve_reg):
(decode_imm):
(decode_modrm_reg):
(decode_modrm_rm):
(decode_o):
(decode_operand):
(decode_operands):
(clear_insn):
(resolve_mode):
(gen_hex):
(decode_insn):
(decode_3dnow):
(decode_ssepfx):
(decode_ext):
(decode_opcode):
(ud_decode):
- disassembler/udis86/udis86_decode.h: Added.
(ud_itab_entry_operand):
(ud_itab_entry):
(ud_lookup_table_list_entry):
(sse_pfx_idx):
(mode_idx):
(modrm_mod_idx):
(vendor_idx):
(is_group_ptr):
(group_idx):
- disassembler/udis86/udis86_extern.h: Added.
- disassembler/udis86/udis86_input.c: Added.
(inp_buff_hook):
(inp_file_hook):
(ud):
(ud_set_user_opaque_data):
(ud_get_user_opaque_data):
(ud_set_input_buffer):
(ud_set_input_file):
(ud_input_skip):
(ud_input_end):
(ud_inp_next):
(ud_inp_back):
(ud_inp_peek):
(ud_inp_move):
(ud_inp_uint8):
(ud_inp_uint16):
(ud_inp_uint32):
(ud_inp_uint64):
- disassembler/udis86/udis86_input.h: Added.
- disassembler/udis86/udis86_itab_holder.c: Added.
- disassembler/udis86/udis86_syn-att.c: Added.
(opr_cast):
(gen_operand):
(ud_translate_att):
- disassembler/udis86/udis86_syn-intel.c: Added.
(opr_cast):
(gen_operand):
(ud_translate_intel):
- disassembler/udis86/udis86_syn.c: Added.
- disassembler/udis86/udis86_syn.h: Added.
(mkasm):
- disassembler/udis86/udis86_types.h: Added.
(ud_operand):
(ud):
- jit/JITCode.h:
(JITCode):
(JSC::JITCode::tryToDisassemble):
Source/WebCore:
Reviewed by Sam Weinig.
Just fixing EFL's build system now that JSC has a new directory.
- CMakeLists.txt:
Source/WTF:
Reviewed by Sam Weinig.
Made changes to Assertions.h to make it friendly to C code again.
Added ENABLE(DISASSEMBLER) and USE(UDIS86) logic to Platform.h.
- wtf/Assertions.h:
- wtf/Platform.h:
- 12:27 PM Changeset in webkit [120744] by
-
- 5 edits2 adds in trunk/Source/WebCore
Adds DeviceMotionClientMock
https://bugs.webkit.org/show_bug.cgi?id=89220
Patch by Amy Ousterhout <aousterh@chromium.org> on 2012-06-19
Reviewed by Steve Block.
Adds DeviceMotionClientMock in WebCore to enable testing of DeviceMotion once it is fully implemented in WebKit.
This addition was originally included in https://bugs.webkit.org/show_bug.cgi?id=89197 but was split into a separate patch for easier review.
No new tests because DeviceMotionClientMock is designed to enable future testing and cannot be tested in itself.
- CMakeLists.txt:
- GNUmakefile.list.am:
- Target.pri:
- WebCore.gypi:
- platform/mock/DeviceMotionClientMock.cpp: Added.
(WebCore):
(WebCore::DeviceMotionClientMock::DeviceMotionClientMock):
(WebCore::DeviceMotionClientMock::setController):
(WebCore::DeviceMotionClientMock::startUpdating):
(WebCore::DeviceMotionClientMock::stopUpdating):
(WebCore::DeviceMotionClientMock::setMotion):
(WebCore::DeviceMotionClientMock::timerFired):
- platform/mock/DeviceMotionClientMock.h: Added.
(WebCore):
(DeviceMotionClientMock):
- 12:22 PM Changeset in webkit [120743] by
-
- 2 edits in trunk/Tools
Fix regression introduced in r120646 where the svn revision isn't saved in results.json
Unreviewed, build fix.
It turns out that we store the svn revision two different ways
in the JSON, and I only caught one of them :(. This should fix
the other.
Also, this part of the code is poorly tested. I will investigate
adding at least a basic test for this in a separate fix.
- Scripts/webkitpy/layout_tests/controllers/manager.py:
(summarize_results):
- 12:17 PM Changeset in webkit [120742] by
-
- 14 edits2 adds in trunk/Source/JavaScriptCore
GCActivityCallback and IncrementalSweeper should share code
https://bugs.webkit.org/show_bug.cgi?id=89400
Reviewed by Geoffrey Garen.
A lot of functionality is duplicated between GCActivityCallback and IncrementalSweeper.
We should extract the common functionality out into a separate class that both of them
can inherit from. This refactoring will be an even greater boon when we add the ability
to shut these two agents down in a thread-safe fashion
- CMakeLists.txt:
- GNUmakefile.list.am:
- JavaScriptCore.vcproj/JavaScriptCore/JavaScriptCore.vcproj:
- JavaScriptCore.xcodeproj/project.pbxproj:
- Target.pri:
- heap/Heap.cpp:
(JSC::Heap::Heap): Move initialization down so that the JSGlobalData has a valid Heap when
we're initializing the GCActivityCallback and the IncrementalSweeper.
- heap/Heap.h:
(Heap):
- heap/HeapTimer.cpp: Added.
(JSC):
(JSC::HeapTimer::HeapTimer): Initialize the various base class data that
DefaultGCActivityCallback::commonConstructor() used to do.
(JSC::HeapTimer::~HeapTimer): Call to invalidate().
(JSC::HeapTimer::synchronize): Same functionality as the old DefaultGCActivityCallback::synchronize().
Virtual so that non-CF subclasses can override.
(JSC::HeapTimer::invalidate): Tears down the runloop timer to prevent any future firing.
(JSC::HeapTimer::timerDidFire): Callback to pass to the timer function. Casts and calls the virtual doWork().
- heap/HeapTimer.h: Added. This is the class that serves as the common base class for
both GCActivityCallback and IncrementalSweeper. It handles setting up and tearing down run loops and synchronizing
across threads for its subclasses.
(JSC):
(HeapTimer):
- heap/IncrementalSweeper.cpp: Changes to accomodate the extraction of common functionality
between IncrementalSweeper and GCActivityCallback into a common ancestor.
(JSC):
(JSC::IncrementalSweeper::doWork):
(JSC::IncrementalSweeper::IncrementalSweeper):
(JSC::IncrementalSweeper::cancelTimer):
(JSC::IncrementalSweeper::create):
- heap/IncrementalSweeper.h:
(IncrementalSweeper):
- runtime/GCActivityCallback.cpp:
(JSC::DefaultGCActivityCallback::DefaultGCActivityCallback):
(JSC::DefaultGCActivityCallback::doWork):
- runtime/GCActivityCallback.h:
(GCActivityCallback):
(JSC::GCActivityCallback::willCollect):
(JSC::GCActivityCallback::GCActivityCallback):
(JSC):
(DefaultGCActivityCallback): Remove the platform data struct. The platform data should be kept in
the class itself so as to be accessible by doWork(). Most of the platform data for CF is kept in
HeapTimer anyways, so we only need the m_delay field now.
- runtime/GCActivityCallbackBlackBerry.cpp:
(JSC):
(JSC::DefaultGCActivityCallback::DefaultGCActivityCallback):
(JSC::DefaultGCActivityCallback::doWork):
(JSC::DefaultGCActivityCallback::didAllocate):
- runtime/GCActivityCallbackCF.cpp:
(JSC):
(JSC::DefaultGCActivityCallback::DefaultGCActivityCallback):
(JSC::DefaultGCActivityCallback::doWork):
(JSC::DefaultGCActivityCallback::scheduleTimer):
(JSC::DefaultGCActivityCallback::cancelTimer):
(JSC::DefaultGCActivityCallback::didAllocate):
(JSC::DefaultGCActivityCallback::willCollect):
(JSC::DefaultGCActivityCallback::cancel):
- 12:09 PM Changeset in webkit [120741] by
-
- 2 edits in trunk/Source/WebCore
Fix link error with fillWithEmptyClients
https://bugs.webkit.org/show_bug.cgi?id=89484
Patch by Alexandre Elias <aelias@google.com> on 2012-06-19
Reviewed by Ryosuke Niwa.
fillWithEmptyClients() was within #if ENABLE() guard in the .cpp file
but not in the header. There is now a caller unrelated to these ENABLEs
in WebHelperPluginImpl, so linking fails if both SVG and PAGE_POPUP
are disabled.
This function looks generic, so just remove the ENABLE guards.
No new tests. (Build fix.)
- loader/EmptyClients.cpp:
(WebCore::fillWithEmptyClients):
- 11:27 AM Changeset in webkit [120740] by
-
- 5 edits in trunk/Tools
REGRESSION (NRWT): Results for new non-text-only tests are always put in the most-specific platform directory
https://bugs.webkit.org/show_bug.cgi?id=78127
Reviewed by Ryosuke Niwa.
Add the --add-platform-exceptions flag from ORWT, and ensure
that --new-baseline is equivalent to --reset-results
--add-platform-exceptions. Also fix the default logic for where to
put PNGs and render tree dumps if --new-test-results is true:
if --add-platform-exceptions is False, baselines should go in
the least-specific platform directory (e.g., platform/mac)
rather than the most-specific (platform/mac-snowleopard).
- Scripts/webkitpy/layout_tests/controllers/single_test_runner.py:
(SingleTestRunner):
(SingleTestRunner.run):
(SingleTestRunner._run_rebaseline):
(SingleTestRunner._add_missing_baselines):
(SingleTestRunner._location_for_new_baseline):
(SingleTestRunner._overwrite_baselines):
(SingleTestRunner._save_baseline_data):
- Scripts/webkitpy/layout_tests/port/base.py:
(Port.baseline_path):
(Port):
(Port.baseline_platform_dir):
(Port.baseline_version_dir):
- Scripts/webkitpy/layout_tests/run_webkit_tests.py:
(_set_up_derived_options):
(parse_args):
- Scripts/webkitpy/layout_tests/run_webkit_tests_integrationtest.py:
(RebaselineTest.assertBaselines):
(RebaselineTest.test_reset_results):
(RebaselineTest.test_missing_results):
(RebaselineTest.test_new_baseline):
- 11:25 AM Changeset in webkit [120739] by
-
- 8 edits in trunk/LayoutTests
Rebaseline some media controller tests.
- platform/mac/media/audio-controls-rendering-expected.txt:
- platform/mac/media/controls-after-reload-expected.txt:
- platform/mac/media/controls-strict-expected.txt:
- platform/mac/media/video-controls-rendering-expected.txt:
- platform/mac/media/video-display-toggle-expected.txt:
- platform/mac/media/video-no-audio-expected.txt:
- platform/mac/media/video-playing-and-pause-expected.txt:
- 11:21 AM Changeset in webkit [120738] by
-
- 9 edits in trunk/Tools
NRWT should not take memory used as disk cache into account when deciding how many processes to launch
https://bugs.webkit.org/show_bug.cgi?id=81379
Reviewed by Eric Seidel.
The 'free memory' calculation we were running on the mac seemed
to underestimate how many children we can run in parallel, and
it was complex. This patch replaces that calculation with a
simpler one that reserves 2GB for overhead and assumes 256MB per
DRT/WTR, so if we had 4GB of RAM we can run up to 8 DRTs.
Also, there was a bug where we were truncating the memory
installed on the machine to 4GB by casting to an int instead of
a long; this was probably the source of some of the earlier
problems when using total memory.
This patch also removes the no-longer-needed restrictions on the
number of workers on beefy Lion boxes for both Apple Mac and Chromium
Mac; we should now use all of the cores by default.
The memory calculations have only been implemented on the mac;
having the calculation in base.default_child_processes() was IMO
misleading, and so this patch also moves the computation into
the MacPort. I have not heard of the # of workers being an issue
on any other ports, so this should be fine.
- Scripts/webkitpy/common/system/platforminfo.py:
(PlatformInfo.total_bytes_memory):
- Scripts/webkitpy/common/system/platforminfo_mock.py:
(PlatformInfo.total_bytes_memory):
(PlatformInfo.total_bytes_memory):
(PlatformInfo._win_version_tuple_from_cmd):
- Scripts/webkitpy/common/system/platforminfo_unittest.py:
(TestPlatformInfo.test_total_bytes_memory):
- Scripts/webkitpy/layout_tests/port/base.py:
(Port.default_child_processes):
- Scripts/webkitpy/layout_tests/port/base_unittest.py:
(PortTest.test_default_child_processes):
- Scripts/webkitpy/layout_tests/port/chromium_mac.py:
(ChromiumMacPort.operating_system):
- Scripts/webkitpy/layout_tests/port/mac.py:
(MacPort.default_child_processes):
- Scripts/webkitpy/layout_tests/port/mac_unittest.py:
(TestMacPort.test_default_child_processes):
- 11:05 AM Changeset in webkit [120737] by
-
- 3 edits2 adds in trunk
Crash in RenderInline::willBeDestroyed.
https://bugs.webkit.org/show_bug.cgi?id=89386
Reviewed by Julien Chaffraix.
Source/WebCore:
We were unable to find the previous continuation during the process
of cleaning up empty anonymous block continuation. The reason being
that we were just looking till our immediate parent and ignoring the
fact that it can be enclosed in anonymous blocks. The change skips
over anonymous blocks in the parent chain.
Test: fullscreen/empty-anonymous-block-continuation-crash.html
- rendering/RenderBlock.cpp:
(WebCore::RenderBlock::removeChild):
LayoutTests:
- fullscreen/empty-anonymous-block-continuation-crash-expected.txt: Added.
- fullscreen/empty-anonymous-block-continuation-crash.html: Added.
- 11:03 AM Changeset in webkit [120736] by
-
- 5 edits in trunk
new-run-webkit-tests reports unexpected pass of pixel tests when pixel testing is disabled
https://bugs.webkit.org/show_bug.cgi?id=85446
Reviewed by Simon Fraser.
Tools:
Embed whether pixel testing was enabled into the results.json.
- Scripts/webkitpy/layout_tests/controllers/manager.py:
(summarize_results):
LayoutTests:
This change makes results.html aware of whether pixel tests were
enabled when the tests were run; if they weren't, we ignore
expected IMAGE tests that pass instead of reporting them as
'unexpected pass'.
- fast/harness/resources/results-test.js:
(mockResults):
- fast/harness/results.html:
- 10:59 AM Changeset in webkit [120735] by
-
- 26 edits in trunk
Remove the -webkit-match-nearest-mail-blockquote-color property.
<http://webkit.org/b/89480>
Reviewed by Antti Koivisto.
Source/WebCore:
This property does nothing except taking up a bit on StyleRareNonInheritedData.
It's apparently famous for being the longest property name we support.
This passes the torch to -webkit-border-bottom-right-radius.
- css/CSSComputedStyleDeclaration.cpp:
(WebCore::CSSComputedStyleDeclaration::getPropertyCSSValue):
- css/CSSParser.cpp:
(WebCore::isValidKeywordPropertyAndValue):
(WebCore::isKeywordPropertyID):
(WebCore::CSSParser::parseValue):
- css/CSSPrimitiveValueMappings.h:
- css/CSSProperty.cpp:
(WebCore::CSSProperty::isInheritedProperty):
- css/CSSPropertyNames.in:
- css/CSSValueKeywords.in:
- css/StyleBuilder.cpp:
(WebCore::StyleBuilder::StyleBuilder):
- css/StyleResolver.cpp:
(WebCore::StyleResolver::collectMatchingRulesForList):
- rendering/style/RenderStyle.h:
- rendering/style/RenderStyleConstants.h:
- rendering/style/StyleRareNonInheritedData.cpp:
(WebCore::StyleRareNonInheritedData::StyleRareNonInheritedData):
(WebCore::StyleRareNonInheritedData::operator==):
- rendering/style/StyleRareNonInheritedData.h:
(StyleRareNonInheritedData):
LayoutTests:
- fast/css/getComputedStyle/resources/property-names.js:
- fast/dom/CSSStyleDeclaration/access-longest-css-property-expected.txt:
- fast/dom/CSSStyleDeclaration/access-longest-css-property.html:
- platform/chromium-mac/fast/css/getComputedStyle/computed-style-without-renderer-expected.txt:
- platform/chromium-win/fast/css/getComputedStyle/computed-style-expected.txt:
- platform/chromium-win/fast/css/getComputedStyle/computed-style-without-renderer-expected.txt:
- platform/gtk/fast/css/getComputedStyle/computed-style-expected.txt:
- platform/mac/fast/css/getComputedStyle/computed-style-expected.txt:
- platform/mac/fast/css/getComputedStyle/computed-style-without-renderer-expected.txt:
- platform/qt/fast/css/getComputedStyle/computed-style-without-renderer-expected.txt:
- platform/win/fast/css/getComputedStyle/computed-style-expected.txt:
- platform/win/fast/css/getComputedStyle/computed-style-without-renderer-expected.txt:
- 10:57 AM Changeset in webkit [120734] by
-
- 4 edits in trunk/Source/WebCore
[BlackBerry] Animations continue to request frames after they have finished
https://bugs.webkit.org/show_bug.cgi?id=89483
Patch by Andrew Lo <anlo@rim.com> on 2012-06-19
Reviewed by Rob Buis.
Don't report that we have running animations if all the animations
are finished. This prevents unnecessary requesting of animation frames.
Internal PR165119
Internally Reviewed by Arvid Nilsson.
- platform/graphics/blackberry/LayerAnimation.cpp:
(WebCore::progress):
(WebCore::fetchIntervalEndpoints):
(WebCore::LayerAnimation::apply):
- platform/graphics/blackberry/LayerAnimation.h:
(WebCore::LayerAnimation::finished):
(WebCore::LayerAnimation::LayerAnimation):
(LayerAnimation):
- platform/graphics/blackberry/LayerCompositingThread.cpp:
(WebCore::LayerCompositingThread::updateAnimations):
- 10:55 AM Changeset in webkit [120733] by
-
- 2 edits in branches/safari-536-branch/LayoutTests
Merged r120642
- 10:55 AM Changeset in webkit [120732] by
-
- 2 edits in trunk/Source/WebCore
Wrong repaintContainerSkipped in RenderObject::container()
when positioned objects are enclosed in a <foreignObject>.
https://bugs.webkit.org/show_bug.cgi?id=89384
Reviewed by Simon Fraser.
No new tests. The bug was found in a code audit.
- rendering/RenderObject.cpp:
(WebCore::RenderObject::container):
- 10:43 AM Changeset in webkit [120731] by
-
- 3 edits2 adds in trunk
Crash in WebCore::RenderSVGModelObject::checkIntersection
https://bugs.webkit.org/show_bug.cgi?id=89059
Reviewed by Rob Buis.
Source/WebCore:
getElementCTM updates layout causing the renderer to be destroyed. We get
the new renderer by storing the element pointer and later accessing it using
the element pointer.
Test: svg/custom/intersection-list-crash.svg
- rendering/svg/RenderSVGModelObject.cpp:
(WebCore::RenderSVGModelObject::checkIntersection):
(WebCore::RenderSVGModelObject::checkEnclosure):
LayoutTests:
- svg/custom/intersection-list-crash-expected.txt: Added.
- svg/custom/intersection-list-crash.svg: Added.
- 10:38 AM Changeset in webkit [120730] by
-
- 1 add in branches/chromium/1180/codereview.settings
adding for easy drover
- 10:37 AM Changeset in webkit [120729] by
-
- 1 copy in branches/chromium/1180
creating branch for m21
- 10:25 AM Changeset in webkit [120728] by
-
- 3 edits in trunk/Source/WebCore
FileWriter progress events should be throttled to 50ms
https://bugs.webkit.org/show_bug.cgi?id=89022
Reviewed by Kent Tamura.
No new tests; existing tests validate our event sequences, and testing
timing is likely to be flaky.
- Modules/filesystem/FileWriter.cpp:
(WebCore):
(WebCore::FileWriter::FileWriter):
(WebCore::FileWriter::didWrite): Throttle sending progress events.
- Modules/filesystem/FileWriter.h:
(FileWriter): Add a variable that remembers when the last event was
sent.
- 10:10 AM Changeset in webkit [120727] by
-
- 21 edits in trunk
Source/WebCore: WebCore part of: Paginated display API doesn’t allow setting the page progression direction
https://bugs.webkit.org/show_bug.cgi?id=89394
Reviewed by Darin Adler.
Updated fast/multicol/pagination-*.html.
- css/StyleResolver.cpp:
(WebCore::StyleResolver::styleForDocument): Changed to handle the new Pagination::Mode enum
values and set the column progression in addition to the column axis.
- page/Page.h: Changed the Pagination::Mode enum values to ones that specify an axis and a
direction along that axis.
- testing/Internals.cpp:
(WebCore::Internals::setPagination): Changed to support the new Pagination::Mode enum values.
Source/WebKit/mac: WebKit/mac part of: Paginated display API doesn’t allow setting the page progression direction
https://bugs.webkit.org/show_bug.cgi?id=89394
Reviewed by Darin Adler.
- WebView/WebView.mm:
(-[WebView _setPaginationMode:]): Changed to map new WebPaginationMode values to new
Pagination::Mode values.
(-[WebView _paginationMode]): Similarly, in the opposite direction.
- WebView/WebViewPrivate.h: Replaced WebPaginationMode values with ones that specify an
axis and a direction along that axis.
Source/WebKit2: WebKit2 part of: Paginated display API doesn’t allow setting the page progression direction
https://bugs.webkit.org/show_bug.cgi?id=89394
Reviewed by Darin Adler.
- UIProcess/API/C/WKPage.cpp:
(WKPageSetPaginationMode): Changed to map new WKPaginationMode values to new
Pagination::Mode values.
(WKPageGetPaginationMode): Similarly, in the opposite direction.
- UIProcess/API/C/WKPagePrivate.h: Replaced WKPaginationMode values with ones that specify
an axis and a direction along that axis.
- UIProcess/API/mac/WKBrowsingContextController.mm:
(-[WKBrowsingContextController setPaginationMode:]): Changed to handle new values.
(-[WKBrowsingContextController paginationMode]): Ditto.
- UIProcess/API/mac/WKBrowsingContextControllerPrivate.h: Replaced
WKBrowsingContextPaginationMode values with ones that specify an axis and a direction along
that axis.
LayoutTests: Paginated display API doesn’t allow setting the page progression direction
https://bugs.webkit.org/show_bug.cgi?id=89394
Reviewed by Darin Adler.
- fast/multicol/pagination-h-horizontal-bt.html:
- fast/multicol/pagination-h-horizontal-tb.html:
- fast/multicol/pagination-h-vertical-lr.html:
- fast/multicol/pagination-h-vertical-rl.html:
- fast/multicol/pagination-v-horizontal-bt.html:
- fast/multicol/pagination-v-horizontal-tb.html:
- fast/multicol/pagination-v-vertical-lr.html:
- fast/multicol/pagination-v-vertical-rl.html:
- 10:09 AM Changeset in webkit [120726] by
-
- 2 edits in trunk/Source/WebKit/mac
[Mac] -[WebPDFView _menuItemsFromPDFKitForEvent:] leaks NSMenuItems
https://bugs.webkit.org/show_bug.cgi?id=89434
Reviewed by Darin Adler.
- WebView/WebPDFView.mm:
(-[WebPDFView _menuItemsFromPDFKitForEvent:]): Release copied
NSMenuItems once they're stored in copiedItems.
- 9:37 AM Changeset in webkit [120725] by
-
- 2 edits in trunk/LayoutTests
[Qt][Mac] Skip new failing and crashing tests to try to paint the bot green.
- platform/qt-mac/Skipped:
- 9:37 AM Changeset in webkit [120724] by
-
- 1 edit2 copies in branches/chromium/1132
Merge 118297 - REGRESSION (115573): Incorrect rounding of margins for floats
https://bugs.webkit.org/show_bug.cgi?id=87319
Reviewed by Eric Seidel.
Source/WebCore:
In RenderBlock::computeInlinePreferredLogicalWidths we used a float to
accumulate margins for floating children while the children themselves
represent their margins as LayoutUnits. Due to lack of rounding this can
cause the block to be too small at certain certain zoom levels, causing
unwanted wrapping.
This patch changes computeInlinePreferredLogicalWidths to use a
LayoutUnit to accumulate the margins and thus ensures that the margin
values are rounded the same way.
Test: fast/block/float/floats-with-margin-should-not-wrap.html
- rendering/RenderBlock.cpp:
(WebCore::RenderBlock::computeInlinePreferredLogicalWidths):
LayoutTests:
Add new test ensuring that using margins with floats doesn't cause
undesired wrapping.
- fast/block/float/floats-with-margin-should-not-wrap-expected.html: Added.
- fast/block/float/floats-with-margin-should-not-wrap.html: Added.
- 9:21 AM Changeset in webkit [120723] by
-
- 1 edit27 deletes in trunk/LayoutTests
[Qt] Unreviewed gardening, remove unnecessary expected files.
- platform/qt/animations/combo-transform-translate+scale-expected.txt: Removed.
- platform/qt/compositing/geometry/object-clip-rects-assertion-expected.txt: Removed.
- platform/qt/css3/calc/getComputedStyle-margin-percentage-expected.txt: Removed.
- platform/qt/css3/filters/remove-filter-rendering-expected.txt: Removed.
- platform/qt/editing/inserting/return-key-in-hidden-field-expected.txt: Removed.
- platform/qt/editing/pasteboard/paste-4039777-fix-expected.txt: Removed.
- platform/qt/editing/pasteboard/paste-noscript-expected.txt: Removed.
- platform/qt/editing/pasteboard/select-element-1-expected.txt: Removed.
- platform/qt/editing/selection/select-line-break-with-opposite-directionality-expected.txt: Removed.
- platform/qt/fast/css/border-solid-single-edge-antialias-expected.txt: Removed.
- platform/qt/fast/dom/Window/window-resize-contents-expected.txt: Removed.
- platform/qt/fast/events/mouse-relative-position-expected.txt: Removed.
- platform/qt/fast/preloader/document-write-noscript-expected.txt: Removed.
- platform/qt/fast/repaint/table-hover-on-link-expected.txt: Removed.
- platform/qt/http/tests/misc/SVGFont-delayed-load-expected.txt: Removed.
- platform/qt/http/tests/security/xssAuditor/script-tag-with-fancy-unicode-expected.txt: Removed.
- platform/qt/svg/custom/acid3-test-77-expected.txt: Removed.
- platform/qt/svg/custom/circular-marker-reference-1-expected.txt: Removed.
- platform/qt/svg/custom/glyph-setting-d-attribute-expected.txt: Removed.
- platform/qt/svg/custom/insertItemBefore-from-non-list-origin-expected.txt: Removed.
- platform/qt/svg/custom/use-transfer-width-height-properties-to-svg-expected.txt: Removed.
- platform/qt/svg/custom/use-transfer-width-height-properties-to-symbol2-expected.txt: Removed.
- platform/qt/svg/dom/baseVal-animVal-crash-expected.txt: Removed.
- platform/qt/svg/dom/css-transforms-expected.txt: Removed.
- platform/qt/svg/dom/font-face-elements-expected.txt: Removed.
- platform/qt/svg/hixie/links/001-expected.txt: Removed.
- platform/qt/tables/mozilla_expected_failures/bugs/bug3517-expected.txt: Removed.
- 9:15 AM Changeset in webkit [120722] by
-
- 1 edit2 deletes in trunk/LayoutTests
[Qt] Unreviewed gardening, remove unnecessary expected files.
- platform/qt-5.0/fast/dom/DeviceMotion/create-event-expected.txt: Removed.
- platform/qt-5.0/fast/dom/DeviceOrientation/create-event-expected.txt: Removed.
- 9:12 AM WebKitGTK/1.8.x edited by
- (diff)
- 9:07 AM Changeset in webkit [120721] by
-
- 3 edits in trunk/Source/WebCore
Make RenderTableSection - nodeAtPoint and paintObject reuse spanning logic.
https://bugs.webkit.org/show_bug.cgi?id=88066
Patch by Allan Sandfeld Jensen <allan.jensen@nokia.com> on 2012-06-19
Reviewed by Julien Chaffraix.
Makes dirtiedRows and dirtiedColumns reuse spannedRows and spannedColumns
internally. This also changes their API to accept a damage-rect flipped
by logicalRectForWritingModeAndDirection.
The generic code in spannedRows and spannedColumns solves a few FIXMEs
surrounding RTL tables, which should improve performance.
- rendering/RenderTableSection.cpp:
(WebCore::RenderTableSection::dirtiedRows):
(WebCore::RenderTableSection::dirtiedColumns):
(WebCore::RenderTableSection::paintObject):
- rendering/RenderTableSection.h:
(WebCore::CellSpan::start):
(WebCore::CellSpan::end):
(CellSpan):
(RenderTableSection):
- 8:59 AM Changeset in webkit [120720] by
-
- 2 edits in trunk/LayoutTests
[Qt] Unreviewed gardening, skip one more failing test.
- platform/qt-5.0-wk2/Skipped:
- 8:47 AM Changeset in webkit [120719] by
-
- 3 edits in trunk/Source/WebCore
[JSC] Implement support for JSDictionary::convertValue(Vector<String>& result)
https://bugs.webkit.org/show_bug.cgi?id=89447
Patch by Christophe Dumez <Christophe Dumez> on 2012-06-19
Reviewed by Adam Barth.
Add implementation for JSDictionary::convertValue(Vector<String>& result)
so that IDBDatabase::createObjectStore() compiles with JSC.
Tests: storage/indexeddb
- bindings/js/JSDictionary.cpp:
(WebCore::JSDictionary::convertValue):
(WebCore):
- bindings/js/JSDictionary.h:
- 8:45 AM Changeset in webkit [120718] by
-
- 2 edits in trunk/Source/WebCore
[Mac] AudioFileReader::createBus() leaks an AudioBufferList if ExtAudioFileRead() returns an error
https://bugs.webkit.org/show_bug.cgi?id=89429
Reviewed by Darin Adler.
- platform/audio/mac/AudioFileReaderMac.cpp:
(WebCore::AudioFileReader::createBus): Call destroyAudioBufferList()
before returning if ExtAudioFileRead() returns an error.
- 8:40 AM Changeset in webkit [120717] by
-
- 5 edits in trunk/LayoutTests
Fixing Typos in DeviceOrientation and DeviceMotion Tests
https://bugs.webkit.org/show_bug.cgi?id=89339
Patch by Amy Ousterhout <aousterh@chromium.org> on 2012-06-19
Reviewed by Steve Block.
- fast/dom/DeviceMotion/create-event-expected.txt:
- fast/dom/DeviceMotion/script-tests/create-event.js:
- fast/dom/DeviceOrientation/no-synchronous-events-expected.txt:
- fast/dom/DeviceOrientation/script-tests/no-synchronous-events.js:
- 8:38 AM Changeset in webkit [120716] by
-
- 2 edits in trunk/Source/WebKit/chromium
[Chromium] Style Fix in WebDeviceOrientationController
https://bugs.webkit.org/show_bug.cgi?id=89337
Patch by Amy Ousterhout <aousterh@chromium.org> on 2012-06-19
Reviewed by Steve Block.
Changing a local variable from a PassRefPtr to a RefPtr in order to be consistent with the style guide.
- src/WebDeviceOrientationController.cpp:
(WebKit::WebDeviceOrientationController::didChangeDeviceOrientation):
- 8:17 AM Changeset in webkit [120715] by
-
- 2 edits in trunk/LayoutTests
[Qt] Unreviewed gardening, skip one more failing test.
- platform/qt-5.0-wk2/Skipped:
- 8:17 AM Changeset in webkit [120714] by
-
- 2 edits in trunk/Source/WebCore
Fix the GTK+ build by adding some missing source files to the source list.
- GNUmakefile.list.am: WebGLDepthTexture.cpp/h to the source list.
- 8:15 AM Changeset in webkit [120713] by
-
- 2 edits in trunk
[Qt] Fix the Windows build when Qt is built without -release or -debug
https://bugs.webkit.org/show_bug.cgi?id=89471
Reviewed by Tor Arne Vestbø.
Make sure that we at least remove build_all on Windows for api.pri,
as WebCore wasn't built for all configurations.
- Source/api.pri:
- 7:25 AM Changeset in webkit [120712] by
-
- 2 edits in trunk/LayoutTests
[Qt] Unreviewed typo fix.
- platform/qt/Skipped:
- 7:15 AM Changeset in webkit [120711] by
-
- 13 edits in trunk/Source
Remove the function ScriptDebugServer::supportsNativeBreakpoints()
https://bugs.webkit.org/show_bug.cgi?id=88759
Patch by Peter Wang <peter.wang@torchmobile.com.cn> on 2012-06-19
Reviewed by Pavel Feldman.
Source/WebCore:
Since both JSC and V8 support DOM breakpoint now, remove the conditional interface and related code.
So there is no new tests.
- bindings/js/ScriptDebugServer.h:
- bindings/v8/ScriptDebugServer.h:
- inspector/Inspector.json:
- inspector/InspectorDebuggerAgent.cpp:
- inspector/InspectorDebuggerAgent.h:
(InspectorDebuggerAgent):
- inspector/front-end/ElementsPanel.js:
(WebInspector.ElementsPanel):
(WebInspector.ElementsPanel.prototype.wasShown):
(WebInspector.ElementsPanel.prototype._populateContextMenu):
- inspector/front-end/NetworkPanel.js:
(WebInspector.NetworkLogView.prototype._createTable):
(WebInspector.NetworkLogView.prototype.switchToDetailedView):
(WebInspector.NetworkLogView.prototype.switchToBriefView):
(WebInspector.NetworkDataGridNode.prototype.createCells):
(WebInspector.NetworkDataGridNode.prototype.refreshRequest):
- inspector/front-end/ScriptsPanel.js:
(WebInspector.ScriptsPanel.prototype.wasShown):
(WebInspector.ScriptsPanel.prototype._clearInterface):
- inspector/front-end/Settings.js:
- inspector/front-end/inspector.js:
(WebInspector.doLoadedDone):
Source/WebKit/chromium:
- src/js/DevTools.js:
- 7:06 AM Changeset in webkit [120710] by
-
- 3 edits in trunk/Source/WebCore
Web Inspector: Make text in WebSocket Frames tab selectable
https://bugs.webkit.org/show_bug.cgi?id=89307
Patch by Nikita Vasilyev <me@elv1s.ru> on 2012-06-19
Reviewed by Yury Semikhatsky.
- inspector/front-end/ResourceWebSocketFrameView.js:
(WebInspector.ResourceWebSocketFrameView):
- inspector/front-end/networkPanel.css:
(.resource-websocket-frame-view):
- 7:03 AM Changeset in webkit [120709] by
-
- 19 edits in trunk/Source/WebCore
Web Inspector: Support 'Restart frame' in protocol and backend
https://bugs.webkit.org/show_bug.cgi?id=89147
Patch by Peter Rybin <peter.rybin@gmail.com> on 2012-06-19
Reviewed by Yury Semikhatsky.
Inspector.json got new command in Debugger domain.
A call path from InspectorDebuggerAgent to V8 Debug.LiveEdit.RestartFrame API
is built.
JSC implementation is a stub with fix me (40300).
- bindings/js/JSJavaScriptCallFrameCustom.cpp:
(WebCore::JSJavaScriptCallFrame::restart):
(WebCore):
- bindings/js/ScriptDebugServer.cpp:
(WebCore):
(WebCore::ScriptDebugServer::updateCallStack):
- bindings/js/ScriptDebugServer.h:
(ScriptDebugServer):
- bindings/v8/DebuggerScript.js:
(.):
- bindings/v8/JavaScriptCallFrame.cpp:
(WebCore::JavaScriptCallFrame::restart):
(WebCore):
- bindings/v8/JavaScriptCallFrame.h:
(JavaScriptCallFrame):
- bindings/v8/ScriptDebugServer.cpp:
(WebCore):
(WebCore::ScriptDebugServer::updateCallStack):
- bindings/v8/ScriptDebugServer.h:
- bindings/v8/ScriptFunctionCall.cpp:
(WebCore::ScriptFunctionCall::call):
- bindings/v8/custom/V8JavaScriptCallFrameCustom.cpp:
(WebCore::V8JavaScriptCallFrame::restartCallback):
(WebCore):
- inspector/InjectedScript.cpp:
(WebCore::InjectedScript::restartFrame):
(WebCore):
- inspector/InjectedScript.h:
(InjectedScript):
- inspector/InjectedScriptExterns.js:
(JavaScriptCallFrame.prototype.restart):
- inspector/InjectedScriptSource.js:
(.):
- inspector/Inspector.json:
- inspector/InspectorDebuggerAgent.cpp:
(WebCore::InspectorDebuggerAgent::restartFrame):
- inspector/InspectorDebuggerAgent.h:
(InspectorDebuggerAgent):
- inspector/JavaScriptCallFrame.idl:
- 7:03 AM Changeset in webkit [120708] by
-
- 4 edits1 copy in trunk/LayoutTests
New inspector/styles/inject-stylesheet.html fails
https://bugs.webkit.org/show_bug.cgi?id=89364
Reviewed by Zoltan Herczeg.
LayoutTestController in Chromium injects testRunner into the Inspector window, while its other implementations don't.
Additionally, testRunner.addUserStyleSheet() behavior differs across platforms: Chromium injects stylesheets
with the "Author" level, while the others do with the "User" level, so the Chromium expectation stands apart.
- inspector/styles/inject-stylesheet-expected.txt:
- inspector/styles/inject-stylesheet.html:
- platform/chromium/inspector/styles/inject-stylesheet-expected.txt: Copied from LayoutTests/inspector/styles/inject-stylesheet-expected.txt.
- platform/qt/Skipped:
- 6:59 AM Changeset in webkit [120707] by
-
- 3 edits in trunk/Source/WebCore
Replaced printf with fprintf in DEBUG_AUDIONODE_REFERENCES
Change printf to fprintf in DEBUG_AUDIONODE_REFERENCES
https://bugs.webkit.org/show_bug.cgi?id=86564
Patch by Andrei Poenaru <poenaru@adobe.com> on 2012-06-19
Reviewed by Csaba Osztrogonác.
There is no need for new tests: writing data to stderr, instead of stdout.
- Modules/webaudio/AudioContext.cpp:
(WebCore::AudioContext::~AudioContext):
- Modules/webaudio/AudioNode.cpp:
(WebCore::AudioNode::~AudioNode):
(WebCore::AudioNode::ref):
(WebCore::AudioNode::finishDeref):
(WebCore::AudioNode::printNodeCounts):
- 5:59 AM Changeset in webkit [120706] by
-
- 5 edits in trunk
[GTK] Assert failure in svg/batik/text/textDecoration.svg,textEffect3.svg for 64bit Debug bot
https://bugs.webkit.org/show_bug.cgi?id=89104
Patch by Dominik Röttsches <dominik.rottsches@intel.com> on 2012-06-19
Reviewed by Nikolas Zimmermann.
Source/WebCore:
Adding SVGInline to assertion to fix assertion failure. Same assertion was hit on Chromium.
Covered by existing tests
svg/batik/text/textDecoration.svg
svg/batik/text/textEffect3.svg
where the assertion failure was hit.
- rendering/svg/RenderSVGResourceContainer.cpp:
(WebCore::RenderSVGResourceContainer::shouldTransformOnTextPainting):
LayoutTests:
Removing CRASH expectations after assertion failure is now fixed.
- platform/chromium/TestExpectations:
- platform/gtk/TestExpectations:
- 4:43 AM Changeset in webkit [120705] by
-
- 3 edits in trunk/Source/WebKit2
[Qt] Create an extra atlas when there is no available buffer space.
https://bugs.webkit.org/show_bug.cgi?id=89012
Reviewed by Noam Rosenthal.
In case of the desktop mode when we scroll the view all the tiles
are updated. It can happen that there no available buffer space for
the currently used atlas(especially when scrolling fast). In that case
we can create a new one. One future improvement could be to make a timer and
delete this extra atlas when it is not used.
- WebProcess/WebPage/qt/LayerTreeHostQt.cpp:
(WebKit::beginContentUpdateInAtlas):
(WebKit::LayerTreeHostQt::beginContentUpdate):
- WebProcess/WebPage/qt/LayerTreeHostQt.h:
(LayerTreeHostQt):
- 4:34 AM Changeset in webkit [120704] by
-
- 14 edits in trunk/Source
[Qt] Buildfix for newer Qt5, use QPointer instead of the deprecated QWeakPointer.
https://bugs.webkit.org/show_bug.cgi?id=89449
Reviewed by Simon Hausmann.
Source/WebCore:
- bridge/qt/qt_instance.h:
(QtInstance):
- bridge/qt/qt_runtime.h:
(QtField):
- platform/Widget.h:
(Widget):
Source/WebKit/qt:
- Api/qwebframe_p.h:
(QWebHitTestResultPrivate):
- Api/qwebpage_p.h:
(QWebPagePrivate):
- WebCoreSupport/FrameLoaderClientQt.cpp:
(WebCore::FrameLoaderClientQt::createFrame):
- WebCoreSupport/PageClientQt.h:
(PageClientQGraphicsWidget):
Source/WebKit2:
- UIProcess/API/qt/qquicknetworkreply_p.h:
- 3:46 AM FeatureFlags edited by
- Add CSP_NEXT (diff)
- 3:44 AM Changeset in webkit [120703] by
-
- 2 edits in trunk/Source/WebKit/qt
[Qt] Document the versions enums where added
https://bugs.webkit.org/show_bug.cgi?id=89452
Patch by Albert Astals Cid <albert.astals@canonical.com> on 2012-06-19
Reviewed by Simon Hausmann.
- Api/qwebpage.cpp:
- 3:38 AM Changeset in webkit [120702] by
-
- 7 edits2 adds in trunk
Web Inspector: FileSystem tree should hide uninitialized FileSystem
https://bugs.webkit.org/show_bug.cgi?id=88602
Patch by Taiju Tsuiki <tzik@chromium.org> on 2012-06-19
Reviewed by Vsevolod Vlasov.
Source/WebCore:
Test: http/tests/inspector/filesystem/get-filesystem-root.html
- inspector/Inspector.json:
- inspector/InspectorFileSystemAgent.cpp:
(WebCore):
(WebCore::InspectorFileSystemAgent::getFileSystemRoot):
- inspector/InspectorFileSystemAgent.h:
(InspectorFileSystemAgent):
- inspector/front-end/FileSystemModel.js:
(WebInspector.FileSystemModel.prototype._originAdded):
(WebInspector.FileSystemModel.prototype.refreshFileSystemList):
(WebInspector.FileSystemModel.prototype._gotFileSystem):
(WebInspector.FileSystemModel.FileSystem):
(WebInspector.FileSystemRequestManager):
(WebInspector.FileSystemRequestManager.prototype.getFileSystemRoot):
(WebInspector.FileSystemRequestManager.prototype._gotFileSystemRoot):
(WebInspector.FileSystemDispatcher.prototype.gotFileSystemRoot):
- inspector/front-end/ResourcesPanel.js:
(WebInspector.FileSystemListTreeElement.prototype._fileSystemRemoved):
LayoutTests:
- http/tests/inspector/filesystem/get-filesystem-root-expected.txt: Added.
- http/tests/inspector/filesystem/get-filesystem-root.html: Added.
- 3:27 AM Changeset in webkit [120701] by
-
- 6 edits in trunk/Source/WebKit/chromium
[Chromium] Web Inspector: remove unnecessary indirection from message loop instrumentation
https://bugs.webkit.org/show_bug.cgi?id=89443
Removed start/stopMainThreadMonitoring methods from WebDevToolsAgentClient
and WebDevToolsAgentImpl. The functionality is now implemented inside
InspectorClientImpl.
Reviewed by Vsevolod Vlasov.
- public/WebDevToolsAgentClient.h:
- src/InspectorClientImpl.cpp:
(WebKit::InspectorClientImpl::clearBrowserCookies):
(WebKit::InspectorClientImpl::startMainThreadMonitoring):
(WebKit::InspectorClientImpl::stopMainThreadMonitoring):
(WebKit::InspectorClientImpl::willProcessTask):
(WebKit):
(WebKit::InspectorClientImpl::didProcessTask):
- src/InspectorClientImpl.h:
(InspectorClientImpl):
- src/WebDevToolsAgentImpl.cpp:
- src/WebDevToolsAgentImpl.h:
(WebDevToolsAgentImpl):
- 3:18 AM Changeset in webkit [120700] by
-
- 2 edits in trunk/Websites/webkit.org
Update webkit.org to refer to layoutTestController instead of testRunner
https://bugs.webkit.org/show_bug.cgi?id=89445
Reviewed by Kent Tamura.
- quality/testwriting.html:
- 2:33 AM Changeset in webkit [120699] by
-
- 2 edits in trunk/Tools
[Qt] Modify HTTPS port to 8443 for performance tests
https://bugs.webkit.org/show_bug.cgi?id=89442
Reviewed by Ryosuke Niwa.
We should be consistent with the default HTTPS port and with the changelog of r119188.
- Scripts/webkitpy/performance_tests/perftest.py:
(ReplayServer.init):
- 2:13 AM Changeset in webkit [120698] by
-
- 19 edits in trunk
Introduce ENABLE_CSP_NEXT configuration flag.
https://bugs.webkit.org/show_bug.cgi?id=89300
Patch by Mike West <mkwst@chromium.org> on 2012-06-19
Reviewed by Adam Barth.
The 1.0 draft of the Content Security Policy spec is just about to
move to Last Call. We'll hide work on the upcoming 1.1 spec behind
this ENABLE flag, disabled by default.
Spec: https://dvcs.w3.org/hg/content-security-policy/raw-file/tip/csp-specification.dev.html
.:
- Source/cmake/WebKitFeatures.cmake:
- Source/cmakeconfig.h.cmake:
Source/JavaScriptCore:
- Configurations/FeatureDefines.xcconfig:
Source/WebCore:
- Configurations/FeatureDefines.xcconfig:
Source/WebKit/chromium:
- features.gypi:
Source/WebKit/mac:
- Configurations/FeatureDefines.xcconfig:
Source/WebKit2:
- Configurations/FeatureDefines.xcconfig:
Tools:
- Scripts/webkitperl/FeatureList.pm:
- qmake/mkspecs/features/features.pri:
WebKitLibraries:
- win/tools/vsprops/FeatureDefines.vsprops:
- win/tools/vsprops/FeatureDefinesCairo.vsprops:
- 2:12 AM Changeset in webkit [120697] by
-
- 2 edits in trunk/Source/WTF
REGRESSION(r120552): Many tests timing out on Windows
https://bugs.webkit.org/show_bug.cgi?id=89433
Reviewed by Andy Estes.
- wtf/CurrentTime.cpp:
(WTF::lowResUTCTime):
It looks like there were two bugs:
1) the function was converting from 100ths of nanoseconds to seconds
rather than 100s of nanoseconds to seconds by dividing by
(nsPerSecond * 100) rather than (nsPerSecond / 100)
2) the function should've been returning milliseconds, not seconds
- 2:10 AM Changeset in webkit [120696] by
-
- 5 edits in trunk/Source/WebCore
Make StylePropertySet::merge() always override on conflict
https://bugs.webkit.org/show_bug.cgi?id=89348
Patch by Thiago Marcos P. Santos <thiago.santos@intel.com> on 2012-06-19
Reviewed by Ryosuke Niwa.
merge() was only used with argOverridesOnConflict = true, which
makes the extra parameter useless. It will also save a boolean check.
- css/StylePropertySet.cpp:
(WebCore::StylePropertySet::mergeAndOverrideOnConflict):
- css/StylePropertySet.h:
(StylePropertySet):
- editing/ApplyStyleCommand.cpp:
(WebCore::ApplyStyleCommand::applyInlineStyleToNodeRange):
- editing/EditingStyle.cpp:
(WebCore::EditingStyle::overrideWithStyle):
(WebCore::styleFromMatchedRulesForElement):
(WebCore::EditingStyle::mergeStyleFromRules):
(WebCore::EditingStyle::mergeStyleFromRulesForSerialization):
- 1:57 AM Changeset in webkit [120695] by
-
- 9 edits in trunk/Source/WebCore
[Forms] Move isKeyboardFocusable and isMouseFocusable to InputType from HTMLInputElement
https://bugs.webkit.org/show_bug.cgi?id=89425
Reviewed by Kent Tamura.
This patch moves implementation of HTMLInpueElement::isKeyboardFocusable
and isMouseFocusable to InputType class to allow each implementation
of InputType can decide focus-ability.
No new tests. This patch doesn't change behavior.
- html/HTMLInputElement.cpp:
(WebCore::HTMLInputElement::isKeyboardFocusable): Changed to call InputType::isKeyboardFocusable.
(WebCore::HTMLInputElement::isMouseFocusable): Changed to call InputType::isMouseFocusable.
(WebCore::HTMLInputElement::isTextFormControlFocusable): Added to allow InputType can call a method in HTMLTextFormControlElement.
(WebCore::HTMLInputElement::isTextFormControlKeyboardFocusable): Added to allow InputType can call a method in HTMLTextFormControlElement.
(WebCore::HTMLInputElement::isTextFormControlMouseFocusable): Added to allow InputType can call a method in HTMLTextFormControlElement.
- html/HTMLInputElement.h:
(HTMLInputElement):
- html/InputType.cpp:
(WebCore::InputType::isKeyboardFocusable): Changed to call isTextFormControlKeyboardFocusable.
(WebCore::InputType::isMouseFocusable): Added for default method of derived InputType classes.
- html/InputType.h:
(InputType): Chagned signature of isKeyboardFocusable.
- html/RadioInputType.cpp:
(WebCore::RadioInputType::isKeyboardFocusable): Changed to call isTextFormControlKeyboardFocusable which was done in HTMLInputElement.
- html/RadioInputType.h:
(RadioInputType):
- html/TextFieldInputType.cpp:
(WebCore::TextFieldInputType::isKeyboardFocusable): Moved implementation from HTMLInputElement.
(WebCore::TextFieldInputType::isMouseFocusable): Moved implementation from HTMLInputElement.
- html/TextFieldInputType.h:
(TextFieldInputType):
- 1:55 AM Changeset in webkit [120694] by
-
- 19 edits in trunk/Source
Calling nativeImageForCurrentFrame() causes assertion failure: m_verifier.isSafeToUse()
https://bugs.webkit.org/show_bug.cgi?id=67582
Reviewed by David Levin.
Source/WebCore:
Added a new synchronous method to the IconDatabase that returns a
native image for the platform instead of a WebCore::Image*.
- loader/icon/IconDatabase.cpp:
(WebCore::IconDatabase::synchronousNativeIconForPageURL):
(WebCore):
- loader/icon/IconDatabase.h:
(IconDatabase):
- loader/icon/IconDatabaseBase.h:
(WebCore::IconDatabaseBase::synchronousNativeIconForPageURL):
(IconDatabaseBase):
Source/WebKit/blackberry:
Use synchronousNativeIconForPageURL() to retrieve favicons.
- WebCoreSupport/FrameLoaderClientBlackBerry.cpp:
(WebCore::FrameLoaderClientBlackBerry::dispatchDidReceiveIcon):
Source/WebKit/cf:
Fixes windows build.
- WebCoreSupport/WebInspectorClientCF.cpp:
Source/WebKit/efl:
Use synchronousNativeIconForPageURL() to retrieve favicons.
- ewk/ewk_history.cpp:
(ewk_history_item_icon_surface_get):
(ewk_history_item_icon_object_add):
- ewk/ewk_settings.cpp:
(ewk_settings_icon_database_icon_surface_get):
(ewk_settings_icon_database_icon_object_add):
Source/WebKit/gtk:
Use synchronousNativeIconForPageURL() to retrieve favicons.
- webkit/webkitfavicondatabase.cpp:
(getIconPixbufSynchronously):
Source/WebKit/qt:
Use synchronousNativeIconForPageURL() to retrieve favicons.
- Api/qwebhistory.cpp:
(QWebHistoryItem::icon):
- Api/qwebsettings.cpp:
(QWebSettings::iconForUrl):
- 1:45 AM Changeset in webkit [120693] by
-
- 2 edits in trunk/Source/WebCore
[Mac] Use AutodrainedPool instead of NSAutoreleasePool in AudioBus::loadPlatformResource()
https://bugs.webkit.org/show_bug.cgi?id=89427
Reviewed by Dan Bernstein.
loadPlatformResource() called -[NSAutoreleasePool release] at each of
its exit points, which is not only an error-prone pattern but is
incorrect in garbage-collected environments. AutodrainedPool solves
both these problems by automatically calling -[NSAutoreleasePool drain]
when going out of scope.
- platform/audio/mac/AudioBusMac.mm:
(WebCore::AudioBus::loadPlatformResource):
- 1:38 AM Changeset in webkit [120692] by
-
- 9 edits in trunk
[EFL] EFL's LayoutTestController does not support setTextDirection
https://bugs.webkit.org/show_bug.cgi?id=87481
Patch by Christophe Dumez <Christophe Dumez> on 2012-06-19
Reviewed by Hajime Morita.
Source/WebKit/efl:
Add new Ewk_View method to set the text direction, by calling
Editor::setBaseWritingDirection() internally, as Mac and Chromium
ports do.
- WebCoreSupport/AssertMatchingEnums.cpp:
- ewk/ewk_view.cpp:
(ewk_view_text_direction_set):
- ewk/ewk_view.h:
Tools:
Implement setTextDirection in EFL's LayoutTestController and
properly reset its value between test cases to avoid flakiness.
- DumpRenderTree/efl/DumpRenderTreeChrome.cpp:
(DumpRenderTreeChrome::resetDefaultsToConsistentValues):
- DumpRenderTree/efl/LayoutTestControllerEfl.cpp:
(LayoutTestController::setTextDirection):
LayoutTests:
Unskip related test case now that EFL's LayoutTestController
implements setTextDirection.
- platform/efl/Skipped:
- 1:36 AM Changeset in webkit [120691] by
-
- 2 edits in trunk/Source/WebCore
Compilation failure in StyleResolver.cpp
https://bugs.webkit.org/show_bug.cgi?id=89446
Patch by Mike West <mkwst@chromium.org> on 2012-06-19
Reviewed by Ryosuke Niwa.
Adding 'CSSPropertyWebkitColumnProgression' to a switch statement from
which it was missing.
- css/StyleResolver.cpp:
(WebCore::StyleResolver::collectMatchingRulesForList):
- 1:23 AM Changeset in webkit [120690] by
-
- 2 edits in trunk/Source/WebCore
[Qt] Unreviewed build system fix after r120051.
- Target.pri:
- 1:18 AM Writing DumpAsMarkup Tests edited by
- s/layoutTestController/testRunner/ (diff)
- 1:17 AM Writing Layout Tests for DumpRenderTree edited by
- s/layoutTestController/testRunner/ (diff)
- 1:16 AM Creating and Submitting Layout Tests and Patches edited by
- s/layoutTestController/testRunner/ (diff)
- 1:15 AM Changeset in webkit [120689] by
-
- 3 edits in trunk/Source/WebKit/chromium
Unreviewed, rolling out r120586.
http://trac.webkit.org/changeset/120586
https://bugs.webkit.org/show_bug.cgi?id=89444
doesn't fix mac issue (Requested by morrita on #webkit).
Patch by Sheriff Bot <webkit.review.bot@gmail.com> on 2012-06-19
- src/ContextMenuClientImpl.cpp:
(WebKit::ContextMenuClientImpl::getCustomMenuFromDefaultItems):
- src/WebFrameImpl.cpp:
(WebKit::WebFrameImpl::replaceMisspelledRange):
- 12:40 AM Changeset in webkit [120688] by
-
- 2 edits in trunk/Tools
[Qt] Temporarily disable xvfb driver in nrwt
https://bugs.webkit.org/show_bug.cgi?id=88414
Reviewed by Csaba Osztrogonác.
- Scripts/webkitpy/layout_tests/port/qt.py:
(QtPort._build_driver):
- 12:35 AM Changeset in webkit [120687] by
-
- 2 edits in trunk/LayoutTests
Unreviewed expectation update.
- platform/chromium/TestExpectations:
- 12:35 AM Changeset in webkit [120686] by
-
- 2 edits in trunk
[Qt] Allow the modules file generation to work on Windows.
https://bugs.webkit.org/show_bug.cgi?id=89010
Patch by Joel Dillon <joel.dillon@codethink.co.uk>, Jocelyn Turcotte <jocelyn.turcotte@nokia.com> on 2012-06-19
Reviewed by Csaba Osztrogonác.
Add calls to toSystemPath and use QMAKE_DIR_SEP where necessary.
- Source/api.pri:
- 12:32 AM Changeset in webkit [120685] by
-
- 2 edits in trunk
[Qt] Fix TARGET not properly being set for QtWebKitd5.dll when building in debug.
https://bugs.webkit.org/show_bug.cgi?id=88880
load(qt_module_config) adjusts the TARGET to add the "d" suffix when doing
the debug build_pass. When doing the first pass, qt_module_config.prf doesn't
adjust the target since it is in debug_and_release and expects the debug
build_pass to come later. However, since we remove debug_and_release from CONFIG
right after this, the TARGET hasn't been adjusted and the debug build_pass
will not happen.
Work around it by disabling this optimization on Windows.
Patch by Jocelyn Turcotte <jocelyn.turcotte@nokia.com> on 2012-06-19
Reviewed by Tor Arne Vestbø.
- Source/api.pri:
- 12:27 AM Changeset in webkit [120684] by
-
- 17 edits2 adds in trunk
Add a scheme registry for bypassing Content Security Policy.
https://bugs.webkit.org/show_bug.cgi?id=89373
Patch by Mike West <mkwst@chromium.org> on 2012-06-19
Reviewed by Adam Barth.
.:
- Source/autotools/symbols.filter:
Source/WebCore:
The Content Security Policy specification notes that "Enforcing a CSP
policy should not interfere with the operation of user-supplied scripts
such as third-party user-agent add-ons and JavaScript bookmarklets."
(https://dvcs.w3.org/hg/content-security-policy/raw-file/tip/csp-specification.dev.html#processing-model)
This use case can be at least partially addressed by adding a scheme
registry for certain types of URLs whose resources should be loaded
regardless of a page's defined policy directives.
See http://crbug.com/133223 for additional context.
Test: http/tests/security/contentSecurityPolicy/register-bypassing-scheme.html
- WebCore.exp.in:
Exporting new symbols for use by Internals.
- page/ContentSecurityPolicy.cpp:
(WebCore::isAllowedByAllWithURL):
Adding a check against the new SchemeRegistry method to determine
whether or not we should proceed with checking the URL against the
policy.
- platform/SchemeRegistry.cpp:
(WebCore::CSPBypassingSchemes):
(WebCore):
(WebCore::SchemeRegistry::registerURLSchemeAsBypassingContentSecurityPolicy):
(WebCore::SchemeRegistry::removeURLSchemeRegisteredAsBypassingContentSecurityPolicy):
(WebCore::SchemeRegistry::schemeShouldBypassContentSecurityPolicy):
- platform/SchemeRegistry.h:
(SchemeRegistry):
Adding the scheme registry for CSP-bypassing URL schemes.
- testing/Internals.cpp:
(WebCore):
(WebCore::Internals::registerURLSchemeAsBypassingContentSecurityPolicy):
(WebCore::Internals::removeURLSchemeRegisteredAsBypassingContentSecurityPolicy):
- testing/Internals.h:
(Internals):
- testing/Internals.idl:
Adding a method to Internals so that we can write a layout test.
Source/WebKit/chromium:
- public/WebSecurityPolicy.h:
(WebSecurityPolicy):
- src/WebSecurityPolicy.cpp:
(WebKit::WebSecurityPolicy::registerURLSchemeAsBypassingContentSecurityPolicy):
(WebKit):
Source/WebKit2:
- win/WebKit2.def:
- win/WebKit2CFLite.def:
LayoutTests:
- http/tests/security/contentSecurityPolicy/register-bypassing-scheme-expected.txt: Added.
- http/tests/security/contentSecurityPolicy/register-bypassing-scheme.html: Added.
- 12:23 AM Changeset in webkit [120683] by
-
- 492 edits in trunk/LayoutTests
Use testRunner instead of layoutTestController in fast/canvas, clip, compact, constructors, cookies, and css* tests
https://bugs.webkit.org/show_bug.cgi?id=89431
Reviewed by Kent Tamura.
- fast/canvas/:
- fast/constructors/constructor-as-function-crash.html:
- fast/css-generated-content/absolute-position-inside-inline.html:
- fast/css-generated-content/after-with-first-letter-float-crash.html:
- fast/css-generated-content/attr-missing.html:
- fast/css-generated-content/before-content-with-list-marker-in-anon-block-crash.html:
- fast/css-generated-content/block-after.html:
- fast/css-generated-content/drag-state.html:
- fast/css-generated-content/empty-content-with-float-crash.html:
- fast/css-generated-content/first-letter-next-sibling-crash.html:
- fast/css-generated-content/first-letter-table-cell-format-block-crash.html:
- fast/css-generated-content/float-first-letter-siblings-convert-to-inline.html:
- fast/css-generated-content/floating-before-content-with-list-marker-crash.html:
- fast/css-generated-content/hit-test-generated-content.html:
- fast/css-generated-content/hover-style-change.html:
- fast/css-generated-content/inline-splitting-with-after-float-crash.html:
- fast/css-generated-content/malformed-url.html:
- fast/css-generated-content/positioned-background-hit-test-crash.html:
- fast/css-generated-content/positioned-div-with-floating-after-content-crash.html:
- fast/css-generated-content/positioned-generated-content-under-run-in-crash.html:
- fast/css-generated-content/remove-div-from-flexible-box-with-floating-after-content-crash.html:
- fast/css-generated-content/reset-content-to-initial.html:
- fast/css-generated-content/resources/pass.html:
- fast/css-generated-content/resources/positioned-div-with-floating-after-content-crash-frame2.html:
- fast/css-generated-content/spellingToolTip-assert.html:
- fast/css-generated-content/table-row-after-no-crash.html:
- fast/css-generated-content/table-with-scrollbar-corner.html:
- fast/css-generated-content/text-before-table-col-crash.html:
- fast/css-grid-layout/display-grid-set-get.html:
- fast/css-grid-layout/grid-columns-rows-get-set-multiple.html:
- fast/css-grid-layout/grid-columns-rows-get-set.html:
- fast/css-grid-layout/grid-item-column-row-get-set.html:
- fast/css/:
- platform/mac/fast/canvas/canvas-draw-xbm-image.html:
- platform/qt/fast/canvas/toDataURL-jpeg-alpha.html:
- platform/qt/fast/canvas/toDataURL-jpeg-primarycolors.html:
- platform/qt/fast/canvas/toDataURL-jpeg-quality-basic.html:
- platform/qt/fast/canvas/toDataURL-jpeg-quality-notnumber.html:
- platform/qt/fast/canvas/toDataURL-jpeg-quality-outsiderange.html:
