Home | History | Annotate | Download | only in toolbar
      1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
      2 // Use of this source code is governed by a BSD-style license that can be
      3 // found in the LICENSE file.
      4 
      5 #include "chrome/browser/ui/toolbar/back_forward_menu_model.h"
      6 
      7 #include "base/path_service.h"
      8 #include "base/strings/string16.h"
      9 #include "base/strings/string_util.h"
     10 #include "base/strings/utf_string_conversions.h"
     11 #include "base/time/time.h"
     12 #include "chrome/browser/favicon/favicon_service_factory.h"
     13 #include "chrome/browser/history/history_service.h"
     14 #include "chrome/browser/history/history_service_factory.h"
     15 #include "chrome/browser/profiles/profile_manager.h"
     16 #include "chrome/browser/ui/browser.h"
     17 #include "chrome/browser/ui/browser_tabstrip.h"
     18 #include "chrome/browser/ui/tabs/tab_strip_model.h"
     19 #include "chrome/common/url_constants.h"
     20 #include "chrome/test/base/chrome_render_view_host_test_harness.h"
     21 #include "chrome/test/base/test_browser_window.h"
     22 #include "chrome/test/base/testing_profile.h"
     23 #include "content/public/browser/navigation_controller.h"
     24 #include "content/public/browser/navigation_entry.h"
     25 #include "content/public/browser/web_contents.h"
     26 #include "content/public/test/web_contents_tester.h"
     27 #include "testing/gtest/include/gtest/gtest.h"
     28 #include "third_party/skia/include/core/SkBitmap.h"
     29 #include "ui/gfx/codec/png_codec.h"
     30 
     31 using base::ASCIIToUTF16;
     32 using content::WebContentsTester;
     33 
     34 namespace {
     35 
     36 // Creates a bitmap of the specified color.
     37 SkBitmap CreateBitmap(SkColor color) {
     38   SkBitmap bitmap;
     39   bitmap.allocN32Pixels(16, 16);
     40   bitmap.eraseColor(color);
     41   return bitmap;
     42 }
     43 
     44 class FaviconDelegate : public ui::MenuModelDelegate {
     45  public:
     46   FaviconDelegate() : was_called_(false) {}
     47 
     48   virtual void OnIconChanged(int model_index) OVERRIDE {
     49     was_called_ = true;
     50     base::MessageLoop::current()->Quit();
     51   }
     52 
     53   bool was_called() const { return was_called_; }
     54 
     55  private:
     56   bool was_called_;
     57 
     58   DISALLOW_COPY_AND_ASSIGN(FaviconDelegate);
     59 };
     60 
     61 }  // namespace
     62 
     63 class BackFwdMenuModelTest : public ChromeRenderViewHostTestHarness {
     64  public:
     65   void ValidateModel(BackForwardMenuModel* model, int history_items,
     66                      int chapter_stops) {
     67     int h = std::min(BackForwardMenuModel::kMaxHistoryItems, history_items);
     68     int c = std::min(BackForwardMenuModel::kMaxChapterStops, chapter_stops);
     69     EXPECT_EQ(h, model->GetHistoryItemCount());
     70     EXPECT_EQ(c, model->GetChapterStopCount(h));
     71     if (h > 0)
     72       h += 2;  // Separator and View History link.
     73     if (c > 0)
     74       ++c;
     75     EXPECT_EQ(h + c, model->GetItemCount());
     76   }
     77 
     78   void LoadURLAndUpdateState(const char* url, const char* title) {
     79     NavigateAndCommit(GURL(url));
     80     controller().GetLastCommittedEntry()->SetTitle(base::UTF8ToUTF16(title));
     81   }
     82 
     83   // Navigate back or forward the given amount and commits the entry (which
     84   // will be pending after we ask to navigate there).
     85   void NavigateToOffset(int offset) {
     86     controller().GoToOffset(offset);
     87     WebContentsTester::For(web_contents())->CommitPendingNavigation();
     88   }
     89 
     90   // Same as NavigateToOffset but goes to an absolute index.
     91   void NavigateToIndex(int index) {
     92     controller().GoToIndex(index);
     93     WebContentsTester::For(web_contents())->CommitPendingNavigation();
     94   }
     95 
     96   // Goes back/forward and commits the load.
     97   void GoBack() {
     98     controller().GoBack();
     99     WebContentsTester::For(web_contents())->CommitPendingNavigation();
    100   }
    101   void GoForward() {
    102     controller().GoForward();
    103     WebContentsTester::For(web_contents())->CommitPendingNavigation();
    104   }
    105 };
    106 
    107 TEST_F(BackFwdMenuModelTest, BasicCase) {
    108   scoped_ptr<BackForwardMenuModel> back_model(new BackForwardMenuModel(
    109       NULL, BackForwardMenuModel::BACKWARD_MENU));
    110   back_model->set_test_web_contents(web_contents());
    111 
    112   scoped_ptr<BackForwardMenuModel> forward_model(new BackForwardMenuModel(
    113       NULL, BackForwardMenuModel::FORWARD_MENU));
    114   forward_model->set_test_web_contents(web_contents());
    115 
    116   EXPECT_EQ(0, back_model->GetItemCount());
    117   EXPECT_EQ(0, forward_model->GetItemCount());
    118   EXPECT_FALSE(back_model->ItemHasCommand(1));
    119 
    120   // Seed the controller with a few URLs
    121   LoadURLAndUpdateState("http://www.a.com/1", "A1");
    122   LoadURLAndUpdateState("http://www.a.com/2", "A2");
    123   LoadURLAndUpdateState("http://www.a.com/3", "A3");
    124   LoadURLAndUpdateState("http://www.b.com/1", "B1");
    125   LoadURLAndUpdateState("http://www.b.com/2", "B2");
    126   LoadURLAndUpdateState("http://www.c.com/1", "C1");
    127   LoadURLAndUpdateState("http://www.c.com/2", "C2");
    128   LoadURLAndUpdateState("http://www.c.com/3", "C3");
    129 
    130   // There're two more items here: a separator and a "Show Full History".
    131   EXPECT_EQ(9, back_model->GetItemCount());
    132   EXPECT_EQ(0, forward_model->GetItemCount());
    133   EXPECT_EQ(ASCIIToUTF16("C2"), back_model->GetLabelAt(0));
    134   EXPECT_EQ(ASCIIToUTF16("A1"), back_model->GetLabelAt(6));
    135   EXPECT_EQ(back_model->GetShowFullHistoryLabel(),
    136             back_model->GetLabelAt(8));
    137 
    138   EXPECT_TRUE(back_model->ItemHasCommand(0));
    139   EXPECT_TRUE(back_model->ItemHasCommand(6));
    140   EXPECT_TRUE(back_model->IsSeparator(7));
    141   EXPECT_TRUE(back_model->ItemHasCommand(8));
    142   EXPECT_FALSE(back_model->ItemHasCommand(9));
    143   EXPECT_FALSE(back_model->ItemHasCommand(9));
    144 
    145   NavigateToOffset(-7);
    146 
    147   EXPECT_EQ(0, back_model->GetItemCount());
    148   EXPECT_EQ(9, forward_model->GetItemCount());
    149   EXPECT_EQ(ASCIIToUTF16("A2"), forward_model->GetLabelAt(0));
    150   EXPECT_EQ(ASCIIToUTF16("C3"), forward_model->GetLabelAt(6));
    151   EXPECT_EQ(forward_model->GetShowFullHistoryLabel(),
    152             forward_model->GetLabelAt(8));
    153 
    154   EXPECT_TRUE(forward_model->ItemHasCommand(0));
    155   EXPECT_TRUE(forward_model->ItemHasCommand(6));
    156   EXPECT_TRUE(forward_model->IsSeparator(7));
    157   EXPECT_TRUE(forward_model->ItemHasCommand(8));
    158   EXPECT_FALSE(forward_model->ItemHasCommand(7));
    159   EXPECT_FALSE(forward_model->ItemHasCommand(9));
    160 
    161   NavigateToOffset(4);
    162 
    163   EXPECT_EQ(6, back_model->GetItemCount());
    164   EXPECT_EQ(5, forward_model->GetItemCount());
    165   EXPECT_EQ(ASCIIToUTF16("B1"), back_model->GetLabelAt(0));
    166   EXPECT_EQ(ASCIIToUTF16("A1"), back_model->GetLabelAt(3));
    167   EXPECT_EQ(back_model->GetShowFullHistoryLabel(),
    168             back_model->GetLabelAt(5));
    169   EXPECT_EQ(ASCIIToUTF16("C1"), forward_model->GetLabelAt(0));
    170   EXPECT_EQ(ASCIIToUTF16("C3"), forward_model->GetLabelAt(2));
    171   EXPECT_EQ(forward_model->GetShowFullHistoryLabel(),
    172             forward_model->GetLabelAt(4));
    173 }
    174 
    175 TEST_F(BackFwdMenuModelTest, MaxItemsTest) {
    176   scoped_ptr<BackForwardMenuModel> back_model(new BackForwardMenuModel(
    177       NULL, BackForwardMenuModel::BACKWARD_MENU));
    178   back_model->set_test_web_contents(web_contents());
    179 
    180   scoped_ptr<BackForwardMenuModel> forward_model(new BackForwardMenuModel(
    181       NULL, BackForwardMenuModel::FORWARD_MENU));
    182   forward_model->set_test_web_contents(web_contents());
    183 
    184   // Seed the controller with 32 URLs
    185   LoadURLAndUpdateState("http://www.a.com/1", "A1");
    186   LoadURLAndUpdateState("http://www.a.com/2", "A2");
    187   LoadURLAndUpdateState("http://www.a.com/3", "A3");
    188   LoadURLAndUpdateState("http://www.b.com/1", "B1");
    189   LoadURLAndUpdateState("http://www.b.com/2", "B2");
    190   LoadURLAndUpdateState("http://www.b.com/3", "B3");
    191   LoadURLAndUpdateState("http://www.c.com/1", "C1");
    192   LoadURLAndUpdateState("http://www.c.com/2", "C2");
    193   LoadURLAndUpdateState("http://www.c.com/3", "C3");
    194   LoadURLAndUpdateState("http://www.d.com/1", "D1");
    195   LoadURLAndUpdateState("http://www.d.com/2", "D2");
    196   LoadURLAndUpdateState("http://www.d.com/3", "D3");
    197   LoadURLAndUpdateState("http://www.e.com/1", "E1");
    198   LoadURLAndUpdateState("http://www.e.com/2", "E2");
    199   LoadURLAndUpdateState("http://www.e.com/3", "E3");
    200   LoadURLAndUpdateState("http://www.f.com/1", "F1");
    201   LoadURLAndUpdateState("http://www.f.com/2", "F2");
    202   LoadURLAndUpdateState("http://www.f.com/3", "F3");
    203   LoadURLAndUpdateState("http://www.g.com/1", "G1");
    204   LoadURLAndUpdateState("http://www.g.com/2", "G2");
    205   LoadURLAndUpdateState("http://www.g.com/3", "G3");
    206   LoadURLAndUpdateState("http://www.h.com/1", "H1");
    207   LoadURLAndUpdateState("http://www.h.com/2", "H2");
    208   LoadURLAndUpdateState("http://www.h.com/3", "H3");
    209   LoadURLAndUpdateState("http://www.i.com/1", "I1");
    210   LoadURLAndUpdateState("http://www.i.com/2", "I2");
    211   LoadURLAndUpdateState("http://www.i.com/3", "I3");
    212   LoadURLAndUpdateState("http://www.j.com/1", "J1");
    213   LoadURLAndUpdateState("http://www.j.com/2", "J2");
    214   LoadURLAndUpdateState("http://www.j.com/3", "J3");
    215   LoadURLAndUpdateState("http://www.k.com/1", "K1");
    216   LoadURLAndUpdateState("http://www.k.com/2", "K2");
    217 
    218   // Also there're two more for a separator and a "Show Full History".
    219   int chapter_stop_offset = 6;
    220   EXPECT_EQ(BackForwardMenuModel::kMaxHistoryItems + 2 + chapter_stop_offset,
    221             back_model->GetItemCount());
    222   EXPECT_EQ(0, forward_model->GetItemCount());
    223   EXPECT_EQ(ASCIIToUTF16("K1"), back_model->GetLabelAt(0));
    224   EXPECT_EQ(back_model->GetShowFullHistoryLabel(),
    225       back_model->GetLabelAt(BackForwardMenuModel::kMaxHistoryItems + 1 +
    226                                chapter_stop_offset));
    227 
    228   // Test for out of bounds (beyond Show Full History).
    229   EXPECT_FALSE(back_model->ItemHasCommand(
    230       BackForwardMenuModel::kMaxHistoryItems + chapter_stop_offset + 2));
    231 
    232   EXPECT_TRUE(back_model->ItemHasCommand(
    233               BackForwardMenuModel::kMaxHistoryItems - 1));
    234   EXPECT_TRUE(back_model->IsSeparator(
    235               BackForwardMenuModel::kMaxHistoryItems));
    236 
    237   NavigateToIndex(0);
    238 
    239   EXPECT_EQ(BackForwardMenuModel::kMaxHistoryItems + 2 + chapter_stop_offset,
    240             forward_model->GetItemCount());
    241   EXPECT_EQ(0, back_model->GetItemCount());
    242   EXPECT_EQ(ASCIIToUTF16("A2"), forward_model->GetLabelAt(0));
    243   EXPECT_EQ(forward_model->GetShowFullHistoryLabel(),
    244       forward_model->GetLabelAt(BackForwardMenuModel::kMaxHistoryItems + 1 +
    245                                     chapter_stop_offset));
    246 
    247   // Out of bounds
    248   EXPECT_FALSE(forward_model->ItemHasCommand(
    249       BackForwardMenuModel::kMaxHistoryItems + 2 + chapter_stop_offset));
    250 
    251   EXPECT_TRUE(forward_model->ItemHasCommand(
    252       BackForwardMenuModel::kMaxHistoryItems - 1));
    253   EXPECT_TRUE(forward_model->IsSeparator(
    254       BackForwardMenuModel::kMaxHistoryItems));
    255 }
    256 
    257 TEST_F(BackFwdMenuModelTest, ChapterStops) {
    258   scoped_ptr<BackForwardMenuModel> back_model(new BackForwardMenuModel(
    259     NULL, BackForwardMenuModel::BACKWARD_MENU));
    260   back_model->set_test_web_contents(web_contents());
    261 
    262   scoped_ptr<BackForwardMenuModel> forward_model(new BackForwardMenuModel(
    263       NULL, BackForwardMenuModel::FORWARD_MENU));
    264   forward_model->set_test_web_contents(web_contents());
    265 
    266   // Seed the controller with 32 URLs.
    267   int i = 0;
    268   LoadURLAndUpdateState("http://www.a.com/1", "A1");
    269   ValidateModel(back_model.get(), i++, 0);
    270   LoadURLAndUpdateState("http://www.a.com/2", "A2");
    271   ValidateModel(back_model.get(), i++, 0);
    272   LoadURLAndUpdateState("http://www.a.com/3", "A3");
    273   ValidateModel(back_model.get(), i++, 0);
    274   LoadURLAndUpdateState("http://www.b.com/1", "B1");
    275   ValidateModel(back_model.get(), i++, 0);
    276   LoadURLAndUpdateState("http://www.b.com/2", "B2");
    277   ValidateModel(back_model.get(), i++, 0);
    278   // i = 5
    279   LoadURLAndUpdateState("http://www.b.com/3", "B3");
    280   ValidateModel(back_model.get(), i++, 0);
    281   LoadURLAndUpdateState("http://www.c.com/1", "C1");
    282   ValidateModel(back_model.get(), i++, 0);
    283   LoadURLAndUpdateState("http://www.c.com/2", "C2");
    284   ValidateModel(back_model.get(), i++, 0);
    285   LoadURLAndUpdateState("http://www.c.com/3", "C3");
    286   ValidateModel(back_model.get(), i++, 0);
    287   LoadURLAndUpdateState("http://www.d.com/1", "D1");
    288   ValidateModel(back_model.get(), i++, 0);
    289   // i = 10
    290   LoadURLAndUpdateState("http://www.d.com/2", "D2");
    291   ValidateModel(back_model.get(), i++, 0);
    292   LoadURLAndUpdateState("http://www.d.com/3", "D3");
    293   ValidateModel(back_model.get(), i++, 0);
    294   LoadURLAndUpdateState("http://www.e.com/1", "E1");
    295   ValidateModel(back_model.get(), i++, 0);
    296   LoadURLAndUpdateState("http://www.e.com/2", "E2");
    297   ValidateModel(back_model.get(), i++, 0);
    298   LoadURLAndUpdateState("http://www.e.com/3", "E3");
    299   ValidateModel(back_model.get(), i++, 0);
    300   // i = 15
    301   LoadURLAndUpdateState("http://www.f.com/1", "F1");
    302   ValidateModel(back_model.get(), i++, 1);
    303   LoadURLAndUpdateState("http://www.f.com/2", "F2");
    304   ValidateModel(back_model.get(), i++, 1);
    305   LoadURLAndUpdateState("http://www.f.com/3", "F3");
    306   ValidateModel(back_model.get(), i++, 1);
    307   LoadURLAndUpdateState("http://www.g.com/1", "G1");
    308   ValidateModel(back_model.get(), i++, 2);
    309   LoadURLAndUpdateState("http://www.g.com/2", "G2");
    310   ValidateModel(back_model.get(), i++, 2);
    311   // i = 20
    312   LoadURLAndUpdateState("http://www.g.com/3", "G3");
    313   ValidateModel(back_model.get(), i++, 2);
    314   LoadURLAndUpdateState("http://www.h.com/1", "H1");
    315   ValidateModel(back_model.get(), i++, 3);
    316   LoadURLAndUpdateState("http://www.h.com/2", "H2");
    317   ValidateModel(back_model.get(), i++, 3);
    318   LoadURLAndUpdateState("http://www.h.com/3", "H3");
    319   ValidateModel(back_model.get(), i++, 3);
    320   LoadURLAndUpdateState("http://www.i.com/1", "I1");
    321   ValidateModel(back_model.get(), i++, 4);
    322   // i = 25
    323   LoadURLAndUpdateState("http://www.i.com/2", "I2");
    324   ValidateModel(back_model.get(), i++, 4);
    325   LoadURLAndUpdateState("http://www.i.com/3", "I3");
    326   ValidateModel(back_model.get(), i++, 4);
    327   LoadURLAndUpdateState("http://www.j.com/1", "J1");
    328   ValidateModel(back_model.get(), i++, 5);
    329   LoadURLAndUpdateState("http://www.j.com/2", "J2");
    330   ValidateModel(back_model.get(), i++, 5);
    331   LoadURLAndUpdateState("http://www.j.com/3", "J3");
    332   ValidateModel(back_model.get(), i++, 5);
    333   // i = 30
    334   LoadURLAndUpdateState("http://www.k.com/1", "K1");
    335   ValidateModel(back_model.get(), i++, 6);
    336   LoadURLAndUpdateState("http://www.k.com/2", "K2");
    337   ValidateModel(back_model.get(), i++, 6);
    338   // i = 32
    339   LoadURLAndUpdateState("http://www.k.com/3", "K3");
    340   ValidateModel(back_model.get(), i++, 6);
    341 
    342   // A chapter stop is defined as the last page the user
    343   // browsed to within the same domain.
    344 
    345   // Check to see if the chapter stops have the right labels.
    346   int index = BackForwardMenuModel::kMaxHistoryItems;
    347   // Empty string indicates item is a separator.
    348   EXPECT_EQ(base::string16(), back_model->GetLabelAt(index++));
    349   EXPECT_EQ(ASCIIToUTF16("F3"), back_model->GetLabelAt(index++));
    350   EXPECT_EQ(ASCIIToUTF16("E3"), back_model->GetLabelAt(index++));
    351   EXPECT_EQ(ASCIIToUTF16("D3"), back_model->GetLabelAt(index++));
    352   EXPECT_EQ(ASCIIToUTF16("C3"), back_model->GetLabelAt(index++));
    353   // The menu should only show a maximum of 5 chapter stops.
    354   EXPECT_EQ(ASCIIToUTF16("B3"), back_model->GetLabelAt(index));
    355   // Empty string indicates item is a separator.
    356   EXPECT_EQ(base::string16(), back_model->GetLabelAt(index + 1));
    357   EXPECT_EQ(back_model->GetShowFullHistoryLabel(),
    358             back_model->GetLabelAt(index + 2));
    359 
    360   // If we go back two we should still see the same chapter stop at the end.
    361   GoBack();
    362   EXPECT_EQ(ASCIIToUTF16("B3"), back_model->GetLabelAt(index));
    363   GoBack();
    364   EXPECT_EQ(ASCIIToUTF16("B3"), back_model->GetLabelAt(index));
    365   // But if we go back again, it should change.
    366   GoBack();
    367   EXPECT_EQ(ASCIIToUTF16("A3"), back_model->GetLabelAt(index));
    368   GoBack();
    369   EXPECT_EQ(ASCIIToUTF16("A3"), back_model->GetLabelAt(index));
    370   GoBack();
    371   EXPECT_EQ(ASCIIToUTF16("A3"), back_model->GetLabelAt(index));
    372   GoBack();
    373   // It is now a separator.
    374   EXPECT_EQ(base::string16(), back_model->GetLabelAt(index));
    375   // Undo our position change.
    376   NavigateToOffset(6);
    377 
    378   // Go back enough to make sure no chapter stops should appear.
    379   NavigateToOffset(-BackForwardMenuModel::kMaxHistoryItems);
    380   ValidateModel(forward_model.get(), BackForwardMenuModel::kMaxHistoryItems, 0);
    381   // Go forward (still no chapter stop)
    382   GoForward();
    383   ValidateModel(forward_model.get(),
    384                 BackForwardMenuModel::kMaxHistoryItems - 1, 0);
    385   // Go back two (one chapter stop should show up)
    386   GoBack();
    387   GoBack();
    388   ValidateModel(forward_model.get(),
    389                 BackForwardMenuModel::kMaxHistoryItems, 1);
    390 
    391   // Go to beginning.
    392   NavigateToIndex(0);
    393 
    394   // Check to see if the chapter stops have the right labels.
    395   index = BackForwardMenuModel::kMaxHistoryItems;
    396   // Empty string indicates item is a separator.
    397   EXPECT_EQ(base::string16(), forward_model->GetLabelAt(index++));
    398   EXPECT_EQ(ASCIIToUTF16("E3"), forward_model->GetLabelAt(index++));
    399   EXPECT_EQ(ASCIIToUTF16("F3"), forward_model->GetLabelAt(index++));
    400   EXPECT_EQ(ASCIIToUTF16("G3"), forward_model->GetLabelAt(index++));
    401   EXPECT_EQ(ASCIIToUTF16("H3"), forward_model->GetLabelAt(index++));
    402   // The menu should only show a maximum of 5 chapter stops.
    403   EXPECT_EQ(ASCIIToUTF16("I3"), forward_model->GetLabelAt(index));
    404   // Empty string indicates item is a separator.
    405   EXPECT_EQ(base::string16(), forward_model->GetLabelAt(index + 1));
    406   EXPECT_EQ(forward_model->GetShowFullHistoryLabel(),
    407       forward_model->GetLabelAt(index + 2));
    408 
    409   // If we advance one we should still see the same chapter stop at the end.
    410   GoForward();
    411   EXPECT_EQ(ASCIIToUTF16("I3"), forward_model->GetLabelAt(index));
    412   // But if we advance one again, it should change.
    413   GoForward();
    414   EXPECT_EQ(ASCIIToUTF16("J3"), forward_model->GetLabelAt(index));
    415   GoForward();
    416   EXPECT_EQ(ASCIIToUTF16("J3"), forward_model->GetLabelAt(index));
    417   GoForward();
    418   EXPECT_EQ(ASCIIToUTF16("J3"), forward_model->GetLabelAt(index));
    419   GoForward();
    420   EXPECT_EQ(ASCIIToUTF16("K3"), forward_model->GetLabelAt(index));
    421 
    422   // Now test the boundary cases by using the chapter stop function directly.
    423   // Out of bounds, first too far right (incrementing), then too far left.
    424   EXPECT_EQ(-1, back_model->GetIndexOfNextChapterStop(33, false));
    425   EXPECT_EQ(-1, back_model->GetIndexOfNextChapterStop(-1, true));
    426   // Test being at end and going right, then at beginning going left.
    427   EXPECT_EQ(-1, back_model->GetIndexOfNextChapterStop(32, true));
    428   EXPECT_EQ(-1, back_model->GetIndexOfNextChapterStop(0, false));
    429   // Test success: beginning going right and end going left.
    430   EXPECT_EQ(2,  back_model->GetIndexOfNextChapterStop(0, true));
    431   EXPECT_EQ(29, back_model->GetIndexOfNextChapterStop(32, false));
    432   // Now see when the chapter stops begin to show up.
    433   EXPECT_EQ(-1, back_model->GetIndexOfNextChapterStop(1, false));
    434   EXPECT_EQ(-1, back_model->GetIndexOfNextChapterStop(2, false));
    435   EXPECT_EQ(2,  back_model->GetIndexOfNextChapterStop(3, false));
    436   // Now see when the chapter stops end.
    437   EXPECT_EQ(32, back_model->GetIndexOfNextChapterStop(30, true));
    438   EXPECT_EQ(32, back_model->GetIndexOfNextChapterStop(31, true));
    439   EXPECT_EQ(-1, back_model->GetIndexOfNextChapterStop(32, true));
    440 
    441   // Bug found during review (two different sites, but first wasn't considered
    442   // a chapter-stop).
    443   // Go to A1;
    444   NavigateToIndex(0);
    445   LoadURLAndUpdateState("http://www.b.com/1", "B1");
    446   EXPECT_EQ(0, back_model->GetIndexOfNextChapterStop(1, false));
    447   EXPECT_EQ(1, back_model->GetIndexOfNextChapterStop(0, true));
    448 
    449   // Now see if it counts 'www.x.com' and 'mail.x.com' as same domain, which
    450   // it should.
    451   // Go to A1.
    452   NavigateToIndex(0);
    453   LoadURLAndUpdateState("http://mail.a.com/2", "A2-mai");
    454   LoadURLAndUpdateState("http://www.b.com/1", "B1");
    455   LoadURLAndUpdateState("http://mail.b.com/2", "B2-mai");
    456   LoadURLAndUpdateState("http://new.site.com", "new");
    457   EXPECT_EQ(1, back_model->GetIndexOfNextChapterStop(0, true));
    458   EXPECT_EQ(3, back_model->GetIndexOfNextChapterStop(1, true));
    459   EXPECT_EQ(3, back_model->GetIndexOfNextChapterStop(2, true));
    460   EXPECT_EQ(4, back_model->GetIndexOfNextChapterStop(3, true));
    461   // And try backwards as well.
    462   EXPECT_EQ(3, back_model->GetIndexOfNextChapterStop(4, false));
    463   EXPECT_EQ(1, back_model->GetIndexOfNextChapterStop(3, false));
    464   EXPECT_EQ(1, back_model->GetIndexOfNextChapterStop(2, false));
    465   EXPECT_EQ(-1, back_model->GetIndexOfNextChapterStop(1, false));
    466 }
    467 
    468 TEST_F(BackFwdMenuModelTest, EscapeLabel) {
    469   scoped_ptr<BackForwardMenuModel> back_model(new BackForwardMenuModel(
    470       NULL, BackForwardMenuModel::BACKWARD_MENU));
    471   back_model->set_test_web_contents(web_contents());
    472 
    473   EXPECT_EQ(0, back_model->GetItemCount());
    474   EXPECT_FALSE(back_model->ItemHasCommand(1));
    475 
    476   LoadURLAndUpdateState("http://www.a.com/1", "A B");
    477   LoadURLAndUpdateState("http://www.a.com/1", "A & B");
    478   LoadURLAndUpdateState("http://www.a.com/2", "A && B");
    479   LoadURLAndUpdateState("http://www.a.com/2", "A &&& B");
    480   LoadURLAndUpdateState("http://www.a.com/3", "");
    481 
    482   EXPECT_EQ(6, back_model->GetItemCount());
    483 
    484   // On Mac ui::MenuModel::GetLabelAt should return unescaped strings.
    485 #if defined(OS_MACOSX)
    486   EXPECT_EQ(ASCIIToUTF16("A B"), back_model->GetLabelAt(3));
    487   EXPECT_EQ(ASCIIToUTF16("A & B"), back_model->GetLabelAt(2));
    488   EXPECT_EQ(ASCIIToUTF16("A && B"), back_model->GetLabelAt(1));
    489   EXPECT_EQ(ASCIIToUTF16("A &&& B"), back_model->GetLabelAt(0));
    490 #else
    491   EXPECT_EQ(ASCIIToUTF16("A B"), back_model->GetLabelAt(3));
    492   EXPECT_EQ(ASCIIToUTF16("A && B"), back_model->GetLabelAt(2));
    493   EXPECT_EQ(ASCIIToUTF16("A &&&& B"), back_model->GetLabelAt(1));
    494   EXPECT_EQ(ASCIIToUTF16("A &&&&&& B"), back_model->GetLabelAt(0));
    495 #endif // defined(OS_MACOSX)
    496 }
    497 
    498 // Test asynchronous loading of favicon from history service.
    499 TEST_F(BackFwdMenuModelTest, FaviconLoadTest) {
    500   ASSERT_TRUE(profile()->CreateHistoryService(true, false));
    501   profile()->CreateFaviconService();
    502   Browser::CreateParams native_params(profile(), chrome::GetActiveDesktop());
    503   scoped_ptr<Browser> browser(
    504       chrome::CreateBrowserWithTestWindowForParams(&native_params));
    505   FaviconDelegate favicon_delegate;
    506 
    507   BackForwardMenuModel back_model(
    508       browser.get(), BackForwardMenuModel::BACKWARD_MENU);
    509   back_model.set_test_web_contents(controller().GetWebContents());
    510   back_model.SetMenuModelDelegate(&favicon_delegate);
    511 
    512   SkBitmap new_icon_bitmap(CreateBitmap(SK_ColorRED));
    513 
    514   GURL url1 = GURL("http://www.a.com/1");
    515   GURL url2 = GURL("http://www.a.com/2");
    516   GURL url1_favicon("http://www.a.com/1/favicon.ico");
    517 
    518   NavigateAndCommit(url1);
    519   // Navigate to a new URL so that url1 will be in the BackForwardMenuModel.
    520   NavigateAndCommit(url2);
    521 
    522   // Set the desired favicon for url1.
    523   HistoryServiceFactory::GetForProfile(
    524       profile(), Profile::EXPLICIT_ACCESS)->AddPage(
    525           url1, base::Time::Now(), history::SOURCE_BROWSED);
    526   FaviconServiceFactory::GetForProfile(profile(), Profile::EXPLICIT_ACCESS)
    527       ->SetFavicons(url1,
    528                     url1_favicon,
    529                     favicon_base::FAVICON,
    530                     gfx::Image::CreateFrom1xBitmap(new_icon_bitmap));
    531 
    532   // Will return the current icon (default) but start an anync call
    533   // to retrieve the favicon from the favicon service.
    534   gfx::Image default_icon;
    535   back_model.GetIconAt(0, &default_icon);
    536 
    537   // Make the favicon service run GetFavIconForURL,
    538   // FaviconDelegate.OnIconChanged will be called.
    539   base::MessageLoop::current()->Run();
    540 
    541   // Verify that the callback executed.
    542   EXPECT_TRUE(favicon_delegate.was_called());
    543 
    544   // Verify the bitmaps match.
    545   gfx::Image valid_icon;
    546   // This time we will get the new favicon returned.
    547   back_model.GetIconAt(0, &valid_icon);
    548 
    549   SkBitmap default_icon_bitmap = *default_icon.ToSkBitmap();
    550   SkBitmap valid_icon_bitmap = *valid_icon.ToSkBitmap();
    551 
    552   SkAutoLockPixels a(new_icon_bitmap);
    553   SkAutoLockPixels b(valid_icon_bitmap);
    554   SkAutoLockPixels c(default_icon_bitmap);
    555   // Verify we did not get the default favicon.
    556   EXPECT_NE(0, memcmp(default_icon_bitmap.getPixels(),
    557                       valid_icon_bitmap.getPixels(),
    558                       default_icon_bitmap.getSize()));
    559   // Verify we did get the expected favicon.
    560   EXPECT_EQ(0, memcmp(new_icon_bitmap.getPixels(),
    561                       valid_icon_bitmap.getPixels(),
    562                       new_icon_bitmap.getSize()));
    563 
    564   // Make sure the browser deconstructor doesn't have problems.
    565   browser->tab_strip_model()->CloseAllTabs();
    566   // This is required to prevent the message loop from hanging.
    567   profile()->DestroyHistoryService();
    568 }
    569