summaryrefslogtreecommitdiffstats
path: root/android_webview/browser
Commit message (Collapse)AuthorAgeFilesLines
* Don't schedule more invokeFunctors than necessary.hush2014-12-131-11/+23
| | | | | | | | | | | | | | | | | | | | | | | | The problematic sequence of events is as follows: 1. ShouldRequestOnNonUiThread, which posts a closure (request_draw_gl_closure_) to UI thread 2. That closure gets run on UI thread, and it schedules the invokeFunctor with Android framework 3. Before the corresponding invokeFunctor actually happens on RT (which is DrawGL process mode), ShouldRequestOnUiTdread is called on the UI thread. At this point, pending_non_ui_ is not null, we cancel the callback, which does nothing, because WebView has already scheduled an invokeFunctor with the Android framework in Step 2. Then we schedule another invokeFunctor immediately on the UI thread. So there are 2 invokeFunctors queued in Android framework in this case. This CL tries keep track of whether or not we've queued an invokeFunctor in Android framework already. BUG=442013 Review URL: https://codereview.chromium.org/801923003 Cr-Commit-Position: refs/heads/master@{#308241}
* Remember user's decisions on SSL errors.hush2014-12-124-1/+172
| | | | | | | | | | | | | | | | | | | | | | | | | | | After CL: https://codereview.chromium.org/369703002, Android WebView just has a NULL SSLHostStateDelegate. So it does not remember any user decisions on SSL errors. This is a regression of behavior from L (m37). History: JB behavior: Larger error codes are assumed to have higher severity. And if the user has allowed an SSL error with a high severity, the user won't be prompted for a lower severity SSL error. K and L behavior: A specific SSL error will be allowed only if the error bit field is a subset of previously allowed error codes. trunk behavior for webview (without this patch): We don't remember user's decision at all. This CL: Maintain the same behavior with K and L. BUG=441065 Review URL: https://codereview.chromium.org/794023002 Cr-Commit-Position: refs/heads/master@{#308160}
* Add data reduction proxy debug info to net-internals#bandwidthjeremyim2014-12-111-1/+1
| | | | | | | | | | | | | | | | - Save off various events in DataReductionProxyEventStore for later use (last bypass event, data reduction proxy configuration, canary request state) - Plumb DataReductionProxyEventStore through ChromeNetworkDelegate to net_internals_ui.cc for consumption by bandwidth_view.js/html - Update bandwidth_view.* for rendering the DataReductionProxyEventStore information BUG=346356 Review URL: https://codereview.chromium.org/775773002 Cr-Commit-Position: refs/heads/master@{#307981}
* aw: Rendering test harness and end-to-end smoke testboliu2014-12-115-33/+406
| | | | | | | | | | | | | | | The test harness is heavily modelled after LayerTreeTests in cc. Add implementation for simulating invalidation and functor behavior. Add in enough hooks to allow a end-to-end smoke test from invalidation to draw to functor draw. BUG=418346 Review URL: https://codereview.chromium.org/786533002 Cr-Commit-Position: refs/heads/master@{#307812}
* Move output surface fallback from cc to embeddersenne2014-12-102-4/+7
| | | | | | | | | | | | | Having this in cc means that all embedders get this logic and API, even though only RenderWidgetCompositor and ui::Compositor use it. This also means that cc no longer has to be robust to null SetOutputSurface, which makes the API to embedders a bit more clear. Review URL: https://codereview.chromium.org/738983002 Cr-Commit-Position: refs/heads/master@{#307767}
* Add argument to pass elastic overscroll delta to Blink (Chrome side)ccameron2014-12-091-4/+6
| | | | | | | | | | | | | Blink will need to use this argument to translate input events during overscroll. R=enne TBR=jam BUG=133097 Review URL: https://codereview.chromium.org/783373002 Cr-Commit-Position: refs/heads/master@{#307525}
* Move data reduction proxy logic out of chrome and android webview network ↵megjablon2014-12-093-66/+31
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | delegates In order to move the data reduction proxy logic from ChromeNetworkDelegate and AwNetworkDelegate to DataReductionProxyNetworkDelegate, pure virtual methods in NetworkDelegate move to the new subclass NetworkDelegateImpl. All subclasses of NetworkDelegate now subclass NetworkDelegateImpl. The DataReductionProxyNetworkDelegate subclasses WrappingNetworkDelegate. WrappingNetworkDelegate subclasses NetworkDelegateand and takes a NetworkDelegate in its constructor and owns it. For every override OnFoo, WrappingNetworkDelegate provides a protected virtual void OnFooInternal(), with an empty default implementation. DataReductionProxyNetworkDelegate subclasses WrappingNetworkDelegate and moves implementations from ChromeNetworkDelegate and AwNetworkDelegate inside the On*Internal() methods. BUG=429734 Review URL: https://codereview.chromium.org/734263003 Cr-Commit-Position: refs/heads/master@{#307523}
* aw: More clean ups for testingboliu2014-12-093-6/+8
| | | | | | | | | | | | | | Have BVR return a pointer to SharedRendererState and do the reinterpret case in AwContents. Update ScopedAppGLStateRestore to handle null binding where most calls are no-op and returns null. BUG=418346 Review URL: https://codereview.chromium.org/735013006 Cr-Commit-Position: refs/heads/master@{#307387}
* Don't key client auth on HttpNetworkSession.davidben2014-12-082-2/+0
| | | | | | | | | | | | | | It's an IO-thread object being maintained (fortunately just as a pointer) on the UI thread. It's used to tie together SSLClientAuthObserver instances, but the BrowserContext is a more suitable UI thread key. This relands part of https://codereview.chromium.org/596873002 BUG=439134 Review URL: https://codereview.chromium.org/780383002 Cr-Commit-Position: refs/heads/master@{#307274}
* Revert of Revert of Cleanup: Update the path to vector2d_f.h header. ↵tfarina2014-12-051-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | (patchset #1 id:1 of https://codereview.chromium.org/771283003/) Reason for revert: This didn't cause the issue. As noted in https://codereview.chromium.org/781863004/#msg8 Original issue's description: > Revert of Cleanup: Update the path to vector2d_f.h header. (patchset #1 id:1 of https://codereview.chromium.org/781863004/) > > Reason for revert: > Breaks Win8 GN and Win8 GN(dbg) builders > > Original issue's description: > > Cleanup: Update the path to vector2d_f.h header. > > > > The geometry headers were moved from /ui/gfx/ to ui/gfx/geometry, so > > we are updating the include paths to fix this. > > > > BUG=395370 > > TEST=gfx_unittests > > TBR=sky@chromium.org,danakj@chromium.org > > > > Committed: https://crrev.com/b486216287a3e1f30e81f70444f889d321b7819a > > Cr-Commit-Position: refs/heads/master@{#306960} > > TBR=danakj@chromium.org,sky@chromium.org,tfarina@chromium.org > NOTREECHECKS=true > NOTRY=true > BUG=395370 > > Committed: https://crrev.com/0dffffd306e8e21bd027b26e947ec8b5ffcdfa51 > Cr-Commit-Position: refs/heads/master@{#306962} TBR=danakj@chromium.org,sky@chromium.org,samli@chromium.org NOTREECHECKS=true NOTRY=true BUG=395370 Review URL: https://codereview.chromium.org/762063006 Cr-Commit-Position: refs/heads/master@{#307020}
* Revert of Extract WebGraphicsContext3DImpl from webkit/common/gpu. (patchset ↵samli2014-12-051-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | #12 id:220001 of https://codereview.chromium.org/761243002/) Reason for revert: Suspecting as cause for Win8 GN compile fail Original issue's description: > Extract WebGraphicsContext3DImpl from webkit/common/gpu. > > We decided that it should be fine to move it to gpu/blink now, similar > to gpu/skia_bindings and cc/blink. > > BUG=338338 > TEST=links > R=piman@chromium.org,jamesr@chromium.org > TBR= > > Committed: https://crrev.com/a962b5f152a0a331bc2e1a5a0d4d38e8adece10d > Cr-Commit-Position: refs/heads/master@{#306957} TBR=jamesr@chromium.org,piman@chromium.org,tfarina@chromium.org NOTREECHECKS=true NOTRY=true BUG=338338 Review URL: https://codereview.chromium.org/763963003 Cr-Commit-Position: refs/heads/master@{#306985}
* Revert of Cleanup: Update the path to vector2d_f.h header. (patchset #1 id:1 ↵samli2014-12-051-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | of https://codereview.chromium.org/781863004/) Reason for revert: Breaks Win8 GN and Win8 GN(dbg) builders Original issue's description: > Cleanup: Update the path to vector2d_f.h header. > > The geometry headers were moved from /ui/gfx/ to ui/gfx/geometry, so > we are updating the include paths to fix this. > > BUG=395370 > TEST=gfx_unittests > TBR=sky@chromium.org,danakj@chromium.org > > Committed: https://crrev.com/b486216287a3e1f30e81f70444f889d321b7819a > Cr-Commit-Position: refs/heads/master@{#306960} TBR=danakj@chromium.org,sky@chromium.org,tfarina@chromium.org NOTREECHECKS=true NOTRY=true BUG=395370 Review URL: https://codereview.chromium.org/771283003 Cr-Commit-Position: refs/heads/master@{#306962}
* Cleanup: Update the path to vector2d_f.h header.tfarina2014-12-051-2/+2
| | | | | | | | | | | | | The geometry headers were moved from /ui/gfx/ to ui/gfx/geometry, so we are updating the include paths to fix this. BUG=395370 TEST=gfx_unittests TBR=sky@chromium.org,danakj@chromium.org Review URL: https://codereview.chromium.org/781863004 Cr-Commit-Position: refs/heads/master@{#306960}
* Extract WebGraphicsContext3DImpl from webkit/common/gpu.tfarina2014-12-051-1/+1
| | | | | | | | | | | | | | We decided that it should be fine to move it to gpu/blink now, similar to gpu/skia_bindings and cc/blink. BUG=338338 TEST=links R=piman@chromium.org,jamesr@chromium.org TBR= Review URL: https://codereview.chromium.org/761243002 Cr-Commit-Position: refs/heads/master@{#306957}
* aw: Add more trace events to drawing functionsboliu2014-12-034-3/+19
| | | | | | | | | | | Add more events with an eye towards debugging as well as accounting all the time spent on RenderThread. BUG= Review URL: https://codereview.chromium.org/778643003 Cr-Commit-Position: refs/heads/master@{#306676}
* Re-revert of "Remove SSLClientAuthHandler's RDH dependency." ↵davidben2014-12-032-0/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | (https://codereview.chromium.org/596873002) Reason for revert: Causes browser crash if URL request is cancelled during client cert loading. This is a reland of https://codereview.chromium.org/766463002/ which reverted https://codereview.chromium.org/596873002. The revert was reverted because it introduced a memory leak. The source of this memory leak was a reference cycle in chrome/browser/chromeos/net/client_cert_filter_chromeos.cc whose fix is included in this CL. If InitIfSlotsAvailable returns true (synchronously), the callback should not be retained. This was not a problem before as the code in question was introduced after https://codereview.chromium.org/596873002 which unrefcounted an object. After this is merged into M-40, https://codereview.chromium.org/596873002 should be reworked with the crash fixed as this reference cycle is too subtle and the object really needn't be reference-counted. TBR=mmenke@chromium.org,benm@chromium.org,jam@chromium.org BUG=422765, 427844, 376003 Review URL: https://codereview.chromium.org/773003004 Cr-Commit-Position: refs/heads/master@{#306649}
* Revert of Revert of "Remove SSLClientAuthHandler's RDH dependency." ↵robliao2014-11-262-2/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | (https://codereview.chromium.org/59687300… (patchset #2 id:20001 of https://codereview.chromium.org/755933002/) Reason for revert: Leak in SSLUITestWithClientCert.TestWSSClientCert http://build.chromium.org/p/chromium.memory/builders/Linux%20Chromium%20OS%20ASan%20LSan%20Tests%20%281%29/builds/4654/steps/browser_tests/logs/TestWSSClientCert First reported leak: Indirect leak of 104 byte(s) in 1 object(s) allocated from: #0 0x57659b in operator new(unsigned long) (/tmp/run_tha_test0QXqvt/out/Release/browser_tests+0x57659b) #1 0x2c7762d in ProfileIOData::ResourceContext::CreateClientCertStore() chrome/browser/profiles/profile_io_data.cc:920:7 #2 0x6c908bd in content::ResourceLoader::OnCertificateRequested(net::URLRequest*, net::SSLCertRequestInfo*) content/browser/loader/resource_loader.cc:298:7 #3 0x3e3e15f in net::URLRequest::NotifyCertificateRequested(net::SSLCertRequestInfo*) net/url_request/url_request.cc:1109:5 #4 0x405ea0d in net::URLRequestHttpJob::OnStartCompleted(int) net/url_request/url_request_http_job.cc:925:5 #5 0x3ccd86c in Run base/callback.h:396:12 #6 0x3ccd86c in net::HttpCache::Transaction::DoCallback(int) net/http/http_cache_transaction.cc:710 #7 0x3cca160 in HandleResult net/http/http_cache_transaction.cc:716:5 #8 0x3cca160 in net::HttpCache::Transaction::DoLoop(int) net/http/http_cache_transaction.cc:977 #9 0x3cc7a68 in net::HttpCache::Transaction::OnIOComplete(int) net/http/http_cache_transaction.cc:2934:3 #10 0xc78765a in Run base/callback.h:396:12 #11 0xc78765a in DevToolsNetworkTransaction::OnCallback(int) chrome/browser/devtools/devtools_network_transaction.cc:73 #12 0x3ceab0c in Run base/callback.h:396:12 #13 0x3ceab0c in DoCallback net/http/http_network_transaction.cc:577 #14 0x3ceab0c in net::HttpNetworkTransaction::OnIOComplete(int) net/http/http_network_transaction.cc:594 #15 0x394b58f in Run base/callback.h:396:12 #16 0x394b58f in base::debug::TaskAnnotator::RunTask(char const*, char const*, base::PendingTask const&) base/debug/task_annotator.cc:63 #17 0x388dd3c in base::MessageLoop::RunTask(base::PendingTask const&) base/message_loop/message_loop.cc:448:3 #18 0x388e49b in base::MessageLoop::DeferOrRunPendingTask(base::PendingTask const&) base/message_loop/message_loop.cc:458:5 #19 0x388ef57 in base::MessageLoop::DoWork() base/message_loop/message_loop.cc:567:13 #20 0x3932db7 in base::MessagePumpLibevent::Run(base::MessagePump::Delegate*) base/message_loop/message_pump_libevent.cc:232:21 #21 0x38c0c1c in base::RunLoop::Run() base/run_loop.cc:55:3 #22 0x388c843 in base::MessageLoop::Run() base/message_loop/message_loop.cc:310:3 #23 0x6af33c1 in content::BrowserThreadImpl::IOThreadRun(base::MessageLoop*) content/browser/browser_thread_impl.cc:218:3 #24 0x6af375e in content::BrowserThreadImpl::Run(base::MessageLoop*) content/browser/browser_thread_impl.cc:253:14 #25 0x390cec1 in base::Thread::ThreadMain() base/threading/thread.cc:228:5 #26 0x3901e17 in base::(anonymous namespace)::ThreadFunc(void*) base/threading/platform_thread_posix.cc:80:3 #27 0x7f99b0016e99 in start_thread /build/buildd/eglibc-2.15/nptl/pthread_create.c:308 Original issue's description: > Revert of "Remove SSLClientAuthHandler's RDH dependency." (https://codereview.chromium.org/596873002) > > Reason for revert: > Causes browser crash if URL request is cancelled during client cert loading. > > BUG=422765, 427844, 376003 > TBR=davidben@chromium.org > > Committed: https://crrev.com/e18bf1d9005b08bba3637b16d744245e5467af0e > Cr-Commit-Position: refs/heads/master@{#305707} TBR=pneubeck@chromium.org,davidben@chromium.org,mattm@chromium.org NOTREECHECKS=true NOTRY=true BUG=422765, 427844, 376003 Review URL: https://codereview.chromium.org/750633005 Cr-Commit-Position: refs/heads/master@{#305828}
* Revert of "Remove SSLClientAuthHandler's RDH dependency." ↵mattm2014-11-252-0/+2
| | | | | | | | | | | | | | (https://codereview.chromium.org/596873002) Reason for revert: Causes browser crash if URL request is cancelled during client cert loading. BUG=422765, 427844, 376003 TBR=davidben@chromium.org Review URL: https://codereview.chromium.org/755933002 Cr-Commit-Position: refs/heads/master@{#305707}
* aw: Clean up unused command line switchboliu2014-11-251-1/+0
| | | | | | | | | | | | Remove the switch used by AOSP to force webview into software mode. AOSP has rolled to Android Lollipop and this backwards compatiblity hack is no longer needed. BUG=430997 Review URL: https://codereview.chromium.org/751583005 Cr-Commit-Position: refs/heads/master@{#305655}
* Add net_log events for data reduction proxy conditions. They will be exposed ↵jeremyim2014-11-243-2/+21
| | | | | | | | | | in the #bandwidth net-internals tab in a future CL. BUG=346356 Review URL: https://codereview.chromium.org/719373002 Cr-Commit-Position: refs/heads/master@{#305496}
* Remove NetworkDelegate::OnBeforeSocketStreamConnectricea2014-11-212-9/+0
| | | | | | | | | | | | | The old WebSocket implementation has been removed, so OnBeforeSocketStreamConnect() will never be called. Remove all implementations. BUG=423201 TEST=compile Review URL: https://codereview.chromium.org/652363005 Cr-Commit-Position: refs/heads/master@{#305176}
* Android platform provides a local proxy that also handles PAC proxy ↵sgurun2014-11-212-12/+20
| | | | | | | | | | | | | | resolution. The CL below modified enabled chrome network stack to start using V8 proxy resolution on Android platform: https://codereview.chromium.org/573013002/ however webview cannot use V8 proxy resolution since Webview is significantly different from chrome: Webview is single process vs. chrome is multithreaded and use of V8 isolates creates concurrency problems with Blink's use of V8 isolates. For now, we want to establish the prior behavior for android webview and consider making changes to v8 proxy resolver in future. BUG=432539 Review URL: https://codereview.chromium.org/742023003 Cr-Commit-Position: refs/heads/master@{#305166}
* Add cc::RendererSettings for use in with cc::Displayjbauman2014-11-201-3/+3
| | | | | | | | The settings are taken from the ui::Compositor's LayerTreeSettings. Previously it was just using the default LayerTreeSettings, which can be suboptimal. Review URL: https://codereview.chromium.org/675083002 Cr-Commit-Position: refs/heads/master@{#305087}
* aw: Workaround qualcomm driver bugboliu2014-11-201-0/+3
| | | | | | | | | | | This is a small webview-specific workaround that affects all drivers. Will need a better driver/version specific workaround later. BUG=434570 Review URL: https://codereview.chromium.org/722683004 Cr-Commit-Position: refs/heads/master@{#305056}
* [Android WebView] Store HTTP cache in a cache directorymnaganov2014-11-184-6/+45
| | | | | | | | | | | | | Previously WebView was storing HTTP cache data in the app's data directory. This change switches to use the directory provided by Context.getCacheDir(). Also, after starting, WebView cleans up the old cache directory to save space. BUG=245549 Review URL: https://codereview.chromium.org/722413002 Cr-Commit-Position: refs/heads/master@{#304670}
* The default Web Notification permission should be denied.peter2014-11-152-29/+0
| | | | | | | | | | | | | When the content embedder does not (or only partially) implement support for Web Notifications, permission calls for the API should not be telling other code-paths that it's granted. This causes confusing behavior for the Web exposed API. BUG=398070 Review URL: https://codereview.chromium.org/420283003 Cr-Commit-Position: refs/heads/master@{#304337}
* Disable channel idsgurun2014-11-151-0/+1
| | | | | | | | | | | | | | Android webview does not use channel id since there is no API to share channel ids and this would break some apps that use different network stacks. This change is to explicitly disable it. BUG=433427 internal bug b/18297802 Review URL: https://codereview.chromium.org/731593003 Cr-Commit-Position: refs/heads/master@{#304316}
* Pass TouchMajor to HitTestResulthush2014-11-132-7/+11
| | | | | | | | | | | | | | | | Blink side change to do hit-test that mimics GestureTap: https://codereview.chromium.org/470833002 Without the touch major information, HitTestResult may not match how touch events are handled in content view core. As a result, it is possible that the user touches a link while the hitTestResult returns unknown type. BUG=403865 Review URL: https://codereview.chromium.org/475633002 Cr-Commit-Position: refs/heads/master@{#304067}
* Migrate HostZoomMap to live in StoragePartition.wjmaclean2014-11-122-0/+8
| | | | | | | | | | | | | | This CL changes the persistence of host zoom levels to be on a per-storage-partition basis, as opposed to (the current) per-profile basis. This is needed to allow WebView content (withing apps) to keep their zoom levels independent of those in the main browser window. BUG=335317 Review URL: https://codereview.chromium.org/393133002 Cr-Commit-Position: refs/heads/master@{#303841}
* aw: Refactor software rendering helperboliu2014-11-122-77/+23
| | | | | | | | | | | | | | | | | | | Move SkCanvas creation entirely out of BrowserViewRenderer. This makes testing software draws much easier. AwContents is now responsible for obtaining a SkCanvas from the java canvas object, and pass it to BrowserViewRenderer. This also implies moving the logic to decide between software and hardware draws to AwContents. Major refactor to JavaHelper to enable this new mode of usage. BUG=418346 Review URL: https://codereview.chromium.org/712243002 Cr-Commit-Position: refs/heads/master@{#303758}
* Partial test harness for android webview renderingboliu2014-11-116-24/+156
| | | | | | | | | | | | | | | | | | | Goal for this class of tests is to test the classes that interacts between the android view system and the chromium compositor. It is modelled after cc::LayerTreeTest This is the first step that brings up a BrowserViewRenderer object in a test. Individual pieces: * Add a TestSynchronousCompositor in content * Remove BrowserViewRenderer dependency on WebContents BUG=418346 Review URL: https://codereview.chromium.org/703953002 Cr-Commit-Position: refs/heads/master@{#303736}
* cc: Make separate interface for BeginFrame ipc from OutputSurfacesimonhong2014-11-111-1/+2
| | | | | | | | | | | | Decouple BeginFrame message from OutputSurface and creates new interface. R=danakj@chromium.org, brianderson@chromium.org, skyostil@chromium.org, piman@chromium.org, boliu@chromium.org BUG=416760 TEST=cc_unittests, content_unittests Review URL: https://codereview.chromium.org/619843002 Cr-Commit-Position: refs/heads/master@{#303715}
* Move data reduction proxy bypass logic to URLRequestInterceptorbengr2014-11-082-8/+6
| | | | | | | | | | | | | Moves the bypass logic, which was in NetworkDelegate to the URLRequestInterceptor. As a result, the mechanism to retry a request without the data reduction proxy is not exposed to higher layers. BUG=429505 Review URL: https://codereview.chromium.org/698383002 Cr-Commit-Position: refs/heads/master@{#303343}
* aw: Move SharedRendererState out of AwContentsboliu2014-11-072-3/+12
| | | | | | | | | | Small little clean up. BUG=418346 Review URL: https://codereview.chromium.org/706033003 Cr-Commit-Position: refs/heads/master@{#303310}
* Adds MaybeInterceptRedirect and MaybeInterceptResponse to the ↵bengr2014-11-072-1/+25
| | | | | | | | | | URLRequestInterceptor interface as well as to URLRequestJobFactory. BUG=429505 Review URL: https://codereview.chromium.org/686343002 Cr-Commit-Position: refs/heads/master@{#303154}
* aw: Destroy ContentViewCore and WebContents togetherboliu2014-11-032-6/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | Even though it's having WebContents without a ContentViewCore is a support scenario in //content, it's not a well tested path, especially the webview only code. This makes sure ContentViewCore and WebContents are destroyed together. Instead of posting on native side to destroy WebContents, post on Java side and have one call destroy both Java and native side. More specific details: * All caller except destroy to AwContents.destroyNatives either have already detached webview so ok to move detach to destroy. * Need to remove some asserts that objects are null when mIsDestroyed is true. * Reordered some native AwContents members so they are naturally destroyed in the correct order. BUG= Review URL: https://codereview.chromium.org/690553002 Cr-Commit-Position: refs/heads/master@{#302513}
* Remove DCHECK from AwResourceDispatcherHostDelegate.mkosiba2014-10-311-5/+1
| | | | | | | | | | | | The DCHECK is triggering quite frequently and causing the test shell to be very unstable. The PingLoader has a legitimate reason for sending "GET" requests without a frame id so the DCHECK no longer makes sense. BUG=389320 Review URL: https://codereview.chromium.org/697693002 Cr-Commit-Position: refs/heads/master@{#302278}
* Part 2: Refactor Android WebView graphics codehush2014-10-306-50/+184
| | | | | | | | | | | | | | | | 1. Change the object ownership to be: AwContents -> BrowserViewRenderer -> SharedRendererState -> HardwareRenderer (The left object owns the right object.) 2. Move DrawGL, InitializeHardwareIfNeeded, ReleaseHardwareIfNeeded from AwContents to SharedRendererState. BUG=418346 Review URL: https://codereview.chromium.org/653173004 Cr-Commit-Position: refs/heads/master@{#302011}
* Move data_reduction_proxy_params* from components/data_reduction_proxy/jeremyim2014-10-292-2/+2
| | | | | | | | | | browser => common BUG=374264 Review URL: https://codereview.chromium.org/665533003 Cr-Commit-Position: refs/heads/master@{#301772}
* gpu: Add CHROMIUM_image support to in-process command buffer.reveman2014-10-291-1/+3
| | | | | | | | | | | | | | | | | | | | | | | | This makes it possible to use the in-process command buffer with CHROMIUM_image extension. GpuMemoryBufferManager and ImageFactory interfaces need to be provided when creating an in-process command buffer. Existing implementations of these interfaces in content/ should work just fine. cc::TestGpuMemoryBufferManager and cc::TestImageFactory are used for cc pixel tests. They provide a minimal implementation of these interfaces using shared memory. Note: shared memory is used for in-process testing as it maps better to the GpuMemoryBuffer framework that has been designed for multi-process usage. Non-shared memory might be a bit more efficient but it would require more complexity and not match real usage as well. BUG=423533 Review URL: https://codereview.chromium.org/665463003 Cr-Commit-Position: refs/heads/master@{#301763}
* Prefix CommandLine usege with base namespace (Part 2: android_webview/)pgal.u-szeged2014-10-282-3/+5
| | | | | | | | | | | Prefix all CommandLine usage in the android_webview/ directory with the base:: namespace. BUG=422426 Review URL: https://codereview.chromium.org/661443003 Cr-Commit-Position: refs/heads/master@{#301593}
* Remove limit on number of resources in ccboliu2014-10-286-531/+17
| | | | | | | | | | | | | | | | This ability was added for Android WebView to limit the number of gralloc buffers, which has a low limit on the number of allocations. Android WebView no longer uses gralloc buffers, so no need to keep this ability. Use memory limit to limit the staging resource pool for 1-copy rasterizer. BUG=426548 Review URL: https://codereview.chromium.org/643993005 Cr-Commit-Position: refs/heads/master@{#301559}
* Remove the strict dependency on WebContents for Web Notifications.peter2014-10-272-2/+4
| | | | | | | | | | | | This patch changes Notification objects created for Web Notifications to go without an associated WebContents. Because we want Web Notifications to be used from Web Workers, this no longer is a valid assumption to make. BUG=415160 Review URL: https://codereview.chromium.org/578883003 Cr-Commit-Position: refs/heads/master@{#301394}
* Group the different permission related methods in the content api.miguelg2014-10-272-74/+56
| | | | | | | | BUG=392145 Review URL: https://codereview.chromium.org/622793002 Cr-Commit-Position: refs/heads/master@{#301338}
* aw: Remove jobject canvas from native RequestDrawGLboliu2014-10-254-17/+15
| | | | | | | | | | | For onDraw with java canvas, the RequestDrawGL can be made from Java side. All other uses pass NULL for java canvas, so remove it. BUG= Review URL: https://codereview.chromium.org/679513002 Cr-Commit-Position: refs/heads/master@{#301261}
* Part 1: Refactor Android WebView graphics codehush2014-10-245-77/+94
| | | | | | | | | | | | | | | | | | | 1. Change the object ownership to be: AwContents -> BrowserViewRenderer -> SharedRendererState AwContents -> HardwareRenderer (The left object owns the right object) HardwareRenderer will be owned by SharedRendererState in Part 2 CL: https://codereview.chromium.org/653173004/. 2. Rename some functions in SharedRendererState to be clear about the threads they are called. BUG=418346 Review URL: https://codereview.chromium.org/655813004 Cr-Commit-Position: refs/heads/master@{#301233}
* gpu: Rewrite MailboxSynchronizersievers2014-10-241-10/+0
| | | | | | | | | | | | The current version that tries to synchronize the tracked textures between separate MailboxManager instances does not get things right, see added tests. Simplify the problem by subclassing MailboxManager. NOTRY=True Review URL: https://codereview.chromium.org/670953004 Cr-Commit-Position: refs/heads/master@{#301165}
* Cleanup: Update the paths to gfx vector2d_conversions.h include.tfarina2014-10-241-1/+1
| | | | | | | | | | | | | The geometry headers were moved from /ui/gfx/ to ui/gfx/geometry, so we are updating the include paths to fix this. BUG=395370 TEST=None TBR=ben@chromium.org,danakj@chromium.org Review URL: https://codereview.chromium.org/669153008 Cr-Commit-Position: refs/heads/master@{#301035}
* Activate data reduction proxy after profile initialization is complete.kundaji2014-10-241-0/+1
| | | | | | | | | | | | | When we activate data reduction proxy, a request is made to the prober URL. If this request is made before ProfileManager::DoFinalInitForServices() completes, it causes a crash because HostContentSettingsMap is not yet initialized. BUG=424260 Review URL: https://codereview.chromium.org/665703002 Cr-Commit-Position: refs/heads/master@{#300995}
* aw: Do not limit number of tilesboliu2014-10-231-2/+3
| | | | | | | | | | By making the limit unrealistically high. BUG=426548 Review URL: https://codereview.chromium.org/660383004 Cr-Commit-Position: refs/heads/master@{#300991}