Remove TODOs to create unecessary EarlGrey utilities.

A few things were tagged as candidates to move to shared code
when creating EarlGrey tests, but these should not move.

Bug: 638674
Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs
Change-Id: Ib80a06abc2c23d729573c594a14cf324284c6591
Reviewed-on: https://chromium-review.googlesource.com/806694Reviewed-by: Eugene But <eugenebut@chromium.org>
Commit-Queue: Mike Baxley <baxley@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521390}

ios/chrome/browser/net/cookies_egtest.mm

@@ -75,7 +75,6 @@ NSString* const kIncognitoCookieValue = @"rainbow";
 }
 
 // Clear cookies to make sure that tests do not interfere each other.
-// TODO(crbug.com/638674): Evaluate if this can move to shared code.
 - (void)tearDown {
   [ChromeEarlGrey
       loadURL:web::test::HttpServer::MakeUrl(kTestUrlNormalBrowsing)];

ios/chrome/browser/ui/keyboard_commands_egtest.mm

@@ -73,7 +73,6 @@ using chrome_test_util::RecentTabsMenuButton;
 }
 
 // Waits for the bookmark editor to display.
-// TODO(crbug.com/638674): Evaluate if this can move to shared code.
 - (void)waitForSingleBookmarkEditorToDisplay {
   BOOL (^block)
   () = ^BOOL {

ios/chrome/browser/ui/settings/settings_egtest.mm

@@ -498,7 +498,6 @@ bool IsCertificateCleared() {
 // Tests that clearing the cookies through the UI does clear all of them. Use a
 // local server to navigate to a page that sets then tests a cookie, and then
 // clears the cookie and tests it is not set.
-// TODO(crbug.com/638674): Evaluate if this can move to shared code.
 - (void)testClearCookies {
   // Creates a map of canned responses and set up the test HTML server.
   std::map<GURL, std::pair<std::string, std::string>> response;