summaryrefslogtreecommitdiffstats
path: root/chrome_frame
Commit message (Collapse)AuthorAgeFilesLines
* Revert 75143 - Reland r74842willchan@chromium.org2011-02-161-21/+21
| | | | | | | | | | | | | | The bug was that the ProfilePolicyContext was getting initialized before the ProfileImplIOData::Handle was initialized. BUG=67237 TEST=none Review URL: http://codereview.chromium.org/6526029 TBR=willchan@chromium.org Review URL: http://codereview.chromium.org/6480120 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@75152 0039d316-1c4b-4281-b951-d872f2087c98
* Reland r74842willchan@chromium.org2011-02-161-21/+21
| | | | | | | | | | | The bug was that the ProfilePolicyContext was getting initialized before the ProfileImplIOData::Handle was initialized. BUG=67237 TEST=none Review URL: http://codereview.chromium.org/6526029 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@75143 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 75106 - New policy protobuf protocol.jkummerow@chromium.org2011-02-161-2/+2
| | | | | | | | | | | | | | | | | (Second attempt to land http://codereview.chromium.org/6409040/) - cloud_policy.proto autogenerated from policy_templats.json - C++ method decoding the protobuf also autogenerated from policy_templates.json - changed policy fetching mechanism to fetch new-style policy protobufs BUG=68309, chromium-os:11253, chromium-os:11255 TEST=CloudPolicyCacheTest.*; also manual test against python testserver Review URL: http://codereview.chromium.org/6523058 TBR=jkummerow@chromium.org git-svn-id: svn://svn.chromium.org/chrome/trunk/src@75115 0039d316-1c4b-4281-b951-d872f2087c98
* New policy protobuf protocol.jkummerow@chromium.org2011-02-161-2/+2
| | | | | | | | | | | | | | | (Second attempt to land http://codereview.chromium.org/6409040/) - cloud_policy.proto autogenerated from policy_templats.json - C++ method decoding the protobuf also autogenerated from policy_templates.json - changed policy fetching mechanism to fetch new-style policy protobufs BUG=68309, chromium-os:11253, chromium-os:11255 TEST=CloudPolicyCacheTest.*; also manual test against python testserver Review URL: http://codereview.chromium.org/6523058 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@75106 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 75082 - New policy protobuf protocol.jkummerow@chromium.org2011-02-161-2/+2
| | | | | | | | | | | | | | | - cloud_policy.proto autogenerated from policy_templates.json - C++ method decoding the protobuf autogenerated from policy_templates.json - changed policy fetching mechanism to fetch new-style policy protobufs BUG=68309, chromium-os:11253, chromium-os:11255 TEST=CloudPolicyCacheTest.*; also manual test against python testserver Review URL: http://codereview.chromium.org/6409040 TBR=jkummerow@chromium.org git-svn-id: svn://svn.chromium.org/chrome/trunk/src@75084 0039d316-1c4b-4281-b951-d872f2087c98
* New policy protobuf protocol.jkummerow@chromium.org2011-02-161-2/+2
| | | | | | | | | | | | | - cloud_policy.proto autogenerated from policy_templates.json - C++ method decoding the protobuf autogenerated from policy_templates.json - changed policy fetching mechanism to fetch new-style policy protobufs BUG=68309, chromium-os:11253, chromium-os:11255 TEST=CloudPolicyCacheTest.*; also manual test against python testserver Review URL: http://codereview.chromium.org/6409040 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@75082 0039d316-1c4b-4281-b951-d872f2087c98
* Remove obsolete dependencies that create needless recompiling.jeanluc@chromium.org2011-02-152-12/+12
| | | | | | | | | BUG=71130 Review URL: http://codereview.chromium.org/6523009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@74966 0039d316-1c4b-4281-b951-d872f2087c98
* Fix for a chrome frame crasher seen in the chrome frame mstone 10 beta channel.ananta@chromium.org2011-02-151-1/+2
| | | | | | | | | | | | | | | The crash occurs while processing an accelerator reflected from chrome and based on the crash dump occurs while dereferencing a NULL IOleInPlaceSite member. Fix is to check for the same. BUG=72957 TEST=none at this point. Review URL: http://codereview.chromium.org/6524003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@74883 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 74842 - It seems to have broken the ChromeOS "PFQ bot"?willchan@chromium.org2011-02-141-21/+21
| | | | | | | | | | | | | | | | r74561 added a DCHECK to make sure users didn't try to access the ChromeURLRequestContextGetter from the Profile, since the Profile should only be read on the UI thread. ChromeOS apparently tried to access it from another thread, and therefore hit the new DCHECK. I'm relanding without the DCHECK. I'm also eagerly initializing the off the record context getter to prevent the ChromeOS race. ChromeOS should fix that code so the eager initialization isn't necessary. BUG=none TEST=none Review URL: http://codereview.chromium.org/6484041 TBR=willchan@chromium.org Review URL: http://codereview.chromium.org/6517021 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@74848 0039d316-1c4b-4281-b951-d872f2087c98
* Revert r74728 which reverted r74561 for exposing a ChromeOS bug.willchan@chromium.org2011-02-141-21/+21
| | | | | | | | | | | | | r74561 added a DCHECK to make sure users didn't try to access the ChromeURLRequestContextGetter from the Profile, since the Profile should only be read on the UI thread. ChromeOS apparently tried to access it from another thread, and therefore hit the new DCHECK. I'm relanding without the DCHECK. I'm also eagerly initializing the off the record context getter to prevent the ChromeOS race. ChromeOS should fix that code so the eager initialization isn't necessary. BUG=none TEST=none Review URL: http://codereview.chromium.org/6484041 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@74842 0039d316-1c4b-4281-b951-d872f2087c98
* Revert r74632 "Reland r74561 after fixing ChromeOS build breakage."oshima@google.com2011-02-121-21/+21
| | | | | | | | | | | | This reverts commit f19e6d524e086ed8182a475680e931a9a870d2a8. TBR=willchan@chromium.org BUG=none TEST=chromeos pfq bot should cycle green Review URL: http://codereview.chromium.org/6512009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@74728 0039d316-1c4b-4281-b951-d872f2087c98
* A number of poorly written IE BHO's crash IE if ChromeFrame is the currently ↵ananta@chromium.org2011-02-116-105/+157
| | | | | | | | | | | | | | | | | | | | | | | loaded document. This is because they expect ChromeFrame to implement interfaces like IHTMLDocument2 on the same lines as regular IE documents. Currently in ChromeFrame we patch the invoke methods of these BHO's prior to firing navigation events from ChromeFrame. However this is not enough as these objects also crash for regular navigation events fired from IE when ChromeFrame is loaded. We now don't fire navigation events for buggy BHO's if ChromeFrame is the current document. The BuggyBho handler instance is now created once for the thread. We patch when we receive navigation notifications from Chrome as before. When we receive a notification on our patched event sink we check if CF is loaded and if yes skip the call. Added helpers to chrome frame utils to check if CF is loaded in the current web browser instance. BUG=55932 TEST=none Review URL: http://codereview.chromium.org/6493002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@74691 0039d316-1c4b-4281-b951-d872f2087c98
* Reland r74561 after fixing ChromeOS build breakage.willchan@chromium.org2011-02-111-21/+21
| | | | | | | | | | | | | | | | | Introduce OffTheRecordProfileIOData and ProfileImplIOData. They both inherit from ProfileIOData. The former is for the off the record (incognito) profile. The latter is for the normal ProfileImpl profile. All of the IO related Profile objects are now initialized at the same time, in the subtype implementations of ProfileIOData::LazyInitializeInternal(). I also took this opportunity to clean URLRequestContext up so it is a class and keeps its member variables private. This required touching a fair number of files. TODO: Remove lots of the refcounting of member variables, since they can now be owned by ProfileIOData. BUG=67237 TEST=none Review URL: http://codereview.chromium.org/6500002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@74632 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 74561 for breaking the ChromeOS buildwillchan@chromium.org2011-02-111-21/+21
| | | | | | | | | | | | | | | | | | | | Introduce OffTheRecordProfileIOData and ProfileImplIOData. They both inherit from ProfileIOData. The former is for the off the record (incognito) profile. The latter is for the normal ProfileImpl profile. All of the IO related Profile objects are now initialized at the same time, in the subtype implementations of ProfileIOData::LazyInitializeInternal(). I also took this opportunity to clean URLRequestContext up so it is a class and keeps its member variables private. This required touching a fair number of files. TODO: Remove lots of the refcounting of member variables, since they can now be owned by ProfileIOData. BUG=67237 TEST=none Review URL: http://codereview.chromium.org/6286133 TBR=willchan@chromium.org Review URL: http://codereview.chromium.org/6489020 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@74564 0039d316-1c4b-4281-b951-d872f2087c98
* Introduce OffTheRecordProfileIOData and ProfileImplIOData.willchan@chromium.org2011-02-111-21/+21
| | | | | | | | | | | | | | | They both inherit from ProfileIOData. The former is for the off the record (incognito) profile. The latter is for the normal ProfileImpl profile. All of the IO related Profile objects are now initialized at the same time, in the subtype implementations of ProfileIOData::LazyInitializeInternal(). I also took this opportunity to clean URLRequestContext up so it is a class and keeps its member variables private. This required touching a fair number of files. TODO: Remove lots of the refcounting of member variables, since they can now be owned by ProfileIOData. BUG=67237 TEST=none Review URL: http://codereview.chromium.org/6286133 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@74561 0039d316-1c4b-4281-b951-d872f2087c98
* Remove includes of message headers in headers.jam@chromium.org2011-02-0915-38/+115
| | | | | | Review URL: http://codereview.chromium.org/6458004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@74257 0039d316-1c4b-4281-b951-d872f2087c98
* Silencing a (stupid) build warning:tommi@chromium.org2011-02-081-1/+1
| | | | | | | | | | | chrome_tab_version.rc(6) : warning RC4093: unescaped newline in character constant in inactive code TEST=none BUG=none Review URL: http://codereview.chromium.org/6451010 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@74151 0039d316-1c4b-4281-b951-d872f2087c98
* Remove DCHECK for an argument that may be NULL.tommi@chromium.org2011-02-081-1/+0
| | | | | | | | | TEST=none BUG=none Review URL: http://codereview.chromium.org/6452009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@74150 0039d316-1c4b-4281-b951-d872f2087c98
* Change includes of gfx/* to ui/gfx/*sail@chromium.org2011-02-053-5/+5
| | | | | | | | | BUG=71063 TEST=compiled Review URL: http://codereview.chromium.org/6312156 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@73890 0039d316-1c4b-4281-b951-d872f2087c98
* Added support in chrome frame for registering and unregistering the NPAPI ↵ananta@chromium.org2011-02-043-19/+45
| | | | | | | | | | | | | plugin as per user. This would ensure that the ChromeFrame Vista and above builders work correctly with UAC on. BUG=none TEST=ChromeFrame tests should pass on Vista with UAC on. Review URL: http://codereview.chromium.org/6368108 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@73860 0039d316-1c4b-4281-b951-d872f2087c98
* Disabling the IEOpenInNewWindow chrome frame test on IE7.ananta@chromium.org2011-02-041-0/+5
| | | | | | | | | | BUG=none TEST=none TBR=amit Review URL: http://codereview.chromium.org/6250163 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@73832 0039d316-1c4b-4281-b951-d872f2087c98
* Invoke the OaEnablePerUserTLibRegistration function in oleaut32.dll during ↵ananta@chromium.org2011-02-041-0/+13
| | | | | | | | | | | | | | | | ChromeFrame per user registration on Vista and beyond. This is to workaround a bug on Vista SP1 and up where the mappings of per user type lib reg keys to HKCR don't work as expected. Windows 7 does not have this bug. The function is a noop there. BUG=none TEST=Per user chrome frame tests should work with UAC off Review URL: http://codereview.chromium.org/6286120 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@73831 0039d316-1c4b-4281-b951-d872f2087c98
* Have a consistent UA string with chromeframe tokenamit@chromium.org2011-02-042-18/+30
| | | | | | | | | | | | | | | | | | | When using 'PostPlatform' registry, chromeframe UA token appears in the middle whereas when we dynamically append the UA we place it at the end. This has caused issues with sites looking for it in a specific place. This change modifies the logic while dynamically appending the UA to insert it consistently in the place where PostPlatform puts it. BUG=70024 TEST=covered by HtmlUtilUnittest.AddChromeFrameToUserAgentValue Review URL: http://codereview.chromium.org/6246088 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@73830 0039d316-1c4b-4281-b951-d872f2087c98
* Add support for IE9 to user-level installations of Chrome Frame.robertshield@chromium.org2011-02-043-13/+164
| | | | | | | | | | BUG=70274 TEST=Install user-level Chrome Frame without admin privileges on a system with IE9. Observe CF being loaded in IE. Review URL: http://codereview.chromium.org/6246040 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@73721 0039d316-1c4b-4281-b951-d872f2087c98
* Fixes for the following chrome frame tests to get them to pass on the per ↵ananta@chromium.org2011-02-042-2/+3
| | | | | | | | | | | | | | | user CF builders. 1. CFBackForward :- Increased the expected timeout for this test. 2. TopLevelPostReissueFromChromeFramePage :- The Onload notification fires only once at times. Adjusted the expectation to account for that. BUG=none TEST=Covered by existing ChromeFrame tests. Review URL: http://codereview.chromium.org/6286095 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@73713 0039d316-1c4b-4281-b951-d872f2087c98
* Refactor HttpNetworkSession construction.willchan@chromium.org2011-02-032-56/+9
| | | | | | | | | | | Introduce HttpNetworkSession::Params. This should make it easy to add new optionally NULL parameters. I also took this opportunity to eliminate some copy/pastes of TestURLRequestContext and make them use the original. I was also able to remove the need for ClientSocketFactory::GetDefaultFactory() calls and new SpdySessionPool() calls in most places. BUG=none TEST=none Review URL: http://codereview.chromium.org/6349028 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@73669 0039d316-1c4b-4281-b951-d872f2087c98
* Fix ChromeFrame full tab refresh tests to get them to work more reliably. We ↵ananta@chromium.org2011-02-034-15/+21
| | | | | | | | | | | | | | | | now call Refresh on the WebBrowser interface when we receive the OnLoad notification from Chrome. Increased the timeout for the refresh tests to get them to work correctly on the slower CF builders. BUG=none TEST=ChromeFrame tests. Review URL: http://codereview.chromium.org/6410049 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@73666 0039d316-1c4b-4281-b951-d872f2087c98
* Move elevation registration into a separate rgs script and leave removal of ↵tommi@chromium.org2011-02-035-23/+36
| | | | | | | | | | | it up to the installer. TEST=Manual registration of npchrome_frame continues to register the elevation policy, but unregistration of the DLL will not remove the elevation policy (since the installer now does that). BUG=none Review URL: http://codereview.chromium.org/6286083 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@73645 0039d316-1c4b-4281-b951-d872f2087c98
* Reverting this as this breaks chrome frame tests everywhere. The problem is ↵ananta@chromium.org2011-02-032-15/+19
| | | | | | | | | | | | | | | | | | | that the chrome frame typelib is not part of chrome frame resources, which causes type library loads to fail. Please fix and reland Revert 73524 - Rework how chrome_tab.idl is processed for chrome_frame. This is to avoid the compiler (at least in VS2010) from needlessly trying to rebuild what is fine. BUG=71130 TEST=Clean compile & usual tests. Review URL: http://codereview.chromium.org/6260049 TBR=jeanluc@chromium.org Review URL: http://codereview.chromium.org/6246062 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@73578 0039d316-1c4b-4281-b951-d872f2087c98
* Rework how chrome_tab.idl is processed for chrome_frame. This is to avoid ↵jeanluc@chromium.org2011-02-022-19/+15
| | | | | | | | | | | the compiler (at least in VS2010) from needlessly trying to rebuild what is fine. BUG=71130 TEST=Clean compile & usual tests. Review URL: http://codereview.chromium.org/6260049 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@73524 0039d316-1c4b-4281-b951-d872f2087c98
* Fix ChromeFrame net tests which regressed with the change to register the ↵ananta@chromium.org2011-02-021-0/+2
| | | | | | | | | | | | | | chrome frame binary for every test run. We need to register the path provider prior to attempting the registration. BUG=none TEST=chrome frame net tests should run TBR=amit Review URL: http://codereview.chromium.org/6312087 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@73433 0039d316-1c4b-4281-b951-d872f2087c98
* More ChromeFrame per user test fixes. ananta@chromium.org2011-02-021-1/+1
| | | | | | | | | BUG=none TEST=chrome frame tests should run in per user mode. Review URL: http://codereview.chromium.org/6410016 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@73412 0039d316-1c4b-4281-b951-d872f2087c98
* Added support for running per user chrome frame tests on the relevant ↵ananta@chromium.org2011-02-0111-29/+82
| | | | | | | | | | | | | | | | | | | | | | ChromeFrame builders. This includes the chrome_frame_net_tests, chrome_frame_tests and the chrome_frame_unittests test suites. We determine whether per user tests are to be run based on a registry config value PerUserTestBed under HKCU\Software\Google\ChromeFrame. As part of this CL we also register chrome frame in the chrome frame net tests suite. We unregister the chrome frame dll on the builder for per user installs after every test run. BUG=none TEST=Existing chrome frame tests. Review URL: http://codereview.chromium.org/6347056 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@73354 0039d316-1c4b-4281-b951-d872f2087c98
* Disabling building of CEEE; we will be moving the code to a separatejoi@chromium.org2011-01-315-181/+2
| | | | | | | | | | | repository. Removing chrome_frame -> ceee dependency. BUG=none TEST=none Review URL: http://codereview.chromium.org/6386025 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@73195 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 73121 - Remove obsolete files from 'sources'. This causes needless ↵nsylvain@chromium.org2011-01-312-12/+16
| | | | | | | | | | | | | | | | | | work on incremental builds in VS2010. This is breaking clobber builds with this error : e:\b\build\slave\google-chrome-rel-win\build\src\chrome_frame\iids.cc(6) : fatal error C1083: Cannot open include file: 'chrome_tab.h': No such file or directory BUG=71130 TEST=Check that Chrome builds. Review URL: http://codereview.chromium.org/6307018 TBR=jeanluc@chromium.org Review URL: http://codereview.chromium.org/6347032 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@73124 0039d316-1c4b-4281-b951-d872f2087c98
* Remove obsolete files from 'sources'. This causes needless work on ↵jeanluc@chromium.org2011-01-302-16/+12
| | | | | | | | | | | incremental builds in VS2010. BUG=71130 TEST=Check that Chrome builds. Review URL: http://codereview.chromium.org/6307018 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@73121 0039d316-1c4b-4281-b951-d872f2087c98
* Fix miscellaneous compilation bugs under Visual Studio 2010.jeanluc@chromium.org2011-01-301-1/+1
| | | | | | | | | BUG=71145 TEST=Run unit tests Review URL: http://codereview.chromium.org/6240017 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@73119 0039d316-1c4b-4281-b951-d872f2087c98
* MASM 10.0 fixes a problem introduced in 8.0 with extra underscore added.jeanluc@chromium.org2011-01-281-1/+2
| | | | | | | | | BUG=71145 TEST=Compile & link cleanly Review URL: http://codereview.chromium.org/6352023 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@72981 0039d316-1c4b-4281-b951-d872f2087c98
* Reland 72784danno@chromium.org2011-01-283-2/+4
| | | | | | | | | | | This is the first step to being able to re-constitute policy from a non-generic protobuf in a policy request in a fully automated way (i.e. the code for the process can be generated). The non-generic protobuf is required for automatic merging of policy on the server-side. BUG=68309 TEST=existing policy/provider tests Review URL: http://codereview.chromium.org/6357021 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@72956 0039d316-1c4b-4281-b951-d872f2087c98
* Simplify HttpCache/HttpNetworkLayer/HttpNetworkSession interaction.willchan@chromium.org2011-01-282-14/+25
| | | | | | | | | | | | | Eliminate lazy initialization of HttpNetworkSession in HttpNetworkLayer. * This eliminates the need to update parameters for HttpNetworkLayer, it just takes a HttpNetworkSession. * It is OK to eliminate lazy initialization since these variables are cheap. BUG=none TEST=none Review URL: http://codereview.chromium.org/6402002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@72931 0039d316-1c4b-4281-b951-d872f2087c98
* Change UTF8ToUTF16 to accept const StringPiece&.suzhe@google.com2011-01-281-0/+2
| | | | | | | | | BUG=70936 TEST=All unit tests should pass. Review URL: http://codereview.chromium.org/6317016 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@72921 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 72784 - Policy: generate boilerplate policy type and constant code.danno@chromium.org2011-01-273-4/+2
| | | | | | | | | | | | | This is the first step to being able to re-constitute policy from a non-generic protobuf in a policy request in a fully automated way (i.e. the code for the process can be generated). The non-generic protobuf is required for automatic merging of policy on the server-side. BUG=68309 TEST=existing policy/provider tests Review URL: http://codereview.chromium.org/6002015 TBR=danno@chromium.org git-svn-id: svn://svn.chromium.org/chrome/trunk/src@72785 0039d316-1c4b-4281-b951-d872f2087c98
* Policy: generate boilerplate policy type and constant code.danno@chromium.org2011-01-273-2/+4
| | | | | | | | | | | This is the first step to being able to re-constitute policy from a non-generic protobuf in a policy request in a fully automated way (i.e. the code for the process can be generated). The non-generic protobuf is required for automatic merging of policy on the server-side. BUG=68309 TEST=existing policy/provider tests Review URL: http://codereview.chromium.org/6002015 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@72784 0039d316-1c4b-4281-b951-d872f2087c98
* Hopefully fix chrome_frame_net_tests.erg@google.com2011-01-271-0/+1
| | | | | | | | | | | | In r72682, I removed this line since I assumed it was including the test utils in the same file, not the unit tests. Put the unit tests themselves back. BUG=68682 TEST=compiles + chrome_frame_net_tests starts running again. Review URL: http://codereview.chromium.org/6350018 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@72734 0039d316-1c4b-4281-b951-d872f2087c98
* Add support for chunked encoding in ChromeFrame for POST requests. This ↵ananta@chromium.org2011-01-265-12/+23
| | | | | | | | | | | | | | | | | | | | fixes the URLRequestTestHTTP.TestPostChunkedDataBeforeStart net test failure in ChromeFrame. To support chunked encoding we need to marshal the corresponding information in the net::UploadData object to ensure that this object gets reconstructed correctly on the other side (CF). Disabled the URLRequestTestHTTP.TestPostChunkedDataAfterStart for ChromeFrame as this test modifies the UploadData object after it has been marshaled over to CF which we don't support in ChromeFrame. BUG=none TEST=Covered by existing net tests. Review URL: http://codereview.chromium.org/6357017 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@72723 0039d316-1c4b-4281-b951-d872f2087c98
* Clean up net unit testing code.erg@google.com2011-01-262-3/+1
| | | | | | | | | | | | | | | | | | | | (Reapply r72562 with willchan's nits + locally tested shlib fixes + removing duplicate code that was partially responsible for the failure.) - Move code included from blah_unittest.h (where blah_unittest.cc has actual unittests) into their own files, often completely out-of-lining the definitions. - Remove anonymous namespaces from headers. - Reorder method declarations. - Make other unit test link against net_test_support instead of reaching in and including .h files in their targets directly. BUG=68682 TEST=compiles First Review URL: http://codereview.chromium.org/6264013 Review URL: http://codereview.chromium.org/6248021 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@72682 0039d316-1c4b-4281-b951-d872f2087c98
* Revert "Clean up net unit testing code." since it breaks the shlib builder.erg@google.com2011-01-252-1/+3
| | | | | | | | | | This reverts commit 006f377bbb65ece3490b7c76e58e0dc4cb330909 (r72562). BUG=68682 TEST=compiles TBR=thomasvl git-svn-id: svn://svn.chromium.org/chrome/trunk/src@72564 0039d316-1c4b-4281-b951-d872f2087c98
* Clean up net unit testing code.erg@google.com2011-01-252-3/+1
| | | | | | | | | | | | | | | | | - Move code included from blah_unittest.h (where blah_unittest.cc has actual unittests) into their own files, often completely out-of-lining the definitions. - Remove anonymous namespaces from headers. - Reorder method declarations. - Make other unit test link against net_test_support instead of reaching in and including .h files in their targets directly. BUG=68682 TEST=compiles Review URL: http://codereview.chromium.org/6264013 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@72562 0039d316-1c4b-4281-b951-d872f2087c98
* More installer refactoring in the interest of fixing some bugs and cleaning ↵grt@chromium.org2011-01-251-4/+9
| | | | | | | | | | | | | | | | | | | | | | | | | things up: - Introduced ProductOperations: an interface implemented for each product that takes care of product-specific functions. Each Product owns an instance and delegates certain operations to it. - Removed the use of MasterPreferences by BrowserDistribution so that the former isn't needed outside of the installer. - Replaced PackageProperties with a new BrowserDistribution type (CHROME_BINARIES) - Plumbed the concept of InstallerState more thoroughly through installer - Removed ProductPackageMapping and Package - Moved more registry read ops into ProductState - Validation of products to be installed is now done in CheckPreInstallConditions - Ignore --chrome-frame --ready-mode if chrome is also being installed/updated and a SxS GCF is found (chrome is updated). - Migrates existing single-install Chrome to multi-install where appropriate. - Fixes update to Chrome's uninstallation arguments when Chrome Frame is uninstalled. - Removed dead code from install.cc. - Added code to update products' "ap" values when ready-mode is accepted. - Skip post-install things such as launching the browser when Chrome was implicitly added to the install/upgrade process by virtue of being part of a multi-install. BUG=61609 TEST=run the installer, see it work. existing tests in installer_util_unittests have been updated; new tests are included for ProductState, ChannelInfo, etc. Review URL: http://codereview.chromium.org/6288009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@72497 0039d316-1c4b-4281-b951-d872f2087c98
* Replace one unfortunate hack with a somewhat better one. The included ↵erikwright@chromium.org2011-01-251-10/+8
| | | | | | | | | | atlctrlx.h relies on min and max macros, which are generally bad and especially so in combination with STL. BUG=None TEST=None Review URL: http://codereview.chromium.org/6298014 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@72496 0039d316-1c4b-4281-b951-d872f2087c98