Restrict width of the Subresource Filtering bubble to 320px.

chrome/browser/ui/views/content_setting_bubble_contents.cc

Index: chrome/browser/ui/views/content_setting_bubble_contents.cc
diff --git a/chrome/browser/ui/views/content_setting_bubble_contents.cc b/chrome/browser/ui/views/content_setting_bubble_contents.cc
index 198251d336ccbce2711934eb5376e1807004f9ad..ea808d84e0e5fba09d901c9a0412771b1daff50f 100644
--- a/chrome/browser/ui/views/content_setting_bubble_contents.cc
+++ b/chrome/browser/ui/views/content_setting_bubble_contents.cc
@@ -53,6 +53,10 @@ const int kMaxContentsWidth = 500;
 // narrow bubbles with lots of line-wrapping.
 const int kMinMultiLineContentsWidth = 250;
 
+// Subresource filter bubble has ling explanation string, so the width should be

[msw, 2016/09/15 18:29:03]

nit: 'a long' consider: 'Maximum width of the subresource filter prompt, which
has a long string.'

[melandory, 2016/09/15 19:47:43]

Done.

+// bounded.
+const int kMaxSubresourceFilterPromptWidth = 320;
+
 }  // namespace
 
 using content::PluginService;
@@ -182,7 +186,11 @@ gfx::Size ContentSettingBubbleContents::GetPreferredSize() const {
        (kMinMultiLineContentsWidth > preferred_size.width()))
           ? kMinMultiLineContentsWidth
           : preferred_size.width();
-  preferred_size.set_width(std::min(preferred_width, kMaxContentsWidth));
+  preferred_size.set_width(

[msw, 2016/09/15 18:29:03]

nit: consider:
  preferred_size.set_width(std::min(preferred_width, kMaxContentsWidth));
  // Clamp the subresource filter prompt even further, because ... <insert text
explaining why this bubble should be less wide than other content bubbles, which
isn't really obvious>
  if (content_setting_bubble_model_->AsSubresourceFilterBubbleModel())
    preferred_size.set_width(std::min(preferred_width,
kMaxSubresourceFilterPromptWidth));

[melandory, 2016/09/15 19:47:43]

Done.

+      std::min(preferred_width,
+               content_setting_bubble_model_->AsSubresourceFilterBubbleModel()
+                   ? kMaxSubresourceFilterPromptWidth
+                   : kMaxContentsWidth));
   return preferred_size;
 }