Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(366)

Side by Side Diff: chrome/browser/ui/webui/options/extension_settings_handler.cc

Issue 8469015: Switch BackgroundContents to use TabContents instead of RenderViewHost. (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: fix unittest Created 9 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2011 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/ui/webui/options/extension_settings_handler.h" 5 #include "chrome/browser/ui/webui/options/extension_settings_handler.h"
6 6
7 #include "base/auto_reset.h" 7 #include "base/auto_reset.h"
8 #include "base/base64.h" 8 #include "base/base64.h"
9 #include "base/bind.h" 9 #include "base/bind.h"
10 #include "base/bind_helpers.h" 10 #include "base/bind_helpers.h"
(...skipping 598 matching lines...) Expand 10 before | Expand all | Expand 10 after
609 // Fall through. 609 // Fall through.
610 case content::NOTIFICATION_RENDER_VIEW_HOST_CREATED: 610 case content::NOTIFICATION_RENDER_VIEW_HOST_CREATED:
611 source_profile = Profile::FromBrowserContext( 611 source_profile = Profile::FromBrowserContext(
612 content::Source<RenderViewHost>(source)->site_instance()-> 612 content::Source<RenderViewHost>(source)->site_instance()->
613 browsing_instance()->browser_context()); 613 browsing_instance()->browser_context());
614 if (!profile->IsSameProfile(source_profile)) 614 if (!profile->IsSameProfile(source_profile))
615 return; 615 return;
616 MaybeUpdateAfterNotification(); 616 MaybeUpdateAfterNotification();
617 break; 617 break;
618 case chrome::NOTIFICATION_BACKGROUND_CONTENTS_DELETED: 618 case chrome::NOTIFICATION_BACKGROUND_CONTENTS_DELETED:
619 deleting_rvh_ = 619 deleting_rvh_ = content::Details<BackgroundContents>(details)->
620 content::Details<BackgroundContents>(details)->render_view_host(); 620 tab_contents()->render_view_host();
621 // Fall through. 621 // Fall through.
622 case chrome::NOTIFICATION_BACKGROUND_CONTENTS_NAVIGATED: 622 case chrome::NOTIFICATION_BACKGROUND_CONTENTS_NAVIGATED:
623 case chrome::NOTIFICATION_EXTENSION_HOST_CREATED: 623 case chrome::NOTIFICATION_EXTENSION_HOST_CREATED:
624 source_profile = content::Source<Profile>(source).ptr(); 624 source_profile = content::Source<Profile>(source).ptr();
625 if (!profile->IsSameProfile(source_profile)) 625 if (!profile->IsSameProfile(source_profile))
626 return; 626 return;
627 MaybeUpdateAfterNotification(); 627 MaybeUpdateAfterNotification();
628 break; 628 break;
629 case chrome::NOTIFICATION_EXTENSION_LOADED: 629 case chrome::NOTIFICATION_EXTENSION_LOADED:
630 case chrome::NOTIFICATION_EXTENSION_UNLOADED: 630 case chrome::NOTIFICATION_EXTENSION_UNLOADED:
(...skipping 145 matching lines...) Expand 10 before | Expand all | Expand 10 after
776 chrome::VIEW_TYPE_EXTENSION_DIALOG == host_type) 776 chrome::VIEW_TYPE_EXTENSION_DIALOG == host_type)
777 continue; 777 continue;
778 778
779 GURL url = host->delegate()->GetURL(); 779 GURL url = host->delegate()->GetURL();
780 RenderProcessHost* process = host->process(); 780 RenderProcessHost* process = host->process();
781 result->push_back( 781 result->push_back(
782 ExtensionPage(url, process->id(), host->routing_id(), 782 ExtensionPage(url, process->id(), host->routing_id(),
783 process->browser_context()->IsOffTheRecord())); 783 process->browser_context()->IsOffTheRecord()));
784 } 784 }
785 } 785 }
OLDNEW
« no previous file with comments | « chrome/browser/ui/search_engines/search_engine_tab_helper.cc ('k') | content/browser/intents/intent_injector.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698