added scss files

imported scss files from Github prototype project; 

BUG=330235
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=244328

[Renato Mangini (chromium), 2013-12-20 13:16:28]

Ready for review. Observe that I've added all scss files from Github, so we can
change site.css by changing the related scss files and compiling. Instructions
were added to docs/README

[not at google - send to devlin, 2013-12-20 23:30:51]

Is this urgent? Kun will soon be submitting native scss support into the
docserver which will avoid the need to do any compilation.

[Renato Mangini (chromium), 2013-12-23 12:41:27]

On 2013/12/20 23:30:51, kalman wrote:
> Is this urgent? Kun will soon be submitting native scss support into the
> docserver which will avoid the need to do any compilation.

When Kun's change is expected to land?

[hukun_google.com, 2013-12-24 02:03:57]

I am ready to submit the CL. Maybe it would be committed in one or two
weeks.


On Mon, Dec 23, 2013 at 8:41 PM, <mangini@chromium.org> wrote:

> On 2013/12/20 23:30:51, kalman wrote:
>
>> Is this urgent? Kun will soon be submitting native scss support into the
>> docserver which will avoid the need to do any compilation.
>>
>
> When Kun's change is expected to land?
>
> https://codereview.chromium.org/113963003/
>

To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.

[Renato Mangini (chromium), 2013-12-27 13:29:38]

I can't commit most of the other CLs until this one (or hukun's) lands, because
they all depend on the sass files. I could send new patch sets to all of them
removing the sass files and only replacing the generated site.css file, but that
might be too much work for a temporary solution.

@hukun, two weeks is tight considering our deadline. Can you link the CL here,
so I can follow its review progress?

Thanks! 

On 2013/12/24 02:03:57, hukun_google.com wrote:
> I am ready to submit the CL. Maybe it would be committed in one or two
> weeks.
> 
> 
> On Mon, Dec 23, 2013 at 8:41 PM, <mailto:mangini@chromium.org> wrote:
> 
> > On 2013/12/20 23:30:51, kalman wrote:
> >
> >> Is this urgent? Kun will soon be submitting native scss support into the
> >> docserver which will avoid the need to do any compilation.
> >>
> >
> > When Kun's change is expected to land?
> >
> > https://codereview.chromium.org/113963003/
> >
> 
> To unsubscribe from this group and stop receiving emails from it, send an
email
> to mailto:chromium-reviews+unsubscribe@chromium.org.

[not at google - send to devlin, 2013-12-27 22:51:03]

So if you want to commit this now that's fine but I'd want to roll it back once
Kun's CL goes in. You'll need to add a LICENSE and a README file for the
third_party stuff though.

Kun, any updates?

[Renato Mangini (chromium), 2013-12-30 12:47:42]

On 2013/12/27 22:51:03, kalman wrote:
> So if you want to commit this now that's fine but I'd want to roll it back
once
> Kun's CL goes in. You'll need to add a LICENSE and a README file for the
> third_party stuff though.
> 
> Kun, any updates?

There is no third-party - the scss files were developed by us (eric and jen). I
definitely don't want to add Compass (the compilation tool) to Chromium repo,
because it depends on Ruby. 

My idea is to keep the scss files as the single source of truth and
compile/generate a new site.css on every CL submitted. Conflicts on the
generated site.css would need to be handled by rebase + recompile scss. How does
that work as a temporary solution?

I assume Kun's CL will perform a on-the-fly compilation of scss files using
libsass. If that's the case, most of this CL will still be useful (the scss
files), but some parts will definitely need to be reworked.

[not at google - send to devlin, 2013-12-31 00:10:04]

Yeah that's what Kun's patch will do, and oops I read those files as being added
to third_party for some reason. Sorry, lgtm.

[commit-bot: I haz the power, 2014-01-03 13:42:29]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/mangini@chromium.org/113963003/70002

[commit-bot: I haz the power, 2014-01-03 13:56:36]

Retried try job too often on chromium_presubmit for step(s) presubmit
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=chromium_p...

[commit-bot: I haz the power, 2014-01-03 14:44:40]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/mangini@chromium.org/113963003/340001

[commit-bot: I haz the power, 2014-01-03 14:44:43]

Failed to apply patch for chrome/common/extensions/docs/config.rb:
While running patch -p1 --forward --force --no-backup-if-mismatch;
  can't find file to patch at input line 6
  Perhaps you used the wrong -p or --strip option?
  The text leading up to this was:
  --------------------------
  |Index: chrome/common/extensions/docs/config.rb
  |diff --git a/chrome/common/extensions/docs/config.rb
b/chrome/common/extensions/docs/config.rb
  |index
6a1226bbfc0726e0006b4a8d8c19005da376f548..911d8b8769f3074f8a5a0d787846ca73c53cee28
100644
  |--- a/chrome/common/extensions/docs/config.rb
  |+++ b/chrome/common/extensions/docs/config.rb
  --------------------------
  No file to patch.  Skipping patch.
  2 out of 2 hunks ignored

Patch:       chrome/common/extensions/docs/config.rb
Index: chrome/common/extensions/docs/config.rb
diff --git a/chrome/common/extensions/docs/config.rb
b/chrome/common/extensions/docs/config.rb
index
6a1226bbfc0726e0006b4a8d8c19005da376f548..911d8b8769f3074f8a5a0d787846ca73c53cee28
100644
--- a/chrome/common/extensions/docs/config.rb
+++ b/chrome/common/extensions/docs/config.rb
@@ -1,3 +1,7 @@
+# Copyright 2013 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.
+
 # Require any additional compass plugins here.
 
 # Change this to :production when ready to deploy the CSS to the live server.
@@ -16,33 +20,11 @@ images_dir = "static/images"
 javascripts_dir = "static/js"
 
 
-# You can select your preferred output style here (can be overridden via the
command line):
-# output_style = :expanded or :nested or :compact or :compressed
-
-# To enable relative paths to assets via compass helper functions. Uncomment:
-# relative_assets = true
-
-# To disable debugging comments that display the original location of your
selectors. Uncomment:
-# line_comments = false
-
-
-
 ##
 ## You probably don't need to edit anything below this.
 ##
 
-# You can select your preferred output style here (can be overridden via the
command line):
-# output_style = :expanded or :nested or :compact or :compressed
 output_style = (environment == :development) ? :expanded : :compressed
-
-# To enable relative paths to assets via compass helper functions. Since Drupal
-# themes can be installed in multiple locations, we don't need to worry about
-# the absolute path to the theme from server root.
 relative_assets = true
-
-# Pass options to sass.
-# - For development, we turn on the FireSass-compatible debug_info.
-# - For production, we force the CSS to be regenerated even though the source
-#   scss may not have changed, since we want the CSS to be compressed and have
-#   the debug info removed.
-sass_options = (environment == :development) ? {:debug_info => false} :
{:always_update => true}
+sass_options = (environment == :development) ? {:debug_info => false} \
+     : {:always_update => true}

[commit-bot: I haz the power, 2014-01-03 14:56:19]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/mangini@chromium.org/113963003/350005

[commit-bot: I haz the power, 2014-01-03 14:56:21]

Failed to apply patch for chrome/common/extensions/docs/config.rb:
While running patch -p1 --forward --force --no-backup-if-mismatch;
  can't find file to patch at input line 6
  Perhaps you used the wrong -p or --strip option?
  The text leading up to this was:
  --------------------------
  |Index: chrome/common/extensions/docs/config.rb
  |diff --git a/chrome/common/extensions/docs/config.rb
b/chrome/common/extensions/docs/config.rb
  |index
6a1226bbfc0726e0006b4a8d8c19005da376f548..911d8b8769f3074f8a5a0d787846ca73c53cee28
100644
  |--- a/chrome/common/extensions/docs/config.rb
  |+++ b/chrome/common/extensions/docs/config.rb
  --------------------------
  No file to patch.  Skipping patch.
  2 out of 2 hunks ignored

Patch:       chrome/common/extensions/docs/config.rb
Index: chrome/common/extensions/docs/config.rb
diff --git a/chrome/common/extensions/docs/config.rb
b/chrome/common/extensions/docs/config.rb
index
6a1226bbfc0726e0006b4a8d8c19005da376f548..911d8b8769f3074f8a5a0d787846ca73c53cee28
100644
--- a/chrome/common/extensions/docs/config.rb
+++ b/chrome/common/extensions/docs/config.rb
@@ -1,3 +1,7 @@
+# Copyright 2013 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.
+
 # Require any additional compass plugins here.
 
 # Change this to :production when ready to deploy the CSS to the live server.
@@ -16,33 +20,11 @@ images_dir = "static/images"
 javascripts_dir = "static/js"
 
 
-# You can select your preferred output style here (can be overridden via the
command line):
-# output_style = :expanded or :nested or :compact or :compressed
-
-# To enable relative paths to assets via compass helper functions. Uncomment:
-# relative_assets = true
-
-# To disable debugging comments that display the original location of your
selectors. Uncomment:
-# line_comments = false
-
-
-
 ##
 ## You probably don't need to edit anything below this.
 ##
 
-# You can select your preferred output style here (can be overridden via the
command line):
-# output_style = :expanded or :nested or :compact or :compressed
 output_style = (environment == :development) ? :expanded : :compressed
-
-# To enable relative paths to assets via compass helper functions. Since Drupal
-# themes can be installed in multiple locations, we don't need to worry about
-# the absolute path to the theme from server root.
 relative_assets = true
-
-# Pass options to sass.
-# - For development, we turn on the FireSass-compatible debug_info.
-# - For production, we force the CSS to be regenerated even though the source
-#   scss may not have changed, since we want the CSS to be compressed and have
-#   the debug info removed.
-sass_options = (environment == :development) ? {:debug_info => false} :
{:always_update => true}
+sass_options = (environment == :development) ? {:debug_info => false} \
+     : {:always_update => true}

[commit-bot: I haz the power, 2014-01-03 15:10:45]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/mangini@chromium.org/113963003/420001

[commit-bot: I haz the power, 2014-01-03 15:37:08]

Retried try job too often on chromium_presubmit for step(s) presubmit
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=chromium_p...

[commit-bot: I haz the power, 2014-01-03 16:17:37]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/mangini@chromium.org/113963003/530001

[commit-bot: I haz the power, 2014-01-03 16:39:57]

Retried try job too often on chromium_presubmit for step(s) presubmit
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=chromium_p...

[hukun_google.com, 2014-01-06 07:36:15]

Sorry. I did not notice this email until today. I will upload scss relative
CL after Ben approved my current markdown CL. So, it still needs one or two
weeks.


On Sat, Dec 28, 2013 at 6:51 AM, <kalman@chromium.org> wrote:

> So if you want to commit this now that's fine but I'd want to roll it back
> once
> Kun's CL goes in. You'll need to add a LICENSE and a README file for the
> third_party stuff though.
>
> Kun, any updates?
>
> https://codereview.chromium.org/113963003/
>

To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.

[commit-bot: I haz the power, 2014-01-09 21:40:40]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/mangini@chromium.org/113963003/690001

[commit-bot: I haz the power, 2014-01-09 22:05:23]

Retried try job too often on chromium_presubmit for step(s) presubmit
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=chromium_p...

[commit-bot: I haz the power, 2014-01-10 00:27:02]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/mangini@chromium.org/113963003/820001

[commit-bot: I haz the power, 2014-01-10 01:05:28]

Retried try job too often on chromium_presubmit for step(s) presubmit
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=chromium_p...

[commit-bot: I haz the power, 2014-01-11 00:20:17]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/mangini@chromium.org/113963003/930001

[commit-bot: I haz the power, 2014-01-11 03:10:36]

Change committed as 244328