summaryrefslogtreecommitdiffstats
path: root/webkit
Commit message (Collapse)AuthorAgeFilesLines
* remove network-shared-worker.html from test_expectations.txthoro@chromium.org2014-02-141-2/+0
| | | | | | | | BUG=329786 Review URL: https://codereview.chromium.org/165793002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@251264 0039d316-1c4b-4281-b951-d872f2087c98
* Move async_file_test_helper.* to content/public/test/pilgrim@chromium.org2014-02-132-379/+0
| | | | | | | | BUG=338338 Review URL: https://codereview.chromium.org/157053010 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@251069 0039d316-1c4b-4281-b951-d872f2087c98
* Remove unused includes & using statements from blink::Platform implementations.ddorwin@chromium.org2014-02-131-13/+0
| | | | | | Review URL: https://codereview.chromium.org/161873002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@250945 0039d316-1c4b-4281-b951-d872f2087c98
* Move webfallbackthemeengine_impl.* to content/child/pilgrim@chromium.org2014-02-133-237/+0
| | | | | | | | BUG=338338 Review URL: https://codereview.chromium.org/160023003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@250930 0039d316-1c4b-4281-b951-d872f2087c98
* Do not include PDF reader assets for platforms with no plugins.aurimas@chromium.org2014-02-121-45/+47
| | | | | | | | BUG=326023 Review URL: https://codereview.chromium.org/160693002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@250853 0039d316-1c4b-4281-b951-d872f2087c98
* DevTools does not display POST bodies when following redirects.eustas@chromium.org2014-02-121-1/+4
| | | | | | | | BUG=342406 Review URL: https://codereview.chromium.org/153403005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@250697 0039d316-1c4b-4281-b951-d872f2087c98
* Updating XTBs based on .GRDs from branch 1750kerz@chromium.org2014-02-121-1/+1
| | | | git-svn-id: svn://svn.chromium.org/chrome/trunk/src@250617 0039d316-1c4b-4281-b951-d872f2087c98
* Move webkit/common/gpu/test_context_provider_factory.* to content/test/pilgrim@chromium.org2014-02-123-78/+0
| | | | | | | | BUG=338338 Review URL: https://codereview.chromium.org/160013003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@250566 0039d316-1c4b-4281-b951-d872f2087c98
* Move common param trait enum's to IPC range-checked macros.tsepez@chromium.org2014-02-111-1/+2
| | | | | | | | | | | This is part of a long-running background task to remove the remaining uses of the unchecked IPC_ENUM_TRAITS() macro. BUG=246708 Review URL: https://codereview.chromium.org/147243004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@250541 0039d316-1c4b-4281-b951-d872f2087c98
* Split preserve3d into: "should-flatten" and "is-3d-sorted"vollick@chromium.org2014-02-112-6/+5
| | | | | | | | | | | | | | | | | | | | | | | | Preserve3d has implications for both sorting and flattening of transforms. This is a bummer. Sometimes we want to allow a transform to remain unflattened, but we don't want that to affect 3d sorting. With this CL, we will have the ability to do that. Note: this is essentially this cl https://codereview.chromium.org/100393005/ ..minus the generalizations to sorting that were being attempted in that patch. Note, this CL is gated on https://codereview.chromium.org/98373011/ R=enne@chromium.org BUG=338980 Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=249974 Review URL: https://codereview.chromium.org/147833003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@250474 0039d316-1c4b-4281-b951-d872f2087c98
* Add transfer size paramater to didFinishLoading [2/3]eustas@chromium.org2014-02-103-10/+13
| | | | | | | | | | | | | | | | | | | Mini DD: https://docs.google.com/a/chromium.org/document/d/1kAKnBKofLc3Tex_JPveubwJcQLT3BhIL3xNQyjvRIps/edit?usp=sharing Most of files contain only signature changes. There are 3 non-trivial changes: 1) ResourceMsg_RequestComplete IPC message receives struct instead of distinct parameters. 2) Use URLRequest::GetTotalReceivedBytes instead of DevToolsNetLogObserver::GetAndResetEncodedDataLength 3) Remove unused code in DevToolsNetLogObserver. BUG=111052 Review URL: https://codereview.chromium.org/143263002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@250069 0039d316-1c4b-4281-b951-d872f2087c98
* Revert of Split preserve3d into: "should-flatten" and "is-3d-sorted" ↵dstockwell@chromium.org2014-02-102-5/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | (https://codereview.chromium.org/147833003/) Reason for revert: Introduces flakiness in Blink layout tests: http://test-results.appspot.com/dashboards/flakiness_dashboard.html#group=%2540ToT%2520Blink&tests=compositing%252Fbackface-visibility-transformed Original issue's description: > Split preserve3d into: "should-flatten" and "is-3d-sorted" > > Preserve3d has implications for both sorting and flattening of > transforms. This is a bummer. Sometimes we want to allow a transform to > remain unflattened, but we don't want that to affect 3d sorting. With > this CL, we will have the ability to do that. > > Note: this is essentially this cl > https://codereview.chromium.org/100393005/ > > ..minus the generalizations to sorting that were being attempted in that > patch. > > Note, this CL is gated on https://codereview.chromium.org/98373011/ > > R=enne@chromium.org > BUG=338980 > > Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=249974 TBR=enne@chromium.org,shawnsingh@chromium.org,vollick@chromium.org NOTREECHECKS=true NOTRY=true BUG=338980 Review URL: https://codereview.chromium.org/158563003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@250031 0039d316-1c4b-4281-b951-d872f2087c98
* Split preserve3d into: "should-flatten" and "is-3d-sorted"vollick@chromium.org2014-02-092-6/+5
| | | | | | | | | | | | | | | | | | | | | | Preserve3d has implications for both sorting and flattening of transforms. This is a bummer. Sometimes we want to allow a transform to remain unflattened, but we don't want that to affect 3d sorting. With this CL, we will have the ability to do that. Note: this is essentially this cl https://codereview.chromium.org/100393005/ ..minus the generalizations to sorting that were being attempted in that patch. Note, this CL is gated on https://codereview.chromium.org/98373011/ R=enne@chromium.org BUG=338980 Review URL: https://codereview.chromium.org/147833003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@249974 0039d316-1c4b-4281-b951-d872f2087c98
* Use incognito split for ChromeVox and TTS in Guest modedpolukhin@chromium.org2014-02-082-3/+3
| | | | | | | | | | | + enable HTML5 filesystem in TTS in Guest mode BUG=337639 TEST=manual Review URL: https://codereview.chromium.org/141733008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@249888 0039d316-1c4b-4281-b951-d872f2087c98
* Plumb network stack information about existence of cached copyrdsmith@chromium.org2014-02-064-4/+12
| | | | | | | | | | | | | | | | | | | through to error page. Specifically, add a "stale_copy_in_cache" argument to all of (ordered from Browser->Renderer): * ResourceMsg_RequestComplete IPC message. * ResourceDispatcher::OnRequestComplete * ResourceLoaderBridge::Peer::OnCompletedRequest. * All subclasses of RLB::P::OnCompleted Request, including WebURLLoaderImpl::context::OnCompletedRequest. * Blink WebURLError and ResourceError classes (https://codereview.chromium.org/138493002). * LocalizedError::GetStrings. This is a paired commit with the blink CL https://codereview.chromium.org/138493002. That CL must be landed before this one. BUG=329620 Review URL: https://codereview.chromium.org/138513002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@249527 0039d316-1c4b-4281-b951-d872f2087c98
* Quota: Cap size of requested quotanhiroki@chromium.org2014-02-063-2/+23
| | | | | | | | | | | | | | | | This change caps size of requested quota as follows: - Temporary quota is capped at the maximum value of int64 (This is an implementation limitation) - Persistent quota is capped at 10GB per host BUG=256288 TEST=content_unittests NOTRY=true Review URL: https://codereview.chromium.org/150903003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@249326 0039d316-1c4b-4281-b951-d872f2087c98
* [FileAPI] Don't fail on ReserveQuota() even on insufficient spacetzik@chromium.org2014-02-068-33/+64
| | | | | | | | | | | Return maximum available quota when there isn't sufficient available space to satisfy the requested amount of quota reservation. BUG=340703 Review URL: https://codereview.chromium.org/155603002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@249323 0039d316-1c4b-4281-b951-d872f2087c98
* Use gyp targets to link in X11 libraries instead of setting -lXfoojamesr@chromium.org2014-02-051-3/+3
| | | | | | | | | | | | | | This sets up gyp targets for the various X11 libraries we use and uses pkg-config to determine the correct cflags/ldflags incantations to use to link against them instead of listing out -lXfoo lines in libraries sections in gyp targets throughout the tree. This is more likely to generate the right lines and makes the dependencies a bit more explicit. BUG=339518 Review URL: https://codereview.chromium.org/153223002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@248939 0039d316-1c4b-4281-b951-d872f2087c98
* Move the worker script loading code to the worker process (phase:4/5)horo@chromium.org2014-02-051-0/+2
| | | | | | | | | | | | | | - Stop sending the worker script from the renderer. - In this change we can remove the pending instances in WorkerServiceImpl. This is step 4 of moving the worker script loading code from the renderer process to the worker process. See: http://crbug.com/329786 BUG=329786 Review URL: https://codereview.chromium.org/133093003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@248869 0039d316-1c4b-4281-b951-d872f2087c98
* Fix dump_file_systemlpique@chromium.org2014-02-041-3/+3
| | | | | | | | | | | | | | | | | | | The change in r247631 broke the ability to compile the dump_file_system tool. This patch allows it to be built once more. Tested with: ninja -C out/Release dump_file_system out/Release/dump_file_system <path-to-a-filesystem-dir> BUG= R=nhiroki@chromium.org, tzik@chromium.org Review URL: https://codereview.chromium.org/153363002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@248769 0039d316-1c4b-4281-b951-d872f2087c98
* Disable a flaky content_unittestpasko@chromium.org2014-02-041-1/+2
| | | | | | | | | | | The test is flaky on Android. More details in the bug. BUG=339534 TBR=michaeln@chromium.org Review URL: https://codereview.chromium.org/150683003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@248725 0039d316-1c4b-4281-b951-d872f2087c98
* Add stubs for setRenderingContext and setShouldFlattenTransformvollick@chromium.org2014-02-042-0/+6
| | | | | | | | | | | | | | Landing this in advance of https://codereview.chromium.org/98373011/ will allow the new API functions being added to WebLayer to be pure virtual. The real implementation of these functions is coming in https://codereview.chromium.org/147833003/ R=jamesr@chromium.org BUG=322216,338980 Review URL: https://codereview.chromium.org/145973018 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@248719 0039d316-1c4b-4281-b951-d872f2087c98
* Remove rendering stats APIernstm@chromium.org2014-02-041-21/+0
| | | | | | | | | | | | | | | | | The rendering stats API is not used anymore, since all benchmarks and tests have been switch to use tracing instead. This patch removes: - the rendering stats API in GpuBenchmarkingExtension - GpuRenderingStats and code that collects it - BrowserRenderingStats (latency info) and code that collects it R=nduca@chromium.org,miletus@chromium.org,kbr@chromium.org,jamesr@chromium.org BUG=310452 Review URL: https://codereview.chromium.org/132373005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@248687 0039d316-1c4b-4281-b951-d872f2087c98
* Don't try to guess the referrer policy, but use the one associated with the ↵jochen@chromium.org2014-02-043-14/+4
| | | | | | | | | | | | | request This is possible now that we store the referrer policy on each request. BUG=334125 R=jamesr@chromium.org Review URL: https://codereview.chromium.org/135973009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@248650 0039d316-1c4b-4281-b951-d872f2087c98
* Appcache::OnCorruptionDetected handling. The general idea is to delete ↵michaeln@chromium.org2014-02-0414-78/+511
| | | | | | | | | | | | | | | | | | | | | | | | everything and start over (reinitialize) if we notice corruption on disk. The is some pre-existing code to perform the reinitialization function, but that was only used in a more narrow case where the disk cache could not have been open. With the changes in this CL, reinitialization can be triggered pretty much at any time. * The AppCacheDatabase classes uses an SQLConnecton ErrorHandler to detect catastrophic errors (corruption). * The AppCacheStorageImpl's DatabaseTask class checks for having seen corruption during a task and initiates a heavy handed operation to delete everything and start over. * The AppCacheStorageImpl::InitTask also checks for the presence of a stale/undeletable DiskCache when there is no SQL database. If found, that also triggers reinitialization. * The AppCacheService class does exponential backoff to avoid thrashing the disk on repeated reinit attempts. * The AppCacheDiskCache class's Disable() method has been modified such that all file handles get released. TBR=jam BUG=318544 Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=247967 Review URL: https://codereview.chromium.org/137493003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@248642 0039d316-1c4b-4281-b951-d872f2087c98
* Transfer appcache state during cross site navigations too.michaeln@chromium.org2014-02-048-8/+118
| | | | | | | | | | | | | | The AppCacheHost is an object that persist across the lifetime of a document. It's created prior to the document and prior to the initiation of the resource load for the documents main resource. Today, when a cross-site xfer occurs the host is deleted and a new host is created that knows nothing of whats going on... so not surprisingly things are broken. Either state of the old host that gets deleted needs to be saved and restored into a new host that gets created... or the old host should not get deleted and instead get transferred along as well. I chose the latter because there's some state that can't be saved/restored so easily and moving the whole object is less tedious. BUG=331270 Review URL: https://codereview.chromium.org/137883005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@248631 0039d316-1c4b-4281-b951-d872f2087c98
* Add gpu::MailboxHolder to hold state for a gpu::Mailboxsheu@chromium.org2014-02-031-3/+6
| | | | | | | | | | | | | | | | | | | | | | | | gpu::Mailbox by itself can hold only a texture id, but in common usage it comes with texture target and syncpoint information for cross-context sharing. To reduce repetition of this pattern, gpu::MailboxHolder holds: * a gpu::Mailbox * a GL texture target * a syncpoint index Refactor other classes to use a gpu::MailboxHolder instead of separate gpu::Mailbox and associated state. Syncpoints are created with uint32 indices; make sure all uses of syncpoints use the appropriate type. BUG=None TEST=local build, unittests on CrOS snow, desktop Linux TBR=piman@chromium.org, enn@chromium.orge, cevans@chromium.org, scherkus@chromium.org Review URL: https://codereview.chromium.org/132233041 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@248612 0039d316-1c4b-4281-b951-d872f2087c98
* Ensure GL initialization only happens once, and provide common init pathdanakj@chromium.org2014-02-031-18/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | Currently tests initialize GL by calling into methods that should be internal to the gl bindings code. Instead, everyone should go through GLSurface::InitializeOneOff. Also GLSurface::InitializeOneOff early outs if it was already called, leading to a pattern of initializing GL all over the place just in case and not having a clear idea of where it should be set up. Instead, DCHECK that it is not called more than once, and move calls to this method to be during process startup for unit test suites instead of mid-test. This adds two test variants of InitializeOneOff for tests to call, that set up OSMesa or Mock GL bindings, via GLSurface::InitializeOneOff. R=ben@chromium.org, piman@chromium.org, sievers@chromium.org, piman, sievers BUG=270918 Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=247793 Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=248049 Review URL: https://codereview.chromium.org/135213003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@248557 0039d316-1c4b-4281-b951-d872f2087c98
* Move the rest of webkit/glue into content/childtfarina@chromium.org2014-02-019-502/+0
| | | | | | | | | | | | | One more step towards removing src/webkit/ BUG=265753 TEST=content_shell and content_unittests R=jochen@chromium.org,jam@chromium.org TBR=darin Review URL: https://codereview.chromium.org/151023002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@248367 0039d316-1c4b-4281-b951-d872f2087c98
* Move webkit/child/webthemeengine_impl_* to content/child/pilgrim@chromium.org2014-01-319-1590/+0
| | | | | | | | BUG=338338 Review URL: https://codereview.chromium.org/150463002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@248292 0039d316-1c4b-4281-b951-d872f2087c98
* Fix blink test expectations.machenbach@chromium.org2014-01-311-3/+0
| | | | | | | | | | | Doesn't pass lint checks due to deleted test. This CL will break blink DEPS builders until next blink roll. BUG= TBR=abarth@chromium.org Review URL: https://codereview.chromium.org/151583003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@248210 0039d316-1c4b-4281-b951-d872f2087c98
* Skip test that has moved to the V8 repository.machenbach@chromium.org2014-01-311-0/+2
| | | | | | | | | | | The test will pass on the next v8 roll and is removed in the next blink roll. BUG=237872 TBR=dglazkov@chromium.org Review URL: https://codereview.chromium.org/151343004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@248190 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 248049 "Ensure GL initialization only happens once, and p..."kbr@chromium.org2014-01-311-12/+18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Caused content_gl_tests to start failing on all Mac Release GPU bots. See Issue 339658 for details. > Ensure GL initialization only happens once, and provide common init path > > Currently tests initialize GL by calling into methods that should be > internal to the gl bindings code. Instead, everyone should go through > GLSurface::InitializeOneOff. > > Also GLSurface::InitializeOneOff early outs if it was already called, > leading to a pattern of initializing GL all over the place just in case > and not having a clear idea of where it should be set up. Instead, DCHECK > that it is not called more than once, and move calls to this method to > be during process startup for unit test suites instead of mid-test. > > This adds two test variants of InitializeOneOff for tests to call, that > set up OSMesa or Mock GL bindings, via GLSurface::InitializeOneOff. > > R=piman, sievers > BUG=270918 > > Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=247793 > > Review URL: https://codereview.chromium.org/135213003 TBR=danakj@chromium.org BUG=270918,339658 Review URL: https://codereview.chromium.org/151003002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@248093 0039d316-1c4b-4281-b951-d872f2087c98
* Pinch/Zoom Infrastructure & Plumbing CLwjmaclean@chromium.org2014-01-314-11/+18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This CL supplies the necessary changes to CC to support the inner/outer viewport model for pinch-zoom and fixed-position elements. The specification for these changes is contained in the document "Layer-based Solution for Pinch Zoom / Fixed Position". It incorporates a change to how scrollbar parameters are computed (removes the notion of max_scroll_offset as a quantity set be the embedder, and instead inferred from the relative sizes of a clip layer w.r.t. the scroll layer). Scrollbars are generalized so that a layer may have more than two scrollbars, and the parameters of the scrollbar are set w.r.t. the sizes and positions of a clip and a scroll layer. Further, changes to the scrip/scroll layer automatically notify any attached scrollbars. The CL also removes existing references to root_scroll_layer from LTH, LTI and LTHI and replaces them with either Inner/OuterViewportScrollLayer (dual-layer operation only exists at present if the --enable-pinch-virtual-viewport flag is specified, otherwise behavior is unchanged). Logic is added to (i) combine scroll offsets for the two viewports before passing it to the embedder, and (ii) splitting any offsets received from the embedder appropriately between the two viewports. This CL relies on https://codereview.chromium.org/138453004/ for changes in Blink to support it. BUG=148816 Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=246812 R=aelias@chromium.org, enne@chromium.org, joi@chromium.org, piman@chromium.org, sky@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=247684 Review URL: https://codereview.chromium.org/23983047 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@248092 0039d316-1c4b-4281-b951-d872f2087c98
* Move webkit/child/web_discardable_memory_impl to content/child/pilgrim@chromium.org2014-01-303-94/+0
| | | | | | | | BUG=338338 Review URL: https://codereview.chromium.org/130713007 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@248057 0039d316-1c4b-4281-b951-d872f2087c98
* Ensure GL initialization only happens once, and provide common init pathdanakj@chromium.org2014-01-301-18/+12
| | | | | | | | | | | | | | | | | | | | | | | | Currently tests initialize GL by calling into methods that should be internal to the gl bindings code. Instead, everyone should go through GLSurface::InitializeOneOff. Also GLSurface::InitializeOneOff early outs if it was already called, leading to a pattern of initializing GL all over the place just in case and not having a clear idea of where it should be set up. Instead, DCHECK that it is not called more than once, and move calls to this method to be during process startup for unit test suites instead of mid-test. This adds two test variants of InitializeOneOff for tests to call, that set up OSMesa or Mock GL bindings, via GLSurface::InitializeOneOff. R=piman, sievers BUG=270918 Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=247793 Review URL: https://codereview.chromium.org/135213003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@248049 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 247967 "Appcache::OnCorruptionDetected handling. The gene..."avi@chromium.org2014-01-3014-524/+78
| | | | | | | | | | | | | | | | | | | | AppCacheDiskCacheTest.DisableWithEntriesOpen started failing on both Android Tests and Android Tests (dbg) with this change. http://build.chromium.org/p/chromium.linux/builders/Android%20Tests%20%28dbg%29/builds/18225/ http://build.chromium.org/p/chromium.linux/builders/Android%20Tests/builds/11615/ > Appcache::OnCorruptionDetected handling. The general idea is to delete everything and start over (reinitialize) if we notice corruption on disk. The is some pre-existing code to perform the reinitialization function, but that was only used in a more narrow case where the disk cache could not have been open. With the changes in this CL, reinitialization can be triggered pretty much at any time. > > TBR=jam > BUG=318544 > > Review URL: https://codereview.chromium.org/137493003 TBR=michaeln@chromium.org Review URL: https://codereview.chromium.org/150683002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@248014 0039d316-1c4b-4281-b951-d872f2087c98
* Appcache::OnCorruptionDetected handling. The general idea is to delete ↵michaeln@chromium.org2014-01-3014-78/+524
| | | | | | | | | | | | | | | | | | | | | | everything and start over (reinitialize) if we notice corruption on disk. The is some pre-existing code to perform the reinitialization function, but that was only used in a more narrow case where the disk cache could not have been open. With the changes in this CL, reinitialization can be triggered pretty much at any time. * The AppCacheDatabase classes uses an SQLConnecton ErrorHandler to detect catastrophic errors (corruption). * The AppCacheStorageImpl's DatabaseTask class checks for having seen corruption during a task and initiates a heavy handed operation to delete everything and start over. * The AppCacheStorageImpl::InitTask also checks for the presence of a stale/undeletable DiskCache when there is no SQL database. If found, that also triggers reinitialization. * The AppCacheService class does exponential backoff to avoid thrashing the disk on repeated reinit attempts. * The AppCacheDiskCache class's Disable() method has been modified such that all file handles get released. TBR=jam BUG=318544 Review URL: https://codereview.chromium.org/137493003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@247967 0039d316-1c4b-4281-b951-d872f2087c98
* Apply automatic enum range checking for appcache enums acros IPC.tsepez@chromium.org2014-01-301-2/+4
| | | | | | | | | | | This is part of a long-running background task to remove the remaining uses of the unchecked IPC_ENUM_TRAITS() macro. BUG=246708 Review URL: https://codereview.chromium.org/149073002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@247905 0039d316-1c4b-4281-b951-d872f2087c98
* Move webkit/child/webkitplatformsupport_child_impl to content/pilgrim@chromium.org2014-01-3010-231/+21
| | | | | | | | BUG=338338 Review URL: https://codereview.chromium.org/148333009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@247904 0039d316-1c4b-4281-b951-d872f2087c98
* QuotaManager: Don't fire callback if it's nullkinuko@chromium.org2014-01-301-1/+1
| | | | | | | | | BUG=n/a R=nhiroki@chromium.org Review URL: https://codereview.chromium.org/147143005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@247879 0039d316-1c4b-4281-b951-d872f2087c98
* Wires through ScrollbarCorner to NativeTheme.sky@chromium.org2014-01-303-9/+5
| | | | | | | | | | BUG=333499 TEST=none R=jamesr@chromium.org Review URL: https://codereview.chromium.org/149513010 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@247876 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 247793 "Ensure GL initialization only happens once, and p..."yoz@chromium.org2014-01-301-12/+18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | due to new static initializers in gl_bindings_autogen_gl.cc > Ensure GL initialization only happens once, and provide common init path > > Currently tests initialize GL by calling into methods that should be > internal to the gl bindings code. Instead, everyone should go through > GLSurface::InitializeOneOff. > > Also GLSurface::InitializeOneOff early outs if it was already called, > leading to a pattern of initializing GL all over the place just in case > and not having a clear idea of where it should be set up. Instead, DCHECK > that it is not called more than once, and move calls to this method to > be during process startup for unit test suites instead of mid-test. > > This adds two test variants of InitializeOneOff for tests to call, that > set up OSMesa or Mock GL bindings, via GLSurface::InitializeOneOff. > > R=piman, sievers > BUG=270918 > > Review URL: https://codereview.chromium.org/135213003 TBR=danakj@chromium.org Review URL: https://codereview.chromium.org/149953003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@247811 0039d316-1c4b-4281-b951-d872f2087c98
* Ensure GL initialization only happens once, and provide common init pathdanakj@chromium.org2014-01-301-18/+12
| | | | | | | | | | | | | | | | | | | | | | Currently tests initialize GL by calling into methods that should be internal to the gl bindings code. Instead, everyone should go through GLSurface::InitializeOneOff. Also GLSurface::InitializeOneOff early outs if it was already called, leading to a pattern of initializing GL all over the place just in case and not having a clear idea of where it should be set up. Instead, DCHECK that it is not called more than once, and move calls to this method to be during process startup for unit test suites instead of mid-test. This adds two test variants of InitializeOneOff for tests to call, that set up OSMesa or Mock GL bindings, via GLSurface::InitializeOneOff. R=piman, sievers BUG=270918 Review URL: https://codereview.chromium.org/135213003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@247793 0039d316-1c4b-4281-b951-d872f2087c98
* Revert of Pinch/Zoom Infrastructure & Plumbing CL ↵ricea@chromium.org2014-01-294-19/+11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | (https://codereview.chromium.org/23983047/) Reason for revert: Broke Linux ASAN Tests (3) bot. Original issue's description: > Pinch/Zoom Infrastructure & Plumbing CL > > This CL supplies the necessary changes to CC to support the > inner/outer viewport model for pinch-zoom and fixed-position > elements. The specification for these changes is contained in > the document "Layer-based Solution for Pinch Zoom / Fixed > Position". > > It incorporates a change to how scrollbar parameters are > computed (removes the notion of max_scroll_offset as a > quantity set be the embedder, and instead inferred from the > relative sizes of a clip layer w.r.t. the scroll layer). > > Scrollbars are generalized so that a layer may have more than > two scrollbars, and the parameters of the scrollbar are set > w.r.t. the sizes and positions of a clip and a scroll layer. > Further, changes to the scrip/scroll layer automatically > notify any attached scrollbars. > > The CL also removes existing references to root_scroll_layer > from LTH, LTI and LTHI and replaces them with either > Inner/OuterViewportScrollLayer (dual-layer operation only > exists at present if the --enable-pinch-virtual-viewport flag > is specified, otherwise behavior is unchanged). > > Logic is added to (i) combine scroll offsets for the two > viewports before passing it to the embedder, and (ii) > splitting any offsets received from the embedder > appropriately between the two viewports. > > This CL relies on https://codereview.chromium.org/138453004/ > for changes in Blink to support it. > > BUG=148816 > > Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=246812 > > R=aelias@chromium.org, enne@chromium.org, joi@chromium.org, piman@chromium.org, sky@chromium.org > > Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=247684 TBR=enne@chromium.org,aelias@chromium.org,joi@chromium.org,sky@chromium.org,piman@chromium.org,tony@chromium.org,wjmaclean@chromium.org NOTREECHECKS=true NOTRY=true BUG=148816 Review URL: https://codereview.chromium.org/135183016 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@247688 0039d316-1c4b-4281-b951-d872f2087c98
* Pinch/Zoom Infrastructure & Plumbing CLwjmaclean@chromium.org2014-01-294-11/+19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This CL supplies the necessary changes to CC to support the inner/outer viewport model for pinch-zoom and fixed-position elements. The specification for these changes is contained in the document "Layer-based Solution for Pinch Zoom / Fixed Position". It incorporates a change to how scrollbar parameters are computed (removes the notion of max_scroll_offset as a quantity set be the embedder, and instead inferred from the relative sizes of a clip layer w.r.t. the scroll layer). Scrollbars are generalized so that a layer may have more than two scrollbars, and the parameters of the scrollbar are set w.r.t. the sizes and positions of a clip and a scroll layer. Further, changes to the scrip/scroll layer automatically notify any attached scrollbars. The CL also removes existing references to root_scroll_layer from LTH, LTI and LTHI and replaces them with either Inner/OuterViewportScrollLayer (dual-layer operation only exists at present if the --enable-pinch-virtual-viewport flag is specified, otherwise behavior is unchanged). Logic is added to (i) combine scroll offsets for the two viewports before passing it to the embedder, and (ii) splitting any offsets received from the embedder appropriately between the two viewports. This CL relies on https://codereview.chromium.org/138453004/ for changes in Blink to support it. BUG=148816 Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=246812 R=aelias@chromium.org, enne@chromium.org, joi@chromium.org, piman@chromium.org, sky@chromium.org Review URL: https://codereview.chromium.org/23983047 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@247684 0039d316-1c4b-4281-b951-d872f2087c98
* [FileAPI] Replace default leveldb::Env with leveldb::MemEnv in tests.tzik@chromium.org2014-01-2916-40/+98
| | | | | | | | | | | Use in-memory LevelDB in tests that don't test LevelDB behavior itself. BUG=331988 R=kinuko@chromium.org, nhiroki@chromium.org, sky@chromium.org Review URL: https://codereview.chromium.org/145693005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@247631 0039d316-1c4b-4281-b951-d872f2087c98
* Remove SetJavaScriptFlags() wrapper function from webkit.tfarina@chromium.org2014-01-292-9/+0
| | | | | | | | | | | | | | | | | This function is just an helper/wrapper function around v8::V8::SetFlagsFromString() and used only by content. John preferred to just remove the wrapper and call the v8 API directly in content. Also removing it from webkit_glue.h helps with the removal of src/webkit/ directory. BUG=265753 TEST=content_shell and content_unittests R=jochen@chromium.org, jam@chromium.org TBR=darin Review URL: https://codereview.chromium.org/147973002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@247626 0039d316-1c4b-4281-b951-d872f2087c98
* Move blob/mock_blob_url_request_context to content/pilgrim@chromium.org2014-01-292-107/+0
| | | | | | | | BUG=338338 Review URL: https://codereview.chromium.org/146963006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@247618 0039d316-1c4b-4281-b951-d872f2087c98
* Remove test_expectations_asan.txt (it moved to Blink).dpranke@chromium.org2014-01-281-84/+0
| | | | | | | | | R=eseidel@chromium.org, ojan@chromium.org BUG=337598 Review URL: https://codereview.chromium.org/143003017 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@247392 0039d316-1c4b-4281-b951-d872f2087c98