summaryrefslogtreecommitdiffstats
path: root/chrome_frame
Commit message (Collapse)AuthorAgeFilesLines
* ChromeFrameTestWithWebServer.WidgetModeIE_SrcProperty is Flaky.mad@chromium.org2011-09-131-1/+1
| | | | | | | | | BUG=96449 TBR=ananta@chromium.org Review URL: http://codereview.chromium.org/7886016 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@100990 0039d316-1c4b-4281-b951-d872f2087c98
* Prevent redirector from returning the current module in cases where it fails ↵robertshield@chromium.org2011-09-133-22/+26
| | | | | | | | | | to look up the first module in. This prevents a bug that can cause user-level Chrome Frame to redirect to itself when a system-level Chrome Frame is present. BUG=96016 TEST=Upgrade a user-level CF install to a system-level CF install while user-level CF is rendering. Observe that the current explorer instance does not hang when new CF tabs are created. Review URL: http://codereview.chromium.org/7866043 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@100904 0039d316-1c4b-4281-b951-d872f2087c98
* Modifying prefetch to account for multi-profile.rlp@chromium.org2011-09-101-4/+4
| | | | | | | | | | | | | | | | | | | | Items of note: - predictor_api is gone. Most functions in predictor_api.cc have moved into the chrome_browser_net::Predictor class or the Profile class. - The predictor state is cleaned up in the Profile dtor. - Predictor is owned by the ProfileIOData of the profile. - InitialObserver class is gone since each profile keeps their own info, the non-OTR don't care if anyone is OTR. - Predictor is created by the profile and then passed to the ProfileIOData. Then its initialization is finished on the IOThread. - ConnectInterceptor now subclasses off of UrlRequestJobFactory::Interceptor. - Updated Profile to create a SimpleShutdownPredictor with limited functionality when in unittests. BUG=89937,90114 TEST=passes existing Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=97446 Review URL: http://codereview.chromium.org/7467012 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@100555 0039d316-1c4b-4281-b951-d872f2087c98
* Clean up reliability_tests: they don't need their ownphajdan.jr@chromium.org2011-09-091-1/+1
| | | | | | | | | | | | test suite. This will make it easier to further reorganize chrome/test BUG=90905 Review URL: http://codereview.chromium.org/7859009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@100522 0039d316-1c4b-4281-b951-d872f2087c98
* Make Chrome Frame registration and unregistration more robust and make it ↵robertshield@chromium.org2011-09-081-290/+397
| | | | | | | | | | | | | | | | | | | | return extra information in the failure code. Specifically, there are a series of registration changes made by Chrome Frame at registration and unregistration time. Previously, if any of those changes failed, the previous changes were left and subsequent changes were not made. During registration this could result in partial registration data being left if one of the items failed which is bad but wasn't a problem in practice, because if ever registration failed, unregistration would then be performed by the installer. Also the most likely step to fail happens first. During unregistration, if any of the steps fail, the remaining unregistration steps will not be taken. This is bad, as uninstallation is a best effort, "clean up everything you can" type thing. Changed this to for registration roll back the previous N-1 steps of step N fails and for unregistration to attempt to perform all N steps disregarding failures. Updated installer logging. BUG=94848 TEST=Unregistration of e.g. the user-agent modification should always be performed even if an earlier step fails. Review URL: http://codereview.chromium.org/7824010 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@100224 0039d316-1c4b-4281-b951-d872f2087c98
* net: Rename URLRequestStatus::os_error_.tfarina@chromium.org2011-09-082-4/+4
| | | | | | | | | | | BUG=92264 TEST=None R=willchan@chromium.org Review URL: http://codereview.chromium.org/7846007 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@100136 0039d316-1c4b-4281-b951-d872f2087c98
* Use precompiled headers for most large projects where the .gyp filejoi@chromium.org2011-09-071-1/+4
| | | | | | | | | | | | | | is not a third party file. On my machine, this speeds up a full recompile of the 'chrome' target in Debug mode by about 18%. BUG=none TEST=it builds, existing tests pass Review URL: http://codereview.chromium.org/7706011 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@99949 0039d316-1c4b-4281-b951-d872f2087c98
* Rename REMOVE_COOKIES to REMOVE_SITE_DATA (which will explicitly include ↵mkwst@chromium.org2011-09-061-2/+2
| | | | | | | | | | | | | | | | | | | REMOVE_LSO_DATA). There's no case in the current codebase when we want to remove _only_ LSO data, and no case where we want to remove cookies, but _not_ LSO data. There are many instances of building removal masks that binary-OR REMOVE_LSO_DATA and REMOVE_COOKIES together; this CL formalizes that by dropping the extra enum value. Moreover, we talk about "cookies" differently when targeting users on the one hand, and developers on the other. REMOVE_COOKIES is a developer-facing name with a user-facing value, which is confusing. Renaming the current functionality to REMOVE_SITE_DATA is much more explicit, and will enable us to _just_ remove cookies at some point in the future via a REMOVE_COOKIES that actually means cookies. REMOVE_SITE_DATA has exactly the same semantics that REMOVE_COOKIES used to, with one exception: REMOVE_COOKIES included deleting cached strict transport security values. This functionality has been moved to REMOVE_CACHE. As a drive-by, this CL also moves a check against the kClearPluginLSODataEnabled pref out of webui (ClearBrowsingDataHandler) and into the BrowsingDataRemover itself, where it seems more likely to be effective. BUG=92628, 94334 TEST=Nothing but removal cached transport-security values should change. Review URL: http://codereview.chromium.org/7717023 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@99753 0039d316-1c4b-4281-b951-d872f2087c98
* Remove use of <iostream> where unnecessary.evan@chromium.org2011-09-011-1/+1
| | | | | | | | | | | | | | Any file which uses <iostream> gains a static initializer, which is both disallowed by our style guide and also harms startup performance. Note that Chrome's logging.h uses <ostream> (note missing "i"), which includes the << operators for logging but does *not* use cin/cout. This is fine. BUG=94794 Review URL: http://codereview.chromium.org/7811002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@99237 0039d316-1c4b-4281-b951-d872f2087c98
* Revamp the multi-profiles testing infrastructure.rsesek@chromium.org2011-08-311-3/+9
| | | | | | | | | | | | | | This creates TestingProfileManager to handle setting up the structures needed to test multi-profiles in unit_tests. It requires there to be a TestingBrowserProcess in place. This also adds a second constructor to TestingProfile for multi-profiles testing. BUG=none TEST=unit_tests Review URL: http://codereview.chromium.org/7748002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@98991 0039d316-1c4b-4281-b951-d872f2087c98
* Move the Chrome/Chromium icon from the locale dlls into npchrome_frame.dll.tony@chromium.org2011-08-302-9/+7
| | | | | | | | | | This avoids us having 50 copies of the same icon (once per locale dll). In practice, this wasn't so bad because they probably zipped well in the 7z file. Review URL: http://codereview.chromium.org/7790011 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@98828 0039d316-1c4b-4281-b951-d872f2087c98
* Stop using the default profile's proxy service for plugin proxy requests, ↵jam@chromium.org2011-08-291-4/+0
| | | | | | | | | and instead use the associated profile's proxy service. I proxy the IPC through the renderer first, as that makes it easy to get to the associated profile. BUG=92361,64339 Review URL: http://codereview.chromium.org/7791005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@98728 0039d316-1c4b-4281-b951-d872f2087c98
* Include the locale dlls (they now only contain dialogs used by chrometony@chromium.org2011-08-261-1/+2
| | | | | | | | | | | | frame) in the mini installer package. BUG=94362 TEST=In a chrome frame page, press ctrl+f. A find dialog should appear and it should be localized. Review URL: http://codereview.chromium.org/7761010 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@98524 0039d316-1c4b-4281-b951-d872f2087c98
* Only put the chrome frame resources in the locale dlls.tony@chromium.org2011-08-266-389/+207
| | | | | | | | | | | | | | | Chrome Frame needs 2 dialogs and an icon in the locale dlls. The rest of the strings and resources can be pulled from the locale pak files. This also removes the icon from the pak files and moves them into the dlls. BUG=94362 Review URL: http://codereview.chromium.org/7766007 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@98513 0039d316-1c4b-4281-b951-d872f2087c98
* Load both language packs and the resource dlls in ChromeFrame and look for ↵ananta@chromium.org2011-08-263-56/+99
| | | | | | | | | | | | | localized strings in the language pack and the other resources like dialogs in the dll. The latter part is done automatically by ATL. BUG=94362 TEST=No change in behavior. The resources should continue to load as before. Review URL: http://codereview.chromium.org/7763006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@98502 0039d316-1c4b-4281-b951-d872f2087c98
* Enabling the CF tests ReadyPromptContentTest and the ↵ananta@chromium.org2011-08-262-6/+2
| | | | | | | | | | | | TopLevelPostReissueFromChromeFramePage as the resource dll issues affecting these tests have now been fixed. BUG=94362 TBR=grt Review URL: http://codereview.chromium.org/7767003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@98477 0039d316-1c4b-4281-b951-d872f2087c98
* Generate locale dlls as a dependency of Chrome Frame.tony@chromium.org2011-08-262-0/+679
| | | | | | | | | | | | Chrome frame has dialogs in generated_resources.grd that are not included in the pak files. They should continue to use the locale dlls. BUG=94362 Review URL: http://codereview.chromium.org/7760002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@98449 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 98424 - Revert 98398 - Disabling the ReadyPromptContentTest and thebattre@chromium.org2011-08-262-2/+6
| | | | | | | | | | | | | | | | | | | TopLevelPostReissueFromChromeFramePage tests as these tests rely on resource dlls being present in the locales folder. After revision 98103 it looks like chrome uses resource packs internally. We need to fix ChromeFrame to not use resource dlls and use the packs instead. BUG=94362 Review URL: http://codereview.chromium.org/7740045 TBR=ananta@chromium.org Review URL: http://codereview.chromium.org/7745050 TBR=battre@chromium.org Review URL: http://codereview.chromium.org/7753037 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@98428 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 98426 - Revert 98103 - Switch to using .pak files for locale data on ↵battre@chromium.org2011-08-261-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Windows. We were using .dlls, but the .pak files are smaller and this will allow us to share more code across platforms. - Remove app/locales.gyp (used on win to generate the locale dlls) and references to it in other gyp(i) files. - Update various packaging scripts. - Move functions from resource_bundle_posix.cc to resource_bundle.cc (LoadResourcesDataPak, UnloadLocaleResources, GetLocalizedString, LoadLocaleResources) and delete the corresponding functions from resource_bundle_win.cc. Fixes from previous patch: - Made dependencies of packed_resources explicit (MSVS can't figure it out from the inputs). - Removed a couple cases where we were looking for dlls on Windows (in l10n_util.cc and l10n_util_unittest.cc). BUG=92724 TBR=cpu Review URL: http://codereview.chromium.org/7714028 TBR=tony@chromium.org Review URL: http://codereview.chromium.org/7744053 TBR=battre@chromium.org Review URL: http://codereview.chromium.org/7759001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@98427 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 98103 - Switch to using .pak files for locale data on Windows.battre@chromium.org2011-08-261-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We were using .dlls, but the .pak files are smaller and this will allow us to share more code across platforms. - Remove app/locales.gyp (used on win to generate the locale dlls) and references to it in other gyp(i) files. - Update various packaging scripts. - Move functions from resource_bundle_posix.cc to resource_bundle.cc (LoadResourcesDataPak, UnloadLocaleResources, GetLocalizedString, LoadLocaleResources) and delete the corresponding functions from resource_bundle_win.cc. Fixes from previous patch: - Made dependencies of packed_resources explicit (MSVS can't figure it out from the inputs). - Removed a couple cases where we were looking for dlls on Windows (in l10n_util.cc and l10n_util_unittest.cc). BUG=92724 TBR=cpu Review URL: http://codereview.chromium.org/7714028 TBR=tony@chromium.org Review URL: http://codereview.chromium.org/7744053 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@98426 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 98398 - Disabling the ReadyPromptContentTest and the ↵battre@chromium.org2011-08-262-6/+2
| | | | | | | | | | | | | | | | | TopLevelPostReissueFromChromeFramePage tests as these tests rely on resource dlls being present in the locales folder. After revision 98103 it looks like chrome uses resource packs internally. We need to fix ChromeFrame to not use resource dlls and use the packs instead. BUG=94362 Review URL: http://codereview.chromium.org/7740045 TBR=ananta@chromium.org Review URL: http://codereview.chromium.org/7745050 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@98424 0039d316-1c4b-4281-b951-d872f2087c98
* Disabling the ReadyPromptContentTest and the ↵ananta@chromium.org2011-08-262-2/+6
| | | | | | | | | | | | | | | TopLevelPostReissueFromChromeFramePage tests as these tests rely on resource dlls being present in the locales folder. After revision 98103 it looks like chrome uses resource packs internally. We need to fix ChromeFrame to not use resource dlls and use the packs instead. BUG=94362 TBR=grt Review URL: http://codereview.chromium.org/7740045 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@98398 0039d316-1c4b-4281-b951-d872f2087c98
* Switch to using .pak files for locale data on Windows.tony@chromium.org2011-08-241-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | We were using .dlls, but the .pak files are smaller and this will allow us to share more code across platforms. - Remove app/locales.gyp (used on win to generate the locale dlls) and references to it in other gyp(i) files. - Update various packaging scripts. - Move functions from resource_bundle_posix.cc to resource_bundle.cc (LoadResourcesDataPak, UnloadLocaleResources, GetLocalizedString, LoadLocaleResources) and delete the corresponding functions from resource_bundle_win.cc. Fixes from previous patch: - Made dependencies of packed_resources explicit (MSVS can't figure it out from the inputs). - Removed a couple cases where we were looking for dlls on Windows (in l10n_util.cc and l10n_util_unittest.cc). BUG=92724 TBR=cpu Review URL: http://codereview.chromium.org/7714028 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@98103 0039d316-1c4b-4281-b951-d872f2087c98
* Tommi: I need an owner review for the chrome frame changes.rogerta@chromium.org2011-08-244-62/+15
| | | | | | | | | | | | | | Moving the test helper class TempRegKeyOverride since I need to do something similar in some new RLZ tests, and I don't want to duplicate the code. Please suggest a better namespace name if needed, I was just following the same pattern found in test_file_util in the same directory. BUG=None TEST=No new tests, just refactoring test helpers Review URL: http://codereview.chromium.org/7669061 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@98100 0039d316-1c4b-4281-b951-d872f2087c98
* Revert "Switch to using .pak files for locale data on Windows." (r97941)mattm@chromium.org2011-08-241-1/+1
| | | | | | | | | | | Revert "Fix a bug where windows can't find the locale paks." (r97951) Revert "Fix the windows build." (r97954) TBR=tony@chromium.org Review URL: http://codereview.chromium.org/7717017 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@97968 0039d316-1c4b-4281-b951-d872f2087c98
* Switch to using .pak files for locale data on Windows.tony@chromium.org2011-08-231-1/+1
| | | | | | | | | | | | | | | We were using .dlls, but the .pak files are smaller and this will allow us to share more code across platforms. - Remove app/locales.gyp (used on win to generate the locale dlls) and references to it in other gyp(i) files. - Update various packaging scripts. - Move functions from resource_bundle_posix.cc to resource_bundle.cc (LoadResourcesDataPak, UnloadLocaleResources, GetLocalizedString, LoadLocaleResources) and delete the corresponding functions from resource_bundle_win.cc. BUG=92724 Review URL: http://codereview.chromium.org/7677004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@97941 0039d316-1c4b-4281-b951-d872f2087c98
* Fix a back forward bug in ChromeFrame reported on the field. The bug occurs ↵ananta@chromium.org2011-08-174-91/+8
| | | | | | | | | | | | | | | | | | | | | | when multiple navigations are performed within IE + CF with some originating from IE and others within Chrome. The CF active document implementation had some code to reuse the active document instance in case a new CF navigation occurred within the current tab. This was for optimization, i.e. to ensure that we don't tear down and create a new instance of Chrome. This basically messed up the document destruction order causing the IE tab history to get populated with information coming from deleted active document instances at times. In any case this optimization is no longer necessary as we have a proxy cache maintained in the automation client in CF which ensures that we don't inadvarently tear down and create new chrome instances. Fixes bug http://code.google.com/p/chromium/issues/detail?id=90424 BUG=90424 Review URL: http://codereview.chromium.org/7627005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@97177 0039d316-1c4b-4281-b951-d872f2087c98
* Remove Purify and Quantify.Doing so will reduce substantially the size of ↵jeanluc@chromium.org2011-08-171-2/+0
| | | | | | | | | the project files. BUG=92102 Review URL: http://codereview.chromium.org/7528010 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@97156 0039d316-1c4b-4281-b951-d872f2087c98
* base: Fix the TODO in string_util.htfarina@chromium.org2011-08-171-0/+1
| | | | | | | | | | | | | This removes the include of stringprintf.h from string_util.h and fix the clients. BUG=None TEST=None R=evan@chromium.org Review URL: http://codereview.chromium.org/7633055 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@97067 0039d316-1c4b-4281-b951-d872f2087c98
* Ensure that webkit ASSERTION's don't fire during CRT cleanup in debug runs ↵ananta@chromium.org2011-08-161-0/+6
| | | | | | | | | | | | | | of the chrome frame net tests suite by terminating the exe using ExitProcess in debug builds. These cause the exe to crash during exit, thus turning the windows try bot runs red. BUG=none TEST=chrome frame net tests should run to completion without crashing in debug runs. Review URL: http://codereview.chromium.org/7663015 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@97041 0039d316-1c4b-4281-b951-d872f2087c98
* Tag IPC::Channel::Listener implementations with OVERRIDEevan@chromium.org2011-08-161-4/+4
| | | | | | Review URL: http://codereview.chromium.org/7661031 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@97021 0039d316-1c4b-4281-b951-d872f2087c98
* Fix obsolete dependencies so that needless recompilation does not happen.jeanluc@chromium.org2011-08-041-2/+2
| | | | | | | BUG=71130 Review URL: http://codereview.chromium.org/7545020 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@95527 0039d316-1c4b-4281-b951-d872f2087c98
* Move more files from chrome/test to chrome/test/base, part #4phajdan.jr@chromium.org2011-08-012-5/+5
| | | | | | | BUG=90905 Review URL: http://codereview.chromium.org/7544021 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@94963 0039d316-1c4b-4281-b951-d872f2087c98
* Include-what-you-use - web_database.h does not include headers for every ↵groby@chromium.org2011-07-291-0/+1
| | | | | | | | | | | | single table any more BUG=none TEST=none Review URL: http://codereview.chromium.org/7523012 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@94702 0039d316-1c4b-4281-b951-d872f2087c98
* Mark FullTabUITest.RendererHasFocus as FLAKY.sergeyu@chromium.org2011-07-281-1/+2
| | | | | | | | | | BUG=90791 TEST=None TBR=kkania@chromium.org Review URL: http://codereview.chromium.org/7522013 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@94407 0039d316-1c4b-4281-b951-d872f2087c98
* Remove explicit keyword from multi-argument (w/o default values) constructorsdilmah@chromium.org2011-07-263-5/+4
| | | | | | | | | BUG=None TEST=None Review URL: http://codereview.chromium.org/7477008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@94115 0039d316-1c4b-4281-b951-d872f2087c98
* Update the url_request_http_job according to the asynchronous CookieMonster API.ycxiao@chromium.org2011-07-251-0/+1
| | | | | | | | | BUG=68657 TEST=XXX Review URL: http://codereview.chromium.org/7210046 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@93966 0039d316-1c4b-4281-b951-d872f2087c98
* Include the UI resources in the Chrome Frame net tests since icons are now ↵robertshield@chromium.org2011-07-221-0/+2
| | | | | | | | | | | required in order to navigate tabs. BUG=chrome_frame_net_tests.exe stop DCHECKing on the try servers TEST=NONE Review URL: http://codereview.chromium.org/7482002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@93573 0039d316-1c4b-4281-b951-d872f2087c98
* Rename CommandLine::GetCommandLineString().msw@chromium.org2011-07-204-8/+8
| | | | | | | | | | | | Fix string hackery in net/tools/dump_cache/dump_cache.cc Fix const casts in chrome/installer/util/product.cc and base/process_util_win.cc. BUG=73195 TEST=none Review URL: http://codereview.chromium.org/7386002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@93165 0039d316-1c4b-4281-b951-d872f2087c98
* Generating separate version resources for:bradnelson@google.com2011-07-191-2/+2
| | | | | | | | | | | | | | chrome.dll chrome.exe nacl64.dll nacl64.exe BUG=http://code.google.com/p/chromium/issues/detail?id=65350 TEST=None R=cpu@chromium.org Review URL: http://codereview.chromium.org/7390037 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@93124 0039d316-1c4b-4281-b951-d872f2087c98
* Deprecate RegisterProtocolFactory/(Un)RegisterInterceptor.willchan@chromium.org2011-07-191-5/+5
| | | | | | | | | | | | Mark those APIs as private and friend existing uses. BUG=none TEST=none Review URL: http://codereview.chromium.org/7397008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@92992 0039d316-1c4b-4281-b951-d872f2087c98
* Convert the remaining callers of LaunchApp to LaunchProcess.evan@chromium.org2011-07-185-7/+10
| | | | | | | | | | And delete temporary shims. BUG=88990 Review URL: http://codereview.chromium.org/7386010 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@92897 0039d316-1c4b-4281-b951-d872f2087c98
* Change base::LaunchProcess API slightlyevan@chromium.org2011-07-152-14/+6
| | | | | | | | | | | Rather than passing the out param process handle via the options, take it as a function argument. This simplifies many callers. BUG=88990 Review URL: http://codereview.chromium.org/7377012 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@92701 0039d316-1c4b-4281-b951-d872f2087c98
* Clean up users of a deprecated base::LaunchApp API.evan@chromium.org2011-07-142-9/+20
| | | | | | | | BUG=88990 Review URL: http://codereview.chromium.org/7351003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@92598 0039d316-1c4b-4281-b951-d872f2087c98
* Convert RenderViewContextMenu to MenuItemView.rhashimoto@chromium.org2011-07-129-68/+76
| | | | | | | | | | | | | | This CL is part of general GTK removal for ChromiumOS. Menu2 uses GTK on linux so we are replacing it with MenuItemView. Chrome Frame currently passes the context menu between processes by using the HMENU. Because MenuItemView does not use HMENU, we need to use another mechanism. This CL creates a ContextMenuModel struct that is serialized into an automation message for Chrome Frame. ContextMenuModel contains the context menu definition in-band replacing the out-of-band HMENU. BUG=chromium-os:13887 TEST=none Review URL: http://codereview.chromium.org/7167002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@92182 0039d316-1c4b-4281-b951-d872f2087c98
* Make version info get stored in the correct resource for ↵robertshield@chromium.org2011-07-112-4/+2
| | | | | | | | | | | | chrome_frame_helper.exe and chrome_frame_helper.dll. Switch to using '1' as the version resource name since that is less fragile against #include path mix ups, etc. BUG=65354 TEST=NONE Review URL: http://codereview.chromium.org/7324054 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@92053 0039d316-1c4b-4281-b951-d872f2087c98
* Remove the comments setting emands and vim tab width and expansion variables.tony@chromium.org2011-07-113-22/+0
| | | | | | | | | | | | | | | | | These were added in r28089 (http://codereview.chromium.org/256059) but are unnecessary bloat for everyone to carry around, even those that don't use emacs or vim. In an earlier change, I added editor config files in src/tools/emacs/ and src/tools/vim/ so users of the appropriate editor can source those instead. BUG=none TEST=none Review URL: http://codereview.chromium.org/7310019 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@92046 0039d316-1c4b-4281-b951-d872f2087c98
* Remove a redundant call to chrome::RegisterPathProvider which causes a ↵robertshield@chromium.org2011-07-111-1/+3
| | | | | | | | | | | DCHECK() while running the Chrome Frame net tests in debug mode. BUG=NONE TEST=Chrome Frame net tests do not dcheck. TBR=ananta Review URL: http://codereview.chromium.org/7335008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@92019 0039d316-1c4b-4281-b951-d872f2087c98
* Fix ananta's listing in chrome_frame/OWNERS.msw@chromium.org2011-07-071-1/+1
| | | | | | | | | BUG=none TEST=none Review URL: http://codereview.chromium.org/7329009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@91782 0039d316-1c4b-4281-b951-d872f2087c98
* Add chrome_frame/OWNERS.thakis@chromium.org2011-07-061-0/+4
| | | | | | | | | BUG=88315 TEST=none Review URL: http://codereview.chromium.org/7304016 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@91525 0039d316-1c4b-4281-b951-d872f2087c98