summaryrefslogtreecommitdiffstats
path: root/chrome/test/automation
Commit message (Collapse)AuthorAgeFilesLines
* `#pragma once` for app, base, chrome, gfx, ipc, net, skia, viewsthakis@chromium.org2010-07-2613-0/+13
| | | | | | | | | BUG=50273 TEST=everything still builds, build is 10% faster on windows, same speed on mac/linux TBR: erg git-svn-id: svn://svn.chromium.org/chrome/trunk/src@53716 0039d316-1c4b-4281-b951-d872f2087c98
* GTTF: test server cleanup:phajdan.jr@chromium.org2010-07-231-2/+1
| | | | | | | | | | | | | - simplify the public interface - remove unneeded methods - make it easier to understand TEST=none BUG=49680 Review URL: http://codereview.chromium.org/2881028 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@53509 0039d316-1c4b-4281-b951-d872f2087c98
* Forward declare more param traits in common_param_traits.erg@google.com2010-07-201-0/+1
| | | | | | | | | BUG=none TEST=none Review URL: http://codereview.chromium.org/2805088 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@53115 0039d316-1c4b-4281-b951-d872f2087c98
* Cleanup: Break another common->app dependency.thestig@chromium.org2010-07-201-4/+5
| | | | | | | | BUG=46666 TEST=none Review URL: http://codereview.chromium.org/3007008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@53113 0039d316-1c4b-4281-b951-d872f2087c98
* Remove the page and app menus.pkasting@chromium.org2010-07-203-10/+9
| | | | | | | | BUG=none TEST=none Review URL: http://codereview.chromium.org/3033007 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@52985 0039d316-1c4b-4281-b951-d872f2087c98
* Reland r52486. Display a tab modal dialog of the allowed/blocked cookies.jochen@chromium.org2010-07-163-1/+17
| | | | | | | | | BUG=45230 TEST=CollectedCookiesTest.* Review URL: http://codereview.chromium.org/3034007 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@52655 0039d316-1c4b-4281-b951-d872f2087c98
* ChromeFrame tabs would hang at times while closing. This would randomly ↵ananta@chromium.org2010-07-151-0/+11
| | | | | | | | | | | | | | | | | | | | | | | | occur if the page had an unload handler. We execute unload handlers in the WM_DESTROY message in the external tab and spin a nested loop waiting for the unload handlers to finish. This causes a deadlock at times if a windows message is dispatched to IE which is blocked in DestroyWindow. The fix is to remove the nested loop mess from the external tab and instead send over a special automation message to Chrome in which context we execute the unload handlers. The message contains the notification window and the actual window message to be posted back when the unload handlers finish executing. The active document/activex spin a GetMessage loop waiting for this message to arrive. To ensure that we don't wait indefinitely we have a 1 second timer and exit the loop if this timer is received. Fixes bug http://code.google.com/p/chromium/issues/detail?id=49132 Bug=49132 Test=Covered by existing unload event test. Review URL: http://codereview.chromium.org/3014001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@52523 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 52486 - Display a tab modal dialog of the allowed/blocked cookies.dmaclach@chromium.org2010-07-153-17/+1
| | | | | | | | | | | | BUG=45230 TEST=CollectedCookiesTest.* Review URL: http://codereview.chromium.org/2907003 TBR=jochen@chromium.org Review URL: http://codereview.chromium.org/3016003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@52505 0039d316-1c4b-4281-b951-d872f2087c98
* Display a tab modal dialog of the allowed/blocked cookies.jochen@chromium.org2010-07-153-1/+17
| | | | | | | | | BUG=45230 TEST=CollectedCookiesTest.* Review URL: http://codereview.chromium.org/2907003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@52486 0039d316-1c4b-4281-b951-d872f2087c98
* ChromeFrame would leave chrome processes running if a page rendered in ↵ananta@chromium.org2010-07-122-2/+4
| | | | | | | | | | | | | | | | | | | | | | | | Chrome would open up a popup window which opened up in IE. This is because of an orphan external tab container created to handle the popup. This tab is destroyed in the normal case when the active document window is destroyed, which is the parent of the external tab container. However in this case the active document is destroyed without its window being created thus resulting in the external tab container lying around which holds a reference to the automation provider. Fix is to pass the ChromeFrameAutomationClient window as the parent window in the AutomationMsg_ConnectExternalTab message. Fixes bug http://code.google.com/p/chromium/issues/detail?id=48732 Bug=48732 Covered by ChromeFrame test FullTabNavigationTest.JavascriptWindowOpenDifferentDomain Review URL: http://codereview.chromium.org/2911006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@52094 0039d316-1c4b-4281-b951-d872f2087c98
* Change a bunch of string types.estade@chromium.org2010-06-293-3/+3
| | | | | | | | | | | | | Started out just trying to change PrefService::GetString and ::SetString. This snowballed a little bit. Had to change a bunch of url strings in search_engines/ from wstring to string (some of them may be better off as GURLs, but UTF-8 is a step in the right direction, since that's what GURL uses internally, as well as externally via its setters/getters). TODO (later patch): things that ask for accepted languages should use std::string, not std::wstring. BUG=none TEST=try bots Review URL: http://codereview.chromium.org/2854015 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@51157 0039d316-1c4b-4281-b951-d872f2087c98
* In pages rendered in ChromeFrame window open requests or link clicks with ↵ananta@chromium.org2010-06-221-9/+1
| | | | | | | | | | | | | | | | | | | | | target blank which target a different origin should initiate the navigation in the host browser. We achieve this by performing an origin check on the opener frame and the URL being opened. If the origins don't match we allow the host browser to handle this navigation. There is still one issue here as a popup window creation request is still initiated and sent out to the host browser which initiates a dummy attach external tab navigation. Subsequently while applying policy the OpenURL IPC is sent out to the host browser which initiates the navigation to the expected URL. This causes a dummy attach external tab entry to be created in the host browser's history which would have to be deleted. Fixes bug http://code.google.com/p/chromium/issues/detail?id=46667 Bug=46667 Test=Covered by new chrome frame unit test. Review URL: http://codereview.chromium.org/2855017 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@50416 0039d316-1c4b-4281-b951-d872f2087c98
* Make new startup test use GetInitialLoadTimes for more accurate times.pathorn@chromium.org2010-06-172-2/+74
| | | | | | | | | | | | | This change modifies RunStartupTest to call GetInitialLoadTimes, if applicable, to print out additional statistics for each tab computed on the browser. Additionally, changes |test_cold| and |important| parameters to be enums instead of booleans to make tests more readable. BUG=44129 TEST=none Review URL: http://codereview.chromium.org/2714015 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@50075 0039d316-1c4b-4281-b951-d872f2087c98
* Flip a test from DISABLED_ to FAILS_ on the Mac, and add a bug annotationstuartmorgan@chromium.org2010-06-141-7/+6
| | | | | | | | | BUG=45892 TEST=AutomationProxyVisibleTest.WindowGetViewBounds should run on Mac Review URL: http://codereview.chromium.org/2698001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@49735 0039d316-1c4b-4281-b951-d872f2087c98
* Clean up external-tab related code in automation provider and automation proxy.estade@chromium.org2010-06-102-52/+54
| | | | | | | | | BUG=44695 TEST=compile Review URL: http://codereview.chromium.org/2743006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@49448 0039d316-1c4b-4281-b951-d872f2087c98
* Add Support for window.close in ChromeFrame. Currently the CloseContents ↵ananta@chromium.org2010-06-071-1/+9
| | | | | | | | | | | | | | | | | | notification on the TabContentsDelegate which the ExternalTabContainer implements is ignored if the window close is not initiated from the host. Fix is to send over an automation message AutomationMsg_CloseExternalTab to the external host where we close the window. Currently this functionality is only provided by the active document. Fixes bug http://code.google.com/p/chromium/issues/detail?id=45930 Bug=45930 Test=Covered by chrome frame unit test FullTabModeIE_WindowCloseInChrome Review URL: http://codereview.chromium.org/2691004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@49093 0039d316-1c4b-4281-b951-d872f2087c98
* Use DISALLOW_COPY_AND_ASSIGN instead of DISALLOW_EVIL_CONSTRUCTORS in chrome ↵tfarina@chromium.org2010-06-062-5/+5
| | | | | | | | | | | directory. BUG=None TEST=trybots Review URL: http://codereview.chromium.org/2657003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@49024 0039d316-1c4b-4281-b951-d872f2087c98
* TTF: Fix and enable AppModalDialogTest on Mac.dmuir@chromium.org2010-06-031-7/+0
| | | | | | | | | | | Also remove FLAKY annotation from test for other platforms -- annotation is likely stale. BUG=25039,5314 TEST=AutomationProxyTest.AppModalDialogTest Review URL: http://codereview.chromium.org/2576002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@48880 0039d316-1c4b-4281-b951-d872f2087c98
* Make webkit_glue not depend on default_plugin.thakis@chromium.org2010-06-021-0/+2
| | | | | | | | | | This removes a dependency cycle and should also make life easier for DRT. Revert http://codereview.chromium.org/1658012 Review URL: http://codereview.chromium.org/2358001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@48731 0039d316-1c4b-4281-b951-d872f2087c98
* Enable passing unload tests on Linux.estade@chromium.org2010-05-271-1/+1
| | | | | | | | | | | Also make one execute significantly faster. BUG=45021 TEST=they pass consistently Review URL: http://codereview.chromium.org/2296001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@48415 0039d316-1c4b-4281-b951-d872f2087c98
* GetEnabledExtensions should also return Loaded Extensions.mad@google.com2010-05-271-6/+5
| | | | | | | | | | | | We use to not need them since we would simply overinstall them all the time. But now that overinstall is allowed and reset the extension state (e.g., close and reopen the background page), we can't simply install exploded extensions at will anymore, we must first check if it was already installed or not (which means GetEnabledExtensions must return them) and only install them when they are not already there. BUG=0 TEST=none Review URL: http://codereview.chromium.org/2303004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@48400 0039d316-1c4b-4281-b951-d872f2087c98
* Marking ExternalTabUITest, TabPostMessage as flaky.robertshield@chromium.org2010-05-201-1/+2
| | | | | | | | | | BUG=44617 TEST=none TBR=cbentzel,stoyan Review URL: http://codereview.chromium.org/2135008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@47758 0039d316-1c4b-4281-b951-d872f2087c98
* Allow extensions to be overinstalled with extensions of sameaa@chromium.org2010-05-192-5/+1
| | | | | | | | | | | | | | | | | | | | | | version. This is useful during development, for switching themes, and for user scripts (since user scripts have no development mode). Since we can't always immediately delete the version directory for an extension after unloading it (because some files might be in use), this required changing the directory layout of the extensions directory to allow multiple copies of the same version of the same extension to be present at once. This was done by adding a counter to the version directory name. Also get rid of all the old "Current Version" cruft, since we no longer use that. BUG=26538 Review URL: http://codereview.chromium.org/1521039 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@47740 0039d316-1c4b-4281-b951-d872f2087c98
* Rename "mixed content" to "insecure content" in as many places as possible, ↵pkasting@chromium.org2010-05-184-15/+15
| | | | | | | | | | to standardize on a consistent naming scheme. BUG=none TEST=none Review URL: http://codereview.chromium.org/2069005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@47531 0039d316-1c4b-4281-b951-d872f2087c98
* Reland r47347 [was reverted in r47357], this time without re-enabling a ↵pkasting@chromium.org2010-05-171-5/+10
| | | | | | | | | | | | | | | | | | | | | DISABLED test that times out on Mac. (Original patch reviewed at http://codereview.chromium.org/2067003 ) Track "display" and "run" separately for mixed content, and make the latter downgrade the SSL state to "authentication broken". Make the "display" state only affect the current tab (not the entire host). Fix an SSL browser test by supplying the appropriate SiteInstance*. Move a test from "disabled" to "flaky" since it at least passes for me. Make the SSLManager header and .cc files put functions in the same order, and make that order somewhat saner. BUG=15072, 18626, 40932, 42758 TEST=Covered by browser tests Review URL: http://codereview.chromium.org/2063008 Review URL: http://codereview.chromium.org/2126005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@47428 0039d316-1c4b-4281-b951-d872f2087c98
* WebKit roll 59478:59607yurys@chromium.org2010-05-171-1/+2
| | | | | | | | | TBR=atwilson BUG=44371,44371 Review URL: http://codereview.chromium.org/2132004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@47420 0039d316-1c4b-4281-b951-d872f2087c98
* [GTTF] Check the result value of StartHTTPServer to avoidphajdan.jr@chromium.org2010-05-151-9/+9
| | | | | | | | | | | | | | | test hangs in cases when the server fails to start. The tests will still fail in that case, obviously, but will no longer hang (or crash), which is an improvement. TEST=browser_tests BUG=none Review URL: http://codereview.chromium.org/2095004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@47362 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 47347 - (Original patch reviewed at ↵rvargas@google.com2010-05-151-10/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | http://codereview.chromium.org/2067003 ) Track "display" and "run" separately for mixed content, and make the latter downgrade the SSL state to "authentication broken". Make the "display" state only affect the current tab (not the entire host). Fix an SSL browser test by supplying the appropriate SiteInstance*. Move a test from "disabled" to "flaky" since it at least passes for me. Make the SSLManager header and .cc files put functions in the same order, and make that order somewhat saner. Falied TestGoodFrameNavigation on Mac 10.5 browser tests among other things: /b/slave/chromium-rel-mac-builder/build/src/chrome/browser/ssl/ssl_browser_tests.cc:43: Failure Value of: entry->ssl().displayed_mixed_content() Actual: false Expected: displayed_mixed_content Which is: true BUG=15072, 18626, 40932, 42758 TEST=Covered by browser tests Review URL: http://codereview.chromium.org/2063008 TBR=pkasting@chromium.org Review URL: http://codereview.chromium.org/2095006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@47357 0039d316-1c4b-4281-b951-d872f2087c98
* (Original patch reviewed at http://codereview.chromium.org/2067003 )pkasting@chromium.org2010-05-151-5/+10
| | | | | | | | | | | | | | | | | | Track "display" and "run" separately for mixed content, and make the latter downgrade the SSL state to "authentication broken". Make the "display" state only affect the current tab (not the entire host). Fix an SSL browser test by supplying the appropriate SiteInstance*. Move a test from "disabled" to "flaky" since it at least passes for me. Make the SSLManager header and .cc files put functions in the same order, and make that order somewhat saner. BUG=15072, 18626, 40932, 42758 TEST=Covered by browser tests Review URL: http://codereview.chromium.org/2063008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@47347 0039d316-1c4b-4281-b951-d872f2087c98
* Removing the app launcher button in the ChromeOS browser and making the app ↵jcivelli@chromium.org2010-05-141-1/+1
| | | | | | | | | | | | | | launcher pinned to the location bar. BUG=3190 TEST=The app launcher icon on the top-left corner of the browser should be gone. Opening a new tab by pressing the + button or typing CTRL-T should bring the app launcher, it should be located on top of the location bar. Review URL: http://codereview.chromium.org/2060004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@47314 0039d316-1c4b-4281-b951-d872f2087c98
* Revert r46909: "Reland r46681: Use IPC to wait for download in DownloadTest."bauerb@chromium.org2010-05-113-23/+0
| | | | | | | | | | TBR=bauerb TEST=ui_tests on 10.6 and ChromiumOS should go green. BUG=43066 Review URL: http://codereview.chromium.org/1985014 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@46915 0039d316-1c4b-4281-b951-d872f2087c98
* Reland r46681: Use IPC to wait for download in DownloadTest.bauerb@chromium.org2010-05-113-0/+23
| | | | | | | | | | | | | Add AutomationMsg_WaitForDownloadShelfVisibilityChange and use it in UITestBase for the download tests. This should decrease flakiness of download tests. TEST=DownloadTest.*, especially DownloadTest.[Dont]CloseNewTab* BUG=43066 Review URL: http://codereview.chromium.org/2051002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@46909 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 46681 - Add AutomationMsg_WaitForDownloadShelfVisibilityChange and ↵bauerb@chromium.org2010-05-073-23/+0
| | | | | | | | | | | | | | | | use it in UITestBase for the download tests. This should decrease flakiness of download tests. TEST=DownloadTest.*, especially DownloadTest.[Dont]CloseNewTab* BUG=43066 Review URL: http://codereview.chromium.org/1933005 TBR=bauerb@chromium.org Review URL: http://codereview.chromium.org/2037003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@46685 0039d316-1c4b-4281-b951-d872f2087c98
* Add AutomationMsg_WaitForDownloadShelfVisibilityChange and use it in ↵bauerb@chromium.org2010-05-073-0/+23
| | | | | | | | | | | | | UITestBase for the download tests. This should decrease flakiness of download tests. TEST=DownloadTest.*, especially DownloadTest.[Dont]CloseNewTab* BUG=43066 Review URL: http://codereview.chromium.org/1933005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@46681 0039d316-1c4b-4281-b951-d872f2087c98
* Mark ExternalTabUITest.CreateExternalTab1 as flaky.phajdan.jr@chromium.org2010-04-291-1/+2
| | | | | | | | | | | TBR=stoyan TEST=flaky BUG=32293 Review URL: http://codereview.chromium.org/1815001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@45951 0039d316-1c4b-4281-b951-d872f2087c98
* Mark ExternalTabUITest.HostNetworkStack failed as flaky.bulach@chromium.org2010-04-261-1/+2
| | | | | | | | | TBR=stoyan@chromium.org BUG=42545 Review URL: http://codereview.chromium.org/1733017 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@45576 0039d316-1c4b-4281-b951-d872f2087c98
* C++ readability review with changelist for providing dom automation support ↵kkania@chromium.org2010-04-226-15/+32
| | | | | | | | | | to browser tests. BUG=none TEST=none Review URL: http://codereview.chromium.org/1135003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@45320 0039d316-1c4b-4281-b951-d872f2087c98
* Add delete cookie option to Automation Proxyjmikhail@google.com2010-04-214-0/+25
| | | | | | | | BUG=none TEST=none git-svn-id: svn://svn.chromium.org/chrome/trunk/src@45256 0039d316-1c4b-4281-b951-d872f2087c98
* Close a newly opened (by cmd-clicking on a link) tab if it resulted in a ↵bauerb@chromium.org2010-04-213-1/+33
| | | | | | | | | | | | | | | | download. To test this, the CL adds an automation message AutomationMsg_NavigateAsyncWithDisposition, and a method NavigateToURLAsyncWithDisposition to TabProxy. The only functional change is in TabContents::OnStartDownload, the rest of the changes is for testing. BUG=10764 TEST=DownloadTest.CloseNewTab Manual test: go to http://build.chromium.org/buildbot/continuous/mac/LATEST/, open the link "chrome-mac.zip" in a new tab. The tab should close as soon as the download starts. Review URL: http://codereview.chromium.org/1151007 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@45158 0039d316-1c4b-4281-b951-d872f2087c98
* Close previous theme info bars when resetting to default themebulach@chromium.org2010-04-204-8/+21
| | | | | | | | | | | | Select and close previous theme info bars when resetting to default theme via the options dialog. As a nice side effect if one tab is showing a theme info bar and the user switches to a different theme in another tab, this change closes the previous theme info bars as well. (landing for satish@chromium.org / http://codereview.chromium.org/1622017/show) BUG=35607 TEST=Install a theme, and with the theme info bar visible open Preferences dialog and click 'reset to default theme' and check the info bar. (full steps given in the above bug entry) Review URL: http://codereview.chromium.org/1650016 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@45034 0039d316-1c4b-4281-b951-d872f2087c98
* Context menu operations like Cut/Copy/Paste etc would not work in pages ↵ananta@chromium.org2010-04-193-4/+11
| | | | | | | | | | | | | | | | | | | rendered by ChromeFrame. The context menu is displayed by the ChromeFrame plugin which grabs focus to ensure that the menu has focus, and then restores focus back once we select an item. The latter step sends over a notification to Chrome via an automation message which then informs the view. This resets the webview item selection which results in this bug. Fix is to send over an additional flag to Chrome in the SetInitialFocus which indicates whether we need to inform the view about the focus change or not. Fixes bug http://code.google.com/p/chromium/issues/detail?id=41523 Bug=41523 Review URL: http://codereview.chromium.org/1574033 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@44951 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 44885 - Select and close previous theme info bars when resetting to ↵jorlow@chromium.org2010-04-184-21/+8
| | | | | | | | | | | | | | | | | default theme via the options dialog. As a nice side effect if one tab is showing a theme info bar and the user switches to a different theme in another tab, this change closes the previous theme info bars as well. BUG=35607 TEST=Install a theme, and with the theme info bar visible open Preferences dialog and click 'reset to default theme' and check the info bar. (full steps given in the above bug entry) Landing http://codereview.chromium.org/1622017/show for Satish. Review URL: http://codereview.chromium.org/1600030 TBR=jorlow@chromium.org Review URL: http://codereview.chromium.org/1599038 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@44886 0039d316-1c4b-4281-b951-d872f2087c98
* Select and close previous theme info bars when resetting to default theme ↵jorlow@chromium.org2010-04-184-8/+21
| | | | | | | | | | | | | | via the options dialog. As a nice side effect if one tab is showing a theme info bar and the user switches to a different theme in another tab, this change closes the previous theme info bars as well. BUG=35607 TEST=Install a theme, and with the theme info bar visible open Preferences dialog and click 'reset to default theme' and check the info bar. (full steps given in the above bug entry) Landing http://codereview.chromium.org/1622017/show for Satish. Review URL: http://codereview.chromium.org/1600030 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@44885 0039d316-1c4b-4281-b951-d872f2087c98
* Add ability to manipulate DOM elements from the automation proxy. Rework the ↵kkania@chromium.org2010-04-168-388/+628
| | | | | | | | | | way that javascript is packaged and parsed in the JavascriptExecutionController, and add some waiting methods. BUG=none TEST=none Review URL: http://codereview.chromium.org/1632001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@44778 0039d316-1c4b-4281-b951-d872f2087c98
* Cleanup: Explicitly add a bunch of missing includes.thestig@chromium.org2010-04-151-1/+2
| | | | | | | | BUG=none TEST=none Review URL: http://codereview.chromium.org/1623014 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@44635 0039d316-1c4b-4281-b951-d872f2087c98
* Switch from using std::min to std::max to fix an obvious bug in tab_proxy.brettw@chromium.org2010-04-141-7/+11
| | | | | | | | | | | | | Using std::min in these cases limits the loops to only 0 or 1 iteration, so the loops were never waiting for the given timeout before returning an error. BUG=none TEST=Tests which were passing before should continue to pass, some flaky ones may start working because we are waiting for the correct timeout now. Patch by satish@chromium.org Original review: http://codereview.chromium.org/1530034 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@44553 0039d316-1c4b-4281-b951-d872f2087c98
* Refactor FileVersionInfo into an interface with platform implementations.tony@chromium.org2010-04-141-1/+2
| | | | | | | | | | | | | This allows us to move the chrome specific version informaton used by Linux into src/chrome. Add a GetChromeVersionInfo() for Linux in src/chrome/app/ and make sure to use this in src/chrome. In src/webkit/glue, add a new glue method for getting the product version. When compiling chrome, use an implementation in src/chrome/renderer (which uses GetChromeVersionInfo()) and a stub implementation for test_shell. Review URL: http://codereview.chromium.org/1560027 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@44435 0039d316-1c4b-4281-b951-d872f2087c98
* [GTTF] Make automation calls for Downloads, Extensions, and History pagesphajdan.jr@chromium.org2010-04-121-0/+31
| | | | | | | | | | | | | reliable. This way we'll be able to write good pyauto tests for them. TEST=ui_tests BUG=none Review URL: http://codereview.chromium.org/1562021 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@44264 0039d316-1c4b-4281-b951-d872f2087c98
* [GTTF] Tests cleanup:phajdan.jr@chromium.org2010-04-081-9/+9
| | | | | | | | | | | | | | - use timeouts correctly - use more automation methods to make test code more clear - use more reliable ways of waiting - remove unnecessary automation calls TEST=none BUG=none Review URL: http://codereview.chromium.org/1553016 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@43932 0039d316-1c4b-4281-b951-d872f2087c98
* Add support for basic extension automation through the Automation Proxy.kkania@chromium.org2010-04-068-11/+539
| | | | | | | | BUG=36171 TEST=none Review URL: http://codereview.chromium.org/1048002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@43730 0039d316-1c4b-4281-b951-d872f2087c98