summaryrefslogtreecommitdiffstats
path: root/chrome_frame
Commit message (Collapse)AuthorAgeFilesLines
* src/: Update the remaining include paths of string_split.h to its new location.tfarina@chromium.org2013-03-061-1/+1
| | | | | | | | | | BUG=175186 TBR=brettw@chromium.org Review URL: https://chromiumcodereview.appspot.com/12473004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@186462 0039d316-1c4b-4281-b951-d872f2087c98
* Fix a crash seen in ChromeFrame when opening a non CF top level tab from a ↵ananta@chromium.org2013-03-041-3/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | CF page. This was a regression caused by the fix for bug http://code.google.com/p/chromium/issues/detail?id=168308 which was to not invalidate the protocol sink mapping for CF pages which are switched in from IE. The crash in this case occurs because the protocol sink mapping is removed in the call to IInternetProtocol::Terminate as this is treated as a ChromeFrame request. This is only a temporary CF url which eventually transitions to the actual URL which is opened in IE. For the curious we intercept IInternetProtocol::LockRequest and for the special gcf://attach_external_tab requests we addref the protocol data and return without calling the original LockRequest API. When UnlockRequest is invoked we rely on the protocol data mapping to exist to release the protocol data and return without calling the original UnlockRequest API. In this case the sequence is IInternetProtocol::LockRequest, IInternetProtocolRoot::Terminate followed by IInternetProtocol::UnlockRequest. In our terminate handler we remove the protocol data mapping for the attach tab request. When UnlockRequest is called we call the original API and end up crashing. Fix is to not invalidate the protocol data mapping for attach tab requests. A test for this is in the works by robertshield. BUG=178415 R=robertshield Review URL: https://chromiumcodereview.appspot.com/12395021 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@185956 0039d316-1c4b-4281-b951-d872f2087c98
* Minimal Chrome Frame with Aura.grt@chromium.org2013-02-281-2/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It builds and runs, but not a whole lot else. This change includes: * ChromeFrameAutomationProvider is now OS_WIN only. In practice, this has been the case for some time. Now it's formalized by giving the implementation files the _win suffix. * Automation messages and datatypes used exclusively by Chrome Frame now use HWND directly rather than a toolkit-specific gfx typedef of one since the requirement is that an actual HWND be sent over the channel. A change in toolkit (e.g., switching to Aura) must not change this. As a consequence of this change, some automation types and messages are now only defined for OS_WIN builds. * ExternalTabContainerWin is no longer derived from a NativeWidget type (this was previously the case so that the ETCW could be notified of NW lifecycle events). Now, in contrast, ETCW registers itself as an observer of its Widget. Two additional lifecycle methods have been added to WidgetObserver: OnWidgetCreated and OnWidgetDestroyed. * ExternalTabContianerWin initializes its Widget with an instance of DesktopNativeWidgetAura when use_aura. * A special note about HWND IPC marshaling: this change adds a type mapping from HWND to a generic HANDLE in ipc_message_utils.h, which allows for the removal of a hack in content_message_generator.h to marshal HWNDs. This change reverts all of: * r178752 -- Remove CF from all.gyp targets if use_aura is defined. * r164590 -- Remove setup -> Chrome Frame dependency. Make it possible to build an installer for Aura. and portions of: * r99993 -- Get chrome to link with USE_AURA * r99787 -- Preliminary work to allow Chrome to build with USE_AURA. BUG=171018 TEST=chrome_frame_tests provides good coverage in non-Aura builds. Review URL: https://chromiumcodereview.appspot.com/12220101 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@185328 0039d316-1c4b-4281-b951-d872f2087c98
* Add Chrome Frame test for large page rendering. This is designed to ensure ↵robertshield@chromium.org2013-02-283-1/+10040
| | | | | | | | | | | | | | that large pages load correctly in their entirety which should help catch bugs like crbug.com/178421. Also fix access violation in code used by tests to shut down IE. BUG=178421 TEST=chrome_frame_tests.exe Review URL: https://chromiumcodereview.appspot.com/12310162 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@185260 0039d316-1c4b-4281-b951-d872f2087c98
* Cleanup: Remove more base/file_paths.h references.thestig@chromium.org2013-02-271-2/+2
| | | | | | | | | R=brettw TBR=grt Review URL: https://chromiumcodereview.appspot.com/12340092 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@184849 0039d316-1c4b-4281-b951-d872f2087c98
* ChromeFrame sites in IE would appear truncated at times.ananta@chromium.org2013-02-261-1/+3
| | | | | | | | | | | | | | | | This was a regression caused by the change to invalidate the cached protocol data when IE (urlmon) terminates the protocol. While this works well for non CF requests, for CF requests when we report the changed mime type to be that of chrome, the protocol is terminated with cached data. When Chrome attempts to read it we don't have anything to report which leads to the problem. Fix is to not invalidate the protocol data for Chrome requests in our Terminate hook. We should invalidate it for Abort because that seems like an error condition. BUG=178421 R=robertshield. Review URL: https://chromiumcodereview.appspot.com/12310145 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@184718 0039d316-1c4b-4281-b951-d872f2087c98
* Make Ash unittests clean up zombie viewer processes between test runs. These ↵robertshield@chromium.org2013-02-255-160/+6
| | | | | | | | | | | | | | zombie viewer processes seem to be a side effect of attempting to quit a metro app while the same app is activating. *while here can mean within a certain number of seconds due to metro life cycle management. Refactor KillAllNamedProcessesWithArgument into base/test. BUG=154081 TEST=Run "ash_unittests.exe --ash-metro-tests" on Win8. Observe that 417 chrome.exe processes do not get created :) Review URL: https://chromiumcodereview.appspot.com/12314043 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@184413 0039d316-1c4b-4281-b951-d872f2087c98
* base: Move MemoryMappedFile out of file_util.h and into its own header file.tfarina@chromium.org2013-02-245-4/+10
| | | | | | | | | | | BUG=175002 TEST=base_unittests R=brettw@chromium.org TBR=erikwright@chromium.org,tommi@chromium.org,fischman@chromium.org,ben@chromium.org Review URL: https://chromiumcodereview.appspot.com/12321062 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@184356 0039d316-1c4b-4281-b951-d872f2087c98
* Move file_path.h to base/files.brettw@chromium.org2013-02-2420-27/+28
| | | | | | TBR=sky git-svn-id: svn://svn.chromium.org/chrome/trunk/src@184344 0039d316-1c4b-4281-b951-d872f2087c98
* Re-enable some network tests that have been disabled formmenke@chromium.org2013-02-201-0/+6
| | | | | | | | | | | | over a year due to failures with Chrome Frame. Keep them disabled for Chrome Frame tests. Also disable two of them on Android. BUG=102991 Review URL: https://codereview.chromium.org/12042058 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@183367 0039d316-1c4b-4281-b951-d872f2087c98
* Move remaining non-test, non-Chrome-specific Prefs code to base/prefs/.joi@chromium.org2013-02-112-5/+5
| | | | | | | | | | | | | | This is a mostly-automated change, using ./tools/git/move_source_file.py to move files and fix up include guards and #includes from other files. Manual edits were limited to fixing up .gypi files and adding BASE_PREFS_EXPORT where needed. TBR=ben@chromium.org BUG=155525 Review URL: https://codereview.chromium.org/12211105 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@181707 0039d316-1c4b-4281-b951-d872f2087c98
* Rename FilePath -> base::FilePath in various toplevel directoriesbrettw@chromium.org2013-02-1045-200/+210
| | | | | | Review URL: https://codereview.chromium.org/12211108 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@181639 0039d316-1c4b-4281-b951-d872f2087c98
* Remove PrefServiceSimple, replacing it with PrefService and PrefRegistrySimple.joi@chromium.org2013-02-062-10/+19
| | | | | | | | | | | | This is a step towards removing registration methods from PrefService and simplifying its class hierarchy. TBR=ben@chromium.org BUG=155525 Review URL: https://chromiumcodereview.appspot.com/11741003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@180873 0039d316-1c4b-4281-b951-d872f2087c98
* Add FilePath to base namespace.brettw@chromium.org2013-02-024-12/+23
| | | | | | | This updates headers that forward-declare it and a few random places to use the namespace explicitly. There us a using declaration in file_path.h that makes the rest compile, which we can do in future passes. Review URL: https://codereview.chromium.org/12163003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@180245 0039d316-1c4b-4281-b951-d872f2087c98
* Move string tokenizer to base/strings.brettw@chromium.org2013-02-023-6/+6
| | | | | | | | BUG= Review URL: https://codereview.chromium.org/12087091 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@180211 0039d316-1c4b-4281-b951-d872f2087c98
* [sync] Componentize sync: Part Final: Target 'sync' is now its own componentrsimha@chromium.org2013-02-011-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | One of the long term goals of the sync team has been to pull sync out of chrome_dll and into its own component. This should result in faster link times for component builds, and cleaner demarcation between sync code and the rest of chrome. This patch does the following: - Splits off sync.gyp into gypi files for sync_core, sync_api, sync_internal_api, sync_notifier and sync_proto. - Audits the dependencies of various targets in sync.gyp, sync_tests.gyp, and other chrome gyp files, and makes sure all dependencies are explicitly declared. - Makes targets declared in gyp files outside sync.gyp directly depend on sync.gyp:sync instead of inner sync targets. - Implements two versions of the target 'sync.gyp:sync': 1) In static mode, the public 'sync' target has a target type of 'none', and is composed of the static library targets 'sync_api', 'sync_core', 'sync_internal_api', 'sync_notifier', and 'sync_proto'. 2) In component mode, we build the public 'sync' target into a single shared library, which includes the contents of sync_api.gypi, sync_core.gypi, sync_internal_api.gypi, sync_notifier.gypi, and sync_proto.gypi. TBR=akalin,robertshield,thakis BUG=136928 TEST=Set GYP_DEFINES="component=shared_library" and build the 'all' target on all platforms. Review URL: https://codereview.chromium.org/11412211 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@180034 0039d316-1c4b-4281-b951-d872f2087c98
* Ensure that ChromeFrame does not interfere with non ChromeFrame requests ↵ananta@chromium.org2013-01-312-3/+48
| | | | | | | | | | | | | | | | | | | | | initiated by IE. ChromeFrame patches the urlmon transaction object vtable to ensure that top level requests initiated by IE are intercepted. Methods patched include the Start/StartEx and Read. The Read method is patched to ensure that we return unprocessed data back to urlmon in case the site is not switched into ChromeFrame. There is a race condition between the time the data is discarded and a new request is created as the IInternetProtocol interface ptr can be reused. This causes us to return stale data from a previous request to IE. The data is keyed off the IInternetProtocol interface ptr. Fix is to patch the Abort and Terminate methods of the transaction vtable and destroy the data for the protocol. Fixes bug http://code.google.com/p/chromium/issues/detail?id=168308 BUG=168308 Review URL: https://chromiumcodereview.appspot.com/12093077 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@179932 0039d316-1c4b-4281-b951-d872f2087c98
* Rename chrome_frame coverage_bundles.py target.chrisha@chromium.org2013-01-291-9/+9
| | | | | | | | | | | chrome_frame and chrome use the same mechanism for generating a list of coverage targets. The chrome_frame targets have been prefixed with gcf_ to make them unique, but this was not done for the coverage_bundles.py target. This causes ninja to complain loudly that a target has multiple definitions. BUG= Review URL: https://chromiumcodereview.appspot.com/12089048 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@179422 0039d316-1c4b-4281-b951-d872f2087c98
* LoadTiming in net, part 3.mmenke@chromium.org2013-01-211-0/+5
| | | | | | | | | | | | | | | Add send and receive times to HttpNetworkTransaction. Add LoadTimingInfo accessors to Http*Transaction, URLRequestJob, and URLRequest classes. Proxy resolution times still need to be added. BUG=77446 Review URL: https://chromiumcodereview.appspot.com/11929017 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@177942 0039d316-1c4b-4281-b951-d872f2087c98
* [Autofill] Access the application locale in a thread-safe manner when ↵isherman@chromium.org2013-01-121-1/+1
| | | | | | | | | | | | migrating WebDB data. BUG=100845 TBR=robertshield@chromium.org Review URL: https://chromiumcodereview.appspot.com/11821044 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@176539 0039d316-1c4b-4281-b951-d872f2087c98
* [Profiles, Fixit] Removing usage of GetDefaultProfile from chrome_frame code.rlp@chromium.org2013-01-041-1/+1
| | | | | | | | | | | I will do a second pass when I fix GetDefaultProfileDir. BUG=83792 Review URL: https://chromiumcodereview.appspot.com/11571089 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@175078 0039d316-1c4b-4281-b951-d872f2087c98
* Split PrefService into PrefService, PrefServiceSimple and PrefServiceSyncable.joi@chromium.org2012-12-221-4/+6
| | | | | | | | | | | | | | | | | | | | | The first two (PrefServiceSimple is a subclass of PrefService) know nothing about sync or any Chrome or content concepts. The third (PrefServiceSyncable, a separate subclass of PrefService) knows about sync and requires users to choose whether each individual preference is syncable or not when it is registered. BrowserProcess::local_state() is a PrefServiceSimple after this change, and Profile::prefs() is a PrefServiceSyncable. COLLABORATOR=kaiwang@chromium.org TBR=ben@chromium.org BUG=155525 Review URL: https://chromiumcodereview.appspot.com/11570009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@174531 0039d316-1c4b-4281-b951-d872f2087c98
* GTTF: No more FLAKY_ .phajdan.jr@chromium.org2012-12-213-21/+22
| | | | | | | | | | | | | | | | | | | | | | | Please note that FLAKY_ tests have been ignored anyway. When tests started crashing, people just flipped that to DISABLED_ . Why not go straight to DISABLED_ then, so that we avoid wasting time on stupid test prefix games? With DISABLED_ it is clear to everyone that there is no coverage from given test. FLAKY_ creates an illusion of coverage, while in fact the test is still ignored. If a FLAKY_ test fails and nobody notices, does it still make a sound? ;-) Finally, note that gtest has a --gtest_also_run_disabled_tests if you need to run tests manually. TBR=jam BUG=none Review URL: https://codereview.chromium.org/11664013 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@174472 0039d316-1c4b-4281-b951-d872f2087c98
* net: Split sync part of UploadFileElementReader as UploadFileElementReaderSynchashimoto@chromium.org2012-12-201-4/+6
| | | | | | | | | | | | | | | Add UploadFileElementReaderSync and use it from Chrome Frame. Remove UploadDataStream::InitSync/ReadSync Remove UploadElementReader::InitSync/ReadSync Allow passing null callbacks for Init/Read when callers can be confident that the operation finishes synchronously. BUG=None TEST=net_unittests, chrome_frame_net_tests TBR=ananta@chromium.org for chrome_frame/urlmon_upload_data_stream.cc Review URL: https://chromiumcodereview.appspot.com/11587007 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@174133 0039d316-1c4b-4281-b951-d872f2087c98
* Extend scoped_ptr to be closer to unique_ptr. Support custom deleters, and ↵ajwong@chromium.org2012-12-201-1/+1
| | | | | | | | | | | | | | | | | | | | | | | deleting arrays. This is based on the modifications by "Geoffrey Romer" <gromer@google.com> to Google's internal version of scoped_ptr<>. All cleaver tricks are his, not mine. :) It brings most of the features of C++11's unique_ptr<> into this class and should eliminate the need for scoped_array<>, scoped_malloc_free<>, and possibly a few other scopers. Please refer to the unique_ptr<> API for documentation. Divergence from unique_ptr<>: 1) DefaultDeleter<T[n]> (sized-arrays) is explicitly disabled because this construct would cause the single-object delete to be called on a pointer to a T[n]. It is impossible to construct a single-object new expression that returns a T[n]*. You can only get this with new[] which should correspond to DefaultDeleter<T[n][]>. This issue has been raised with the C++ LWG as it is possible a unique_ptr<> spec defect. 2) Reference types for Deleters are not supported. This simplifies implementation significantly because there is no need to distinguish between the converting constructor + converting assignment operator and their move constructor + move assignment operator. 4) Move-only Deleters are not supported. Avoids the need to emulate std::forward(). BUG=109874 TBR=dhollowa,kinuko,ananta,hans,scherkus Review URL: https://codereview.chromium.org/11149006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@174057 0039d316-1c4b-4281-b951-d872f2087c98
* Introduce PrefServiceBuilder, use for ChromePrefServiceBuilder.joi@chromium.org2012-12-191-1/+5
| | | | | | | | | | | | | | | | This change extracts a non-test PrefServiceBuilder from PrefServiceMockBuilder, and extracts the Chrome-specific logic previously in pref_service.cc that was used to put together a PrefService for Chrome into a ChromePrefServiceBuilder subclass thereof. TBR=ben@chromium.org BUG=155525 Review URL: https://chromiumcodereview.appspot.com/11570007 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@173988 0039d316-1c4b-4281-b951-d872f2087c98
* Disable CF web server tests temporarily on IE9 while we hunt down the cause ↵robertshield@chromium.org2012-12-141-0/+6
| | | | | | | | | | | | of the first-request failures. BUG=143699 TEST=chrome_frame_tests Review URL: https://chromiumcodereview.appspot.com/11316280 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@173124 0039d316-1c4b-4281-b951-d872f2087c98
* Disable ContextMenuTest.CFTxtFieldSelectAlldgrogan@chromium.org2012-12-121-1/+2
| | | | | | | | | BUG=144664 TBR=robertshield@chromium.org Review URL: https://codereview.chromium.org/11547030 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@172679 0039d316-1c4b-4281-b951-d872f2087c98
* net: Make UploadDataStream independent from UploadDatahashimoto@chromium.org2012-12-111-3/+54
| | | | | | | | | | | | | | Change argument of UploadDataStream's ctor from UploadData to ScopedVector<UploadElementReader> Move chunk related logic from UploadData to UploadDataStream. Move UploadElementReader creation code from UploadElementReader::Create to URLRequest::set_upload. (this code will soon be moved again to src/webkit/glue/resource_request_body.cc) BUG=156574 TEST=net_unittests TBR=jam@chromium.org for chrome/browser/automation (already reviewed by ananta@) Review URL: https://chromiumcodereview.appspot.com/11419220 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@172295 0039d316-1c4b-4281-b951-d872f2087c98
* Fix typo so that ChromeFrameTestWithWebServer.FullTabModeIE_SubFrame is ↵dmazzoni@chromium.org2012-12-041-1/+1
| | | | | | | | | | | actually disabled. BUG=163728 TBR=robertshield Review URL: https://codereview.chromium.org/11411359 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@170979 0039d316-1c4b-4281-b951-d872f2087c98
* build fix attemptthakis@chromium.org2012-12-031-1/+1
| | | | | | | | BUG=none Review URL: https://codereview.chromium.org/11308333 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@170818 0039d316-1c4b-4281-b951-d872f2087c98
* Move many ProcessSingleton methods to "protected" visibility as an upcoming ↵gab@chromium.org2012-12-031-0/+2
| | | | | | | | | | | | refactoring of ProcessSingleton on Windows requires that only NotifyOtherProcessOrCreate() be callable from the public interface. This doesn't change any code logic, just moves things around. BUG=None Review URL: https://codereview.chromium.org/11415237 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@170805 0039d316-1c4b-4281-b951-d872f2087c98
* Disable ChromeFrame tests that flakily time out.isherman@chromium.org2012-12-011-5/+8
| | | | | | | | | BUG=163728 TBR=robertshield@chromium.org Review URL: https://codereview.chromium.org/11411303 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@170616 0039d316-1c4b-4281-b951-d872f2087c98
* GTTF: Make chrome_frame_tests faster.phajdan.jr@chromium.org2012-11-281-2/+2
| | | | | | | BUG=none Review URL: https://codereview.chromium.org/11316120 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@170089 0039d316-1c4b-4281-b951-d872f2087c98
* POSTs to HSTS domains are no longer converted to GETs mmenke@chromium.org2012-11-281-1/+2
| | | | | | | | | | | | | | | on upgrade to https. URLRequestRedirectJobs now take a status code on construction, to reduce the change of regressions, and the status code can no longer be changed after construction. TBR=jhawkins@chromium.org BUG=159343 Review URL: https://chromiumcodereview.appspot.com/11420013 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@170029 0039d316-1c4b-4281-b951-d872f2087c98
* chrome_frame: Update the remaining calls from RunAllPending() to RunUntilIdle().tfarina@chromium.org2012-11-263-4/+4
| | | | | | | | | | | | RunAllPending() is deprecated and we should switch to RunUntilIdle(). BUG=131220 TBR=erikwright@chromium.org Review URL: https://chromiumcodereview.appspot.com/11299168 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@169401 0039d316-1c4b-4281-b951-d872f2087c98
* Make all pdb file names follow the same naming convention.iannucci@chromium.org2012-11-171-1/+0
| | | | | | | | | | | | | | | | | | | The new naming scheme simply appends .pdb to the target (e.g. foo.dll.pdb and foo.exe.pdb). This is beneficial for a couple reasons: * no more chrome_dll.pdb and chrome_exe.pdb silliness * also avoids this issue if we end up with similarly named targets in the future (target.dll and target.exe) * ninja already does this, and, by extension, either ninja needs to be broken to use the old scheme, or the we fix the broken in the existing system. One less difference between msvs and ninja. Fixes for chrome-internal stuff: https://chromereviews.googleplex.com/5681015 BUG= Review URL: https://codereview.chromium.org/11384003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@168341 0039d316-1c4b-4281-b951-d872f2087c98
* Move scoped_temp_dir from base to base/filesbrettw@chromium.org2012-11-164-9/+10
| | | | | | | | | | Also add to base namespace. BUG= Review URL: https://codereview.chromium.org/11359217 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@168281 0039d316-1c4b-4281-b951-d872f2087c98
* net: Change type of UploadData::elements from std::vector to ScopedVectorhashimoto@chromium.org2012-11-141-2/+2
| | | | | | | | | | | | | | Using std::vector to hold UploadElement is bad for two reasons: 1. It results in a lot of unnecessary copy of uploaded data. 2. Appending new chunks may result in invalidating the pointer held by UploadBytesElementReader. BUG=160028 TEST=git try TBR=ananta@chromium.org, tony@chromium.org Review URL: https://chromiumcodereview.appspot.com/11275223 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@167611 0039d316-1c4b-4281-b951-d872f2087c98
* [sync] Componentize sync: Part 1: Clean up sync.gyp and update chrome / test ↵rsimha@chromium.org2012-11-141-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | dependencies One of the long term goals of the sync team is to pull sync code out of chrome.dll and into its own component. As of today, several chrome tests depend on various sync targets as defined in sync.gyp. This is the first in a series of CLs to achieve the above goal. This patch does the following: - Renames / reorganizes several of the targets in sync.gyp - Updates the declarations in various chrome gyp files by depending on the smallest sync target possible to reflect the actual dependency. - Takes a step closer toward the ideal IWYU guideline. TBR=erikwright@chromium.org BUG=136928 TEST=all chrome targets build on all platforms; all tests pass. Review URL: https://chromiumcodereview.appspot.com/11348052 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@167558 0039d316-1c4b-4281-b951-d872f2087c98
* Establish /extensions directory and move URLPattern there.aa@chromium.org2012-11-133-2/+5
| | | | | | | | | | | | | This doesn't create an extensions gyp target, or a module with a public API. It does set of DEPS rules and seems like it will make it easier for people to write correctly layered code today, while we are in transition. BUG=159265 TBR=ben@chromium.org Review URL: https://codereview.chromium.org/11410015 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@167454 0039d316-1c4b-4281-b951-d872f2087c98
* Reland 167337 - Move url_request_test_util into net namespacejochen@chromium.org2012-11-133-5/+8
| | | | | | | | | | | | | | | This file contains some rather generic names like "TestDelegate". Move it to the net namespace to avoid collisions BUG=none TBR=wtc@chromium.org,darin@chromium.org Original Review URL: https://codereview.chromium.org/11369179 Review URL: https://chromiumcodereview.appspot.com/11365227 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@167413 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 167337 - Move url_request_test_util into net namespacejochen@chromium.org2012-11-133-8/+5
| | | | | | | | | | | | | | | This file contains some rather generic names like "TestDelegate". Move it to the net namespace to avoid collisions BUG=none TBR=darin@chromium.org Review URL: https://codereview.chromium.org/11369179 TBR=jochen@chromium.org Review URL: https://codereview.chromium.org/11410066 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@167356 0039d316-1c4b-4281-b951-d872f2087c98
* Move url_request_test_util into net namespacejochen@chromium.org2012-11-133-5/+8
| | | | | | | | | | | | This file contains some rather generic names like "TestDelegate". Move it to the net namespace to avoid collisions BUG=none TBR=darin@chromium.org Review URL: https://codereview.chromium.org/11369179 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@167337 0039d316-1c4b-4281-b951-d872f2087c98
* Revert "Deleted TestWithWebServer from chrome_frame_tests."ygorshenin@chromium.org2012-11-121-0/+2
| | | | | | | | | | | | | This reverts commit 0ed667765967ffa7ab1bfa455e7aa570f433eb07. Reverted deletion of TestWithWebServer from chrome_frame_tests. BUG=160497 TBR=grt@chromium.org Review URL: https://codereview.chromium.org/11275258 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@167170 0039d316-1c4b-4281-b951-d872f2087c98
* Deleted TestWithWebServer from chrome_frame_tests.ygorshenin@chromium.org2012-11-121-2/+0
| | | | | | | | | BUG=160497 TBR=grt@chromium.org Review URL: https://codereview.chromium.org/11275256 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@167169 0039d316-1c4b-4281-b951-d872f2087c98
* Some chrome frame tests are marked as flaky.ygorshenin@chromium.org2012-11-121-7/+15
| | | | | | | | | | | | | | | | | | Following tests: FullTabModeIE_XHRTest FullTabModeIE_MetaTag FullTabModeIE_TestMultipleGet WidgetModeIE_CFInstanceNoSrc WidgetModeIE_Version WidgetModeIE_NavigateURLRelative WidgetModeIE_iframeBasic Are marked as FLAKY. BUG=160497 TBR=grt@chromium.org Review URL: https://codereview.chromium.org/11275254 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@167150 0039d316-1c4b-4281-b951-d872f2087c98
* Move stack_container and linked_list to the new containers subdirectory.brettw@chromium.org2012-11-101-2/+2
| | | | | | | | | | This also replaces the StackWString with StackString16 and adds a char traits definition to the string16 header file to make this possible without ifdefs. BUG= Review URL: https://codereview.chromium.org/11360174 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@167022 0039d316-1c4b-4281-b951-d872f2087c98
* Moved JsonPrefStore to use SequencedWorkerPool instead of FILE thread. The ↵zelidrag@chromium.org2012-11-081-3/+11
| | | | | | | | | | | pool also ensures that the same file requests are written in order received and that they block on shutdown. BUG=153367 TEST=existing unit/browser tests Review URL: https://codereview.chromium.org/11027070 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@166603 0039d316-1c4b-4281-b951-d872f2087c98
* Provide mutable members of UrlRequestContext via pure-virtual interface. ↵pauljensen@chromium.org2012-11-072-2/+5
| | | | | | | | | | Create a pure-virtual interface called HttpUserAgentSettings that provides access to the Accept-Language, Accept-Charset, and User-Agent HTTP headers. Each UrlRequestContext should have a HttpUserAgentSettings implementation attached via set_http_user_agent_settings(). BUG=146596 Review URL: https://chromiumcodereview.appspot.com/10918279 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@166425 0039d316-1c4b-4281-b951-d872f2087c98