summaryrefslogtreecommitdiffstats
path: root/chrome/browser/debugger
Commit message (Collapse)AuthorAgeFilesLines
* Get rid of a bunch of tab_contents.h includes from chrome. These are all ↵jam@chromium.org2012-01-043-4/+4
| | | | | | | | | trivial changes to use WebContents instead of TabContents. BUG=98716 Review URL: http://codereview.chromium.org/9030032 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@116392 0039d316-1c4b-4281-b951-d872f2087c98
* Get rid of content::NavigationController in cc file and use "using" instead.jam@chromium.org2012-01-042-6/+8
| | | | | | | | BUG=98716 TBR=joi Review URL: http://codereview.chromium.org/8983012 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@116328 0039d316-1c4b-4281-b951-d872f2087c98
* Replace Browser::GetTabContentsAt with GetWebContentsAt and update all callers.jam@chromium.org2012-01-041-4/+6
| | | | | | | | BUG=98716 TBR=joi Review URL: http://codereview.chromium.org/9085006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@116322 0039d316-1c4b-4281-b951-d872f2087c98
* Convert WebContents to return a content::NavigationController instead of the ↵jam@chromium.org2012-01-041-2/+2
| | | | | | | | | | implementation. Update all the headers in chrome to use the interface only. In a subsequent cl, I'll rename the implementation to NavigationControllerImpl and also get rid of content::NavigationController everywhere. BUG=98716 TBR=joi Review URL: http://codereview.chromium.org/8983010 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@116244 0039d316-1c4b-4281-b951-d872f2087c98
* Convert all the notifications which use NavigationController to ↵jam@chromium.org2012-01-032-9/+9
| | | | | | | | | | content::NavigationController. After this is done, we can then make WebContents return a content::NavigationEntry instead of the impl. BUG=98716 TBR=joi Review URL: http://codereview.chromium.org/9016047 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@116150 0039d316-1c4b-4281-b951-d872f2087c98
* Replace most of Browser::GetSelectedTabContents calls into ↵jam@chromium.org2012-01-032-2/+4
| | | | | | | | | | | | Browser::GetSelectedWebContents. I've converted the easy ones, I'll do the ones with more dependencies in a separate change to keep things trivial to review. I considered taking out GetSelectedTabContents altogether and having people just use GetSelectedTabContentsWrapper()->web_contents() per the existing comment in browser.h, but there are a lot of callers and it seemed too long to type. BUG=98716 TBR=joi Review URL: http://codereview.chromium.org/9015022 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@116122 0039d316-1c4b-4281-b951-d872f2087c98
* Create an API around WorkerService that chrome consumes. Rename the existing ↵jam@chromium.org2011-12-291-2/+4
| | | | | | | | | WorkerService to WorkerServiceImpl and put it in the content namespace while at it. Also move WorkerServiceObserver to the public directory and into the content namespace. BUG=98716 Review URL: http://codereview.chromium.org/9052007 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@116007 0039d316-1c4b-4281-b951-d872f2087c98
* DevTools: make dock side type a part of the devtools location url.pfeldman@chromium.org2011-12-291-2/+7
| | | | | | Review URL: http://codereview.chromium.org/9022029 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@115968 0039d316-1c4b-4281-b951-d872f2087c98
* TBR=jampfeldman@chromium.org2011-12-292-0/+24
| | | | | | | | John: I've added a method and a message in the devtools subsystem. Review URL: http://codereview.chromium.org/9008043 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@115967 0039d316-1c4b-4281-b951-d872f2087c98
* Convert all of the WebContentsDelegate to use WebContents instead of ↵jam@chromium.org2011-12-282-6/+6
| | | | | | | | | | TabContents, and update all the dependent code. BUG=98716 TBR=joi Review URL: http://codereview.chromium.org/9008047 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@115932 0039d316-1c4b-4281-b951-d872f2087c98
* Rename NavigationController to NavigationControllerImpl and put it into the ↵jam@chromium.org2011-12-281-1/+2
| | | | | | | | | | content namespace. Also make all users of content::NavigationController interface use the "using" keyword so they don't have to put content:: everywhere (similar to other Content API and WebKit API classes). BUG=98716 TBR=joi Review URL: http://codereview.chromium.org/8956059 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@115916 0039d316-1c4b-4281-b951-d872f2087c98
* Convert a few methods in NavigationController to return a ↵jam@chromium.org2011-12-281-2/+3
| | | | | | | | | | content::NavigationEntry, in preparation for creating an interface around NavigationController. BUG=98716 TBR=joi Review URL: http://codereview.chromium.org/9008034 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@115872 0039d316-1c4b-4281-b951-d872f2087c98
* Move the PageNavigator interface and GlobalRequestID struct to ↵jam@chromium.org2011-12-282-3/+6
| | | | | | | | | | content\public and put them in the content namespace. Make PageNavigator use WebContents instead of TabContents. While I'm touching all the callers, I've removed the deprecated PageNavigator function and converted users to the new one. BUG=98716 TBR=joi Review URL: http://codereview.chromium.org/8991012 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@115858 0039d316-1c4b-4281-b951-d872f2087c98
* Move FaviconStatus and SSLStatus out of NavigationEntry into their own files ↵jam@chromium.org2011-12-271-2/+2
| | | | | | | | | in content/public and in the content namespace. I've also made them structs instead of classes. This was because I didn't want to wrap them with Content API for what were really a collection of member variables. The one exception was SSLStatus::content_status which had helper functions around it to set and get the bitfield. Each of the two setter helpers were only called in one non-test code, and read in a few places, so I just converted them to do it directly. BUG=98716 Review URL: http://codereview.chromium.org/9048002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@115847 0039d316-1c4b-4281-b951-d872f2087c98
* Added command line flag for enabling DevTools experiments.vsevik@chromium.org2011-12-271-2/+8
| | | | | | | | | BUG=108691 Review URL: http://codereview.chromium.org/9047004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@115838 0039d316-1c4b-4281-b951-d872f2087c98
* Convert SpeechInputBubble and BalloonHost to use WebContents instead of ↵jam@chromium.org2011-12-271-1/+1
| | | | | | | | | | TabContents, as well as all the dependent code. BUG=98716 TBR=joi Review URL: http://codereview.chromium.org/9016038 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@115810 0039d316-1c4b-4281-b951-d872f2087c98
* Adjusted tests to changes to chrome.(experimental.)?devtools namespaces in WK.caseq@google.com2011-12-261-2/+17
| | | | | | | | | BUG=none TEST=yep! Review URL: http://codereview.chromium.org/9018012 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@115779 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 115346. The change didn't break anything, the test was faulty. Will ↵jam@chromium.org2011-12-212-8/+8
| | | | | | | | | | | | | | | | | | | | disable in a followup. - Revert r115276, it broke PanelDownloadTest.Download in interactive_ui_tests on win,linux,mac. Original change: Rename TabContents::controller() to GetController and put it into the WebContents namespace. BUG=98716 TBR=dpranke Review URL: http://codereview.chromium.org/8956050 TBR=jam Review URL: http://codereview.chromium.org/9018016 TBR=thakis@chromium.org git-svn-id: svn://svn.chromium.org/chrome/trunk/src@115428 0039d316-1c4b-4281-b951-d872f2087c98
* Revert r115276, it broke PanelDownloadTest.Download in interactive_ui_teststhakis@chromium.org2011-12-212-8/+8
| | | | | | | | | | | | | | | | on win,linux,mac. Original change: Rename TabContents::controller() to GetController and put it into the WebContents namespace. BUG=98716 TBR=dpranke Review URL: http://codereview.chromium.org/8956050 TBR=jam Review URL: http://codereview.chromium.org/9018016 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@115346 0039d316-1c4b-4281-b951-d872f2087c98
* Temporarily disable DevToolsExtensionTest.TestDevToolsExtensionAPIcaseq@google.com2011-12-211-1/+1
| | | | | | | | | | | | | | | while landing WK bug 74941 (https://bugs.webkit.org/show_bug.cgi?id=74941) these will be re-enabled with this change: http://codereview.chromium.org/9018012/ once WebKit is rolled past the revision that contains patch for bug 74941. BUG=none TEST=none TBR=pfeldman Review URL: http://codereview.chromium.org/9016015 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@115341 0039d316-1c4b-4281-b951-d872f2087c98
* Take chrome.devtools out of experimental.caseq@google.com2011-12-212-11/+9
| | | | | | | | | BUG=105920 TEST=DevToolsExtensionAPITest.TestDevToolsExtensionAPI, ExtensionManifestTest.DevToolsExtensions Review URL: http://codereview.chromium.org/8995004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@115320 0039d316-1c4b-4281-b951-d872f2087c98
* Rename TabContents::view() to GetView() and put it in the WebContents interface.jam@chromium.org2011-12-211-3/+3
| | | | | | | | BUG=98716 TBR=tburkard Review URL: http://codereview.chromium.org/8999005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@115278 0039d316-1c4b-4281-b951-d872f2087c98
* Rename TabContents::controller() to GetController and put it into the ↵jam@chromium.org2011-12-212-8/+8
| | | | | | | | | | WebContents namespace. BUG=98716 TBR=dpranke Review URL: http://codereview.chromium.org/8956050 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@115276 0039d316-1c4b-4281-b951-d872f2087c98
* [Coverity] Change PASS_BY_VALUE of large-ish value to pass by ref indbeam@chromium.org2011-12-211-1/+1
| | | | | | | | | | | | chrome/browser/debugger/devtools_file_util.cc R=groby@chromium.org,binji@chromium.org CID=100209 Review URL: http://codereview.chromium.org/9005028 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@115240 0039d316-1c4b-4281-b951-d872f2087c98
* Move TabContentsDelegate to content/public/browser and put it in the content ↵tfarina@chromium.org2011-12-202-5/+5
| | | | | | | | | | | | | | namespace. - Rename TabContentsDelegate to WebContentsDelegate and move it to web_contents_delegate.* BUG=98716,105875 R=jam@chromium.org TBR=avi@chromium.org Review URL: http://codereview.chromium.org/9008013 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@115213 0039d316-1c4b-4281-b951-d872f2087c98
* Rename TabContents::render_view_host() to GetRenderViewHost and put it into ↵jam@chromium.org2011-12-202-12/+12
| | | | | | | | | the WebContents interface. BUG=98716 Review URL: http://codereview.chromium.org/9007026 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@115200 0039d316-1c4b-4281-b951-d872f2087c98
* Start creating the content::WebPage interface around TabContents. This is ↵jam@chromium.org2011-12-201-4/+4
| | | | | | | | | | | just the start of moving 4 very commonly used unix_hacker functions into CamelCase. I've added the interface with just these methods, it's not actually used yet. The long term plan is that TabContentsWrapper would become TabContents, while TabContents will become WebPageImpl BUG=98716 Review URL: http://codereview.chromium.org/8997020 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@115043 0039d316-1c4b-4281-b951-d872f2087c98
* Added a smoke test for DevTools extension API (detailed API tests are in WebKit)caseq@google.com2011-12-191-6/+19
| | | | | | | | | | | TEST=yes BUG=none Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=114962 Review URL: http://codereview.chromium.org/8899008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@114983 0039d316-1c4b-4281-b951-d872f2087c98
* Revert "Added a smoke test for DevTools extension API (detailed API tests ↵caseq@google.com2011-12-191-20/+7
| | | | | | | | | | | | | | are in WebKit)" This caused PauseInSharedWorkerInitialization to fail on linux. BUG=none TEST=none TBR=yurys,finnur Review URL: http://codereview.chromium.org/8995006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@114979 0039d316-1c4b-4281-b951-d872f2087c98
* Added a smoke test for DevTools extension API (detailed API tests are in WebKit)caseq@google.com2011-12-191-7/+20
| | | | | | | | | TEST=yes BUG=none Review URL: http://codereview.chromium.org/8899008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@114962 0039d316-1c4b-4281-b951-d872f2087c98
* Clean up TCW, make it solely a hub for 1:1 observer/helper objects.avi@chromium.org2011-12-122-23/+33
| | | | | | | | | BUG=105872 TEST=no functional change Review URL: http://codereview.chromium.org/8892011 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@114017 0039d316-1c4b-4281-b951-d872f2087c98
* Removing MessageLoop::QuitTask() from chrome/dcheng@chromium.org2011-12-101-3/+3
| | | | | | | | | | | | base/task.h is going away and being replaced by base/bind.h and base/callback.h. This CL was automatically generated by sed. BUG=none TEST=none Review URL: http://codereview.chromium.org/8873032 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@113896 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 113071 - Revert 113047 - Make ExtensionService use ExtensionSet.yoz@chromium.org2011-12-061-2/+2
| | | | | | | | | | | | | | | | | | | ExtensionManagementApiBrowserTest.LaunchApp fixed by r113141. Make ExtensionSet iterate like a set instead of a map. BUG=104091 TEST=existing tests Review URL: http://codereview.chromium.org/8733004 TBR=yoz@chromium.org Review URL: http://codereview.chromium.org/8789018 TBR=yoz@chromium.org Review URL: http://codereview.chromium.org/8822021 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@113233 0039d316-1c4b-4281-b951-d872f2087c98
* Flaky on Mac: DevToolsSanityTest.TestReattachAfterCrash.joi@chromium.org2011-12-061-1/+1
| | | | | | | | | | | | Was already marked Flaky for Linux. Now failing ~5-10% of the time on Mac as well starting Friday December 2nd, first failure has Chrome blamelist 112587:112582. TBR=yurys@chromium.org BUG=103539 Review URL: http://codereview.chromium.org/8822011 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@113209 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 113047 - Make ExtensionService use ExtensionSet.yoz@chromium.org2011-12-061-2/+2
| | | | | | | | | | | | | | | | It broke ExtensionManagementApiBrowserTest.LaunchApp. Make ExtensionSet iterate like a set instead of a map. BUG=104091 TEST=existing tests Review URL: http://codereview.chromium.org/8733004 TBR=yoz@chromium.org Review URL: http://codereview.chromium.org/8789018 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@113071 0039d316-1c4b-4281-b951-d872f2087c98
* Make ExtensionService use ExtensionSet.yoz@chromium.org2011-12-051-2/+2
| | | | | | | | | | | Make ExtensionSet iterate like a set instead of a map. BUG=104091 TEST=existing tests Review URL: http://codereview.chromium.org/8733004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@113047 0039d316-1c4b-4281-b951-d872f2087c98
* Change NavigationController::LoadURL to take a Referrer class instead of a ↵jochen@chromium.org2011-12-051-1/+1
| | | | | | | | | | | GURL as referrer BUG=105028 TEST=no functional change Review URL: http://codereview.chromium.org/8785004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@112968 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 112688 - DevTools: remove devtools interactive tests suppressions.shess@chromium.org2011-12-021-2/+6
| | | | | | | | | | | BUG=106114 Review URL: http://codereview.chromium.org/8770038 TBR=pfeldman@chromium.org Review URL: http://codereview.chromium.org/8775068 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@112772 0039d316-1c4b-4281-b951-d872f2087c98
* Remove deprecated TabContentsDelegate::OpenURLFromTab variantjochen@chromium.org2011-12-022-19/+0
| | | | | | | | | | BUG=105028 TEST=no functional change Review URL: http://codereview.chromium.org/8772041 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@112706 0039d316-1c4b-4281-b951-d872f2087c98
* DevTools: remove devtools interactive tests suppressions.pfeldman@chromium.org2011-12-021-6/+2
| | | | | | | | BUG=106114 Review URL: http://codereview.chromium.org/8770038 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@112688 0039d316-1c4b-4281-b951-d872f2087c98
* Add content API for DevTools HTTP handleryurys@chromium.org2011-12-024-16/+24
| | | | | | | | | BUG=104625 TEST=Existing tests Review URL: http://codereview.chromium.org/8554008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@112662 0039d316-1c4b-4281-b951-d872f2087c98
* Suppression of DevToolsSanityTest TestPauseWhenLoadingDevTools ↵scheib@chromium.org2011-12-021-2/+8
| | | | | | | | | | PauseInSharedWorkerInitialization TBR=pfeldman Review URL: http://codereview.chromium.org/8775032 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@112581 0039d316-1c4b-4281-b951-d872f2087c98
* Define DevTools content APIyurys@chromium.org2011-12-013-55/+62
| | | | | | | | | | | | | | | The API consists of the following parts: * DevToolsManager routes messages between devtools agents and clients * DevToolsAgentHost provides an abstract interface to the debuggee, currently it is either RenderViewHost or Shared Worker. Client can obtain DevToolsAgentHost from DevToolsAgentHostRegistry. * DevToolsClientHost is an API that should be implemented by DevTools front-end. There is a default Chromium implementation living in chrome/ and a remote debugging server which also implements this interface. Clients can extend it in order to provide custom front-end. There is a default DevTools front-end implementation and content/ provides a way for creating corresponding DevToolsClientHost by means of DevToolsClientHost::CreateDevToolsFrontendHost. The embedder just needs to provide a concrete delegate. * This patch also removes DevToolsHost_ForwardToAgent and DevToolsHost_ForwardToClient IPC messages which were used to forward only one message. DevTools IPC messages are now hidden behind the devtools content API. BUG=104625 TEST=Existing tests Review URL: http://codereview.chromium.org/8549022 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@112473 0039d316-1c4b-4281-b951-d872f2087c98
* Move DevTools front-end IPC message handling to contentyurys@chromium.org2011-12-012-68/+53
| | | | | | | | | BUG=104625 TEST=Existing tests Review URL: http://codereview.chromium.org/8729024 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@112415 0039d316-1c4b-4281-b951-d872f2087c98
* Move the ProcessType enum out to its own file. This is in preparation for ↵jam@chromium.org2011-12-011-3/+2
| | | | | | | | | getting rid of ChildProcessInfo. BUG=98716 Review URL: http://codereview.chromium.org/8760011 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@112353 0039d316-1c4b-4281-b951-d872f2087c98
* base::Bind: Implement a 1-arity CancelableCallback and use this to implementjhawkins@chromium.org2011-11-301-2/+2
| | | | | | | | | | | | | net::CancelableCompletionClosure. BUG=none TEST=none R=ajwong@chromium.org Review URL: http://codereview.chromium.org/8662047 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@112279 0039d316-1c4b-4281-b951-d872f2087c98
* Reenable NavigateToPrerenderedPageWhenDevToolsAttached after WebKit r101176yurys@chromium.org2011-11-291-2/+1
| | | | | | | | | | | | Reenable PrerenderBrowserTest.NavigateToPrerenderedPageWhenDevToolsAttached and DevToolsSanityTest.TestReattachAfterCrash The problem was fixed in WebKit r101176. BUG=105433 TEST= Review URL: http://codereview.chromium.org/8720004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@111901 0039d316-1c4b-4281-b951-d872f2087c98
* base::Bind: Convert devtools_sanity_unittest.cc.jhawkins@chromium.org2011-11-281-36/+16
| | | | | | | | | | | BUG=none TEST=none R=csilv@chromium.org Review URL: http://codereview.chromium.org/8715001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@111723 0039d316-1c4b-4281-b951-d872f2087c98
* Revert InspectSharedWorker and PauseInSharedWorkerInitialization tests to ↵caseq@google.com2011-11-281-2/+16
| | | | | | | | | | | | | pass/flaky as the reason of their failure was fixed in WebKit r101047 BUG=89845,100538 TEST=yes, it is! Review URL: http://codereview.chromium.org/8700005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@111684 0039d316-1c4b-4281-b951-d872f2087c98
* Move RenderViewHostObserver to content/public/browser/ and put it into the ↵tfarina@chromium.org2011-11-252-8/+7
| | | | | | | | | | | content namespace. BUG=98716 R=jam@chromium.org Review URL: http://codereview.chromium.org/8676032 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@111604 0039d316-1c4b-4281-b951-d872f2087c98