Show a custom message in the page info bubble for view-source URLs

chrome/browser/ui/cocoa/website_settings/website_settings_bubble_controller.mm

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #import "chrome/browser/ui/cocoa/website_settings/website_settings_bubble_controller.h"
 
 #import <AppKit/AppKit.h>
 
 #include <cmath>
 
@@ skipping 165 matching lines @@
 }
 
 @end
 
 @implementation WebsiteSettingsBubbleController
 
 - (CGFloat)defaultWindowWidth {
   return kDefaultWindowWidth;
 }
 
+bool IsInternalURL(const GURL& url) {
+  return url.SchemeIs(content::kChromeUIScheme) ||
+         url.SchemeIs(extensions::kExtensionScheme) ||
+         url.SchemeIs(content::kViewSourceScheme);
+}
+
 - (id)initWithParentWindow:(NSWindow*)parentWindow
     websiteSettingsUIBridge:(WebsiteSettingsUIBridge*)bridge
                 webContents:(content::WebContents*)webContents
-                 bubbleType:(BubbleType)bubbleType
+                        url:(const GURL&)url
          isDevToolsDisabled:(BOOL)isDevToolsDisabled {
   DCHECK(parentWindow);
 
   webContents_ = webContents;
   permissionsPresent_ = NO;
   isDevToolsDisabled_ = isDevToolsDisabled;
+  url_ = url;
 
   // Use an arbitrary height; it will be changed in performLayout.
   NSRect contentRect = NSMakeRect(0, 0, [self defaultWindowWidth], 1);
   // Create an empty window into which content is placed.
   base::scoped_nsobject<InfoBubbleWindow> window(
       [[InfoBubbleWindow alloc] initWithContentRect:contentRect
                                           styleMask:NSBorderlessWindowMask
                                             backing:NSBackingStoreBuffered
                                               defer:NO]);
 
   if ((self = [super initWithWindow:window.get()
                        parentWindow:parentWindow
                          anchoredAt:NSZeroPoint])) {
     [[self bubble] setArrowLocation:info_bubble::kTopLeft];
 
     // Create the container view that uses flipped coordinates.
     NSRect contentFrame = NSMakeRect(0, 0, [self defaultWindowWidth], 300);
     contentView_.reset(
         [[FlippedView alloc] initWithFrame:contentFrame]);
 
     // Replace the window's content.
     [[[self window] contentView] setSubviews:
         [NSArray arrayWithObject:contentView_.get()]];
 
-    if (bubbleType == INTERNAL_PAGE)
-      [self initializeContentsForInternalPage:false];
-    else if (bubbleType == EXTENSION_PAGE)
-      [self initializeContentsForInternalPage:true];
-    else
+    if (IsInternalURL(url_)) {
+      [self initializeContentsForInternalPage:url_];
+    } else {
       [self initializeContents];
+    }
 
     bridge_.reset(bridge);
     bridge_->set_bubble_controller(self);
   }
   return self;
 }
 
 - (void)windowWillClose:(NSNotification*)notification {
   if (presenter_.get())
     presenter_->OnUIClosing();
   presenter_.reset();
   [super windowWillClose:notification];
 }
 
 - (void)setPresenter:(WebsiteSettings*)presenter {
   presenter_.reset(presenter);
 }
 
 // Create the subviews for the bubble for internal Chrome pages.
-- (void)initializeContentsForInternalPage:(BOOL)isExtensionPage {
+- (void)initializeContentsForInternalPage:(const GURL&)url {
   ui::ResourceBundle& rb = ui::ResourceBundle::GetSharedInstance();
 
-  NSPoint controlOrigin = NSMakePoint(
-      kInternalPageFramePadding,
-      kInternalPageFramePadding + info_bubble::kBubbleArrowHeight);
-  NSImage* productLogoImage =
-      rb.GetNativeImageNamed(isExtensionPage ? IDR_PLUGINS_FAVICON
-                                             : IDR_PRODUCT_LOGO_16)
-          .ToNSImage();
+  int text = IDR_PRODUCT_LOGO_16;
+  int icon = IDR_PLUGINS_FAVICON;
+  if (url.SchemeIs(extensions::kExtensionScheme)) {
+    text = IDS_PAGE_INFO_EXTENSION_PAGE;
+    icon = IDR_PLUGINS_FAVICON;
+  } else if (url.SchemeIs(content::kViewSourceScheme)) {
+    // view-source scheme uses the same icon as other internal pages.
+    text = IDS_PAGE_INFO_VIEW_SOURCE_PAGE;
+  }

[felt, 2016/08/12 18:58:39]

If I'm reading this right, the default is the content::kChromeUIScheme because
this can only be reached for one of those three schemes. However it is not
explicitly called out here. I'm worried that in a future CL someone will add a
new scheme to IsInternalURL, and kChromeUIScheme will get the wrong treatment.

Would it be possible to keep the concept of a BubbleType so that you can have
switch statements, making sure all the BubbleTypes are handled?

At the very least, please be explicit about the kChromeUIScheme here.

[meacer, 2016/08/12 21:13:49]

I removed BubbleType for two reasons:
- Corresponding views code uses URLs, so this is more consistent
- BubbleType is only used here, and it seemed overkill to add an enum for only
this reason. (it was me who added it in a previous CL)

Hopefully it's clearer now with the NOTREACHED.

+
+  NSPoint controlOrigin =
+      NSMakePoint(kInternalPageFramePadding,
+                  kInternalPageFramePadding + info_bubble::kBubbleArrowHeight);
+  NSImage* productLogoImage = rb.GetNativeImageNamed(icon).ToNSImage();
   NSImageView* imageView = [self addImageWithSize:[productLogoImage size]
                                            toView:contentView_
                                           atPoint:controlOrigin];
   [imageView setImage:productLogoImage];
 
   NSRect imageFrame = [imageView frame];
   controlOrigin.x += NSWidth(imageFrame) + kInternalPageImageSpacing;
-  base::string16 text =
-      l10n_util::GetStringUTF16(isExtensionPage ? IDS_PAGE_INFO_EXTENSION_PAGE
-                                                : IDS_PAGE_INFO_INTERNAL_PAGE);
-  NSTextField* textField = [self addText:text
+  NSTextField* textField = [self addText:l10n_util::GetStringUTF16(text)
                                 withSize:[NSFont smallSystemFontSize]
                                     bold:NO
                                   toView:contentView_
                                  atPoint:controlOrigin];
   // Center the image vertically with the text. Previously this code centered
   // the text vertically while holding the image in place. That produced correct
   // results when the image, at 26x26, was taller than (or just slightly
   // shorter) than the text, but produced incorrect results once the icon
   // shrank to 16x16. The icon should now always be shorter than the text.
   // See crbug.com/572044 .
@@ skipping 851 matching lines @@
         web_contents, virtual_url, security_info);
     return;
   }
 
   // Don't show the popup if it's already being shown. Since this method is
   // called each time the location icon is clicked, each click toggles the popup
   // in and out.
   if (g_is_popup_showing)
     return;
 
-  BubbleType bubble_type = WEB_PAGE;
-  if (virtual_url.SchemeIs(content::kChromeUIScheme))
-    bubble_type = INTERNAL_PAGE;
-  else if (virtual_url.SchemeIs(extensions::kExtensionScheme))
-    bubble_type = EXTENSION_PAGE;
-
   // Create the bridge. This will be owned by the bubble controller.
   WebsiteSettingsUIBridge* bridge = new WebsiteSettingsUIBridge(web_contents);
 
   bool is_devtools_disabled =
       profile->GetPrefs()->GetBoolean(prefs::kDevToolsDisabled);
 
   // Create the bubble controller. It will dealloc itself when it closes,
   // resetting |g_is_popup_showing|.
   WebsiteSettingsBubbleController* bubble_controller =
       [[WebsiteSettingsBubbleController alloc]
              initWithParentWindow:parent
           websiteSettingsUIBridge:bridge
                       webContents:web_contents
-                       bubbleType:bubble_type
+                              url:virtual_url
                isDevToolsDisabled:is_devtools_disabled];
 
-  if (bubble_type == WEB_PAGE) {
+  if (!IsInternalURL(virtual_url)) {
     // Initialize the presenter, which holds the model and controls the UI.
     // This is also owned by the bubble controller.
     WebsiteSettings* presenter = new WebsiteSettings(
         bridge, profile,
         TabSpecificContentSettings::FromWebContents(web_contents), web_contents,
         virtual_url, security_info, content::CertStore::GetInstance());
     [bubble_controller setPresenter:presenter];
   }
 
   [bubble_controller showWindow:nil];
@@ skipping 19 matching lines @@
 void WebsiteSettingsUIBridge::SetPermissionInfo(
     const PermissionInfoList& permission_info_list,
     const ChosenObjectInfoList& chosen_object_info_list) {
   [bubble_controller_ setPermissionInfo:permission_info_list
                        andChosenObjects:chosen_object_info_list];
 }
 
 void WebsiteSettingsUIBridge::SetSelectedTab(TabId tab_id) {
   // TODO(lgarron): Remove this from the interface. (crbug.com/571533)
 }