GitLab

Follow-up to 113090 to see which ModelChangingSyncerCommand triggers
a perf regression.

BUG=97832
TEST=
TBR=tim

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

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

Revert 113300 - Revert of 112134 of Revert 112130 - Close idle connections / SPDY sessions when needed.

Due to the idle connection state being held by different socket pools, it's possible for one socket pool to hold an idle socket in a lower layer socket pool. From the lower level socket pool's perspective, the socket is being "actively" used. From the higher socket pool's (including SpdySession, which is more of a connection manager) perspective, the connection is idle and can be closed if we have hit a limit.

Normally this isn't a big deal, except when we have a lot of idle SPDY connections and are connecting via a proxy, so we have low connection limits through the proxy server. We address this problem by allowing lower-level socket pools to tell higher level socket pools to close a socket. 

Fixed ASAN test failures by removing  .Times(1) and .Times(2)  from CloseMultipleIdleSocketsHeldByLayeredPoolWhenNeeded unittest (this removes the tests relying on the order of std::set in CloseOneIdleConnectionInLayeredPool). ASAN is probably causing the memory allocator to allocate the pools differently. The std::set is ordered by LayeredPool* which is the address of the LayeredPool (willchan).


BUG=62364,92244, 105839
TEST=none


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

TBR=willchan@chromium.org
Review URL: http://codereview.chromium.org/8745007

TBR=willchan@chromium.org

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

TBR=rtenneti@chromium.org
Review URL: http://codereview.chromium.org/8825014

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

Disabling 2 tests on pyauto chromium vista bot only.  They seem fine on other
bots / setups.

R=frankf@chromium.org
BUG=
TEST=

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

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

BUG=none
TEST=compiles
TBR=sky
Review URL: http://codereview.chromium.org/8771015

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

I keep getting confused between things like AudioImpl and PPB_Audio_Impl. This uses _shared for the names, so now we have _impl, _proxy, and _shared which makes more sense.

I also removed the ppb_opengles2_impl file since it was just a forward to the shared version.

BUG=
TEST=

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

TBR=brettw@chromium.org
Review URL: http://codereview.chromium.org/8824015

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

Due to the idle connection state being held by different socket pools, it's possible for one socket pool to hold an idle socket in a lower layer socket pool. From the lower level socket pool's perspective, the socket is being "actively" used. From the higher socket pool's (including SpdySession, which is more of a connection manager) perspective, the connection is idle and can be closed if we have hit a limit.

Normally this isn't a big deal, except when we have a lot of idle SPDY connections and are connecting via a proxy, so we have low connection limits through the proxy server. We address this problem by allowing lower-level socket pools to tell higher level socket pools to close a socket. 

Fixed ASAN test failures by removing  .Times(1) and .Times(2)  from CloseMultipleIdleSocketsHeldByLayeredPoolWhenNeeded unittest (this removes the tests relying on the order of std::set in CloseOneIdleConnectionInLayeredPool). ASAN is probably causing the memory allocator to allocate the pools differently. The std::set is ordered by LayeredPool* which is the address of the LayeredPool (willchan).


BUG=62364,92244, 105839
TEST=none


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

TBR=willchan@chromium.org
Review URL: http://codereview.chromium.org/8745007

TBR=willchan@chromium.org

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

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

This is temporary.  Until nduca has time to look at the failure.

BUG=106557
TEST=gpu win release green again
TBR=nduca
Review URL: http://codereview.chromium.org/8823004

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

BUG=
TEST=GpuFeatureBrowsertest
R=jbates
Review URL: http://codereview.chromium.org/8816027

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

BUG=none
TEST=DownloadProtectionServiceTest


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

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

Add ONC VPN support for OpenVPN and L2TP/IPsec VPNs

Also simplifies some of the parsing code, adds type validation, brings
the WiFi security parsing up to spec, and improves the unit tests for
wifi so that they all verify the property_map_ fields (the only ones
actually used to configure the network). Also fixed a general parser
problem where certificates wouldn't be parsed if network configs were.

BUG=chromium-os:23476
chromium-os:23477
TEST=manual import of ONC file with openvpn network, unit_tests

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=113120
Review URL: http://codereview.chromium.org/8821023

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

or sent ping (like below):

t=1323200233634 [st=196851]     SPDY_SESSION_PING
                                --> type = "sent"
                                --> unique_id = 3
t=1323200233638 [st=196855]     SPDY_SESSION_PING
                                --> type = "received"
                                --> unique_id = 3
BUG=106499
R=willchan
TEST=spdy unit tests

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

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

Also, add padding around the tooltip text and disable the
border when shadows are enabled.

I'm also updating the existing shadow code in aura_shell to
clip corner images as needed when they're too large to fit
within the requested size.

BUG=106507
TEST=added


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

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

BUG=106529
TEST=removed suppressions from mem/heap check tools


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

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

R=estade@chromium.org,csilv@chromium.org
BUG=92912
TEST=Drag a [removable] app over the trash in either LTR or RTL and be greeted
by a sweet trash can lid opening animation.

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=113167

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

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

Write the ActiveX control from an external script in all test documents.  This works around a "feature" of IE versions released between April, 2006 and April, 2008 that required the user to click on an ActiveX control to "activate" it.  See http://msdn.microsoft.com/en-us/library/ms537508.aspx.

I also added asserts for the copying of CFInstance and CFInstall so that the whole test aborts if the files can't be copied, since otherwise test results are bogus.

BUG=60987
TEST=no manual testing needed; chrome_frame_tests.exe will either become better or worse.

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

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

I keep getting confused between things like AudioImpl and PPB_Audio_Impl. This uses _shared for the names, so now we have _impl, _proxy, and _shared which makes more sense.

I also removed the ppb_opengles2_impl file since it was just a forward to the shared version.

BUG=
TEST=

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

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

failure.

BUG=none
TEST=none
R=oshima@chromium.org


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

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

BUG=106157
TEST=browser_tests

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

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

HttpStreamFactoryImpl::Job::OnStreamReadyCallback.

As a side-effect of OnPipelineFeedback() calling OnPipelineHasCapacity() on all
of the pipelines, some requests are late-bound to existing pipelines with
capacity. However, those requests may have been in the process of establishing
new pipelines. By stealing those requests, the new pipelines become empty and
are deleted. Later iterations of the loop in OnPipelineFeedback() weren't
careful about checking to see if the pipeline was deleted before calling
OnPipelineHasCapacity() on it.

BUG=106313
TEST=net_unittests under valgrind


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

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

First in a series of patches to replace ModelType{Bit,}Set with
ModelEnumSet (which will then be renamed to ModelTypeSet).

BUG=79970
TEST=


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

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

Get rid of the last view_messages.h include in chrome, other than tab_contents_views_mac.mm and render_view_host_delegate_helper.cc, both of which Avi is working on. I added a test method to fake a UpdateRect IPC message for the thumbnail test.

BUG=98716

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

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

Cleanup the connection member of the engine object when destructing.
Properly unregister IMEs when unloading extensions.

BUG=chromium-os:21357,21514,21355
TEST=See bugs


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

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

BUG=none
TEST=none
R=csilv

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

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

DnsClient provides a generic DNS client that allows fetching resource records. 
DnsClient is very lightweight and does not support aggregation, queuing or 
prioritization of requests. 

This is the first CL in a series to merge AsyncHostResolver into 
HostResolverImpl. 

Also introduces general-purpose BigEndianReader/Writer. 

BUG=90881
TEST=./net_unittests


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

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

Pull the CrAppProtocol autorelease-pool handling down into 
MessagePumpCrApplication, which is selected at Create() if NSApp 
implements the right protocol. UsingCrApp() allows clients to 
confirm the correct setup (unfortunately, synchronizing NSApp 
initialization and MessagePump::Create() would be intrusive).

Also push CrAppProtocol and CrAppControlProtocol implementation into
BrowserCrApplication, and reparent that class from NSApplication.

Reparent ServiceCrApplication on NSApplication and rename.

Remove CrApplication registration from gpu, plugin, and renderer
mains.

Remove MockCrApp dependency from remoting sample code.

BUG=102224


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

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

BUG=None
TEST=None


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

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

BUG=none
TEST=manual


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

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

BUG=None
TEST=


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

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

BUG=101214
BUG=106490

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=113007

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

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

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

Update the pre-built ICU data file for the following changes:

1. Update Olson timezone data file to 2011n (the latest)

  Among other things, Moscow time should be *always* 4 hrs ahead of UTC.

2. Fix a typoe in the timezone name for US Pacific time in Korean locale.
3. Fix AmPm markers in Indonesian locale
4. Fix the abbrebiated day name for Wednesday in Spanish
5. Add timezone display data for the following locales:
   am, bn, fa, gu, kn, ml, mr, sw, ta, te
  (they're supposed to be included, but were missing in the previous pre-built data check-in.) 

For the first two changes, see
   http://codereview.chromium.org/8724009


See  also http://crrev.com/113037 for more details.

BUG=74566, chromium-os:23256
TEST=1. Change the MSK (Moscow timezone) during winter. It should be UTC+4:00 instead of UTC+3:00
2. In Korean locale, the US Pacific time display name should start with '북미 태평양' instead of '북미 평양'.
3. In Indonesian locale, AmPm markers should be 'AM' and 'PM'.
4. In locales listed for item 5 above, the timezone selection menu shows time zone names in the language on CrOS. 
TBR=nkostylev
Review URL: http://codereview.chromium.org/8800022

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

Adds XInput redist DLL which is a new dependency of recently added gamepad code,
and is required on Win XP. laforge has OK'd the size addition to the final
package (~100K).

BUG=79050

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=112812

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

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

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

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

1102: Lots of python style cleanups; let gyp-mac-tool use `/usr/bin/env python`.
1103: Add pylint presubmit check.

BUG=none
TEST=none
TBR=mark

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

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

For now, the trybot will be compile-only to help with reliability.

BUG=None
TEST=


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

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

Follow-up to 113090 to see which ModelChangingSyncerCommand triggers
a perf regression.

BUG=97832
TEST=
TBR=tim@chromium.org

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

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

TBR=dtseng

BUG=
TEST=

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

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

- Enabled by default for new versions of the plugin
- Now it does not auto-disable if enabled

BUG=89248
TEST=see bug
Review URL: http://codereview.chromium.org/8806034

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

BUG=none
TEST=none
R=csilv

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

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

BUG=none
TEST=none
R=csilv

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

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