Timeline


and

10/21/09:

23:20 Changeset [49931] by eric@webkit.org

2009-10-21 Eric Seidel <eric@webkit.org>

Reviewed by Adam Barth.

bugzilla-tool's "patch failed to download an apply" error should give more information
https://bugs.webkit.org/show_bug.cgi?id=30632

  • Scripts/modules/scm.py:
    • Use the common run_command method instead of custom POpen code.
    • Make run_command know how to take pipes as input.
  • Scripts/modules/scm_unittest.py:
    • Add new tests to cover change.
    • Also move test_error_handlers into new SCMClassTests so we don't run it 3 times.
21:32 Changeset [49930] by tkent@chromium.org

committers

16:57 Changeset [49929] by dumi@chromium.org

Adding the ability to enable/disable functions in V8 at runtime.

Patch by Dumitru Daniliuc <dumi@chromium.org> on 2009-10-21
Reviewed by Dimitri Glazkov.

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

  • bindings/scripts/CodeGeneratorV8.pm:
16:47 Changeset [49928] by jorlow@chromium.org

Combine local and session storage tests into one and use script-tests properly
https://bugs.webkit.org/show_bug.cgi?id=30640

Patch by Jeremy Orlow <jorlow@chromium.org> on 2009-10-21
Reviewed by Darin Adler.

I was apparently misusing the script-tests stuff. I changed all the tests to
test both local storage and session storage so that we can use the
make-test-script-wrappers stuff as is.

  • storage/domstorage/clear-expected.txt: Added.
  • storage/domstorage/clear.html: Added.
  • storage/domstorage/complex-keys-expected.txt: Added.
  • storage/domstorage/complex-keys.html: Added.
  • storage/domstorage/complex-values-expected.txt: Added.
  • storage/domstorage/complex-values.html: Added.
  • storage/domstorage/localstorage/clear-expected.txt: Removed.
  • storage/domstorage/localstorage/clear.html: Removed.
  • storage/domstorage/localstorage/complex-keys-expected.txt: Removed.
  • storage/domstorage/localstorage/complex-keys.html: Removed.
  • storage/domstorage/localstorage/complex-values-expected.txt: Removed.
  • storage/domstorage/localstorage/complex-values.html: Removed.
  • storage/domstorage/localstorage/quota-expected.txt: Removed.
  • storage/domstorage/localstorage/quota.html: Removed.
  • storage/domstorage/localstorage/remove-item-expected.txt: Removed.
  • storage/domstorage/localstorage/remove-item.html: Removed.
  • storage/domstorage/quota-expected.txt: Added.
  • storage/domstorage/quota.html: Added.
  • storage/domstorage/remove-item-expected.txt: Added.
  • storage/domstorage/remove-item.html: Added.
  • storage/domstorage/script-tests/TEMPLATE.html: Added.
  • storage/domstorage/script-tests/clear.js:

(test):
(runTest):

  • storage/domstorage/script-tests/complex-keys.js:

(test):
(runTest):

  • storage/domstorage/script-tests/complex-values.js:

(test):
(runTest):

  • storage/domstorage/script-tests/no-quota.js: Removed.
  • storage/domstorage/script-tests/quota.js:

(testQuota):
(testNoQuota):
(runTest):

  • storage/domstorage/script-tests/remove-item.js:

(test):
(runTest):

  • storage/domstorage/sessionstorage/clear-expected.txt: Removed.
  • storage/domstorage/sessionstorage/clear.html: Removed.
  • storage/domstorage/sessionstorage/complex-keys-expected.txt: Removed.
  • storage/domstorage/sessionstorage/complex-keys.html: Removed.
  • storage/domstorage/sessionstorage/complex-values-expected.txt: Removed.
  • storage/domstorage/sessionstorage/complex-values.html: Removed.
  • storage/domstorage/sessionstorage/no-quota-expected.txt: Removed.
  • storage/domstorage/sessionstorage/no-quota.html: Removed.
  • storage/domstorage/sessionstorage/remove-item-expected.txt: Removed.
  • storage/domstorage/sessionstorage/remove-item.html: Removed.
16:39 Changeset [49927] by kevino@webkit.org

Reviewed by Kevin Ollivier.

Update the Windows installer builder to work with Vista / Win 7 and with git.

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

16:10 Changeset [49926] by darin@apple.com

WebCore: Swedish search (and other languages as well) is broken while fixing Japanese search
https://bugs.webkit.org/show_bug.cgi?id=30646

Patch by Darin Adler <darin@apple.com> on 2009-10-21
Rolled Japanese tailoring out that was done to fix
https://bugs.webkit.org/show_bug.cgi?id=30437 earlier.
It was overriding the locale-specific tailoring.
We'll land a fix once we figure out how to add the
Japanese tailoring without removing anything.

  • editing/TextIterator.cpp: Rolled out r49924.

LayoutTests: Update since we rolled Japanese tailoring out that was done to fix
https://bugs.webkit.org/show_bug.cgi?id=30437 earlier.

Patch by Darin Adler <darin@apple.com> on 2009-10-21

  • fast/text/international/japanese-kana-letters-expected.txt: Update

test results to expect failure. We'll land results that expect
success once we re-fix the bug.

16:06 Changeset [49925] by eric@webkit.org

2009-10-21 Andrew Scherkus <scherkus@chromium.org>

Reviewed by Eric Seidel.

Update more media layout tests to use findMediaFile().

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

  • media/controls-drag-timebar.html: Switch to using findMediaFile().
  • media/video-currentTime-set.html: Remove src attribute and rely on findMediaFile() instead.
15:16 Changeset [49924] by bweinstein@apple.com

Roll out r49922 because it broke Tiger and SL in strange ways.

14:59 Changeset [49923] by pfeldman@chromium.org

2009-10-21 Pavel Feldman <pfeldman@chromium.org>

Not reviewed: chromium build fix - add missing image to gypi.

  • WebCore.gypi:
14:51 Changeset [49922] by bweinstein@apple.com

WebCore: Fixes part of <http://webkit.org/b/30522>.
Web Inspector: DOM Exceptions throughout the Inspector should be more human readable.

Reviewed by Eric Seidel.

Expose the description attribute that is now a member of
ExceptionBase as of r49723, so when a user logs an exception, they
can see the description in the Web Inspector.

Test: fast/dom/dom-exception-description.html

  • dom/DOMCoreException.idl:
  • dom/EventException.idl:
  • dom/RangeException.idl:
  • svg/SVGException.idl:
  • xml/XMLHttpRequestException.idl:
  • xml/XPathException.idl:

LayoutTests: Fixes part of <http://webkit.org/b/30522>.
Web Inspector: DOM Exceptions throughout the Inspector should be more human readable.

Reviewed by Eric Seidel.

Adds a test to check the name and description of some DOM Exceptions.

  • fast/dom/dom-exception-description-expected.txt: Added.
  • fast/dom/dom-exception-description.html: Added.
  • fast/dom/script-tests/dom-exception-description.js: Added.
13:47 Changeset [49921] by eric@webkit.org

2009-10-21 Alejandro G. Castro <alex@igalia.com>

Reviewed by Gustavo Noronha.

[GTK] Added conditional code to avoid using
gdk_window_get_root_coords if we do not have a gtk+ release newer
than 2.17.3.
https://bugs.webkit.org/show_bug.cgi?id=30636

  • WebKitTools/DumpRenderTree/gtk/EventSender.cpp:
13:28 Writing Layout Tests for DumpRenderTree edited by jorlow@chromium.org
(diff)
13:22 WebKit Team edited by tkent@chromium.org
(diff)
13:02 Changeset [49920] by sullivan@apple.com

Rubber-stamped by Dan Bernstein.

  • WebView/WebDynamicScrollBarsView.h:

Corrected a couple of long-obsolete comments regarding Safari's use of this class.

12:53 Changeset [49919] by bweinstein@apple.com

Fixes <https://bugs.webkit.org/show_bug.cgi?id=30616>.
REGRESSION (r49036): Web Inspector: Summary graph no longer switching to size summary.

Reviewed by Timothy Hatcher.

Add a check to make sure a function we need to calculate percentages is
defined before we use it.

  • inspector/front-end/ResourcesPanel.js:

(WebInspector.ResourcesPanel.prototype._updateGraphDividersIfNeeded):

12:43 Changeset [49918] by eric@webkit.org

2009-10-21 Alejandro G. Castro <alex@igalia.com>

Reviewed by Xan Lopez.

[GTK] Removed 5333725.html from Skipped and modified the expected
result of 5232159-expected.txt and fake-drag-expected.txt, this
modifications come from the changes in EventSender (bug 30633 and
bug 30636).
https://bugs.webkit.org/show_bug.cgi?id=30593

  • LayoutTests/platform/gtk/Skipped:
  • LayoutTests/platform/gtk/editing/selection/5232159-expected.txt:
  • LayoutTests/platform/gtk/editing/selection/fake-drag-expected.tx :
12:36 Changeset [49917] by eric@webkit.org

2009-10-21 Keishi Hattori <casey.hattori@gmail.com>

Reviewed by Timothy Hatcher.

REGRESSION: Clicking on nodes in the console should take you to the element in the DOM
https://bugs.webkit.org/show_bug.cgi?id=27231

  • inspector/front-end/ConsoleView.js: (WebInspector.ConsoleView.prototype._formatnode): Turns on ElementsTreeOutline.showInElementsPanelEnabled
  • inspector/front-end/ElementsTreeOutline.js: (WebInspector.ElementsTreeOutline): Added ElementsTreeOutline.showInElementsPanelEnabled (WebInspector.ElementsTreeElement.prototype.onmousedown): Reveals the node in the Elements panel if treeOutline.showInElementsPanelEnabled is true.
12:22 Changeset [49916] by jorlow@chromium.org

https://bugs.webkit.org/show_bug.cgi?id=30637
Fix a compiler warning in windows.

Patch by Jeremy Orlow <jorlow@chromium.org> on 2009-10-21
Reviewed by Eric Seidel.

  • loader/ResourceLoadNotifier.h:
12:18 Changeset [49915] by eric@webkit.org

2009-10-21 Shu Chang <Chang.Shu@nokia.com>

Reviewed by Eric Seidel.

[Qt] Added dummy implementation for keepWebHistory()
https://bugs.webkit.org/show_bug.cgi?id=30592

  • DumpRenderTree/qt/LayoutTestControllerQt.cpp: (LayoutTestController::keepWebHistory):
  • DumpRenderTree/qt/LayoutTestControllerQt.h:
12:10 Changeset [49914] by eric@webkit.org

2009-10-21 Victor Wang <victorw@chromium.org>

Reviewed by Eric Seidel.

Fix click-focus-anchor layout test to mimic click event.

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

  • fast/events/click-focus-anchor.html:
12:03 Changeset [49913] by eric@webkit.org

2009-10-21 Alejandro G. Castro <alex@igalia.com>

Reviewed by Xan Lopez.

[GTK] Fixed the double click condition, it is not double click if
we move in just in one direction.
https://bugs.webkit.org/show_bug.cgi?id=30636

  • WebKitTools/DumpRenderTree/gtk/EventSender.cpp:
11:55 Changeset [49912] by eric@webkit.org

2009-10-21 Alejandro G. Castro <alex@igalia.com>

Reviewed by Xan Lopez.

[GTK] Initialize the events completly before emitting them.
https://bugs.webkit.org/show_bug.cgi?id=30633

  • WebKitTools/DumpRenderTree/gtk/EventSender.cpp:
11:32 Changeset [49911] by dimich@chromium.org

Fix the crash when a node not in a document receives dispatchEvent.
https://bugs.webkit.org/show_bug.cgi?id=30611

Reviewed by Darin Adler.

WebCore:

Test: fast/events/dispatch-event-no-document.html

  • dom/EventTarget.cpp: Check for scriptExecutionContext() at the moment of dispatchEvent; do nothing if no context.

(WebCore::EventTarget::dispatchEvent):

LayoutTests:

  • fast/events/dispatch-event-no-document-expected.txt: Added.
  • fast/events/dispatch-event-no-document.html: Added.
11:25 Changeset [49910] by kevino@webkit.org

wx build fix. Fix for when linking using --as-needed with gcc.

11:23 Changeset [49909] by agl@chromium.org

2009-10-21 Adam Langley <agl@google.com>

Reviewed by Eric Seidel.

Chromium Linux: disable subpixel text on layers.

https://bugs.webkit.org/show_bug.cgi?id=30635
http://code.google.com/p/chromium/issues/detail?id=25365

With the addition of layers for drawing rounded corners in r49641,
subpixel text on rounded rectangles broke. This is because the layer
only contains a single alpha channel and this is insufficient to
compose subpixel text correctly.

On Windows, a large body of code in TransparencyWin.cpp exists to try
to deal with this. Even then, in some cases, it downgrades to
anti-aliased text. We need a fix for the grevious effects quickly thus
this patch disables subpixel text when rendering into a layer.

This would be covered by existing tests except that subpixel text is
disabled for pixel tests on Chromium Linux.

  • platform/graphics/chromium/FontLinux.cpp: (WebCore::isCanvasMultiLayered): (WebCore::adjustTextRenderMode): (WebCore::Font::drawGlyphs): (WebCore::Font::drawComplexText):
11:13 Changeset [49908] by kevino@webkit.org

Reviewed by Kevin Ollivier.

Include 'WebFrame.h' declared classes in wxPython bindings.

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

11:06 Changeset [49907] by kevino@webkit.org

wxMac build fixes. Fix issues with linking against wrong versions of sqlite3 and curl on Mac 10.4.

09:47 Changeset [49906] by mitz@apple.com

WebCore: Fixed background-clip parsing regressions introduced in r46240.

Reviewed by Sam Weinig.

Test: fast/css/background-clip-text.html

  • css/CSSParser.cpp:

(WebCore::parseBackgroundClip): Added this helper method.
(WebCore::CSSParser::parseFillShorthand): Use parseBackgroundClip() to
reparse the value. The old code called parseFillProperty() again, which
didn’t work, because parseFillProperty() advances the value list.
(WebCore::CSSParser::parseFillProperty): Changed to use
parseBackgroundClip().

  • css/CSSPrimitiveValueMappings.h:

(WebCore::CSSPrimitiveValue::operator EFillBox): Added a case for
CSSValueWebkitText.

LayoutTests: Added a background-clip parsing test that would have caught some
regressions that were introduced in r46240.

Reviewed by Sam Weinig.

  • fast/css/background-clip-text-expected.txt: Added.
  • fast/css/background-clip-text.html: Added.
  • fast/css/script-tests: Added.
  • fast/css/script-tests/TEMPLATE.html: Copied from LayoutTests/fast/backgrounds/repeat/script-tests/TEMPLATE.html.
  • fast/css/script-tests/background-clip-text.js: Added.

(test):

06:57 Changeset [49905] by eric@webkit.org

2009-10-21 Alexander Pavlov <apavlov@chromium.org>

Reviewed by Timothy Hatcher.

Enable creation of custom SidebarTreeElements for different ProfileTypes

ProfileTypes can now create sidebar tree elements of custom types.
https://bugs.webkit.org/show_bug.cgi?id=30520

  • inspector/front-end/ProfileView.js: (WebInspector.CPUProfileType.prototype.setRecordingProfile): (WebInspector.CPUProfileType.prototype.createSidebarTreeElementForProfile): (WebInspector.CPUProfileType.prototype.createView): (WebInspector.CPUProfile.prototype.get head):
  • inspector/front-end/ProfilesPanel.js: (WebInspector.ProfileType.prototype.buttonClicked): (WebInspector.ProfileType.prototype.viewForProfile): (WebInspector.ProfileType.prototype.createView): (WebInspector.ProfileType.prototype.createSidebarTreeElementForProfile): (WebInspector.ProfilesPanel.prototype.addProfileHeader): (WebInspector.ProfilesPanel.prototype.showProfile):
06:45 Changeset [49904] by joepeck@webkit.org

2009-10-20 Joseph Pecoraro <joepeck@webkit.org>

Reviewed by Timothy Hatcher.

Web Inspector: Selectors in the Styles pane should trigger a search for that selector when clicked
https://bugs.webkit.org/show_bug.cgi?id=17126

  • inspector/front-end/ElementsPanel.js: (WebInspector.ElementsPanel.prototype.addNodesToSearchResult): only change focusedDOMNode on a manual search
  • inspector/front-end/StylesSidebarPane.js: (WebInspector.StylePropertiesSection.prototype._clickSelector): trigger a search
  • inspector/front-end/ElementsTreeOutline.js: (WebInspector.ElementsTreeElement.prototype._startEditing): correctly start editing attribute value instead of name
06:33 Changeset [49903] by pfeldman@chromium.org

2009-10-20 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Timothy Hatcher.

Web Inspector: Fix DOM Agent leak.

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

  • inspector/InspectorController.cpp: (WebCore::InspectorController::~InspectorController):
06:08 Changeset [49902] by eric@webkit.org

2009-10-21 Girish Ramakrishnan <girish@forwardbias.in>

Reviewed by Holger Freyther.

Plugins: Create manual tests for testing plugins

Create two test files:

  1. windowed.html - Test plugin in windowed mode.
  2. windowless.html - Test plugin in windowless mode.

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

  • manual-tests/plugins/windowed.html: Added.
  • manual-tests/plugins/windowless.html: Added.
02:10 Changeset [49901] by xan@webkit.org

2009-10-21 Xan Lopez <xlopez@igalia.com>

Skip another failing test.

  • platform/gtk/Skipped:
00:52 Changeset [49900] by xan@webkit.org

2009-10-21 Xan Lopez <xlopez@igalia.com>

Skip a few more libxml2 related tests.

  • platform/gtk/Skipped:
00:11 Changeset [49899] by eric@webkit.org

2009-10-21 Satoshi Nakagawa <psychs@limechat.net>

Reviewed by Darin Adler.

Fixed Japanese text search problems.
Treat small kana letters and kana letters as different characters in search.
Do not ignore diacritic marks in search for Japanese texts.

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

  • fast/text/international/japanese-kana-letters-expected.txt: Added.
  • fast/text/international/japanese-kana-letters.html: Added.

2009-10-21 Satoshi Nakagawa <psychs@limechat.net>

Reviewed by Darin Adler.

Fixed Japanese text search problems.
Treat small kana letters and kana letters as different characters in search.
Do not ignore diacritic marks in search for Japanese texts.

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

Test: fast/text/international/japanese-kana-letters.html

  • editing/TextIterator.cpp: (WebCore::createCollator): (WebCore::collator): (WebCore::createSearcher):

10/20/09:

21:56 Changeset [49898] by eric@webkit.org

2009-10-20 Eric Z. Ayers <zundel@google.com>

Reviewed by Timothy Hatcher.

Changes TimelineItems to be created with raw timestamps as opposed
to a time relative to a start of session. Normalized timestamps cause
problems when monitoring a browsing session across multiple page
transitions.

  • inspector/InspectorTimelineAgent.cpp: (WebCore::InspectorTimelineAgent::InspectorTimelineAgent): (WebCore::InspectorTimelineAgent::willDispatchDOMEvent): (WebCore::InspectorTimelineAgent::willLayout): (WebCore::InspectorTimelineAgent::willRecalculateStyle): (WebCore::InspectorTimelineAgent::willPaint): (WebCore::InspectorTimelineAgent::willWriteHTML): (WebCore::InspectorTimelineAgent::reset): (WebCore::InspectorTimelineAgent::didCompleteCurrentRecord):
  • inspector/InspectorTimelineAgent.h:
20:37 Changeset [49897] by pfeldman@chromium.org

2009-10-20 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Timothy Hatcher.

Web Inspector: simple console tests do not require testing harness.

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

  • inspector/uncaught-dom1-exception-expected.txt:
  • inspector/uncaught-dom1-exception.html:
  • inspector/uncaught-dom3-exception-expected.txt:
  • inspector/uncaught-dom3-exception.html:
  • inspector/uncaught-dom8-exception-expected.txt:
  • inspector/uncaught-dom8-exception.html:
19:31 Changeset [49896] by mrowe@apple.com

Fix leak of WebSocketChannel by adopting the newly-created reference.

Reviewed by Oliver Hunt.

  • websockets/WebSocketChannel.h:

(WebCore::WebSocketChannel::create):

17:42 Changeset [49895] by mrowe@apple.com

Roll out r49876 as it causes many assertion failures during regression tests on SnowLeopard.

17:40 Changeset [49894] by jorlow@chromium.org

DOM Storage runtime flag changes
https://bugs.webkit.org/show_bug.cgi?id=30602

Patch by Jeremy Orlow <jorlow@chromium.org> on 2009-10-20
Reviewed by Adam Barth.

Part 1/2. Removing sessionStorageEnabled in next patch after the
Chromium side of the changes lands.

Revert my changes to Settings and instead implement DOM Storage enabling via
the methods agreed upon in https://bugs.webkit.org/show_bug.cgi?id=30240

This stuff was (intentionally) never exposed to web pages or DRT, so there's no
LayoutTest visible changes and thus no tests.

  • bindings/v8/custom/V8CustomBinding.h:
  • bindings/v8/custom/V8DOMWindowCustom.cpp:

(WebCore::ACCESSOR_RUNTIME_ENABLER):

  • page/DOMWindow.idl:
  • storage/Storage.cpp:

(WebCore::Storage::setLocalStorageAvailable):
(WebCore::Storage::localStorageAvailable):
(WebCore::Storage::setSessionStorageAvailable):
(WebCore::Storage::sessionStorageAvailable):

  • storage/Storage.h:
17:22 Changeset [49893] by atwilson@chromium.org

Need to turn off notifications properly at runtime
https://bugs.webkit.org/show_bug.cgi?id=30409

Patch by John Gregg <johnnyg@google.com> on 2009-10-20
Reviewed by David Levin.

This code only affects chromium, and is all behind a compile time
flag current turned off, so no new tests.

  • bindings/v8/custom/V8CustomBinding.h:
  • bindings/v8/custom/V8DOMWindowCustom.cpp:

(WebCore::ACCESSOR_RUNTIME_ENABLER):

  • bindings/v8/custom/V8WorkerContextCustom.cpp:

(WebCore::ACCESSOR_RUNTIME_ENABLER):

  • notifications/NotificationCenter.cpp:

(WebCore::NotificationCenter::setIsAvailable):
(WebCore::NotificationCenter::isAvailable):

  • notifications/NotificationCenter.h:
  • page/DOMWindow.idl:
  • workers/WorkerContext.idl:
17:08 Changeset [49892] by eric@webkit.org

2009-10-20 James Robinson <jamesr@chromium.org>

Reviewed by Adam Barth.

Fixes RefPtr initialization in the V8 implementation of WebCore::ScriptString to use the ::create() idiom and
use adoptRef() properly. I failed to read the RefPtr docs the first time through :(

No new tests, error was caught by valgrind on the Chromium builders.

  • bindings/v8/ScriptString.h: (WebCore::ScriptString::ScriptString): (WebCore::ScriptString::operator=):
  • bindings/v8/ScriptStringImpl.h: (WebCore::ScriptStringImpl::create):
16:10 Changeset [49891] by eric@webkit.org

2009-10-20 Mikhail Naganov <mnaganov@chromium.org>

Reviewed by Pavel Feldman.

Web Inspector: populate child nodes before sorting them.

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

  • inspector/front-end/BottomUpProfileDataGridTree.js: (WebInspector.BottomUpProfileDataGridNode): Swapped with BottomUpProfileDataGridTree to be consistent with TopDownProfileDataGridNode. (WebInspector.BottomUpProfileDataGridNode.prototype._takePropertiesFromProfileDataGridNode): (WebInspector.BottomUpProfileDataGridNode.prototype._keepOnlyChild): (WebInspector.BottomUpProfileDataGridNode.prototype._exclude): (WebInspector.BottomUpProfileDataGridNode.prototype._merge): (WebInspector.BottomUpProfileDataGridNode.prototype._sharedPopulate): (WebInspector.BottomUpProfileDataGridTree.prototype.exclude):
  • inspector/front-end/ProfileDataGridTree.js: (WebInspector.ProfileDataGridNode.prototype.sort): Added missing parentheses. (WebInspector.ProfileDataGridNode.prototype.get _parent): (WebInspector.ProfileDataGridNode.prototype._populate):
  • inspector/front-end/TopDownProfileDataGridTree.js: (WebInspector.TopDownProfileDataGridNode.prototype._sharedPopulate):
14:56 Changeset [49890] by jorlow@chromium.org

Remove redundant String ref/deref calls in generated V8 bindings.
https://bugs.webkit.org/show_bug.cgi?id=30579

Patch by Jens Alfke <snej@chromium.org> on 2009-10-20
Reviewed by Eric Seidel.

  • bindings/v8/V8Binding.h:

(WebCore::toString): Fix return type of 'toString' to make it truly a no-op

instead of constructing/destructing a String.

14:56 Changeset [49889] by ggaren@apple.com

Windows build fix: updated variable name.

Patch by Geoffrey Garen <ggaren@apple.com> on 2009-10-20

  • runtime/DatePrototype.cpp:

(JSC::formatLocaleDate):

14:55 Changeset [49888] by ggaren@apple.com
  • jit/JITOpcodes.cpp:

(JSC::JIT::emit_op_next_pname): Slightly tweaked this #ifdef to match the
size of a JSValue because m_jsStrings is an array of JSValues.

Reviewed by Mark Rowe.

14:49 Changeset [49887] by ggaren@apple.com

Fixed a 64-bit regression caused by the fix for
https://bugs.webkit.org/show_bug.cgi?id=30570.

Patch by Geoffrey Garen <ggaren@apple.com> on 2009-10-20
Reviewed by Mark Rowe.

  • jit/JITOpcodes.cpp:

(JSC::JIT::emit_op_next_pname): Use TimesEight stepping on 64-bit, since
64-bit pointers are eight bytes long.

14:39 Changeset [49886] by ggaren@apple.com

Refactored DateInstance::msToGregorianDateTime so that a DateInstance's
caller doesn't need to supply the DateInstance's own internal value to
the DateInstance.

Patch by Geoffrey Garen <ggaren@apple.com> on 2009-10-20
Reviewed by Sam Weinig.

  • runtime/DateInstance.cpp:

(JSC::DateInstance::getGregorianDateTime): Renamed from "msToGregorianDateTime".

  • runtime/DateInstance.h:
  • runtime/DatePrototype.cpp:

(JSC::formatLocaleDate):
(JSC::dateProtoFuncToString):
(JSC::dateProtoFuncToUTCString):
(JSC::dateProtoFuncToISOString):
(JSC::dateProtoFuncToDateString):
(JSC::dateProtoFuncToTimeString):
(JSC::dateProtoFuncToLocaleString):
(JSC::dateProtoFuncToLocaleDateString):
(JSC::dateProtoFuncToLocaleTimeString):
(JSC::dateProtoFuncGetTime):
(JSC::dateProtoFuncGetFullYear):
(JSC::dateProtoFuncGetUTCFullYear):
(JSC::dateProtoFuncToGMTString):
(JSC::dateProtoFuncGetMonth):
(JSC::dateProtoFuncGetUTCMonth):
(JSC::dateProtoFuncGetDate):
(JSC::dateProtoFuncGetUTCDate):
(JSC::dateProtoFuncGetDay):
(JSC::dateProtoFuncGetUTCDay):
(JSC::dateProtoFuncGetHours):
(JSC::dateProtoFuncGetUTCHours):
(JSC::dateProtoFuncGetMinutes):
(JSC::dateProtoFuncGetUTCMinutes):
(JSC::dateProtoFuncGetSeconds):
(JSC::dateProtoFuncGetUTCSeconds):
(JSC::dateProtoFuncGetTimezoneOffset):
(JSC::setNewValueFromTimeArgs):
(JSC::setNewValueFromDateArgs):
(JSC::dateProtoFuncSetYear):
(JSC::dateProtoFuncGetYear): Also renamed "utc" to "outputIsUTC", for clarity.

14:20 Changeset [49885] by eric@webkit.org

2009-10-20 Joanmarie Diggs <joanmarie.diggs@gmail.com>

Reviewed by Xan Lopez.

https://bugs.webkit.org/show_bug.cgi?id=25411
[GTK] ATK accessible ancestry broken

Work around for the problem of bogus additional objects in the ancestry.
We now set the parent when we ref the child, then ask the Atk child if
it knows its parent. This solves the bulk of the cases. For those it
doesn't, fall back to the existing logic.

  • accessibility/gtk/AccessibilityObjectWrapperAtk.cpp: (webkit_accessible_get_parent): (webkit_accessible_ref_child):
13:56 Changeset [49884] by mitz@apple.com

Rubber-stamped by Mark Rowe.

<rdar://problem/7319749> REGRESSION(r49806): Contextual Menu items missing
https://bugs.webkit.org/show_bug.cgi?id=30568

Revert accidental change from r49806.

  • WebCoreSupport/WebContextMenuClient.mm:

(WebContextMenuClient::getCustomMenuFromDefaultItems):

13:50 Changeset [49883] by bweinstein@apple.com

Fixes <http://webkit.org/b/30421>.
Web Inpsector: Local Files can show up on Cookies List with Cookies from other Domain.

Reviewed by Alice Liu.

When a page like Google generates an about:blank, it shows up as a local file on
the list of Cookie domains, but has all of the cookies of Google. When we are adding
cookie domains to the inspector, we should only add the domain is the URL of the resource
is in the HTTP protocol family or a file URL.

  • inspector/InspectorController.cpp:

(WebCore::InspectorController::populateScriptObjects):
(WebCore::InspectorController::didFinishLoading):

13:38 Changeset [49882] by japhet@chromium.org

2009-10-20 Mark Mentovai <mark@chromium.org>

Reviewed by Darin Fisher.

Use a version of libWebKitSystemInterface with global symbols marked
private_extern for Chromium Mac.

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

  • WebCore.gyp/WebCore.gyp:
  • WebCore.gyp/mac: Added.
  • WebCore.gyp/mac/Empty.cpp: Added.
  • WebCore.gyp/mac/adjust_visibility.sh: Added.
13:12 Changeset [49881] by zoltan@webkit.org

2009-10-20 Zoltan Horvath <zoltan@webkit.org>

Reviewed by Eric Seidel.

Allow custom memory allocation control for WebCore's CSSProperty
https://bugs.webkit.org/show_bug.cgi?id=30564

Inherits CSSProperty class from FastAllocBase because it has been
instantiated by 'new' in WebCore/css/CSSParser.cpp:367.

  • css/CSSProperty.h:
13:03 Changeset [49880] by xan@webkit.org

2009-10-20 Xan Lopez <xlopez@igalia.com>

Another batch of skipped tests with links to their bugs.

  • platform/gtk/Skipped:
12:52 Changeset [49879] by kenneth@webkit.org

Reviewed By Adam Barth.

Patch by Kenneth Rohde Christiansen <kenneth@webkit.org> on 2009-10-20
Add some actions to the menu for cursor debugging.

GraphicsView based launcher only.

  • QGVLauncher/main.cpp:

(MainView::setWaitCursor):
(MainView::resetCursor):
(MainView::flip):
(MainWindow::setWaitCursor):
(MainWindow::resetCursor):
(MainWindow::buildUI):

12:08 Changeset [49878] by eric@webkit.org

2009-10-20 Andrew Scherkus <scherkus@chromium.org>

Reviewed by Eric Seidel.

Hide Chromium's media slider thumb if no source has been loaded.

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

Covered by existing layout tests, notably:

LayoutTests/media/controls-rendering.html
LayoutTests/media/unsupported-tracks.html
LayoutTests/media/video-src-none.html

  • rendering/RenderMediaControlsChromium.cpp: (WebCore::hasSource): Returns true if an HTMLMediaElement has a valid source set. (WebCore::paintMediaMuteButton): Refactored to use hasSource(). (WebCore::paintMediaPlayButton): Ditto. (WebCore::paintMediaSliderThumb): Add call to hasSource() to determine if we should paint the thumb.
12:02 Changeset [49877] by xan@webkit.org

2009-10-20 Xan Lopez <xlopez@igalia.com>

Another batch of skipped tests with links to their bugs.

  • platform/gtk/Skipped:
12:00 Changeset [49876] by eric@webkit.org

2009-10-20 Satoshi Nakagawa <psychs@limechat.net>

Reviewed by Darin Adler.

Fixed Japanese text search problems.
Treat small kana letters and kana letters as different characters in search.
Do not ignore diacritic marks in search for Japanese texts.

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

  • fast/text/international/japanese-kana-letters-expected.txt: Added.
  • fast/text/international/japanese-kana-letters.html: Added.

2009-10-20 Satoshi Nakagawa <psychs@limechat.net>

Reviewed by Darin Adler.

Fixed Japanese text search problems.
Treat small kana letters and kana letters as different characters in search.
Do not ignore diacritic marks in search for Japanese texts.

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

Test: fast/text/international/japanese-kana-letters.html

  • editing/TextIterator.cpp: (WebCore::createCollator): (WebCore::collator): (WebCore::createSearcher):
11:46 Changeset [49875] by hamaji@chromium.org

2009-10-20 Shinichiro Hamaji <hamaji@chromium.org>

Reviewed by Darin Adler.

fast/css/getComputedStyle/marginComputedStyle is missing
https://bugs.webkit.org/show_bug.cgi?id=30566

  • fast/css/getComputedStyle/marginComputedStyle-expected.txt: Added.
  • fast/css/getComputedStyle/marginComputedStyle.html: Added.
  • platform/mac/fast/css/getComputedStyle/marginComputedStyle-expected.checksum: Removed.
  • platform/mac/fast/css/getComputedStyle/marginComputedStyle-expected.png: Removed.
  • platform/mac/fast/css/getComputedStyle/marginComputedStyle-expected.txt: Removed.
  • platform/qt/fast/css/marginComputedStyle-expected.txt: Removed.
11:41 Changeset [49874] by eric@webkit.org

2009-10-20 Gabor Loki <loki@inf.u-szeged.hu>

Reviewed by Geoffrey Garen.

The op_next_pname should use 4 bytes addressing mode in case of JSValue32
https://bugs.webkit.org/show_bug.cgi?id=30570

  • jit/JITOpcodes.cpp: (JSC::JIT::emit_op_next_pname):
11:20 Changeset [49873] by xan@webkit.org

2009-10-20 Xan Lopez <xlopez@igalia.com>

Another batch of skipped tests with links to their bugs.

  • platform/gtk/Skipped:
11:17 Changeset [49872] by japhet@chromium.org

2009-10-20 Nate Chapin <japhet@chromium.org>

Reviewed by Adam Barth.

Added getter for FrameLoader:m_suppressOpenerInNewFrame.
This will allow Chromium to more intelligently detect
noreferrer links and therefore open them in a new process.

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

  • loader/FrameLoader.h: (WebCore::FrameLoader::suppressOpenerInNewFrame): Added.
10:46 Changeset [49871] by abarth@webkit.org

2009-10-20 Anton Muhin <antonm@chromium.org>

Reviewed by Adam Barth.

Add {ager,antonm,yurys}@chromium.org into committers.py
https://bugs.webkit.org/show_bug.cgi?id=30560

  • Scripts/modules/committers.py:
10:45 Changeset [49870] by kenneth@webkit.org

Remove clipRenderToViewport as agreed upon in
https://bugs.webkit.org/show_bug.cgi?id=29843

Patch by Kenneth Rohde Christiansen <kenneth@webkit.org> on 2009-10-20
Rubberstamped by Adam Barth.

  • Api/qwebframe.cpp:
  • Api/qwebframe.h:
  • Api/qwebframe_p.h:

(QWebFramePrivate::QWebFramePrivate):

10:37 Changeset [49869] by xan@webkit.org

2009-10-20 Xan Lopez <xlopez@igalia.com>

Skip 3 failing xmlhttprequest tests.

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

  • platform/gtk/Skipped:
10:22 Changeset [49868] by xan@webkit.org

2009-10-20 Xan Lopez <xlopez@igalia.com>

Skip two failing security canvas tests.

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

  • platform/gtk/Skipped:
10:10 Changeset [49867] by xan@webkit.org

2009-10-20 Xan Lopez <xlopez@igalia.com>

Skip failing editing test.

  • platform/gtk/Skipped:
10:03 Changeset [49866] by kenneth@webkit.org

Update the tests to test the new render functionality, and take
into consideration that render() clips to the frame itself as well
as the viewport.

Patch by Kenneth Rohde Christiansen <kenneth@webkit.org> on 2009-10-20
Reviewed by Adam Barth.

QWebFrame::render() now always clips, so the old tests were bogus.

Rendering pure contents (no scrollbars etc) without clipping can now
be accomplished using QWebFrame::documentElement()->render(...)

  • Api/qwebframe.cpp:
  • Api/qwebframe.h:
  • Api/qwebframe_p.h:

(QWebFramePrivate::QWebFramePrivate):

  • tests/qwebframe/tst_qwebframe.cpp:
10:03 Changeset [49865] by kenneth@webkit.org

As we do not support rendering a QWebFrame without it being clipped
the the frame as well as the viewport, we now set the viewport size
to the size of the contents.

Patch by Kenneth Rohde Christiansen <kenneth@webkit.org> on 2009-10-20
Reviewed by Adam Barth.

Rendering pure contents (no scrollbars etc) without clipping can be
acomplished using QWebFrame::documentElement()->render(...)

  • tests/qwebelement/tst_qwebelement.cpp:

(tst_QWebElement::render):

09:43 Changeset [49864] by xan@webkit.org

2009-10-20 Xan Lopez <xlopez@igalia.com>

Skip a bunch of failing DnD tests.

  • platform/gtk/Skipped:
09:40 Changeset [49863] by zoltan@webkit.org

2009-10-20 Zoltan Horvath <zoltan@webkit.org>

Reviewed by Darin Adler.

Allow custom memory allocation control for WebCore's CSSParserFunction
https://bugs.webkit.org/show_bug.cgi?id=30563

Inherits CSSParserFunction struct from FastAllocBase because it has been
instantiated by 'new' in WebCore/css/CSSParser.cpp:4827.

  • css/CSSProperty.h:
09:29 Changeset [49862] by xan@webkit.org

2009-10-20 Xan Lopez <xlopez@igalia.com>

Skip two zoom related tests.

  • platform/gtk/Skipped:
08:55 Changeset [49861] by xan@webkit.org

2009-10-20 Xan Lopez <xlopez@igalia.com>

Skip a couple of failing media tests.

  • platform/gtk/Skipped:
08:47 Changeset [49860] by xan@webkit.org

2009-10-20 Xan Lopez <xlopez@igalia.com>

Skip failing a11y test while we investigate what's going on.

  • platform/gtk/Skipped:
08:41 Changeset [49859] by xan@webkit.org

2009-10-20 Xan Lopez <xlopez@igalia.com>

Skip tests expecting new results.

  • platform/gtk/Skipped:
08:30 Changeset [49858] by joepeck@webkit.org

2009-10-20 Joseph Pecoraro <joepeck@webkit.org>

Reviewed by Timothy Hatcher.

Web Inspector: Watched Expressions Buttons Do Not Match Inspector Styles
https://bugs.webkit.org/show_bug.cgi?id=30554

  • inspector/front-end/inspector.css: added style to pane buttons
08:29 Changeset [49857] by xan@webkit.org

2009-10-20 Xan Lopez <xlopez@igalia.com>

Skip media/restore-from-page-cache.html since we are missing some
of the needed API for the test.

  • platform/gtk/Skipped:
08:06 Changeset [49856] by yurys@chromium.org

=fix for 30559

08:00 Changeset [49855] by xan@webkit.org

2009-10-20 Xan Lopez <xlopez@igalia.com>

Skip another failing test because we don't currently support
changing the locale after gtk_init has been called.

Also, add another test to the list of libxml2 issues.

  • platform/gtk/Skipped:
07:25 Changeset [49854] by ossy@webkit.org

2009-10-20 Csaba Osztrogonác <ossy@webkit.org>

Reviewed by Tor Arne Vestbø.

[Qt] Remove http/tests/xmlhttprequest tests
from skiplist, and add only now failing tests.

  • platform/qt/Skipped:
07:18 Changeset [49853] by xan@webkit.org

2009-10-20 Xan Lopez <xlopez@igalia.com>

Skip a bunch of tests failing in the debug bots because WebKit
does not work correctly with very recent versions of libxml2. The
tracker bug for the issue is
https://bugs.webkit.org/show_bug.cgi?id=30508

  • platform/gtk/Skipped:
07:15 Changeset [49852] by kenneth@webkit.org

Make the Netscape Test plugin available to the Qt launcher.

Patch by Kenneth Rohde Christiansen <kenneth@webkit.org> on 2009-10-20
Reviewed by Tor Arne Vestbø

  • Scripts/run-launcher:
07:14 Changeset [49851] by kenneth@webkit.org

Add menu item to dump the plugin list to the console,
which can be handy for debugging.

Patch by Kenneth Rohde Christiansen <kenneth@webkit.org> on 2009-10-20
Reviewed by Tor Arne Vestbø.

  • QtLauncher/main.cpp:

(MainWindow::dumpPlugins):
(MainWindow::setupUI):

07:14 Changeset [49850] by kenneth@webkit.org

Introduce new render method on QWebFrame, which supports specifying
which layers to render (scrollbars, contents, pan-icon).

Patch by Kenneth Rohde Christiansen <kenneth@webkit.org> on 2009-10-19
Reviewed by Tor Arne Vestbø.

Deprecate render().

  • Api/qwebframe.cpp:

(QWebFramePrivate::renderPrivate):
(QWebFrame::render):

  • Api/qwebframe.h:
  • Api/qwebframe_p.h:
05:16 Changeset [49849] by ossy@webkit.org

[Qt] Enable previously disabled fast/css tests after r48873 landed which
fixed https://bugs.webkit.org/show_bug.cgi?id=28288

Patch by Andras Becsi <becsi.andras@stud.u-szeged.hu> on 2009-10-20
Reviewed by Ariya Hidayat.

  • platform/qt/Skipped:
04:18 Changeset [49848] by xan@webkit.org

2009-10-20 Xan Lopez <xlopez@igalia.com>

Skip failing test, with comment pointing to the bug to track it.

  • platform/gtk/Skipped:
03:44 Changeset [49847] by xan@webkit.org

2009-10-20 Xan Lopez <xlopez@igalia.com>

Skip plugins/private-browsing-mode.html, since our plugin
implementation lacks that feature.

  • platform/gtk/Skipped:
02:54 Changeset [49846] by tonikitoo@webkit.org

[Qt] Infinite loop (leading to crash) when setting cursor in QGraphicsWebView
https://bugs.webkit.org/show_bug.cgi?id=30549

Patch by Antonio Gomes <tonikitoo@webkit.org> on 2009-10-19
Reviewed by Ariya Hidayat.

Patch reimplements QGraphicsItem's itemChange method, and make
CursorChange event to be emitted after cursor has already been
set.

QWidget::setCursor send the event just after it sets the cursor,
then patch makes both behaviors compatible.

  • Api/qgraphicswebview.cpp:

(QGraphicsWebView::itemChange):

  • Api/qgraphicswebview.h:
02:42 Changeset [49845] by ossy@webkit.org

Move OverridesMarkChildren flag from DatePrototype to its parent class
https://bugs.webkit.org/show_bug.cgi?id=30372

Patch by Gabor Loki <loki@inf.u-szeged.hu> on 2009-10-20
Reviewed by Oliver Hunt.

  • runtime/DateInstance.h:

(JSC::DateInstance::createStructure):

  • runtime/DatePrototype.h:
01:52 Changeset [49844] by ossy@webkit.org

Unreviewed buildfix.
[Qt] Add FEGaussianBlur.h and FEGaussianBlur.cpp which were refactored in r49778 to WebCore.pro.

Patch by Andras Becsi <becsi.andras@stud.u-szeged.hu> on 2009-10-20

  • WebCore.pro:
00:36 Changeset [49843] by ukai@chromium.org

2009-10-20 Fumitoshi Ukai <ukai@chromium.org>

Reviewed by David Levin.

Set EnabledAtRuntime for WebSocket in DOMWindow
https://bugs.webkit.org/show_bug.cgi?id=29896

Supported by chromium/v8 only.
Add WebSocket::isAvailable()/setIsAvailable(bool) to control v8
bindings.
Remove Settings::experimentalWebSocketsEnabled() and
setExperimentalWebSocketsEnabled(bool).

  • WebCore.base.exp:
  • bindings/js/JSDOMWindowCustom.cpp: (WebCore::JSDOMWindow::webSocket):
  • bindings/v8/V8DOMWrapper.cpp: (WebCore::V8DOMWrapper::getConstructor):
  • bindings/v8/custom/V8CustomBinding.h:
  • bindings/v8/custom/V8DOMWindowCustom.cpp: (WebCore::ACCESSOR_RUNTIME_ENABLER):
  • page/DOMWindow.idl:
  • page/Settings.cpp: (WebCore::Settings::Settings):
  • page/Settings.h:
  • websockets/WebSocket.cpp: (WebCore::WebSocket::setIsAvailable): (WebCore::WebSocket::isAvailable):
  • websockets/WebSocket.h:

2009-10-20 Fumitoshi Ukai <ukai@chromium.org>

Reviewed by David Levin.

Removed WebSocket runtime settings.
https://bugs.webkit.org/show_bug.cgi?id=29896

WebSocket runtime configuration is supported by chromium/v8 only.

  • webkit/webkitwebview.cpp: (webkit_web_view_update_settings): (webkit_web_view_settings_notify):

2009-10-20 Fumitoshi Ukai <ukai@chromium.org>

Reviewed by David Levin.

Removed WebSocket runtime settings.
https://bugs.webkit.org/show_bug.cgi?id=29896

WebSocket runtime configuration is supported by chromium/v8 only.

  • WebView/WebPreferenceKeysPrivate.h:
  • WebView/WebPreferences.mm: (+[WebPreferences initialize]):
  • WebView/WebPreferencesPrivate.h:
  • WebView/WebView.mm: (-[WebView _preferencesChangedNotification:]):

2009-10-20 Fumitoshi Ukai <ukai@chromium.org>

Reviewed by David Levin.

Removed WebSocket runtime settings.
https://bugs.webkit.org/show_bug.cgi?id=29896

WebSocket runtime configuration is supported by chromium/v8 only.

  • Interfaces/IWebPreferencesPrivate.idl:
  • WebPreferenceKeysPrivate.h:
  • WebPreferences.cpp: (WebPreferences::initializeDefaultSettings): (WebPreferences::experimentalNotificationsEnabled):
  • WebPreferences.h:
  • WebView.cpp: (WebView::notifyPreferencesChanged):

2009-10-20 Fumitoshi Ukai <ukai@chromium.org>

Reviewed by David Levin.

Removed WebSocket runtime settings.
https://bugs.webkit.org/show_bug.cgi?id=29896

WebSocket runtime configuration is supported by chromium/v8 only.

  • DumpRenderTree/mac/DumpRenderTree.mm: (resetDefaultsToConsistentValues):
  • DumpRenderTree/win/DumpRenderTree.cpp: (resetDefaultsToConsistentValues):
00:23 Changeset [49842] by ossy@webkit.org

Rubber-stamped by Tor Arne Vestbø.

Patch by Andras Becsi <becsi.andras@stud.u-szeged.hu> on 2009-10-20
[Qt] Skip failing tests introduced by disabling Phonon support in Buildbot's Qt version
until a proper solution for the Phonon related craches and timeouts is found.

  • platform/qt/Skipped:

10/19/09:

23:56 Changeset [49841] by oliver@apple.com

REGRESSION: Dromaeo DOM test is 14% slower
https://bugs.webkit.org/show_bug.cgi?id=30273

Reviewed by Sam Weinig.

Whoops, make prototype bindings actually use the StructureFlags.

22:01 Changeset [49840] by abarth@webkit.org

2009-10-19 James Robinson <jamesr@chromium.org>

Reviewed by Adam Barth.

Better implementation of WebCore::ScriptString for the V8 bindings.

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

WebCore::ScriptString is used for XMLHttpRequest's responseText attribute which is
shared with JavaScript. Thus, simply using a WebCore::String and copying the value
is pretty inefficient, especially since responseText is built up with a sequence of
operator+= calls. JSC builds use a JSC::UString to share the buffer when possible,
this patch adopts a similar approach for V8.

No new tests, behavior is unchanged and covered by LayoutTests/http/tests/xmlhttprequest

  • WebCore.gypi:
  • bindings/v8/ScriptString.h: (WebCore::ScriptString::ScriptString): (WebCore::ScriptString::operator String): (WebCore::ScriptString::isNull): (WebCore::ScriptString::size): (WebCore::ScriptString::operator=): (WebCore::ScriptString::operator+=): (WebCore::ScriptString::v8StringOrNull):
  • bindings/v8/ScriptStringImpl.cpp: Added. (WebCore::ScriptStringImpl::ScriptStringImpl): (WebCore::ScriptStringImpl::~ScriptStringImpl): (WebCore::ScriptStringImpl::toString): (WebCore::ScriptStringImpl::isNull): (WebCore::ScriptStringImpl::size): (WebCore::ScriptStringImpl::append):
  • bindings/v8/ScriptStringImpl.h: Added. (WebCore::ScriptStringImpl::ScriptStringImpl): (WebCore::ScriptStringImpl::v8StringHandle):
  • bindings/v8/custom/V8XMLHttpRequestCustom.cpp: (WebCore::ACCESSOR_GETTER):
21:46 Changeset [49839] by abarth@webkit.org

2009-10-19 Adam Barth <abarth@webkit.org>

No review, rolling out r49837.
http://trac.webkit.org/changeset/49837

  • page/Settings.cpp: (WebCore::Settings::setStandardFontFamily): (WebCore::Settings::setFixedFontFamily): (WebCore::Settings::setSerifFontFamily): (WebCore::Settings::setSansSerifFontFamily): (WebCore::Settings::setCursiveFontFamily): (WebCore::Settings::setFantasyFontFamily):
  • page/Settings.h: (WebCore::Settings::standardFontFamily): (WebCore::Settings::fixedFontFamily): (WebCore::Settings::serifFontFamily): (WebCore::Settings::sansSerifFontFamily): (WebCore::Settings::cursiveFontFamily): (WebCore::Settings::fantasyFontFamily):
  • platform/text/UScriptCode.h: Removed.
21:39 Changeset [49838] by abarth@webkit.org

2009-10-19 Kwang Yul Seo <skyul@company100.net>

Reviewed by Eric Seidel.

Check COMPILER(MSVC) instead of PLATFORM(WIN_OS) to avoid the use of vasprintf.
https://bugs.webkit.org/show_bug.cgi?id=30473

vasprintf is missing in MSVC. Use COMPILER(MSVC) guards instead of
PLALTFORM(WIN_OS) guards.

  • dom/XMLTokenizerLibxml2.cpp: (WebCore::XMLTokenizer::error):
21:37 Changeset [49837] by abarth@webkit.org

2009-10-19 Jungshik Shin <jshin@chromium.org>

Reviewed by Eric Seidel.

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

Make generic font family getters/setters accept an additional
argument (script code). It has a default value so that if an embedder
does not have/want a per-script font family setting, call-sites
don't have to be changed.
This is to prepare for fixing bug 10874 (font selection is not
language-dependent) and bug 18085.

There should be no change in layout and no new layout test
is added.

  • WebCore.base.exp:
  • page/Settings.cpp:
  • page/Settings.h:
  • platform/text/UScriptCode.h: Added. This is for ports that do not use ICU. the part of ICU's common/unicode/uscript.h that defines script code enum was copied. To keep enums compatible with those in ICU, we don't generate the list out of Scripts.txt of the Unicode Data base or CLDR's data.
21:28 Changeset [49836] by abarth@webkit.org

2009-10-19 Evan Stade <estade@chromium.org>

Reviewed by Darin Adler.

Clarify usage of SuddenTermination API. No code change.

  • platform/SuddenTermination.h:
21:05 Changeset [49835] by oliver@apple.com

REGRESSION: Dromaeo DOM test is 14% slower
https://bugs.webkit.org/show_bug.cgi?id=30273

Reviewed by Gavin Barraclough.

Make DOM bindings automatically inherit correct structure
flags rather than being needlessly conservative. This is
done by making the bindings generator use the same model
for TypeInfo flags that we now use in JSC.

This gains us about 1% of this regression back.

21:01 Changeset [49834] by abarth@webkit.org

2009-10-19 John Gregg <johnnyg@google.com>

Reviewed by Dimitri Glazkov.

set ENABLE_NOTIFICATIONS=1 for Chromium
https://bugs.webkit.org/show_bug.cgi?id=29949

  • chromium/features.gypi:
20:19 Changeset [49833] by eric@webkit.org

2009-10-19 Robin Qiu <robin.qiu@torchmobile.com.cn>

Reviewed by Nikolas Zimmermann.

Add a layout test for nested SVG <use> element which has a child.
https://bugs.webkit.org/show_bug.cgi?id=26117

  • svg/custom/use-on-use-with-child-expected.txt: Added.
  • svg/custom/use-on-use-with-child.svg: Added.

2009-10-19 Robin Qiu <robin.qiu@torchmobile.com.cn>

Reviewed by Nikolas Zimmermann.

Fixed a bug on nested SVG <use> elements.
https://bugs.webkit.org/show_bug.cgi?id=26117
When a <use> element refer to another <use> element which has
child/children, the instance tree built for this <use> element
is incorrect (more nodes than expected).

Test: svg/dom/use-on-use-with-child.svg

  • svg/SVGUseElement.cpp: (WebCore::SVGUseElement::buildInstanceTree):
19:59 Changeset [49832] by timothy@apple.com

Small changes to fully invalidate and update the JavaScriptCallFrame.

Patch by Oliver Hunt <oliver@apple.com> on 2009-10-19
<rdar://problem/7020755> JSDebugger crashes after reloading from a breakpoint
https://bugs.webkit.org/show_bug.cgi?id=27146

Reviewed by Timothy Hatcher.

  • inspector/JavaScriptCallFrame.h:

(WebCore::JavaScriptCallFrame::invalidate):
(WebCore::JavaScriptCallFrame::update):

19:51 Changeset [49831] by jorlow@chromium.org

2009-10-19 Jeremy Orlow <jorlow@chromium.org>

Reviewed by Adam Barth.

Fix fallout from last change.
https://bugs.webkit.org/show_bug.cgi?id=30550

Update init-events to match reality. I only ran the domstorage layout tests
and thus missed one in fast/events/. This addresses fallout from
https://bugs.webkit.org/show_bug.cgi?id=30536

  • fast/events/init-events-expected.txt:
  • fast/events/script-tests/init-events.js:
18:32 Changeset [49830] by jorlow@chromium.org

2009-10-19 Jeremy Orlow <jorlow@chromium.org>

Reviewed by Dmitri Titov.

Remove "source" from storage events
https://bugs.webkit.org/show_bug.cgi?id=30536

Remove "source" from storage events per
http://lists.whatwg.org/htdig.cgi/whatwg-whatwg.org/2009-October/023703.html

This was removed because it makes it introduces synchronous access that can
cross the event loop boundry (since a storage event can fire from one process
and be handled in another).

  • inspector/InspectorDOMStorageResource.cpp: (WebCore::InspectorDOMStorageResource::handleEvent):
  • storage/StorageEvent.cpp: (WebCore::StorageEvent::create): (WebCore::StorageEvent::StorageEvent): (WebCore::StorageEvent::initStorageEvent):
  • storage/StorageEvent.h: (WebCore::StorageEvent::uri):
  • storage/StorageEvent.idl:
  • storage/StorageEventDispatcher.cpp: (WebCore::StorageEventDispatcher::dispatch):

2009-10-19 Jeremy Orlow <jorlow@chromium.org>

Reviewed by Dmitri Titov.

Remove "source" from storage events
https://bugs.webkit.org/show_bug.cgi?id=30536

Remove "source" from storage events per
http://lists.whatwg.org/htdig.cgi/whatwg-whatwg.org/2009-October/023703.html

  • storage/domstorage/localstorage/iframe-events-expected.txt:
  • storage/domstorage/localstorage/iframe-events.html:
  • storage/domstorage/localstorage/resources/iframe-events-second.html:
  • storage/domstorage/localstorage/simple-events-expected.txt:
  • storage/domstorage/localstorage/simple-events.html:
  • storage/domstorage/sessionstorage/iframe-events-expected.txt:
  • storage/domstorage/sessionstorage/iframe-events.html:
  • storage/domstorage/sessionstorage/resources/iframe-events-second.html:
  • storage/domstorage/sessionstorage/simple-events-expected.txt:
  • storage/domstorage/sessionstorage/simple-events.html:
18:29 Changeset [49829] by abarth@webkit.org

2009-10-19 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.

Remove ResourceLoadNotifier::activeDocumentLoader
https://bugs.webkit.org/show_bug.cgi?id=30533

Removing this method from ResourceLoadNotifier better decouples the
notifier from FrameLoader.

  • loader/FrameLoader.cpp: (WebCore::FrameLoader::continueLoadAfterWillSubmitForm): (WebCore::FrameLoader::requestFromDelegate):
  • loader/ResourceLoadNotifier.cpp: (WebCore::ResourceLoadNotifier::didReceiveResponse):
  • loader/ResourceLoadNotifier.h:
  • loader/ResourceLoader.cpp: (WebCore::ResourceLoader::willSendRequest):
18:28 Changeset [49828] by abarth@webkit.org

2009-10-19 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.

Move sendRemainingDelegateMessages to ResourceLoadNotifier
https://bugs.webkit.org/show_bug.cgi?id=30531

This method is about notifying folks and doesn't interact with the rest
of FrameLoader.

  • loader/FrameLoader.cpp: (WebCore::FrameLoader::willLoadMediaElementURL): (WebCore::FrameLoader::commitProvisionalLoad): (WebCore::FrameLoader::loadResourceSynchronously): (WebCore::FrameLoader::loadedResourceFromMemoryCache):
  • loader/FrameLoader.h:
  • loader/ResourceLoadNotifier.cpp: (WebCore::ResourceLoadNotifier::sendRemainingDelegateMessages):
  • loader/ResourceLoadNotifier.h:
18:19 Changeset [49827] by abarth@webkit.org

2009-10-19 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.

Bypass popup blocker using click event
https://bugs.webkit.org/show_bug.cgi?id=21501

Test that a fake event can't get around the popup blocker.

  • http/tests/security/popup-blocked-from-fake-event-expected.txt: Added.
  • http/tests/security/popup-blocked-from-fake-event.html: Added.

2009-10-19 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.

Bypass popup blocker using click event
https://bugs.webkit.org/show_bug.cgi?id=21501

Keep track of which events were generated by JavaScript and use that
inforation when figuring out if we're processing a user gesture.

Test: http/tests/security/popup-blocked-from-fake-event.html

  • bindings/js/ScriptController.cpp: (WebCore::ScriptController::processingUserGestureEvent):
  • bindings/v8/ScriptController.cpp: (WebCore::ScriptController::processingUserGesture):
  • dom/Document.cpp: (WebCore::Document::createEvent):
  • dom/Event.cpp: (WebCore::Event::Event):
  • dom/Event.h: (WebCore::Event::createdByDOM): (WebCore::Event::setCreatedByDOM):
18:09 Changeset [49826] by dumi@chromium.org

Fixing a typo.

Patch by Dumitru Daniliuc <dumi@chromium.org> on 2009-10-19
Reviewed by Darin Adler.

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

  • platform/sql/SQLiteDatabase.h:
18:05 Changeset [49825] by dumi@chromium.org

Adding myself to committers.py.

17:41 Changeset [49824] by dimich@chromium.org

Manual Test for crash caused by JS accessing DOMWindow which is disconnected from the Frame.
https://bugs.webkit.org/show_bug.cgi?id=30544

Reviewed by Darin Adler.

  • manual-tests/crash-on-accessing-domwindow-without-frame.html: Added.
17:38 Changeset [49823] by simon.fraser@apple.com

2009-10-19 Simon Fraser <simon.fraser@apple.com>

Reviewed by Darin Adler.

Flash at end of opacity/transform transition sometimes
https://bugs.webkit.org/show_bug.cgi?id=30501

When a transition finishes, there window of time between when the animation is
removed, and the final style set on the GraphicsLayer. This caused the layer to revert
to its old appearance for one or two frames. To avoid this, we set the final
transform or opacity at the start of the transition; we know that the animation
will override the final value for as long as its running.

No test because this is a very transient effect that can't be captured
in a test.

  • rendering/RenderLayerBacking.cpp: (WebCore::RenderLayerBacking::createGraphicsLayer): (WebCore::RenderLayerBacking::updateLayerOpacity): (WebCore::RenderLayerBacking::updateLayerTransform): (WebCore::RenderLayerBacking::updateGraphicsLayerGeometry): (WebCore::RenderLayerBacking::startTransition):
  • rendering/RenderLayerBacking.h:
17:12 Changeset [49822] by cmarrin@apple.com

Added one more style rule

17:03 Changeset [49821] by cmarrin@apple.com

Moved WebGL blog CSS to a separate file

15:59 Changeset [49820] by ggaren@apple.com

Tightened up some put_by_id_transition code generation.
https://bugs.webkit.org/show_bug.cgi?id=30539

Patch by Geoffrey Garen <ggaren@apple.com> on 2009-10-19
Reviewed by Oliver Hunt.

  • jit/JIT.h:
  • jit/JITPropertyAccess.cpp:

(JSC::JIT::testPrototype):
(JSC::JIT::privateCompilePutByIdTransition): No need to do object type
checks or read Structures and prototypes from objects: they're all known
constants at compile time.

15:44 Changeset [49819] by eric@webkit.org

2009-10-19 Jason Yan <tailofthesun@gmail.com>

Reviewed by Eric Seidel.

Fixed issue with LayoutTests/fast/cookies/local-file-can-set-cookies.html
which can fail when the system under test has local file cookies set already.
https://bugs.webkit.org/show_bug.cgi?id=30281

  • fast/cookies/local-file-can-set-cookies-expected.txt: Modified. Updated expected results.
  • fast/cookies/local-file-can-set-cookies.html: Modified. Check for existing cookie and set non-persistent cookie.
15:36 Changeset [49818] by eric@webkit.org

2009-10-19 Shu Chang <Chang.Shu@nokia.com>

Reviewed by Eric Seidel.

Language tag should include both language and country sub-tags,
case insensitive.
https://bugs.webkit.org/show_bug.cgi?id=30440

  • fast/js/navigator-language-expected.txt:
  • fast/js/navigator-language.html:
15:28 Changeset [49817] by eric@webkit.org

2009-10-19 Marshall Culpepper <mculpepper@appcelerator.com>

Reviewed by Eric Seidel.

added cairo include and lib directories to debug_wincairo.vsprops
https://bugs.webkit.org/show_bug.cgi?id=29831

  • win/tools/vsprops/debug_wincairo.vsprops:
15:10 Changeset [49816] by eric@webkit.org

2009-10-19 Andrew Scherkus <scherkus@chromium.org>

Reviewed by Eric Seidel.

Replace boolean operator indentation example with an if statement.

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

  • coding/coding-style.html: Replace boolean indenting example to use "if"
15:00 Changeset [49815] by pfeldman@chromium.org

2009-10-17 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Timothy Hatcher.

Web Inspector: Add more stop characters into console completions.

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

  • inspector/front-end/ConsoleView.js: (WebInspector.ConsoleView.prototype.completions): (WebInspector.ConsoleView.prototype._reportCompletions):
15:00 Changeset [49814] by aroben@apple.com

Fix crashes/assertions when calling WebLocalizedString from multiple threads concurrently

Fixes <http://webkit.org/b/30534> WebLocalizedString asserts if called
from multiple threads concurrently

Reviewed by John Sullivan.

  • WebLocalizableStrings.cpp:

(mainBundleLocStringsMutex):
(frameworkLocStringsMutex):
Added these new getters.

(findCachedString):
(cacheString):
Lock the relevant mutex before accessing each string map. Otherwise
bad things could happen if two threads end up here at the same time.

15:00 Changeset [49813] by aroben@apple.com

Get rid of a few static initializers/exit-time destructors in WebLocalizableStrings

Reviewed by Dan Bernstein.

  • WebLocalizableStrings.cpp:

(mainBundleLocStrings):
(frameworkLocStrings):
Added these functions instead of file-scope variables.

(findCachedString):
(cacheString):
Updated to use the new functions.

14:41 Changeset [49812] by japhet@chromium.org

2009-10-19 Nate Chapin <japhet@chromium.org>

Unreviewed, build fix.

Update call to FrameLoader::loadFrameRequest() in qt and
FrameLoader::urlSelected() in win.

14:39 Changeset [49811] by dbates@webkit.org

2009-10-19 Daniel Bates <dbates@webkit.org>

Reviewed by Eric Seidel.

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


Added the test case for bug #30266 to the Windows skip list because
event.dataTransfer.types does not behave correctly. See bug #30527.

  • platform/win/Skipped: Added test case to Windows skipped list.
14:31 Changeset [49810] by eric@webkit.org

2009-10-19 Marshall Culpepper <mculpepper@appcelerator.com>

Reviewed by Adam Roben.

implemented ClipboardWin::files()
https://bugs.webkit.org/show_bug.cgi?id=29666

  • platform/win/ClipboardWin.cpp: (WebCore::ClipboardWin::files):
14:15 Changeset [49809] by japhet@chromium.org

2009-10-19 Nate Chapin <japhet@chromium.org>

Reviewed by Alexey Proskuryakov.

Add support for link relations in HTMLAnchorElement and implement rel="noreferrer".

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

14:08 Changeset [49808] by ddkilzer@apple.com

Documentation fix for -[NSObject(WebResourceLoadDelegate) webView:resource:willSendRequest:redirectResponse:fromDataSource:]

Reviewed by Dan Bernstein.

  • WebView/WebResourceLoadDelegate.h: Prepended "webView:" to the

@method declaration of
-webView:resource:willSendRequest:redirectResponse:fromDataSource:
which was absent before r3953 and was missed during the great
documentation clean up of r4269.

14:01 Changeset [49807] by andersca@apple.com

Fix build.

  • Plugins/WebNetscapePluginView.mm:

(-[WebNetscapePluginView saveAndSetNewPortStateForUpdate:]):

13:30 Changeset [49806] by mitz@apple.com

Split assertions of conjunctions into separate statements or remove
redundant assertions.

Reviewed by Anders Carlsson.

  • Plugins/Hosted/NetscapePluginHostProxy.mm:

(WebKit::NetscapePluginHostProxy::deadNameNotificationCallback):

  • Plugins/Hosted/WebHostedNetscapePluginView.mm:

(-[WebHostedNetscapePluginView setAttributeKeys:andValues:]):
(-[WebHostedNetscapePluginView _webPluginContainerCancelCheckIfAllowedToLoadRequest:]):

  • Plugins/WebBaseNetscapePluginView.mm:

(-[WebBaseNetscapePluginView halt]):
(-[WebBaseNetscapePluginView resumeFromHalt]):

  • Plugins/WebNetscapePluginView.mm:

(-[WebNetscapePluginView saveAndSetNewPortStateForUpdate:]):
(-[WebNetscapePluginView _webPluginContainerCancelCheckIfAllowedToLoadRequest:]):

  • WebCoreSupport/WebContextMenuClient.mm:

(WebContextMenuClient::getCustomMenuFromDefaultItems):

  • WebCoreSupport/WebEditorClient.mm:

(WebEditorClient::checkGrammarOfString):
(WebEditorClient::checkTextOfParagraph):

  • WebView/WebTextCompletionController.mm:

(-[WebTextCompletionController _reflectSelection]):

13:25 Changeset [49805] by japhet@chromium.org

2009-10-19 Nate Chapin <japhet@chromium.org>

Reviewed by Geoffrey Garen.

Add plugins/netscape-invoke-failures.html to Snow Leopard skipped list.

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

  • platform/mac-snowleopard/Skipped: Add plugins/netscape-invoke-failures.html.
13:25 Changeset [49804] by kenneth@webkit.org

Refactor out some of the code in paint() into paintPanScrollIcon()
and paintScrollbars() in preparation of some Qt API changes.

Patch by Kenneth Rohde Christiansen <kenneth@webkit.org> on 2009-10-19
Reviewed by David Hyatt.

  • platform/ScrollView.cpp:

(WebCore::ScrollView::paint):

  • platform/ScrollView.h:
13:18 Changeset [49803] by hclam@chromium.org

[v8] typeof(HTMLMediaElement) should return undefined if media
engine is not available
https://bugs.webkit.org/show_bug.cgi?id=30343

Patch by Alpha Lam <hclam@chromium.org> on 2009-10-19
Reviewed by Dimitri Glazkov (dglazkov@chromium.org).

Check for availability of the media engine to disable
HTMLMediaElement, HTMLAudioElement, HTMLVideoElement, MediaError
in runtime.

Try runs and review in Chromium:
http://codereview.chromium.org/276011

Test: manual-tests/chromium/media-player-not-available.html

The above test can work work in Chromium. Testing procedures:

  1. Remove all media support libraries in Chromium (e.g. ffmpeg libraries)
  2. Open Chromium with above test page
  3. Verify the test results according to the test page
  • bindings/v8/custom/V8CustomBinding.h:

Declare enabler methods.

  • bindings/v8/custom/V8DOMWindowCustom.cpp:

Report enabled only if media player is available.

  • manual-tests/chromium/media-player-not-available.html: Added.

Manual test, following instruction in this test for procedures.

  • page/DOMWindow.idl:

Mark HTMLMediaElement, HTMLAudioElement, HTMLVideElement, MediaError
as enabled at runtime.

12:59 Changeset [49802] by ggaren@apple.com

Added a private API for getting a global context from a context, for
clients who want to preserve a context for a later callback.

Patch by Geoffrey Garen <ggaren@apple.com> on 2009-10-19
Reviewed by Sam Weinig.

  • API/APICast.h:

(toGlobalRef): Added an ASSERT, since this function is used more often
than before.

  • API/JSContextRef.cpp:
  • API/JSContextRefPrivate.h: Added. The new API.
  • API/tests/testapi.c:

(print_callAsFunction):
(main): Test the new API.

12:38 Changeset [49801] by jmalonzo@webkit.org

2009-10-19 Jan Michael Alonzo <jmalonzo@webkit.org>

Reviewed by Alexey Proskuryakov.

[Gtk] ApplicationCacheStorage error while compiling WebKit/gtk/webkit/webkitapplicationcache.cpp
https://bugs.webkit.org/show_bug.cgi?id=30358

Guard calls to cacheStorage() with ENABLE(OFFLINE_WEB_APPLICATIONS).

  • webkit/webkitapplicationcache.cpp:
12:24 Changeset [49800] by kenneth@webkit.org

Make the local static panScrollIcon into a Image* instead
of a RefPtr<Image>.

Patch by Kenneth Rohde Christiansen <kenneth@webkit.org> on 2009-10-19
Reviewed by Adam Roben.

  • platform/ScrollView.cpp:

(WebCore::ScrollView::wheelEvent):

12:19 Changeset [49799] by ajwong@chromium.org

Forcefully set antialiasing for Chromium media controls.

Patch by Andrew Scherkus <scherkus@chromium.org> on 2009-10-19
Reviewed by NOBODY (OOPS!).

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

Before it was nondeterministiacally being enabled based on the previous state of GraphicsContext.

Covered by existing layout tests.

  • rendering/RenderMediaControlsChromium.cpp:

(WebCore::paintMediaSlider): Added setShouldAntialias(true).

12:11 Changeset [49798] by eric@webkit.org

2009-10-19 Jens Alfke <jens@mooseyard.com>

Reviewed by Darin Adler.

Optimize string upper/lowercasing
https://bugs.webkit.org/show_bug.cgi?id=30261

  • Added AtomicString::upper() and lower()
  • Further optimized StringImpl::lower()
  • Removed StringImpl::isLower()
  • Added QualifiedName::localNameUpper(), which is cached, thereby saving thousands of upper() calls and string allocations.
  • dom/Element.cpp: (WebCore::Element::setAttribute): Call AtomicString::lower()
  • dom/QualifiedName.cpp: (WebCore::QualifiedName::localNameUpper): New method
  • dom/QualifiedName.h: Added localNameUpper() method
  • dom/StyledElement.cpp: (WebCore::StyledElement::parseMappedAttribute): Call AtomicString::lower()
  • html/HTMLDocument.cpp: (WebCore::HTMLDocument::createElement): Call AtomicString::lower()
  • html/HTMLElement.cpp: (WebCore::HTMLElement::nodeName): Call localNameUpper()
  • platform/text/AtomicString.cpp: (WebCore::AtomicString::lower): New method (WebCore::AtomicString::upper): New method
  • platform/text/AtomicString.h: Added lower() and upper()
  • platform/text/StringImpl.cpp: Removed isLower() (WebCore::StringImpl::lower): Further optimization of initial loop
  • platform/text/StringImpl.h: Removed isLower()
11:59 Changeset [49797] by eric@webkit.org

2009-10-19 Zan Dobersek <zandobersek@gmail.com>

Reviewed by Gustavo Noronha.

Update test results and Skipped list after changes in r49791.

Some test results were accidentally committed in r49192. At that time
results were incorrect, but testing still succeeded because of
unimplemented features. After r49791 tests, though generating semantically
equal output to Mac's render trees, started to fail and need to have
generated results updated. This patch does just that.

  • platform/gtk/Skipped: Skip a test that does not generate semantically equal render trees.
  • platform/gtk/editing/inserting/insert-3907422-fix-expected.txt:
  • platform/gtk/editing/pasteboard/4076267-2-expected.txt:
  • platform/gtk/editing/pasteboard/4076267-3-expected.txt:
  • platform/gtk/editing/pasteboard/4076267-expected.txt:
  • platform/gtk/editing/pasteboard/paste-table-001-expected.txt:
  • platform/gtk/editing/pasteboard/paste-text-001-expected.txt:
  • platform/gtk/editing/pasteboard/paste-text-002-expected.txt:
  • platform/gtk/editing/pasteboard/paste-text-003-expected.txt:
  • platform/gtk/editing/pasteboard/paste-text-004-expected.txt:
  • platform/gtk/editing/pasteboard/paste-text-005-expected.txt:
  • platform/gtk/editing/pasteboard/paste-text-006-expected.txt:
  • platform/gtk/editing/pasteboard/paste-text-007-expected.txt:
  • platform/gtk/editing/pasteboard/paste-text-008-expected.txt:
  • platform/gtk/editing/pasteboard/paste-text-009-expected.txt:
  • platform/gtk/editing/pasteboard/paste-text-010-expected.txt:
  • platform/gtk/editing/pasteboard/paste-text-015-expected.txt:
  • platform/gtk/editing/pasteboard/paste-xml-expected.txt:
  • platform/gtk/editing/pasteboard/select-element-1-expected.txt: Removed.
11:50 Changeset [49796] by eric@webkit.org

2009-10-19 Nate Chapin <japhet@chromium.org>

Reviewed by Adam Barth.

Test that the proper exception is thrown into JS when a plugin
invoke call returns false.

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

  • plugins/netscape-invoke-failure-expected.txt: Added.
  • plugins/netscape-invoke-failure.html: Added.

2009-10-19 Nate Chapin <japhet@chromium.org>

Reviewed by Adam Barth.

If a call to a plugin's invokeMethod, invokeDefault or construct
returns false, throw an exception into JS.

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

Test: plugins/netscape-invoke-failure.html

  • bindings/v8/V8NPObject.cpp: Check return values of invokeMethod, invokeDefault and construct. (npObjectInvokeImpl):
  • bridge/c/c_instance.cpp: (JSC::Bindings::CInstance::invokeMethod): Check return value (JSC::Bindings::CInstance::invokeDefaultMethod): Check return value (JSC::Bindings::CInstance::invokeConstruct): Check return value

2009-10-19 Nate Chapin <japhet@chromium.org>

Reviewed by Adam Barth.

Add testFail() to test plugin so we can test our handling of a
plugin invoke call returning false.

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

  • DumpRenderTree/TestNetscapePlugIn.subproj/PluginObject.cpp: Add testFail(). (pluginInvoke): (testIdentifierToString): Always return true, since returning false will now cause an exception to be thrown.
11:47 Changeset [49795] by ggaren@apple.com

Tightened up some instanceof code generation.
https://bugs.webkit.org/show_bug.cgi?id=30488

Patch by Geoffrey Garen <ggaren@apple.com> on 2009-10-17
Reviewed by Oliver Hunt.

  • jit/JITOpcodes.cpp:

(JSC::JIT::emit_op_instanceof):
(JSC::JIT::emitSlow_op_instanceof): No need to do object type checks -
cell type checks and ImplementsDefaultHasIntance checks implicitly
supersede object type checks.

11:38 Changeset [49794] by senorblanco@chromium.org

WebCore: Fix for chromium/skia canvas arc start/end positions, when the arc is

360 degrees.

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

Reviewed by Adam Barth.

Covered by new test:
LayoutTests/fast/canvas/arc360.html

  • platform/graphics/skia/PathSkia.cpp:

(WebCore::Path::addArc):

LayoutTests: New test for 360 degree arc endpoints, and expected results.
http://bugs.webkit.org/show_bug.cgi?id=30449

Reviewed by Adam Barth.

  • fast/canvas/arc360.html: Added.
  • platform/mac/fast/canvas/arc360-expected.checksum: Added.
  • platform/mac/fast/canvas/arc360-expected.png: Added.
  • platform/mac/fast/canvas/arc360-expected.txt: Added.
11:14 Changeset [49793] by bweinstein@apple.com

Rubber-stamped by Adam Roben.

Add tests that have never passed on Windows to the Skipped list, and reopen
bugs.

  • platform/win/Skipped:
10:35 Changeset [49792] by eric@webkit.org

2009-10-19 Kent Tamura <tkent@chromium.org>

Reviewed by Alexey Proskuryakov.

Fix uninitialized variable reference in Element::removeAttribute().
https://bugs.webkit.org/show_bug.cgi?id=30502

  • dom/Element.cpp: (WebCore::Element::removeAttribute):
10:25 Changeset [49791] by eric@webkit.org

2009-10-19 Zan Dobersek <zandobersek@gmail.com>

Reviewed by Gustavo Noronha.

Enable tests that pass when DOM pasting is allowed.

  • platform/gtk/Skipped:

2009-10-19 Zan Dobersek <zandobersek@gmail.com>

Reviewed by Gustavo Noronha.

Add a settings to allow DOM pastes to be performed.

  • webkit/webkitwebsettings.cpp: (webkit_web_settings_class_init): (webkit_web_settings_set_property): (webkit_web_settings_get_property): (webkit_web_settings_copy):
  • webkit/webkitwebview.cpp: (webkit_web_view_update_settings): (webkit_web_view_settings_notify):

2009-10-19 Zan Dobersek <zandobersek@gmail.com>

Reviewed by Gustavo Noronha.

Enable DOM pasting when running layout tests.

  • DumpRenderTree/gtk/DumpRenderTree.cpp: (resetDefaultsToConsistentValues):
10:13 Changeset [49790] by eric.carlson@apple.com

2009-10-19 Eric Carlson <eric.carlson@apple.com>

Reviewed by Simon Fraser.

https://bugs.webkit.org/show_bug.cgi?id=30463
Remove HTML5 media element 'loadend' event

  • dom/EventNames.h:
  • html/HTMLMediaElement.cpp: (WebCore::HTMLMediaElement::parseMappedAttribute): Remove 'loadend'. (WebCore::HTMLMediaElement::loadInternal): Ditto. (WebCore::HTMLMediaElement::noneSupported): Ditto. (WebCore::HTMLMediaElement::mediaEngineError): Ditto. (WebCore::HTMLMediaElement::setNetworkState): Ditto. (WebCore::HTMLMediaElement::userCancelledLoad): Ditto.

2009-10-19 Eric Carlson <eric.carlson@apple.com>

Reviewed by Simon Fraser.

https://bugs.webkit.org/show_bug.cgi?id=30463
Remove HTML5 media element 'loadend' event

  • media/event-attributes-expected.txt:
  • media/event-attributes.html: Switch from 'loadend' to 'canplaythrough'.
  • media/media-load-event-expected.txt:
  • media/media-load-event.html: Ditto.
  • media/progress-event-at-least-one-expected.txt:
  • media/progress-event-at-least-one.html: Ditto.
09:26 Changeset [49789] by eric@webkit.org

2009-10-19 Jocelyn Turcotte <jocelyn.turcotte@nokia.com>

Reviewed by Adam Barth.

[Qt] Remove deletion of outgoing FormData object when reply is finished.
The buffers will be destroyed a bit later, when the QNetworkReply is deleted.
Bug: https://bugs.webkit.org/show_bug.cgi?id=29551

  • platform/network/qt/QNetworkReplyHandler.cpp:
  • platform/network/qt/QNetworkReplyHandler.h:
09:18 Changeset [49788] by eric@webkit.org

2009-10-19 Dimitri Glazkov <dglazkov@chromium.org>

Reviewed by Darin Adler.

Fix hard-to-reproduce crash in HTMLTokenizer by avoiding a rare
fastRealloc edge case.
https://bugs.webkit.org/show_bug.cgi?id=29313

No test, the crash shows up occasionally in crash dumps, we weren't able
to reproduce it locally.

  • html/HTMLTokenizer.cpp: (WebCore::HTMLTokenizer::enlargeScriptBuffer): Added an early exit to

avoid calling fastRealloc with the size of 0.

09:08 Changeset [49787] by eric@webkit.org

2009-10-19 Andrew Scherkus <scherkus@chromium.org>

Reviewed by Eric Seidel.

Use fillRect() instead of drawRect() to fix Chromium media controls rendering.

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

Chromium Mac layout tests will need to be rebaselined. Existing layout tests for Linux/Win still pass.

  • rendering/RenderMediaControlsChromium.cpp: (WebCore::paintMediaSlider): Use fillRect() instead of drawRect().
09:00 Changeset [49786] by eric@webkit.org

2009-10-19 Dmitry Titov <dimich@chromium.org>

Reviewed by Adam Barth.

Crash in DOMWindow::clearTimeout etc when DOMWindow is not connected to a frame.
https://bugs.webkit.org/show_bug.cgi?id=29832

Need to make sure the script caches are reset when frame gets disconnected from still-alive DOMWindow.
This will prevent JS from calling DOMWindow methods that can not be completed w/o the frame.

I am not sure it's possible to test this since the only file that repros the problem need ~10 seconds to cause crash.

  • page/Frame.cpp: (WebCore::Frame::~Frame): Right after frame disconnects from DOMWindow, clear WindowShell.
08:53 Changeset [49785] by laszlo.1.gombos@nokia.com

2009-10-19 Viatcheslav Ostapenko <ostapenko.viatcheslav@nokia.com>

Reviewed by Ariya Hidayat.

Add QWebElement::render API which allows rendering of single
element.

  • Api/qwebelement.cpp: (QWebElement::render):
  • Api/qwebelement.h:
  • tests/qwebelement/tst_qwebelement.cpp: (tst_QWebElement::render):
  • tests/qwebelement/qwebelement.qrc:
  • tests/qwebelement/image.png: Added.
08:16 Changeset [49784] by ariya@webkit.org

QWebPage: Doc: setNetworkAccessManager should only be called once.

Patch by Markus Goetz <Markus.Goetz@nokia.com> on 2009-10-19
Reviewed by Ariya Hidayat.

08:02 Changeset [49783] by ossy@webkit.org

2009-10-19 Csaba Osztrogonác <ossy@webkit.org>

Unreviewed build warning fix.
[Qt] Apply changes to WebCore.pro introduced in r49778

  • WebCore.pro: Removed svg/graphics/filters/SVGFEGaussianBlur.h from HEADERS
08:01 Changeset [49782] by tonikitoo@webkit.org

Wrong ifdef combination in QGraphicsWebView's event method.

Patch by Antonio Gomes <tonikitoo@webkit.org> on 2009-10-19
Reviewed by Tor Arne.

  • Api/qgraphicswebview.cpp:

(QGraphicsWebView::event):

07:42 Changeset [49781] by ossy@webkit.org

[Qt] Skip failing tests for now and add test-specific information about the cause to the Skipped list.

Patch by Andras Becsi <becsi.andras@stud.u-szeged.hu> on 2009-10-19
Reviewed by Tor Arne Vestbø.

  • platform/qt/Skipped:
07:16 Changeset [49780] by kov@webkit.org

Make WebKitHitTestResult actually appear in the index

07:00 Changeset [49779] by ossy@webkit.org

[Qt] Add platform expected file for canvas test.

Patch by Andras Becsi <becsi.andras@stud.u-szeged.hu> on 2009-10-19
Reviewed by Tor Arne Vestbø.

  • platform/qt/fast/canvas/canvas-empty-image-pattern-expected.txt: Added.
05:57 Changeset [49778] by krit@webkit.org

2009-10-19 Dirk Schulze <krit@webkit.org>

Reviewed by Darin Adler.

Move feGaussianBlur from WebCore/svg to WebCore/platform
https://bugs.webkit.org/show_bug.cgi?id=30495

This patch moves SVGFRGaussianBlur from svg/graphics/filters
to platform/graphics/filters/FEGaussianBlur. This is needed
for shadow support on Qt and Cairo.
No change in functionality, therfore no new test case needed.

  • GNUmakefile.am:
  • WebCore.gypi:
  • WebCore.pro:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • platform/graphics/filters/FEGaussianBlur.cpp: Added.
  • platform/graphics/filters/FEGaussianBlur.h: Added.
  • svg/SVGFEGaussianBlurElement.h:
  • svg/graphics/filters/SVGFEGaussianBlur.cpp: Removed.
  • svg/graphics/filters/SVGFEGaussianBlur.h: Removed.
05:49 Changeset [49777] by yael.aharon@nokia.com

Unreviewed.

Patch by Yael Aharon <yael.aharon@nokia.com> on 2009-10-19
Added reviewer name to r49574.

05:48 Changeset [49776] by ossy@webkit.org

2009-10-19 Andras Becsi <becsi.andras@stud.u-szeged.hu>

Reviewed by Tor Arne Vestbø.

[Qt] Update outdated metrics for these passing tests.

  • platform/qt/css1/font_properties/font-expected.txt:
  • platform/qt/fast/xsl/document-function-expected.txt:
04:58 Changeset [49775] by eric@webkit.org

2009-10-19 Joanmarie Diggs <joanmarie.diggs@gmail.com>

Reviewed by Xan Lopez.

https://bugs.webkit.org/show_bug.cgi?id=30489
[Gtk] The document frame/html container claims to be parentless

Work around for the parentless claim made by the accessible associated
with the embedded WebView. When we identify this condition, get the
widget's parent, and then get the AtkObject from that. Admittedly an ugly
hack, but we'll need it for Yelp and other Gtk+/GNOME applications which
plan to switch over to WebKit.

  • accessibility/gtk/AccessibilityObjectWrapperAtk.cpp: (webkit_accessible_get_parent):
04:51 Changeset [49774] by jmalonzo@webkit.org

2009-10-19 Benjamin Otte <otte@gnome.org>

Reviewed by Jan Alonzo.

Handle duration queries properly
https://bugs.webkit.org/show_bug.cgi?id=29999

Previously duration queries failed to handle unknown duration
and didn't treat the returned duration as an unsigned.

  • platform/graphics/gtk/MediaPlayerPrivateGStreamer.cpp: (WebCore::MediaPlayerPrivate::duration):
04:50 Changeset [49773] by eric@webkit.org

2009-10-19 Daniel Macks <dmacks@netspace.org>

Reviewed by Adam Barth.

Inconsistent handling of gettext domain can cause compile failure
https://bugs.webkit.org/show_bug.cgi?id=30292

  • GNUmakefile.am: Use DOMAIN variable instead of hard-coded value
04:28 Changeset [49772] by eric@webkit.org

2009-10-19 Jocelyn Turcotte <jocelyn.turcotte@nokia.com>

Reviewed by Tor Arne Vestbø.

[Qt] Fix build on Windows.

  • plugins/win/PluginViewWin.cpp: (windowHandleForPageClient):
04:17 Changeset [49771] by zecke@webkit.org

Windowed Plugins: Don't crash when client is 0.

Client is 0 when we use QWebPage without a QWebView or QGraphicsWebView.
In addition, setFrameRect()/updatePluginWidget() is called even if the
plugin was not succesfully loaded. updatePluginWidget() updates the
window rect which is, in theory, useful to draw something that indicates
that we didn't load successfully.

So, a status check is added to setNPWindowIfNeeded.

03:44 Changeset [49770] by eric@webkit.org

2009-10-19 Girish Ramakrishnan <girish@forwardbias.in>

Reviewed by Holger Freyther.

[Qt] Windowed Plugins: Fix crash when QWebPage is deleted after QWebView.

Fixes various sources of crashes:

  1. The PluginContainer is a child of QWebView. When the view gets deleted, the PluginView is not notified about the deletion of PluginContainer.
  2. QWebView destructor does not set client to 0.
  3. Sometimes pending paint events are sent after the plugin has died, so add a check in PluginView::setNPWindowIfNeeded.

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

  • plugins/qt/PluginContainerQt.cpp: (PluginContainerQt::~PluginContainerQt):
  • plugins/qt/PluginViewQt.cpp: (WebCore::PluginView::setNPWindowIfNeeded):

2009-10-19 Girish Ramakrishnan <girish@forwardbias.in>

Reviewed by Holger Freyther.

[Qt] Windowed Plugins: Fix crash when QWebPage is deleted after QWebView.

Fixes various sources of crashes:

  1. The PluginContainer is a child of QWebView. When the view gets deleted, the PluginView is not notified about the deletion of PluginContainer.
  2. QWebView destructor does not set client to 0.
  3. Sometimes pending paint events are sent after the plugin has died, so add a check in PluginView::setNPWindowIfNeeded.

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

  • Api/qwebview.cpp: (QWebView::~QWebView):
  • tests/qwebview/qwebview.pro:
  • tests/qwebview/tst_qwebview.cpp: (tst_QWebView::reusePage_data): (tst_QWebView::reusePage):
00:17 Changeset [49769] by eric@webkit.org

2009-10-19 Jakob Truelsen <antialize@gmail.com>

Reviewed by Adam Barth.

https://bugs.webkit.org/show_bug.cgi?id=29042
Allow one to customize the minimal and maximal shrink factors used when printing.

  • page/PrintContext.cpp: (WebCore::PrintContext::begin):
  • page/Settings.cpp: (WebCore::Settings::Settings): (WebCore::Settings::setPrintingMinimumShrinkFactor): (WebCore::Settings::setPrintingMaximumShrinkFactor):
  • page/Settings.h: (WebCore::Settings::printingMinimumShrinkFactor): (WebCore::Settings::printingMaximumShrinkFactor):

2009-10-19 Jakob Truelsen <antialize@gmail.com>

Reviewed by Adam Barth.

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

Allow one to costumize the minimal and maximal shrink factors,
Added methods setPrintingMinimumShrinkFactor, printingMinimumShrinkFactor,
setPrintingMaximumShrinkFactor, printingMaximumShrinkFactor to QWebSettings.

  • Api/qwebsettings.cpp: (QWebSettingsPrivate::apply): (QWebSettings::QWebSettings): (QWebSettings::setPrintingMinimumShrinkFactor): (QWebSettings::printingMinimumShrinkFactor): (QWebSettings::setPrintingMaximumShrinkFactor): (QWebSettings::printingMaximumShrinkFactor):
  • Api/qwebsettings.h:
00:04 Changeset [49768] by abarth@webkit.org

Fix ChangeLog spelling.

10/18/09:

23:59 Changeset [49767] by eric@webkit.org

2009-10-18 Erik Arvidsson <arv@chromium.org>

Reviewed by Adam Barth.

Fixes issue where doubleclicking a word could select following adjacent newlines.
https://bugs.webkit.org/show_bug.cgi?id=30234

  • platform/mac/editing/selection/doubleclick-should-not-expand-across-lines-expected.txt: Added.
  • platform/mac/editing/selection/doubleclick-should-not-expand-across-lines.html: Added.
  • platform/mac/editing/selection/script-tests/TEMPLATE.html: Added.
  • platform/mac/editing/selection/script-tests/doubleclick-should-not-expand-across-lines.js: Added. (description.doubleClickOnElement): (doubleClickAt): (selectionShouldBe):
  • platform/win/editing/selection/doubleclick-should-not-expand-across-lines-expected.txt: Added.
  • platform/win/editing/selection/doubleclick-should-not-expand-across-lines.html: Added.
  • platform/win/editing/selection/script-tests/TEMPLATE.html: Added.
  • platform/win/editing/selection/script-tests/doubleclick-should-not-expand-across-lines.js: Added. (description.doubleClickOnElement): (doubleClickAt): (selectionShouldBe):

2009-10-18 Erik Arvidsson <arv@chromium.org>

Reviewed by Adam Barth.

Fixes issue where doubleclicking a word could select following adjacent newlines.
https://bugs.webkit.org/show_bug.cgi?id=30234

Tests: platform/mac/editing/selection/doubleclick-should-not-expand-across-lines.html

platform/mac/editing/selection/script-tests/TEMPLATE.html
platform/win/editing/selection/doubleclick-should-not-expand-across-lines.html
platform/win/editing/selection/script-tests/TEMPLATE.html

  • editing/VisibleSelection.cpp: (WebCore::VisibleSelection::appendTrailingWhitespace):
23:51 Changeset [49766] by eric@webkit.org

2009-10-18 Kenneth Rohde Christiansen <kenneth@webkit.org>

Reviewed by Simon Hausmann.

Rename fixedContentsSize property to preferredContentsSize as
agreed upon with Siman Hausmann and Matthias Ettrich.

  • Api/qwebpage.cpp: (QWebPage::preferredContentsSize): (QWebPage::setPreferredContentsSize):
  • Api/qwebpage.h:
  • WebCoreSupport/FrameLoaderClientQt.cpp: (WebCore::FrameLoaderClientQt::transitionToCommittedForNewPage):

2009-10-18 Kenneth Rohde Christiansen <kenneth@webkit.org>

Reviewed by Simon Hausmann.

Use the setPreferredContentsSize method instead
of setFixedContentsSize, as the method has been renamed.

  • DumpRenderTree/qt/DumpRenderTree.cpp: (WebCore::DumpRenderTree::open):
  • DumpRenderTree/qt/LayoutTestControllerQt.cpp: (LayoutTestController::setFixedContentsSize):
23:50 QtBackLog edited by zecke@selfish.org
(diff)
23:46 QtBackLog edited by zecke@selfish.org
Move QtWebKit problems into a table. (diff)
23:43 Changeset [49765] by eric@webkit.org

2009-10-18 Kwang Yul Seo <skyul@company100.net>

Reviewed by Darin Adler.

Use _stricmp and _strnicmp instead of deprecated stricmp and strnicmp.
https://bugs.webkit.org/show_bug.cgi?id=30474

stricmp and strnicmp are deprecated beginning in Visual
C++ 2005. Use _stricmp and _strnicmp instead in StringExtras.h.

  • wtf/StringExtras.h: (strncasecmp): (strcasecmp):
23:42 QtBackLog edited by zecke@selfish.org
Change Qt items into a table (diff)
23:34 QtBackLog edited by zecke@selfish.org
Add completed items to the bottom (diff)
20:53 Changeset [49764] by kevino@webkit.org

CURL build fix, use proper header name.

19:38 Changeset [49763] by mrowe@apple.com

<rdar://problem/7309062> Crashes in WebCore::DocumentLoader::isStopping below WebKit::NetscapePluginInstanceProxy::loadRequest.

Reviewed by Dan Bernstein.

  • Plugins/Hosted/NetscapePluginInstanceProxy.mm:

(WebKit::NetscapePluginInstanceProxy::loadRequest): Null-check the DocumentLoader before deferencing it.

19:09 Changeset [49762] by pfeldman@chromium.org

2009-10-18 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Timothy Hatcher.

Web Inspector: Properties on Arrays and NodeLists are not logged correctly.

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

WebCore:

  • inspector/front-end/ConsoleView.js: (WebInspector.ConsoleView.prototype._formatarray): (WebInspector.ConsoleView.prototype._printArray):

LayoutTests:

  • inspector/console-format-expected.txt:
  • inspector/console-format.html:
18:20 Changeset [49761] by pfeldman@chromium.org

2009-10-18 Pavel Feldman <pfeldman@chromium.org>

Not reviewed: touching WebCore so that it
initiated inspector frontend deployment on Windows.

Web Inspector: frontend files are not deployed in Windows
incremental build.

  • inspector/InspectorDOMAgent.cpp:
17:20 Changeset [49760] by pfeldman@chromium.org

2009-10-18 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Timothy Hatcher.

Web Inspector: make console tests more granular.

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

  • inspector/console-dir-expected.txt: Added.
  • inspector/console-dir.html: Added.
  • inspector/console-dirxml-expected.txt: Added.
  • inspector/console-dirxml.html: Added.
  • inspector/console-format-expected.txt: Added.
  • inspector/console-format.html: Added.
  • inspector/console-tests-expected.txt:
  • inspector/console-tests.html:
  • platform/gtk/Skipped:
  • platform/qt/Skipped:
16:48 Changeset [49759] by mrowe@apple.com

Versioning.

16:17 Changeset [49758] by pfeldman@chromium.org

2009-10-18 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Timothy Hatcher.

Web Inspector: Title of dir(["test", "test2"]) should be Array

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

Test: inspector/console-tests.html

  • inspector/front-end/InjectedScript.js:
14:15 Changeset [49757] by krit@webkit.org

2009-10-18 Dirk Schulze <krit@webkit.org>

Reviewed by Nikolas Zimmermann.

REGRESSION: SVG Mask doesn't work for maskContentUnits="objectBoundingBox"
https://bugs.webkit.org/show_bug.cgi?id=30480

Patch for SVGMaskElement landed in r49598 should correct the location of
the mask image graphics context. It only corrects the location for userSpaceOnUse
mode in maskContentUnits but breaks objectBoundingBoxMode.
The maskDestRect shouldn't be moved. It is not responsible for the correct context
postioin but for the correct position of the mask image.
This patch calculates the context postion independently from the maskDestRect.
It also uses lineareRGB color space for masking on CG now. This is the default
color space for masking operations.
We already have tests for both maskContentUnits modes.

*svg/W3C-SVG-1.1/masking-intro-01-f.svg
*svg/custom/mask-with-default-value.svg

Some tests needed new pixel tests because of the new color space.

  • svg/SVGMaskElement.cpp: (WebCore::SVGMaskElement::drawMaskerContent):

The color space of masking operations in SVG changed to linearSVG.
This needs an update of some masking pixel tests.

  • platform/mac/svg/W3C-SVG-1.1/masking-intro-01-f-expected.checksum:
  • platform/mac/svg/W3C-SVG-1.1/masking-intro-01-f-expected.png:
  • platform/mac/svg/batik/masking/maskRegions-expected.checksum:
  • platform/mac/svg/batik/masking/maskRegions-expected.png:
  • platform/mac/svg/custom/grayscale-gradient-mask-expected.checksum:
  • platform/mac/svg/custom/grayscale-gradient-mask-expected.png:
12:55 Changeset [49756] by eric@webkit.org

2009-10-18 Anton Muhin <antonm@chromium.org>

Reviewed by Adam Barth.

Use v8::Integer::NewFromUnsigned when converting unsigneds into V8's numbers.
That is notably faster for small numbers (most common case).
https://bugs.webkit.org/show_bug.cgi?id=30493

  • bindings/scripts/CodeGeneratorV8.pm:
12:29 Changeset [49755] by jmalonzo@webkit.org

2009-10-18 Jan Michael Alonzo <jmalonzo@webkit.org>

Reviewed by Holger Freyther.

[GTK] Add MathML to the build system
https://bugs.webkit.org/show_bug.cgi?id=30487

Add --enable-mathml to configure.

  • configure.ac:

Add MathML sources if MathML support is enabled.

  • GNUmakefile.am:

Include layout test baseline for Gtk.

  • platform/gtk/mathml/presentation/sub-expected.txt: Added.
  • platform/gtk/mathml/presentation/sup-expected.txt: Added.
  • platform/gtk/mathml/presentation/tables-expected.txt: Added.
11:21 Changeset [49754] by kevino@webkit.org

Non-PCH build fix. Added missing header.

10:17 Changeset [49753] by simon.fraser@apple.com

2009-10-18 Simon Fraser <simon.fraser@apple.com>

Reviewed by Dan Bernstein.

Compositing layers not correctly updated after partial layout
https://bugs.webkit.org/show_bug.cgi?id=30425

When a partial layout happened, for example as the result of a postion-change-only
layout, then some compositing layer positions were not correctly updated. To fix
this, updateLayerPositions() now carries along a flag that is set at the rootmost
layer being updated, and used to determine when we hit the first compositing layer
in this update. RenderLayerBacking::updateAfterLayout() makes use of this information
to do a full geometry update on that layer, which is thus the rootmost compositing
layer that is being updated.

Test: compositing/geometry/partial-layout-update.html

  • page/FrameView.cpp: (WebCore::FrameView::layout):
  • rendering/RenderLayer.cpp: (WebCore::RenderLayer::updateLayerPositions): (WebCore::RenderLayer::scrollToOffset):
  • rendering/RenderLayer.h: (WebCore::RenderLayer::):
  • rendering/RenderLayerBacking.cpp: (WebCore::RenderLayerBacking::updateAfterLayout):
  • rendering/RenderLayerBacking.h:
10:02 Changeset [49752] by kevino@webkit.org

Reviewed by Kevin Ollivier.

Add the ability to specify a proxy for wxWebKit.

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

09:23 Changeset [49751] by eric@webkit.org

2009-10-18 Joanmarie Diggs <joanmarie.diggs@gmail.com>

Reviewed by Xan Lopez

https://bugs.webkit.org/show_bug.cgi?id=25901
Use ATK_ROLE_SECTION for divTag and ATK_ROLE_LABEL for labelTag

Replaces the use of ATK_ROLE_PANEL with the expected accessible roles.

  • accessibility/gtk/AccessibilityObjectWrapperAtk.cpp: (webkit_accessible_get_role):
09:08 Changeset [49750] by kevino@webkit.org

Reviewed by Kevin Ollivier.

Add basic database support to wx API

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

09:00 Changeset [49749] by eric.carlson@apple.com

2009-10-18 Eric Carlson <eric.carlson@apple.com>

Skip media/video-empty-source.html until <rdar://problem/6563117> has been fixed.

  • platform/mac/Skipped:
07:50 QtBackLog edited by zecke@selfish.org
(diff)
02:35 QtWebKitPlugins edited by girish@forwardbias.in
(diff)

10/17/09:

23:27 Changeset [49748] by eric@webkit.org

2009-10-17 Yuta Kitamura <yutak@chromium.org>

Reviewed by Darin Adler.

Fix inefficient string concatenation.

LayoutTests/fast/css/large-list-of-rules-crash.html contains code that
concatenate strings in an inefficient way. Concatenation of strings should be
done with Array.join method. This patch fixes this issue, and also fixes typos
in the test.

fast/css/large-list-of-rules-crash.html concatenates strings in an inefficient way
https://bugs.webkit.org/show_bug.cgi?id=30436

  • fast/css/large-list-of-rules-crash.html:
18:46 QtBackLog edited by zecke@selfish.org
(diff)
18:41 QtBackLog edited by zecke@selfish.org
(diff)
18:22 QtBackLog edited by zecke@selfish.org
(diff)
18:15 QtBackLog edited by zecke@selfish.org
(diff)
16:07 Changeset [49747] by mrowe@apple.com

Versioning.

16:06 Changeset [49746] by mrowe@apple.com

New tag.

15:52 Changeset [49745] by zimmermann@webkit.org

2009-10-17 Nikolas Zimmermann <nzimmermann@rim.com>

Reviewed by George Staikos.

Cleanup SVGElement code, preparing for animVal support
https://bugs.webkit.org/show_bug.cgi?id=30466

Add new SynchronizablePropertyController and move the code for SVG property <-> XML attribute synchronization
from SVGElement. This is a further preparation for animVal support, where SynchronizablePropertyController will
be used to control the start/end state of an animated property.

We're currently tracking animated properties that need synchronization, generalize this concept and use
it to track their creation (bound to certain SVG*Element classes) as well as a flag determing the need
of synchronization (no SVG DOM object wrappers, no synchronization needed).

No change in functionality, thus no tests.

  • GNUmakefile.am: Add 'SynchronizablePropertyController.cpp/h' to build
  • WebCore.gypi: Ditto.
  • WebCore.pro: Ditto.
  • WebCore.vcproj/WebCore.vcproj: Ditto.
  • WebCore.xcodeproj/project.pbxproj: Ditto.
  • svg/SVGAllInOne.cpp: Ditto.
  • svg/SVGAnimatedProperty.h: Add registerProperty() function (WebCore::::SVGAnimatedProperty): Call registerProperty() function (WebCore::::registerProperty): Add new registerProperty() function, announcing new properties to SynchronizablePropertyController.
  • svg/SVGAnimatedTemplate.h: Use propertyController() method, to access the new functions moved from SVGElement. (WebCore::lookupOrCreateWrapper):
  • svg/SVGElement.cpp: Adapt to code moving to SynchronizablePropertyController. (WebCore::SVGElement::updateAnimatedSVGAttribute):
  • svg/SVGElement.h: Move handling SVG property synchronization to SynchronizablePropertyController. (WebCore::SVGElement::propertyController): Expose reference to SynchronizablePropertyController object.
  • svg/SVGViewSpec.cpp: Reorder initialization order of contextElement, leading to possible crashes. (WebCore::SVGViewSpec::SVGViewSpec): (WebCore::SVGViewSpec::viewTarget):
  • svg/SVGViewSpec.h: Ditto. (WebCore::SVGViewSpec::contextElement):
  • svg/SynchronizablePropertyController.cpp: Added. (WebCore::SynchronizablePropertyController::SynchronizablePropertyController): (WebCore::SynchronizablePropertyController::registerProperty): (WebCore::SynchronizablePropertyController::setPropertyNeedsSynchronization): (WebCore::SynchronizablePropertyController::synchronizeProperty): (WebCore::SynchronizablePropertyController::synchronizeAllProperties): (WebCore::SynchronizablePropertyController::startAnimation): (WebCore::SynchronizablePropertyController::stopAnimation):
  • svg/SynchronizablePropertyController.h: Added. (WebCore::SynchronizableProperty::SynchronizableProperty): (WebCore::SynchronizableProperty::isHashTableDeletedValue): (WebCore::SynchronizableProperty::operator==): (WebCore::SynchronizablePropertyHash::hash): (WebCore::SynchronizablePropertyHash::equal): (WebCore::SynchronizablePropertyHashTraits::constructDeletedValue): (WebCore::SynchronizablePropertyHashTraits::isDeletedValue):
15:38 Changeset [49744] by ap@apple.com

Reviewed by Eric Carlson.

https://bugs.webkit.org/show_bug.cgi?id=30470
Make marking a bug as duplicate easier

  • js/field.js: (setResolutionToDuplicate): Focus the edit field, too.
13:00 Changeset [49743] by eric@webkit.org

2009-10-17 Joanmarie Diggs <joanmarie.diggs@gmail.com>

Reviewed by Xan Lopez.

https://bugs.webkit.org/show_bug.cgi?id=25413
[GTK] Please expose the level of headings

Exposes the heading level as an attribute of the AtkObject.

  • accessibility/gtk/AccessibilityObjectWrapperAtk.cpp: (addAttributeToSet): (webkit_accessible_get_attributes): (webkit_accessible_class_init):
08:54 Changeset [49742] by eric@webkit.org

2009-10-17 Kwang Yul Seo <skyul@company100.net>

Reviewed by Eric Seidel.

EventHandler::eventLoopHandleMouseDragged needs ENABLE(DRAG_SUPPORT) guards
https://bugs.webkit.org/show_bug.cgi?id=30472

Put ENABLE(DRAG_SUPPORT) guards around EventHandler::eventLoopHandleMouseDragged
in EventHandler.cpp. MSVC fails to compile when DRAG_SUPPORT not enabled.

  • page/EventHandler.cpp:
08:49 Changeset [49741] by kevino@webkit.org

Reviewed by Kevin Ollivier.

Optionally allow the user to zoom text using the mouse wheel.

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

08:45 Changeset [49740] by eric@webkit.org

2009-10-17 Alpha Lam <hclam@chromium.org>

Reviewed by Eric Seidel.

[chromium] Video controls not zoomed / transformed correctly
https://bugs.webkit.org/show_bug.cgi?id=30461

Draw the images in the controls scaled.

No new tests. With this change Chromium will pass the following tests:
LayoutTests/media/video-controls-zoomed.html
LayoutTests/media/video-controls-transformed.html

  • rendering/RenderMediaControlsChromium.cpp: (WebCore::paintMediaButton): Pass along the target rect directly. (WebCore::RenderMediaControlsChromium::adjustMediaSliderThumbSize): Adjust the thumb slide applied with the scale factor.
00:53 Changeset [49739] by laszlo.1.gombos@nokia.com

2009-10-17 Laszlo Gombos <laszlo.1.gombos@nokia.com>

Unreviewed.

[Gtk] Build fix for WebKitGtk after r49723.

Add bindings/js/JSExceptionBase.* to the build.

  • GNUmakefile.am:
00:46 Changeset [49738] by laszlo.1.gombos@nokia.com

2009-10-17 Laszlo Gombos <laszlo.1.gombos@nokia.com>

Unreviewed.

[Qt] Build fix for QtWebKit after r49723.

Add bindings/js/JSExceptionBase.cpp to the build.

  • WebCore.pro:
00:32 Changeset [49737] by pfeldman@chromium.org

2009-10-17 Pavel Feldman <pfeldman@chromium.org>

Not reviewed: reduce inspector console test size - it times out.

  • inspector/console-tests-expected.txt:
  • inspector/console-tests.html:
Note: See TracTimeline for information about the timeline view.