[ios] Fix omnibox baseWritingDirection and dismissal iOS11 bugs.

ios/chrome/browser/ui/omnibox/omnibox_popup_material_view_controller.mm

 // Copyright (c) 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 "ios/chrome/browser/ui/omnibox/omnibox_popup_material_view_controller.h"
 
 #include <memory>
 
 #include "base/ios/ios_util.h"
 #include "base/mac/scoped_cftyperef.h"
@@ skipping 17 matching lines @@
 #include "net/base/escape.h"
 
 #if !defined(__has_feature) || !__has_feature(objc_arc)
 #error "This file requires ARC support."
 #endif
 
 namespace {
 const int kRowCount = 6;
 const CGFloat kRowHeight = 48.0;
 const CGFloat kAnswerRowHeight = 64.0;
 const CGFloat kTopAndBottomPadding = 8.0;

[rohitrao (ping after 24h), 2017/06/14 16:47:03]

Why do we need to inset with this padding?  If you remember why, this could use
a comment.

 // The color of the main text of a suggest cell.
 UIColor* SuggestionTextColor() {
   return [UIColor colorWithWhite:(51 / 255.0) alpha:1.0];
 }
 // The color of the detail text of a suggest cell.
 UIColor* SuggestionDetailTextColor() {
   return [UIColor colorWithRed:(85 / 255.0)
                          green:(149 / 255.0)
                           blue:(254 / 255.0)
                          alpha:1.0];
@@ skipping 508 matching lines @@
   size_t size = _currentResult.size();
   if (animation && size > 0) {
     [self fadeInRows];
   }
 }
 
 - (void)layoutRows {
   size_t size = _currentResult.size();
 
   [self.tableView reloadData];
-  [self.tableView beginUpdates];
-  for (size_t i = 0; i < kRowCount; i++) {
-    OmniboxPopupMaterialRow* row = _rows[i];
 
-    if (i < size) {
-      const AutocompleteMatch& match =
-          ((const AutocompleteResult&)_currentResult).match_at((NSUInteger)i);
-      [self updateRow:row withMatch:match];
-      row.hidden = NO;
-    } else {
-      row.hidden = YES;
+  // We animate the rows ourselves in -fadeInRows, so make sure the table view
+  // doens't try to apply it's own animation.

[rohitrao (ping after 24h), 2017/06/14 16:47:03]

doesn't and its.

+  [UIView performWithoutAnimation:^{
+    [self.tableView beginUpdates];
+    for (size_t i = 0; i < kRowCount; i++) {
+      OmniboxPopupMaterialRow* row = _rows[i];
+
+      if (i < size) {
+        const AutocompleteMatch& match =
+            ((const AutocompleteResult&)_currentResult).match_at((NSUInteger)i);
+        [self updateRow:row withMatch:match];
+        row.hidden = NO;
+      } else {
+        row.hidden = YES;
+      }
     }
-  }
-  [self.tableView endUpdates];
+    [self.tableView endUpdates];
+  }];
 
   if (IsIPadIdiom())
     [self updateContentInsetForKeyboard];
 }
 
 - (void)keyboardDidShow:(NSNotification*)notification {
   NSDictionary* keyboardInfo = [notification userInfo];
   NSValue* keyboardFrameValue =
       [keyboardInfo valueForKey:UIKeyboardFrameEndUserInfoKey];
   keyboardHeight_ = CurrentKeyboardHeight(keyboardFrameValue);
@@ skipping 58 matching lines @@
   // Make a defensive copy of |match.contents|, as CopyToOmnibox() will trigger
   // a new round of autocomplete and modify |_currentResult|.
   base::string16 contents(match.contents);
   _popupView->CopyToOmnibox(contents);
 }
 
 #pragma mark -
 #pragma mark UIScrollViewDelegate
 
 - (void)scrollViewDidScroll:(UIScrollView*)scrollView {
-  // Setting the top inset of the scrollView to |kTopAndBottomPadding| causes a
-  // one time scrollViewDidScroll to |-kTopAndBottomPadding|.  It's easier to
-  // just ignore this one scroll tick.
-  if (scrollView.contentOffset.y == 0 - kTopAndBottomPadding)
+  if (!scrollView.dragging)

[rohitrao (ping after 24h), 2017/06/14 16:47:03]

Please add a comment here explaining why we need this check.  Seems like it's
because we programmatically set insets, which triggers a call to didScroll that
we want to ignore.  We only want to inform the machinery of a scroll if we know
that the user is directly scrolling.

     return;
 
   _popupView->DidScroll();
   for (OmniboxPopupMaterialRow* row in _rows) {
     row.highlighted = NO;
   }
 }
 
 // Set text alignment for popup cells.
 - (void)setTextAlignment:(NSTextAlignment)alignment {
@@ skipping 113 matching lines @@
     // The delete button never disappears if you don't call this after a tap.
     // It doesn't seem to be required anywhere else.
     [_rows[indexPath.row] prepareForReuse];
     const AutocompleteMatch& match =
         ((const AutocompleteResult&)_currentResult).match_at(indexPath.row);
     _popupView->DeleteMatch(match);
   }
 }
 
 @end