New Sad Tab strings.

chrome/browser/ui/sad_tab.h

 // Copyright 2012 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.
 
 #ifndef CHROME_BROWSER_UI_SAD_TAB_H_
 #define CHROME_BROWSER_UI_SAD_TAB_H_
 
 #include "base/process/kill.h"
 #include "chrome/browser/ui/sad_tab_types.h"
 
@@ skipping 20 matching lines @@
   virtual ~SadTab() {}
 
   // These functions return resource string IDs for UI text. They may be
   // different for each sad tab. (Right now, the first sad tab in a session
   // suggests reloading and subsequent ones suggest sending feedback.)
   int GetTitle();
   int GetMessage();
   int GetButtonTitle();
   int GetHelpLinkTitle();
 
+  // Returns the resource string Id for bullet point |bullet_id| or 0 if no
+  // bullet should be added for the |bullet_id|. Bullet ids count from 0.
+  int GetBulletText(size_t bullet_id);

[sky, 2017/05/14 13:46:16]

optional: consider naming this Submessage. While the text does in fact have a
bullet in front of it, the name 'bullettext' doesn't really describe what it is.

[Will Harris, 2017/05/17 18:25:19]

Done.

+
   // Returns the target of the "Learn more" link. Use it for the context menu
   // and to show the URL on hover, but call PerformAction() for regular clicks.
   const char* GetHelpLinkURL();
 
   // Virtual for testing.
   virtual void RecordFirstPaint();
   virtual void PerformAction(Action);
 
  protected:
   SadTab(content::WebContents* web_contents, SadTabKind kind);
 
  private:
   content::WebContents* web_contents_;
   SadTabKind kind_;
   bool show_feedback_button_;
   bool recorded_paint_;
+  bool other_tabs_open_;

[sky, 2017/05/14 13:46:16]

Please add description. AFAICT this is always true. Is it needed?

[Will Harris, 2017/05/17 18:25:19]

I removed this and now actually detect other tabs being open.

 
   DISALLOW_COPY_AND_ASSIGN(SadTab);
 };
 
 }  // namespace chrome
 
 #endif  // CHROME_BROWSER_UI_SAD_TAB_H_