This includes temporary test_expectations, will push a fix to webkit later today.

Review URL: http://codereview.chromium.org/5805003

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@69121 0039d316-1c4b-4281-b951-d872f2087c98

BUG=60243
TEST=none

Review URL: http://codereview.chromium.org/5633008

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@69089 0039d316-1c4b-4281-b951-d872f2087c98

BUG=64263
TEST=trybots

Review URL: http://codereview.chromium.org/5755004

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68959 0039d316-1c4b-4281-b951-d872f2087c98

BUG=64263
TEST=compiled locally, trybots

Review URL: http://codereview.chromium.org/5607004

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68935 0039d316-1c4b-4281-b951-d872f2087c98

BUG=none
TEST=none


git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68931 0039d316-1c4b-4281-b951-d872f2087c98

BUG=none
TEST=none


git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68926 0039d316-1c4b-4281-b951-d872f2087c98

Make it possible to write simple unit tests for the pepper implementation. This adds a mock plugin delegate that does nothing, and a helper unit test base class that sets up a dummy plugin instance and module for use in writing tests.

This implements some basic tests on the resource tracker and fixes a bug I found!

TEST=this is
BUG=none
Review URL: http://codereview.chromium.org/5685002

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68781 0039d316-1c4b-4281-b951-d872f2087c98

BUG=64371
TEST=none


git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68760 0039d316-1c4b-4281-b951-d872f2087c98

TBR=yutak
TEST=none
BUG=none


git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68491 0039d316-1c4b-4281-b951-d872f2087c98

TBR=yutak
TEST=none
BUG=none


git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68415 0039d316-1c4b-4281-b951-d872f2087c98

BUG=41405
TEST=green bot

Review URL: http://codereview.chromium.org/5598007

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68412 0039d316-1c4b-4281-b951-d872f2087c98

Attempt #2 after fixing some race conditions between Initialize() and Abort() (see r68363).

One thing to notice is that both buffered_data_source_unittest and simple_data_source_unittest need to have a way to inject a MockWebURLLoader into the BufferedResourceLoader and the SimpleDataSource.  In order to make sure a new one is not created during a Start(), I introduced the function SetURLLoaderForTest and keep_test_loader flag.

Patch by annacc@chromium.org:
http://codereview.chromium.org/3863002/

BUG=16751
TEST=src/xcodebuild/Debug/test_shell_tests --gtest_filter=Buffered*
src/xcodebuild/Debug/test_shell_tests --gtest_filter=Simple*
src/webkit/tools/layout_tests/run_webkit_tests.sh --debug media
webkit/tools/layout_tests/run_webkit_tests.sh --debug http/tests/media

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68378 0039d316-1c4b-4281-b951-d872f2087c98

TBR=yutak
TESt=none
BUG=65404


git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68357 0039d316-1c4b-4281-b951-d872f2087c98

TBR=yutak
BUG=65404
TEST=none


git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68281 0039d316-1c4b-4281-b951-d872f2087c98

TBR=yutak
TEST=none
BUG=none


git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68276 0039d316-1c4b-4281-b951-d872f2087c98

BUG=23581
TEST=compiles

Review URL: http://codereview.chromium.org/5591004

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68208 0039d316-1c4b-4281-b951-d872f2087c98

Confusingly, we had a ToString() that returned a UTF-8 string and
a ToStringVector() that returned wstrings.  Since CppVariant is
a small wrapper around NPVariant I made them both use UTF-8; it
simplified most of the users anyway (which only dealt with ASCII
anyway).

BUG=23581
TEST=compiles

Review URL: http://codereview.chromium.org/5631002

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68187 0039d316-1c4b-4281-b951-d872f2087c98

Also clean up downstream test expectations,
they were added upstream http://trac.webkit.org/changeset/73212

TBR=yutak
TEST=none
BUG=none


git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68127 0039d316-1c4b-4281-b951-d872f2087c98

Revert 68094 - Refactoring BufferedDataSource to work with WebURLLoader instead of a MediaResourceLoaderBridge.

One thing to notice is that both buffered_data_source_unittest and simple_data_source_unittest need to have a way to inject a MockWebURLLoader into the BufferedResourceLoader and the SimpleDataSource.  In order to make sure a new one is not created during a Start(), I introduced the function SetURLLoaderForTest and keep_test_loader flag.

Patch by annacc@chromium.org:
http://codereview.chromium.org/3863002/

BUG=16751
TEST=src/xcodebuild/Debug/test_shell_tests --gtest_filter=Buffered*
src/xcodebuild/Debug/test_shell_tests --gtest_filter=Simple*
src/webkit/tools/layout_tests/run_webkit_tests.sh --debug media
webkit/tools/layout_tests/run_webkit_tests.sh --debug http/tests/media

TBR=scherkus@chromium.org
Review URL: http://codereview.chromium.org/5619002

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68126 0039d316-1c4b-4281-b951-d872f2087c98

TBR=yutak
TEST=none
BUG=none


git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68116 0039d316-1c4b-4281-b951-d872f2087c98

BUG=none
TEST=compiles

Review URL: http://codereview.chromium.org/5527003

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68115 0039d316-1c4b-4281-b951-d872f2087c98

One thing to notice is that both buffered_data_source_unittest and simple_data_source_unittest need to have a way to inject a MockWebURLLoader into the BufferedResourceLoader and the SimpleDataSource.  In order to make sure a new one is not created during a Start(), I introduced the function SetURLLoaderForTest and keep_test_loader flag.

Patch by annacc@chromium.org:
http://codereview.chromium.org/3863002/

BUG=16751
TEST=src/xcodebuild/Debug/test_shell_tests --gtest_filter=Buffered*
src/xcodebuild/Debug/test_shell_tests --gtest_filter=Simple*
src/webkit/tools/layout_tests/run_webkit_tests.sh --debug media
webkit/tools/layout_tests/run_webkit_tests.sh --debug http/tests/media

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68094 0039d316-1c4b-4281-b951-d872f2087c98

It's been moved upstream with http://trac.webkit.org/changeset/73050

Review URL: http://codereview.chromium.org/5539004

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68070 0039d316-1c4b-4281-b951-d872f2087c98

http://test-results.appspot.com/dashboards/flakiness_dashboard.html#tests=svg%2FW3C-SVG-1.1%2Fstruct-image-10-t.svg%20
http://test-results.appspot.com/dashboards/flakiness_dashboard.html#tests=tables%2Fmozilla%2Fbugs%2Fbug14929.html

Fixed upstream here:
http://trac.webkit.org/changeset/73089

BUG=64395,64394
TEST=webkit tests stay green

Review URL: http://codereview.chromium.org/5539003

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68048 0039d316-1c4b-4281-b951-d872f2087c98

It was moved upstream in r73093.

TEST=none
BUG=none

Review URL: http://codereview.chromium.org/5557001

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68032 0039d316-1c4b-4281-b951-d872f2087c98

TBR=tony
TEST=none
BUG=none



git-svn-id: svn://svn.chromium.org/chrome/trunk/src@67980 0039d316-1c4b-4281-b951-d872f2087c98

BUG=None
TEST=None

Review URL: http://codereview.chromium.org/5503001

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@67934 0039d316-1c4b-4281-b951-d872f2087c98

to allow the dump including the heap memory.

TEST=none
BUG=none

Review URL: http://codereview.chromium.org/5262011

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@67925 0039d316-1c4b-4281-b951-d872f2087c98

TBR=ojan

Review URL: http://codereview.chromium.org/5534001

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@67915 0039d316-1c4b-4281-b951-d872f2087c98

gwtquake lets the renderer create > 300 threads (one per audio element?), and eventually thread creation fails. This CL makes the media code more robust against thread creation failure (currently, it just crashes the renderer).

The Real Fix probably is to have a thread pool for media stuff instead of one thread per media object. And maybe threads just leak under some circumstances. I will file a follow-up bug for that case, hopefully with a reduced test case.

BUG=53867,61293
TEST=Completing the first level in gwtquake shouldn't crash the renderer.

Review URL: http://codereview.chromium.org/5362003

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@67824 0039d316-1c4b-4281-b951-d872f2087c98

Disable failing NaClUITest.SrpcBasicTest and add back in test
expectation that was erroneously removed.

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@67811 0039d316-1c4b-4281-b951-d872f2087c98

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@67807 0039d316-1c4b-4281-b951-d872f2087c98

BUG=64263
TEST=compiled locally, trybots

Review URL: http://codereview.chromium.org/5384002

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@67762 0039d316-1c4b-4281-b951-d872f2087c98

BUG=none
TEST=none
TBR=ojan

Review URL: http://codereview.chromium.org/5364008

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@67759 0039d316-1c4b-4281-b951-d872f2087c98

In the current code calling dispatcher->DidXxx in an operation's DidXxx method MAY indirectly delete the operation itself depending on the dispatcher's implementation.  I was confused by this several times and I want to make this flow more explicit.

This patch lets FileSystemOperation control its lifetime by itself so that each callback dispatcher implementation does not need to take care of it.

Also moved BrowserFileSystemCallbackDispatcher into file_system_dispatcher_host.cc as it's only used in it and its implementation is tightly coupled with the DispatcherHost.

BUG=60243
TEST=none

Review URL: http://codereview.chromium.org/4821005

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@67732 0039d316-1c4b-4281-b951-d872f2087c98

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@67685 0039d316-1c4b-4281-b951-d872f2087c98

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@67684 0039d316-1c4b-4281-b951-d872f2087c98

All dependencies were already removed.

BUG=64260

Review URL: http://codereview.chromium.org/5318009

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@67586 0039d316-1c4b-4281-b951-d872f2087c98

I haven't deleted it yet because it might be useful for debugging
failures.  I'll delete it in a separate change so it's easy for
someone to cherry pick the change back.

BUG=64260

Review URL: http://codereview.chromium.org/5301003

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@67571 0039d316-1c4b-4281-b951-d872f2087c98

to mock menu item strings.
This change was once reverted at r67248 due to hitting a webkit bug
and is re-landing because the bug was fixed at
http://trac.webkit.org/changeset/72721.

BUG=63460
TEST=LayoutTest/editing/spelling/context-menu-suggestions.html
Review URL: http://codereview.chromium.org/5120002

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@67505 0039d316-1c4b-4281-b951-d872f2087c98