Add in DOMBrowserView and Frame related classes

chrome/chrome_browser.gypi

 # Copyright (c) 2011 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.
 
 {
   'targets': [
     {
       'target_name': 'browser',
       'type': '<(library)',
       'msvs_guid': '5BF908A7-68FB-4A4B-99E3-8C749F1FE4EA',
@@ skipping 690 matching lines @@
         'browser/chromeos/webui/language_options_util.h',
         'browser/chromeos/webui/language_pinyin_options_handler.cc',
         'browser/chromeos/webui/language_pinyin_options_handler.h',
         'browser/chromeos/webui/login/authenticator_facade.h',
         'browser/chromeos/webui/login/authenticator_facade_cros.cc',
         'browser/chromeos/webui/login/authenticator_facade_cros.h',
         'browser/chromeos/webui/login/authenticator_facade_cros_helpers.cc',
         'browser/chromeos/webui/login/authenticator_facade_cros_helpers.h',
         'browser/chromeos/webui/login/browser/dom_browser.cc',
         'browser/chromeos/webui/login/browser/dom_browser.h',
+        'browser/chromeos/webui/login/browser/dom_browser_view.cc',
+        'browser/chromeos/webui/login/browser/dom_browser_view.h',
+        'browser/chromeos/webui/login/browser/dom_browser_view_layout.cc',
+        'browser/chromeos/webui/login/browser/dom_browser_view_layout.h',
+        'browser/chromeos/webui/login/frame/dom_browser_frame.cc',
+        'browser/chromeos/webui/login/frame/dom_browser_frame.h',
+        'browser/chromeos/webui/login/frame/dom_browser_frame_view.cc',
+        'browser/chromeos/webui/login/frame/dom_browser_frame_view.h',

[oshima, 2011/03/16 21:10:25]

sorry for missing this in previous review.

webui should be for the content code, not a frame/window of webui.
Can you move them to
browser/chromeos/frame? It's consistent with chrome/browser/ui/views/frame.

[rharrison, 2011/03/17 17:00:50]

Since this would involve moving a lot of code around I would prefer to leave it
where it is for this CL and then follow this CL up with a cleanup CL that covers
moving all of the code. Is this acceptable to you?

[oshima, 2011/03/17 18:49:31]

That's fine.

         'browser/chromeos/webui/login/login_ui.cc',
         'browser/chromeos/webui/login/login_ui.h',
         'browser/chromeos/webui/login/login_container_ui.cc',
         'browser/chromeos/webui/login/login_container_ui.h',
         'browser/chromeos/webui/login/login_ui_helpers.cc',
         'browser/chromeos/webui/login/login_ui_helpers.h',
         'browser/chromeos/webui/menu_ui.cc',
         'browser/chromeos/webui/menu_ui.h',
         'browser/chromeos/webui/mobile_setup_ui.cc',
         'browser/chromeos/webui/mobile_setup_ui.h',
@@ skipping 3626 matching lines @@
       ],
     },
   ],
 }
 
 # Local Variables:
 # tab-width:2
 # indent-tabs-mode:nil
 # End:
 # vim: set expandtab tabstop=2 shiftwidth=2: