More code cleanups found by -Wextra.

(I'm still waiting on a v8 change to turn this on for the whole codebase...)

Review URL: http://codereview.chromium.org/746001

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@41080 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 15 insertions, 19 deletions

diff --git a/chrome/browser/views/location_bar_view.cc b/chrome/browser/views/location_bar_view.cc
index b1c8d5b..d52ac39 100644
--- a/chrome/browser/views/location_bar_view.cc
+++ b/chrome/browser/views/location_bar_view.cc
@@ -1746,15 +1746,13 @@ ExtensionAction* LocationBarView::GetPageAction(size_t index) {
 }
 
 ExtensionAction* LocationBarView::GetVisiblePageAction(size_t index) {
-  if (index >= 0) {
-    size_t current = 0;
-    for (size_t i = 0; i < page_action_views_.size(); ++i) {
-      if (page_action_views_[i]->IsVisible()) {
-        if (current == index)
-          return page_action_views_[i]->image_view()->page_action();
-
-        ++current;
-      }
+  size_t current = 0;
+  for (size_t i = 0; i < page_action_views_.size(); ++i) {
+    if (page_action_views_[i]->IsVisible()) {
+      if (current == index)
+        return page_action_views_[i]->image_view()->page_action();
+
+      ++current;
     }
   }
 
@@ -1763,17 +1761,15 @@ ExtensionAction* LocationBarView::GetVisiblePageAction(size_t index) {
 }
 
 void LocationBarView::TestPageActionPressed(size_t index) {
-  if (index >= 0) {
-    size_t current = 0;
-    for (size_t i = 0; i < page_action_views_.size(); ++i) {
-      if (page_action_views_[i]->IsVisible()) {
-        if (current == index) {
-          const int kLeftMouseButton = 1;
-          page_action_views_[i]->image_view()->ExecuteAction(kLeftMouseButton);
-          return;
-        }
-        ++current;
+  size_t current = 0;
+  for (size_t i = 0; i < page_action_views_.size(); ++i) {
+    if (page_action_views_[i]->IsVisible()) {
+      if (current == index) {
+        const int kLeftMouseButton = 1;
+        page_action_views_[i]->image_view()->ExecuteAction(kLeftMouseButton);
+        return;
       }
+      ++current;
     }
   }