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

Timeline



Jul 16, 2009:

11:41 PM Changeset in webkit [46020] by abarth@webkit.org
  • 1 edit
    5 adds in trunk/WebCore

2009-07-16 Maxime Simon <Maxime Simon>

Reviewed by Oliver Hunt.

Added a third bunch of Haiku-specific files for WebCore.
​https://bugs.webkit.org/show_bug.cgi?id=26952

Adding five files, EventLoopHaiku.cpp, FileChooserHaiku.cpp,
FileSystemHaiku.cpp, KeyboardCodes.h and MIMETypeRegistryHaiku.cpp

  • platform/haiku/EventLoopHaiku.cpp: Added. (WebCore::EventLoop::cycle):
  • platform/haiku/FileChooserHaiku.cpp: Added. (WebCore::FileChooser::FileChooser): (WebCore::FileChooser::basenameForWidth):
  • platform/haiku/FileSystemHaiku.cpp: Added. (WebCore::fileSystemRepresentation): (WebCore::homeDirectoryPath): (WebCore::openTemporaryFile): (WebCore::closeFile): (WebCore::writeToFile): (WebCore::unloadModule): (WebCore::listDirectory):
  • platform/haiku/KeyboardCodes.h: Added.
  • platform/haiku/MIMETypeRegistryHaiku.cpp: Added. (WebCore::): (WebCore::MIMETypeRegistry::getMIMETypeForExtension):
11:41 PM Changeset in webkit [46019] by abarth@webkit.org
  • 1 edit
    5 adds in trunk/WebCore

2009-07-16 Maxime Simon <Maxime Simon>

Reviewed by Oliver Hunt.

Added a second bunch of Haiku-specific files for WebCore.
​https://bugs.webkit.org/show_bug.cgi?id=26952

Adding four files, ContextMenuHaiku.cpp, ContextMenuItemHaiku.cpp,
DragDataHaiku.cpp and DragImageHaiku.cpp

  • platform/haiku/ContextMenuHaiku.cpp: Added. (WebCore::ContextMenuReceiver::ContextMenuReceiver): (WebCore::ContextMenuReceiver::HandleMessage): (WebCore::ContextMenuReceiver::Result): (WebCore::ContextMenu::ContextMenu): (WebCore::ContextMenu::~ContextMenu): (WebCore::ContextMenu::appendItem): (WebCore::ContextMenu::itemCount): (WebCore::ContextMenu::insertItem): (WebCore::ContextMenu::platformDescription): (WebCore::ContextMenu::setPlatformDescription):
  • platform/haiku/ContextMenuItemHaiku.cpp: Added. (ContextMenuItem::ContextMenuItem): (ContextMenuItem::~ContextMenuItem): (ContextMenuItem::releasePlatformDescription): (ContextMenuItem::type): (ContextMenuItem::setType): (ContextMenuItem::action): (ContextMenuItem::setAction): (ContextMenuItem::title): (ContextMenuItem::setTitle): (ContextMenuItem::platformSubMenu): (ContextMenuItem::setSubMenu): (ContextMenuItem::setChecked): (ContextMenuItem::setEnabled): (ContextMenuItem::enabled):
  • platform/haiku/DragDataHaiku.cpp: Added. (WebCore::DragData::canSmartReplace): (WebCore::DragData::containsColor): (WebCore::DragData::containsFiles): (WebCore::DragData::asFilenames): (WebCore::DragData::containsPlainText): (WebCore::DragData::asPlainText): (WebCore::DragData::asColor): (WebCore::DragData::createClipboard): (WebCore::DragData::containsCompatibleContent): (WebCore::DragData::containsURL): (WebCore::DragData::asURL): (WebCore::DragData::asFragment):
  • platform/haiku/DragImageHaiku.cpp: Added. (WebCore::dragImageSize): (WebCore::deleteDragImage): (WebCore::scaleDragImage): (WebCore::dissolveDragImageToFraction): (WebCore::createDragImageFromImage): (WebCore::createDragImageIconForCachedImage):
11:30 PM Changeset in webkit [46018] by abarth@webkit.org
  • 1 edit
    2 adds in trunk/JavaScriptCore

2009-07-16 Maxime Simon <Maxime Simon>

Reviewed by Eric Seidel.

Added Haiku-specific files for JavaScriptCore.
​https://bugs.webkit.org/show_bug.cgi?id=26620

  • wtf/haiku/MainThreadHaiku.cpp: Added. (WTF::initializeMainThreadPlatform): (WTF::scheduleDispatchFunctionsOnMainThread):
11:28 PM Changeset in webkit [46017] by darin@chromium.org
  • 11 edits in trunk/WebCore

2009-07-16 Stephen White <​senorblanco@chromium.org>

Reviewed by Darin Fisher and Brett Wilson.

Refactor Skia implementation of gradients and patterns.

​http://bugs.webkit.org/show_bug.cgi?id=26618

The following layout tests were breaking on Chromium/Linux:

LayoutTests/svg/custom/js-late-gradient-creation.svg (bad baseline PNG)
LayoutTests/svg/custom/js-late-gradient-and-object.creation.svg
LayoutTests/svg/custom/js-late-pattern-creation.svg (bad baseline PNG)
LayoutTests/svg/custom/js-late-pattern-and-object-creation.svg

I could've fixed these the easy way, by copying the same 5
lines of code we use everywhere we need patterns or gradients, but
I decided to fix it the hard way: by refactoring the code so that
PlatformContextSkia::setupPaintForFilling() and
PlatformContextSkia::setupPaintForStroking() do the right thing,
and also handle gradients and patterns.


This required pushing the gradients and patterns set in
(generic) GraphicsContext::setFillPattern() and friends down into
PlatformContextSkia. For this, I followed the setPlatformXXX()
pattern used elsewhere in GraphicsContext, and stubbed them out on
the other platforms with #if !PLATFORM(SKIA). This also required
pushing changes to the gradientSpaceTransform from the Gradient into
GradientSkia.

Since it's a Skia context, I decided to cache the values as
SkShaders. There were existing m_pattern and m_gradient SkShaders,
but they were unused, and whose use was ambiguous, so I
replaced them with one SkShader each for filling and stroking.

  • platform/graphics/Gradient.cpp: (WebCore::Gradient::setGradientSpaceTransform): (WebCore::Gradient::setPlatformGradientSpaceTransform):
  • platform/graphics/Gradient.h:
  • platform/graphics/GraphicsContext.cpp: (WebCore::GraphicsContext::setStrokePattern): (WebCore::GraphicsContext::setFillPattern): (WebCore::GraphicsContext::setStrokeGradient): (WebCore::GraphicsContext::setFillGradient): (WebCore::GraphicsContext::setPlatformFillGradient): (WebCore::GraphicsContext::setPlatformFillPattern): (WebCore::GraphicsContext::setPlatformStrokeGradient): (WebCore::GraphicsContext::setPlatformStrokePattern):
  • platform/graphics/GraphicsContext.h:
  • platform/graphics/skia/GradientSkia.cpp: (WebCore::Gradient::setPlatformGradientSpaceTransform):
  • platform/graphics/skia/GraphicsContextSkia.cpp: (WebCore::GraphicsContext::fillPath): (WebCore::GraphicsContext::fillRect): (WebCore::GraphicsContext::setPlatformFillGradient): (WebCore::GraphicsContext::setPlatformFillPattern): (WebCore::GraphicsContext::setPlatformStrokeGradient): (WebCore::GraphicsContext::setPlatformStrokePattern): (WebCore::GraphicsContext::strokePath): (WebCore::GraphicsContext::strokeRect):
  • platform/graphics/skia/PlatformContextSkia.cpp: (PlatformContextSkia::State::State): (PlatformContextSkia::State::~State): (PlatformContextSkia::drawRect): (PlatformContextSkia::setupPaintCommon): (PlatformContextSkia::setupPaintForFilling): (PlatformContextSkia::setupPaintForStroking): (PlatformContextSkia::setFillColor): (PlatformContextSkia::setStrokeColor): (PlatformContextSkia::setStrokeShader): (PlatformContextSkia::setFillShader):
  • platform/graphics/skia/PlatformContextSkia.h:
  • platform/graphics/skia/SkiaFontWin.cpp: (WebCore::skiaDrawText): (WebCore::paintSkiaText):
  • svg/graphics/SVGPaintServer.cpp: (WebCore::SVGPaintServer::teardown):
11:03 PM Changeset in webkit [46016] by abarth@webkit.org
  • 1 edit
    2 adds in trunk/WebKit

2009-07-16 Maxime Simon <Maxime Simon>

Reviewed by Eric Seidel.

Added InspectorClient for Haiku WebCore support.
​https://bugs.webkit.org/show_bug.cgi?id=26952

Adding two files, InspectorClientHaiku.h and InspectorClientHaiku.cpp

  • haiku/WebCoreSupport/InspectorClientHaiku.cpp: Added. (WebCore::InspectorClientHaiku::inspectorDestroyed): (WebCore::InspectorClientHaiku::createPage): (WebCore::InspectorClientHaiku::localizedStringsURL): (WebCore::InspectorClientHaiku::hiddenPanels): (WebCore::InspectorClientHaiku::showWindow): (WebCore::InspectorClientHaiku::closeWindow): (WebCore::InspectorClientHaiku::attachWindow): (WebCore::InspectorClientHaiku::detachWindow): (WebCore::InspectorClientHaiku::setAttachedWindowHeight): (WebCore::InspectorClientHaiku::highlight): (WebCore::InspectorClientHaiku::hideHighlight): (WebCore::InspectorClientHaiku::inspectedURLChanged): (WebCore::InspectorClientHaiku::populateSetting): (WebCore::InspectorClientHaiku::storeSetting): (WebCore::InspectorClientHaiku::removeSetting):
  • haiku/WebCoreSupport/InspectorClientHaiku.h: Added.
11:02 PM Changeset in webkit [46015] by abarth@webkit.org
  • 1 edit
    2 adds in trunk/WebKit

2009-07-16 Maxime Simon <Maxime Simon>

Reviewed by Oliver Hunt.

Added EditorClient for Haiku WebCore support.
​https://bugs.webkit.org/show_bug.cgi?id=26952

Adding two files, EditorClientHaiku.h and EditorClientHaiku.cpp

  • haiku/WebCoreSupport/EditorClientHaiku.cpp: Added. (WebCore::EditorClientHaiku::EditorClientHaiku): (WebCore::EditorClientHaiku::setPage): (WebCore::EditorClientHaiku::pageDestroyed): (WebCore::EditorClientHaiku::shouldDeleteRange): (WebCore::EditorClientHaiku::shouldShowDeleteInterface): (WebCore::EditorClientHaiku::smartInsertDeleteEnabled): (WebCore::EditorClientHaiku::isSelectTrailingWhitespaceEnabled): (WebCore::EditorClientHaiku::isContinuousSpellCheckingEnabled): (WebCore::EditorClientHaiku::toggleContinuousSpellChecking): (WebCore::EditorClientHaiku::isGrammarCheckingEnabled): (WebCore::EditorClientHaiku::toggleGrammarChecking): (WebCore::EditorClientHaiku::spellCheckerDocumentTag): (WebCore::EditorClientHaiku::isEditable): (WebCore::EditorClientHaiku::shouldBeginEditing): (WebCore::EditorClientHaiku::shouldEndEditing): (WebCore::EditorClientHaiku::shouldInsertNode): (WebCore::EditorClientHaiku::shouldInsertText): (WebCore::EditorClientHaiku::shouldChangeSelectedRange): (WebCore::EditorClientHaiku::shouldApplyStyle): (WebCore::EditorClientHaiku::shouldMoveRangeAfterDelete): (WebCore::EditorClientHaiku::didBeginEditing): (WebCore::EditorClientHaiku::respondToChangedContents): (WebCore::EditorClientHaiku::respondToChangedSelection): (WebCore::EditorClientHaiku::didEndEditing): (WebCore::EditorClientHaiku::didWriteSelectionToPasteboard): (WebCore::EditorClientHaiku::didSetSelectionTypesForPasteboard): (WebCore::EditorClientHaiku::registerCommandForUndo): (WebCore::EditorClientHaiku::registerCommandForRedo): (WebCore::EditorClientHaiku::clearUndoRedoOperations): (WebCore::EditorClientHaiku::canUndo): (WebCore::EditorClientHaiku::canRedo): (WebCore::EditorClientHaiku::undo): (WebCore::EditorClientHaiku::redo): (WebCore::EditorClientHaiku::handleKeyboardEvent): (WebCore::EditorClientHaiku::handleInputMethodKeydown): (WebCore::EditorClientHaiku::textFieldDidBeginEditing): (WebCore::EditorClientHaiku::textFieldDidEndEditing): (WebCore::EditorClientHaiku::textDidChangeInTextField): (WebCore::EditorClientHaiku::doTextFieldCommandFromEvent): (WebCore::EditorClientHaiku::textWillBeDeletedInTextField): (WebCore::EditorClientHaiku::textDidChangeInTextArea): (WebCore::EditorClientHaiku::ignoreWordInSpellDocument): (WebCore::EditorClientHaiku::learnWord): (WebCore::EditorClientHaiku::checkSpellingOfString): (WebCore::EditorClientHaiku::getAutoCorrectSuggestionForMisspelledWord): (WebCore::EditorClientHaiku::checkGrammarOfString): (WebCore::EditorClientHaiku::updateSpellingUIWithGrammarString): (WebCore::EditorClientHaiku::updateSpellingUIWithMisspelledWord): (WebCore::EditorClientHaiku::showSpellingUI): (WebCore::EditorClientHaiku::spellingUIIsShowing): (WebCore::EditorClientHaiku::getGuessesForWord): (WebCore::EditorClientHaiku::setInputMethodState): (WebCore::EditorClientHaiku::isEditing):
  • haiku/WebCoreSupport/EditorClientHaiku.h: Added.
11:02 PM Changeset in webkit [46014] by abarth@webkit.org
  • 1 edit
    2 adds in trunk/WebKit

2009-07-16 Maxime Simon <Maxime Simon>

Reviewed by Eric Seidel.

Added DragClient for Haiku WebCore support.
​https://bugs.webkit.org/show_bug.cgi?id=26952

Adding two files, DragClientHaiku.h and DragClientHaiku.cpp

  • haiku/WebCoreSupport/DragClientHaiku.cpp: Added. (WebCore::DragClientHaiku::actionMaskForDrag): (WebCore::DragClientHaiku::willPerformDragDestinationAction): (WebCore::DragClientHaiku::dragControllerDestroyed): (WebCore::DragClientHaiku::dragSourceActionMaskForPoint): (WebCore::DragClientHaiku::willPerformDragSourceAction): (WebCore::DragClientHaiku::startDrag): (WebCore::DragClientHaiku::createDragImageForLink):
  • haiku/WebCoreSupport/DragClientHaiku.h: Added.
11:02 PM Changeset in webkit [46013] by abarth@webkit.org
  • 1 edit
    4 adds in trunk/WebKit

2009-07-16 Maxime Simon <Maxime Simon>

Reviewed by Oliver Hunt.

Added ContextMenuClient for Haiku WebCore support.
​https://bugs.webkit.org/show_bug.cgi?id=26952

Adding two files, ContextMenuClientHaiku.h
and ContextMenuClientHaiku.cpp

  • haiku/WebCoreSupport/ContextMenuClientHaiku.cpp: Added. (WebCore::ContextMenuClientHaiku::contextMenuDestroyed): (WebCore::ContextMenuClientHaiku::getCustomMenuFromDefaultItems): (WebCore::ContextMenuClientHaiku::contextMenuItemSelected): (WebCore::ContextMenuClientHaiku::downloadURL): (WebCore::ContextMenuClientHaiku::lookUpInDictionary): (WebCore::ContextMenuClientHaiku::speak): (WebCore::ContextMenuClientHaiku::isSpeaking): (WebCore::ContextMenuClientHaiku::stopSpeaking): (WebCore::ContextMenuClientHaiku::searchWithGoogle):
  • haiku/WebCoreSupport/ContextMenuClientHaiku.h: Added.
10:47 PM Changeset in webkit [46012] by abarth@webkit.org
  • 1 edit
    2 adds in trunk/WebCore

2009-07-16 Maxime Simon <Maxime Simon>

Reviewed by Oliver Hunt.

Added Haiku-specific files for WebCore/platform/image-decoders/.
​https://bugs.webkit.org/show_bug.cgi?id=26949

Adding a new file, ImageDecoderHaiku.cpp.

  • platform/image-decoders/haiku/ImageDecoderHaiku.cpp: Added. (WebCore::RGBA32Buffer::RGBA32Buffer): (WebCore::RGBA32Buffer::clear): (WebCore::RGBA32Buffer::zeroFill): (WebCore::RGBA32Buffer::copyBitmapData): (WebCore::RGBA32Buffer::setSize): (WebCore::RGBA32Buffer::asNewNativeImage): (WebCore::RGBA32Buffer::hasAlpha): (WebCore::RGBA32Buffer::setHasAlpha): (WebCore::RGBA32Buffer::setStatus): (WebCore::RGBA32Buffer::operator=): (WebCore::RGBA32Buffer::width): (WebCore::RGBA32Buffer::height):
10:47 PM Changeset in webkit [46011] by abarth@webkit.org
  • 1 edit
    4 adds in trunk/WebCore

2009-07-16 Maxime Simon <Maxime Simon>

Reviewed by Eric Seidel.

Added Haiku-specific files for WebCore/page/.
​https://bugs.webkit.org/show_bug.cgi?id=26949

Adding three new files, DragControllerHaiku.cpp, EventHandlerHaiku.cpp
and FrameHaiku.cpp

  • page/haiku/DragControllerHaiku.cpp: Added. (WebCore::DragController::isCopyKeyDown): (WebCore::DragController::dragOperation): (WebCore::DragController::maxDragImageSize): (WebCore::DragController::cleanupAfterSystemDrag):
  • page/haiku/EventHandlerHaiku.cpp: Added. (WebCore::isKeyboardOptionTab): (WebCore::EventHandler::invertSenseOfTabsToLinks): (WebCore::EventHandler::tabsToAllControls): (WebCore::EventHandler::focusDocumentView): (WebCore::EventHandler::passWidgetMouseDownEventToWidget): (WebCore::EventHandler::passMouseDownEventToWidget): (WebCore::EventHandler::eventActivatedView): (WebCore::EventHandler::passSubframeEventToSubframe): (WebCore::EventHandler::passWheelEventToWidget): (WebCore::EventHandler::createDraggingClipboard): (WebCore::EventHandler::passMousePressEventToSubframe): (WebCore::EventHandler::passMouseMoveEventToSubframe): (WebCore::EventHandler::passMouseReleaseEventToSubframe): (WebCore::EventHandler::accessKeyModifiers):
  • page/haiku/FrameHaiku.cpp: Added. (WebCore::Frame::dragImageForSelection):
10:46 PM Changeset in webkit [46010] by abarth@webkit.org
  • 1 edit
    2 adds in trunk/WebCore

2009-07-16 Maxime Simon <Maxime Simon>

Reviewed by Eric Seidel.

Added Haiku-specific files for WebCore/editing/.
​https://bugs.webkit.org/show_bug.cgi?id=26949

Adding one new file, EditorHaiku.cpp

  • editing/haiku/EditorHaiku.cpp: Added. (WebCore::Editor::newGeneralClipboard):
10:46 PM Changeset in webkit [46009] by abarth@webkit.org
  • 1 edit
    1 add in trunk/WebCore

2009-07-16 Maxime Simon <Maxime Simon>

Reviewed by Eric Seidel.

Added Haiku-specific files for WebCore/bindings/js/.
​https://bugs.webkit.org/show_bug.cgi?id=26949

Adding a new file, ScriptControllerHaiku.cpp

  • bindings/js/ScriptControllerHaiku.cpp: Added. (WebCore::ScriptController::createScriptInstanceForWidget):
10:46 PM Changeset in webkit [46008] by abarth@webkit.org
  • 1 edit
    3 adds in trunk/WebCore

2009-07-16 Maxime Simon <Maxime Simon>

Reviewed by Eric Seidel.

Added Haiku-specific files for WebCore/platform/text/.
​https://bugs.webkit.org/show_bug.cgi?id=26949

Adding two new files, StringHaiku.cpp
and TextBreakIteratorInternalICUHaiku.cpp

  • platform/text/haiku/StringHaiku.cpp: Added. (WebCore::String::String): (WebCore::String::operator BString):
  • platform/text/haiku/TextBreakIteratorInternalICUHaiku.cpp: Added. (WebCore::currentTextBreakLocaleID):
10:29 PM Changeset in webkit [46007] by abarth@webkit.org
  • 2 edits in trunk/WebKit/gtk

2009-07-16 Holger Hans Peter Freyther <​zecke@selfish.org>

Reviewed by Jan Alonzo.

[GTK+] Update the focus handling code.

Change focus-in to always call setActive(true), call
FocusController::setFocusedFrame when no focused frame exists
other SelectionController::setActive. This is matching Windows
and Qt.

On focus-out call FocusController::setActive(false) and
SelectionController::setFocued(false). This is matching Windows
and Qt as well.

  • webkit/webkitwebview.cpp: (webkit_web_view_focus_in_event): (webkit_web_view_focus_out_event):
9:36 PM Changeset in webkit [46006] by abarth@webkit.org
  • 6 edits
    2 adds in trunk

2009-07-16 Kent Tamura <​tkent@chromium.org>

Reviewed by Eric Seidel and Adam Barth.

A test for for non-multipart file upload form submission.
This test is disabled for non-Mac platforms because of beginDragWithFiles().
<​https://bugs.webkit.org/show_bug.cgi?id=26505>

  • fast/forms/get-file-upload-expected.txt: Added.
  • fast/forms/get-file-upload.html: Added.
  • platform/gtk/Skipped:
  • platform/qt/Skipped:
  • platform/win/Skipped:

2009-07-16 Kent Tamura <​tkent@chromium.org>

Reviewed by Eric Seidel.

Sends the basename of a selected file for non-multipart form submission.
<​https://bugs.webkit.org/show_bug.cgi?id=26505>

Test: fast/forms/get-file-upload.html

  • html/HTMLInputElement.cpp: (WebCore::HTMLInputElement::appendFormData):
9:23 PM Changeset in webkit [46005] by abarth@webkit.org
  • 4 edits
    2 adds in trunk/WebCore

2009-07-16 Adam Barth <​abarth@webkit.org>

Reviewed by David Levin.

[V8] Centralize hidden property names
​https://bugs.webkit.org/show_bug.cgi?id=27359

No behavior change. Just moving these names to a central location.
I'll move the rest of our hidden property names as I sweep though the
bindings.

  • WebCore.gypi:
  • bindings/v8/V8HiddenPropertyName.cpp: Added. (WebCore::V8HiddenPropertyName::objectPrototype): (WebCore::V8HiddenPropertyName::isolatedWorld):
  • bindings/v8/V8HiddenPropertyName.h: Added.
  • bindings/v8/V8IsolatedWorld.cpp: (WebCore::V8IsolatedWorld::V8IsolatedWorld): (WebCore::V8IsolatedWorld::getEntered):
  • bindings/v8/V8Proxy.cpp: (WebCore::V8Proxy::getHiddenObjectPrototype): (WebCore::V8Proxy::installHiddenObjectPrototype):
8:24 PM Changeset in webkit [46004] by barraclough@apple.com
  • 4 edits in trunk/JavaScriptCore

2009-07-16 Gavin Barraclough <​barraclough@apple.com>

RS by Oliver Hunt.

Revert r45969, this fix does not appear to be valid.
​https://bugs.webkit.org/show_bug.cgi?id=27077

  • bytecode/CodeBlock.cpp: (JSC::CodeBlock::~CodeBlock): (JSC::CodeBlock::unlinkCallers):
  • jit/JIT.cpp:
  • jit/JIT.h:
7:55 PM Changeset in webkit [46003] by treat@webkit.org
  • 3 edits in trunk/WebKitTools

2009-07-16 Adam Treat <​adam.treat@torchmobile.com>

Reviewed by Dave Levin.

cpplint should check for one line control clauses that are surrounded
by braces
​https://bugs.webkit.org/show_bug.cgi?id=27354

  • Scripts/modules/cpplint.py: Added the new lint check.
  • Scripts/modules/cpplint_unittest.py: Add tests for the new lint check and fix the other tests as they were not passing this new lint check.
5:47 PM Changeset in webkit [46002] by mitz@apple.com
  • 8 edits
    4 adds in trunk

WebCore:

Reviewed by Simon Fraser.

REGRESSION (r41238) Repainted portion of a scaled image does not line up with full image
​https://bugs.webkit.org/show_bug.cgi?id=26747
rdar://problem/7009243

Test: fast/repaint/background-misaligned.html

  • platform/graphics/Image.cpp: (WebCore::Image::drawTiled): Moved a variable definition closer to where it is used.
  • platform/graphics/cg/ImageCG.cpp: (WebCore::BitmapImage::draw): In the subimage code path, compute a pixel-aligned source rect, because the subiamge is always pixel-aligned in source space, and adjust the destination rect to preserve the source -> destination mapping. Clip to the (original) destination rect to prevent bleeding out.

LayoutTests:

Reviewed by Simon Fraser.

REGRESSION (r41238) Repainted portion of a scaled image does not line up with full image
​https://bugs.webkit.org/show_bug.cgi?id=26747
rdar://problem/7009243

  • fast/repaint/background-misaligned.html: Added.
  • platform/mac/fast/backgrounds/size/backgroundSize15-expected.checksum:
  • platform/mac/fast/backgrounds/size/backgroundSize15-expected.png:
  • platform/mac/fast/borders/border-image-rotate-transform-expected.checksum:
  • platform/mac/fast/borders/border-image-rotate-transform-expected.png:
  • platform/mac/fast/repaint/background-misaligned-expected.checksum: Added.
  • platform/mac/fast/repaint/background-misaligned-expected.png: Added.
  • platform/mac/fast/repaint/background-misaligned-expected.txt: Added.
5:41 PM Changeset in webkit [46001] by abarth@webkit.org
  • 4 edits in trunk/WebCore

2009-07-16 Jeremy Orlow <​jorlow@chromium.org>

Reviewed by Adam Barth.

Add a sessionStorageEnabled setting to the settings class.
​https://bugs.webkit.org/show_bug.cgi?id=27318

Allow LocalStorage to be enabled without enabling SessionStorage at
runtime. There is a settings class setting for localStorage, but not
for sessionStorage. We want to be able to test one of these features
without necessarily enabling the other.

SessionStorage defaults to true so as to not change behavior and
because there really aren't any security concerns around SessionStorage
(unlike LocalsStorage). The flag is needed in Chromium only because
we want to enable the compile time flag in the default build, but don't
want it on by default until it's been thoroughly tested.

  • page/DOMWindow.cpp: (WebCore::DOMWindow::sessionStorage): Check the new flag (WebCore::DOMWindow::localStorage): A bit of cleanup
  • page/Settings.cpp: (WebCore::Settings::Settings): Default the flag to true (WebCore::Settings::setSessionStorageEnabled): Add the new flag
  • page/Settings.h: (WebCore::Settings::sessionStorageEnabled): Get the new flag
5:35 PM Changeset in webkit [46000] by mitz@apple.com
  • 5 edits in trunk/WebKit

../gtk:

2009-07-16 Xiaomei Ji <​xji@chromium.org>

Reviewed by Dan Bernstein.

This is the 2nd part of fixing "RTL: tooltip does not get its directionlity from its element's."
​https://bugs.webkit.org/show_bug.cgi?id=24187

Add one extra parameter to the callee of HitTestResult::title() due to the signature change.

  • WebCoreSupport/ChromeClientGtk.cpp: (WebKit::ChromeClient::mouseDidMoveOverElement): Add direction as a parameter to the callee of HitTestResult::title().

../qt:

2009-07-16 Xiaomei Ji <​xji@chromium.org>

Reviewed by Dan Bernstein.

This is the 2nd part of fixing "RTL: tooltip does not get its directionlity from its element's."
​https://bugs.webkit.org/show_bug.cgi?id=24187

Add one extra parameter to the callee of HitTestResult::title() due to the signature change.

  • Api/qwebframe.cpp: (QWebHitTestResultPrivate::QWebHitTestResultPrivate): Add direction as a parameter to the callee of HitTestResult::title().
  • WebCoreSupport/ChromeClientQt.cpp: (WebCore::ChromeClientQt::mouseDidMoveOverElement): Add direction as a parameter to the callee of HitTestResult::title().
5:16 PM Changeset in webkit [45999] by abarth@webkit.org
  • 3 edits
    2 deletes in trunk

WebCore:

2009-07-16 Adam Barth <​abarth@webkit.org>

Unreviewed.

Revert 45987. Tests did not pass on Windows.

  • html/HTMLInputElement.cpp: (WebCore::HTMLInputElement::appendFormData):

LayoutTests:

2009-07-16 Adam Barth <​abarth@webkit.org>

Unreviewed.

Revert 45987. Tests did not pass on Windows.

  • fast/forms/get-file-upload-expected.txt: Removed.
  • fast/forms/get-file-upload.html: Removed.
4:38 PM Changeset in webkit [45998] by pkasting@chromium.org
  • 2 edits in trunk/WebKitTools

2009-07-16 Peter Kasting <​pkasting@google.com>

Reviewed by David Kilzer.

​https://bugs.webkit.org/show_bug.cgi?id=27323
Improve support for WebKit checkouts hosted inside other checkouts
(possible for some ports, e.g. Chromium).

  • Scripts/svn-create-patch: Determine SVN root by looking for Repository Root string and aborting when it's missing or different than what we've already seen.
4:29 PM Changeset in webkit [45997] by levin@chromium.org
  • 4 edits in trunk/WebCore

2009-07-16 Drew Wilson <​atwilson@google.com>

Reviewed by David Levin.

Added SHARED_WORKER flag to Windows build files, as well as associated .ccp/.h files.
Added missing V8 bindings to the AllInOne file

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

  • WebCore.vcproj/WebCore.vcproj: Added missing files to build.
  • bindings/v8/DerivedSourcesAllInOne.cpp: Added missing V8 bindings (V8AbstractWorker.cpp and V8SharedWorker.cpp)
  • DerivedSources.cpp Added missing JS bindings (JSAbstractWorker.cpp and JSSharedWorker.cpp)
4:29 PM Changeset in webkit [45996] by levin@chromium.org
  • 2 edits in trunk/WebCore

2009-07-16 John Abd-El-Malek <​jam@chromium.org>

Reviewed by David Levin.

Add a getter in MessagePortChannel for the PlatformMessagePortChannel.

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

  • dom/MessagePortChannel.h: (WebCore::MessagePortChannel::channel):
4:28 PM Changeset in webkit [45995] by abarth@webkit.org
  • 2 edits in trunk/JavaScriptCore

2009-07-16 Zoltan Horvath <​hzoltan@inf.u-szeged.hu>

Reviewed by Oliver Hunt.

Allow custom memory allocation control in ExceptionInfo and RareData struct
​https://bugs.webkit.org/show_bug.cgi?id=27336

Inherits ExceptionInfo and RareData struct from FastAllocBase because these
have been instantiated by 'new' in JavaScriptCore/bytecode/CodeBlock.cpp:1289 and
in JavaScriptCore/bytecode/CodeBlock.h:453.

Remove unnecessary WTF
namespace from CodeBlock inheritance.
  • bytecode/CodeBlock.h:
4:25 PM Changeset in webkit [45994] by pkasting@chromium.org
  • 1 edit in trunk/WebKitTools/ChangeLog

Fix accidentally blowing away a ChangeLog entry, plus a spurious comment
that no longer applied to the patch version I checked in.

3:55 PM Changeset in webkit [45993] by pkasting@chromium.org
  • 15 edits in trunk

2009-07-16 Peter Kasting <​pkasting@google.com>

Reviewed by Adam Roben.

​https://bugs.webkit.org/show_bug.cgi?id=27323
Improve support for non-Cygwin SVNs on Windows, as well as WebKit
checkouts hosted inside other checkouts.

  • Scripts/commit-log-editor: Modify regex so that trailing whitespace (e.g. \r) isn't included in filenames.
  • Scripts/prepare-ChangeLog: Fix a case of adding "\n" to the ChangeLog without normalizing. Normalize file paths early instead of late so all stages of the script work. Modify regexes so that trailing whitespace (e.g. \r) isn't included in filenames.
  • Scripts/svn-create-patch: Use a regex instead of chomp so we cut off line endings even if they don't match Perl's.
3:50 PM Changeset in webkit [45992] by abarth@webkit.org
  • 3 edits in trunk/WebKit/qt

2009-07-16 Benjamin C Meyer <​benjamin.meyer@torchmobile.com>

Reviewed by Adam Treat.

Add new action to qwebpage to reload without cache.

  • Api/qwebpage.cpp: (QWebPagePrivate::updateAction): (QWebPagePrivate::updateNavigationActions): (QWebPage::triggerAction):
  • Api/qwebpage.h:
3:43 PM Changeset in webkit [45991] by abarth@webkit.org
  • 25 edits in trunk

2009-07-16 Xiaomei Ji <​xji@chromium.org>

Reviewed by Darin Adler.

Fix tooltip does not get its directionality from its element's directionality.
​https://bugs.webkit.org/show_bug.cgi?id=24187

Per mitz's suggestion in comment #6, while getting the plain-text
title, we also get the directionality of the title. How to handle
the directionality is up to clients. Clients could ignore it,
or use attribute or unicode control characters to display the title
as what they want.

WARNING: NO TEST CASES ADDED OR CHANGED

  • WebCore.base.exp: Replace 2 names due to signature change.
  • loader/EmptyClients.h: (WebCore::EmptyChromeClient::setToolTip): Add direction as 2nd parameter.
  • page/Chrome.cpp: (WebCore::Chrome::setToolTip): Calculate tooltip direction as well and pass it to client to take care when display tooltip.
  • page/ChromeClient.h: Add direction as 2nd parameter to pure virtual function setToolTip().
  • page/chromium/ChromeClientChromium.h: (WebCore::ChromeClientChromium::setToolTip): Add setToolTip() temprarily to make chromium compile after pick up this webkit patch.
  • rendering/HitTestResult.cpp: (WebCore::HitTestResult::spellingToolTip): Besides getting the spelling tooltip, get its directionality as well. (WebCore::HitTestResult::title): Besides getting the title, get its directionality as well.
  • rendering/HitTestResult.h: Add 2 more methods.

2009-07-16 Xiaomei Ji <​xji@chromium.org>

Reviewed by Darin Adler.

Fix tooltip does not get its directionality from its element's directionality.
​https://bugs.webkit.org/show_bug.cgi?id=24187

Per mitz's suggestion in comment #6, while getting the plain-text
title, we also get the directionality of the title. How to handle
the directionality is up to clients. Clients could ignore it,
or use attribute or unicode control characters to display the title
as what they want.

  • WebCoreSupport/ChromeClientGtk.cpp: (WebKit::ChromeClient::setToolTip): Add directionality as 2nd parameter to setToopTip() (without handling it yet).
  • WebCoreSupport/ChromeClientGtk.h: Add directionality as 2nd parameter to setToolTip().

2009-07-16 Xiaomei Ji <​xji@chromium.org>

Reviewed by Darin Adler.

Fix tooltip does not get its directionality from its element's directionality.
​https://bugs.webkit.org/show_bug.cgi?id=24187

Per mitz's suggestion in comment #6, while getting the plain-text
title, we also get the directionality of the title. How to handle
the directionality is up to clients. Clients could ignore it,
or use attribute or unicode control characters to display the title
as what they want.

  • Misc/WebElementDictionary.mm: (-[WebElementDictionary _spellingToolTip]): Change spellingToolTip caller due to signature change. (-[WebElementDictionary _title]): Change title caller due to signature change.
  • WebCoreSupport/WebChromeClient.h: Add directionality as 2nd parameter to setToolTip().
  • WebCoreSupport/WebChromeClient.mm: (WebChromeClient::setToolTip): Add directionality as 2nd parameter to setToopTip() (without handling it yet).

2009-07-16 Xiaomei Ji <​xji@chromium.org>

Reviewed by Darin Adler.

Fix tooltip does not get its directionality from its element's directionality.
​https://bugs.webkit.org/show_bug.cgi?id=24187

Per mitz's suggestion in comment #6, while getting the plain-text
title, we also get the directionality of the title. How to handle
the directionality is up to clients. Clients could ignore it,
or use attribute or unicode control characters to display the title
as what they want.

  • WebCoreSupport/ChromeClientQt.cpp: (WebCore::ChromeClientQt::setToolTip): Add directionality as 2nd parameter to setToopTip() (without handling it yet).
  • WebCoreSupport/ChromeClientQt.h: Add directionality as 2nd parameter to setToolTip().

2009-07-16 Xiaomei Ji <​xji@chromium.org>

Reviewed by Darin Adler.

Fix tooltip does not get its directionality from its element's directionality.
​https://bugs.webkit.org/show_bug.cgi?id=24187

Per mitz's suggestion in comment #6, while getting the plain-text
title, we also get the directionality of the title. How to handle
the directionality is up to clients. Clients could ignore it,
or use attribute or unicode control characters to display the title
as what they want.

  • WebCoreSupport/WebChromeClient.cpp: (WebChromeClient::setToolTip): Add directionality as 2nd parameter to setToopTip() (without handling it yet).
  • WebCoreSupport/WebChromeClient.h: Add directionality as 2nd parameter to setToolTip().
  • WebElementPropertyBag.cpp: (WebElementPropertyBag::Read): Change call of spellingToolTip() and title() due to signature change.

2009-07-16 Xiaomei Ji <​xji@chromium.org>

Reviewed by Darin Adler.

Fix tooltip does not get its directionality from its element's directionality.
​https://bugs.webkit.org/show_bug.cgi?id=24187

Per mitz's suggestion in comment #6, while getting the plain-text
title, we also get the directionality of the title. How to handle
the directionality is up to clients. Clients could ignore it,
or use attribute or unicode control characters to display the title
as what they want.

  • WebKitSupport/ChromeClientWx.cpp: (WebCore::ChromeClientWx::setToolTip): Add directionality as 2nd parameter to setToopTip() (without handling it yet).
  • WebKitSupport/ChromeClientWx.h: Add directionality as 2nd parameter to setToolTip().
3:36 PM Changeset in webkit [45990] by mrowe@apple.com
  • 2 edits in trunk/JavaScriptCore

Rubber-stamped by Geoff Garen.

Fix FeatureDefines.xcconfig to not be out of sync with the rest of the world.

  • Configurations/FeatureDefines.xcconfig:
3:23 PM Changeset in webkit [45989] by abarth@webkit.org
  • 3 edits
    3 adds in trunk

2009-07-16 Shinichiro Hamaji <​hamaji@chromium.org>

Reviewed by Oliver Hunt.

[CAIRO] pattern of a canvas-element changes after modifications on canvas-element
​https://bugs.webkit.org/show_bug.cgi?id=20578

Copy pixel image in ImageBuffer::image() just like CG and Skia glue.

  • fast/canvas/canvas-pattern-modify-expected.txt: Added.
  • fast/canvas/canvas-pattern-modify.html: Added.
  • fast/canvas/canvas-pattern-modify.js: Added. (dataToArray): (getPixel): (pixelShouldBe): (createCanvasImage):

2009-07-16 Shinichiro Hamaji <​hamaji@chromium.org>

Reviewed by Oliver Hunt.

[CAIRO] pattern of a canvas-element changes after modifications on canvas-element
​https://bugs.webkit.org/show_bug.cgi?id=20578

Copy pixel image in ImageBuffer::image() just like CG and Skia glue.

Test: fast/canvas/canvas-pattern-modify.html

  • platform/graphics/cairo/ImageBufferCairo.cpp: (copySurface): (WebCore::ImageBuffer::image):
3:15 PM Changeset in webkit [45988] by hyatt@apple.com
  • 3 edits
    4 adds in trunk

WebCore:

2009-07-16 David Hyatt <​hyatt@apple.com>

Reviewed by Beth Dakin.

​https://bugs.webkit.org/show_bug.cgi?id=27353
Images mispositioned because of bug in percentage-based relative positioning.

Added fast/css/nested-floating-relative-position-percentages.html

  • rendering/RenderBoxModelObject.cpp: (WebCore::RenderBoxModelObject::relativePositionOffsetX):

LayoutTests:

2009-07-16 David Hyatt <​hyatt@apple.com>

Reviewed by Beth Dakin.

​https://bugs.webkit.org/show_bug.cgi?id=27353
Images mispositioned because of bug in percentage-based relative positioning.

  • fast/css/nested-floating-relative-position-percentages.html: Added.
  • platform/mac/fast/css/nested-floating-relative-position-percentages-expected.checksum: Added.
  • platform/mac/fast/css/nested-floating-relative-position-percentages-expected.png: Added.
  • platform/mac/fast/css/nested-floating-relative-position-percentages-expected.txt: Added.
3:02 PM Changeset in webkit [45987] by abarth@webkit.org
  • 3 edits
    2 adds in trunk

2009-07-16 Kent Tamura <​tkent@chromium.org>

Reviewed by Eric Seidel.

A test for for non-multipart file upload form submission.
<​https://bugs.webkit.org/show_bug.cgi?id=26505>

  • fast/forms/get-file-upload-expected.txt: Added.
  • fast/forms/get-file-upload.html: Added.

2009-07-16 Kent Tamura <​tkent@chromium.org>

Reviewed by Eric Seidel.

Sends the basename of a selected file for non-multipart form submission.
<​https://bugs.webkit.org/show_bug.cgi?id=26505>

Test: fast/forms/get-file-upload.html

  • html/HTMLInputElement.cpp: (WebCore::HTMLInputElement::appendFormData):
2:52 PM Changeset in webkit [45986] by Simon Fraser
  • 3 edits in trunk/WebCore

2009-07-16 Simon Fraser <Simon Fraser>

Reviewed by Darin Adler.

Video size sometimes jumps just after the video starts loading
​https://bugs.webkit.org/show_bug.cgi?id=27352

Ensure that the media player is at or after the 'HaveMetadata' state so that
the instrinsic size is known before we create the layer for video. This avoids
a flash caused by computing the video rect using the default intrinsic size, and then
re-computing it when that size changes.

  • platform/graphics/mac/MediaPlayerPrivateQTKit.h:
  • platform/graphics/mac/MediaPlayerPrivateQTKit.mm: (WebCore::MediaPlayerPrivate::isReadyForRendering): (WebCore::MediaPlayerPrivate::updateStates): (WebCore::MediaPlayerPrivate::supportsAcceleratedRendering):
2:17 PM Changeset in webkit [45985] by abarth@webkit.org
  • 2 edits in trunk/WebKitTools

2009-07-16 Joseph Pecoraro <​joepeck02@gmail.com>

Reviewed by Darin Adler.

WebKitTools/Scripts/svn-create-patch is broken
​https://bugs.webkit.org/show_bug.cgi?id=27328

  • Scripts/svn-create-patch: one line fix for unusual perl behavior
1:57 PM Changeset in webkit [45984] by abarth@webkit.org
  • 2 edits in trunk/LayoutTests/http/tests/security/isolatedWorld

Unreviewed typo fix in two tests.

1:26 PM Changeset in webkit [45983] by abarth@webkit.org
  • 5 edits
    31 adds in trunk/LayoutTests

2009-07-15 Adam Barth <​abarth@webkit.org>

Reviewed by Darin Fisher.

Add tests for isolated worlds
​https://bugs.webkit.org/show_bug.cgi?id=27331

These tests are currently disabled on JSC ports.

  • http/tests/security/isolatedWorld/all-window-properties-expected.txt: Added.
  • http/tests/security/isolatedWorld/all-window-properties.html: Added.
  • http/tests/security/isolatedWorld/all-window-prototypes-expected.txt: Added.
  • http/tests/security/isolatedWorld/all-window-prototypes.html: Added.
  • http/tests/security/isolatedWorld/body-properties-expected.txt: Added.
  • http/tests/security/isolatedWorld/body-properties.html: Added.
  • http/tests/security/isolatedWorld/body-prototype-expected.txt: Added.
  • http/tests/security/isolatedWorld/body-prototype.html: Added.
  • http/tests/security/isolatedWorld/document-properties-expected.txt: Added.
  • http/tests/security/isolatedWorld/document-properties.html: Added.
  • http/tests/security/isolatedWorld/document-prototype-expected.txt: Added.
  • http/tests/security/isolatedWorld/document-prototype.html: Added.
  • http/tests/security/isolatedWorld/global-variables-expected.txt: Added.
  • http/tests/security/isolatedWorld/global-variables.html: Added.
  • http/tests/security/isolatedWorld/image-properties-expected.txt: Added.
  • http/tests/security/isolatedWorld/image-properties.html: Added.
  • http/tests/security/isolatedWorld/image-prototype-expected.txt: Added.
  • http/tests/security/isolatedWorld/image-prototype.html: Added.
  • http/tests/security/isolatedWorld/location-properties-expected.txt: Added.
  • http/tests/security/isolatedWorld/location-properties.html: Added.
  • http/tests/security/isolatedWorld/location-prototype-expected.txt: Added.
  • http/tests/security/isolatedWorld/location-prototype.html: Added.
  • http/tests/security/isolatedWorld/number-prototype-expected.txt: Added.
  • http/tests/security/isolatedWorld/number-prototype.html: Added.
  • http/tests/security/isolatedWorld/object-prototype-expected.txt: Added.
  • http/tests/security/isolatedWorld/object-prototype.html: Added.
  • http/tests/security/isolatedWorld/string-prototype-expected.txt: Added.
  • http/tests/security/isolatedWorld/string-prototype.html: Added.
  • http/tests/security/isolatedWorld/window-properties-expected.txt: Added.
  • http/tests/security/isolatedWorld/window-properties.html: Added.
  • platform/gtk/Skipped:
  • platform/mac/Skipped:
  • platform/qt/Skipped:
  • platform/win/Skipped:
1:01 PM Changeset in webkit [45982] by mrowe@apple.com
  • 2 edits in branches/safari-4-branch/WebCore

Merge r45979.

12:23 PM Changeset in webkit [45981] by levin@chromium.org
  • 3 edits in trunk/WebKitTools

2009-07-16 David Levin <​levin@chromium.org>

Reviewed by David Kilzer.

cpplint should check for equality comparisons to 0/true/false
​https://bugs.webkit.org/show_bug.cgi?id=27333

  • Scripts/modules/cpplint.py: Added the new lint check.
  • Scripts/modules/cpplint_unittest.py: Add tests for the new lint check and fix a regex that in another unit test that caused it to fail when you have a directory with a number in it (like WebKit-2)
11:36 AM Changeset in webkit [45980] by Simon Fraser
  • 1 edit
    7 adds in trunk/WebKitSite

2009-07-16 Simon Fraser <Simon Fraser>

New demo files for an up-coming blog post.

  • blog-files/3d-transforms/mighty-cubes.png: Added.
  • blog-files/3d-transforms/morphing-cubes.html: Added.
  • blog-files/3d-transforms/perspective-by-example.html: Added.
  • blog-files/3d-transforms/poster-circle.html: Added.
  • blog-files/3d-transforms/poster-circle.png: Added.
  • blog-files/3d-transforms/transform-style.html: Added.
11:35 AM Changeset in webkit [45979] by beidson@apple.com
  • 2 edits in trunk/WebCore

2009-07-16 Brady Eidson <​beidson@apple.com>

Reviewed by Antti Koivisto.
Patch by Brady Eidson and Alexey Proskuryakov.

​https://bugs.webkit.org/show_bug.cgi?id=26496
<rdar://problem/7065391> Microsoft Outlook Web Access fails because XHR stream connection blocks script loading/revalidation

After refreshing a page and when all CachedResources on that page are in validation mode, they got an exemption
from the connection-per-host limit. Removing that exemption makes the test case load smoothly after reloads.

  • loader/loader.cpp: (WebCore::Loader::Host::servePendingRequests): Remove the resourceIsCacheValidator exemption to the connection-per-host limit.
10:26 AM Changeset in webkit [45978] by ddkilzer@apple.com
  • 2 edits in trunk/WebKitTools

<​http://webkit.org/b/27241> bugzilla-tool post-commits silently fails with bad args

Reviewed by Eric Seidel.

  • Scripts/bugzilla-tool: (PostCommitsAsPatchesToBug.execute): Added more error checking when consuming arguments.
10:21 AM Changeset in webkit [45977] by abarth@webkit.org
  • 2 edits in trunk/WebCore

2009-07-16 Adam Barth <​abarth@webkit.org>

Reviewed by Dimitri Glazkov.

[V8] V8IsolatedWorld::evaluate needs to call didCreateIsolatedScriptContext
​https://bugs.webkit.org/show_bug.cgi?id=27335

evaluateInNewContext makes this delegate call. evaluateInNewWorld
needs to make the same call. This does not appear to be testable with
our current technology.

  • bindings/v8/V8IsolatedWorld.cpp: (WebCore::V8IsolatedWorld::evaluate):
10:19 AM Changeset in webkit [45976] by ddkilzer@apple.com
  • 2 edits in trunk/WebKitTools

TOOL FIX: scm.py: CommitMessage.message is ambiguous

Fixes this error when running bugzilla-tool apply-patches
--local-commit:

File "bugzilla-tool", line 188, in apply_patches

scm.commit_locally_with_message(commit_message.message() or patchname)

TypeError: 'list' object is not callable

  • Scripts/modules/scm.py: After r45940 (and r45971), rename CommitMessage.message attribute to CommitMessage.message_lines. (CommitMessage.init): (CommitMessage.body): (CommitMessage.description): (CommitMessage.message): (CommitMessage.parse_bug_id):
9:43 AM Changeset in webkit [45975] by Simon Hausmann
  • 2 edits in trunk

2009-07-16 Laszlo Gombos <Laszlo Gombos>

Reviewed by Simon Hausmann.

[Qt] Enable GNU compiler extensions to the ARM compiler
for all Qt ports using RVCT
​https://bugs.webkit.org/show_bug.cgi?id=27348

  • WebKit.pri:
8:43 AM Changeset in webkit [45974] by Simon Hausmann
  • 2 edits in trunk/WebCore

2009-07-15 Jakub Wieczorek <​faw217@gmail.com>

Reviewed by Simon Hausmann.

Fix a typo: application/atom=xml -> application/atom+xml.

  • dom/ProcessingInstruction.cpp: (WebCore::ProcessingInstruction::checkStyleSheet):
8:10 AM Changeset in webkit [45973] by Dimitri Glazkov
  • 6 edits in trunk

WebCore:

2009-07-16 Dean McNamee <​deanm@chromium.org>

Reviewed by Oliver Hunt.

​https://bugs.webkit.org/show_bug.cgi?id=27292
Improve handling of <canvas> path operations on an empty path.
Implement Skia's Path::hasCurrentPoint().

  • html/CanvasRenderingContext2D.cpp: (WebCore::CanvasRenderingContext2D::lineTo): (WebCore::CanvasRenderingContext2D::quadraticCurveTo): (WebCore::CanvasRenderingContext2D::bezierCurveTo):
  • platform/graphics/skia/PathSkia.cpp: (WebCore::Path::hasCurrentPoint):

LayoutTests:

2009-07-16 Dean McNamee <​deanm@chromium.org>

Reviewed by Oliver Hunt.

​https://bugs.webkit.org/show_bug.cgi?id=27292
Improve handling of <canvas> path operations on an empty path.
Implement Skia's Path::hasCurrentPoint().

  • fast/canvas/canvas-modify-emptyPath-expected.txt:
  • fast/canvas/resources/canvas-modify-emptyPath.js:
7:13 AM Changeset in webkit [45972] by treat@webkit.org
  • 2 edits in trunk/JavaScriptCore

2009-07-16 Yong Li <​yong.li@torchmobile.com>

Reviewed by George Staikos.

​https://bugs.webkit.org/show_bug.cgi?id=27320
_countof is only included in CE6; for CE5 we need to define it ourself

  • wtf/Platform.h:
6:43 AM Changeset in webkit [45971] by ddkilzer@apple.com
  • 2 edits in trunk/WebKitTools

TOOL FIX: bugzilla-tool: import CommitMessage class

  • Scripts/bugzilla-tool: After r45940, the CommitMessage class needs to be imported for commit_message_for_this_commit().
6:15 AM Changeset in webkit [45970] by Simon Hausmann
  • 2 edits in trunk/LayoutTests

2009-07-16 Simon Hausmann <​simon.hausmann@nokia.com>

Reviewed by Adam Treat.

Skip two tests for the Qt DRT to keep the results green.

fast/js/instanceof-operator.html because it requires XSLT support.

editing/selection/extend-selection-after-double-click.html needs
investigation for the Qt port after r45945

  • platform/qt/Skipped:
4:16 AM Changeset in webkit [45969] by oliver@apple.com
  • 4 edits in trunk/JavaScriptCore

2009-07-16 Zoltan Herczeg <​zherczeg@inf.u-szeged.hu>

Reviewed by Oliver Hunt.

Workers + garbage collector: weird crashes
​https://bugs.webkit.org/show_bug.cgi?id=27077

We need to unlink cached method call sites when a function is destroyed.

  • JavaScriptCore.xcodeproj/project.pbxproj:
  • bytecode/CodeBlock.cpp: (JSC::CodeBlock::~CodeBlock): (JSC::CodeBlock::unlinkCallers):
  • jit/JIT.cpp: (JSC::JIT::unlinkMethodCall):
  • jit/JIT.h:
1:17 AM Changeset in webkit [45968] by levin@chromium.org
  • 2 edits in trunk/WebKitTools

2009-07-16 David Levin <​levin@chromium.org>

Reviewed by Maciej Stachowiak.

prepare-ChangeLog should display the --bug option in its help text.
​https://bugs.webkit.org/show_bug.cgi?id=27334

  • Scripts/prepare-ChangeLog: Added the help text.

Jul 15, 2009:

11:10 PM Changeset in webkit [45967] by levin@chromium.org
  • 2 edits in trunk/LayoutTests

2009-07-15 David Levin <​levin@chromium.org>

Rubber-stamped by Oliver Hunt.

Setting white-space and word-wrap via CSS in textarea doesn't override the wrap attribute
​https://bugs.webkit.org/show_bug.cgi?id=26254

Change the expected Windows result (similar to the mac result change).

  • platform/win/fast/forms/basic-textareas-expected.txt:
10:32 PM Changeset in webkit [45966] by levin@chromium.org
  • 9 edits in trunk

2009-07-15 Shinichiro Hamaji <​hamaji@chromium.org>

Reviewed by Eric Seidel.

Setting white-space and word-wrap via CSS in textarea doesn't override the wrap attribute
​https://bugs.webkit.org/show_bug.cgi?id=26254

WebCore:

Make it so that setting white-space and word-wrap via CSS
overrides the wrap attribute.

This involves having the shadow div in the textarea inherit
the CSS from its parent instead of hard-coding it in
RenderTextControlMultiline.

Committer note: Earlier I reverted this change because I did it incorrectly
by leaving out css/html.css. In the patch, the filename was the old name
css/html4.css and that led to my error.

  • css/html.css:
  • html/HTMLTextAreaElement.cpp: (WebCore::HTMLTextAreaElement::parseMappedAttribute):
  • rendering/RenderTextControlMultiLine.cpp: (WebCore::RenderTextControlMultiLine::createInnerTextStyle):

LayoutTests:

Make it so that setting white-space and word-wrap via CSS
overrides the wrap attribute.

This involves having the shadow div in the textarea inherit
the CSS from its parent instead of hard-coding it in
RenderTextControlMultiline.

  • fast/forms/basic-textareas.html:
  • platform/mac/fast/forms/basic-textareas-expected.checksum:
  • platform/mac/fast/forms/basic-textareas-expected.png:
  • platform/mac/fast/forms/basic-textareas-expected.txt:
10:03 PM Changeset in webkit [45965] by abarth@webkit.org
  • 2 edits in trunk/WebCore

2009-07-15 James Hawkins <​jhawkins@google.com>

Reviewed by Adam Barth.

[V8] Remove a local variable that is shadowing a function parameter.
​https://bugs.webkit.org/show_bug.cgi?id=27309

No test required as this modification does not change the current behavior.

  • bindings/v8/V8DOMWrapper.cpp: (WebCore::V8DOMWrapper::instantiateV8Object):
7:28 PM Changeset in webkit [45964] by agl@chromium.org
  • 3 edits
    2 deletes in trunk/WebCore

2009-07-15 Adam Langley <​agl@google.com>

No review: reverting previous change.

Revert r45959:

2009-07-15 Dumitru Daniliuc <​dumi@chromium.org>
Reviewed by Dimitri Glazkov.

Adding the Win SQLite VFS implementation for Chromium.

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

The Chromium side of this patch was landed in 20839, but broke the build. It
was reverted in r20840. Thus, I'm reverting this side of the patch too.

  • WebCore.gypi:
  • platform/chromium/ChromiumBridge.h:
  • platform/sql/chromium/SQLiteFileSystemChromium.cpp: Removed.
  • platform/sql/chromium/SQLiteFileSystemChromiumWin.cpp: Removed.
5:14 PM Changeset in webkit [45963] by levin@chromium.org
  • 8 edits in trunk

WebCore:

2009-07-15 David Levin <​levin@chromium.org>

Layout test fix, reverting previous change.

Reverting r45962 as it caused several layout test failures.

  • html/HTMLTextAreaElement.cpp: (WebCore::HTMLTextAreaElement::parseMappedAttribute):
  • rendering/RenderTextControlMultiLine.cpp: (WebCore::RenderTextControlMultiLine::createInnerTextStyle):

LayoutTests:

2009-07-15 David Levin <​levin@chromium.org>

Layout test fix, reverting previous change.

Reverting r45962 as it caused several layout test failures.

  • fast/forms/basic-textareas.html:
  • platform/mac/fast/forms/basic-textareas-expected.checksum:
  • platform/mac/fast/forms/basic-textareas-expected.png:
  • platform/mac/fast/forms/basic-textareas-expected.txt:
4:46 PM Changeset in webkit [45962] by levin@chromium.org
  • 8 edits in trunk

2009-06-08 Shinichiro Hamaji <​hamaji@chromium.org>

Reviewed by Eric Seidel.

Setting white-space and word-wrap via CSS in textarea doesn't override the wrap attribute
​https://bugs.webkit.org/show_bug.cgi?id=26254

WebCore:

Make it so that setting white-space and word-wrap via CSS
overrides the wrap attribute.

This involves having the shadow div in the textarea inherit
the CSS from its parent instead of hard-coding it in
RenderTextControlMultiline.

  • css/html4.css:
  • html/HTMLTextAreaElement.cpp: (WebCore::HTMLTextAreaElement::parseMappedAttribute):
  • rendering/RenderTextControlMultiLine.cpp: (WebCore::RenderTextControlMultiLine::createInnerTextStyle):

LayoutTests:

Make it so that setting white-space and word-wrap via CSS
overrides the wrap attribute.

This involves having the shadow div in the textarea inherit
the CSS from its parent instead of hard-coding it in
RenderTextControlMultiline.

  • fast/forms/basic-textareas.html:
  • platform/mac/fast/forms/basic-textareas-expected.checksum:
  • platform/mac/fast/forms/basic-textareas-expected.png:
  • platform/mac/fast/forms/basic-textareas-expected.txt:
4:35 PM Changeset in webkit [45961] by mrowe@apple.com
  • 4 edits in branches/safari-4-branch

Versioning.

4:34 PM Changeset in webkit [45960] by mrowe@apple.com
  • 1 copy in tags/Safari-6531.5

New tag.

4:18 PM Changeset in webkit [45959] by Nate Chapin
  • 3 edits
    3 adds in trunk/WebCore

2009-07-15 Dumitru Daniliuc <​dumi@chromium.org>

Reviewed by Dimitri Glazkov.

Adding the Win SQLite VFS implementation for Chromium.

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

4:15 PM Changeset in webkit [45958] by jianli@chromium.org
  • 13 edits in trunk

WebCore:

2009-07-15 Jian Li <​jianli@chromium.org>

Reviewed by David Levin.

Bug 25151 - workers that fail to load scripts not firing error event.
​https://bugs.webkit.org/show_bug.cgi?id=25151

This fixes the problem that an error event is not fired when the worker
script fails to load. Some reasons this may occur are an invalid URL for
the worker script or a cross-origin redirect.

We also moves the code to complete the URL and check its origin from
Worker constructor to WorkerScriptLoader loading functions in order to
move the exception throwing logic out of the scope of Worker constructor.
Due to this change, we also remove the output ExceptionCode parameter
in the worker constructor. Corresponding JS/V8 binding codes have been
updated to reflect this change.

  • bindings/js/JSWorkerConstructor.cpp: (WebCore::constructWorker):
  • bindings/v8/custom/V8WorkerCustom.cpp: (WebCore::CALLBACK_FUNC_DECL):
  • workers/Worker.cpp: (WebCore::Worker::Worker): (WebCore::Worker::notifyFinished):
  • workers/Worker.h: (WebCore::Worker::create):
  • workers/WorkerContext.cpp: (WebCore::WorkerContext::importScripts):
  • workers/WorkerScriptLoader.cpp: (WebCore::toCrossOriginRedirectPolicy): (WebCore::WorkerScriptLoader::loadSynchronously): (WebCore::WorkerScriptLoader::loadAsynchronously): (WebCore::notifyLoadErrorTask): (WebCore::WorkerScriptLoader::createResourceRequest): (WebCore::WorkerScriptLoader::didFail): (WebCore::WorkerScriptLoader::didFailRedirectCheck): (WebCore::WorkerScriptLoader::didReceiveAuthenticationCancellation): (WebCore::WorkerScriptLoader::notifyError):
  • workers/WorkerScriptLoader.h: (WebCore::): (WebCore::WorkerScriptLoader::url):

LayoutTests:

2009-07-15 Jian Li <​jianli@chromium.org>

Reviewed by David Levin.

Bug 25151 - workers that fail to load scripts not firing error event.
​https://bugs.webkit.org/show_bug.cgi?id=25151

Updates layout test cases for bug 25151.

  • fast/workers/worker-constructor-expected.txt:
  • fast/workers/worker-constructor.html:
  • http/tests/workers/worker-redirect-expected.txt:
  • http/tests/workers/worker-redirect.html:
4:11 PM Changeset in webkit [45957] by mrowe@apple.com
  • 4 edits in branches/safari-4-branch

Versioning.

4:11 PM Changeset in webkit [45956] by mitz@apple.com
  • 5 edits
    4 adds in trunk

WebCore:

Reviewed by Dave Hyatt.

text-shadow is not drawn for text with transparent colour
​https://bugs.webkit.org/show_bug.cgi?id=21374

Test: fast/text/shadow-translucent-fill.html

  • rendering/InlineTextBox.cpp: (WebCore::paintTextWithShadows): If the text fill color is not opaque, paint all shadows separately from the text, by casting them from clipped-out opaque text.

LayoutTests:

Reviewed by Dave Hyatt.

text-shadow is not drawn for text with transparent colour
​https://bugs.webkit.org/show_bug.cgi?id=21374

  • fast/text/shadow-translucent-fill.html: Added.
  • platform/mac/fast/css/shadow-multiple-expected.checksum:
  • platform/mac/fast/css/shadow-multiple-expected.png:
  • platform/mac/fast/text/shadow-translucent-fill-expected.checksum: Added.
  • platform/mac/fast/text/shadow-translucent-fill-expected.png: Added.
  • platform/mac/fast/text/shadow-translucent-fill-expected.txt: Added.
4:02 PM Changeset in webkit [45955] by treat@webkit.org
  • 2 edits in trunk/WebCore

2009-07-15 Adam Treat <​adam.treat@torchmobile.com>

Fix the Qt build.

  • html/HTMLAreaElement.cpp:
3:56 PM Changeset in webkit [45954] by darin@chromium.org
  • 18 edits in trunk/WebCore

2009-07-15 Jeremy Orlow <​jorlow@chromium.org>

Reviewed by Darin Fisher.

Cleanup DOM Storage dependencies.
​https://bugs.webkit.org/show_bug.cgi?id=27180

DOM Storage had several unnecessary (and probably unintended)
dependencies. This patch replaces many includes of header files with
forward declaration of classes, making some destructors explicit, and
taking some factories out of the header files.

This will allow things like StorageAreaSync to take a StorageAreaImpl*
(as it should) rather than a StorageArea* which previously weren't
possible because the dependencies were such a tangled mess.

  • storage/LocalStorageTask.cpp: (WebCore::LocalStorageTask::~LocalStorageTask):
  • storage/LocalStorageTask.h:
  • storage/Storage.cpp: (WebCore::Storage::~Storage):
  • storage/Storage.h:
  • storage/StorageArea.cpp:
  • storage/StorageArea.h:
  • storage/StorageAreaImpl.cpp:
  • storage/StorageAreaImpl.h:
  • storage/StorageAreaSync.cpp: (WebCore::StorageAreaSync::~StorageAreaSync):
  • storage/StorageAreaSync.h:
  • storage/StorageEvent.cpp: (WebCore::StorageEvent::create): (WebCore::StorageEvent::StorageEvent):
  • storage/StorageEvent.h:
  • storage/StorageNamespace.h:
  • storage/StorageNamespaceImpl.cpp:
  • storage/StorageNamespaceImpl.h:
  • storage/StorageSyncManager.cpp: (WebCore::StorageSyncManager::~StorageSyncManager):
  • storage/StorageSyncManager.h:
3:33 PM Changeset in webkit [45953] by cmarrin@apple.com
  • 3 edits
    2 adds in trunk

Incorrect animation when trying to duplicate effect of transform-origin

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

The bug is that matrix animation is being used when animating
a list of transform functions that match in the from and to states.
This sometimes works. But because of the way CA does matrix animation
function lists like the one shown in the testcase animate incorrectly.

This fixes the bug by always doing component animation
as long as the function lists match. This allows CA
to animate the components and then recompose the result
into the correct matrix.

3:29 PM Changeset in webkit [45952] by jianli@chromium.org
  • 2 edits in trunk/WebCore

2009-07-15 Albert J. Wong <​ajwong@chromium.org>

Reviewed by David Levin.

Upstream the V8NPObject and NPV8Object build changes for WebCore.gypi.

Add upstreamed V8 bindings files into WebCore.gypi so they can be seen
downstream
​https://bugs.webkit.org/show_bug.cgi?id=27274

Changes the build file for chromium. Test built the chromium tree
to verify.

  • WebCore.gypi:
3:24 PM Changeset in webkit [45951] by mrowe@apple.com
  • 2 edits in trunk/WebCore

I like it when the code compiles.

  • WebCore.base.exp:
3:07 PM Changeset in webkit [45950] by sfalken@apple.com
  • 3 edits in branches/safari-4-branch/JavaScriptCore

Merge 45949.

3:02 PM Changeset in webkit [45949] by sfalken@apple.com
  • 3 edits in trunk/JavaScriptCore

2009-07-15 Steve Falkenburg <​sfalken@apple.com>

Windows Build fix.

Visual Studio reset our intermediate directory on us.
This sets it back.


2:56 PM Changeset in webkit [45948] by sfalken@apple.com
  • 16 edits in branches/safari-4-branch

Merge r45892.

2:50 PM Changeset in webkit [45947] by Darin Adler
  • 40 edits in trunk

WebCore:

2009-07-15 Darin Adler <Darin Adler>

Reviewed by Sam Weinig.

Renamed parseURL to deprecatedParseURL.

  • bindings/js/JSAttrCustom.cpp: (WebCore::JSAttr::setValue): Renamed.
  • bindings/js/JSElementCustom.cpp: (WebCore::allowSettingSrcToJavascriptURL): Renamed.
  • bindings/js/JSHTMLFrameElementCustom.cpp: (WebCore::allowSettingJavascriptURL): Renamed.
  • bindings/js/JSHTMLIFrameElementCustom.cpp: (WebCore::JSHTMLIFrameElement::setSrc): Renamed.
  • bindings/objc/DOM.mm: (-[DOMElement _getURLAttribute:]): Renamed.
  • bindings/objc/DOMHTML.mm: (-[DOMHTMLDocument _createDocumentFragmentWithMarkupString:baseURLString:]): Renamed.
  • bindings/v8/custom/V8CustomBinding.cpp: (WebCore::allowSettingFrameSrcToJavascriptUrl): Renamed.
  • css/CSSHelper.cpp: (WebCore::deprecatedParseURL): Renamed.
  • css/CSSHelper.h: Renamed and updated comment.
  • html/HTMLAnchorElement.cpp: (WebCore::HTMLAnchorElement::defaultEventHandler): Renamed. (WebCore::HTMLAnchorElement::parseMappedAttribute): Renamed.
  • html/HTMLBaseElement.cpp: (WebCore::HTMLBaseElement::parseMappedAttribute): Renamed.
  • html/HTMLBodyElement.cpp: (WebCore::HTMLBodyElement::parseMappedAttribute): Renamed.
  • html/HTMLEmbedElement.cpp: (WebCore::HTMLEmbedElement::parseMappedAttribute): Renamed.
  • html/HTMLFormElement.cpp: (WebCore::HTMLFormElement::parseMappedAttribute): Renamed.
  • html/HTMLFrameElementBase.cpp: (WebCore::HTMLFrameElementBase::parseMappedAttribute): Renamed.
  • html/HTMLImageElement.cpp: (WebCore::HTMLImageElement::parseMappedAttribute): Renamed.
  • html/HTMLImageLoader.cpp: (WebCore::HTMLImageLoader::sourceURI): Renamed.
  • html/HTMLLinkElement.cpp: (WebCore::HTMLLinkElement::parseMappedAttribute): Renamed.
  • html/HTMLObjectElement.cpp: (WebCore::HTMLObjectElement::parseMappedAttribute): Renamed.
  • html/HTMLTableElement.cpp: (WebCore::HTMLTableElement::parseMappedAttribute): Renamed.
  • html/HTMLTablePartElement.cpp: (WebCore::HTMLTablePartElement::parseMappedAttribute): Renamed.
  • html/HTMLTokenizer.cpp: (WebCore::HTMLTokenizer::parseTag): Renamed.
  • html/PreloadScanner.cpp: (WebCore::PreloadScanner::processAttribute): Renamed. (WebCore::PreloadScanner::emitCSSRule): Renamed.
  • platform/chromium/ClipboardChromium.cpp: (WebCore::ClipboardChromium::declareAndWriteDragImage): Renamed.
  • platform/chromium/PasteboardChromium.cpp: (WebCore::Pasteboard::writeImage): Renamed.
  • platform/qt/ClipboardQt.cpp: (WebCore::ClipboardQt::declareAndWriteDragImage): Renamed.
  • platform/win/ClipboardWin.cpp: (WebCore::ClipboardWin::declareAndWriteDragImage): Renamed.
  • rendering/HitTestResult.cpp: (WebCore::HitTestResult::absoluteImageURL): Renamed. (WebCore::HitTestResult::absoluteMediaURL): Renamed. (WebCore::HitTestResult::absoluteLinkURL): Renamed.
  • svg/SVGAElement.cpp: (WebCore::SVGAElement::defaultEventHandler): Renamed.
  • svg/SVGImageLoader.cpp: (WebCore::SVGImageLoader::sourceURI): Renamed.
  • wml/WMLAElement.cpp: (WebCore::WMLAElement::defaultEventHandler): Renamed.
  • wml/WMLImageLoader.cpp: (WebCore::WMLImageLoader::sourceURI): Renamed.

WebKit/mac:

2009-07-15 Darin Adler <Darin Adler>

Reviewed by Sam Weinig.

Renamed parseURL to deprecatedParseURL.

  • DOM/WebDOMOperations.mm: (-[DOMDocument URLWithAttributeString:]): Renamed.
  • WebCoreSupport/WebFrameLoaderClient.mm: (WebFrameLoaderClient::createPlugin): Renamed.

WebKit/win:

2009-07-15 Darin Adler <Darin Adler>

Reviewed by Sam Weinig.

Renamed parseURL to deprecatedParseURL.

  • WebCoreSupport/WebFrameLoaderClient.cpp: (WebFrameLoaderClient::createPlugin): Renamed.

WebKitTools:

2009-07-15 Darin Adler <Darin Adler>

  • Scripts/do-webcore-rename: Check in the version used to rename parseURL to deprecatedParseURL.
2:44 PM Changeset in webkit [45946] by Darin Adler
  • 6 edits
    2 adds in trunk

WebCore:

2009-07-15 Darin Adler <Darin Adler>

Reviewed by Dan Bernstein.

CSSHelper.h's parseURL is a function that no one should ever call
Part 1: Eliminate callers in the CSS parser.
​https://bugs.webkit.org/show_bug.cgi?id=26599

Test: fast/css/uri-token-parsing.html

  • css/CSSHelper.h: Added a comment explaining why nobody should ever call this function. A FIXME suggests a next step, which would be to rename it deprecatedParseURL.
  • css/CSSParser.cpp: (WebCore::CSSParser::parseValue): Removed unneeded call to parseURL; CSSParser::text already takes care of parsing the URI token syntax, and the parseURL function does no good. (WebCore::CSSParser::parseContent): Ditto. (WebCore::CSSParser::parseFillImage): Ditto. (WebCore::CSSParser::parseFontFaceSrc): Ditto. (WebCore::CSSParser::parseBorderImage): Ditto. (WebCore::isCSSWhitespace): Added. Helper function that makes the text function easier to read. (WebCore::CSSParser::text): Tweak logic so that leading and trailing whitespace are both trimmed before removing the quote marks. Changed to use the isCSSWhitespace, isASCIIHexDigit, and toASCIIHexValue functions for clarity.
  • css/CSSParser.h: Removed stray "public:" in this header.
  • platform/text/StringImpl.cpp: (WebCore::StringImpl::substring): Optimized the case where the substring covers the entire string, so we just share the StringImpl instead of making a new one. This case came up in earlier versions of the CSS parser changes above. (WebCore::StringImpl::substringCopy): Streamlined the logic here and made it not call substring any more. Before, this was relying on the substring function always making a copy of any non-empty substring.

LayoutTests:

2009-07-15 Darin Adler <Darin Adler>

Reviewed by Dan Bernstein.

CSSHelper.h's parseURL is a function that no one should ever call
Part 1: Eliminate callers in the CSS parser.
​https://bugs.webkit.org/show_bug.cgi?id=26599

  • fast/css/uri-token-parsing-expected.txt: Added.
  • fast/css/uri-token-parsing.html: Added. Tests URI token parsing cases, many of which were handled wrong before due to stripping quotes and URL() twice.
2:35 PM Changeset in webkit [45945] by Darin Adler
  • 3 edits
    1 copy
    1 add in trunk

WebCore:

2009-07-15 Darin Adler <Darin Adler>

Reviewed by John Sullivan.

After double-clicking a word, using Shift-arrow to select behaves unpredictably
​https://bugs.webkit.org/show_bug.cgi?id=27177
rdar://problem/7034324

Test: editing/selection/extend-selection-after-double-click.html

The bug was due to the m_lastChangeWasHorizontalExtension flag, which was not
being cleared in many cases where it should have been.

  • editing/SelectionController.cpp: (WebCore::SelectionController::setSelection): Set m_lastChangeWasHorizontalExtension to false. This catches all sorts of cases that don't flow through the modify function. Before, the flag would reflect the last call to the modify function, which was not necessarily the last selection change. (WebCore::SelectionController::willBeModified): Rearrange function for clarity. Remove code that sets m_lastChangeWasHorizontalExtension; that is now handled elsewhere. (WebCore::SelectionController::modify): Call setLastChangeWasHorizontalExtension after setSelection when setting up a trial selection controller, since setSelection now clears that flag. Also changed both trial selection controller cases to set the flag, although it's not strictly necessary in both cases. Added code to set m_lastChangeWasHorizontalExtension when extending the selection, which used to be handled in willBeModified. Now we need to do it after the selection change.

LayoutTests:

2009-07-15 Darin Adler <Darin Adler>

Reviewed by John Sullivan.

After double-clicking a word, using Shift-arrow to select behaves unpredictably
​https://bugs.webkit.org/show_bug.cgi?id=27177
rdar://problem/7034324

  • editing/selection/extend-selection-after-double-click-expected.txt: Added.
  • editing/selection/extend-selection-after-double-click.html: Copied from LayoutTests/editing/selection/word-granularity.html. Then turned it into a new test.
2:16 PM Changeset in webkit [45944] by Dimitri Glazkov
  • 2 edits in trunk/WebCore

2009-07-15 Jeremy Orlow <​jorlow@chromium.org>

Reviewed by Dimitri Glazkov.

Need to update DOM Storage files in GYPI file.
​https://bugs.webkit.org/show_bug.cgi?id=27317

Need to update DOM Storage files in the GYPI file. They're pretty out
of date and we're on the path towards enabling them for everyone!

  • WebCore.gypi:
1:55 PM Changeset in webkit [45943] by bfulgham@webkit.org
  • 5 edits in trunk

JavaScriptCore:

2009-07-15 Kwang Yul Seo <​skyul@company100.net>

Reviewed by Eric Seidel.

​https://bugs.webkit.org/show_bug.cgi?id=26794
Make Yacc-generated parsers to use fastMalloc/fastFree.


Define YYMALLOC and YYFREE to fastMalloc and fastFree
respectively.

  • parser/Grammar.y:

WebCore:

2009-07-15 Kwang Yul Seo <​skyul@company100.net>

Reviewed by Eric Seidel.

​https://bugs.webkit.org/show_bug.cgi?id=26794
Make Yacc-generated parsers to use fastMalloc/fastFree.


Define YYMALLOC and YYFREE to fastMalloc and fastFree
respectively.

  • css/CSSGrammar.y:
  • xml/XPathGrammar.y:
1:49 PM Changeset in webkit [45942] by kov@webkit.org
  • 2 edits in trunk/WebKit/gtk

2009-07-15 Gustavo Noronha Silva <​gustavo.noronha@collabora.co.uk>

Rubber-stamped by Jan Alonzo.

Fix enable-xss-auditor property documentation, and default value
to match the fact that it is now enabled by default.

  • webkit/webkitwebsettings.cpp: (webkit_web_settings_class_init):
1:43 PM Changeset in webkit [45941] by hyatt@apple.com
  • 11 edits in trunk

WebCore:

2009-07-15 David Hyatt <​hyatt@apple.com>

Reviewed by Adam Roben.

​https://bugs.webkit.org/show_bug.cgi?id=27193
Don't run in to anonymous blocks. No other browsers do this, and our implementation of run-in
is effectively broken as a result.

No new tests. Changed fast/runin/001.html and fast/runin/generated.html to match new behavior.

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

LayoutTests:

2009-07-15 David Hyatt <​hyatt@apple.com>

Reviewed by Adam Roben.

​https://bugs.webkit.org/show_bug.cgi?id=27193
Don't let run-ins run in to anonymous blocks.

  • fast/runin/001.html:
  • fast/runin/generated.html:
  • platform/mac/fast/runin/001-expected.checksum:
  • platform/mac/fast/runin/001-expected.png:
  • platform/mac/fast/runin/001-expected.txt:
  • platform/mac/fast/runin/generated-expected.checksum:
  • platform/mac/fast/runin/generated-expected.png:
  • platform/mac/fast/runin/generated-expected.txt:
1:15 PM Changeset in webkit [45940] by bfulgham@webkit.org
  • 3 edits in trunk/WebKitTools

2009-07-10 David Kilzer <​ddkilzer@apple.com>

bugzilla-tool: create CommitMessage class

Reviewed by Eric Seidel.

Create a CommitMessage class to encapsulate related code.

  • Scripts/bugzilla-tool: (bug_id_from_commit_message): Moved to CommitMessage.parse_bug_id(). (commit_message_for_this_commit): Return a CommitMessage. (ApplyPatchesFromBug.apply_patches): Use CommitMessage.message(). (LandPatchesFromBugs.build_and_commit): Ditto. (CommitMessageForCurrentDiff.execute): Ditto. (PostCommitsAsPatchesToBug.execute): Switched from Git.commit_message_for_commit() to Git.commit_message_for_local_commit(). Switched from bug_id_from_commit_message() to CommitMessage.parse_bug_id().
  • Scripts/modules/scm.py: (first_non_empty_line_after_index): Added. (CommitMessage.init): Added. (CommitMessage.body): Added. (CommitMessage.description): Added. (CommitMessage.message): Added. (CommitMessage.parse_bug_id): Added. Moved from bug_id_from_commit_message() in bugzilla-tool. (Git.commit_message_for_local_commit): Renamed from commit_message_for_commit(). Return a CommitMessage.
1:11 PM Changeset in webkit [45939] by bfulgham@webkit.org
  • 2 edits in trunk/WebKitTools

2009-07-15 Joseph Pecoraro <​joepeck02@gmail.com>

Reviewed by David Kilzer.

bugzilla-tool/svn-apply can't handle patches made from a non-root directory
​https://bugs.webkit.org/show_bug.cgi?id=26999

  • Scripts/svn-create-patch:
1:08 PM Changeset in webkit [45938] by bfulgham@webkit.org
  • 21 edits
    4 adds in trunk

WebCore:

2009-07-15 Yuzo Fujishima <​yuzo@google.com>

Reviewed by Darin Adler.

Test: fast/js/instanceof-operator.html

Fix for: Bug 25205 - XMLHttpRequest instance is not an instanceof XMLHttpRequest
​https://bugs.webkit.org/show_bug.cgi?id=25205

In addition to for XMLHttpRequest, this also fixes for:

  • Audio
  • Image
  • MessageChannel
  • Option
  • WebKitCSSMatrix
  • WebKitPoint
  • Worker
  • XSLTProcessor
  • bindings/js/JSAudioConstructor.cpp: (WebCore::JSAudioConstructor::JSAudioConstructor):
  • bindings/js/JSAudioConstructor.h:
  • bindings/js/JSDOMBinding.h: (WebCore::DOMConstructorObject::createStructure): (WebCore::DOMConstructorObject::DOMConstructorObject):
  • bindings/js/JSImageConstructor.cpp: (WebCore::JSImageConstructor::JSImageConstructor):
  • bindings/js/JSImageConstructor.h:
  • bindings/js/JSMessageChannelConstructor.cpp: (WebCore::JSMessageChannelConstructor::JSMessageChannelConstructor):
  • bindings/js/JSMessageChannelConstructor.h:
  • bindings/js/JSOptionConstructor.cpp: (WebCore::JSOptionConstructor::JSOptionConstructor):
  • bindings/js/JSOptionConstructor.h:
  • bindings/js/JSWebKitCSSMatrixConstructor.cpp: (WebCore::JSWebKitCSSMatrixConstructor::JSWebKitCSSMatrixConstructor):
  • bindings/js/JSWebKitCSSMatrixConstructor.h:
  • bindings/js/JSWebKitPointConstructor.cpp: (WebCore::JSWebKitPointConstructor::JSWebKitPointConstructor):
  • bindings/js/JSWebKitPointConstructor.h:
  • bindings/js/JSWorkerConstructor.cpp: (WebCore::JSWorkerConstructor::JSWorkerConstructor):
  • bindings/js/JSWorkerConstructor.h:
  • bindings/js/JSXMLHttpRequestConstructor.cpp: (WebCore::JSXMLHttpRequestConstructor::JSXMLHttpRequestConstructor):
  • bindings/js/JSXMLHttpRequestConstructor.h:
  • bindings/js/JSXSLTProcessorConstructor.cpp: (WebCore::JSXSLTProcessorConstructor::JSXSLTProcessorConstructor):
  • bindings/js/JSXSLTProcessorConstructor.h:

LayoutTests:

2009-07-15 Yuzo Fujishima <​yuzo@google.com>

Reviewed by Darin Adler.

Fix for: Bug 25205 - XMLHttpRequest instance is not an instanceof XMLHttpRequest
​https://bugs.webkit.org/show_bug.cgi?id=25205

In addition to for XMLHttpRequest, this also fixes for:

  • Audio
  • Image
  • MessageChannel
  • Option
  • WebKitCSSMatrix
  • WebKitPoint
  • Worker
  • XSLTProcessor
  • fast/js/instanceof-operator-expected.txt: Added.
  • fast/js/instanceof-operator.html: Added.
  • fast/js/resources/instanceof-operator-dummy-worker.js: Added.
  • fast/js/resources/instanceof-operator.js: Added.
12:51 PM Changeset in webkit [45937] by bfulgham@webkit.org
  • 5 edits
    4 adds in trunk

WebCore:

2009-07-15 Kai Bruning <​kai@granus.net>

Reviewed by Dave Hyatt.

CSS21 attribute selectors not dynamic for xml.
​https://bugs.webkit.org/show_bug.cgi?id=25072

Moved the relevant test in StyledElement::attributeChanged()
to a new function Element::recalcStyleIfNeededAfterAttributeChanged()
so it can be called from both StyledElement::attributeChanged()
and Element::attributeChanged().
Refactored Element::attributeChanged() into
Element::updateAfterAttributeChanged() and
Element::recalcStyleIfNeededAfterAttributeChanged(), which are called
separately from StyledElement::attributeChanged().

Test: fast/css/attribute-selector-dynamic.xml

  • dom/Element.cpp: (WebCore::Element::attributeChanged): (WebCore::Element::updateAfterAttributeChanged): (WebCore::Element::recalcStyleIfNeededAfterAttributeChanged):
  • dom/Element.h:
  • dom/StyledElement.cpp: (WebCore::StyledElement::attributeChanged):

LayoutTests:

2009-07-15 Kai Bruning <​kai@granus.net>

Reviewed by Dave Hyatt.

CSS21 attribute selectors not dynamic for xml.
​https://bugs.webkit.org/show_bug.cgi?id=25072

Test that CSS21 attribute selectors take effect when the attribute is
dynamically changed in an xml dom.

  • fast/css/attribute-selector-dynamic.xml: Added.
  • platform/mac/fast/css/attribute-selector-dynamic-expected.checksum: Added.
  • platform/mac/fast/css/attribute-selector-dynamic-expected.png: Added.
  • platform/mac/fast/css/attribute-selector-dynamic-expected.txt: Added.
12:27 PM Changeset in webkit [45936] by levin@chromium.org
  • 2 edits in trunk/WebCore

2009-07-15 Alpha Lam <​hclam@chromium.org>

Reviewed by David Levin.

[V8] Layout test failures for drawImage in Canvas
​https://bugs.webkit.org/show_bug.cgi?id=27311

Fixing several canvas layout tests failures due to a
missing return statement in CanvasRenderingContext2DDrawImage() which
was accidentally removed in r45929.

  • bindings/v8/custom/V8CanvasRenderingContext2DCustom.cpp:
12:16 PM Changeset in webkit [45935] by bfulgham@webkit.org
  • 2 edits in trunk/WebCore

2009-07-15 Robert Hogan <​robert@roberthogan.net>

Reviewed by Eric Seidel.

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

If the httpMethod() of the request passed to SubresourceLoader::create is not
supported by the client we must expect to call didFail() while m_loader is still null.

  • loader/DocumentThreadableLoader.cpp: (DocumentThreadableLoader::didFail):Changed.
11:34 AM Changeset in webkit [45934] by abarth@webkit.org
  • 2 edits in trunk/WebKitSite

2009-07-15 Joseph Pecoraro <​joepeck02@gmail.com>

Reviewed by Sam Weinig.

Side bar linking to "DOM Interfaces" returns 404
​https://bugs.webkit.org/show_bug.cgi?id=24000

  • nav.inc:
11:28 AM Changeset in webkit [45933] by Darin Adler
  • 2 edits in trunk/JavaScriptCore

2009-07-15 Darin Adler <Darin Adler>

Fix a build for a particular Apple configuration.

  • wtf/FastAllocBase.h: Change include to use "" style for including another wtf header. This is the style we use for including other public headers in the same directory.
11:25 AM Changeset in webkit [45932] by levin@chromium.org
  • 1 edit in trunk/WebCore/ChangeLog

Add bug link to changelog entry.

11:23 AM Changeset in webkit [45931] by treat@webkit.org
  • 2 edits in trunk/JavaScriptCore

2009-07-15 George Staikos <​george.staikos@torchmobile.com>

Reviewed by Adam Treat.

​https://bugs.webkit.org/show_bug.cgi?id=27303
Implement createThreadInternal for WinCE.
Contains changes by George Staikos <​george.staikos@torchmobile.com> and Joe Mason <​joe.mason@torchmobile.com>

  • wtf/ThreadingWin.cpp: (WTF::createThreadInternal):
11:22 AM Changeset in webkit [45930] by mrowe@apple.com
  • 6 edits in trunk/WebCore

Fix the Mac build.

  • WebCore.base.exp:
  • css/MediaQueryEvaluator.cpp:
  • rendering/SVGRenderTreeAsText.cpp:
  • rendering/style/SVGRenderStyle.cpp:
  • svg/graphics/SVGPaintServer.cpp:
11:21 AM Changeset in webkit [45929] by levin@chromium.org
  • 2 edits in trunk/WebCore

2009-07-15 Alpha Lam <​hclam@chromium.org>

Reviewed by Dimitri Glazkov.

Changed CanvasRenderingContext2DDrawImage() to accept HTMLVideoElement
as a parameter of drawImage() for HTMLCanvasElement.

  • bindings/v8/custom/V8CanvasRenderingContext2DCustom.cpp:
11:14 AM Changeset in webkit [45928] by abarth@webkit.org
  • 7 edits in trunk/WebCore

2009-07-15 Adam Barth <​abarth@webkit.org>

Reviewed by Dimitri Glazkov.

[V8] Fix isolated world constructors
​https://bugs.webkit.org/show_bug.cgi?id=27287

Don't enter V8Proxy::m_context before creating DOM constructors.
Instead, use getWrapperContext to get the right context.

After this patch, all my tests pass. I'll enable the feature
downstream and land the tests.

  • bindings/scripts/CodeGeneratorV8.pm:
  • bindings/v8/V8DOMWrapper.cpp: (WebCore::V8DOMWrapper::getConstructor): (WebCore::V8DOMWrapper::lookupDOMWrapper):
  • bindings/v8/V8DOMWrapper.h:
  • bindings/v8/V8IsolatedWorld.cpp: (WebCore::V8IsolatedWorld::evaluate):
  • bindings/v8/V8Proxy.cpp: (WebCore::V8Proxy::createWrapperFromCache): (WebCore::V8Proxy::isContextInitialized): (WebCore::V8Proxy::disposeContextHandles): (WebCore::V8Proxy::installDOMWindow): (WebCore::V8Proxy::initContextIfNeeded): (WebCore::V8Proxy::getHiddenObjectPrototype): (WebCore::V8Proxy::installHiddenObjectPrototype):
  • bindings/v8/V8Proxy.h:
11:12 AM Changeset in webkit [45927] by treat@webkit.org
  • 2 edits in trunk/JavaScriptCore

2009-07-15 Joe Mason <​joe.mason@torchmobile.com>

Reviewed by George Staikos.

​https://bugs.webkit.org/show_bug.cgi?id=27298
Platform defines for WINCE.
Contains changes by Yong Li <​yong.li@torchmobile.com>,
George Staikos <​george.staikos@torchmobile.com> and Joe Mason <​joe.mason@torchmobile.com>

  • wtf/Platform.h:
11:07 AM Changeset in webkit [45926] by bfulgham@webkit.org
  • 2 edits in trunk/WebCore

2009-07-15 Antonio Gomes <​antonio.gomes@openbossa.org>

Reviewed by Darin Adler.

useless null-check statement in visible_units.cpp@logicalStartOfLine
​https://bugs.webkit.org/show_bug.cgi?id=27154

Simple fix.

  • editing/visible_units.cpp: (WebCore::logicalStartOfLine): Doubled honorEditableBoundaryAtOrAfter() call removed.
10:44 AM Changeset in webkit [45925] by bweinstein@apple.com
  • 1 edit in trunk/WebCore/WebCore.base.exp

Attempted Mac Build Fix.

10:34 AM Changeset in webkit [45924] by treat@webkit.org
  • 2 edits in trunk/JavaScriptCore

2009-07-15 Yong Li <​yong.li@torchmobile.com>

Reviewed by Adam Treat.

​https://bugs.webkit.org/show_bug.cgi?id=27306
Use RegisterClass instead of RegisterClassEx on WinCE.

  • wtf/win/MainThreadWin.cpp: (WTF::initializeMainThreadPlatform):
10:32 AM Changeset in webkit [45923] by beidson@apple.com
  • 12 edits in trunk

WebCore:

2009-07-15 Brady Eidson <​beidson@apple.com>

Reviewed by Dan Bernstein.

​https://bugs.webkit.org/show_bug.cgi?id=27304
WebKit should provide usage and eligibility information about the page cache.

  • WebCore.base.exp:
  • history/CachedFrame.cpp: (WebCore::CachedFrame::childFrameCount):
  • history/CachedFrame.h:
  • history/PageCache.cpp: (WebCore::PageCache::frameCount): (WebCore::PageCache::autoreleasedPageCount):
  • history/PageCache.h: (WebCore::PageCache::pageCount):

WebKit/mac:

2009-07-15 Brady Eidson <​beidson@apple.com>

Reviewed by Dan Bernstein.

​https://bugs.webkit.org/show_bug.cgi?id=27304
WebKit should provide usage and eligibility information about the page cache.

  • Misc/WebCoreStatistics.h:
  • Misc/WebCoreStatistics.mm: (+[WebCoreStatistics numberCachedPages]): (+[WebCoreStatistics numberCachedFrames]): (+[WebCoreStatistics numberAutoreleasedPages]):
  • WebKit.exp:
  • WebView/WebFrame.mm: (-[WebFrame _cacheabilityDictionary]): Returns a dictionary with all the data about why the frame is not cacheable. If the frame *is* cacheable, the dictionary is empty.
  • WebView/WebFramePrivate.h:
10:31 AM Changeset in webkit [45922] by levin@chromium.org
  • 2 edits in trunk/WebCore

2009-07-15 Shinichiro Hamaji <​hamaji@chromium.org>

Reviewed by David Levin.

Chromium's canvas forgets its context after fillText again
​https://bugs.webkit.org/show_bug.cgi?id=27293

No new tests because the test for this was already added in
​https://bugs.webkit.org/show_bug.cgi?id=26436

  • platform/graphics/chromium/TransparencyWin.cpp: (WebCore::TransparencyWin::compositeTextComposite):
10:20 AM Changeset in webkit [45921] by treat@webkit.org
  • 2 edits in trunk/JavaScriptCore

2009-07-15 Yong Li <​yong.li@torchmobile.com>

Reviewed by George Staikos.

​https://bugs.webkit.org/show_bug.cgi?id=27301
Use OutputDebugStringW on WinCE since OutputDebugStringA is not supported
Originally written by Yong Li <​yong.li@torchmobile.com> and refactored by
Joe Mason <​joe.mason@torchmobile.com>

  • wtf/Assertions.cpp: vprintf_stderr_common
10:07 AM Changeset in webkit [45920] by levin@chromium.org
  • 1 edit
    2 moves in trunk/WebKitTools

2009-07-15 Shinichiro Hamaji <​hamaji@chromium.org>

Reviewed by David Levin.

Move cpplint.py to module directory
​https://bugs.webkit.org/show_bug.cgi?id=27302

  • Scripts/modules/cpplint.py: Renamed from WebKitTools/Scripts/cpplint.py.
  • Scripts/modules/cpplint_unittest.py: Renamed from WebKitTools/Scripts/cpplint_unittest.py.
10:02 AM Changeset in webkit [45919] by hyatt@apple.com
  • 19 edits
    10 adds in trunk

WebCore:

2009-07-14 David Hyatt <​hyatt@apple.com>

Reviewed by Simon Fraser.

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

Implement the new 'rem' unit from CSS3.

Added some rem-* tests in fast/css.

  • css/CSSGrammar.y:
  • css/CSSParser.cpp: (WebCore::CSSParser::validUnit): (WebCore::unitFromString): (WebCore::CSSParser::parseValue): (WebCore::CSSParser::lex):
  • css/CSSParserValues.cpp: (WebCore::CSSParserValue::createCSSValue):
  • css/CSSPrimitiveValue.cpp: (WebCore::CSSPrimitiveValue::computeLengthInt): (WebCore::CSSPrimitiveValue::computeLengthIntForLength): (WebCore::CSSPrimitiveValue::computeLengthShort): (WebCore::CSSPrimitiveValue::computeLengthFloat): (WebCore::CSSPrimitiveValue::computeLengthDouble): (WebCore::CSSPrimitiveValue::cssText): (WebCore::CSSPrimitiveValue::parserValue):
  • css/CSSPrimitiveValue.h: (WebCore::CSSPrimitiveValue::): (WebCore::CSSPrimitiveValue::isUnitTypeLength):
  • css/CSSStyleSelector.cpp: (WebCore::CSSStyleSelector::initForStyleResolve): (WebCore::convertToLength): (WebCore::CSSStyleSelector::applyProperty): (WebCore::CSSStyleSelector::mapFillSize): (WebCore::CSSStyleSelector::mapFillXPosition): (WebCore::CSSStyleSelector::mapFillYPosition): (WebCore::CSSStyleSelector::createTransformOperations):
  • css/CSSStyleSelector.h:
  • css/MediaQueryEvaluator.cpp: (WebCore::device_heightMediaFeatureEval): (WebCore::device_widthMediaFeatureEval): (WebCore::heightMediaFeatureEval): (WebCore::widthMediaFeatureEval):
  • css/WebKitCSSMatrix.cpp: (WebCore::WebKitCSSMatrix::setMatrixValue):
  • css/tokenizer.flex:
  • dom/Document.cpp: (WebCore::Document::Document):
  • dom/Document.h: (WebCore::Document::usesRemUnits): (WebCore::Document::setUsesRemUnits):
  • dom/Element.cpp: (WebCore::Element::recalcStyle):
  • rendering/SVGRenderTreeAsText.cpp: (WebCore::writeStyle):
  • rendering/style/SVGRenderStyle.cpp: (WebCore::SVGRenderStyle::cssPrimitiveToLength):
  • svg/graphics/SVGPaintServer.cpp: (WebCore::applyStrokeStyleToContext): (WebCore::dashArrayFromRenderingStyle):
  • svg/graphics/SVGPaintServer.h:

LayoutTests:

2009-07-14 David Hyatt <​hyatt@apple.com>

Reviewed by Simon Fraser.

Add layout tests for the new CSS3 'rem' unit type.

  • fast/css/rem-dynamic-scaling.html: Added.
  • fast/css/rem-units-on-root.html: Added.
  • fast/css/remove-shorthand-expected.checksum: Added.
  • fast/css/remove-shorthand-expected.png: Added.
  • platform/mac/fast/css/rem-dynamic-scaling-expected.checksum: Added.
  • platform/mac/fast/css/rem-dynamic-scaling-expected.png: Added.
  • platform/mac/fast/css/rem-dynamic-scaling-expected.txt: Added.
  • platform/mac/fast/css/rem-units-on-root-expected.checksum: Added.
  • platform/mac/fast/css/rem-units-on-root-expected.png: Added.
  • platform/mac/fast/css/rem-units-on-root-expected.txt: Added.
9:58 AM Changeset in webkit [45918] by treat@webkit.org
  • 2 edits in trunk/JavaScriptCore

2009-07-15 Yong Li <​yong.li@torchmobile.com>

Reviewed by George Staikos.

​https://bugs.webkit.org/show_bug.cgi?id=27020
msToGregorianDateTime should set utcOffset to 0 when outputIsUTC is false

  • wtf/DateMath.cpp: (WTF::gregorianDateTimeToMS):
9:56 AM Changeset in webkit [45917] by bweinstein@apple.com
  • 2 edits in trunk/LayoutTests

2009-07-15 Brian Weinstein <​bweinstein@apple.com>

Rubberstamped by Mark Rowe.

Add one more test to Windows Skipped List to hopefully get it to stay green.

  • platform/win/Skipped:
9:27 AM Changeset in webkit [45916] by Dimitri Glazkov
  • 2 edits in trunk/WebCore

2009-07-15 Dimitri Glazkov <Dimitri Glazkov>

Unreviewed, build fix.

Remove extraneous qualifier, accidentally added in ​http://trac.webkit.org/changeset/45884.

  • bindings/v8/V8DOMWrapper.h: Removed extraneous qualifier.
9:17 AM Changeset in webkit [45915] by Dimitri Glazkov
  • 5 edits in trunk/WebCore

2009-07-15 Dimitri Glazkov <Dimitri Glazkov>

Reviewed by Darin Fisher.

[V8] Update bindings for ValiditeState patch.
​https://bugs.webkit.org/show_bug.cgi?id=19562

  • bindings/v8/DOMObjectsInclude.h:
  • bindings/v8/DerivedSourcesAllInOne.cpp:
  • bindings/v8/V8Index.cpp:
  • bindings/v8/V8Index.h:
8:49 AM Changeset in webkit [45914] by Simon Hausmann
  • 5 edits in trunk

JavaScriptCore:

2009-07-15 Laszlo Gombos <Laszlo Gombos>

Reviewed by Simon Hausmann.

[Qt] Cleanup - Remove obsolete code from the make system
​https://bugs.webkit.org/show_bug.cgi?id=27299

WebCore:

2009-07-15 Laszlo Gombos <Laszlo Gombos>

Reviewed by Simon Hausmann.

[Qt] Cleanup - Remove obsolete code from the make system
​https://bugs.webkit.org/show_bug.cgi?id=27299

  • WebCore.pro:
8:41 AM Changeset in webkit [45913] by Simon Hausmann
  • 2 edits in trunk/WebKit/qt

2009-07-15 Yael Aharon <​yael.aharon@nokia.com>

Reviewed by Simon Hausmann.

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

When the user clicks a link with a target attribute, the newly created window should be visible.
Make new windows created in Qtlauncher visible.

  • QtLauncher/main.cpp: (WebPage::createWindow):
8:33 AM Changeset in webkit [45912] by Simon Hausmann
  • 2 edits in trunk/WebCore

2009-07-15 Simon Hausmann <​simon.hausmann@nokia.com>

Reviewed by Ariya Hidayat.

Fix the build without media elements.

  • rendering/HitTestResult.cpp: (WebCore::HitTestResult::absoluteMediaURL): Add #if ENABLE(VIDEO) markers around the body of the method.
8:18 AM Changeset in webkit [45911] by Simon Hausmann
  • 2 edits in trunk/JavaScriptCore

2009-07-07 Norbert Leser <​norbert.leser@nokia.com>

Reviewed by Simon Hausmann.

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

Alternate bool operator for codewarrior compiler (WINSCW).
Compiler (latest b482) reports error for UnspecifiedBoolType construct:
"illegal explicit conversion from 'WTF::OwnArrayPtr<JSC::Register>' to 'bool'"

Same fix as in r38391.

8:14 AM Changeset in webkit [45910] by vestbo@webkit.org
  • 2 edits in trunk

2009-07-15 Tor Arne Vestbø <​tor.arne.vestbo@nokia.com>

Rubber-stamped by Simon Hausmann.

Fix the Qt/Mac build by disabling TestNetscapePlugin

We should fix and enable this once we run DRT for Qt/Mac

  • WebKit.pro:
8:14 AM Changeset in webkit [45909] by Simon Hausmann
  • 2 edits in trunk/JavaScriptCore

2009-07-15 Norbert Leser <​norbert.leser@nokia.com>

Reviewed by Darin Adler.

Qualify include path with wtf to fix compilation
on Symbian.
​https://bugs.webkit.org/show_bug.cgi?id=27055

  • interpreter/Interpreter.h:
7:18 AM Changeset in webkit [45908] by Simon Hausmann
  • 4 edits in trunk/JavaScriptCore

2009-07-15 Laszlo Gombos <Laszlo Gombos>

Reviewed by Dave Kilzer.

Turn off non-portable date manipulations for SYMBIAN
​https://bugs.webkit.org/show_bug.cgi?id=27064

Introduce HAVE(TM_GMTOFF), HAVE(TM_ZONE) and HAVE(TIMEGM) guards
and place the rules for controlling the guards in Platform.h.
Turn off these newly introduced guards for SYMBIAN.

  • wtf/DateMath.cpp: (WTF::calculateUTCOffset):
  • wtf/DateMath.h: (WTF::GregorianDateTime::GregorianDateTime): (WTF::GregorianDateTime::operator tm):
  • wtf/Platform.h:
7:04 AM Changeset in webkit [45907] by Simon Hausmann
  • 2 edits in trunk/JavaScriptCore

2009-07-15 Norbert Leser <​norbert.leser@nokia.com>

Reviewed by Simon Hausmann.

Undef ASSERT on Symbian, to avoid excessive warnings
​https://bugs.webkit.org/show_bug.cgi?id=27052

  • wtf/Assertions.h:
5:51 AM Changeset in webkit [45906] by Simon Hausmann
  • 2 edits in trunk/WebKitTools

2009-07-15 Simon Hausmann <​simon.hausmann@nokia.com>

Reviewed by Adam Treat.

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

Re-implement QWebPage::shouldInterruptJavaScript to disable
js interruption and avoid showing a messagebox during Qt DRT
runs when script execution takes a bit longer.

  • DumpRenderTree/qt/DumpRenderTree.cpp: (WebCore::WebPage::shouldInterruptJavaScript): Return false in re-implemented slot.
5:49 AM Changeset in webkit [45905] by jmalonzo@webkit.org
  • 1 edit
    1 add in trunk/WebKit/gtk/po

2009-07-15 Aron Xu <​aronmalache@163.com>

Rubber-stamped by Jan Alonzo.

Simplified Chinese translation
​https://bugs.webkit.org/show_bug.cgi?id=27252

  • zh_CN.po: Added.
5:16 AM Changeset in webkit [45904] by oliver@apple.com
  • 2 edits in trunk/JavaScriptCore

REGRESSION: fast/js/postfix-syntax.html fails with interpreter
​https://bugs.webkit.org/show_bug.cgi?id=27294

Reviewed by Simon Hausmann.

When postfix operators operating on locals assign to the same local
the order of operations has to be to store the incremented value, then
store the unmodified number. Rather than implementing this subtle
semantic in the interpreter I've just made the logic explicit in the
bytecode generator, so x=x++ effectively becomes x=ToNumber(x) (for a
local var x).

2:59 AM Changeset in webkit [45903] by oliver@apple.com
  • 2 edits in trunk/JavaScriptCore

REGRESSION(43559): fast/js/kde/arguments-scope.html fails with interpreter
​https://bugs.webkit.org/show_bug.cgi?id=27259

Reviewed by Simon Hausmann

The interpreter was incorrectly basing its need to create the arguments object
based on the presence of the callframe's argument reference rather than the local
arguments reference. Based on this it then overrode the local variable reference.

2:32 AM Changeset in webkit [45902] by jmalonzo@webkit.org
  • 1 edit in trunk/WebKit/gtk/po/ChangeLog

Fix ChangeLog attribution to Bruce Cowan

2:29 AM Changeset in webkit [45901] by jmalonzo@webkit.org
  • 1 edit
    1 add in trunk/WebKit/gtk/po

2009-07-15 Jan Michael Alonzo <​jmalonzo@webkit.org>

Rubber-stamped by Jan Alonzo.

[Gtk] en_GB translation
​https://bugs.webkit.org/show_bug.cgi?id=27244

  • en_GB.po: Added.
2:12 AM Changeset in webkit [45900] by jmalonzo@webkit.org
  • 2 edits in trunk/WebKitTools

2009-07-15 Zoltan Horvath <​hzoltan@inf.u-szeged.hu>

Reviewed by Jan Alonzo.

Fix the DumpRenderTree GTK+ build
​https://bugs.webkit.org/show_bug.cgi?id=27290

Set USE_SYSTEM_MALLOC macro for the DumpRenderTree build to
disable using TCmalloc in DumpRenderTree.

  • GNUmakefile.am:
1:10 AM Changeset in webkit [45899] by pfeldman@chromium.org
  • 3 edits in trunk/WebCore

2009-07-14 Pavel Feldman <​pfeldman@chromium.org>

Reviewed by Timothy Hatcher.

WebInspector: Move storeLastActivePanel out of the
ifdef ENABLE_JAVASCRIPT_DEBUGGER section in IDL;
Add default panel for the first opening of the
WebInspector.

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

  • inspector/InspectorController.cpp: (InspectorController::setWindowVisible):
  • inspector/InspectorController.idl:
Note: See TracTimeline for information about the timeline view.