summaryrefslogtreecommitdiffstats
path: root/chrome/chrome.gyp
Commit message (Collapse)AuthorAgeFilesLines
* Revert 155950 - Add initial Chromium TestShell support for Androidthakis@chromium.org2012-09-111-7/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This makes check_deps fail on linux thusly: Using base directory: src Checking: src WARNING: no package definition found in src/content/test/data/plugin/java_applet.java WARNING: no package definition found in src/content/test/data/plugin/applet-tests/cookies/CookieTest.java WARNING: no package definition found in src/content/test/data/plugin/applet-tests/authenticator/AuthenticatorTest.java WARNING: multiple definitions of com.example.HelloLicenseServlet: src/chrome/common/extensions/docs/examples/apps/hello-java/workspace/HelloLicense/src/com/example/HelloLicenseServlet.java src/chrome/common/extensions/docs/examples/apps/hello-java/HelloLicenseServlet.java ERROR in src/chrome/android/testshell/java/src/org/chromium/chrome/testshell/ChromiumTestShellActivity.java Illegal include: "content/public/android/java/src/org/chromium/content/app/LibraryLoader.java" Because of no rule applying. Illegal include: "content/public/android/java/src/org/chromium/content/browser/ContentView.java" Because of no rule applying. Illegal include: "content/public/android/java/src/org/chromium/content/common/CommandLine.java" Because of no rule applying. ERROR in src/chrome/android/testshell/java/src/org/chromium/chrome/testshell/ChromiumTestShellApplication.java Illegal include: "content/public/android/java/src/org/chromium/content/app/LibraryLoader.java" Because of no rule applying. Illegal include: "content/public/android/java/src/org/chromium/content/browser/ResourceExtractor.java" Because of no rule applying. FAILED Add a build target and tie it into android_all. This will build and link Chrome code for a test apk for Android. When it runs it properly loads the library but crashes. Will fix the crashes in subsequent patches, but can fork that work after this is in. BUG=136786 Review URL: https://chromiumcodereview.appspot.com/10918113 TBR=dtrainor@chromium.org Review URL: https://chromiumcodereview.appspot.com/10911202 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@155958 0039d316-1c4b-4281-b951-d872f2087c98
* Add initial Chromium TestShell support for Androiddtrainor@chromium.org2012-09-111-2/+7
| | | | | | | | | | | | | | Add a build target and tie it into android_all. This will build and link Chrome code for a test apk for Android. When it runs it properly loads the library but crashes. Will fix the crashes in subsequent patches, but can fork that work after this is in. BUG=136786 Review URL: https://chromiumcodereview.appspot.com/10918113 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@155950 0039d316-1c4b-4281-b951-d872f2087c98
* Remove {base,net}_java dependencies from GYP client targets on Android.pliard@chromium.org2012-09-101-4/+2
| | | | | | | | | | | | | | | | | | | | | | | | On Android, some GYP client targets were explicitly depending on both the native and Java sides of a library. This removes the dependencies to {base,net}_java and make the native side ('base' and 'net) of these libraries depend on their Java counterpart. On Android it rarely makes sense to depend on a single side of a Java/C++ library. The {base,net}_java can now be considered as "private" targets although GYP does not support this concept unfortunately (AFAICT). Note that I made sure that the resulting APKs' size is unchanged. Additionally, this CL removes 'base_java' (i.e. does not replace it with 'base') from the targets including 'build/apk_test.gypi'. This dependency should not have been there (in the wrong layer) in the first place. It's needed by ChromeNativeTestActivity.java which clients should not know about. BUG=146323 TBR=lipalani,sky,willchan,brettw Review URL: https://chromiumcodereview.appspot.com/10913083 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@155737 0039d316-1c4b-4281-b951-d872f2087c98
* Moving google_apis and GaiaClient to src/google_apis.joi@chromium.org2012-09-071-0/+1
| | | | | | | | | | TBR=mechanicalowners@chromium.org BUG=145584 Review URL: https://chromiumcodereview.appspot.com/10928017 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@155312 0039d316-1c4b-4281-b951-d872f2087c98
* Move chrome/browser/ui files to their own target, chrome_browser_uithakis@chromium.org2012-09-061-0/+1
| | | | | | | | | | | | | | | | This makes chrome_browser.gypi a lot smaller, which helps with the "ar commandline to long" that some platforms have. On Mac, rebuild time after touching a file below chrome/ui improves from 12s to 8s on my MBP while plugged in, and goes from 20s to 14s while on battery. (This is because only the chrome_browser_ui lib needs to be rebuilt, instead of all of libbrowser.a) BUG=none Review URL: https://chromiumcodereview.appspot.com/10911127 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@155251 0039d316-1c4b-4281-b951-d872f2087c98
* Use ChromeBrowserMainPartsAndroid.yfriedman@chromium.org2012-08-301-0/+2
| | | | | | | | | | | | | | | | | | | | | | | Includes several changes: 1) Stubs out jankometer 2) Makes some background-related profile code guarded by ENABLE_BACKGROUND 3) Remove SingletonProcess because it's guaranteed by the OS. 4) If-def out first-run code and process startup code as it's necessary handled in the Java layer on Android. In order to improve startup performance, Chrome for Android runs startup code in the java layer asynchronously with loading the native shared library. Re-using this code would result in us blocking until the entire library is loaded into memory. 5) Removes UMA code. UMA sessions are handled differently on Android as the browser isn't ever killed and is instead paused/resumed. 6) Also removes things like ManagedMode/PageCycler which are unused on Android. BUG=138184 Review URL: https://chromiumcodereview.appspot.com/10885023 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@154029 0039d316-1c4b-4281-b951-d872f2087c98
* Split browser/chromeos into its own targetstbarzic@chromium.org2012-08-281-0/+3
| | | | | | | | | BUG=144717 TEST=compiles Review URL: https://chromiumcodereview.appspot.com/10867071 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@153738 0039d316-1c4b-4281-b951-d872f2087c98
* Add missing dependency for chrome_java target.mkosiba@chromium.org2012-08-241-0/+2
| | | | | | | | | | | | | | | | The chrome_java target should depend on browser/components/web_contents_delegate_android. This is a re-land of r153154 which was reverted in r153160. Sadly this change needs a clobber. BUG=None TBR=yfriedman,thakis Review URL: https://chromiumcodereview.appspot.com/10880043 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@153209 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 153154 - Add missing dependency for chrome_java target.dbeam@chromium.org2012-08-241-2/+0
| | | | | | | | | | | | | | | The chrome_java target should depend on browser/components/web_contents_delegate_android. BUG=None Review URL: https://chromiumcodereview.appspot.com/10883016 TBR=nyquist@chromium.org Review URL: https://chromiumcodereview.appspot.com/10879060 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@153160 0039d316-1c4b-4281-b951-d872f2087c98
* Add missing dependency for chrome_java target.nyquist@chromium.org2012-08-241-0/+2
| | | | | | | | | | | | The chrome_java target should depend on browser/components/web_contents_delegate_android. BUG=None Review URL: https://chromiumcodereview.appspot.com/10883016 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@153154 0039d316-1c4b-4281-b951-d872f2087c98
* Make the Mac Info.plist SCM keys genericstuartmorgan@chromium.org2012-08-231-6/+6
| | | | | | | | | | | | | | This allows the file_version_info_mac code to make sense for any SCM (e.g., git). It also aligns the use of FetchVersionInfo to its documented behavior, which is to return a (branch, revision) tuple for "some appropriate revision control system". BUG=None Review URL: https://chromiumcodereview.appspot.com/10828228 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@153101 0039d316-1c4b-4281-b951-d872f2087c98
* Make java.gypi export jar path to input_jars_paths variablecjhopman@chromium.org2012-08-091-0/+5
| | | | | | | | | | | | | | | | | Both java.gypi and apk_test.gypi expect the jar path in input_jars_paths. This means that we were specifying the dependence in both 'dependencies' and 'input_jars_paths'. This change makes it so that we don't need that redundancy. Also, make java.gypi use input_jars_paths in its input so that we actually rebuild targets when we should. BUG=136756 Review URL: https://chromiumcodereview.appspot.com/10837143 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@150850 0039d316-1c4b-4281-b951-d872f2087c98
* Don't use NXArgc or NXArgv. Access them via _NSGetArgc() and _NSGetArgv().mark@chromium.org2012-08-071-10/+0
| | | | | | | | | | | This allows dead code stripping to be turned back on for the main executables and for the NXArgc and NXArgv symbols to be stripped. BUG=139902 Review URL: https://chromiumcodereview.appspot.com/10837149 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@150419 0039d316-1c4b-4281-b951-d872f2087c98
* [Android] Add chrome_java target for building Java code in the chromium layer.yfriedman@chromium.org2012-08-061-0/+19
| | | | | | | | | | | Also includes a refactoring of the Ant xml scripts to use a common template (contributed by shashishekhar@chromium.org). As part of this, I also continued Torne's effort of removing our reliance on environment variables. Unfortunately this currently means that you have to specify 5 gyp flags: ANDROID_SDK, ANDROID_SDK_ROOT, ANDROID_SDK_TOOLS, ANDROID_SDK_VERSION, ANDROID_TOOLCHAIN. This'll get better as we make further use of the checked in sdk both upstream and downstream. The problem stems from the android tree and released sdk having different configurations. Review URL: https://chromiumcodereview.appspot.com/10830012 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@150172 0039d316-1c4b-4281-b951-d872f2087c98
* Do not pass -dead_strip to ld for the Mac helper and main apps.rsesek@chromium.org2012-08-011-0/+10
| | | | | | | | | | | | | On certain SDK versions and OSes with the 10.6 deployment target, the linker, if invoked with -dead_strip, incorrectly removes _NXArgc and _NXArgv. Since the apps are just shims into the Framework, there's little to strip anyways. BUG=139902 TEST=Build on 10.7+ with GYP variable mac_strip_release=1. Review URL: https://chromiumcodereview.appspot.com/10830126 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149531 0039d316-1c4b-4281-b951-d872f2087c98
* Simplify how LayoutBrowserTest scraps a tab. This allows removal of a whole ↵jam@chromium.org2012-07-241-4/+0
| | | | | | | | | bunch of ancient code. BUG=90448 Review URL: https://chromiumcodereview.appspot.com/10800090 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@148111 0039d316-1c4b-4281-b951-d872f2087c98
* Disable uses of ProfileImportProcess on Android.pliard@chromium.org2012-07-181-0/+7
| | | | | | | | | | | | | This is part of the unit_tests linking effort on Android. This moves the profile import-related code excluded on Android to its own class. BUG=136787 Review URL: https://chromiumcodereview.appspot.com/10777012 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@147285 0039d316-1c4b-4281-b951-d872f2087c98
* mac: Make plugins work in the components build.thakis@chromium.org2012-06-011-0/+6
| | | | | | | | | BUG=130789 TEST=youtube Review URL: https://chromiumcodereview.appspot.com/10473003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@140085 0039d316-1c4b-4281-b951-d872f2087c98
* The LASTCHANGE file is created and updated by gclient runhooks undertorne@chromium.org2012-05-311-6/+0
| | | | | | | | | | | | | | | | | | normal circumstances, so it makes more sense to treat it as if it were a checked-in source file, not a generated file. The rule for its generation was still getting run once on a clean build, because the other targets that depended on it expected a gyp timestamp file to exist for the target. There's no need for any other targets to actually depend on lastchange (and in fact several of the targets that make use of the file already don't depend on the target); it should already exist, and if it doesn't then the source checkout is probably broken (as the other steps in runhooks are also mandatory). So, remove the rule entirely. Review URL: https://chromiumcodereview.appspot.com/10388213 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@139765 0039d316-1c4b-4281-b951-d872f2087c98
* mac: Don't require DYLD_LIBRARY_PATH to be set when using the shared build.thakis@chromium.org2012-05-301-0/+9
| | | | | | | | | | | | | | | | | | | | | | | | Set the install name of all dylibs to @rpath. Set the rpath of all executables to '@loader_path/.' (for normal binaries) and '@loader_path/../../..' (for bundled binaries). Also, Chromium Helper.app doesn't end up in out/Release but somewhere below Chromium.app, so set its rpath to '@loader_path/../../../../../../..' to get all the way back to out/Release. Also add "(allow file-read-metadata)" to the sandbox definition when running on 10.6 or earlier when using the component build, to work around a bug in dyld (see http://crbug.com/127465). BUG=90078,127465 TEST=Do a components build. DYLD_LIBRARY_PATH isn't necessary at build time, and isn't necessary when running chromium or test binaries. Review URL: https://chromiumcodereview.appspot.com/10389047 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@139550 0039d316-1c4b-4281-b951-d872f2087c98
* [Sync] Move chrome/browser/sync/api to sync/apiakalin@chromium.org2012-05-151-37/+0
| | | | | | | | | | | | Update all references. BUG=128061 TEST= TBR=sky@chromium.org Review URL: https://chromiumcodereview.appspot.com/10389134 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@137062 0039d316-1c4b-4281-b951-d872f2087c98
* Make sure only the main browser process and service processes are allowed to ↵pastarmovj@chromium.org2012-05-091-0/+4
| | | | | | | | | | | | | | create the profile directory. This patch lets Chrome start with profile located on a network share on Windows Vista and newer. BUG=120388 TEST=Start Chrome with --user-data-dir pointing to a network share location and try to navigate to a web page. This should not lead to a hang of the renderer. NaCl and NPAPI plugins should run fine too. Review URL: https://chromiumcodereview.appspot.com/10390003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@136020 0039d316-1c4b-4281-b951-d872f2087c98
* Added wipeout functionality when Cloud Print Connector get disabled by user ↵gene@chromium.org2012-05-041-0/+2
| | | | | | | | | | | | in settings. This will unregister all local printers from the server. TEST=Disable CLoud Print in Chrome and verify printers get unregistered. Review URL: http://codereview.chromium.org/10373002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@135420 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 135321 - Make sure only the main browser process and service proceses ↵pastarmovj@chromium.org2012-05-041-4/+0
| | | | | | | | | | | | | | | | | | are allowed to create the profile directory. This patch lets Chrome start with profile located on a network share on Windows Vista and newer. BUG=120388 TEST=Start Chrome with --user-data-dir pointing to a network share location and try to navigate to a web page. This should not lead to a hang of the renderer. Review URL: http://codereview.chromium.org/10306009 TBR=pastarmovj@chromium.org Review URL: https://chromiumcodereview.appspot.com/10382012 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@135347 0039d316-1c4b-4281-b951-d872f2087c98
* Make sure only the main browser process and service proceses are allowed to ↵pastarmovj@chromium.org2012-05-041-0/+4
| | | | | | | | | | | | | | | create the profile directory. This patch lets Chrome start with profile located on a network share on Windows Vista and newer. BUG=120388 TEST=Start Chrome with --user-data-dir pointing to a network share location and try to navigate to a web page. This should not lead to a hang of the renderer. Review URL: http://codereview.chromium.org/10306009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@135321 0039d316-1c4b-4281-b951-d872f2087c98
* Build fixes to get use_aura && !use_ash to build.ben@chromium.org2012-05-011-0/+8
| | | | | | | | | | Also, build more stuff on win_aura. BUG=none TEST=none Review URL: https://chromiumcodereview.appspot.com/10260033 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@134806 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 134269 - Revert 134234 - Turn thread restrictions on for release ↵kevers@chromium.org2012-04-271-0/+2
| | | | | | | | | | | | | | | | | builds as well so that they're triggered in (default) try/commit jobs. I let them be compiled out only in official builds. Revert breaks ChromeOS Builder (dbg) Also update bug for net::FileStream usage. Review URL: https://chromiumcodereview.appspot.com/10238009 TBR=jam@chromium.org Review URL: https://chromiumcodereview.appspot.com/10256003 TBR=kevers@chromium.org Review URL: https://chromiumcodereview.appspot.com/10255004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@134270 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 134234 - Turn thread restrictions on for release builds as well so ↵kevers@chromium.org2012-04-271-2/+0
| | | | | | | | | | | | | | that they're triggered in (default) try/commit jobs. I let them be compiled out only in official builds. Breaks vmtest on builders chromiumOS x86 and amd64. Also update bug for net::FileStream usage. Review URL: https://chromiumcodereview.appspot.com/10238009 TBR=jam@chromium.org Review URL: https://chromiumcodereview.appspot.com/10256003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@134269 0039d316-1c4b-4281-b951-d872f2087c98
* Turn thread restrictions on for release builds as well so that they're ↵jam@chromium.org2012-04-271-0/+2
| | | | | | | | | triggered in (default) try/commit jobs. I let them be compiled out only in official builds. Also update bug for net::FileStream usage. Review URL: https://chromiumcodereview.appspot.com/10238009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@134234 0039d316-1c4b-4281-b951-d872f2087c98
* [Sync] Move 'syncapi_core' and 'sync_unit_tests' targets to sync/akalin@chromium.org2012-04-211-67/+1
| | | | | | | | | | | | | | | Also move related test files. Lock down deps for sync/internal_api. Clean up some deps on chrome/browser/sync. BUG=117585 TEST= Review URL: https://chromiumcodereview.appspot.com/10147003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@133349 0039d316-1c4b-4281-b951-d872f2087c98
* Make the Datatypes using new syncapi upload unrecoverable errors to breakpad.lipalani@chromium.org2012-04-201-0/+2
| | | | | | | | | BUG= TEST= Review URL: http://codereview.chromium.org/10061002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@133251 0039d316-1c4b-4281-b951-d872f2087c98
* [Sync] Move 'sync_notifier' target to sync/akalin@chromium.org2012-04-191-46/+1
| | | | | | | | | | | | | | | | | | Also move related test files. Remove some unneeded includes. Lock down deps for sync/notifier. Clean up some deps in chrome/browser/sync. BUG=117585,124139 TEST= Review URL: http://codereview.chromium.org/10115046 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@132990 0039d316-1c4b-4281-b951-d872f2087c98
* Update other gypi that referenced extension_proxy.cc. TBR=ahsujam@chromium.org2012-04-181-2/+0
| | | | git-svn-id: svn://svn.chromium.org/chrome/trunk/src@132802 0039d316-1c4b-4281-b951-d872f2087c98
* mac: Add pkg-dmg as input to the build_app_dmg action.thakis@chromium.org2012-04-161-0/+2
| | | | | | | | | | | | | That way, the action reruns if pkg-dmg (called from the build_app_dmg shell script) is touched. BUG=none TEST=none Review URL: http://codereview.chromium.org/10073026 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@132426 0039d316-1c4b-4281-b951-d872f2087c98
* Fix an input and the output of the build_app_dmg target for Google-branded ↵thakis@chromium.org2012-04-111-2/+11
| | | | | | | | | | | | builds. BUG=none TEST=build target 'all' with make or ninja doing an official build. 'build_app_dmg' doesn't complain about missing input out/Release/Chrome.app. Review URL: http://codereview.chromium.org/10048013 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@131695 0039d316-1c4b-4281-b951-d872f2087c98
* Filter out devtools_window.cc from Chrome on Android buildjgreenwald@chromium.org2012-04-061-0/+5
| | | | | | | | | | | | This eliminates some link errors when trying to remove browser.cc from the build. BUG=117407 TEST= Review URL: http://codereview.chromium.org/9546036 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@131061 0039d316-1c4b-4281-b951-d872f2087c98
* DevTools: make Save As dialog optional for the paths that have already been ↵pfeldman@chromium.org2012-04-041-2/+2
| | | | | | | | saved. Review URL: https://chromiumcodereview.appspot.com/9974001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@130630 0039d316-1c4b-4281-b951-d872f2087c98
* Make content_shell show up in chrome.sln. There are a number of people ↵jam@chromium.org2012-04-021-1/+2
| | | | | | | | who're starting to use it, and it makes sense that it shows up there. Review URL: https://chromiumcodereview.appspot.com/9965063 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@130162 0039d316-1c4b-4281-b951-d872f2087c98
* Remove an out-dated TODO. All content-side debugger stuff is already built ↵joi@chromium.org2012-03-231-1/+0
| | | | | | | | | | | via rules in GYP files under content/. TBR=jam@chromium.org BUG=98716 Review URL: https://chromiumcodereview.appspot.com/9835017 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@128476 0039d316-1c4b-4281-b951-d872f2087c98
* Completely remove the service process for Android.yfriedman@chromium.org2012-03-161-1/+6
| | | | | | | | | | | | It's only used by cloud print which isn't supported. BUG=117407 TEST= Review URL: http://codereview.chromium.org/9702025 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@127084 0039d316-1c4b-4281-b951-d872f2087c98
* [Sync] Move 'sync' target to sync/akalin@chromium.org2012-03-151-209/+13
| | | | | | | | | | | | | | | | | | | | | | Also move related test files. Move WriteNode::UpdateEntryWithEncryption to nigori_util.h. Clean up defines and dependencies. In particular, get rid of SYNC_ENGINE_VERSION_STRING and hard-code the string in the single place it's used. Rename data_encryption.* to data_encryption_win.* and add a pragma for crypt32.lib. Clean up exit-time constructor warnings in sync{able,er}_unittest.cc. Remove some unused files. BUG=117585 TEST= TBR=jhawkins@chromium.org Review URL: https://chromiumcodereview.appspot.com/9699057 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@126872 0039d316-1c4b-4281-b951-d872f2087c98
* [Sync] Create new top-level sync/ directoryakalin@chromium.org2012-03-101-6/+6
| | | | | | | | | | | | | Move sync .proto files to sync/protocol. Update all references. This begins the Great Sync Migration. BUG=117585,43624 TEST= Review URL: https://chromiumcodereview.appspot.com/9668028 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@125967 0039d316-1c4b-4281-b951-d872f2087c98
* Move all files below chrome/extensions into its own target.thakis@chromium.org2012-03-091-0/+1
| | | | | | | | | | | | This is a bandaid to work around the ar command for libbrowser.a failing with "command line too long". BUG=117512 TEST=none Review URL: https://chromiumcodereview.appspot.com/9651027 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@125812 0039d316-1c4b-4281-b951-d872f2087c98
* Refactor sync tests to use in-memory databaserlarocque@chromium.org2012-03-081-5/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit makes the DBS a pure-virtual class and introduces three concrete sub-classes: OnDiskDirectoryBackingStore, InMemoryDirectoryBackingStore and TestDirectoryBackingStore. This allows us to separate test-specific functionality (ie. OpenConnectionForTest() and some friend declarations) from the main DirectoryBackingStore interface. OnDiskDirectoryBackingStore is drop-in replacement to the "classic" DirectoryBackingStore. InMemoryDirectoryBackingStore is an easy to use class that is meant to be injected into test classes that require a DirectoryBackingStore. The TestDirectoryBackingStore is a bit more complicated, and is used only when directly testing the DirectoryBackingStore. TestDirectorySetterUpper and most test fixtures now use syncable::Directory::OpenInMemory() instead of Open(). This makes the Directory initialize itself with InMemoryDirectoryBackingStore. Tests that requiring an on-disk database have been modified to use OnDiskDirectoryBackingStore. The DirectoryBackingStore unit tests now use TestDirectoryBackingStore. The unit tests must manage their own sql::Connection and provide it to the TestDirectoryBackingStore. This allows them to inject initial database state into the test without having to create the database on disk. These changes have a significant impact on the time taken to run sync unit tests on my local machine. The average run time has gone from over 1 minute to under 20 seconds. BUG=116328 TEST= Review URL: http://codereview.chromium.org/9549023 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@125574 0039d316-1c4b-4281-b951-d872f2087c98
* [Sync] First pass at removing unneeded dependencies of syncakalin@chromium.org2012-03-081-8/+1
| | | | | | | | | | | | | | | | | | | | | In particular, remove dependency on 'common', 'content', and 'test_support_common' which was pulling in tons of chrome stuff as well as webkit. Make sync_unit_tests depend only on (mainly) sync, base, net, crypto, sql. Move syncable_mock.{h,cc} to test_support_sync. Add dependency on sql. Pull in run_all_unittests.cc from base for sync_unit_tests. BUG=113723 TEST= Review URL: https://chromiumcodereview.appspot.com/9600076 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@125548 0039d316-1c4b-4281-b951-d872f2087c98
* Fix tweak_info_plist.py to be explicit about arguments.avi@chromium.org2012-03-071-14/+10
| | | | | | | | | | BUG=115784 TEST=none Review URL: http://codereview.chromium.org/9617032 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@125349 0039d316-1c4b-4281-b951-d872f2087c98
* [Sync] Clean up GetSessionName filesakalin@chromium.org2012-03-071-0/+3
| | | | | | | | | | | | | | | | Move platform-specific stuff into their own files. Move GetSessionName unit tests to sync_unit_tests. Make GetSessionName's unit tests platform-agnostic. BUG=113723 TEST= Review URL: http://codereview.chromium.org/9600039 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@125270 0039d316-1c4b-4281-b951-d872f2087c98
* First pass at content shell for the Mac. Doesn't work yet.avi@chromium.org2012-03-061-2/+2
| | | | | | | | | BUG=115784 TEST=none Review URL: https://chromiumcodereview.appspot.com/9513006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@125254 0039d316-1c4b-4281-b951-d872f2087c98
* [Sync] Simplify GetSessionName interfaceakalin@chromium.org2012-03-031-3/+3
| | | | | | | | | | | | | | | | Turn the GetSessionNameTask class into a simple GetSessionName function. Use the browser thread blocking pool instead of the FILE thread. Remove dependency on BrowserThread from internal_api. BUG=113723 TEST= Review URL: http://codereview.chromium.org/9565050 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@124838 0039d316-1c4b-4281-b951-d872f2087c98
* [Sync] Move BridgedSyncNotifier and ChromeSyncNotificationBridge to glue/akalin@chromium.org2012-03-011-4/+0
| | | | | | | | | | | sync_notifier isn't supposed to have dependencies on chrome. BUG=113723 TEST= Review URL: https://chromiumcodereview.appspot.com/9512005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@124381 0039d316-1c4b-4281-b951-d872f2087c98
generated by cgit v1.1 at 2025-03-10 22:51:30 +0000