Context Navigation

← Previous Period
Next Period →

Timeline

May 9, 2010:

10:07 PM Changeset in webkit [59066] by tkent@chromium.org

4 edits in trunk/LayoutTests

Unreviewed, test expectation update.

Update Chromium/Linux expectation for fast/forms/input-file-re-render.html.
http://crbug.com/40027

platform/chromium-linux/fast/forms/input-file-re-render-expected.checksum:
platform/chromium-linux/fast/forms/input-file-re-render-expected.png:
platform/chromium/test_expectations.txt:

7:17 PM Changeset in webkit [59065] by oliver@apple.com

2 edits in trunk/JavaScriptCore

Fix accidental repeat addition of emit_op_new_regexp

6:41 PM Changeset in webkit [59064] by oliver@apple.com

14 edits
2 moves
1 add
1 delete in trunk

2010-05-09 Geoffrey Garen <ggaren@apple.com>

Reviewed by Oliver Hunt.

Reserve a large-ish initial capacity for Lexer::m_buffer16.

SunSpider says 0.3% faster.

m_buffer16 is used when parsing complex strings -- for example, strings
with escape sequences in them. These kinds of strings can be really long,
and we want to avoid repeatedly copying as we grow m_buffer16.

The net memory cost is quite low, since it's proporitional to source
code we already have in memory, and we throw away m_buffer16 right when
we're done parsing.

parser/Lexer.cpp: (JSC::Lexer::Lexer): No need to reserve initial capacity in our constructor, since setCode will be called before we're asked to lex anything. (JSC::Lexer::setCode): Reserve enough space to lex half the source code as a complex string without having to copy. (JSC::Lexer::clear): No need to reserve initial capacity here either, since setCode will be called before we're asked to lex anything.

2010-05-09 Oliver Hunt <oliver@apple.com>

Reviewed by Gavin Barraclough.

REGRESSION(r57955): RegExp literals should not actually be cached, so r57955 should be rolled out.
https://bugs.webkit.org/show_bug.cgi?id=38828

Replace incorrect test for caching regexp literals with ones that tests that they are not cached.

fast/js/regexp-literals-are-constants-expected.txt: Removed.
fast/js/regexp-literals-are-constants.html: Removed.
fast/js/regexp-literals-arent-constants-expected.txt: Added.
fast/js/regexp-literals-arent-constants.html: Added.
fast/js/script-tests/regexp-literals-are-constants.js: Removed.
fast/js/script-tests/regexp-literals-arent-constants.js: Added. (test1): (returnRegExpLiteral): (returnConditionalRegExpLiteral):

3:09 PM Changeset in webkit [59063] by loislo@chromium.org

3 edits in trunk/WebCore

2010-05-10 Ilya Tikhonovsky <loislo@chromium.org>

Not reviewed: rollback the commit r59062

inspector/front-end/TimelinePanel.js: (WebInspector.TimelinePanel.prototype._innerAddRecordToTimeline): (WebInspector.TimelinePanel.FormattedRecord.prototype._generatePopupContent): (WebInspector.TimelinePanel.FormattedRecord.prototype._getRecordDetails):
inspector/front-end/inspector.js: (WebInspector.linkifyURLAsNode):

2:38 PM Changeset in webkit [59062] by loislo@chromium.org

3 edits in trunk/WebCore

2010-05-10 Ilya Tikhonovsky <loislo@chromium.org>

Reviewed by Pavel Feldman.

WebInspector: Cosmetic changes.
1) Inner Function Call event of Timer Fire should be merged into parent even if it is not the only child.
2) Standard tooltips for resource links are clashing with Popover.
3) Recalculate Style caller info are duplicating in Details Popover (Caller and Details rows).
https://bugs.webkit.org/show_bug.cgi?id=38732

inspector/front-end/TimelinePanel.js: (WebInspector.TimelinePanel.prototype._innerAddRecordToTimeline): (WebInspector.TimelinePanel.FormattedRecord.prototype._generatePopupContent): (WebInspector.TimelinePanel.FormattedRecord.prototype._getRecordDetails):
inspector/front-end/inspector.js: (WebInspector.linkifyURLAsNode):

2:16 PM Changeset in webkit [59061] by ggaren@apple.com

2 edits in trunk/JavaScriptCore

Reserve a large-ish initial capacity for Lexer::m_buffer16.

Reviewed by Oliver Hunt.

SunSpider says 0.3% faster.

m_buffer16 is used when parsing complex strings -- for example, strings
with escape sequences in them. These kinds of strings can be really long,
and we want to avoid repeatedly copying as we grow m_buffer16.

The net memory cost is quite low, since it's proporitional to source
code we already have in memory, and we throw away m_buffer16 right when
we're done parsing.

parser/Lexer.cpp:

(JSC::Lexer::Lexer): No need to reserve initial capacity in our constructor,
since setCode will be called before we're asked to lex anything.
(JSC::Lexer::setCode): Reserve enough space to lex half the source code
as a complex string without having to copy.
(JSC::Lexer::clear): No need to reserve initial capacity here either,
since setCode will be called before we're asked to lex anything.

2:11 PM Changeset in webkit [59060] by eric@webkit.org

2 edits in trunk/JavaScriptCore

2010-05-09 Laszlo Gombos <Laszlo Gombos>

Reviewed by Eric Seidel.

[Qt] Remove YARR configuration rules from Qt make system
https://bugs.webkit.org/show_bug.cgi?id=38819

Setting YARR based on JIT seeting is now in Platform.h
for all ports. This patch essentially reverses r49238.

JavaScriptCore.pri:

1:58 PM Changeset in webkit [59059] by loislo@chromium.org

2 edits in trunk/WebCore

2010-05-08 Ilya Tikhonovsky <loislo@chromium.org>

Reviewed by Pavel Feldman.

WebInspector: If some record is not in visible area then it's expand element also is not
visible even if it has children in visible area.
https://bugs.webkit.org/show_bug.cgi?id=38643

inspector/front-end/TimelinePanel.js: (WebInspector.TimelinePanel): (WebInspector.TimelinePanel.prototype._refreshRecords):

1:34 PM Changeset in webkit [59058] by weinig@apple.com

5 edits in trunk/WebCore

Fix for https://bugs.webkit.org/show_bug.cgi?id=38557
r58526 introduced a ~30% regression on Dromaeo JS lib

Reviewed by Darin Adler.

This fix does two things.

Don't use QualifiedName as the key to a HashMap, use a RefPtr<QualifiedNameImpl> instead. We should remove the HashTraits for QualifiedName and that will happen in a follow up patch.
Only mark cached NodeLists on Documents instead of all Nodes. This is okay since the marking of NodeLists is an optimization to keep NodeList wrappers alive and is not mandated by any spec.

bindings/js/JSDocumentCustom.cpp:

(WebCore::JSDocument::markChildren):

bindings/js/JSNodeCustom.cpp:

(WebCore::JSNode::markChildren):

dom/Node.cpp:

(WebCore::Node::removeCachedTagNodeList):
(WebCore::Node::getElementsByTagNameNS):

dom/NodeRareData.h:

9:37 AM Changeset in webkit [59057] by eric@webkit.org

2 edits in trunk/WebKit/qt

2010-05-09 Noam Rosenthal <noam.rosenthal@nokia.com>

Reviewed by Kenneth Rohde Christiansen.

[Qt] Crash in QGraphicsWebViewPrivate::~QGraphicsWebViewPrivate when animation were used
https://bugs.webkit.org/show_bug.cgi?id=38574

The fix uses a QWeakPointer for rootGraphicsLayer, protecting from a crash in case the layer is deleted before the QGraphicsWebView.

Api/qgraphicswebview.cpp: (QGraphicsWebViewPrivate::QGraphicsWebViewPrivate): (QGraphicsWebViewPrivate::~QGraphicsWebViewPrivate): (QGraphicsWebViewPrivate::setRootGraphicsLayer): (QGraphicsWebViewPrivate::updateCompositingScrollPosition):

4:42 AM Changeset in webkit [59056] by oliver@apple.com

7 edits in trunk/JavaScriptCore

2010-05-09 Oliver Hunt <oliver@apple.com>

Reviewed by Maciej Stachowiak.

Improve string indexing performance
https://bugs.webkit.org/show_bug.cgi?id=38814

Add an assembly stub to do indexed loads from strings much
more cheaply than the current stub dispatch logic. We can
do this because we are able to make guarantees about the
register contents when entering the stub so the call overhead
is negligible.

jit/JIT.h:
jit/JITInlineMethods.h:
jit/JITOpcodes.cpp:
jit/JITPropertyAccess.cpp: (JSC::JIT::stringGetByValStubGenerator): (JSC::JIT::emitSlow_op_get_by_val):

Moved from JITOpcodes.cpp to keep the slowcase next to
the normal case codegen as we do for everything else.

jit/JITPropertyAccess32_64.cpp: (JSC::JIT::stringGetByValStubGenerator): (JSC::JIT::emitSlow_op_get_by_val):
jit/JSInterfaceJIT.h: (JSC::JSInterfaceJIT::emitFastArithImmToInt):

4:18 AM Changeset in webkit [59055] by mjs@apple.com

3 edits in trunk/JavaScriptCore

2010-05-09 Maciej Stachowiak <mjs@apple.com>

Fixed version of: "Optimized o[s] where o is a cell and s is a string"
https://bugs.webkit.org/show_bug.cgi?id=38815

Fixed the previous patch for this from Geoff Garen.

The two problems were a missing exception check and a PropertySlot
initialized improperly, leading to crashes and failures in the case
of getters accessed with bracket syntax.

Previous patch:

Optimized o[s] where o is a cell and s is a string, removing some old
code that wasn't really tuned for the JIT.

SunSpider says 0.8% faster.

jit/JITStubs.cpp: (JSC::DEFINE_STUB_FUNCTION):
runtime/JSCell.h:

3:24 AM Changeset in webkit [59054] by robert@webkit.org

1 edit
1 delete in trunk/LayoutTests

2010-05-09 Robert Hogan <robert@webkit.org>

[Qt] Update expected results for unskipped editing/inserting tests

Unreviewed, remove platform-specific results for editing/inserting/6633727.html

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

platform/qt/editing/inserting/6633727-expected.txt: Removed.

3:15 AM Changeset in webkit [59053] by robert@webkit.org

97 edits
5 adds in trunk/LayoutTests

[Qt] Update expected results for unskipped editing/inserting tests

Unreviewed, add expected results for unskipped tests. Results are taken
from Qt bot.

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

2:52 AM Changeset in webkit [59052] by robert@webkit.org

2 edits in trunk/LayoutTests

[Qt] Unskip editing/inserting tests that fail due to x/y differences

2010-05-03 Robert Hogan <robert@webkit.org>

Reviewed by Eric Seidel.

[Qt] Unskip editing/inserting tests that fail due to x/y differences

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

Unskip the following tests:

editing/inserting/12882.html
editing/inserting/4840662.html
editing/inserting/4875189-1.html
editing/inserting/4959067.html
editing/inserting/4960120-2.html
editing/inserting/5002441.html
editing/inserting/5058163-2.html
editing/inserting/5156401-2.html
editing/inserting/5418891.html
editing/inserting/5510537.html
editing/inserting/5549929-2.html
editing/inserting/5549929-3.html
editing/inserting/6633727.html
editing/inserting/6703873.html
editing/inserting/editable-inline-element.html
editing/inserting/insert-3654864-fix.html
editing/inserting/insert-3659587-fix.html
editing/inserting/insert-3775316-fix.html
editing/inserting/insert-3778059-fix.html
editing/inserting/insert-3786362-fix.html
editing/inserting/insert-3800346-fix.html
editing/inserting/insert-3851164-fix.html
editing/inserting/insert-3907422-fix.html
editing/inserting/insert-after-delete-001.html
editing/inserting/insert-at-end-01.html
editing/inserting/insert-at-end-02.html
editing/inserting/insert-before-link-1.html
editing/inserting/insert-br-001.html
editing/inserting/insert-br-002.html
editing/inserting/insert-br-003.html
editing/inserting/insert-br-004.html
editing/inserting/insert-br-005.html
editing/inserting/insert-br-006.html
editing/inserting/insert-br-007.html
editing/inserting/insert-br-008.html
editing/inserting/insert-br-009.html
editing/inserting/insert-br-at-tabspan-001.html
editing/inserting/insert-br-at-tabspan-002.html
editing/inserting/insert-br-at-tabspan-003.html
editing/inserting/insert-br-quoted-001.html
editing/inserting/insert-br-quoted-002.html
editing/inserting/insert-br-quoted-003.html
editing/inserting/insert-br-quoted-004.html
editing/inserting/insert-br-quoted-005.html
editing/inserting/insert-br-quoted-006.html
editing/inserting/insert-div-001.html
editing/inserting/insert-div-002.html
editing/inserting/insert-div-003.html
editing/inserting/insert-div-004.html
editing/inserting/insert-div-005.html
editing/inserting/insert-div-006.html
editing/inserting/insert-div-007.html
editing/inserting/insert-div-008.html
editing/inserting/insert-div-009.html
editing/inserting/insert-div-010.html
editing/inserting/insert-div-011.html
editing/inserting/insert-div-012.html
editing/inserting/insert-div-013.html
editing/inserting/insert-div-014.html
editing/inserting/insert-div-015.html
editing/inserting/insert-div-016.html
editing/inserting/insert-div-017.html
editing/inserting/insert-div-018.html
editing/inserting/insert-div-019.html
editing/inserting/insert-div-020.html
editing/inserting/insert-div-021.html
editing/inserting/insert-div-022.html
editing/inserting/insert-div-024.html
editing/inserting/insert-div-025.html
editing/inserting/insert-div-026.html
editing/inserting/insert-div-027.html
editing/inserting/insert-paragraph-01.html
editing/inserting/insert-paragraph-02.html
editing/inserting/insert-paragraph-03.html
editing/inserting/insert-paragraph-04.html
editing/inserting/insert-paragraph-05.html
editing/inserting/insert-space-in-empty-doc.html
editing/inserting/insert-tab-001.html
editing/inserting/insert-tab-002.html
editing/inserting/insert-tab-004.html
editing/inserting/insert-text-at-tabspan-001.html
editing/inserting/insert-text-at-tabspan-002.html
editing/inserting/insert-text-at-tabspan-003.html
editing/inserting/line-break.html
editing/inserting/paragraph-separator-01.html
editing/inserting/paragraph-separator-02.html
editing/inserting/paragraph-separator-in-table-1.html
editing/inserting/paragraph-separator-in-table-2.html
editing/inserting/redo.html
editing/inserting/return-key-with-selection-003.html
editing/inserting/typing-001.html
editing/inserting/typing-002.html
editing/inserting/typing-003.html
editing/inserting/typing-around-br-001.html
editing/inserting/typing-around-image-001.html

editing/inserting/5058163-1.html
editing/inserting/edited-whitespace-1.html
editing/inserting/paragraph-separator-03.html
editing/inserting/return-key-with-selection-001.html
editing/inserting/return-key-with-selection-002.html
editing/inserting/4875189-2.html

platform/qt/Skipped:

2:50 AM Changeset in webkit [59051] by dbates@webkit.org

2 edits in trunk/WebKitTools

2010-05-09 Daniel Bates <dbates@rim.com>

Reviewed by Chris Jerdonek.

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

Make the regular expressions for parsing the start of an SVN
and Git header global variables since they are used throughout
VCSUtils.pm.

Scripts/VCSUtils.pm:

2:08 AM Changeset in webkit [59050] by dbates@webkit.org

3 edits in trunk

2010-05-07 Daniel Bates <dbates@rim.com>

Unreviewed, change log fix.

Add back some entries inadvertently deleted in changeset 59000
<http://trac.webkit.org/changeset/59000> and not restored (as
intended) in changeset 59005 <http://trac.webkit.org/changeset/59005>.

1:42 AM Changeset in webkit [59049] by mjs@apple.com

1 edit in trunk/JavaScriptCore/ChangeLog

Add back some entries accidentally deleted in r59000 and not restored as intended in r59005.

12:21 AM Changeset in webkit [59048] by Chris Jerdonek

7 edits in trunk/WebKitTools

2010-05-09 Chris Jerdonek <Chris Jerdonek>

Reviewed by Daniel Bates.

Finished moving the header-parsing logic from svn-apply and -unapply
to VCSUtils.pm's parsing methods.

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

Scripts/VCSUtils.pm:
- Added to parseGitDiffHeader() the ability to parse and store whether a file is new or deleted.
- Also reordered in parseGitDiffHeader() some of the else statements to a more readable ordering.
- Added to parseSvnDiffHeader() the ability to parse and store whether a file is new.
Scripts/svn-apply:
- Changed handleGitBinaryChange() to use the new "isNew" and "isDeletion" diffHash key-values.
- Changed patch() to use the new "isNew" diffHash key-value.
Scripts/svn-unapply:
- Changed patch() to use the new "isNew" and "isDeletion" diffHash key-values.
Scripts/webkitperl/VCSUtils_unittest/parseDiff.pl:
- Added unit tests for new and deleted files.
Scripts/webkitperl/VCSUtils_unittest/parseGitDiffHeader.pl:
- Updated the unit tests as necessary.
- Added a unit test for a deleted file.
Scripts/webkitperl/VCSUtils_unittest/parseSvnDiffHeader.pl:
- Updated the unit tests as necessary.

May 8, 2010:

11:50 PM Changeset in webkit [59047] by Chris Jerdonek

4 edits in trunk/BugsSite

2010-05-08 Chris Jerdonek <Chris Jerdonek>

Reviewed by Maciej Stachowiak.

Allow long comment lines to wrap to the window width instead
of to 80 characters.

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

Bugzilla/Constants.pm:
- Introduced a new constant COMMENT_COLS_WRAP to replace COMMENT_COLS inside the wrap_comment() subroutine. The new constant effectively disables text wrapping by increasing the value used in wrap_comment() from 80 characters to a very large value (8000).
Bugzilla/Util.pm:
- Replaced COMMENT_COLS with COMMENT_COLS_WRAP inside wrap_comment().
skins/custom/global.css: (.bz_comment_text):
- Eliminate the width styling on comments to allow long lines to wrap at the window width.

(.bz_comment_text > span.quote):

Inherit the "white-space" property. This prevents long quoted lines in comments from causing non-quoted lines to extend beyond the window width.

8:21 PM Changeset in webkit [59046] by tonikitoo@webkit.org

2 edits in trunk/WebCore

2010-05-08 Antonio Gomes <tonikitoo@webkit.org>

Unreviewed attempt to fix Windows build.

page/SpatialNavigation.h: (WebCore::fudgeFactor):

8:05 PM Changeset in webkit [59045] by tonikitoo@webkit.org

3 edits in trunk/WebCore

Spatial Navigation: create a getter for the "fudgeFactor"
https://bugs.webkit.org/show_bug.cgi?id=38488

Reviewed by Kenneth Christiansen.
Patch by Antonio Gomes <tonikitoo@webkit.org>

A couple of places in the Spatial Navigation code make use of a "fudge factor"
to improve precision by working around outline focus metrics and such. Patch adds
a helper method for unify getter operations of this value, instead of having it
declared locally in the various methods it is used.

No behaviour change.

page/SpatialNavigation.cpp:

(WebCore::scrollIntoView):
(WebCore::deflateIfOverlapped):

page/SpatialNavigation.h:

(WebCore::fudgeFactor):

7:52 PM Changeset in webkit [59044] by Chris Jerdonek

7 edits in trunk/WebKitTools

Added to svn-apply support for git renames and copies with similarity
index less than 100%.

Reviewed by Daniel Bates.

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

Scripts/VCSUtils.pm:
- Added to parseGitDiffHeader() support for renames and similarity index less than 100%.
- Added to parseDiff() support for processing renames, renames with changes, and copies with changes.
- Added to parsePatch() the ability to process multiple return values from parseDiff().
Scripts/svn-apply:
- Added to patch() the ability to process diff hashes with the isDeletion key-value set.
Scripts/webkitperl/VCSUtils_unittest/parseDiff.pl:
- Updated the unit tests as necessary.
- Added unit tests for rename with similarity index 100%, rename with similarity index < 100%, and rename with a change to the executable bit.
Scripts/webkitperl/VCSUtils_unittest/parseGitDiffHeader.pl:
- Added unit tests for rename with similarity index 100%, rename with similarity index < 100%, and rename with a change to the executable bit.
Scripts/webkitperl/VCSUtils_unittest/parsePatch.pl:
- Updated the unit tests as necessary.
Scripts/webkitperl/VCSUtils_unittest/parseSvnDiffHeader.pl:
- Simplified the carriage-return unit test to more narrowly test only carriage returns.

7:37 PM Changeset in webkit [59043] by Chris Jerdonek

4 edits in trunk/WebKitTools

This revision suppresses the misleading "error: pathspec..." messages
when using svn-apply to add a binary file from a Git diff.

Reviewed by Daniel Bates.

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

When adding a new binary file from a Git diff, svn-apply prints
a misleading error of the form -- "error: pathspec '<filename>' did
not match any file(s) known to git. Did you forget to 'git add'?"
This patch suppresses these messages since they are normal.

Scripts/VCSUtils.pm: Added the callSilently() subroutine from runPatchCommand.pl, which executes a Perl function while suppressing STDERR.
Scripts/svn-apply: Refactored the Git portion of scmKnowsOfFile() into a gitKnowsOfFile(), and called this new subroutine using callSilently().
Scripts/webkitperl/VCSUtils_unittest/runPatchCommand.pl: Removed callSilently() since it was moved to VCSUtils.pm in this patch.

7:02 PM Changeset in webkit [59042] by Laszlo Gombos

2 edits in trunk/JavaScriptCore

2010-05-08 Laszlo Gombos <Laszlo Gombos>

Unreviewed, build fix.

Fix missing terminating ' character warning

The warning has been only seen on the Gtk buildbots.

wtf/Platform.h:

5:38 PM Changeset in webkit [59041] by eric@webkit.org

2 edits in trunk/JavaScriptCore

2010-05-08 Laszlo Gombos <Laszlo Gombos>

Reviewed by Gavin Barraclough.

Refactor YARR and YARR_JIT default rules in Platform.h
https://bugs.webkit.org/show_bug.cgi?id=38727

Turn on YARR and YARR JIT by default of JIT is enabled.
Ports can overrule this default decisions as they wish.

wtf/Platform.h:

3:59 PM Changeset in webkit [59040] by oliver@apple.com

7 edits
1 add in trunk/JavaScriptCore

2010-05-08 Oliver Hunt <oliver@apple.com>

Reviewed by Gavin Barraclough.

Split JSVALUE32_64 code out of JITOpcodes.cpp and into JITOpcodes32_64.cpp
https://bugs.webkit.org/show_bug.cgi?id=38808

GNUmakefile.am:
JavaScriptCore.gypi:
JavaScriptCore.pro:
JavaScriptCore.vcproj/JavaScriptCore/JavaScriptCore.vcproj:
JavaScriptCore.xcodeproj/project.pbxproj:
jit/JITOpcodes.cpp:
jit/JITOpcodes32_64.cpp: Added. (JSC::JIT::privateCompileCTIMachineTrampolines): (JSC::JIT::emit_op_mov): (JSC::JIT::emit_op_end): (JSC::JIT::emit_op_jmp): (JSC::JIT::emit_op_loop_if_lesseq): (JSC::JIT::emitSlow_op_loop_if_lesseq): (JSC::JIT::emit_op_new_object): (JSC::JIT::emit_op_instanceof): (JSC::JIT::emitSlow_op_instanceof): (JSC::JIT::emit_op_new_func): (JSC::JIT::emit_op_get_global_var): (JSC::JIT::emit_op_put_global_var): (JSC::JIT::emit_op_get_scoped_var): (JSC::JIT::emit_op_put_scoped_var): (JSC::JIT::emit_op_tear_off_activation): (JSC::JIT::emit_op_tear_off_arguments): (JSC::JIT::emit_op_new_array): (JSC::JIT::emit_op_resolve): (JSC::JIT::emit_op_to_primitive): (JSC::JIT::emitSlow_op_to_primitive): (JSC::JIT::emit_op_strcat): (JSC::JIT::emit_op_resolve_base): (JSC::JIT::emit_op_resolve_skip): (JSC::JIT::emit_op_resolve_global): (JSC::JIT::emitSlow_op_resolve_global): (JSC::JIT::emit_op_not): (JSC::JIT::emitSlow_op_not): (JSC::JIT::emit_op_jfalse): (JSC::JIT::emitSlow_op_jfalse): (JSC::JIT::emit_op_jtrue): (JSC::JIT::emitSlow_op_jtrue): (JSC::JIT::emit_op_jeq_null): (JSC::JIT::emit_op_jneq_null): (JSC::JIT::emit_op_jneq_ptr): (JSC::JIT::emit_op_jsr): (JSC::JIT::emit_op_sret): (JSC::JIT::emit_op_eq): (JSC::JIT::emitSlow_op_eq): (JSC::JIT::emit_op_neq): (JSC::JIT::emitSlow_op_neq): (JSC::JIT::compileOpStrictEq): (JSC::JIT::emit_op_stricteq): (JSC::JIT::emitSlow_op_stricteq): (JSC::JIT::emit_op_nstricteq): (JSC::JIT::emitSlow_op_nstricteq): (JSC::JIT::emit_op_eq_null): (JSC::JIT::emit_op_neq_null): (JSC::JIT::emit_op_resolve_with_base): (JSC::JIT::emit_op_new_func_exp): (JSC::JIT::emit_op_throw): (JSC::JIT::emit_op_get_pnames): (JSC::JIT::emit_op_next_pname): (JSC::JIT::emit_op_push_scope): (JSC::JIT::emit_op_pop_scope): (JSC::JIT::emit_op_to_jsnumber): (JSC::JIT::emitSlow_op_to_jsnumber): (JSC::JIT::emit_op_push_new_scope): (JSC::JIT::emit_op_catch): (JSC::JIT::emit_op_jmp_scopes): (JSC::JIT::emit_op_switch_imm): (JSC::JIT::emit_op_switch_char): (JSC::JIT::emit_op_switch_string): (JSC::JIT::emit_op_new_error): (JSC::JIT::emit_op_debug): (JSC::JIT::emit_op_enter): (JSC::JIT::emit_op_enter_with_activation): (JSC::JIT::emit_op_create_arguments): (JSC::JIT::emit_op_init_arguments): (JSC::JIT::emit_op_convert_this): (JSC::JIT::emitSlow_op_convert_this): (JSC::JIT::emit_op_profile_will_call): (JSC::JIT::emit_op_profile_did_call):

1:47 PM Changeset in webkit [59039] by eric@webkit.org

2 edits in trunk/WebCore

2010-05-08 Stuart Morgan <stuartmorgan@chromium.org>

Reviewed by Darin Adler.

Change the npapi.h header guards to match the npapi-headers repository
copy.

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

bridge/npapi.h:

1:03 PM Changeset in webkit [59038] by eric@webkit.org

3 edits in trunk/JavaScriptCore

2010-05-08 Gabor Loki <loki@webkit.org>

Reviewed by Gavin Barraclough.

Fix halfword loads on ARM
https://bugs.webkit.org/show_bug.cgi?id=38741

The BaseIndex and ImplicitAddress are contain 32bit wide offset, but
the load16 functions were working with 8 bit data (encoded in the
instruction). If the offset cannot be encoded in an instruction, it
should be stored in a temporary register.

assembler/ARMAssembler.h: (JSC::ARMAssembler::getOffsetForHalfwordDataTransfer):
assembler/MacroAssemblerARM.h: (JSC::MacroAssemblerARM::load16):

12:51 PM Changeset in webkit [59037] by eric@webkit.org

2 edits in trunk/JavaScriptCore

2010-05-08 Gabor Loki <loki@webkit.org>

Reviewed by Gavin Barraclough.

Fix spanning branch instruction on Cortex-A8 with Thumb-2 JIT
https://bugs.webkit.org/show_bug.cgi?id=38280

If the 32-bit Thumb-2 branch instruction spans two 4KiB regions and
the target of the branch falls within the first region it is
possible for the processor to incorrectly determine the branch
instruction, and it is also possible in some cases for the processor
to enter a deadlock state.

assembler/ARMv7Assembler.h: (JSC::ARMv7Assembler::linkJumpAbsolute):

12:36 PM Changeset in webkit [59036] by eric@webkit.org

2 edits
1 add in trunk/WebKitTools

2010-05-08 Victor Wang <victorw@chromium.org>

Reviewed by Adam Barth.

[chromium]: Upload test results json files to app engine server
Add an option to run_webkit_tests.py to upload generated
JSON files to app engine server. These JSON files will be used
by chromium layout test falkiness dashboard.

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

Scripts/webkitpy/layout_tests/layout_package/test_results_uploader.py: Added.
Scripts/webkitpy/layout_tests/run_webkit_tests.py:

12:23 PM Changeset in webkit [59035] by eric@webkit.org

3 edits in trunk/WebCore

2010-05-08 Stuart Morgan <stuartmorgan@chromium.org>

Reviewed by Darin Fisher.

Resolve most of the whitespace/comment/order differences in npapi.h
and npruntime.h relative to the npapi-headers repository versions.
No changes to the substance of the code.

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

bridge/npapi.h: (_NPPrint::): (_NPCocoaEvent::):
bridge/npruntime.h:

12:05 PM Changeset in webkit [59034] by eric@webkit.org

2 edits in trunk/WebCore

2010-05-08 Laszlo Gombos <Laszlo Gombos>

Reviewed by Darin Adler.

Fix guard around IndexedDatabase
https://bugs.webkit.org/show_bug.cgi?id=38710

No new tests, there is no new functionality.

page/PageGroup.h:

11:51 AM Changeset in webkit [59033] by eric@webkit.org

6 edits
3 adds in trunk

2010-05-08 Luiz Agostini <luiz.agostini@openbossa.org>

Reviewed by Simon Hausmann.

[Qt] Platform plugin
https://bugs.webkit.org/show_bug.cgi?id=38438

Select popups delegate private API. To provide select popup delegates
a plugin that implements QWebKitPlatformPlugin must be found
in QCoreApplication::libraryPaths.

WebCore.pro:

2010-05-08 Luiz Agostini <luiz.agostini@openbossa.org>

Reviewed by Simon Hausmann.

[Qt] Platform plugin
https://bugs.webkit.org/show_bug.cgi?id=38438

Select popups delegate private API. To provide select popup delegates
a plugin that implements QWebKitPlatformPlugin must be found
in QCoreApplication::libraryPaths.

Api/headers.pri:
Api/qwebkitplatformplugin.h: Added. (QWebSelectData::~QWebSelectData): (QWebSelectData::): (QWebSelectMethod::~QWebSelectMethod): (QWebKitPlatformPlugin::~QWebKitPlatformPlugin): (QWebKitPlatformPlugin::):
WebCoreSupport/ChromeClientQt.cpp: (WebCore::ChromeClientQt::createSelectPopup):
WebCoreSupport/ChromeClientQt.h:
WebCoreSupport/QtPlatformPlugin.cpp: Added. (WebCore::SelectData::SelectData): (WebCore::SelectData::itemText): (WebCore::SelectData::itemToolTip): (WebCore::SelectData::itemIsEnabled): (WebCore::SelectData::itemCount): (WebCore::SelectData::itemIsSelected): (WebCore::SelectData::multiple): (WebCore::SelectData::itemType): (WebCore::SelectInputMethodWrapper::SelectInputMethodWrapper): (WebCore::SelectInputMethodWrapper::show): (WebCore::SelectInputMethodWrapper::hide): (WebCore::SelectInputMethodWrapper::selectItem): (WebCore::SelectInputMethodWrapper::didHide): (WebCore::getPluginObject): (WebCore::QtPlatformPlugin::~QtPlatformPlugin): (WebCore::QtPlatformPlugin::plugin): (WebCore::QtPlatformPlugin::createSelectInputMethod):
WebCoreSupport/QtPlatformPlugin.h: Added. (WebCore::QtPlatformPlugin::QtPlatformPlugin):

11:40 AM Changeset in webkit [59032] by eric@webkit.org

2 edits in trunk/WebKit

2010-05-08 Leandro Pereira <leandro@profusion.mobi>

Reviewed by Laszlo Gombos.

[EFL] Only compile database stuff if ENABLE_DATABASE is set.
http://webkit.org/b/38777

efl/WebCoreSupport/ChromeClientEfl.cpp:

11:29 AM Changeset in webkit [59031] by eric@webkit.org

4 edits in trunk

2010-05-08 Laszlo Gombos <Laszlo Gombos>

Reviewed by Kenneth Rohde Christiansen.

[Qt] Build the ImageDiff tool for all platforms including Windows and Symbian
https://bugs.webkit.org/show_bug.cgi?id=38706

WebKit.pro:

2010-05-08 Laszlo Gombos <Laszlo Gombos>

Reviewed by Kenneth Rohde Christiansen.

[Qt] Build the ImageDiff tool for all platforms including Windows and Symbian
https://bugs.webkit.org/show_bug.cgi?id=38706

Use qmath.h instead of math.h to make it portable.

DumpRenderTree/qt/ImageDiff.cpp: (main):

11:17 AM Changeset in webkit [59030] by eric@webkit.org

2 edits in trunk/WebCore

2010-05-08 Leandro Pereira <leandro@profusion.mobi>

Reviewed by Gustavo Noronha Silva.

[EFL] It does not make sense to use RenderThemeEfl::RenderThemeEfl, so
use RenderThemeEfl (GCC 4.5 seems to be pickier about this).
http://webkit.org/b/38770

platform/efl/RenderThemeEfl.cpp: (WebCore::renderThemeEflColorClassSelectionActive): (WebCore::renderThemeEflColorClassSelectionInactive): (WebCore::renderThemeEflColorClassFocusRing): (WebCore::renderThemeEflColorClassButtonText): (WebCore::renderThemeEflColorClassComboText): (WebCore::renderThemeEflColorClassEntryText): (WebCore::renderThemeEflColorClassSearchText): (WebCore::RenderThemeEfl::adjustCheckboxStyle): (WebCore::RenderThemeEfl::adjustRadioStyle): (WebCore::RenderThemeEfl::adjustButtonStyle): (WebCore::RenderThemeEfl::adjustMenuListStyle): (WebCore::RenderThemeEfl::adjustTextFieldStyle): (WebCore::RenderThemeEfl::adjustSearchFieldDecorationStyle): (WebCore::RenderThemeEfl::adjustSearchFieldResultsButtonStyle): (WebCore::RenderThemeEfl::adjustSearchFieldResultsDecorationStyle): (WebCore::RenderThemeEfl::adjustSearchFieldCancelButtonStyle): (WebCore::RenderThemeEfl::adjustSearchFieldStyle):

11:06 AM Changeset in webkit [59029] by eric@webkit.org

4 edits in trunk/WebKit/chromium

2010-05-08 Jens Alfke <snej@chromium.org>

Reviewed by Darin Fisher.

[chromium] WebFrame::registerPasswordListener shouldn't assert on duplicate listener.
https://bugs.webkit.org/show_bug.cgi?id=38765

public/WebFrame.h: (WebKit::WebFrameImpl::registerPasswordListener): Changed return type to bool. Updated comment.
src/WebFrameImpl.cpp: (WebKit::WebFrameImpl::registerPasswordListener): Return false instead of asserting.
src/WebFrameImpl.h: (WebKit::WebFrameImpl::registerPasswordListener): Changed return type to bool.

10:53 AM Changeset in webkit [59028] by eric@webkit.org

2 edits in trunk/WebCore

2010-05-08 Leandro Pereira <leandro@profusion.mobi>

Reviewed by Gustavo Noronha Silva.

ResourceHandleSoup should depend only on GLib, not on GTK+.
http://webkit.org/b/38763

platform/network/soup/ResourceHandleSoup.cpp: Include glib.h instead of gtk/gtk.h.

9:49 AM Changeset in webkit [59027] by eric@webkit.org

13 edits
1 delete in trunk/WebCore

2010-05-08 Eric Seidel <eric@webkit.org>

Unreviewed, rolling out r59023.
http://trac.webkit.org/changeset/59023
https://bugs.webkit.org/show_bug.cgi?id=37874

Caused Chromium Windows build to fail to link.

WebCore.gypi:
loader/CachedMetadata.h: Removed.
loader/CachedResource.cpp:
loader/CachedResource.h:
loader/ResourceLoader.h:
loader/SubresourceLoader.cpp:
loader/SubresourceLoader.h:
loader/SubresourceLoaderClient.h:
loader/loader.cpp:
loader/loader.h:
platform/network/ResourceHandle.cpp:
platform/network/ResourceHandle.h:
platform/network/ResourceHandleClient.h:

6:31 AM Changeset in webkit [59026] by eric@webkit.org

2 edits in trunk/WebKit/qt

2010-05-08 Jocelyn Turcotte <jocelyn.turcotte@nokia.com>

Reviewed by Simon Hausmann.

[Qt] Document a limitation of QWebPage::linkClicked
https://bugs.webkit.org/show_bug.cgi?id=37694

Api/qwebpage.cpp:

6:20 AM Changeset in webkit [59025] by eric@webkit.org

2 edits in trunk/WebCore

2010-05-08 Ilya Tikhonovsky <loislo@chromium.org>

Reviewed by Timothy Hatcher.

WebInspector: Autocompletion in console is not usable because it is not add to the entered text the common prefix of suggestions.
As example if you entered $0.getE then getElementByClassName will be suggested as gray text.
If you press tab then getElementByClassName be suggested again and lementByClassName becomes selected as black text with selection background.
Really after pressing the TAB the next suggestion should be selected (getElementByTagName) and only flexible part of suggestions should be selected.
In this case it should be TagName because getElementBy is the common part for all getE prefix suggestions.
https://bugs.webkit.org/show_bug.cgi?id=38753

inspector/front-end/TextPrompt.js: (WebInspector.TextPrompt.prototype._completionsReady):

6:09 AM Changeset in webkit [59024] by eric@webkit.org

1 edit
1 add in trunk/JavaScriptCore

2010-05-08 Adam Barth <abarth@webkit.org>

Reviewed by Darin Fisher.

[WTFURL] Add a buffer for managing the output of the canonicalizer
https://bugs.webkit.org/show_bug.cgi?id=38671

The canonicalizer produces a stream of canonicalized characters, which
we will store in a URLBuffer. URLBuffer is quite similar to Vector,
but serves as an abstraction to isolate us from the underlying
character storage. In the case of WebKit, we'll probably implement a
subclass of URLBuffer with Vector. In Chromium, the backing store is a
std::string, which avoids an unnecessary memcpy that would be caused by
using Vector instead.

wtf/url/src/URLBuffer.h: Added. (WTF::URLBuffer::URLBuffer): (WTF::URLBuffer::~URLBuffer): (WTF::URLBuffer::at): (WTF::URLBuffer::set): (WTF::URLBuffer::capacity): (WTF::URLBuffer::length): (WTF::URLBuffer::data): (WTF::URLBuffer::setLength): (WTF::URLBuffer::append): (WTF::URLBuffer::grow):

5:58 AM Changeset in webkit [59023] by eric@webkit.org

13 edits
1 add in trunk/WebCore

2010-05-08 Tony Gentilcore <tonyg@chromium.org>

Reviewed by Darin Fisher.

Provide mechanism to store cached metadata on a CachedResource.
https://bugs.webkit.org/show_bug.cgi?id=37874

No new tests because no new functionality.

WebCore.gypi:
loader/CachedMetadata.h: Added. (WebCore::CachedMetadata::~CachedMetadata): (WebCore::CachedMetadata::data): (WebCore::CachedMetadata::size): (WebCore::CachedMetadata::create): (WebCore::CachedMetadata::deserialize): (WebCore::CachedMetadata::serialize): (WebCore::CachedMetadata::dataTypeID): (WebCore::CachedMetadata::readUnsigned): (WebCore::CachedMetadata::appendUnsigned): (WebCore::CachedMetadata::CachedMetadata):
loader/CachedResource.cpp: (WebCore::CachedResource::setSerializedCachedMetadata): (WebCore::CachedResource::setCachedMetadata): (WebCore::CachedResource::cachedMetadata):
loader/CachedResource.h:
loader/ResourceLoader.h: (WebCore::ResourceLoader::didReceiveCachedMetadata):
loader/SubresourceLoader.cpp: (WebCore::SubresourceLoader::didReceiveCachedMetadata):
loader/SubresourceLoader.h:
loader/SubresourceLoaderClient.h: (WebCore::SubresourceLoaderClient::didReceiveCachedMetadata):
loader/loader.cpp: (WebCore::Loader::Host::didReceiveCachedMetadata):
loader/loader.h:
platform/network/ResourceHandle.cpp: (WebCore::ResourceHandle::cacheMetadata):
platform/network/ResourceHandle.h:
platform/network/ResourceHandleClient.h: (WebCore::ResourceHandleClient::didReceiveCachedMetadata):

5:39 AM Changeset in webkit [59022] by eric@webkit.org

9 edits in trunk

2010-05-08 Michael Nordman <Michael Nordman>

Reviewed by Dmitry Titov.

Have WorkerScriptLoader provide access to the ResourceResponse for the loaded script
through the WorkerScriptLoaderClient interface.
https://bugs.webkit.org/show_bug.cgi?id=38605

No new tests. There are no script visible changes.

workers/WorkerScriptLoader.cpp: Call the client with the response. (WebCore::WorkerScriptLoader::didReceiveResponse):
workers/WorkerScriptLoaderClient.h: Define a new method to receive the response. (WebCore::WorkerScriptLoaderClient::didReceiveResponse):

2010-05-08 Michael Nordman <Michael Nordman>

Reviewed by Dmitry Titov.

Add an additional parameter to the WebSharedWorker::startWorkerContext method
to indicate which appcache the shared worker script resource was loaded from,
and provide that value when calling out to chrome via this interface.
https://bugs.webkit.org/show_bug.cgi?id=38605

I'm keeping the old method around until patches can land in both repositories
that make the old method obsolete and unneeded. There's a FIXME for that.

Also somewhat related but independent change to propagate the runtime enabled
flag thru to the page settings instance used by the 'shadow page'.

public/WebSharedWorker.h: Add the new param (but keep the old interface around for now too) (WebKit::WebSharedWorker::startWorkerContext):
src/SharedWorkerRepository.cpp: Provide the value when calling out to chrome. (WebCore::SharedWorkerScriptLoader::notifyFinished):
src/WebSharedWorkerImpl.cpp: (WebKit::WebSharedWorkerImpl::startWorkerContext):
src/WebSharedWorkerImpl.h:
src/WebWorkerBase.cpp: Propagate runtime enabled flag thru to the page settings. (WebKit::WebWorkerBase::initializeLoader):

5:13 AM Changeset in webkit [59021] by robert@webkit.org

2 edits in trunk/LayoutTests

2010-05-08 Robert Hogan <robert@webkit.org>

Unreviewed Qt bot fix: test unskipped accidentally in r59020.

platform/qt/Skipped: Uncommented http/tests/xmlhttprequest/failed-auth.html again.

4:50 AM Changeset in webkit [59020] by robert@webkit.org

7 edits
1 add in trunk

[Qt] Fix http/tests/xmlhttprequest/cross-origin-no-authorization.html

2010-05-08 Robert Hogan <robert@roberthogan.net>

Reviewed by Simon Hausmann.

[Qt] Fix http/tests/xmlhttprequest/cross-origin-no-authorization.html

and http/tests/xmlhttprequest/cross-origin-authorization.html

QHttpNetworkRequest adds Authorization and Cookie headers to XHRs
without knowing if this is valid behaviour or not. In order to allow
Qt to decide whether Cookie/Authorization headers should be added
to an XHR QtWebKit needs to use an attribute added to QNetworkRequest.
These new attributes are: QNetworkRequest::CookieLoadControlAttribute,
QNetworkRequest::CookieSaveControlAttribute,and
QNetworkRequest::AuthenticationReuseControlAttribute.

QtWebKit will set QNetworkRequest::AuthenticationReuseControlAttribute to false
unless withCredentials is set to true in the XHR.

QtWebkit will set CookieLoad/SaveControlAttribute to false unless withCredentials
is set to true in the XHR.

Qt will pass the values onto QHttpNetworkRequest and this will permit
the Qt network access processing to decide whether or not to add either
or both of the Cookie/Authorisation headers, and whether to save cookies
returned from such requests. By default the attribute
will always be true so unless QtWebKit sets it to false normal
header processing is unaffected.

There are platform-specific results for cross-origin-no-authorization.html:

Qt's DRT does not capture the final 'DONE'.
QNetworkReplyHandler.cpp's ignoreHttpError() means that async requests do not execute didFail() when they receive a 401 HTTP response. This results in async tests passing with '401 Authorisation required' rather than 'received error event'.

The Qt part of these changes was merged at: http://gitorious.org/qt/qt/merge_requests/592

In order to properly support the tests, Qt's DRT needs to use one
NetworkAccessManager for all pages. This allows it to use cached
credentials where appropriate.

The tests now pass when run individually but there seems to be a problem with
leaking the results of requests across tests when run with the others in
http/tests. This will be addressed in a separate patch.

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

DumpRenderTree/qt/DumpRenderTreeQt.cpp: (WebCore::WebPage::WebPage): (WebCore::DumpRenderTree::DumpRenderTree):
DumpRenderTree/qt/DumpRenderTreeQt.h:
platform/network/qt/ResourceRequestQt.cpp: (WebCore::ResourceRequest::toNetworkRequest):
platform/qt/Skipped: Group together the tests that are waiting for 4.7 to be unskipped.
platform/qt/http/tests/xmlhttprequest/cross-origin-no-authorization-expected.txt: Added.

4:48 AM Changeset in webkit [59019] by eric@webkit.org

2 edits in trunk/WebCore

2010-05-08 Ilya Tikhonovsky <loislo@chromium.org>

Reviewed by Pavel Feldman.

WebInsector: Record expand UI element should be extracted from TimelineRecordGraphRow as separate class for future reuse.
https://bugs.webkit.org/show_bug.cgi?id=38726

inspector/front-end/TimelinePanel.js: (WebInspector.TimelinePanel): (WebInspector.TimelinePanel.prototype._refreshRecords): (WebInspector.TimelineRecordGraphRow): (WebInspector.TimelineRecordGraphRow.prototype.update): (WebInspector.TimelineExpandableElement): (WebInspector.TimelineExpandableElement.prototype._update): (WebInspector.TimelineExpandableElement.prototype._dispose):

4:08 AM Changeset in webkit [59018] by Nikolas Zimmermann

2 edits in trunk/LayoutTests

2010-05-08 Nikolas Zimmermann <nzimmermann@rim.com>

Not reviewed. Forgot to rebase fast/dom/prototype-inheritance-2.html after the <vkern> patches.

platform/win/fast/dom/prototype-inheritance-2-expected.txt:

4:01 AM Changeset in webkit [59017] by robert@webkit.org

4 edits in trunk

[Qt] unskip http/tests/plugins/npapi-response-headers.html

2010-05-04 Robert Hogan <robert@webkit.org>

Reviewed by Eric Seidel.

[Qt] unskip http/tests/plugins/npapi-response-headers.html

Turns out this failed because run-webkit-tests was eating the first occurrence
of 'Content-Type: text/plain' in the test output as a header. Strange but true.
So do as Chromium does and preface all text dumps with the
'Content-Type: text/plain' header.

Take this opportunity to unskip plugins/private-browsing-mode.html too, which
has passed since r55433. See https://bugs.webkit.org/show_bug.cgi?id=33180
and https://bugs.webkit.org/show_bug.cgi?id=35576.

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

DumpRenderTree/qt/DumpRenderTreeQt.cpp: (WebCore::DumpRenderTree::dump):
platform/qt/Skipped:

3:55 AM Changeset in webkit [59016] by robert@webkit.org

4 edits in trunk

[Qt] Fix encoding of status message in Plugin API

2010-05-05 Robert Hogan <robert@webkit.org>

Reviewed by Alexey Proskuryakov.

[Qt] Fix encoding of status message in Plugin API

Spotted by Alexey Proskuryakov.

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

platform/qt/Skipped: Unskip plugins/set-status.html
plugins/PluginView.cpp: (WebCore::PluginView::status):

3:11 AM Changeset in webkit [59015] by krit@webkit.org

2 edits in trunk/WebCore

2010-05-08 Dirk Schulze <krit@webkit.org>

Unreviewed win build fix.

JS bindings were missing in DerivedSources.cpp

DerivedSources.cpp:

3:01 AM Changeset in webkit [59014] by Nikolas Zimmermann

4 edits in trunk/LayoutTests

2010-05-08 Nikolas Zimmermann <nzimmermann@rim.com>

Not reviewed. Rebaseline Win specific results after exposing JSSVGVKernElement/JSSVGHKernElement.

platform/win/fast/dom/Window/window-property-descriptors-expected.txt:
platform/win/fast/dom/prototype-inheritance-expected.txt:
platform/win/fast/js/global-constructors-expected.txt:

2:56 AM Changeset in webkit [59013] by krit@webkit.org

2 edits in trunk/WebCore

2010-05-08 Dirk Schulze <krit@webkit.org>

Uneviewed fix of Win bot

Added JS bindings to Win build system.

WebCore.vcproj/WebCore.vcproj:

2:54 AM Changeset in webkit [59012] by Nikolas Zimmermann

4 edits in trunk/LayoutTests

2010-05-08 Nikolas Zimmermann <nzimmermann@rim.com>

Not reviewed. Rebaseline GTK specific results after exposing JSSVGVKernElement/JSSVGHKernElement.

platform/gtk/fast/dom/Window/window-property-descriptors-expected.txt:
platform/gtk/fast/dom/prototype-inheritance-expected.txt:
platform/gtk/fast/js/global-constructors-expected.txt:

2:28 AM Changeset in webkit [59011] by krit@webkit.org

2 edits in trunk/WebCore

2010-05-08 Dirk Schulze <krit@webkit.org>

Unreviewed try to fix Win bot.

SVGFont's VKern implementation missing
https://bugs.webkit.org/show_bug.cgi?id=38663

bindings/scripts/CodeGeneratorJS.pm:

2:06 AM Changeset in webkit [59010] by krit@webkit.org

31 edits
7 adds in trunk

2010-05-08 Dirk Schulze <krit@webkit.org>

Reviewed by Nikolas Zimmermann.

SVGFont's VKern implementation missing
https://bugs.webkit.org/show_bug.cgi?id=38663

Implementation of vkern, needed by SVGFont. Also added DOM and JS bindings
for vkern and hkern. Made some clean-up in SVGParserUtilities and SVGFontElement.
SVGHKernElement and SVGVKernElement share most of the code now.

Test: svg/text/text-vkern.svg

Android.derived.jscbindings.mk:
Android.derived.v8bindings.mk:
Android.mk:
DerivedSources.make:
GNUmakefile.am:
WebCore.gyp/WebCore.gyp:
WebCore.gypi:
WebCore.pri:
WebCore.pro:
WebCore.vcproj/WebCore.vcproj:
WebCore.xcodeproj/project.pbxproj:
page/DOMWindow.idl:
rendering/SVGRootInlineBox.cpp: (WebCore::applySVGKerning): (WebCore::SVGRootInlineBox::buildLayoutInformationForTextBox):
svg/SVGAllInOne.cpp:
svg/SVGFontElement.cpp: (WebCore::SVGFontElement::invalidateGlyphCache): (WebCore::SVGFontElement::ensureGlyphCache): (WebCore::matches): (WebCore::kerningForPairOfStringsAndGlyphs): (WebCore::SVGFontElement::horizontalKerningForPairOfStringsAndGlyphs): (WebCore::SVGFontElement::verticalKerningForPairOfStringsAndGlyphs):
svg/SVGFontElement.h: (WebCore::SVGKerningPair::SVGKerningPair): (WebCore::SVGFontElement::rendererIsNeeded):
svg/SVGHKernElement.cpp: (WebCore::SVGHKernElement::buildHorizontalKerningPair):
svg/SVGHKernElement.h:
svg/SVGParserUtilities.cpp: (WebCore::parseGlyphName): (WebCore::parseKerningUnicodeString):
svg/SVGParserUtilities.h:
svg/SVGVKernElement.cpp: Added. (WebCore::SVGVKernElement::SVGVKernElement): (WebCore::SVGVKernElement::~SVGVKernElement): (WebCore::SVGVKernElement::insertedIntoDocument): (WebCore::SVGVKernElement::removedFromDocument): (WebCore::SVGVKernElement::buildVerticalKerningPair):
svg/SVGVKernElement.h: Added. (WebCore::SVGVKernElement::rendererIsNeeded):
svg/SVGVKernElement.idl: Added.
svg/svgtags.in:

2010-05-08 Dirk Schulze <krit@webkit.org>

Reviewed by Nikolas Zimmermann.

SVGFont's VKern implementation missing
https://bugs.webkit.org/show_bug.cgi?id=38663

The changes in DOMWindow.idl and the implementation of the DOM
and JS bindings of VKern and HKern made some updates necessary.
Added new test case for SVGFont's VKern.

fast/dom/Window/window-properties-expected.txt:
fast/dom/Window/window-property-descriptors-expected.txt:
fast/dom/prototype-inheritance-2-expected.txt:
fast/dom/prototype-inheritance-expected.txt:
fast/js/global-constructors-expected.txt:
platform/mac/svg/text/text-vkern-expected.checksum: Added.
platform/mac/svg/text/text-vkern-expected.png: Added.
platform/mac/svg/text/text-vkern-expected.txt: Added.
svg/custom/global-constructors-expected.txt:
svg/text/text-vkern.svg: Added.

2010-05-08 Dirk Schulze <krit@webkit.org>

Reviewed by Nikolas Zimmermann.

SVGFont's VKern implementation missing
https://bugs.webkit.org/show_bug.cgi?id=38663

projects/svg/status.xml:

1:44 AM Changeset in webkit [59009] by jer.noble@apple.com

2 edits in trunk/WebCore

Unreviewed, just fixing windows build.

platform/graphics/win/MediaPlayerPrivateQuickTimeVisualContext.h: Add #if USE(ACCELERATED_COMPOSITING) guards around GraphicsLayer areas of the code. Include GraphicsLayer.h directly.
platform/graphics/win/MediaPlayerPrivateQuickTimeVisualContext.cpp: As above.
platform/graphics/win/QTMovieVisualContext.cpp: Remove the #include of d3d9types.h.

1:26 AM Changeset in webkit [59008] by jer.noble@apple.com

2 edits in trunk/WebCore/platform/graphics/win

Fix the windows build (by adding #if USE(ACCELERATED_COMPOSITING) guards).

12:38 AM Changeset in webkit [59007] by jer.noble@apple.com

1 edit in trunk/WebCore/platform/graphics/win/MediaPlayerPrivateQuickTimeVisualContext.h

Fix the windows build.

12:24 AM Changeset in webkit [59006] by eric@webkit.org

6 edits in trunk/WebKitTools

2010-05-08 Eric Seidel <eric@webkit.org>

Reviewed by Daniel Bates.

test-webkitpy fails under cygwin
https://bugs.webkit.org/show_bug.cgi?id=38718

Scripts/webkitpy/common/system/executive.py:
- Add _KILL_PROCESS_KILLED_PROCESS_EXIT_CODE and _KILL_ALL_KILLED_PROCESS_EXIT_CODE to store the expected exit codes of processes killed by kill_process and kill_all. These two constants are only used by the unit tests but are stored in executive.py so they can be right next to the platform ifs.
- Remove unnecessary str() conversion, run_command does that for us.
- Make os.kill retry on cygwin on EAGAIN. It's unclear why CYGWIN throws EAGAIN, but it only does so sometimes. 3 may not be enough retries, but we'll try it to start with.
- Add _windows_image_name to automatically convert "yes" to "yes.exe" for use with taskkill.exe /im. Various callers to kill_all could be updated to remove the .exe, but that can be done in another patch.
- Use taskkill.exe for killall on cygwin.
Scripts/webkitpy/common/system/executive_unittest.py:
- Use the new *_KILLED_PROCESS_EXIT_CODE constants which are correctly set to 0 on windows/cygwin systems where taskkill.exe is used.
- Test _windows_image_name
Scripts/webkitpy/layout_tests/port/chromium_win.py:
- Add FIXME about including mac-snowleopard in baseline_search_path.
Scripts/webkitpy/layout_tests/port/webkit.py:
- Make default_configuration actually read from the Configuration file.
Scripts/webkitpy/layout_tests/port/webkit_unittest.py: Added.
- Test default_configuration
Scripts/webkitpy/layout_tests/port/win.py:
- Need a basic baseline_search_path if --platform dryrun is to work.

12:14 AM Changeset in webkit [59005] by eric@webkit.org

25 edits
3 adds
1 delete in trunk

2010-05-08 Eric Seidel <eric@webkit.org>

Unreviewed, just reverting commit.

REGRESSION(59000): r59000 contained all sorts of changes it should not have, needs revert.
https://bugs.webkit.org/show_bug.cgi?id=38798

bytecompiler/BytecodeGenerator.cpp: (JSC::BytecodeGenerator::emitResolve):
jit/JITOpcodes.cpp: (JSC::JIT::emit_op_resolve_global):

2010-05-08 Eric Seidel <eric@webkit.org>

Unreviewed, just reverting commit.

REGRESSION(59000): r59000 contained all sorts of changes it should not have, needs revert.
https://bugs.webkit.org/show_bug.cgi?id=38798

fast/js/global-resolve-through-eval-expected.txt: Added.
fast/js/global-resolve-through-eval.html: Added.
fast/js/script-tests/global-resolve-through-eval.js: Added. (accessGlobal): (accessLocal):
platform/win/Skipped:

2010-05-08 Eric Seidel <eric@webkit.org>

Unreviewed, just reverting commit.

REGRESSION(59000): r59000 contained all sorts of changes it should not have, needs revert.
https://bugs.webkit.org/show_bug.cgi?id=38798

Test: fast/js/global-resolve-through-eval.html

bindings/js/JSCallbackData.h: (WebCore::JSCallbackData::JSCallbackData): (WebCore::JSCallbackData::~JSCallbackData): (WebCore::DeleteCallbackDataTask::create): (WebCore::DeleteCallbackDataTask::performTask): (WebCore::DeleteCallbackDataTask::isCleanupTask): (WebCore::DeleteCallbackDataTask::DeleteCallbackDataTask):
bindings/js/JSCustomVoidCallback.cpp: (WebCore::JSCustomVoidCallback::JSCustomVoidCallback): (WebCore::JSCustomVoidCallback::~JSCustomVoidCallback):
bindings/js/JSCustomVoidCallback.h:
bindings/scripts/CodeGeneratorJS.pm:
bindings/scripts/test/JS/JSTestInterface.cpp: (WebCore::):
bindings/scripts/test/JS/JSTestObj.cpp: (WebCore::):
workers/WorkerThread.cpp: (WebCore::WorkerThreadShutdownFinishTask::performTask): (WebCore::WorkerThreadShutdownStartTask::performTask):

2010-05-08 Eric Seidel <eric@webkit.org>

Unreviewed, just reverting commit.

REGRESSION(59000): r59000 contained all sorts of changes it should not have, needs revert.
https://bugs.webkit.org/show_bug.cgi?id=38798

Scripts/webkitpy/common/system/executive.py:
Scripts/webkitpy/common/system/executive_unittest.py:
Scripts/webkitpy/layout_tests/port/chromium_linux.py:
Scripts/webkitpy/layout_tests/port/chromium_mac.py:
Scripts/webkitpy/layout_tests/port/chromium_win.py:
Scripts/webkitpy/layout_tests/port/mac.py:
Scripts/webkitpy/layout_tests/port/webkit.py:
Scripts/webkitpy/layout_tests/port/webkit_unittest.py: Removed.
Scripts/webkitpy/layout_tests/port/win.py:
Scripts/webkitpy/layout_tests/run_webkit_tests_unittest.py:
Scripts/webkitpy/tool/commands/download.py:
Scripts/webkitpy/tool/commands/download_unittest.py:

12:08 AM Changeset in webkit [59004] by Simon Hausmann

2 edits in trunk/WebKit/qt

[Qt] Unreviewed, QtWebKit versioning for the trunk.

The next version is going to be 2.1, based off the trunk.

Api/qwebkitglobal.h:

May 7, 2010:

11:56 PM Changeset in webkit [59003] by jer.noble@apple.com

1 edit in trunk/WebCore/platform/graphics/win/QTMovieVisualContext.cpp

Fix the windows build.

11:27 PM Changeset in webkit [59002] by zherczeg@webkit.org

2 edits in trunk/WebCore

Fixed the inverse function of f(x) = ((x + 0.055) / 1.055) ^{2.4

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

Reviewed by Dirk Schulze

It is ((x ^{(1 / 2.4)) * 1.055) - 0.055}

Chromium pixel tests cover the issue.

platform/graphics/ImageBuffer.cpp:

(WebCore::ImageBuffer::transformColorSpace):

11:24 PM Changeset in webkit [59001] by jer.noble@apple.com

14 edits
4 copies
2 moves
10 adds in trunk

JavaScriptCore: Safari pegs CPU and drops tons of frames using HTML5 Vimeo player
<https://bugs.webkit.org/show_bug.cgi?id=34005>
<rdar://problem/7569713>

Reviewed by Maciej Stachowiak

Added cancelCallOnMainThread. callOnMainThread should always now be paired
with cancelCallOnMainThread in situations where the refcon passed to callOnMainThread
may be dealloced before the main thread function can be dispatched.

wtf/MainThread.cpp:

(WTF::FunctionWithContext::operator == ): Supports the FunctionWithContextFinder predicate functor.
(WTF::FunctionWithContextFinder::FunctionWithContextFinder): Predicate functor for use with Dequeue::findIf
(WTF::FunctionWithContextFinder::operator()):
(WTF::cancelCallOnMainThread):

wtf/MainThread.h:

WebCore: Safari pegs CPU and drops tons of frames using HTML5 Vimeo player
<https://bugs.webkit.org/show_bug.cgi?id=34005>
<rdar://problem/7569713>

Reviewed by Eric Carlson

The original functionality of QTMovieWin has been split between QTMovieGWorld,
and a new class: QTMovie. QTMovie contains all the "controller" parts (changing
the rate, seeking, etc) while QTMovieGWorld retains all the drawing code. QTMovieGWorld
now takes a QTMovie, and as such QTMovie is now retainable. QTMovieGWorld registers
itself as a client of QTMovie, so that it can receive load-state notifications,
and thus QTMovie must now support multiple clients. Movie tasking timer support
has been moved into its own class (QTMovieTask) and will be addressed in a future
patch. Most of the functions listed below only changed so much as their class name changed.

platform/graphics/win/QTMovie.cpp: Copied from WebCore/platform/graphics/win/QTMovieWin.cpp.

(QTMoviePrivate::QTMoviePrivate):
(QTMoviePrivate::~QTMoviePrivate):
(QTMoviePrivate::startTask):
(QTMoviePrivate::endTask):
(QTMoviePrivate::task):
(QTMoviePrivate::createMovieController):
(QTMovie::QTMovie):
(QTMovie::~QTMovie):
(QTMovie::addClient):
(QTMovie::removeClient):
(QTMovie::play):
(QTMovie::pause):
(QTMovie::rate):
(QTMovie::setRate):
(QTMovie::duration):
(QTMovie::currentTime):
(QTMovie::setCurrentTime):
(QTMovie::setVolume):
(QTMovie::setPreservesPitch):
(QTMovie::dataSize):
(QTMovie::maxTimeLoaded):
(QTMovie::loadState):
(QTMovie::getNaturalSize):
(QTMovie::load):
(QTMovie::disableUnsupportedTracks):
(QTMovie::isDisabled):
(QTMovie::setDisabled):
(QTMovie::hasVideo):
(QTMovie::hasAudio):
(QTMovie::hasClosedCaptions):
(QTMovie::setClosedCaptionsVisible):
(QTMovie::countSupportedTypes):
(QTMovie::getSupportedType):
(QTMovie::initializeQuickTime):
(QTMovie::getMovieHandle):

platform/graphics/win/QTMovie.h: Copied from WebCore/platform/graphics/win/QTMovieWin.h.
platform/graphics/win/QTMovieGWorld.cpp: Copied from WebCore/platform/graphics/win/QTMovieWin.cpp.

(QTMovieGWorldPrivate::QTMovieGWorldPrivate):
(QTMovieGWorldPrivate::~QTMovieGWorldPrivate):
(QTMovieGWorldPrivate::cacheMovieScale):
(movieDrawingCompleteProc):
(QTMovieGWorldPrivate::registerDrawingCallback):
(QTMovieGWorldPrivate::unregisterDrawingCallback):
(QTMovieGWorldPrivate::drawingComplete):
(QTMovieGWorldPrivate::updateGWorld):
(QTMovieGWorldPrivate::createGWorld):
(QTMovieGWorldPrivate::clearGWorld):
(QTMovieGWorldPrivate::setSize):
(QTMovieGWorldPrivate::updateMovieSize):
(QTMovieGWorldPrivate::deleteGWorld):
(QTMovieGWorldPrivate::movieEnded):
(QTMovieGWorldPrivate::movieLoadStateChanged):
(QTMovieGWorldPrivate::movieTimeChanged):
(QTMovieGWorld::QTMovieGWorld):
(QTMovieGWorld::~QTMovieGWorld):
(QTMovieGWorld::setSize):
(QTMovieGWorld::setVisible):
(QTMovieGWorld::getCurrentFrameInfo):
(QTMovieGWorld::paint):
(QTMovieGWorld::setDisabled):
(QTMovieGWorld::isDisabled):
(QTMovieGWorld::fullscreenWndProc):
(QTMovieGWorld::enterFullscreen):
(QTMovieGWorld::exitFullscreen):
(QTMovieGWorld::setMovie):
(QTMovieGWorld::movie):

platform/graphics/win/QTMovieGWorld.h: Copied from WebCore/platform/graphics/win/QTMovieWin.h.
platform/graphics/win/QTMovieTask.cpp: Added.

(QTMovieTask::QTMovieTask):
(QTMovieTask::~QTMovieTask):
(QTMovieTask::sharedTask):
(QTMovieTask::updateTaskTimer):
(QTMovieTask::fireTaskClients):
(QTMovieTask::addTaskClient):
(QTMovieTask::removeTaskClient):
(QTMovieTask::setTaskTimerFuncs):

platform/graphics/win/QTMovieTask.h: Added.

WebKit/win: Safari pegs CPU and drops tons of frames using HTML5 Vimeo player
https://bugs.webkit.org/show_bug.cgi?id=34005

Reviewed by Adele Peterson.

QTMovieWin is now QTMovieGWorld.

FullscreenVideoController.cpp:

(FullscreenVideoController::movie):

FullscreenVideoController.h:

11:10 PM Changeset in webkit [59000] by eric@webkit.org

25 edits
1 add
3 deletes in trunk

2010-05-06 Eric Seidel <eric@webkit.org>

Reviewed by Daniel Bates.

test-webkitpy fails under cygwin
https://bugs.webkit.org/show_bug.cgi?id=38718

Scripts/webkitpy/common/system/executive.py:
- Add _KILL_PROCESS_KILLED_PROCESS_EXIT_CODE and _KILL_ALL_KILLED_PROCESS_EXIT_CODE to store the expected exit codes of processes killed by kill_process and kill_all. These two constants are only used by the unit tests but are stored in executive.py so they can be right next to the platform ifs.
- Remove unnecessary str() conversion, run_command does that for us.
- Make os.kill retry on cygwin on EAGAIN. It's unclear why CYGWIN throws EAGAIN, but it only does so sometimes. 3 may not be enough retries, but we'll try it to start with.
- Add _windows_image_name to automatically convert "yes" to "yes.exe" for use with taskkill.exe /im. Various callers to kill_all could be updated to remove the .exe, but that can be done in another patch.
- Use taskkill.exe for killall on cygwin.
Scripts/webkitpy/common/system/executive_unittest.py:
- Use the new *_KILLED_PROCESS_EXIT_CODE constants which are correctly set to 0 on windows/cygwin systems where taskkill.exe is used.
- Test _windows_image_name
Scripts/webkitpy/layout_tests/port/chromium_win.py:
- Add FIXME about including mac-snowleopard in baseline_search_path.
Scripts/webkitpy/layout_tests/port/webkit.py:
- Make default_configuration actually read from the Configuration file.
Scripts/webkitpy/layout_tests/port/webkit_unittest.py: Added.
- Test default_configuration
Scripts/webkitpy/layout_tests/port/win.py:
- Need a basic baseline_search_path if --platform dryrun is to work.

11:03 PM Changeset in webkit [58999] by eric@webkit.org

2 edits in trunk/WebKitTools

2010-05-07 Eric Seidel <eric@webkit.org>

Reviewed by Nikolas Zimmermann.

DryrunTest fails on every platform other than mac
https://bugs.webkit.org/show_bug.cgi?id=38796

The test uses the port detection logic to find a suitable
port to use results from. However that detection logic assumes
chromium on linux, which requires a chromium checkout which the
bots don't have. The test is broken and we'll need to fix it.
For now I'm just going to disable the test on all platforms besides mac.

Scripts/webkitpy/layout_tests/run_webkit_tests_unittest.py:

11:01 PM Changeset in webkit [58998] by eric@webkit.org

6 edits in trunk/WebCore

2010-05-07 Eric Uhrhane <ericu@chromium.org>

Reviewed by Dmitry Titov.

JSCallbackData is deleted on the Main thread, not the Context thread.
https://bugs.webkit.org/show_bug.cgi?id=38623

No new tests; you'll only see a problem after the worker-access-to-DB
code goes in. Without this fix, the GTK bots assert, but it's
timing-sensitive and not truly platform-specific.

bindings/js/JSCallbackData.h: (WebCore::JSCallbackData::JSCallbackData): (WebCore::JSCallbackData::~JSCallbackData): Store the allocating thread, then assert that we're on it at deletion.

(WebCore::DeleteCallbackDataTask):
This is the task that can get posted to the context thread for cleanup.

bindings/js/JSCustomVoidCallback.cpp:
bindings/js/JSCustomVoidCallback.h: This callback isn't autogenerated, so we have to do the cleanup manually.

bindings/scripts/CodeGeneratorJS.pm: Add the proper cleanup to all autogenerated callbacks.

workers/WorkerThread.cpp: (WebCore::WorkerThreadShutdownFinishTask::performTask): (WebCore::WorkerThreadShutdownStartTask::performTask): WorkerContext::clearScript can't be called until the posted deletion tasks have completed; move the call from shutdown start to shutdown finish.

10:51 PM Changeset in webkit [58997] by Nikolas Zimmermann

2 edits in trunk/LayoutTests

2010-05-07 Nikolas Zimmermann <nzimmermann@rim.com>

Not reviewed. Attempt to turn Windows bots green again.

Skip two failing tests:

https://bugs.webkit.org/show_bug.cgi?id=38794
fast/css/display-none-inline-style-change-crash.html (fails on Windows Release bot)

https://bugs.webkit.org/show_bug.cgi?id=38795
inspector/timeline-paint.html

platform/win/Skipped:

10:35 PM Changeset in webkit [58996] by eric@webkit.org

5 edits in trunk/WebKitTools

2010-05-07 Eric Seidel <eric@webkit.org>

Reviewed by Daniel Bates.

Clean up baseline_search_path to use map to reduce copy/paste code
https://bugs.webkit.org/show_bug.cgi?id=38792

Reading which portnames a port falls back to is easier if
we convert port names to paths with map instead of using copy/paste code.

Scripts/webkitpy/layout_tests/port/chromium_linux.py:
Scripts/webkitpy/layout_tests/port/chromium_mac.py:
Scripts/webkitpy/layout_tests/port/chromium_win.py:
Scripts/webkitpy/layout_tests/port/mac.py:

9:17 PM Changeset in webkit [58995] by eric@webkit.org

3 edits in trunk/JavaScriptCore

2010-05-07 Eric Seidel <eric@webkit.org>

Unreviewed, rolling out r58990.
http://trac.webkit.org/changeset/58990

Broke 4 tests on Snow Leopard and Qt.

jit/JITStubs.cpp: (JSC::DEFINE_STUB_FUNCTION):
runtime/JSCell.h:

9:00 PM Changeset in webkit [58994] by eric@webkit.org

3 edits in trunk/WebKitTools

2010-05-07 Eric Seidel <eric@webkit.org>

Reviewed by Daniel Bates.

rollout commands fail when commit is missing bug number
https://bugs.webkit.org/show_bug.cgi?id=38791

Scripts/webkitpy/tool/commands/download.py:
- _commit_info failed to actually return the CommitInfo in the no-bug-id case.
Scripts/webkitpy/tool/commands/download_unittest.py:
- Test that the fix worked.

8:19 PM Changeset in webkit [58993] by oliver@apple.com

3 edits
3 adds in trunk

Optimize access to the global object from a function that uses eval
https://bugs.webkit.org/show_bug.cgi?id=38644

Reviewed by Gavin Barraclough.

JavaScriptCore:

Fix bug where cross scope access to a global var (vs. property) would
be allowed without checking for intervening dynamic scopes.

bytecompiler/BytecodeGenerator.cpp:

(JSC::BytecodeGenerator::emitResolve):

LayoutTests:

Add test case to ensure we don't incorrectly allow dynamic scopes
to be skipped when doing direct var access.

fast/js/global-resolve-through-eval-expected.txt: Added.
fast/js/global-resolve-through-eval.html: Added.
fast/js/script-tests/global-resolve-through-eval.js: Added.

(accessGlobal):
(accessLocal):

8:16 PM Changeset in webkit [58992] by eric@webkit.org

4 edits in trunk/WebCore

2010-05-07 Eric Seidel <eric@webkit.org>

Reviewed by Adam Barth.

Fix JSC binding generation to match webkit style
https://bugs.webkit.org/show_bug.cgi?id=38173

Add spaces around "|" to match webkit style.

bindings/scripts/CodeGeneratorJS.pm:
bindings/scripts/test/JS/JSTestObj.cpp:

7:49 PM Changeset in webkit [58991] by oliver@apple.com

2 edits in trunk/JavaScriptCore

Macro expansion I stab at thee!

32bit test fix

7:18 PM Changeset in webkit [58990] by ggaren@apple.com

3 edits in trunk/JavaScriptCore

Optimized o[s] where o is a cell and s is a string, removing some old
code that wasn't really tuned for the JIT.

Reviewed by Darin Adler.

SunSpider says 0.8% faster.

jit/JITStubs.cpp:

(JSC::DEFINE_STUB_FUNCTION):

runtime/JSCell.h:

6:27 PM Changeset in webkit [58989] by dumi@chromium.org

25 edits
8 adds in trunk/WebCore

Adding the IDL files for the sync DB API and the stubs for the JS and V8 bindings.
https://bugs.webkit.org/show_bug.cgi?id=34994

Reviewed by Brady Eidson.

DerivedSources.cpp:
DerivedSources.make:
GNUmakefile.am:
WebCore.gypi:
WebCore.pri:
WebCore.pro:
WebCore.vcproj/WebCore.vcproj:
WebCore.xcodeproj/project.pbxproj:
bindings/js/JSBindingsAllInOne.cpp:
bindings/js/JSDatabaseSyncCustom.cpp: Added.

(WebCore::JSDatabaseSync::changeVersion):
(WebCore::createTransaction):
(WebCore::JSDatabaseSync::transaction):
(WebCore::JSDatabaseSync::readTransaction):

bindings/js/JSSQLTransactionSyncCustom.cpp: Added.

(WebCore::JSSQLTransactionSync::executeSql):

bindings/js/JSWorkerContextCustom.cpp:

(WebCore::JSWorkerContext::openDatabaseSync):

bindings/v8/custom/V8BindingMacros.h: Added.
bindings/v8/custom/V8DatabaseSyncCustom.cpp: Added.

(WebCore::V8DatabaseSync::changeVersionCallback):
(WebCore::createTransaction):
(WebCore::V8DatabaseSync::transactionCallback):
(WebCore::V8DatabaseSync::readTransactionCallback):

bindings/v8/custom/V8SQLTransactionSyncCustom.cpp: Added.

(WebCore::V8SQLTransactionSync::executeSqlCallback):

bindings/v8/custom/V8WorkerContextCustom.cpp:

(WebCore::V8WorkerContext::openDatabaseSyncCallback):

storage/Database.idl:
storage/DatabaseCallback.h:
storage/DatabaseCallback.idl:
storage/DatabaseSync.cpp:

(WebCore::DatabaseSync::changeVersion):
(WebCore::DatabaseSync::transaction):

storage/DatabaseSync.h:
storage/DatabaseSync.idl: Added.
storage/SQLError.idl:
storage/SQLResultSet.idl:
storage/SQLResultSetRowList.idl:
storage/SQLTransaction.idl:
storage/SQLTransactionSync.idl: Added.
storage/SQLTransactionSyncCallback.h:
storage/SQLTransactionSyncCallback.idl: Added.
workers/WorkerContext.cpp:

(WebCore::WorkerContext::openDatabaseSync):

workers/WorkerContext.h:
workers/WorkerContext.idl:

6:05 PM Changeset in webkit [58988] by oliver@apple.com

2 edits in trunk/JavaScriptCore

Windows build fix

5:51 PM Changeset in webkit [58987] by oliver@apple.com

2 edits in trunk/JavaScriptCore

Windows build fix.

5:05 PM Changeset in webkit [58986] by oliver@apple.com

20 edits in trunk/JavaScriptCore

2010-05-07 Oliver Hunt <oliver@apple.com>

Reviewed by Geoffrey Garen.

Optimize access to the global object from a function that uses eval
https://bugs.webkit.org/show_bug.cgi?id=38644

Add op_resolve_global_dynamic, a variant of op_resolve_global that
checks each node in the scope chain for dynamically inserted properties
and falls back to the normal resolve logic in that case.

JavaScriptCore.exp:
bytecode/CodeBlock.cpp: (JSC::isGlobalResolve): (JSC::CodeBlock::printStructures): (JSC::CodeBlock::dump): (JSC::CodeBlock::derefStructures):
bytecode/Opcode.h:
bytecompiler/BytecodeGenerator.cpp: (JSC::BytecodeGenerator::findScopedProperty):

Now take an additional reference parameter to used to indicate that
there were nodes that may gain dynamic properties

(JSC::BytecodeGenerator::emitResolve):
(JSC::BytecodeGenerator::emitResolveBase):
(JSC::BytecodeGenerator::emitResolveWithBase):

deal with additional argument to findScopedProperty

bytecompiler/BytecodeGenerator.h:
bytecompiler/NodesCodegen.cpp: (JSC::FunctionCallResolveNode::emitBytecode): (JSC::PostfixResolveNode::emitBytecode): (JSC::PrefixResolveNode::emitBytecode): (JSC::ReadModifyResolveNode::emitBytecode): (JSC::AssignResolveNode::emitBytecode):

These functions use findScopedProperty directly in order to
optimise lookup. They cannot trivially handle any degree of
dynamism in the lookup so we just give up in such case.

interpreter/Interpreter.cpp: (JSC::Interpreter::resolveGlobalDynamic): (JSC::Interpreter::execute): (JSC::Interpreter::privateExecute):
interpreter/Interpreter.h:
jit/JIT.cpp: (JSC::JIT::privateCompileMainPass): (JSC::JIT::privateCompileSlowCases):
jit/JIT.h:
jit/JITOpcodes.cpp: (JSC::JIT::emit_op_resolve_global): (JSC::JIT::emit_op_resolve_global_dynamic): (JSC::JIT::emitSlow_op_resolve_global): (JSC::JIT::emitSlow_op_resolve_global_dynamic):

Happily resolve_global_dynamic can share the slow case!

jit/JITStubs.h: (JSC::):
runtime/JSActivation.cpp: (JSC::JSActivation::isDynamicScope):
runtime/JSActivation.h:
runtime/JSGlobalObject.cpp: (JSC::JSGlobalObject::isDynamicScope):
runtime/JSGlobalObject.h:
runtime/JSStaticScopeObject.cpp: (JSC::JSStaticScopeObject::isDynamicScope):
runtime/JSStaticScopeObject.h:
runtime/JSVariableObject.h:

4:53 PM Changeset in webkit [58985] by darin@chromium.org

2 edits in trunk/WebKit/chromium

2010-05-07 Darin Fisher <darin@chromium.org>

Fix build bustage. Update to Chromium code that avoids a bad
dependency from net/ on chrome/.

DEPS:

4:51 PM Changeset in webkit [58984] by dpranke@chromium.org

3 edits in trunk/LayoutTests

2010-05-06 Dirk Pranke <dpranke@chromium.org>

Reviewed by Alexey Proskuryakov.

Rewrite test to fail faster if the inner frame doesn't load at all
for some reason (previous version would raise a TypeError before
the timeout was set).

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

http/tests/security/sandbox-inherit-to-initial-document-2.html:
platform/chromium/test_expectations.txt:

4:35 PM Changeset in webkit [58983] by Beth Dakin

2 edits in trunk/WebCore

Fix for <rdar://problem/7956972> REGRESSION: Can't focus and type
in GMail due to bad repainting
-and corresponding-
https://bugs.webkit.org/show_bug.cgi?id=38782

Reviewed by Simon Fraser.

This patch reverts back to pre-r58797 behavior when
shouldPropagateCompositingToIFrameParent() is false.

rendering/RenderLayerBacking.cpp:

(WebCore::RenderLayerBacking::paintingGoesToWindow):

4:23 PM Changeset in webkit [58982] by andersca@apple.com

8 edits in trunk/WebKit/mac

<rdar://problem/7947356>
QT Plug-in in hardware-accelerated WebKit is missing the controller

Reviewed by Sam Weinig and Simon Fraser.

Replace the useSoftwareRenderer boolean with an enum that lets the plug-in host opt into using a layer
backed NSView instead of inserting the layer into the WebCore layer hierarchy.

Plugins/Hosted/NetscapePluginHostManager.mm:

(WebKit::NetscapePluginHostManager::instantiatePlugin):

Plugins/Hosted/NetscapePluginHostProxy.mm:

(WKPCInstantiatePluginReply):

Plugins/Hosted/NetscapePluginInstanceProxy.h:

(WebKit::NetscapePluginInstanceProxy::rendererType):
(WebKit::NetscapePluginInstanceProxy::setRendererType):
(WebKit::NetscapePluginInstanceProxy::InstantiatePluginReply::InstantiatePluginReply):

Plugins/Hosted/NetscapePluginInstanceProxy.mm:

(WebKit::NetscapePluginInstanceProxy::NetscapePluginInstanceProxy):
Replace the useSoftwareRenderer boolean everywhere with the RendererType enum.

Plugins/Hosted/WebHostedNetscapePluginView.mm:

(-[WebHostedNetscapePluginView createPlugin]):
Only go into full compositing mode when the renderer type returned is UseAcceleratedCompositing.
Otherwise, use a layer backed NSView.

Plugins/Hosted/WebKitPluginHostTypes.h:

Add a RendererType enum.

4:15 PM Changeset in webkit [58981] by darin@chromium.org

2 edits in trunk/WebKit/chromium

2010-05-07 Darin Fisher <darin@chromium.org>

Fix build bustage. Update to Chromium code that no longer uses
deprecated methods.

DEPS:

4:10 PM Changeset in webkit [58980] by crogers@google.com

1 edit in branches/audio/WebCore/audio/AudioBasicProcessorNode.cpp

Pull more AudioBasicProcessorNode::process() code inside the lock

3:55 PM Changeset in webkit [58979] by crogers@google.com

2 edits in branches/audio/WebCore/audio

Fix AudioBasicProcessorNode::inputHasBeenConnected() to allow dynamic re-initialization.

this requires a lock around process()
also fix uninitialize() method

3:55 PM Changeset in webkit [58978] by crogers@google.com

2 edits in branches/audio/WebCore/audio

Initial rough implementation for buffer source gain - later to be more optimized

3:55 PM Changeset in webkit [58977] by crogers@google.com

1 edit in branches/audio/WebCore/audio/AudioResampler.cpp

More descriptive debugging message for resampler channel mismatch

3:55 PM Changeset in webkit [58976] by crogers@google.com

2 edits in branches/audio/WebCore/audio

Add temporary mono and stereo processing busses for use by any processing node.

also change local variables from PassRefPtr to RefPtr

3:50 PM Changeset in webkit [58975] by darin@chromium.org

2 edits in trunk/WebKitTools

2010-05-07 Darin Fisher <darin@chromium.org>

Fix build bustage: toElement<T> should be to<T>.

DumpRenderTree/chromium/LayoutTestController.cpp: (LayoutTestController::elementDoesAutoCompleteForElementWithId):

3:18 PM Changeset in webkit [58974] by ggaren@apple.com

2 edits in trunk/JavaScriptCore

Fixed Sputnik failure seen on buildbot.

Reviewed by Gavin Barraclough.

runtime/NumberPrototype.cpp:

(JSC::numberProtoFuncToString): Cast to unsigned before checking magnitude,
to efficiently exclude negative numbers, which require two characters
instead of one.

2:44 PM Changeset in webkit [58973] by johnnyg@google.com

2 edits in trunk/WebCore

2010-05-07 Sheriff Bot <webkit.review.bot@gmail.com>

Unreviewed, rolling out r58963.
http://trac.webkit.org/changeset/58963
https://bugs.webkit.org/show_bug.cgi?id=38773

Broke Chromium layout tests. (Requested by dcheng on #webkit).

platform/chromium/DragDataChromium.cpp: (WebCore::DragData::asURL):

2:18 PM Changeset in webkit [58972] by johnnyg@google.com

2 edits
6 adds in trunk/LayoutTests

2010-05-07 John Gregg <johnnyg@google.com>

Unreviewed [re-]baseline of chromium tests.

platform/chromium-linux/fast/frames/take-focus-from-iframe-expected.checksum: Added.
platform/chromium-linux/fast/frames/take-focus-from-iframe-expected.png: Added.
platform/chromium-mac/fast/frames/take-focus-from-iframe-expected.checksum: Added.
platform/chromium-win/fast/frames/take-focus-from-iframe-expected.checksum: Added.
platform/chromium-win/fast/frames/take-focus-from-iframe-expected.png: Added.
platform/chromium-win/fast/frames/take-focus-from-iframe-expected.txt: Added.
platform/chromium-win/svg/hixie/error/017-expected.txt:

1:28 PM Changeset in webkit [58971] by Martin Robinson

2 edits in trunk/LayoutTests

2010-05-07 Martin Robinson <mrobinson@igalia.com>

Reviewed by Gustavo Noronha Silva.

Adjust the position of the comment on the WOFF test and
link to the bug report. This should fix a spurious error
while running layout tests.

platform/gtk/Skipped:

1:13 PM Changeset in webkit [58970] by ggaren@apple.com

2 edits in trunk/JavaScriptCore

Slightly more effective way to guarantee a compile-time constant, at
least on gcc.

Reviewed by Alexey Proskuryakov.

runtime/NumberPrototype.cpp:

(JSC::numberProtoFuncToString): Use pointer notation instead of array
notation, so the compiler treats the string and its contents as constant.

1:13 PM Changeset in webkit [58969] by krit@webkit.org

1 edit
1 add in trunk/LayoutTests

2010-05-07 Dirk Schulze <krit@webkit.org>

Unreviewed fix of Windows test.

Windows need an platform dependent DRT result for clip-path-referencing-use2.svg,
Win doesn't have console output during DRT.

platform/win/svg/custom/clip-path-referencing-use2-expected.txt: Added.

1:03 PM Changeset in webkit [58968] by Martin Robinson

2 edits in trunk/WebCore

2010-05-07 Martin Robinson <mrobinson@igalia.com>

Reviewed by Gustavo Noronha Silva.

https://bugs.webkit.org/show_bug.cgi?id=38769
ClipboardGtk should write the URL label in the text portion of the DataObject

No tests as this code path is currently unused in the GTK+ port.

platform/gtk/ClipboardGtk.cpp: (WebCore::ClipboardGtk::writeURL): Write the label to the text portion of the DataObject, instead of the URL.

12:57 PM Changeset in webkit [58967] by darin@chromium.org

6 edits in trunk/WebKit/chromium

2010-05-07 Darin Fisher <darin@chromium.org>

Reviewed by Dimitri Glazkov.

[chromium] clear out more deprecated methods that are no longer used
https://bugs.webkit.org/show_bug.cgi?id=38702

public/WebFrame.h:
public/WebNode.h:
src/WebFrameImpl.cpp:
src/WebFrameImpl.h:
src/WebNode.cpp:

12:56 PM Changeset in webkit [58966] by abarth@webkit.org

2 edits
1 add in trunk/JavaScriptCore

2010-05-07 Adam Barth <abarth@webkit.org>

Reviewed by Alexey Proskuryakov.

[WTFURL] Add core URL parser
https://bugs.webkit.org/show_bug.cgi?id=38572

This patch adds the core of the URL parser. The URL parser uses a
templated notion of a code unit to support different string types.
Later we'll add some non-templated APIs with concrete types that
clients can use more conveniently.

The URLParser has a couple calls to ASSERT(), which I've commented out
for now. I'm not 100% sure how to handle the dependency issues there
yet, so I've punted on the issue for now.

Also, there are a number of methods that are declared public in this
patch that are only used by the canonicalizer. My plan is to make
those private or protected and make the canonicalizer a friend of the
parser. The details will be in a later patch.

wtf/url/src/URLComponent.h: (WTF::URLComponent::fromRange): (WTF::URLComponent::isNonEmpty): (WTF::URLComponent::isEmptyOrInvalid):
wtf/url/src/URLParser.h: Added. (WTF::URLParser::): (WTF::URLParser::isPossibleAuthorityTerminator): (WTF::URLParser::parseAuthority): (WTF::URLParser::extractScheme): (WTF::URLParser::parseAfterScheme): (WTF::URLParser::parseStandardURL): (WTF::URLParser::parsePath): (WTF::URLParser::parsePathURL): (WTF::URLParser::parseMailtoURL): (WTF::URLParser::parsePort): (WTF::URLParser::extractFileName): (WTF::URLParser::extractQueryKeyValue): (WTF::URLParser::isURLSlash): (WTF::URLParser::shouldTrimFromURL): (WTF::URLParser::trimURL): (WTF::URLParser::consecutiveSlashes): (WTF::URLParser::isPortDigit): (WTF::URLParser::nextAuthorityTerminator): (WTF::URLParser::parseUserInfo): (WTF::URLParser::parseServerInfo):

12:35 PM Changeset in webkit [58965] by Nikolas Zimmermann

10 edits in trunk

2010-05-07 Nikolas Zimmermann <nzimmermann@rim.com>

Reviewed by Dirk Schulze.

RenderSVGResourceClipper assigns a temporary mutated RenderStyle but does not correctly preserve the old style
https://bugs.webkit.org/show_bug.cgi?id=38767

Fix small logic error leading to a problem in RenderSVGResourceClipper. It assigns a new temporary RenderStyle
to the target object, but fails to reset it correctly to the old style. Fixes all svg/clip-path errors on the windows bots.

rendering/RenderSVGResourceClipper.cpp: (WebCore::RenderSVGResourceClipper::createClipData):

2010-05-07 Nikolas Zimmermann <nzimmermann@rim.com>

Reviewed by Dirk Schulze.

RenderSVGResourceClipper assigns a temporary mutated RenderStyle but does not correctly preserve the old style
https://bugs.webkit.org/show_bug.cgi?id=38767

Update baseline after fixing a logic error in RenderSVGResourceClipper.

platform/mac/svg/clip-path/clip-path-child-clipped-expected.txt:
platform/mac/svg/clip-path/clip-path-childs-clipped-expected.txt:
platform/mac/svg/clip-path/clip-path-clipped-expected.txt:
platform/mac/svg/clip-path/clip-path-evenodd-expected.txt:
platform/mac/svg/clip-path/clip-path-evenodd-nonzero-expected.txt:
platform/mac/svg/clip-path/clip-path-nonzero-evenodd-expected.txt:
platform/mac/svg/clip-path/clip-path-text-and-shape-expected.txt:

12:30 PM Changeset in webkit [58964] by ggaren@apple.com

2 edits in trunk/JavaScriptCore

Added a fast path for number-to-character conversion via
Number.prototype.toString base 36.

Reviewed by Oliver Hunt.

0.7% speedup on SunSpider.

runtime/NumberPrototype.cpp:

(JSC::numberProtoFuncToString): Made radix interpretation a little more
efficient by keeping it in int space. Turned "const char" into
"static const char" just in case. Added a fast path for base 36
conversion of something that will turn into a character.

12:23 PM Changeset in webkit [58963] by jianli@chromium.org

2 edits in trunk/WebCore

Add a few more files with Windows linefeeds to the .gitattributes file.

Patch by Simon Hausmann <simon.hausmann@nokia.com> on 2010-05-07

.gitattributes:

11:46 AM Changeset in webkit [58962] by Simon Fraser

2 edits in trunk/WebCore

2010-05-07 Simon Fraser <Simon Fraser>

Reviewed by Dan Bernstein.

Fix debug-only render tree output to show object addresses in hex.

rendering/RenderTreeAsText.cpp: (WebCore::writeRenderObject): (WebCore::write):

11:40 AM Changeset in webkit [58961] by ap@apple.com

3 edits
4 adds in trunk

Reviewed by Adele Peterson.

https://bugs.webkit.org/show_bug.cgi?id=38546
Node.focus() fails to move focus from subframe properly

Test: fast/frames/take-focus-from-iframe.html

html/HTMLFrameElementBase.cpp: (WebCore::HTMLFrameElementBase::setFocus): Don't clear focus if this frame doesn't have it. This can happen if page's and HTMLFrameElement's ideas of focused frame get out of sync temporarily.

11:24 AM Changeset in webkit [58960] by Nikolas Zimmermann

9 edits in trunk

2010-05-07 Nikolas Zimmermann <nzimmermann@rim.com>

Reviewed by Dirk Schulze.

svg/custom/use-instanceRoot-as-event-target.xhtml crashes randomly
https://bugs.webkit.org/show_bug.cgi?id=37798

When creating JSEventListeners through attributes (onclick, etc..) or add/removeEventListener
calls on a SVGElementInstance, do NOT pass the element instance itself as JS wrapper object
but the correspondingElement(). SVGElementInstance redirects all event listener registrations
to the correspondingElement(), as they share an event listener list, per SVG spec. The old
code was very dangerous, leading to random assertions, when garbage collection teared down
the JSSVGElementInstance and event listeners fired, whose listeners were registered with
it, but residing in the correspondingElement() event listener list.

Removes the need for CustomToJS code for JSSVGElementInstance, which was wrong anyways. We hoped
to keep the event listeers alive by just creating a js wrapper of the correspondingElement(), that
could only work as long as garbage collection didn't tear it down, just luck. Also remove the
CustomPushEventHandlerScope marker, as it is only used for JSLazyEventListeners and only works
for JSNode derived objects, it was a no-op for SVGElementInstance, thus removed it.

Should fix all random crashes/assertions seen with svg/custom/use-instanceRoot-as-event-target.xhtml
See bug report for a detailed crash analysis.

bindings/js/JSSVGElementInstanceCustom.cpp: Remove custom toJS()/pushEventHandlerScope() handling, not necessary anymore. (WebCore::JSSVGElementInstance::markChildren):
bindings/scripts/CodeGeneratorJS.pm: For JSSVGElementInstance pass the correspondingElement() as JS wrapper object, not itself.
svg/SVGElementInstance.cpp: (WebCore::SVGElementInstance::invalidateAllInstancesOfElement): Be sure to trigger a style update here, so dirty shadow trees for <use> get rebuild. Otherwhise DOM may be out-of-sync. (WebCore::SVGElementInstance::eventTargetData): Add ASSERT_NOT_REACHED(), all event listener calls are forwarded to the correspondingElement(). (WebCore::SVGElementInstance::ensureEventTargetData): Ditto.
svg/SVGElementInstance.idl: Remove CustomToJS, CustomPushEventHandlerScope markers.

2010-05-07 Nikolas Zimmermann <nzimmermann@rim.com>

Reviewed by Dirk Schulze.

svg/custom/use-instanceRoot-as-event-target.xhtml crashes randomly
https://bugs.webkit.org/show_bug.cgi?id=37798

Make use-instanceRoot-as-event-target.xhtml behave more correctly. Due a copy & paste problem test #7 was not executed. Fixed.
Make destruction more reliable, by using the standard js-test-post.js injection through <script> before calling notifyDone(), and "TEST COMPLETE" magically appears.

platform/mac/svg/hixie/error/017-expected.txt: Update result, now that invalidateAllInstancesOfElement(), correctly triggers <use> shadow tree rebuilds.
svg/custom/use-instanceRoot-as-event-target-expected.txt: Update result, was missing a "TEST COMPLETE" marker, and Test #7 was not executed before.
svg/custom/use-instanceRoot-as-event-target.xhtml: Fix typo, test #7 is now executed, it also had a bug which has been corrected.

11:20 AM Changeset in webkit [58959] by eric@webkit.org

1 edit
1 add in trunk/JavaScriptCore

2010-05-07 Adam Barth <abarth@webkit.org>

Reviewed by Darin Fisher.

[WTFURL] Add a malloc-backed URLBuffer
https://bugs.webkit.org/show_bug.cgi?id=38694

This patch adds a URLBuffer that uses new/delete to manage the backing
store for the buffer.

wtf/url/src/RawURLBuffer.h: Added. (WTF::RawURLBuffer::RawURLBuffer): (WTF::RawURLBuffer::~RawURLBuffer): (WTF::RawURLBuffer::resize):

11:05 AM Changeset in webkit [58958] by eric@webkit.org

2 edits in trunk/WebKit/chromium

2010-05-07 Finnur Thorarinsson <finnur.webkit@gmail.com>

Reviewed by Dimitri Glazkov.

[chromium]: Fixing Find in page crash on sites.google.com
https://bugs.webkit.org/show_bug.cgi?id=38591

setMarkerActive should not try to set a collapsed marker as
active. This can happen when you Edit a Sites page after doing
a Find operation and then you press FindNext.

src/WebFrameImpl.cpp: (WebKit::WebFrameImpl::setMarkerActive):

10:52 AM Changeset in webkit [58957] by kbr@google.com

38 edits
4 adds in trunk

2010-05-06 Kenneth Russell <kbr@google.com>

Reviewed by Dimitri Glazkov.

WebCore::WebGLUnsignedIntArrayInternal::getCallback ReadAV@Arbitrary (deef89ee3d0345edebeaf13cf974c47c)
https://bugs.webkit.org/show_bug.cgi?id=38039

Web IDL now allows indexed getters and setters to be unnamed. Per
discussion in WebGL working group and recent update to spec,
removed the buggy get() and single-element set() methods from the
JavaScript bindings to the WebGL array types. Refactored set()
implementation in JSC bindings to share more code and modified V8
binding to look more like it. Added unit tests for indexed getter
with out-of-range indices and verifying removal of get and
single-element set methods. Updated existing WebGL array tests.

Tests: fast/canvas/webgl/array-get-and-set-method-removal.html

fast/canvas/webgl/array-get-out-of-bounds.html

bindings/js/JSWebGLArrayHelper.h: (WebCore::setWebGLArrayHelper):
bindings/js/JSWebGLByteArrayCustom.cpp: (WebCore::JSWebGLByteArray::set):
bindings/js/JSWebGLFloatArrayCustom.cpp: (WebCore::JSWebGLFloatArray::set):
bindings/js/JSWebGLIntArrayCustom.cpp: (WebCore::JSWebGLIntArray::set):
bindings/js/JSWebGLShortArrayCustom.cpp: (WebCore::JSWebGLShortArray::set):
bindings/js/JSWebGLUnsignedByteArrayCustom.cpp: (WebCore::JSWebGLUnsignedByteArray::set):
bindings/js/JSWebGLUnsignedIntArrayCustom.cpp: (WebCore::JSWebGLUnsignedIntArray::set):
bindings/js/JSWebGLUnsignedShortArrayCustom.cpp: (WebCore::JSWebGLUnsignedShortArray::set):
bindings/v8/custom/V8WebGLArrayCustom.h: (WebCore::setWebGLArrayHelper):
bindings/v8/custom/V8WebGLByteArrayCustom.cpp: (WebCore::V8WebGLByteArray::setCallback):
bindings/v8/custom/V8WebGLFloatArrayCustom.cpp: (WebCore::V8WebGLFloatArray::setCallback):
bindings/v8/custom/V8WebGLIntArrayCustom.cpp: (WebCore::V8WebGLIntArray::setCallback):
bindings/v8/custom/V8WebGLShortArrayCustom.cpp: (WebCore::V8WebGLShortArray::setCallback):
bindings/v8/custom/V8WebGLUnsignedByteArrayCustom.cpp: (WebCore::V8WebGLUnsignedByteArray::setCallback):
bindings/v8/custom/V8WebGLUnsignedIntArrayCustom.cpp: (WebCore::V8WebGLUnsignedIntArray::setCallback):
bindings/v8/custom/V8WebGLUnsignedShortArrayCustom.cpp: (WebCore::V8WebGLUnsignedShortArray::setCallback):
html/canvas/WebGLByteArray.h:
html/canvas/WebGLByteArray.idl:
html/canvas/WebGLFloatArray.h:
html/canvas/WebGLFloatArray.idl:
html/canvas/WebGLIntArray.h:
html/canvas/WebGLIntArray.idl:
html/canvas/WebGLShortArray.h:
html/canvas/WebGLShortArray.idl:
html/canvas/WebGLUnsignedByteArray.h:
html/canvas/WebGLUnsignedByteArray.idl:
html/canvas/WebGLUnsignedIntArray.h:
html/canvas/WebGLUnsignedIntArray.idl:
html/canvas/WebGLUnsignedShortArray.h:
html/canvas/WebGLUnsignedShortArray.idl:

2010-05-06 Kenneth Russell <kbr@google.com>

Reviewed by Dimitri Glazkov.

WebCore::WebGLUnsignedIntArrayInternal::getCallback ReadAV@Arbitrary (deef89ee3d0345edebeaf13cf974c47c)
https://bugs.webkit.org/show_bug.cgi?id=38039

Web IDL now allows indexed getters and setters to be unnamed. Per
discussion in WebGL working group and recent update to spec,
removed the buggy get() and single-element set() methods from the
JavaScript bindings to the WebGL array types. Refactored set()
implementation in JSC bindings to share more code and modified V8
binding to look more like it. Added unit tests for indexed getter
with out-of-range indices and verifying removal of get and
single-element set methods. Updated existing WebGL array tests.

fast/canvas/webgl/array-get-and-set-method-removal-expected.txt: Added.
fast/canvas/webgl/array-get-and-set-method-removal.html: Added.
fast/canvas/webgl/array-get-out-of-bounds-expected.txt: Added.
fast/canvas/webgl/array-get-out-of-bounds.html: Added.
fast/canvas/webgl/array-setters-expected.txt:
fast/canvas/webgl/array-setters.html:
fast/canvas/webgl/array-unit-tests-expected.txt:
fast/canvas/webgl/array-unit-tests.html:
fast/canvas/webgl/bug-32456-expected.txt:
fast/canvas/webgl/bug-32456.html:

10:32 AM Changeset in webkit [58956] by pfeldman@chromium.org

2 edits in trunk/WebCore

2010-05-07 Pavel Feldman <pfeldman@chromium.org>

Not reviewed: chromium dev tools tests fix.
Add a more strict check prior to accessing constructor property in
Web Inspector frontend.

inspector/front-end/InjectedScript.js: (injectedScriptConstructor):

10:30 AM Changeset in webkit [58955] by johnnyg@google.com

3 edits in trunk/LayoutTests

2010-05-07 John Gregg <johnnyg@google.com>

Unreviewed, rebaseline for chromium.

platform/chromium-mac/fast/layers/video-layer-expected.checksum:
platform/chromium-mac/fast/layers/video-layer-expected.png:

10:03 AM Changeset in webkit [58954] by pfeldman@chromium.org

3 edits in trunk/WebCore

2010-05-07 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Joseph Pecoraro.

Web Inspector: On Linux/Windows panel history is
traversed while iterating over words in text prompt.

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

inspector/front-end/TextPrompt.js: (WebInspector.TextPrompt):
inspector/front-end/inspector.js: (WebInspector.documentKeyDown):

9:59 AM Changeset in webkit [58953] by pfeldman@chromium.org

2 edits in trunk/WebCore

2010-05-07 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Timothy Hatcher.

Web Inspector: come up with InjectedScript._className that works for both JSC and V8.

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

inspector/front-end/InjectedScript.js: (injectedScriptConstructor):

9:32 AM Changeset in webkit [58952] by johnnyg@google.com

17 edits in trunk/LayoutTests

2010-05-07 John Gregg <johnnyg@google.com>

Unreviewed, rebaseline for chromium.

platform/chromium-mac/media/audio-controls-rendering-expected.checksum:
platform/chromium-mac/media/audio-controls-rendering-expected.png:
platform/chromium-mac/media/controls-after-reload-expected.checksum:
platform/chromium-mac/media/controls-after-reload-expected.png:
platform/chromium-mac/media/controls-strict-expected.checksum:
platform/chromium-mac/media/controls-strict-expected.png:
platform/chromium-mac/media/video-controls-rendering-expected.checksum:
platform/chromium-mac/media/video-controls-rendering-expected.png:
platform/chromium-mac/media/video-display-toggle-expected.checksum:
platform/chromium-mac/media/video-display-toggle-expected.png:
platform/chromium-mac/media/video-empty-source-expected.checksum:
platform/chromium-mac/media/video-empty-source-expected.png:
platform/chromium-mac/media/video-no-audio-expected.checksum:
platform/chromium-mac/media/video-no-audio-expected.png:
platform/chromium-mac/media/video-volume-slider-expected.checksum:
platform/chromium-mac/media/video-volume-slider-expected.png:

8:54 AM Changeset in webkit [58951] by Chris Jerdonek

6 edits in trunk/WebKitTools

Refactored VCSUtils.pm's parse-related methods to leave inapplicable
hash values unset instead of setting them to "undef".

Reviewed by Daniel Bates.

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

Preferring "not set" over "undef" keeps the unit tests smaller and
easier to maintain. Otherwise, we would have to update every unit
test case each time we add support for a new key-value pair --
instead of just the relevant ones.

Scripts/VCSUtils.pm:
- In parseGitDiffHeader(), adjusted the handling of these key-values: executableBitDelta and isBinary.
- In parseSvnDiffHeader(), adjusted the handling of these key-values: copiedFromPath, isBinary, and sourceRevision.
- In parseDiffHeader(), adjusted the handling of these key-values: isGit and isSvn.
- In parseDiff(), adjusted the handling of these key-values: isBinary, isGit, isSvn, and sourceRevision.

Scripts/webkitperl/VCSUtils_unittest/parseDiff.pl:
- Updated the unit tests as necessary.

Scripts/webkitperl/VCSUtils_unittest/parseDiffHeader.pl:
- Updated the unit tests as necessary.

Scripts/webkitperl/VCSUtils_unittest/parseGitDiffHeader.pl:
- Updated the unit tests as necessary.

Scripts/webkitperl/VCSUtils_unittest/parseSvnDiffHeader.pl:
- Updated the unit tests as necessary.

8:42 AM Changeset in webkit [58950] by Joseph Pecoraro

4 edits
1 add in trunk/WebCore

2010-05-07 Joseph Pecoraro <Joseph Pecoraro>

Reviewed by Adam Barth.

document.write is not synchronous after page load
https://bugs.webkit.org/show_bug.cgi?id=38146

If there are no pending scripts, a document.write call should be
synchronous. This matches other browsers and the HTML5 spec. Forcing
the tokenizer to be synchronous in Document::write does not affect
external scripts written by the write call. This should only change
behavior of document.write after the page is done loading.

Difficult to test reliably due to HTMLTokenizing relying on processing time.
I made a manual test because the test requires processing very large strings
synchronously and therefore can take some time.

Test: WebCore/manual-tests/dom/document-write-synchronous-after-page-load.html

dom/Document.cpp: (WebCore::SynchronousHTMLTokenizerGuard::SynchronousHTMLTokenizerGuard): if the provided tokenizer is an HTMLTokenizer make it synchronous (WebCore::SynchronousHTMLTokenizerGuard::~SynchronousHTMLTokenizerGuard): if the provided tokenizer was an HTMLTokenizer return its synchronous state (WebCore::Document::write): temporarily set the tokenizer to synchronous during document.write
dom/Tokenizer.h: (WebCore::Tokenizer::asHTMLTokenizer): default implementation returns 0, to be overridden by HTMLTokenizer
html/HTMLTokenizer.h: allow access to to the force synchronous state (WebCore::HTMLTokenizer::forceSynchronous): accessor (WebCore::HTMLTokenizer::asHTMLTokenizer): override the default to return itself
manual-tests/dom/document-write-synchronous-after-page-load.html: Added.

8:15 AM QtWebKitJournal edited by Simon Hausmann

(diff)

8:13 AM Changeset in webkit [58949] by vestbo@webkit.org

2 edits in trunk/WebCore

[Qt] Fix rendering of -webkit-user-select: none

Reviewed by Simon Hausmann.

-webkit-user-select: none is implemented by filling
the area with an invalid default-constructed Color.
In most ports passing an invalid color down to the
graphics backend seems to produce transparent fills.

In Qt the behavior of painting with an invalid QColor
is undefined, and in practice it results in painting
black opaque areas.

One way to fix this would be to use Qt::transparent
when converting an undefined Color to a QColor, but
Qt does not have short circuits for fully transparent
painting, and we actually end up in slow code paths
due to the transparency. So, we're better of doing the
short circuit in WebKit.

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

platform/graphics/qt/GraphicsContextQt.cpp:

8:04 AM Changeset in webkit [58948] by eric@webkit.org

2 edits in trunk/WebCore

2010-05-07 Ben Murdoch <benm@google.com>

Reviewed by Darin Adler.

Potential crash in EventHandler::handleTouchEvent
https://bugs.webkit.org/show_bug.cgi?id=38646

Fix a ref counting bug that can cause a crash if the m_originatingouchPointTargets
hashmap holds the last ref to an EventTarget when the user lifts their finger.

This is very hard to reproduce in a consistent way and clearly a
simple logic error in the code, therefore no new tests.

page/EventHandler.cpp: (WebCore::EventHandler::handleTouchEvent): Don't let the RefPtr we get back from

the hasmap go out of scope so soon as it could delete the wrapped ptr if the
hashmap held the last ref (and we use the raw ptr that the RefPtr
wraps later in the WebCore::Touch constructor).

8:03 AM Changeset in webkit [58947] by Simon Hausmann

2 edits in trunk

Add a few more files with Windows linefeeds to the .gitattributes file.

.gitattributes:

7:08 AM Changeset in webkit [58946] by kov@webkit.org

2 edits in trunk/WebCore

2010-05-07 Gustavo Noronha Silva <gustavo.noronha@collabora.co.uk>

Rubber-stamped by Kenneth Rohde Christiansen.

[GTK] Fails to build on GTK+ < 2.14
https://bugs.webkit.org/show_bug.cgi?id=38746

Build fix only. Do basically the same as gtk_adjustment_configure
would do if we are bellow GTK+ 2.14.

platform/gtk/ScrollViewGtk.cpp: (WebCore::AdjustmentConfigure):

6:43 AM Changeset in webkit [58945] by kov@webkit.org

2 edits in trunk/WebCore

2010-05-07 Gustavo Noronha Silva <gustavo.noronha@collabora.co.uk>

Rubber-stamped by Xan Lopez.

Crash with MgOpenModerna font, with non-UTF-8 character
https://bugs.webkit.org/show_bug.cgi?id=37795

NULL-check the font platform data, for now, to avoid the crash.

Could not yet figure out a way to get a test written that does not
involve adding the font to the tree, but there are doubts about
licensing.

platform/graphics/cairo/FontCacheCairo.cpp: (WebCore::FontCache::getFontDataForCharacters):

5:33 AM Changeset in webkit [58944] by Nikolas Zimmermann

2 edits in trunk/LayoutTests

2010-05-07 Nikolas Zimmermann <nzimmermann@rim.com>

Not reviewed. Turn Tiger core builder green again, filed https://bugs.webkit.org/show_bug.cgi?id=38743 for failing webarchive/test-link-rel-icon.html.

platform/mac-tiger/Skipped:

5:31 AM Changeset in webkit [58943] by morrita@google.com

6 edits in trunk/WebCore

2010-05-07 MORITA Hajime <morrita@google.com>

Reviewed by Eric Seidel.

[Chromium] RenderThemeChromiumMac.mm should share the code with RenderThemeMac.mm
https://bugs.webkit.org/show_bug.cgi?id=37204

Made RenderThemeChromiumMac subclass of RenderThemeMac, instead of RenderTheme.
Added some method to RenderThemeMac for overriding on RenderThemeChromiumMac.

No new tests; Should have no behavioural change.

WebCore.gyp/WebCore.gyp:
rendering/RenderThemeChromiumMac.h:
rendering/RenderThemeChromiumMac.mm: (WebCore::RenderThemeChromiumMac::usesTestModeFocusRingColor): (WebCore::RenderThemeChromiumMac::documentViewFor): (WebCore::RenderThemeChromiumMac::adjustMediaSliderThumbSize): (WebCore::RenderThemeChromiumMac::paintMediaVolumeSliderTrack): (WebCore::RenderThemeChromiumMac::paintMediaVolumeSliderThumb):
rendering/RenderThemeMac.h: (WebCore::RenderThemeMac::updateActiveState): Added for overriding
rendering/RenderThemeMac.mm: (WebCore::RenderThemeMac::usesTestModeFocusRingColor): Added for overriding (WebCore::RenderThemeMac::documentViewFor): Added for overriding (WebCore::RenderThemeMac::paintMenuList): (WebCore::RenderThemeMac::setPopupButtonCellState): (WebCore::RenderThemeMac::paintSliderThumb): (WebCore::RenderThemeMac::paintSearchField): (WebCore::RenderThemeMac::setSearchCellState): (WebCore::RenderThemeMac::paintSearchFieldCancelButton): (WebCore::RenderThemeMac::paintSearchFieldResultsDecoration): (WebCore::RenderThemeMac::paintSearchFieldResultsButton): (WebCore::RenderThemeMac::adjustSliderThumbSize): (WebCore::RenderThemeMac::adjustMediaSliderThumbSize): Added for overriding (WebCore::RenderThemeMac::extraMediaControlsStyleSheet):

4:53 AM Changeset in webkit [58942] by Simon Hausmann

2 edits in trunk/WebKit/qt

Symbian build fix.

[Qt] Updated the def file with absent exports.

symbian/eabi/QtWebKitu.def:

12:32 AM Changeset in webkit [58941] by ukai@chromium.org

5 edits in trunk

2010-05-06 Fumitoshi Ukai <ukai@chromium.org>

Reviewed by Alexey Proskuryakov.

JavaScriptCore/wtf/RandomNumber.h should provide using WTF::*
https://bugs.webkit.org/show_bug.cgi?id=38719

wtf/RandomNumber.h: Add using directives.

2010-05-06 Fumitoshi Ukai <ukai@chromium.org>

Reviewed by Alexey Proskuryakov.

JavaScriptCore/wtf/RandomNumber.h should provide using WTF::*
https://bugs.webkit.org/show_bug.cgi?id=38719

platform/graphics/wince/FontCustomPlatformData.cpp: (WebCore::createUniqueFontName):

Remove WTF

prefix from randomNumber()

platform/network/FormDataBuilder.cpp: (WebCore::FormDataBuilder::generateUniqueBoundaryString): Ditto.

May 6, 2010:

11:42 PM Changeset in webkit [58940] by mrowe@apple.com

2 edits in trunk/WebKitTools

Exclude leaks due to <rdar://problem/7815391> from the output.

Rubber-stamped by Dan Bernstein.

Scripts/old-run-webkit-tests:

11:22 PM Changeset in webkit [58939] by Laszlo Gombos

2 edits in trunk/JavaScriptCore

2010-05-06 Laszlo Gombos <Laszlo Gombos>

Reviewed by Kenneth Rohde Christiansen.

[Qt] Enable YARR_JIT for X86 Mac for QtWebKit
https://bugs.webkit.org/show_bug.cgi?id=38668

wtf/Platform.h:

11:18 PM Changeset in webkit [58938] by mitz@apple.com

2 edits in trunk/LayoutTests

Skip compositing/layer-creation/overflow-scroll-overlap.html from r58936, since hardware
compositing is disabled on the Leopard test bot.

platform/mac-leopard/Skipped:

10:12 PM Changeset in webkit [58937] by tony@chromium.org

4 edits
3 adds in trunk

2010-04-27 Tony Chang <tony@chromium.org>

Reviewed by Eric Seidel.

[chromium] clicking a scrollbar in an iframe shouldn't lose the selection
https://bugs.webkit.org/show_bug.cgi?id=38189

fast/events/mousedown-in-subframe-scrollbar-expected.txt: Added.
fast/events/mousedown-in-subframe-scrollbar.html: Added.
fast/events/resources/mousedown-in-subframe-scrollbar.html: Added.
platform/win/Skipped:

2010-04-27 Tony Chang <tony@chromium.org>

Reviewed by Eric Seidel.

[chromium] clicking a scrollbar in an iframe shouldn't lose the selection
https://bugs.webkit.org/show_bug.cgi?id=38189

Test: fast/events/mousedown-in-subframe-scrollbar.html

page/chromium/EventHandlerChromium.cpp: (WebCore::EventHandler::passMousePressEventToSubframe): If clicking on

a scrollbar, pass the event directly to the scrollbar. This matches
mac which passes the event to the NSScroller.

10:04 PM Changeset in webkit [58936] by Simon Fraser

3 edits
3 adds in trunk

2010-05-06 Simon Fraser <Simon Fraser>

Reviewed by Dan Bernstein.

Compositing layers that are clipped out by an overflow:scroll parent fail to get created on reveal
https://bugs.webkit.org/show_bug.cgi?id=38712

When compositing layer creation is testing layer overlap, we need to re-run the
algorithm when an overflow:scroll element scrolls, to create new layers for revealed
elements.

Test: compositing/layer-creation/overflow-scroll-overlap.html

rendering/RenderLayer.cpp: (WebCore::RenderLayer::scrollToOffset): First, move the compositing updates until after the call to updateWidgetPositions(), so that this order is similar to what we done when a FrameView scrolls. This change has no known side effects. Second, if compositingConsultsOverlap() is true, we need to actually do a compositing layer update to compute whether revealed/hidden layers should be created/destroyed.

9:58 PM Changeset in webkit [58935] by barraclough@apple.com

5 edits
1 add in trunk/JavaScriptCore

Bug 38714 - Add result caching for Math.sin

Reviewed by Geoff Garen.

Math.sin is frequently called with the same operand, caching the result should
improve performance. CachedTranscendentalFunction adds a generic mechanism to
cache results for pure functions with the signature "double func(double)", and
where NaN maps to NaN.

CachedTranscendentalFunction uses a very simple hash function designed to be
fast to execute since we cannot cache the result, and trivial to JIT generate,
should we wish to do so.

JavaScriptCore.xcodeproj/project.pbxproj:
runtime/CachedTranscendentalFunction.h: Added.

(JSC::CachedTranscendentalFunction::CachedTranscendentalFunction):
(JSC::CachedTranscendentalFunction::~CachedTranscendentalFunction):
(JSC::CachedTranscendentalFunction::operator()):
(JSC::CachedTranscendentalFunction::initialize):
(JSC::CachedTranscendentalFunction::hash):

runtime/JSGlobalData.h:
runtime/MathObject.cpp:

(JSC::mathProtoFuncSin):

wtf/Platform.h:

9:51 PM Changeset in webkit [58934] by eric@webkit.org

6 edits
5 copies
30 adds in trunk/WebKitTools

2010-05-06 Sheriff Bot <webkit.review.bot@gmail.com>

Unreviewed, rolling out r58933.
http://trac.webkit.org/changeset/58933
https://bugs.webkit.org/show_bug.cgi?id=38717

"Broke all websocket tests on Tiger" (Requested by eseidel on
#webkit).

Scripts/new-run-webkit-websocketserver:
Scripts/old-run-webkit-tests:
Scripts/run-webkit-websocketserver:
Scripts/webkitpy/layout_tests/port/websocket_server.py:
Scripts/webkitpy/thirdparty/init.py:
Scripts/webkitpy/thirdparty/pywebsocket/COPYING: Added.
Scripts/webkitpy/thirdparty/pywebsocket/MANIFEST.in: Added.
Scripts/webkitpy/thirdparty/pywebsocket/README: Added.
Scripts/webkitpy/thirdparty/pywebsocket/README.webkit: Added.
Scripts/webkitpy/thirdparty/pywebsocket/example/echo_client.py: Added.
Scripts/webkitpy/thirdparty/pywebsocket/example/echo_wsh.py: Added.
Scripts/webkitpy/thirdparty/pywebsocket/example/handler_map.txt: Added.
Scripts/webkitpy/thirdparty/pywebsocket/mod_pywebsocket/init.py: Added.
Scripts/webkitpy/thirdparty/pywebsocket/mod_pywebsocket/dispatch.py: Added.
Scripts/webkitpy/thirdparty/pywebsocket/mod_pywebsocket/handshake.py: Added.
Scripts/webkitpy/thirdparty/pywebsocket/mod_pywebsocket/headerparserhandler.py: Added.
Scripts/webkitpy/thirdparty/pywebsocket/mod_pywebsocket/memorizingfile.py: Added.
Scripts/webkitpy/thirdparty/pywebsocket/mod_pywebsocket/msgutil.py: Added.
Scripts/webkitpy/thirdparty/pywebsocket/mod_pywebsocket/standalone.py: Added.
Scripts/webkitpy/thirdparty/pywebsocket/mod_pywebsocket/util.py: Added.
Scripts/webkitpy/thirdparty/pywebsocket/setup.py: Added.
Scripts/webkitpy/thirdparty/pywebsocket/test/config.py: Added.
Scripts/webkitpy/thirdparty/pywebsocket/test/mock.py: Added.
Scripts/webkitpy/thirdparty/pywebsocket/test/run_all.py: Added.
Scripts/webkitpy/thirdparty/pywebsocket/test/test_dispatch.py: Added.
Scripts/webkitpy/thirdparty/pywebsocket/test/test_handshake.py: Added.
Scripts/webkitpy/thirdparty/pywebsocket/test/test_memorizingfile.py: Added.
Scripts/webkitpy/thirdparty/pywebsocket/test/test_mock.py: Added.
Scripts/webkitpy/thirdparty/pywebsocket/test/test_msgutil.py: Added.
Scripts/webkitpy/thirdparty/pywebsocket/test/test_util.py: Added.
Scripts/webkitpy/thirdparty/pywebsocket/test/testdata/README: Added.
Scripts/webkitpy/thirdparty/pywebsocket/test/testdata/handlers/blank_wsh.py: Added.
Scripts/webkitpy/thirdparty/pywebsocket/test/testdata/handlers/origin_check_wsh.py: Added.
Scripts/webkitpy/thirdparty/pywebsocket/test/testdata/handlers/sub/exception_in_transfer_wsh.py: Added.
Scripts/webkitpy/thirdparty/pywebsocket/test/testdata/handlers/sub/no_wsh_at_the_end.py: Added.
Scripts/webkitpy/thirdparty/pywebsocket/test/testdata/handlers/sub/non_callable_wsh.py: Added.
Scripts/webkitpy/thirdparty/pywebsocket/test/testdata/handlers/sub/plain_wsh.py: Added.
Scripts/webkitpy/thirdparty/pywebsocket/test/testdata/handlers/sub/wrong_handshake_sig_wsh.py: Added.
Scripts/webkitpy/thirdparty/pywebsocket/test/testdata/handlers/sub/wrong_transfer_sig_wsh.py: Added.
Scripts/webkitpy/thirdparty/pywebsocket/test/testdata/hello.pl: Added.

8:56 PM Changeset in webkit [58933] by ukai@chromium.org

6 edits
35 deletes in trunk/WebKitTools

2010-05-06 Fumitoshi Ukai <ukai@chromium.org>

Reviewed by David Levin.

WebSocket: pywebsocket 0.5
https://bugs.webkit.org/show_bug.cgi?id=38034

Remove pywebsocket from webkitpy/thirdparty.
Make pywebsocket autoinstalled.

Scripts/new-run-webkit-websocketserver: Add --output-dir option.
Scripts/old-run-webkit-tests: Use new-run-webkit-websocketserver, rather than directly run pywebsocket's standalone.py
Scripts/run-webkit-websocketserver: Ditto.
Scripts/webkitpy/layout_tests/port/websocket_server.py: Use autoinstalled pywebsocket.
Scripts/webkitpy/thirdparty/init.py: Autoinstall pywebsocket
Scripts/webkitpy/thirdparty/pywebsocket: Removed.

8:13 PM Changeset in webkit [58932] by Chris Jerdonek

4 edits in trunk/WebKitTools

svn-apply now understands the Git diff "copy from" syntax when the
similarity index is 100%.

Reviewed by Daniel Bates.

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

Scripts/VCSUtils.pm:
- Adjusted parseGitDiffHeader() to parse the "copy from" and "similarity index" lines.

Scripts/svn-unapply:
- Adjusted the patch() subroutine so that copies are recognized as file additions.

Scripts/webkitperl/VCSUtils_unittest/parseGitDiffHeader.pl:
- Added unit tests for the cases of a copy with similarity index 100% and less than 100%.

7:58 PM Changeset in webkit [58931] by mrowe@apple.com

5 edits in trunk

Versioning.

7:58 PM Changeset in webkit [58930] by mrowe@apple.com

1 copy in tags/Safari-533.8

New tag.

7:54 PM Changeset in webkit [58929] by Chris Jerdonek

8 edits in trunk/WebKitTools

2010-05-06 Chris Jerdonek <Chris Jerdonek>

Reviewed by Daniel Bates.

Removed the need for svn-apply and -unapply to re-parse whether
a diff is binary or not.

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

Scripts/VCSUtils.pm:
- Adjusted parseGitDiffHeader() to set the isBinary key.
- Adjusted parseSvnDiffHeader() to set the isBinary key.
- Adjusted parseDiffHeader() to set the isBinary key.
- Changed the scmFormat key set by parseDiffHeader() to isGit and isSvn keys.
- Adjusted parseDiff() to set the isBinary, isGit, and isSvn keys.

Scripts/svn-apply:
- Updated the patch() method to use the isBinary, isGit, and isSvn keys.

Scripts/svn-unapply:
- Updated the patch() method to use the isBinary and isSvn keys.

Scripts/webkitperl/VCSUtils_unittest/parseDiff.pl:
- Updated the unit tests as necessary.
- Added a test case to test that the isBinary key is getting set properly.

Scripts/webkitperl/VCSUtils_unittest/parseDiffHeader.pl:
- Updated the unit tests as necessary.

Scripts/webkitperl/VCSUtils_unittest/parseGitDiffHeader.pl:
- Updated the unit tests as necessary.

Scripts/webkitperl/VCSUtils_unittest/parseSvnDiffHeader.pl:
- Updated the unit tests as necessary.

7:37 PM Changeset in webkit [58928] by dpranke@chromium.org

6 edits
2 moves
2 adds in trunk/LayoutTests

2010-05-06 Dirk Pranke <dpranke@chromium.org>

Unreviewed, rebaseline and expectation changes from WK roll from a
few days ago.

platform/chromium-linux/http/tests/loading/bad-scheme-subframe-expected.txt: Copied from LayoutTests/platform/chromium-mac/http/tests/loading/bad-scheme-subframe-expected.txt.
platform/chromium-linux/svg/W3C-SVG-1.1/fonts-kern-01-t-expected.checksum:
platform/chromium-linux/svg/W3C-SVG-1.1/fonts-kern-01-t-expected.png:
platform/chromium-linux/svg/text/kerning-expected.checksum:
platform/chromium-linux/svg/text/kerning-expected.png:
platform/chromium-mac/editing/selection/caret-rtl-2-expected.checksum: Renamed from LayoutTests/platform/chromium-mac/editing/selection/caret-rtl-2-actual.checksum.
platform/chromium-mac/editing/selection/caret-rtl-2-expected.png: Renamed from LayoutTests/platform/chromium-mac/editing/selection/caret-rtl-2-actual.png.
platform/chromium-mac/svg/text/kerning-expected.checksum: Added.
platform/chromium/test_expectations.txt:

6:56 PM Changeset in webkit [58927] by tkent@chromium.org

4 edits in trunk

2010-05-06 Kent Tamura <tkent@chromium.org>

Reviewed by Dimitri Glazkov.

[DRT/Chromium] Add support for resources on Mac
https://bugs.webkit.org/show_bug.cgi?id=38637

DEPS: Check out tools/data_pack.

2010-05-06 Kent Tamura <tkent@chromium.org>

Reviewed by Dimitri Glazkov.

[DRT/Chromium] Add support for resources on Mac
https://bugs.webkit.org/show_bug.cgi?id=38637

Repack webkit_chromium_resources.pak, webkit_strings_en-US.pak,
and webkit_resources.pak, and put them as Mac bundle resource.
The 'actions' section is almost same as a part of test_shell.gypi.

DumpRenderTree/DumpRenderTree.gyp/DumpRenderTree.gyp:

6:29 PM Changeset in webkit [58926] by dpranke@chromium.org

1 edit
2 adds in trunk/LayoutTests

2010-05-06 Dirk Pranke <dpranke@chromium.org>

Unreviewed, rebaseline a chromium test.

Need a linux baseline for fast/notifications/notifications-replace-expected.txt to override the windows version.

platform/chromium-linux/fast/notifications/notifications-replace-expected.txt: Added.

5:47 PM Changeset in webkit [58925] by mjs@apple.com

2 edits in trunk/WebCore

Not reviewed, build fix.

Try again - with quotes this time. *facepalm*

dom/Notation.cpp:

5:39 PM Changeset in webkit [58924] by mjs@apple.com

2 edits in trunk/WebCore

Not reviewed, build fix.

Speculative fix for Qt and Gtk.

dom/Notation.cpp:

5:34 PM BuildingQtOnWindows edited by Laszlo Gombos

few words on how to generate MSVC project files (diff)

5:25 PM Changeset in webkit [58923] by mjs@apple.com

2 edits in trunk/WebCore

Not reviewed, build fix.

Fix an assert that I failed to update in my earlier change.

dom/CharacterData.h:

(WebCore::CharacterData::CharacterData):

5:03 PM Changeset in webkit [58922] by ap@apple.com

4 edits
2 adds in trunk

Reviewed by Geoff Garen.

https://bugs.webkit.org/show_bug.cgi?id=38697
REGRESSION (r58299): Replying on reddit.com no longer works

css/CSSSelector.cpp: (WebCore::CSSSelector::extractPseudoType): Don't recognize :first, :left and :right. The aren't allowed in all contexts, and properly implementing them is out of scope for this regression fix.

5:01 PM Changeset in webkit [58921] by jianli@chromium.org

11 edits in trunk/WebCore

Improve code generator scripts to support converting ScriptString.
https://bugs.webkit.org/show_bug.cgi?id=38699

Reviewed by Adam Barth.

Change both JSC and V8 generators to introduce "ConvertScriptString"
attribute to allow converting from ScriptString. Also updated the
bindings test result.

These changes are necessary in order to avoid adding custom binding codes
when we add the FileReader interface.

bindings/scripts/CodeGeneratorJS.pm:
bindings/scripts/CodeGeneratorV8.pm:
bindings/scripts/test/GObject/WebKitDOMTestObj.cpp:

(webkit_dom_test_obj_get_script_string_attr):
(webkit_dom_test_obj_get_property):
(webkit_dom_test_obj_class_init):

bindings/scripts/test/GObject/WebKitDOMTestObj.h:
bindings/scripts/test/JS/JSTestObj.cpp:

(WebCore::):
(WebCore::jsTestObjScriptStringAttr):

bindings/scripts/test/JS/JSTestObj.h:
bindings/scripts/test/ObjC/DOMTestObj.h:
bindings/scripts/test/ObjC/DOMTestObj.mm:

(-[DOMTestObj scriptStringAttr]):

bindings/scripts/test/TestObj.idl:
bindings/scripts/test/V8/V8TestObj.cpp:

(WebCore::TestObjInternal::scriptStringAttrAttrGetter):
(WebCore::):

4:45 PM Changeset in webkit [58920] by Csaba Osztrogonác

2 edits in trunk/WebKitTools

[Qt] Unreviewed buildfix after r58917.

DumpRenderTree/qt/LayoutTestControllerQt.h: Missing function declaration added.

4:33 PM Changeset in webkit [58919] by jianli@chromium.org

4 edits
11 adds in trunk/WebCore

Improve code generator scripts to pass additional ScriptExecutionContext
argument to the constructor.
https://bugs.webkit.org/show_bug.cgi?id=38687

Reviewed by Adam Barth.

Change both JSC and V8 generators to introduce "CallWith=ScriptExecutionContext"
attribute to allow passing the additional ScriptExecutionContext argument to
the constructor. Also add another test IDL file to test interface-level
features.

These changes are necessary in order to avoid adding custom binding codes
when we add the FileReader interface.

bindings/scripts/CodeGeneratorJS.pm:
bindings/scripts/CodeGeneratorV8.pm:
bindings/scripts/test/GObject/WebKitDOMTestInterface.cpp: Added.
bindings/scripts/test/GObject/WebKitDOMTestInterface.h: Added.
bindings/scripts/test/GObject/WebKitDOMTestInterfacePrivate.h: Added.
bindings/scripts/test/JS/JSTestInterface.cpp: Added.
bindings/scripts/test/JS/JSTestInterface.h: Added.
bindings/scripts/test/ObjC/DOMTestInterface.h: Added.
bindings/scripts/test/ObjC/DOMTestInterface.mm: Added.
bindings/scripts/test/ObjC/DOMTestInterfaceInternal.h: Added.
bindings/scripts/test/TestInterface.idl: Added.
bindings/scripts/test/V8/V8TestInterface.cpp: Added.
bindings/scripts/test/V8/V8TestInterface.h: Added.
bindings/v8/V8Proxy.h:

(WebCore::V8Proxy::constructDOMObjectWithScriptExecutionContext):

4:23 PM Changeset in webkit [58918] by mjs@apple.com

2 edits in trunk/WebCore

Build fix, not reviewed.

Speculative Windows and Debug build fixes for the last change.

dom/Node.h:

(WebCore::Node::inDocument):
(WebCore::Node::setFlag):

4:23 PM Changeset in webkit [58917] by andersca@apple.com

14 edits
2 adds in trunk

2010-05-06 Anders Carlsson <andersca@apple.com>

Reviewed by Darin Adler and Dan Bernstein..

REGRESSION (r51617): when plugins are disabled, plugins show up as garbage characters
https://bugs.webkit.org/show_bug.cgi?id=38698
<rdar://problem/7942075>

When the plug-in database is initialized, we will register all the MIME types it supports with the global
WebView dictionary. When plug-ins are disabled for a single web view, the MIME types still need to be
in the global mapping (because other web views might still have plug-ins enabled).

Prior to r51617 we would always look at the plug-in database to determine that the MIME type belongs to a
plug-in, but now we won't even touch the plug-in database when plug-ins are disabled.

In order to fix this, a new set of registered MIME types that are known to be plug-ins is added. When
+[WebView _viewClass:andRepresentationClass:forMIMEType:allowingPlugins:] is called and allowingPlugins is FALSE
we check if the MIME type is a known plug-in MIME type and return false in that case.

Plugins/WebPluginDatabase.mm: (-[WebPluginDatabase refresh]): (-[WebPluginDatabase _removePlugin:]):
WebView/WebView.mm: (knownPluginMIMETypes): (+[WebView _registerPluginMIMEType:]): (+[WebView _unregisterPluginMIMEType:]): (+[WebView _viewClass:andRepresentationClass:forMIMEType:allowingPlugins:]):
WebView/WebViewInternal.h:

2010-05-06 Anders Carlsson <andersca@apple.com>

Reviewed by Darin Adler and Dan Bernstein..

REGRESSION (r51617): when plugins are disabled, plugins show up as garbage characters
https://bugs.webkit.org/show_bug.cgi?id=38698
<rdar://problem/7942075>

Add tests.

platform/mac/plugins/disable-plugins-expected.txt: Added.
platform/mac/plugins/disable-plugins.html: Added.

2010-05-06 Anders Carlsson <andersca@apple.com>

Reviewed by Darin Adler and Dan Bernstein..

REGRESSION (r51617): when plugins are disabled, plugins show up as garbage characters
https://bugs.webkit.org/show_bug.cgi?id=38698
<rdar://problem/7942075>

Add a 'setPluginsEnabled' layoutTestController function for disabling plug-ins. This is only implemented on Mac currently
because the bug that needs this functionality is mac specific.

DumpRenderTree/LayoutTestController.cpp: (setPluginsEnabledCallback): (LayoutTestController::staticFunctions):
DumpRenderTree/LayoutTestController.h:
DumpRenderTree/gtk/LayoutTestControllerGtk.cpp: (LayoutTestController::setPluginsEnabled):
DumpRenderTree/mac/DumpRenderTree.mm: (resetDefaultsToConsistentValues):
DumpRenderTree/mac/LayoutTestControllerMac.mm: (LayoutTestController::setPluginsEnabled):
DumpRenderTree/qt/LayoutTestControllerQt.cpp: (LayoutTestController::setPluginsEnabled):
DumpRenderTree/win/LayoutTestControllerWin.cpp: (LayoutTestController::setPluginsEnabled):
DumpRenderTree/wx/LayoutTestControllerWx.cpp: (LayoutTestController::setPluginsEnabled):

4:18 PM Changeset in webkit [58916] by jamesr@google.com

10 edits in trunk/WebCore

2010-05-06 James Robinson <jamesr@chromium.org>

Reviewed by Eric Seidel.

Fix warnings emitted by gcc 4.4.1 on linux in chromium-specific platform graphics files.
https://bugs.webkit.org/show_bug.cgi?id=38158

Fixes:

replace NULL with 0
remove unusued locals

add parens around ambiguous looking compound predicates like (a

b && c)

This also adds a check for x >= 0 to FontLinux.cpp's in this statement:

if (x < walker.width())

This is more documentation than anything else since walker.width() returns
an unsigned the current behavior is that x is promoted to unsigned and as
long as x + walker.width() is less than 2^{31 all negative values of x

end up wrapping around and not being < walker.width(). This behavior is

tested by fast/text/international/khmer-selection.html}

platform/graphics/chromium/FontLinux.cpp: (WebCore::adjustTextRenderMode): (WebCore::TextRunWalker::TextRunWalker): (WebCore::TextRunWalker::length): (WebCore::TextRunWalker::width): (WebCore::TextRunWalker::getTextRun): (WebCore::TextRunWalker::getNormalizedTextRun): (WebCore::Font::offsetForPositionForComplexText):
platform/graphics/chromium/FontPlatformDataLinux.cpp: (WebCore::FontPlatformData::setupPaint):
platform/graphics/chromium/HarfbuzzSkia.cpp: (WebCore::getOutlinePoint):
platform/graphics/skia/GraphicsContext3DSkia.cpp: (WebCore::GraphicsContext3D::getImageData):
platform/graphics/skia/GraphicsContextSkia.cpp: (WebCore::isCoordinateSkiaSafe): (WebCore::GraphicsContext::fillRect): (WebCore::GraphicsContext::strokePath): (WebCore::GraphicsContext::strokeRect):

3:53 PM Changeset in webkit [58915] by adachan@apple.com

2 edits in trunk/WebCore

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

Reviewed by David Kilzer.

Check the result from widget() for NULL before accessing it.

rendering/RenderIFrame.cpp:

(WebCore::RenderIFrame::calcHeight):
(WebCore::RenderIFrame::calcWidth):

3:41 PM Changeset in webkit [58914] by mjs@apple.com

44 edits in trunk/WebCore

2010-05-06 Maciej Stachowiak <mjs@apple.com>

Reviewed by Darin Adler.

further fixes towards REGRESSION (r57292): 1% PLT regression from visited link information leak fix
https://bugs.webkit.org/show_bug.cgi?id=38682
<rdar://problem/7859794>

Looks like a 1-2% speedup on PLT.

Reorder CSS properties.
Remove short circuit tag check in matchRulesForList which costs more than it saves.
Inline initForStyleResolve.
Optimize applyDeclarations to avoid switch and take fewer branches in the inner loop.

Change the way Node handles flags - replace bitfield with a uint32_t and explicit masking, to make it cheaper to initialize the bits and give faster access.
Added new Node flags to check for isStyledElement, isHTMLElement, isSVGElement, isComment, and devirtualize those methods.
Inline constructors for Node, Element, Text, CharacterData, StyledElement, etc since they are very simple and lots of nodes get constructed.

css/CSSPropertyNames.in: Move a few of the properties up front so we can check for them with < instead of switch statements
css/CSSStyleSelector.cpp: (WebCore::CSSStyleSelector::matchRulesForList): Remove unnecessary preflight check for tag match before checking selector. This check very rarely short circuits anything, since most rules with a tag end up in the appropriate tag bucket. So doing the check cost more time than the time saved. (WebCore::CSSStyleSelector::initForStyleResolve): Inline. Create RenderStyle in a better way. (WebCore::CSSStyleSelector::applyDeclarations): Get rid of switch statement and use <= to check for the high priority properties. Convert to template to avoid checking "applyFirst" each time through the loop. (WebCore::CSSStyleSelector::styleForElement): Adjust for new applyDeclarations() signature. (WebCore::CSSStyleSelector::keyframeStylesForAnimation): ditto (WebCore::CSSStyleSelector::pseudoStyleForElement): ditto
css/CSSStyleSelector.h: Adjust for new applyDeclarations() signature.
rendering/style/RenderStyle.cpp: (WebCore::RenderStyle::RenderStyle): Inline constructor.
rendering/style/RenderStyle.h: Make constructor private so it can be inline.
dom/Node.h: (WebCore::): See high-level description of changes above. Redid the way boolean flags work. (WebCore::Node::isElementNode): (WebCore::Node::isContainerNode): (WebCore::Node::isTextNode): (WebCore::Node::isHTMLElement): (WebCore::Node::isSVGElement): (WebCore::Node::isStyledElement): (WebCore::Node::isCommentNode): (WebCore::Node::hasID): (WebCore::Node::hasClass): (WebCore::Node::active): (WebCore::Node::inActiveChain): (WebCore::Node::inDetach): (WebCore::Node::hovered): (WebCore::Node::attached): (WebCore::Node::setAttached): (WebCore::Node::needsStyleRecalc): (WebCore::Node::styleChangeType): (WebCore::Node::childNeedsStyleRecalc): (WebCore::Node::isLink): (WebCore::Node::setHasID): (WebCore::Node::setHasClass): (WebCore::Node::setChildNeedsStyleRecalc): (WebCore::Node::clearChildNeedsStyleRecalc): (WebCore::Node::setInDocument): (WebCore::Node::clearInDocument): (WebCore::Node::setInActiveChain): (WebCore::Node::clearInActiveChain): (WebCore::Node::setIsLink): (WebCore::Node::clearIsLink): (WebCore::Node::setActive): (WebCore::Node::setHovered): (WebCore::Node::inDocument): (WebCore::Node::): (WebCore::Node::getFlag): (WebCore::Node::setFlag): (WebCore::Node::clearFlag): (WebCore::Node::hasRareData): (WebCore::Node::isParsingChildrenFinished): (WebCore::Node::setIsParsingChildrenFinished): (WebCore::Node::clearIsParsingChildrenFinished): (WebCore::Node::isStyleAttributeValid): (WebCore::Node::setIsStyleAttributeValid): (WebCore::Node::clearIsStyleAttributeValid): (WebCore::Node::isSynchronizingStyleAttribute): (WebCore::Node::setIsSynchronizingStyleAttribute): (WebCore::Node::clearIsSynchronizingStyleAttribute): (WebCore::Node::areSVGAttributesValid): (WebCore::Node::setAreSVGAttributesValid): (WebCore::Node::clearAreSVGAttributesValid): (WebCore::Node::isSynchronizingSVGAttributes): (WebCore::Node::setIsSynchronizingSVGAttributes): (WebCore::Node::clearIsSynchronizingSVGAttributes): (WebCore::Node::hasRareSVGData): (WebCore::Node::setHasRareSVGData): (WebCore::Node::clearHasRareSVGData): (WebCore::Node::initialRefCount):
dom/Node.cpp: (WebCore::Node::trackForDebugging): Adjusted for changes in flag handling. (WebCore::Node::ensureRareData): ditto (WebCore::Node::setStyleChange): ditto (WebCore::Node::setNeedsStyleRecalc): ditto (WebCore::Node::lazyAttach): ditto (WebCore::Node::attach): ditto (WebCore::Node::detach): ditto (WebCore::Node::insertedIntoDocument): ditto (WebCore::Node::removedFromDocument): ditto
dom/CharacterData.cpp:
dom/CharacterData.h: (WebCore::CharacterData::CharacterData): Inline the constructor (moved from .cpp)
dom/Comment.cpp: (WebCore::Comment::Comment): Tell the base class that we're a comment.
dom/Comment.h: Remove isCommentNode override.
dom/ContainerNode.cpp: (WebCore::ContainerNode::detach): Adjusted for changes in flag handling. (WebCore::ContainerNode::removedFromDocument): ditto
dom/Document.cpp: (WebCore::Document::Document): Adjusted for changes in flag handling. (WebCore::Document::recalcStyle): ditto (WebCore::Document::setFocusedNode): ditto
dom/Document.h: (WebCore::Node::Node): Inline the Node constructor - goes here because it uses Document.
dom/DocumentFragment.cpp: include Document.h due to above change
dom/EditingText.cpp: ditto
dom/EntityReference.cpp: ditto
dom/Element.cpp: (WebCore::Element::getAttribute): Adjusted for changes in flag handling. (WebCore::Element::setAttribute): ditto (WebCore::Element::hasAttributes): ditto (WebCore::Element::recalcStyle): ditto (WebCore::Element::finishParsingChildren): ditto
dom/Element.h: (WebCore::Element::Element): Inline (moved from .cpp) (WebCore::Element::isFinishedParsingChildren): (WebCore::Element::beginParsingChildren): (WebCore::Element::attributes): Adjusted for changes in flag handling.
dom/StyledElement.cpp: (WebCore::StyledElement::updateStyleAttribute): Adjust for changes to flag handling. (WebCore::StyledElement::mapToEntry): ditto (WebCore::StyledElement::parseMappedAttribute): ditto (WebCore::StyledElement::copyNonAttributeProperties): ditto
dom/StyledElement.h: (WebCore::StyledElement::StyledElement): Inline (moved from.cpp) (WebCore::StyledElement::invalidateStyleAttribute): Adjust for changes in flag handling.
dom/Text.h: (WebCore::Text::Text): Inline (moved from .cpp)
dom/Text.cpp:
html/HTMLAnchorElement.cpp: (WebCore::HTMLAnchorElement::HTMLAnchorElement): Adjust for changes in flag handling. (WebCore::HTMLAnchorElement::parseMappedAttribute): ditto
html/HTMLElement.cpp: (WebCore::HTMLElement::create): Tell base class we're an HTML element.
html/HTMLElement.h: ditto above; remove isHTMLElement override.
html/HTMLFormControlElement.h: Tell base class we're an HTML element.
html/HTMLFrameOwnerElement.cpp: (WebCore::HTMLFrameOwnerElement::HTMLFrameOwnerElement): ditto
html/HTMLProgressElement.cpp: (WebCore::HTMLProgressElement::HTMLProgressElement): ditto
mathml/MathMLElement.cpp: (WebCore::MathMLElement::MathMLElement): Tell base class we're a styled element.
rendering/MediaControlElements.cpp: (WebCore::MediaControlShadowRootElement::MediaControlShadowRootElement): Adjust for changes in flag handling. (WebCore::MediaControlElement::MediaControlElement): ditto (WebCore::MediaControlInputElement::MediaControlInputElement): ditto
rendering/RenderFileUploadControl.cpp: (WebCore::RenderFileUploadControl::updateFromElement): ditto
rendering/RenderLayer.cpp: (WebCore::RenderLayer::updateHoverActiveState): ditto
rendering/RenderProgress.cpp: (WebCore::RenderProgress::updateValuePartState): ditto
rendering/RenderSlider.cpp: (WebCore::RenderSlider::updateFromElement): ditto
rendering/SVGShadowTreeElements.cpp: (WebCore::SVGShadowTreeRootElement::SVGShadowTreeRootElement): ditto
rendering/TextControlInnerElements.cpp: (WebCore::TextControlInnerElement::attachInnerElement): ditto
svg/SVGAnimatedProperty.h: (WebCore::SVGAnimatedPropertyTearOff::setBaseVal): ditto (WebCore::SVGAnimatedPropertyTearOff::setAnimVal): ditto
svg/SVGElement.cpp: (WebCore::SVGElement::SVGElement): Tell base class we're an svg element. (WebCore::SVGElement::ensureRareSVGData): Adjust for flag handling changes. (WebCore::SVGElement::updateAnimatedSVGAttribute): ditto
svg/SVGElement.h: (WebCore::SVGElement::invalidateSVGAttributes): ditto
svg/SVGPolyElement.cpp: (WebCore::SVGPolyElement::svgAttributeChanged): ditto
wml/WMLAnchorElement.cpp: (WebCore::WMLAnchorElement::WMLAnchorElement): ditto
wml/WMLElement.cpp: (WebCore::WMLElement::WMLElement): Tell base class we're a styled element.

3:28 PM Changeset in webkit [58913] by mitz@apple.com

4 edits in trunk/WebKit/mac

<rdar://problem/7951285> REGRESSION (r58847): Composited iframe content obscures Safari's application chrome

Reviewed by Simon Fraser.

Fixed this other regression from r58847. The regression was caused by overriding -visibleRect to
return the WebClipView’s full bounds. AppKit uses -visibleRect to determine the geometry
of the surface for the child WebFrameView. The fix is to restrict the special behavior of
-[WebClipView visibleRect] to when AppKit is consulting it for the purpose of invalidating
areas while scrolling.

WebView/WebClipView.h:
WebView/WebClipView.mm:

(-[WebClipView visibleRect]): If the WebClipView is not scrolling, always return
[super visibleRect].
(-[WebClipView _immediateScrollToPoint:]): Override this internal NSClipView method
to set a flag telling -visibleRect that the view is scrolling.

WebView/WebView.mm:

(layerSyncRunLoopObserverCallBack): Ensure that screen updates, disabled by AppKit
when it thinks an upcoming window flush will re-enable them, are enabled here in
case the -setNeedsDisplayInRect: override has prevented the window from needing to be
flushed.

3:28 PM Changeset in webkit [58912] by johnnyg@google.com

1 edit
2 adds in trunk/LayoutTests

2010-05-06 John Gregg <johnnyg@google.com>

Unreviewed, rebaseline for chromium test.

[chromium] Deal with different local path structure on windows.

platform/chromium-win/fast/notifications: Added.
platform/chromium-win/fast/notifications/notifications-replace-expected.txt: Added.

3:19 PM Changeset in webkit [58911] by ggaren@apple.com

3 edits in trunk/JavaScriptCore

Windows build fix.

bytecode/SamplingTool.cpp:

(JSC::SamplingFlags::sample): Use a cast, so Windows will be happy when
this code is enabled.

wtf/Platform.h: Reverted last change to this file, which seems to have

been accidental.

3:12 PM Changeset in webkit [58910] by abarth@webkit.org

2 edits in trunk/WebCore

2010-05-06 Adam Barth <abarth@webkit.org>

Unreviewed. Fix indent. Sorry, my OCD was acting up.

page/EventHandler.cpp: (WebCore::EventHandler::handleMouseReleaseEvent):

3:09 PM Changeset in webkit [58909] by dpranke@chromium.org

2 edits in trunk/LayoutTests

2010-05-06 Dirk Pranke <dpranke@chromium.org>

Unreviewed, expectations fix.

Delete a line for a test that is no longer failing on chromium linux.

platform/chromium/test_expectations.txt:

2:20 PM BuildingQtOnSymbian edited by Laszlo Gombos

(diff)

1:50 PM Changeset in webkit [58908] by eric@webkit.org

3 edits in trunk/WebCore

2010-05-06 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Joseph Pecoraro.

Web Inspector: store selected Headers / Content tab on explicit switch and/or explicit navigate only.

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

inspector/front-end/ResourceView.js: (WebInspector.ResourceView): (WebInspector.ResourceView.prototype._selectHeadersTab): (WebInspector.ResourceView.prototype.selectContentTab):
inspector/front-end/ResourcesPanel.js: (WebInspector.ResourcesPanel.prototype.showResource):

1:47 PM Changeset in webkit [58907] by barraclough@apple.com

4 edits in trunk/JavaScriptCore

Add compile switch to make debugger keyword plant breakpoint instructions.

Reviewed by Oliver Hunt.

bytecompiler/BytecodeGenerator.cpp:

(JSC::BytecodeGenerator::emitDebugHook):

jit/JITOpcodes.cpp:

(JSC::JIT::emit_op_debug):

wtf/Platform.h:

1:38 PM Changeset in webkit [58906] by eric@webkit.org

2 edits in trunk/WebCore

2010-05-06 Luiz Agostini <luiz.agostini@openbossa.org>

Rubber-stamped by Simon Hausmann.

[Qt] use QT_MOBILE_THEME in Symbian
https://bugs.webkit.org/show_bug.cgi?id=38440

Putting QT_MOBILE_THEME into use for Symbian.

WebCore.pro:

1:26 PM Changeset in webkit [58905] by eric@webkit.org

3 edits
1 move in trunk/WebCore

2010-05-06 Laszlo Gombos <Laszlo Gombos>

Unreviewed, build fix WinCE for QtWebKit.

[Qt] Compilation with Plugins disabled is broken
https://bugs.webkit.org/show_bug.cgi?id=31407

Rename platform/qt/TemporaryLinkStubs.cpp to avoid name collition on
Windows.

Thanks for Ismail "cartman" Donmez for help.

No new tests, as there is no new functionality.

WebCore.gypi:
WebCore.pro:
platform/qt/TemporaryLinkStubs.cpp: Removed.
platform/qt/TemporaryLinkStubsQt.cpp: Copied from WebCore/platform/qt/TemporaryLinkStubs.cpp.

1:04 PM Changeset in webkit [58904] by oliver@apple.com

3 edits in trunk/JavaScriptCore

Fix ARM builds

1:01 PM Changeset in webkit [58903] by eric@webkit.org

2 edits in trunk/WebCore

2010-05-06 Simon Hausmann <simon.hausmann@nokia.com>

Reviewed by Kenneth Rohde Christiansen.

[Qt] REGRESSION: Loading of external CSS and JS files over network fails in some cases
https://bugs.webkit.org/show_bug.cgi?id=36755

Enable the direct connection between QtNetwork and QtWebKit only for Qt versions
greater than 4.6.2, due to a bug in Qt that's been fixed after 4.6.2.

platform/network/qt/QNetworkReplyHandler.cpp:

12:39 PM Changeset in webkit [58902] by oliver@apple.com

14 edits
9 adds in trunk

2010-05-06 Oliver Hunt <oliver@apple.com>

Reviewed by Geoffrey Garen.

Improve performance of single character string compares
https://bugs.webkit.org/show_bug.cgi?id=38659

Add logic to the jit to identify comparisons to single character string literals
and then just perform the comparison inline, rather than ignoring the evidence
and attempting to perform an integer comparison.

Multiple changes required -- add jnlesseq opcode, add helper function to identify
single character string constants, add a helper to load single character strings.
Then add the 32_64 and normal codepaths to the JIT.

assembler/MacroAssemblerX86Common.h: (JSC::MacroAssemblerX86Common::load16):
bytecode/CodeBlock.cpp: (JSC::CodeBlock::dump):
bytecode/Opcode.h:
bytecompiler/BytecodeGenerator.cpp: (JSC::BytecodeGenerator::emitJumpIfTrue):
interpreter/Interpreter.cpp: (JSC::Interpreter::privateExecute):
jit/JIT.cpp: (JSC::JIT::privateCompileMainPass): (JSC::JIT::privateCompileSlowCases):
jit/JIT.h:
jit/JITArithmetic.cpp: (JSC::JIT::emit_op_jnless): (JSC::JIT::emitSlow_op_jnless): (JSC::JIT::emit_op_jless): (JSC::JIT::emitSlow_op_jless): (JSC::JIT::emit_op_jlesseq): (JSC::JIT::emit_op_jnlesseq): (JSC::JIT::emitSlow_op_jlesseq): (JSC::JIT::emitSlow_op_jnlesseq):
jit/JITArithmetic32_64.cpp: (JSC::JIT::emit_op_jnless): (JSC::JIT::emitSlow_op_jnless): (JSC::JIT::emit_op_jless): (JSC::JIT::emitSlow_op_jless): (JSC::JIT::emit_op_jlesseq): (JSC::JIT::emit_op_jnlesseq): (JSC::JIT::emitSlow_op_jlesseq): (JSC::JIT::emitSlow_op_jnlesseq): (JSC::JIT::emitBinaryDoubleOp):
jit/JITInlineMethods.h: (JSC::JIT::emitLoadCharacterString): (JSC::JIT::isOperandConstantImmediateChar):
jit/JSInterfaceJIT.h: (JSC::ThunkHelpers::stringImplDataOffset): (JSC::ThunkHelpers::jsStringLengthOffset): (JSC::ThunkHelpers::jsStringValueOffset): Moved from ThunkGenerators to make it possible to share.
jit/ThunkGenerators.cpp:

2010-05-06 Oliver Hunt <oliver@apple.com>

Reviewed by Geoffrey Garen.

Improve performance of single character string compares
https://bugs.webkit.org/show_bug.cgi?id=38659

Add many tests of <, >, <=, >=, ==, ===, !=, !== as the existing
tests were woefully inadequate.

fast/js/comparison-operators-expected.txt: Added.
fast/js/comparison-operators-greater-expected.txt: Added.
fast/js/comparison-operators-greater.html: Added.
fast/js/comparison-operators-less-expected.txt: Added.
fast/js/comparison-operators-less.html: Added.
fast/js/comparison-operators.html: Added.
fast/js/script-tests/comparison-operators-greater.js: Added. (description.makeTest.func.f.toString): (description.makeTest): (doTest):
fast/js/script-tests/comparison-operators-less.js: Added. (description.makeTest.func.f.toString): (description.makeTest): (doTest):
fast/js/script-tests/comparison-operators.js: Added. (description.makeTest.func.f.toString): (description.makeTest): (doTest):

12:06 PM Changeset in webkit [58901] by Martin Robinson

6 edits in trunk/WebKit/gtk

2010-05-06 Martin Robinson <Martin Robinson>

Reviewed by Xan Lopez.

Remove use of GOwnPtr to hold GObject types.
https://bugs.webkit.org/show_bug.cgi?id=38669

Change use of GOwnPtr to GRefPtr to types which are natively reference-counted.

WebCoreSupport/FrameLoaderClientGtk.cpp: (WebKit::FrameLoaderClient::dispatchWillSendRequest): Change GOwnPtr to GRefPtr in some places. (WebKit::FrameLoaderClient::dispatchDecidePolicyForMIMEType): Ditto.
webkit/webkitnetworkrequest.cpp: (webkit_network_request_new_with_core_request): Ditto.
webkit/webkitnetworkresponse.cpp: (webkit_network_response_new_with_core_response): Ditto.
webkit/webkitprivate.cpp: Remove GOwnPtr reference counting template specialization.
webkit/webkitprivate.h: Ditto.

11:50 AM Changeset in webkit [58900] by Adam Roben

2 edits in trunk/WebKit/win

2010-05-06 Adam Roben <Adam Roben>

Bail out of WebView::paint when there's nothing to paint

Fixes <http://webkit.org/b/38670> <rdar://problem/7947105> REGRESSION
(r58067): Crash in WebView::paint when Web Inspector is docked and
window is resized so small that WebView disappears

When the WebView is 0-sized, ensureBackingStore() bails out without
creating a bitmap, leaving m_backingStoreBitmap null. Before r58067,
m_backingStoreBitmap was an HBITMAP, so we were happily passing along
a null HBITMAP to various Windows APIs. These calls would fail but not
crash. r58067 changed m_backingStoreBitmap to a RefCountedHBITMAP, and
dereferencing a null RefCountedHBITMAP* of course crashes.

Reviewed by Steve Falkenburg.

WebView.cpp: (WebView::paint): Bail if the rect to paint is empty.

11:47 AM Changeset in webkit [58899] by johnnyg@google.com

2 edits in trunk/LayoutTests

2010-05-06 John Gregg <johnnyg@google.com>

Unreviewed. Updating test expectations.

[chromium] enabling notifications tests after test_shell change.

platform/chromium/test_expectations.txt:

11:46 AM Changeset in webkit [58898] by kov@webkit.org

2 edits in trunk/WebCore

2010-05-06 Gustavo Noronha Silva <gustavo.noronha@collabora.co.uk>

Reviewed by Xan Lopez.

Rework where we get page step and line step from, so we do not
depend on scrollbars existing. Caught by API test I forgot to run.

platform/gtk/ScrollViewGtk.cpp: (WebCore::ScrollView::setGtkAdjustments):

10:47 AM Changeset in webkit [58897] by kov@webkit.org

2 edits in trunk/WebCore

2010-05-06 Gustavo Noronha Silva <gustavo.noronha@collabora.co.uk>

Reviewed by Xan Lopez.

[GTK] Adjustment resetting uses wrong values, and misses page_size and steps
https://bugs.webkit.org/show_bug.cgi?id=38657

Fix resetting adjustment values. In the page cache case, we were
confusing page_size and upper, leading to stray scrollbars
sometimes.

platform/gtk/ScrollViewGtk.cpp: (WebCore::ScrollView::setGtkAdjustments):

10:42 AM Changeset in webkit [58896] by ojan@chromium.org

3 edits in trunk/LayoutTests

2010-05-06 Ojan Vafai <ojan@chromium.org>

Reviewed by Adam Barth.

editing/selection/shift-click.html fails on GTK and QT
https://bugs.webkit.org/show_bug.cgi?id=38656

r58892 added this test. It's the first test for shift+click behavior,
so it exposed bugs in GTK/QT. Skipping to green the bots.

platform/gtk/Skipped:
platform/qt/Skipped:

10:37 AM Changeset in webkit [58895] by sfalken@apple.com

2 edits in trunk/WebKit/win

2010-05-06 Steve Falkenburg <sfalken@apple.com>

Reviewed by Adam Roben.

WebFrame::paintDocumentRectToContext paints content at the wrong location
https://bugs.webkit.org/show_bug.cgi?id=38651

WebFrame.cpp: (WebFrame::paintDocumentRectToContext):

10:12 AM Changeset in webkit [58894] by Darin Adler

2 edits in trunk/WebCore

2010-05-06 Darin Adler <Darin Adler>

Reviewed by Beth Dakin.

Page::setCanStartMedia does not properly handle the case where a media listener is removed
https://bugs.webkit.org/show_bug.cgi?id=38602

We can't find any real case where this causes a crash at this time, but
we want to harden the code anyway. Thus there are no new regression tests.

page/Page.cpp: (WebCore::Page::removeMediaCanStartListener): Removed incorrect assertion. (WebCore::Page::setCanStartMedia): Change algorithm so we notify listeners one at a time and don't notify any listener that has already been removed from the set.

10:05 AM Changeset in webkit [58893] by evan@chromium.org

9 edits in trunk/LayoutTests

2010-05-06 Evan Martin <evan@chromium.org>

Unreviewed. Updating test expectations.

[chromium] new baselines for r58886
https://bugs.webkit.org/show_bug.cgi?id=38652

platform/chromium-linux/fast/lists/w3-css3-list-styles-numeric-expected.txt:
platform/chromium-linux/fast/text/international/danda-space-expected.checksum:
platform/chromium-linux/fast/text/international/danda-space-expected.png:
platform/chromium-linux/fast/text/international/danda-space-expected.txt:
platform/chromium-linux/fast/text/international/thai-baht-space-expected.checksum:
platform/chromium-linux/fast/text/international/thai-baht-space-expected.png:
platform/chromium-linux/fast/text/international/thai-baht-space-expected.txt:
platform/chromium/test_expectations.txt:

9:58 AM Changeset in webkit [58892] by ojan@chromium.org

3 edits
4 adds in trunk

2010-05-05 Ojan Vafai <ojan@chromium.org>

Reviewed by Darin Adler.

shift+click on an existing selection doesn't work right
https://bugs.webkit.org/show_bug.cgi?id=36542

NSTextView behavior is to move the end of the selection
closest to the shift-click. Win/Linux behavior is to always
move the focus end of the selection.

editing/selection/script-tests/shift-click.js: Added. (shiftClick): (assertSelectionString):
editing/selection/shift-click-expected.txt: Added.
editing/selection/shift-click.html: Added.
platform/win/editing/selection/shift-click-expected.txt: Added.

2010-05-05 Ojan Vafai <ojan@chromium.org>

Reviewed by Darin Adler.

shift+click on an existing selection doesn't work right
https://bugs.webkit.org/show_bug.cgi?id=36542

NSTextView behavior is to move the end of the selection
closest to the shift-click. Win/Linux behavior is to always
move the focus end of the selection.

Test: editing/selection/shift-click.html

page/EventHandler.cpp: (WebCore::textDistance): (WebCore::EventHandler::handleMousePressEventSingleClick):

9:53 AM Changeset in webkit [58891] by pfeldman@chromium.org

2 edits in trunk/WebCore

2010-05-06 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Timothy Hatcher.

Web Inspector: implement panels history traversal on Cmd+Left/Right.

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

inspector/front-end/inspector.js: (WebInspector.set currentPanel): (WebInspector.loaded): (WebInspector.documentKeyDown): (WebInspector.PanelHistory): (WebInspector.PanelHistory.prototype.canGoBack): (WebInspector.PanelHistory.prototype.goBack): (WebInspector.PanelHistory.prototype.canGoForward): (WebInspector.PanelHistory.prototype.goForward): (WebInspector.PanelHistory.prototype.setPanel):

9:49 AM Changeset in webkit [58890] by pfeldman@chromium.org

4 edits in trunk

2010-05-06 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Timothy Hatcher.

Chromium Dev Tools: Large toolbar icons flash when dev
tools is opened in docked mode.

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

src/js/DevTools.js: (devtools.domContentLoaded):

9:46 AM Changeset in webkit [58889] by eric@webkit.org

2 edits in trunk/WebKitTools

2010-05-06 Jochen Eisinger <jochen@chromium.org>

Reviewed by Dimitri Glazkov.

Make ImageDiff depend on WebKit. When compiled from within Chromium, WTF is not a standalone dynamic library but depends on WebKit.
https://bugs.webkit.org/show_bug.cgi?id=38632

DumpRenderTree/DumpRenderTree.gyp/DumpRenderTree.gyp:

9:43 AM Changeset in webkit [58888] by tonikitoo@webkit.org

4 edits
4 deletes in trunk/LayoutTests

2010-05-05 Antonio Gomes <tonikitoo@webkit.org>

Rubber-stamped by Kenneth Christiansen.

Spatial Navigation: use data url in layout tests
https://bugs.webkit.org/show_bug.cgi?id=38584

In order to make Spatial Navigation layout tests more readable, patch
changes the use or src=resources/iframeXXX.html by inlined sources via
data uri schema.

It is a polishment.

fast/events/spatial-navigation/resources/iframe.html: Removed.
fast/events/spatial-navigation/resources/iframe2.html: Removed.
fast/events/spatial-navigation/resources/iframe3.html: Removed.
fast/events/spatial-navigation/resources/iframe4.html: Removed.
fast/events/spatial-navigation/snav-iframe-no-focusable-content.html:
fast/events/spatial-navigation/snav-iframe-no-scrollable-content.html:
fast/events/spatial-navigation/snav-iframe-with-offscreen-focusable-element.html:

9:42 AM Changeset in webkit [58887] by tonikitoo@webkit.org

1 edit in trunk/LayoutTests/platform/gtk/Skipped

2010-05-06 Antonio Gomes <tonikitoo@webkit.org>

[Gtk] Skipped fast/events/spatial-navigation/snav-iframe-flattening-simple.html

DRT does not support frame flattening testing.
https://bugs.webkit.org/show_bug.cgi?id=38650

platform/gtk/Skipped:

9:34 AM Changeset in webkit [58886] by evan@chromium.org

2 edits in trunk/WebCore

2010-05-03 Evan Martin <evan@chromium.org>

Reviewed by Eric Seidel.

[chromium] use glyphDataForCharacter so we hit the glyphdata cache
https://bugs.webkit.org/show_bug.cgi?id=38500

Calling fontDataForCharacters() while rendering text is expensive.
Examining the relevant Mac code in WebKit revealed that calling the
similarly-named glyphDataForCharacters() results in caching the font
data on a per-glyph basis.

Since we now choose a font based on the first character in a run, we
need to be careful that all the glyphs within the run use the same font.
This is also similar to the WebKit Mac code. We need to remove all of
the script run bits eventually and this is a step in that direction.

Tests: fast/text/international/danda-space.html spacing changes slightly
(now matches Firefox better) and fast/text/international/thai-baht-space.html
now picks the correct (serif) font for the digits.

platform/graphics/chromium/FontLinux.cpp: (WebCore::TextRunWalker::nextScriptRun): (WebCore::TextRunWalker::setupFontForScriptRun):

9:32 AM Changeset in webkit [58885] by Martin Robinson

11 edits in trunk

2010-05-06 Martin Robinson <Martin Robinson>

Reviewed by Gustavo Noronha Silva.

[GTK] Enable DOM clipboard and drag-and-drop access
https://bugs.webkit.org/show_bug.cgi?id=30623

Convert dragging portion of drag-and-drop to use DataObjectGtk.

wtf/gobject/GRefPtr.h: Add forward declarations for GObject functions.

2010-05-06 Martin Robinson <Martin Robinson>

Reviewed by Gustavo Noronha Silva.

[GTK] Enable DOM clipboard and drag-and-drop access
https://bugs.webkit.org/show_bug.cgi?id=30623

Convert dragging portion of drag-and-drop to use DataObjectGtk.

No new tests, because functionality has not changed.

page/gtk/EventHandlerGtk.cpp: (WebCore::EventHandler::createDraggingClipboard): Pass the DataObjectGtk as a parameter here.
platform/gtk/ClipboardGtk.h: (WebCore::ClipboardGtk::create): Take the DataObject as a parameter instead of creating it here. (WebCore::ClipboardGtk::helper): Added. (WebCore::ClipboardGtk::dataObject): Added.
platform/gtk/PasteboardHelper.h: Make targetListForDataObject a public method.

2010-05-06 Martin Robinson <Martin Robinson>

Reviewed by Gustavo Noronha Silva.

[GTK] Enable DOM clipboard and drag-and-drop access
https://bugs.webkit.org/show_bug.cgi?id=30623

Convert dragging portion of drag-and-drop to use DataObjectGtk.

WebCoreSupport/DragClientGtk.cpp: (WebKit::DragClient::willPerformDragDestinationAction): Remove the notImplemented. It is implemented, it's just a no-op. (WebKit::DragClient::startDrag): Start the drag context via the ClipboardGtk and PasteboardHelper now.
WebCoreSupport/DragClientGtk.h: Small style fix.
webkit/webkitprivate.h: Add a HashMap of contexts and DataObjects here to to represent all current drag operations.
webkit/webkitwebview.cpp: (webkit_web_view_dispose): Clear all data objects during disposal. (webkit_web_view_drag_end): When a drag is over, just remove it from the map. (webkit_web_view_drag_data_get): To get the drag data, just grab it from the DataObject.

9:30 AM Changeset in webkit [58884] by pfeldman@chromium.org

6 edits in trunk/WebCore

2010-05-06 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Timothy Hatcher.

Web Inspector: Do not show content tab for resources
not supporting content preview.

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

English.lproj/localizedStrings.js:
inspector/front-end/FontView.js: (WebInspector.FontView.prototype.hasContentTab):
inspector/front-end/ImageView.js: (WebInspector.ImageView.prototype.hasContentTab):
inspector/front-end/ResourceView.js: (WebInspector.ResourceView): (WebInspector.ResourceView.prototype._selectTab): (WebInspector.ResourceView.prototype.hasContentTab):
inspector/front-end/SourceView.js: (WebInspector.SourceView.prototype.hasContentTab):

9:23 AM Changeset in webkit [58883] by tonikitoo@webkit.org

1 edit
2 adds in trunk/LayoutTests

2010-05-05 Antonio Gomes <tonikitoo@webkit.org>

Reviewed by Kenneth Christiansen.

Spatial Navigation: Add LayoutTest with frame flattening feature on.
https://bugs.webkit.org/show_bug.cgi?id=38585

fast/events/spatial-navigation/snav-iframe-flattening-simple-expected.txt: Added.
fast/events/spatial-navigation/snav-iframe-flattening-simple.html: Added.

9:17 AM Changeset in webkit [58882] by tonikitoo@webkit.org

3 edits in trunk/WebCore

2010-05-06 Antonio Gomes <tonikitoo@webkit.org>

Reviewed by Kenneth Christiansen.

Spatial Navigation: adapt the logic of {deep}findFocusableNodeInDirection to do traversal starting from Node* not Document*
https://bugs.webkit.org/show_bug.cgi?id=37803

Instead of receiving a Document pointer as incoming parameter, patch modifies
findFocusableNodeInDirection and deepFindFocusableNodeInDirection methods to
receive a Node pointer as start for content traversal.
This way we can make good use of deepFindFocusableNodeInDirection to traverse
other scrollable container like scrollable div's, and not only frames or iframes.

Patch also makes use of 'while' instead of 'for' to loop control, that gives move
flexibility to the incremental step: e.g. if a scrollable div was processed, the incremental
step in the loop does not have to do node->traverseNextNode() but node->traverseNextSibling().

No behavior change. It is a preparation for supporting scrollable containers in Spatial
Navigation.

page/FocusController.cpp: (WebCore::FocusController::advanceFocusDirectionally): (WebCore::FocusController::findFocusableNodeInDirection): (WebCore::FocusController::deepFindFocusableNodeInDirection):
page/FocusController.h:

9:04 AM Changeset in webkit [58881] by Csaba Osztrogonác

2 edits in trunk/WebCore

Unreviewed WinCE buildfix after r58842.
Preprocessor doesn't understand "true", changed to "1"

platform/graphics/MediaPlayer.cpp:

8:31 AM Changeset in webkit [58880] by mnaganov@chromium.org

2 edits in trunk/WebCore

2010-05-06 Mikhail Naganov <mnaganov@chromium.org>

Reviewed by Pavel Feldman.

Temporarily disable 'console.profiles' until we can distinguish
functions from different frames

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

bindings/v8/custom/V8ConsoleCustom.cpp: (WebCore::V8Console::profilesAccessorGetter):

8:13 AM Changeset in webkit [58879] by vitalyr@chromium.org

2 edits
6 adds in trunk/LayoutTests

2010-05-06 Vitaly Repeshko <vitalyr@chromium.org>

[chromium] Unreviewed. Updating expectations.

platform/chromium-linux/fast/url/segments-expected.txt: Added.
platform/chromium-linux/fast/url/segments-from-data-url-expected.txt: Added.
platform/chromium-mac/fast/url/segments-expected.txt: Added.
platform/chromium-mac/fast/url/segments-from-data-url-expected.txt: Added.
platform/chromium-win/fast/url/segments-expected.txt: Added.
platform/chromium-win/fast/url/segments-from-data-url-expected.txt: Added.
platform/chromium/test_expectations.txt:

7:12 AM QtWebKitTriageRoster edited by Simon Hausmann

(diff)

7:09 AM Changeset in webkit [58878] by eric@webkit.org

26 edits in trunk

2010-05-06 Steve Block <steveblock@google.com>

Reviewed by Eric Seidel.

MAC_JAVA_BRIDGE should be renamed JAVA_BRIDGE
https://bugs.webkit.org/show_bug.cgi?id=38544

wtf/Platform.h:

2010-05-06 Steve Block <steveblock@google.com>

Reviewed by Eric Seidel.

MAC_JAVA_BRIDGE should be renamed JAVA_BRIDGE
https://bugs.webkit.org/show_bug.cgi?id=38544

No new tests, build fix only.

DerivedSources.make:
bindings/js/ScriptController.cpp: (WebCore::ScriptController::ScriptController):
bindings/js/ScriptController.h:
bindings/js/ScriptControllerMac.mm: (WebCore::ScriptController::createScriptInstanceForWidget):
bridge/jni/JNIBridge.cpp:
bridge/jni/JNIBridge.h:
bridge/jni/JNIUtility.cpp:
bridge/jni/JNIUtility.h:
bridge/jni/jni_jsobject.h:
bridge/jni/jni_jsobject.mm:
bridge/jni/jni_objc.mm:
bridge/jni/jsc/JNIBridgeJSC.cpp:
bridge/jni/jsc/JNIBridgeJSC.h:
bridge/jni/jsc/JNIUtilityPrivate.cpp:
bridge/jni/jsc/JNIUtilityPrivate.h:
bridge/jni/jsc/JavaClassJSC.cpp:
bridge/jni/jsc/JavaClassJSC.h:
bridge/jni/jsc/JavaInstanceJSC.cpp:
bridge/jni/jsc/JavaInstanceJSC.h:
loader/FrameLoaderClient.h:

2010-05-06 Steve Block <steveblock@google.com>

Reviewed by Eric Seidel.

MAC_JAVA_BRIDGE should be renamed JAVA_BRIDGE
https://bugs.webkit.org/show_bug.cgi?id=38544

WebCoreSupport/WebFrameLoaderClient.h:
WebCoreSupport/WebFrameLoaderClient.mm:

6:12 AM Changeset in webkit [58877] by eric@webkit.org

2 edits in trunk/WebKit/qt

2010-05-06 Andreas Kling <andreas.kling@nokia.com>

Reviewed by Simon Hausmann.

[Qt] Patch to fix compilation warnings for QGraphicsWebView
https://bugs.webkit.org/show_bug.cgi?id=37428

Patch by Alexis Menard <alexis.menard@nokia.com>

Api/qgraphicswebview.cpp: (QGraphicsWebView::itemChange):

6:00 AM Changeset in webkit [58876] by eric@webkit.org

8 edits in trunk

2010-05-06 Simon Hausmann <simon.hausmann@nokia.com>

Reviewed by Kenneth Rohde Christiansen.

[Qt] Replace public inspector url with private property for QtLauncher
https://bugs.webkit.org/show_bug.cgi?id=35340

Replace the public API with a private dynamic property until this feature
is ready.

Api/qwebsettings.cpp:
Api/qwebsettings.h:
WebCoreSupport/InspectorClientQt.cpp: (WebCore::InspectorClientQt::openInspectorFrontend):
symbian/bwins/QtWebKitu.def:
symbian/eabi/QtWebKitu.def:

2010-05-06 Simon Hausmann <simon.hausmann@nokia.com>

Reviewed by Kenneth Rohde Christiansen.

[Qt] Replace public inspector url with private property for QtLauncher
https://bugs.webkit.org/show_bug.cgi?id=35340

Replace the public API with a private dynamic property until this feature
is ready.

QtLauncher/main.cpp: (LauncherWindow::init):

2:44 AM Changeset in webkit [58875] by alex

4 edits in trunk

2010-05-05 Alejandro G. Castro <alex@igalia.com>

Reviewed by Xan Lopez.

Fixed the gobject introspection compilation with the new DOM
bindings, we needed to add DOM objects.

GNUmakefile.am:

2:31 AM Changeset in webkit [58874] by eric@webkit.org

2 edits
2 adds in trunk/JavaScriptCore

2010-05-06 Adam Barth <abarth@webkit.org>

Reviewed by Maciej Stachowiak.

[WTFURL] Add a class to represent the segments of a URL
https://bugs.webkit.org/show_bug.cgi?id=38566

When paired with a string, this class represents a parsed URL.

wtf/url/src/URLSegments.cpp: Added. (WTF::URLSegments::length): (WTF::URLSegments::charactersBefore):
wtf/url/src/URLSegments.h: Added. (WTF::URLSegments::): (WTF::URLSegments::URLSegments):
wtf/url/wtfurl.gyp:

2:19 AM Changeset in webkit [58873] by eric@webkit.org

11 edits in trunk

2010-05-06 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.

drop support for sessionStorage in sandbox iframes
https://bugs.webkit.org/show_bug.cgi?id=38151

Update test expectation. This test was originally created to show that
we complied with the letter of the spec, but our behavior is insecure.
In the future, we might revert this change and implement a separate
storage area for each unique origin.

fast/frames/resources/sandboxed-iframe-storage-disallowed.html:
fast/frames/sandboxed-iframe-storage-expected.txt:

2010-05-06 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.

drop support for sessionStorage in sandbox iframes
https://bugs.webkit.org/show_bug.cgi?id=38151

This patch causes us to throw a security exception when a sandboxed
iframe attempts to access sessionStorage, matching our behavior for
localStorage. The letter of the spec asks us to create a separate
storage area for each unique origin. We might want to do that in a
future patch, but throwing a security error seems like a safe move now.

page/DOMWindow.cpp: (WebCore::DOMWindow::sessionStorage): (WebCore::DOMWindow::localStorage):
page/DOMWindow.h:
page/DOMWindow.idl:
page/SecurityOrigin.h: (WebCore::SecurityOrigin::canAccessSessionStorage):
storage/StorageEventDispatcher.cpp: (WebCore::StorageEventDispatcher::dispatch):

2010-05-06 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.

drop support for sessionStorage in sandbox iframes
https://bugs.webkit.org/show_bug.cgi?id=38151

Update client of sessionStorage to handle exceptions.

src/StorageAreaProxy.cpp: (WebCore::StorageAreaProxy::storageEvent):

1:51 AM Changeset in webkit [58872] by aestes@apple.com

4 edits
3 adds in trunk

WebCore: Allow forms submitting to target="_blank" to open popups if the submission
originated from a user gesture.

Reviewed by Maciej Stachowiak.

https://bugs.webkit.org/show_bug.cgi?id=37335
<rdar://problem/7884980>

Test: fast/events/popup-allowed-from-gesture-initiated-form-submit.html

bindings/js/ScriptController.cpp:

(WebCore::ScriptController::processingUserGesture): If no DOM event is
being processed, consult UserGestureIndicator to determine return value.

bindings/js/ScriptController.h: Moved the logic of processingUserGestureEvent()

into processingUserGesture().

LayoutTests: Add a test to ensure popups are allowed when the user submits a form with target="_blank".

Reviewed by Maciej Stachowiak.

https://bugs.webkit.org/show_bug.cgi?id=37335
<rdar://problem/7884980>

fast/events/popup-allowed-from-gesture-initiated-form-submit-expected.txt: Added.
fast/events/popup-allowed-from-gesture-initiated-form-submit.html: Added.
fast/events/resources/popup-allowed-from-gesture-initiated-form-submit-target.html: Added.

1:10 AM Changeset in webkit [58871] by xan@webkit.org

3 edits in trunk/WebCore

2010-05-06 Xan Lopez <xlopez@igalia.com>

Rubber-stamped by Eric Seidel.

[GTK] Refactor GenerateProperties in CodeGenerationGObject.pm
https://bugs.webkit.org/show_bug.cgi?id=38577

Thinko caught while refactoring, $custom variable was not
initialized.

bindings/scripts/CodeGeneratorGObject.pm:
bindings/scripts/test/GObject/WebKitDOMTestObj.cpp: (webkit_dom_test_obj_set_property): (webkit_dom_test_obj_get_property): (webkit_dom_test_obj_class_init):

1:09 AM Changeset in webkit [58870] by xan@webkit.org

2 edits in trunk/WebCore

2010-05-06 Xan Lopez <xlopez@igalia.com>

Rubber-stamped by Eric Seidel.

[GTK] Refactor GenerateProperties in CodeGenerationGObject.pm
https://bugs.webkit.org/show_bug.cgi?id=38577

Refactor GenerateProperty out of GenerateProperties. This is in
preparation for EventListeren attributes, which won't generate
normal GObject properties.

bindings/scripts/CodeGeneratorGObject.pm:

1:01 AM Changeset in webkit [58869] by eric@webkit.org

5 edits in trunk/WebCore

2010-05-06 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Joseph Pecoraro.

Web Inspector: follow up to linkify event listeners.

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

bindings/js/ScriptEventListener.cpp: (WebCore::eventListenerHandlerLocation):
bindings/v8/ScriptEventListener.cpp: (WebCore::eventListenerHandlerLocation):
inspector/front-end/ElementsPanel.js: (WebInspector.ElementsPanel.prototype.linkifyNodeReference):
inspector/front-end/EventListenersSidebarPane.js:

12:05 AM Changeset in webkit [58868] by eric@webkit.org

2 edits in trunk/WebKitTools

2010-05-05 Joseph Pecoraro <Joseph Pecoraro>

Reviewed by Pavel Feldman.

Web Inspector: build-webkit --inspector-frontend Should Exclude *.re2js
https://bugs.webkit.org/show_bug.cgi?id=38449

Scripts/webkitdirs.pm:

May 5, 2010:

11:54 PM Changeset in webkit [58867] by eric@webkit.org

14 edits
4 adds in trunk

2010-05-05 Charles Wei <charles.wei@torchmobile.com.cn>

Reviewed by George Staikos

https://bugs.webkit.org/show_bug.cgi?id=37848
Add test cases for -wap-input-format and -wap-input-required of WCSS

fast/wcss/wap-input-format-expected.txt: Added.
fast/wcss/wap-input-format.xhtml: Added.
fast/wcss/wap-input-required-expected.txt: Added.
fast/wcss/wap-input-required.xhtml: Added.

2010-05-05 Charles Wei <charles.wei@torchmobile.com.cn>

Reviewed by George Staikos

https://bugs.webkit.org/show_bug.cgi?id=37848
This patch adds WCSS -wap-input-format and -wap-input-required support to WebKit

Tests: fast/wcss/wap-input-format.xhtml

fast/wcss/wap-input-required.xhtml

css/CSSParser.cpp: (WebCore::CSSParser::parseValue): (WebCore::CSSParser::parseWCSSInputProperty):
css/CSSParser.h:
css/CSSStyleSelector.cpp: (WebCore::CSSStyleSelector::applyProperty):
css/WCSSPropertyNames.in:
dom/InputElement.cpp: (WebCore::InputElement::sanitizeValue): (WebCore::InputElement::handleBeforeTextInsertedEvent): (WebCore::InputElementData::InputElementData): (WebCore::formatCodes): (WebCore::cursorPositionToMaskIndex): (WebCore::InputElement::isConformToInputMask): (WebCore::InputElement::validateInputMask):
dom/InputElement.h: (WebCore::InputElementData::inputFormatMask): (WebCore::InputElementData::setInputFormatMask): (WebCore::InputElementData::maxInputCharsAllowed): (WebCore::InputElementData::setMaxInputCharsAllowed):
html/HTMLInputElement.cpp: (WebCore::HTMLInputElement::setWapInputFormat):
html/HTMLInputElement.h: (WebCore::HTMLInputElement::data):
wml/WMLInputElement.h: (WebCore::WMLInputElement::data):

2010-05-05 Charles Wei <charles.wei@torchmobile.com.cn>

Reviewed by George Staikos

This patch adds WCSS -wap-input-format and -wap-input-required support to WebKit
Make the test cases in fast/wcss optionional only when WCSS is enabled.

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

Scripts/old-run-webkit-tests:
Scripts/webkitperl/features.pm:

11:23 PM Changeset in webkit [58866] by hamaji@chromium.org

2 edits in trunk/LayoutTests

2010-05-05 Shinichiro Hamaji <hamaji@chromium.org>

Reviewed by Eric Seidel.

[GTK] Markup of copied non ASCII characters is broken
https://bugs.webkit.org/show_bug.cgi?id=37918

This test was fixed by http://trac.webkit.org/changeset/58468

platform/gtk/Skipped:

11:06 PM Changeset in webkit [58865] by mitz@apple.com

2 edits in trunk/WebKit/mac

Fixed a crash when closing Top Sites after r58847.

Reviewed by Mark Rowe.

WebView/WebFrameView.mm:

(-[WebFrameView webFrame]): Null-check _private.

11:02 PM Changeset in webkit [58864] by morrita@google.com

6 edits in trunk/WebCore

2010-05-05 MORITA Hajime <morrita@google.com>

Reviewed by Darin Adler.

https://bugs.webkit.org/show_bug.cgi?id=38150
Refactoring: webkitEditableContentChangedEvent should be handled by the owner of appropriate the renderer.

Moved a part of code chunk in HTMLFormControlElementWithState::defaultEventHandler()
which accesses the renderer from foreign node,
to TextControlInnerTextElement::defaultEventHandler() which owns the renderer.

No new tests. No behavioral change.

html/HTMLFormControlElement.cpp: (WebCore::HTMLFormControlElementWithState::defaultEventHandler):
html/HTMLFormControlElement.h:
html/HTMLInputElement.h:
html/HTMLTextAreaElement.h:
rendering/TextControlInnerElements.cpp: (WebCore::TextControlInnerTextElement::defaultEventHandler):

10:47 PM Changeset in webkit [58863] by Csaba Osztrogonác

2 edits in trunk/LayoutTests

[Qt] Enable Sputnik tests.
https://bugs.webkit.org/show_bug.cgi?id=38457

Rubber-stamped by Alexey Proskuryakov.

platform/qt/Skipped: Only skip tests that fail.

10:40 PM Changeset in webkit [58862] by tkent@chromium.org

3 edits in trunk/WebKitTools

2010-05-05 Kent Tamura <tkent@chromium.org>

Reviewed by Eric Seidel.

[DRT/Chromium] Remove InitWebCoreSystemInterface() call
https://bugs.webkit.org/show_bug.cgi?id=38624

Chromium r45167 <http://src.chromium.org/viewvc/chrome?view=rev&revision=45167>
added InitWebCoreSystemInterface() to webkit/support/platform_support_mac.mm.
So we don't need to call it from DumpRenderTree.cpp anymore.

DumpRenderTree/DumpRenderTree.gyp/DumpRenderTree.gyp:
DumpRenderTree/chromium/DumpRenderTree.cpp: (main): Remove InitWebCoreSystemInterface().

10:34 PM Changeset in webkit [58861] by eric@webkit.org

3 edits
1 delete in trunk/LayoutTests

2010-05-05 Steven Lai <steven_lai@asia.apple.com>

Reviewed by Alexey Proskuryakov.

Remove platform specific test result
Don't print the formatted string of the z-index in the test case
since there's a risk that the output on different c stdlib implementations/platforms/locale could be different
https://bugs.webkit.org/show_bug.cgi?id=38617.

fast/dom/clone-node-z-index-expected.txt:
fast/dom/clone-node-z-index.html:
platform/win/fast/dom/clone-node-z-index-expected.txt: Removed.

10:23 PM Changeset in webkit [58860] by eric@webkit.org

1 edit
3 adds in trunk/LayoutTests

2010-05-05 Adam Barth <abarth@webkit.org>

Reviewed by Alexey Proskuryakov.

Add a test of URL segmentation from a data URL base
https://bugs.webkit.org/show_bug.cgi?id=38606

As requested by Alexey. These tests find some interesting behavior in
both KURL and GURL. We'll need to go through them in more detail later
to understand what the right behavior is.

fast/url/script-tests/segments-from-data-url.js: Added.
fast/url/segments-from-data-url-expected.txt: Added.
fast/url/segments-from-data-url.html: Added.

10:11 PM Changeset in webkit [58859] by eric@webkit.org

2 edits
6 adds in trunk/LayoutTests

2010-05-05 Adam Barth <abarth@webkit.org>

Reviewed by Alexey Proskuryakov.

Add a test of URL segmentation
https://bugs.webkit.org/show_bug.cgi?id=38600

These tests are based on URLParser.Standard from
http://code.google.com/p/google-url/source/browse/trunk/src/url_parse_unittest.cc

fast/url/resources/utilities.js: (canonicalize):
- I changed this to use deterministic IDs because (theoretically) the old way was flaky in case the IDs collided.

(segments):

Here we're using HTMLAnchorElement's attributes to infer the internal segmentation. Unforunately, there don't seem to be accessors for usernames and passwords of URLs.

fast/url/script-tests/segments.js: Added.
- I'm not 100% sure all the test cases here represent the intent of the original test author. The tricky bit is we don't have a good way to test absolute URL parsing from the web platform, which means a bunch of these become tests of how we handle relative URLs. However, I'm not one to turn down more test cases.
fast/url/script-tests/trivial-segments.js: Added.
- This is more of a test of the testing harness to make sure its working properly when running on other browsers.
fast/url/segments-expected.txt: Added.
fast/url/trivial-segments-expected.txt: Added.
- As usual, these have their "PASS" expectations set to the Chromium behavior but have their -expected.txt set to the normal WebKit behavior, letting us track differences between the two.
fast/url/segments.html: Added.
fast/url/trivial-segments.html: Added.

9:50 PM Changeset in webkit [58858] by eric@webkit.org

2 edits in trunk/JavaScriptCore

2010-05-05 Tony Gentilcore <tonyg@chromium.org>

Reviewed by Adam Barth.

Adding comment with link to RefPtr documentation.
https://bugs.webkit.org/show_bug.cgi?id=38601

wtf/RefPtr.h:

9:28 PM Changeset in webkit [58857] by eric@webkit.org

2 edits in trunk/WebCore

2010-05-05 Joseph Pecoraro <Joseph Pecoraro>

Reviewed by Timothy Hatcher.

Web Inspector: Line Numbers should be Aligned with the Source Code Line
https://bugs.webkit.org/show_bug.cgi?id=38593

If there is a message bubble, such as an error message, in a source view
the line number should be in line with the source code line, not centered
between the source and bubble.

inspector/front-end/textViewer.css: (.webkit-line-number): added vertical-align: top

8:16 PM Changeset in webkit [58856] by mitz@apple.com

2 edits in trunk/WebKit/mac

Fixed test crashes after r58847.

Rubber-stamped by Mark Rowe.

WebView/WebHTMLView.mm:

(setNeedsDisplayInRect): Null-check the frame.

6:51 PM Changeset in webkit [58855] by dpranke@chromium.org

2 edits in trunk/LayoutTests

2010-05-05 Dirk Pranke <dpranke@chromium.org>

Unreviewed, expectations change.

platform/chromium/test_expectations.txt:

6:36 PM Changeset in webkit [58854] by mitz@apple.com

3 edits in trunk/WebCore

Fix the decelerated compositing build.

page/FrameView.cpp:

(WebCore::FrameView::isEnclosedInCompositingLayer):

page/FrameView.h:

6:33 PM Changeset in webkit [58853] by dpranke@chromium.org

6 edits
3 adds in trunk/WebKitTools

2010-05-05 Dirk Pranke <dpranke@chromium.org>

Reviewed by Eric Seidel.

new-run-webkit-tests: clean up newline handling in printing

The new printing module seems to handle newlines somewhat
inconsistently, especially in --verbose mode. This change cleans up
the code to make things more consistent and adds a bunch of unit tests.

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

Scripts/webkitpy/common/array_stream.py: Added.
Scripts/webkitpy/common/array_stream_unittest.py: Added.
Scripts/webkitpy/layout_tests/layout_package/metered_stream.py:
Scripts/webkitpy/layout_tests/layout_package/metered_stream_unittest.py: Added.
Scripts/webkitpy/layout_tests/layout_package/printing.py:
Scripts/webkitpy/layout_tests/layout_package/printing_unittest.py:
Scripts/webkitpy/layout_tests/run_webkit_tests.py:
Scripts/webkitpy/layout_tests/run_webkit_tests_unittest.py:

6:12 PM Changeset in webkit [58852] by jamesr@google.com

3 edits in trunk/WebKitTools

2010-05-05 James Robinson <jamesr@chromium.org>

Reviewed by Adam Barth.

Ban the single letter 'l' as an identifier name
http://trac.webkit.org/changeset/58844

Add a lint rule to ban the single letter 'l' as an identifier name
since it is very easy to confuse with the numeral '1', especially
in code like WebCore/css/CSSHelper.cpp.

See http://trac.webkit.org/changeset/58844 as an example of a bug
caused by confusing short variable names.

Scripts/webkitpy/style/checkers/cpp.py:
Scripts/webkitpy/style/checkers/cpp_unittest.py:

6:08 PM Changeset in webkit [58851] by barraclough@apple.com

9 edits in trunk

Bug 38604 - workers-gc2 crashing on Qt

Reviewed by Darin Adler.

This appears to be due to a couple of issues.
(1) When the atomic string table is deleted it does not clear the 'inTable' bit
on AtomicStrings - it implicitly assumes that all AtomicStrings have already
been deleted at this point (otherwise they will crash in their destructor when
they try to remove themselves from the atomic string table).
(2) We don't fix the ordering in which WTF::WTFThreadData and
WebCore::ThreadGlobalData are destructed.

We should make sure that ThreadGlobalData is cleaned up before worker threads
terminate and WTF::WTFThreadData is destroyed, and we should clear the inTable
bit of members on atomic string table destruction.

JavaScriptCore:

WTF changes (fix issue 1, above) - ensure inTable property is cleared when the
atomic string table is destroyed (also, rename InTable to IsAtomic, to make it
clear which table we are refering to!)

wtf/text/AtomicString.cpp:

(WebCore::AtomicStringTable::destroy):
(WebCore::CStringTranslator::translate):
(WebCore::UCharBufferTranslator::translate):
(WebCore::HashAndCharactersTranslator::translate):
(WebCore::AtomicString::add):

wtf/text/StringImpl.cpp:

(WebCore::StringImpl::~StringImpl):

wtf/text/StringImpl.h:

(WebCore::StringImpl::isAtomic):
(WebCore::StringImpl::setIsAtomic):

wtf/text/StringImplBase.h:

WebCore:

WTF changes (fix issue 2, above) - clean up the thread data on worker termination.

platform/ThreadGlobalData.cpp:

(WebCore::ThreadGlobalData::~ThreadGlobalData):
(WebCore::ThreadGlobalData::destroy):

platform/ThreadGlobalData.h:
workers/WorkerThread.cpp:

(WebCore::WorkerThread::workerThread):

6:05 PM Changeset in webkit [58850] by bweinstein@apple.com

1 edit
1 add in trunk/LayoutTests

Add Windows-specific results for clone-node-z-index.html. The difference in results is harmless (the number is the same),
but the difference is being tracked by https://bugs.webkit.org/show_bug.cgi?id=38617.

Rubber-stamped by Mark Rowe.

platform/win/fast/dom/clone-node-z-index-expected.txt: Added.

5:53 PM Changeset in webkit [58849] by crogers@google.com

8 edits in branches/audio/WebCore/audio

Fix problems with node disconnection

add notifyInputConnection() method to AudioNode

5:53 PM Changeset in webkit [58848] by crogers@google.com

2 edits in branches/audio/WebCore/audio

Cleanup AudioContext - clarify memory management a little bit

5:37 PM Changeset in webkit [58847] by mitz@apple.com

10 edits
1 move
4 adds in trunk

<rdar://problem/7932072> Iframes in composited layers don’t repaint correctly (affects Yahoo! Mail with Flash Player 10.1)
https://bugs.webkit.org/show_bug.cgi?id=38427

Reviewed by Simon Fraser.

WebCore:

Test: compositing/iframes/iframe-in-composited-layer.html

WebCore.base.exp: Export FrameView::isEnclosedInCompositingLayer().
page/FrameView.cpp:

(WebCore::FrameView::isEnclosedInCompositingLayer): Added this predicate.

page/FrameView.h:

WebKit:

WebKit.xcodeproj/project.pbxproj: Renamed WebClipView.m to WebClipView.mm and changed it to

Objective-C++.

WebKit/mac:

WebView/WebClipView.m: Renamed to WebClipView.mm.
WebView/WebClipView.mm:

(-[WebClipView visibleRect]): Added this override, which for instances used for WebFrameViews in
composited layers, returns the clip view’s entire bounds. This prevents drawing from being clipped to
AppKit’s idea of what part of the view would be visible if it was drawn as part of the view hierarchy.
Since it is drawn into a compositing layer, that’s irrelevant, and we should not be clipping.

WebView/WebHTMLView.mm:

(setCursor): Style tweak.
(setNeedsDisplayInRect): Added. Replaces the default implementation of -[NSView setNeedsDisplayInRect:],
so that if the receiver is a descendant of a WebFrameView that draws into a composited layer, the invalidation
is routed back through the WebCore FrameView, which propagates it to the layer.
(+[WebHTMLViewPrivate initialize]): Swizzle the setNeedsDisplayInRect: override in.
(-[WebHTMLView visibleRect]): Removed whitespace.

WebView/WebView.mm:

(layerSyncRunLoopObserverCallBack): If we bailed out on syncing, due to pending layout, do an eager layout
in preparation for the displaying of compositing layers.

LayoutTests:

compositing/iframes/iframe-in-composited-layer.html: Added.
platform/mac/compositing/iframes/iframe-in-composited-layer-expected.checksum: Added.
platform/mac/compositing/iframes/iframe-in-composited-layer-expected.png: Added.
platform/mac/compositing/iframes/iframe-in-composited-layer-expected.txt: Added.

5:20 PM Changeset in webkit [58846] by sullivan@apple.com

2 edits in trunk/WebKit/mac

<rdar://problem/7942606> Output appears in Console when exiting Safari with multiple windows opened

Reviewed by Mark Rowe.

WebView/WebView.mm:

(+[WebView closeAllWebViews]):
Make copy of allWebViewsSet to avoid mutating it while iterating through it.

5:02 PM Changeset in webkit [58845] by cmarrin@apple.com

3 edits
4 adds in trunk

2010-05-05 Chris Marrin <cmarrin@apple.com>

Reviewed by Simon Fraser.

Got composited iframes showing up on Mac again
https://bugs.webkit.org/show_bug.cgi?id=38565

This was broken by http://trac.webkit.org/changeset/58798. That
change connected iframes through the parent document to the root
of the layer tree. That is correct for Windows, but not for Mac.
So the places where the linkage is made were wrapped in
shouldPropagateCompositingToIFrameParent() calls, which is
always false for Mac.

Test: compositing/iframes/composited-iframe-alignment.html

rendering/RenderLayerCompositor.cpp:Avoid doing composited iframe linkage on Mac (WebCore::RenderLayerCompositor::rebuildCompositingLayerTree): (WebCore::RenderLayerCompositor::didMoveOnscreen): (WebCore::RenderLayerCompositor::willMoveOffscreen): (WebCore::RenderLayerCompositor::ensureRootPlatformLayer):

4:24 PM Changeset in webkit [58844] by ap@apple.com

3 edits
2 adds in trunk

Reviewed by Darin Adler.

https://bugs.webkit.org/show_bug.cgi?id=38260
<rdar://problem/7917548> Fix whitespace removing in deprecatedParseURL().

Broken all the way since r4 (yes, that's a revision number).

Test: http/tests/security/xss-DENIED-javascript-with-spaces.html

css/CSSHelper.cpp: (WebCore::deprecatedParseURL): Fixed loop conditions for remaining length.

4:12 PM Changeset in webkit [58843] by Chris Fleizach

5 edits
2 adds in trunk

Many AXUnknown showing up in the hierarchy
https://bugs.webkit.org/show_bug.cgi?id=38607

Reviewed by Beth Dakin.

WebCore:

The RenderTableSection should not be an accessible element.

Test: platform/mac/accessibility/parent-of-table-row-is-table.html

accessibility/AccessibilityRenderObject.cpp:

(WebCore::AccessibilityRenderObject::determineAccessibilityRole):

LayoutTests:

platform/mac-leopard/Skipped:
platform/mac-tiger/Skipped:
platform/mac/accessibility/parent-of-table-row-is-table-expected.txt: Added.
platform/mac/accessibility/parent-of-table-row-is-table.html: Added.

3:59 PM Changeset in webkit [58842] by Csaba Osztrogonác

2 edits in trunk/WebCore

[Qt]Disable Qt Multimedia backend for HTML 5 Audio and Video elements
Fall back to the Phonon backend until the release situation has been cleared up.

Reviewed by Simon Hausmann.

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

Buildfix for QT_VERSION >= 0x040700 after r58810.

platform/graphics/MediaPlayer.cpp:

3:42 PM Changeset in webkit [58841] by jianli@chromium.org

9 edits
3 adds in trunk

XMLHttpRequestUpload events do not fire when sending a raw file or FormData object.
https://bugs.webkit.org/show_bug.cgi?id=37771

Reviewed by David Levin.

WebCore:

Test: http/tests/local/formdata/upload-events.html

xml/XMLHttpRequest.cpp:

(WebCore::XMLHttpRequest::createRequest):

LayoutTests:

Add a layout test to test upload events firing.

http/tests/local/formdata/resources/send-form-data-common.js:

(dumpResponse):
(sendFormData):
(testSendingFormData):

http/tests/local/formdata/script-tests/send-form-data-with-sliced-file.js:
http/tests/local/formdata/script-tests/send-form-data.js:
http/tests/local/formdata/script-tests/upload-events.js: Added.
http/tests/local/formdata/upload-events-expected.txt: Added.
http/tests/local/formdata/upload-events.html: Added.
platform/gtk/Skipped: Skip the new test because eventSender.beginDragWithFiles is not supported in gtk platform.
platform/qt/Skipped: Skip the new test because eventSender.beginDragWithFiles is not supported in qt platform.
platform/win/Skipped: Skip the new test because eventSender.beginDragWithFiles is not supported in win platform.

3:28 PM Changeset in webkit [58840] by Csaba Osztrogonác

2 edits in trunk/WebCore

[Qt] Qt port should use its own QUuid class to create UUID string.
https://bugs.webkit.org/show_bug.cgi?id=38581

Reviewed by Simon Hausmann.

platform/UUID.cpp:

(WebCore::createCanonicalUUIDString): Added PLATFORM(QT) case.

2:51 PM Changeset in webkit [58839] by darin@chromium.org

11 edits in trunk/WebKit/chromium

2010-05-05 Darin Fisher <darin@chromium.org>

Reviewed by Dimitri Glazkov.

[chromium] clear out deprecated methods that are no longer used
https://bugs.webkit.org/show_bug.cgi?id=38610

public/WebFormElement.h:
public/WebInputElement.h:
public/WebKitClient.h:
public/WebView.h:
public/WebViewClient.h: (WebKit::WebViewClient::createPopupMenu):
src/ChromeClientImpl.cpp: (WebKit::ChromeClientImpl::popupOpened):
src/WebFormElement.cpp:
src/WebInputElement.cpp: (WebKit::WebInputElement::setSelectionRange):
src/WebViewImpl.cpp: (WebKit::WebViewImpl::hideSuggestionsPopup):
src/WebViewImpl.h:

2:16 PM QtWebKitTriageRoster edited by Simon Hausmann

(diff)

1:16 PM Changeset in webkit [58838] by dpranke@chromium.org

2 edits in trunk/LayoutTests

2010-05-05 Dirk Pranke <dpranke@chromium.org>

Unreviewed, expectations fix.

Update expectations with tests that are now passing (possibly flaky?).

platform/chromium/test_expectations.txt:

1:06 PM Changeset in webkit [58837] by dpranke@chromium.org

2 edits in trunk/LayoutTests

2010-05-05 Dirk Pranke <dpranke@chromium.org>

Unreviewed, expectations fix.

Mark svg/custom/mask-changes.svg and http/tests/loading/bad-scheme-subframe.html as failing on Chromium Linux until they can be triaged.

platform/chromium/test_expectations.txt:

12:19 PM Changeset in webkit [58836] by ap@apple.com

3 edits
4 deletes in trunk

Roll out r58830 for breaking tests.

Was: https://bugs.webkit.org/show_bug.cgi?id=38546
Node.focus() fails to move focus from subframe properly

12:11 PM Changeset in webkit [58835] by crogers@google.com

2 edits in branches/audio/WebCore/audio

Call to pull() must take return value into account for non-inplace case

12:11 PM Changeset in webkit [58834] by crogers@google.com

1 edit in branches/audio/WebCore/audio/AudioBufferSourceNode.cpp

Audio buffer source node needs default number of channels

12:10 PM Changeset in webkit [58833] by crogers@google.com

2 edits in branches/audio/WebCore/audio

Keep track of mixer input availability

11:52 AM Changeset in webkit [58832] by jianli@chromium.org

12 edits
2 adds in trunk/WebCore

Implement FileReader class.
https://bugs.webkit.org/show_bug.cgi?id=38157

Reviewed by Adam Barth.

This patch only contains the implementation of FileReader class as defined
in the File API spec: http://www.w3.org/TR/file-upload/#dfn-filereader.

New test will be added when a IDL is exposed and the FILE_READER is turned
on.

GNUmakefile.am:
WebCore.gypi:
WebCore.pro:
WebCore.vcproj/WebCore.vcproj:
WebCore.xcodeproj/project.pbxproj:
dom/EventNames.h:
dom/EventTarget.cpp:

(WebCore::EventTarget::toFileReader):

dom/EventTarget.h:
html/FileError.h:

(WebCore::FileError::FileError):

html/FileReader.cpp: Added.
html/FileReader.h: Added.
html/FileStream.cpp:

(WebCore::FileStream::FileStream):
(WebCore::FileStream::openForRead):
(WebCore::FileStream::close):
(WebCore::FileStream::read):

html/FileStream.h:

11:35 AM Changeset in webkit [58831] by sfalken@apple.com

2 edits in trunk/WebCore

2010-05-05 Steve Falkenburg <sfalken@apple.com>

Reviewed by Maciej Stachowiak.

REGRESSION(r57969) Image decoder is repeatedly destroyed/created in CoreGraphics-based Windows WebKit
https://bugs.webkit.org/show_bug.cgi?id=38595

platform/graphics/cg/ImageSourceCG.cpp: (WebCore::ImageSource::setData):

11:26 AM Changeset in webkit [58830] by ap@apple.com

3 edits
4 adds in trunk

Reviewed by Adele Peterson.

https://bugs.webkit.org/show_bug.cgi?id=38546
Node.focus() fails to move focus from subframe properly

Test: fast/frames/take-focus-from-iframe.html

html/HTMLFrameElementBase.cpp: (WebCore::HTMLFrameElementBase::setFocus): Don't clear focus if this frame doesn't have it. This can happen if page's and HTMLFrameElement's ideas of focused frame get out of sync temporarily.

11:24 AM How to CC yourself on (or make other changes to) many bugs at once created by Adam Roben

Initial version

11:20 AM WikiStart edited by Adam Roben

(diff)

11:17 AM Changeset in webkit [58829] by ap@apple.com

2 edits
1 add in trunk/WebCore

Reviewed by Adele Peterson.

https://bugs.webkit.org/show_bug.cgi?id=26824
<rdar://problem/7018610> EventHandler can operate on a wrong frame if focus changes during
keyboard event dispatch.

EventHandler object is tied to a frame, so it's wrong for it to continue processing a keyboard
event if focused frame changes between keydown and keypress.

manual-tests/focus-change-between-key-events.html: Added.

page/EventHandler.cpp: (WebCore::EventHandler::keyEvent): Bail out early if focused frame changes while dispatching keydown. Also made similar changes for Windows to maintain matching behavior, even though EventHandler was re-entered anyway due to WM_KEYDOWN and WM_CHAR being separate events.

11:14 AM Changeset in webkit [58828] by eric@webkit.org

3 edits in trunk/WebKit/chromium

2010-05-05 Hans Wennborg <hans@chromium.org>

Reviewed by Jeremy Orlow.

[chromium] Add quota parameter to WebStorageNamespace::createSessionStorageNamespace
https://bugs.webkit.org/show_bug.cgi?id=38589

public/WebStorageNamespace.h: (WebKit::WebStorageNamespace::createSessionStorageNamespace):
src/WebStorageNamespaceImpl.cpp: (WebKit::WebStorageNamespace::createSessionStorageNamespace):

10:53 AM Changeset in webkit [58827] by hamaji@chromium.org

2 edits in trunk/WebKitSite

2010-05-05 Shinichiro Hamaji <hamaji@chromium.org>

Reviewed by Adam Barth.

Commit and Review policy should encourage unofficial reviews
https://bugs.webkit.org/show_bug.cgi?id=38517

coding/commit-review-policy.html:

10:20 AM Changeset in webkit [58826] by eric@webkit.org

2 edits in trunk/WebCore

2010-05-05 Steve Block <steveblock@google.com>

Reviewed by Adam Barth.

MediaError.h is missing PassRefPtr.h include
https://bugs.webkit.org/show_bug.cgi?id=38575

No new tests, build fix only.

html/MediaError.h:

10:06 AM Changeset in webkit [58825] by eric@webkit.org

2 edits in trunk/WebCore

2010-05-05 Alexander Pavlov <apavlov@chromium.org>

Reviewed by Pavel Feldman.

Web Inspector: Audits panel: Resource counters get reset when switching panels while reloading page
https://bugs.webkit.org/show_bug.cgi?id=38579

inspector/front-end/AuditLauncherView.js: (WebInspector.AuditLauncherView.prototype.updateResourceTrackingState): (WebInspector.AuditLauncherView.prototype._updateResourceProgress):

10:03 AM Changeset in webkit [58824] by oliver@apple.com

2 edits in trunk/JavaScriptCore

2010-05-05 Oliver Hunt <oliver@apple.com>

Reviewed by Geoffrey Garen.

Make Array.join specialised for actual arrays
https://bugs.webkit.org/show_bug.cgi?id=38592

Bring Array.join in line with the other array methods by specialising for
this === an array.

runtime/ArrayPrototype.cpp: (JSC::arrayProtoFuncJoin):

9:48 AM Changeset in webkit [58823] by eric@webkit.org

2 edits in trunk/WebCore

2010-05-05 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Joseph Pecoraro.

Web Inspector: Doubleclick on line number should not trigger source editing.

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

inspector/front-end/TextViewer.js: (WebInspector.TextViewer.prototype._handleDoubleClick):

9:30 AM Changeset in webkit [58822] by Martin Robinson

2 edits in trunk/WebCore

2010-05-05 Martin Robinson <Martin Robinson>

Reviewed by Gustavo Noronha.

[GTK] Enable DOM clipboard and drag-and-drop access
https://bugs.webkit.org/show_bug.cgi?id=30623

Add support in the PasteboardHelper for images, in preparation for drag-and-drop data.

No new tests as functionality has not changed.

platform/gtk/PasteboardHelper.cpp: (WebCore::PasteboardHelper::fillSelectionData): Add support for images here. (WebCore::PasteboardHelper::targetListForDataObject): Add support for images here.

9:22 AM Changeset in webkit [58821] by eric@webkit.org

26 edits in trunk

2010-05-05 Stuart Morgan <stuartmorgan@chromium.org>

Reviewed by Darin Fisher.

Add a "focused" parameter to Widget::setFocus, and updates Document
so that Widget is informed of focus loss as well as focus gain.
Changes all existing setFocus implementations so that they ignore
the 'false' case, so no behavior is changed until individual
platforms are updated to handle the new case (if they want to).

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

dom/Document.cpp: (WebCore::Document::setFocusedNode):
platform/Widget.h:
platform/android/WidgetAndroid.cpp: (WebCore::Widget::setFocus):
platform/chromium/WidgetChromium.cpp: (WebCore::Widget::setFocus):
platform/efl/WidgetEfl.cpp: (WebCore::Widget::setFocus):
platform/gtk/WidgetGtk.cpp: (WebCore::Widget::setFocus):
platform/haiku/WidgetHaiku.cpp: (WebCore::Widget::setFocus):
platform/mac/WidgetMac.mm: (WebCore::Widget::setFocus):
platform/qt/WidgetQt.cpp: (WebCore::Widget::setFocus):
platform/win/WidgetWin.cpp: (WebCore::Widget::setFocus):
platform/wx/WidgetWx.cpp: (WebCore::Widget::setFocus):
plugins/PluginView.h:
plugins/PluginViewNone.cpp: (WebCore::PluginView::setFocus):
plugins/gtk/PluginViewGtk.cpp: (WebCore::PluginView::setFocus):
plugins/mac/PluginViewMac.cpp: (WebCore::PluginView::setFocus):
plugins/qt/PluginViewQt.cpp: (WebCore::PluginView::setFocus):
plugins/symbian/PluginViewSymbian.cpp: (WebCore::PluginView::setFocus):
plugins/win/PluginViewWin.cpp: (WebCore::PluginView::setFocus):

2010-05-05 Stuart Morgan <stuartmorgan@chromium.org>

Reviewed by Darin Fisher.

Update the plugin container's setFocus implementation to pass the new
boolean argument to the next layer.

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

src/WebPluginContainerImpl.cpp: (WebKit::WebPluginContainerImpl::setFocus):
src/WebPluginContainerImpl.h:

2010-05-05 Stuart Morgan <stuartmorgan@chromium.org>

Reviewed by Darin Fisher.

Update setFocus for the new boolean argument; no behavioral change.

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

WebCoreSupport/EmbeddedWidget.cpp: (EmbeddedWidget::setFocus):
WebCoreSupport/EmbeddedWidget.h:

9:01 AM Changeset in webkit [58820] by jeremy@chromium.org

2 edits in trunk/LayoutTests

2010-05-05 Jeremy Moskovich <jeremy@chromium.org>

Unreviewed, updated baselines.

skia r561 broke a bunch of chromium image baselines.
Update test_expectations to include remaining failures.

platform/chromium/test_expectations.txt:

8:18 AM Changeset in webkit [58819] by jeremy@chromium.org

4 edits
2 adds in trunk/LayoutTests

2010-05-05 Jeremy Moskovich <jeremy@chromium.org>

Unreviewed, updated baselines.

skia r561 broke a bunch of chromium image baselines.
Rebase a few tests that were missed by previous passes.

platform/chromium-linux/svg/W3C-SVG-1.1/filters-diffuse-01-f-expected.checksum:
platform/chromium-linux/svg/W3C-SVG-1.1/filters-diffuse-01-f-expected.png:
platform/chromium-linux/svg/custom/mask-changes-expected.checksum: Added.
platform/chromium-linux/svg/custom/mask-changes-expected.png: Added.
platform/chromium/test_expectations.txt:

8:08 AM Changeset in webkit [58818] by eric@webkit.org

5 edits in trunk/WebCore

2010-05-05 Steve Block <steveblock@google.com>

Reviewed by Pavel Feldman.

Move V8 ScriptDebugServer::topStackFrame() to ScriptCallStack
https://bugs.webkit.org/show_bug.cgi?id=38531

ScriptCallStack::create() uses ScriptDebugServer::topStackFrame(), but ScriptDebugServer
is guarded with ENABLE(JAVASCRIPT_DEBUGGER). This prevents ScriptCallStack from being
built on platforms that do not define ENABLE(JAVASCRIPT_DEBUGGER).

No new tests, build fix only.

bindings/v8/ScriptCallStack.cpp: (WebCore::ScriptCallStack::callLocation): (WebCore::ScriptCallStack::createUtilityContext): Helper for topStackFrame, moved from ScriptDebugServer (WebCore::ScriptCallStack::topStackFrame): Moved from ScriptDebugServer
bindings/v8/ScriptCallStack.h: (WebCore::ScriptCallStack::utilityContext): Helper for topStackFrame, moved from ScriptDebugServer
bindings/v8/ScriptDebugServer.cpp:
bindings/v8/ScriptDebugServer.h:

7:42 AM Changeset in webkit [58817] by jeremy@chromium.org

7 edits
1 delete in trunk/LayoutTests

2010-05-05 Jeremy Moskovich <jeremy@chromium.org>

Unreviewed, updated baselines - 6th and final batch.

skia r561 broke a bunch of chromium image baselines.
<long list of new image baselines redacted>

platform/chromium/test_expectations.txt:

7:18 AM Changeset in webkit [58816] by jeremy@chromium.org

671 edits
10 adds
62 deletes in trunk/LayoutTests

2010-05-05 Jeremy Moskovich <jeremy@chromium.org>

Unreviewed, updated baselines - 5th batch.

skia r561 broke a bunch of chromium image baselines.
<long list of new image baselines redacted>

platform/chromium/test_expectations.txt:

7:01 AM BuildingQtOnWindows edited by jocelyn.turcotte@nokia.com

(diff)

6:52 AM Changeset in webkit [58815] by jeremy@chromium.org

852 edits
4 adds
41 deletes in trunk/LayoutTests

2010-05-05 Jeremy Moskovich <jeremy@chromium.org>

Unreviewed, updated baselines - 4th batch.

skia r561 broke a bunch of chromium image baselines.
<long list of new image baselines redacted>

platform/chromium/test_expectations.txt:

6:49 AM QtWebKitJournal edited by Simon Hausmann

(diff)

6:31 AM Changeset in webkit [58814] by jeremy@chromium.org

820 edits
2 adds
69 deletes in trunk/LayoutTests

2010-05-05 Jeremy Moskovich <jeremy@chromium.org>

Unreviewed, updated baselines - 3rd batch.

skia r561 broke a bunch of chromium image baselines.
<long list of new image baselines redacted>

platform/chromium/test_expectations.txt:

5:57 AM Changeset in webkit [58813] by Csaba Osztrogonác

2 edits in trunk/WebCore

[Qt] REGRESSION(r56869): WinCE build is broken
https://bugs.webkit.org/show_bug.cgi?id=36929

Rubber-stamped by Simon Hausmann.

WebCore.pro: LIBS += -lOle32 added.

5:44 AM BuildingQtOnWindows edited by jocelyn.turcotte@nokia.com

Added python as a dependency (diff)

5:36 AM Changeset in webkit [58812] by jeremy@chromium.org

1080 edits
8 adds
111 deletes in trunk/LayoutTests

Unreviewed, updated baselines - 2nd batch.

skia r561 broke a bunch of chromium image baselines.
<long list of new image baselines redacted>

platform/chromium/test_expectations.txt:

4:54 AM Changeset in webkit [58811] by jeremy@chromium.org

379 edits
39 deletes in trunk/LayoutTests

Unreviewed, updated baselines - 1st batch.

skia r561 broke a bunch of chromium image baselines.
<long list of new image baselines redacted>

platform/chromium/test_expectations.txt:

4:30 AM Changeset in webkit [58810] by Simon Hausmann

2 edits in trunk/WebCore

[Qt] Disable Qt Multimedia backend for HTML 5 Audio and Video elements

Reviewed by Tor Arne Vestbø.

Fall back to the Phonon backend until the release situation has been cleared up.

WebCore.pro:

4:30 AM Changeset in webkit [58809] by loislo@chromium.org

2 edits in trunk/WebCore

2010-05-05 Ilya Tikhonovsky <loislo@chromium.org>

Reviewed by Pavel Feldman.

WebInspector: Clickable links to resources should be used as 'details' part for the all suitable kinds of records.
https://bugs.webkit.org/show_bug.cgi?id=38542

inspector/front-end/TimelinePanel.js: (WebInspector.TimelinePanel.prototype._innerAddRecordToTimeline): (WebInspector.TimelineRecordListRow): (WebInspector.TimelineRecordListRow.prototype.update): (WebInspector.TimelinePanel.FormattedRecord): (WebInspector.TimelinePanel.FormattedRecord.prototype._generatePopupContent): (WebInspector.TimelinePanel.FormattedRecord.prototype._getRecordDetails): (WebInspector.TimelinePanel.PopupContentHelper): (WebInspector.TimelinePanel.PopupContentHelper.prototype._createCell): (WebInspector.TimelinePanel.PopupContentHelper.prototype._appendTextRow): (WebInspector.TimelinePanel.PopupContentHelper.prototype._appendElementRow): (WebInspector.TimelinePanel.PopupContentHelper.prototype._appendLinkRow):

2:33 AM Changeset in webkit [58808] by Nikolas Zimmermann

1 edit
6 adds in trunk/WebCore

2010-05-05 Nikolas Zimmermann <nzimmermann@rim.com>

Not reviewed. Add missing files generated by "run-bindings-test", after the DOMTestCallback.idl addition.

bindings/scripts/test/GObject/WebKitDOMTestCallback.cpp: Added. (WebKit::kit): (webkit_dom_test_callback_callback_with_class1param): (webkit_dom_test_callback_callback_with_class2param): (webkit_dom_test_callback_callback_with_non_bool_return_type): (WebKit::wrapTestCallback): (WebKit::core): (webkit_dom_test_callback_finalize): (webkit_dom_test_callback_set_property): (webkit_dom_test_callback_get_property): (webkit_dom_test_callback_class_init): (webkit_dom_test_callback_init):
bindings/scripts/test/GObject/WebKitDOMTestCallback.h: Added.
bindings/scripts/test/GObject/WebKitDOMTestCallbackPrivate.h: Added.
bindings/scripts/test/ObjC/DOMTestCallback.h: Added.
bindings/scripts/test/ObjC/DOMTestCallback.mm: Added. (-[DOMTestCallback dealloc]): (-[DOMTestCallback finalize]): (-[DOMTestCallback callbackWithClass1Param:]): (-[DOMTestCallback callbackWithClass2Param:strArg:]): (-[DOMTestCallback callbackWithNonBoolReturnType:]): (-[DOMTestCallback customCallback:class6Param:]): (core): (kit):
bindings/scripts/test/ObjC/DOMTestCallbackInternal.h: Added.

1:00 AM Changeset in webkit [58807] by jeremy@chromium.org

2 edits in trunk/LayoutTests

Unreviewed, expectations change.

recommit r58747 - Preparing for chromium skia roll incorporating changes up to r561.
Add expected failures so bots don't turn red while rolling.

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

platform/chromium/test_expectations.txt:

12:57 AM Changeset in webkit [58806] by xan@webkit.org

2 edits in trunk/LayoutTests

2010-05-05 Xan Lopez <xlopez@igalia.com>

Skip flakey workers test.

platform/gtk/Skipped:

12:57 AM Changeset in webkit [58805] by eric@webkit.org

2 edits in trunk/WebCore

2010-05-05 Joseph Pecoraro <Joseph Pecoraro>

Reviewed by Pavel Feldman.

Web Inspector: FontView needs Cleanup
https://bugs.webkit.org/show_bug.cgi?id=38567

FontView was missing some of the newer practices for initialization
that other Resource Views had. Also its width calculations could
result in a single character wrapping. This is general cleanup.

inspector/front-end/FontView.js: (WebInspector.FontView): move initialization into contentTabSelected (WebInspector.FontView.prototype.contentTabSelected): handle initialization like other Resource Views (WebInspector.FontView.prototype.updateFontPreviewSize): use a narrower width to prevent text from widowing

12:31 AM Changeset in webkit [58804] by krit@webkit.org

2 edits in trunk/WebCore

2010-05-05 Dirk Schulze <krit@webkit.org>

Reviewed by Nikolas Zimmermann.

SVG hit testing is *way* too slow
https://bugs.webkit.org/show_bug.cgi?id=19312

Use the cached stroke and fill boundaries in RenderPath as a heuristik to
speed up SVG's hit testing.

No new tests added.

rendering/RenderPath.cpp: (WebCore::RenderPath::fillContains): (WebCore::RenderPath::strokeContains):

Note: See TracTimeline for information about the timeline view.

Download in other formats:

RSS Feed