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

Timeline



Feb 22, 2010:

11:53 PM Changeset in webkit [55123] by eric@webkit.org
  • 7 edits in trunk/WebKitTools

2010-02-22 Dirk Pranke <​dpranke@chromium.org>

Reviewed by Eric Siedel.

Add more checking for missing binaries and packages to check_sys_deps()

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

  • Scripts/webkitpy/layout_tests/port/base.py:
  • Scripts/webkitpy/layout_tests/port/chromium.py:
  • Scripts/webkitpy/layout_tests/port/chromium_linux.py:
  • Scripts/webkitpy/layout_tests/port/chromium_mac.py:
  • Scripts/webkitpy/layout_tests/port/chromium_win.py:
  • Scripts/webkitpy/layout_tests/run_chromium_webkit_tests.py:
11:37 PM Changeset in webkit [55122] by eric@webkit.org
  • 3 edits in trunk/WebCore

2010-02-22 Stephan Aßmus <​superstippi@gmx.de>

Reviewed by Eric Seidel.

[Haiku] Implemented PopupMenu support.
​https://bugs.webkit.org/show_bug.cgi?id=35078

Covered by existing tests.

The implementation is currently very simple: Added a PopupMenuHaiku
class that derives from a native BPopUpMenu. It attaches a BHandler
derivative to the BApplication (WebCore main thread) which receives
the item invokation and menu hidden events and informs the PopupMenuClient
accordingly.

  • platform/PopupMenu.h:

Changed type of m_menu for Haiku.

  • platform/haiku/PopupMenuHaiku.cpp: (WebCore::PopupMenuHandler::PopupMenuHandler): (WebCore::PopupMenuHandler::MessageReceived): (WebCore::PopupMenuHaiku::PopupMenuHaiku): (WebCore::PopupMenuHaiku::~PopupMenuHaiku): (WebCore::PopupMenuHaiku::show): (WebCore::PopupMenuHaiku::hide): (WebCore::PopupMenuHaiku::Hide): (WebCore::PopupMenu::PopupMenu): (WebCore::PopupMenu::~PopupMenu):

Removed bogus code.

(WebCore::PopupMenu::show):

Implemented using new PopupMenuHaiku class.

(WebCore::PopupMenu::hide):

Implemented using new PopupMenuHaiku class.

(WebCore::PopupMenu::updateFromElement):

Implemented.

(WebCore::PopupMenu::itemWritingDirectionIsNatural):

Implemented according to Gtk port.

10:28 PM Changeset in webkit [55121] by hamaji@chromium.org
  • 3 edits in trunk/WebKitTools

2010-02-22 Shinichiro Hamaji <​hamaji@chromium.org>

Reviewed by David Levin.

[Gtk] check-webkit-style: GTK style should be allowed in WebKitTools/DumpRenderTree/gtk
​https://bugs.webkit.org/show_bug.cgi?id=35229

  • Scripts/webkitpy/style/checker.py:
  • Scripts/webkitpy/style/checker_unittest.py:
9:03 PM Changeset in webkit [55120] by jchaffraix@webkit.org
  • 8 edits in trunk/JavaScriptCore

Remove auto_ptr usage in JavaScriptCore.
​https://bugs.webkit.org/show_bug.cgi?id=35221

Reviewed by Darin Adler.

  • parser/Nodes.h: Removed now unneeded adopt method.
  • parser/Parser.cpp: Removed <memory> include as it is not required anymore.
  • wtf/OwnPtr.h: Removed the constructor from auto_ptr.
  • wtf/VectorTraits.h: Removed a template specialization for auto_ptr.
  • wtf/unicode/Collator.h: Made userDefault return a PassOwnPtr.
  • wtf/unicode/CollatorDefault.cpp:

(WTF::Collator::userDefault): Changed the method to match the next signature.

  • wtf/unicode/icu/CollatorICU.cpp:

(WTF::Collator::userDefault): Ditto.

8:50 PM Changeset in webkit [55119] by jamesr@google.com
  • 2 edits in trunk/WebKitTools

2010-02-22 James Robinson <​jamesr@chromium.org>

Unreviewed. Adding myself to committers list.

  • Scripts/webkitpy/committers.py:
8:29 PM Changeset in webkit [55118] by Simon Fraser
  • 2 edits in trunk/WebKitTools

2010-02-22 Simon Fraser <Simon Fraser>

Reviewed by Darin Adler.

Remove DRT hack that turns off hardware acceleration with older QuickTimes
​https://bugs.webkit.org/show_bug.cgi?id=35275

Now that WebKit does a version check to avoid a QuickTime-related
crash (r55100), DumpRenderTree does not need to.

  • DumpRenderTree/mac/DumpRenderTree.mm: (resetDefaultsToConsistentValues):
5:55 PM Changeset in webkit [55117] by eric@webkit.org
  • 5 edits in trunk

2010-02-22 Huahui Wu <​hwu@google.com>

Reviewed by Eric Seidel.

Add code that enables SquirrelFish Extreme (a.k.a JSCX, JSC JIT)
in Android. It's disabled by default, but is enabled when the
enveronment variable ENABLE_JSC_JIT is set to true.
​https://bugs.webkit.org/show_bug.cgi?id=34855

  • Android.mk:

2010-02-22 Huahui Wu <​hwu@google.com>

Reviewed by Eric Seidel.

Add code that enables SquirrelFish Extreme (a.k.a JSCX, JSC JIT)
in Android. It's disabled by default, but is enabled when the
enveronment variable ENABLE_JSC_JIT is set to true.
​https://bugs.webkit.org/show_bug.cgi?id=34855

  • Android.mk:
  • wtf/Platform.h:
5:29 PM Changeset in webkit [55116] by eric@webkit.org
  • 2 edits in trunk/WebCore

2010-02-22 Stephan Aßmus <​superstippi@gmx.de>

Reviewed by Eric Seidel.

Fix various issues in PlatformMouseEventHaiku.
​https://bugs.webkit.org/show_bug.cgi?id=34685

Covered by existing tests.

Mapping Haiku button constants (bit field) to WebCore buttons was broken.
Extracting event time was broken (supposed to be in seconds).
Wrong coordinate was being extracted, needs to be content local.
Added extracting modifier key flags.

  • platform/haiku/PlatformMouseEventHaiku.cpp: (WebCore::PlatformMouseEvent::PlatformMouseEvent):
5:12 PM Changeset in webkit [55115] by eric@webkit.org
  • 3 edits in trunk/WebCore

2010-02-22 Stephan Aßmus <​superstippi@gmx.de>

Reviewed by Eric Seidel.

[Haiku] Implement creating and filling platform gradients.
​https://bugs.webkit.org/show_bug.cgi?id=34683

Covered by existing tests.

  • platform/graphics/Gradient.h:

Typedef PlatformGradient to BGradient

  • platform/graphics/haiku/GradientHaiku.cpp: (WebCore::Gradient::platformDestroy):

Delete the cached BGradient object.

(WebCore::Gradient::platformGradient):

Create a BGradient object according to the type of Gradient.
Return the cached object.

(WebCore::Gradient::fill):

use BView API to fill with the platform gradient.

5:01 PM Changeset in webkit [55114] by tony@chromium.org
  • 1 edit
    2 adds in trunk/LayoutTests

2010-02-22 Chris Evans <​cevans@chromium.org>

Reviewed by Darin Adler.

Add test for ​https://bugs.webkit.org/show_bug.cgi?id=32842
It has been fixed on trunk for a while, but we want to make sure we
do not regress.

  • fast/text/find-layout-crash.html: Added.
  • fast/text/find-layout-crash-expected.txt: Added.
4:55 PM Changeset in webkit [55113] by eric@webkit.org
  • 2 edits in trunk/WebCore

2010-02-22 Stephan Aßmus <​superstippi@gmx.de>

Reviewed by Eric Seidel.

Build fix for debug builds of GlyphPageTreeNode.cpp
​https://bugs.webkit.org/show_bug.cgi?id=34528

Covered by existing tests.

  • platform/graphics/GlyphPageTreeNode.cpp:

include <stdio.h> since printf() is used if NDEBUG is not defined.

4:45 PM Changeset in webkit [55112] by sfalken@apple.com
  • 6 edits in trunk/WebKit/win

WebKit on Windows should pick up system setting changes without requiring explicit API calls
​https://bugs.webkit.org/show_bug.cgi?id=35269

Reviewed by Darin Adler.

  • WebKit.vcproj/WebKit.def: Removed WebKitSystemParameterChanged.
  • WebKit.vcproj/WebKit_debug.def: Removed WebKitSystemParameterChanged.
  • WebKitGraphics.cpp: Removed WebKitSystemParameterChanged.
  • WebKitGraphics.h: Removed WebKitSystemParameterChanged.
  • WebView.cpp:

(systemParameterChanged): Call through to wkSystemFontSmoothingChanged for font changes.
(WebView::windowReceivedMessage): Pick up WM_SETTINGCHANGE from windowReceivedMessage.

4:12 PM Changeset in webkit [55111] by Nate Chapin
  • 2 edits in trunk/WebCore

2010-02-22 Nate Chapin <Nate Chapin>

Unreviewed, Chromium build fix.

Add special case in CodeGeneratorV8.pm for named getters for html plugin-related elements.

CodeGeneratorV8.pm was relying on HasOverridingNameGetter to hint
that a custom named getter was needed. That hint was removed in
​http://trac.webkit.org/changeset/55104.

  • bindings/scripts/CodeGeneratorV8.pm:
3:28 PM Changeset in webkit [55110] by Csaba Osztrogonác
  • 1 edit
    1 delete in trunk/LayoutTests

Rubber-stamped by Alexey Proskuryakov.

[Qt] fast/encoding/invalid-UTF-8-2.html fails
​https://bugs.webkit.org/show_bug.cgi?id=35263

Qt Linux Release buildbot got the platform independent result for
fast/encoding/invalid-UTF-8-2.html instead of Qt specific one.

Qt specific expected file is removed to make buildbot green,
but it needs more investigation.

  • platform/qt/fast/encoding/invalid-UTF-8-2-expected.txt: Removed.
3:27 PM Changeset in webkit [55109] by ap@apple.com
  • 21 edits in trunk

Rubber-stamped by Geoff Garen.

Rename RuntimeObjectImp to RuntimeObject.

3:23 PM Changeset in webkit [55108] by pkasting@chromium.org
  • 3 edits
    3 adds in trunk

Avoid recursion when trying to get the size of a PNG; it's unnecessary
and in the worst case can lead to heap corruption.
​https://bugs.webkit.org/show_bug.cgi?id=35167

Reviewed by Eric Seidel.

WebCore:

Test: fast/images/bad-png.html

  • platform/image-decoders/png/PNGImageDecoder.cpp:

(WebCore::PNGImageReader::decode):

LayoutTests:

  • fast/images/bad-png-expected.txt: Added.
  • fast/images/bad-png.html: Added.
  • fast/images/resources/bad-png.png: Added.
3:20 PM BuildingOnWindows edited by Adam Roben
(diff)
3:10 PM Changeset in webkit [55107] by beidson@apple.com
  • 8 edits in trunk

Disable WebView docking to views that are too small.
<rdar://problem/7248409> and ​https://bugs.webkit.org/show_bug.cgi?id=35254

Reviewed by Tim Hatcher.

WebCore:

  • WebCore.base.exp:
  • inspector/InspectorController.cpp:

(WebCore::InspectorController::canAttachWindow): Provide a utility for WebKits to make a showWindow()

decision based on if attachment would be allowed or not.

(WebCore::InspectorController::attachWindow): Don't attach if the view is too small to attach to.

  • inspector/InspectorController.h:

WebKit/mac:

  • WebCoreSupport/WebInspectorClient.mm:

(-[WebInspectorWindowController showWindow:]): No matter the preference, don't open the inspector

window attached if WebCore says it shouldn't be attached.

WebKit/win:

  • WebCoreSupport/WebInspectorClient.cpp:

(WebInspectorClient::showWindowWithoutNotifications): No matter the preference, don't open the inspector

window attached if WebCore says it shouldn't be attached.

2:32 PM Changeset in webkit [55106] by ap@apple.com
  • 2 edits in trunk/WebCore

Build fix.

  • WebCore.base.exp: Export Instance::newRuntimeObject, it's virtual!
2:25 PM Changeset in webkit [55105] by ap@apple.com
  • 2 edits in trunk/WebCore

Undo a small part of the previous commit.

  • bridge/runtime_method.cpp: (JSC::callRuntimeMethod): Let's keep the instance local variable as RefPtr for safety.
2:21 PM Changeset in webkit [55104] by ap@apple.com
  • 12 edits in trunk/WebCore

Reviewed by Darin Adler.

​https://bugs.webkit.org/show_bug.cgi?id=35260
Eliminate apple_runtime_object

No change in functionality, so no tests.

  • WebCore.base.exp: Don't export Instance::newRuntimeObject, WebKit doesn't need it.
  • bindings/js/JSHTMLAppletElementCustom.cpp:
  • bindings/js/JSHTMLEmbedElementCustom.cpp:
  • bindings/js/JSHTMLObjectElementCustom.cpp:
  • html/HTMLAppletElement.idl:
  • html/HTMLEmbedElement.idl:
  • html/HTMLObjectElement.idl: These objects no longer need overriding name getters, as they no longer intercept the apple_runtime_object property.
  • bindings/js/JSPluginElementFunctions.cpp: (WebCore::pluginInstance): This is no longer static. It was used for callPlugin() in this file, and now it's also used elsewhere when calling plug-in methods. (WebCore::runtimeObjectGetter): Removed. This function was only used by the intercepting name getters.
  • bindings/js/JSPluginElementFunctions.h: Export pluginInstance().
  • bindings/objc/WebScriptObject.mm: (+[WebScriptObject _convertValueToObjcValue:originRootObject:rootObject:]):
  • bridge/runtime_method.cpp: (JSC::callRuntimeMethod): Take plug-in element's instance directly, without relying on fake attribute lookup. One change resulting from this is that RuntimeObjectImp may not be created in some cases - this code only needs an instance, but the old code had to pass the instance wrapped into RuntimeObjectImp.
2:17 PM Changeset in webkit [55103] by ap@apple.com
  • 5 edits
    3 adds in trunk/LayoutTests

2010-02-22 Robert Hogan <​robert@roberthogan.net>

Reviewed by Alexey Proskuryakov.

​https://bugs.webkit.org/show_bug.cgi?id=32937
LayoutTests/fast/encoding/invalid-UTF-8.html fails on Qt.

The test was verifying two things: first, that U+FFFD is rendered by the browser, and second,
that ill-formed UTF-8 sequences are turned into U+FFFD. Some aspects of the latter are not
precisely defined by the Unicode specification, so different platforms may legitimately
have different results.

Split the test into two: one to test the rendering of invalid UTF-8 sequences, and the other
to protect against regressions in each platform's decoding behaviour.

  • fast/encoding/invalid-UTF-8-2-expected.txt: Added.
  • fast/encoding/invalid-UTF-8-2.html: Added.
  • fast/encoding/invalid-UTF-8.html:
  • platform/mac/fast/encoding/invalid-UTF-8-expected.checksum:
  • platform/mac/fast/encoding/invalid-UTF-8-expected.png:
  • platform/mac/fast/encoding/invalid-UTF-8-expected.txt:
  • platform/qt/fast/encoding/invalid-UTF-8-2-expected.txt:
1:07 PM Changeset in webkit [55102] by Chris Fleizach
  • 3 edits in trunk/LayoutTests

Bug 35186 - AX: AXFocused is not writable like it should be on nodes
​https://bugs.webkit.org/show_bug.cgi?id=35186

Fixing layout test failure.

12:59 PM Changeset in webkit [55101] by sullivan@apple.com
  • 2 edits in trunk/WebCore

​https://bugs.webkit.org/show_bug.cgi?id=35256
HTMLSelectElement::setSelectedIndexByUser() can trigger unnecessary JS when there is no change to the selected index

Reviewed by Tim Hatcher.

No new tests because this code path is not used for JS-initiated changes.

  • html/HTMLSelectElement.cpp:

(WebCore::HTMLSelectElement::setSelectedIndexByUser):
Bail out if the to-be-selected index matches the already-selected index.

12:51 PM Changeset in webkit [55100] by Simon Fraser
  • 2 edits in trunk/WebKit/mac

2010-02-22 Simon Fraser <Simon Fraser>

Reviewed by John Sullivan.

<rdar://problem/7285392>
On Leopard, we have to disable hardware acceleration if we detect that the
installed Core Video framework has bug <rdar://problem/7228836>.

  • WebView/WebView.mm: (coreVideoHas7228836Fix): (-[WebView _preferencesChangedNotification:]):
12:18 PM Changeset in webkit [55099] by kov@webkit.org
  • 1 copy in releases/WebKitGTK/webkit-1.1.22

Tagging 1.1.22.

12:05 PM Changeset in webkit [55098] by enrica@apple.com
  • 3 edits
    2 adds in trunk

Reproducible crash in WebCore::nextLinePosition on Tweeteorites.com
<rdar://problem/7615758>
​https://bugs.webkit.org/show_bug.cgi?id=35060

Reviewed by Dan Bernstein.

WebCore:

Test: editing/selection/extend-byline-withfloat.html

  • editing/visible_units.cpp:

(WebCore::previousLinePosition): Skip elements with zero height.
(WebCore::nextLinePosition): Skip elements with zero height.

LayoutTests:

  • editing/selection/extend-byline-withfloat-expected.txt: Added.
  • editing/selection/extend-byline-withfloat.html: Added.
12:03 PM Changeset in webkit [55097] by Nate Chapin
  • 2 edits in trunk/WebCore

2010-02-22 Nate Chapin <Nate Chapin>

Unreviewed, Chromium build fix.

  • bindings/scripts/CodeGeneratorV8.pm: Compile break due to bad patch merge.
11:53 AM Changeset in webkit [55096] by Nate Chapin
  • 27 edits
    5 deletes in trunk/WebCore

2010-02-22 Nate Chapin <Nate Chapin>

Reviewed by Dimitri Glazkov.

[V8] Streamline V8 event listener code. Merge all the variants of
V8DOMWrapper::getEventListner() into a single version and generate
addEventListener() and removeEventListener() bindings for all objects
except DOMWindow.

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

11:29 AM Changeset in webkit [55095] by Chris Fleizach
  • 3 edits
    2 adds in trunk

AX: AXFocused is not writable like it should be on nodes
​https://bugs.webkit.org/show_bug.cgi?id=35186

Reviewed by Beth Dakin.

WebCore:

Test: platform/mac/accessibility/element-focus.html

  • accessibility/AccessibilityRenderObject.cpp:

(WebCore::AccessibilityRenderObject::canSetFocusAttribute):

LayoutTests:

  • platform/mac/accessibility/element-focus-expected.txt: Added.
  • platform/mac/accessibility/element-focus.html: Added.
11:24 AM Changeset in webkit [55094] by Laszlo Gombos
  • 2 edits in trunk/WebKit/qt

2010-02-22 Laszlo Gombos <Laszlo Gombos>

Unreviewed Symbian build fix.

[Qt] Updated the def file with new exports to enable
building DumpRenderTree.

  • symbian/eabi/QtWebKitu.def:
11:12 AM Changeset in webkit [55093] by barraclough@apple.com
  • 5 edits in trunk/JavaScriptCore

JSStringBuilder should not CRASH if allocation fails, it should throw a JSException.

Reviewed by Oliver Hunt.

  • runtime/JSGlobalObjectFunctions.cpp:
  • runtime/JSStringBuilder.h:

(JSC::JSStringBuilder::JSStringBuilder):
(JSC::JSStringBuilder::append):
(JSC::JSStringBuilder::build):

  • runtime/StringBuilder.h:

(JSC::StringBuilder::build):

  • wtf/Vector.h:

(WTF::VectorBufferBase::tryAllocateBuffer):
(WTF::):
(WTF::VectorBuffer::tryAllocateBuffer):
(WTF::::tryExpandCapacity):
(WTF::::tryReserveCapacity):
(WTF::::tryAppend):

11:07 AM Changeset in webkit [55092] by xan@webkit.org
  • 4 edits in trunk

2010-02-22 Xan Lopez <​xlopez@igalia.com>

Reviewed by Gustavo Noronha.

Bump library versioning for 1.1.22 release.

  • configure.ac:

WebKit/gtk:

2010-02-22 Xan Lopez <​xlopez@igalia.com>

Reviewed by Gustavo Noronha.

Update for release.

  • NEWS:
11:02 AM Changeset in webkit [55091] by eric@webkit.org
  • 2 edits
    1 add in trunk/JavaScriptCore

2010-02-22 Kwang Yul Seo <​skyul@company100.net>

Reviewed by Eric Seidel.

[BREWMP] Map FastMalloc to BREW memory allocator
​https://bugs.webkit.org/show_bug.cgi?id=33570

Use MALLOC macro instead of the standard malloc function.
Although RVCT provides malloc, we can't use it in BREW
because the loader does not initialize the base address properly.

  • wtf/FastMalloc.cpp:
  • wtf/brew/SystemMallocBrew.h: Added. (mallocBrew): (callocBrew): (freeBrew): (reallocBrew):
10:48 AM Changeset in webkit [55090] by Simon Fraser
  • 11 edits in trunk/WebCore

2010-02-22 Simon Fraser <Simon Fraser>

Reviewed by Dan Bernstein, Darin Adler.

Remove addScrolledContentOffset/subtractScrolledContentOffset
​https://bugs.webkit.org/show_bug.cgi?id=35225

Remove RenderLayer's addScrolledContentOffset() and subtractScrolledContentOffset()
methods, and instead use the existing scrolledContentOffset(), and use
IntSize and IntPoint instead of lots of x, y variables.

Added new IntPoint toPoint(const IntSize&) method as a convenience to convert a size to a point,
which is needed in a few places.

  • page/EventHandler.cpp: (WebCore::EventHandler::handleDrag): Use the new toPoint() convenience.
  • platform/graphics/IntPoint.h: (WebCore::toPoint): New convenience method to convert an IntSize to an IntPoint.
  • rendering/LayoutState.cpp: (WebCore::LayoutState::LayoutState):
  • rendering/RenderBlock.cpp: (WebCore::RenderBlock::layoutBlock): (WebCore::RenderBlock::paintObject): (WebCore::RenderBlock::selectionGapRectsForRepaint): (WebCore::RenderBlock::nodeAtPoint): (WebCore::RenderBlock::offsetForContents):
  • rendering/RenderBoxModelObject.cpp: (WebCore::RenderBoxModelObject::paintFillLayerExtended):
  • rendering/RenderInline.cpp: (WebCore::RenderInline::clippedOverflowRectForRepaint):
  • rendering/RenderLayer.cpp: (WebCore::RenderLayer::updateLayerPosition):
  • rendering/RenderLayer.h: (WebCore::RenderLayer::size):
  • rendering/RenderObject.cpp: (WebCore::RenderObject::computeRectForRepaint):
  • rendering/TextControlInnerElements.cpp: (WebCore::RenderTextControlInnerBlock::positionForPoint):
10:36 AM Changeset in webkit [55089] by agl@chromium.org
  • 12 edits
    3 copies in trunk

2009-02-22 Adam Langley <​agl@google.com>

Reviewed by Darin Fisher.

fontconfig on Linux can change the render preferences on a per strike
basis (a strike a combination of face and size). Because of this, we
need to query fontconfig each time a new FontPlatformData is created
for a new size.

This patch adds support for querying this via ChromiumBridge.

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

  • platform/chromium/ChromiumBridge.h: add getRenderStyleForStrike
  • platform/graphics/chromium/FontCacheLinux.cpp: (WebCore::FontCache::createFontPlatformData):
  • platform/graphics/chromium/FontCustomPlatformData.cpp: (WebCore::FontCustomPlatformData::fontPlatformData):
  • platform/graphics/chromium/FontPlatformDataLinux.cpp: (WebCore::FontPlatformData::FontPlatformData): (WebCore::FontPlatformData::operator=): (WebCore::FontPlatformData::setupPaint): (WebCore::FontPlatformData::queryStyle): add code to query fontconfig via ChromiumBridge
  • platform/graphics/chromium/FontPlatformDataLinux.h:
10:11 AM Changeset in webkit [55088] by kov@webkit.org
  • 4 edits in trunk

Build fix for make distcheck.

9:25 AM Changeset in webkit [55087] by eric@webkit.org
  • 2 edits in trunk/WebCore

2010-02-22 Laszlo Gombos <Laszlo Gombos>

Reviewed by Kenneth Rohde Christiansen.

Fix compiler warning "suggest parentheses around"
​https://bugs.webkit.org/show_bug.cgi?id=35197

No new tests as there is no new functionality.

  • wml/WMLVariables.cpp: (WebCore::substituteVariableReferences):
9:01 AM Changeset in webkit [55086] by kov@webkit.org
  • 8 edits in trunk

WebCore

2010-02-22 Gustavo Noronha Silva <​gustavo.noronha@collabora.co.uk>

Reviewed by Xan Lopez.

[Soup] loses information related to message flags when converting from/to Resource{Request,Response}
​https://bugs.webkit.org/show_bug.cgi?id=35093

Store message flags in WebCore objects, and set them on the
SoupMessage, when creating one from them.

No behaviour change.

  • platform/network/soup/ResourceHandleSoup.cpp: (WebCore::fillResponseFromMessage):
  • platform/network/soup/ResourceRequest.h: (WebCore::ResourceRequest::soupMessageFlags): (WebCore::ResourceRequest::setSoupMessageFlags):
  • platform/network/soup/ResourceRequestSoup.cpp: (WebCore::ResourceRequest::toSoupMessage): (WebCore::ResourceRequest::updateFromSoupMessage):
  • platform/network/soup/ResourceResponse.h: (WebCore::ResourceResponse::soupMessageFlags): (WebCore::ResourceResponse::setSoupMessageFlags):
  • platform/network/soup/ResourceResponseSoup.cpp: (WebCore::ResourceResponse::toSoupMessage): (WebCore::ResourceResponse::updateFromSoupMessage):

WebKit/gtk

2010-02-22 Gustavo Noronha Silva <​gustavo.noronha@collabora.co.uk>

Reviewed by Xan Lopez.

[Soup] loses information related to message flags when converting from/to Resource{Request,Response}
​https://bugs.webkit.org/show_bug.cgi?id=35093

Update the flags that are stored in the request, when the response
is received.

  • WebCoreSupport/FrameLoaderClientGtk.cpp: (WebKit::FrameLoaderClient::dispatchDidReceiveResponse):
8:40 AM Changeset in webkit [55085] by vestbo@webkit.org
  • 2 edits in trunk/WebKitTools

[Qt] Build the DRT in debug on Mac OS X

Reviewed by Kenneth Rohde Christiansen.

  • DumpRenderTree/qt/DumpRenderTree.pro:
8:17 AM Changeset in webkit [55084] by eric@webkit.org
  • 2 edits in trunk/WebCore

2010-02-22 Steve Block <​steveblock@google.com>

Reviewed by Nate Chapin.

Shared worker types used in globalObjectPrototypeIsDOMWindow are not properly guarded
​https://bugs.webkit.org/show_bug.cgi?id=35238

No new tests, build fix only.

  • bindings/v8/V8DOMWrapper.cpp: (WebCore::globalObjectPrototypeIsDOMWindow):
7:16 AM Changeset in webkit [55083] by Laszlo Gombos
  • 2 edits in trunk/JavaScriptCore

2010-02-22 Laszlo Gombos <Laszlo Gombos>

Unreviewed build fix.

[Qt] Build fix for RVCT.

Fix after r55024. The "-i" option is for perl not for the
script.

  • DerivedSources.pro:
7:12 AM Changeset in webkit [55082] by eric@webkit.org
  • 6 edits in trunk

2010-02-22 Laszlo Gombos <Laszlo Gombos>

Reviewed by Kenneth Rohde Christiansen.

[Qt] Add support for layout tests on Symbian
​https://bugs.webkit.org/show_bug.cgi?id=31589

  • WebKit.pro:

2010-02-22 Laszlo Gombos <Laszlo Gombos>

Reviewed by Kenneth Rohde Christiansen.

[Qt] Add support for layout tests on Symbian
​https://bugs.webkit.org/show_bug.cgi?id=31589

  • DumpRenderTree/qt/DumpRenderTree.pro:
  • DumpRenderTree/qt/DumpRenderTreeQt.cpp:
  • DumpRenderTree/qt/main.cpp: (main):
6:26 AM Changeset in webkit [55081] by eric@webkit.org
  • 2 edits in trunk/WebCore

2010-02-22 Steve Block <​steveblock@google.com>

Reviewed by Simon Fraser.

Simplifies calculation of the transform in RenderLayer::paintLayer
​https://bugs.webkit.org/show_bug.cgi?id=35101

No new tests, optimization only.

  • rendering/RenderLayer.cpp: (WebCore::RenderLayer::paintLayer):
6:21 AM Changeset in webkit [55080] by vestbo@webkit.org
  • 2 edits in trunk/BugsSite

2010-02-22 Tor Arne Vestbø <​tor.arne.vestbo@nokia.com>

Reviewed by Maciej Stachowiak.

Don't reset the assignee back to default on component change

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

There's a risk of reseting a valid assignee for example when
triaging a bug. All components in WebKit have the default
assignee set to ​webkit-unassigned@webkit.org so this should
not cause problems for people relying on this functionality.

  • template/en/custom/bug/edit.html.tmpl:
5:42 AM Changeset in webkit [55079] by vestbo@webkit.org
  • 6 edits
    2 adds in trunk/WebCore

[Qt] Switching from Phonon to QtMultimedia Backend for Qt 4.7

Patch by Nicholas Young <​nicholas.young@nokia.com> on 2010-02-22
Reviewed by Tor Arne Vestbø.

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

No new tests. This patch only affects multimedia backend.

  • WebCore.pro: Build depends on Qt version.
  • css/mediaControlsQt.css: Updated media controls.
  • platform/graphics/MediaPlayer.cpp: Different backend included depending on Qt version.
  • platform/graphics/qt/MediaPlayerPrivateQt.cpp: Added new QtMultimedia Backend.

(WebCore::MediaPlayerPrivate::create):
(WebCore::MediaPlayerPrivate::registerMediaEngine):
(WebCore::MediaPlayerPrivate::getSupportedTypes):
(WebCore::MediaPlayerPrivate::supportsType):
(WebCore::MediaPlayerPrivate::MediaPlayerPrivate):
(WebCore::MediaPlayerPrivate::~MediaPlayerPrivate):
(WebCore::MediaPlayerPrivate::hasVideo):
(WebCore::MediaPlayerPrivate::hasAudio):
(WebCore::MediaPlayerPrivate::load):
(WebCore::MediaPlayerPrivate::cancelLoad):
(WebCore::MediaPlayerPrivate::play):
(WebCore::MediaPlayerPrivate::pause):
(WebCore::MediaPlayerPrivate::paused):
(WebCore::MediaPlayerPrivate::seek):
(WebCore::MediaPlayerPrivate::seeking):
(WebCore::MediaPlayerPrivate::duration):
(WebCore::MediaPlayerPrivate::currentTime):
(WebCore::MediaPlayerPrivate::buffered):
(WebCore::MediaPlayerPrivate::maxTimeSeekable):
(WebCore::MediaPlayerPrivate::bytesLoaded):
(WebCore::MediaPlayerPrivate::totalBytes):
(WebCore::MediaPlayerPrivate::setRate):
(WebCore::MediaPlayerPrivate::setVolume):
(WebCore::MediaPlayerPrivate::supportsMuting):
(WebCore::MediaPlayerPrivate::setMuted):
(WebCore::MediaPlayerPrivate::networkState):
(WebCore::MediaPlayerPrivate::readyState):
(WebCore::MediaPlayerPrivate::setVisible):
(WebCore::MediaPlayerPrivate::mediaStatusChanged):
(WebCore::MediaPlayerPrivate::handleError):
(WebCore::MediaPlayerPrivate::stateChanged):
(WebCore::MediaPlayerPrivate::nativeSizeChanged):
(WebCore::MediaPlayerPrivate::queuedSeekTimeout):
(WebCore::MediaPlayerPrivate::seekTimeout):
(WebCore::MediaPlayerPrivate::positionChanged):
(WebCore::MediaPlayerPrivate::durationChanged):
(WebCore::MediaPlayerPrivate::volumeChanged):
(WebCore::MediaPlayerPrivate::mutedChanged):
(WebCore::MediaPlayerPrivate::updateStates):
(WebCore::MediaPlayerPrivate::setSize):
(WebCore::MediaPlayerPrivate::naturalSize):
(WebCore::MediaPlayerPrivate::paint):
(WebCore::MediaPlayerPrivate::repaint):

  • platform/graphics/qt/MediaPlayerPrivateQt.h: Added new QtMultimedia Backend.

(WebCore::MediaPlayerPrivate::isAvailable):
(WebCore::MediaPlayerPrivate::supportsFullscreen):

  • platform/qt/RenderThemeQt.cpp:

(WebCore::RenderThemeQt::mediaControlsBaselineOpacity): New method.
(WebCore::RenderThemeQt::paintMediaBackground): Background depends on baseline opacity.
(WebCore::RenderThemeQt::paintMediaMuteButton): Changed styling.
(WebCore::RenderThemeQt::paintMediaCurrentTime): Added current time display.
(WebCore::RenderThemeQt::formatMediaControlsCurrentTime): Added time formatting.
(WebCore::RenderThemeQt::formatMediaControlsRemainingTime): Added time formatting.
(WebCore::RenderThemeQt::paintMediaVolumeSliderTrack): Volume slider added.
(WebCore::RenderThemeQt::paintMediaVolumeSliderThumb): Volume slider added.
(WebCore::RenderThemeQt::paintMediaSliderTrack): Updated for QtMultimedia.
(WebCore::RenderThemeQt::paintMediaSliderThumb): Dropped an unnecessary check.
(WebCore::RenderThemeQt::adjustSliderThumbSize): Handle a missing case.

  • platform/qt/RenderThemeQt.h: Reimplemented a few more methods.
5:02 AM Changeset in webkit [55078] by apavlov@chromium.org
  • 2 edits in trunk/WebCore

2010-02-22 Alexander Pavlov <​apavlov@chromium.org>

Not reviewed: mac build fix

  • bindings/js/ScriptDebugServer.cpp: (WebCore::ScriptDebugServer::ScriptDebugServer):
4:14 AM Changeset in webkit [55077] by apavlov@chromium.org
  • 15 edits
    2 adds in trunk

2010-02-22 Alexander Pavlov <​apavlov@chromium.org>

Reviewed by Pavel Feldman.

Web Inspector: there should be a way to "deactivate" or "skip" all breakpoints while debugging.
​https://bugs.webkit.org/show_bug.cgi?id=33217

WebCore:

  • English.lproj/localizedStrings.js:
  • bindings/js/ScriptDebugServer.cpp: (WebCore::ScriptDebugServer::ScriptDebugServer): (WebCore::ScriptDebugServer::hasBreakpoint): (WebCore::ScriptDebugServer::setBreakpointsActivated):
  • bindings/js/ScriptDebugServer.h:
  • bindings/v8/ScriptDebugServer.h: (WebCore::ScriptDebugServer::setBreakpointsActivated):
  • inspector/InspectorBackend.cpp: (WebCore::InspectorBackend::activateBreakpoints): (WebCore::InspectorBackend::deactivateBreakpoints):
  • inspector/InspectorBackend.h:
  • inspector/InspectorBackend.idl:
  • inspector/front-end/Images/deactivateBreakpointsButtonGlyph.png: Added.
  • inspector/front-end/Images/deactivateBreakpointsDisabledButtonGlyph.png: Added.
  • inspector/front-end/InspectorBackendStub.js: (.WebInspector.InspectorBackendStub.prototype.activateBreakpoints): (.WebInspector.InspectorBackendStub.prototype.deactivateBreakpoints):
  • inspector/front-end/ScriptsPanel.js: (WebInspector.ScriptsPanel): (WebInspector.ScriptsPanel.prototype.get breakpointsActivated): (WebInspector.ScriptsPanel.prototype.addBreakpoint): (WebInspector.ScriptsPanel.prototype._stepOutClicked): (WebInspector.ScriptsPanel.prototype._toggleBreakpointsClicked):
  • inspector/front-end/inspector.css:
  • inspector/front-end/textViewer.css:

WebKit/chromium:

  • src/js/DebuggerAgent.js: (devtools.DebuggerAgent): (devtools.DebuggerAgent.prototype.setBreakpointsActivated): (devtools.DebuggerAgent.prototype.handleBreakEvent_): (devtools.DebuggerAgent.prototype.handleExceptionEvent_):
  • src/js/InspectorControllerImpl.js: (devtools.InspectorBackendImpl.prototype.activateBreakpoints): (devtools.InspectorBackendImpl.prototype.deactivateBreakpoints):
3:18 AM Changeset in webkit [55076] by mrowe@apple.com
  • 2 edits in trunk/WebCore

REGRESSION (r55039): Animation starts from near end when loaded over slow network
<​http://webkit.org/b/35222> / <rdar://problem/7673523>

Rubber-stamped by Sam Weinig and Maciej Stachowiak.

Roll out r55039 (related to <​http://webkit.org/b/35115>) as it causes animated GIFs
to skip intermediate frames when loading over a slower network.

  • platform/graphics/BitmapImage.cpp:

(WebCore::BitmapImage::startAnimation):

2:31 AM Changeset in webkit [55075] by mjs@apple.com
  • 12 edits
    1 delete in trunk/WebCore

Not reviewed, build fix.

Revert the previous change.

1:28 AM Changeset in webkit [55074] by mjs@apple.com
  • 12 edits
    1 add in trunk/WebCore

2010-02-21 Maciej Stachowiak <​mjs@apple.com>

Reviewed by Darin Adler.

Cache JavaScript wrappers inline in DOM nodes
​https://bugs.webkit.org/show_bug.cgi?id=35226
<rdar://problem/7664202>

8.8% speedup on Dromaeo DOM Core tests.
3.3% speedup on Hixie DOM Core tests.


  • bindings/js/JSDOMBinding.cpp: (WebCore::forgetDOMNode): Clear wrapper pointer. (WebCore::cacheDOMNodeWrapper): Cache inline too if caching for normal world.
  • bindings/js/JSDOMBinding.h: (WebCore::DOMObjectWrapperMapFor):
  • bindings/js/JSDocumentCustom.cpp: (WebCore::toJS): Remove unneeded argument from getCachedDOMNodeWrapper.
  • bindings/js/JSNodeCustom.cpp: (WebCore::createWrapperInline): Renamed version of original createWrapper. (WebCore::createWrapper): Call createWrapperInline. Out-of-line version. (WebCore::toJSNewlyCreated): Call createWrapperInline instead of createWrapper.
  • bindings/js/JSNodeCustom.h: Added. (WebCore::getCachedDOMNodeWrapper): Moved here so it can be inlined. (WebCore::toJS): Moved here so it can be inlined.
  • bindings/js/ScriptWrappable.h: (WebCore::ScriptWrappable::ScriptWrappable): Implement this in the obvious way for JavaScriptCore. (WebCore::ScriptWrappable::wrapper): (WebCore::ScriptWrappable::setWrapper): (WebCore::ScriptWrappable::clearWrapper):
  • bindings/scripts/CodeGeneratorJS.pm: Include CustomHeader heaaders in the header, not just the impl file, so they can add inlining.
  • dom/Node.idl: Add CustomHeader directive.


Add new files to build.

  • GNUmakefile.am:
  • WebCore.gypi:
  • WebCore.pro:
  • WebCore.vcproj/WebCore.vcproj:
1:13 AM Changeset in webkit [55073] by pfeldman@chromium.org
  • 9 edits
    1 add in trunk/WebCore

2010-02-22 Pavel Feldman <​pfeldman@chromium.org>

Not reviewed: windows build fix.

12:08 AM Changeset in webkit [55072] by bweinstein@apple.com
  • 1 edit in trunk/JavaScriptCore/JavaScriptCore.vcproj/JavaScriptCore/JavaScriptCore.def

Part 2 of Windows build fix.

Feb 21, 2010:

11:59 PM Changeset in webkit [55071] by pfeldman@chromium.org
  • 34 edits
    1 move
    2 deletes in trunk

2010-02-21 Pavel Feldman <​pfeldman@chromium.org>

Reviewed by Timothy Hatcher.

Web Inspector: stop on inline breakpoints on reload.

  • ScriptBreakpoint struct has been introduced and reused in InspectorController and debug server.
  • JavaScriptDebugServer was moved to bindings/js and renamed to ScriptDebugServer There were no changes to semantics, only some mechanical changes:
    • addBreakpoint and updateBreakpoint were merged into setBreakpoint
    • ScriptDebugServer now operates ScriptBreakpoint instances instead of BreakpointInfo.
    • It no longer allocates maps and breakpoint info in heap - all done on stack.
    • JavaScriptDebugListener is now ScriptDebugServer::Listener
    • Listener methods no longer have ExecState (was unused).
    • addListener/removeListener pair wuth no page argument removed (was unused).
  • InspectorController now binds sourceID to url and maintains a map of 'sticky' breakpoints. Whenever script is loaded and there is a sticky breakpoint url matching, breakpoint is being 'restored' synchronously in debug server and pushed to frontend. Front-end no longer stores map of sticky breakpoints.
    • setBreakpoint/removeBreakpoint/didParseSource trio handle this logic.
  • A bunch of if(USE_JSC/V8/other) forks removed.
  • InspectorFrontend now operates primitive types only, got rid of USE_JSC as well.

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

  • GNUmakefile.am:
  • WebCore.base.exp:
  • WebCore.gypi:
  • WebCore.order:
  • WebCore.pro:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • bindings/js/JSInjectedScriptHostCustom.cpp: (WebCore::JSInjectedScriptHost::currentCallFrame): (WebCore::JSInjectedScriptHost::isActivation):
  • bindings/js/ScriptDebugServer.cpp: (WebCore::ScriptDebugServer::shared): (WebCore::ScriptDebugServer::ScriptDebugServer): (WebCore::ScriptDebugServer::~ScriptDebugServer): (WebCore::ScriptDebugServer::addListener): (WebCore::ScriptDebugServer::removeListener): (WebCore::ScriptDebugServer::pageCreated): (WebCore::ScriptDebugServer::hasListenersInterestedInPage): (WebCore::ScriptDebugServer::setBreakpoint): (WebCore::ScriptDebugServer::removeBreakpoint): (WebCore::ScriptDebugServer::hasBreakpoint): (WebCore::ScriptDebugServer::clearBreakpoints): (WebCore::ScriptDebugServer::setPauseOnExceptionsState): (WebCore::ScriptDebugServer::pauseProgram): (WebCore::ScriptDebugServer::continueProgram): (WebCore::ScriptDebugServer::stepIntoStatement): (WebCore::ScriptDebugServer::stepOverStatement): (WebCore::ScriptDebugServer::stepOutOfFunction): (WebCore::ScriptDebugServer::currentCallFrame): (WebCore::ScriptDebugServer::currentCallFrameState): (WebCore::ScriptDebugServer::dispatchDidParseSource): (WebCore::ScriptDebugServer::dispatchFailedToParseSource): (WebCore::toPage): (WebCore::ScriptDebugServer::detach): (WebCore::ScriptDebugServer::sourceParsed): (WebCore::ScriptDebugServer::dispatchFunctionToListeners): (WebCore::ScriptDebugServer::setJavaScriptPaused): (WebCore::ScriptDebugServer::pauseIfNeeded): (WebCore::ScriptDebugServer::callEvent): (WebCore::ScriptDebugServer::atStatement): (WebCore::ScriptDebugServer::returnEvent): (WebCore::ScriptDebugServer::exception): (WebCore::ScriptDebugServer::willExecuteProgram): (WebCore::ScriptDebugServer::didExecuteProgram): (WebCore::ScriptDebugServer::didReachBreakpoint): (WebCore::ScriptDebugServer::recompileAllJSFunctionsSoon): (WebCore::ScriptDebugServer::recompileAllJSFunctions): (WebCore::ScriptDebugServer::didAddListener): (WebCore::ScriptDebugServer::didRemoveListener): (WebCore::ScriptDebugServer::didRemoveLastListener):
  • bindings/js/ScriptDebugServer.h: (WebCore::ScriptDebugServer::Listener::~Listener): (WebCore::ScriptDebugServer::): (WebCore::ScriptDebugServer::pauseOnExceptionsState): (WebCore::ScriptDebugServer::hasListeners): (WebCore::ScriptDebugServer::hasGlobalListeners):
  • bindings/v8/ScriptDebugServer.cpp: (WebCore::ScriptDebugServer::shared):
  • bindings/v8/ScriptDebugServer.h: (WebCore::ScriptDebugServer::Listener::~Listener): (WebCore::ScriptDebugServer::addListener): (WebCore::ScriptDebugServer::removeListener): (WebCore::ScriptDebugServer::setBreakpoint): (WebCore::ScriptDebugServer::removeBreakpoint): (WebCore::ScriptDebugServer::clearBreakpoints): (WebCore::ScriptDebugServer::): (WebCore::ScriptDebugServer::pauseOnExceptionsState): (WebCore::ScriptDebugServer::setPauseOnExceptionsState): (WebCore::ScriptDebugServer::pauseProgram): (WebCore::ScriptDebugServer::continueProgram): (WebCore::ScriptDebugServer::stepIntoStatement): (WebCore::ScriptDebugServer::stepOverStatement): (WebCore::ScriptDebugServer::stepOutOfFunction): (WebCore::ScriptDebugServer::recompileAllJSFunctionsSoon): (WebCore::ScriptDebugServer::recompileAllJSFunctions): (WebCore::ScriptDebugServer::currentCallFrameState): (WebCore::ScriptDebugServer::pageCreated): (WebCore::ScriptDebugServer::ScriptDebugServer): (WebCore::ScriptDebugServer::~ScriptDebugServer):
  • bindings/v8/custom/V8InjectedScriptHostCustom.cpp: (WebCore::V8InjectedScriptHost::currentCallFrameCallback): (WebCore::V8InjectedScriptHost::isActivationCallback):
  • inspector/InjectedScriptHost.cpp:
  • inspector/InjectedScriptHost.h:
  • inspector/InjectedScriptHost.idl:
  • inspector/InspectorBackend.cpp: (WebCore::InspectorBackend::setBreakpoint): (WebCore::InspectorBackend::removeBreakpoint): (WebCore::InspectorBackend::pauseInDebugger): (WebCore::InspectorBackend::stepOverStatementInDebugger): (WebCore::InspectorBackend::stepIntoStatementInDebugger): (WebCore::InspectorBackend::stepOutOfFunctionInDebugger): (WebCore::InspectorBackend::pauseOnExceptionsState): (WebCore::InspectorBackend::setPauseOnExceptionsState):
  • inspector/InspectorBackend.h:
  • inspector/InspectorBackend.idl:
  • inspector/ScriptBreakpoint.h: Added. (WebCore::InspectorBreakpoint::InspectorBreakpoint):
  • inspector/InspectorController.cpp: (WebCore::InspectorController::InspectorController): (WebCore::InspectorController::setWindowVisible): (WebCore::InspectorController::scriptObjectReady): (WebCore::InspectorController::close): (WebCore::InspectorController::didCommitLoad): (WebCore::InspectorController::addProfile): (WebCore::InspectorController::startUserInitiatedProfiling): (WebCore::InspectorController::stopUserInitiatedProfiling): (WebCore::InspectorController::enableProfiler): (WebCore::InspectorController::disableProfiler): (WebCore::InspectorController::enableDebuggerFromFrontend): (WebCore::InspectorController::disableDebugger): (WebCore::InspectorController::resumeDebugger): (WebCore::InspectorController::setBreakpoint): (WebCore::InspectorController::removeBreakpoint): (WebCore::InspectorController::didParseSource): (WebCore::InspectorController::failedToParseSource): (WebCore::InspectorController::didPause):
  • inspector/InspectorController.h:
  • inspector/InspectorFrontend.cpp: (WebCore::InspectorFrontend::parsedScriptSource): (WebCore::InspectorFrontend::restoredBreakpoint): (WebCore::InspectorFrontend::failedToParseScriptSource): (WebCore::InspectorFrontend::resumedScript):
  • inspector/InspectorFrontend.h:
  • inspector/JavaScriptDebugListener.h: Removed.
  • inspector/JavaScriptDebugServer.cpp: Removed.
  • inspector/JavaScriptDebugServer.h: Removed.
  • inspector/front-end/Breakpoint.js: (WebInspector.Breakpoint.prototype.set condition):
  • inspector/front-end/BreakpointsSidebarPane.js: (WebInspector.BreakpointsSidebarPane.prototype.addBreakpoint): (WebInspector.BreakpointsSidebarPane.prototype._breakpointEnableChanged):
  • inspector/front-end/InspectorBackendStub.js: (.WebInspector.InspectorBackendStub.prototype.addBreakpoint):
  • inspector/front-end/ResourcesPanel.js: (WebInspector.ResourcesPanel.prototype.sourceFrameForResource):
  • inspector/front-end/ScriptsPanel.js: (WebInspector.ScriptsPanel): (WebInspector.ScriptsPanel.prototype.addScript): (WebInspector.ScriptsPanel.prototype.addBreakpoint): (WebInspector.ScriptsPanel.prototype.removeBreakpoint): (WebInspector.ScriptsPanel.prototype._showScriptOrResource):
  • inspector/front-end/inspector.js: (WebInspector.restoredBreakpoint):
  • page/Page.cpp: (WebCore::Page::Page):
  • platform/android/TemporaryLinkStubs.cpp:
11:56 PM Changeset in webkit [55070] by bweinstein@apple.com
  • 1 edit in trunk/JavaScriptCore/JavaScriptCore.vcproj/JavaScriptCore/JavaScriptCore.def

Part 1 of Windows build fix.

10:50 PM Changeset in webkit [55069] by barraclough@apple.com
  • 7 edits in trunk

JavaScriptCore: Make UString::m_data be const, and make the UChar owned/ref-counted by CrossThreadRefCounted be const too.

Reviewed by Oliver Hunt.

  • runtime/UStringImpl.cpp:

(JSC::UStringImpl::baseSharedBuffer):
(JSC::UStringImpl::~UStringImpl):

  • runtime/UStringImpl.h:

(JSC::UStringImpl::create):
(JSC::UStringImpl::data):
(JSC::UStringImpl::UStringImpl):

  • wtf/OwnFastMallocPtr.h:

(WTF::OwnFastMallocPtr::~OwnFastMallocPtr):

WebCore: Make the UChar owned/ref-counted by StringImpl::CrossThreadRefCounted be const.

Reviewed by Oliver Hunt.

  • platform/text/StringImpl.cpp:

(WebCore::StringImpl::sharedBuffer):

  • platform/text/StringImpl.h:
10:12 PM Changeset in webkit [55068] by eric@webkit.org
  • 2 edits in trunk/JavaScriptCore

2010-02-21 Yuta Kitamura <​yutak@chromium.org>

Reviewed by Darin Adler.

HashMapTranslatorAdapter::translate() needs to set the mapped value.

HTTPHeaderMap::add(const char*, const String&) does not work
​https://bugs.webkit.org/show_bug.cgi?id=35227

  • wtf/HashMap.h: (WTF::HashMapTranslatorAdapter::translate):
8:12 PM Changeset in webkit [55067] by yuzo@google.com
  • 4 edits
    3 adds in trunk

2010-02-07 Yuzo Fujishima <​yuzo@google.com>

Reviewed by Eric Seidel.

When page-break-{after,before} is set to always, force page breaks even for overflow-specified elements.
RenderBlock::inRootBlockContext() was introduced by Changeset 5611. Although it is a reasonable criteria for choosing an optional page break location, it is not for a mandatory page break as specified by ​http://dev.w3.org/csswg/css3-page/#forced-pg-brk. The method is removed because it is not used anywhere else.
Note: this patch makes page break work for overflow-specified elements. For tables and floated elements, more work is needed.
​https://bugs.webkit.org/show_bug.cgi?id=9526

  • printing/page-break-always-for-overflow-expected.txt: Added.
  • printing/page-break-always-for-overflow.html: Added.
  • printing/script-tests/page-break-always-for-overflow.js: Added.

2010-02-07 Yuzo Fujishima <​yuzo@google.com>

Reviewed by Eric Seidel.

When page-break-{after,before} is set to always, force page breaks even for overflow-specified elements.
RenderBlock::inRootBlockContext() was introduced by Changeset 5611. Although it is a reasonable criteria for choosing an optional page break location, it is not for a mandatory page break as specified by ​http://dev.w3.org/csswg/css3-page/#forced-pg-brk. The method is removed because it is not used anywhere else.
Note: this patch makes page break work for overflow-specified elements. For tables and floated elements, more work is needed.
​https://bugs.webkit.org/show_bug.cgi?id=9526

Test: printing/page-break-always-for-overflow.html

  • rendering/RenderBlock.cpp: (WebCore::RenderBlock::paintChildren):
  • rendering/RenderBlock.h:
3:40 PM Changeset in webkit [55066] by jchaffraix@webkit.org
  • 19 edits in trunk/WebCore

Remove auto_ptr usage in WebCore.
​https://bugs.webkit.org/show_bug.cgi?id=35157

Reviewed by Eric Seidel.

The changes consists of:

  • Changing auto_ptr arguments or return types to PassOwnPtr.
  • Replacing local auto_ptr by OwnPtr.
  • Removing now unneeded <memory> inclusion.
  • css/CSSParser.cpp:

(WebCore::CSSParser::addProperty):

  • loader/WorkerThreadableLoader.cpp:

(WebCore::WorkerThreadableLoader::MainThreadBridge::mainThreadCreateLoader):
(WebCore::workerContextDidReceiveResponse):
(WebCore::workerContextDidReceiveData):
(WebCore::WorkerThreadableLoader::MainThreadBridge::didReceiveData):
(WebCore::workerContextDidReceiveAuthenticationCancellation):

  • loader/WorkerThreadableLoader.h:
  • loader/icon/IconLoader.cpp:

(WebCore::IconLoader::create):

  • loader/icon/IconLoader.h:
  • platform/ContextMenu.cpp:

(WebCore::separatorItem):

  • platform/CrossThreadCopier.h:

(WebCore::):

  • platform/network/HTTPHeaderMap.cpp:

(WebCore::HTTPHeaderMap::copyData):
(WebCore::HTTPHeaderMap::adopt):

  • platform/network/HTTPHeaderMap.h:
  • platform/network/ResourceRequestBase.cpp:

(WebCore::ResourceRequestBase::adopt):
(WebCore::ResourceRequestBase::copyData):

  • platform/network/ResourceRequestBase.h:
  • platform/network/ResourceResponseBase.cpp:

(WebCore::ResourceResponseBase::adopt):
(WebCore::ResourceResponseBase::copyData):

  • platform/network/ResourceResponseBase.h:
  • svg/SVGDocumentExtensions.cpp:

(WebCore::SVGDocumentExtensions::removePendingResource):

  • svg/SVGDocumentExtensions.h:
  • svg/SVGElement.cpp:

(WebCore::SVGElement::insertedIntoDocument):

  • workers/GenericWorkerTask.h:
  • workers/WorkerThread.cpp:

(WebCore::WorkerThreadStartupData::create):

1:47 PM Changeset in webkit [55065] by Simon Fraser
  • 4 edits in trunk/WebCore

2010-02-21 Simon Fraser <Simon Fraser>

Reviewed by Dan Bernstein.

​https://bugs.webkit.org/show_bug.cgi?id=22215
Avoid calling absoluteClippedOverflowRect() so many times

RenderLayer::updateLayerPositions() computes the clipped overflow rect
and the outline bounds for repaint, and then calls repaintAfterLayoutIfNeeded()
which can compute the same rects all over again. Avoid this by passing
these two rects into repaintAfterLayoutIfNeeded() if known. This measurably
reduces the time spent in updateLayerPositions() for some content.

  • rendering/RenderLayer.cpp: (WebCore::RenderLayer::updateLayerPositions):
  • rendering/RenderObject.cpp: (WebCore::RenderObject::repaintAfterLayoutIfNeeded):
  • rendering/RenderObject.h:
1:19 PM Changeset in webkit [55064] by mitz@apple.com
  • 2 edits in trunk/WebKit/mac

False warnings about needing layout in
-[WebHTMLView _recursiveDisplayRectIfNeededIgnoringOpacity:isVisibleRect:rectIsVisibleRectForView:topView:]
​https://bugs.webkit.org/show_bug.cgi?id=35218

Reviewed by Darin Adler.

  • WebView/WebHTMLView.mm:

(-[WebHTMLView _recursiveDisplayRectIfNeededIgnoringOpacity:isVisibleRect:rectIsVisibleRectForView:topView:]):
Suppress the warning and the forced layout if the view is not being drawn
in this display operation.

12:20 PM Changeset in webkit [55063] by kdecker@apple.com
  • 2 edits in trunk/WebKit/mac

Reviewed by Sam Weinig.

​https://bugs.webkit.org/show_bug.cgi?id=35215
<rdar://problem/7673157>


The plugInViewWithArguments: API passes a dictionary of plugin arguments. One of the parameters
is WebPlugInBaseURLKey, which is a key that represents the base URL of the document containing
the plug-in's view. Instead of sending the base URL, code in WebFrameLoaderClient::createPlugin
would incorrectly pass the source URL of the plug-in resource.

  • WebCoreSupport/WebFrameLoaderClient.mm: (WebFrameLoaderClient::createPlugin): When building the plug-in arguments dictionary, pass the real base URL for the WebPlugInBaseURLKey key.
3:32 AM Changeset in webkit [55062] by mjs@apple.com
  • 3 edits
    17 adds in trunk

2010-02-20 Maciej Stachowiak <​mjs@apple.com>

Rubber stamped by Adam Barth.

Add sunspider-0.9.1 to the site in preparation for announcing it.
​https://bugs.webkit.org/show_bug.cgi?id=35206

  • perf/sunspider-0.9.1: Added.
  • perf/sunspider-0.9.1/json2.js: Added.
  • perf/sunspider-0.9.1/sunspider-0.9: Added.
  • perf/sunspider-0.9.1/sunspider-0.9.1: Added.
  • perf/sunspider-0.9.1/sunspider-0.9.1/driver.html: Added.
  • perf/sunspider-0.9.1/sunspider-0.9.1/results.html: Added.
  • perf/sunspider-0.9.1/sunspider-0.9.1/sunspider-test-contents.js: Added.
  • perf/sunspider-0.9.1/sunspider-0.9.1/sunspider-test-prefix.js: Added.
  • perf/sunspider-0.9.1/sunspider-0.9/driver.html: Added.
  • perf/sunspider-0.9.1/sunspider-0.9/results.html: Added.
  • perf/sunspider-0.9.1/sunspider-0.9/sunspider-test-contents.js: Added.
  • perf/sunspider-0.9.1/sunspider-0.9/sunspider-test-prefix.js: Added.
  • perf/sunspider-0.9.1/sunspider-analyze-results.js: Added.
  • perf/sunspider-0.9.1/sunspider-compare-results.js: Added.
  • perf/sunspider-0.9.1/sunspider.css: Added.
  • perf/sunspider-0.9.1/sunspider.html: Added.
  • perf/sunspider-0.9.1/versions.html: Added.

2010-02-20 Maciej Stachowiak <​mjs@apple.com>

Reviewed by Adam Barth.

Add sunspider-0.9.1 to the site in preparation for announcing it.
​https://bugs.webkit.org/show_bug.cgi?id=35206

  • hosted/sunspider.css: Remove tabs, which I noticed where present while copying to the WebKitSite directory.
Note: See TracTimeline for information about the timeline view.